From d21d1dac8f0212764de5a2c3e352ceebc2d63ef5 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 630d252116..ab674effb0 100644 --- a/Open-ILS/src/perlmods/lib/OpenILS/Application/AppUtils.pm +++ b/Open-ILS/src/perlmods/lib/OpenILS/Application/AppUtils.pm @@ -2045,7 +2045,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