Bug 19531

Summary: When editing patrons without circulation permission redirect should be to the patron's detail page
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: PatronsAssignee: Bugs List <koha-bugs>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: gmcharlt, kyle.m.hall, nick
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 19531: When saving patron record do not redirect to circ if not authorised
Bug 19531: When saving patron record do not redirect to circ if not authorised
Bug 19531: When saving patron record do not redirect to circ if not authorised

Description Katrin Fischer 2017-10-25 20:07:38 UTC
This was just reported for 16.11.10, but was able to confirm in master:

1) create a staff account with catalog and borrowers permission
2) create a new user with this patron account
3) try to edit the patron account - you get a log in screen (Error: You do not have permission to access this page.)
Comment 1 Jonathan Druart 2017-10-25 20:31:18 UTC
Created attachment 68557 [details] [review]
Bug 19531: When saving patron record do not redirect to circ if not authorised

If the logged in patron does not have the necessary permission we should
not redirect to circulation.pl but moremember.pl instead

Test plan:
With the borrowers permission, you should be able to edit a patron and
be redirect to the moremember page
Comment 2 Katrin Fischer 2017-10-25 20:50:10 UTC
Created attachment 68568 [details] [review]
Bug 19531: When saving patron record do not redirect to circ if not authorised

If the logged in patron does not have the necessary permission we should
not redirect to circulation.pl but moremember.pl instead

Test plan:
With the borrowers permission, you should be able to edit a patron and
be redirect to the moremember page

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 3 Nick Clemens 2017-10-26 16:05:06 UTC
Created attachment 68631 [details] [review]
Bug 19531: When saving patron record do not redirect to circ if not authorised

If the logged in patron does not have the necessary permission we should
not redirect to circulation.pl but moremember.pl instead

Test plan:
With the borrowers permission, you should be able to edit a patron and
be redirect to the moremember page

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 4 Jonathan Druart 2017-10-27 17:18:12 UTC
Pushed to master for 17.11, thanks to everybody involved!