Bug 16029

Summary: Do not show patron toolbar when showing the "patron does not exist" message
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Aleisha Amohia <aleisha>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: aleisha, brendan, f.demians, jonathan.druart, julian.maurice, veron
Version: Main   
Hardware: All   
OS: All   
URL: /cgi-bin/koha/members/moremember.pl?borrowernumber=X
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16082
Change sponsored?: Sponsored Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 16029: Hide patron toolbar if patron does not exist
Bug 16029: Hide patron toolbar if patron does not exist
Bug 16029: Hide patron toolbar if patron does not exist

Description Owen Leonard 2016-03-09 16:08:45 UTC
If you happen to try to view the details of a patron record which no longer exists you will see a "This patron does not exist." message. This page should not also show the patron toolbar (Edit, Change password, etc.) because none of those functions will work.
Comment 1 Aleisha Amohia 2016-03-13 23:43:22 UTC Comment hidden (obsolete)
Comment 2 Marc Véron 2016-03-15 16:20:20 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2016-03-16 09:38:33 UTC
Created attachment 49204 [details] [review]
Bug 16029: Hide patron toolbar if patron does not exist

To test:
1) Create a patron, take note of the borrower number
2) Delete the patron
3) Navigate to the page of the patron you just deleted by typing the url (ie /cgi-bin/koha/members/moremember.pl?borrowernumber=X)
4) Confirm that the patron toolbar is not showing on the page
5) The message now has a link that says 'Find another patron?'. Click this link and confirm you are taken to the member home pgae.

Sponsored-by: Catalyst IT

Followed test plan, works as expected.
Signed-off-by: Marc Véron <veron@veron.ch>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 4 Jonathan Druart 2016-03-16 10:07:00 UTC
See similar patch for circulation.pl on bug 16082
Comment 5 Brendan Gallagher 2016-03-22 00:57:27 UTC
Pushed to Master - Should be in the May 2016 release.  Thanks!
Comment 6 Julian Maurice 2016-04-08 05:52:09 UTC
Patch pushed to 3.22.x, will be in 3.22.6
Comment 7 Frédéric Demians 2016-04-27 15:34:22 UTC
Pushed to 3.22.x, will be in 3.20.11.