You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@myfaces.apache.org by mc...@apache.org on 2010/06/12 02:10:43 UTC

svn commit: r953909 - /myfaces/trinidad/branches/trinidad-1.2.x/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXHierarchy.java

Author: mcooper
Date: Sat Jun 12 00:10:42 2010
New Revision: 953909

URL: http://svn.apache.org/viewvc?rev=953909&view=rev
Log:
TRINIDAD-1830 UIXHierarchy.visitHierarchy() has a bad if statement

Fixed a bad semicolon in an if statement that causes UIXHierarchy to terminate prematurely; the following "return true" was always executing regardless of the result of the if clause.

Modified:
    myfaces/trinidad/branches/trinidad-1.2.x/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXHierarchy.java

Modified: myfaces/trinidad/branches/trinidad-1.2.x/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXHierarchy.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/branches/trinidad-1.2.x/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXHierarchy.java?rev=953909&r1=953908&r2=953909&view=diff
==============================================================================
--- myfaces/trinidad/branches/trinidad-1.2.x/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXHierarchy.java (original)
+++ myfaces/trinidad/branches/trinidad-1.2.x/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXHierarchy.java Sat Jun 12 00:10:42 2010
@@ -355,7 +355,7 @@ public abstract class UIXHierarchy exten
           try
           {
             // visit this container.  If we have visited all of the visit targets then return early
-            if (visitHierarchy(visitContext, callback, stamps, disclosedRowKeys));
+            if (visitHierarchy(visitContext, callback, stamps, disclosedRowKeys))
               return true;
           }
           finally