chore: some fixes

Author: farhan

xmodule/assets/video/public/js/01_initialize.js

@@ -16,9 +16,6 @@ import i18n from './00_i18n.js';
 import _ from 'underscore';
 import moment from 'moment';
 
-// TODO: Review this removal of line
-// const moment = moment || window.moment;
-
 /**
  * @function
  *
@@ -732,7 +729,8 @@ function youtubeId(speed) {
 
 function getDuration() {
     try {
-        return moment.duration(this.metadata[this.youtubeId()].duration, moment.ISO_8601).asSeconds();
+        const safeMoment = typeof moment !== 'undefined' ? moment : window.moment;
+        return safeMoment.duration(this.metadata[this.youtubeId()].duration, safeMoment.ISO_8601).asSeconds();
     } catch (err) {
         return _.result(this.metadata[this.youtubeId('1.0')], 'duration') || 0;
     }

xmodule/assets/video/public/js/10_main.js

@@ -38,7 +38,6 @@ if (typeof window.gettext === 'undefined') {
 
 (function () {
     let youtubeXhr = null;
-    const oldVideo = window.Video;
 
     window.Video = function (runtime, element) {
         console.log('In Video initialize method');
@@ -147,6 +146,7 @@ if (typeof window.gettext === 'undefined') {
     // TODO: Review following code and remove it as it's not needed anymore
     // Invoke the mock Video constructor so that the elements stored within it can be processed by the real
     // `window.Video` constructor.
+    // const oldVideo = window.Video;
     // oldVideo(null, true);
 
 }());