Re: cvs commit: jakarta-velocity/src/java/org/apache/velocity/runtime/configurationConfiguration.java Configurations.java ConfigurationsRepository.javaExtendedProperties.java

[jv...@periapt.com]

Sam Ruby wrote:

> jvanzyl:
> >
> > - fixes to make the Configurations.subset(prefix) work correctly,
> >   and added a setProperty(key,value) that takes into account
> >   duplicate keys. when reading in a file the duplicate keys
> >   were being accounted for but using setProperty() was using
> >   Hashtable.put(key,value) which was wiping out duplicate
>
> This fix will break jakarta-site2.
>
> The default for resource.loader.1.resource.path is ".".
>
> The provided velocity.properties defines this as "./xdocs/stylesheets".
>
> With the changes made between 7pm and 8pm EST, these two are combined into
> a vector, and the result is:
> java.lang.ClassCastException: java.util.Vector
>            at
> org.apache.velocity.runtime.configuration.Configurations.subset(Configurations.java:214)
>
> It is also worth noting that Anakia task catches exceptions, prints stack
> tracebacks, and then returns successfully on error.  It would be preferred
> if BuildExceptions were thrown with the original exception as the root
> cause.
>
> - Sam Ruby

Thanks Sam, I will look into it ASAP.

--
jvz.

Jason van Zyl
jvanzyl@periapt.com

http://jakarta.apache.org/velocity
http://jakarta.apache.org/turbine