Merge pull request #2 from dlbptfs/PT56448214

[#56448214] Random drop down menu order when updating member attribute S...
liblime · Jan 15, 2015 · d675771 · d675771
2 parents 4545a9c + 92e0d59
commit d675771
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/cgi/members/memberentry.pl b/cgi/members/memberentry.pl
@@ -789,11 +789,11 @@ sub patron_attributes_form {
                 $newentry->{use_dropdown} = 0;
                 if ($attr_type->authorised_value_category()) {
                     $newentry->{use_dropdown} = 1;
-                    $newentry->{auth_val_loop} = GetAuthorisedValues($attr_type->authorised_value_category(), $attr->{value});
+                    my $authvals = GetAuthorisedValues($attr_type->authorised_value_category(), $attr->{value});
+                    $newentry->{auth_val_loop} = [ sort {lc($a->{lib}) cmp lc($b->{lib})} @{$authvals} ];
                 }
                 $i++;
                 $newentry->{form_id} = "patron_attr_$i";
-                #use Data::Dumper; die Dumper($entry) if  $entry->{use_dropdown};
                 push @attribute_loop, $newentry;
             }
         } else {