[jira] Created: (WW-3455) ServletRedirectResult ignoring supressEmptyParameters option

[Thomas Howard (JIRA)]

ServletRedirectResult ignoring supressEmptyParameters option
------------------------------------------------------------

                 Key: WW-3455
                 URL: https://issues.apache.org/jira/browse/WW-3455
             Project: Struts 2
          Issue Type: Bug
    Affects Versions: 2.1.8.1
            Reporter: Thomas Howard


I've been attempting to use the supressEmptyParameters option on 
ServletRedirectResult but it appears that this option is basically ignored in 
the code.  

Looking at the source it seems there's a line that adds the value to the 
requestParameters map:

requestParameters.put(e.getKey().toString(),
                                e.getValue() == null ? "" :
                                        
conditionalParse(e.getValue().toString(), invocation));

Which is then followed by the check of the empty parameters, which then 
attempts to add the value again:

String potentialValue = e.getValue() == null ? "" : 
conditionalParse(e.getValue().toString(), invocation);
                        if (!supressEmptyParameters || ((potentialValue != 
null) && (potentialValue.length() > 0))) {
                            requestParameters.put(e.getKey().toString(), 
potentialValue);
                        }

I'm guessing that the first line was either left in by mistake, or maybe got 
merged back into place by mistake, but basically needs to be removed.

Thanks

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.