X-Git-Url: http://git.heureux-cyclage.org/?a=blobdiff_plain;f=includes%2FQueryPage.php;h=2371444aba2348124d9ffb6e7dcbc15edb6c366d;hb=ce335a08902456a66726ab7ac092fa466e3918fc;hp=027df13741a28ecc850ffb8e00aa5019f2c4c96c;hpb=9194bdac2ecf238520ebdd7d8b276f4bc6e4b061;p=lhc%2Fweb%2Fwiklou.git diff --git a/includes/QueryPage.php b/includes/QueryPage.php index 027df13741..2371444aba 100644 --- a/includes/QueryPage.php +++ b/includes/QueryPage.php @@ -1,6 +1,7 @@ getName(), 'QueryPage::recache', 'vslow' ) ); + if ( !$dbw || !$dbr ) { + return false; + } + + $querycache = $dbr->tableName( 'querycache' ); + + if ( $ignoreErrors ) { + $ignoreW = $dbw->ignoreErrors( true ); + $ignoreR = $dbr->ignoreErrors( true ); + } + + # Clear out any old cached data + $dbw->delete( 'querycache', array( 'qc_type' => $this->getName() ), $fname ); + # Do query + $res = $dbr->query( $this->getSQL() . $this->getOrder() . $dbr->limitResult( 1000,0 ), $fname ); + $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; + while ( $res && $row = $dbr->fetchObject( $res ) ) { + if ( $first ) { + $first = false; + } else { + $insertSql .= ','; + } + if ( isset( $row->value ) ) { + $value = $row->value; + } else { + $value = ''; + } + + $insertSql .= '(' . + $dbw->addQuotes( $row->type ) . ',' . + $dbw->addQuotes( $row->namespace ) . ',' . + $dbw->addQuotes( $row->title ) . ',' . + $dbw->addQuotes( $value ) . ')'; + } + + # Save results into the querycache table on the master + if ( !$first ) { + if ( !$dbw->query( $insertSql, $fname ) ) { + // Set result to false to indicate error + $dbr->freeResult( $res ); + $res = false; + } + } + if ( $res ) { + $dbr->freeResult( $res ); + } + if ( $ignoreErrors ) { + $dbw->ignoreErrors( $ignoreW ); + $dbr->ignoreErrors( $ignoreR ); + } + } + return $num; + } /** * This is the actual workhorse. It does everything needed to make a @@ -88,9 +209,10 @@ class QueryPage { * * @param $offset database query offset * @param $limit database query limit + * @param $shownavigation show navigation like "next 200"? */ - function doQuery( $offset, $limit ) { - global $wgUser, $wgOut, $wgLang, $wgRequest; + function doQuery( $offset, $limit, $shownavigation=true ) { + global $wgUser, $wgOut, $wgLang, $wgRequest, $wgContLang; global $wgMiserMode; $sname = $this->getName(); @@ -100,89 +222,70 @@ class QueryPage { $dbw =& wfGetDB( DB_MASTER ); $querycache = $dbr->tableName( 'querycache' ); - $wgOut->setSyndicated( true ); - $res = false; + $wgOut->setSyndicated( $this->isSyndicated() ); if ( $this->isExpensive() ) { - $recache = $wgRequest->getBool( 'recache' ); - if( $recache ) { - # Clear out any old cached data - $dbw->delete( 'querycache', array( 'qc_type' => $sname ), $fname ); - - # Do query on the (possibly out of date) slave server - $maxstored = 1000; - $res = $dbr->query( $sql . $this->getOrder() . $dbr->limitResult( $maxstored,0 ), $fname ); - - # Fetch results - $insertSql = "INSERT INTO $querycache (qc_type,qc_namespace,qc_title,qc_value) VALUES "; - $first = true; - while ( $row = $dbr->fetchObject( $res ) ) { - if ( $first ) { - $first = false; - } else { - $insertSql .= ','; - } - $insertSql .= '(' . - $dbw->addQuotes( $row->type ) . ',' . - $dbw->addQuotes( $row->namespace ) . ',' . - $dbw->addQuotes( $row->title ) . ',' . - $dbw->addQuotes( $row->value ) . ')'; - } - - # Save results into the querycache table on the master - $dbw->query( $insertSql, $fname ); - - # Set result pointer to allow reading for display - $numRows = $dbr->numRows( $res ); - if ( $numRows <= $offset ) { - $num = 0; - } else { - $dbr->dataSeek( $res, $offset ); - $num = max( $limit, $numRows - $offset ); - } - } - if( $wgMiserMode || $recache ) { + // Disabled recache parameter due to retry problems -- TS + if( $wgMiserMode ) { $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( $wgMiserMode ) { $wgOut->addWikiText( wfMsg( 'perfcached' ) ); } } - if ( $res === false ) { - $res = $dbr->query( $sql . $this->getOrder() . - $dbr->limitResult( $limit,$offset ), $fname ); - $num = $dbr->numRows($res); - } - - + + $res = $dbr->query( $sql . $this->getOrder() . + $dbr->limitResult( $limit,$offset ), $fname ); + $num = $dbr->numRows($res); + $sk = $wgUser->getSkin( ); - $wgOut->addHTML( $this->getPageHeader() ); - - $top = wfShowingResults( $offset, $num); - $wgOut->addHTML( "

{$top}\n" ); - - # often disable 'next' link when we reach the end - if($num < $limit) { $atend = true; } else { $atend = false; } - - $sl = wfViewPrevNext( $offset, $limit , $wgLang->specialPage( $sname ), "" ,$atend ); - $wgOut->addHTML( "
{$sl}

\n" ); + if($shownavigation) { + $wgOut->addHTML( $this->getPageHeader() ); + $top = wfShowingResults( $offset, $num); + $wgOut->addHTML( "

{$top}\n" ); + + # often disable 'next' link when we reach the end + if($num < $limit) { $atend = true; } else { $atend = false; } + + $sl = wfViewPrevNext( $offset, $limit , + $wgContLang->specialPage( $sname ), + wfArrayToCGI( $this->linkParameters() ), $atend ); + $wgOut->addHTML( "
{$sl}

\n" ); + } + if ( $num > 0 ) { + $s = "
    "; + + # Only read at most $num rows, because $res may contain the whole 1000 + for ( $i = 0; $i < $num && $obj = $dbr->fetchObject( $res ); $i++ ) { + $format = $this->formatResult( $sk, $obj ); + if ( $format ) { + $attr = ( isset ( $obj->usepatrol ) && $obj->usepatrol && + $obj->patrolled == 0 ) ? ' class="not-patrolled"' : ''; + $s .= "{$format}\n"; + } + } - $s = "
      "; - # Only read at most $num rows, because $res may contain the whole 1000 - for ( $i = 0; $i < $num && $obj = $dbr->fetchObject( $res ); $i++ ) { - $format = $this->formatResult( $sk, $obj ); - $attr = ( isset ( $obj->usepatrol ) && $obj->usepatrol && - $obj->patrolled == 0 ) ? ' class="not_patrolled"' : ''; - $s .= "{$format}\n"; + if($this->tryLastResult()) { + // flush the very last result + $obj = null; + $format = $this->formatResult( $sk, $obj ); + if( $format ) { + $attr = ( isset ( $obj->usepatrol ) && $obj->usepatrol && + $obj->patrolled == 0 ) ? ' class="not-patrolled"' : ''; + $s .= "{$format}\n"; + } + } + + $dbr->freeResult( $res ); + $s .= '
    '; + $wgOut->addHTML( $s ); } - $dbr->freeResult( $res ); - $s .= '
'; - $wgOut->addHTML( $s ); - $wgOut->addHTML( "

{$sl}

\n" ); + if($shownavigation) { + $wgOut->addHTML( "

{$sl}

\n" ); + } + return $num; } /** @@ -237,7 +340,7 @@ class QueryPage { } return new FeedItem( - $title->getText(), + $title->getPrefixedText(), $this->feedItemDesc( $row ), $title->getFullURL(), $date, @@ -249,18 +352,9 @@ class QueryPage { } function feedItemDesc( $row ) { - $text = ''; - if( isset( $row->comment ) ) { - $text = htmlspecialchars( $row->comment ); - } else { - $text = ''; - } - - if( isset( $row->text ) ) { - $text = '

' . htmlspecialchars( wfMsg( 'summary' ) ) . ': ' . $text . "

\n
\n
" . - nl2br( htmlspecialchars( $row->text ) ) . "
";; - } - return $text; + return isset( $row->comment ) + ? htmlspecialchars( $row->comment ) + : ''; } function feedItemAuthor( $row ) { @@ -293,12 +387,15 @@ class QueryPage { * This is a subclass for very simple queries that are just looking for page * titles that match some criteria. It formats each result item as a link to * that page. + * + * @package MediaWiki */ class PageQueryPage extends QueryPage { function formatResult( $skin, $result ) { + global $wgContLang; $nt = Title::makeTitle( $result->namespace, $result->title ); - return $skin->makeKnownLinkObj( $nt, '' ); + return $skin->makeKnownLinkObj( $nt, $wgContLang->convert( $nt->getPrefixedText() ) ); } }