Merge "Action::checkCanExecute doesn't return anything on success, but WatchAction...

diff --git a/includes/Action.php b/includes/Action.php
index 84c0f80..a25e8aa 100644 (file)
--- a/includes/Action.php
+++ b/includes/Action.php
@@ -266,6 +266,7 @@ abstract class Action {
         *
         * @param $user User: the user to check, or null to use the context user
         * @throws ErrorPageError
+        * @return bool True on success
         */
        protected function checkCanExecute( User $user ) {
                $right = $this->getRestriction();
@@ -287,6 +288,7 @@ abstract class Action {
                if ( $this->requiresWrite() && wfReadOnly() ) {
                        throw new ReadOnlyError();
                }
+               return true;
        }
 
        /**