pauamma: Cartooney crab wearing hot pink and acid green facemask holding drink with straw (Default)
Res facta quae tamen fingi potuit ([personal profile] pauamma) wrote in [site community profile] changelog2009-03-09 08:15 pm

[dw-free] Manage Filters page does not alphabetize lists

[commit: http://hg.dwscoalition.org/dw-free/rev/790d6028b787]

http://bugs.dwscoalition.org/show_bug.cgi?id=450

Alphabetize filter lists.

Patch by [personal profile] janine.

Files modified:
  • htdocs/manage/circle/editfilters.bml
--------------------------------------------------------------------------------
diff -r acc9d0a50cd9 -r 790d6028b787 htdocs/manage/circle/editfilters.bml
--- a/htdocs/manage/circle/editfilters.bml	Mon Mar 09 18:29:10 2009 +0000
+++ b/htdocs/manage/circle/editfilters.bml	Mon Mar 09 20:14:48 2009 +0000
@@ -123,9 +123,8 @@
         $body .= "<input type='hidden' name='efg_delete_${i}' value='0' />";
         $body .= "<input type='hidden' name='efg_set_${i}_public' value='$public' />";
     }
-    foreach my $uid ( keys %$trust_list ) {
+    foreach my $uid ( sort { $trusted_us->{$a}->display_username cmp $trusted_us->{$b}->display_username } keys %$trust_list ) {
         my $trusted_u = $trusted_us->{$uid};
-        next unless LJ::isu( $trusted_u );
 
         my $user = $trusted_u->user;
         my $groupmask = $trust_list->{$uid}->{groupmask} || 1;
--------------------------------------------------------------------------------