Page MenuHomePhabricator

list=logevents leuser= and list=usercontribs ucuser= parameters inconsistent
Closed, ResolvedPublic




Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 10:26 PM
bzimport set Reference to bz16421.

(In reply to comment #0)

First: leuser does not allow underscores in place of spaces, but ucuser does,
can they be made consistent? Preferred is to have _ always an alias for %20.

Fixed in r43864.

Second: leuser errors out if the user doesn't exist, but ucuser simply returns
an empty contributions list. I am not sure which is ideal.

That's because the revision table (which usercontribs uses) stores the user ID and the user name (0 and the IP respectively for anons), while the logging table (which logevents uses) only stores the user ID. That means anons can't have log entries (bug 3732) and that you can't search the logging table for users that don't exist, because you need to look up their user ID first.