NPE free

git-svn-id: https://svn.apache.org/repos/asf/archiva/trunk@1436973 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/api/DefaultJavascriptLogger.java b/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/api/DefaultJavascriptLogger.java
index 0594a5f189d490965bd07faddb7dffa78bda22d4..37a320a2fd36123d29b4d98a14b861b3291192b7 100644 (file)
--- a/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/api/DefaultJavascriptLogger.java
+++ b/archiva-modules/archiva-web/archiva-web-common/src/main/java/org/apache/archiva/web/api/DefaultJavascriptLogger.java
@@ -27,7 +27,7 @@ import org.springframework.stereotype.Service;
  * @author Olivier Lamy
  * @since 1.4-M4
  */
-@Service( "javascriptLogger#default" )
+@Service("javascriptLogger#default")
 public class DefaultJavascriptLogger
     implements JavascriptLogger
 {
@@ -37,7 +37,10 @@ public class DefaultJavascriptLogger
     {
         Logger toUse =
             javascriptLog.getLoggerName() == null ? logger : LoggerFactory.getLogger( javascriptLog.getLoggerName() );
-
+        if ( javascriptLog.getMessage() == null )
+        {
+            return Boolean.TRUE;
+        }
         toUse.trace( javascriptLog.getMessage() );
         return Boolean.TRUE;
     }
@@ -47,6 +50,11 @@ public class DefaultJavascriptLogger
         Logger toUse =
             javascriptLog.getLoggerName() == null ? logger : LoggerFactory.getLogger( javascriptLog.getLoggerName() );
 
+        if ( javascriptLog.getMessage() == null )
+        {
+            return Boolean.TRUE;
+        }
+
         toUse.debug( javascriptLog.getMessage() );
         return Boolean.TRUE;
     }
@@ -56,6 +64,11 @@ public class DefaultJavascriptLogger
         Logger toUse =
             javascriptLog.getLoggerName() == null ? logger : LoggerFactory.getLogger( javascriptLog.getLoggerName() );
 
+        if ( javascriptLog.getMessage() == null )
+        {
+            return Boolean.TRUE;
+        }
+
         toUse.info( javascriptLog.getMessage() );
         return Boolean.TRUE;
     }
@@ -65,6 +78,11 @@ public class DefaultJavascriptLogger
         Logger toUse =
             javascriptLog.getLoggerName() == null ? logger : LoggerFactory.getLogger( javascriptLog.getLoggerName() );
 
+        if ( javascriptLog.getMessage() == null )
+        {
+            return Boolean.TRUE;
+        }
+
         toUse.warn( javascriptLog.getMessage() );
         return Boolean.TRUE;
     }
@@ -74,6 +92,11 @@ public class DefaultJavascriptLogger
         Logger toUse =
             javascriptLog.getLoggerName() == null ? logger : LoggerFactory.getLogger( javascriptLog.getLoggerName() );
 
+        if ( javascriptLog.getMessage() == null )
+        {
+            return Boolean.TRUE;
+        }
+
         toUse.error( javascriptLog.getMessage() );
         return Boolean.TRUE;
     }