diff --git a/common/lib/xmodule/xmodule/annotatable_module.py b/common/lib/xmodule/xmodule/annotatable_module.py index 5b167dd0be..db2aa13cb7 100644 --- a/common/lib/xmodule/xmodule/annotatable_module.py +++ b/common/lib/xmodule/xmodule/annotatable_module.py @@ -5,8 +5,6 @@ from pkg_resources import resource_string, resource_listdir from xmodule.x_module import XModule from xmodule.raw_module import RawDescriptor -from xmodule.modulestore.mongo import MongoModuleStore -from xmodule.modulestore.django import modulestore from xmodule.contentstore.content import StaticContent from xblock.core import Scope, String diff --git a/common/lib/xmodule/xmodule/modulestore/django.py b/common/lib/xmodule/xmodule/modulestore/django.py index 8e48154188..88471191d2 100644 --- a/common/lib/xmodule/xmodule/modulestore/django.py +++ b/common/lib/xmodule/xmodule/modulestore/django.py @@ -8,13 +8,7 @@ from __future__ import absolute_import from importlib import import_module from os import environ -# cdodge: ICK! Sorry about this but I'm not sure how to resolve. We have some unit tests which do not setup a -# Django runtime. This import expects to find a "CACHE =" in a configuration, which doesn't exist in the -# test environment -try: - from django.core.cache import get_cache, InvalidCacheBackendError -except: - pass +from django.core.cache import get_cache, InvalidCacheBackendError from django.conf import settings diff --git a/common/lib/xmodule/xmodule/video_module.py b/common/lib/xmodule/xmodule/video_module.py index 4fc87b0750..0203299b40 100644 --- a/common/lib/xmodule/xmodule/video_module.py +++ b/common/lib/xmodule/xmodule/video_module.py @@ -8,8 +8,6 @@ from django.http import Http404 from xmodule.x_module import XModule from xmodule.raw_module import RawDescriptor -from xmodule.modulestore.xml import XMLModuleStore -from xmodule.modulestore.django import modulestore from xmodule.contentstore.content import StaticContent from xblock.core import Integer, Scope, String @@ -120,13 +118,6 @@ class VideoModule(VideoFields, XModule): return self.youtube def get_html(self): - if isinstance(modulestore(), XMLModuleStore): - # VS[compat] - # cdodge: filesystem static content support. - caption_asset_path = "/static/{0}/subs/".format(self.descriptor.data_dir) - else: - caption_asset_path = StaticContent.get_base_url_path_for_course_assets(self.location) + '/subs_' - # We normally let JS parse this, but in the case that we need a hacked # out player because YouTube has broken their