Make sure we only avoid this path when the filter argument is zero.

        Patch by kareila. 

diffs (14 lines):

diff -r b61bdc1afbd2 -r 458e61bf06ef cgi-bin/LJ/S2/FriendsPage.pm
--- a/cgi-bin/LJ/S2/FriendsPage.pm      Sat Oct 23 11:39:13 2010 +0800
+++ b/cgi-bin/LJ/S2/FriendsPage.pm      Fri Oct 22 22:51:27 2010 -0500
@@ -132,8 +132,8 @@ sub FriendsPage
             $p->{filter_name} = $opts->{securityfilter};
     } else {
     # but we can't just use a filter, we have to make sure the person is 
allowed to
-        if ( ( ! $get->{filter} || $get->{filter} && $get->{filter} ne "0" )
-            && $cf && ( $u->equals( $remote ) || $cf->public ) ) {
+        if ( ( ! defined $get->{filter} || $get->{filter} ne "0" )
+                && $cf && ( $u->equals( $remote ) || $cf->public ) ) {
             $filter = $cf;
 
         # if we couldn't use the group, then we can throw an error, but ONLY 
IF they specified
_______________________________________________
dw-changelog mailing list
[email protected]
http://lists.dwscoalition.org/mailman/listinfo/dw-changelog

Reply via email to