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 2015/10/06 00:46:19 UTC

[1/2] wicket git commit: WICKET-5993 change type attribute when submit only

Repository: wicket
Updated Branches:
  refs/heads/master 1febddd2e -> 796d9418d


WICKET-5993 change type attribute when submit only


Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/672f9f79
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/672f9f79
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/672f9f79

Branch: refs/heads/master
Commit: 672f9f792f0788c730df12bc1313ef835e8bfa9f
Parents: 1febddd
Author: Sven Meier <sv...@apache.org>
Authored: Tue Oct 6 00:30:05 2015 +0200
Committer: Sven Meier <sv...@apache.org>
Committed: Tue Oct 6 00:32:26 2015 +0200

----------------------------------------------------------------------
 .../java/org/apache/wicket/ajax/markup/html/form/AjaxButton.java | 4 +++-
 .../org/apache/wicket/ajax/markup/html/form/AjaxButtonTest.java  | 2 ++
 2 files changed, 5 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/wicket/blob/672f9f79/wicket-core/src/main/java/org/apache/wicket/ajax/markup/html/form/AjaxButton.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/main/java/org/apache/wicket/ajax/markup/html/form/AjaxButton.java b/wicket-core/src/main/java/org/apache/wicket/ajax/markup/html/form/AjaxButton.java
index caa7e39..85201f7 100644
--- a/wicket-core/src/main/java/org/apache/wicket/ajax/markup/html/form/AjaxButton.java
+++ b/wicket-core/src/main/java/org/apache/wicket/ajax/markup/html/form/AjaxButton.java
@@ -169,7 +169,9 @@ public abstract class AjaxButton extends Button
 	protected void onComponentTag(ComponentTag tag)
 	{
 		// WICKET-5594 prevent non-Ajax submit
-		tag.put("type", "button");
+		if ("submit".equals(tag.getAttribute("type"))) {
+			tag.put("type", "button");
+		}
 
 		super.onComponentTag(tag);
 	}

http://git-wip-us.apache.org/repos/asf/wicket/blob/672f9f79/wicket-core/src/test/java/org/apache/wicket/ajax/markup/html/form/AjaxButtonTest.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/test/java/org/apache/wicket/ajax/markup/html/form/AjaxButtonTest.java b/wicket-core/src/test/java/org/apache/wicket/ajax/markup/html/form/AjaxButtonTest.java
index 18bd35a..91ae973 100644
--- a/wicket-core/src/test/java/org/apache/wicket/ajax/markup/html/form/AjaxButtonTest.java
+++ b/wicket-core/src/test/java/org/apache/wicket/ajax/markup/html/form/AjaxButtonTest.java
@@ -34,6 +34,8 @@ public class AjaxButtonTest extends WicketTestCase
 		tester.startPage(HomePage.class);
 		// assert rendered page class
 		tester.assertRenderedPage(HomePage.class);
+		// assert button type unmodified, WICKET-5993W
+		assertEquals("image", tester.getTagById("cancel3").getAttribute("type"));
 		// assert button type for AjaxButtons, WICKET-5594
 		assertEquals("button", tester.getTagById("submit2").getAttribute("type"));
 		// assert rendered label component


[2/2] wicket git commit: WICKET-5993 change type attribute if submit only

Posted by sv...@apache.org.
WICKET-5993 change type attribute if submit only


Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/796d9418
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/796d9418
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/796d9418

Branch: refs/heads/master
Commit: 796d9418d336dc0b278260fd27331194ee4d8f83
Parents: 672f9f7
Author: Sven Meier <sv...@apache.org>
Authored: Tue Oct 6 00:41:04 2015 +0200
Committer: Sven Meier <sv...@apache.org>
Committed: Tue Oct 6 00:43:04 2015 +0200

----------------------------------------------------------------------
 .../java/org/apache/wicket/ajax/markup/html/form/HomePage.html     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/wicket/blob/796d9418/wicket-core/src/test/java/org/apache/wicket/ajax/markup/html/form/HomePage.html
----------------------------------------------------------------------
diff --git a/wicket-core/src/test/java/org/apache/wicket/ajax/markup/html/form/HomePage.html b/wicket-core/src/test/java/org/apache/wicket/ajax/markup/html/form/HomePage.html
index 6a7eb7e..cf581fb 100644
--- a/wicket-core/src/test/java/org/apache/wicket/ajax/markup/html/form/HomePage.html
+++ b/wicket-core/src/test/java/org/apache/wicket/ajax/markup/html/form/HomePage.html
@@ -8,7 +8,7 @@
         <span wicket:id="message">message will be here</span>
         
         <form wicket:id="form">
-          <input type="submit" wicket:id="cancel" value="Submit"/>
+          <input type="image" wicket:id="cancel" value="Submit"/>
           <input type="submit" wicket:id="submit" value="Submit"/>
         </form>