From: Penny Leach <thispicture@rambler.ru>
Date: Mon, 13 Apr 2009 13:58:26 +0300
Subject: [PATCH] Array fixes - change name of arrays in unique name.

---
 question/type/dragdrop/templates/dragdroparrange.tpl |  14 +++++++------- 
 question/type/dragdrop/templates/dragdropshow.tpl  |  12 ++++++------
 question/type/dragdrop/dragdrop.php   |     31 ++++++++++++++++---------------
 3 files changed, 29 insertions(+), 28 deletions(-)

--- D:/shea/moodle/вопросы/dragdrop/dragdrop.php	Sun Jul 06 08:21:34 2008
+++ D:/shea/moodle/вопросы/drugdrop_main/dragdrop.php	Fri Apr 10 15:59:15 2009
@@ -184,12 +184,13 @@
             $mediahotspot = $media->hotspots[$media->primary_hotspot];
             $suffix = "{$this->dragdropqcount}_$i";
             $container = "container$suffix";
-            $hotspot = "hotspot$suffix";
+            $hotspot = "hotspot$suffix".$id;
 
             $isimage = $this->is_image($media->media);
 
             $maxheight = max($maxheight, ($media->displayheight > 0 ? $media->displayheight : $media->height));
-            $displayname = "gapimage$suffix";
+            $displayname = "gapimage$suffix".$id;
             $displaylayers[$i]['tag'] = $this->dragdrop_tag($question->id, $media, $displayname, !$isimage);
 
             $displaylayers[$i]['name'] = $displayname;
@@ -306,6 +307,7 @@
 
         // Javascriptcode for DDGapImage object added here because we need it twice.
         // In dragdropshow.tpl and in dragdroparrange.tpl.

         $text = "<script type='text/javascript'>"."\n".
                 "// Our custom drag and drop implementation, extending YAHOO.util.DD"."\n".
                 "YAHOO.util.DDGapImage = function(id, sGroup, config) {"."\n".
@@ -357,7 +359,7 @@
 
                 "         // return to the start position"."\n".
                 "         var attr = {"."\n".
               "                       points: { to: myAbsolutPos}"."\n".
                 "                    };"."\n".
                 "         // we store this in a dummy for calling the setFormValues function"."\n".
                 "         // at the end of the animation"."\n".
@@ -420,34 +422,36 @@
                 "     }"."\n".
                 "});"."\n"."\n".
 
-                "var arrOfGapimageObjects = new Array();"."\n".
-                "var arrOfGapimages = new Array();"."\n".
+                "var arrOfGapimageObjects".$id." = new Array();"."\n".
+                "var arrOfGapimages".$id." = new Array();"."\n".
                 "</script>"."\n"."\n";
+        $smarty->assign('id1',$id);
         $smarty->assign('script1', $text);
 

         $text  = '<script type="text/javascript">'."\n".
                  '// Initializing the background as valid drop target'."\n".
                  'var background = new YAHOO.util.DDTarget(backgroundId);'."\n"."\n".
 
                  '// Initializing the gapimages'."\n".
                  'function init() {'."\n".
-                 '    for (var i = 0; i < arrOfGapimageObjects.length; ++i) {'."\n".
-                 '        arrOfGapimages[i] = new YAHOO.util.DDGapImage(arrOfGapimageObjects[i].name);'."\n".
-                 '        arrOfGapimages[i].backgroundId = arrOfGapimageObjects[i].backgroundId;'."\n".
-                 '        arrOfGapimages[i].width = arrOfGapimageObjects[i].width;'."\n".
-                 '        arrOfGapimages[i].height = arrOfGapimageObjects[i].height;'."\n";
+                 '    for (var i = 0; i < arrOfGapimageObjects'.$id.'.length; ++i) {'."\n".
+                 '        arrOfGapimages'.$id.'[i] = new YAHOO.util.DDGapImage(arrOfGapimageObjects'.$id.'[i].name);'."\n".
+                 '        arrOfGapimages'.$id.'[i].backgroundId = arrOfGapimageObjects'.$id.'[i].backgroundId;'."\n".
+                 '        arrOfGapimages'.$id.'[i].width = arrOfGapimageObjects'.$id.'[i].width;'."\n".
+                 '        arrOfGapimages'.$id.'[i].height = arrOfGapimageObjects'.$id.'[i].height;'."\n";
 
         // the arrange template gets another script than the view template
         if ($displaytype == 'edit') {
-            $text .= '        arrOfGapimages[i].targetx = arrOfGapimageObjects[i].targetx;'."\n";
-            $text .= '        arrOfGapimages[i].targety = arrOfGapimageObjects[i].targety;'."\n";
-            $text .= '        arrOfGapimages[i].initDDGapImage();'."\n";
-            $text .= '        arrOfHotspots[i].gapImage = arrOfGapimages[i];'."\n";
+            $text .= '        arrOfGapimages'.$id.'[i].targetx = arrOfGapimageObjects'.$id.'[i].targetx;'."\n";
+            $text .= '        arrOfGapimages'.$id.'[i].targety = arrOfGapimageObjects'.$id.'[i].targety;'."\n";
+            $text .= '        arrOfGapimages'.$id.'[i].initDDGapImage();'."\n";
+            $text .= '        arrOfHotspots[i].gapImage = arrOfGapimages'.$id.'[i];'."\n";
             $text .= '        arrOfHotspots[i].initDDHotSpot();'."\n";
         }
         else {
-            $text .= '        arrOfGapimages[i].responseForm = YAHOO.util.Dom.get(arrOfGapimageObjects[i].responseFormId);'."\n";
-            $text .= '        arrOfGapimages[i].initDDGapImage();'."\n";
+            $text .= '        arrOfGapimages'.$id.'[i].responseForm = YAHOO.util.Dom.get(arrOfGapimageObjects'.$id.'[i].responseFormId);'."\n";
+            $text .= '        arrOfGapimages'.$id.'[i].initDDGapImage();'."\n";
         }
 
         $text .= '    }'."\n".
--- D:/shea/moodle/вопросы/dragdrop/templates/dragdroparrange.tpl	Sat Jul 05 18:58:56 2008
+++ D:/shea/moodle/вопросы/drugdrop_main/templates/dragdroparrange.tpl	Thu Apr 09 16:38:34 2009
@@ -295,13 +295,13 @@
                             {$gapimages[img].tag}
 
                             <script type="text/javascript">
-                            arrOfGapimageObjects[{$smarty.section.img.index}]                   = new Object();
-                            arrOfGapimageObjects[{$smarty.section.img.index}]["name"]           = "{$gapimages[img].name}";
-                            arrOfGapimageObjects[{$smarty.section.img.index}]["backgroundId"]   = backgroundId;
-                            arrOfGapimageObjects[{$smarty.section.img.index}]["width"]          = {$gapimages[img].width};
-                            arrOfGapimageObjects[{$smarty.section.img.index}]["height"]         = {$gapimages[img].height};
-                            arrOfGapimageObjects[{$smarty.section.img.index}]["targetx"]        = {$gapimages[img].targetx};
-                            arrOfGapimageObjects[{$smarty.section.img.index}]["targety"]        = {$gapimages[img].targety};
+                            arrOfGapimageObjects{$id1}[{$smarty.section.img.index}]                   = new Object();
+                            arrOfGapimageObjects{$id1}[{$smarty.section.img.index}]["name"]           = "{$gapimages[img].name}";
+                            arrOfGapimageObjects{$id1}[{$smarty.section.img.index}]["backgroundId"]   = backgroundId;
+                            arrOfGapimageObjects{$id1}[{$smarty.section.img.index}]["width"]          = {$gapimages[img].width};
+                            arrOfGapimageObjects{$id1}[{$smarty.section.img.index}]["height"]         = {$gapimages[img].height};
+                            arrOfGapimageObjects{$id1}[{$smarty.section.img.index}]["targetx"]        = {$gapimages[img].targetx};
+                            arrOfGapimageObjects{$id1}[{$smarty.section.img.index}]["targety"]        = {$gapimages[img].targety};
                             </script>
                         </td>
                     </tr>
--- D:/shea/moodle/вопросы/dragdrop/templates/dragdropshow.tpl	Sat Jul 05 18:58:56 2008
+++ D:/shea/moodle/вопросы/drugdrop_main/templates/dragdropshow.tpl	Thu Apr 09 15:53:40 2009
@@ -57,12 +57,12 @@
                             {$gapimages[img].tag}
 
                             <script type="text/javascript">
-                            arrOfGapimageObjects[{$smarty.section.img.index}]                   = new Object();
-                            arrOfGapimageObjects[{$smarty.section.img.index}]["name"]           = "{$gapimages[img].name}";
-                            arrOfGapimageObjects[{$smarty.section.img.index}]["responseFormId"] = "{$gapimages[img].responseFormId}";
-                            arrOfGapimageObjects[{$smarty.section.img.index}]["backgroundId"]   = backgroundId;
-                            arrOfGapimageObjects[{$smarty.section.img.index}]["width"]          = {$gapimages[img].width};
-                            arrOfGapimageObjects[{$smarty.section.img.index}]["height"]         = {$gapimages[img].height};
+                            arrOfGapimageObjects{$id1}[{$smarty.section.img.index}]                   = new Object();
+                            arrOfGapimageObjects{$id1}[{$smarty.section.img.index}]["name"]           = "{$gapimages[img].name}";
+                            arrOfGapimageObjects{$id1}[{$smarty.section.img.index}]["responseFormId"] = "{$gapimages[img].responseFormId}";
+                            arrOfGapimageObjects{$id1}[{$smarty.section.img.index}]["backgroundId"]   = backgroundId;
+                            arrOfGapimageObjects{$id1}[{$smarty.section.img.index}]["width"]          = {$gapimages[img].width};
+                            arrOfGapimageObjects{$id1}[{$smarty.section.img.index}]["height"]         = {$gapimages[img].height};
                             </script>
                         </td>
                     </tr>