Bug 5029 - Update patron deletion error page
Summary: Update patron deletion error page
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: PATCH-Sent (DO NOT USE) normal (vote)
Assignee: Owen Leonard
QA Contact: Bugs List
URL: /cgi-bin/koha/members/deletemem.pl?me...
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-16 15:27 UTC by Owen Leonard
Modified: 2012-10-25 22:53 UTC (History)
1 user (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Proposed fix for HEAD (3.10 KB, patch)
2010-08-03 16:54 UTC, Owen Leonard
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Owen Leonard 2010-07-16 15:27:30 UTC
If you try to delete a patron and they have checkouts or fines, a page appears telling you so, deletemem.pl. The template for this page should style the message like other warning dialogs and the page should offer a link back to the patron record in question.
Comment 1 Owen Leonard 2010-08-03 16:54:27 UTC
Created attachment 2447 [details] [review]
Proposed fix for HEAD

- Styling error message with "dialog alert" class
- Adding sidebar patron details to match other patron pages

I didn't add a link back to anywhere from the dialog because a deletion could have been initiated from any page with the patron toolbar. Adding the patron details in the sidebar should improve recognition on the part of the user that this is like any other patron information page and all patron-related links are available in the sidebar.
Comment 2 Chris Cormack 2010-12-01 20:21:26 UTC
Patch pushed, please test
Comment 3 MJ Ray (software.coop) 2010-12-14 10:22:51 UTC
This bug is mentioned in:
Fix for Bug 5029 - Update	patron deletion error page http://lists.koha-community.org/pipermail/koha-patches/2010-November/012988.html