Skip to content

Commit 84b028d

Browse files
committed
Merge pull request #392 from jleyva/MOBILE-1428
MOBILE-1428 mod_scorm: LMSFinish must return strings representing bool
2 parents 6afdc3d + 335f706 commit 84b028d

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

www/addons/mod_scorm/services/data_model_12.js

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -513,6 +513,9 @@ angular.module('mm.addons.mod_scorm')
513513

514514
// Trigger TOC update.
515515
triggerEvent(mmaModScormEventUpdateToc);
516+
517+
// Conver to string representing a boolean.
518+
result = result ? "true" : "false";
516519
return result;
517520
} else {
518521
errorCode = "301";
@@ -685,6 +688,9 @@ angular.module('mm.addons.mod_scorm')
685688
triggerEvent(mmaModScormEventUpdateToc);
686689

687690
errorCode = result ? '0' : '101';
691+
692+
// Conver to string representing a boolean.
693+
result = result ? "true" : "false";
688694
return result;
689695
} else {
690696
errorCode = "301";

0 commit comments

Comments
 (0)