X-Git-Url: http://git.cyclocoop.org/?a=blobdiff_plain;f=includes%2FArticle.php;h=cca2bb0fa7370e1d859292105a91ca1b93bfbdac;hb=d73935a5f0fb09882b4a9269868a9bb68bb29d72;hp=6dea6747dca4845ba7d6eaf040996927080d9d62;hpb=f912c43a127270f56e52732a64895f13e5d63b5e;p=lhc%2Fweb%2Fwiklou.git diff --git a/includes/Article.php b/includes/Article.php index 6dea6747dc..cca2bb0fa7 100644 --- a/includes/Article.php +++ b/includes/Article.php @@ -7,7 +7,11 @@ # Note: edit user interface and cache support functions have been # moved to separate EditPage and CacheManager classes. -require_once( 'CacheManager.php' ); +require_once ( 'CacheManager.php' ); +include_once ( 'SpecialValidate.php' ) ; + +$wgArticleCurContentFields = false; +$wgArticleOldContentFields = false; class Article { /* private */ var $mContent, $mContentLoaded; @@ -16,12 +20,12 @@ class Article { /* private */ var $mMinorEdit, $mRedirectedFrom; /* private */ var $mTouched, $mFileCache, $mTitle; /* private */ var $mId, $mTable; - + function Article( &$title ) { $this->mTitle =& $title; $this->clear(); } - + /* private */ function clear() { $this->mContentLoaded = false; @@ -62,7 +66,7 @@ class Article { } return $text; } - + /* static */ function compressRevisionText( &$text ) { global $wgCompressRevisions; if( !$wgCompressRevisions ) { @@ -75,7 +79,7 @@ class Article { $text = gzdeflate( $text ); return 'gzip'; } - + # Returns the text associated with a "link" type old table row /* static */ function followLink( $link ) { # Split the link into fields and values @@ -119,7 +123,7 @@ class Article { global $wgLoadBalancer; $fname = 'fetchFromLocation'; wfProfileIn( $fname ); - + $p = strpos( $location, ':' ); if ( $p === false ) { wfProfileOut( $fname ); @@ -132,7 +136,7 @@ class Article { case 'mysql': # MySQL locations are specified by mysql:///// # Machine ID 0 is the current connection - if ( preg_match( '/^mysql:\/\/(\d+)\/([A-Za-z_]+)\/([A-Za-z_]+)\/([A-Za-z_]+)$/', + if ( preg_match( '/^mysql:\/\/(\d+)\/([A-Za-z_]+)\/([A-Za-z_]+)\/([A-Za-z_]+)$/', $location, $matches ) ) { $machineID = $matches[1]; $dbName = $matches[2]; @@ -140,20 +144,20 @@ class Article { $index = $matches[4]; if ( $machineID == 0 ) { # Current connection - $db =& wfGetDB(); + $db =& wfGetDB( DB_SLAVE ); } else { # Alternate connection $db =& $wgLoadBalancer->getConnection( $machineID ); - + if ( array_key_exists( $machineId, $wgKnownMysqlServers ) ) { # Try to open, return false on failure $params = $wgKnownDBServers[$machineId]; - $db = Database::newFromParams( $params['server'], $params['user'], $params['password'], - $dbName, 1, false, true, true ); + $db = Database::newFromParams( $params['server'], $params['user'], $params['password'], + $dbName, 1, DBO_IGNORE ); } } if ( $db->isOpen() ) { - $index = wfStrencode( $index ); + $index = $db->strencode( $index ); $res = $db->query( "SELECT blob_data FROM $dbName.$tblName WHERE blob_index='$index'", $fname ); $row = $db->fetchObject( $res ); $text = $row->text_data; @@ -214,17 +218,14 @@ class Article { return wfMsg( 'noarticletext' ); } else { $this->loadContent( $noredir ); - - if( - # check if we're displaying a [[User talk:x.x.x.x]] anonymous talk page - ( $this->mTitle->getNamespace() == Namespace::getTalk( Namespace::getUser()) ) && - preg_match('/^\d{1,3}\.\d{1,3}.\d{1,3}\.\d{1,3}$/',$this->mTitle->getText()) && - $action=='view' - ) - { + # check if we're displaying a [[User talk:x.x.x.x]] anonymous talk page + if ( $this->mTitle->getNamespace() == NS_USER_TALK && + preg_match('/^\d{1,3}\.\d{1,3}.\d{1,3}\.\d{1,3}$/',$this->mTitle->getText()) && + $action=='view' + ) { wfProfileOut( $fname ); - return $this->mContent . "\n" .wfMsg('anontalkpagetext'); } - else { + return $this->mContent . "\n" .wfMsg('anontalkpagetext'); + } else { if($action=='edit') { if($section!='') { if($section=='new') { @@ -244,15 +245,15 @@ class Article { } } } - + # This function returns the text of a section, specified by a number ($section). - # A section is text under a heading like == Heading == or

Heading

, or + # A section is text under a heading like == Heading == or

Heading

, or # the first section before any such heading (section 0). # # If a section contains subsections, these are also returned. # function getSection($text,$section) { - + # strip NOWIKI etc. to avoid confusion (true-parameter causes HTML # comments to be stripped as well) $striparray=array(); @@ -273,23 +274,23 @@ class Article { $headline=$secs[$section*2-1]; preg_match( '/^(=+).*?=+|^.*?<\/h[1-6].*?' . '>/mi',$headline,$matches); $hlevel=$matches[1]; - + # translate wiki heading into level if(strpos($hlevel,'=')!==false) { - $hlevel=strlen($hlevel); + $hlevel=strlen($hlevel); } - + $rv=$headline. $secs[$section*2]; $count=$section+1; - + $break=false; while(!empty($secs[$count*2-1]) && !$break) { - + $subheadline=$secs[$count*2-1]; preg_match( '/^(=+).*?=+|^.*?<\/h[1-6].*?' . '>/mi',$subheadline,$matches); $subhlevel=$matches[1]; if(strpos($subhlevel,'=')!==false) { - $subhlevel=strlen($subhlevel); + $subhlevel=strlen($subhlevel); } if($subhlevel > $hlevel) { $rv.=$subheadline.$secs[$count*2]; @@ -298,7 +299,7 @@ class Article { $break=true; } $count++; - + } } # reinsert stripped tags @@ -308,79 +309,88 @@ class Article { return $rv; } - + # Return an array of the columns of the "cur"-table + function &getCurContentFields() { + global $wgArticleCurContentFields; + if ( !$wgArticleCurContentFields ) { + $wgArticleCurContentFields = array( 'cur_text','cur_timestamp','cur_user', 'cur_user_text', + 'cur_comment','cur_counter','cur_restrictions','cur_touched' ); + } + return $wgArticleCurContentFields; + } + + # Return an array of the columns of the "old"-table + function &getOldContentFields() { + global $wgArticleOldContentFields; + if ( !$wgArticleOldContentFields ) { + $wgArticleOldContentFields = array( 'old_namespace','old_title','old_text','old_timestamp', + 'old_user','old_user_text','old_comment','old_flags' ); + } + return $wgArticleOldContentFields; + } + # Load the revision (including cur_text) into this object function loadContent( $noredir = false ) { - global $wgOut, $wgMwRedir, $wgRequest, $wgIsPg; + global $wgOut, $wgMwRedir, $wgRequest; + $dbr =& wfGetDB( DB_SLAVE ); # Query variables :P $oldid = $wgRequest->getVal( 'oldid' ); $redirect = $wgRequest->getVal( 'redirect' ); if ( $this->mContentLoaded ) return; $fname = 'Article::loadContent'; - - # Pre-fill content with error message so that if something + + # Pre-fill content with error message so that if something # fails we'll have something telling us what we intended. - $t = $this->mTitle->getPrefixedText(); - if ( isset( $oldid ) ) { - $oldid = IntVal( $oldid ); + $t = $this->mTitle->getPrefixedText(); + if ( isset( $oldid ) ) { + $oldid = IntVal( $oldid ); $t .= ",oldid={$oldid}"; - } - if ( isset( $redirect ) ) { - $redirect = ($redirect == 'no') ? 'no' : 'yes'; - $t .= ",redirect={$redirect}"; - } + } + if ( isset( $redirect ) ) { + $redirect = ($redirect == 'no') ? 'no' : 'yes'; + $t .= ",redirect={$redirect}"; + } $this->mContent = wfMsg( 'missingarticle', $t ); - + if ( ! $oldid ) { # Retrieve current version $id = $this->getID(); if ( 0 == $id ) return; - $sql = 'SELECT ' . - 'cur_text,cur_timestamp,cur_user,cur_user_text,cur_comment,cur_counter,cur_restrictions,cur_touched ' . - "FROM cur WHERE cur_id={$id}"; - wfDebug( "$sql\n" ); - $res = wfQuery( $sql, DB_READ, $fname ); - if ( 0 == wfNumRows( $res ) ) { - return; + $s = $dbr->getArray( 'cur', $this->getCurContentFields(), array( 'cur_id' => $id ), $fname ); + if ( $s === false ) { + return; } - $s = wfFetchObject( $res ); # If we got a redirect, follow it (unless we've been told # not to by either the function parameter or the query - if ( ( 'no' != $redirect ) && ( false == $noredir ) && - ( $wgMwRedir->matchStart( $s->cur_text ) ) ) { - if ( preg_match( '/\\[\\[([^\\]\\|]+)[\\]\\|]/', - $s->cur_text, $m ) ) { - $rt = Title::newFromText( $m[1] ); - if( $rt ) { - # Gotta hand redirects to special pages differently: - # Fill the HTTP response "Location" header and ignore - # the rest of the page we're on. - - if ( $rt->getInterwiki() != '' ) { - $wgOut->redirect( $rt->getFullURL() ) ; - return; - } - if ( $rt->getNamespace() == Namespace::getSpecial() ) { - $wgOut->redirect( $rt->getFullURL() ); - return; - } - $rid = $rt->getArticleID(); - if ( 0 != $rid ) { - $sql = 'SELECT cur_text,cur_timestamp,cur_user,cur_user_text,cur_comment,' . - "cur_counter,cur_restrictions,cur_touched FROM cur WHERE cur_id={$rid}"; - $res = wfQuery( $sql, DB_READ, $fname ); - - if ( 0 != wfNumRows( $res ) ) { - $this->mRedirectedFrom = $this->mTitle->getPrefixedText(); - $this->mTitle = $rt; - $s = wfFetchObject( $res ); - } + if ( ( 'no' != $redirect ) && ( false == $noredir ) ) { + $rt = Title::newFromRedirect( $s->cur_text ); + if ( $rt ) { + # Gotta hand redirects to special pages differently: + # Fill the HTTP response "Location" header and ignore + # the rest of the page we're on. + + if ( $rt->getInterwiki() != '' ) { + $wgOut->redirect( $rt->getFullURL() ) ; + return; + } + if ( $rt->getNamespace() == NS_SPECIAL ) { + $wgOut->redirect( $rt->getFullURL() ); + return; + } + $rid = $rt->getArticleID(); + if ( 0 != $rid ) { + $redirRow = $dbr->getArray( 'cur', $this->getCurContentFields(), array( 'cur_id' => $rid ), $fname ); + + if ( $redirRow !== false ) { + $this->mRedirectedFrom = $this->mTitle->getPrefixedText(); + $this->mTitle = $rt; + $s = $redirRow; } } } @@ -395,18 +405,12 @@ class Article { $this->mTouched = $s->cur_touched; $this->mTitle->mRestrictions = explode( ',', trim( $s->cur_restrictions ) ); $this->mTitle->mRestrictionsLoaded = true; - wfFreeResult( $res ); } else { # oldid set, retrieve historical version - $oldtable=$wgIsPg?'"old"':'old'; - $sql = "SELECT old_namespace,old_title,old_text,old_timestamp,". - "old_user,old_user_text,old_comment,old_flags FROM old ". - "WHERE old_id={$oldid}"; - $res = wfQuery( $sql, DB_READ, $fname ); - if ( 0 == wfNumRows( $res ) ) { + $s = $dbr->getArray( 'old', $this->getOldContentFields(), array( 'old_id' => $oldid ), $fname ); + if ( $s === false ) { return; } - $s = wfFetchObject( $res ); if( $this->mTitle->getNamespace() != $s->old_namespace || $this->mTitle->getDBkey() != $s->old_title ) { $oldTitle = Title::makeTitle( $s->old_namesapce, $s->old_title ); @@ -419,7 +423,6 @@ class Article { $this->mComment = $s->old_comment; $this->mCounter = 0; $this->mTimestamp = $s->old_timestamp; - wfFreeResult( $res ); } $this->mContentLoaded = true; return $this->mContent; @@ -428,48 +431,40 @@ class Article { # Gets the article text without using so many damn globals # Returns false on error function getContentWithoutUsingSoManyDamnGlobals( $oldid = 0, $noredir = false ) { - global $wgMwRedir, $wgIsPg; + global $wgMwRedir; if ( $this->mContentLoaded ) { return $this->mContent; } $this->mContent = false; - - $fname = 'Article::loadContent'; - + + $fname = 'Article::getContentWithout'; + $dbr =& wfGetDB( DB_SLAVE ); + if ( ! $oldid ) { # Retrieve current version $id = $this->getID(); if ( 0 == $id ) { return false; } - $sql = 'SELECT ' . - 'cur_text,cur_timestamp,cur_user,cur_counter,cur_restrictions,cur_touched ' . - "FROM cur WHERE cur_id={$id}"; - $res = wfQuery( $sql, DB_READ, $fname ); - if ( 0 == wfNumRows( $res ) ) { - return false; + $s = $dbr->getArray( 'cur', $this->getCurContentFields(), array( 'cur_id' => $id ), $fname ); + if ( $s === false ) { + return false; } - $s = wfFetchObject( $res ); # If we got a redirect, follow it (unless we've been told # not to by either the function parameter or the query - if ( !$noredir && $wgMwRedir->matchStart( $s->cur_text ) ) { - if ( preg_match( '/\\[\\[([^\\]\\|]+)[\\]\\|]/', - $s->cur_text, $m ) ) { - $rt = Title::newFromText( $m[1] ); - if( $rt && $rt->getInterwiki() == '' && $rt->getNamespace() != Namespace::getSpecial() ) { - $rid = $rt->getArticleID(); - if ( 0 != $rid ) { - $sql = 'SELECT cur_text,cur_timestamp,cur_user,' . - "cur_counter,cur_restrictions,cur_touched FROM cur WHERE cur_id={$rid}"; - $res = wfQuery( $sql, DB_READ, $fname ); - - if ( 0 != wfNumRows( $res ) ) { - $this->mRedirectedFrom = $this->mTitle->getPrefixedText(); - $this->mTitle = $rt; - $s = wfFetchObject( $res ); - } + if ( !$noredir ) { + $rt = Title::newFromRedirect( $s->cur_text ); + if( $rt && $rt->getInterwiki() == '' && $rt->getNamespace() != NS_SPECIAL ) { + $rid = $rt->getArticleID(); + if ( 0 != $rid ) { + $redirRow = $dbr->getArray( 'cur', $this->getCurContentFields(), array( 'cur_id' => $rid ), $fname ); + + if ( $redirRow !== false ) { + $this->mRedirectedFrom = $this->mTitle->getPrefixedText(); + $this->mTitle = $rt; + $s = $redirRow; } } } @@ -477,27 +472,24 @@ class Article { $this->mContent = $s->cur_text; $this->mUser = $s->cur_user; + $this->mUserText = $s->cur_user_text; + $this->mComment = $s->cur_comment; $this->mCounter = $s->cur_counter; $this->mTimestamp = $s->cur_timestamp; $this->mTouched = $s->cur_touched; $this->mTitle->mRestrictions = explode( ",", trim( $s->cur_restrictions ) ); $this->mTitle->mRestrictionsLoaded = true; - wfFreeResult( $res ); } else { # oldid set, retrieve historical version - $oldtable=$wgIsPg?'"old"':'old'; - $sql = "SELECT old_text,old_timestamp,old_user,old_flags FROM $oldtable " . - "WHERE old_id={$oldid}"; - $res = wfQuery( $sql, DB_READ, $fname ); - if ( 0 == wfNumRows( $res ) ) { - return false; + $s = $dbr->getArray( 'old', $this->getOldContentFields(), array( 'old_id' => $oldid ) ); + if ( $s === false ) { + return false; } - - $s = wfFetchObject( $res ); $this->mContent = Article::getRevisionText( $s ); $this->mUser = $s->old_user; + $this->mUserText = $s->old_user_text; + $this->mComment = $s->old_comment; $this->mCounter = 0; $this->mTimestamp = $s->old_timestamp; - wfFreeResult( $res ); } $this->mContentLoaded = true; return $this->mContent; @@ -515,7 +507,8 @@ class Article { { if ( -1 == $this->mCounter ) { $id = $this->getID(); - $this->mCounter = wfGetSQL( 'cur', 'cur_counter', "cur_id={$id}" ); + $dbr =& wfGetDB( DB_SLAVE ); + $this->mCounter = $dbr->getField( 'cur', 'cur_counter', "cur_id={$id}" ); } return $this->mCounter; } @@ -526,7 +519,7 @@ class Article { function isCountable( $text ) { global $wgUseCommaCount, $wgMwRedir; - + if ( 0 != $this->mTitle->getNamespace() ) { return 0; } if ( $wgMwRedir->matchStart( $text ) ) { return 0; } $token = ($wgUseCommaCount ? ',' : '[[' ); @@ -541,14 +534,15 @@ class Article { { global $wgOut; if ( -1 != $this->mUser ) return; + + $fname = 'Article::loadLastEdit'; - $sql = 'SELECT cur_user,cur_user_text,cur_timestamp,' . - 'cur_comment,cur_minor_edit FROM cur WHERE ' . - 'cur_id=' . $this->getID(); - $res = wfQuery( $sql, DB_READ, 'Article::loadLastEdit' ); + $dbr =& wfGetDB( DB_SLAVE ); + $s = $dbr->getArray( 'cur', + array( 'cur_user','cur_user_text','cur_timestamp', 'cur_comment','cur_minor_edit' ), + array( 'cur_id' => $this->getID() ), $fname ); - if ( wfNumRows( $res ) > 0 ) { - $s = wfFetchObject( $res ); + if ( $s !== false ) { $this->mUser = $s->cur_user; $this->mUserText = $s->cur_user_text; $this->mTimestamp = $s->cur_timestamp; @@ -592,56 +586,47 @@ class Article { $fname = 'Article::getContributors'; # XXX: this is expensive; cache this info somewhere. - - $title = $this->mTitle; + $title = $this->mTitle; $contribs = array(); - - $sql = 'SELECT old.old_user, old.old_user_text, ' . - ' user.user_real_name, MAX(old.old_timestamp) as timestamp' . - ' FROM old, user ' . - ' WHERE old.old_user = user.user_id ' . - ' AND old.old_namespace = ' . $title->getNamespace() . - ' AND old.old_title = "' . $title->getDBkey() . '"' . - ' AND old.old_user != 0 ' . - ' AND old.old_user != ' . $this->getUser() . - ' GROUP BY old.old_user ' . - ' ORDER BY timestamp DESC '; + $dbr =& wfGetDB( DB_SLAVE ); + $oldTable = $dbr->tableName( 'old' ); + $userTable = $dbr->tableName( 'user' ); + $encDBkey = $dbr->strencode( $title->getDBkey() ); + $ns = $title->getNamespace(); + $user = $this->getUser(); + + $sql = "SELECT old_user, old_user_text, user_real_name, MAX(old_timestamp) as timestamp + FROM $oldTable LEFT JOIN $userTable ON old_user = user_id + WHERE old_namespace = $user + AND old_title = $encDBkey + AND old_user != $user + GROUP BY old_user + ORDER BY timestamp DESC"; if ($limit > 0) { $sql .= ' LIMIT '.$limit; } - $res = wfQuery($sql, DB_READ, $fname); - - while ( $line = wfFetchObject( $res ) ) { - $contribs[$line->old_user] = - array($line->old_user_text, $line->user_real_name); - } - - # Count anonymous users - - $res = wfQuery('SELECT COUNT(*) AS cnt ' . - ' FROM old ' . - ' WHERE old_namespace = ' . $title->getNamespace() . - " AND old_title = '" . $title->getDBkey() . "'" . - ' AND old_user = 0 ', DB_READ, $fname); - - while ( $line = wfFetchObject( $res ) ) { - $contribs[0] = array($line->cnt, 'Anonymous'); - } + $res = $dbr->query($sql, $fname); + + while ( $line = $dbr->fetchObject( $res ) ) { + $contribs[] = array($line->old_user, $line->old_user_text, $line->user_real_name); + } + + $dbr->freeResult($res); return $contribs; } - + # This is the default action of the script: just view the page of # the given title. function view() { - global $wgUser, $wgOut, $wgLang, $wgRequest; - global $wgLinkCache, $IP, $wgEnableParserCache; - + global $wgUser, $wgOut, $wgLang, $wgRequest, $wgMwRedir; + global $wgLinkCache, $IP, $wgEnableParserCache, $wgStylePath; + $fname = 'Article::view'; wfProfileIn( $fname ); @@ -660,10 +645,14 @@ class Article { $de = new DifferenceEngine( intval($oldid), intval($diff) ); $de->showDiffPage(); wfProfileOut( $fname ); + if( $diff == 0 ) { + # Run view updates for current revision only + $this->viewUpdates(); + } return; } - if ( !is_null( $oldid ) and $this->checkTouched() ) { + if ( empty( $oldid ) && $this->checkTouched() ) { if( $wgOut->checkLastModified( $this->mTouched ) ){ return; } else if ( $this->tryFileCache() ) { @@ -673,14 +662,14 @@ class Article { return; } } - + # Should the parser cache be used? if ( $wgEnableParserCache && intval($wgUser->getOption( 'stubthreshold' )) == 0 && empty( $oldid ) ) { $pcache = true; } else { $pcache = false; } - + $outputDone = false; if ( $pcache ) { if ( $wgOut->tryParserCache( $this, $wgUser ) ) { @@ -690,14 +679,14 @@ class Article { if ( !$outputDone ) { $text = $this->getContent( false ); # May change mTitle by following a redirect - + # Another whitelist check in case oldid or redirects are altering the title if ( !$this->mTitle->userCanRead() ) { $wgOut->loginToUse(); $wgOut->output(); exit; } - + # We're looking at an old revision @@ -711,29 +700,38 @@ class Article { 'redirect=no' ); $s = wfMsg( 'redirectedfrom', $redir ); $wgOut->setSubtitle( $s ); - - # Can't cache redirects + + # Can't cache redirects $pcache = false; } - $wgLinkCache->preFill( $this->mTitle ); - # wrap user css and user js in pre and don't parse # XXX: use $this->mTitle->usCssJsSubpage() when php is fixed/ a workaround is found - if ( - $this->mTitle->getNamespace() == Namespace::getUser() && + if ( + $this->mTitle->getNamespace() == NS_USER && preg_match('/\\/[\\w]+\\.(css|js)$/', $this->mTitle->getDBkey()) ) { $wgOut->addWikiText( wfMsg('clearyourcache')); $wgOut->addHTML( '
'.htmlspecialchars($this->mContent)."\n
" ); + } else if ( $rt = Title::newFromRedirect( $text ) ) { + # Display redirect + $sk = $wgUser->getSkin(); + $imageUrl = "$wgStylePath/images/redirect.png"; + $targetUrl = $rt->escapeLocalURL(); + $titleText = htmlspecialchars( $rt->getPrefixedText() ); + $link = $sk->makeLinkObj( $rt ); + $wgOut->addHTML( "" . + "$link" ); } else if ( $pcache ) { + # Display content and save to parser cache $wgOut->addWikiText( $text, true, $this ); } else { + # Display content, don't attempt to save to parser cache $wgOut->addWikiText( $text ); } } $wgOut->setPageTitle( $this->mTitle->getPrefixedText() ); - + # Add link titles as META keywords $wgOut->addMetaTags() ; @@ -748,9 +746,9 @@ class Article { /* private */ function insertNewArticle( $text, $summary, $isminor, $watchthis ) { - global $wgOut, $wgUser, $wgLinkCache, $wgMwRedir; - global $wgUseSquid, $wgDeferredUpdateList, $wgInternalServer, $wgIsPg, $wgIsMySQL; - + global $wgOut, $wgUser, $wgMwRedir; + global $wgUseSquid, $wgDeferredUpdateList, $wgInternalServer; + $fname = 'Article::insertNewArticle'; $this->mCountAdjustment = $this->isCountable( $text ); @@ -765,48 +763,49 @@ class Article { $won = wfInvertTimestamp( $now ); wfSeedRandom(); $rand = number_format( mt_rand() / mt_getrandmax(), 12, '.', '' ); - - if ($wgIsPg) { - $cur_id_column="cur_id,"; - $cur_id=wfGetSQL(""," nextval('cur_cur_id_seq')"); - $cur_id_value="{$cur_id},"; - } else { - $cur_id_column=""; - $cur_id=""; - $cur_id_value=""; - } + $dbw =& wfGetDB( DB_MASTER ); + + $cur_id = $dbw->nextSequenceValue( 'cur_cur_id_seq' ); $isminor = ( $isminor && $wgUser->getID() ) ? 1 : 0; - $sql = "INSERT INTO cur ({$cur_id_column}cur_namespace,cur_title,cur_text," . - 'cur_comment,cur_user,cur_timestamp,cur_minor_edit,cur_counter,' . - 'cur_restrictions,cur_user_text,cur_is_redirect,' . - "cur_is_new,cur_random,cur_touched,inverse_timestamp) VALUES ({$cur_id_value}{$ns},'" . wfStrencode( $ttl ) . "', '" . - wfStrencode( $text ) . "', '" . - wfStrencode( $summary ) . "', '" . - $wgUser->getID() . "', '{$now}', " . - $isminor . ", 0, '', '" . - wfStrencode( $wgUser->getName() ) . "', $redir, 1, $rand, '{$now}', '{$won}')"; - $res = wfQuery( $sql, DB_WRITE, $fname ); - - $newid = $wgIsPg?$cur_id:wfInsertId(); + + $dbw->insertArray( 'cur', array( + 'cur_id' => $cur_id, + 'cur_namespace' => $ns, + 'cur_title' => $ttl, + 'cur_text' => $text, + 'cur_comment' => $summary, + 'cur_user' => $wgUser->getID(), + 'cur_timestamp' => $now, + 'cur_minor_edit' => $isminor, + 'cur_counter' => 0, + 'cur_restrictions' => '', + 'cur_user_text' => $wgUser->getName(), + 'cur_is_redirect' => $redir, + 'cur_is_new' => 1, + 'cur_random' => $rand, + 'cur_touched' => $now, + 'inverse_timestamp' => $won, + ), $fname ); + + $newid = $dbw->insertId(); $this->mTitle->resetArticleID( $newid ); Article::onArticleCreate( $this->mTitle ); RecentChange::notifyNew( $now, $this->mTitle, $isminor, $wgUser, $summary ); - - if ($watchthis) { - if(!$this->mTitle->userIsWatching()) $this->watch(); + + if ($watchthis) { + if(!$this->mTitle->userIsWatching()) $this->watch(); } else { if ( $this->mTitle->userIsWatching() ) { $this->unwatch(); } } - + # The talk page isn't in the regular link tables, so we need to update manually: $talkns = $ns ^ 1; # talk -> normal; normal -> talk - $sql = "UPDATE cur set cur_touched='$now' WHERE cur_namespace=$talkns AND cur_title='" . wfStrencode( $ttl ) . "'"; - wfQuery( $sql, DB_WRITE ); - + $dbw->updateArray( 'cur', array( 'cur_touched' => $now ), array( 'cur_namespace' => $talkns, 'cur_title' => $ttl ), $fname ); + # standard deferred updates $this->editUpdates( $text ); @@ -817,7 +816,7 @@ class Article { /* Side effects: loads last edit */ function getTextOfLastEditWithSectionReplacedOrAdded($section, $text, $summary = ''){ $this->loadLastEdit(); - $oldtext = $this->getContent( true ); + $oldtext = $this->getContent( true ); if ($section != '') { if($section=='new') { if($summary) $subject="== {$summary} ==\n\n"; @@ -838,10 +837,10 @@ class Article { $secs=preg_split('/(^=+.*?=+|^.*?<\/h[1-6].*?' . '>)/mi', $oldtext,-1,PREG_SPLIT_DELIM_CAPTURE); $secs[$section*2]=$text."\n\n"; // replace with edited - + # section 0 is top (intro) section - if($section!=0) { - + if($section!=0) { + # headline of old section - we need to go through this section # to determine if there are any subsections that now need to # be erased, as the mother section has been replaced with @@ -849,23 +848,23 @@ class Article { $headline=$secs[$section*2-1]; preg_match( '/^(=+).*?=+|^.*?<\/h[1-6].*?' . '>/mi',$headline,$matches); $hlevel=$matches[1]; - + # determine headline level for wikimarkup headings if(strpos($hlevel,'=')!==false) { - $hlevel=strlen($hlevel); + $hlevel=strlen($hlevel); } - + $secs[$section*2-1]=''; // erase old headline $count=$section+1; $break=false; while(!empty($secs[$count*2-1]) && !$break) { - + $subheadline=$secs[$count*2-1]; preg_match( '/^(=+).*?=+|^.*?<\/h[1-6].*?' . '>/mi',$subheadline,$matches); $subhlevel=$matches[1]; if(strpos($subhlevel,'=')!==false) { - $subhlevel=strlen($subhlevel); + $subhlevel=strlen($subhlevel); } if($subhlevel > $hlevel) { // erase old subsections @@ -876,30 +875,31 @@ class Article { $break=true; } $count++; - + } - + } $text=join('',$secs); # reinsert the stuff that we stripped out earlier - $text=$parser->unstrip($text,$striparray); - $text=$parser->unstripNoWiki($text,$striparray); + $text=$parser->unstrip($text,$striparray); + $text=$parser->unstripNoWiki($text,$striparray); } - + } return $text; } function updateArticle( $text, $summary, $minor, $watchthis, $forceBot = false, $sectionanchor = '' ) { - global $wgOut, $wgUser, $wgLinkCache; + global $wgOut, $wgUser; global $wgDBtransactions, $wgMwRedir; global $wgUseSquid, $wgInternalServer; - global $wgIsPg; + $fname = 'Article::updateArticle'; - + $good = true; + if ( $this->mMinorEdit ) { $me1 = 1; } else { $me1 = 0; } - if ( $minor && $wgUser->getID() ) { $me2 = 1; } else { $me2 = 0; } + if ( $minor && $wgUser->getID() ) { $me2 = 1; } else { $me2 = 0; } if ( preg_match( "/^((" . $wgMwRedir->getBaseRegex() . ')[^\\n]+)/i', $text, $m ) ) { $redir = 1; $text = $m[1] . "\n"; # Remove all content but redirect @@ -907,80 +907,89 @@ class Article { else { $redir = 0; } $text = $this->preSaveTransform( $text ); - + $dbw =& wfGetDB( DB_MASTER ); + # Update article, but only if changed. + # It's important that we either rollback or complete, otherwise an attacker could + # overwrite cur entries by sending precisely timed user aborts. Random bored users + # could conceivably have the same effect, especially if cur is locked for long periods. if( $wgDBtransactions ) { - $sql = 'BEGIN'; - wfQuery( $sql, DB_WRITE ); + $dbw->query( 'BEGIN', $fname ); + } else { + $userAbort = ignore_user_abort( true ); } + $oldtext = $this->getContent( true ); if ( 0 != strcmp( $text, $oldtext ) ) { $this->mCountAdjustment = $this->isCountable( $text ) - $this->isCountable( $oldtext ); - $now = wfTimestampNow(); $won = wfInvertTimestamp( $now ); - $sql = "UPDATE cur SET cur_text='" . wfStrencode( $text ) . - "',cur_comment='" . wfStrencode( $summary ) . - "',cur_minor_edit={$me2}, cur_user=" . $wgUser->getID() . - ",cur_timestamp='{$now}',cur_user_text='" . - wfStrencode( $wgUser->getName() ) . - "',cur_is_redirect={$redir}, cur_is_new=0, cur_touched='{$now}', inverse_timestamp='{$won}' " . - "WHERE cur_id=" . $this->getID() . - " AND cur_timestamp='" . $this->getTimestamp() . "'"; - $res = wfQuery( $sql, DB_WRITE, $fname ); - - if( wfAffectedRows() == 0 ) { - /* Belated edit conflict! Run away!! */ - return false; - } - # This overwrites $oldtext if revision compression is on - $flags = Article::compressRevisionText( $oldtext ); + # First update the cur row + $dbw->updateArray( 'cur', + array( /* SET */ + 'cur_text' => $text, + 'cur_comment' => $summary, + 'cur_minor_edit' => $me2, + 'cur_user' => $wgUser->getID(), + 'cur_timestamp' => $now, + 'cur_user_text' => $wgUser->getName(), + 'cur_is_redirect' => $redir, + 'cur_is_new' => 0, + 'cur_touched' => $now, + 'inverse_timestamp' => $won + ), array( /* WHERE */ + 'cur_id' => $this->getID(), + 'cur_timestamp' => $this->getTimestamp() + ), $fname + ); - $oldtable=$wgIsPg?'"old"':'old'; - if ($wgIsPg) { - $oldtable='"old"'; - $old_id_column='old_id,'; - $old_id=wfGetSQL(""," nextval('old_old_id_seq')"); - $old_id_value=$old_id.','; + if( $dbw->affectedRows() == 0 ) { + /* Belated edit conflict! Run away!! */ + $good = false; } else { - $oldtable='old'; - $old_id_column=''; - $old_id_value=''; - } + # Now insert the previous revision into old - $sql = "INSERT INTO $oldtable ({$old_id_column}old_namespace,old_title,old_text," . - 'old_comment,old_user,old_user_text,old_timestamp,' . - 'old_minor_edit,inverse_timestamp,old_flags) VALUES (' . - $old_id_value. - $this->mTitle->getNamespace() . ", '" . - wfStrencode( $this->mTitle->getDBkey() ) . "', '" . - wfStrencode( $oldtext ) . "', '" . - wfStrencode( $this->getComment() ) . "', " . - $this->getUser() . ", '" . - wfStrencode( $this->getUserText() ) . "', '" . - $this->getTimestamp() . "', " . $me1 . ", '" . - wfInvertTimestamp( $this->getTimestamp() ) . "','$flags')"; - $res = wfQuery( $sql, DB_WRITE, $fname ); - - $oldid = $wgIsPg?$old_id:wfInsertId( $res ); + # This overwrites $oldtext if revision compression is on + $flags = Article::compressRevisionText( $oldtext ); + + $dbw->insertArray( 'old', + array( + 'old_id' => $dbw->nextSequenceValue( 'old_old_id_seq' ), + 'old_namespace' => $this->mTitle->getNamespace(), + 'old_title' => $this->mTitle->getDBkey(), + 'old_text' => $oldtext, + 'old_comment' => $this->getComment(), + 'old_user' => $this->getUser(), + 'old_user_text' => $this->getUserText(), + 'old_timestamp' => $this->getTimestamp(), + 'old_minor_edit' => $me1, + 'inverse_timestamp' => wfInvertTimestamp( $this->getTimestamp() ), + 'old_flags' => $flags, + ), $fname + ); + + $oldid = $dbw->insertId(); - $bot = (int)($wgUser->isBot() || $forceBot); - RecentChange::notifyEdit( $now, $this->mTitle, $me2, $wgUser, $summary, - $oldid, $this->getTimestamp(), $bot ); - Article::onArticleEdit( $this->mTitle ); + $bot = (int)($wgUser->isBot() || $forceBot); + RecentChange::notifyEdit( $now, $this->mTitle, $me2, $wgUser, $summary, + $oldid, $this->getTimestamp(), $bot ); + Article::onArticleEdit( $this->mTitle ); + } } if( $wgDBtransactions ) { - $sql = 'COMMIT'; - wfQuery( $sql, DB_WRITE ); + $dbw->query( 'COMMIT', $fname ); + } else { + ignore_user_abort( $userAbort ); } - - if ($watchthis) { - if (!$this->mTitle->userIsWatching()) $this->watch(); + + if ( $good ) { + if ($watchthis) { + if (!$this->mTitle->userIsWatching()) $this->watch(); } else { if ( $this->mTitle->userIsWatching() ) { $this->unwatch(); @@ -988,8 +997,8 @@ class Article { } # standard deferred updates $this->editUpdates( $text ); - - + + $urls = array(); # Template namespace # Purge all articles linking here @@ -997,12 +1006,12 @@ class Article { $titles = $this->mTitle->getLinksTo(); Title::touchArray( $titles ); if ( $wgUseSquid ) { - foreach ( $titles as $title ) { - $urls[] = $title->getInternalURL(); - } + foreach ( $titles as $title ) { + $urls[] = $title->getInternalURL(); + } } } - + # Squid updates if ( $wgUseSquid ) { $urls = array_merge( $urls, $this->mTitle->getSquidURLs() ); @@ -1011,7 +1020,8 @@ class Article { } $this->showArticle( $text, wfMsg( 'updated' ), $sectionanchor ); - return true; + } + return $good; } # After we've either updated or inserted the article, update @@ -1023,12 +1033,18 @@ class Article { global $wgMwRedir; $wgLinkCache = new LinkCache(); - + # Select for update + $wgLinkCache->forUpdate( true ); + # Get old version of link table to allow incremental link updates $wgLinkCache->preFill( $this->mTitle ); $wgLinkCache->clear(); - - # Now update the link cache by parsing the text + + # Switch on use of link cache in the skin + $sk =& $wgUser->getSkin(); + $sk->postParseLinkColour( false ); + + # Now update the link cache by parsing the text $wgOut = new OutputPage(); $wgOut->addWikiText( $text ); @@ -1039,6 +1055,22 @@ class Article { $wgOut->redirect( $this->mTitle->getFullURL( $r ).$sectionanchor ); } + # Validate article + + function validate () + { + global $wgOut ; + $wgOut->setPagetitle( wfMsg( 'validate' ) . " : " . $this->mTitle->getPrefixedText() ); + $wgOut->setRobotpolicy( 'noindex,follow' ); + if ( $this->mTitle->getNamespace() != 0 ) + { + $wgOut->addHTML ( wfMsg ( 'val_validate_article_namespace_only' ) ) ; + return ; + } + $v = new Validation ; + $v->validate_form ( $this->mTitle->getDBkey() ) ; + } + # Add this page to my watchlist function watch( $add = true ) @@ -1074,7 +1106,7 @@ class Article { $up = new UserUpdate(); array_push( $wgDeferredUpdateList, $up ); - $wgOut->returnToMain( false ); + $wgOut->returnToMain( true, $this->mTitle->getPrefixedText() ); } function unwatch() @@ -1104,18 +1136,23 @@ class Article { $reason = $wgRequest->getText( 'wpReasonProtect' ); if ( $confirm ) { + $dbw =& wfGetDB( DB_MASTER ); + $dbw->updateArray( 'cur', + array( /* SET */ + 'cur_touched' => wfTimestampNow(), + 'cur_restrictions' => (string)$limit + ), array( /* WHERE */ + 'cur_id' => $id + ), 'Article::protect' + ); - $sql = "UPDATE cur SET cur_touched='" . wfTimestampNow() . "'," . - "cur_restrictions='{$limit}' WHERE cur_id={$id}"; - wfQuery( $sql, DB_WRITE, 'Article::protect' ); - - $log = new LogPage( wfMsg( 'protectlogpage' ), wfMsg( 'protectlogtext' ) ); - if ( $limit === "" ) { - $log->addEntry( wfMsg( 'unprotectedarticle', $this->mTitle->getPrefixedText() ), $reason ); - } else { - $log->addEntry( wfMsg( 'protectedarticle', $this->mTitle->getPrefixedText() ), $reason ); - } - $wgOut->redirect( $this->mTitle->getFullURL() ); + $log = new LogPage( wfMsg( 'protectlogpage' ), wfMsg( 'protectlogtext' ) ); + if ( $limit === "" ) { + $log->addEntry( wfMsg( 'unprotectedarticle', $this->mTitle->getPrefixedText() ), $reason ); + } else { + $log->addEntry( wfMsg( 'protectedarticle', $this->mTitle->getPrefixedText() ), $reason ); + } + $wgOut->redirect( $this->mTitle->getFullURL() ); return; } else { $reason = htmlspecialchars( wfMsg( 'protectreason' ) ); @@ -1137,12 +1174,14 @@ class Article { $protcom = ''; if ( $limit === '' ) { + $wgOut->setPageTitle( wfMsg( 'confirmunprotect' ) ); $wgOut->setSubtitle( wfMsg( 'unprotectsub', $sub ) ); $wgOut->addWikiText( wfMsg( 'confirmunprotecttext' ) ); $check = htmlspecialchars( wfMsg( 'confirmunprotect' ) ); $protcom = htmlspecialchars( wfMsg( 'unprotectcomment' ) ); $formaction = $this->mTitle->escapeLocalURL( 'action=unprotect' . $par ); } else { + $wgOut->setPageTitle( wfMsg( 'confirmprotect' ) ); $wgOut->setSubtitle( wfMsg( 'protectsub', $sub ) ); $wgOut->addWikiText( wfMsg( 'confirmprotecttext' ) ); $check = htmlspecialchars( wfMsg( 'confirmprotect' ) ); @@ -1191,16 +1230,16 @@ class Article { return $this->protect( '' ); } - # UI entry point for page deletion + # UI entry point for page deletion function delete() { - global $wgUser, $wgOut, $wgMessageCache, $wgRequest, $wgIsPg; + global $wgUser, $wgOut, $wgMessageCache, $wgRequest; $fname = 'Article::delete'; $confirm = $wgRequest->getBool( 'wpConfirm' ) && $wgRequest->wasPosted(); $reason = $wgRequest->getText( 'wpReason' ); - + # This code desperately needs to be totally rewritten - + # Check permissions if ( ( ! $wgUser->isSysop() ) ) { $wgOut->sysopRequired(); @@ -1227,22 +1266,33 @@ class Article { # determine whether this page has earlier revisions # and insert a warning if it does # we select the text because it might be useful below + $dbr =& wfGetDB( DB_SLAVE ); $ns = $this->mTitle->getNamespace(); $title = $this->mTitle->getDBkey(); - $etitle = wfStrencode( $title ); - $oldtable=$wgIsPg?'"old"':'old'; - $sql = "SELECT old_text,old_flags FROM $oldtable WHERE old_namespace=$ns and old_title='$etitle' ORDER BY inverse_timestamp LIMIT 1"; - $res = wfQuery( $sql, DB_READ, $fname ); - if( ($old=wfFetchObject($res)) && !$confirm ) { + $old = $dbr->getArray( 'old', + array( 'old_text', 'old_flags' ), + array( + 'old_namespace' => $ns, + 'old_title' => $title, + ), $fname, array( 'ORDER BY' => 'inverse_timestamp' ) + ); + + if( $old !== false && !$confirm ) { $skin=$wgUser->getSkin(); $wgOut->addHTML(''.wfMsg('historywarning')); $wgOut->addHTML( $skin->historyLink() .''); } - - $sql="SELECT cur_text FROM cur WHERE cur_namespace=$ns and cur_title='$etitle'"; - $res=wfQuery($sql, DB_READ, $fname); - if( ($s=wfFetchObject($res))) { - + + # Fetch cur_text + $s = $dbr->getArray( 'cur', + array( 'cur_text' ), + array( + 'cur_namespace' => $ns, + 'cur_title' => $title, + ), $fname + ); + + if( $s !== false ) { # if this is a mini-text, we can paste part of it into the deletion reason #if this is empty, an earlier revision may contain "useful" text @@ -1254,19 +1304,19 @@ class Article { $text = Article::getRevisionText( $old ); $blanked = true; } - + } - - $length=strlen($text); - + + $length=strlen($text); + # this should not happen, since it is not possible to store an empty, new # page. Let's insert a standard text in case it does, though - if($length == 0 && $reason === '') { + if($length == 0 && $reason === '') { $reason = wfMsg('exblank'); } - + if($length < 500 && $reason === '') { - + # comment field=255, let's grep the first 150 to have some user # space left $text=substr($text,0,150); @@ -1281,27 +1331,27 @@ class Article { $reason=wfMsg('exbeforeblank') . " '".$text; } if($length>150) { $reason .= '...'; } # we've only pasted part of the text - $reason.="'"; + $reason.="'"; } } return $this->confirmDelete( '', $reason ); } - + # Output deletion confirmation dialog function confirmDelete( $par, $reason ) { global $wgOut; wfDebug( "Article::confirmDelete\n" ); - + $sub = htmlspecialchars( $this->mTitle->getPrefixedText() ); $wgOut->setSubtitle( wfMsg( 'deletesub', $sub ) ); $wgOut->setRobotpolicy( 'noindex,nofollow' ); $wgOut->addWikiText( wfMsg( 'confirmdeletetext' ) ); $formaction = $this->mTitle->escapeLocalURL( 'action=delete' . $par ); - + $confirm = htmlspecialchars( wfMsg( 'confirm' ) ); $check = htmlspecialchars( wfMsg( 'confirmcheck' ) ); $delcom = htmlspecialchars( wfMsg( 'deletecomment' ) ); @@ -1347,15 +1397,14 @@ class Article { $fname = 'Article::doDelete'; wfDebug( "$fname\n" ); - if ( $this->doDeleteArticle( $reason ) ) { + if ( $this->doDeleteArticle( $reason ) ) { $deleted = $this->mTitle->getPrefixedText(); $wgOut->setPagetitle( wfMsg( 'actioncomplete' ) ); $wgOut->setRobotpolicy( 'noindex,nofollow' ); $sk = $wgUser->getSkin(); - $loglink = $sk->makeKnownLink( $wgLang->getNsText( - Namespace::getWikipedia() ) . + $loglink = $sk->makeKnownLink( $wgLang->getNsText( NS_WIKIPEDIA ) . ':' . wfMsg( 'dellogpage' ), wfMsg( 'deletionlog' ) ); $text = wfMsg( "deletedtext", $deleted, $loglink ); @@ -1377,9 +1426,10 @@ class Article { $fname = 'Article::doDeleteArticle'; wfDebug( $fname."\n" ); - + + $dbw =& wfGetDB( DB_MASTER ); $ns = $this->mTitle->getNamespace(); - $t = wfStrencode( $this->mTitle->getDBkey() ); + $t = $this->mTitle->getDBkey(); $id = $this->mTitle->getArticleID(); if ( '' == $t || $id == 0 ) { @@ -1388,14 +1438,14 @@ class Article { $u = new SiteStatsUpdate( 0, 1, -$this->isCountable( $this->getContent( true ) ) ); array_push( $wgDeferredUpdateList, $u ); - + $linksTo = $this->mTitle->getLinksTo(); # Squid purging if ( $wgUseSquid ) { - $urls = array( + $urls = array( $this->mTitle->getInternalURL(), - $this->mTitle->getInternalURL( 'history' ) + $this->mTitle->getInternalURL( 'history' ) ); foreach ( $linksTo as $linkTo ) { $urls[] = $linkTo->getInternalURL(); @@ -1410,68 +1460,75 @@ class Article { Title::touchArray( $linksTo ); # Move article and history to the "archive" table - $sql = 'INSERT INTO archive (ar_namespace,ar_title,ar_text,' . - 'ar_comment,ar_user,ar_user_text,ar_timestamp,ar_minor_edit,' . - 'ar_flags) SELECT cur_namespace,cur_title,cur_text,cur_comment,' . - 'cur_user,cur_user_text,cur_timestamp,cur_minor_edit,0 FROM cur ' . - "WHERE cur_namespace={$ns} AND cur_title='{$t}'"; - wfQuery( $sql, DB_WRITE, $fname ); - - $sql = 'INSERT INTO archive (ar_namespace,ar_title,ar_text,' . - 'ar_comment,ar_user,ar_user_text,ar_timestamp,ar_minor_edit,' . - 'ar_flags) SELECT old_namespace,old_title,old_text,old_comment,' . - 'old_user,old_user_text,old_timestamp,old_minor_edit,old_flags ' . - "FROM old WHERE old_namespace={$ns} AND old_title='{$t}'"; - wfQuery( $sql, DB_WRITE, $fname ); - + $archiveTable = $dbw->tableName( 'archive' ); + $oldTable = $dbw->tableName( 'old' ); + $curTable = $dbw->tableName( 'cur' ); + $recentchangesTable = $dbw->tableName( 'recentchanges' ); + $linksTable = $dbw->tableName( 'links' ); + $brokenlinksTable = $dbw->tableName( 'brokenlinks' ); + + $dbw->insertSelect( 'archive', 'cur', + array( + 'ar_namespace' => 'cur_namespace', + 'ar_title' => 'cur_title', + 'ar_text' => 'cur_text', + 'ar_comment' => 'cur_comment', + 'ar_user' => 'cur_user', + 'ar_user_text' => 'cur_user_text', + 'ar_timestamp' => 'cur_timestamp', + 'ar_minor_edit' => 'cur_minor_edit', + 'ar_flags' => 0, + ), array( + 'cur_namespace' => $ns, + 'cur_title' => $t, + ), $fname + ); + + $dbw->insertSelect( 'archive', 'old', + array( + 'ar_namespace' => 'old_namespace', + 'ar_title' => 'old_title', + 'ar_text' => 'old_text', + 'ar_comment' => 'old_comment', + 'ar_user' => 'old_user', + 'ar_user_text' => 'old_user_text', + 'ar_timestamp' => 'old_timestamp', + 'ar_minor_edit' => 'old_minor_edit', + 'ar_flags' => 'old_flags' + ), array( + 'old_namespace' => $ns, + 'old_title' => $t, + ), $fname + ); + # Now that it's safely backed up, delete it - $sql = "DELETE FROM cur WHERE cur_namespace={$ns} AND " . - "cur_title='{$t}'"; - wfQuery( $sql, DB_WRITE, $fname ); - - $sql = "DELETE FROM old WHERE old_namespace={$ns} AND " . - "old_title='{$t}'"; - wfQuery( $sql, DB_WRITE, $fname ); - - $sql = "DELETE FROM recentchanges WHERE rc_namespace={$ns} AND " . - "rc_title='{$t}'"; - wfQuery( $sql, DB_WRITE, $fname ); + $dbw->delete( 'cur', array( 'cur_namespace' => $ns, 'cur_title' => $t ), $fname ); + $dbw->delete( 'old', array( 'old_namespace' => $ns, 'old_title' => $t ), $fname ); + $dbw->delete( 'recentchanges', array( 'rc_namespace' => $ns, 'rc_title' => $t ), $fname ); # Finally, clean up the link tables - $t = wfStrencode( $this->mTitle->getPrefixedDBkey() ); + $t = $this->mTitle->getPrefixedDBkey(); Article::onArticleDelete( $this->mTitle ); - - $sql = 'INSERT INTO brokenlinks (bl_from,bl_to) VALUES '; - $first = true; + # Insert broken links + $brokenLinks = array(); foreach ( $linksTo as $titleObj ) { - if ( ! $first ) { $sql .= ','; } - $first = false; # Get article ID. Efficient because it was loaded into the cache by getLinksTo(). - $linkID = $titleObj->getArticleID(); - $sql .= "({$linkID},'{$t}')"; - } - if ( ! $first ) { - wfQuery( $sql, DB_WRITE, $fname ); + $linkID = $titleObj->getArticleID(); + $brokenLinks[] = array( 'bl_from' => $linkID, 'bl_to' => $t ); } - - $sql = "DELETE FROM links WHERE l_to={$id}"; - wfQuery( $sql, DB_WRITE, $fname ); - - $sql = "DELETE FROM links WHERE l_from={$id}"; - wfQuery( $sql, DB_WRITE, $fname ); - - $sql = "DELETE FROM imagelinks WHERE il_from={$id}"; - wfQuery( $sql, DB_WRITE, $fname ); - - $sql = "DELETE FROM brokenlinks WHERE bl_from={$id}"; - wfQuery( $sql, DB_WRITE, $fname ); - - $sql = "DELETE FROM categorylinks WHERE cl_from={$id}"; - wfQuery( $sql, DB_WRITE, $fname ); + $dbw->insert( 'brokenlinks', $brokenLinks, $fname, 'IGNORE' ); + # Delete live links + $dbw->delete( 'links', array( 'l_to' => $id ) ); + $dbw->delete( 'links', array( 'l_from' => $id ) ); + $dbw->delete( 'imagelinks', array( 'il_from' => $id ) ); + $dbw->delete( 'brokenlinks', array( 'bl_from' => $id ) ); + $dbw->delete( 'categorylinks', array( 'cl_from' => $id ) ); + + # Log the deletion $log = new LogPage( wfMsg( 'dellogpage' ), wfMsg( 'dellogpagetext' ) ); $art = $this->mTitle->getPrefixedText(); $log->addEntry( wfMsg( 'deletedarticle', $art ), $reason ); @@ -1484,8 +1541,9 @@ class Article { function rollback() { - global $wgUser, $wgLang, $wgOut, $wgRequest, $wgIsMySQL, $wgIsPg; - + global $wgUser, $wgLang, $wgOut, $wgRequest; + $fname = "Article::rollback"; + if ( ! $wgUser->isSysop() ) { $wgOut->sysopRequired(); return; @@ -1494,28 +1552,30 @@ class Article { $wgOut->readOnlyPage( $this->getContent( true ) ); return; } - + $dbw =& wfGetDB( DB_MASTER ); + # Enhanced rollback, marks edits rc_bot=1 $bot = $wgRequest->getBool( 'bot' ); - + # Replace all this user's current edits with the next one down - $tt = wfStrencode( $this->mTitle->getDBKey() ); + $tt = $this->mTitle->getDBKey(); $n = $this->mTitle->getNamespace(); - - # Get the last editor - $sql = 'SELECT cur_id,cur_user,cur_user_text,cur_comment ' . - "FROM cur WHERE cur_title='{$tt}' AND cur_namespace={$n}"; - $res = wfQuery( $sql, DB_READ ); - if( ($x = wfNumRows( $res )) != 1 ) { + + # Get the last editor, lock table exclusively + $s = $dbw->getArray( 'cur', + array( 'cur_id','cur_user','cur_user_text','cur_comment' ), + array( 'cur_title' => $tt, 'cur_namespace' => $n ), + $fname, 'FOR UPDATE' + ); + if( $s === false ) { # Something wrong $wgOut->addHTML( wfMsg( 'notanarticle' ) ); return; } - $s = wfFetchObject( $res ); - $ut = wfStrencode( $s->cur_user_text ); + $ut = $dbw->strencode( $s->cur_user_text ); $uid = $s->cur_user; $pid = $s->cur_id; - + $from = str_replace( '_', ' ', $wgRequest->getVal( 'from' ) ); if( $from != $s->cur_user_text ) { $wgOut->setPageTitle(wfmsg('rollbackfailed')); @@ -1530,30 +1590,33 @@ class Article { } return; } - - # Get the last edit not by this guy - $use_index=$wgIsMySQL?"USE INDEX (name_title_timestamp)":""; - $oldtable=$wgIsPg?'"old"':'old'; - $sql = 'SELECT old_text,old_user,old_user_text,old_timestamp,old_flags ' . - "FROM $oldtable {$use_index} " . - "WHERE old_namespace={$n} AND old_title='{$tt}' " . - "AND (old_user <> {$uid} OR old_user_text <> '{$ut}') " . - 'ORDER BY inverse_timestamp LIMIT 1'; - $res = wfQuery( $sql, DB_READ ); - if( wfNumRows( $res ) != 1 ) { + # Get the last edit not by this guy + $s = $dbw->getArray( 'old', + array( 'old_text','old_user','old_user_text','old_timestamp','old_flags' ), + array( + 'old_namespace' => $n, + 'old_title' => $tt, + "old_user <> {$uid} OR old_user_text <> '{$ut}'" + ), $fname, array( 'FOR UPDATE', 'USE INDEX' => 'name_title_timestamp' ) + ); + if( $s === false ) { # Something wrong $wgOut->setPageTitle(wfMsg('rollbackfailed')); $wgOut->addHTML( wfMsg( 'cantrollback' ) ); return; } - $s = wfFetchObject( $res ); - + if ( $bot ) { # Mark all reverted edits as bot - $sql = 'UPDATE recentchanges SET rc_bot=1 WHERE ' . - "rc_cur_id=$pid AND rc_user=$uid AND rc_timestamp > '{$s->old_timestamp}'"; - wfQuery( $sql, DB_WRITE, $fname ); + $dbw->updateArray( 'recentchanges', + array( /* SET */ + 'rc_bot' => 1 + ), array( /* WHERE */ + 'rc_user' => $uid, + "rc_timestamp > '{$s->old_timestamp}'", + ), $fname + ); } # Save it! @@ -1565,7 +1628,7 @@ class Article { Article::onArticleEdit( $this->mTitle ); $wgOut->returnToMain( false ); } - + # Do standard deferred updates after page view @@ -1579,10 +1642,10 @@ class Article { $u = new SiteStatsUpdate( 1, 0, 0 ); array_push( $wgDeferredUpdateList, $u ); } - $u = new UserTalkUpdate( 0, $this->mTitle->getNamespace(), - $this->mTitle->getDBkey() ); - array_push( $wgDeferredUpdateList, $u ); } + $u = new UserTalkUpdate( 0, $this->mTitle->getNamespace(), + $this->mTitle->getDBkey() ); + array_push( $wgDeferredUpdateList, $u ); } # Do standard deferred updates after page edit. @@ -1592,17 +1655,18 @@ class Article { { global $wgDeferredUpdateList, $wgDBname, $wgMemc; global $wgMessageCache; - + wfSeedRandom(); if ( 0 == mt_rand( 0, 999 ) ) { + $dbw =& wfGetDB( DB_MASTER ); $cutoff = wfUnix2Timestamp( time() - ( 7 * 86400 ) ); $sql = "DELETE FROM recentchanges WHERE rc_timestamp < '{$cutoff}'"; - wfQuery( $sql, DB_WRITE ); + $dbw->query( $sql ); } $id = $this->getID(); $title = $this->mTitle->getPrefixedDBkey(); $shortTitle = $this->mTitle->getDBkey(); - + $adj = $this->mCountAdjustment; if ( 0 != $id ) { @@ -1639,7 +1703,7 @@ class Article { global $wgParser, $wgUser; return $wgParser->preSaveTransform( $text, $this->mTitle, $wgUser, ParserOptions::newFromUser( $wgUser ) ); } - + /* Caching functions */ # checkLastModified returns true if it has taken care of all @@ -1666,7 +1730,7 @@ class Article { $cache->loadFromFileCache(); return true; } else { - wfDebug( " tryFileCache() - starting buffer\n" ); + wfDebug( " tryFileCache() - starting buffer\n" ); ob_start( array(&$cache, 'saveToFileCache' ) ); } } else { @@ -1677,27 +1741,30 @@ class Article { function isFileCacheable() { global $wgUser, $wgUseFileCache, $wgShowIPinHeader, $wgRequest; extract( $wgRequest->getValues( 'action', 'oldid', 'diff', 'redirect', 'printable' ) ); - + return $wgUseFileCache and (!$wgShowIPinHeader) and ($this->getID() != 0) and ($wgUser->getId() == 0) and (!$wgUser->getNewtalk()) - and ($this->mTitle->getNamespace() != Namespace::getSpecial()) - and ($action == 'view') + and ($this->mTitle->getNamespace() != NS_SPECIAL ) + and ($action == 'view' || empty( $action )) and (!isset($oldid)) and (!isset($diff)) and (!isset($redirect)) and (!isset($printable)) and (!$this->mRedirectedFrom); } - + # Loads cur_touched and returns a value indicating if it should be used function checkTouched() { + $fname = 'Article::checkTouched'; + $id = $this->getID(); - $sql = 'SELECT cur_touched,cur_is_redirect FROM cur WHERE cur_id='.$id; - $res = wfQuery( $sql, DB_READ, 'Article::checkTouched' ); - if( $s = wfFetchObject( $res ) ) { + $dbr =& wfGetDB( DB_SLAVE ); + $s = $dbr->getArray( 'cur', array( 'cur_touched', 'cur_is_redirect' ), + array( 'cur_id' => $id ), $fname ); + if( $s !== false ) { $this->mTouched = $s->cur_touched; return !$s->cur_is_redirect; } else { @@ -1707,24 +1774,35 @@ class Article { # Edit an article without doing all that other stuff function quickEdit( $text, $comment = '', $minor = 0 ) { - global $wgUser, $wgMwRedir, $wgIsPg; + global $wgUser, $wgMwRedir; $fname = 'Article::quickEdit'; wfProfileIn( $fname ); + $dbw =& wfGetDB( DB_MASTER ); $ns = $this->mTitle->getNamespace(); $dbkey = $this->mTitle->getDBkey(); - $encDbKey = wfStrencode( $dbkey ); + $encDbKey = $dbw->strencode( $dbkey ); $timestamp = wfTimestampNow(); - + # Save to history - $oldtable=$wgIsPg?'"old"':'old'; - $sql = "INSERT INTO $oldtable (old_namespace,old_title,old_text,old_comment,old_user,old_user_text,old_timestamp,inverse_timestamp) - SELECT cur_namespace,cur_title,cur_text,cur_comment,cur_user,cur_user_text,cur_timestamp,99999999999999-cur_timestamp - FROM cur WHERE cur_namespace=$ns AND cur_title='$encDbKey'"; - wfQuery( $sql, DB_WRITE ); - + $dbw->insertSelect( 'old', 'cur', + array( + 'old_namespace' => 'cur_namespace', + 'old_title' => 'cur_title', + 'old_text' => 'cur_text', + 'old_comment' => 'cur_comment', + 'old_user' => 'cur_user', + 'old_user_text' => 'cur_user_text', + 'old_timestamp' => 'cur_timestamp', + 'inverse_timestamp' => '99999999999999-cur_timestamp', + ), array( + 'cur_namespace' => $ns, + 'cur_title' => $dbkey, + ), $fname + ); + # Use the affected row count to determine if the article is new - $numRows = wfAffectedRows(); + $numRows = $dbw->affectedRows(); # Make an array of fields to be inserted $fields = array( @@ -1742,14 +1820,14 @@ class Article { if ( $numRows ) { # Update article $fields['cur_is_new'] = 0; - wfUpdateArray( 'cur', $fields, array( 'cur_namespace' => $ns, 'cur_title' => $dbkey ), $fname ); + $dbw->updateArray( 'cur', $fields, array( 'cur_namespace' => $ns, 'cur_title' => $dbkey ), $fname ); } else { # Insert new article $fields['cur_is_new'] = 1; $fields['cur_namespace'] = $ns; $fields['cur_title'] = $dbkey; $fields['cur_random'] = $rand = number_format( mt_rand() / mt_getrandmax(), 12, '.', '' ); - wfInsertArray( 'cur', $fields, $fname ); + $dbw->insertArray( 'cur', $fields, $fname ); } wfProfileOut( $fname ); } @@ -1759,60 +1837,64 @@ class Article { $id = intval( $id ); global $wgHitcounterUpdateFreq; - if( $wgHitcounterUpdateFreq <= 1 ){ // - wfQuery('UPDATE cur SET cur_counter = cur_counter + 1 ' . - 'WHERE cur_id = '.$id, DB_WRITE); + $dbw =& wfGetDB( DB_MASTER ); + $curTable = $dbw->tableName( 'cur' ); + $hitcounterTable = $dbw->tableName( 'hitcounter' ); + $acchitsTable = $dbw->tableName( 'acchits' ); + + if( $wgHitcounterUpdateFreq <= 1 ){ // + $dbw->query( "UPDATE $curTable SET cur_counter = cur_counter + 1 WHERE cur_id = $id" ); return; } # Not important enough to warrant an error page in case of failure - $oldignore = wfIgnoreSQLErrors( true ); + $oldignore = $dbw->ignoreErrors( true ); - wfQuery("INSERT INTO hitcounter (hc_id) VALUES ({$id})", DB_WRITE); + $dbw->query( "INSERT INTO $hitcounterTable (hc_id) VALUES ({$id})" ); $checkfreq = intval( $wgHitcounterUpdateFreq/25 + 1 ); - if( (rand() % $checkfreq != 0) or (wfLastErrno() != 0) ){ + if( (rand() % $checkfreq != 0) or ($dbw->lastErrno() != 0) ){ # Most of the time (or on SQL errors), skip row count check - wfIgnoreSQLErrors( $oldignore ); + $dbw->ignoreErrors( $oldignore ); return; } - $res = wfQuery('SELECT COUNT(*) as n FROM hitcounter', DB_WRITE); - $row = wfFetchObject( $res ); + $res = $dbw->query("SELECT COUNT(*) as n FROM $hitcounterTable"); + $row = $dbw->fetchObject( $res ); $rown = intval( $row->n ); - if( $rown >= $wgHitcounterUpdateFreq ){ + if( $rown >= $wgHitcounterUpdateFreq ){ wfProfileIn( 'Article::incViewCount-collect' ); $old_user_abort = ignore_user_abort( true ); - wfQuery('LOCK TABLES hitcounter WRITE', DB_WRITE); - wfQuery('CREATE TEMPORARY TABLE acchits TYPE=HEAP '. - 'SELECT hc_id,COUNT(*) AS hc_n FROM hitcounter '. - 'GROUP BY hc_id', DB_WRITE); - wfQuery('DELETE FROM hitcounter', DB_WRITE); - wfQuery('UNLOCK TABLES', DB_WRITE); - wfQuery('UPDATE cur,acchits SET cur_counter=cur_counter + hc_n '. - 'WHERE cur_id = hc_id', DB_WRITE); - wfQuery('DROP TABLE acchits', DB_WRITE); + $dbw->query("LOCK TABLES $hitcounterTable WRITE"); + $dbw->query("CREATE TEMPORARY TABLE $acchitsTable TYPE=HEAP ". + "SELECT hc_id,COUNT(*) AS hc_n FROM $hitcounterTable ". + 'GROUP BY hc_id'); + $dbw->query("DELETE FROM $hitcounterTable"); + $dbw->query('UNLOCK TABLES'); + $dbw->query("UPDATE $curTable,$acchitsTable SET cur_counter=cur_counter + hc_n ". + 'WHERE cur_id = hc_id'); + $dbw->query("DROP TABLE $acchitsTable"); ignore_user_abort( $old_user_abort ); wfProfileOut( 'Article::incViewCount-collect' ); } - wfIgnoreSQLErrors( $oldignore ); + $dbw->ignoreErrors( $oldignore ); } # The onArticle*() functions are supposed to be a kind of hooks - # which should be called whenever any of the specified actions - # are done. + # which should be called whenever any of the specified actions + # are done. # - # This is a good place to put code to clear caches, for instance. + # This is a good place to put code to clear caches, for instance. - # This is called on page move and undelete, as well as edit + # This is called on page move and undelete, as well as edit /* static */ function onArticleCreate($title_obj){ global $wgUseSquid, $wgDeferredUpdateList; $titles = $title_obj->getBrokenLinksTo(); - - # Purge squid + + # Purge squid if ( $wgUseSquid ) { $urls = $title_obj->getSquidURLs(); foreach ( $titles as $linkTitle ) { @@ -1833,6 +1915,75 @@ class Article { /* static */ function onArticleEdit($title_obj){ LinkCache::linksccClearPage( $title_obj->getArticleID() ); } + + # Info about this page + + function info() + { + global $wgUser, $wgTitle, $wgOut, $wgLang, $wgAllowPageInfo; + $fname = 'Article::info'; + + if ( !$wgAllowPageInfo ) { + $wgOut->errorpage( "nosuchaction", "nosuchactiontext" ); + return; + } + + $dbr =& wfGetDB( DB_SLAVE ); + + $basenamespace = $wgTitle->getNamespace() & (~1); + $cur_clause = array( 'cur_title' => $wgTitle->getDBkey(), 'cur_namespace' => $basenamespace ); + $old_clause = array( 'old_title' => $wgTitle->getDBkey(), 'old_namespace' => $basenamespace ); + $wl_clause = array( 'wl_title' => $wgTitle->getDBkey(), 'wl_namespace' => $basenamespace ); + $fullTitle = $wgTitle->makeName($basenamespace, $wgTitle->getDBKey()); + $wgOut->setPagetitle( $fullTitle ); + $wgOut->setSubtitle( wfMsg( "infosubtitle" )); + + # first, see if the page exists at all. + $exists = $dbr->selectField( 'cur', 'COUNT(*)', $cur_clause, $fname ); + if ($exists < 1) { + $wgOut->addHTML( wfMsg("noarticletext") ); + } else { + $numwatchers = $dbr->selectField( 'watchlist', 'COUNT(*)', $wl_clause, $fname ); + $wgOut->addHTML( "
  • " . wfMsg("numwatchers", $numwatchers) . "
  • " ); + $old = $dbr->selectField( 'old', 'COUNT(*)', $old_clause, $fname ); + $wgOut->addHTML( "
  • " . wfMsg("numedits", $old + 1) . "
  • "); + + # to find number of distinct authors, we need to do some + # funny stuff because of the cur/old table split: + # - first, find the name of the 'cur' author + # - then, find the number of *other* authors in 'old' + + # find 'cur' author + $cur_author = $dbr->selectField( 'cur', 'cur_user_text', $cur_clause, $fname ); + + # find number of 'old' authors excluding 'cur' author + $authors = $dbr->selectField( 'old', 'COUNT(DISTINCT old_user_text)', + $old_clause + array( 'old_user_text<>' . $dbr->addQuotes( $cur_author ) ), $fname ) + 1; + + # now for the Talk page ... + $cur_clause = array( 'cur_title' => $wgTitle->getDBkey(), 'cur_namespace' => $basenamespace+1 ); + $old_clause = array( 'old_title' => $wgTitle->getDBkey(), 'old_namespace' => $basenamespace+1 ); + + # does it exist? + $exists = $dbr->selectField( 'cur', 'COUNT(*)', $cur_clause, $fname ); + + # number of edits + if ($exists > 0) { + $old = $dbr->selectField( 'old', 'COUNT(*)', $old_clause, $fname ); + $wgOut->addHTML( "
  • " . wfMsg("numtalkedits", $old + 1) . "
  • "); + } + $wgOut->addHTML( "
  • " . wfMsg("numauthors", $authors) . "
  • " ); + + # number of authors + if ($exists > 0) { + $cur_author = $dbr->selectField( 'cur', 'cur_user_text', $cur_clause, $fname ); + $authors = $dbr->selectField( 'cur', 'COUNT(DISTINCT old_user_text)', + $old_clause + array( 'old_user_text<>' . $dbr->addQuotes( $cur_author ) ), $fname ); + + $wgOut->addHTML( "
  • " . wfMsg("numtalkauthors", $authors) . "
" ); + } + } + } } ?>