Author: evenisse
Date: Mon Apr 30 02:46:38 2007
New Revision: 533652

URL: http://svn.apache.org/viewvc?view=rev&rev=533652
Log:
[SCM-299] Fix message when developerConnection is missing.
Submitted by: Tom Parker

Modified:
    
maven/scm/trunk/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/AbstractScmMojo.java

Modified: 
maven/scm/trunk/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/AbstractScmMojo.java
URL: 
http://svn.apache.org/viewvc/maven/scm/trunk/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/AbstractScmMojo.java?view=diff&rev=533652&r1=533651&r2=533652
==============================================================================
--- 
maven/scm/trunk/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/AbstractScmMojo.java
 (original)
+++ 
maven/scm/trunk/maven-scm-plugin/src/main/java/org/apache/maven/scm/plugin/AbstractScmMojo.java
 Mon Apr 30 02:46:38 2007
@@ -157,7 +157,8 @@
 
     public String getConnectionUrl()
     {
-        if ( StringUtils.isNotEmpty( connectionUrl ) && "connection".equals( 
connectionType.toLowerCase() ) )
+        boolean requireDeveloperConnection = !"connection".equals( 
connectionType.toLowerCase() );
+        if ( StringUtils.isNotEmpty( connectionUrl ) && 
!requireDeveloperConnection )
         {
             return connectionUrl;
         }
@@ -165,8 +166,14 @@
         {
             return developerConnectionUrl;
         }
-
-        throw new NullPointerException( "You need to define a connectionUrl 
parameter" );
+        if ( requireDeveloperConnection )
+        {
+            throw new NullPointerException( "You need to define a 
developerConnectionUrl parameter" );
+        }
+        else
+        {
+            throw new NullPointerException( "You need to define a 
connectionUrl parameter" );
+        }
     }
 
     public void setConnectionUrl( String connectionUrl )


Reply via email to