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 <tomitakyle@gmail.com>
modified:   Open-ILS/src/perlmods/lib/OpenILS/Application/AppUtils.pm

diff --git a/Open-ILS/src/perlmods/lib/OpenILS/Application/AppUtils.pm b/Open-ILS/src/perlmods/lib/OpenILS/Application/AppUtils.pm
index 630d252..ab674ef 100644 (file)
--- 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,