Author: fmeschbe
Date: Tue Feb 16 10:59:31 2010
New Revision: 910468
URL: http://svn.apache.org/viewvc?rev=910468&view=rev
Log:
SLING-1377 use the actual searchType to find principals and remove old comment
Modified:
sling/trunk/bundles/jcr/jackrabbit-usermanager/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/resource/AuthorizableResourceProvider.java
Modified:
sling/trunk/bundles/jcr/jackrabbit-usermanager/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/resource/AuthorizableResourceProvider.java
URL:
http://svn.apache.org/viewvc/sling/trunk/bundles/jcr/jackrabbit-usermanager/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/resource/AuthorizableResourceProvider.java?rev=910468&r1=910467&r2=910468&view=diff
==============================================================================
---
sling/trunk/bundles/jcr/jackrabbit-usermanager/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/resource/AuthorizableResourceProvider.java
(original)
+++
sling/trunk/bundles/jcr/jackrabbit-usermanager/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/resource/AuthorizableResourceProvider.java
Tue Feb 16 10:59:31 2010
@@ -169,15 +169,10 @@
}
if (searchType != -1) {
PrincipalIterator principals = null;
-
- // TODO: this actually does not work correctly since the
- // jackrabbit findPrincipals API
- // currently does an exact match of the search filter so it
- // won't match a wildcard
Session session = resourceResolver.adaptTo(Session.class);
if (session != null) {
PrincipalManager principalManager =
AccessControlUtil.getPrincipalManager(session);
- principals =
principalManager.getPrincipals(PrincipalManager.SEARCH_TYPE_NOT_GROUP);
+ principals = principalManager.getPrincipals(searchType);
}
if (principals != null) {
