You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by sv...@apache.org on 2019/04/24 07:33:49 UTC
[wicket] branch wicket-8.x updated: WICKET-6658 allow non-form tag
for nested forms
This is an automated email from the ASF dual-hosted git repository.
svenmeier pushed a commit to branch wicket-8.x
in repository https://gitbox.apache.org/repos/asf/wicket.git
The following commit(s) were added to refs/heads/wicket-8.x by this push:
new 62638b9 WICKET-6658 allow non-form tag for nested forms
62638b9 is described below
commit 62638b92d48f295bdbdab2921074958e3c970891
Author: Sven Meier <sv...@apache.org>
AuthorDate: Wed Apr 24 09:13:21 2019 +0200
WICKET-6658 allow non-form tag for nested forms
---
.../src/main/java/org/apache/wicket/markup/html/form/Form.java | 10 +++++++---
.../org/apache/wicket/markup/html/form/NestedFormsPage.html | 5 +++--
2 files changed, 10 insertions(+), 5 deletions(-)
diff --git a/wicket-core/src/main/java/org/apache/wicket/markup/html/form/Form.java b/wicket-core/src/main/java/org/apache/wicket/markup/html/form/Form.java
index 551eb3c..19cce91 100644
--- a/wicket-core/src/main/java/org/apache/wicket/markup/html/form/Form.java
+++ b/wicket-core/src/main/java/org/apache/wicket/markup/html/form/Form.java
@@ -1575,10 +1575,10 @@ public class Form<T> extends WebMarkupContainer
{
super.onComponentTag(tag);
- checkComponentTag(tag, "form");
-
if (isRootForm())
{
+ checkComponentTag(tag, "form");
+
String method = getMethod().toLowerCase(Locale.ROOT);
tag.put("method", method);
String url = getActionUrl().toString();
@@ -1631,7 +1631,11 @@ public class Form<T> extends WebMarkupContainer
}
else
{
- tag.setName("div");
+ // WICKET-6658 form is not allowed, anything else can stay as is
+ if ("form".equals(tag.getName()))
+ {
+ tag.setName("div");
+ }
tag.remove("method");
tag.remove("action");
tag.remove("enctype");
diff --git a/wicket-core/src/test/java/org/apache/wicket/markup/html/form/NestedFormsPage.html b/wicket-core/src/test/java/org/apache/wicket/markup/html/form/NestedFormsPage.html
index 826f9be..5b3ace3 100644
--- a/wicket-core/src/test/java/org/apache/wicket/markup/html/form/NestedFormsPage.html
+++ b/wicket-core/src/test/java/org/apache/wicket/markup/html/form/NestedFormsPage.html
@@ -42,13 +42,14 @@
<a wicket:id="toggle">[toggle]</a><br/>
<input type="button" wicket:id="submit"/>
- <form wicket:id="innerForm">
+ <!-- WICKET-6658 for nested forms any tag is allowed -->
+ <span wicket:id="innerForm">
<input type="text" wicket:id="first"/>
<input type="text" wicket:id="second"/><br/>
<a wicket:id="ajaxSubmit">submit via ajax</a><br/>
<a wicket:id="toggle">[toggle]</a><br/>
<input type="button" wicket:id="submit"/>
- </form>
+ </span>
</form>
</form>
</body>