User::newFromId should not assume ID exists
Open, LowPublic
Actions

Assigned To

None

Authored By

	Krinkle
	Sep 18 2012, 11:38 PM

Description

In other words:

"User:newFromId should not set ItemLoaded['id'] = true"

$ php maintenance/eval.php:

< $u = User:newFromId( 1000000000000000000 ); // Exa-, 10^18
< $u->getId();
1000000000000000000;

var_dump( $u->isLoggedIn() );
true
$u->isItemLoaded( 'id' );
true

Bumped into this while fixing T42340 (bug 40340).

Details

Reference: bz40342

Related Objects

Mentioned Here: T42340: migrateUserGroup.php should invalidate object cache and update user_touched for affected users

Event Timeline

• bzimport raised the priority of this task from to Low.Nov 22 2014, 12:59 AM

• bzimport added projects: Future-Release, MediaWiki-General.

• bzimport set Reference to bz40342.

Krinkle created this task.Sep 18 2012, 11:38 PM

If you're going to set a milestone, it would be reasonable to set a priority.

aravikn wrote:

User.php, Line 278.

$this->mLoadedItems = true;
// Set it now to avoid infinite recursion in accessors

Isn't this the thing that was asked to be changed? But would infinite recursion occur as specified?

Krinkle raised the priority of this task from Low to Medium.Nov 24 2014, 3:22 PM

Krinkle removed a project: Future-Release.

Krinkle set Security to None.

Krinkle added a subscriber: Reedy.

Krinkle added a project: Technical-Debt.Dec 10 2014, 2:02 PM

Krinkle removed Krinkle as the assignee of this task.Dec 20 2014, 11:17 PM

Krinkle lowered the priority of this task from Medium to Low.

Krinkle edited projects, added MediaWiki-User-management; removed MediaWiki-General.May 8 2017, 12:27 AM

Krinkle moved this task from Backlog to Other on the MediaWiki-User-management board.

Krinkle updated the task description. (Show Details)

User::newFromId should not assume ID existsOpen, LowPublicActions

Description

Details

Related Objects

Event Timeline

User::newFromId should not assume ID exists
Open, LowPublic
Actions