From: Michael J. Rubinsky <mrubinsk@horde.org>
Date: Sun, 8 Nov 2009 21:33:54 +0000 (-0500)
Subject: Fix toggle actions to work properly with only POST data
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=3e5cbe4a13ed9243b432d4d7db55bca1f54920d9;p=horde.git

Fix toggle actions to work properly with only POST data
---

diff --git a/ansel/js/togglewidget.js b/ansel/js/togglewidget.js
index c299065f9..1dbf970c7 100755
--- a/ansel/js/togglewidget.js
+++ b/ansel/js/togglewidget.js
@@ -29,10 +29,6 @@ function togglePlusMinus(node, requestType)
     }
 
     var url = Ansel.widgets[requestType].url;
- 
-    var params = new Object();
-    params.pref_value = pref_value;
-    new Ajax.Request(url, {
-        parameters: params
-    });
+    var params = { "value": "value=" + pref_value };
+    new Ajax.Request(url, { parameters: params });
 }
\ No newline at end of file
diff --git a/ansel/lib/Ajax/Imple/ToggleGalleryActions.php b/ansel/lib/Ajax/Imple/ToggleGalleryActions.php
index 357474e19..7f2094636 100644
--- a/ansel/lib/Ajax/Imple/ToggleGalleryActions.php
+++ b/ansel/lib/Ajax/Imple/ToggleGalleryActions.php
@@ -15,7 +15,7 @@ class Ansel_Ajax_Imple_ToggleGalleryActions extends Horde_Ajax_Imple_Base
         // Include the js
         Horde::addScriptFile('togglewidget.js');
 
-        $url = $this->_getUrl('ToggleGalleryActions', 'ansel', array('input' => 'pref_value', 'sessionWrite' => true));
+        $url = $this->_getUrl('ToggleGalleryActions', 'ansel', array('post' => 'value', 'sessionWrite' => true));
 
         $js = array();
         $js[] = "Ansel.widgets['galleryActions'] = {'bindTo': '" . $this->_params['bindTo'] . "', 'url': '" . $url . "'}";
@@ -26,10 +26,10 @@ class Ansel_Ajax_Imple_ToggleGalleryActions extends Horde_Ajax_Imple_Base
 
     public function handle($args, $post)
     {
-         if (empty($args['input'])) {
+         if (!isset($post['value'])) {
             return 0;
          }
-         $input = Horde_Util::getPost($args['input']);
+         $input = Horde_Util::getPost($post['value']);
          $GLOBALS['prefs']->setValue('show_actions', $input);
 
         return 1;
diff --git a/ansel/lib/Ajax/Imple/ToggleOtherGalleries.php b/ansel/lib/Ajax/Imple/ToggleOtherGalleries.php
index 715c17167..8bc74eeda 100644
--- a/ansel/lib/Ajax/Imple/ToggleOtherGalleries.php
+++ b/ansel/lib/Ajax/Imple/ToggleOtherGalleries.php
@@ -15,7 +15,7 @@ class Ansel_Ajax_Imple_ToggleOtherGalleries extends Horde_Ajax_Imple_Base
         // Include the js
         Horde::addScriptFile('togglewidget.js');
 
-        $url = $this->_getUrl('ToggleOtherGalleries', 'ansel', array('input' => 'pref_value', 'sessionWrite' => true));
+        $url = $this->_getUrl('ToggleOtherGalleries', 'ansel', array('post' => 'value', 'sessionWrite' => true));
         $js = array();
         $js[] = "Ansel.widgets['otherGalleries'] = {'bindTo': '" . $this->_params['bindTo'] . "', 'url': '" . $url . "'}";
         $js[] = "Event.observe(Ansel.widgets.otherGalleries.bindTo + '-toggle', 'click', function(event) {doActionToggle('" . $this->_params['bindTo'] . "', 'otherGalleries'); Event.stop(event)});";
@@ -25,14 +25,13 @@ class Ansel_Ajax_Imple_ToggleOtherGalleries extends Horde_Ajax_Imple_Base
 
     public function handle($args, $post)
     {
-         if (empty($args['input'])) {
+         if (!isset($post['value'])) {
             return 0;
          }
-        $input = Horde_Util::getPost($args['input']);
-        $GLOBALS['prefs']->setValue('show_othergalleries', $input);
-        echo $GLOBALS['prefs']->getValue('show_othergalleries');
 
-        return 1;
+        $GLOBALS['prefs']->setValue('show_othergalleries', $post['value']);
+
+        return $GLOBALS['prefs']->getValue('show_othergalleries');
     }
 
 }