Bug 18573: (bug 17847 follow-up) Update av-build-dropbox.inc for OPAC
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Wed, 10 May 2017 16:13:09 +0000 (13:13 -0300)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 12 May 2017 12:46:35 +0000 (08:46 -0400)
On bug 17847, av-build-dropbox.inc has been updated for the intranet,
not OPAC

There is only one call to av-build-dropbox.inc at the OPAC, from
opac-suggestion.tt

Test plan:
Create a new suggestion, anonymous or with a logged in user
=> Without this patch you got
  Template process failed: undef error - The method default is not
  covered by tests! at /home/vagrant/kohaclone/C4/Templates.pm line 121.
=> With this patch applied you should see the item type dropdown list
correctly filled

Reproduced without patch, OK with patch
Signed-off-by: Marc VĂ©ron <veron@veron.ch>
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
koha-tmpl/opac-tmpl/bootstrap/en/includes/av-build-dropbox.inc

index 0853685..17c12d0 100644 (file)
@@ -8,7 +8,7 @@
     size: the size to use for the input (generated if the authorised value category does not exist).
 %]
 
-[% SET avs = AuthorisedValues.GetAuthValueDropbox( category, default ) %]
+[% SET avs = AuthorisedValues.GetAuthValueDropbox( category ) %]
 [% DEFAULT
     class = ''
     size = 20
 [% IF avs %]
   <select id="[% name %]" name="[% name %]" class="[% class %]" >
   [% FOR av IN avs %]
-    [% IF av.default %]
-      <option value="[% av.value %]" selected="selected">[% av.label | html_entity %]</option>
+    [% IF av.authorised_value == default %]
+      <option value="[% av.authorised_value %]" selected="selected">[% av.lib | html_entity %]</option>
     [% ELSE %]
-      <option value="[% av.value %]">[% av.label | html_entity %]</option>
+      <option value="[% av.authorised_value %]">[% av.lib | html_entity %]</option>
     [% END %]
   [% END %]
   </select>