* (bug 33101) Viewing a User or User talk of username resembling IP ending with .xxx causes Internal error

Patch by Brad Jorsch

Also simplify double return null in method
This commit is contained in:
Sam Reed 2011-12-15 18:57:53 +00:00
parent 451af61e33
commit 85ee2d2db2
2 changed files with 6 additions and 7 deletions

View file

@ -183,6 +183,8 @@ production.
* (bug 32960) Remove EmailAuthenticationTimestamp from database when a
email address is removed.
* (bug 32414) Empty page get a empty bytes attribute in Export/Dump.
* (bug 33101) Viewing a User or User talk of username resembling IP ending
with .xxx causes Internal error
=== API changes in 1.19 ===
* (bug 19838) siprop=interwikimap can now use the interwiki cache.

View file

@ -430,7 +430,7 @@ class Block {
$row = $this->getDatabaseArray();
$row['ipb_id'] = $dbw->nextSequenceValue("ipblocks_ipb_id_seq");
$dbw->insert(
'ipblocks',
$row,
@ -969,7 +969,7 @@ class Block {
*/
public static function formatExpiry( $encoded_expiry ) {
wfDeprecated( __METHOD__, '1.18' );
global $wgContLang;
static $msg = null;
@ -1039,7 +1039,7 @@ class Block {
# passed by some callers (bug 29116)
return null;
} elseif( in_array( $type, array( Block::TYPE_USER, Block::TYPE_IP, Block::TYPE_RANGE, null ) ) ) {
} elseif( in_array( $type, array( Block::TYPE_USER, Block::TYPE_IP, Block::TYPE_RANGE ) ) ) {
$block = new Block();
$block->fromMaster( $fromMaster );
@ -1049,12 +1049,9 @@ class Block {
if( $block->newLoad( $vagueTarget ) ){
return $block;
} else {
return null;
}
} else {
return null;
}
return null;
}
/**