Author: evenisse
Date: Tue Apr 24 05:58:01 2007
New Revision: 531916

URL: http://svn.apache.org/viewvc?view=rev&rev=531916
Log:
[MRELEASE-147] Allow ${project.groupId} in dependencies groupId

Modified:
    
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/AbstractRewritePomsPhase.java
    
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-with-parent-and-properties/subproject2/expected-pom.xml
    
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-with-parent-and-properties/subproject2/pom.xml

Modified: 
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/AbstractRewritePomsPhase.java
URL: 
http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/AbstractRewritePomsPhase.java?view=diff&rev=531916&r1=531915&r2=531916
==============================================================================
--- 
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/AbstractRewritePomsPhase.java
 (original)
+++ 
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/AbstractRewritePomsPhase.java
 Tue Apr 24 05:58:01 2007
@@ -423,7 +423,20 @@
                 "' and artifactId='" + artifactId + "']" );
         }
 
-        return (Element) xpath.selectSingleNode( dependencyRoot );
+        Element elem = (Element) xpath.selectSingleNode( dependencyRoot );
+
+        //MRELEASE-147
+        if ( elem == null && groupId.indexOf( "${" ) == -1 )
+        {
+            elem = getDependency( "${project.groupId}", artifactId, 
groupTagName, tagName, dependencyRoot );
+
+            if ( elem == null )
+            {
+                elem = getDependency( "${pom.groupId}", artifactId, 
groupTagName, tagName, dependencyRoot );
+            }
+        }
+
+        return elem;
     }
 
     private void updateDomVersion( String groupId, String artifactId, Map 
mappedVersions,
@@ -437,7 +450,7 @@
         String resolvedSnapshotVersion = getResolvedSnapshotVersion( key, 
resolvedSnapshotDepedencies );
         Object originalVersion = originalVersions.get( key );
 
-        // workaround 
+        // workaround
         if ( originalVersion == null )
         {
             originalVersion = getOriginalResolvedSnapshotVersion( key, 
resolvedSnapshotDepedencies );

Modified: 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-with-parent-and-properties/subproject2/expected-pom.xml
URL: 
http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-with-parent-and-properties/subproject2/expected-pom.xml?view=diff&rev=531916&r1=531915&r2=531916
==============================================================================
--- 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-with-parent-and-properties/subproject2/expected-pom.xml
 (original)
+++ 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-with-parent-and-properties/subproject2/expected-pom.xml
 Tue Apr 24 05:58:01 2007
@@ -29,7 +29,7 @@
 
   <dependencies>
     <dependency>
-      <groupId>groupId</groupId>
+      <groupId>${project.groupId}</groupId>
       <artifactId>subproject1</artifactId>
       <version>${module.version}</version>
     </dependency>

Modified: 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-with-parent-and-properties/subproject2/pom.xml
URL: 
http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-with-parent-and-properties/subproject2/pom.xml?view=diff&rev=531916&r1=531915&r2=531916
==============================================================================
--- 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-with-parent-and-properties/subproject2/pom.xml
 (original)
+++ 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-with-parent-and-properties/subproject2/pom.xml
 Tue Apr 24 05:58:01 2007
@@ -29,7 +29,7 @@
 
   <dependencies>
     <dependency>
-      <groupId>groupId</groupId>
+      <groupId>${project.groupId}</groupId>
       <artifactId>subproject1</artifactId>
       <version>${module.version}</version>
     </dependency>


Reply via email to