Do not replace if literals are evaluated to null

Author: Fabrice Tiercelin

plugin/src/main/java/org/autorefactor/jdt/internal/ui/fix/LogParametersRatherThanLogMessageCleanUp.java

@@ -96,7 +96,7 @@ private boolean maybeReplaceConcatenation(final MethodInvocation visited, final
 				hasLiteral= true;
 				String literal= (String) string.resolveConstantExpressionValue();
 
-				if (literal != null && (literal.contains("{") || literal.contains("}"))) { //$NON-NLS-1$ //$NON-NLS-2$
+				if (literal == null || literal.contains("{") || literal.contains("}")) { //$NON-NLS-1$ //$NON-NLS-2$
 					return true;
 				}
 

samples/src/test/java/org/autorefactor/refactoring/rules/samples_in/LogParametersRatherThanLogMessageSample.java

@@ -25,6 +25,8 @@
  */
 package org.autorefactor.refactoring.rules.samples_in;
 
+import java.util.List;
+
 import org.slf4j.Logger;
 
 public class LogParametersRatherThanLogMessageSample {
@@ -93,4 +95,8 @@ public void doNotReplaceConcatenateSplitLiteral(Logger slf4jLog) {
         slf4jLog.debug("This text is very long and it can't be kept on a single line"
                 + " so we have to break it in order to keep readability.");
     }
+
+    public void doNotReplaceIfLiteralsAreEvaluatedToNull(Logger slf4jLog) {
+        slf4jLog.debug("aa" + "aa" + 1);
+    }
 }

samples/src/test/java/org/autorefactor/refactoring/rules/samples_out/LogParametersRatherThanLogMessageSample.java

@@ -25,6 +25,8 @@
  */
 package org.autorefactor.refactoring.rules.samples_out;
 
+import java.util.List;
+
 import org.slf4j.Logger;
 
 public class LogParametersRatherThanLogMessageSample {
@@ -93,4 +95,8 @@ public void doNotReplaceConcatenateSplitLiteral(Logger slf4jLog) {
         slf4jLog.debug("This text is very long and it can't be kept on a single line"
                 + " so we have to break it in order to keep readability.");
     }
+
+    public void doNotReplaceIfLiteralsAreEvaluatedToNull(Logger slf4jLog) {
+        slf4jLog.debug("aa" + "aa" + 1);
+    }
 }