Merge "Remove methods in Block deprecated in 1.18"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Fri, 17 Jan 2014 20:40:06 +0000 (20:40 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Fri, 17 Jan 2014 20:40:06 +0000 (20:40 +0000)
RELEASE-NOTES-1.23
includes/Block.php
tests/phpunit/includes/BlockTest.php

index fe17e10..9047be3 100644 (file)
@@ -220,6 +220,15 @@ changes to languages because of Bugzilla reports.
 * Article::markpatrolled() (deprecated in 1.18)
 * Article::unwatch() (deprecated in 1.18)
 * Article::watch() (deprecated in 1.18)
+* Block::clear() (deprecated in 1.18)
+* Block::decodeExpiry() (deprecated in 1.18)
+* Block::encodeExpiry() (deprecated in 1.18)
+* Block::forUpdate() (deprecated in 1.18)
+* Block::infinity() (deprecated in 1.18)
+* Block::load() (deprecated in 1.18)
+* Block::newFromDB() (deprecated in 1.18)
+* Block::normaliseRange() (deprecated in 1.18)
+* Block::parseExpiryInput() (deprecated in 1.18)
 * CategoryViewer::addSubcategory() (deprecated in 1.17)
 * EditPage::spamPage() (deprecated since 1.17)
 * Exif::getFormattedData() (deprecated in 1.18)
index 8673a3d..3c22f9b 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
@@ -859,17 +806,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 +882,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 +894,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 +910,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:
index f0049fe..480d7d2 100644 (file)
@@ -91,28 +91,6 @@ class BlockTest extends MediaWikiLangTestCase {
                $this->assertLessThan( 2, $delta, "If no timestamp is specified, the block is recorded as time()" );
        }
 
-       /**
-        * This is the method previously used to load block info in CheckUser etc
-        * passing an empty value (empty string, null, etc) as the ip parameter bypasses IP lookup checks.
-        *
-        * This stopped working with r84475 and friends: regression being fixed for bug 29116.
-        *
-        * @dataProvider provideBug29116Data
-        * @covers Block::load
-        */
-       public function testBug29116LoadWithEmptyIp( $vagueTarget ) {
-               $this->hideDeprecated( 'Block::load' );
-
-               $uid = User::idFromName( 'UTBlockee' );
-               $this->assertTrue( ( $uid > 0 ), 'Must be able to look up the target user during tests' );
-
-               $block = new Block();
-               $ok = $block->load( $vagueTarget, $uid );
-               $this->assertTrue( $ok, "Block->load() with empty IP and user ID '$uid' should return a block" );
-
-               $this->assertTrue( $this->block->equals( $block ), "Block->load() returns the same block as the one that was made when given empty ip param " . var_export( $vagueTarget, true ) );
-       }
-
        /**
         * CheckUser since being changed to use Block::newFromTarget started failing
         * because the new function didn't accept empty strings like Block::load()