Bug 12944 [Patron lists follow-up] There is no way to search orders by creator
authorOwen Leonard <oleonard@myacpl.org>
Wed, 29 Oct 2014 14:53:07 +0000 (10:53 -0400)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Mon, 9 Feb 2015 18:48:13 +0000 (15:48 -0300)
This bug modifies not just the order search form but also the patron
lists feature, making use of the new script for building a patron
autocomplete form. However, the test plan does not specify that the
patron lists form be tested as well, and it doesn't work.

This patch corrects a minor flaw which broke the patron search
autocomplete when adding patrons to a patron list.

To test, create a new patron list if necessary. Click the "add patrons"
button to add patrons to the list. In the patron search form type a
partial patron name. You should get an autocomplete dropdown of existing
patrons. Selecting one should work correctly.

Signed-off-by: Paola Rossi <paola.rossi@cineca.it>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
koha-tmpl/intranet-tmpl/prog/en/modules/acqui/histsearch.tt
koha-tmpl/intranet-tmpl/prog/en/modules/patron_lists/list.tt

index 75bc842..76ebc8e 100644 (file)
@@ -6,7 +6,7 @@
 [% INCLUDE 'calendar.inc' %]
 [% INCLUDE 'datatables.inc' %]
 
-<script type="text/javascript" src="[% interface %]/js/autocomplete/patrons.pl"></script>
+<script type="text/javascript" src="[% interface %]/js/autocomplete/patrons.js"></script>
 <script type="text/javascript">
 //<![CDATA[
  $(document).ready(function() {
index 1090dbf..a93798f 100644 (file)
@@ -25,7 +25,7 @@ $(document).ready(function() {
     patron_autocomplete({
         patron_container: $("#patrons_to_add"),
         input_autocomplete: $("#find_patron"),
-        patron_input_name = 'patrons_to_add'
+        patron_input_name: 'patrons_to_add'
     });
 
     var checkBoxes = $("input[type='checkbox']","#patron-list-table");