The MessageFormat class must not be used to format check message when there isn't any parameter. Indeed, otherwise the message can't contain some patterns like "if(){".

1 files changed, 7 insertions, 3 deletions

diff --git a/sonar-squid/src/main/java/org/sonar/squid/api/CheckMessage.java b/sonar-squid/src/main/java/org/sonar/squid/api/CheckMessage.java
index a8ce8ed8c17..425b26af8be 100644
--- a/sonar-squid/src/main/java/org/sonar/squid/api/CheckMessage.java
+++ b/sonar-squid/src/main/java/org/sonar/squid/api/CheckMessage.java
@@ -73,11 +73,15 @@ public class CheckMessage implements Message {
 
   @Override
   public String toString() {
-    return new ToStringBuilder(this).append("source", sourceCode).append("check", codeCheck).append("msg", defaultMessage).append("line",
-        line).toString();
+    return new ToStringBuilder(this).append("source", sourceCode).append("check", codeCheck).append("msg", defaultMessage)
+        .append("line", line).toString();
   }
 
   public String formatDefaultMessage() {
-    return MessageFormat.format(defaultMessage, messageArguments);
+    if (messageArguments.length == 0) {
+      return defaultMessage;
+    } else {
+      return MessageFormat.format(defaultMessage, messageArguments);
+    }
   }
 }