You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@karaf.apache.org by jb...@apache.org on 2017/04/13 11:16:10 UTC

karaf git commit: Expose Dependency.hasVersion() in feature model. This closes #282

Repository: karaf
Updated Branches:
  refs/heads/master 99ec8ddff -> 442b5024d


Expose Dependency.hasVersion() in feature model. This closes #282


Project: http://git-wip-us.apache.org/repos/asf/karaf/repo
Commit: http://git-wip-us.apache.org/repos/asf/karaf/commit/442b5024
Tree: http://git-wip-us.apache.org/repos/asf/karaf/tree/442b5024
Diff: http://git-wip-us.apache.org/repos/asf/karaf/diff/442b5024

Branch: refs/heads/master
Commit: 442b5024dab1f9050a969715457e58a60811f0d1
Parents: 99ec8dd
Author: Jean-Baptiste Onofr� <jb...@apache.org>
Authored: Thu Apr 13 13:15:25 2017 +0200
Committer: Jean-Baptiste Onofr� <jb...@apache.org>
Committed: Thu Apr 13 13:15:25 2017 +0200

----------------------------------------------------------------------
 .../main/java/org/apache/karaf/features/Dependency.java   |  2 ++
 .../apache/karaf/features/internal/model/Dependency.java  | 10 ++++++++++
 2 files changed, 12 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/karaf/blob/442b5024/features/core/src/main/java/org/apache/karaf/features/Dependency.java
----------------------------------------------------------------------
diff --git a/features/core/src/main/java/org/apache/karaf/features/Dependency.java b/features/core/src/main/java/org/apache/karaf/features/Dependency.java
index 13a4381..3f8fda8 100644
--- a/features/core/src/main/java/org/apache/karaf/features/Dependency.java
+++ b/features/core/src/main/java/org/apache/karaf/features/Dependency.java
@@ -23,6 +23,8 @@ public interface Dependency {
 
     String getVersion();
 
+    boolean hasVersion();
+
     boolean isPrerequisite();
 
     boolean isDependency();

http://git-wip-us.apache.org/repos/asf/karaf/blob/442b5024/features/core/src/main/java/org/apache/karaf/features/internal/model/Dependency.java
----------------------------------------------------------------------
diff --git a/features/core/src/main/java/org/apache/karaf/features/internal/model/Dependency.java b/features/core/src/main/java/org/apache/karaf/features/internal/model/Dependency.java
index 0963b3d..3d81425 100644
--- a/features/core/src/main/java/org/apache/karaf/features/internal/model/Dependency.java
+++ b/features/core/src/main/java/org/apache/karaf/features/internal/model/Dependency.java
@@ -65,6 +65,7 @@ public class Dependency implements org.apache.karaf.features.Dependency {
      * @return possible object is
      * {@link String }
      */
+    @Override
     public String getName() {
         return name;
     }
@@ -85,6 +86,7 @@ public class Dependency implements org.apache.karaf.features.Dependency {
      * @return possible object is
      * {@link String }
      */
+    @Override
     public String getVersion() {
         if (version == null) {
             return Feature.DEFAULT_VERSION;
@@ -103,6 +105,14 @@ public class Dependency implements org.apache.karaf.features.Dependency {
         this.version = value;
     }
 
+    /**
+     * Since version has a default value ("0.0.0"), returns whether or not the version has been set.
+     */
+    @Override
+    public boolean hasVersion() {
+        return version != null;
+    }
+
     @Override
     public boolean isPrerequisite() {
         return prerequisite;