kareila: (Default)
kareila ([personal profile] kareila) wrote in [site community profile] changelog2009-12-23 07:26 pm

[dw-free] More accurate nav strip reading page filter drop down behavior

[commit: http://hg.dwscoalition.org/dw-free/rev/304d266c3019]

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

Fix case sensitivity issue in subscription filter dropdown on reading page.

Patch by [personal profile] yvi.

Files modified:
  • cgi-bin/weblib.pl
--------------------------------------------------------------------------------
diff -r ddae2ed90eae -r 304d266c3019 cgi-bin/weblib.pl
--- a/cgi-bin/weblib.pl	Wed Dec 23 18:10:08 2009 +0000
+++ b/cgi-bin/weblib.pl	Wed Dec 23 13:26:20 2009 -0600
@@ -2719,7 +2719,7 @@ sub control_strip
                 my @custom_filters = $journal->content_filters;
 
                 foreach my $f ( @custom_filters ) {
-                     push @filters, "filter:" . $f->name, $f->name;
+                    push @filters, "filter:" . lc( $f->name ), $f->name;
                 }
 
                 my $selected = "all";
@@ -2729,7 +2729,7 @@ sub control_strip
                     $selected = "showsyndicated"  if $r->query_string eq "show=F&filter=0";
                 } elsif ($r->uri =~ /^\/read\/?(.+)?/i) {
                     my $filter = $1 || "default view";
-                    $selected = "filter:" . LJ::durl(lc($filter));
+                    $selected = "filter:" . LJ::durl( lc( $filter ) );
                 } elsif ($r->uri eq "/network" && $r->query_string ne "") {
                     $selected = "showpeople"      if $r->query_string eq "show=P&filter=0";
                     $selected = "showcommunities" if $r->query_string eq "show=C&filter=0";
--------------------------------------------------------------------------------