From 44cf04e1b9bc2c782a7193170a117c6ad621bc77 Mon Sep 17 00:00:00 2001 From: Jane Sandberg Date: Wed, 3 Oct 2018 09:04:54 -0700 Subject: [PATCH] LP1795469: Opac holdings sort now considers CN suffix To test: 1) Create a bunch of Call Number suffixes in Administration -> Server Administration -> Call Number Suffixes. 2) Go to a bib record, and add a bunch of holdings, all with the same call number label, owning/circ libraries, item numbers, and parts (if using parts) but with different barcodes and CN suffixes. 3) Look at the OPAC view of these holdings. Note that they are in order by barcode, without any consideration for the CN suffix. 4) Apply this commit. 5) Look at the OPAC view again. Note that they are now sorted by CN suffix, and then by barcode. Signed-off-by: Jane Sandberg Signed-off-by: Josh Stompro Signed-off-by: Galen Charlton --- Open-ILS/src/perlmods/lib/OpenILS/Application/AppUtils.pm | 1 + 1 file changed, 1 insertion(+) diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Application/AppUtils.pm b/Open-ILS/src/perlmods/lib/OpenILS/Application/AppUtils.pm index f44e6ef049..9190ad81b8 100644 --- a/Open-ILS/src/perlmods/lib/OpenILS/Application/AppUtils.pm +++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/AppUtils.pm @@ -2153,6 +2153,7 @@ sub basic_opac_copy_query { order_by => [ {class => 'aou', field => 'name'}, {class => 'acn', field => 'label_sortkey'}, + {class => 'acns', field => 'label_sortkey'}, {class => 'bmp', field => 'label_sortkey'}, {class => 'acp', field => 'copy_number'}, {class => 'acp', field => 'barcode'} -- 2.11.0