Yup, +1 from me too.

We might also want to rename them to DenouncePlugin.properties and ProfanityFilter.properties to make sure they are aligned with the plugin names. ATM they're not that descriptive.

/Janne

On 14 Feb 2009, at 01:31, Andrew Jaquith wrote:

+1

On Feb 13, 2009, at 14:44, Harry Metske <[email protected]> wrote:

then I'd also like to move

src/java/org/apache/wiki/plugin/denounce.properties
src/java/org/apache/wiki/filters/profanity.properties

to

src/WebContent/WEB-INF/classes/denounce.properties
src/WebContent/WEB-INF/classes/profanity.properties

we then have most property files together.

/Harry

2009/2/10 Janne Jalkanen <[email protected]>


Yeah, the idea of having WebContent is that we can put all the resources in
the right places from the get-go...

/Janne


On 10 Feb 2009, at 22:31, Harry Metske wrote:

you mean moving it to WEB-INF/classes ?

that's also a possibility, needs some tweaks to build.xml

Harry

2009/2/9 Janne Jalkanen <[email protected]>

* added etc/i18n to the classpath and changed project setting "Allow

output folders
as source folders", this makes it possible to run individual JUnit
tests again.


Shouldn't etc/i18n -files be rather under WEB-INF? Or should they go
under
src/resources?

/Janne




Reply via email to