diff --git a/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/JavaAnnotationsMojoDescriptorExtractor.java b/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/JavaAnnotationsMojoDescriptorExtractor.java index 8b17b9c..d595438 100644 --- a/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/JavaAnnotationsMojoDescriptorExtractor.java +++ b/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/JavaAnnotationsMojoDescriptorExtractor.java @@ -186,9 +186,7 @@ public class JavaAnnotationsMojoDescriptorExtractor private boolean isMojoAnnnotatedClassCandidate( MojoAnnotatedClass mojoAnnotatedClass ) { - return mojoAnnotatedClass != null - && !( mojoAnnotatedClass.getComponents().isEmpty() && mojoAnnotatedClass.getParameters().isEmpty() - && mojoAnnotatedClass.getExecute() == null && mojoAnnotatedClass.getMojo() == null ); + return mojoAnnotatedClass != null && mojoAnnotatedClass.hasAnnotations(); } protected Map discoverClassesFromSourcesJar( Artifact artifact, PluginToolsRequest request, diff --git a/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/DefaultMojoAnnotationsScanner.java b/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/DefaultMojoAnnotationsScanner.java index d882a70..1be6561 100644 --- a/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/DefaultMojoAnnotationsScanner.java +++ b/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/DefaultMojoAnnotationsScanner.java @@ -212,7 +212,7 @@ public class DefaultMojoAnnotationsScanner if ( mojoAnnotatedClass != null ) // see MPLUGIN-206 we can have intermediate classes without annotations { - if ( getLogger().isDebugEnabled() && hasMojoAnnotations( mojoAnnotatedClass ) ) + if ( getLogger().isDebugEnabled() && mojoAnnotatedClass.hasAnnotations() ) { getLogger().debug( "found MojoAnnotatedClass:" + mojoAnnotatedClass.getClassName() + ":" + mojoAnnotatedClass ); @@ -222,12 +222,6 @@ public class DefaultMojoAnnotationsScanner } } - private boolean hasMojoAnnotations( MojoAnnotatedClass mojoAnnotatedClass ) - { - return !( mojoAnnotatedClass.getComponents().isEmpty() && mojoAnnotatedClass.getParameters().isEmpty() - && mojoAnnotatedClass.getExecute() == null && mojoAnnotatedClass.getMojo() == null ); - } - protected void populateAnnotationContent( Object content, MojoAnnotationVisitor mojoAnnotationVisitor ) throws ReflectorException { diff --git a/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/MojoAnnotatedClass.java b/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/MojoAnnotatedClass.java index 0f810ca..b775890 100644 --- a/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/MojoAnnotatedClass.java +++ b/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/MojoAnnotatedClass.java @@ -146,6 +146,11 @@ public class MojoAnnotatedClass this.artifact = artifact; } + public boolean hasAnnotations() + { + return !( getComponents().isEmpty() && getParameters().isEmpty() && execute == null && mojo == null ); + } + @Override public String toString() {