Mailing-List: contact issues-help@struts.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@struts.apache.org
Date: Wed, 17 Jul 2013 15:12:56 +0000 (UTC)
From: "ASF subversion and git services (JIRA)" <jira@apache.org>
To: issues@struts.apache.org
Message-ID: <JIRA.12654251.1371850322270.65990.1374073976372@arcas>
In-Reply-To: <JIRA.12654251.1371850322270@arcas>
References: <JIRA.12654251.1371850322270@arcas>
Subject: [jira] [Commented] (WW-4126) Incorrect behavior for
 ELSupport.containsNulls()
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


    [ https://issues.apache.org/jira/browse/WW-4126?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13711158#comment-13711158 ] 

ASF subversion and git services commented on WW-4126:
-----------------------------------------------------

Commit 1504168 from [~bphillips] in branch 'struts2/trunk'
[ https://svn.apache.org/r1504168 ]

WW-4126 - changed logic of is statement so that every element in the Object array is compared to null
                
> 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
>            Assignee: Bruce Phillips
>              Labels: patch
>             Fix For: 2.3.16
>
>         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