From: jamiesensei <jamiesensei>
Date: Tue, 29 May 2007 06:14:09 +0000 (+0000)
Subject: reverting last commit "added another parameter to moodle_url out, out_action and... 
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=d65cb30f630d69f13c7a3bedf2f43d12ef130c17;p=moodle.git

reverting last commit "added another parameter to moodle_url out, out_action and get_query_string so that developer can output url with query string joined by '&' and not '&amp;'. &amp; breaks redirect on IE."  It turned out to be unnecessary.
---

diff --git a/lib/listlib.php b/lib/listlib.php
index 87ec53b95a..75c266d2bd 100644
--- a/lib/listlib.php
+++ b/lib/listlib.php
@@ -275,7 +275,7 @@ class moodle_list{
                     $html .= " $currentpage \n";
                 }
                 else {
-                    $html .= "<a href=\"".$this->pageurl->out('', false, array($this->pageparamname => $currentpage))."\">";
+                    $html .= "<a href=\"".$this->pageurl->out(false, array($this->pageparamname => $currentpage))."\">";
                     $html .= " $currentpage </a>\n";
                 }
             }
@@ -429,7 +429,7 @@ class moodle_list{
             return false;
         }
 
-        redirect($this->pageurl->out('&'));
+        redirect($this->pageurl->out());
     }
 }
 
@@ -541,26 +541,26 @@ class list_item{
             } else {
                 $action = $strmoveleft;
             }
-            $icons['left'] = $this->image_icon($action, $this->parentlist->pageurl->out_action('', array('left'=>$this->id)), 'left'); 
+            $icons['left'] = $this->image_icon($action, $this->parentlist->pageurl->out_action(array('left'=>$this->id)), 'left'); 
         } else {
             $icons['left'] =  $this->image_spacer();
         }
 
         if (!$first) {
-             $icons['up'] = $this->image_icon($strmoveup, $this->parentlist->pageurl->out_action('', array('moveup'=>$this->id)), 'up');
+             $icons['up'] = $this->image_icon($strmoveup, $this->parentlist->pageurl->out_action(array('moveup'=>$this->id)), 'up');
         } else {
             $icons['up'] =  $this->image_spacer();
         }
 
         if (!$last) {
-            $icons['down'] = $this->image_icon($strmovedown, $this->parentlist->pageurl->out_action('', array('movedown'=>$this->id)), 'down');
+            $icons['down'] = $this->image_icon($strmovedown, $this->parentlist->pageurl->out_action(array('movedown'=>$this->id)), 'down');
         } else {
             $icons['down'] =  $this->image_spacer();
         }
 
         if (!empty($lastitem)) {
             $makechildof = get_string('makechildof', 'question', $lastitem->name);
-            $icons['right'] = $this->image_icon($makechildof, $this->parentlist->pageurl->out_action('', array('right'=>$this->id)), 'right'); 
+            $icons['right'] = $this->image_icon($makechildof, $this->parentlist->pageurl->out_action(array('right'=>$this->id)), 'right'); 
         } else {
             $icons['right'] =  $this->image_spacer();
         }