Hi,

On 09.03.2010 09:53, Bertrand Delacretaz wrote:
> On Tue, Mar 9, 2010 at 9:34 AM, Carsten Ziegeler <[email protected]> wrote:
>> Is everyone fine if we add the following interface to the API?
>>
>> interface ResourceDecorator {
>>    Resource decorade(Resource)
>>    Resource decorade(Resource, HttpServletRequest)
>> }
> 
> Why not SlingHttpServletRequest?

The resolve method is called with a HttpServletRequest because at the
time this method is called, the SlingHttpServletRequest does not exist
yet. [ The resolve method is called to create that object ]

Regards
Felix

> 
> +1 anyway if there's a good reason to use HttpServletRequest.
> 
> (and typo: should be "decorate" of course)
> 
> -Bertrand
> 

Reply via email to