The getMessages factory method is currently defined as:
public static Messages getMessages(String package)

which concatenates package.LocalStrings to find the a default messages properties file called LocalStrings.

I suggest changing the behavior to take the full package name + properties file name in the String argument. Instead of calling
Messages.getMessages("org.apache.commons") we would call
Messages.getMessages("org.apache.commons.LocalStrings"). The LocalStrings name seems to be a Struts convention and I don't believe a generic component like resources should be assuming anything about how you name your properties files.


Thoughts?

David






_________________________________________________________________
The new MSN 8: advanced junk mail protection and 2 months FREE* http://join.msn.com/?page=features/junkmail



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



Reply via email to