Bug 19621

Summary: Routing lists tab not present when viewing 'Holds history' tab for a patron
Product: Koha Reporter: Nick Clemens <nick>
Component: PatronsAssignee: Nick Clemens <nick>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: fridolin.somers, gmcharlt, katrin.fischer, kyle.m.hall, m.de.rooy, pierre.genty
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 19621 - Use preference check on template side to display/not display Routing lists tab
Bug 19621 - Use preference check on template side to display/not display Routing lists tab
Bug 19621 - Use preference check on template side to display/not display Routing lists tab

Description Nick Clemens 2017-11-13 17:18:35 UTC
To recreate:
1 - Enable RoutingSerials syspref
2 - Open a patron record
3 - Click through the tabs on the left side of the screen
4 - Note that 'Routing lists' is visible on all screens except 'Holds history'
Comment 1 Nick Clemens 2017-11-13 17:21:00 UTC
Created attachment 69132 [details] [review]
Bug 19621 - Use preference check on template side to display/not display Routing lists tab

Currently we pass a param to control display - the param is a just a
check of syspref so can be done directly form template

To test:
1 - Enable RoutingSerials syspref
2 - Open a patron record
3 - Click through the tabs on the left side of the screen
4 - Note that 'Routing lists' is visible on all screens except 'Holds
history'
5 - Apply patch
6 - 'Routing lists' should now be visible in all tabs.
Comment 2 Dilan Johnpullé 2017-11-16 00:06:44 UTC
Created attachment 69171 [details] [review]
Bug 19621 - Use preference check on template side to display/not display Routing lists tab

Patch applies without issue and functions as described.

Signed-off-by: Dilan Johnpullé <dilan@calyx.net.au>
Comment 3 Marcel de Rooy 2017-11-27 13:10:51 UTC
Seems an enh to me
Comment 4 Nick Clemens 2017-11-27 18:34:34 UTC
I's argue for bug - it appears on all other tabs and disappears on one, not adding a new page, just fixing to how it 'should' work

It does hit a few pages, so fine either way
Comment 5 Katrin Fischer 2017-12-22 00:08:47 UTC
I agree with Nick - inconsistent behaviour, you don't expect for menu items to go missing when changing the tab.
Comment 6 Katrin Fischer 2017-12-22 00:10:23 UTC
Created attachment 70091 [details] [review]
Bug 19621 - Use preference check on template side to display/not display Routing lists tab

Patch applies without issue and functions as described.

Signed-off-by: Dilan Johnpullé <dilan@calyx.net.au>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 7 Jonathan Druart 2017-12-22 16:16:55 UTC
Pushed to master for 18.05, thanks to everybody involved!
Comment 8 Nick Clemens 2018-01-15 16:45:02 UTC
Awesome work all, pushed to stable for 17.11.02
Comment 9 Fridolin Somers 2018-01-19 12:44:08 UTC
Pushed to 17.05.x for v17.05.08