From 50d8efd175223c708cb6f060a44ac88a3a722a13 Mon Sep 17 00:00:00 2001 From: dbs Date: Thu, 28 Oct 2010 14:55:33 +0000 Subject: [PATCH] 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 --- 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 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; -- 2.11.0