X-Git-Url: https://git.heureux-cyclage.org/?a=blobdiff_plain;f=includes%2FQueryPage.php;h=ef675c0e53c473f14a6d45082e2034cb8ae52193;hb=e5167d224bd3a192b113d834055686cc7aeb622e;hp=16dc7c04698c2cb4f0280a94104e8f5b9f688335;hpb=0236e8f144053a08d68f4acb3863cc22d93b2cf6;p=lhc%2Fweb%2Fwiklou.git diff --git a/includes/QueryPage.php b/includes/QueryPage.php index 16dc7c0469..ef675c0e53 100644 --- a/includes/QueryPage.php +++ b/includes/QueryPage.php @@ -15,40 +15,45 @@ global $wgQueryPages; // not redundant $wgQueryPages = array( // QueryPage subclass Special page name Limit (false for none, none for the default) -//---------------------------------------------------------------------------- +// ---------------------------------------------------------------------------- array( 'AncientPagesPage', 'Ancientpages' ), array( 'BrokenRedirectsPage', 'BrokenRedirects' ), array( 'DeadendPagesPage', 'Deadendpages' ), array( 'DisambiguationsPage', 'Disambiguations' ), array( 'DoubleRedirectsPage', 'DoubleRedirects' ), - array( 'ListredirectsPage', 'Listredirects' ), + array( 'FileDuplicateSearchPage', 'FileDuplicateSearch' ), + array( 'LinkSearchPage', 'LinkSearch' ), + array( 'ListredirectsPage', 'Listredirects' ), array( 'LonelyPagesPage', 'Lonelypages' ), array( 'LongPagesPage', 'Longpages' ), + array( 'MIMEsearchPage', 'MIMEsearch' ), array( 'MostcategoriesPage', 'Mostcategories' ), array( 'MostimagesPage', 'Mostimages' ), array( 'MostlinkedCategoriesPage', 'Mostlinkedcategories' ), - array( 'SpecialMostlinkedtemplates', 'Mostlinkedtemplates' ), + array( 'MostlinkedtemplatesPage', 'Mostlinkedtemplates' ), array( 'MostlinkedPage', 'Mostlinked' ), array( 'MostrevisionsPage', 'Mostrevisions' ), array( 'FewestrevisionsPage', 'Fewestrevisions' ), array( 'ShortPagesPage', 'Shortpages' ), array( 'UncategorizedCategoriesPage', 'Uncategorizedcategories' ), array( 'UncategorizedPagesPage', 'Uncategorizedpages' ), - array( 'UncategorizedImagesPage', 'Uncategorizedimages' ), - array( 'UncategorizedTemplatesPage', 'Uncategorizedtemplates' ), + array( 'UncategorizedImagesPage', 'Uncategorizedimages' ), + array( 'UncategorizedTemplatesPage', 'Uncategorizedtemplates' ), array( 'UnusedCategoriesPage', 'Unusedcategories' ), array( 'UnusedimagesPage', 'Unusedimages' ), array( 'WantedCategoriesPage', 'Wantedcategories' ), + array( 'WantedFilesPage', 'Wantedfiles' ), array( 'WantedPagesPage', 'Wantedpages' ), + array( 'WantedTemplatesPage', 'Wantedtemplates' ), array( 'UnwatchedPagesPage', 'Unwatchedpages' ), - array( 'UnusedtemplatesPage', 'Unusedtemplates' ), - array( 'WithoutInterwikiPage', 'Withoutinterwiki' ), + array( 'UnusedtemplatesPage', 'Unusedtemplates' ), + array( 'WithoutInterwikiPage', 'Withoutinterwiki' ), ); wfRunHooks( 'wgQueryPages', array( &$wgQueryPages ) ); global $wgDisableCounters; if ( !$wgDisableCounters ) - $wgQueryPages[] = array( 'PopularPagesPage', 'Popularpages' ); + $wgQueryPages[] = array( 'PopularPagesPage', 'Popularpages' ); /** @@ -57,7 +62,7 @@ if ( !$wgDisableCounters ) * subclasses derive from it. * @ingroup SpecialPage */ -class QueryPage { +abstract class QueryPage extends SpecialPage { /** * Whether or not we want plain listoutput rather than an ordered list * @@ -74,78 +79,130 @@ class QueryPage { var $limit = 0; /** - * A mutator for $this->listoutput; - * - * @param bool $bool + * The number of rows returned by the query. Reading this variable + * only makes sense in functions that are run after the query has been + * done, such as preprocessResults() and formatRow(). */ - function setListoutput( $bool ) { - $this->listoutput = $bool; - } + protected $numRows; + + protected $cachedTimestamp = null; /** - * Subclasses return their name here. Make sure the name is also - * specified in SpecialPage.php and in Language.php as a language message - * param. + * Wheter to show prev/next links */ - function getName() { - return ''; - } + protected $shownavigation = true; /** - * Return title object representing this page + * A mutator for $this->listoutput; * - * @return Title + * @param $bool Boolean */ - function getTitle() { - return SpecialPage::getTitleFor( $this->getName() ); + function setListoutput( $bool ) { + $this->listoutput = $bool; } /** - * Subclasses return an SQL query here. + * 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 four columns: - * 'type' (your special page's name), 'namespace', 'title', and 'value' - * *in that order*. 'value' is used for sorting. + * 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. + * 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. * - * Don't include an ORDER or LIMIT clause, this 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 stronly deprecated; getQueryInfo() should be overridden instead. + * @return string */ function getSQL() { - return "SELECT 'sample' as type, 0 as namespace, 'Sample result' as title, 42 as value"; + /* 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 Boolean */ function sortDescending() { return true; } - function getOrder() { - return ' ORDER BY value ' . - ($this->sortDescending() ? 'DESC' : ''); - } - /** * 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 Boolean */ - function isExpensive( ) { + function isExpensive() { global $wgDisableQueryPages; return $wgDisableQueryPages; } /** - * Whether or not the output of the page in question is retrived from + * 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 Boolean + * @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 + * @return Boolean */ function isCached() { global $wgMiserMode; @@ -155,6 +212,8 @@ class QueryPage { /** * Sometime we dont want to build rss / atom feeds. + * + * @return Boolean */ function isSyndicated() { return true; @@ -164,16 +223,22 @@ class QueryPage { * 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 return "false", the line output will be skipped. + * If the function returns false, the line output will be skipped. + * @param $skin Skin + * @param $result object Result row + * @return mixed String or false to skip + * + * @param $skin Skin object + * @param $result Object: database row */ - function formatResult( $skin, $result ) { - return ''; - } + abstract function formatResult( $skin, $result ); /** * The content returned by this function will be output before any result + * + * @return String */ - function getPageHeader( ) { + function getPageHeader() { return ''; } @@ -181,7 +246,8 @@ class QueryPage { * 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 + * + * @return Array */ function linkParameters() { return array(); @@ -190,26 +256,32 @@ class QueryPage { /** * 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 call one more time wfFormatResult to - * be sure that the very last result is formatted and shown. + * 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. */ - function tryLastResult( ) { + function tryLastResult() { return false; } /** * Clear the cache and save new results + * + * @param $limit Integer: limit for SQL statement + * @param $ignoreErrors Boolean: whether to ignore database errors */ function recache( $limit, $ignoreErrors = true ) { - $fname = get_class($this) . '::recache'; + if ( !$this->isCacheable() ) { + return 0; + } + + $fname = get_class( $this ) . '::recache'; $dbw = wfGetDB( DB_MASTER ); - $dbr = wfGetDB( DB_SLAVE, array( $this->getName(), 'QueryPage::recache', 'vslow' ) ); + $dbr = wfGetDB( DB_SLAVE, array( $this->getName(), __METHOD__, 'vslow' ) ); if ( !$dbw || !$dbr ) { return false; } - $querycache = $dbr->tableName( 'querycache' ); - if ( $ignoreErrors ) { $ignoreW = $dbw->ignoreErrors( true ); $ignoreR = $dbr->ignoreErrors( true ); @@ -218,46 +290,37 @@ class QueryPage { # Clear out any old cached data $dbw->delete( 'querycache', array( 'qc_type' => $this->getName() ), $fname ); # Do query - $sql = $this->getSQL() . $this->getOrder(); - if ($limit !== false) - $sql = $dbr->limitResult($sql, $limit, 0); - $res = $dbr->query($sql, $fname); + $res = $this->reallyDoQuery( $limit, false ); $num = false; if ( $res ) { $num = $dbr->numRows( $res ); # Fetch results - $insertSql = "INSERT INTO $querycache (qc_type,qc_namespace,qc_title,qc_value) VALUES "; - $first = true; + $vals = array(); while ( $res && $row = $dbr->fetchObject( $res ) ) { - if ( $first ) { - $first = false; - } else { - $insertSql .= ','; - } if ( isset( $row->value ) ) { - $value = $row->value; + if ( $this->usesTimestamps() ) { + $value = wfTimestamp( TS_UNIX, + $row->value ); + } else { + $value = intval( $row->value ); // @bug 14414 + } } else { $value = 0; } - $insertSql .= '(' . - $dbw->addQuotes( $row->type ) . ',' . - $dbw->addQuotes( $row->namespace ) . ',' . - $dbw->addQuotes( $row->title ) . ',' . - $dbw->addQuotes( $value ) . ')'; + $vals[] = array( 'qc_type' => $this->getName(), + 'qc_namespace' => $row->namespace, + 'qc_title' => $row->title, + 'qc_value' => $value ); } # Save results into the querycache table on the master - if ( !$first ) { - if ( !$dbw->query( $insertSql, $fname ) ) { + if ( count( $vals ) ) { + if ( !$dbw->insert( 'querycache', $vals, __METHOD__ ) ) { // Set result to false to indicate error - $dbr->freeResult( $res ); - $res = false; + $num = false; } } - if ( $res ) { - $dbr->freeResult( $res ); - } if ( $ignoreErrors ) { $dbw->ignoreErrors( $ignoreW ); $dbr->ignoreErrors( $ignoreR ); @@ -271,85 +334,185 @@ class QueryPage { return $num; } + /** + * Run the query and return the result + * @param $limit mixed Numerical limit or false for no limit + * @param $offset mixed Numerical offset or false for no offset + * @return ResultWrapper + * @since 1.18 + */ + function reallyDoQuery( $limit, $offset = false ) { + $fname = get_class( $this ) . "::reallyDoQuery"; + $dbr = wfGetDB( DB_SLAVE ); + $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'] = implode( ', ', $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() + */ + 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 $limit mixed Numerical limit or false for no limit + * @param $offset mixed 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', + 'qc_namespace AS namespace', + 'qc_title AS title', + 'qc_value AS 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 $offset database query offset - * @param $limit database query limit - * @param $shownavigation show navigation like "next 200"? */ - function doQuery( $offset, $limit, $shownavigation=true ) { - global $wgUser, $wgOut, $wgLang, $wgContLang; - - $this->offset = $offset; - $this->limit = $limit; + function execute( $par ) { + if ( !$this->userCanExecute( $this->getUser() ) ) { + $this->displayRestrictionError(); + return; + } - $sname = $this->getName(); - $fname = get_class($this) . '::doQuery'; + if ( $this->limit == 0 && $this->offset == 0 ) { + list( $this->limit, $this->offset ) = $this->getRequest()->getLimitOffset(); + } $dbr = wfGetDB( DB_SLAVE ); - $wgOut->setSyndicated( $this->isSyndicated() ); + $this->setHeaders(); + $out = $this->getOutput(); + $out->setSyndicated( $this->isSyndicated() ); + if ( $this->isCached() && !$this->isCacheable() ) { + $out->setSyndicated( false ); + $out->addWikiMsg( 'querypage-disabled' ); + return 0; + } + + // TODO: Use doQuery() + // $res = null; if ( !$this->isCached() ) { - $sql = $this->getSQL(); + $res = $this->reallyDoQuery( $this->limit, $this->offset ); } else { # Get the cached result - $querycache = $dbr->tableName( 'querycache' ); - $type = $dbr->strencode( $sname ); - $sql = - "SELECT qc_type as type, qc_namespace as namespace,qc_title as title, qc_value as value - FROM $querycache WHERE qc_type='$type'"; - - if( !$this->listoutput ) { + $res = $this->fetchFromCache( $this->limit, $this->offset ); + if ( !$this->listoutput ) { # Fetch the timestamp of this update - $tRes = $dbr->select( 'querycache_info', array( 'qci_timestamp' ), array( 'qci_type' => $type ), $fname ); - $tRow = $dbr->fetchObject( $tRes ); - - if( $tRow ) { - $updated = $wgLang->timeAndDate( $tRow->qci_timestamp, true, true ); - $wgOut->addMeta( 'Data-Cache-Time', $tRow->qci_timestamp ); - $wgOut->addInlineScript( "var dataCacheTime = '{$tRow->qci_timestamp}';" ); - $wgOut->addWikiMsg( 'perfcachedts', $updated ); + $ts = $this->getCachedTimestamp(); + + if ( $ts ) { + $lang = $this->getLanguage(); + $updated = $lang->timeanddate( $ts, true, true ); + $updateddate = $lang->date( $ts, true, true ); + $updatedtime = $lang->time( $ts, true, true ); + $out->addMeta( 'Data-Cache-Time', $ts ); + $out->addInlineScript( "var dataCacheTime = '$ts';" ); + $out->addWikiMsg( 'perfcachedts', $updated, $updateddate, $updatedtime ); } else { - $wgOut->addWikiMsg( 'perfcached' ); + $out->addWikiMsg( 'perfcached' ); } # If updates on this page have been disabled, let the user know # that the data set won't be refreshed for now global $wgDisableQueryPageUpdate; - if( is_array( $wgDisableQueryPageUpdate ) && in_array( $this->getName(), $wgDisableQueryPageUpdate ) ) { - $wgOut->addWikiMsg( 'querypage-no-updates' ); + if ( is_array( $wgDisableQueryPageUpdate ) && in_array( $this->getName(), $wgDisableQueryPageUpdate ) ) { + $out->addWikiMsg( 'querypage-no-updates' ); } } } - $sql .= $this->getOrder(); - $sql = $dbr->limitResult($sql, $limit, $offset); - $res = $dbr->query( $sql ); - $num = $dbr->numRows($res); + $this->numRows = $dbr->numRows( $res ); $this->preprocessResults( $dbr, $res ); - $wgOut->addHtml( XML::openElement( 'div', array('class' => 'mw-spcontent') ) ); + $out->addHTML( Xml::openElement( 'div', array( 'class' => 'mw-spcontent' ) ) ); # Top header and navigation - if( $shownavigation ) { - $wgOut->addHtml( $this->getPageHeader() ); - if( $num > 0 ) { - $wgOut->addHtml( '

' . wfShowingResults( $offset, $num ) . '

' ); + if ( $this->shownavigation ) { + $out->addHTML( $this->getPageHeader() ); + if ( $this->numRows > 0 ) { + $out->addHTML( $this->msg( 'showingresults' )->numParams( + $this->numRows, $this->offset + 1 )->parseAsBlock() ); # Disable the "next" link when we reach the end - $paging = wfViewPrevNext( $offset, $limit, $wgContLang->specialPage( $sname ), - wfArrayToCGI( $this->linkParameters() ), ( $num < $limit ) ); - $wgOut->addHtml( '

' . $paging . '

' ); + $paging = $this->getLanguage()->viewPrevNext( $this->getTitle( $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 - $wgOut->addHtml( '

' . wfMsgHtml( 'specialpage-empty' ) . '

' ); - $wgOut->addHtml( XML::closeElement( 'div' ) ); + $out->addWikiMsg( 'specialpage-empty' ); + $out->addHTML( Xml::closeElement( 'div' ) ); return; } } @@ -357,47 +520,48 @@ class QueryPage { # 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( $wgOut, - $wgUser->getSkin(), + $this->outputResults( $out, + $this->getSkin(), $dbr, # Should use a ResultWrapper for this $res, - $dbr->numRows( $res ), - $offset ); + $this->numRows, + $this->offset ); # Repeat the paging links at the bottom - if( $shownavigation ) { - $wgOut->addHtml( '

' . $paging . '

' ); + if ( $this->shownavigation ) { + $out->addHTML( '

' . $paging . '

' ); } - $wgOut->addHtml( XML::closeElement( 'div' ) ); + $out->addHTML( Xml::closeElement( 'div' ) ); - return $num; + return $this->numRows; } /** * 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 Database $dbr Database (read) connection to use - * @param int $res Result pointer - * @param int $num Number of available result rows - * @param int $offset Paging offset + * @param $out OutputPage to print to + * @param $skin Skin: user skin to use + * @param $dbr Database (read) connection to use + * @param $res Integer: result pointer + * @param $num Integer: number of available result rows + * @param $offset Integer: paging offset */ protected function outputResults( $out, $skin, $dbr, $res, $num, $offset ) { global $wgContLang; - if( $num > 0 ) { + if ( $num > 0 ) { $html = array(); - if( !$this->listoutput ) + 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] - for( $i = 0; $i < $num && $row = $dbr->fetchObject( $res ); $i++ ) { + for ( $i = 0; $i < $num && $row = $dbr->fetchObject( $res ); $i++ ) { $line = $this->formatResult( $skin, $row ); - if( $line ) { + if ( $line ) { $attr = ( isset( $row->usepatrol ) && $row->usepatrol && $row->patrolled == 0 ) ? ' class="not-patrolled"' : ''; @@ -408,10 +572,10 @@ class QueryPage { } # Flush the final result - if( $this->tryLastResult() ) { + if ( $this->tryLastResult() ) { $row = null; $line = $this->formatResult( $skin, $row ); - if( $line ) { + if ( $line ) { $attr = ( isset( $row->usepatrol ) && $row->usepatrol && $row->patrolled == 0 ) ? ' class="not-patrolled"' : ''; @@ -421,21 +585,29 @@ class QueryPage { } } - if( !$this->listoutput ) + if ( !$this->listoutput ) { $html[] = $this->closeList(); + } $html = $this->listoutput ? $wgContLang->listToText( $html ) : implode( '', $html ); - $out->addHtml( $html ); + $out->addHTML( $html ); } } + /** + * @param $offset + * @return string + */ function openList( $offset ) { return "\n
    \n"; } + /** + * @return string + */ function closeList() { return "
\n"; } @@ -452,32 +624,29 @@ class QueryPage { global $wgFeed, $wgFeedClasses; if ( !$wgFeed ) { - global $wgOut; - $wgOut->addWikiMsg( 'feed-unavailable' ); + $this->getOutput()->addWikiMsg( 'feed-unavailable' ); return; } - + global $wgFeedLimit; - if( $limit > $wgFeedLimit ) { + if ( $limit > $wgFeedLimit ) { $limit = $wgFeedLimit; } - if( isset($wgFeedClasses[$class]) ) { + if ( isset( $wgFeedClasses[$class] ) ) { $feed = new $wgFeedClasses[$class]( $this->feedTitle(), $this->feedDesc(), $this->feedUrl() ); $feed->outHeader(); - $dbr = wfGetDB( DB_SLAVE ); - $sql = $this->getSQL() . $this->getOrder(); - $sql = $dbr->limitResult( $sql, $limit, 0 ); - $res = $dbr->query( $sql, 'QueryPage::doFeed' ); - while( $obj = $dbr->fetchObject( $res ) ) { + $res = $this->reallyDoQuery( $limit, 0 ); + foreach ( $res as $obj ) { $item = $this->feedResult( $obj ); - if( $item ) $feed->outItem( $item ); + if ( $item ) { + $feed->outItem( $item ); + } } - $dbr->freeResult( $res ); $feed->outFooter(); return true; @@ -491,14 +660,14 @@ class QueryPage { * feedItemDesc() */ function feedResult( $row ) { - if( !isset( $row->title ) ) { - return NULL; + if ( !isset( $row->title ) ) { + return null; } $title = Title::MakeTitle( intval( $row->namespace ), $row->title ); - if( $title ) { + if ( $title ) { $date = isset( $row->timestamp ) ? $row->timestamp : ''; $comments = ''; - if( $title ) { + if ( $title ) { $talkpage = $title->getTalkPage(); $comments = $talkpage->getFullURL(); } @@ -509,9 +678,9 @@ class QueryPage { $title->getFullURL(), $date, $this->feedItemAuthor( $row ), - $comments); + $comments ); } else { - return NULL; + return null; } } @@ -524,18 +693,106 @@ class QueryPage { } function feedTitle() { - global $wgContLanguageCode, $wgSitename; - $page = SpecialPage::getPage( $this->getName() ); - $desc = $page->getDescription(); - return "$wgSitename - $desc [$wgContLanguageCode]"; + global $wgLanguageCode, $wgSitename; + $desc = $this->getDescription(); + return "$wgSitename - $desc [$wgLanguageCode]"; } function feedDesc() { - return wfMsg( 'tagline' ); + return $this->msg( 'tagline' )->text(); } function feedUrl() { - $title = SpecialPage::getTitleFor( $this->getName() ); - return $title->getFullURL(); + return $this->getTitle()->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 + */ + function preprocessResults( $db, $res ) { + $batch = new LinkBatch; + foreach ( $res as $row ) { + $batch->add( $row->namespace, $row->title ); + } + $batch->execute(); + + // Back to start for display + if ( $db->numRows( $res ) > 0 ) + // If there are no rows we get an error seeking. + $db->dataSeek( $res, 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). + */ + function forceExistenceCheck() { + return false; + } + + /** + * Format an individual result + * + * @param $skin Skin to use for UI elements + * @param $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 to make the link for + * @param $result Object: 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 ); } }