API: Improvements to action=emailuser:
[lhc/web/wiklou.git] / includes / api / ApiMain.php
index e44a5f2..c3946e5 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,
@@ -39,7 +43,7 @@ if (!defined('MEDIAWIKI')) {
  * 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
+ * @ingroup API
  */
 class ApiMain extends ApiBase {
 
@@ -72,6 +76,8 @@ class ApiMain extends ApiBase {
                'unblock' => 'ApiUnblock',
                'move' => 'ApiMove',
                'edit' => 'ApiEditPage',
+               'emailuser' => 'ApiEmailUser',
+               'watch' => 'ApiWatch',
        );
 
        /**
@@ -94,6 +100,23 @@ class ApiMain extends ApiBase {
                'dbg' => 'ApiFormatDbg',
                'dbgfm' => 'ApiFormatDbg'
        );
+       
+       /**
+        * List of user roles that are specifically relevant to the API.
+        * array( 'right' => array ( 'msg'    => 'Some message with a $1',
+        *                           'params' => array ( $someVarToSubst ) ),
+        *                          );
+        */
+       private static $mRights = array('writeapi' => array(
+                                               'msg' => 'Use of the write API',
+                                               'params' => array()
+                                       ),
+                                       'apihighlimits' => array(
+                                               'msg' => 'Use higher limits in API queries (Slow queries: $1 results; Fast queries: $2 results). The limits for slow queries also apply to multivalue parameters.',
+                                               'params' => array (ApiMain::LIMIT_SML2, ApiMain::LIMIT_BIG2)
+                                       )
+       );
+
 
        private $mPrinter, $mModules, $mModuleNames, $mFormats, $mFormatNames;
        private $mResult, $mAction, $mShowVersions, $mEnableWrite, $mRequest, $mInternalMode, $mSquidMaxage;
@@ -149,7 +172,7 @@ class ApiMain extends ApiBase {
 
                $this->mRequest = & $request;
 
-               $this->mSquidMaxage = 0;
+               $this->mSquidMaxage = -1; // flag for executeActionWithErrorHandling()
                $this->mCommit = false;
        }
 
@@ -175,12 +198,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');
        }
 
        /**
@@ -248,11 +278,21 @@ class ApiMain extends ApiBase {
                        $this->printResult(true);
                }
 
+               if($this->mSquidMaxage == -1)
+               {
+                       # Nobody called setCacheMaxAge(), use the (s)maxage parameters
+                       $smaxage = $this->mRequest->getVal('smaxage', 0);
+                       $maxage = $this->mRequest->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();
@@ -291,17 +331,27 @@ class ApiMain extends ApiBase {
                                        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();
+                       // Re-add the id
+                       if($this->mRequest->getCheck('requestid'))
+                               $this->getResult()->addValue(null, 'requestid', $this->mRequest->getVal('requestid'));
                        $this->getResult()->addValue(null, 'error', $errMessage);
 
                return $errMessage['code'];
@@ -311,12 +361,19 @@ class ApiMain extends ApiBase {
         * Execute the actual module, without any error handling
         */
        protected function executeAction() {
+               // First add the id to the top element
+               if($this->mRequest->getCheck('requestid'))
+                       $this->getResult()->addValue(null, 'requestid', $this->mRequest->getVal('requestid'));
 
                $params = $this->extractRequestParams();
 
                $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);
 
@@ -399,6 +456,15 @@ 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
+                       ),
+                       'requestid' => null,
                );
        }
 
@@ -410,7 +476,10 @@ 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',
+                       'requestid' => 'Request ID to distinguish requests. This will just be output back to you',
                );
        }
 
@@ -453,6 +522,7 @@ class ApiMain extends ApiBase {
                        'API developers:',
                        '    Roan Kattouw <Firstname>.<Lastname>@home.nl (lead developer Sep 2007-present)',
                        '    Victor Vasiliev - vasilvv at gee mail dot com',
+                       '    Bryan Tong Minh - bryan . tongminh @ gmail . com',
                        '    Yuri Astrakhan <Firstname><Lastname>@gmail.com (creator, lead developer Sep 2006-Sep 2007)',
                        '',
                        'Please send your comments, suggestions and questions to mediawiki-api@lists.wikimedia.org',
@@ -481,6 +551,14 @@ class ApiMain extends ApiBase {
                        $msg .= "\n";
                }
 
+               $msg .= "\n$astriks Permissions $astriks\n\n";
+               foreach ( self :: $mRights as $right => $rightMsg ) {
+                       $groups = User::getGroupsWithPermission( $right );
+                       $msg .= "* " . $right . " *\n  " . wfMsgReplaceArgs( $rightMsg[ 'msg' ], $rightMsg[ 'params' ] ) . 
+                                               "\nGranted to:\n  " . str_replace( "*", "all", implode( ", ", $groups ) ) . "\n";
+
+               }
+
                $msg .= "\n$astriks Formats  $astriks\n\n";
                foreach( $this->mFormats as $formatName => $unused ) {
                        $module = $this->createPrinterByName($formatName);
@@ -562,7 +640,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();
@@ -608,7 +686,7 @@ 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 {