Tweak for r39314: don't add a line break before the "Go" button on the watchlist...
[lhc/web/wiklou.git] / includes / LogPage.php
index df6b9da..647cc01 100644 (file)
@@ -20,7 +20,7 @@
 
 /**
  * Contain log classes
- *
+ * @file
  */
 
 /**
  *
  */
 class LogPage {
+       const DELETED_ACTION = 1;
+       const DELETED_COMMENT = 2;
+       const DELETED_USER = 4;
+    const DELETED_RESTRICTED = 8;
        /* @access private */
-       var $type, $action, $comment, $params, $target;
+       var $type, $action, $comment, $params, $target, $doer;
        /* @acess public */
        var $updateRecentChanges;
 
@@ -47,41 +51,40 @@ class LogPage {
                $this->updateRecentChanges = $rc;
        }
 
-       function saveContent() {
-               if( wfReadOnly() ) return false;
-
-               global $wgUser;
+       protected function saveContent() {
+               global $wgUser, $wgLogRestrictions;
                $fname = 'LogPage::saveContent';
 
                $dbw = wfGetDB( DB_MASTER );
-               $uid = $wgUser->getID();
                $log_id = $dbw->nextSequenceValue( 'log_log_id_seq' );
 
                $this->timestamp = $now = wfTimestampNow();
                $data = array(
+                       'log_id' => $log_id,
                        'log_type' => $this->type,
                        'log_action' => $this->action,
                        'log_timestamp' => $dbw->timestamp( $now ),
-                       'log_user' => $uid,
+                       'log_user' => $this->doer->getId(),
                        'log_namespace' => $this->target->getNamespace(),
                        'log_title' => $this->target->getDBkey(),
                        'log_comment' => $this->comment,
                        'log_params' => $this->params
                );
-
-               # log_id doesn't exist on Wikimedia servers yet, and it's a tricky 
-               # schema update to do. Hack it for now to ignore the field on MySQL.
-               if ( !is_null( $log_id ) ) {
-                       $data['log_id'] = $log_id;
-               }
                $dbw->insert( 'logging', $data, $fname );
+               $newId = !is_null($log_id) ? $log_id : $dbw->insertId();
 
+               if( !($dbw->affectedRows() > 0) ) {
+                       wfDebugLog( "logging", "LogPage::saveContent failed to insert row - Error {$dbw->lastErrno()}: {$dbw->lastError()}" );
+               }
                # And update recentchanges
-               if ( $this->updateRecentChanges ) {
-                       $titleObj = SpecialPage::getTitleFor( 'Log', $this->type );
-                       $rcComment = $this->getRcComment();
-                       RecentChange::notifyLog( $now, $titleObj, $wgUser, $rcComment, '',
-                               $this->type, $this->action, $this->target, $this->comment, $this->params );
+               if( $this->updateRecentChanges ) {
+                       # Don't add private logs to RC!
+                       if( !isset($wgLogRestrictions[$this->type]) || $wgLogRestrictions[$this->type]=='*' ) {
+                               $titleObj = SpecialPage::getTitleFor( 'Log', $this->type );
+                               $rcComment = $this->getRcComment();
+                               RecentChange::notifyLog( $now, $titleObj, $this->doer, $rcComment, '',
+                                       $this->type, $this->action, $this->target, $this->comment, $this->params, $newId );
+                       }
                }
                return true;
        }
@@ -129,24 +132,28 @@ class LogPage {
 
        /**
         * @todo handle missing log types
-        * @static
+        * @param string $type logtype
+        * @return string Headertext of this logtype
         */
        static function logHeader( $type ) {
-               global $wgLogHeaders;
-               return wfMsg( $wgLogHeaders[$type] );
+               global $wgLogHeaders, $wgMessageCache;
+               $wgMessageCache->loadAllMessages();
+               return wfMsgExt($wgLogHeaders[$type],array('parseinline'));
        }
 
        /**
         * @static
+        * @return HTML string
         */
        static function actionText( $type, $action, $title = NULL, $skin = NULL, $params = array(), $filterWikilinks=false ) {
-               global $wgLang, $wgContLang, $wgLogActions;
+               global $wgLang, $wgContLang, $wgLogActions, $wgMessageCache;
 
+               $wgMessageCache->loadAllMessages();
                $key = "$type/$action";
-               
+
                if( $key == 'patrol/patrol' )
                        return PatrolLog::makeActionText( $title, $params, $skin );
-               
+
                if( isset( $wgLogActions[$key] ) ) {
                        if( is_null( $title ) ) {
                                $rv=wfMsg( $wgLogActions[$key] );
@@ -155,7 +162,7 @@ class LogPage {
 
                                        switch( $type ) {
                                                case 'move':
-                                                       $titleLink = $skin->makeLinkObj( $title, $title->getPrefixedText(), 'redirect=no' );
+                                                       $titleLink = $skin->makeLinkObj( $title, htmlspecialchars( $title->getPrefixedText() ), 'redirect=no' );
                                                        $params[0] = $skin->makeLinkObj( Title::newFromText( $params[0] ), htmlspecialchars( $params[0] ) );
                                                        break;
                                                case 'block':
@@ -188,6 +195,11 @@ class LogPage {
                                if( $key == 'rights/rights' ) {
                                        if ($skin) {
                                                $rightsnone = wfMsg( 'rightsnone' );
+                                               foreach ( $params as &$param ) {
+                                                       $groupArray = array_map( 'trim', explode( ',', $param ) );
+                                                       $groupArray = array_map( array( 'User', 'getGroupName' ), $groupArray );
+                                                       $param = $wgLang->listToText( $groupArray );
+                                               }
                                        } else {
                                                $rightsnone = wfMsgForContent( 'rightsnone' );
                                        }
@@ -204,7 +216,7 @@ class LogPage {
                                        }
                                } else {
                                        array_unshift( $params, $titleLink );
-                                       if ( $key == 'block/block' ) {
+                                       if ( $key == 'block/block' || $key == 'suppress/block' ) {
                                                if ( $skin ) {
                                                        $params[1] = '<span title="' . htmlspecialchars( $params[1] ). '">' . $wgLang->translateBlockExpiry( $params[1] ) . '</span>';
                                                } else {
@@ -218,8 +230,14 @@ class LogPage {
                                }
                        }
                } else {
-                       wfDebug( "LogPage::actionText - unknown action $key\n" );
-                       $rv = "$action";
+                       global $wgLogActionsHandlers;
+                       if( isset( $wgLogActionsHandlers[$key] ) ) {
+                               $args = func_get_args();
+                               $rv = call_user_func_array( $wgLogActionsHandlers[$key], $args );
+                       } else {
+                               wfDebug( "LogPage::actionText - unknown action $key\n" );
+                               $rv = "$action";
+                       }
                }
                if( $filterWikilinks ) {
                        $rv = str_replace( "[[", "", $rv );
@@ -234,8 +252,9 @@ class LogPage {
         * @param object &$target A title object.
         * @param string $comment Description associated
         * @param array $params Parameters passed later to wfMsg.* functions
+        * @param User $doer The user doing the action
         */
-       function addEntry( $action, $target, $comment, $params = array() ) {
+       function addEntry( $action, $target, $comment, $params = array(), $doer = null ) {
                if ( !is_array( $params ) ) {
                        $params = array( $params );
                }
@@ -244,6 +263,15 @@ class LogPage {
                $this->target = $target;
                $this->comment = $comment;
                $this->params = LogPage::makeParamBlob( $params );
+               
+               if ($doer === null) {
+                       global $wgUser;
+                       $doer = $wgUser;
+               } elseif (!is_object( $doer ) ) {
+                       $doer = User::newFromId( $doer );
+               }
+               
+               $this->doer = $doer;
 
                $this->actionText = LogPage::actionText( $this->type, $action, $target, NULL, $params );
 
@@ -269,13 +297,13 @@ class LogPage {
                        return explode( "\n", $blob );
                }
        }
-       
+
        /**
         * Convert a comma-delimited list of block log flags
         * into a more readable (and translated) form
         *
         * @param $flags Flags to format
-        * @param $forContent Whetever to localize the message depending of the user
+        * @param $forContent Whether to localize the message depending of the user
         *                    language
         * @return string
         */
@@ -289,12 +317,12 @@ class LogPage {
                        return '';
                }
        }
-       
+
        /**
         * Translate a block log flag if possible
         *
         * @param $flag Flag to translate
-        * @param $forContent Whetever to localize the message depending of the user
+        * @param $forContent Whether to localize the message depending of the user
         *                    language
         * @return string
         */
@@ -310,7 +338,12 @@ class LogPage {
                }
                return $messages[$flag];
        }
-       
 }
 
-
+/**
+ * Aliases for backwards compatibility with 1.6
+ */
+define( 'MW_LOG_DELETED_ACTION', LogPage::DELETED_ACTION );
+define( 'MW_LOG_DELETED_USER', LogPage::DELETED_USER );
+define( 'MW_LOG_DELETED_COMMENT', LogPage::DELETED_COMMENT );
+define( 'MW_LOG_DELETED_RESTRICTED', LogPage::DELETED_RESTRICTED );