Ensure create server updates plugin config xml after copying config f…

…iles
cherylking · Feb 13, 2024 · 1ce4996 · 1ce4996
1 parent 80fea34
commit 1ce4996
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/liberty-maven-plugin/src/main/java/io/openliberty/tools/maven/applications/DeployMojo.java b/liberty-maven-plugin/src/main/java/io/openliberty/tools/maven/applications/DeployMojo.java
@@ -72,7 +72,6 @@ private void doDeploy() throws IOException, MojoExecutionException, TransformerE
 
         // update target server configuration
         copyConfigFiles();
-        exportParametersToXml();
 
         boolean installDependencies = false;
         boolean installProject = false;

diff --git a/...rty-maven-plugin/src/main/java/io/openliberty/tools/maven/server/PluginConfigSupport.java b/...rty-maven-plugin/src/main/java/io/openliberty/tools/maven/server/PluginConfigSupport.java
@@ -98,6 +98,20 @@ protected String getDeployPackages() {
         return deployPackages;
     }
 
+    /**
+     * @throws IOException
+     * @throws MojoExecutionException
+     */
+    @Override
+    protected void copyConfigFiles() throws IOException, MojoExecutionException {
+        try {
+            super.copyConfigFiles();
+            exportParametersToXml();
+        } catch (IOException | ParserConfigurationException |  TransformerException e) {
+            throw new MojoExecutionException("Error copying configuration files to Liberty server directory.", e);            
+        }
+    }
+
     /*
      * Export plugin configuration parameters to
      * target/liberty-plugin-config.xml