[Koha-bugs] [Bug 26170] Add protected status for patrons

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26170

David Nind <david at davidnind.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Needs Signoff               |Failed QA

--- Comment #27 from David Nind <david at davidnind.com> ---
Everything works as per the test plan.

However, while the "More > Delete" option for a patron is greyed out, it is not
disabled.

If you click on Delete you get:
- a message saying "Are you sure you want to delete the patron NAME?", then
- if you say Yes, then an error trace:

 [Deleting patron failed, patron is protected] at
/usr/share/perl5/Exception/Class/Base.pm line 88


Some other suggestions (maybe future enhancements):

1. Change "More > Delete" to "More > Delete patron". I'm not sure whether this
makes a difference for translation, but is also consistent with "Renew patron"
option.

2. Maybe add a hint for the protected file? "Prevents deleting a patron."


Testing notes (using KTD):
1. Run the database update after applying the patch: updatedatabase
2. Run dbic: dbic
3. Restart everything: restart_all

-- 
You are receiving this mail because:
You are watching all bug changes.