Index: src/main/java/org/apache/jackrabbit/core/config/RepositoryConfigurationParser.java =================================================================== --- src/main/java/org/apache/jackrabbit/core/config/RepositoryConfigurationParser.java (revision 881285) +++ src/main/java/org/apache/jackrabbit/core/config/RepositoryConfigurationParser.java (working copy) @@ -593,8 +593,8 @@ /** - * Read the WorkspaceSecurity Element of Workspace's configuration. It uses - * the following format: + * Read the optional WorkspaceSecurity Element of Workspace's configuration. + * It uses the following format: *
      *   <WorkspaceSecurity>
      *     <AccessControlProvider class="..." (optional)>
@@ -602,7 +602,8 @@
      * 
* * @param parent Workspace-Root-Element - * @return a new WorkspaceSecurityConfig + * @return a new WorkspaceSecurityConfig or null + * if none is configured. * @throws ConfigurationException */ public WorkspaceSecurityConfig parseWorkspaceSecurityConfig(Element parent) @@ -616,8 +617,9 @@ acProviderConfig = parseBeanConfig(element, AC_PROVIDER_ELEMENT); acProviderConfig.setValidate(false); // JCR-1920 } + return new WorkspaceSecurityConfig(acProviderConfig); } - return new WorkspaceSecurityConfig(acProviderConfig); + return null; } /** @@ -633,12 +635,13 @@ * * * @param parent Workspace-Root-Element - * @return a new XmlImportConfig + * @return a new ImportConfig or null if none is + * configured. * @throws ConfigurationException */ public ImportConfig parseImportConfig(Element parent) throws ConfigurationException { - List protectedNodeImporters = new ArrayList(); - List protectedPropertyImporters = new ArrayList(); + List protectedNodeImporters = new ArrayList(); + List protectedPropertyImporters = new ArrayList(); Element element = getElement(parent, IMPORT_ELEMENT, false); if (element != null) { @@ -659,8 +662,9 @@ } // else: some other entry -> ignore. } } + return new ImportConfig(protectedNodeImporters, protectedPropertyImporters); } - return new ImportConfig(protectedNodeImporters, protectedPropertyImporters); + return null; } /**