jackrabbit-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From dbros...@apache.org
Subject svn commit: r1185434 - /jackrabbit/trunk/jackrabbit-webdav/src/main/java/org/apache/jackrabbit/webdav/security/report/AclPrincipalReport.java
Date Mon, 17 Oct 2011 23:52:38 GMT
Author: dbrosius
Date: Mon Oct 17 23:52:38 2011
New Revision: 1185434

URL: http://svn.apache.org/viewvc?rev=1185434&view=rev
Log:
fix ignoring of duplicate principal processing

Modified:
    jackrabbit/trunk/jackrabbit-webdav/src/main/java/org/apache/jackrabbit/webdav/security/report/AclPrincipalReport.java

Modified: jackrabbit/trunk/jackrabbit-webdav/src/main/java/org/apache/jackrabbit/webdav/security/report/AclPrincipalReport.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-webdav/src/main/java/org/apache/jackrabbit/webdav/security/report/AclPrincipalReport.java?rev=1185434&r1=1185433&r2=1185434&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-webdav/src/main/java/org/apache/jackrabbit/webdav/security/report/AclPrincipalReport.java
(original)
+++ jackrabbit/trunk/jackrabbit-webdav/src/main/java/org/apache/jackrabbit/webdav/security/report/AclPrincipalReport.java
Mon Oct 17 23:52:38 2011
@@ -82,7 +82,7 @@ public class AclPrincipalReport extends 
         List<AclProperty.Ace> list = (List<AclProperty.Ace>) ((AclProperty)acl).getValue();
         for (AclProperty.Ace ace : list) {
             String href = ace.getPrincipal().getHref();
-            if (href == null && respMap.containsKey(href)) {
+            if (href == null || respMap.containsKey(href)) {
                 // ignore non-href principals and principals that have been listed before
                 continue;
             }



Mime
View raw message