* (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:
parent
451af61e33
commit
85ee2d2db2
2 changed files with 6 additions and 7 deletions
|
|
@ -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.
|
||||
|
|
|
|||
|
|
@ -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;
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
|
|||
Loading…
Reference in a new issue