FU r106752: use "media-" instead of "images-" in container names. Long live books...
[lhc/web/wiklou.git] / includes / api / ApiBase.php
index be1be86..e1ba493 100644 (file)
@@ -1,9 +1,8 @@
 <?php
-
 /**
- * Created on Sep 5, 2006
  *
- * API for MediaWiki 1.8+
+ *
+ * Created on Sep 5, 2006
  *
  * Copyright © 2006, 2010 Yuri Astrakhan <Firstname><Lastname>@gmail.com
  *
@@ -21,6 +20,8 @@
  * with this program; if not, write to the Free Software Foundation, Inc.,
  * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
  * http://www.gnu.org/copyleft/gpl.html
+ *
+ * @file
  */
 
 /**
@@ -29,7 +30,7 @@
  * The class functions are divided into several areas of functionality:
  *
  * Module parameters: Derived classes can define getAllowedParams() to specify
- *     which parameters to expect,ow to parse and validate them.
+ *     which parameters to expect, how to parse and validate them.
  *
  * Profiling: various methods to allow keeping tabs on various tasks and their
  *     time costs
@@ -38,7 +39,7 @@
  *
  * @ingroup API
  */
-abstract class ApiBase {
+abstract class ApiBase extends ContextSource {
 
        // These constants allow modules to specify exactly how to treat incoming parameters.
 
@@ -51,6 +52,7 @@ abstract class ApiBase {
        const PARAM_ALLOW_DUPLICATES = 6; // Boolean, do we allow the same value to be set more than once when ISMULTI=true
        const PARAM_DEPRECATED = 7; // Boolean, is the parameter deprecated (will show a warning)
        const PARAM_REQUIRED = 8; // Boolean, is the parameter required?
+       const PARAM_RANGE_ENFORCE = 9; // Boolean, if MIN/MAX are set, enforce (die) these? Only applies if TYPE='integer' Use with extreme caution
 
        const LIMIT_BIG1 = 500; // Fast query, std user limit
        const LIMIT_BIG2 = 5000; // Fast query, bot/sysop limit
@@ -70,6 +72,10 @@ abstract class ApiBase {
                $this->mMainModule = $mainModule;
                $this->mModuleName = $moduleName;
                $this->mModulePrefix = $modulePrefix;
+
+               if ( !$this->isMain() ) {
+                       $this->setContext( $mainModule->getContext() );
+               }
        }
 
        /*****************************************************************************
@@ -120,6 +126,9 @@ abstract class ApiBase {
 
        /**
         * Get the name of the module as shown in the profiler log
+        *
+        * @param $db DatabaseBase
+        *
         * @return string
         */
        public function getModuleProfileName( $db = false ) {
@@ -168,6 +177,20 @@ abstract class ApiBase {
                return $this->getResult()->getData();
        }
 
+       /**
+        * Create a new RequestContext object to use e.g. for calls to other parts
+        * the software.
+        * The object will have the WebRequest and the User object set to the ones
+        * used in this instance.
+        *
+        * @deprecated since 1.19 use getContext to get the current context
+        * @return DerivativeContext
+        */
+       public function createContext() {
+               wfDeprecated( __METHOD__, '1.19' );
+               return new DerivativeContext( $this->getContext() );
+       }
+
        /**
         * Set warning section for this module. Users should monitor this
         * section to notice any changes in API. Multiple calls to this
@@ -176,24 +199,24 @@ abstract class ApiBase {
         * @param $warning string Warning message
         */
        public function setWarning( $warning ) {
-               $data = $this->getResult()->getData();
+               $result = $this->getResult();
+               $data = $result->getData();
                if ( isset( $data['warnings'][$this->getModuleName()] ) ) {
                        // Don't add duplicate warnings
                        $warn_regex = preg_quote( $warning, '/' );
-                       if ( preg_match( "/{$warn_regex}(\\n|$)/", $data['warnings'][$this->getModuleName()]['*'] ) )
-                       {
+                       if ( preg_match( "/{$warn_regex}(\\n|$)/", $data['warnings'][$this->getModuleName()]['*'] ) ) {
                                return;
                        }
                        $oldwarning = $data['warnings'][$this->getModuleName()]['*'];
                        // If there is a warning already, append it to the existing one
                        $warning = "$oldwarning\n$warning";
-                       $this->getResult()->unsetValue( 'warnings', $this->getModuleName() );
+                       $result->unsetValue( 'warnings', $this->getModuleName() );
                }
                $msg = array();
                ApiResult::setContent( $msg, $warning );
-               $this->getResult()->disableSizeCheck();
-               $this->getResult()->addValue( 'warnings', $this->getModuleName(), $msg );
-               $this->getResult()->enableSizeCheck();
+               $result->disableSizeCheck();
+               $result->addValue( 'warnings', $this->getModuleName(), $msg );
+               $result->enableSizeCheck();
        }
 
        /**
@@ -213,7 +236,7 @@ abstract class ApiBase {
        public function makeHelpMsg() {
                static $lnPrfx = "\n  ";
 
-               $msg = $this->getDescription();
+               $msg = $this->getFinalDescription();
 
                if ( $msg !== false ) {
 
@@ -234,8 +257,7 @@ abstract class ApiBase {
                                $msg .= "\nThis module only accepts POST requests";
                        }
                        if ( $this->isReadMode() || $this->isWriteMode() ||
-                                       $this->mustBePosted() )
-                       {
+                                       $this->mustBePosted() ) {
                                $msg .= "\n";
                        }
 
@@ -245,17 +267,8 @@ abstract class ApiBase {
                                $msg .= "Parameters:\n$paramsMsg";
                        }
 
-                       // Examples
-                       $examples = $this->getExamples();
-                       if ( $examples !== false ) {
-                               if ( !is_array( $examples ) ) {
-                                       $examples = array(
-                                               $examples
-                                       );
-                               }
-                               $msg .= 'Example' . ( count( $examples ) > 1 ? 's' : '' ) . ":\n  ";
-                               $msg .= implode( $lnPrfx, $examples ) . "\n";
-                       }
+                       $msg .= $this->makeHelpArrayToString( $lnPrfx, "Example", $this->getExamples() );
+                       $msg .= $this->makeHelpArrayToString( $lnPrfx, "Help page", $this->getHelpUrls() );
 
                        if ( $this->getMain()->getShowVersions() ) {
                                $versions = $this->getVersion();
@@ -278,10 +291,34 @@ abstract class ApiBase {
                return $msg;
        }
 
+       /**
+        * @param $prefix string Text to split output items
+        * @param $title string What is being output
+        * @param $input string|array
+        * @return string
+        */
+       protected function makeHelpArrayToString( $prefix, $title, $input ) {
+               if ( $input === false ) {
+                       return '';
+               }
+               if ( !is_array( $input ) ) {
+                       $input = array(
+                               $input
+                       );
+               }
+
+               if ( count( $input ) > 0 ) {
+                       $msg = $title . ( count( $input ) > 1 ? 's' : '' ) . ":\n  ";
+                       $msg .= implode( $prefix, $input ) . "\n";
+                       return $msg;
+               }
+               return '';
+       }
+
        /**
         * Generates the parameter descriptions for this module, to be displayed in the
         * module's help.
-        * @return string
+        * @return string or false
         */
        public function makeHelpMsgParameters() {
                $params = $this->getFinalParams();
@@ -289,7 +326,8 @@ abstract class ApiBase {
 
                        $paramsDescription = $this->getFinalParamDescription();
                        $msg = '';
-                       $paramPrefix = "\n" . str_repeat( ' ', 19 );
+                       $paramPrefix = "\n" . str_repeat( ' ', 24 );
+                       $descWordwrap = "\n" . str_repeat( ' ', 28 );
                        foreach ( $params as $paramName => $paramSettings ) {
                                $desc = isset( $paramsDescription[$paramName] ) ? $paramsDescription[$paramName] : '';
                                if ( is_array( $desc ) ) {
@@ -307,7 +345,7 @@ abstract class ApiBase {
                                if ( $deprecated ) {
                                        $desc = "DEPRECATED! $desc";
                                }
-                               
+
                                $required = isset( $paramSettings[self::PARAM_REQUIRED] ) ?
                                        $paramSettings[self::PARAM_REQUIRED] : false;
                                if ( $required ) {
@@ -332,15 +370,23 @@ abstract class ApiBase {
                                                                $choices[] =  $t;
                                                        }
                                                }
-                                               $desc .= $paramPrefix . $nothingPrompt . $prompt . implode( ', ', $choices );
+                                               $desc .= $paramPrefix . $nothingPrompt . $prompt;
+                                               $choicesstring = implode( ', ', $choices );
+                                               $desc .= wordwrap( $choicesstring, 100, $descWordwrap );
                                        } else {
                                                switch ( $type ) {
                                                        case 'namespace':
                                                                // Special handling because namespaces are type-limited, yet they are not given
-                                                               $desc .= $paramPrefix . $prompt . implode( ', ', MWNamespace::getValidNamespaces() );
+                                                               $desc .= $paramPrefix . $prompt;
+                                                               $desc .= wordwrap( implode( ', ', MWNamespace::getValidNamespaces() ),
+                                                                       100, $descWordwrap );
                                                                break;
                                                        case 'limit':
-                                                               $desc .= $paramPrefix . "No more than {$paramSettings[self :: PARAM_MAX]} ({$paramSettings[self::PARAM_MAX2]} for bots) allowed";
+                                                               $desc .= $paramPrefix . "No more than {$paramSettings[self :: PARAM_MAX]}";
+                                                               if ( isset( $paramSettings[self::PARAM_MAX2] ) ) {
+                                                                       $desc .= " ({$paramSettings[self::PARAM_MAX2]} for bots)";
+                                                               }
+                                                               $desc .= ' allowed';
                                                                break;
                                                        case 'integer':
                                                                $hasMin = isset( $paramSettings[self::PARAM_MIN] );
@@ -358,15 +404,27 @@ abstract class ApiBase {
                                                                }
                                                                break;
                                                }
+
+                                               if ( isset( $paramSettings[self::PARAM_ISMULTI] ) ) {
+                                                       $isArray = is_array( $paramSettings[self::PARAM_TYPE] );
+
+                                                       if ( !$isArray
+                                                                       || $isArray && count( $paramSettings[self::PARAM_TYPE] ) > self::LIMIT_SML1 ) {
+                                                               $desc .= $paramPrefix . "Maximum number of values " .
+                                                                       self::LIMIT_SML1 . " (" . self::LIMIT_SML2 . " for bots)";
+                                                       }
+                                               }
                                        }
                                }
 
-                               $default = is_array( $paramSettings ) ? ( isset( $paramSettings[self::PARAM_DFLT] ) ? $paramSettings[self::PARAM_DFLT] : null ) : $paramSettings;
+                               $default = is_array( $paramSettings )
+                                               ? ( isset( $paramSettings[self::PARAM_DFLT] ) ? $paramSettings[self::PARAM_DFLT] : null )
+                                               : $paramSettings;
                                if ( !is_null( $default ) && $default !== false ) {
                                        $desc .= $paramPrefix . "Default: $default";
                                }
 
-                               $msg .= sprintf( "  %-14s - %s\n", $this->encodeParamName( $paramName ), $desc );
+                               $msg .= sprintf( "  %-19s - %s\n", $this->encodeParamName( $paramName ), $desc );
                        }
                        return $msg;
 
@@ -378,9 +436,14 @@ abstract class ApiBase {
        /**
         * Callback for preg_replace_callback() call in makeHelpMsg().
         * Replaces a source file name with a link to ViewVC
+        *
+        * @param $matches array
+        * @return string
         */
        public function makeHelpMsg_callback( $matches ) {
                global $wgAutoloadClasses, $wgAutoloadLocalClasses;
+
+               $file = '';
                if ( isset( $wgAutoloadLocalClasses[get_class( $this )] ) ) {
                        $file = $wgAutoloadLocalClasses[get_class( $this )];
                } elseif ( isset( $wgAutoloadClasses[get_class( $this )] ) ) {
@@ -401,7 +464,7 @@ abstract class ApiBase {
                // This is necessary to make stuff like ApiMain::getVersion()
                // returning the version string for ApiBase work
                if ( $path ) {
-                       return "{$matches[0]}\n   http://svn.wikimedia.org/" .
+                       return "{$matches[0]}\n   https://svn.wikimedia.org/" .
                                "viewvc/mediawiki/trunk/" . dirname( $path ) .
                                "/{$matches[2]}";
                }
@@ -417,8 +480,8 @@ abstract class ApiBase {
        }
 
        /**
-        * Returns usage examples for this module. Return null if no examples are available.
-        * @return mixed string or array of strings
+        * Returns usage examples for this module. Return false if no examples are available.
+        * @return false|string|array
         */
        protected function getExamples() {
                return false;
@@ -429,7 +492,7 @@ abstract class ApiBase {
         * value) or (parameter name) => (array with PARAM_* constants as keys)
         * Don't call this function directly: use getFinalParams() to allow
         * hooks to modify parameters as needed.
-        * @return array
+        * @return array or false
         */
        protected function getAllowedParams() {
                return false;
@@ -439,7 +502,7 @@ abstract class ApiBase {
         * Returns an array of parameter descriptions.
         * Don't call this functon directly: use getFinalParamDescription() to
         * allow hooks to modify descriptions as needed.
-        * @return array
+        * @return array or false
         */
        protected function getParamDescription() {
                return false;
@@ -448,7 +511,8 @@ abstract class ApiBase {
        /**
         * Get final list of parameters, after hooks have had a chance to
         * tweak it as needed.
-        * @return array
+        *
+        * @return array or false
         */
        public function getFinalParams() {
                $params = $this->getAllowedParams();
@@ -457,8 +521,9 @@ abstract class ApiBase {
        }
 
        /**
-        * Get final description, after hooks have had a chance to tweak it as
+        * Get final parameter descriptions, after hooks have had a chance to tweak it as
         * needed.
+        *
         * @return array
         */
        public function getFinalParamDescription() {
@@ -467,6 +532,18 @@ abstract class ApiBase {
                return $desc;
        }
 
+       /**
+        * Get final module description, after hooks have had a chance to tweak it as
+        * needed.
+        *
+        * @return array
+        */
+       public function getFinalDescription() {
+               $desc = $this->getDescription();
+               wfRunHooks( 'APIGetDescription', array( &$this, &$desc ) );
+               return $desc;
+       }
+
        /**
         * This method mangles parameter name based on the prefix supplied to the constructor.
         * Override this method to change parameter name during runtime
@@ -494,12 +571,8 @@ abstract class ApiBase {
 
                        if ( $params ) { // getFinalParams() can return false
                                foreach ( $params as $paramName => $paramSettings ) {
-                                       $results[$paramName] = $this->getParameterFromSettings( 
+                                       $results[$paramName] = $this->getParameterFromSettings(
                                                $paramName, $paramSettings, $parseLimit );
-
-                                       if( isset( $paramSettings[self::PARAM_REQUIRED] ) && !isset( $results[$paramName] ) ) {
-                                               $this->dieUsageMsg( array( 'missingparam', $paramName ) );
-                                       }
                                }
                        }
                        $this->mParamCache[$parseLimit] = $results;
@@ -528,8 +601,8 @@ abstract class ApiBase {
                array_shift( $required );
 
                $intersection = array_intersect( array_keys( array_filter( $params,
-                               create_function( '$x', 'return !is_null($x) && $x !== false;' )
-                       ) ), $required );
+                               array( $this, "parameterNotEmpty" ) ) ), $required );
+
                if ( count( $intersection ) > 1 ) {
                        $this->dieUsage( 'The parameters ' . implode( ', ', $intersection ) . ' can not be used together', 'invalidparammix' );
                } elseif ( count( $intersection ) == 0 ) {
@@ -538,9 +611,70 @@ abstract class ApiBase {
        }
 
        /**
-        * @deprecated use MWNamespace::getValidNamespaces()
+        * Generates the possible errors requireOnlyOneParameter() can die with
+        *
+        * @param $params array
+        * @return array
+        */
+       public function getRequireOnlyOneParameterErrorMessages( $params ) {
+               $p = $this->getModulePrefix();
+               $params = implode( ", {$p}", $params );
+
+               return array(
+                       array( 'code' => "{$p}missingparam", 'info' => "One of the parameters {$p}{$params} is required" ),
+                       array( 'code' => "{$p}invalidparammix", 'info' => "The parameters {$p}{$params} can not be used together" )
+               );
+       }
+
+       /**
+        * Die if more than one of a certain set of parameters is set and not false.
+        *
+        * @param $params array
+        */
+       public function requireMaxOneParameter( $params ) {
+               $required = func_get_args();
+               array_shift( $required );
+
+               $intersection = array_intersect( array_keys( array_filter( $params,
+                               array( $this, "parameterNotEmpty" ) ) ), $required );
+
+               if ( count( $intersection ) > 1 ) {
+                       $this->dieUsage( 'The parameters ' . implode( ', ', $intersection ) . ' can not be used together', 'invalidparammix' );
+               }
+       }
+
+       /**
+        * Generates the possible error requireMaxOneParameter() can die with
+        *
+        * @param $params array
+        * @return array
+        */
+       public function getRequireMaxOneParameterErrorMessages( $params ) {
+               $p = $this->getModulePrefix();
+               $params = implode( ", {$p}", $params );
+
+               return array(
+                       array( 'code' => "{$p}invalidparammix", 'info' => "The parameters {$p}{$params} can not be used together" )
+               );
+       }
+
+       /**
+        * Callback function used in requireOnlyOneParameter to check whether reequired parameters are set
+        *
+        * @param  $x object Parameter to check is not null/false
+        * @return bool
+        */
+       private function parameterNotEmpty( $x ) {
+               return !is_null( $x ) && $x !== false;
+       }
+
+       /**
+        * @deprecated since 1.17 use MWNamespace::getValidNamespaces()
+        *
+        * @return array
         */
        public static function getValidNamespaces() {
+               wfDeprecated( __METHOD__, '1.17' );
                return MWNamespace::getValidNamespaces();
        }
 
@@ -548,11 +682,14 @@ abstract class ApiBase {
         * Return true if we're to watch the page, false if not, null if no change.
         * @param $watchlist String Valid values: 'watch', 'unwatch', 'preferences', 'nochange'
         * @param $titleObj Title the page under consideration
-        * @param $userOption The user option to consider when $watchlist=preferences.
+        * @param $userOption String The user option to consider when $watchlist=preferences.
         *      If not set will magically default to either watchdefault or watchcreations
-        * @returns mixed
+        * @return bool
         */
        protected function getWatchlistValue ( $watchlist, $titleObj, $userOption = null ) {
+
+               $userWatching = $titleObj->userIsWatching();
+
                switch ( $watchlist ) {
                        case 'watch':
                                return true;
@@ -561,24 +698,23 @@ abstract class ApiBase {
                                return false;
 
                        case 'preferences':
-                               global $wgUser;
                                # If the user is already watching, don't bother checking
-                               if ( $titleObj->userIsWatching() ) {
-                                       return null;
+                               if ( $userWatching ) {
+                                       return true;
                                }
                                # If no user option was passed, use watchdefault or watchcreation
                                if ( is_null( $userOption ) ) {
                                        $userOption = $titleObj->exists()
                                                ? 'watchdefault' : 'watchcreations';
                                }
-                               # If the corresponding user option is true, watch, else no change
-                               return $wgUser->getOption( $userOption ) ? true : null;
+                               # Watch the article based on the user preference
+                               return (bool)$this->getUser()->getOption( $userOption );
 
                        case 'nochange':
-                               return null;
+                               return $userWatching;
 
                        default:
-                               return null;
+                               return $userWatching;
                }
        }
 
@@ -586,19 +722,19 @@ abstract class ApiBase {
         * Set a watch (or unwatch) based the based on a watchlist parameter.
         * @param $watch String Valid values: 'watch', 'unwatch', 'preferences', 'nochange'
         * @param $titleObj Title the article's title to change
-        * @param $userOption The user option to consider when $watch=preferences
+        * @param $userOption String The user option to consider when $watch=preferences
         */
-       protected function setWatch ( $watch, $titleObj, $userOption = null ) {
+       protected function setWatch( $watch, $titleObj, $userOption = null ) {
                $value = $this->getWatchlistValue( $watch, $titleObj, $userOption );
                if ( $value === null ) {
                        return;
                }
 
-               $articleObj = new Article( $titleObj );
+               $user = $this->getUser();
                if ( $value ) {
-                       $articleObj->doWatch();
+                       WatchAction::doWatch( $titleObj, $user );
                } else {
-                       $articleObj->doUnwatch();
+                       WatchAction::doUnwatch( $titleObj, $user );
                }
        }
 
@@ -621,12 +757,14 @@ abstract class ApiBase {
                        $type = gettype( $paramSettings );
                        $dupes = false;
                        $deprecated = false;
+                       $required = false;
                } else {
                        $default = isset( $paramSettings[self::PARAM_DFLT] ) ? $paramSettings[self::PARAM_DFLT] : null;
                        $multi = isset( $paramSettings[self::PARAM_ISMULTI] ) ? $paramSettings[self::PARAM_ISMULTI] : false;
                        $type = isset( $paramSettings[self::PARAM_TYPE] ) ? $paramSettings[self::PARAM_TYPE] : null;
                        $dupes = isset( $paramSettings[self::PARAM_ALLOW_DUPLICATES] ) ? $paramSettings[self::PARAM_ALLOW_DUPLICATES] : false;
                        $deprecated = isset( $paramSettings[self::PARAM_DEPRECATED] ) ? $paramSettings[self::PARAM_DEPRECATED] : false;
+                       $required = isset( $paramSettings[self::PARAM_REQUIRED] ) ? $paramSettings[self::PARAM_REQUIRED] : false;
 
                        // When type is not given, and no choices, the type is the same as $default
                        if ( !isset( $type ) ) {
@@ -644,9 +782,9 @@ abstract class ApiBase {
                                ApiBase::dieDebug( __METHOD__, "Boolean param $encParamName's default is set to '$default'" );
                        }
 
-                       $value = $this->getMain()->getRequest()->getCheck( $encParamName );
+                       $value = $this->getRequest()->getCheck( $encParamName );
                } else {
-                       $value = $this->getMain()->getRequest()->getVal( $encParamName, $default );
+                       $value = $this->getRequest()->getVal( $encParamName, $default );
 
                        if ( isset( $value ) && $type == 'namespace' ) {
                                $type = MWNamespace::getValidNamespaces();
@@ -664,18 +802,29 @@ abstract class ApiBase {
                                switch ( $type ) {
                                        case 'NULL': // nothing to do
                                                break;
-                                       case 'string': // nothing to do
+                                       case 'string':
+                                               if ( $required && $value === '' ) {
+                                                       $this->dieUsageMsg( array( 'missingparam', $paramName ) );
+                                               }
+
                                                break;
                                        case 'integer': // Force everything using intval() and optionally validate limits
-
-                                               $value = is_array( $value ) ? array_map( 'intval', $value ) : intval( $value );
                                                $min = isset ( $paramSettings[self::PARAM_MIN] ) ? $paramSettings[self::PARAM_MIN] : null;
                                                $max = isset ( $paramSettings[self::PARAM_MAX] ) ? $paramSettings[self::PARAM_MAX] : null;
-
-                                               if ( !is_null( $min ) || !is_null( $max ) ) {
-                                                       $values = is_array( $value ) ? $value : array( $value );
-                                                       foreach ( $values as &$v ) {
-                                                               $this->validateLimit( $paramName, $v, $min, $max );
+                                               $enforceLimits = isset ( $paramSettings[self::PARAM_RANGE_ENFORCE] )
+                                                               ? $paramSettings[self::PARAM_RANGE_ENFORCE] : false;
+
+                                               if ( is_array( $value ) ) {
+                                                       $value = array_map( 'intval', $value );
+                                                       if ( !is_null( $min ) || !is_null( $max ) ) {
+                                                               foreach ( $value as &$v ) {
+                                                                       $this->validateLimit( $paramName, $v, $min, $max, null, $enforceLimits );
+                                                               }
+                                                       }
+                                               } else {
+                                                       $value = intval( $value );
+                                                       if ( !is_null( $min ) || !is_null( $max ) ) {
+                                                               $this->validateLimit( $paramName, $value, $min, $max, null, $enforceLimits );
                                                        }
                                                }
                                                break;
@@ -705,19 +854,18 @@ abstract class ApiBase {
                                                }
                                                break;
                                        case 'timestamp':
-                                               if ( $multi ) {
-                                                       ApiBase::dieDebug( __METHOD__, "Multi-values not supported for $encParamName" );
-                                               }
-                                               $value = wfTimestamp( TS_UNIX, $value );
-                                               if ( $value === 0 ) {
-                                                       $this->dieUsage( "Invalid value '$value' for timestamp parameter $encParamName", "badtimestamp_{$encParamName}" );
+                                               if ( is_array( $value ) ) {
+                                                       foreach ( $value as $key => $val ) {
+                                                               $value[$key] = $this->validateTimestamp( $val, $encParamName );
+                                                       }
+                                               } else {
+                                                       $value = $this->validateTimestamp( $value, $encParamName );
                                                }
-                                               $value = wfTimestamp( TS_MW, $value );
                                                break;
                                        case 'user':
                                                if ( !is_array( $value ) ) {
-                            $value = array( $value );
-                        }
+                                                       $value = array( $value );
+                                               }
 
                                                foreach ( $value as $key => $val ) {
                                                        $title = Title::makeTitleSafe( NS_USER, $val );
@@ -728,9 +876,9 @@ abstract class ApiBase {
                                                }
 
                                                if ( !$multi ) {
-                            $value = $value[0];
-                        }
-                        break;
+                                                       $value = $value[0];
+                                               }
+                                               break;
                                        default:
                                                ApiBase::dieDebug( __METHOD__, "Param $encParamName's type is unknown - $type" );
                                }
@@ -745,6 +893,8 @@ abstract class ApiBase {
                        if ( $deprecated && $value !== false ) {
                                $this->setWarning( "The $encParamName parameter has been deprecated." );
                        }
+               } elseif ( $required ) {
+                       $this->dieUsageMsg( array( 'missingparam', $paramName ) );
                }
 
                return $value;
@@ -806,13 +956,16 @@ abstract class ApiBase {
         * Prints usage info on failure.
         * @param $paramName string Parameter name
         * @param $value int Parameter value
-        * @param $min int Minimum value
-        * @param $max int Maximum value for users
+        * @param $min int|null Minimum value
+        * @param $max int|null Maximum value for users
         * @param $botMax int Maximum value for sysops/bots
+        * @param $enforceLimits Boolean Whether to enforce (die) if value is outside limits
         */
-       function validateLimit( $paramName, &$value, $min, $max, $botMax = null ) {
+       function validateLimit( $paramName, &$value, $min, $max, $botMax = null, $enforceLimits = false ) {
                if ( !is_null( $min ) && $value < $min ) {
-                       $this->setWarning( $this->encodeParamName( $paramName ) . " may not be less than $min (set to $value)" );
+
+                       $msg = $this->encodeParamName( $paramName ) . " may not be less than $min (set to $value)";
+                       $this->warnOrDie( $msg, $enforceLimits );
                        $value = $min;
                }
 
@@ -826,16 +979,45 @@ abstract class ApiBase {
                if ( !is_null( $max ) && $value > $max ) {
                        if ( !is_null( $botMax ) && $this->getMain()->canApiHighLimits() ) {
                                if ( $value > $botMax ) {
-                                       $this->setWarning( $this->encodeParamName( $paramName ) . " may not be over $botMax (set to $value) for bots or sysops" );
+                                       $msg = $this->encodeParamName( $paramName ) . " may not be over $botMax (set to $value) for bots or sysops";
+                                       $this->warnOrDie( $msg, $enforceLimits );
                                        $value = $botMax;
                                }
                        } else {
-                               $this->setWarning( $this->encodeParamName( $paramName ) . " may not be over $max (set to $value) for users" );
+                               $msg = $this->encodeParamName( $paramName ) . " may not be over $max (set to $value) for users";
+                               $this->warnOrDie( $msg, $enforceLimits );
                                $value = $max;
                        }
                }
        }
 
+       /**
+        * @param $value string
+        * @param $paramName string
+        * @return string
+        */
+       function validateTimestamp( $value, $paramName ) {
+               $value = wfTimestamp( TS_UNIX, $value );
+               if ( $value === 0 ) {
+                       $this->dieUsage( "Invalid value '$value' for timestamp parameter $paramName", "badtimestamp_{$paramName}" );
+               }
+               return wfTimestamp( TS_MW, $value );
+       }
+
+       /**
+        * Adds a warning to the output, else dies
+        *
+        * @param  $msg String Message to show as a warning, or error message if dying
+        * @param  $enforceLimits Boolean Whether this is an enforce (die)
+        */
+       private function warnOrDie( $msg, $enforceLimits = false ) {
+               if ( $enforceLimits ) {
+                       $this->dieUsage( $msg, 'integeroutofrange' );
+               } else {
+                       $this->setWarning( $msg );
+               }
+       }
+
        /**
         * Truncate an array to a certain length.
         * @param $arr array Array to truncate
@@ -863,7 +1045,7 @@ abstract class ApiBase {
         * @param $extradata array Data to add to the <error> element; array in ApiResult format
         */
        public function dieUsage( $description, $errorCode, $httpRespCode = 0, $extradata = null ) {
-               wfProfileClose();
+               Profiler::instance()->close();
                throw new UsageException( $description, $this->encodeParamName( $errorCode ), $httpRespCode, $extradata );
        }
 
@@ -879,7 +1061,8 @@ abstract class ApiBase {
                'ns-specialprotected' => array( 'code' => 'unsupportednamespace', 'info' => "Pages in the Special namespace can't be edited" ),
                'protectedinterface' => array( 'code' => 'protectednamespace-interface', 'info' => "You're not allowed to edit interface messages" ),
                'namespaceprotected' => array( 'code' => 'protectednamespace', 'info' => "You're not allowed to edit pages in the ``\$1'' namespace" ),
-               'customcssjsprotected' => array( 'code' => 'customcssjsprotected', 'info' => "You're not allowed to edit custom CSS and JavaScript pages" ),
+               'customcssprotected' => array( 'code' => 'customcssprotected', 'info' => "You're not allowed to edit custom CSS pages" ),
+               'customjsprotected' => array( 'code' => 'customjsprotected', 'info' => "You're not allowed to edit custom JavaScript pages" ),
                'cascadeprotected' => array( 'code' => 'cascadeprotected', 'info' => "The page you're trying to edit is protected because it's included in a cascade-protected page" ),
                'protectedpagetext' => array( 'code' => 'protectedpage', 'info' => "The ``\$1'' right is required to edit this page" ),
                'protect-cantedit' => array( 'code' => 'cantedit', 'info' => "You can't protect this page because you can't edit it" ),
@@ -962,7 +1145,6 @@ abstract class ApiBase {
                'createonly-exists' => array( 'code' => 'articleexists', 'info' => "The article you tried to create has been created already" ),
                'nocreate-missing' => array( 'code' => 'missingtitle', 'info' => "The article you tried to edit doesn't exist" ),
                'nosuchrcid' => array( 'code' => 'nosuchrcid', 'info' => "There is no change with rcid ``\$1''" ),
-               'cantpurge' => array( 'code' => 'cantpurge', 'info' => "Only users with the 'purge' right can purge pages via the API" ),
                'protect-invalidaction' => array( 'code' => 'protect-invalidaction', 'info' => "Invalid protection type ``\$1''" ),
                'protect-invalidlevel' => array( 'code' => 'protect-invalidlevel', 'info' => "Invalid protection level ``\$1''" ),
                'toofewexpiries' => array( 'code' => 'toofewexpiries', 'info' => "\$1 expiry timestamps were provided where \$2 were needed" ),
@@ -981,6 +1163,9 @@ abstract class ApiBase {
                'sharedfile-exists' => array( 'code' => 'fileexists-sharedrepo-perm', 'info' => 'The target file exists on a shared repository. Use the ignorewarnings parameter to override it.' ),
                'mustbeposted' => array( 'code' => 'mustbeposted', 'info' => "The \$1 module requires a POST request" ),
                'show' => array( 'code' => 'show', 'info' => 'Incorrect parameter - mutually exclusive values may not be supplied' ),
+               'specialpage-cantexecute' => array( 'code' => 'specialpage-cantexecute', 'info' => "You don't have permission to view the results of this special page" ),
+               'invalidoldimage' => array( 'code' => 'invalidoldimage', 'info' => 'The oldimage parameter has invalid format' ),
+               'nodeleteablefile' => array( 'code' => 'nodeleteablefile', 'info' => 'No such old version of the file' ),
 
                // ApiEditPage messages
                'noimageredirect-anon' => array( 'code' => 'noimageredirect-anon', 'info' => "Anonymous users can't create image redirects" ),
@@ -999,11 +1184,22 @@ abstract class ApiBase {
                'revwrongpage' => array( 'code' => 'revwrongpage', 'info' => "r\$1 is not a revision of ``\$2''" ),
                'undo-failure' => array( 'code' => 'undofailure', 'info' => 'Undo failed due to conflicting intermediate edits' ),
 
+               // Messages from WikiPage::doEit()
+               'edit-hook-aborted' => array( 'code' => 'edit-hook-aborted', 'info' => "Your edit was aborted by an ArticleSave hook" ),
+               'edit-gone-missing' => array( 'code' => 'edit-gone-missing', 'info' => "The page you tried to edit doesn't seem to exist anymore" ),
+               'edit-conflict' => array( 'code' => 'editconflict', 'info' => "Edit conflict detected" ),
+               'edit-already-exists' => array( 'code' => 'edit-already-exists', 'info' => "It seems the page you tried to create already exist" ),
+
                // uploadMsgs
                'invalid-session-key' => array( 'code' => 'invalid-session-key', 'info' => 'Not a valid session key' ),
                'nouploadmodule' => array( 'code' => 'nouploadmodule', 'info' => 'No upload module set' ),
                'uploaddisabled' => array( 'code' => 'uploaddisabled', 'info' => 'Uploads are not enabled.  Make sure $wgEnableUploads is set to true in LocalSettings.php and the PHP ini setting file_uploads is true' ),
                'copyuploaddisabled' => array( 'code' => 'copyuploaddisabled', 'info' => 'Uploads by URL is not enabled.  Make sure $wgAllowCopyUploads is set to true in LocalSettings.php.' ),
+
+               'filename-tooshort' => array( 'code' => 'filename-tooshort', 'info' => 'The filename is too short' ),
+               'filename-toolong' => array( 'code' => 'filename-toolong', 'info' => 'The filename is too long' ),
+               'illegal-filename' => array( 'code' => 'illegal-filename', 'info' => 'The filename is not allowed' ),
+               'filetype-missing' => array( 'code' => 'filetype-missing', 'info' => 'The file is missing an extension' ),
        );
 
        /**
@@ -1017,9 +1213,14 @@ abstract class ApiBase {
 
        /**
         * Output the error message related to a certain array
-        * @param $error array Element of a getUserPermissionsErrors()-style array
+        * @param $error (array|string) Element of a getUserPermissionsErrors()-style array
         */
        public function dieUsageMsg( $error ) {
+               # most of the time we send a 1 element, so we might as well send it as
+               # a string and make this an array here.
+               if( is_string( $error ) ) {
+                       $error = array( $error );
+               }
                $parsed = $this->parseMsg( $error );
                $this->dieUsage( $parsed['info'], $parsed['code'] );
        }
@@ -1031,6 +1232,14 @@ abstract class ApiBase {
         */
        public function parseMsg( $error ) {
                $key = array_shift( $error );
+
+               // Check whether the error array was nested
+               // array( array( <code>, <params> ), array( <another_code>, <params> ) )
+               if( is_array( $key ) ){
+                       $error = $key;
+                       $key = array_shift( $error );
+               }
+
                if ( isset( self::$messageMap[$key] ) ) {
                        return array( 'code' =>
                                wfMsgReplaceArgs( self::$messageMap[$key]['code'], $error ),
@@ -1038,6 +1247,7 @@ abstract class ApiBase {
                                wfMsgReplaceArgs( self::$messageMap[$key]['info'], $error )
                        );
                }
+
                // If the key isn't present, throw an "unknown error"
                return $this->parseMsg( array( 'unknownerror', $key ) );
        }
@@ -1082,24 +1292,32 @@ abstract class ApiBase {
                return false;
        }
 
+       /**
+        * Returns whether this module requires a Token to execute
+        * @return bool
+        */
+       public function needsToken() {
+               return false;
+       }
+
        /**
         * Returns the token salt if there is one, '' if the module doesn't require a salt, else false if the module doesn't need a token
-        * @returns bool
+        * @return bool|string
         */
        public function getTokenSalt() {
                return false;
        }
 
        /**
-       * Gets the user for whom to get the watchlist
-       *
-       * @returns User
-       */
+        * Gets the user for whom to get the watchlist
+        *
+        * @param $params array
+        * @return User
+        */
        public function getWatchlistUser( $params ) {
-               global $wgUser;
                if ( !is_null( $params['owner'] ) && !is_null( $params['token'] ) ) {
                        $user = User::newFromName( $params['owner'], false );
-                       if ( !$user->getId() ) {
+                       if ( !($user && $user->getId()) ) {
                                $this->dieUsage( 'Specified user does not exist', 'bad_wlowner' );
                        }
                        $token = $user->getOption( 'watchlisttoken' );
@@ -1107,14 +1325,21 @@ abstract class ApiBase {
                                $this->dieUsage( 'Incorrect watchlist token provided -- please set a correct token in Special:Preferences', 'bad_wltoken' );
                        }
                } else {
-                       if ( !$wgUser->isLoggedIn() ) {
+                       if ( !$this->getUser()->isLoggedIn() ) {
                                $this->dieUsage( 'You must be logged-in to have a watchlist', 'notloggedin' );
                        }
-                       $user = $wgUser;
+                       $user = $this->getUser();
                }
                return $user;
        }
 
+       /**
+        * @return false|string|array Returns a false if the module has no help url, else returns a (array of) string
+        */
+       public function getHelpUrls() {
+               return false;
+       }
+
        /**
         * Returns a list of all possible errors returned by the module
         * @return array in the format of array( key, param1, param2, ... ) or array( 'code' => ..., 'info' => ... )
@@ -1125,7 +1350,7 @@ abstract class ApiBase {
                $params = $this->getFinalParams();
                if ( $params ) {
                        foreach ( $params as $paramName => $paramSettings ) {
-                               if( isset( $paramSettings[ApiBase::PARAM_REQUIRED] ) ) {
+                               if ( isset( $paramSettings[ApiBase::PARAM_REQUIRED] ) ) {
                                        $ret[] = array( 'missingparam', $paramName );
                                }
                        }
@@ -1144,7 +1369,7 @@ abstract class ApiBase {
                        $ret[] = array( 'writedisabled' );
                }
 
-               if ( $this->getTokenSalt() !== false ) {
+               if ( $this->needsToken() ) {
                        $ret[] = array( 'missingparam', 'token' );
                        $ret[] = array( 'sessionfailure' );
                }
@@ -1275,6 +1500,13 @@ abstract class ApiBase {
                return $this->mDBTime;
        }
 
+       /**
+        * @return DatabaseBase
+        */
+       protected function getDB() {
+               return wfGetDB( DB_SLAVE, 'api' );
+       }
+
        /**
         * Debugging function that prints a value and an optional backtrace
         * @param $value mixed Value to print