You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by an...@apache.org on 2015/03/03 10:28:57 UTC

svn commit: r1663570 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/SecurityProviderImpl.java

Author: angela
Date: Tue Mar  3 09:28:56 2015
New Revision: 1663570

URL: http://svn.apache.org/r1663570
Log:
minor improvement : @SuppressWarnings, simplify usage of ConfigurationParameters

Modified:
    jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/SecurityProviderImpl.java

Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/SecurityProviderImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/SecurityProviderImpl.java?rev=1663570&r1=1663569&r2=1663570&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/SecurityProviderImpl.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/security/SecurityProviderImpl.java Tue Mar  3 09:28:56 2015
@@ -207,28 +207,31 @@ public class SecurityProviderImpl implem
         userAuthenticationFactory.stop();
     }
 
+    @SuppressWarnings("UnusedDeclaration")
     protected void bindPrincipalConfiguration(@Nonnull PrincipalConfiguration reference) {
         principalConfiguration.addConfiguration(initConfiguration(reference));
     }
 
+    @SuppressWarnings("UnusedDeclaration")
     protected void unbindPrincipalConfiguration(@Nonnull PrincipalConfiguration reference) {
         principalConfiguration.removeConfiguration(reference);
     }
 
+    @SuppressWarnings("UnusedDeclaration")
     protected void bindTokenConfiguration(@Nonnull TokenConfiguration reference) {
         tokenConfiguration.addConfiguration(initConfiguration(reference));
     }
 
+    @SuppressWarnings("UnusedDeclaration")
     protected void unbindTokenConfiguration(@Nonnull TokenConfiguration reference) {
         tokenConfiguration.removeConfiguration(reference);
     }
 
     //------------------------------------------------------------< private >---
     private void initializeConfigurations() {
-        Map<String, WhiteboardRestrictionProvider> authorizMap = ImmutableMap.of(
-                AccessControlConstants.PARAM_RESTRICTION_PROVIDER, restrictionProvider
+        initConfiguration(authorizationConfiguration, ConfigurationParameters.of(
+                AccessControlConstants.PARAM_RESTRICTION_PROVIDER, restrictionProvider)
         );
-        initConfiguration(authorizationConfiguration, ConfigurationParameters.of(authorizMap));
 
         Map<String, Object> userMap = ImmutableMap.<String,Object>of(
                 UserConstants.PARAM_AUTHORIZABLE_ACTION_PROVIDER, authorizableActionProvider,