* (bug 10117) Special:Wantedpages doesn't handle invalid titles in result - now print...
[lhc/web/wiklou.git] / includes / RecentChange.php
index 659e688..88a4598 100644 (file)
@@ -1,7 +1,6 @@
 <?php
 /**
  *
- * @package MediaWiki
  */
 
 /**
@@ -24,6 +23,8 @@
  *     rc_ip           IP address of the user in dotted quad notation
  *     rc_new          obsolete, use rc_type==RC_NEW
  *     rc_patrolled    boolean whether or not someone has marked this edit as patrolled
+ *     rc_old_len      integer byte length of the text before the edit
+ *     rc_new_len      the same after the edit
  *
  * mExtra:
  *     prefixedDBkey   prefixed db key, used by external app via msg queue
@@ -37,7 +38,6 @@
  *      numberofWatchingusers
  *
  * @todo document functions and variables
- * @package MediaWiki
  */
 class RecentChange
 {
@@ -47,14 +47,14 @@ class RecentChange
 
        # Factory methods
 
-       /* static */ function newFromRow( $row )
+       public static function newFromRow( $row )
        {
                $rc = new RecentChange;
                $rc->loadFromRow( $row );
                return $rc;
        }
 
-       /* static */ function newFromCurRow( $row, $rc_this_oldid = 0 )
+       public static function newFromCurRow( $row, $rc_this_oldid = 0 )
        {
                $rc = new RecentChange;
                $rc->loadFromCurRow( $row, $rc_this_oldid );
@@ -62,6 +62,24 @@ class RecentChange
                $rc->numberofWatchingusers = false;
                return $rc;
        }
+       
+       /**
+        * Obtain the recent change with a given rc_id value
+        *
+        * @param $rcid rc_id value to retrieve
+        * @return RecentChange
+        */
+       public static function newFromId( $rcid ) {
+               $dbr = wfGetDB( DB_SLAVE );
+               $res = $dbr->select( 'recentchanges', '*', array( 'rc_id' => $rcid ), __METHOD__ );
+               if( $res && $dbr->numRows( $res ) > 0 ) {
+                       $row = $dbr->fetchObject( $res );
+                       $dbr->freeResult( $res );
+                       return self::newFromRow( $row );
+               } else {
+                       return NULL;
+               }
+       }
 
        # Accessors
 
@@ -95,10 +113,10 @@ class RecentChange
        # Writes the data in this object to the database
        function save()
        {
-               global $wgLocalInterwiki, $wgPutIPinRC, $wgRC2UDPAddress, $wgRC2UDPPort, $wgRC2UDPPrefix, $wgUseRCPatrol;
+               global $wgLocalInterwiki, $wgPutIPinRC, $wgRC2UDPAddress, $wgRC2UDPPort, $wgRC2UDPPrefix;
                $fname = 'RecentChange::save';
 
-               $dbw =& wfGetDB( DB_MASTER );
+               $dbw = wfGetDB( DB_MASTER );
                if ( !is_array($this->mExtra) ) {
                        $this->mExtra = array();
                }
@@ -108,11 +126,21 @@ class RecentChange
                        $this->mAttribs['rc_ip'] = '';
                }
 
+               ## If our database is strict about IP addresses, use NULL instead of an empty string
+               if ( $dbw->strictIPs() and $this->mAttribs['rc_ip'] == '' ) {
+                       unset( $this->mAttribs['rc_ip'] );
+               }
+
                # Fixup database timestamps
                $this->mAttribs['rc_timestamp'] = $dbw->timestamp($this->mAttribs['rc_timestamp']);
                $this->mAttribs['rc_cur_time'] = $dbw->timestamp($this->mAttribs['rc_cur_time']);
                $this->mAttribs['rc_id'] = $dbw->nextSequenceValue( 'rc_rc_id_seq' );
 
+               ## If we are using foreign keys, an entry of 0 for the page_id will fail, so use NULL
+               if ( $dbw->cascadingDeletes() and $this->mAttribs['rc_cur_id']==0 ) {
+                       unset ( $this->mAttribs['rc_cur_id'] );
+               }
+
                # Insert new row
                $dbw->insert( 'recentchanges', $this->mAttribs, $fname );
 
@@ -163,20 +191,23 @@ class RecentChange
                        }
                }
 
-               // E-mail notifications
+               # E-mail notifications
                global $wgUseEnotif;
                if( $wgUseEnotif ) {
                        # this would be better as an extension hook
+                       global $wgUser;
                        include_once( "UserMailer.php" );
                        $enotif = new EmailNotification();
                        $title = Title::makeTitle( $this->mAttribs['rc_namespace'], $this->mAttribs['rc_title'] );
-                       $enotif->notifyOnPageChange( $title,
+                       $enotif->notifyOnPageChange( $wgUser, $title,
                                $this->mAttribs['rc_timestamp'],
                                $this->mAttribs['rc_comment'],
                                $this->mAttribs['rc_minor'],
                                $this->mAttribs['rc_last_oldid'] );
                }
 
+               # Notify extensions
+               wfRunHooks( 'RecentChange_save', array( &$this ) );
        }
 
        # Marks a certain row as patrolled
@@ -184,7 +215,7 @@ class RecentChange
        {
                $fname = 'RecentChange::markPatrolled';
 
-               $dbw =& wfGetDB( DB_MASTER );
+               $dbw = wfGetDB( DB_MASTER );
 
                $dbw->update( 'recentchanges',
                        array( /* SET */
@@ -200,8 +231,9 @@ class RecentChange
                $oldId, $lastTimestamp, $bot = "default", $ip = '', $oldSize = 0, $newSize = 0,
                $newId = 0)
        {
-               if ( $bot == 'default ' ) {
-                       $bot = $user->isBot();
+
+               if ( $bot === 'default' ) {
+                       $bot = $user->isAllowed( 'bot' );
                }
 
                if ( !$ip ) {
@@ -228,9 +260,11 @@ class RecentChange
                        'rc_bot'        => $bot ? 1 : 0,
                        'rc_moved_to_ns'        => 0,
                        'rc_moved_to_title'     => '',
-                       'rc_ip' => $ip,
-                       'rc_patrolled' => 0,
-                       'rc_new'        => 0 # obsolete
+                       'rc_ip'         => $ip,
+                       'rc_patrolled'  => 0,
+                       'rc_new'        => 0,  # obsolete
+                       'rc_old_len'    => $oldSize,
+                       'rc_new_len'    => $newSize
                );
 
                $rc->mExtra =  array(
@@ -243,9 +277,12 @@ class RecentChange
                return( $rc->mAttribs['rc_id'] );
        }
 
-       # Makes an entry in the database corresponding to page creation
-       # Note: the title object must be loaded with the new id using resetArticleID()
-       /*static*/ function notifyNew( $timestamp, &$title, $minor, &$user, $comment, $bot = "default",
+       /**
+        * Makes an entry in the database corresponding to page creation
+        * Note: the title object must be loaded with the new id using resetArticleID()
+        * @todo Document parameters and return
+        */
+       public static function notifyNew( $timestamp, &$title, $minor, &$user, $comment, $bot = 'default',
          $ip='', $size = 0, $newId = 0 )
        {
                if ( !$ip ) {
@@ -254,8 +291,8 @@ class RecentChange
                                $ip = '';
                        }
                }
-               if ( $bot == 'default' ) {
-                       $bot = $user->isBot();
+               if ( $bot === 'default' ) {
+                       $bot = $user->isAllowed( 'bot' );
                }
 
                $rc = new RecentChange;
@@ -277,7 +314,9 @@ class RecentChange
                        'rc_moved_to_title' => '',
                        'rc_ip'             => $ip,
                        'rc_patrolled'      => 0,
-                       'rc_new'        => 1 # obsolete
+                       'rc_new'            => 1, # obsolete
+                       'rc_old_len'        => 0,
+                       'rc_new_len'        => $size
                );
 
                $rc->mExtra =  array(
@@ -314,12 +353,14 @@ class RecentChange
                        'rc_comment'    => $comment,
                        'rc_this_oldid' => 0,
                        'rc_last_oldid' => 0,
-                       'rc_bot'        => $user->isBot() ? 1 : 0,
+                       'rc_bot'        => $user->isAllowed( 'bot' ) ? 1 : 0,
                        'rc_moved_to_ns'        => $newTitle->getNamespace(),
                        'rc_moved_to_title'     => $newTitle->getDBkey(),
                        'rc_ip'         => $ip,
                        'rc_new'        => 0, # obsolete
-                       'rc_patrolled' => 1
+                       'rc_patrolled'  => 1,
+                       'rc_old_len'    => NULL,
+                       'rc_new_len'    => NULL,
                );
 
                $rc->mExtra = array(
@@ -364,12 +405,14 @@ class RecentChange
                        'rc_comment'    => $comment,
                        'rc_this_oldid' => 0,
                        'rc_last_oldid' => 0,
-                       'rc_bot'        => $user->isBot() ? 1 : 0,
+                       'rc_bot'        => $user->isAllowed( 'bot' ) ? 1 : 0,
                        'rc_moved_to_ns'        => 0,
                        'rc_moved_to_title'     => '',
                        'rc_ip' => $ip,
                        'rc_patrolled' => 1,
-                       'rc_new'        => 0 # obsolete
+                       'rc_new'        => 0, # obsolete
+                       'rc_old_len'    => NULL,
+                       'rc_new_len'    => NULL,
                );
                $rc->mExtra =  array(
                        'prefixedDBkey' => $title->getPrefixedDBkey(),
@@ -391,7 +434,7 @@ class RecentChange
                $this->mExtra = array();
        }
 
-       # Makes a pseudo-RC entry from a cur row, for watchlists and things
+       # Makes a pseudo-RC entry from a cur row
        function loadFromCurRow( $row )
        {
                $this->mAttribs = array(
@@ -411,14 +454,25 @@ class RecentChange
                        'rc_moved_to_ns'        => 0,
                        'rc_moved_to_title'     => '',
                        'rc_ip' => '',
-                       'rc_patrolled' => '1',  # we can't support patrolling on the Watchlist
-                                               # currently because it uses cur, not recentchanges
-                       'rc_new' => $row->page_is_new # obsolete
+                       'rc_id' => $row->rc_id,
+                       'rc_patrolled' => $row->rc_patrolled,
+                       'rc_new' => $row->page_is_new, # obsolete
+                       'rc_old_len' => $row->rc_old_len,
+                       'rc_new_len' => $row->rc_new_len,
                );
 
                $this->mExtra = array();
        }
 
+       /**
+        * Get an attribute value
+        *
+        * @param $name Attribute name
+        * @return mixed
+        */
+       public function getAttribute( $name ) {
+               return isset( $this->mAttribs[$name] ) ? $this->mAttribs[$name] : NULL;
+       }
 
        /**
         * Gets the end part of the diff URL associated with this object
@@ -447,6 +501,8 @@ class RecentChange
        function getIRCLine() {
                global $wgUseRCPatrol;
 
+               // FIXME: Would be good to replace these 2 extract() calls with something more explicit
+               // e.g. list ($rc_type, $rc_id) = array_values ($this->mAttribs); [or something like that]
                extract($this->mAttribs);
                extract($this->mExtra);
 
@@ -505,5 +561,37 @@ class RecentChange
                return $fullString;
        }
 
+       /**
+        * Returns the change size (HTML).
+        * The lengths can be given optionally.
+        */
+       function getCharacterDifference( $old = 0, $new = 0 ) {
+               global $wgRCChangedSizeThreshold, $wgLang;
+
+               if( $old === 0 ) {
+                       $old = $this->mAttribs['rc_old_len'];
+               }
+               if( $new === 0 ) {
+                       $new = $this->mAttribs['rc_new_len'];
+               }
+
+               if( $old === NULL || $new === NULL ) {
+                       return '';
+               }
+
+               $szdiff = $new - $old;
+               $formatedSize = wfMsgExt( 'rc-change-size', array( 'parsemag', 'escape'),
+                       $wgLang->formatNum($szdiff) );
+
+               if( $szdiff < $wgRCChangedSizeThreshold ) {
+                       return '<strong class=\'mw-plusminus-neg\'>(' . $formatedSize . ')</strong>';
+               } elseif( $szdiff === 0 ) {
+                       return '<span class=\'mw-plusminus-null\'>(' . $formatedSize . ')</span>';
+               } elseif( $szdiff > 0 ) {
+                       return '<span class=\'mw-plusminus-pos\'>(+' . $formatedSize . ')</span>';
+               } else {
+                       return '<span class=\'mw-plusminus-neg\'>(' . $formatedSize . ')</span>';
+               }
+       }
 }
 ?>