Revision: 1123
Author:   jhoskens
Date:     2006-05-08 06:10:56 -0700 (Mon, 08 May 2006)
ViewCVS:  http://svn.sourceforge.net/spring-rich-c/?rev=1123&view=rev

Log Message:
-----------
Functionality reverted. When complex hierarchical structures are used, 
resetting child models can break the top-down structure.

Modified Paths:
--------------
    
trunk/spring-richclient/support/src/main/java/org/springframework/binding/form/support/AbstractFormModel.java
Modified: 
trunk/spring-richclient/support/src/main/java/org/springframework/binding/form/support/AbstractFormModel.java
===================================================================
--- 
trunk/spring-richclient/support/src/main/java/org/springframework/binding/form/support/AbstractFormModel.java
       2006-05-07 20:28:46 UTC (rev 1122)
+++ 
trunk/spring-richclient/support/src/main/java/org/springframework/binding/form/support/AbstractFormModel.java
       2006-05-08 13:10:56 UTC (rev 1123)
@@ -547,13 +547,15 @@
     }
 
     /**
-     * If formModel has children, these are reset first.
+     * Complex forms with parent-child relations can use derived formModels.
+     * Such a Hierarchical tree cannot have its children reset on its own as it
+     * would break the top-down structure. see RCP-329 and the cvs maillist.
+     * 
+     * TODO add a unit test with such a complex use case
+     * 
      * @see FormModel#reset()
      */
     public void reset() {
-        for (Iterator i = children.iterator(); i.hasNext();) {
-            ((FormModel)i.next()).reset();
-        }
         setFormObject(null);
     }
 


This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.



-------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
spring-rich-c-cvs mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/spring-rich-c-cvs

Reply via email to