diff --git a/common/lib/xmodule/xmodule/js/src/video/03_video_player.js b/common/lib/xmodule/xmodule/js/src/video/03_video_player.js index 649dd9cf668d4cd5a9854a5045dbc318fe5a7319..03fcb193518ab3afdc9642838da63fae7edc72a8 100644 --- a/common/lib/xmodule/xmodule/js/src/video/03_video_player.js +++ b/common/lib/xmodule/xmodule/js/src/video/03_video_player.js @@ -420,12 +420,15 @@ function (HTML5Video) { this.videoPlayer.player.setPlaybackRate(this.speed); } + /* The following has been commented out to make sure autoplay is + disabled for students. if ( !onTouchBasedDevice() && $('.video:first').data('autoplay') === 'True' ) { this.videoPlayer.play(); } + */ } function onStateChange(event) { diff --git a/lms/djangoapps/courseware/tests/test_video_mongo.py b/lms/djangoapps/courseware/tests/test_video_mongo.py index ea4949e4c9b3294736dcb13e7502503aa1cca177..75cf55c1cc3c93e209f0d5307877798199c7bf17 100644 --- a/lms/djangoapps/courseware/tests/test_video_mongo.py +++ b/lms/djangoapps/courseware/tests/test_video_mongo.py @@ -61,7 +61,7 @@ class TestVideo(BaseTestXmodule): 'sub': u'a_sub_file.srt.sjson', 'track': '', 'youtube_streams': _create_youtube_string(self.item_module), - 'autoplay': settings.MITX_FEATURES.get('AUTOPLAY_VIDEOS', True), + 'autoplay': settings.MITX_FEATURES.get('AUTOPLAY_VIDEOS', False), 'yt_test_timeout': 1500, 'yt_test_url': 'https://gdata.youtube.com/feeds/api/videos/' } diff --git a/lms/djangoapps/courseware/tests/test_video_xml.py b/lms/djangoapps/courseware/tests/test_video_xml.py index ea17af723db697035c4515ac950e2439ce2c1563..28cee28e7cbbffeedbdde707875b402dca959222 100644 --- a/lms/djangoapps/courseware/tests/test_video_xml.py +++ b/lms/djangoapps/courseware/tests/test_video_xml.py @@ -94,7 +94,7 @@ class VideoModuleUnitTest(unittest.TestCase): 'sources': sources, 'youtube_streams': _create_youtube_string(module), 'track': '', - 'autoplay': settings.MITX_FEATURES.get('AUTOPLAY_VIDEOS', True), + 'autoplay': settings.MITX_FEATURES.get('AUTOPLAY_VIDEOS', False), 'yt_test_timeout': 1500, 'yt_test_url': 'https://gdata.youtube.com/feeds/api/videos/' } diff --git a/lms/envs/common.py b/lms/envs/common.py index 3fa191876927590ced1939a6082b447793addc73..81b01179d7cab94836520da2d794528a72a25972 100644 --- a/lms/envs/common.py +++ b/lms/envs/common.py @@ -144,8 +144,8 @@ MITX_FEATURES = { # Toggle to indicate use of a custom theme 'USE_CUSTOM_THEME': False, - # Do autoplay videos for students - 'AUTOPLAY_VIDEOS': True, + # Don't autoplay videos for students + 'AUTOPLAY_VIDEOS': False, # Enable instructor dash to submit background tasks 'ENABLE_INSTRUCTOR_BACKGROUND_TASKS': True,