Fix: model.set moved to model, empty cc added to avoid error on load

cgkineo · Jan 25, 2024 · 28143c1 · 28143c1
1 parent 06a0c8a
commit 28143c1
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 10 deletions.
diff --git a/js/simpleVideoBranchingModel.js b/js/simpleVideoBranchingModel.js
@@ -28,8 +28,14 @@ define([
         }
       });
 
-      // totals are useful to have too
+      // totals are useful to have too, set initial player state attributes
       this.set({
+        _isMediaEnded: false,
+        _isMediaPlaying: false,
+        _media: {
+          cc: [],
+          mp4: ' '// prevent console warning 'No media is selected in components.json'
+        },
         _mediaCount: mediaIndex,
         _questionCount: questionIndex
       });

diff --git a/js/simpleVideoBranchingView.js b/js/simpleVideoBranchingView.js
@@ -29,15 +29,6 @@ define([
 
       _.bindAll(this, 'onMediaElementPlay', 'onMediaElementPause', 'onMediaElementEnded', 'onMediaElementTimeUpdate', 'onMediaElementSeeking');
 
-      // set initial player state attributes
-      this.model.set({
-        _isMediaEnded: false,
-        _isMediaPlaying: false,
-        _media: {
-          mp4: ' '// prevent console warning 'No media is selected in components.json'
-        }
-      });
-
       this.checkIfResetOnRevisit?.();
     },