Apache/struts
1
0
Fork 0
mirror of https://github.com/apache/struts.git synced 2026-08-06 23:27:07 +00:00
Code Issues Packages Projects Releases Wiki Activity

fixed

Browse Source 
getValidationGroups get called if not skipped Validation
removed unnecessary LOG.isDebugEnabled()
This commit is contained in:
Hedju Hor
2018-01-12 18:36:08 +01:00
parent 2b2de35519
commit 7518e809dd
1 changed files with 3 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
plugins/bean-validation/src/main/java/org/apache/struts/beanvalidation/validation/interceptor/BeanValidationInterceptor.java
+3 -7
View File
@@ -94,13 +94,11 @@ public class BeanValidationInterceptor extends MethodFilterInterceptor {
ActionProxy actionProxy = invocation.getProxy();
String methodName = actionProxy.getMethod();
if (LOG.isDebugEnabled()) {
LOG.debug("Validating [{}/{}] with method [{}]", invocation.getProxy().getNamespace(), invocation.getProxy().getActionName(), methodName);
}
Class<?>[] validationGroup = getValidationGroups(action, methodName);
LOG.debug("Validating [{}/{}] with method [{}]", invocation.getProxy().getNamespace(), invocation.getProxy().getActionName(), methodName);
if (null == MethodUtils.getAnnotation(getActionMethod(action.getClass(), methodName), SkipValidation.class,
true, true)) {
Class<?>[] validationGroup = getValidationGroups(action, methodName);
// performing bean validation on action
performBeanValidation(action, validator, validationGroup);
}
@@ -156,9 +154,7 @@ public class BeanValidationInterceptor extends MethodFilterInterceptor {
if (action instanceof ModelDriven && fieldName.startsWith(ValidatorConstants.MODELDRIVEN_PREFIX)) {
fieldName = fieldName.replace("model.", ValidatorConstants.EMPTY_SPACE);
}
if (LOG.isDebugEnabled()) {
LOG.debug("Adding field error [{}] with message [{}]", fieldName, validationError.getMessage());
}
LOG.debug("Adding field error [{}] with message [{}]", fieldName, validationError.getMessage());
validatorContext.addFieldError(fieldName, validationError.getMessage());
}
}