You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@struts.apache.org by bu...@apache.org on 2003/07/02 04:44:16 UTC

DO NOT REPLY [Bug 21250] - Action.getLocale() should call RequestUtils.retrieveUserLocale()

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21250>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=21250

Action.getLocale() should call RequestUtils.retrieveUserLocale()

dgraham@apache.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|Normal                      |Enhancement
            Summary|Action.getLocale does not   |Action.getLocale() should
                   |retrieve locale from request|call
                   |                            |RequestUtils.retrieveUserLoc
                   |                            |ale()



------- Additional Comments From dgraham@apache.org  2003-07-02 02:44 -------
Action.getLocale() also has the nasty side effect of creating a session if one 
doesn't exist.  I agree that we should be calling RequestUtils.
retrieveUserLocale() but that method will need to be refactored to accept an 
HttpServletRequest object instead of a PageContext.

Note that RequestUtils.retrieveUserLocale() doesn't actually retrieve the Locale 
from the request.

---------------------------------------------------------------------
To unsubscribe, e-mail: struts-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: struts-dev-help@jakarta.apache.org