You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@velocity.apache.org by nb...@apache.org on 2008/10/25 01:26:22 UTC
svn commit: r707785 -
/velocity/tools/trunk/src/main/java/org/apache/velocity/tools/config/ToolConfiguration.java
Author: nbubna
Date: Fri Oct 24 16:26:22 2008
New Revision: 707785
URL: http://svn.apache.org/viewvc?rev=707785&view=rev
Log:
fix skipSetters setting handling
Modified:
velocity/tools/trunk/src/main/java/org/apache/velocity/tools/config/ToolConfiguration.java
Modified: velocity/tools/trunk/src/main/java/org/apache/velocity/tools/config/ToolConfiguration.java
URL: http://svn.apache.org/viewvc/velocity/tools/trunk/src/main/java/org/apache/velocity/tools/config/ToolConfiguration.java?rev=707785&r1=707784&r2=707785&view=diff
==============================================================================
--- velocity/tools/trunk/src/main/java/org/apache/velocity/tools/config/ToolConfiguration.java (original)
+++ velocity/tools/trunk/src/main/java/org/apache/velocity/tools/config/ToolConfiguration.java Fri Oct 24 16:26:22 2008
@@ -47,7 +47,7 @@
private String key;
private String classname;
private String restrictTo;
- private boolean skipSetters;
+ private Boolean skipSetters;
private Status status;
private Throwable problem;
@@ -78,7 +78,7 @@
this.restrictTo = path;
}
- public void setSkipSetters(boolean cfgOnly)
+ public void setSkipSetters(Boolean cfgOnly)
{
this.skipSetters = cfgOnly;
}
@@ -262,7 +262,7 @@
return this.restrictTo;
}
- public boolean getSkipSetters()
+ public Boolean getSkipSetters()
{
return this.skipSetters;
}
@@ -284,7 +284,10 @@
}
info.restrictTo(getRestrictTo());
- info.setSkipSetters(getSkipSetters());
+ if (getSkipSetters() != null)
+ {
+ info.setSkipSetters(getSkipSetters());
+ }
// it's ok to use this here, because we know it's the
// first time properties have been added to this ToolInfo
info.addProperties(getPropertyMap());