From eaa15ed1adbe1e85004ab962bc7673d15ab2332c Mon Sep 17 00:00:00 2001
From: csantossaenz <csantossaenz>
Date: Fri, 15 Jun 2007 09:26:17 +0000
Subject: [PATCH] Fixed bug mdl-7516

---
 mod/scorm/datamodels/scorm_12lib.php | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

diff --git a/mod/scorm/datamodels/scorm_12lib.php b/mod/scorm/datamodels/scorm_12lib.php
index 9a18e7b710..e20da14cc6 100644
--- a/mod/scorm/datamodels/scorm_12lib.php
+++ b/mod/scorm/datamodels/scorm_12lib.php
@@ -264,8 +264,8 @@ function scorm_get_toc($user,$scorm,$liststyle,$currentorg='',$scoid='',$mode='n
                 if (isset($_COOKIE['hide:SCORMitem'.$nextsco->id])) {
                     $icon = 'plus';
                 }
-                //$result->toc .= '<a href="javascript:expandCollide(\'img'.$sublist.'\',\''.$sublist.'\','.$nextsco->id.');"><img id="img'.$sublist.'" src="'.$scormpixdir.'/'.$icon.'.gif" alt="'.$strexpand.'" title="'.$strexpand.'"/></a>';
-                $result->toc .= '<a href="#" onclick="elementToggleHide(\''.$sublist.'\',true);"><img id="img'.$sublist.'" src="'.$scormpixdir.'/'.$icon.'.gif" alt="'.$strexpand.'" title="'.$strexpand.'"/></a>';
+               $result->toc .= $result->toc .= '<a href="javascript:expandCollide(\'img'.$sublist.'\','.$sublist.','.$nextsco->id.');"><img id="img'.$sublist.'" src="'.$scormpixdir.'/'.$icon.'.gif" alt="'.$strexpand.'" title="'.$strexpand.'"/></a>';
+               // $result->toc .= '<a href="#" onclick="elementToggleHide(\''.$sublist.'\',true);"><img id="img'.$sublist.'" src="'.$scormpixdir.'/'.$icon.'.gif" alt="'.$strexpand.'" title="'.$strexpand.'"/></a>';
             } else if ($isvisible) {
                 $result->toc .= '<img src="'.$scormpixdir.'/spacer.gif" />';
             }
@@ -369,20 +369,21 @@ function scorm_get_toc($user,$scorm,$liststyle,$currentorg='',$scoid='',$mode='n
         $result->toc .= '
           <script type="text/javascript">
           //<![CDATA[
-              function expandCollide(which,list,item) {
+               function expandCollide(which,list,item) {
                   var nn=document.ids?true:false
                   var w3c=document.getElementById?true:false
                   var beg=nn?"document.ids.":w3c?"document.getElementById(":"document.all.";
-                  var style=w3c?").style":".style";
-                  var mid=w3c?")":"";
+                  
+                  var mid=w3c?").style":".style";
+				  which = which.substring(0,(which.length));
 
                   if (eval(beg+list+mid+".display") != "none") {
-                      eval(beg+which+mid+".src=\''.$scormpixdir.'/plus.gif\';");
-                      eval(beg+list+style+".display=\'none\';");
+                      document.getElementById(which).src = "'.$scormpixdir.'/plus.gif";
+                      eval(beg+list+mid+".display=\'none\';");
                       new cookie("hide:SCORMitem" + item, 1, 356, "/").set();
                   } else {
-                      eval(beg+which+mid+".src=\''.$scormpixdir.'/minus.gif\';");
-                      eval(beg+list+style+".display=\'block\';");
+                      document.getElementById(which).src = "'.$scormpixdir.'/minus.gif";
+                      eval(beg+list+mid+".display=\'block\';");
                       new cookie("hide:SCORMitem" + item, 1, -1, "/").set();
                   }
               }
-- 
2.39.5