From acd364bb34d1288e3999c5c6ccf5fa8fba752e22 Mon Sep 17 00:00:00 2001
From: vasraz <vasyl.razinkov@est.tech>
Date: Thu, 8 Jun 2023 15:39:40 +0100
Subject: Remove unused Neo4jErrorsConfiguration

Signed-off-by: Vasyl Razinkov <vasyl.razinkov@est.tech>
Change-Id: Ib9aa060057718144766c6061823bbae076647fca
Issue-ID: SDC-4532
---
 .../sdc/be/config/ConfigurationManager.java        |  9 +----
 .../sdc/be/config/Neo4jErrorsConfiguration.java    | 45 ----------------------
 .../sdc/common/impl/FSConfigurationSource.java     | 16 +++-----
 3 files changed, 7 insertions(+), 63 deletions(-)
 delete mode 100644 common-app-api/src/main/java/org/openecomp/sdc/be/config/Neo4jErrorsConfiguration.java

(limited to 'common-app-api/src/main')

diff --git a/common-app-api/src/main/java/org/openecomp/sdc/be/config/ConfigurationManager.java b/common-app-api/src/main/java/org/openecomp/sdc/be/config/ConfigurationManager.java
index 7241decd62..4ff8da7760 100644
--- a/common-app-api/src/main/java/org/openecomp/sdc/be/config/ConfigurationManager.java
+++ b/common-app-api/src/main/java/org/openecomp/sdc/be/config/ConfigurationManager.java
@@ -58,7 +58,6 @@ public class ConfigurationManager implements FileChangeCallback, IEcompConfigura
     private void loadConfigurationFiles() {
         loadConfigurationClass(Configuration.class);
         loadConfigurationClass(ErrorConfiguration.class);
-        loadConfigurationClass(Neo4jErrorsConfiguration.class);
         loadConfigurationClass(EcompErrorConfiguration.class);
         loadConfigurationClass(DistributionEngineConfiguration.class);
     }
@@ -78,8 +77,8 @@ public class ConfigurationManager implements FileChangeCallback, IEcompConfigura
         configurations.put(getKey(clazz), object);
     }
 
-    private <T> String getKey(Class<T> class1) {
-        return class1.getSimpleName();
+    private <T> String getKey(Class<T> clazz) {
+        return clazz.getSimpleName();
     }
 
     public Configuration getConfiguration() {
@@ -98,10 +97,6 @@ public class ConfigurationManager implements FileChangeCallback, IEcompConfigura
         configurations.put(getKey(ErrorConfiguration.class), configuration);
     }
 
-    public Neo4jErrorsConfiguration getNeo4jErrorsConfiguration() {
-        return (Neo4jErrorsConfiguration) configurations.get(getKey(Neo4jErrorsConfiguration.class));
-    }
-
     @Override
     public EcompErrorConfiguration getEcompErrorConfiguration() {
         return (EcompErrorConfiguration) configurations.get(getKey(EcompErrorConfiguration.class));
diff --git a/common-app-api/src/main/java/org/openecomp/sdc/be/config/Neo4jErrorsConfiguration.java b/common-app-api/src/main/java/org/openecomp/sdc/be/config/Neo4jErrorsConfiguration.java
deleted file mode 100644
index 835621fcf2..0000000000
--- a/common-app-api/src/main/java/org/openecomp/sdc/be/config/Neo4jErrorsConfiguration.java
+++ /dev/null
@@ -1,45 +0,0 @@
-/*-
- * ============LICENSE_START=======================================================
- * SDC
- * ================================================================================
- * Copyright (C) 2017 AT&T Intellectual Property. All rights reserved.
- * ================================================================================
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *      http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- * ============LICENSE_END=========================================================
- */
-package org.openecomp.sdc.be.config;
-
-import java.util.Map;
-import org.openecomp.sdc.common.api.BasicConfiguration;
-
-public class Neo4jErrorsConfiguration extends BasicConfiguration {
-
-    private Map<String, String> errors;
-
-    public Map<String, String> getErrors() {
-        return errors;
-    }
-
-    public void setErrors(Map<String, String> errors) {
-        this.errors = errors;
-    }
-
-    public String getErrorMessage(String key) {
-        return errors.get(key);
-    }
-
-    @Override
-    public String toString() {
-        return "Neo4jErrorsConfiguration [errors=" + errors + "]";
-    }
-}
diff --git a/common-app-api/src/main/java/org/openecomp/sdc/common/impl/FSConfigurationSource.java b/common-app-api/src/main/java/org/openecomp/sdc/common/impl/FSConfigurationSource.java
index cbb2de7dbc..0d1df424b7 100644
--- a/common-app-api/src/main/java/org/openecomp/sdc/common/impl/FSConfigurationSource.java
+++ b/common-app-api/src/main/java/org/openecomp/sdc/common/impl/FSConfigurationSource.java
@@ -23,6 +23,7 @@ package org.openecomp.sdc.common.impl;
 
 import java.util.Arrays;
 import java.util.stream.Collectors;
+import lombok.AllArgsConstructor;
 import org.openecomp.sdc.common.api.ConfigurationListener;
 import org.openecomp.sdc.common.api.ConfigurationSource;
 import org.openecomp.sdc.common.api.Constants;
@@ -35,18 +36,12 @@ import org.openecomp.sdc.exception.YamlConversionException;
  *
  * @author esofer
  */
+@AllArgsConstructor
 public class FSConfigurationSource implements ConfigurationSource {
 
-    private final YamlToObjectConverter yamlToObjectConverter = new YamlToObjectConverter();
     private final ConfigFileChangeListener changeListener;
     private final String appConfigDir;
 
-    public FSConfigurationSource(ConfigFileChangeListener changeListener, String appConfigDir) {
-        super();
-        this.changeListener = changeListener;
-        this.appConfigDir = appConfigDir;
-    }
-
     /**
      * convert camel case string to list of words separated by "-" where each word is in lower case format. For example, MyClass will be calculated to
      * be my-class.yaml .
@@ -71,7 +66,7 @@ public class FSConfigurationSource implements ConfigurationSource {
         final String configFileName = calculateFileName(className);
         T object;
         try {
-            object = yamlToObjectConverter.convert(this.appConfigDir, className, configFileName);
+            object = (new YamlToObjectConverter()).convert(this.appConfigDir, className, configFileName);
         } catch (final YamlConversionException e) {
             final String errorMsg = String.format("Could not load '%s' in '%s' for class '%s'", configFileName, appConfigDir, className);
             throw new LoadConfigurationException(errorMsg, e);
@@ -83,9 +78,8 @@ public class FSConfigurationSource implements ConfigurationSource {
     }
 
     public <T> void addWatchConfiguration(Class<T> className, ConfigurationListener configurationListener) {
-        String configFileName = calculateFileName(className);
-        if (configurationListener != null) {
-            changeListener.register(configFileName, configurationListener);
+        if (configurationListener != null && changeListener != null) {
+            changeListener.register(calculateFileName(className), configurationListener);
         }
     }
 }
-- 
cgit 1.2.3-korg