You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by jm...@apache.org on 2007/07/03 21:04:02 UTC
svn commit: r552950 -
/maven/sandbox/trunk/plugins/maven-pom-plugin/src/main/java/org/apache/maven/plugins/pom/AddByXPathMojo.java
Author: jmcconnell
Date: Tue Jul 3 12:04:01 2007
New Revision: 552950
URL: http://svn.apache.org/viewvc?view=rev&rev=552950
Log:
commit bugfix I missed a while back
Modified:
maven/sandbox/trunk/plugins/maven-pom-plugin/src/main/java/org/apache/maven/plugins/pom/AddByXPathMojo.java
Modified: maven/sandbox/trunk/plugins/maven-pom-plugin/src/main/java/org/apache/maven/plugins/pom/AddByXPathMojo.java
URL: http://svn.apache.org/viewvc/maven/sandbox/trunk/plugins/maven-pom-plugin/src/main/java/org/apache/maven/plugins/pom/AddByXPathMojo.java?view=diff&rev=552950&r1=552949&r2=552950
==============================================================================
--- maven/sandbox/trunk/plugins/maven-pom-plugin/src/main/java/org/apache/maven/plugins/pom/AddByXPathMojo.java (original)
+++ maven/sandbox/trunk/plugins/maven-pom-plugin/src/main/java/org/apache/maven/plugins/pom/AddByXPathMojo.java Tue Jul 3 12:04:01 2007
@@ -87,7 +87,7 @@
xmlTool.removeNamespaces();
- if( xmlTool.hasElement( skipXPath ) )
+ if( skipXPath != null && xmlTool.hasElement( skipXPath ) )
{
getLog().info( "skipXPath element located, skipping execution" );
return;
@@ -97,11 +97,11 @@
{
Element node = xmlTool.getElement( xpath );
- node.addElement( newElement );
+ Element newNode = node.addElement( newElement );
if ( newValue != null )
{
- node.setText( newValue );
+ newNode.setText( newValue );
}
}
else if ( failFast )