Code Optimization.

Change config item 'msbServerAddr' to 'serviceAddr'.

Issue-ID: SDC-1079

Change-Id: I18275728ade2bc7016ea661b70a55f5550546ee7
Signed-off-by: YuanHu <yuan.hu1@zte.com.cn>

diff --git a/distribution/src/main/assembly/conf/workflow-designer.yml b/distribution/src/main/assembly/conf/workflow-designer.yml
index b824207..56d93f3 100644 (file)
--- a/distribution/src/main/assembly/conf/workflow-designer.yml
+++ b/distribution/src/main/assembly/conf/workflow-designer.yml
@@ -14,9 +14,8 @@ template: Hello, %s!
 
 defaultName: ${DW_DEFAULT_NAME:-Stranger}
 
-msbServerAddr: http://127.0.0.1:80
-
 sdcServiceProxy:
+  serviceAddr: http://127.0.0.1:8080
   xEcompInstanceId: workflow
   authorization: Basic YWFpOktwOGJKNFNYc3pNMFdYbGhhazNlSGxjc2UyZ0F3ODR2YW9HR21KdlV5MlU=
 

diff --git a/sdc-workflow-designer-server/src/main/java/org/onap/sdc/workflowdesigner/SDCServiceProxyInfo.java b/sdc-workflow-designer-server/src/main/java/org/onap/sdc/workflowdesigner/SDCServiceProxyInfo.java
index 5d8b8ea..a4853f7 100644 (file)
--- a/sdc-workflow-designer-server/src/main/java/org/onap/sdc/workflowdesigner/SDCServiceProxyInfo.java
+++ b/sdc-workflow-designer-server/src/main/java/org/onap/sdc/workflowdesigner/SDCServiceProxyInfo.java
@@ -15,10 +15,26 @@ package org.onap.sdc.workflowdesigner;
  *\r
  */\r
 public class SDCServiceProxyInfo {\r
+  private String serviceAddr;\r
+  \r
   private String xEcompInstanceId;\r
   \r
   private String authorization;\r
 \r
+  /**\r
+   * @return the serviceAddr\r
+   */\r
+  public String getServiceAddr() {\r
+    return serviceAddr;\r
+  }\r
+\r
+  /**\r
+   * @param serviceAddr the serviceAddr to set\r
+   */\r
+  public void setServiceAddr(String serviceAddr) {\r
+    this.serviceAddr = serviceAddr;\r
+  }\r
+\r
   /**\r
    * @return the xEcompInstanceId\r
    */\r

diff --git a/sdc-workflow-designer-server/src/main/java/org/onap/sdc/workflowdesigner/WorkflowDesignerApp.java b/sdc-workflow-designer-server/src/main/java/org/onap/sdc/workflowdesigner/WorkflowDesignerApp.java
index ba0fcd7..8e8d9a1 100644 (file)
--- a/sdc-workflow-designer-server/src/main/java/org/onap/sdc/workflowdesigner/WorkflowDesignerApp.java
+++ b/sdc-workflow-designer-server/src/main/java/org/onap/sdc/workflowdesigner/WorkflowDesignerApp.java
@@ -53,7 +53,6 @@ public class WorkflowDesignerApp extends Application<WorkflowDesignerConfigurati
   public void run(WorkflowDesignerConfiguration configuration, Environment environment) {
     LOGGER.info("Start to initialize Workflow Designer.");
     
-    AppConfig.setMsbServerAddr(configuration.getMsbServerAddr());
     AppConfig.setSdcServiceProxy(configuration.getSdcServiceProxy());
 
     environment.jersey().register(new WorkflowModelerResource());

diff --git a/sdc-workflow-designer-server/src/main/java/org/onap/sdc/workflowdesigner/WorkflowDesignerConfiguration.java b/sdc-workflow-designer-server/src/main/java/org/onap/sdc/workflowdesigner/WorkflowDesignerConfiguration.java
index dfdffcf..6ce2a1e 100644 (file)
--- a/sdc-workflow-designer-server/src/main/java/org/onap/sdc/workflowdesigner/WorkflowDesignerConfiguration.java
+++ b/sdc-workflow-designer-server/src/main/java/org/onap/sdc/workflowdesigner/WorkflowDesignerConfiguration.java
@@ -25,9 +25,6 @@ public class WorkflowDesignerConfiguration extends Configuration {
   @NotEmpty
   private String defaultName = "Workflow Designer";
   
-  @NotEmpty
-  private String msbServerAddr;
-  
   @NotEmpty
   private SDCServiceProxyInfo sdcServiceProxy;
 
@@ -50,16 +47,6 @@ public class WorkflowDesignerConfiguration extends Configuration {
   public void setDefaultName(String name) {
     this.defaultName = name;
   }
-  
-  @JsonProperty
-  public String getMsbServerAddr() {
-      return msbServerAddr;
-  }
-
-  @JsonProperty
-  public void setMsbServerAddr(String msbServerAddr) {
-      this.msbServerAddr = msbServerAddr;
-  }
 
   /**
    * @return the sdcServiceProxy

diff --git a/sdc-workflow-designer-server/src/main/java/org/onap/sdc/workflowdesigner/config/AppConfig.java b/sdc-workflow-designer-server/src/main/java/org/onap/sdc/workflowdesigner/config/AppConfig.java
index f9aa060..c617c1f 100644 (file)
--- a/sdc-workflow-designer-server/src/main/java/org/onap/sdc/workflowdesigner/config/AppConfig.java
+++ b/sdc-workflow-designer-server/src/main/java/org/onap/sdc/workflowdesigner/config/AppConfig.java
@@ -21,23 +21,10 @@ import org.onap.sdc.workflowdesigner.SDCServiceProxyInfo;
  * 
  */
 public class AppConfig {
-  private static String msbServerAddr;
   private static SDCServiceProxyInfo sdcServiceProxy;
 
   private AppConfig() {}
 
-  public static String getMsbServerAddr() {
-    return msbServerAddr;
-  }
-
-  public static void setMsbServerAddr(String msbServerAddr) {
-    AppConfig.msbServerAddr = msbServerAddr;
-  }
-
-  public static String getSDCAddr() {
-    return msbServerAddr + "/api/sdc/v1";
-  }
-
   /**
    * @param sdcServiceProxy
    */

diff --git a/sdc-workflow-designer-server/src/main/java/org/onap/sdc/workflowdesigner/externalservice/sdc/SDCServiceProxy.java b/sdc-workflow-designer-server/src/main/java/org/onap/sdc/workflowdesigner/externalservice/sdc/SDCServiceProxy.java
index c9c0f81..6319cbf 100644 (file)
--- a/sdc-workflow-designer-server/src/main/java/org/onap/sdc/workflowdesigner/externalservice/sdc/SDCServiceProxy.java
+++ b/sdc-workflow-designer-server/src/main/java/org/onap/sdc/workflowdesigner/externalservice/sdc/SDCServiceProxy.java
@@ -30,13 +30,24 @@ import com.eclipsesource.jaxrs.consumer.ConsumerFactory;
 public class SDCServiceProxy {
   private static final Logger LOGGER = LoggerFactory.getLogger(SDCService.class);
 
+  private static final String AUTHORIZATION = AppConfig.getSdcServiceProxy().getAuthorization();
+
+  private static final String X_ECOMP_INSTANCE_ID = AppConfig.getSdcServiceProxy().getxEcompInstanceId();
+  /** */
+  private static final String SDC_ROOT_PATH = "/sdc/v1";
+  
+
+  private static String getSDCRootPath() {
+    return AppConfig.getSdcServiceProxy().getServiceAddr() + SDC_ROOT_PATH;
+  }
+  
   /**
    * @return
    */
   private SDCService getSDCServiceProxy() {
     ClientConfig config = new ClientConfig();
     SDCService sdcServiceProxy =
-        ConsumerFactory.createConsumer(AppConfig.getSDCAddr(), config, SDCService.class);
+        ConsumerFactory.createConsumer(getSDCRootPath(), config, SDCService.class);
     return sdcServiceProxy;
   }
 
@@ -53,8 +64,8 @@ public class SDCServiceProxy {
     SDCService sdcServiceProxy = getSDCServiceProxy();
     try {
       sdcServiceProxy.saveWorkflowArtifact(uuid, operationId, workflowId,
-          AppConfig.getSdcServiceProxy().getxEcompInstanceId(),
-          AppConfig.getSdcServiceProxy().getAuthorization(), workflowArtifactInfo);
+          X_ECOMP_INSTANCE_ID,
+          AUTHORIZATION, workflowArtifactInfo);
     } catch (Exception e) {
       LOGGER.error("Save WorkflowArtifact Failed.", e);
       throw new WorkflowDesignerException("Save WorkflowArtifact Failed.", e);
@@ -74,8 +85,8 @@ public class SDCServiceProxy {
     SDCService sdcServiceProxy = getSDCServiceProxy();
     try {
       return sdcServiceProxy.getWorkflowArtifact(uuid, operationId, workflowId,
-          AppConfig.getSdcServiceProxy().getxEcompInstanceId(),
-          AppConfig.getSdcServiceProxy().getAuthorization());
+          X_ECOMP_INSTANCE_ID,
+          AUTHORIZATION);
     } catch (Exception e) {
       LOGGER.error("Get WorkflowArtifact Failed.", e);
       throw new WorkflowDesignerException("Save WorkflowArtifact Failed.", e);