diff --git a/common/lib/xmodule/xmodule/tests/test_video.py b/common/lib/xmodule/xmodule/tests/test_video.py index 8fc4aabef3..490e65ef6b 100644 --- a/common/lib/xmodule/xmodule/tests/test_video.py +++ b/common/lib/xmodule/xmodule/tests/test_video.py @@ -272,7 +272,7 @@ class VideoDescriptorImportTestCase(unittest.TestCase): 'end_time': datetime.timedelta(seconds=0.0), 'track': '', 'download_track': False, - 'download_video': False, + 'download_video': True, 'html5_sources': ['http://www.example.com/source.mp4'], 'data': '' }) diff --git a/common/lib/xmodule/xmodule/video_module.py b/common/lib/xmodule/xmodule/video_module.py index 9764066141..0ed9303403 100644 --- a/common/lib/xmodule/xmodule/video_module.py +++ b/common/lib/xmodule/xmodule/video_module.py @@ -563,6 +563,10 @@ class VideoDescriptor(VideoFields, TabsEditingDescriptor, EmptyDataRawDescriptor value = deserialize_field(cls.fields[attr], value) field_data[attr] = value + # Add `source` for backwards compatibility if xml doesn't have `download_video`. + if 'download_video' not in field_data and sources: + field_data['source'] = field_data['html5_sources'][0] + return field_data diff --git a/lms/djangoapps/courseware/tests/test_video_mongo.py b/lms/djangoapps/courseware/tests/test_video_mongo.py index e6a24154bd..b5b277cb43 100644 --- a/lms/djangoapps/courseware/tests/test_video_mongo.py +++ b/lms/djangoapps/courseware/tests/test_video_mongo.py @@ -198,6 +198,7 @@ class TestGetHtmlMethod(BaseTestXmodule): 'end': 3610.0, 'id': None, 'sources': { + 'main': u'example.mp4', u'mp4': u'example.mp4', u'webm': u'example.webm' },