Bug 13335

Summary: Holds and priority display via OPACShowHoldQueueDetails confusing
Product: Koha Reporter: Kyle M Hall <kyle>
Component: OPACAssignee: Kyle M Hall <kyle>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: chris, tomascohen
Version: unspecified   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 13335 - Holds and priority display via OPACShowHoldQueueDetails confusing
Bug 13335 - Holds and priority display via OPACShowHoldQueueDetails confusing
Bug 13335 - Holds and priority display via OPACShowHoldQueueDetails confusing
[SIGNED OFF]Bug 13335 - Holds and priority display via OPACShowHoldQueueDetails confusing
Bug 13335 - Holds and priority display via OPACShowHoldQueueDetails confusing

Description Kyle M Hall 2014-11-25 17:14:39 UTC
No matter what the selection of OPACShowHoldQueueDetails is, if it is enabled it displays a line "Holds and priority:" even if you've opted to hide one of those!
Comment 1 Kyle M Hall 2014-11-25 17:15:24 UTC Comment hidden (obsolete)
Comment 2 Kyle M Hall 2014-11-25 17:19:20 UTC Comment hidden (obsolete)
Comment 3 Christopher Brannon 2014-11-26 19:38:11 UTC Comment hidden (obsolete)
Comment 4 Christopher Brannon 2014-11-26 19:39:28 UTC Comment hidden (obsolete)
Comment 5 Jonathan Druart 2014-12-19 11:03:13 UTC
Created attachment 34527 [details] [review]
Bug 13335 - Holds and priority display via OPACShowHoldQueueDetails confusing

No matter what the selection of OPACShowHoldQueueDetails is, if it is
enabled it displays a line "Holds and priority:" even if you've opted to
hide one of those!

Test Plan:
1) Apply this patch
2) Test each setting of OPACShowHoldQueueDetails
3) Ensure each setting displays the correct fields ( or lack thereof )

Signed-off-by: Christopher Brannon <cbrannon@debian.localdomain>

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 6 Tomás Cohen Arazi 2015-02-12 18:31:18 UTC
Patch pushed to master.

Thanks Kyle!
Comment 7 Chris Cormack 2015-02-14 08:12:18 UTC
Pushes to 3.18.x will be in 3.18.4