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

Posted to commits@myfaces.apache.org by mm...@apache.org on 2006/02/21 16:45:54 UTC

svn commit: r379497 - /myfaces/tomahawk/trunk/sandbox/core/src/main/java/org/apache/myfaces/custom/subform/SubForm.java

Author: mmarinschek
Date: Tue Feb 21 07:45:53 2006
New Revision: 379497

URL: http://svn.apache.org/viewcvs?rev=379497&view=rev
Log:
small bug fix

Modified:
    myfaces/tomahawk/trunk/sandbox/core/src/main/java/org/apache/myfaces/custom/subform/SubForm.java

Modified: myfaces/tomahawk/trunk/sandbox/core/src/main/java/org/apache/myfaces/custom/subform/SubForm.java
URL: http://svn.apache.org/viewcvs/myfaces/tomahawk/trunk/sandbox/core/src/main/java/org/apache/myfaces/custom/subform/SubForm.java?rev=379497&r1=379496&r2=379497&view=diff
==============================================================================
--- myfaces/tomahawk/trunk/sandbox/core/src/main/java/org/apache/myfaces/custom/subform/SubForm.java (original)
+++ myfaces/tomahawk/trunk/sandbox/core/src/main/java/org/apache/myfaces/custom/subform/SubForm.java Tue Feb 21 07:45:53 2006
@@ -147,6 +147,15 @@
             _submitted = true;
         }
 
+        // This idea is taken from ADF faces - my approach didn't go as far
+        // as necessary - I still believe this to be a hack as well.
+        // If the event is being queued for anything *after* APPLY_REQUEST_VALUES,
+        // then this subform is active.
+        if (PhaseId.APPLY_REQUEST_VALUES.compareTo(event.getPhaseId()) < 0)
+        {
+            setSubmitted(true);
+        }
+
         super.queueEvent(event);
     }