From ba5f337fdbe8804d8d227aa0ad2f7079b98a2d66 Mon Sep 17 00:00:00 2001 From: Valera Rozuvan Date: Wed, 23 Oct 2013 11:52:22 +0300 Subject: [PATCH] Fix bug. Add validation to HTML5 sources field. --- cms/static/js/views/transcripts/editor.js | 16 ++++++++++++---- cms/static/js/views/transcripts/utils.js | 2 -- 2 files changed, 12 insertions(+), 6 deletions(-) diff --git a/cms/static/js/views/transcripts/editor.js b/cms/static/js/views/transcripts/editor.js index d369161cd6..21e48a03b0 100644 --- a/cms/static/js/views/transcripts/editor.js +++ b/cms/static/js/views/transcripts/editor.js @@ -110,11 +110,19 @@ function($, Backbone, _, Utils, MetadataView, MetadataCollection) { // Get values from `Advanced` tab fields (`html5_sources`, // `youtube_id_1_0`) that should be synchronized. - values.html5Sources = getField(metadataCollection, 'html5_sources') - .getDisplayValue(); + html5Sources = getField(metadataCollection, 'html5_sources') +                                    .getDisplayValue(); + +            values.youtube = getField(metadataCollection, 'youtube_id_1_0') +                                    .getDisplayValue(); + +            values.html5Sources = _.filter(html5Sources, function (value) { +                var link = Utils.parseLink(value), + mode = link && link.mode; + +                return mode === 'html5' && mode; +            }); - values.youtube = getField(metadataCollection, 'youtube_id_1_0') - .getDisplayValue(); // The length of youtube video_id should be 11 characters. if (values.youtube.length === 11) { diff --git a/cms/static/js/views/transcripts/utils.js b/cms/static/js/views/transcripts/utils.js index fa26d7fb55..7965d0b9e3 100644 --- a/cms/static/js/views/transcripts/utils.js +++ b/cms/static/js/views/transcripts/utils.js @@ -317,8 +317,6 @@ define(["jquery", "underscore", "jquery.ajaxQueue"], function($, _) { return function (action, component_id, videoList, extraParams) { var params, data; - console.log('[_command]: arguments = ', arguments); - if (extraParams) { if ($.isPlainObject(extraParams)) { params = extraParams;