Commit 821e1549 by Alan Boudreault

revert deprecated block_type change for local_resource_url

parent 68610ec1
...@@ -91,8 +91,8 @@ class PreviewModuleSystem(ModuleSystem): # pylint: disable=abstract-method ...@@ -91,8 +91,8 @@ class PreviewModuleSystem(ModuleSystem): # pylint: disable=abstract-method
'suffix': suffix, 'suffix': suffix,
}) + '?' + query }) + '?' + query
def local_resource_url(self, block, uri, block_type=None): def local_resource_url(self, block, uri):
return local_resource_url(block, uri, block_type) return local_resource_url(block, uri)
class StudioUserService(object): class StudioUserService(object):
......
...@@ -25,11 +25,11 @@ def handler_url(block, handler_name, suffix='', query='', thirdparty=False): ...@@ -25,11 +25,11 @@ def handler_url(block, handler_name, suffix='', query='', thirdparty=False):
return url return url
def local_resource_url(block, uri, block_type=None): def local_resource_url(block, uri):
""" """
local_resource_url for Studio local_resource_url for Studio
""" """
return reverse('xblock_resource_url', kwargs={ return reverse('xblock_resource_url', kwargs={
'block_type': block_type or block.scope_ids.block_type, 'block_type': block.scope_ids.block_type,
'uri': uri, 'uri': uri,
}) })
...@@ -1178,13 +1178,13 @@ class DescriptorSystem(MetricsMixin, ConfigurableFragmentWrapper, Runtime): # p ...@@ -1178,13 +1178,13 @@ class DescriptorSystem(MetricsMixin, ConfigurableFragmentWrapper, Runtime): # p
# global function that the application can override. # global function that the application can override.
return descriptor_global_handler_url(block, handler_name, suffix, query, thirdparty) return descriptor_global_handler_url(block, handler_name, suffix, query, thirdparty)
def local_resource_url(self, block, uri, block_type=None): def local_resource_url(self, block, uri):
""" """
See :meth:`xblock.runtime.Runtime:local_resource_url` for documentation. See :meth:`xblock.runtime.Runtime:local_resource_url` for documentation.
""" """
xmodule_runtime = getattr(block, 'xmodule_runtime', None) xmodule_runtime = getattr(block, 'xmodule_runtime', None)
if xmodule_runtime is not None: if xmodule_runtime is not None:
return xmodule_runtime.local_resource_url(block, uri, block_type) return xmodule_runtime.local_resource_url(block, uri)
else: else:
# Currently, Modulestore is responsible for instantiating DescriptorSystems # Currently, Modulestore is responsible for instantiating DescriptorSystems
# This means that LMS/CMS don't have a way to define a subclass of DescriptorSystem # This means that LMS/CMS don't have a way to define a subclass of DescriptorSystem
......
...@@ -114,12 +114,12 @@ class LmsHandlerUrls(object): ...@@ -114,12 +114,12 @@ class LmsHandlerUrls(object):
return url return url
def local_resource_url(self, block, uri, block_type=None): def local_resource_url(self, block, uri):
""" """
local_resource_url for Studio local_resource_url for Studio
""" """
path = reverse('xblock_resource_url', kwargs={ path = reverse('xblock_resource_url', kwargs={
'block_type': block_type or block.scope_ids.block_type, 'block_type': block.scope_ids.block_type,
'uri': uri, 'uri': uri,
}) })
return '//{}{}'.format(settings.SITE_NAME, path) return '//{}{}'.format(settings.SITE_NAME, path)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment