Tweak the version info for MW just a bit, bring it more in line with the module listi...
[lhc/web/wiklou.git] / includes / api / ApiMain.php
index 5765c97..63e5377 100644 (file)
@@ -28,6 +28,10 @@ if (!defined('MEDIAWIKI')) {
        require_once ('ApiBase.php');
 }
 
+/**
+ * @defgroup API API
+ */
+
 /**
  * This is the main API class, used for both external and internal processing.
  * When executed, it will create the requested formatter object,
@@ -37,9 +41,9 @@ if (!defined('MEDIAWIKI')) {
  *
  * To use API from another application, run it using FauxRequest object, in which
  * case any internal exceptions will not be handled but passed up to the caller.
- * After successful execution, use getResult() for the resulting data.   
- *  
- * @addtogroup API
+ * After successful execution, use getResult() for the resulting data.
+ *
+ * @ingroup API
  */
 class ApiMain extends ApiBase {
 
@@ -62,7 +66,7 @@ class ApiMain extends ApiBase {
                'help' => 'ApiHelp',
                'paraminfo' => 'ApiParamInfo',
        );
-       
+
        private static $WriteModules = array (
                'rollback' => 'ApiRollback',
                'delete' => 'ApiDelete',
@@ -72,6 +76,7 @@ class ApiMain extends ApiBase {
                'unblock' => 'ApiUnblock',
                'move' => 'ApiMove',
                'edit' => 'ApiEditPage',
+               'emailuser' => 'ApiEmailUser',
        );
 
        /**
@@ -95,7 +100,7 @@ class ApiMain extends ApiBase {
                'dbgfm' => 'ApiFormatDbg'
        );
 
-       private $mPrinter, $mModules, $mModuleNames, $mFormats, $mFormatNames, $mCommit;
+       private $mPrinter, $mModules, $mModuleNames, $mFormats, $mFormatNames;
        private $mResult, $mAction, $mShowVersions, $mEnableWrite, $mRequest, $mInternalMode, $mSquidMaxage;
 
        /**
@@ -112,25 +117,25 @@ class ApiMain extends ApiBase {
                parent :: __construct($this, $this->mInternalMode ? 'main_int' : 'main');
 
                if (!$this->mInternalMode) {
-                       
+
                        // Impose module restrictions.
-                       // If the current user cannot read, 
+                       // If the current user cannot read,
                        // Remove all modules other than login
                        global $wgUser;
-                       
+
                        if( $request->getVal( 'callback' ) !== null ) {
                                // JSON callback allows cross-site reads.
                                // For safety, strip user credentials.
                                wfDebug( "API: stripping user credentials for JSON callback\n" );
                                $wgUser = new User();
                        }
-                       
+
                        if (!$wgUser->isAllowed('read')) {
                                self::$Modules = array(
                                        'login'  => self::$Modules['login'],
                                        'logout' => self::$Modules['logout'],
                                        'help'   => self::$Modules['help'],
-                                       ); 
+                                       );
                        }
                }
 
@@ -149,7 +154,7 @@ class ApiMain extends ApiBase {
 
                $this->mRequest = & $request;
 
-               $this->mSquidMaxage = 0;
+               $this->mSquidMaxage = -1; // flag for executeActionWithErrorHandling()
                $this->mCommit = false;
        }
 
@@ -175,12 +180,19 @@ class ApiMain extends ApiBase {
        }
 
        /**
-        * This method will simply cause an error if the write mode was disabled for this api.
+        * This method will simply cause an error if the write mode was disabled
+        * or if the current user doesn't have the right to use it
         */
        public function requestWriteMode() {
+               global $wgUser;
                if (!$this->mEnableWrite)
-                       $this->dieUsage('Editing of this site is disabled. Make sure the $wgEnableWriteAPI=true; ' .
-                       'statement is included in the site\'s LocalSettings.php file', 'noapiwrite');
+                       $this->dieUsage('Editing of this wiki through the API' .
+                       ' is disabled. Make sure the $wgEnableWriteAPI=true; ' .
+                       'statement is included in the wiki\'s ' .
+                       'LocalSettings.php file', 'noapiwrite');
+               if (!$wgUser->isAllowed('writeapi'))
+                       $this->dieUsage('You\'re not allowed to edit this ' .
+                       'wiki through the API', 'writeapidenied');
        }
 
        /**
@@ -196,16 +208,9 @@ class ApiMain extends ApiBase {
        public function createPrinterByName($format) {
                return new $this->mFormats[$format] ($this, $format);
        }
-       
-       /**
-        * Schedule a database commit
-        */
-       public function scheduleCommit() {
-               $this->mCommit = true;
-       }
 
        /**
-        * Execute api request. Any errors will be handled if the API was called by the remote client. 
+        * Execute api request. Any errors will be handled if the API was called by the remote client.
         */
        public function execute() {
                $this->profileIn();
@@ -213,11 +218,7 @@ class ApiMain extends ApiBase {
                        $this->executeAction();
                else
                        $this->executeActionWithErrorHandling();
-               if($this->mCommit)
-               {
-                       $dbw = wfGetDb(DB_MASTER);
-                       $dbw->immediateCommit();
-               }
+       
                $this->profileOut();
        }
 
@@ -259,11 +260,22 @@ class ApiMain extends ApiBase {
                        $this->printResult(true);
                }
 
+               global $wgRequest;
+               if($this->mSquidMaxage == -1)
+               {
+                       # Nobody called setCacheMaxAge(), use the (s)maxage parameters
+                       $smaxage = $wgRequest->getVal('smaxage', 0);
+                       $maxage = $wgRequest->getVal('maxage', 0);
+               }
+               else
+                       $smaxage = $maxage = $this->mSquidMaxage;
+
                // Set the cache expiration at the last moment, as any errors may change the expiration.
                // if $this->mSquidMaxage == 0, the expiry time is set to the first second of unix epoch
-               $expires = $this->mSquidMaxage == 0 ? 1 : time() + $this->mSquidMaxage;
+               $exp = min($smaxage, $maxage);
+               $expires = ($exp == 0 ? 1 : time() + $exp);
                header('Expires: ' . wfTimestamp(TS_RFC2822, $expires));
-               header('Cache-Control: s-maxage=' . $this->mSquidMaxage . ', must-revalidate, max-age=0');
+               header('Cache-Control: s-maxage=' . $smaxage . ', must-revalidate, max-age=' . $maxage);
 
                if($this->mPrinter->getIsHtml())
                        echo wfReportTime();
@@ -273,10 +285,10 @@ class ApiMain extends ApiBase {
 
        /**
         * Replace the result data with the information about an exception.
-        * Returns the error code 
+        * Returns the error code
         */
        protected function substituteResultWithError($e) {
-       
+
                        // Printer may not be initialized if the extractRequestParams() fails for the main module
                        if (!isset ($this->mPrinter)) {
                                // The printer has not been created yet. Try to manually get formatter value.
@@ -296,20 +308,27 @@ class ApiMain extends ApiBase {
                                $errMessage = array (
                                'code' => $e->getCodeString(),
                                'info' => $e->getMessage());
-                               
+
                                // Only print the help message when this is for the developer, not runtime
                                if ($this->mPrinter->getIsHtml() || $this->mAction == 'help')
                                        ApiResult :: setContent($errMessage, $this->makeHelpMsg());
 
                        } else {
+                               global $wgShowSQLErrors, $wgShowExceptionDetails;
                                //
                                // Something is seriously wrong
                                //
+                               if ( ( $e instanceof DBQueryError ) && !$wgShowSQLErrors ) {
+                                       $info = "Database query error";
+                               } else {
+                                       $info = "Exception Caught: {$e->getMessage()}";
+                               }
+
                                $errMessage = array (
                                        'code' => 'internal_api_error_'. get_class($e),
-                                       'info' => "Exception Caught: {$e->getMessage()}"
+                                       'info' => $info,
                                );
-                               ApiResult :: setContent($errMessage, "\n\n{$e->getTraceAsString()}\n\n");
+                               ApiResult :: setContent($errMessage, $wgShowExceptionDetails ? "\n\n{$e->getTraceAsString()}\n\n" : "" );                               
                        }
 
                        $this->getResult()->reset();
@@ -328,9 +347,13 @@ class ApiMain extends ApiBase {
                $this->mShowVersions = $params['version'];
                $this->mAction = $params['action'];
 
+               if( !is_string( $this->mAction ) ) {
+                       $this->dieUsage( "The API requires a valid action parameter", 'unknown_action' );
+               }
+
                // Instantiate the module requested by the user
                $module = new $this->mModules[$this->mAction] ($this, $this->mAction);
-               
+
                if( $module->shouldCheckMaxlag() && isset( $params['maxlag'] ) ) {
                        // Check for maxlag
                        global $wgShowHostnames;
@@ -345,7 +368,7 @@ class ApiMain extends ApiBase {
                                return;
                        }
                }
-               
+
                if (!$this->mInternalMode) {
                        // Ignore mustBePosted() for internal calls
                        if($module->mustBePosted() && !$this->mRequest->wasPosted())
@@ -379,11 +402,11 @@ class ApiMain extends ApiBase {
        protected function printResult($isError) {
                $printer = $this->mPrinter;
                $printer->profileIn();
-       
+
                /* If the help message is requested in the default (xmlfm) format,
                 * tell the printer not to escape ampersands so that our links do
                 * not break. */
-               $printer->setUnescapeAmps ( ( $this->mAction == 'help' || $isError ) 
+               $printer->setUnescapeAmps ( ( $this->mAction == 'help' || $isError )
                                && $this->getParameter('format') == ApiMain::API_DEFAULT_FORMAT );
 
                $printer->initPrinter($isError);
@@ -410,6 +433,14 @@ class ApiMain extends ApiBase {
                        'maxlag'  => array (
                                ApiBase :: PARAM_TYPE => 'integer'
                        ),
+                       'smaxage' => array (
+                               ApiBase :: PARAM_TYPE => 'integer',
+                               ApiBase :: PARAM_DFLT => 0
+                       ),
+                       'maxage' => array (
+                               ApiBase :: PARAM_TYPE => 'integer',
+                               ApiBase :: PARAM_DFLT => 0
+                       ),
                );
        }
 
@@ -421,7 +452,9 @@ class ApiMain extends ApiBase {
                        'format' => 'The format of the output',
                        'action' => 'What action you would like to perform',
                        'version' => 'When showing help, include version for each module',
-                       'maxlag' => 'Maximum lag'
+                       'maxlag' => 'Maximum lag',
+                       'smaxage' => 'Set the s-maxage header to this many seconds. Errors are never cached',
+                       'maxage' => 'Set the max-age header to this many seconds. Errors are never cached',
                );
        }
 
@@ -455,7 +488,7 @@ class ApiMain extends ApiBase {
                        '',
                );
        }
-       
+
        /**
         * Returns an array of strings with credits for the API
         */
@@ -475,7 +508,7 @@ class ApiMain extends ApiBase {
         * Override the parent to generate help messages for all available modules.
         */
        public function makeHelpMsg() {
-               
+
                $this->mPrinter->setHelp();
 
                // Use parent to make default message for the main module
@@ -501,9 +534,9 @@ class ApiMain extends ApiBase {
                                $msg .= $msg2;
                        $msg .= "\n";
                }
-               
+
                $msg .= "\n*** Credits: ***\n   " . implode("\n   ", $this->getCredits()) . "\n";
-               
+
 
                return $msg;
        }
@@ -511,15 +544,15 @@ class ApiMain extends ApiBase {
        public static function makeHelpMsgHeader($module, $paramName) {
                $modulePrefix = $module->getModulePrefix();
                if (!empty($modulePrefix))
-                       $modulePrefix = "($modulePrefix) "; 
-               
+                       $modulePrefix = "($modulePrefix) ";
+
                return "* $paramName={$module->getModuleName()} $modulePrefix*";
-       } 
+       }
 
        private $mIsBot = null;
        private $mIsSysop = null;
        private $mCanApiHighLimits = null;
-       
+
        /**
         * Returns true if the currently logged in user is a bot, false otherwise
         * OBSOLETE, use canApiHighLimits() instead
@@ -531,7 +564,7 @@ class ApiMain extends ApiBase {
                }
                return $this->mIsBot;
        }
-       
+
        /**
         * Similar to isBot(), this method returns true if the logged in user is
         * a sysop, and false if not.
@@ -545,7 +578,11 @@ class ApiMain extends ApiBase {
 
                return $this->mIsSysop;
        }
-       
+
+       /**
+        * Check whether the current user is allowed to use high limits
+        * @return bool
+        */
        public function canApiHighLimits() {
                if (!isset($this->mCanApiHighLimits)) {
                        global $wgUser;
@@ -555,6 +592,10 @@ class ApiMain extends ApiBase {
                return $this->mCanApiHighLimits;
        }
 
+       /**
+        * Check whether the user wants us to show version information in the API help
+        * @return bool
+        */
        public function getShowVersions() {
                return $this->mShowVersions;
        }
@@ -565,7 +606,7 @@ class ApiMain extends ApiBase {
         */
        public function getVersion() {
                $vers = array ();
-               $vers[] = 'MediaWiki ' . SpecialVersion::getVersion();
+               $vers[] = 'MediaWiki: ' . SpecialVersion::getVersion() . "\n    http://svn.wikimedia.org/viewvc/mediawiki/trunk/phase3/";
                $vers[] = __CLASS__ . ': $Id$';
                $vers[] = ApiBase :: getBaseVersion();
                $vers[] = ApiFormatBase :: getBaseVersion();
@@ -576,7 +617,7 @@ class ApiMain extends ApiBase {
 
        /**
         * Add or overwrite a module in this ApiMain instance. Intended for use by extending
-        * classes who wish to add their own modules to their lexicon or override the 
+        * classes who wish to add their own modules to their lexicon or override the
         * behavior of inherent ones.
         *
         * @access protected
@@ -598,7 +639,7 @@ class ApiMain extends ApiBase {
        protected function addFormat( $fmtName, $fmtClass ) {
                $this->mFormats[$fmtName] = $fmtClass;
        }
-       
+
        /**
         * Get the array mapping module names to class names
         */
@@ -610,8 +651,8 @@ class ApiMain extends ApiBase {
 /**
  * This exception will be thrown when dieUsage is called to stop module execution.
  * The exception handling code will print a help screen explaining how this API may be used.
- * 
- * @addtogroup API
+ *
+ * @ingroup API
  */
 class UsageException extends Exception {
 
@@ -628,6 +669,3 @@ class UsageException extends Exception {
                return "{$this->getCodeString()}: {$this->getMessage()}";
        }
 }
-
-
-