Bugzilla – Attachment 59138 Details for
Bug 17886
Don't show option to add to existing list if there are no lists in staff
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 17886 Don't show option to add to existing list if there are no
Bug-17886-Dont-show-option-to-add-to-existing-list.patch (text/plain), 4.94 KB, created by
Caitlin Goodger
on 2017-01-18 03:27:36 UTC
(
hide
)
Description:
Bug 17886 Don't show option to add to existing list if there are no
Filename:
MIME Type:
Creator:
Caitlin Goodger
Created:
2017-01-18 03:27:36 UTC
Size:
4.94 KB
patch
obsolete
>From 25036ed61d0d2fb25146bcb8a26d01708c3ae51e Mon Sep 17 00:00:00 2001 >From: Caitlin Goodger <caitlingoodger.student@wegc.school.nz> >Date: Wed, 18 Jan 2017 03:23:02 +0000 >Subject: [PATCH] Bug 17886 Don't show option to add to existing list if there > are no > >lists > >Test Plan >1: Make sure that you don't have any lists and then add a Biblical >Record to a list >2: You should see that you can add it to an existing list even though >you don't have one. >3: Apply patch >4: Add a Biblical record and if you don't have a list then you shouldn't >see the add to existing list. >5: Create a list and then add a Bibical Record and you should see the >add to existing list >--- > .../en/modules/virtualshelves/addbybiblionumber.tt | 77 +++++++++++----------- > 1 file changed, 40 insertions(+), 37 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/virtualshelves/addbybiblionumber.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/virtualshelves/addbybiblionumber.tt >index 63974a9..6d14c5f 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/virtualshelves/addbybiblionumber.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/virtualshelves/addbybiblionumber.tt >@@ -37,45 +37,48 @@ > [% UNLESS ( shelfnumber ) %] > > [% UNLESS ( newshelf ) %] >+[% IF ( private_shelves.count) %] > <fieldset class="rows"> >- <legend>Select an existing list</legend> >- <ol> >- <li> >- <label>Choose list</label> >- <select name="shelfnumber" id="shelfnumber"> >- [% IF private_shelves.count %] >- <optgroup label="Private lists"> >- [% FOREACH private_shelf IN private_shelves %] >- <option value="[% private_shelf.shelfnumber %]">[% private_shelf.shelfname | html%]</option> >- [% END %] >- </optgroup> >- [% END %] >- [% IF private_shelves_shared_with_me.count %] >- <optgroup label="Private lists shared with me"> >- [% FOREACH private_shelf IN private_shelves_shared_with_me %] >- <option value="[% private_shelf.shelfnumber %]">[% private_shelf.shelfname | html%]</option> >- [% END %] >- </optgroup> >- [% END %] >- [% IF public_shelves.count %] >- <optgroup label="Public lists"> >- [% FOREACH public_shelf IN public_shelves %] >- <option value="[% public_shelf.shelfnumber %]">[% public_shelf.shelfname | html%]</option> >- [% END %] >- </optgroup> >- [% END %] >- </select> >- </li> >- </ol> >- [% FOREACH biblio IN biblios %]<input type="hidden" name="biblionumber" value="[% biblio.biblionumber %]" />[% END %] >- [% UNLESS ( biblionumbers ) %] >- <input type="hidden" name="biblionumber" value="[% biblionumber %]" /> >- [% ELSE %] >- <input type="hidden" name="biblionumbers" value="[% biblionumbers %]" /> >- [% END %] >- </fieldset> >+ <legend>Select an existing list</legend> >+ <ol> >+ <li> >+ <label>Choose list</label> >+ <select name="shelfnumber" id="shelfnumber"> >+ [% IF private_shelves.count %] >+ <optgroup label="Private lists"> >+ [% FOREACH private_shelf IN private_shelves %] >+ <option value="[% private_shelf.shelfnumber %]">[% private_shelf.shelfname | html%]</option> >+ [% END %] >+ </optgroup> >+ [% END %] >+ [% IF private_shelves_shared_with_me.count %] >+ <optgroup label="Private lists shared with me"> >+ [% FOREACH private_shelf IN private_shelves_shared_with_me %] >+ <option value="[% private_shelf.shelfnumber %]">[% private_shelf.shelfname | html%]</option> >+ [% END %] >+ </optgroup> >+ [% END %] >+ [% IF public_shelves.count %] >+ <optgroup label="Public lists"> >+ [% FOREACH public_shelf IN public_shelves %] >+ <option value="[% public_shelf.shelfnumber %]">[% public_shelf.shelfname | html%]</option> >+ [% END %] >+ </optgroup> >+ [% END %] >+ </select> >+ </li> >+ </ol> >+ >+ [% FOREACH biblio IN biblios %]<input type="hidden" name="biblionumber" value="[% biblio.biblionumber %]" />[% END %] >+ [% UNLESS ( biblionumbers ) %] >+ <input type="hidden" name="biblionumber" value="[% biblionumber %]" /> >+ [% ELSE %] >+ <input type="hidden" name="biblionumbers" value="[% biblionumbers %]" /> >+ [% END %] >+ </fieldset> > >- <p>... or...</p> >+ <p>... or...</p> >+ [% END %] > [% END %] > > <fieldset class="rows"><legend>Add to a new list:</legend> >-- >2.1.4
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 17886
:
59138
|
59142
|
59282
|
59381
|
61117
|
75640
|
76306
|
76428