[
https://issues.apache.org/jira/browse/WW-4275?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13867732#comment-13867732
]
Greg Huber commented on WW-4275:
--------------------------------
I have tested the patch against primitive types, Integers, Doubles, Lists etc.
Committed revision 1557084.
Cheers Greg.
> Cast exception on Supressing empty parameters on <s:a> tag
> -----------------------------------------------------------
>
> Key: WW-4275
> URL: https://issues.apache.org/jira/browse/WW-4275
> Project: Struts 2
> Issue Type: Bug
> Components: Plugin - Java Templates
> Affects Versions: 2.3.16
> Environment: Tomcat/Centos
> Reporter: Greg Huber
> Assignee: Greg Huber
> Priority: Minor
> Fix For: 2.3.17
>
>
> I have noticed that my patch does not handle integers correctly, alas, we
> need to do another change
> It does kind of the same check, but this works with integers/doubles.
> {code:java}
> Object value = findValue(this.value);
> if (suppressEmptyParameters) {
> if (value != null && !value.toString().isEmpty()){
> component.addParameter(name, value); }
> } else{
> component.addParameter(name, value);
> }
> {code}
> existing code:
> {code:java}
> if (suppressEmptyParameters) {
> String potentialValue = (String) value;
> if (potentialValue != null && potentialValue.length() > 0) {
> component.addParameter(name, value); }
> } else { component.addParameter(name, value); }
> {code}
--
This message was sent by Atlassian JIRA
(v6.1.5#6160)
