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;
--------------------------------------------------------------------------------

Post a comment in response:

This account has disabled anonymous posting.
If you don't have an account you can create one now.
HTML doesn't work in the subject.
More info about formatting

If you are unable to use this captcha for any reason, please contact us by email at support@dreamwidth.org