Fix Sonar -ComponentDependencyModelDaoZusammenImpl 83/27283/2
authorAbhai Singh <Abhai.Singh@amdocs.com>
Wed, 3 Jan 2018 13:27:21 +0000 (18:57 +0530)
committerVitaly Emporopulo <Vitaliy.Emporopulo@amdocs.com>
Wed, 3 Jan 2018 14:34:13 +0000 (14:34 +0000)
Fixed reported sonar issue

Change-Id: I6c8b86cb54bcc6d21920e9ea806dcd40717c2383
Issue-ID: SDC-343
Signed-off-by: Abhai Singh <Abhai.Singh@amdocs.com>
openecomp-be/lib/openecomp-sdc-vendor-software-product-lib/openecomp-sdc-vendor-software-product-core/src/main/java/org/openecomp/sdc/vendorsoftwareproduct/dao/impl/zusammen/ComponentDependencyModelDaoZusammenImpl.java

index 6741d72..d98a67c 100644 (file)
@@ -10,8 +10,6 @@ import com.amdocs.zusammen.datatypes.item.ElementContext;
 import com.amdocs.zusammen.datatypes.item.Info;
 import org.openecomp.core.zusammen.api.ZusammenAdaptor;
 import org.openecomp.sdc.datatypes.model.ElementType;
-import org.openecomp.sdc.logging.api.Logger;
-import org.openecomp.sdc.logging.api.LoggerFactory;
 import org.openecomp.sdc.vendorsoftwareproduct.dao.ComponentDependencyModelDao;
 import org.openecomp.sdc.vendorsoftwareproduct.dao.impl.zusammen.convertor.ElementToComponentDependencyModelConvertor;
 import org.openecomp.sdc.vendorsoftwareproduct.dao.type.ComponentDependencyModelEntity;
@@ -31,9 +29,6 @@ import static org.openecomp.core.zusammen.api.ZusammenUtil.createSessionContext;
  */
 public class ComponentDependencyModelDaoZusammenImpl implements ComponentDependencyModelDao {
 
-  private static final Logger logger =
-      LoggerFactory.getLogger(OrchestrationTemplateCandidateDaoZusammenImpl.class);
-
   private ZusammenAdaptor zusammenAdaptor;
 
   public ComponentDependencyModelDaoZusammenImpl(ZusammenAdaptor zusammenAdaptor) {
@@ -151,11 +146,10 @@ public class ComponentDependencyModelDaoZusammenImpl implements ComponentDepende
 
     Info info = new Info();
     info.addProperty(ElementPropertyName.elementType.name(), ElementType.ComponentDependency);
-    //info.addProperty(ComponentDependencyModelPropertyName.id.name(), entity.getId());
-    info.addProperty(ComponentDependencyModelPropertyName.relation.name(), compDep.getRelation());
-    info.addProperty(ComponentDependencyModelPropertyName.sourcecomponent_id.name(),
+    info.addProperty(ComponentDependencyModelPropertyName.RELATION.getVal(), compDep.getRelation());
+    info.addProperty(ComponentDependencyModelPropertyName.SOURCE_COMPONENT_ID.getVal(),
         compDep.getSourceComponentId());
-    info.addProperty(ComponentDependencyModelPropertyName.targetcomponent_id.name(),
+    info.addProperty(ComponentDependencyModelPropertyName.TARGET_COMPONENT_ID.getVal(),
         compDep.getTargetComponentId());
 
     componentDependencyElement.setInfo(info);
@@ -164,9 +158,19 @@ public class ComponentDependencyModelDaoZusammenImpl implements ComponentDepende
   }
 
   private enum ComponentDependencyModelPropertyName {
-    id,
-    relation,
-    sourcecomponent_id,
-    targetcomponent_id,
+    ID("id"),
+    RELATION("relation"),
+    SOURCE_COMPONENT_ID("sourcecomponent_id"),
+    TARGET_COMPONENT_ID("targetcomponent_id");
+
+    private String val;
+
+    ComponentDependencyModelPropertyName(String val){
+      this.val=val;
+    }
+
+    public String getVal() {
+      return val;
+    }
   }
 }