From: dbs <dbs@dcc99617-32d9-48b4-a31d-7c20da2025e4>
Date: Thu, 28 Oct 2010 14:55:33 +0000 (+0000)
Subject: Ensure call numbers are returned to the holdings editor in ascending sorted order
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=50d8efd175223c708cb6f060a44ac88a3a722a13;p=evergreen%2Ftadl.git

Ensure call numbers are returned to the holdings editor in ascending sorted order

James Fournie in https://bugs.launchpad.net/evergreen/+bug/635121 reported
that call numbers were being displayed in the order that they had been
added to the database, rather than in sorted call number label order.
Although I have been unable to reproduce this problem on two different
test systems, the patch he provided for 1.6.1 (which fixes the problem on their
test system) shouldn't hurt other systems.

For sortkey-aware branches, I modified James' patch to take advantage of the
asset.call_number.label_sortkey column and the oils_text_as_bytea() function.


git-svn-id: svn://svn.open-ils.org/ILS/trunk@18516 dcc99617-32d9-48b4-a31d-7c20da2025e4
---

diff --git a/Open-ILS/src/perlmods/OpenILS/Application/Cat.pm b/Open-ILS/src/perlmods/OpenILS/Application/Cat.pm
index dad96eed8b..10c9da566f 100644
--- a/Open-ILS/src/perlmods/OpenILS/Application/Cat.pm
+++ b/Open-ILS/src/perlmods/OpenILS/Application/Cat.pm
@@ -660,7 +660,9 @@ sub _build_volume_list {
 	$search_hash->{deleted} = 'f';
 	my $e = new_editor();
 
-	my $vols = $e->search_asset_call_number($search_hash);
+	my $vols = $e->search_asset_call_number([$search_hash, { 'order_by' => {
+        'acn' => 'oils_text_as_bytea(label_sortkey), oils_text_as_bytea(label), id, owning_lib'
+    } } ] );
 
 	my @volumes;