Open Bug 1507812 Opened 2 years ago Updated 2 years ago

merge_user.pl should not continue if the old user id has an account in Phabricator

Categories

(bugzilla.mozilla.org :: General, defect)

Production
defect
Not set
normal

Tracking

()

ASSIGNED

People

(Reporter: dkl, Assigned: dkl)

References

Details

Attachments

(1 file)

46 bytes, text/x-github-pull-request
Details | Review
merge_user.pl should throw an error if the old user id that is going to be removed has an account in Phabricator.

The old account in Phabricator will need to be updated first to match the destication account in BMO. This will probably require the attention of a Phabricator admin and a separate bug should be filed.
Note: Phabricator now supports allowing others than Admins to disable users through the admin UI. Applications → People → Configure has a "Can Disable Users" permission which can be set to a broader set of users than "Administrators". Unfortunately calling the conduit API method user.edit to disable a user still requires full admin rights (legacy issue) which means phab-bot cannot do the task. Which means we cannot yet automate this part inside of merge_users.pl.
Attached file GitHub Pull Request
You need to log in before you can comment on or make changes to this bug.