Koha/cataloguing
Bernardo Gonzalez Kriegel 2d9f5fc69e Bug 9457: [ENH] Ordering branches should be case independent (2)
This patch replace use of CGI::scroll_list() to show list of branches.

In two files, marc21_linking_section.pl and unimarc_field_4XX.pl,
the scrolling list is created but not used in the template file,
so the code is removed.

Also minor renaming/normalizing of variables.

To test:
1) Install with some branches, records and patrons

2.1) Select a record, click 'Place hold', select user,
     there is a library pull-down next to 'Pickup at:',
     list is ordered case sensitive
2.2) Go to Reports > Average loan time,
     next to Library is a pull-down,
     list without order
2.3) Go to Reports > Catalog by item type,
     next to 'Select a library' is a pull-down,
     list is ordered case sensitive
2.4) This is tricky, go to Reports home,
     change last part of URL 'reports-home.pl' with
     'manager.pl?report_name=issues_by_borrower_category'
     (can't find a direct link), next to 'Select a library'
     is a library pull-down,
     list without order
2.5) Edit/Add a patron, on section 'Library management'
     there is a library pull-down, case sensitive
2.6) OPAC, as logged user, make a suggestion or hold,
     there is library pull-down, correct order

3) Apply the patch

4.1) Repeat 2.1), correctly ordered list
4.2) Repeat 2.2), correctly ordered list
4.3) Repeat 2.3), correctly ordered list
4.4) Repeat 2.4), correctly ordered list
4.5) This is a bit more work
     There are 3 possible situations to test:
     A) No branches, must show a message that are no
        libraries defined
     B) New patron, must show a correctly ordered
        list of branches, current branch selected
     C) Edit patron, must show a correctly ordered
        list of branches, patron branch selected
4.6) Small changes on variable names, so retest 2.6)

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
2013-03-20 15:35:58 -04:00
..
value_builder Bug 9457: [ENH] Ordering branches should be case independent (2) 2013-03-20 15:35:58 -04:00
addbiblio.pl Bug 5333 - z3950 normalization should apply only on UNIMARC 2013-02-28 19:11:24 -05:00
addbooks.pl Bug 9239: Allow the use of QueryParser for all queries 2013-03-16 21:32:32 -04:00
additem.pl Bug 7253: fast cataloging not always checking out 2013-02-12 10:00:37 -05:00
linkitem.pl Bug 5528: Analytic records support 2011-10-13 10:03:39 +13:00
merge.pl Bug 8862 follow-up s/\t/4spaces/g 2012-10-12 22:19:25 +02:00
merge_ajax.pl Bug 8860 - can't merge if you don't have fast cataloging permissions 2012-10-05 16:51:32 +02:00
moveitem.pl
plugin_launcher.pl Bug 7356 - Fix various typos and mis-spellings 2012-01-13 11:51:26 +01:00
ysearch.pl Bug 7400: Add auto-completion on auth_finder 2012-03-19 18:20:30 +01:00
z3950_search.pl Bug 9662 - ISSN search field not passed to Z3950Search 2013-02-20 09:55:54 -05:00