[
https://issues.apache.org/struts/browse/WW-3109?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Wes Wannemacher resolved WW-3109.
---------------------------------
Resolution: Fixed
Fix Version/s: (was: 2.1.8)
2.1.7
Assignee: Musachy Barroso
Musachy fixed this in XWork back in March... Unfortunately, Fisheye on
opensymphony is out of sync, but here is the patch from the fix -
Index:
src/java/com/opensymphony/xwork2/validator/validators/ValidatorSupport.java
===================================================================
--- src/java/com/opensymphony/xwork2/validator/validators/ValidatorSupport.java
(revision 1956)
+++ src/java/com/opensymphony/xwork2/validator/validators/ValidatorSupport.java
(revision 1957)
@@ -13,7 +13,9 @@
import java.util.ArrayList;
import java.util.List;
+import org.apache.commons.lang.StringUtils;
+
/**
* Abstract implementation of the Validator interface suitable for subclassing.
*
@@ -96,7 +98,8 @@
message = defaultMessage;
}
- message = TextParseUtil.translateVariables(message, stack);
+ if (StringUtils.isNotBlank(message))
+ message = TextParseUtil.translateVariables(message, stack);
stack.pop();
You can try to create a patched copy of xwork, or wait on the next release of
Struts, which should be pretty soon.
> Visitor validator crashes when <message> is blank
> -------------------------------------------------
>
> Key: WW-3109
> URL: https://issues.apache.org/struts/browse/WW-3109
> Project: Struts 2
> Issue Type: Bug
> Components: XML Validators
> Affects Versions: 2.1.6
> Environment: Struts 2.1.6 on Websphere 6.1
> Reporter: Jon
> Assignee: Musachy Barroso
> Fix For: 2.1.7
>
>
> Using a visitor validator crashes application if the <message> element is
> empty. e.g.
> In BeanName-validation.xml this will crash.
> <field name="lifeAssured">
> <field-validator type="visitor">
> <param name="appendPrefix">true</param>
> <message></message>
> </field-validator>
> </field>
> But this will work fine.
> <field name="lifeAssured">
> <field-validator type="visitor">
> <param name="appendPrefix">true</param>
> <message>Hi</message>
> </field-validator>
> </field>
> Message shouldn't be mandatory - I'm happy for the messages defined in the
> nested beans validator to be displayed against the field without anything
> prepended.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
