Explicit variable definition, tweak documentation
[lhc/web/wiklou.git] / includes / api / ApiBase.php
index be1be86..df7ca1b 100644 (file)
@@ -1,10 +1,9 @@
 <?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
  *
  * This program is free software; you can redistribute it and/or modify
@@ -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
  */
 
 /**
@@ -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
@@ -180,8 +182,7 @@ abstract class ApiBase {
                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()]['*'];
@@ -253,8 +254,11 @@ abstract class ApiBase {
                                                $examples
                                        );
                                }
-                               $msg .= 'Example' . ( count( $examples ) > 1 ? 's' : '' ) . ":\n  ";
-                               $msg .= implode( $lnPrfx, $examples ) . "\n";
+
+                               if ( count( $examples ) > 0 ) {
+                                       $msg .= 'Example' . ( count( $examples ) > 1 ? 's' : '' ) . ":\n  ";
+                                       $msg .= implode( $lnPrfx, $examples ) . "\n";
+                               }
                        }
 
                        if ( $this->getMain()->getShowVersions() ) {
@@ -307,7 +311,7 @@ abstract class ApiBase {
                                if ( $deprecated ) {
                                        $desc = "DEPRECATED! $desc";
                                }
-                               
+
                                $required = isset( $paramSettings[self::PARAM_REQUIRED] ) ?
                                        $paramSettings[self::PARAM_REQUIRED] : false;
                                if ( $required ) {
@@ -340,7 +344,11 @@ abstract class ApiBase {
                                                                $desc .= $paramPrefix . $prompt . implode( ', ', MWNamespace::getValidNamespaces() );
                                                                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] );
@@ -494,12 +502,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 +532,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 ) {
@@ -537,6 +541,16 @@ abstract class ApiBase {
                }
        }
 
+       /**
+        * 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 use MWNamespace::getValidNamespaces()
         */
@@ -548,11 +562,12 @@ 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
         */
        protected function getWatchlistValue ( $watchlist, $titleObj, $userOption = null ) {
+               global $wgUser;
                switch ( $watchlist ) {
                        case 'watch':
                                return true;
@@ -561,7 +576,6 @@ abstract class ApiBase {
                                return false;
 
                        case 'preferences':
-                               global $wgUser;
                                # If the user is already watching, don't bother checking
                                if ( $titleObj->userIsWatching() ) {
                                        return null;
@@ -586,7 +600,7 @@ 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 ) {
                $value = $this->getWatchlistValue( $watch, $titleObj, $userOption );
@@ -621,12 +635,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 ) ) {
@@ -664,19 +680,28 @@ 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;
+                                               $enforceLimits = isset ( $paramSettings[self::PARAM_RANGE_ENFORCE] )
+                                                               ? $paramSettings[self::PARAM_RANGE_ENFORCE] : false;
 
                                                if ( !is_null( $min ) || !is_null( $max ) ) {
-                                                       $values = is_array( $value ) ? $value : array( $value );
-                                                       foreach ( $values as &$v ) {
-                                                               $this->validateLimit( $paramName, $v, $min, $max );
-                                                       }
+                                                   if ( is_array( $value ) ) {
+                                                           $value = array_map( 'intval', $value );
+                                                           foreach ( $value as &$v ) {
+                                                                       $this->validateLimit( $paramName, $v, $min, $max, null, $enforceLimits );
+                                                               }
+                                                   } else {
+                                                           $value = intval( $value );
+                                                           $this->validateLimit( $paramName, $value, $min, $max, null, $enforceLimits );
+                                                   }
                                                }
                                                break;
                                        case 'limit':
@@ -716,8 +741,8 @@ abstract class ApiBase {
                                                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 +753,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 +770,8 @@ abstract class ApiBase {
                        if ( $deprecated && $value !== false ) {
                                $this->setWarning( "The $encParamName parameter has been deprecated." );
                        }
+               } else if ( $required ) {
+                       $this->dieUsageMsg( array( 'missingparam', $paramName ) );
                }
 
                return $value;
@@ -809,10 +836,13 @@ abstract class ApiBase {
         * @param $min int Minimum value
         * @param $max int 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 +856,32 @@ 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;
                        }
                }
        }
 
+       /**
+        * 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
@@ -962,7 +1008,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" ),
@@ -1082,6 +1127,14 @@ abstract class ApiBase {
                return false;
        }
 
+       /**
+        * Returns whether this module requires a Token to execute
+        * @returns 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
@@ -1125,7 +1178,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 +1197,7 @@ abstract class ApiBase {
                        $ret[] = array( 'writedisabled' );
                }
 
-               if ( $this->getTokenSalt() !== false ) {
+               if ( $this->needsToken() ) {
                        $ret[] = array( 'missingparam', 'token' );
                        $ret[] = array( 'sessionfailure' );
                }