You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by mg...@apache.org on 2014/02/02 10:22:37 UTC

[2/3] git commit: Fix #setPreventDefault and #isPreventDefault's javadoc.

Fix #setPreventDefault and #isPreventDefault's javadoc.

Add back #setAllowDefault and #isAllowDefault as deprecated for easier migration to 7.x


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

Branch: refs/heads/master
Commit: 7c2f79a6f2f84ee970729205f5e574c23340c4e1
Parents: c97f577
Author: Martin Tzvetanov Grigorov <mg...@apache.org>
Authored: Sun Feb 2 10:17:29 2014 +0100
Committer: Martin Tzvetanov Grigorov <mg...@apache.org>
Committed: Sun Feb 2 10:17:29 2014 +0100

----------------------------------------------------------------------
 .../ajax/attributes/AjaxRequestAttributes.java  | 54 ++++++++++++++------
 1 file changed, 37 insertions(+), 17 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/wicket/blob/7c2f79a6/wicket-core/src/main/java/org/apache/wicket/ajax/attributes/AjaxRequestAttributes.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/main/java/org/apache/wicket/ajax/attributes/AjaxRequestAttributes.java b/wicket-core/src/main/java/org/apache/wicket/ajax/attributes/AjaxRequestAttributes.java
index b8b4e2e..9eff742 100644
--- a/wicket-core/src/main/java/org/apache/wicket/ajax/attributes/AjaxRequestAttributes.java
+++ b/wicket-core/src/main/java/org/apache/wicket/ajax/attributes/AjaxRequestAttributes.java
@@ -289,14 +289,14 @@ public final class AjaxRequestAttributes
 	}
 
 	/**
-	 * Only applies for event behaviors. Returns whether the behavior should allow the default event
+	 * Only applies for event behaviors. Returns whether the behavior should prevent the default event
 	 * handler to be invoked. For example if the behavior is attached to a link and
-	 * {@link #isPreventDefault()} returns <code>false</code> (which is default value), the link's
+	 * isPreventDefault() returns <code>true</code>, the link's
 	 * URL will not be followed. If the Ajax behavior is attached to a checkbox or a radio button
 	 * then the default behavior should be allowed to actually check the box or radio button, i.e.
-	 * this method should return <code>true</code>.
+	 * this method should return <code>false</code>.
 	 * 
-	 * @return {@code true} if the default event handler should be invoked, {@code false} otherwise.
+	 * @return {@code false} if the default event handler should be invoked
 	 */
 	public boolean isPreventDefault()
 	{
@@ -304,20 +304,11 @@ public final class AjaxRequestAttributes
 	}
 
 	/**
-	 * Only applies for event behaviors. Returns whether the behavior should allow the JavaScript
-	 * event to propagate to the parent of its target.
-	 */
-	public EventPropagation getEventPropagation()
-	{
-		return eventPropagation;
-	}
-
-	/**
-	 * Only applies for event behaviors. Determines whether the behavior should allow the default
+	 * Only applies for event behaviors. Determines whether the behavior should prevent the default
 	 * event handler to be invoked.
-	 * 
+	 *
 	 * @see #isPreventDefault()
-	 * 
+	 *
 	 * @param preventDefault
 	 * @return {@code this} object for chaining
 	 * @see #isPreventDefault()
@@ -329,6 +320,35 @@ public final class AjaxRequestAttributes
 	}
 
 	/**
+	 * @return {@code this} object for chaining
+	 * @deprecated Use #setPreventDefault() instead
+	 */
+	@Deprecated
+	public AjaxRequestAttributes setAllowDefault(boolean allowDefault)
+	{
+		this.preventDefault = !allowDefault;
+		return this;
+	}
+
+	/**
+	 * @deprecated Use #isPreventDefault() instead
+	 */
+	@Deprecated
+	public boolean isAllowDefault()
+	{
+		return !preventDefault;
+	}
+
+	/**
+	 * Only applies for event behaviors. Returns whether the behavior should allow the JavaScript
+	 * event to propagate to the parent of its target.
+	 */
+	public EventPropagation getEventPropagation()
+	{
+		return eventPropagation;
+	}
+
+	/**
 	 * Only applies to event behaviors. Determines whether the behavior should allow the JavaScript
 	 * event to propagate to the parent of its target.
 	 * 
@@ -502,4 +522,4 @@ public final class AjaxRequestAttributes
 		return this;
 	}
 
-}
\ No newline at end of file
+}