DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT <http://issues.apache.org/bugzilla/show_bug.cgi?id=30034>. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND INSERTED IN THE BUG DATABASE.
http://issues.apache.org/bugzilla/show_bug.cgi?id=30034 Locale feature is unsutisfuctory [EMAIL PROTECTED] changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|Normal |Enhancement ------- Additional Comments From [EMAIL PROTECTED] 2004-07-11 15:27 ------- Firstly I'm changing this to enhancement because Struts is working as designed and therefore its not a bug. I think your suggestion is good except for configuring it through the serlvet <init-param> - its would be more in keeping with struts to configure it though the <controller> element in the struts-config.xml. Having said that this is exactly the kind of behaviour provided in struts-chain (in the contrib area) and I believe that other committers are planning to move to a "ComposableRequestProcessor" based on that for the next Struts version. At the moment I personally wouldn't apply a patch to do your change until the decision on that has been taken. If you look at struts-chain: http://cvs.apache.org/viewcvs.cgi/jakarta-struts/contrib/struts- chain/src/java/org/apache/struts/chain/ The AbstractSelectLocale class is pretty much what you are proposing with LocaleSupplier. Niall --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
