You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by da...@apache.org on 2008/08/08 10:39:43 UTC

svn commit: r683909 - /wicket/trunk/wicket/src/main/java/org/apache/wicket/feedback/FeedbackMessage.java

Author: dashorst
Date: Fri Aug  8 01:39:42 2008
New Revision: 683909

URL: http://svn.apache.org/viewvc?rev=683909&view=rev
Log:
WICKET-1780 NPE in feedback panel - added null check in constructor of feedback message to signal mistakes earlier.

Modified:
    wicket/trunk/wicket/src/main/java/org/apache/wicket/feedback/FeedbackMessage.java

Modified: wicket/trunk/wicket/src/main/java/org/apache/wicket/feedback/FeedbackMessage.java
URL: http://svn.apache.org/viewvc/wicket/trunk/wicket/src/main/java/org/apache/wicket/feedback/FeedbackMessage.java?rev=683909&r1=683908&r2=683909&view=diff
==============================================================================
--- wicket/trunk/wicket/src/main/java/org/apache/wicket/feedback/FeedbackMessage.java (original)
+++ wicket/trunk/wicket/src/main/java/org/apache/wicket/feedback/FeedbackMessage.java Fri Aug  8 01:39:42 2008
@@ -90,13 +90,15 @@
 	 * @param reporter
 	 *            The message reporter
 	 * @param message
-	 *            The actual message
+	 *            The actual message. Must not be <code>null</code>.
 	 * @param level
 	 *            The level of the message
 	 */
 	public FeedbackMessage(final Component reporter, final Serializable message,
 		final int level)
 	{
+		if(message == null)
+			throw new IllegalArgumentException("Parameter message can't be null");
 		this.reporter = reporter;
 		this.message = message;
 		this.level = level;