From: Kevin Israel Date: Tue, 3 Jun 2014 03:02:29 +0000 (-0400) Subject: Move QueryPage and abstract subclasses to specialpage folder X-Git-Tag: 1.31.0-rc.0~15178^2 X-Git-Url: http://git.cyclocoop.org/%27-%20%20.%20url_absolue%28find_in_path%28%27spip_style.css%27%29%29%20%20%20.%20url_absolue%28find_in_path%28%27prive/spip_style.css%27%29%29%20.%20%27?a=commitdiff_plain;h=ffa3518ed54bf1240195af6ec72e2e0a66a1e54e;p=lhc%2Fweb%2Fwiklou.git Move QueryPage and abstract subclasses to specialpage folder ... now that f508561f7bde ("Moved $wgQueryPages stuff out of the global scope and into a function") has been merged, which eliminated hardcoding of the pathname except in the AutoLoader. WantedQueryPage is moved to a separate file. Bug: 60445 Change-Id: If2019492722083819b82afa6302670b94b181098 --- diff --git a/includes/AutoLoader.php b/includes/AutoLoader.php index 3ac4722e27..bf3f42eb0e 100644 --- a/includes/AutoLoader.php +++ b/includes/AutoLoader.php @@ -110,7 +110,6 @@ $wgAutoloadLocalClasses = array( 'ICacheHelper' => 'includes/CacheHelper.php', 'IcuCollation' => 'includes/Collation.php', 'IdentityCollation' => 'includes/Collation.php', - 'ImageQueryPage' => 'includes/ImageQueryPage.php', 'ImportStreamSource' => 'includes/Import.php', 'ImportStringSource' => 'includes/Import.php', 'IndexPager' => 'includes/Pager.php', @@ -140,7 +139,6 @@ $wgAutoloadLocalClasses = array( 'MWInit' => 'includes/Init.php', 'MWNamespace' => 'includes/Namespace.php', 'OutputPage' => 'includes/OutputPage.php', - 'PageQueryPage' => 'includes/PageQueryPage.php', 'Pager' => 'includes/Pager.php', 'PasswordError' => 'includes/User.php', 'PathRouter' => 'includes/PathRouter.php', @@ -156,7 +154,6 @@ $wgAutoloadLocalClasses = array( 'PreferencesForm' => 'includes/Preferences.php', 'PrefixSearch' => 'includes/PrefixSearch.php', 'ProtectionForm' => 'includes/ProtectionForm.php', - 'QueryPage' => 'includes/QueryPage.php', 'QuickTemplate' => 'includes/SkinTemplate.php', 'RawMessage' => 'includes/Message.php', 'ReverseChronologicalPager' => 'includes/Pager.php', @@ -196,7 +193,6 @@ $wgAutoloadLocalClasses = array( 'UserCache' => 'includes/cache/UserCache.php', 'UserMailer' => 'includes/UserMailer.php', 'UserRightsProxy' => 'includes/UserRightsProxy.php', - 'WantedQueryPage' => 'includes/QueryPage.php', 'WatchedItem' => 'includes/WatchedItem.php', 'WebRequest' => 'includes/WebRequest.php', 'WebRequestUpload' => 'includes/WebRequest.php', @@ -927,13 +923,17 @@ $wgAutoloadLocalClasses = array( # includes/specialpage 'ChangesListSpecialPage' => 'includes/specialpage/ChangesListSpecialPage.php', 'FormSpecialPage' => 'includes/specialpage/FormSpecialPage.php', + 'ImageQueryPage' => 'includes/specialpage/ImageQueryPage.php', 'IncludableSpecialPage' => 'includes/specialpage/IncludableSpecialPage.php', + 'PageQueryPage' => 'includes/specialpage/PageQueryPage.php', + 'QueryPage' => 'includes/specialpage/QueryPage.php', 'RedirectSpecialArticle' => 'includes/specialpage/RedirectSpecialPage.php', 'RedirectSpecialPage' => 'includes/specialpage/RedirectSpecialPage.php', 'SpecialPage' => 'includes/specialpage/SpecialPage.php', 'SpecialPageFactory' => 'includes/specialpage/SpecialPageFactory.php', 'SpecialRedirectToSpecial' => 'includes/specialpage/RedirectSpecialPage.php', 'UnlistedSpecialPage' => 'includes/specialpage/UnlistedSpecialPage.php', + 'WantedQueryPage' => 'includes/specialpage/WantedQueryPage.php', # includes/specials 'ActiveUsersPager' => 'includes/specials/SpecialActiveusers.php', diff --git a/includes/ImageQueryPage.php b/includes/ImageQueryPage.php deleted file mode 100644 index b0266cbaf7..0000000000 --- a/includes/ImageQueryPage.php +++ /dev/null @@ -1,80 +0,0 @@ - - */ -abstract class ImageQueryPage extends QueryPage { - /** - * Format and output report results using the given information plus - * OutputPage - * - * @param OutputPage $out OutputPage to print to - * @param Skin $skin User skin to use [unused] - * @param DatabaseBase $dbr (read) connection to use - * @param int $res Result pointer - * @param int $num Number of available result rows - * @param int $offset Paging offset - */ - protected function outputResults( $out, $skin, $dbr, $res, $num, $offset ) { - if ( $num > 0 ) { - $gallery = ImageGalleryBase::factory(); - $gallery->setContext( $this->getContext() ); - - # $res might contain the whole 1,000 rows, so we read up to - # $num [should update this to use a Pager] - $i = 0; - foreach ( $res as $row ) { - $i++; - $namespace = isset( $row->namespace ) ? $row->namespace : NS_FILE; - $title = Title::makeTitleSafe( $namespace, $row->title ); - if ( $title instanceof Title && $title->getNamespace() == NS_FILE ) { - $gallery->add( $title, $this->getCellHtml( $row ) ); - } - if ( $i === $num ) { - break; - } - } - - $out->addHTML( $gallery->toHtml() ); - } - } - - // Gotta override this since it's abstract - function formatResult( $skin, $result ) { - } - - /** - * Get additional HTML to be shown in a results' cell - * - * @param object $row Result row - * @return string - */ - protected function getCellHtml( $row ) { - return ''; - } -} diff --git a/includes/PageQueryPage.php b/includes/PageQueryPage.php deleted file mode 100644 index afc02271c5..0000000000 --- a/includes/PageQueryPage.php +++ /dev/null @@ -1,72 +0,0 @@ -numRows() ) { - return; - } - - $batch = new LinkBatch(); - foreach ( $res as $row ) { - $batch->add( $row->namespace, $row->title ); - } - $batch->execute(); - - $res->seek( 0 ); - } - - /** - * Format the result as a simple link to the page - * - * @param Skin $skin - * @param object $row Result row - * @return string - */ - public function formatResult( $skin, $row ) { - global $wgContLang; - - $title = Title::makeTitleSafe( $row->namespace, $row->title ); - - if ( $title instanceof Title ) { - $text = $wgContLang->convert( $title->getPrefixedText() ); - return Linker::link( $title, htmlspecialchars( $text ) ); - } else { - return Html::element( 'span', array( 'class' => 'mw-invalidtitle' ), - Linker::getInvalidTitleDescription( $this->getContext(), $row->namespace, $row->title ) ); - } - } -} diff --git a/includes/QueryPage.php b/includes/QueryPage.php deleted file mode 100644 index 82dea0d96f..0000000000 --- a/includes/QueryPage.php +++ /dev/null @@ -1,855 +0,0 @@ -listoutput; - * - * @param bool $bool - */ - function setListoutput( $bool ) { - $this->listoutput = $bool; - } - - /** - * Subclasses return an SQL query here, formatted as an array with the - * following keys: - * tables => Table(s) for passing to Database::select() - * fields => Field(s) for passing to Database::select(), may be * - * conds => WHERE conditions - * options => options - * join_conds => JOIN conditions - * - * Note that the query itself should return the following three columns: - * 'namespace', 'title', and 'value'. 'value' is used for sorting. - * - * These may be stored in the querycache table for expensive queries, - * and that cached data will be returned sometimes, so the presence of - * extra fields can't be relied upon. The cached 'value' column will be - * an integer; non-numeric values are useful only for sorting the - * initial query (except if they're timestamps, see usesTimestamps()). - * - * Don't include an ORDER or LIMIT clause, they will be added. - * - * If this function is not overridden or returns something other than - * an array, getSQL() will be used instead. This is for backwards - * compatibility only and is strongly deprecated. - * @return array - * @since 1.18 - */ - function getQueryInfo() { - return null; - } - - /** - * For back-compat, subclasses may return a raw SQL query here, as a string. - * This is strongly deprecated; getQueryInfo() should be overridden instead. - * @throws MWException - * @return string - */ - function getSQL() { - /* Implement getQueryInfo() instead */ - throw new MWException( "Bug in a QueryPage: doesn't implement getQueryInfo() nor " - . "getQuery() properly" ); - } - - /** - * Subclasses return an array of fields to order by here. Don't append - * DESC to the field names, that'll be done automatically if - * sortDescending() returns true. - * @return array - * @since 1.18 - */ - function getOrderFields() { - return array( 'value' ); - } - - /** - * Does this query return timestamps rather than integers in its - * 'value' field? If true, this class will convert 'value' to a - * UNIX timestamp for caching. - * NOTE: formatRow() may get timestamps in TS_MW (mysql), TS_DB (pgsql) - * or TS_UNIX (querycache) format, so be sure to always run them - * through wfTimestamp() - * @return bool - * @since 1.18 - */ - function usesTimestamps() { - return false; - } - - /** - * Override to sort by increasing values - * - * @return bool - */ - function sortDescending() { - return true; - } - - /** - * Is this query expensive (for some definition of expensive)? Then we - * don't let it run in miser mode. $wgDisableQueryPages causes all query - * pages to be declared expensive. Some query pages are always expensive. - * - * @return bool - */ - function isExpensive() { - global $wgDisableQueryPages; - return $wgDisableQueryPages; - } - - /** - * Is the output of this query cacheable? Non-cacheable expensive pages - * will be disabled in miser mode and will not have their results written - * to the querycache table. - * @return bool - * @since 1.18 - */ - public function isCacheable() { - return true; - } - - /** - * Whether or not the output of the page in question is retrieved from - * the database cache. - * - * @return bool - */ - function isCached() { - global $wgMiserMode; - - return $this->isExpensive() && $wgMiserMode; - } - - /** - * Sometime we don't want to build rss / atom feeds. - * - * @return bool - */ - function isSyndicated() { - return true; - } - - /** - * Formats the results of the query for display. The skin is the current - * skin; you can use it for making links. The result is a single row of - * result data. You should be able to grab SQL results off of it. - * If the function returns false, the line output will be skipped. - * @param Skin $skin - * @param object $result Result row - * @return string|bool String or false to skip - */ - abstract function formatResult( $skin, $result ); - - /** - * The content returned by this function will be output before any result - * - * @return string - */ - function getPageHeader() { - return ''; - } - - /** - * If using extra form wheely-dealies, return a set of parameters here - * as an associative array. They will be encoded and added to the paging - * links (prev/next/lengths). - * - * @return array - */ - function linkParameters() { - return array(); - } - - /** - * Some special pages (for example SpecialListusers) might not return the - * current object formatted, but return the previous one instead. - * Setting this to return true will ensure formatResult() is called - * one more time to make sure that the very last result is formatted - * as well. - * @return bool - */ - function tryLastResult() { - return false; - } - - /** - * Clear the cache and save new results - * - * @param int|bool $limit Limit for SQL statement - * @param bool $ignoreErrors Whether to ignore database errors - * @throws DBError|Exception - * @return bool|int - */ - function recache( $limit, $ignoreErrors = true ) { - if ( !$this->isCacheable() ) { - return 0; - } - - $fname = get_class( $this ) . '::recache'; - $dbw = wfGetDB( DB_MASTER ); - if ( !$dbw ) { - return false; - } - - try { - # Do query - $res = $this->reallyDoQuery( $limit, false ); - $num = false; - if ( $res ) { - $num = $res->numRows(); - # Fetch results - $vals = array(); - foreach ( $res as $row ) { - if ( isset( $row->value ) ) { - if ( $this->usesTimestamps() ) { - $value = wfTimestamp( TS_UNIX, - $row->value ); - } else { - $value = intval( $row->value ); // @bug 14414 - } - } else { - $value = 0; - } - - $vals[] = array( 'qc_type' => $this->getName(), - 'qc_namespace' => $row->namespace, - 'qc_title' => $row->title, - 'qc_value' => $value ); - } - - $dbw->begin( __METHOD__ ); - # Clear out any old cached data - $dbw->delete( 'querycache', array( 'qc_type' => $this->getName() ), $fname ); - # Save results into the querycache table on the master - if ( count( $vals ) ) { - $dbw->insert( 'querycache', $vals, __METHOD__ ); - } - # Update the querycache_info record for the page - $dbw->delete( 'querycache_info', array( 'qci_type' => $this->getName() ), $fname ); - $dbw->insert( 'querycache_info', - array( 'qci_type' => $this->getName(), 'qci_timestamp' => $dbw->timestamp() ), - $fname ); - $dbw->commit( __METHOD__ ); - } - } catch ( DBError $e ) { - if ( !$ignoreErrors ) { - throw $e; // report query error - } - $num = false; // set result to false to indicate error - } - - return $num; - } - - /** - * Get a DB connection to be used for slow recache queries - * @return DatabaseBase - */ - function getRecacheDB() { - return wfGetDB( DB_SLAVE, array( $this->getName(), 'QueryPage::recache', 'vslow' ) ); - } - - /** - * Run the query and return the result - * @param int|bool $limit Numerical limit or false for no limit - * @param int|bool $offset Numerical offset or false for no offset - * @return ResultWrapper - * @since 1.18 - */ - function reallyDoQuery( $limit, $offset = false ) { - $fname = get_class( $this ) . "::reallyDoQuery"; - $dbr = $this->getRecacheDB(); - $query = $this->getQueryInfo(); - $order = $this->getOrderFields(); - - if ( $this->sortDescending() ) { - foreach ( $order as &$field ) { - $field .= ' DESC'; - } - } - - if ( is_array( $query ) ) { - $tables = isset( $query['tables'] ) ? (array)$query['tables'] : array(); - $fields = isset( $query['fields'] ) ? (array)$query['fields'] : array(); - $conds = isset( $query['conds'] ) ? (array)$query['conds'] : array(); - $options = isset( $query['options'] ) ? (array)$query['options'] : array(); - $join_conds = isset( $query['join_conds'] ) ? (array)$query['join_conds'] : array(); - - if ( count( $order ) ) { - $options['ORDER BY'] = $order; - } - - if ( $limit !== false ) { - $options['LIMIT'] = intval( $limit ); - } - - if ( $offset !== false ) { - $options['OFFSET'] = intval( $offset ); - } - - $res = $dbr->select( $tables, $fields, $conds, $fname, - $options, $join_conds - ); - } else { - // Old-fashioned raw SQL style, deprecated - $sql = $this->getSQL(); - $sql .= ' ORDER BY ' . implode( ', ', $order ); - $sql = $dbr->limitResult( $sql, $limit, $offset ); - $res = $dbr->query( $sql, $fname ); - } - - return $dbr->resultObject( $res ); - } - - /** - * Somewhat deprecated, you probably want to be using execute() - * @param int|bool $offset - * @param int|bool $limit - * @return ResultWrapper - */ - function doQuery( $offset = false, $limit = false ) { - if ( $this->isCached() && $this->isCacheable() ) { - return $this->fetchFromCache( $limit, $offset ); - } else { - return $this->reallyDoQuery( $limit, $offset ); - } - } - - /** - * Fetch the query results from the query cache - * @param int|bool $limit Numerical limit or false for no limit - * @param int|bool $offset Numerical offset or false for no offset - * @return ResultWrapper - * @since 1.18 - */ - function fetchFromCache( $limit, $offset = false ) { - $dbr = wfGetDB( DB_SLAVE ); - $options = array(); - if ( $limit !== false ) { - $options['LIMIT'] = intval( $limit ); - } - if ( $offset !== false ) { - $options['OFFSET'] = intval( $offset ); - } - if ( $this->sortDescending() ) { - $options['ORDER BY'] = 'qc_value DESC'; - } else { - $options['ORDER BY'] = 'qc_value ASC'; - } - $res = $dbr->select( 'querycache', array( 'qc_type', - 'namespace' => 'qc_namespace', - 'title' => 'qc_title', - 'value' => 'qc_value' ), - array( 'qc_type' => $this->getName() ), - __METHOD__, $options - ); - return $dbr->resultObject( $res ); - } - - public function getCachedTimestamp() { - if ( is_null( $this->cachedTimestamp ) ) { - $dbr = wfGetDB( DB_SLAVE ); - $fname = get_class( $this ) . '::getCachedTimestamp'; - $this->cachedTimestamp = $dbr->selectField( 'querycache_info', 'qci_timestamp', - array( 'qci_type' => $this->getName() ), $fname ); - } - return $this->cachedTimestamp; - } - - /** - * This is the actual workhorse. It does everything needed to make a - * real, honest-to-gosh query page. - * @param string $par - * @return int - */ - function execute( $par ) { - global $wgQueryCacheLimit, $wgDisableQueryPageUpdate; - - $user = $this->getUser(); - if ( !$this->userCanExecute( $user ) ) { - $this->displayRestrictionError(); - return; - } - - $this->setHeaders(); - $this->outputHeader(); - - $out = $this->getOutput(); - - if ( $this->isCached() && !$this->isCacheable() ) { - $out->addWikiMsg( 'querypage-disabled' ); - return 0; - } - - $out->setSyndicated( $this->isSyndicated() ); - - if ( $this->limit == 0 && $this->offset == 0 ) { - list( $this->limit, $this->offset ) = $this->getRequest()->getLimitOffset(); - } - - // @todo Use doQuery() - if ( !$this->isCached() ) { - # select one extra row for navigation - $res = $this->reallyDoQuery( $this->limit + 1, $this->offset ); - } else { - # Get the cached result, select one extra row for navigation - $res = $this->fetchFromCache( $this->limit + 1, $this->offset ); - if ( !$this->listoutput ) { - - # Fetch the timestamp of this update - $ts = $this->getCachedTimestamp(); - $lang = $this->getLanguage(); - $maxResults = $lang->formatNum( $wgQueryCacheLimit ); - - if ( $ts ) { - $updated = $lang->userTimeAndDate( $ts, $user ); - $updateddate = $lang->userDate( $ts, $user ); - $updatedtime = $lang->userTime( $ts, $user ); - $out->addMeta( 'Data-Cache-Time', $ts ); - $out->addJsConfigVars( 'dataCacheTime', $ts ); - $out->addWikiMsg( 'perfcachedts', $updated, $updateddate, $updatedtime, $maxResults ); - } else { - $out->addWikiMsg( 'perfcached', $maxResults ); - } - - # If updates on this page have been disabled, let the user know - # that the data set won't be refreshed for now - if ( is_array( $wgDisableQueryPageUpdate ) - && in_array( $this->getName(), $wgDisableQueryPageUpdate ) - ) { - $out->wrapWikiMsg( - "
\n$1\n
", - 'querypage-no-updates' - ); - } - } - } - - $this->numRows = $res->numRows(); - - $dbr = wfGetDB( DB_SLAVE ); - $this->preprocessResults( $dbr, $res ); - - $out->addHTML( Xml::openElement( 'div', array( 'class' => 'mw-spcontent' ) ) ); - - # Top header and navigation - if ( $this->shownavigation ) { - $out->addHTML( $this->getPageHeader() ); - if ( $this->numRows > 0 ) { - $out->addHTML( $this->msg( 'showingresultsinrange' )->numParams( - min( $this->numRows, $this->limit ), # do not show the one extra row, if exist - $this->offset + 1, ( min( $this->numRows, $this->limit ) + $this->offset ) )->parseAsBlock() ); - # Disable the "next" link when we reach the end - $paging = $this->getLanguage()->viewPrevNext( $this->getPageTitle( $par ), $this->offset, - $this->limit, $this->linkParameters(), ( $this->numRows <= $this->limit ) ); - $out->addHTML( '

' . $paging . '

' ); - } else { - # No results to show, so don't bother with "showing X of Y" etc. - # -- just let the user know and give up now - $out->addWikiMsg( 'specialpage-empty' ); - $out->addHTML( Xml::closeElement( 'div' ) ); - return; - } - } - - # The actual results; specialist subclasses will want to handle this - # with more than a straight list, so we hand them the info, plus - # an OutputPage, and let them get on with it - $this->outputResults( $out, - $this->getSkin(), - $dbr, # Should use a ResultWrapper for this - $res, - min( $this->numRows, $this->limit ), # do not format the one extra row, if exist - $this->offset ); - - # Repeat the paging links at the bottom - if ( $this->shownavigation ) { - $out->addHTML( '

' . $paging . '

' ); - } - - $out->addHTML( Xml::closeElement( 'div' ) ); - - return min( $this->numRows, $this->limit ); # do not return the one extra row, if exist - } - - /** - * Format and output report results using the given information plus - * OutputPage - * - * @param OutputPage $out OutputPage to print to - * @param Skin $skin User skin to use - * @param DatabaseBase $dbr Database (read) connection to use - * @param ResultWrapper $res Result pointer - * @param int $num Number of available result rows - * @param int $offset Paging offset - */ - protected function outputResults( $out, $skin, $dbr, $res, $num, $offset ) { - global $wgContLang; - - if ( $num > 0 ) { - $html = array(); - if ( !$this->listoutput ) { - $html[] = $this->openList( $offset ); - } - - # $res might contain the whole 1,000 rows, so we read up to - # $num [should update this to use a Pager] - // @codingStandardsIgnoreStart Generic.CodeAnalysis.ForLoopWithTestFunctionCall.NotAllowed - for ( $i = 0; $i < $num && $row = $res->fetchObject(); $i++ ) { - // @codingStandardsIgnoreEnd - $line = $this->formatResult( $skin, $row ); - if ( $line ) { - $attr = ( isset( $row->usepatrol ) && $row->usepatrol && $row->patrolled == 0 ) - ? ' class="not-patrolled"' - : ''; - $html[] = $this->listoutput - ? $line - : "{$line}\n"; - } - } - - # Flush the final result - if ( $this->tryLastResult() ) { - $row = null; - $line = $this->formatResult( $skin, $row ); - if ( $line ) { - $attr = ( isset( $row->usepatrol ) && $row->usepatrol && $row->patrolled == 0 ) - ? ' class="not-patrolled"' - : ''; - $html[] = $this->listoutput - ? $line - : "{$line}\n"; - } - } - - if ( !$this->listoutput ) { - $html[] = $this->closeList(); - } - - $html = $this->listoutput - ? $wgContLang->listToText( $html ) - : implode( '', $html ); - - $out->addHTML( $html ); - } - } - - /** - * @param int $offset - * @return string - */ - function openList( $offset ) { - return "\n
    \n"; - } - - /** - * @return string - */ - function closeList() { - return "
\n"; - } - - /** - * Do any necessary preprocessing of the result object. - * @param DatabaseBase $db - * @param ResultWrapper $res - */ - function preprocessResults( $db, $res ) { - } - - /** - * Similar to above, but packaging in a syndicated feed instead of a web page - * @param string $class - * @param int $limit - * @return bool - */ - function doFeed( $class = '', $limit = 50 ) { - global $wgFeed, $wgFeedClasses, $wgFeedLimit; - - if ( !$wgFeed ) { - $this->getOutput()->addWikiMsg( 'feed-unavailable' ); - return false; - } - - $limit = min( $limit, $wgFeedLimit ); - - if ( isset( $wgFeedClasses[$class] ) ) { - /** @var RSSFeed|AtomFeed $feed */ - $feed = new $wgFeedClasses[$class]( - $this->feedTitle(), - $this->feedDesc(), - $this->feedUrl() ); - $feed->outHeader(); - - $res = $this->reallyDoQuery( $limit, 0 ); - foreach ( $res as $obj ) { - $item = $this->feedResult( $obj ); - if ( $item ) { - $feed->outItem( $item ); - } - } - - $feed->outFooter(); - return true; - } else { - return false; - } - } - - /** - * Override for custom handling. If the titles/links are ok, just do - * feedItemDesc() - * @param object $row - * @return FeedItem|null - */ - function feedResult( $row ) { - if ( !isset( $row->title ) ) { - return null; - } - $title = Title::makeTitle( intval( $row->namespace ), $row->title ); - if ( $title ) { - $date = isset( $row->timestamp ) ? $row->timestamp : ''; - $comments = ''; - if ( $title ) { - $talkpage = $title->getTalkPage(); - $comments = $talkpage->getFullURL(); - } - - return new FeedItem( - $title->getPrefixedText(), - $this->feedItemDesc( $row ), - $title->getFullURL(), - $date, - $this->feedItemAuthor( $row ), - $comments ); - } else { - return null; - } - } - - function feedItemDesc( $row ) { - return isset( $row->comment ) ? htmlspecialchars( $row->comment ) : ''; - } - - function feedItemAuthor( $row ) { - return isset( $row->user_text ) ? $row->user_text : ''; - } - - function feedTitle() { - global $wgLanguageCode, $wgSitename; - $desc = $this->getDescription(); - return "$wgSitename - $desc [$wgLanguageCode]"; - } - - function feedDesc() { - return $this->msg( 'tagline' )->text(); - } - - function feedUrl() { - return $this->getPageTitle()->getFullURL(); - } -} - -/** - * Class definition for a wanted query page like - * WantedPages, WantedTemplates, etc - */ -abstract class WantedQueryPage extends QueryPage { - function isExpensive() { - return true; - } - - function isSyndicated() { - return false; - } - - /** - * Cache page existence for performance - * @param DatabaseBase $db - * @param ResultWrapper $res - */ - function preprocessResults( $db, $res ) { - if ( !$res->numRows() ) { - return; - } - - $batch = new LinkBatch; - foreach ( $res as $row ) { - $batch->add( $row->namespace, $row->title ); - } - $batch->execute(); - - // Back to start for display - $res->seek( 0 ); - } - - /** - * Should formatResult() always check page existence, even if - * the results are fresh? This is a (hopefully temporary) - * kluge for Special:WantedFiles, which may contain false - * positives for files that exist e.g. in a shared repo (bug - * 6220). - * @return bool - */ - function forceExistenceCheck() { - return false; - } - - /** - * Format an individual result - * - * @param Skin $skin Skin to use for UI elements - * @param object $result Result row - * @return string - */ - public function formatResult( $skin, $result ) { - $title = Title::makeTitleSafe( $result->namespace, $result->title ); - if ( $title instanceof Title ) { - if ( $this->isCached() || $this->forceExistenceCheck() ) { - $pageLink = $title->isKnown() - ? '' . Linker::link( $title ) . '' - : Linker::link( - $title, - null, - array(), - array(), - array( 'broken' ) - ); - } else { - $pageLink = Linker::link( - $title, - null, - array(), - array(), - array( 'broken' ) - ); - } - return $this->getLanguage()->specialList( $pageLink, $this->makeWlhLink( $title, $result ) ); - } else { - return $this->msg( 'wantedpages-badtitle', $result->title )->escaped(); - } - } - - /** - * Make a "what links here" link for a given title - * - * @param Title $title Title to make the link for - * @param object $result Result row - * @return string - */ - private function makeWlhLink( $title, $result ) { - $wlh = SpecialPage::getTitleFor( 'Whatlinkshere', $title->getPrefixedText() ); - $label = $this->msg( 'nlinks' )->numParams( $result->value )->escaped(); - return Linker::link( $wlh, $label ); - } -} diff --git a/includes/specialpage/ImageQueryPage.php b/includes/specialpage/ImageQueryPage.php new file mode 100644 index 0000000000..b0266cbaf7 --- /dev/null +++ b/includes/specialpage/ImageQueryPage.php @@ -0,0 +1,80 @@ + + */ +abstract class ImageQueryPage extends QueryPage { + /** + * Format and output report results using the given information plus + * OutputPage + * + * @param OutputPage $out OutputPage to print to + * @param Skin $skin User skin to use [unused] + * @param DatabaseBase $dbr (read) connection to use + * @param int $res Result pointer + * @param int $num Number of available result rows + * @param int $offset Paging offset + */ + protected function outputResults( $out, $skin, $dbr, $res, $num, $offset ) { + if ( $num > 0 ) { + $gallery = ImageGalleryBase::factory(); + $gallery->setContext( $this->getContext() ); + + # $res might contain the whole 1,000 rows, so we read up to + # $num [should update this to use a Pager] + $i = 0; + foreach ( $res as $row ) { + $i++; + $namespace = isset( $row->namespace ) ? $row->namespace : NS_FILE; + $title = Title::makeTitleSafe( $namespace, $row->title ); + if ( $title instanceof Title && $title->getNamespace() == NS_FILE ) { + $gallery->add( $title, $this->getCellHtml( $row ) ); + } + if ( $i === $num ) { + break; + } + } + + $out->addHTML( $gallery->toHtml() ); + } + } + + // Gotta override this since it's abstract + function formatResult( $skin, $result ) { + } + + /** + * Get additional HTML to be shown in a results' cell + * + * @param object $row Result row + * @return string + */ + protected function getCellHtml( $row ) { + return ''; + } +} diff --git a/includes/specialpage/PageQueryPage.php b/includes/specialpage/PageQueryPage.php new file mode 100644 index 0000000000..afc02271c5 --- /dev/null +++ b/includes/specialpage/PageQueryPage.php @@ -0,0 +1,72 @@ +numRows() ) { + return; + } + + $batch = new LinkBatch(); + foreach ( $res as $row ) { + $batch->add( $row->namespace, $row->title ); + } + $batch->execute(); + + $res->seek( 0 ); + } + + /** + * Format the result as a simple link to the page + * + * @param Skin $skin + * @param object $row Result row + * @return string + */ + public function formatResult( $skin, $row ) { + global $wgContLang; + + $title = Title::makeTitleSafe( $row->namespace, $row->title ); + + if ( $title instanceof Title ) { + $text = $wgContLang->convert( $title->getPrefixedText() ); + return Linker::link( $title, htmlspecialchars( $text ) ); + } else { + return Html::element( 'span', array( 'class' => 'mw-invalidtitle' ), + Linker::getInvalidTitleDescription( $this->getContext(), $row->namespace, $row->title ) ); + } + } +} diff --git a/includes/specialpage/QueryPage.php b/includes/specialpage/QueryPage.php new file mode 100644 index 0000000000..3a83d2bad8 --- /dev/null +++ b/includes/specialpage/QueryPage.php @@ -0,0 +1,761 @@ +listoutput; + * + * @param bool $bool + */ + function setListoutput( $bool ) { + $this->listoutput = $bool; + } + + /** + * Subclasses return an SQL query here, formatted as an array with the + * following keys: + * tables => Table(s) for passing to Database::select() + * fields => Field(s) for passing to Database::select(), may be * + * conds => WHERE conditions + * options => options + * join_conds => JOIN conditions + * + * Note that the query itself should return the following three columns: + * 'namespace', 'title', and 'value'. 'value' is used for sorting. + * + * These may be stored in the querycache table for expensive queries, + * and that cached data will be returned sometimes, so the presence of + * extra fields can't be relied upon. The cached 'value' column will be + * an integer; non-numeric values are useful only for sorting the + * initial query (except if they're timestamps, see usesTimestamps()). + * + * Don't include an ORDER or LIMIT clause, they will be added. + * + * If this function is not overridden or returns something other than + * an array, getSQL() will be used instead. This is for backwards + * compatibility only and is strongly deprecated. + * @return array + * @since 1.18 + */ + function getQueryInfo() { + return null; + } + + /** + * For back-compat, subclasses may return a raw SQL query here, as a string. + * This is strongly deprecated; getQueryInfo() should be overridden instead. + * @throws MWException + * @return string + */ + function getSQL() { + /* Implement getQueryInfo() instead */ + throw new MWException( "Bug in a QueryPage: doesn't implement getQueryInfo() nor " + . "getQuery() properly" ); + } + + /** + * Subclasses return an array of fields to order by here. Don't append + * DESC to the field names, that'll be done automatically if + * sortDescending() returns true. + * @return array + * @since 1.18 + */ + function getOrderFields() { + return array( 'value' ); + } + + /** + * Does this query return timestamps rather than integers in its + * 'value' field? If true, this class will convert 'value' to a + * UNIX timestamp for caching. + * NOTE: formatRow() may get timestamps in TS_MW (mysql), TS_DB (pgsql) + * or TS_UNIX (querycache) format, so be sure to always run them + * through wfTimestamp() + * @return bool + * @since 1.18 + */ + function usesTimestamps() { + return false; + } + + /** + * Override to sort by increasing values + * + * @return bool + */ + function sortDescending() { + return true; + } + + /** + * Is this query expensive (for some definition of expensive)? Then we + * don't let it run in miser mode. $wgDisableQueryPages causes all query + * pages to be declared expensive. Some query pages are always expensive. + * + * @return bool + */ + function isExpensive() { + global $wgDisableQueryPages; + return $wgDisableQueryPages; + } + + /** + * Is the output of this query cacheable? Non-cacheable expensive pages + * will be disabled in miser mode and will not have their results written + * to the querycache table. + * @return bool + * @since 1.18 + */ + public function isCacheable() { + return true; + } + + /** + * Whether or not the output of the page in question is retrieved from + * the database cache. + * + * @return bool + */ + function isCached() { + global $wgMiserMode; + + return $this->isExpensive() && $wgMiserMode; + } + + /** + * Sometime we don't want to build rss / atom feeds. + * + * @return bool + */ + function isSyndicated() { + return true; + } + + /** + * Formats the results of the query for display. The skin is the current + * skin; you can use it for making links. The result is a single row of + * result data. You should be able to grab SQL results off of it. + * If the function returns false, the line output will be skipped. + * @param Skin $skin + * @param object $result Result row + * @return string|bool String or false to skip + */ + abstract function formatResult( $skin, $result ); + + /** + * The content returned by this function will be output before any result + * + * @return string + */ + function getPageHeader() { + return ''; + } + + /** + * If using extra form wheely-dealies, return a set of parameters here + * as an associative array. They will be encoded and added to the paging + * links (prev/next/lengths). + * + * @return array + */ + function linkParameters() { + return array(); + } + + /** + * Some special pages (for example SpecialListusers) might not return the + * current object formatted, but return the previous one instead. + * Setting this to return true will ensure formatResult() is called + * one more time to make sure that the very last result is formatted + * as well. + * @return bool + */ + function tryLastResult() { + return false; + } + + /** + * Clear the cache and save new results + * + * @param int|bool $limit Limit for SQL statement + * @param bool $ignoreErrors Whether to ignore database errors + * @throws DBError|Exception + * @return bool|int + */ + function recache( $limit, $ignoreErrors = true ) { + if ( !$this->isCacheable() ) { + return 0; + } + + $fname = get_class( $this ) . '::recache'; + $dbw = wfGetDB( DB_MASTER ); + if ( !$dbw ) { + return false; + } + + try { + # Do query + $res = $this->reallyDoQuery( $limit, false ); + $num = false; + if ( $res ) { + $num = $res->numRows(); + # Fetch results + $vals = array(); + foreach ( $res as $row ) { + if ( isset( $row->value ) ) { + if ( $this->usesTimestamps() ) { + $value = wfTimestamp( TS_UNIX, + $row->value ); + } else { + $value = intval( $row->value ); // @bug 14414 + } + } else { + $value = 0; + } + + $vals[] = array( 'qc_type' => $this->getName(), + 'qc_namespace' => $row->namespace, + 'qc_title' => $row->title, + 'qc_value' => $value ); + } + + $dbw->begin( __METHOD__ ); + # Clear out any old cached data + $dbw->delete( 'querycache', array( 'qc_type' => $this->getName() ), $fname ); + # Save results into the querycache table on the master + if ( count( $vals ) ) { + $dbw->insert( 'querycache', $vals, __METHOD__ ); + } + # Update the querycache_info record for the page + $dbw->delete( 'querycache_info', array( 'qci_type' => $this->getName() ), $fname ); + $dbw->insert( 'querycache_info', + array( 'qci_type' => $this->getName(), 'qci_timestamp' => $dbw->timestamp() ), + $fname ); + $dbw->commit( __METHOD__ ); + } + } catch ( DBError $e ) { + if ( !$ignoreErrors ) { + throw $e; // report query error + } + $num = false; // set result to false to indicate error + } + + return $num; + } + + /** + * Get a DB connection to be used for slow recache queries + * @return DatabaseBase + */ + function getRecacheDB() { + return wfGetDB( DB_SLAVE, array( $this->getName(), 'QueryPage::recache', 'vslow' ) ); + } + + /** + * Run the query and return the result + * @param int|bool $limit Numerical limit or false for no limit + * @param int|bool $offset Numerical offset or false for no offset + * @return ResultWrapper + * @since 1.18 + */ + function reallyDoQuery( $limit, $offset = false ) { + $fname = get_class( $this ) . "::reallyDoQuery"; + $dbr = $this->getRecacheDB(); + $query = $this->getQueryInfo(); + $order = $this->getOrderFields(); + + if ( $this->sortDescending() ) { + foreach ( $order as &$field ) { + $field .= ' DESC'; + } + } + + if ( is_array( $query ) ) { + $tables = isset( $query['tables'] ) ? (array)$query['tables'] : array(); + $fields = isset( $query['fields'] ) ? (array)$query['fields'] : array(); + $conds = isset( $query['conds'] ) ? (array)$query['conds'] : array(); + $options = isset( $query['options'] ) ? (array)$query['options'] : array(); + $join_conds = isset( $query['join_conds'] ) ? (array)$query['join_conds'] : array(); + + if ( count( $order ) ) { + $options['ORDER BY'] = $order; + } + + if ( $limit !== false ) { + $options['LIMIT'] = intval( $limit ); + } + + if ( $offset !== false ) { + $options['OFFSET'] = intval( $offset ); + } + + $res = $dbr->select( $tables, $fields, $conds, $fname, + $options, $join_conds + ); + } else { + // Old-fashioned raw SQL style, deprecated + $sql = $this->getSQL(); + $sql .= ' ORDER BY ' . implode( ', ', $order ); + $sql = $dbr->limitResult( $sql, $limit, $offset ); + $res = $dbr->query( $sql, $fname ); + } + + return $dbr->resultObject( $res ); + } + + /** + * Somewhat deprecated, you probably want to be using execute() + * @param int|bool $offset + * @param int|bool $limit + * @return ResultWrapper + */ + function doQuery( $offset = false, $limit = false ) { + if ( $this->isCached() && $this->isCacheable() ) { + return $this->fetchFromCache( $limit, $offset ); + } else { + return $this->reallyDoQuery( $limit, $offset ); + } + } + + /** + * Fetch the query results from the query cache + * @param int|bool $limit Numerical limit or false for no limit + * @param int|bool $offset Numerical offset or false for no offset + * @return ResultWrapper + * @since 1.18 + */ + function fetchFromCache( $limit, $offset = false ) { + $dbr = wfGetDB( DB_SLAVE ); + $options = array(); + if ( $limit !== false ) { + $options['LIMIT'] = intval( $limit ); + } + if ( $offset !== false ) { + $options['OFFSET'] = intval( $offset ); + } + if ( $this->sortDescending() ) { + $options['ORDER BY'] = 'qc_value DESC'; + } else { + $options['ORDER BY'] = 'qc_value ASC'; + } + $res = $dbr->select( 'querycache', array( 'qc_type', + 'namespace' => 'qc_namespace', + 'title' => 'qc_title', + 'value' => 'qc_value' ), + array( 'qc_type' => $this->getName() ), + __METHOD__, $options + ); + return $dbr->resultObject( $res ); + } + + public function getCachedTimestamp() { + if ( is_null( $this->cachedTimestamp ) ) { + $dbr = wfGetDB( DB_SLAVE ); + $fname = get_class( $this ) . '::getCachedTimestamp'; + $this->cachedTimestamp = $dbr->selectField( 'querycache_info', 'qci_timestamp', + array( 'qci_type' => $this->getName() ), $fname ); + } + return $this->cachedTimestamp; + } + + /** + * This is the actual workhorse. It does everything needed to make a + * real, honest-to-gosh query page. + * @param string $par + * @return int + */ + function execute( $par ) { + global $wgQueryCacheLimit, $wgDisableQueryPageUpdate; + + $user = $this->getUser(); + if ( !$this->userCanExecute( $user ) ) { + $this->displayRestrictionError(); + return; + } + + $this->setHeaders(); + $this->outputHeader(); + + $out = $this->getOutput(); + + if ( $this->isCached() && !$this->isCacheable() ) { + $out->addWikiMsg( 'querypage-disabled' ); + return 0; + } + + $out->setSyndicated( $this->isSyndicated() ); + + if ( $this->limit == 0 && $this->offset == 0 ) { + list( $this->limit, $this->offset ) = $this->getRequest()->getLimitOffset(); + } + + // @todo Use doQuery() + if ( !$this->isCached() ) { + # select one extra row for navigation + $res = $this->reallyDoQuery( $this->limit + 1, $this->offset ); + } else { + # Get the cached result, select one extra row for navigation + $res = $this->fetchFromCache( $this->limit + 1, $this->offset ); + if ( !$this->listoutput ) { + + # Fetch the timestamp of this update + $ts = $this->getCachedTimestamp(); + $lang = $this->getLanguage(); + $maxResults = $lang->formatNum( $wgQueryCacheLimit ); + + if ( $ts ) { + $updated = $lang->userTimeAndDate( $ts, $user ); + $updateddate = $lang->userDate( $ts, $user ); + $updatedtime = $lang->userTime( $ts, $user ); + $out->addMeta( 'Data-Cache-Time', $ts ); + $out->addJsConfigVars( 'dataCacheTime', $ts ); + $out->addWikiMsg( 'perfcachedts', $updated, $updateddate, $updatedtime, $maxResults ); + } else { + $out->addWikiMsg( 'perfcached', $maxResults ); + } + + # If updates on this page have been disabled, let the user know + # that the data set won't be refreshed for now + if ( is_array( $wgDisableQueryPageUpdate ) + && in_array( $this->getName(), $wgDisableQueryPageUpdate ) + ) { + $out->wrapWikiMsg( + "
\n$1\n
", + 'querypage-no-updates' + ); + } + } + } + + $this->numRows = $res->numRows(); + + $dbr = wfGetDB( DB_SLAVE ); + $this->preprocessResults( $dbr, $res ); + + $out->addHTML( Xml::openElement( 'div', array( 'class' => 'mw-spcontent' ) ) ); + + # Top header and navigation + if ( $this->shownavigation ) { + $out->addHTML( $this->getPageHeader() ); + if ( $this->numRows > 0 ) { + $out->addHTML( $this->msg( 'showingresultsinrange' )->numParams( + min( $this->numRows, $this->limit ), # do not show the one extra row, if exist + $this->offset + 1, ( min( $this->numRows, $this->limit ) + $this->offset ) )->parseAsBlock() ); + # Disable the "next" link when we reach the end + $paging = $this->getLanguage()->viewPrevNext( $this->getPageTitle( $par ), $this->offset, + $this->limit, $this->linkParameters(), ( $this->numRows <= $this->limit ) ); + $out->addHTML( '

' . $paging . '

' ); + } else { + # No results to show, so don't bother with "showing X of Y" etc. + # -- just let the user know and give up now + $out->addWikiMsg( 'specialpage-empty' ); + $out->addHTML( Xml::closeElement( 'div' ) ); + return; + } + } + + # The actual results; specialist subclasses will want to handle this + # with more than a straight list, so we hand them the info, plus + # an OutputPage, and let them get on with it + $this->outputResults( $out, + $this->getSkin(), + $dbr, # Should use a ResultWrapper for this + $res, + min( $this->numRows, $this->limit ), # do not format the one extra row, if exist + $this->offset ); + + # Repeat the paging links at the bottom + if ( $this->shownavigation ) { + $out->addHTML( '

' . $paging . '

' ); + } + + $out->addHTML( Xml::closeElement( 'div' ) ); + + return min( $this->numRows, $this->limit ); # do not return the one extra row, if exist + } + + /** + * Format and output report results using the given information plus + * OutputPage + * + * @param OutputPage $out OutputPage to print to + * @param Skin $skin User skin to use + * @param DatabaseBase $dbr Database (read) connection to use + * @param ResultWrapper $res Result pointer + * @param int $num Number of available result rows + * @param int $offset Paging offset + */ + protected function outputResults( $out, $skin, $dbr, $res, $num, $offset ) { + global $wgContLang; + + if ( $num > 0 ) { + $html = array(); + if ( !$this->listoutput ) { + $html[] = $this->openList( $offset ); + } + + # $res might contain the whole 1,000 rows, so we read up to + # $num [should update this to use a Pager] + // @codingStandardsIgnoreStart Generic.CodeAnalysis.ForLoopWithTestFunctionCall.NotAllowed + for ( $i = 0; $i < $num && $row = $res->fetchObject(); $i++ ) { + // @codingStandardsIgnoreEnd + $line = $this->formatResult( $skin, $row ); + if ( $line ) { + $attr = ( isset( $row->usepatrol ) && $row->usepatrol && $row->patrolled == 0 ) + ? ' class="not-patrolled"' + : ''; + $html[] = $this->listoutput + ? $line + : "{$line}\n"; + } + } + + # Flush the final result + if ( $this->tryLastResult() ) { + $row = null; + $line = $this->formatResult( $skin, $row ); + if ( $line ) { + $attr = ( isset( $row->usepatrol ) && $row->usepatrol && $row->patrolled == 0 ) + ? ' class="not-patrolled"' + : ''; + $html[] = $this->listoutput + ? $line + : "{$line}\n"; + } + } + + if ( !$this->listoutput ) { + $html[] = $this->closeList(); + } + + $html = $this->listoutput + ? $wgContLang->listToText( $html ) + : implode( '', $html ); + + $out->addHTML( $html ); + } + } + + /** + * @param int $offset + * @return string + */ + function openList( $offset ) { + return "\n
    \n"; + } + + /** + * @return string + */ + function closeList() { + return "
\n"; + } + + /** + * Do any necessary preprocessing of the result object. + * @param DatabaseBase $db + * @param ResultWrapper $res + */ + function preprocessResults( $db, $res ) { + } + + /** + * Similar to above, but packaging in a syndicated feed instead of a web page + * @param string $class + * @param int $limit + * @return bool + */ + function doFeed( $class = '', $limit = 50 ) { + global $wgFeed, $wgFeedClasses, $wgFeedLimit; + + if ( !$wgFeed ) { + $this->getOutput()->addWikiMsg( 'feed-unavailable' ); + return false; + } + + $limit = min( $limit, $wgFeedLimit ); + + if ( isset( $wgFeedClasses[$class] ) ) { + /** @var RSSFeed|AtomFeed $feed */ + $feed = new $wgFeedClasses[$class]( + $this->feedTitle(), + $this->feedDesc(), + $this->feedUrl() ); + $feed->outHeader(); + + $res = $this->reallyDoQuery( $limit, 0 ); + foreach ( $res as $obj ) { + $item = $this->feedResult( $obj ); + if ( $item ) { + $feed->outItem( $item ); + } + } + + $feed->outFooter(); + return true; + } else { + return false; + } + } + + /** + * Override for custom handling. If the titles/links are ok, just do + * feedItemDesc() + * @param object $row + * @return FeedItem|null + */ + function feedResult( $row ) { + if ( !isset( $row->title ) ) { + return null; + } + $title = Title::makeTitle( intval( $row->namespace ), $row->title ); + if ( $title ) { + $date = isset( $row->timestamp ) ? $row->timestamp : ''; + $comments = ''; + if ( $title ) { + $talkpage = $title->getTalkPage(); + $comments = $talkpage->getFullURL(); + } + + return new FeedItem( + $title->getPrefixedText(), + $this->feedItemDesc( $row ), + $title->getFullURL(), + $date, + $this->feedItemAuthor( $row ), + $comments ); + } else { + return null; + } + } + + function feedItemDesc( $row ) { + return isset( $row->comment ) ? htmlspecialchars( $row->comment ) : ''; + } + + function feedItemAuthor( $row ) { + return isset( $row->user_text ) ? $row->user_text : ''; + } + + function feedTitle() { + global $wgLanguageCode, $wgSitename; + $desc = $this->getDescription(); + return "$wgSitename - $desc [$wgLanguageCode]"; + } + + function feedDesc() { + return $this->msg( 'tagline' )->text(); + } + + function feedUrl() { + return $this->getPageTitle()->getFullURL(); + } +} diff --git a/includes/specialpage/WantedQueryPage.php b/includes/specialpage/WantedQueryPage.php new file mode 100644 index 0000000000..678c803d19 --- /dev/null +++ b/includes/specialpage/WantedQueryPage.php @@ -0,0 +1,117 @@ +numRows() ) { + return; + } + + $batch = new LinkBatch; + foreach ( $res as $row ) { + $batch->add( $row->namespace, $row->title ); + } + $batch->execute(); + + // Back to start for display + $res->seek( 0 ); + } + + /** + * Should formatResult() always check page existence, even if + * the results are fresh? This is a (hopefully temporary) + * kluge for Special:WantedFiles, which may contain false + * positives for files that exist e.g. in a shared repo (bug + * 6220). + * @return bool + */ + function forceExistenceCheck() { + return false; + } + + /** + * Format an individual result + * + * @param Skin $skin Skin to use for UI elements + * @param object $result Result row + * @return string + */ + public function formatResult( $skin, $result ) { + $title = Title::makeTitleSafe( $result->namespace, $result->title ); + if ( $title instanceof Title ) { + if ( $this->isCached() || $this->forceExistenceCheck() ) { + $pageLink = $title->isKnown() + ? '' . Linker::link( $title ) . '' + : Linker::link( + $title, + null, + array(), + array(), + array( 'broken' ) + ); + } else { + $pageLink = Linker::link( + $title, + null, + array(), + array(), + array( 'broken' ) + ); + } + return $this->getLanguage()->specialList( $pageLink, $this->makeWlhLink( $title, $result ) ); + } else { + return $this->msg( 'wantedpages-badtitle', $result->title )->escaped(); + } + } + + /** + * Make a "what links here" link for a given title + * + * @param Title $title Title to make the link for + * @param object $result Result row + * @return string + */ + private function makeWlhLink( $title, $result ) { + $wlh = SpecialPage::getTitleFor( 'Whatlinkshere', $title->getPrefixedText() ); + $label = $this->msg( 'nlinks' )->numParams( $result->value )->escaped(); + return Linker::link( $wlh, $label ); + } +} diff --git a/maintenance/updateSpecialPages.php b/maintenance/updateSpecialPages.php index fae9063c00..61642828c9 100644 --- a/maintenance/updateSpecialPages.php +++ b/maintenance/updateSpecialPages.php @@ -34,8 +34,8 @@ class UpdateSpecialPages extends Maintenance { parent::__construct(); $this->addOption( 'list', 'List special page names' ); $this->addOption( 'only', 'Only update "page"; case sensitive, ' . - 'check correct case by calling this script with --list or on ' . - 'includes/QueryPage.php. Ex: --only=BrokenRedirects', false, true ); + 'check correct case by calling this script with --list. ' . + 'Ex: --only=BrokenRedirects', false, true ); $this->addOption( 'override', 'Also update pages that have updates disabled' ); }