Page MenuHomePhabricator

User::edits() and User::incEditCount() should be refactored
Closed, ResolvedPublic

Description

User::edits() let you fetch a cached number of edit from a slave database. Whenever the field is not yet, we initialize the stats by hitting revision table and saving he user editcount.

User::incEditCounts() update the user counter but also does a a lazy initialization albeit from the slave database and is inaccurate (though we probably do not care that much).

Both methods uses the same $dbw->update() statement.

I think there is some potential there to merge both methods in a new clean one :-]


Version: unspecified
Severity: normal

Details

Reference
bz41171