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-Tag: sprint4-merge-nov22~2369 X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=d21d1dac8f0212764de5a2c3e352ceebc2d63ef5;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 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,