From: Kyle Tomita Date: Thu, 6 Feb 2014 00:23:18 +0000 (-0800) Subject: LP1155769: Items repeated in tpac display, others not visible X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=bc0a3bed1fcca684d96a925547b962d597cdc692;p=working%2FEvergreen.git 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 --- 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,