This is an automated email from the ASF dual-hosted git repository.

elharo pushed a commit to branch clean
in repository https://gitbox.apache.org/repos/asf/maven-ear-plugin.git

commit a5bbd80c98ceb90fee7cde9ae04fb8551f13a81f
Author: Elliotte Rusty Harold <elh...@ibiblio.org>
AuthorDate: Mon Nov 18 09:08:37 2024 -0500

    Clean up a bit of wonky code
---
 src/main/java/org/apache/maven/plugins/ear/AbstractEarMojo.java | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/src/main/java/org/apache/maven/plugins/ear/AbstractEarMojo.java 
b/src/main/java/org/apache/maven/plugins/ear/AbstractEarMojo.java
index 9ed24ab..078f97c 100644
--- a/src/main/java/org/apache/maven/plugins/ear/AbstractEarMojo.java
+++ b/src/main/java/org/apache/maven/plugins/ear/AbstractEarMojo.java
@@ -110,6 +110,7 @@ public abstract class AbstractEarMojo extends AbstractMojo {
      *
      * @deprecated
      */
+    @Deprecated
     @Parameter
     private String fileNameMapping;
 
@@ -158,6 +159,7 @@ public abstract class AbstractEarMojo extends AbstractMojo {
     private JbossConfiguration jbossConfiguration;
 
     /** {@inheritDoc} */
+    @Override
     public void execute() throws MojoExecutionException, MojoFailureException {
         if (fileNameMapping != null) {
             getLog().error("fileNameMapping has been removed with version 
3.0.0. You are still using it.");
@@ -197,12 +199,9 @@ public abstract class AbstractEarMojo extends AbstractMojo 
{
         getLog().debug("Resolving ear modules ...");
         List<EarModule> allModules = new ArrayList<>();
         try {
-            if (modules != null && modules.length > 0) {
+            if (modules != null) {
                 // Let's validate user-defined modules
-                EarModule module;
-
-                for (EarModule module1 : modules) {
-                    module = module1;
+                for (EarModule module : modules) {
                     getLog().debug("Resolving ear module[" + module + "]");
                     module.setEarExecutionContext(earExecutionContext);
                     module.resolveArtifact(project.getArtifacts());

Reply via email to