The only disadvantage of including the settings in struts-default.xml file is we lose the comments above each setting. A recent change I did was to plug in a Properties file loader that not only remembers line numbers but also comments above an entry. These comments are saved in Location objects so that debugging and error logs later will include them.

However, having default.properties override struts.xml is definitely not what we want...We could also split up the LegacyPropertiesLoader to only load one at a time (default.properties and struts.properties).

Don

Ted Husted wrote:
I'm thinking that a problem I having with setting properties in the
struts.xml (see r477484) may be because the default properties file is
being loaded after the XML configurations, and overriding my changes.

In any event, I'm going to try moving the default.properties settings
to the struts-default.xml, so that everything is configured in one
place.

-Ted.

On 11/20/06, Ted Husted <[EMAIL PROTECTED]> wrote:
Now that we can configure constants via the XML files

* http://struts.apache.org/2.x/docs/constant-configuration.html

is using the struts.properties deprecated?

Or, would there be other valid reasons to keep it around?

-T.





---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to