From 4759037b1de3be1fce398a6eaefd16230b50ff2a Mon Sep 17 00:00:00 2001 From: dbs Date: Thu, 28 Oct 2010 15:10:53 +0000 Subject: [PATCH] Backport r18516: 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/branches/rel_2_0@18518 dcc99617-32d9-48b4-a31d-7c20da2025e4 --- Open-ILS/src/perlmods/OpenILS/Application/Cat.pm | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/Open-ILS/src/perlmods/OpenILS/Application/Cat.pm b/Open-ILS/src/perlmods/OpenILS/Application/Cat.pm index 6b39ece8c2..4d688124ba 100644 --- a/Open-ILS/src/perlmods/OpenILS/Application/Cat.pm +++ b/Open-ILS/src/perlmods/OpenILS/Application/Cat.pm @@ -550,7 +550,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; -- 2.11.0