Move more POST handling into ManagedAction

diff --git a/lib/formaction.php b/lib/formaction.php
index 7c68583268449860f79cd090272ba621e79784c2..0cc455e9681df7462a7d804ef819f7094e068d5b 100644 (file)
--- a/lib/formaction.php
+++ b/lib/formaction.php
@@ -116,24 +116,4 @@ class FormAction extends ManagedAction
         $form = new $class($this, $this->formOpts);
         return $form;
     }
-
-    /**
-     * Gets called from handle() if isPost() is true;
-     * @return void
-     */
-    protected function handlePost()
-    {
-        // check for this before token since all POST and FILES data
-        // is losts when size is exceeded
-        if (empty($_POST) && $_SERVER['CONTENT_LENGTH']>0) {
-            // TRANS: Client error displayed when the number of bytes in a POST request exceeds a limit.
-            // TRANS: %s is the number of bytes of the CONTENT_LENGTH.
-            $msg = _m('The server was unable to handle that much POST data (%s MiB) due to its current configuration.',
-                      'The server was unable to handle that much POST data (%s MiB) due to its current configuration.',
-                      round($_SERVER['CONTENT_LENGTH']/1024/1024,2));
-            throw new ClientException($msg);
-        }
-
-        return parent::handlePost();
-    }
 }

diff --git a/lib/managedaction.php b/lib/managedaction.php
index 22752aff61951f144fe65515efa280507602a201..4caf977210f3d6b9897b5114ca4806c6ce6ea281 100644 (file)
--- a/lib/managedaction.php
+++ b/lib/managedaction.php
@@ -75,6 +75,17 @@ class ManagedAction extends Action
         // This will only be run if the Action has the property canPost==true
         assert($this->canPost);
 
+        // check for this before token since all POST and FILES data
+        // is losts when size is exceeded
+        if (empty($_POST) && $_SERVER['CONTENT_LENGTH']>0) {
+            // TRANS: Client error displayed when the number of bytes in a POST request exceeds a limit.
+            // TRANS: %s is the number of bytes of the CONTENT_LENGTH.
+            $msg = _m('The server was unable to handle that much POST data (%s MiB) due to its current configuration.',
+                      'The server was unable to handle that much POST data (%s MiB) due to its current configuration.',
+                      round($_SERVER['CONTENT_LENGTH']/1024/1024,2));
+            throw new ClientException($msg);
+        }
+
         $this->checkSessionToken();
         return $this->doPost();
     }
@@ -85,6 +96,6 @@ class ManagedAction extends Action
      * exception on failure, with a descriptive message.
      */
     protected function doPost() {
-        throw new Exception('Unhandled POST');
+        return false;
     }
 }