From 07b5ee31c074af467bb7e9e4305b2af31e31d51f Mon Sep 17 00:00:00 2001 From: Piers Harding Date: Fri, 13 Aug 2010 01:26:33 +0000 Subject: [PATCH] MDL-23751 fix the clearing down of the TOC before reloading the next after navigation. --- mod/scorm/datamodels/callback.js.php | 20 +++++++++++--------- mod/scorm/datamodels/scorm_12.js.php | 2 +- 2 files changed, 12 insertions(+), 10 deletions(-) diff --git a/mod/scorm/datamodels/callback.js.php b/mod/scorm/datamodels/callback.js.php index 103bbafb4bd..ee053084742 100644 --- a/mod/scorm/datamodels/callback.js.php +++ b/mod/scorm/datamodels/callback.js.php @@ -4,15 +4,17 @@ success: function(o) { scorm_tree_node = YAHOO.widget.TreeView.getTree('scorm_tree'); if (o.responseText !== undefined) { - if (scorm_tree_node && o.responseText) { - var hnode = scorm_tree_node.getHighlightedNode(); - var hidx = null; - if (hnode) { - hidx = hnode.index + scorm_tree_node.getNodeCount(); - } - // all gone - var root_node = scorm_tree_node.getRoot(); - scorm_tree_node.removeNode(root_node.children[0]); + if (scorm_tree_node && o.responseText) { + var hnode = scorm_tree_node.getHighlightedNode(); + var hidx = null; + if (hnode) { + hidx = hnode.index + scorm_tree_node.getNodeCount(); + } + // all gone + var root_node = scorm_tree_node.getRoot(); + while (root_node.children.length > 0) { + scorm_tree_node.removeNode(root_node.children[0]); + } } // make sure the temporary tree element is not there var el_old_tree = document.getElementById('scormtree123'); diff --git a/mod/scorm/datamodels/scorm_12.js.php b/mod/scorm/datamodels/scorm_12.js.php index 2aa1f51bdb4..2a8596f91b4 100644 --- a/mod/scorm/datamodels/scorm_12.js.php +++ b/mod/scorm/datamodels/scorm_12.js.php @@ -436,7 +436,7 @@ function SCORMapi1_2() { if (scorm_debugging($scorm)) { //echo 'alert("Finished SCORM 1.2");'; echo 'LogAPICall("LMSCommit", "result", result, 0);'; - echo 'LogAPICall("LMSCommit", param, "", 0);'; + echo 'LogAPICall("LMSCommit", "errorCode", errorCode, 0);'; } ?> return result;