From bc0a3bed1fcca684d96a925547b962d597cdc692 Mon Sep 17 00:00:00 2001 From: Kyle Tomita Date: Wed, 5 Feb 2014 16:23:18 -0800 Subject: [PATCH] LP1155769: Items repeated in tpac display, others not visible This was an issue with the query and ordering by a column that was sometimes null (copy_number). Added barcode to be a sort rather than replacing copy_number, per bshum's suggestion. Not replacing allows libraries that use copy_number to continue to use them as well. Signed-off-by: Kyle Tomita Signed-off-by: Ben Shum --- Open-ILS/src/perlmods/lib/OpenILS/Application/AppUtils.pm | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Application/AppUtils.pm b/Open-ILS/src/perlmods/lib/OpenILS/Application/AppUtils.pm index 5a6d38582d..7dd658b639 100644 --- a/Open-ILS/src/perlmods/lib/OpenILS/Application/AppUtils.pm +++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/AppUtils.pm @@ -2038,7 +2038,8 @@ sub basic_opac_copy_query { order_by => [ {class => 'aou', field => 'name'}, {class => 'acn', field => 'label_sortkey'}, - {class => 'acp', field => 'copy_number'} + {class => 'acp', field => 'copy_number'}, + {class => 'acp', field => 'barcode'} ], limit => $copy_limit, -- 2.11.0