From 2c09f8298861b32edac2ac570883a109582b9ce9 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.43.2