Page MenuHomePhabricator

grabLogs.php: default log_user to 0, and add option to import existing user ids only if specifically desired
Open, Needs TriagePublic

Description

Special:Log etc expect there to be an actual user attached to any non-zero id.

Unless we're actually maintaining all the users and importing/creating them somehow at the same time as we run this (such as by using populateUserTable.php), we need to set the id to zero here or it will break stuff.

Event Timeline

Isarra created this task.Apr 2 2019, 3:21 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 2 2019, 3:21 PM

grabLogs (and the grabbers suite in general) expects an initial empty database and to mirror everything using the same IDs on both wikis. I'm not against making this configurable, but it should be the opposite: only import them with user id = 0 if explicitly requested.