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

[si...@apache.org]

Author: sisbell
Date: Tue Apr 28 21:52:43 2009
New Revision: 769560

URL: http://svn.apache.org/viewvc?rev=769560&view=rev
Log:
Verify that plugin has version as precondition in PluginManager.

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

Modified: maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java
URL: http://svn.apache.org/viewvc/maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java?rev=769560&r1=769559&r2=769560&view=diff
==============================================================================
--- maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java (original)
+++ maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java Tue Apr 28 21:52:43 2009
@@ -47,6 +47,7 @@
 import org.apache.maven.artifact.versioning.DefaultArtifactVersion;
 import org.apache.maven.execution.MavenSession;
 import org.apache.maven.execution.RuntimeInformation;
+import org.apache.maven.lifecycle.LifecycleExecutionException;
 import org.apache.maven.model.Dependency;
 import org.apache.maven.model.Plugin;
 import org.apache.maven.model.PluginExecution;
@@ -1024,6 +1025,11 @@
     public MojoDescriptor getMojoDescriptor( Plugin plugin, String goal, MavenSession session )
         throws PluginLoaderException
     {
+        if ( plugin.getVersion() == null )
+        {
+        	throw new IllegalArgumentException("plugin.version: null");
+        }
+        
         PluginDescriptor pluginDescriptor =  loadPlugin( plugin, session.getCurrentProject(), session );
 
         MojoDescriptor mojoDescriptor = pluginDescriptor.getMojo( goal );