[jira] Resolved: (TRINIDAD-1830) UIXHierarchy.visitHierarchy() has a bad if statement

["Matt Cooper (JIRA)" <de...@myfaces.apache.org>]

     [ https://issues.apache.org/jira/browse/TRINIDAD-1830?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Matt Cooper resolved TRINIDAD-1830.
-----------------------------------

    Fix Version/s: 1.2.14-core 
                    2.0.0.2-core 
       Resolution: Fixed

> UIXHierarchy.visitHierarchy() has a bad if statement
> ----------------------------------------------------
>
>                 Key: TRINIDAD-1830
>                 URL: https://issues.apache.org/jira/browse/TRINIDAD-1830
>             Project: MyFaces Trinidad
>          Issue Type: Bug
>    Affects Versions:  2.0.0.2-core 
>            Reporter: Matt Cooper
>            Assignee: Matt Cooper
>             Fix For: 1.2.14-core ,  2.0.0.2-core 
>
>
> I have found some code in UIXHierarchy.visitHierarchy() where according to the code's indentation, a return true should happen only if the if statement is true.
> However, this if clause ends with a semicolon so regardless of whether the the clause is true, the return true will always happen.
> This ends up causing UIXHierarchy.visitHierarchy() to short circuit prematurely (and a call to UIXTree.visitChildren() to in turn short circuit visitation of the view).

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