X-Git-Url: https://git.cyclocoop.org/?a=blobdiff_plain;f=includes%2FRevision.php;h=05a4a68ade5ead3f15226a5544716740823753b0;hb=1e7e73444a9357be1c58301e8707c029f31f5ef7;hp=67eb4be6fcff0f34c3b798a82164db9ce5f215f4;hpb=a49698b0f775c195131eac1f8e887af41e42d872;p=lhc%2Fweb%2Fwiklou.git diff --git a/includes/Revision.php b/includes/Revision.php index 67eb4be6fc..05a4a68ade 100644 --- a/includes/Revision.php +++ b/includes/Revision.php @@ -1,32 +1,31 @@ IntVal( $id ) ) ); + 'rev_id' => intval( $id ) ) ); } - + /** * Load either the current, or a specified, revision * that's attached to a given title. If not attached @@ -36,10 +35,11 @@ class Revision { * @param int $id * @return Revision * @access public + * @static */ - function &newFromTitle( &$title, $id = 0 ) { + public static function newFromTitle( &$title, $id = 0 ) { if( $id ) { - $matchId = IntVal( $id ); + $matchId = intval( $id ); } else { $matchId = 'page_latest'; } @@ -47,9 +47,24 @@ class Revision { array( "rev_id=$matchId", 'page_id=rev_page', 'page_namespace' => $title->getNamespace(), - 'page_title' => $title->getDbkey() ) ); + 'page_title' => $title->getDBkey() ) ); } - + + /** + * Load a page revision from a given revision ID number. + * Returns null if no such revision can be found. + * + * @param Database $db + * @param int $id + * @access public + * @static + */ + public static function loadFromId( &$db, $id ) { + return Revision::loadFromConds( $db, + array( 'page_id=rev_page', + 'rev_id' => intval( $id ) ) ); + } + /** * Load either the current, or a specified, revision * that's attached to a given page. If not attached @@ -60,33 +75,83 @@ class Revision { * @param int $id * @return Revision * @access public + * @static + */ + public static function loadFromPageId( $db, $pageid, $id = 0 ) { + $conds=array('page_id=rev_page','rev_page'=>intval( $pageid ), 'page_id'=>intval( $pageid )); + if( $id ) { + $conds['rev_id']=intval($id); + } else { + $conds[]='rev_id=page_latest'; + } + return Revision::loadFromConds( $db, $conds ); + } + + /** + * Load either the current, or a specified, revision + * that's attached to a given page. If not attached + * to that page, will return null. + * + * @param Database $db + * @param Title $title + * @param int $id + * @return Revision + * @access public + * @static */ - function &loadFromPageId( &$db, $pageid, $id = 0 ) { + public static function loadFromTitle( &$db, $title, $id = 0 ) { if( $id ) { - $matchId = IntVal( $id ); + $matchId = intval( $id ); } else { $matchId = 'page_latest'; } return Revision::loadFromConds( $db, array( "rev_id=$matchId", - 'rev_page' => IntVal( $pageid ), - 'page_id=rev_page' ) ); + 'page_id=rev_page', + 'page_namespace' => $title->getNamespace(), + 'page_title' => $title->getDBkey() ) ); } - + + /** + * Load the revision for the given title with the given timestamp. + * WARNING: Timestamps may in some circumstances not be unique, + * so this isn't the best key to use. + * + * @param Database $db + * @param Title $title + * @param string $timestamp + * @return Revision + * @access public + * @static + */ + public static function loadFromTimestamp( &$db, &$title, $timestamp ) { + return Revision::loadFromConds( + $db, + array( 'rev_timestamp' => $db->timestamp( $timestamp ), + 'page_id=rev_page', + 'page_namespace' => $title->getNamespace(), + 'page_title' => $title->getDBkey() ) ); + } + /** * Given a set of conditions, fetch a revision. * * @param array $conditions * @return Revision - * @static * @access private + * @static */ - function &newFromConds( $conditions ) { - $db =& wfGetDB( DB_SLAVE ); - return Revision::loadFromConds( $db, $conditions ); + private static function newFromConds( $conditions ) { + $db = wfGetDB( DB_SLAVE ); + $row = Revision::loadFromConds( $db, $conditions ); + if( is_null( $row ) ) { + $dbw = wfGetDB( DB_MASTER ); + $row = Revision::loadFromConds( $dbw, $conditions ); + } + return $row; } - + /** * Given a set of conditions, fetch a revision from * the given database connection. @@ -94,21 +159,23 @@ class Revision { * @param Database $db * @param array $conditions * @return Revision - * @static * @access private + * @static */ - function &loadFromConds( &$db, $conditions ) { - $res =& Revision::fetchFromConds( $db, $conditions ); + private static function loadFromConds( $db, $conditions ) { + $res = Revision::fetchFromConds( $db, $conditions ); if( $res ) { $row = $res->fetchObject(); $res->free(); if( $row ) { - return new Revision( $row ); + $ret = new Revision( $row ); + return $ret; } } - return null; + $ret = null; + return $ret; } - + /** * Return a wrapper for a series of database rows to * fetch all of a given page's revisions in turn. @@ -116,17 +183,17 @@ class Revision { * * @param Title $title * @return ResultWrapper - * @static * @access public + * @static */ - function &fetchAllRevisions( &$title ) { + public static function fetchAllRevisions( &$title ) { return Revision::fetchFromConds( wfGetDB( DB_SLAVE ), array( 'page_namespace' => $title->getNamespace(), - 'page_title' => $title->getDbkey(), - 'page_id=rev_page' ) ); + 'page_title' => $title->getDBkey(), + 'page_id=rev_page' ) ); } - + /** * Return a wrapper for a series of database rows to * fetch all of a given page's revisions in turn. @@ -134,18 +201,18 @@ class Revision { * * @param Title $title * @return ResultWrapper - * @static * @access public + * @static */ - function &fetchRevision( &$title ) { + public static function fetchRevision( &$title ) { return Revision::fetchFromConds( wfGetDB( DB_SLAVE ), array( 'rev_id=page_latest', 'page_namespace' => $title->getNamespace(), - 'page_title' => $title->getDbkey(), - 'page_id=rev_page' ) ); + 'page_title' => $title->getDBkey(), + 'page_id=rev_page' ) ); } - + /** * Given a set of conditions, return a ResultWrapper * which will return matching database rows with the @@ -154,10 +221,10 @@ class Revision { * @param Database $db * @param array $conditions * @return ResultWrapper - * @static * @access private + * @static */ - function &fetchFromConds( &$db, $conditions ) { + private static function fetchFromConds( $db, $conditions ) { $res = $db->select( array( 'page', 'revision' ), array( 'page_namespace', @@ -171,145 +238,255 @@ class Revision { 'rev_user', 'rev_minor_edit', 'rev_timestamp', - 'rev_deleted' ), + 'rev_deleted', + 'rev_len' ), $conditions, - 'Revision::fetchRow' ); - return $db->resultObject( $res ); + 'Revision::fetchRow', + array( 'LIMIT' => 1 ) ); + $ret = $db->resultObject( $res ); + return $ret; } - + + /** + * Return the list of revision fields that should be selected to create + * a new revision. + */ + static function selectFields() { + return array( + 'rev_id', + 'rev_page', + 'rev_text_id', + 'rev_timestamp', + 'rev_comment', + 'rev_minor_edit', + 'rev_user', + 'rev_user_text,'. + 'rev_deleted', + 'rev_len' + ); + } + /** * @param object $row * @access private */ function Revision( $row ) { if( is_object( $row ) ) { - $this->mId = IntVal( $row->rev_id ); - $this->mPage = IntVal( $row->rev_page ); - $this->mTextId = IntVal( $row->rev_text_id ); + $this->mId = intval( $row->rev_id ); + $this->mPage = intval( $row->rev_page ); + $this->mTextId = intval( $row->rev_text_id ); $this->mComment = $row->rev_comment; $this->mUserText = $row->rev_user_text; - $this->mUser = IntVal( $row->rev_user ); - $this->mMinorEdit = IntVal( $row->rev_minor_edit ); + $this->mUser = intval( $row->rev_user ); + $this->mMinorEdit = intval( $row->rev_minor_edit ); $this->mTimestamp = $row->rev_timestamp; - $this->mDeleted = IntVal( $row->rev_deleted ); - - $this->mCurrent = ( $row->rev_id == $row->page_latest ); - $this->mTitle = Title::makeTitle( $row->page_namespace, - $row->page_title ); + $this->mDeleted = intval( $row->rev_deleted ); + if( !isset( $row->rev_len ) || is_null( $row->rev_len ) ) + $this->mSize = null; + else + $this->mSize = intval( $row->rev_len ); + + if( isset( $row->page_latest ) ) { + $this->mCurrent = ( $row->rev_id == $row->page_latest ); + $this->mTitle = Title::makeTitle( $row->page_namespace, + $row->page_title ); + } else { + $this->mCurrent = false; + $this->mTitle = null; + } + + // Lazy extraction... + $this->mText = null; if( isset( $row->old_text ) ) { - $this->mText = $this->getRevisionText( $row ); + $this->mTextRow = $row; } else { - $this->mText = null; + // 'text' table row entry will be lazy-loaded + $this->mTextRow = null; } } elseif( is_array( $row ) ) { // Build a new revision to be saved... global $wgUser; + + $this->mId = isset( $row['id'] ) ? intval( $row['id'] ) : null; + $this->mPage = isset( $row['page'] ) ? intval( $row['page'] ) : null; + $this->mTextId = isset( $row['text_id'] ) ? intval( $row['text_id'] ) : null; + $this->mUserText = isset( $row['user_text'] ) ? strval( $row['user_text'] ) : $wgUser->getName(); + $this->mUser = isset( $row['user'] ) ? intval( $row['user'] ) : $wgUser->getId(); + $this->mMinorEdit = isset( $row['minor_edit'] ) ? intval( $row['minor_edit'] ) : 0; + $this->mTimestamp = isset( $row['timestamp'] ) ? strval( $row['timestamp'] ) : wfTimestamp( TS_MW ); + $this->mDeleted = isset( $row['deleted'] ) ? intval( $row['deleted'] ) : 0; + $this->mSize = isset( $row['len'] ) ? intval( $row['len'] ) : null; - $this->mId = isset( $row['id'] ) ? IntVal( $row['id'] ) : null; - $this->mPage = isset( $row['page'] ) ? IntVal( $row['page'] ) : null; - $this->mTextId = isset( $row['text_id'] ) ? IntVal( $row['text_id'] ) : null; - $this->mComment = isset( $row['comment'] ) ? StrVal( $row['comment'] ) : null; - $this->mUserText = isset( $row['user_text'] ) ? StrVal( $row['user_text'] ) : $wgUser->getName(); - $this->mUser = isset( $row['user'] ) ? IntVal( $row['user'] ) : $wgUser->getId(); - $this->mMinorEdit = isset( $row['minor_edit'] ) ? IntVal( $row['minor_edit'] ) : 0; - $this->mTimestamp = isset( $row['timestamp'] ) ? StrVal( $row['timestamp'] ) : wfTimestamp( TS_MW ); - $this->mDeleted = isset( $row['deleted'] ) ? IntVal( $row['deleted'] ) : 0; - $this->mText = isset( $row['text'] ) ? StrVal( $row['text'] ) : null; - + // Enforce spacing trimming on supplied text + $this->mComment = isset( $row['comment'] ) ? trim( strval( $row['comment'] ) ) : null; + $this->mText = isset( $row['text'] ) ? rtrim( strval( $row['text'] ) ) : null; + $this->mTextRow = null; + $this->mTitle = null; # Load on demand if needed $this->mCurrent = false; + # If we still have no len_size, see it we have the text to figure it out + if ( !$this->mSize ) + $this->mSize = is_null($this->mText) ? null : strlen($this->mText); } else { - wfDebugDieBacktrace( 'Revision constructor passed invalid row format.' ); + throw new MWException( 'Revision constructor passed invalid row format.' ); } } - + /**#@+ * @access public */ - + /** * @return int */ function getId() { return $this->mId; } - + /** * @return int */ function getTextId() { return $this->mTextId; } - + + /** + * Returns the length of the text in this revision, or null if unknown. + */ + function getSize() { + return $this->mSize; + } + /** * Returns the title of the page associated with this entry. * @return Title */ - function &getTitle() { + function getTitle() { if( isset( $this->mTitle ) ) { return $this->mTitle; } - $dbr =& wfGetDB( DB_SLAVE ); + $dbr = wfGetDB( DB_SLAVE ); $row = $dbr->selectRow( array( 'page', 'revision' ), array( 'page_namespace', 'page_title' ), array( 'page_id=rev_page', 'rev_id' => $this->mId ), - 'Revision::getTItle' ); + 'Revision::getTitle' ); if( $row ) { - $this->mTitle =& Title::makeTitle( $row->page_namespace, + $this->mTitle = Title::makeTitle( $row->page_namespace, $row->page_title ); } return $this->mTitle; } - + + /** + * Set the title of the revision + * @param Title $title + */ + function setTitle( $title ) { + $this->mTitle = $title; + } + /** * @return int */ function getPage() { return $this->mPage; } - + /** + * Fetch revision's user id if it's available to all users * @return int */ function getUser() { + if( $this->isDeleted( self::DELETED_USER ) ) { + return 0; + } else { + return $this->mUser; + } + } + + /** + * Fetch revision's user id without regard for the current user's permissions + * @return string + */ + function getRawUser() { return $this->mUser; } - + /** + * Fetch revision's username if it's available to all users * @return string */ function getUserText() { + if( $this->isDeleted( self::DELETED_USER ) ) { + return ""; + } else { + return $this->mUserText; + } + } + + /** + * Fetch revision's username without regard for view restrictions + * @return string + */ + function getRawUserText() { return $this->mUserText; } /** + * Fetch revision comment if it's available to all users * @return string */ function getComment() { + if( $this->isDeleted( self::DELETED_COMMENT ) ) { + return ""; + } else { + return $this->mComment; + } + } + + /** + * Fetch revision comment without regard for the current user's permissions + * @return string + */ + function getRawComment() { return $this->mComment; } - + /** * @return bool */ function isMinor() { return (bool)$this->mMinorEdit; } - + /** + * int $field one of DELETED_* bitfield constants * @return bool */ - function isDeleted() { - return (bool)$this->mDeleted; + function isDeleted( $field ) { + return ($this->mDeleted & $field) == $field; } - + /** + * Fetch revision text if it's available to all users * @return string */ function getText() { + if( $this->isDeleted( self::DELETED_TEXT ) ) { + return ""; + } else { + return $this->getRawText(); + } + } + + /** + * Fetch revision text without regard for view restrictions + * @return string + */ + function getRawText() { if( is_null( $this->mText ) ) { // Revision text is immutable. Load on demand: $this->mText = $this->loadText(); @@ -317,34 +494,54 @@ class Revision { return $this->mText; } + /** + * Fetch revision text if it's available to THIS user + * @return string + */ + function revText() { + if( !$this->userCan( self::DELETED_TEXT ) ) { + return ""; + } else { + return $this->getRawText(); + } + } + /** * @return string */ function getTimestamp() { - return $this->mTimestamp; + return wfTimestamp(TS_MW, $this->mTimestamp); } - + /** * @return bool */ function isCurrent() { return $this->mCurrent; } - + /** * @return Revision */ - function &getPrevious() { + function getPrevious() { $prev = $this->mTitle->getPreviousRevisionID( $this->mId ); - return Revision::newFromTitle( $this->mTitle, $prev ); + if ( $prev ) { + return Revision::newFromTitle( $this->mTitle, $prev ); + } else { + return null; + } } /** * @return Revision */ - function &getNext() { + function getNext() { $next = $this->mTitle->getNextRevisionID( $this->mId ); - return Revision::newFromTitle( $this->mTitle, $next ); + if ( $next ) { + return Revision::newFromTitle( $this->mTitle, $next ); + } else { + return null; + } } /**#@-*/ @@ -357,10 +554,10 @@ class Revision { * @param string $prefix table prefix (default 'old_') * @return string $text|false the text requested */ - function getRevisionText( $row, $prefix = 'old_' ) { + public static function getRevisionText( $row, $prefix = 'old_' ) { $fname = 'Revision::getRevisionText'; wfProfileIn( $fname ); - + # Get data $textField = $prefix . 'text'; $flagsField = $prefix . 'flags'; @@ -381,40 +578,41 @@ class Revision { # Use external methods for external objects, text in table is URL-only then if ( in_array( 'external', $flags ) ) { $url=$text; - @list($proto,$path)=explode('://',$url,2); + @list(/* $proto */,$path)=explode('://',$url,2); if ($path=="") { wfProfileOut( $fname ); return false; } - require_once('ExternalStore.php'); $text=ExternalStore::fetchFromURL($url); } - if( in_array( 'gzip', $flags ) ) { - # Deal with optional compression of archived pages. - # This can be done periodically via maintenance/compressOld.php, and - # as pages are saved if $wgCompressRevisions is set. - $text = gzinflate( $text ); - } - - if( in_array( 'object', $flags ) ) { - # Generic compressed storage - $obj = unserialize( $text ); + // If the text was fetched without an error, convert it + if ( $text !== false ) { + if( in_array( 'gzip', $flags ) ) { + # Deal with optional compression of archived pages. + # This can be done periodically via maintenance/compressOld.php, and + # as pages are saved if $wgCompressRevisions is set. + $text = gzinflate( $text ); + } - # Bugger, corrupted my test database by double-serializing - if ( !is_object( $obj ) ) { - $obj = unserialize( $obj ); + if( in_array( 'object', $flags ) ) { + # Generic compressed storage + $obj = unserialize( $text ); + if ( !is_object( $obj ) ) { + // Invalid object + wfProfileOut( $fname ); + return false; + } + $text = $obj->getText(); } - $text = $obj->getText(); - } - - global $wgLegacyEncoding; - if( $wgLegacyEncoding && !in_array( 'utf-8', $flags ) ) { - # Old revisions kept around in a legacy encoding? - # Upconvert on demand. - global $wgInputEncoding, $wgContLang; - $text = $wgContLang->iconv( $wgLegacyEncoding, $wgInputEncoding, $text ); + global $wgLegacyEncoding; + if( $wgLegacyEncoding && !in_array( 'utf-8', $flags ) ) { + # Old revisions kept around in a legacy encoding? + # Upconvert on demand. + global $wgInputEncoding, $wgContLang; + $text = $wgContLang->iconv( $wgLegacyEncoding, $wgInputEncoding, $text ); + } } wfProfileOut( $fname ); return $text; @@ -434,11 +632,11 @@ class Revision { function compressRevisionText( &$text ) { global $wgCompressRevisions; $flags = array(); - + # Revisions not marked this way will be converted # on load if $wgLegacyCharset is set in the future. $flags[] = 'utf-8'; - + if( $wgCompressRevisions ) { if( function_exists( 'gzdeflate' ) ) { $text = gzdeflate( $text ); @@ -449,7 +647,7 @@ class Revision { } return implode( ',', $flags ); } - + /** * Insert a new revision into the database, returning the new revision ID * number on success and dies horribly on failure. @@ -458,25 +656,47 @@ class Revision { * @return int */ function insertOn( &$dbw ) { + global $wgDefaultExternalStore; + $fname = 'Revision::insertOn'; wfProfileIn( $fname ); - - $mungedText = $this->mText; - $flags = Revision::compressRevisionText( $mungedText ); - - # Record the text to the text table + + $data = $this->mText; + $flags = Revision::compressRevisionText( $data ); + + # Write to external storage if required + if ( $wgDefaultExternalStore ) { + if ( is_array( $wgDefaultExternalStore ) ) { + // Distribute storage across multiple clusters + $store = $wgDefaultExternalStore[mt_rand(0, count( $wgDefaultExternalStore ) - 1)]; + } else { + $store = $wgDefaultExternalStore; + } + // Store and get the URL + $data = ExternalStore::insert( $store, $data ); + if ( !$data ) { + # This should only happen in the case of a configuration error, where the external store is not valid + throw new MWException( "Unable to store text to external storage $store" ); + } + if ( $flags ) { + $flags .= ','; + } + $flags .= 'external'; + } + + # Record the text (or external storage URL) to the text table if( !isset( $this->mTextId ) ) { $old_id = $dbw->nextSequenceValue( 'text_old_id_val' ); $dbw->insert( 'text', array( 'old_id' => $old_id, - 'old_text' => $mungedText, + 'old_text' => $data, 'old_flags' => $flags, ), $fname ); $this->mTextId = $dbw->insertId(); } - + # Record the edit in revisions $rev_id = isset( $this->mId ) ? $this->mId @@ -492,15 +712,15 @@ class Revision { 'rev_user_text' => $this->mUserText, 'rev_timestamp' => $dbw->timestamp( $this->mTimestamp ), 'rev_deleted' => $this->mDeleted, + 'rev_len' => $this->mSize, ), $fname ); - - $this->mId = $dbw->insertId(); - + + $this->mId = !is_null($rev_id) ? $rev_id : $dbw->insertId(); wfProfileOut( $fname ); return $this->mId; } - + /** * Lazy-load the revision's text. * Currently hardcoded to the 'text' table storage engine. @@ -512,15 +732,51 @@ class Revision { $fname = 'Revision::loadText'; wfProfileIn( $fname ); - $dbr =& wfGetDB( DB_SLAVE ); - $row = $dbr->selectRow( 'text', - array( 'old_text', 'old_flags' ), - array( 'old_id' => $this->getTextId() ), - $fname); + // Caching may be beneficial for massive use of external storage + global $wgRevisionCacheExpiry, $wgMemc; + $key = wfMemcKey( 'revisiontext', 'textid', $this->getTextId() ); + if( $wgRevisionCacheExpiry ) { + $text = $wgMemc->get( $key ); + if( is_string( $text ) ) { + wfProfileOut( $fname ); + return $text; + } + } + // If we kept data for lazy extraction, use it now... + if ( isset( $this->mTextRow ) ) { + $row = $this->mTextRow; + $this->mTextRow = null; + } else { + $row = null; + } + + if( !$row ) { + // Text data is immutable; check slaves first. + $dbr = wfGetDB( DB_SLAVE ); + $row = $dbr->selectRow( 'text', + array( 'old_text', 'old_flags' ), + array( 'old_id' => $this->getTextId() ), + $fname); + } + + if( !$row ) { + // Possible slave lag! + $dbw = wfGetDB( DB_MASTER ); + $row = $dbw->selectRow( 'text', + array( 'old_text', 'old_flags' ), + array( 'old_id' => $this->getTextId() ), + $fname); + } + $text = Revision::getRevisionText( $row ); - wfProfileOut( $fname ); + if( $wgRevisionCacheExpiry ) { + $wgMemc->set( $key, $text, $wgRevisionCacheExpiry ); + } + + wfProfileOut( $fname ); + return $text; } @@ -538,10 +794,9 @@ class Revision { * @param bool $minor * @return Revision */ - function &newNullRevision( &$dbw, $pageId, $summary, $minor ) { - $fname = 'Revision::newNullRevision'; - wfProfileIn( $fname ); - + public static function newNullRevision( &$dbw, $pageId, $summary, $minor ) { + wfProfileIn( __METHOD__ ); + $current = $dbw->selectRow( array( 'page', 'revision' ), array( 'page_latest', 'rev_text_id' ), @@ -549,8 +804,8 @@ class Revision { 'page_id' => $pageId, 'page_latest=rev_id', ), - $fname ); - + __METHOD__ ); + if( $current ) { $revision = new Revision( array( 'page' => $pageId, @@ -561,10 +816,75 @@ class Revision { } else { $revision = null; } - - wfProfileOut( $fname ); + + wfProfileOut( __METHOD__ ); return $revision; } + /** + * Determine if the current user is allowed to view a particular + * field of this revision, if it's marked as deleted. + * @param int $field one of self::DELETED_TEXT, + * self::DELETED_COMMENT, + * self::DELETED_USER + * @return bool + */ + function userCan( $field ) { + if( ( $this->mDeleted & $field ) == $field ) { + global $wgUser; + $permission = ( $this->mDeleted & self::DELETED_RESTRICTED ) == self::DELETED_RESTRICTED + ? 'hiderevision' + : 'deleterevision'; + wfDebug( "Checking for $permission due to $field match on $this->mDeleted\n" ); + return $wgUser->isAllowed( $permission ); + } else { + return true; + } + } + + + /** + * Get rev_timestamp from rev_id, without loading the rest of the row + * @param integer $id + */ + static function getTimestampFromID( $id ) { + $dbr = wfGetDB( DB_SLAVE ); + $timestamp = $dbr->selectField( 'revision', 'rev_timestamp', + array( 'rev_id' => $id ), __METHOD__ ); + if ( $timestamp === false ) { + # Not in slave, try master + $dbw = wfGetDB( DB_MASTER ); + $timestamp = $dbw->selectField( 'revision', 'rev_timestamp', + array( 'rev_id' => $id ), __METHOD__ ); + } + return $timestamp; + } + + static function countByPageId( $db, $id ) { + $row = $db->selectRow( 'revision', 'COUNT(*) AS revCount', + array( 'rev_page' => $id ), __METHOD__ ); + if( $row ) { + return $row->revCount; + } + return 0; + } + + static function countByTitle( $db, $title ) { + $id = $title->getArticleId(); + if( $id ) { + return Revision::countByPageId( $db, $id ); + } + return 0; + } } -?> + +/** + * Aliases for backwards compatibility with 1.6 + */ +define( 'MW_REV_DELETED_TEXT', Revision::DELETED_TEXT ); +define( 'MW_REV_DELETED_COMMENT', Revision::DELETED_COMMENT ); +define( 'MW_REV_DELETED_USER', Revision::DELETED_USER ); +define( 'MW_REV_DELETED_RESTRICTED', Revision::DELETED_RESTRICTED ); + + +