diff --git a/config/alfresco/audit/alfresco-audit-3.2.xsd b/config/alfresco/audit/alfresco-audit-3.2.xsd index f050c984e0..c7df8902e1 100644 --- a/config/alfresco/audit/alfresco-audit-3.2.xsd +++ b/config/alfresco/audit/alfresco-audit-3.2.xsd @@ -110,7 +110,7 @@ - + @@ -118,7 +118,7 @@ - + @@ -126,7 +126,7 @@ - + diff --git a/source/java/org/alfresco/repo/audit/AuditComponentImpl.java b/source/java/org/alfresco/repo/audit/AuditComponentImpl.java index 5a7eec6772..9b624e7197 100644 --- a/source/java/org/alfresco/repo/audit/AuditComponentImpl.java +++ b/source/java/org/alfresco/repo/audit/AuditComponentImpl.java @@ -1096,10 +1096,13 @@ public class AuditComponentImpl implements AuditComponent if (disabledPaths.contains(AuditApplication.buildPath(rootKey))) { // The root key has been disabled for this application - logger.debug( - "Audit values root path has been excluded by disabled paths: \n" + - " Application: " + application + "\n" + - " Root Path: " + AuditApplication.buildPath(rootKey)); + if (logger.isDebugEnabled()) + { + logger.debug( + "Audit values root path has been excluded by disabled paths: \n" + + " Application: " + application + "\n" + + " Root Path: " + AuditApplication.buildPath(rootKey)); + } continue; } // Do the audit diff --git a/source/java/org/alfresco/repo/audit/model/AuditApplication.java b/source/java/org/alfresco/repo/audit/model/AuditApplication.java index 2ec374cb0a..d6e56414d2 100644 --- a/source/java/org/alfresco/repo/audit/model/AuditApplication.java +++ b/source/java/org/alfresco/repo/audit/model/AuditApplication.java @@ -50,8 +50,8 @@ import org.apache.commons.logging.LogFactory; public class AuditApplication { public static final String AUDIT_PATH_SEPARATOR = "/"; - public static final String AUDIT_KEY_REGEX = "[a-zA-Z0-9\\-\\.]+"; - public static final String AUDIT_PATH_REGEX = "(/[a-zA-Z0-9\\-\\.]+)+"; + public static final String AUDIT_KEY_REGEX = "[a-zA-Z0-9\\-\\_\\.]+"; + public static final String AUDIT_PATH_REGEX = "(/[a-zA-Z0-9\\-\\_\\.]+)+"; private static final Log logger = LogFactory.getLog(AuditApplication.class); diff --git a/source/java/org/alfresco/repo/audit/model/_3/ObjectFactory.java b/source/java/org/alfresco/repo/audit/model/_3/ObjectFactory.java index e5d10ed115..422c3785ff 100644 --- a/source/java/org/alfresco/repo/audit/model/_3/ObjectFactory.java +++ b/source/java/org/alfresco/repo/audit/model/_3/ObjectFactory.java @@ -33,14 +33,6 @@ public class ObjectFactory { public ObjectFactory() { } - /** - * Create an instance of {@link DataExtractor } - * - */ - public DataExtractor createDataExtractor() { - return new DataExtractor(); - } - /** * Create an instance of {@link Audit } * @@ -49,46 +41,6 @@ public class ObjectFactory { return new Audit(); } - /** - * Create an instance of {@link DataExtractors } - * - */ - public DataExtractors createDataExtractors() { - return new DataExtractors(); - } - - /** - * Create an instance of {@link AuditPath } - * - */ - public AuditPath createAuditPath() { - return new AuditPath(); - } - - /** - * Create an instance of {@link PathMap } - * - */ - public PathMap createPathMap() { - return new PathMap(); - } - - /** - * Create an instance of {@link KeyedAuditDefinition } - * - */ - public KeyedAuditDefinition createKeyedAuditDefinition() { - return new KeyedAuditDefinition(); - } - - /** - * Create an instance of {@link PathMappings } - * - */ - public PathMappings createPathMappings() { - return new PathMappings(); - } - /** * Create an instance of {@link DataGenerator } * @@ -98,19 +50,19 @@ public class ObjectFactory { } /** - * Create an instance of {@link Application } + * Create an instance of {@link AuditPath } * */ - public Application createApplication() { - return new Application(); + public AuditPath createAuditPath() { + return new AuditPath(); } /** - * Create an instance of {@link DataGenerators } + * Create an instance of {@link KeyedAuditDefinition } * */ - public DataGenerators createDataGenerators() { - return new DataGenerators(); + public KeyedAuditDefinition createKeyedAuditDefinition() { + return new KeyedAuditDefinition(); } /** @@ -121,6 +73,22 @@ public class ObjectFactory { return new GenerateValue(); } + /** + * Create an instance of {@link DataGenerators } + * + */ + public DataGenerators createDataGenerators() { + return new DataGenerators(); + } + + /** + * Create an instance of {@link DataExtractors } + * + */ + public DataExtractors createDataExtractors() { + return new DataExtractors(); + } + /** * Create an instance of {@link RecordValue } * @@ -129,6 +97,38 @@ public class ObjectFactory { return new RecordValue(); } + /** + * Create an instance of {@link Application } + * + */ + public Application createApplication() { + return new Application(); + } + + /** + * Create an instance of {@link DataExtractor } + * + */ + public DataExtractor createDataExtractor() { + return new DataExtractor(); + } + + /** + * Create an instance of {@link PathMappings } + * + */ + public PathMappings createPathMappings() { + return new PathMappings(); + } + + /** + * Create an instance of {@link PathMap } + * + */ + public PathMap createPathMap() { + return new PathMap(); + } + /** * Create an instance of {@link JAXBElement }{@code <}{@link Audit }{@code >}} *