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 5c3336eb..33518e77 100644 --- a/src/main/java/org/apache/maven/report/projectinfo/MailingListsReport.java +++ b/src/main/java/org/apache/maven/report/projectinfo/MailingListsReport.java @@ -157,16 +157,43 @@ public void renderBody() { List textRow = new ArrayList<>(); - // Validate here subsribe/unsubsribe lists and archives? - textRow.add( mailingList.getName() ); + if ( StringUtils.isNotEmpty( mailingList.getName() ) ) + { + textRow.add( mailingList.getName() ); + } + else + { + textRow.add( "-" ); + } - textRow.add( createEmailLinkPatternedText( subscribe, mailingList.getSubscribe(), null ) ); + if ( StringUtils.isNotEmpty( mailingList.getSubscribe() ) ) + { + textRow.add( createEmailLinkPatternedText( subscribe, mailingList.getSubscribe(), null ) ); + } + else + { + textRow.add( "-" ); + } - textRow.add( createEmailLinkPatternedText( unsubscribe, mailingList.getUnsubscribe(), null ) ); + if ( StringUtils.isNotEmpty( mailingList.getUnsubscribe() ) ) + { + textRow.add( createEmailLinkPatternedText( unsubscribe, mailingList.getUnsubscribe(), null ) ); + } + else + { + textRow.add( "-" ); + } - textRow.add( createEmailLinkPatternedText( post, mailingList.getPost(), "-" ) ); + if ( StringUtils.isNotEmpty( mailingList.getPost() ) ) + { + textRow.add( createEmailLinkPatternedText( post, mailingList.getPost(), null ) ); + } + else + { + textRow.add( "-" ); + } - if ( mailingList.getArchive() != null && mailingList.getArchive().length() > 0 ) + if ( mailingList.getArchive() != null && !mailingList.getArchive().isEmpty() ) { textRow.add( createLinkPatternedText( ProjectInfoReportUtils.getArchiveServer( mailingList.getArchive() ),