You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by sl...@apache.org on 2020/01/17 21:53:00 UTC

[maven-project-info-reports-plugin] 02/02: [MPIR-385] Avoid double adding mailto: prefix

This is an automated email from the ASF dual-hosted git repository.

slachiewicz pushed a commit to branch MPIR-385
in repository https://gitbox.apache.org/repos/asf/maven-project-info-reports-plugin.git

commit 2b11b844e60806cefc5aff79bda88b9d8f5a1e9a
Author: Sylwester Lachiewicz <sl...@apache.org>
AuthorDate: Fri Jan 17 22:52:11 2020 +0100

    [MPIR-385] Avoid double adding mailto: prefix
---
 .../java/org/apache/maven/report/projectinfo/MailingListsReport.java    | 2 +-
 src/test/resources/plugin-configs/mailing-lists-plugin-config.xml       | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/main/java/org/apache/maven/report/projectinfo/MailingListsReport.java b/src/main/java/org/apache/maven/report/projectinfo/MailingListsReport.java
index 39b415f..fd0c8df 100644
--- a/src/main/java/org/apache/maven/report/projectinfo/MailingListsReport.java
+++ b/src/main/java/org/apache/maven/report/projectinfo/MailingListsReport.java
@@ -247,7 +247,7 @@ public class MailingListsReport
             {
                 return createLinkPatternedText( text, defaultHref );
             }
-            return createLinkPatternedText( text, "mailto:" + href );
+            return createLinkPatternedText( text, href.startsWith( "mailto:" ) ? href : "mailto:" + href );
         }
 
         /**
diff --git a/src/test/resources/plugin-configs/mailing-lists-plugin-config.xml b/src/test/resources/plugin-configs/mailing-lists-plugin-config.xml
index 7d833c6..9228396 100644
--- a/src/test/resources/plugin-configs/mailing-lists-plugin-config.xml
+++ b/src/test/resources/plugin-configs/mailing-lists-plugin-config.xml
@@ -36,7 +36,7 @@ under the License.
     <mailingList>
       <name>Test List</name>
       <post>test@maven.apache.org</post>
-      <subscribe>test-subscribe@maven.apache.org</subscribe>
+      <subscribe>mailto:test-subscribe@maven.apache.org</subscribe>
     </mailingList>
   </mailingLists>
   <build>