Comment 1 for bug 2043218

Revision history for this message
Terran McCanna (tmccanna) wrote :

I've made a stab at this. If I've done it right, the upgrade script should first remove the setting from any of the tables where it could have been referenced (according to the database schema), and then remove the setting itself.

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/mccanna/lp2043218_remove_perm_update_org_setting

To test on a new install:
1) Verify that the setting does not appear under Admin > Server Admin > Permissions

To test on an existing install:
1) Apply the setting to a user permission group and to an individual user (or identify ones where it is already applied)
2) Run the upgrade script
3) Verify that the permission has been removed from the permission group, from the user, and from the list of possible permissions

(you could also try manually creating a new permission with the same name and then running the upgrade script to be sure it deletes it)

Did I miss anything? Feedback desired!