From: piers <piers>
Date: Tue, 29 Jul 2008 22:18:43 +0000 (+0000)
Subject: MDL-12222 Error adding cmi.session_time in SCORM 1.3 (patch thanks to Hector Gomis... 
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=2f60a9cb5b13a7ae4ea253bd084ba5e54979cac4;p=moodle.git

MDL-12222 Error adding cmi.session_time in SCORM 1.3 (patch thanks to Hector Gomis).  Also - some syntax formating to stop errors in Eclipse highlighting).
---

diff --git a/mod/scorm/datamodels/scorm_13.js.php b/mod/scorm/datamodels/scorm_13.js.php
index 035eafc1ce..c1f0e77e82 100644
--- a/mod/scorm/datamodels/scorm_13.js.php
+++ b/mod/scorm/datamodels/scorm_13.js.php
@@ -550,7 +550,7 @@ function SCORMapi1_3() {
                                                             }
                                                             if ((nodes.length > 0) && (nodes.length <= correct_responses[interactiontype].max)) {
                                                                 expression = new RegExp(correct_responses[interactiontype].format);
-                                                                for (var i=0; (i<nodes.length) && (errorCode=="0"); i++) {
+                                                                for (var i=0; (i < nodes.length) && (errorCode=="0"); i++) {
                                                                     if (typeof correct_responses[interactiontype].delimiter2 != 'undefined') {
                                                                         values = nodes[i].split(correct_responses[interactiontype].delimiter2);
                                                                         if (values.length == 2) {
@@ -654,7 +654,7 @@ function SCORMapi1_3() {
                                                     }
                                                     if ((nodes.length > 0) && (nodes.length <= learner_response[interactiontype].max)) {
                                                         expression = new RegExp(learner_response[interactiontype].format);
-                                                        for (var i=0; (i<nodes.length) && (errorCode=="0"); i++) {
+                                                        for (var i=0; (i < nodes.length) && (errorCode=="0"); i++) {
                                                             if (typeof learner_response[interactiontype].delimiter2 != 'undefined') {
                                                                 values = nodes[i].split(learner_response[interactiontype].delimiter2);
                                                                 if (values.length == 2) {
@@ -720,7 +720,7 @@ function SCORMapi1_3() {
 	
                                                     if ((nodes.length > 0) && (nodes.length <= correct_responses[interactiontype].max)) {
                                                         expression = new RegExp(correct_responses[interactiontype].format);
-                                                        for (var i=0; (i<nodes.length) && (errorCode=="0"); i++) {
+                                                        for (var i=0; (i < nodes.length) && (errorCode=="0"); i++) {
                                                             if (typeof correct_responses[interactiontype].delimiter2 != 'undefined') {
 		
 	
@@ -968,7 +968,7 @@ function SCORMapi1_3() {
     function duplicatedID (element, parent, value) {
         var found = false;
         var elements = eval(parent+'._count');
-        for (var n=0;(n<elements) && (!found);n++) {
+        for (var n=0;(n < elements) && (!found);n++) {
             if ((parent+'.N'+n+'.id' != element) && (eval(parent+'.N'+n+'.id') == value)) {
                 found = true;
             }
@@ -979,7 +979,7 @@ function SCORMapi1_3() {
     function duplicatedPA (element, parent, value) {
         var found = false;
         var elements = eval(parent+'._count');
-        for (var n=0;(n<elements) && (!found);n++) {
+        for (var n=0;(n < elements) && (!found);n++) {
             if ((parent+'.N'+n+'.pattern' != element) && (eval(parent+'.N'+n+'.pattern') == value)) {
                 found = true;
             }
@@ -1017,7 +1017,7 @@ function SCORMapi1_3() {
             var mins = parseInt(firstarray[11],10)+parseInt(secondarray[11],10)+change;   //Minutes
             change = Math.floor(mins / 60);
             mins = mins - (change * 60);
-            var hours = parseInt(firstarray[10],10)+parseInt(secondarray[10],10)+change;  //Hours
+            var hours = parseInt(firstarray[9],10)+parseInt(secondarray[9],10)+change;  //Hours
             change = Math.floor(hours / 24);
             hours = hours - (change * 24);
             var days = parseInt(firstarray[6],10)+parseInt(secondarray[6],10)+change; // Days