From: Julien HENRY <julien.henry@sonarsource.com>
Date: Tue, 30 Jul 2013 15:56:43 +0000 (+0200)
Subject: SONAR-4480 Improve email notification title of New Issues
X-Git-Tag: 3.7.1-RC1-~184
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=ef9ec75e79af80ce5d7de774ab4c7d29f6f82e72;p=sonarqube.git

SONAR-4480 Improve email notification title of New Issues
---

diff --git a/plugins/sonar-core-plugin/src/main/java/org/sonar/plugins/core/issue/notification/NewIssuesEmailTemplate.java b/plugins/sonar-core-plugin/src/main/java/org/sonar/plugins/core/issue/notification/NewIssuesEmailTemplate.java
index d479c508d1f..d8f96c2fb8d 100644
--- a/plugins/sonar-core-plugin/src/main/java/org/sonar/plugins/core/issue/notification/NewIssuesEmailTemplate.java
+++ b/plugins/sonar-core-plugin/src/main/java/org/sonar/plugins/core/issue/notification/NewIssuesEmailTemplate.java
@@ -57,7 +57,7 @@ public class NewIssuesEmailTemplate extends EmailTemplate {
 
     EmailMessage message = new EmailMessage()
       .setMessageId("new-issues/" + notification.getFieldValue("projectKey"))
-      .setSubject("Project " + projectName + ", new issues")
+      .setSubject(projectName + ": new issues")
       .setMessage(sb.toString());
 
     return message;
diff --git a/plugins/sonar-core-plugin/src/test/java/org/sonar/plugins/core/issue/notification/NewIssuesEmailTemplateTest.java b/plugins/sonar-core-plugin/src/test/java/org/sonar/plugins/core/issue/notification/NewIssuesEmailTemplateTest.java
index eb09ae9c7d9..aad5c762186 100644
--- a/plugins/sonar-core-plugin/src/test/java/org/sonar/plugins/core/issue/notification/NewIssuesEmailTemplateTest.java
+++ b/plugins/sonar-core-plugin/src/test/java/org/sonar/plugins/core/issue/notification/NewIssuesEmailTemplateTest.java
@@ -68,7 +68,7 @@ public class NewIssuesEmailTemplateTest {
 
     EmailMessage message = template.format(notification);
     assertThat(message.getMessageId()).isEqualTo("new-issues/org.apache:struts");
-    assertThat(message.getSubject()).isEqualTo("Project Struts, new issues");
+    assertThat(message.getSubject()).isEqualTo("Struts: new issues");
     assertThat(message.getMessage()).isEqualTo("" +
       "Project: Struts\n" +
       "32 new issues\n" +