r78256 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r78255‎ | r78256 | r78257 >
Date:16:15, 12 December 2010
Author:platonides
Status:ok
Tags:
Comment:
This is not an array, but a parameter. Revert r78254. Is the precommit hook not working?
PHP Parse error: syntax error, unexpected T_DOUBLE_ARROW in includes/Title.php on line 3752
Modified paths:
  • /trunk/phase3/includes/Title.php (modified) (history)

Diff [purge]

Index: trunk/phase3/includes/Title.php
@@ -3749,7 +3749,7 @@
37503750 public function countRevisionsBetween( $old, $new ) {
37513751 $dbr = wfGetDB( DB_SLAVE );
37523752 return (int)$dbr->selectField( 'revision', 'count(*)',
3753 - 'rev_page' => intval( $this->getArticleId() ) .
 3753+ 'rev_page = ' . intval( $this->getArticleId() ) .
37543754 ' AND rev_id > ' . intval( $old ) .
37553755 ' AND rev_id < ' . intval( $new ),
37563756 __METHOD__
@@ -3770,7 +3770,7 @@
37713771 $db = ( $flags & self::GAID_FOR_UPDATE ) ? wfGetDB( DB_MASTER ) : wfGetDB( DB_SLAVE );
37723772 $res = $db->select( 'revision', 'DISTINCT rev_user_text',
37733773 array(
3774 - 'rev_page' => $this->getArticleID(),
 3774+ 'rev_page = ' . $this->getArticleID(),
37753775 'rev_id > ' . (int)$fromRevId,
37763776 'rev_id < ' . (int)$toRevId
37773777 ), __METHOD__,

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r78254Folllowup r72866: use array syntax for rev_page = foo conditioncatrope15:55, 12 December 2010

Status & tagging log