<optional>false</optional> is no longer being written to pom.xml

forge · May 20, 2014 · 849143e · 849143e
1 parent 9a3a42a
commit 849143e
Showing 1 changed file with 12 additions and 3 deletions.
diff --git a/maven/api/src/main/java/org/jboss/forge/addon/maven/dependencies/MavenDependencyAdapter.java b/maven/api/src/main/java/org/jboss/forge/addon/maven/dependencies/MavenDependencyAdapter.java
@@ -37,7 +37,10 @@ public MavenDependencyAdapter(final org.apache.maven.model.Dependency dep)
       this.setGroupId(clone.getGroupId());
       this.setClassifier("".equals(clone.getClassifier()) ? null : clone.getClassifier());
       this.setExclusions(clone.getExclusions());
-      this.setOptional(clone.isOptional());
+      if (clone.isOptional())
+      {
+         this.setOptional(clone.isOptional());
+      }
       this.setScope(clone.getScope());
       this.setType(clone.getType());
       this.setVersion(clone.getVersion());
@@ -58,7 +61,10 @@ public MavenDependencyAdapter(final Dependency dep)
       this.setType(dep.getCoordinate().getPackaging());
       this.setClassifier(dep.getCoordinate().getClassifier());
       this.setSystemPath(dep.getCoordinate().getSystemPath());
-      this.setOptional(dep.isOptional());
+      if (dep.isOptional())
+      {
+         this.setOptional(dep.isOptional());
+      }
 
       if (dep.getExcludedCoordinates() != null)
       {
@@ -83,7 +89,10 @@ public MavenDependencyAdapter(final org.eclipse.aether.graph.Dependency dep)
       this.setGroupId(dep.getArtifact().getGroupId());
       this.setClassifier("".equals(dep.getArtifact().getClassifier()) ? null : dep.getArtifact().getClassifier());
       this.setExclusions(dep.getExclusions());
-      this.setOptional(dep.isOptional());
+      if (dep.isOptional())
+      {
+         this.setOptional(dep.isOptional());
+      }
       this.setScope(dep.getScope());
       this.setType(dep.getArtifact().getExtension());
       this.setVersion(dep.getArtifact().getBaseVersion());