[Policy-52, Policy-92, Policy-93] Policy Enhancements and bugfixes

[policy/engine.git] / ECOMP-ControlloopPolicy / src / main / java / org / openecomp / policy / controlloop / policy / guard / GuardPolicy.java

diff --git a/ECOMP-ControlloopPolicy/src/main/java/org/openecomp/policy/controlloop/policy/guard/GuardPolicy.java b/ECOMP-ControlloopPolicy/src/main/java/org/openecomp/policy/controlloop/policy/guard/GuardPolicy.java
index 0195fac..830736b 100644 (file)
--- a/ECOMP-ControlloopPolicy/src/main/java/org/openecomp/policy/controlloop/policy/guard/GuardPolicy.java
+++ b/ECOMP-ControlloopPolicy/src/main/java/org/openecomp/policy/controlloop/policy/guard/GuardPolicy.java
@@ -26,47 +26,85 @@ import java.util.UUID;
 
 public class GuardPolicy {
 
-       public String id = UUID.randomUUID().toString();
-       public String name;
-       public String description;
-       public String actor;
-       public String recipe;
-       public LinkedList<Constraint> limit_constraints;
+       private String id = UUID.randomUUID().toString();
+       private String name;
+       private String description;
+       private MatchParameters match_parameters;
+       private LinkedList<Constraint> limit_constraints;
        
        public GuardPolicy() {
-               
+               //Do Nothing Empty Constructor. 
        }
        
+       public String getId() {
+               return id;
+       }
+
+       public void setId(String id) {
+               this.id = id;
+       }
+
+       public String getName() {
+               return name;
+       }
+
+       public void setName(String name) {
+               this.name = name;
+       }
+
+       public String getDescription() {
+               return description;
+       }
+
+       public void setDescription(String description) {
+               this.description = description;
+       }
+
+       public MatchParameters getMatch_parameters() {
+               return match_parameters;
+       }
+
+       public void setMatch_parameters(MatchParameters match_parameters) {
+               this.match_parameters = match_parameters;
+       }
+
+       public LinkedList<Constraint> getLimit_constraints() {
+               return  limit_constraints;
+       }
+
+       public void setLimit_constraints(LinkedList<Constraint> limit_constraints) {
+               this.limit_constraints = limit_constraints;
+       }
+
        public GuardPolicy(String id) {
                this.id = id;
        }
        
-       public GuardPolicy(String name, String actor, String recipe) {
+       public GuardPolicy(String name, MatchParameters matchParameters) {
                this.name = name;
-               this.actor = actor;
-               this.recipe = recipe;
+               this.match_parameters = matchParameters;
        }
        
-       public GuardPolicy(String id, String name, String description, String actor, String recipe) {
-               this(name, actor, recipe);
+       public GuardPolicy(String id, String name, String description, MatchParameters matchParameters) {
+               this(name, matchParameters);
                this.id = id;
                this.description = description;
        }
        
-       public GuardPolicy(String name, String actor, String recipe, List<Constraint> limit_constraints) {
-               this(name, actor, recipe);
+       public GuardPolicy(String name, MatchParameters matchParameters, List<Constraint> limitConstraints) {
+               this(name, matchParameters);
                if (limit_constraints != null) {
-                       this.limit_constraints = (LinkedList<Constraint>) Collections.unmodifiableList(limit_constraints);
+                       this.limit_constraints = (LinkedList<Constraint>) Collections.unmodifiableList(limitConstraints);
                }
        }
        
-       public GuardPolicy(String name, String description, String actor, String recipe, List<Constraint> limit_constraints) {
-               this(name, actor, recipe, limit_constraints);
+       public GuardPolicy(String name, String description, MatchParameters matchParameters, List<Constraint> limitConstraints) {
+               this(name, matchParameters, limitConstraints);
                this.description = description;
        }
        
-       public GuardPolicy(String id, String name, String description, String actor, String recipe, List<Constraint> limit_constraints) {
-               this(name, description, actor, recipe, limit_constraints);
+       public GuardPolicy(String id, String name, String description, MatchParameters matchParameters, List<Constraint> limitConstraints) {
+               this(name, description, matchParameters, limitConstraints);
                this.id = id;
        }
        
@@ -74,49 +112,31 @@ public class GuardPolicy {
                this.id = policy.id;
                this.name = policy.name;
                this.description = policy.description;
-               this.actor = policy.actor;
-               this.recipe = policy.recipe;
+               this.match_parameters = new MatchParameters(policy.match_parameters);
                if (policy.limit_constraints != null) {
                        this.limit_constraints = (LinkedList<Constraint>) Collections.unmodifiableList(policy.limit_constraints);
                }
        }
        
        public boolean isValid() {
-               try {
-                       if (id == null) {
-                               throw new NullPointerException();
-                       }
-                       if (name == null) {
-                               throw new NullPointerException();
-                       }
-                       if (actor == null) {
-                               throw new NullPointerException();
-                       }
-                       if (recipe == null) {
-                               throw new NullPointerException();
-                       }
-               } catch (Exception e) {
-                       return false;
-               }
-               return true;
+               return (id==null || name ==null)? false : true;
        }
        
        @Override
        public String toString() {
-               return "Policy [id=" + id + ", name=" + name + ", description=" + description + ", actor=" + actor + ", recipe="
-                               + recipe + ", limit_constraints=" + limit_constraints + "]";
+               return "Policy [id=" + id + ", name=" + name + ", description=" + description + ", match_parameters=" 
+                               +match_parameters + ", limitConstraints=" + limit_constraints + "]";
        }
 
        @Override
        public int hashCode() {
                final int prime = 31;
                int result = 1;
-               result = prime * result + ((actor == null) ? 0 : actor.hashCode());
                result = prime * result + ((description == null) ? 0 : description.hashCode());
                result = prime * result + ((id == null) ? 0 : id.hashCode());
                result = prime * result + ((name == null) ? 0 : name.hashCode());
                result = prime * result + ((limit_constraints == null) ? 0 : limit_constraints.hashCode());
-               result = prime * result + ((recipe == null) ? 0 : recipe.hashCode());
+               result = prime * result + ((match_parameters == null) ? 0 : match_parameters.hashCode());
                return result;
        }
 
@@ -129,11 +149,6 @@ public class GuardPolicy {
                if (getClass() != obj.getClass())
                        return false;
                GuardPolicy other = (GuardPolicy) obj;
-               if (actor == null) {
-                       if (other.actor != null) 
-                               return false;
-               } else if (!actor.equals(other.actor))
-                       return false;
                if (description == null) {
                        if (other.description != null)
                                return false;
@@ -154,10 +169,10 @@ public class GuardPolicy {
                                return false;
                } else if (!limit_constraints.equals(other.limit_constraints))
                        return false;
-               if (recipe == null) {
-                       if (other.recipe != null)
+               if (match_parameters==null){
+                       if(other.match_parameters !=null)
                                return false;
-               } else if (!recipe.equals(other.recipe))
+               } else if(!match_parameters.equals(other.match_parameters))
                        return false;
                return true;
        }