You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by si...@apache.org on 2008/11/12 18:31:07 UTC
svn commit: r713429 - in /maven/components/trunk:
maven-project/src/main/java/org/apache/maven/project/builder/PomClassicTransformer.java
maven-shared-model/src/main/java/org/apache/maven/shared/model/ModelTransformerContext.java
Author: sisbell
Date: Wed Nov 12 09:31:07 2008
New Revision: 713429
URL: http://svn.apache.org/viewvc?rev=713429&view=rev
Log:
Fix for IT 3827. On joins between model containers during the applying of plugin management, transform rule was reversing the plugin config element order. Now do a reverse sort prior to join, so elements end up in correct order.
Modified:
maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/builder/PomClassicTransformer.java
maven/components/trunk/maven-shared-model/src/main/java/org/apache/maven/shared/model/ModelTransformerContext.java
Modified: maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/builder/PomClassicTransformer.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/builder/PomClassicTransformer.java?rev=713429&r1=713428&r2=713429&view=diff
==============================================================================
--- maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/builder/PomClassicTransformer.java (original)
+++ maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/builder/PomClassicTransformer.java Wed Nov 12 09:31:07 2008
@@ -184,7 +184,9 @@
ModelContainerAction action = dependencyContainer.containerAction( managementContainer );
if ( action.equals( ModelContainerAction.JOIN ) || action.equals( ModelContainerAction.DELETE ) )
{
- source.join( dependencyContainer, managementContainer );
+ ModelContainer reverseSortedContainer = new ArtifactModelContainerFactory().create(
+ ModelTransformerContext.sort(dependencyContainer.getProperties(), ProjectUri.Build.Plugins.Plugin.xUri) );
+ source.join( reverseSortedContainer, managementContainer );
}
}
}
Modified: maven/components/trunk/maven-shared-model/src/main/java/org/apache/maven/shared/model/ModelTransformerContext.java
URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-shared-model/src/main/java/org/apache/maven/shared/model/ModelTransformerContext.java?rev=713429&r1=713428&r2=713429&view=diff
==============================================================================
--- maven/components/trunk/maven-shared-model/src/main/java/org/apache/maven/shared/model/ModelTransformerContext.java (original)
+++ maven/components/trunk/maven-shared-model/src/main/java/org/apache/maven/shared/model/ModelTransformerContext.java Wed Nov 12 09:31:07 2008
@@ -340,7 +340,7 @@
* @param baseUri the base URI of every model property
* @return sorted list of model properties
*/
- protected static List<ModelProperty> sort( List<ModelProperty> properties, String baseUri )
+ public static List<ModelProperty> sort( List<ModelProperty> properties, String baseUri )
{
if ( properties == null )
{