Merge "Drop index oi_name_archive_name on table oldimage"
[lhc/web/wiklou.git] / includes / api / ApiQueryDeletedrevs.php
index fe2ec8f..2959151 100644 (file)
@@ -37,21 +37,12 @@ class ApiQueryDeletedrevs extends ApiQueryBase {
        }
 
        public function execute() {
-               $user = $this->getUser();
                // Before doing anything at all, let's check permissions
-               if ( !$user->isAllowed( 'deletedhistory' ) ) {
-                       $this->dieUsage(
-                               'You don\'t have permission to view deleted revision information',
-                               'permissiondenied'
-                       );
-               }
+               $this->checkUserRightsAny( 'deletedhistory' );
 
-               $this->setWarning(
-                       'list=deletedrevs has been deprecated. Please use prop=deletedrevisions or ' .
-                       'list=alldeletedrevisions instead.'
-               );
-               $this->logFeatureUsage( 'action=query&list=deletedrevs' );
+               $this->addDeprecation( 'apiwarn-deprecation-deletedrevs', 'action=query&list=deletedrevs' );
 
+               $user = $this->getUser();
                $db = $this->getDB();
                $params = $this->extractRequestParams( false );
                $prop = array_flip( $params['prop'] );
@@ -70,9 +61,6 @@ class ApiQueryDeletedrevs extends ApiQueryBase {
 
                if ( isset( $prop['token'] ) ) {
                        $p = $this->getModulePrefix();
-                       $this->setWarning(
-                               "{$p}prop=token has been deprecated. Please use action=query&meta=tokens instead."
-                       );
                }
 
                // If we're in a mode that breaks the same-origin policy, no tokens can
@@ -103,25 +91,25 @@ class ApiQueryDeletedrevs extends ApiQueryBase {
 
                if ( $mode == 'revs' || $mode == 'user' ) {
                        // Ignore namespace and unique due to inability to know whether they were purposely set
-                       foreach ( array( 'from', 'to', 'prefix', /*'namespace', 'unique'*/ ) as $p ) {
+                       foreach ( [ 'from', 'to', 'prefix', /*'namespace', 'unique'*/ ] as $p ) {
                                if ( !is_null( $params[$p] ) ) {
-                                       $this->dieUsage( "The '{$p}' parameter cannot be used in modes 1 or 2", 'badparams' );
+                                       $this->dieWithError( [ 'apierror-deletedrevs-param-not-1-2', $p ], 'badparams' );
                                }
                        }
                } else {
-                       foreach ( array( 'start', 'end' ) as $p ) {
+                       foreach ( [ 'start', 'end' ] as $p ) {
                                if ( !is_null( $params[$p] ) ) {
-                                       $this->dieUsage( "The {$p} parameter cannot be used in mode 3", 'badparams' );
+                                       $this->dieWithError( [ 'apierror-deletedrevs-param-not-3', $p ], 'badparams' );
                                }
                        }
                }
 
                if ( !is_null( $params['user'] ) && !is_null( $params['excludeuser'] ) ) {
-                       $this->dieUsage( 'user and excludeuser cannot be used together', 'badparams' );
+                       $this->dieWithError( 'user and excludeuser cannot be used together', 'badparams' );
                }
 
                $this->addTables( 'archive' );
-               $this->addFields( array( 'ar_title', 'ar_namespace', 'ar_timestamp', 'ar_deleted', 'ar_id' ) );
+               $this->addFields( [ 'ar_title', 'ar_namespace', 'ar_timestamp', 'ar_deleted', 'ar_id' ] );
 
                $this->addFieldsIf( 'ar_parent_id', $fld_parentid );
                $this->addFieldsIf( 'ar_rev_id', $fld_revid );
@@ -135,7 +123,7 @@ class ApiQueryDeletedrevs extends ApiQueryBase {
                if ( $fld_tags ) {
                        $this->addTables( 'tag_summary' );
                        $this->addJoinConds(
-                               array( 'tag_summary' => array( 'LEFT JOIN', array( 'ar_rev_id=ts_rev_id' ) ) )
+                               [ 'tag_summary' => [ 'LEFT JOIN', [ 'ar_rev_id=ts_rev_id' ] ] ]
                        );
                        $this->addFields( 'ts_tags' );
                }
@@ -143,7 +131,7 @@ class ApiQueryDeletedrevs extends ApiQueryBase {
                if ( !is_null( $params['tag'] ) ) {
                        $this->addTables( 'change_tag' );
                        $this->addJoinConds(
-                               array( 'change_tag' => array( 'INNER JOIN', array( 'ar_rev_id=ct_rev_id' ) ) )
+                               [ 'change_tag' => [ 'INNER JOIN', [ 'ar_rev_id=ct_rev_id' ] ] ]
                        );
                        $this->addWhereFld( 'ct_tag', $params['tag'] );
                }
@@ -157,17 +145,12 @@ class ApiQueryDeletedrevs extends ApiQueryBase {
                        // to be able to tell the difference.
                        $this->addTables( 'text' );
                        $this->addJoinConds(
-                               array( 'text' => array( 'LEFT JOIN', array( 'ar_text_id=old_id' ) ) )
+                               [ 'text' => [ 'LEFT JOIN', [ 'ar_text_id=old_id' ] ] ]
                        );
-                       $this->addFields( array( 'ar_text', 'ar_flags', 'ar_text_id', 'old_text', 'old_flags' ) );
+                       $this->addFields( [ 'ar_text', 'ar_flags', 'ar_text_id', 'old_text', 'old_flags' ] );
 
                        // This also means stricter restrictions
-                       if ( !$user->isAllowedAny( 'undelete', 'deletedtext' ) ) {
-                               $this->dieUsage(
-                                       'You don\'t have permission to view deleted revision content',
-                                       'permissiondenied'
-                               );
-                       }
+                       $this->checkUserRightsAny( [ 'deletedtext', 'undelete' ] );
                }
                // Check limits
                $userMax = $fld_content ? ApiBase::LIMIT_SML1 : ApiBase::LIMIT_BIG1;
@@ -220,7 +203,7 @@ class ApiQueryDeletedrevs extends ApiQueryBase {
                }
 
                if ( !is_null( $params['user'] ) || !is_null( $params['excludeuser'] ) ) {
-                       // Paranoia: avoid brute force searches (bug 17342)
+                       // Paranoia: avoid brute force searches (T19342)
                        // (shouldn't be able to get here without 'deletedhistory', but
                        // check it again just in case)
                        if ( !$user->isAllowed( 'deletedhistory' ) ) {
@@ -267,7 +250,7 @@ class ApiQueryDeletedrevs extends ApiQueryBase {
                $this->addOption( 'LIMIT', $limit + 1 );
                $this->addOption(
                        'USE INDEX',
-                       array( 'archive' => ( $mode == 'user' ? 'usertext_timestamp' : 'name_title_timestamp' ) )
+                       [ 'archive' => ( $mode == 'user' ? 'usertext_timestamp' : 'name_title_timestamp' ) ]
                );
                if ( $mode == 'all' ) {
                        if ( $params['unique'] ) {
@@ -275,11 +258,11 @@ class ApiQueryDeletedrevs extends ApiQueryBase {
                                $this->addOption( 'GROUP BY', 'ar_title' );
                        } else {
                                $sort = ( $dir == 'newer' ? '' : ' DESC' );
-                               $this->addOption( 'ORDER BY', array(
+                               $this->addOption( 'ORDER BY', [
                                        'ar_title' . $sort,
                                        'ar_timestamp' . $sort,
                                        'ar_id' . $sort,
-                               ) );
+                               ] );
                        }
                } else {
                        if ( $mode == 'revs' ) {
@@ -292,7 +275,7 @@ class ApiQueryDeletedrevs extends ApiQueryBase {
                        $this->addWhereRange( 'ar_id', $dir, null, null );
                }
                $res = $this->select( __METHOD__ );
-               $pageMap = array(); // Maps ns&title to (fake) pageid
+               $pageMap = []; // Maps ns&title to (fake) pageid
                $count = 0;
                $newPageID = 0;
                foreach ( $res as $row ) {
@@ -308,7 +291,7 @@ class ApiQueryDeletedrevs extends ApiQueryBase {
                                break;
                        }
 
-                       $rev = array();
+                       $rev = [];
                        $anyHidden = false;
 
                        $rev['timestamp'] = wfTimestamp( TS_ISO_8601, $row->ar_timestamp );
@@ -389,7 +372,7 @@ class ApiQueryDeletedrevs extends ApiQueryBase {
                                        ApiResult::setIndexedTagName( $tags, 'tag' );
                                        $rev['tags'] = $tags;
                                } else {
-                                       $rev['tags'] = array();
+                                       $rev['tags'] = [];
                                }
                        }
 
@@ -400,18 +383,18 @@ class ApiQueryDeletedrevs extends ApiQueryBase {
                        if ( !isset( $pageMap[$row->ar_namespace][$row->ar_title] ) ) {
                                $pageID = $newPageID++;
                                $pageMap[$row->ar_namespace][$row->ar_title] = $pageID;
-                               $a['revisions'] = array( $rev );
+                               $a['revisions'] = [ $rev ];
                                ApiResult::setIndexedTagName( $a['revisions'], 'rev' );
                                $title = Title::makeTitle( $row->ar_namespace, $row->ar_title );
                                ApiQueryBase::addTitleInfo( $a, $title );
                                if ( $fld_token ) {
                                        $a['token'] = $token;
                                }
-                               $fit = $result->addValue( array( 'query', $this->getModuleName() ), $pageID, $a );
+                               $fit = $result->addValue( [ 'query', $this->getModuleName() ], $pageID, $a );
                        } else {
                                $pageID = $pageMap[$row->ar_namespace][$row->ar_title];
                                $fit = $result->addValue(
-                                       array( 'query', $this->getModuleName(), $pageID, 'revisions' ),
+                                       [ 'query', $this->getModuleName(), $pageID, 'revisions' ],
                                        null, $rev );
                        }
                        if ( !$fit ) {
@@ -425,7 +408,7 @@ class ApiQueryDeletedrevs extends ApiQueryBase {
                                break;
                        }
                }
-               $result->addIndexedTagName( array( 'query', $this->getModuleName() ), 'page' );
+               $result->addIndexedTagName( [ 'query', $this->getModuleName() ], 'page' );
        }
 
        public function isDeprecated() {
@@ -433,52 +416,52 @@ class ApiQueryDeletedrevs extends ApiQueryBase {
        }
 
        public function getAllowedParams() {
-               return array(
-                       'start' => array(
+               return [
+                       'start' => [
                                ApiBase::PARAM_TYPE => 'timestamp',
-                               ApiBase::PARAM_HELP_MSG_INFO => array( array( 'modes', 1, 2 ) ),
-                       ),
-                       'end' => array(
+                               ApiBase::PARAM_HELP_MSG_INFO => [ [ 'modes', 1, 2 ] ],
+                       ],
+                       'end' => [
                                ApiBase::PARAM_TYPE => 'timestamp',
-                               ApiBase::PARAM_HELP_MSG_INFO => array( array( 'modes', 1, 2 ) ),
-                       ),
-                       'dir' => array(
-                               ApiBase::PARAM_TYPE => array(
+                               ApiBase::PARAM_HELP_MSG_INFO => [ [ 'modes', 1, 2 ] ],
+                       ],
+                       'dir' => [
+                               ApiBase::PARAM_TYPE => [
                                        'newer',
                                        'older'
-                               ),
+                               ],
                                ApiBase::PARAM_DFLT => 'older',
                                ApiBase::PARAM_HELP_MSG => 'api-help-param-direction',
-                               ApiBase::PARAM_HELP_MSG_INFO => array( array( 'modes', 1, 3 ) ),
-                       ),
-                       'from' => array(
-                               ApiBase::PARAM_HELP_MSG_INFO => array( array( 'modes', 3 ) ),
-                       ),
-                       'to' => array(
-                               ApiBase::PARAM_HELP_MSG_INFO => array( array( 'modes', 3 ) ),
-                       ),
-                       'prefix' => array(
-                               ApiBase::PARAM_HELP_MSG_INFO => array( array( 'modes', 3 ) ),
-                       ),
-                       'unique' => array(
+                               ApiBase::PARAM_HELP_MSG_INFO => [ [ 'modes', 1, 3 ] ],
+                       ],
+                       'from' => [
+                               ApiBase::PARAM_HELP_MSG_INFO => [ [ 'modes', 3 ] ],
+                       ],
+                       'to' => [
+                               ApiBase::PARAM_HELP_MSG_INFO => [ [ 'modes', 3 ] ],
+                       ],
+                       'prefix' => [
+                               ApiBase::PARAM_HELP_MSG_INFO => [ [ 'modes', 3 ] ],
+                       ],
+                       'unique' => [
                                ApiBase::PARAM_DFLT => false,
-                               ApiBase::PARAM_HELP_MSG_INFO => array( array( 'modes', 3 ) ),
-                       ),
-                       'namespace' => array(
+                               ApiBase::PARAM_HELP_MSG_INFO => [ [ 'modes', 3 ] ],
+                       ],
+                       'namespace' => [
                                ApiBase::PARAM_TYPE => 'namespace',
                                ApiBase::PARAM_DFLT => NS_MAIN,
-                               ApiBase::PARAM_HELP_MSG_INFO => array( array( 'modes', 3 ) ),
-                       ),
+                               ApiBase::PARAM_HELP_MSG_INFO => [ [ 'modes', 3 ] ],
+                       ],
                        'tag' => null,
-                       'user' => array(
+                       'user' => [
                                ApiBase::PARAM_TYPE => 'user'
-                       ),
-                       'excludeuser' => array(
+                       ],
+                       'excludeuser' => [
                                ApiBase::PARAM_TYPE => 'user'
-                       ),
-                       'prop' => array(
+                       ],
+                       'prop' => [
                                ApiBase::PARAM_DFLT => 'user|comment',
-                               ApiBase::PARAM_TYPE => array(
+                               ApiBase::PARAM_TYPE => [
                                        'revid',
                                        'parentid',
                                        'user',
@@ -491,24 +474,24 @@ class ApiQueryDeletedrevs extends ApiQueryBase {
                                        'content',
                                        'token',
                                        'tags'
-                               ),
+                               ],
                                ApiBase::PARAM_ISMULTI => true
-                       ),
-                       'limit' => array(
+                       ],
+                       'limit' => [
                                ApiBase::PARAM_DFLT => 10,
                                ApiBase::PARAM_TYPE => 'limit',
                                ApiBase::PARAM_MIN => 1,
                                ApiBase::PARAM_MAX => ApiBase::LIMIT_BIG1,
                                ApiBase::PARAM_MAX2 => ApiBase::LIMIT_BIG2
-                       ),
-                       'continue' => array(
+                       ],
+                       'continue' => [
                                ApiBase::PARAM_HELP_MSG => 'api-help-param-continue',
-                       ),
-               );
+                       ],
+               ];
        }
 
        protected function getExamplesMessages() {
-               return array(
+               return [
                        'action=query&list=deletedrevs&titles=Main%20Page|Talk:Main%20Page&' .
                                'drprop=user|comment|content'
                                => 'apihelp-query+deletedrevs-example-mode1',
@@ -518,7 +501,7 @@ class ApiQueryDeletedrevs extends ApiQueryBase {
                                => 'apihelp-query+deletedrevs-example-mode3-main',
                        'action=query&list=deletedrevs&drdir=newer&drlimit=50&drnamespace=1&drunique='
                                => 'apihelp-query+deletedrevs-example-mode3-talk',
-               );
+               ];
        }
 
        public function getHelpUrls() {