You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@maven.apache.org by vs...@apache.org on 2009/02/27 12:22:04 UTC

svn commit: r748487 - /maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/LicenseReport.java

Author: vsiveton
Date: Fri Feb 27 11:22:04 2009
New Revision: 748487

URL: http://svn.apache.org/viewvc?rev=748487&view=rev
Log:
MPIR-135: NPE if license URL is not defined

o applied Benno's comment

Modified:
    maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/LicenseReport.java

Modified: maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/LicenseReport.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/LicenseReport.java?rev=748487&r1=748486&r2=748487&view=diff
==============================================================================
--- maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/LicenseReport.java (original)
+++ maven/plugins/trunk/maven-project-info-reports-plugin/src/main/java/org/apache/maven/report/projectinfo/LicenseReport.java Fri Feb 27 11:22:04 2009
@@ -153,7 +153,7 @@
         // URLs do not contain a valid authority (no hostname).
         // As a workaround accept license URLs that start with the
         // file scheme.
-        if ( urlValidator.isValid( url ) || url.startsWith( "file://" ) )
+        if ( urlValidator.isValid( url ) || StringUtils.defaultString( url ).startsWith( "file://" ) )
         {
             try
             {