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:53:21 UTC
svn commit: r683913 -
/wicket/branches/wicket-1.3.x/jdk-1.4/wicket/src/main/java/org/apache/wicket/feedback/FeedbackMessage.java
Author: dashorst
Date: Fri Aug 8 01:53:21 2008
New Revision: 683913
URL: http://svn.apache.org/viewvc?rev=683913&view=rev
Log:
WICKET-1780 NPE in feedback panel - added null check in constructor of feedback message to signal mistakes earlier.
Modified:
wicket/branches/wicket-1.3.x/jdk-1.4/wicket/src/main/java/org/apache/wicket/feedback/FeedbackMessage.java
Modified: wicket/branches/wicket-1.3.x/jdk-1.4/wicket/src/main/java/org/apache/wicket/feedback/FeedbackMessage.java
URL: http://svn.apache.org/viewvc/wicket/branches/wicket-1.3.x/jdk-1.4/wicket/src/main/java/org/apache/wicket/feedback/FeedbackMessage.java?rev=683913&r1=683912&r2=683913&view=diff
==============================================================================
--- wicket/branches/wicket-1.3.x/jdk-1.4/wicket/src/main/java/org/apache/wicket/feedback/FeedbackMessage.java (original)
+++ wicket/branches/wicket-1.3.x/jdk-1.4/wicket/src/main/java/org/apache/wicket/feedback/FeedbackMessage.java Fri Aug 8 01:53:21 2008
@@ -90,12 +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 not be null.");
+
this.reporter = reporter;
this.message = message;
this.level = level;