svn commit: r778142 - /maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/plugin/MavenPluginValidator.java

[be...@apache.org]

Author: bentmann
Date: Sun May 24 13:17:47 2009
New Revision: 778142

URL: http://svn.apache.org/viewvc?rev=778142&view=rev
Log:
o Fixed plugin validator to consider only the first plugin descriptor

Modified:
    maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/plugin/MavenPluginValidator.java

Modified: maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/plugin/MavenPluginValidator.java
URL: http://svn.apache.org/viewvc/maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/plugin/MavenPluginValidator.java?rev=778142&r1=778141&r2=778142&view=diff
==============================================================================
--- maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/plugin/MavenPluginValidator.java (original)
+++ maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/plugin/MavenPluginValidator.java Sun May 24 13:17:47 2009
@@ -27,6 +27,8 @@
 
     private List<String> errors = new ArrayList<String>();
 
+    private boolean firstDescriptor = true;
+
     public MavenPluginValidator( Artifact pluginArtifact )
     {
         this.pluginArtifact = pluginArtifact;
@@ -34,6 +36,16 @@
 
     public void validate( PluginDescriptor pluginDescriptor )
     {
+        /*
+         * NOTE: For plugins that depend on other plugin artifacts the plugin realm contains more than one plugin
+         * descriptor. However, only the first descriptor is of interest.
+         */
+        if ( !firstDescriptor )
+        {
+            return;
+        }
+        firstDescriptor = false;
+
         if ( !pluginArtifact.getGroupId().equals( pluginDescriptor.getGroupId() ) )
         {
             errors.add( "Plugin's descriptor contains the wrong group ID: " + pluginDescriptor.getGroupId() );