Author: pauls
Date: Fri Oct 13 12:27:01 2017
New Revision: 1812124

URL: http://svn.apache.org/viewvc?rev=1812124&view=rev
Log:
SLING-7198: Remove duplicated lookup for mapping without subserviceName.

Modified:
    
sling/trunk/bundles/extensions/serviceusermapper/src/main/java/org/apache/sling/serviceusermapping/impl/ServiceUserMapperImpl.java

Modified: 
sling/trunk/bundles/extensions/serviceusermapper/src/main/java/org/apache/sling/serviceusermapping/impl/ServiceUserMapperImpl.java
URL: 
http://svn.apache.org/viewvc/sling/trunk/bundles/extensions/serviceusermapper/src/main/java/org/apache/sling/serviceusermapping/impl/ServiceUserMapperImpl.java?rev=1812124&r1=1812123&r2=1812124&view=diff
==============================================================================
--- 
sling/trunk/bundles/extensions/serviceusermapper/src/main/java/org/apache/sling/serviceusermapping/impl/ServiceUserMapperImpl.java
 (original)
+++ 
sling/trunk/bundles/extensions/serviceusermapper/src/main/java/org/apache/sling/serviceusermapping/impl/ServiceUserMapperImpl.java
 Fri Oct 13 12:27:01 2017
@@ -479,14 +479,6 @@ public class ServiceUserMapperImpl imple
             }
         }
 
-        for (Mapping mapping : this.activeMappings) {
-            final Iterable<String> principalNames = 
mapping.mapPrincipals(serviceName, null);
-            if (principalNames != null) {
-                log.debug("Got principalNames [{}] from {}/{}", new Object[] 
{principalNames, serviceName });
-                return principalNames;
-            }
-        }
-
         // second round without serviceInfo
         log.debug(
                 "internalGetPrincipalNames: {} active mappings, looking for 
mapping for {}/<no subServiceName>",


Reply via email to