diff options
author | Arindam Mondal <arind.mondal@samsung.com> | 2019-09-10 15:02:53 +0900 |
---|---|---|
committer | Ofir Sonsino <ofir.sonsino@intl.att.com> | 2019-10-03 13:57:18 +0000 |
commit | b0f96a46e2720c75eb035bbe94015e5a1c6b0b32 (patch) | |
tree | 5505f6910d94281a4d858c320f2013bfbddc715b | |
parent | 0c2d872da4b39f3427e761e57c86c089882c6171 (diff) |
Sonar fix NullPointerException
Issue-ID: SDC-2542
Change-Id: I08a93932d2776c40a982bb802aeac994c1f35841
Signed-off-by: arindamm <arind.mondal@samsung.com>
3 files changed, 4 insertions, 6 deletions
diff --git a/catalog-dao/src/main/java/org/openecomp/sdc/be/dao/jsongraph/JanusGraphDao.java b/catalog-dao/src/main/java/org/openecomp/sdc/be/dao/jsongraph/JanusGraphDao.java index 5917b14fd3..7a8d1b7ca7 100644 --- a/catalog-dao/src/main/java/org/openecomp/sdc/be/dao/jsongraph/JanusGraphDao.java +++ b/catalog-dao/src/main/java/org/openecomp/sdc/be/dao/jsongraph/JanusGraphDao.java @@ -956,7 +956,7 @@ public class JanusGraphDao { toVertex.property(GraphPropertyEnum.UNIQUE_ID.getProperty())); result = JanusGraphOperationStatus.NOT_FOUND; } - if (result == null) { + if (result == null && org.apache.commons.collections4.IteratorUtils.isEmpty(prevEdgeIter)) { result = replaceEdgeLabel(fromVertex, toVertex, prevEdgeIter.next(), prevLabel, newLabel); } return result; diff --git a/catalog-model/src/main/java/org/openecomp/sdc/be/model/jsonjanusgraph/operations/ToscaElementLifecycleOperation.java b/catalog-model/src/main/java/org/openecomp/sdc/be/model/jsonjanusgraph/operations/ToscaElementLifecycleOperation.java index 771620d758..a0e6e7e12d 100644 --- a/catalog-model/src/main/java/org/openecomp/sdc/be/model/jsonjanusgraph/operations/ToscaElementLifecycleOperation.java +++ b/catalog-model/src/main/java/org/openecomp/sdc/be/model/jsonjanusgraph/operations/ToscaElementLifecycleOperation.java @@ -560,7 +560,7 @@ public class ToscaElementLifecycleOperation extends BaseOperation { result = StorageOperationStatus.NOT_FOUND; } } - if (result == null) { + if (result == null && foundEdge!=null) { JanusGraphOperationStatus createEdgeRes = janusGraphDao .createEdge(foundEdge.outVertex(), certifiedToscaElement.getVertex(), EdgeLabelEnum.LAST_STATE, foundEdge); diff --git a/common-be/src/main/java/org/openecomp/sdc/be/datatypes/components/ResourceMetadataDataDefinition.java b/common-be/src/main/java/org/openecomp/sdc/be/datatypes/components/ResourceMetadataDataDefinition.java index 3a48286280..7d9d475943 100644 --- a/common-be/src/main/java/org/openecomp/sdc/be/datatypes/components/ResourceMetadataDataDefinition.java +++ b/common-be/src/main/java/org/openecomp/sdc/be/datatypes/components/ResourceMetadataDataDefinition.java @@ -179,16 +179,14 @@ public class ResourceMetadataDataDefinition extends ComponentMetadataDataDefinit } else if (!vendorName.equals(other.vendorName)) { return false; } - if (vendorRelease == null) { - if (other.vendorRelease != null) { + if (vendorRelease == null && other.vendorRelease != null) { return false; - } } if (toscaResourceName == null) { if (other.toscaResourceName != null) { return false; } - } else if (!vendorRelease.equals(other.vendorRelease)) { + } else if (vendorRelease!=null && !vendorRelease.equals(other.vendorRelease)) { return false; } if (resourceVendorModelNumber == null) { |