svn commit: r942297 - in /click/trunk/click/documentation/docs: roadmap-changes.html upgrade-path.html

[sa...@apache.org]

Author: sabob
Date: Sat May  8 02:57:16 2010
New Revision: 942297

URL: http://svn.apache.org/viewvc?rev=942297&view=rev
Log:
doco

Modified:
    click/trunk/click/documentation/docs/roadmap-changes.html
    click/trunk/click/documentation/docs/upgrade-path.html

Modified: click/trunk/click/documentation/docs/roadmap-changes.html
URL: http://svn.apache.org/viewvc/click/trunk/click/documentation/docs/roadmap-changes.html?rev=942297&r1=942296&r2=942297&view=diff
==============================================================================
--- click/trunk/click/documentation/docs/roadmap-changes.html (original)
+++ click/trunk/click/documentation/docs/roadmap-changes.html Sat May  8 02:57:16 2010
@@ -201,7 +201,7 @@ includes improved Ajax support and @Bind
         </ul>
       </li>
       <li class="change">
-        Added new method <a href="click-api/org/apache/click/control/Container.html#replace(org.apache.click.control Control, org.apache.click.control Control)">Container.replace</a>
+        Added new method <a href="click-api/org/apache/click/control/Container.html#replace(org.apache.click.Control, org.apache.click.Control)">Container.replace</a>
         for replacing an existing control with a new control. In previous releases,
         an exception would be raised if a page or container already contained a
         control with the same name as the newly added control. In 2.2.0 the

Modified: click/trunk/click/documentation/docs/upgrade-path.html
URL: http://svn.apache.org/viewvc/click/trunk/click/documentation/docs/upgrade-path.html?rev=942297&r1=942296&r2=942297&view=diff
==============================================================================
--- click/trunk/click/documentation/docs/upgrade-path.html (original)
+++ click/trunk/click/documentation/docs/upgrade-path.html Sat May  8 02:57:16 2010
@@ -173,6 +173,22 @@ public class MyPage extends BorderPage {
         Please note, for backward compatibility the autobinding attribute still
         accepts the <tt>public</tt> value.
       </li>
+      <li class="change">
+        Added new method <a href="click-api/org/apache/click/control/Container.html#replace(org.apache.click.control Control, org.apache.click.control Control)">Container.replace</a>
+        for replacing an existing control with a new control. AbstractContainer
+        has an implementation of this method which should cater for most use cases.
+        <p/>
+        If however you have custom controls that override the
+        <a href="click-api/org/apache/click/control/Container.html#insert(org.apache.click.Control, int)">insert</a> or
+        <a href="click-api/org/apache/click/control/Container.html#add(org.apache.click.Control)">add</a>
+        methods, you need to ensure that
+        <a href="click-api/org/apache/click/control/Container.html#replace(org.apache.click.Control, org.apache.click.Control)">replace</a>
+        will function correctly for your control.
+        <p/>
+        For example, if a custom insert method adds the control to an
+        internal list the <tt>replace</tt> method must be overridden to replace
+        the control on that list as well.
+      </li>
     </ul>
   </dd>