Author: jdcasey
Date: Mon Nov 12 16:30:39 2007
New Revision: 594363

URL: http://svn.apache.org/viewvc?rev=594363&view=rev
Log:
Remove UnsupportedOperationExceptions, ignore invalid mutator methods so we 
will continue to derive information from the parent artifact where necessary 
and still work in the changing maven-artifact framework.

Modified:
    
maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/AttachedArtifact.java

Modified: 
maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/AttachedArtifact.java
URL: 
http://svn.apache.org/viewvc/maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/AttachedArtifact.java?rev=594363&r1=594362&r2=594363&view=diff
==============================================================================
--- 
maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/AttachedArtifact.java
 (original)
+++ 
maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/AttachedArtifact.java
 Mon Nov 12 16:30:39 2007
@@ -41,11 +41,11 @@
     {
         super( parent.getGroupId(), parent.getArtifactId(), 
parent.getVersionRange(), parent.getScope(), type,
                classifier, artifactHandler, parent.isOptional() );
-        
+
         setDependencyTrail( Collections.singletonList( parent.getId() ) );
-        
+
         this.parent = parent;
-        
+
         if ( getId().equals( parent.getId() ) )
         {
             throw new InvalidArtifactRTException( parent.getGroupId(), 
parent.getArtifactId(), parent.getVersion(), parent.getType(), "An attached 
artifact must have a different ID than its corresponding main artifact." );
@@ -56,102 +56,102 @@
     {
         this( parent, type, null, artifactHandler );
     }
-    
+
     public void setArtifactId( String artifactId )
     {
-        throw new UnsupportedOperationException( "Cannot change the artifactId 
for an attached artifact.  It is derived from the main artifact." );
+        // ignore this. We should ALWAYS use the information from the parent 
artifact.
     }
 
     public List getAvailableVersions()
     {
         return parent.getAvailableVersions();
     }
-    
+
     public void setAvailableVersions( List availableVersions )
     {
-        throw new UnsupportedOperationException( "Cannot change the version 
information for an attached artifact. It is derived from the main artifact." );
+        // ignore this. We should ALWAYS use the information from the parent 
artifact.
     }
 
     public String getBaseVersion()
     {
         return parent.getBaseVersion();
     }
-    
+
     public void setBaseVersion( String baseVersion )
     {
-        throw new UnsupportedOperationException( "Cannot change the version 
information for an attached artifact. It is derived from the main artifact." );
+        // ignore this. We should ALWAYS use the information from the parent 
artifact.
     }
 
     public String getDownloadUrl()
     {
         return parent.getDownloadUrl();
     }
-    
+
     public void setDownloadUrl( String downloadUrl )
     {
-        throw new UnsupportedOperationException( "Cannot change the download 
information for an attached artifact. It is derived from the main artifact." );
+        // ignore this. We should ALWAYS use the information from the parent 
artifact.
     }
 
     public void setGroupId( String groupId )
     {
-        throw new UnsupportedOperationException( "Cannot change the groupId on 
attached artifacts. It is derived from the main artifact." );
+        // ignore this. We should ALWAYS use the information from the parent 
artifact.
     }
 
     public ArtifactRepository getRepository()
     {
         return parent.getRepository();
     }
-    
+
     public void setRepository( ArtifactRepository repository )
     {
-        throw new UnsupportedOperationException( "Cannot change the repository 
information for an attached artifact. It is derived from the main artifact." );
+        // ignore this. We should ALWAYS use the information from the parent 
artifact.
     }
 
     public String getScope()
     {
         return parent.getScope();
     }
-    
+
     public void setScope( String scope )
     {
-        throw new UnsupportedOperationException( "Cannot change the scoping 
information for an attached artifact. It is derived from the main artifact." );
+        // ignore this. We should ALWAYS use the information from the parent 
artifact.
     }
 
     public String getVersion()
     {
         return parent.getVersion();
     }
-    
+
     public void setVersion( String version )
     {
-        throw new UnsupportedOperationException( "Cannot change the version 
information for an attached artifact. It is derived from the main artifact." );
+        // ignore this. We should ALWAYS use the information from the parent 
artifact.
     }
 
     public VersionRange getVersionRange()
     {
         return parent.getVersionRange();
     }
-    
+
     public void setVersionRange( VersionRange range )
     {
-        throw new UnsupportedOperationException( "Cannot change the version 
information for an attached artifact. It is derived from the main artifact." );
+        // ignore this. We should ALWAYS use the information from the parent 
artifact.
     }
 
     public boolean isRelease()
     {
         return parent.isRelease();
     }
-    
+
     public void setRelease( boolean release )
     {
-        throw new UnsupportedOperationException( "Cannot change the version 
information for an attached artifact. It is derived from the main artifact." );
+        // ignore this. We should ALWAYS use the information from the parent 
artifact.
     }
 
     public boolean isSnapshot()
     {
         return parent.isSnapshot();
     }
-    
+
     public void addMetadata( ArtifactMetadata metadata )
     {
         // ignore. The parent artifact will handle metadata.


Reply via email to