Bug 7783

Summary: Show private shelves in opac when deleting non-empty private list from shelf view
Product: Koha Reporter: Marcel de Rooy <m.de.rooy>
Component: ListsAssignee: Marcel de Rooy <m.de.rooy>
Status: CLOSED FIXED QA Contact: Ian Walls <koha.sekjal>
Severity: enhancement    
Priority: P5 - low CC: mathilde.formery, paul.poulain
Version: 3.8   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 7310    
Bug Blocks:    
Attachments: Patch

Description Marcel de Rooy 2012-03-21 15:41:22 UTC
This patch builds on top of 7310, but the problem is in master already.
If you delete a non-empty list from shelf view (looking at the biblio records of the list), the confirm message is shown but listing public lists instead of private lists.

Report 7310 resolved this for staff. This patch adds it for OPAC.
Comment 1 Marcel de Rooy 2012-03-21 15:46:39 UTC
Created attachment 8440 [details] [review]

One-line-fix: easy to test (after 7310 is through)
Comment 2 MathildeF 2012-03-23 10:00:34 UTC
Tested on sandbox.
it works correct.
Comment 3 Ian Walls 2012-03-25 20:20:56 UTC
Template only change, adding a new template parameter to the submit to opac-shelves.pl if showprivateshelves is set.  Marking Passed QA.
Comment 4 Paul Poulain 2012-03-26 14:00:13 UTC
tiny comment: it would have been more clear to write the patch on 2 lines:
+ [% IF ( showprivateshelves ) %]<input type="hidden" name="display" value="privateshelves"/>[% END %]
alone, but will push the patch as it is ;-)