You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jspwiki.apache.org by ju...@apache.org on 2013/01/28 20:12:35 UTC

svn commit: r1439582 - /incubator/jspwiki/trunk/src/org/apache/wiki/WikiSession.java

Author: juanpablo
Date: Mon Jan 28 19:12:35 2013
New Revision: 1439582

URL: http://svn.apache.org/viewvc?rev=1439582&view=rev
Log:
simplified a bit addMessage(..) by using StringUtils.defaultString()

Modified:
    incubator/jspwiki/trunk/src/org/apache/wiki/WikiSession.java

Modified: incubator/jspwiki/trunk/src/org/apache/wiki/WikiSession.java
URL: http://svn.apache.org/viewvc/incubator/jspwiki/trunk/src/org/apache/wiki/WikiSession.java?rev=1439582&r1=1439581&r2=1439582&view=diff
==============================================================================
--- incubator/jspwiki/trunk/src/org/apache/wiki/WikiSession.java (original)
+++ incubator/jspwiki/trunk/src/org/apache/wiki/WikiSession.java Mon Jan 28 19:12:35 2013
@@ -27,6 +27,7 @@ import javax.security.auth.Subject;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpSession;
 
+import org.apache.commons.lang.StringUtils;
 import org.apache.log4j.Logger;
 
 import org.apache.wiki.auth.*;
@@ -286,17 +287,13 @@ public final class WikiSession implement
         {
             throw new IllegalArgumentException( "addMessage: topic cannot be null." );
         }
-        if ( message == null )
-        {
-            message = "";
-        }
         Set<String> messages = m_messages.get( topic );
         if (messages == null )
         {
             messages = new LinkedHashSet<String>();
             m_messages.put( topic, messages );
         }
-        messages.add( message );
+        messages.add( StringUtils.defaultString( message ) );
     }
 
     /**