You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Stefan Seifert (JIRA)" <ji...@apache.org> on 2016/02/05 22:35:40 UTC
[jira] [Created] (SLING-5490) servlet-helpers: Add locale support
to MockSlingHttpServletRequest and MockSlingHttpServletResponse
Stefan Seifert created SLING-5490:
-------------------------------------
Summary: servlet-helpers: Add locale support to MockSlingHttpServletRequest and MockSlingHttpServletResponse
Key: SLING-5490
URL: https://issues.apache.org/jira/browse/SLING-5490
Project: Sling
Issue Type: Improvement
Components: Extensions
Reporter: Stefan Seifert
Assignee: Stefan Seifert
Fix For: Servlet Helpers 1.0.0
{quote}
-----Original Message-----
From: Justin Edelson [mailto:justin@justinedelson.com]
Sent: Friday, January 22, 2016 5:54 PM
To: dev@sling.apache.org
Subject: MockSlingHttpServletResponse doesn't support getLocale()
Hi,
I was looking at MockSlingHttpServletResponse (
https://github.com/apache/sling/blob/trunk/bundles/extensions/servlet-helpers/src/main/java/org/apache/sling/servlethelpers/MockSlingHttpServletResponse.java)
and was wondering if there was a specific reason why getLocale() isn't
supported. IIUC (and to be honest, I wasn't 100% following this
discussion), this class is now intended to be used in concert with the
SlingRequestProcessor interface (as opposed to just unit tests). If so, it
seems like supporting getLocale() would be important as servlets/scripts
might depend upon this.
I can make this change super-quick, but wanted to check first to see if
there was a reason not to implement it.
Thanks,
Justin
{quote}
additionally we should make the locale in MockSlingHttpServletRequest configurable.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)