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 )