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 ) );
}
/**