Page MenuHomePhabricator

patch-drop-user-fields.sql should not be applied until after cleanupUsersWithNoId.php
Open, Needs TriagePublic


As per title. The patch drops fields that cleanupUsersWithNoId needs.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 10 2020, 12:13 AM

One option would be to drop the fields in cleanupUsersWithNoId.php. Another would be to incorporate cleanupUsersWithNoId.php into update.php, but this would mean using a fixed username prefix.

I'm leaning towards dropping in cleanupUsersWithNoId.php.