Author: ptahchiev Date: Thu Mar 6 13:29:31 2008 New Revision: 634422 URL: http://svn.apache.org/viewvc?rev=634422&view=rev Log: check if the server property has a null key or value added.
Modified: jakarta/cactus/trunk/integration/ant/src/main/java/org/apache/cactus/integration/ant/CactusTask.java Modified: jakarta/cactus/trunk/integration/ant/src/main/java/org/apache/cactus/integration/ant/CactusTask.java URL: http://svn.apache.org/viewvc/jakarta/cactus/trunk/integration/ant/src/main/java/org/apache/cactus/integration/ant/CactusTask.java?rev=634422&r1=634421&r2=634422&view=diff ============================================================================== --- jakarta/cactus/trunk/integration/ant/src/main/java/org/apache/cactus/integration/ant/CactusTask.java (original) +++ jakarta/cactus/trunk/integration/ant/src/main/java/org/apache/cactus/integration/ant/CactusTask.java Thu Mar 6 13:29:31 2008 @@ -34,8 +34,8 @@ import org.apache.cactus.integration.ant.deployment.DeployableFile; import org.apache.cactus.integration.ant.deployment.EarParser; import org.apache.cactus.integration.ant.deployment.WarParser; - import org.apache.cactus.integration.ant.util.PropertySet; + import org.apache.tools.ant.BuildException; import org.apache.tools.ant.Project; import org.apache.tools.ant.taskdefs.optional.junit.JUnitTask; @@ -263,6 +263,9 @@ public void addSysproperty(Environment.Variable theProperty) { addCactusServerProperty(theProperty); + if(theProperty.getKey() != null && !theProperty.getKey().trim().equals("") && + theProperty.getValue() != null && !theProperty.getValue().trim().equals("")) + addCactusClientProperty(theProperty.getKey(), theProperty.getValue()); super.addSysproperty(theProperty); } @@ -346,6 +349,13 @@ */ private void addCactusServerProperty(Variable theProperty) { + //TODO We always need to check this below, because null properties + // break the cargo execution + if(theProperty.getKey() == null || theProperty.getKey().trim().equals("")|| + theProperty.getValue() == null || theProperty.getValue().trim().equals("")) { + return; + } + log("Adding Cactus server system property [" + theProperty.getKey() + "] with value [" + theProperty.getValue() + "]", Project.MSG_VERBOSE); --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]