Revert r19877; no reason is given for it but it breaks things such as parameter subst...
[lhc/web/wiklou.git] / includes / SpecialLog.php
index a7c01e5..409ca23 100644 (file)
@@ -1,26 +1,25 @@
 <?php
 # Copyright (C) 2004 Brion Vibber <brion@pobox.com>
 # http://www.mediawiki.org/
-# 
+#
 # This program is free software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
-# the Free Software Foundation; either version 2 of the License, or 
+# the Free Software Foundation; either version 2 of the License, or
 # (at your option) any later version.
-# 
+#
 # This program is distributed in the hope that it will be useful,
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
 # GNU General Public License for more details.
-# 
+#
 # You should have received a copy of the GNU General Public License along
 # with this program; if not, write to the Free Software Foundation, Inc.,
-# 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
 # http://www.gnu.org/copyleft/gpl.html
 
 /**
  *
- * @package MediaWiki
- * @subpackage SpecialPage
+ * @addtogroup SpecialPage
  */
 
 /**
  */
 function wfSpecialLog( $par = '' ) {
        global $wgRequest;
-       $logReader =& new LogReader( $wgRequest );
-       if( '' == $wgRequest->getVal( 'type' ) && !empty( $par ) ) {
+       $logReader = new LogReader( $wgRequest );
+       if( $wgRequest->getVal( 'type' ) == '' && $par != '' ) {
                $logReader->limitType( $par );
        }
-       $logViewer =& new LogViewer( $logReader );
+       $logViewer = new LogViewer( $logReader );
        $logViewer->show();
 }
 
 /**
  *
- * @package MediaWiki
- * @subpackage SpecialPage
+ * @addtogroup SpecialPage
  */
 class LogReader {
        var $db, $joinClauses, $whereClauses;
        var $type = '', $user = '', $title = null;
-       
+
        /**
         * @param WebRequest $request For internal use use a FauxRequest object to pass arbitrary parameters.
         */
        function LogReader( $request ) {
-               $this->db =& wfGetDB( DB_SLAVE );
+               $this->db = wfGetDB( DB_SLAVE );
                $this->setupQuery( $request );
        }
-       
+
        /**
         * Basic setup and applies the limiting factors from the WebRequest object.
         * @param WebRequest $request
@@ -61,18 +59,20 @@ class LogReader {
        function setupQuery( $request ) {
                $page = $this->db->tableName( 'page' );
                $user = $this->db->tableName( 'user' );
-               $this->joinClauses = array( "LEFT OUTER JOIN $page ON log_namespace=page_namespace AND log_title=page_title" );
-               $this->whereClauses = array( 'user_id=log_user' );
-               
+               $this->joinClauses = array( 
+                       "LEFT OUTER JOIN $page ON log_namespace=page_namespace AND log_title=page_title",
+                       "INNER JOIN $user ON user_id=log_user" );
+               $this->whereClauses = array();
+
                $this->limitType( $request->getVal( 'type' ) );
                $this->limitUser( $request->getText( 'user' ) );
                $this->limitTitle( $request->getText( 'page' ) );
                $this->limitTime( $request->getVal( 'from' ), '>=' );
                $this->limitTime( $request->getVal( 'until' ), '<=' );
-               
+
                list( $this->limit, $this->offset ) = $request->getLimitOffset();
        }
-       
+
        /**
         * Set the log reader to return only entries of the given type.
         * @param string $type A log type ('upload', 'delete', etc)
@@ -86,23 +86,30 @@ class LogReader {
                $safetype = $this->db->strencode( $type );
                $this->whereClauses[] = "log_type='$safetype'";
        }
-       
+
        /**
         * Set the log reader to return only entries by the given user.
-        * @param string $name Valid user name
+        * @param string $name (In)valid user name
         * @private
         */
        function limitUser( $name ) {
-               $title = Title::makeTitle( NS_USER, $name );
-               if( empty( $name ) || is_null( $title ) ) {
+               if ( $name == '' )
                        return false;
-               }
-               $this->user = str_replace( '_', ' ', $title->getDBkey() );
-               $safename = $this->db->strencode( $this->user );
-               $user = $this->db->tableName( 'user' );
-               $this->whereClauses[] = "user_name='$safename'";
+               $usertitle = Title::makeTitleSafe( NS_USER, $name );
+               if ( is_null( $usertitle ) )
+                       return false;
+               $this->user = $usertitle->getText();
+               
+               /* Fetch userid at first, if known, provides awesome query plan afterwards */
+               $userid = $this->db->selectField('user','user_id',array('user_name'=>$this->user));
+               if (!$userid)
+                       /* It should be nicer to abort query at all, 
+                          but for now it won't pass anywhere behind the optimizer */
+                       $this->whereClauses[] = "NULL";
+               else
+                       $this->whereClauses[] = "log_user=$userid";
        }
-       
+
        /**
         * Set the log reader to return only entries affecting the given page.
         * (For the block and rights logs, this is a user page.)
@@ -119,7 +126,7 @@ class LogReader {
                $ns = $title->getNamespace();
                $this->whereClauses[] = "log_namespace=$ns AND log_title='$safetitle'";
        }
-       
+
        /**
         * Set the log reader to return only entries in a given time range.
         * @param string $time Timestamp of one endpoint
@@ -134,7 +141,7 @@ class LogReader {
                $safetime = $this->db->strencode( wfTimestamp( TS_MW, $time ) );
                $this->whereClauses[] = "log_timestamp $direction '$safetime'";
        }
-       
+
        /**
         * Build an SQL query from all the set parameters.
         * @return string the SQL query
@@ -142,44 +149,44 @@ class LogReader {
         */
        function getQuery() {
                $logging = $this->db->tableName( "logging" );
-               $user = $this->db->tableName( 'user' );
-               $sql = "SELECT log_type, log_action, log_timestamp,
+               $sql = "SELECT /*! STRAIGHT_JOIN */ log_type, log_action, log_timestamp,
                        log_user, user_name,
                        log_namespace, log_title, page_id,
-                       log_comment, log_params FROM $user, $logging ";
+                       log_comment, log_params FROM $logging ";
                if( !empty( $this->joinClauses ) ) {
-                       $sql .= implode( ',', $this->joinClauses );
+                       $sql .= implode( ' ', $this->joinClauses );
                }
                if( !empty( $this->whereClauses ) ) {
                        $sql .= " WHERE " . implode( ' AND ', $this->whereClauses );
                }
                $sql .= " ORDER BY log_timestamp DESC ";
-               $sql .= $this->db->limitResult( $this->limit, $this->offset );
+               $sql = $this->db->limitResult($sql, $this->limit, $this->offset );
                return $sql;
        }
-       
+
        /**
         * Execute the query and start returning results.
         * @return ResultWrapper result object to return the relevant rows
         */
        function getRows() {
-               return $this->db->resultObject( $this->db->query( $this->getQuery() ) );
+               $res = $this->db->query( $this->getQuery(), 'LogReader::getRows' );
+               return $this->db->resultObject( $res );
        }
-       
+
        /**
         * @return string The query type that this LogReader has been limited to.
         */
        function queryType() {
                return $this->type;
        }
-       
+
        /**
         * @return string The username type that this LogReader has been limited to, if any.
         */
        function queryUser() {
                return $this->user;
        }
-       
+
        /**
         * @return string The text of the title that this LogReader has been limited to.
         */
@@ -194,24 +201,24 @@ class LogReader {
 
 /**
  *
- * @package MediaWiki
- * @subpackage SpecialPage
+ * @addtogroup SpecialPage
  */
 class LogViewer {
        /**
         * @var LogReader $reader
         */
        var $reader;
-       
+       var $numResults = 0;
+
        /**
         * @param LogReader &$reader where to get our data from
         */
        function LogViewer( &$reader ) {
                global $wgUser;
-               $this->skin =& $wgUser->getSkin();
+               $this->skin = $wgUser->getSkin();
                $this->reader =& $reader;
        }
-       
+
        /**
         * Take over the whole output page in $wgOut with the log display.
         */
@@ -219,11 +226,46 @@ class LogViewer {
                global $wgOut;
                $this->showHeader( $wgOut );
                $this->showOptions( $wgOut );
+               $result = $this->getLogRows();
                $this->showPrevNext( $wgOut );
-               $this->showList( $wgOut );
+               $this->doShowList( $wgOut, $result );
                $this->showPrevNext( $wgOut );
        }
-       
+
+       /**
+        * Load the data from the linked LogReader
+        * Preload the link cache
+        * Initialise numResults
+        *
+        * Must be called before calling showPrevNext
+        *
+        * @return object database result set
+        */
+       function getLogRows() {
+               $result = $this->reader->getRows();
+               $this->numResults = 0;
+
+               // Fetch results and form a batch link existence query
+               $batch = new LinkBatch;
+               while ( $s = $result->fetchObject() ) {
+                       // User link
+                       $batch->addObj( Title::makeTitleSafe( NS_USER, $s->user_name ) );
+                       $batch->addObj( Title::makeTitleSafe( NS_USER_TALK, $s->user_name ) );
+
+                       // Move destination link
+                       if ( $s->log_type == 'move' ) {
+                               $paramArray = LogPage::extractParams( $s->log_params );
+                               $title = Title::newFromText( $paramArray[0] );
+                               $batch->addObj( $title );
+                       }
+                       ++$this->numResults;
+               }
+               $batch->execute();
+
+               return $result;
+       }
+
+
        /**
         * Output just the list of entries given by the linked LogReader,
         * with extraneous UI elements. Use for displaying log fragments in
@@ -231,16 +273,25 @@ class LogViewer {
         * @param OutputPage $out where to send output
         */
        function showList( &$out ) {
-               $html = "\n<ul>\n";
-               $result = $this->reader->getRows();
-               while( $s = $result->fetchObject() ) {
-                       $html .= $this->logLine( $s );
+               $this->doShowList( $out, $this->getLogRows() );
+       }
+
+       function doShowList( &$out, $result ) {
+               // Rewind result pointer and go through it again, making the HTML
+               if ($this->numResults > 0) {
+                       $html = "\n<ul>\n";
+                       $result->seek( 0 );
+                       while( $s = $result->fetchObject() ) {
+                               $html .= $this->logLine( $s );
+                       }
+                       $html .= "\n</ul>\n";
+                       $out->addHTML( $html );
+               } else {
+                       $out->addWikiText( wfMsg( 'logempty' ) );
                }
                $result->free();
-               $html .= "\n</ul>\n";
-               $out->addHTML( $html );
        }
-       
+
        /**
         * @param Object $s a single row from the result set
         * @return string Formatted HTML list item
@@ -249,22 +300,38 @@ class LogViewer {
        function logLine( $s ) {
                global $wgLang;
                $title = Title::makeTitle( $s->log_namespace, $s->log_title );
-               $user = Title::makeTitleSafe( NS_USER, $s->user_name );
-               $time = $wgLang->timeanddate( $s->log_timestamp, true );
+               $time = $wgLang->timeanddate( wfTimestamp(TS_MW, $s->log_timestamp), true );
+
+               // Enter the existence or non-existence of this page into the link cache,
+               // for faster makeLinkObj() in LogPage::actionText()
+               $linkCache =& LinkCache::singleton();
                if( $s->page_id ) {
-                       $titleLink = $this->skin->makeKnownLinkObj( $title );
+                       $linkCache->addGoodLinkObj( $s->page_id, $title );
                } else {
-                       $titleLink = $this->skin->makeBrokenLinkObj( $title );
+                       $linkCache->addBadLinkObj( $title );
                }
-               $userLink = $this->skin->makeLinkObj( $user, htmlspecialchars( $s->user_name ) );
+
+               $userLink = $this->skin->userLink( $s->log_user, $s->user_name ) . $this->skin->userToolLinksRedContribs( $s->log_user, $s->user_name );
                $comment = $this->skin->commentBlock( $s->log_comment );
                $paramArray = LogPage::extractParams( $s->log_params );
-               
-               $action = LogPage::actionText( $s->log_type, $s->log_action, $titleLink, $paramArray );
-               $out = "<li>$time $userLink $action $comment</li>\n";
+               $revert = '';
+               if ( $s->log_type == 'move' && isset( $paramArray[0] ) ) {
+                       $specialTitle = SpecialPage::getTitleFor( 'Movepage' );
+                       $destTitle = Title::newFromText( $paramArray[0] );
+                       if ( $destTitle ) {
+                               $revert = '(' . $this->skin->makeKnownLinkObj( $specialTitle, wfMsg( 'revertmove' ),
+                                       'wpOldTitle=' . urlencode( $destTitle->getPrefixedDBkey() ) .
+                                       '&wpNewTitle=' . urlencode( $title->getPrefixedDBkey() ) .
+                                       '&wpReason=' . urlencode( wfMsgForContent( 'revertmove' ) ) .
+                                       '&wpMovetalk=0' ) . ')';
+                       }
+               }
+
+               $action = LogPage::actionText( $s->log_type, $s->log_action, $title, $this->skin, $paramArray, true, true );
+               $out = "<li>$time $userLink $action $comment $revert</li>\n";
                return $out;
        }
-       
+
        /**
         * @param OutputPage &$out where to send output
         * @private
@@ -276,7 +343,7 @@ class LogViewer {
                        $out->addWikiText( LogPage::logHeader( $type ) );
                }
        }
-       
+
        /**
         * @param OutputPage &$out where to send output
         * @private
@@ -284,17 +351,17 @@ class LogViewer {
        function showOptions( &$out ) {
                global $wgScript;
                $action = htmlspecialchars( $wgScript );
-               $title = Title::makeTitle( NS_SPECIAL, 'Log' );
+               $title = SpecialPage::getTitleFor( 'Log' );
                $special = htmlspecialchars( $title->getPrefixedDBkey() );
                $out->addHTML( "<form action=\"$action\" method=\"get\">\n" .
-                       "<input type='hidden' name='title' value=\"$special\" />\n" .
-                       $this->getTypeMenu() .
-                       $this->getUserInput() .
-                       $this->getTitleInput() .
-                       "<input type='submit' value=\"" . wfMsg( 'querybtn' ) . "\" />" .
+                       Xml::hidden( 'title', $special ) . "\n" .
+                       $this->getTypeMenu() . "\n" .
+                       $this->getUserInput() . "\n" .
+                       $this->getTitleInput() . "\n" .
+                       Xml::submitButton( wfMsg( 'allpagessubmit' ) ) . "\n" .
                        "</form>" );
        }
-       
+
        /**
         * @return string Formatted HTML
         * @private
@@ -302,32 +369,32 @@ class LogViewer {
        function getTypeMenu() {
                $out = "<select name='type'>\n";
                foreach( LogPage::validTypes() as $type ) {
-                       $text = htmlspecialchars( LogPage::logName( $type ) );
-                       $selected = ($type == $this->reader->queryType()) ? ' selected="selected"' : '';
-                       $out .= "<option value=\"$type\"$selected>$text</option>\n";
+                       $text = LogPage::logName( $type );
+                       $selected = ($type == $this->reader->queryType());
+                       $out .= Xml::option( $text, $type, $selected ) . "\n";
                }
-               $out .= "</select>\n";
+               $out .= '</select>';
                return $out;
        }
-       
+
        /**
         * @return string Formatted HTML
         * @private
         */
        function getUserInput() {
-               $user = htmlspecialchars( $this->reader->queryUser() );
-               return wfMsg('specialloguserlabel') . "<input type='text' name='user' size='12' value=\"$user\" />\n";
+               $user =  $this->reader->queryUser();
+               return Xml::inputLabel( wfMsg( 'specialloguserlabel' ), 'user', 'user', 12, $user );
        }
-       
+
        /**
         * @return string Formatted HTML
         * @private
         */
        function getTitleInput() {
-               $title = htmlspecialchars( $this->reader->queryTitle() );
-               return wfMsg('speciallogtitlelabel') . "<input type='text' name='page' size='20' value=\"$title\" />\n";
+               $title = $this->reader->queryTitle();
+               return Xml::inputLabel( wfMsg( 'speciallogtitlelabel' ), 'page', 'page', 20, $title );
        }
-       
+
        /**
         * @param OutputPage &$out where to send output
         * @private
@@ -335,18 +402,18 @@ class LogViewer {
        function showPrevNext( &$out ) {
                global $wgContLang,$wgRequest;
                $pieces = array();
-               $pieces[] = 'type=' . htmlspecialchars( $this->reader->queryType() );
-               $pieces[] = 'user=' . htmlspecialchars( $this->reader->queryUser() );
-               $pieces[] = 'page=' . htmlspecialchars( $this->reader->queryTitle() );
+               $pieces[] = 'type=' . urlencode( $this->reader->queryType() );
+               $pieces[] = 'user=' . urlencode( $this->reader->queryUser() );
+               $pieces[] = 'page=' . urlencode( $this->reader->queryTitle() );
                $bits = implode( '&', $pieces );
                list( $limit, $offset ) = $wgRequest->getLimitOffset();
-               
+
                # TODO: use timestamps instead of offsets to make it more natural
                # to go huge distances in time
                $html = wfViewPrevNext( $offset, $limit,
                        $wgContLang->specialpage( 'Log' ),
                        $bits,
-                       false);
+                       $this->numResults < $limit);
                $out->addHTML( '<p>' . $html . '</p>' );
        }
 }