[
https://issues.apache.org/jira/browse/WW-4126?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Adrian Nistor updated WW-4126:
------------------------------
Attachment: patch2.diff
patch.diff
> Incorrect behavior for ELSupport.containsNulls()
> ------------------------------------------------
>
> Key: WW-4126
> URL: https://issues.apache.org/jira/browse/WW-4126
> Project: Struts 2
> Issue Type: Bug
> Components: Plugin - Embedded JSP
> Affects Versions: 2.3.14.3
> Environment: any
> Reporter: Adrian Nistor
> Labels: patch
> Attachments: patch2.diff, patch.diff
>
>
> The incorrect behavior appears in version 2.3.14.3 and in revision
> 1495522. I attached a one-line patch (patch.diff) that fixes it.
> Method "ELSupport.containsNulls" checks if the "Object[] obj" array
> contains nulls like this:
> {code:java|borderStyle=solid}
> for (int i = 0; i < obj.length; i++) {
> if (obj[0] == null) {
> return true;
> }
> }
> return false;
> {code}
> Instead of "obj[0]", the code should be "obj[i]", as in patch.diff.
> If indeed the code is intended to check only "obj[0]", then there is
> no need for a loop and the entire method body should be only "return
> obj[0] == null;" (I attached patch2.diff, though it's unlikely that
> this is the correct behavior).
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira