svn commit: r530534 - /incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/ajax/AjaxRequestTarget.java

[iv...@apache.org]

Author: ivaynberg
Date: Thu Apr 19 13:05:46 2007
New Revision: 530534

URL: http://svn.apache.org/viewvc?view=rev&rev=530534
Log:
added argument guards

Modified:
    incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/ajax/AjaxRequestTarget.java

Modified: incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/ajax/AjaxRequestTarget.java
URL: http://svn.apache.org/viewvc/incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/ajax/AjaxRequestTarget.java?view=diff&rev=530534&r1=530533&r2=530534
==============================================================================
--- incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/ajax/AjaxRequestTarget.java (original)
+++ incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/ajax/AjaxRequestTarget.java Thu Apr 19 13:05:46 2007
@@ -294,6 +294,18 @@
 	 */
 	public final void addChildren(MarkupContainer parent, Class childCriteria)
 	{
+		if (parent == null)
+		{
+			throw new IllegalArgumentException("Argument `parent` cannot be null");
+		}
+		if (childCriteria == null)
+		{
+			throw new IllegalArgumentException(
+					"Argument `childCriteria` cannot be null. If you want to traverse all components use `"
+							+ Component.class.getName() + ".class` as the value for this argument");
+		}
+		
+		
 		parent.visitChildren(childCriteria, new Component.IVisitor()
 		{
 
@@ -368,7 +380,8 @@
 
 	/**
 	 * Sets the focus in the browser to the given component. The markup id must
-	 * be set. If the component is null the focus will not be set to any component.
+	 * be set. If the component is null the focus will not be set to any
+	 * component.
 	 * 
 	 * @param component
 	 *            The component to get the focus or null.