Merge "Put callback function within class in SiteConfigurationTest"
[lhc/web/wiklou.git] / includes / Block.php
index 8673a3d..3575b9d 100644 (file)
@@ -101,21 +101,6 @@ class Block {
                $this->mFromMaster = false;
        }
 
-       /**
-        * Load a block from the database, using either the IP address or
-        * user ID. Tries the user ID first, and if that doesn't work, tries
-        * the address.
-        *
-        * @param string $address IP address of user/anon
-        * @param $user Integer: user id of user
-        * @return Block Object
-        * @deprecated since 1.18
-        */
-       public static function newFromDB( $address, $user = 0 ) {
-               wfDeprecated( __METHOD__, '1.18' );
-               return self::newFromTarget( User::whoIs( $user ), $address );
-       }
-
        /**
         * Load a blocked user from their block id.
         *
@@ -186,44 +171,6 @@ class Block {
                );
        }
 
-       /**
-        * Clear all member variables in the current object. Does not clear
-        * the block from the DB.
-        * @deprecated since 1.18
-        */
-       public function clear() {
-               wfDeprecated( __METHOD__, '1.18' );
-               # Noop
-       }
-
-       /**
-        * Get a block from the DB, with either the given address or the given username
-        *
-        * @param string $address The IP address of the user, or blank to skip IP blocks
-        * @param int $user The user ID, or zero for anonymous users
-        * @return Boolean: the user is blocked from editing
-        * @deprecated since 1.18
-        */
-       public function load( $address = '', $user = 0 ) {
-               wfDeprecated( __METHOD__, '1.18' );
-               if ( $user ) {
-                       $username = User::whoIs( $user );
-                       $block = self::newFromTarget( $username, $address );
-               } else {
-                       $block = self::newFromTarget( null, $address );
-               }
-
-               if ( $block instanceof Block ) {
-                       # This is mildly evil, but hey, it's B/C :D
-                       foreach ( $block as $variable => $value ) {
-                               $this->$variable = $value;
-                       }
-                       return true;
-               } else {
-                       return false;
-               }
-       }
-
        /**
         * Load a block from the database which affects the already-set $this->target:
         *     1) A block directly on the given user or IP
@@ -488,13 +435,14 @@ class Block {
         * Update a block in the DB with new parameters.
         * The ID field needs to be loaded first.
         *
-        * @return Int number of affected rows, which should probably be 1 or something has
-        *     gone slightly awry
+        * @return bool|array False on failure, array on success: ('id' => block ID, 'autoIds' => array of autoblock IDs)
         */
        public function update() {
                wfDebug( "Block::update; timestamp {$this->mTimestamp}\n" );
                $dbw = wfGetDB( DB_MASTER );
 
+               $dbw->startAtomic( __METHOD__ );
+
                $dbw->update(
                        'ipblocks',
                        $this->getDatabaseArray( $dbw ),
@@ -502,7 +450,23 @@ class Block {
                        __METHOD__
                );
 
-               return $dbw->affectedRows();
+               $affected = $dbw->affectedRows();
+
+               $dbw->update(
+                       'ipblocks',
+                       $this->getAutoblockUpdateArray(),
+                       array( 'ipb_parent_block_id' => $this->getId() ),
+                       __METHOD__
+               );
+
+               $dbw->endAtomic( __METHOD__ );
+
+               if ( $affected ) {
+                       $auto_ipd_ids = $this->doRetroactiveAutoblock();
+                       return array( 'id' => $this->mId, 'autoIds' => $auto_ipd_ids );
+               }
+
+               return false;
        }
 
        /**
@@ -545,6 +509,20 @@ class Block {
                return $a;
        }
 
+       /**
+        * @return Array
+        */
+       protected function getAutoblockUpdateArray() {
+               return array(
+                       'ipb_by'               => $this->getBy(),
+                       'ipb_by_text'          => $this->getByName(),
+                       'ipb_reason'           => $this->mReason,
+                       'ipb_create_account'   => $this->prevents( 'createaccount' ),
+                       'ipb_deleted'          => (int)$this->mHideName, // typecast required for SQLite
+                       'ipb_allow_usertalk'   => !$this->prevents( 'editownusertalk' ),
+               );
+       }
+
        /**
         * Retroactively autoblocks the last IP used by the user (if it is a user)
         * blocked by this Block.
@@ -859,17 +837,6 @@ class Block {
                return $this->mId;
        }
 
-       /**
-        * Get/set the SELECT ... FOR UPDATE flag
-        * @deprecated since 1.18
-        *
-        * @param $x Bool
-        */
-       public function forUpdate( $x = null ) {
-               wfDeprecated( __METHOD__, '1.18' );
-               # noop
-       }
-
        /**
         * Get/set a flag determining whether the master is used for reads
         *
@@ -946,33 +913,6 @@ class Block {
                }
        }
 
-       /**
-        * Encode expiry for DB
-        *
-        * @param string $expiry timestamp for expiry, or
-        * @param $db DatabaseBase object
-        * @return String
-        * @deprecated since 1.18; use $dbw->encodeExpiry() instead
-        */
-       public static function encodeExpiry( $expiry, $db ) {
-               wfDeprecated( __METHOD__, '1.18' );
-               return $db->encodeExpiry( $expiry );
-       }
-
-       /**
-        * Decode expiry which has come from the DB
-        *
-        * @param string $expiry Database expiry format
-        * @param int $timestampType Requested timestamp format
-        * @return String
-        * @deprecated since 1.18; use $wgLang->formatExpiry() instead
-        */
-       public static function decodeExpiry( $expiry, $timestampType = TS_MW ) {
-               wfDeprecated( __METHOD__, '1.18' );
-               global $wgContLang;
-               return $wgContLang->formatExpiry( $expiry, $timestampType );
-       }
-
        /**
         * Get a timestamp of the expiry for autoblocks
         *
@@ -985,18 +925,6 @@ class Block {
                return wfTimestamp( TS_MW, wfTimestamp( TS_UNIX, $timestamp ) + $wgAutoblockExpiry );
        }
 
-       /**
-        * Gets rid of unneeded numbers in quad-dotted/octet IP strings
-        * For example, 127.111.113.151/24 -> 127.111.113.0/24
-        * @param string $range IP address to normalize
-        * @return string
-        * @deprecated since 1.18, call IP::sanitizeRange() directly
-        */
-       public static function normaliseRange( $range ) {
-               wfDeprecated( __METHOD__, '1.18' );
-               return IP::sanitizeRange( $range );
-       }
-
        /**
         * Purge expired blocks from the ipblocks table
         */
@@ -1013,29 +941,6 @@ class Block {
                } );
        }
 
-       /**
-        * Get a value to insert into expiry field of the database when infinite expiry
-        * is desired
-        * @deprecated since 1.18, call $dbr->getInfinity() directly
-        * @return String
-        */
-       public static function infinity() {
-               wfDeprecated( __METHOD__, '1.18' );
-               return wfGetDB( DB_SLAVE )->getInfinity();
-       }
-
-       /**
-        * Convert a submitted expiry time, which may be relative ("2 weeks", etc) or absolute
-        * ("24 May 2034"), into an absolute timestamp we can put into the database.
-        * @param string $expiry whatever was typed into the form
-        * @return String: timestamp or "infinity" string for th DB implementation
-        * @deprecated since 1.18 moved to SpecialBlock::parseExpiryInput()
-        */
-       public static function parseExpiryInput( $expiry ) {
-               wfDeprecated( __METHOD__, '1.18' );
-               return SpecialBlock::parseExpiryInput( $expiry );
-       }
-
        /**
         * Given a target and the target's type, get an existing Block object if possible.
         * @param $specificTarget String|User|Int a block target, which may be one of several types: