https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34460 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david@davidnind.com --- Comment #6 from David Nind <david@davidnind.com> --- Testing notes (using KTD): 1. Apply the patch. 2. Update the database: updatedatabase 3. Restart everything: restart_all Using Henry - only changing the patron category "Set permissions" setting: 1. Change user name and password so can log in as them 2. Set permissions: - Staff access, allows viewing of catalogue in staff interface (catalogue) - Manage Koha system settings (Administration panel) (parameters) 3. Change Henry from "Staff" to "Patron" category 4. "Set permissions" is greyed out 5. Can still log in to the staff interface 6. Change the "Patron" category so that "Set permissions" is set to "Yes" 7 "Set permissions" no longer greyed out (after refreshing Henry's details page) Using Henry - using the new ClearPermissionsAutomatically system preference: 1. Set ClearPermissionsAutomatically = "Enable" 2. Change "Set permissions" for the "Patron" category to "No" 3. "Set permissions" is greyed out for Henry 4. Henry can still log in to the staff interface 5. Change the "Patron" category so that it doesn't have permissions 6. "Set permissions" for Henry is greyed out 7. Change Henry's category to the "Staff" category 8. Permissions from previously remain the same and can still log in to the staff interface 9. Change Henry's category to the "Patron" category 10. Refresh Henry's details page - "Set permissions" greyed out 11. Henry can no longer log in to the staff interface 12. Change Henry's category back to the "Staff" category. 13. Henry can't log in to the staff interface 14. Refresh Henry's details page - "Set permissions" available 15. Click "Set permissions" - no permissions are set, so permissions were cleared Notes: 1. New field added to patron categories: Can have permissions: [Yes|No] (With a hint: Determine if patrons in this category can have permissions assigned.) 2. Set to "No" by default for new patron categories 3. If a patron's category is changed from one with permissions to one with no permissions, the "Set permissions" setting for the patron's account is greyed out 4. Their permissions are not cleared, and they can still do what they did before (log in to the staff interface, and so on) 5. If the new ClearPermissionsAutomatically system preference is set to "Enable", then changing the patron's category from one with permissions to no permissions, clears any existing permissions they may have Observation 1 ============= Wording for the new ClearPermissionsAutomatically system preference: [Enable/Don't enable] the ability for staff permissions to be automatically cleared when a staff member's category is changed to one that does not have permissions. Suggested changes: [Enable/Don't enable] automatically removing permissions for patrons when their patron category is changed, and that patron category has the "Can have permissions" setting set to "No". Observation 2 ============= When the "Can have permissions" setting is changed for a patron category, the success alert at the top of the page is "Patron category updated successfully." If the ClearPermissionsAutomatically system preference is enabled and the setting for "Can have permissions" is changed from "Yes" to "No", should there be an additional message along the lines: Any patrons that are changed to this category from a patron category that has permissions, will have their permissions removed. Observation 3 ============= A bug currently waiting for QA has some settings and improvements to messages when changing a patron's category, so that there aren't "silent" changes: Bug 42112 - Add option when changing a patron's category: allow choosing what to do for the expiry date and messaging preferences Should this change be added as a pop-up message there as well when ClearPermissionsAutomatically is enabled? -- You are receiving this mail because: You are watching all bug changes.