Setting default user option using the content language rather than the UI language
[lhc/web/wiklou.git] / includes / PageHistory.php
index 5b57eec..2bc77a8 100644 (file)
@@ -1,9 +1,15 @@
 <?php
-
-/* Page history
-   Split off from Article.php and Skin.php, 2003-12-22
-*/
-
+/**
+ * Page history
+ * 
+ * Split off from Article.php and Skin.php, 2003-12-22
+ * @package MediaWiki
+ */
+
+/**
+ * @todo document
+ * @package MediaWiki
+ */
 class PageHistory {
        var $mArticle, $mTitle, $mSkin;
        var $lastline, $lastdate;
@@ -12,36 +18,35 @@ class PageHistory {
                $this->mArticle =& $article;
                $this->mTitle =& $article->mTitle;
        }
-       
+
        # This shares a lot of issues (and code) with Recent Changes
 
-       function history()
-       {
+       function history() {
                global $wgUser, $wgOut, $wgLang;
 
                # If page hasn't changed, client can cache this
-               
+
                if( $wgOut->checkLastModified( $this->mArticle->getTimestamp() ) ){
                        # Client cache fresh and headers sent, nothing more to do.
                        return;
                }
-               $fname = "PageHistory::history";
+               $fname = 'PageHistory::history';
                wfProfileIn( $fname );
 
                $wgOut->setPageTitle( $this->mTitle->getPRefixedText() );
-               $wgOut->setSubtitle( wfMsg( "revhistory" ) );
+               $wgOut->setSubtitle( wfMsg( 'revhistory' ) );
                $wgOut->setArticleFlag( false );
                $wgOut->setArticleRelated( true );
-               $wgOut->setRobotpolicy( "noindex,nofollow" );
+               $wgOut->setRobotpolicy( 'noindex,nofollow' );
 
                if( $this->mTitle->getArticleID() == 0 ) {
-                       $wgOut->addHTML( wfMsg( "nohistory" ) );
+                       $wgOut->addHTML( wfMsg( 'nohistory' ) );
                        wfProfileOut( $fname );
                        return;
                }
-               
+
                list( $limit, $offset ) = wfCheckLimits();
-               
+
                /* We have to draw the latest revision from 'cur' */
                $rawlimit = $limit;
                $rawoffset = $offset - 1;
@@ -51,42 +56,49 @@ class PageHistory {
                }
                /* Check one extra row to see whether we need to show 'next' and diff links */
                $limitplus = $rawlimit + 1;
-               
+
                $namespace = $this->mTitle->getNamespace();
                $title = $this->mTitle->getText();
+
+               $db =& wfGetDB( DB_SLAVE );
+               $use_index = $db->useIndexClause( 'name_title_timestamp' );
+               $oldtable = $db->tableName( 'old' );
+
                $sql = "SELECT old_id,old_user," .
                  "old_comment,old_user_text,old_timestamp,old_minor_edit ".
-                 "FROM old USE INDEX (name_title_timestamp) " .
+                 "FROM $oldtable $use_index " .
                  "WHERE old_namespace={$namespace} AND " .
-                 "old_title='" . wfStrencode( $this->mTitle->getDBkey() ) . "' " .
-                 "ORDER BY inverse_timestamp LIMIT $rawoffset, $limitplus";
-               $res = wfQuery( $sql, DB_READ, $fname );
+                 "old_title='" . $db->strencode( $this->mTitle->getDBkey() ) . "' " .
+                 "ORDER BY inverse_timestamp".$db->limitResult($limitplus,$rawoffset);
+               $res = $db->query( $sql, $fname );
+
+               $revs = $db->numRows( $res );
 
-               $revs = wfNumRows( $res );
-               
                if( $revs < $limitplus ) // the sql above tries to fetch one extra
                        $this->linesonpage = $revs;
                else
                        $this->linesonpage = $revs - 1;
 
                $atend = ($revs < $limitplus);
-               
+
                $this->mSkin = $wgUser->getSkin();
                $numbar = wfViewPrevNext(
                        $offset, $limit,
                        $this->mTitle->getPrefixedText(),
-                       "action=history", $atend );
+                       'action=history', $atend );
                $s = $numbar;
-               $this->submitbuttonhtml = ( $this->linesonpage > 0) ? '<input class="historysubmit" 
-               type="submit" accesskey="'.wfMsg('accesskey-compareselectedversions').
-                       '" title="'.wfMsg('tooltip-compareselectedversions').'" value="'.wfMsg('compareselectedversions')."\" />" :
-                       '';
+               if($this->linesonpage > 0) {
+                       $submitpart1 = '<input class="historysubmit" type="submit" accesskey="'.wfMsg('accesskey-compareselectedversions').
+                       '" title="'.wfMsg('tooltip-compareselectedversions').'" value="'.wfMsg('compareselectedversions').'"';
+                       $this->submitbuttonhtml1 = $submitpart1 . ' />';
+                       $this->submitbuttonhtml2 = $submitpart1 . ' id="historysubmit" />';
+               }
                $s .= $this->beginHistoryList();
                $counter = 1;
                if( $offset == 0 ){
                        $this->linesonpage++;
-                       $s .= $this->historyLine( 
-                               $this->mArticle->getTimestamp(), 
+                       $s .= $this->historyLine(
+                               $this->mArticle->getTimestamp(),
                                $this->mArticle->getUser(),
                                $this->mArticle->getUserText(), $namespace,
                                $title, 0, $this->mArticle->getComment(),
@@ -94,8 +106,8 @@ class PageHistory {
                                $counter++
                        );
                }
-               while ( $line = wfFetchObject( $res ) ) {
-                       $s .= $this->historyLine( 
+               while ( $line = $db->fetchObject( $res ) ) {
+                       $s .= $this->historyLine(
                                $line->old_timestamp, $line->old_user,
                                $line->old_user_text, $namespace,
                                $title, $line->old_id,
@@ -109,72 +121,70 @@ class PageHistory {
                wfProfileOut( $fname );
        }
 
-       function beginHistoryList()
-       {
+       function beginHistoryList() {
                global $wgTitle;
-               $this->lastdate = $this->lastline = "";
-               $s = "\n<p>" . wfMsg( "histlegend" ).'</p>'; 
-               $s .="\n<form action=\"" . $wgTitle->escapeLocalURL( '-' ) . "\" method=\"get\">";
-               $s .= "<input type=\"hidden\" name=\"title\" value=\"".wfEscapeHTML($wgTitle->getPrefixedDbKey())."\"/>\n";
-               $s .= !empty($this->submitbuttonhtml) ? $this->submitbuttonhtml."\n":'';
-               $s .= "" . "\n<ul id=\"pagehistory\" >";
+               $this->lastdate = $this->lastline = '';
+               $s = '<p>' . wfMsg( 'histlegend' ) . '</p>';
+               $s .= '<form action="' . $wgTitle->escapeLocalURL( '-' ) . '" method="get">';
+               $prefixedkey = htmlspecialchars($wgTitle->getPrefixedDbKey());
+               $s .= "<input type='hidden' name='title' value='{$prefixedkey}' />\n";
+               $s .= !empty($this->submitbuttonhtml1) ? $this->submitbuttonhtml1."\n":'';
+               $s .= '<ul id="pagehistory">';
                return $s;
        }
 
-       function endHistoryList( $skip = false )
-       {
-               $last = wfMsg( "last" );
+       function endHistoryList( $skip = false ) {
+               $last = wfMsg( 'last' );
 
-               $s = $skip ? "" : preg_replace( "/!OLDID![0-9]+!/", $last, $this->lastline );
-               $s .= "</ul>";
-               $s .= !empty($this->submitbuttonhtml) ? $this->submitbuttonhtml."\n":'';
-               $s .= "</form>\n";
+               $s = $skip ? '' : preg_replace( "/!OLDID![0-9]+!/", $last, $this->lastline );
+               $s .= '</ul>';
+               $s .= !empty($this->submitbuttonhtml2) ? $this->submitbuttonhtml2 : '';
+               $s .= '</form>';
                return $s;
        }
 
-       function historyLine( $ts, $u, $ut, $ns, $ttl, $oid, $c, $isminor, $counter = '' )
-       {
+       function historyLine( $ts, $u, $ut, $ns, $ttl, $oid, $c, $isminor, $counter = '' ) {
                global $wgLang;
 
                $artname = Title::makeName( $ns, $ttl );
-               $last = wfMsg( "last" );
-               $cur = wfMsg( "cur" );
-               $cr = wfMsg( "currentrev" );
+               $last = wfMsg( 'last' );
+               $cur = wfMsg( 'cur' );
+               $cr = wfMsg( 'currentrev' );
 
                if ( $oid && $this->lastline ) {
                        $ret = preg_replace( "/!OLDID!([0-9]+)!/", $this->mSkin->makeKnownLink(
                          $artname, $last, "diff=\\1&oldid={$oid}",'' ,'' ,' tabindex="'.$counter.'"' ), $this->lastline );
                } else {
-                       $ret = "";
+                       $ret = '';
                }
                $dt = $wgLang->timeanddate( $ts, true );
 
                if ( $oid ) {
-                       $q = "oldid={$oid}";
+                       $q = 'oldid='.$oid;
                } else {
-                       $q = "";
+                       $q = '';
                }
                $link = $this->mSkin->makeKnownLink( $artname, $dt, $q );
 
                if ( 0 == $u ) {
-                       $ul = $this->mSkin->makeKnownLink( $wgLang->specialPage( "Contributions" ),
-                               $ut, "target=" . $ut );
-               } else { 
+                       $ul = $this->mSkin->makeKnownLink( $wgLang->specialPage( 'Contributions' ),
+                               htmlspecialchars( $ut ), 'target=' . urlencode( $ut ) );
+               } else {
                        $ul = $this->mSkin->makeLink( $wgLang->getNsText(
-                               Namespace::getUser() ) . ":{$ut}", $ut );
+                               Namespace::getUser() ) . ':'.$ut , htmlspecialchars( $ut ) );
                }
 
-               $s = "<li>";
+               $s = '<li>';
                if ( $oid ) {
                        $curlink = $this->mSkin->makeKnownLink( $artname, $cur,
-                         "diff=0&oldid={$oid}" );
+                         'diff=0&oldid='.$oid );
                } else {
                        $curlink = $cur;
                }
-               $arbitrary = "";
+               $arbitrary = '';
                if( $this->linesonpage > 1) {
                        # XXX: move title texts to javascript
-                       $checkmark = "";
+                       $checkmark = '';
                        if ( !$oid ) {
                                $arbitrary = '<input type="radio" style="visibility:hidden" name="oldid" value="'.$oid.'" title="'.wfMsg('selectolderversionfordiff').'" />';
                                $checkmark = ' checked="checked"';
@@ -187,14 +197,13 @@ class PageHistory {
                }
                $s .= "({$curlink}) (!OLDID!{$oid}!) $arbitrary {$link} <span class='user'>{$ul}</span>";
                $s .= $isminor ? ' <span class="minor">'.wfMsg( "minoreditletter" ).'</span>': '' ;
-               
 
-               if ( "" != $c && "*" != $c ) {
 
-                       $c = $this->mSkin->formatcomment($c);
-                       $s .= " <em>(" . $c . ")</em>";
+               if ( '' != $c && '*' != $c ) {
+                       $c = $this->mSkin->formatcomment($c,$this->mTitle);
+                       $s .= " <em>($c)</em>";
                }
-               $s .= "</li>\n";
+               $s .= '</li>';
 
                $this->lastline = $s;
                return $ret;