Page MenuHomePhabricator

Recent changes rebuild script doesn't restore rc_bot value
Closed, ResolvedPublic

Description

Author: gunter.schmidt

Description:
The script rebuildrecentchanges.php has a litte error. It does not check, if the user is a bot. Thus all
changes by bots appear in the recent change list.

It should check if the user is a bot and set the bot-flag to 1.


Version: 1.11.x
Severity: normal

Details

Reference
bz9166

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 9:37 PM
bzimport set Reference to bz9166.
bzimport added a subscriber: Unknown Object (MLST).

suisui wrote:

proposed patch for svn r20183

I made a patch for this trunk/ r20183

Attached:

robchur wrote:

Comments for Database::set() indicate it is deprecated. Use Database::update()
instead.

suisui wrote:

Database::update doesnt hang
ug_user=rc_user part cause Database::update uses makeList() to expand
so it is treated as ug_user = 'rc_user'

end_invites wrote:

shouldn't this patch be reviewed?