From: Jane Sandberg Date: Wed, 3 Oct 2018 16:04:54 +0000 (-0700) Subject: LP1795469: Opac holdings sort now considers CN suffix X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=2c09f8298861b32edac2ac570883a109582b9ce9;p=evergreen%2Fjoelewis.git 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 --- 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'}