aboutsummaryrefslogtreecommitdiffstats
path: root/catalog-be/src/main/java/org
diff options
context:
space:
mode:
Diffstat (limited to 'catalog-be/src/main/java/org')
-rw-r--r--catalog-be/src/main/java/org/openecomp/sdc/be/components/impl/ResourceImportManager.java13
1 files changed, 7 insertions, 6 deletions
diff --git a/catalog-be/src/main/java/org/openecomp/sdc/be/components/impl/ResourceImportManager.java b/catalog-be/src/main/java/org/openecomp/sdc/be/components/impl/ResourceImportManager.java
index 6b0b6bf81c..79543ce5cf 100644
--- a/catalog-be/src/main/java/org/openecomp/sdc/be/components/impl/ResourceImportManager.java
+++ b/catalog-be/src/main/java/org/openecomp/sdc/be/components/impl/ResourceImportManager.java
@@ -461,11 +461,11 @@ public class ResourceImportManager {
Boolean validateVsParentCap = validateCapNameVsDerived(reqName2TypeMap, requirementDef
.getCapability(), requirementDef.getName());
if (!validateVsParentCap) {
- log.debug("Requirement with name {} already exists in parent {}", requirementDef.getName(), parentResource
- .getName());
+ String parentResourceName = parentResource != null ? parentResource.getName() : "";
+ log.debug("Requirement with name {} already exists in parent {}", requirementDef.getName(), parentResourceName);
throw new ByActionStatusComponentException(ActionStatus.IMPORT_REQ_CAP_NAME_EXISTS_IN_DERIVED, "requirement", requirementDef
.getName()
- .toLowerCase(), parentResource.getName());
+ .toLowerCase(), parentResourceName);
}
}
if (moduleRequirements.size() > 0) {
@@ -623,11 +623,12 @@ public class ResourceImportManager {
if (!validateVsParentCap) {
// Here parentResource is for sure not null, so it's
// null-safe
- log.debug("Capability with name {} already exists in parent {}", capabilityDef.getName(), parentResource
- .getName());
+ // Check added to avoid sonar warning
+ String parentResourceName = parentResource != null ? parentResource.getName() : "";
+ log.debug("Capability with name {} already exists in parent {}", capabilityDef.getName(), parentResourceName);
throw new ByActionStatusComponentException(ActionStatus.IMPORT_REQ_CAP_NAME_EXISTS_IN_DERIVED, "capability", capabilityDef
.getName()
- .toLowerCase(), parentResource.getName());
+ .toLowerCase(), parentResourceName);
}
}
if (moduleCapabilities.size() > 0) {