Pages crash if trackVideo is not set

thexerteproject · Jul 7, 2022 · 83ec7a7 · 83ec7a7
1 parent 2613309
commit 83ec7a7
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/modules/xerte/parent_templates/Nottingham/models_html5/interactiveVideo.html b/modules/xerte/parent_templates/Nottingham/models_html5/interactiveVideo.html
@@ -304,7 +304,12 @@
                 // Calculate progress
                 progress = XThelperDetermineProgress(videoState);
             }
-            switch (x_currentPageXML.getAttribute('trackVideo')) {
+            let trackVideoSetting = 'no_mcqs';
+            if (x_currentPageXML.getAttribute('trackVideo') != undefined)
+            {
+                trackVideoSetting = x_currentPageXML.getAttribute('trackVideo');
+            }
+            switch (trackVideoSetting) {
                 case 'yes':
                     totalScore = (mcqScore + progress) / 2;
                     break;

diff --git a/modules/xerte/parent_templates/Nottingham/models_html5/mediaLesson.html b/modules/xerte/parent_templates/Nottingham/models_html5/mediaLesson.html
@@ -265,7 +265,12 @@
 				// Calculate progress
 				progress = XThelperDetermineProgress(videoState);
 			}
-			switch (x_currentPageXML.getAttribute('trackVideo')) {
+			let trackVideoSetting = 'no_mcqs';
+			if (x_currentPageXML.getAttribute('trackVideo') != undefined)
+			{
+				trackVideoSetting = x_currentPageXML.getAttribute('trackVideo');
+			}
+			switch (trackVideoSetting) {
 				case 'yes':
 					totalScore = (mcqScore + progress) / 2;
 					break;