Fixed invalid artifact URL that is created when no packaging is speci…

…fied (which actually means default packaging 'jar').
jenkinsci · Jul 22, 2015 · 35d8239 · 35d8239
1 parent 3901c4d
commit 35d8239
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/main/lombok/eu/markov/jenkins/plugin/mvnmeta/MavenMetadataParameterDefinition.java b/src/main/lombok/eu/markov/jenkins/plugin/mvnmeta/MavenMetadataParameterDefinition.java
@@ -128,16 +128,16 @@ public Pattern getVersionFilterPattern() {
     return versionFilterPattern;
   }
 
-  public String readPackaging() {
+  public String getPackaging() {
     if (StringUtils.isNotBlank(this.packaging)) {
       return this.packaging;
     }
     return "jar";
   }
 
   private ParameterValue createValue(String version) {
-    return new MavenMetadataParameterValue(getName(), getDescription(), getGroupId(), getArtifactId(), version, readPackaging(),
-        getFullArtifactUrl(getGroupId(), getArtifactId(), version, readPackaging()));
+    return new MavenMetadataParameterValue(getName(), getDescription(), getGroupId(), getArtifactId(), version, getPackaging(),
+        getFullArtifactUrl(getGroupId(), getArtifactId(), version, getPackaging()));
   }
 
   // Create a parameter value from the string given in the CLI.
@@ -196,7 +196,7 @@ public ParameterValue getDefaultParameterValue() {
     }
 
     return new MavenMetadataParameterValue(getName(), getDescription(), getGroupId(), getArtifactId(), defaultVersion,
-        readPackaging(), getFullArtifactUrl(getGroupId(), getArtifactId(), defaultVersion, readPackaging()));
+        getPackaging(), getFullArtifactUrl(getGroupId(), getArtifactId(), defaultVersion, getPackaging()));
   }
 
   @Override