From 63a5affbda0b94da296704d42e0ef537afae787f Mon Sep 17 00:00:00 2001
From: Dan Marsden <Dan@digitalclay.co.nz>
Date: Thu, 24 Jun 2010 02:21:15 +0000
Subject: [PATCH] SCORM MDL-17891 fix loss of session data - thanks to Ron and
 Matteo for report/fixes merged from HEAD

---
 mod/scorm/datamodels/scorm_12.js.php | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/mod/scorm/datamodels/scorm_12.js.php b/mod/scorm/datamodels/scorm_12.js.php
index 9bab6e53960..7cc4ffec5be 100644
--- a/mod/scorm/datamodels/scorm_12.js.php
+++ b/mod/scorm/datamodels/scorm_12.js.php
@@ -198,14 +198,22 @@ function SCORMapi1_2() {
                     if (<?php echo $scorm->auto ?> == 1) {
                         setTimeout('top.document.location=top.next;',500);
                     }
-                }    
+                }
+               <?php
+                    if (debugging('',DEBUG_DEVELOPER)) {
+                        echo 'LogAPICall("LMSFinish", "AJAXResult", result, 0);';
+                    }
+                ?>
+                result = ('true' == result) ? 'true' : 'false';
+                errorCode = (result == 'true')? '0' : '101';
                 <?php 
                     if (debugging('',DEBUG_DEVELOPER)) {
                         //echo 'alert("Finished SCORM 1.2");';
+                        echo 'LogAPICall("LMSFinish", "result", result, 0);';
                         echo 'LogAPICall("LMSFinish", param, "", 0);';
                     }
                 ?>
-                return "true";
+                return result;
             } else {
                 errorCode = "301";
             }