[Koha-patches] [PATCH] Bug 4307: Sorts library locations in inventory drop-down list.

[Garry Collum]

Replacement patch for previous patch submitted.
---
 .../prog/en/modules/tools/inventory.tmpl           |    1 +
 tools/inventory.pl                                 |    2 +-
 2 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tmpl b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tmpl
index d521761..0b35000 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tmpl
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tmpl
@@ -77,6 +77,7 @@ $(document).ready(function(){
         <legend>Select items you want to check</legend>
         <ol><li>
         <label for="branchloop">Library</label><select id="branchloop" name="branchcode" style="width:12em;">
+            <option value="">All Locations</option>
         <!-- TMPL_LOOP NAME="branchloop" -->
             <!-- TMPL_IF name="selected" -->
                 <option value="<!-- TMPL_VAR NAME="value" -->" selected="selected"><!-- TMPL_VAR NAME="branchname" --></option>
diff --git a/tools/inventory.pl b/tools/inventory.pl
index 56f0bcb..c87a7e9 100755
--- a/tools/inventory.pl
+++ b/tools/inventory.pl
@@ -65,13 +65,13 @@ my ($template, $borrowernumber, $cookie)
 
 my $branches = GetBranches();
 my @branch_loop;
-push @branch_loop, {value => "", branchname => "All Locations", };
 for my $branch_hash (keys %$branches) {
 	push @branch_loop, {value => "$branch_hash",
 	                   branchname => $branches->{$branch_hash}->{'branchname'}, 
 	                   selected => ($branch_hash eq $branchcode?1:0)};	
 }
 
+ at branch_loop = sort {$a->{branchname} cmp $b->{branchname}} @branch_loop;
 my @authorised_value_list;
 my $authorisedvalue_categories;
 
-- 
1.5.6.5