Re: svn commit: r419092 - in /myfaces: maven/trunk/maven-archetype-myfaces/src/main/resources/archetype-resources/src/main/webapp/WEB-INF/ shared/trunk/core/src/main/java/org/apache/myfaces/shared/renderkit/html/ tomahawk/trunk/examples/blank/src/mai

[Mike Kienenberger]

Hey Martin, now that you're back :--)

Shouldn't you be checking against a Form Family member rather than a
UIForm?   I know ADFFaces/Trinidad doesn't descend from UIForm.


On 7/4/06, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote:

Author: mmarinschek
Date: Tue Jul  4 13:41:53 2006
New Revision: 419092

URL: http://svn.apache.org/viewvc?rev=419092&view=rev
Log:
implemented a check for nested forms. Nested forms are not allowed in HTML - 
and if you do use them, they cause unpredictable behaviour, depending on the 
browser family.

org.apache.myfaces.shared.renderkit.RendererUtils.checkParamValidity(facesContext,
component, UIForm.class);

+                if(parent instanceof UIForm)
+                {
+                    throw new FacesException("You should never nest HTML-forms. 
"+
+                            "This leads to unpredictable behaviour in all major 
browsers. "+
+                                "You can use multiple forms on a page, but they may 
not be nested!");
+                }

         UIForm htmlForm = (UIForm)component;