MediaWiki 1.30.2

[autoinstallsdev/mediawiki.git] / includes / specials / SpecialWantedpages.php

diff --git a/includes/specials/SpecialWantedpages.php b/includes/specials/SpecialWantedpages.php
index 4e1611bce4a7c3cdfce9cc6b4da639de27a4287b..8cea6ccb7768ef55c85fc7d9433fe10d39ed4ee8 100644 (file)
--- a/includes/specials/SpecialWantedpages.php
+++ b/includes/specials/SpecialWantedpages.php
@@ -27,60 +27,72 @@
  * @ingroup SpecialPage
  */
 class WantedPagesPage extends WantedQueryPage {
-       var $nlinks;
 
-       function __construct( $inc = false, $nlinks = true ) {
-               $this->setListoutput( $inc );
-               $this->nlinks = $nlinks;
+       function __construct( $name = 'Wantedpages' ) {
+               parent::__construct( $name );
        }
 
-       function getName() {
-               return 'Wantedpages';
+       function isIncludable() {
+               return true;
        }
 
-       function getSQL() {
-               global $wgWantedPagesThreshold;
-               $count = $wgWantedPagesThreshold - 1;
-               $dbr = wfGetDB( DB_SLAVE );
-               $pagelinks = $dbr->tableName( 'pagelinks' );
-               $page      = $dbr->tableName( 'page' );
-               $sql = "SELECT 'Wantedpages' AS type,
-                               pl_namespace AS namespace,
-                               pl_title AS title,
-                               COUNT(*) AS value
-                       FROM $pagelinks
-                       LEFT JOIN $page AS pg1
-                       ON pl_namespace = pg1.page_namespace AND pl_title = pg1.page_title
-                       LEFT JOIN $page AS pg2
-                       ON pl_from = pg2.page_id
-                       WHERE pg1.page_namespace IS NULL
-                       AND pl_namespace NOT IN ( " . NS_USER . ", ". NS_USER_TALK . ")
-                       AND pg2.page_namespace != " . NS_MEDIAWIKI . "
-                       GROUP BY pl_namespace, pl_title
-                       HAVING COUNT(*) > $count";
+       function execute( $par ) {
+               $inc = $this->including();
 
-               wfRunHooks( 'WantedPages::getSQL', array( &$this, &$sql ) );
-               return $sql;
+               if ( $inc ) {
+                       $this->limit = (int)$par;
+                       $this->offset = 0;
+               }
+               $this->setListoutput( $inc );
+               $this->shownavigation = !$inc;
+               parent::execute( $par );
        }
-}
 
-/**
- * constructor
- */
-function wfSpecialWantedpages( $par = null, $specialPage ) {
-       $inc = $specialPage->including();
+       function getQueryInfo() {
+               $dbr = wfGetDB( DB_REPLICA );
+               $count = $this->getConfig()->get( 'WantedPagesThreshold' ) - 1;
+               $query = [
+                       'tables' => [
+                               'pagelinks',
+                               'pg1' => 'page',
+                               'pg2' => 'page'
+                       ],
+                       'fields' => [
+                               'namespace' => 'pl_namespace',
+                               'title' => 'pl_title',
+                               'value' => 'COUNT(*)'
+                       ],
+                       'conds' => [
+                               'pg1.page_namespace IS NULL',
+                               'pl_namespace NOT IN (' . $dbr->makeList( [ NS_USER, NS_USER_TALK ] ) . ')',
+                               'pg2.page_namespace != ' . $dbr->addQuotes( NS_MEDIAWIKI ),
+                       ],
+                       'options' => [
+                               'HAVING' => [
+                                       'COUNT(*) > ' . $dbr->addQuotes( $count ),
+                                       'COUNT(*) > SUM(pg2.page_is_redirect)'
+                               ],
+                               'GROUP BY' => [ 'pl_namespace', 'pl_title' ]
+                       ],
+                       'join_conds' => [
+                               'pg1' => [
+                                       'LEFT JOIN', [
+                                               'pg1.page_namespace = pl_namespace',
+                                               'pg1.page_title = pl_title'
+                                       ]
+                               ],
+                               'pg2' => [ 'LEFT JOIN', 'pg2.page_id = pl_from' ]
+                       ]
+               ];
+               // Replacement for the WantedPages::getSQL hook
+               // Avoid PHP 7.1 warning from passing $this by reference
+               $wantedPages = $this;
+               Hooks::run( 'WantedPages::getQueryInfo', [ &$wantedPages, &$query ] );
 
-       if ( $inc ) {
-               @list( $limit, $nlinks ) = explode( '/', $par, 2 );
-               $limit = (int)$limit;
-               $nlinks = $nlinks === 'nlinks';
-               $offset = 0;
-       } else {
-               list( $limit, $offset ) = wfCheckLimits();
-               $nlinks = true;
+               return $query;
        }
 
-       $wpp = new WantedPagesPage( $inc, $nlinks );
-
-       $wpp->doQuery( $offset, $limit, !$inc );
+       protected function getGroupName() {
+               return 'maintenance';
+       }
 }