track imple post-only changes in ajax actions for face editing
authorMichael J. Rubinsky <mrubinsk@horde.org>
Mon, 9 Nov 2009 19:12:15 +0000 (14:12 -0500)
committerMichael J. Rubinsky <mrubinsk@horde.org>
Mon, 9 Nov 2009 19:12:15 +0000 (14:12 -0500)
ansel/js/editfaces.js
ansel/lib/Ajax/Imple/EditFaces.php

index 4657ac1..47443b4 100644 (file)
@@ -1,30 +1,20 @@
 document.observe('dom:loaded', function() {
     Ansel.deleteFace = function(image_id, face_id)
     {
-        new Ajax.Request(Ansel.ajax.editFaces.url, 
+        new Ajax.Request(Ansel.ajax.editFaces.url + "/action=delete/post=values"
             {
                 method: 'post',
-                parameters: {
-                                 action: 'delete',
-                                 image: image_id,
-                                 face: face_id
-                            }
+                parameters: { "values": "image=" + image_id + "/face=" + face_id }
             });
         $('face' + face_id).remove();
     };
     
     Ansel.setFaceName = function(image_id, face_id)
     {
-        new Ajax.Request(Ansel.ajax.editFaces.url,
+        new Ajax.Request(Ansel.ajax.editFaces.url + "/action=setname/post=values",
                          {    
                               method: 'post',
-                              parameters: 
-                              {
-                                  action: 'setname',
-                                  face: face_id,
-                                  image: image_id,
-                                  facename: encodeURIComponent($F('facename' + face_id))
-                              },
+                              parameters: { "values": "face=" + face_id + "/image=" + image_id + "/facename=" + encodeURIComponent($F('facename' + face_id)) },
                               onComplete: function(r) {
                                   if (r.responseJSON.response == 1) {
                                       $('faces_widget_content').update(r.responseJSON.message);
@@ -37,14 +27,10 @@ document.observe('dom:loaded', function() {
     Ansel.doFaceEdit = function(image_id)
     {
         $('faces_widget_content').update(Ansel.ajax.editFaces.text.loading);
-        new Ajax.Request(Ansel.ajax.editFaces.url,
+        new Ajax.Request(Ansel.ajax.editFaces.url + "/action=process/post=values",
                          {
                              method: 'post',
-                             parameters:
-                             {
-                                 action: 'process',
-                                 image: image_id
-                             },
+                             parameters: { "values": "image=" + image_id },
                              onComplete: function(r) {
                                  if (r.responseJSON.response == 1) {
                                      $('faces_widget_content').update(r.responseJSON.message);
index 2c23b38..3f33ce0 100644 (file)
@@ -33,10 +33,9 @@ class Ansel_Ajax_Imple_EditFaces extends Horde_Ajax_Imple_Base
         include_once dirname(__FILE__) . '/../../base.php';
 
         if (Horde_Auth::getAuth()) {
-            /* Require POST for these actions */
-            $action = Horde_Util::getPost('action');
-            $image_id = (int)Horde_Util::getPost('image');
-            $reload = Horde_Util::getPost('reload', 0);
+            $action = $args['action'];
+            $image_id = (int)$post['image'];
+            $reload = empty($post['reload']) ? 0 : $post['reload'];
 
             if (empty($action)) {
                 return array('response' => 0);
@@ -73,7 +72,7 @@ class Ansel_Ajax_Imple_EditFaces extends Horde_Ajax_Imple_Base
 
             case 'delete':
                 // delete - deletes a single face from an image.
-                $face_id = (int)Horde_Util::getPost('face');
+                $face_id = (int)$post['face'];
                 $image = &$GLOBALS['ansel_storage']->getImage($image_id);
                 $gallery = &$GLOBALS['ansel_storage']->getGallery($image->gallery);
                 if (!$gallery->hasPermission(Horde_Auth::getAuth(), PERMS_EDIT)) {
@@ -86,12 +85,12 @@ class Ansel_Ajax_Imple_EditFaces extends Horde_Ajax_Imple_Base
 
             case 'setname':
                 // setname - sets the name of a single image.
-                $face_id = (int)Horde_Util::getPost('face');
+                $face_id = (int)$post['face'];
                 if (!$face_id) {
                     return array('response' => 0);
                 }
 
-                $name = Horde_Util::getPost('facename');
+                $name = $post['facename'];
                 $image = &$GLOBALS['ansel_storage']->getImage($image_id);
                 $gallery = &$GLOBALS['ansel_storage']->getGallery($image->gallery);
                 if (!$gallery->hasPermission(Horde_Auth::getAuth(), PERMS_EDIT)) {