Hi Bertrand,

> On 2 Dec 2019, at 12:54, Bertrand Delacretaz <[email protected]> wrote:
> 
> On Fri, Nov 29, 2019 at 4:39 PM Radu Cotescu <[email protected]> wrote:
>> 
>> wrapped the default PageContext in order to be able to provide the JSP
>> scripts with bindings entries, through the pageContext...
> 
> I think it would be good to add a brief comment to
> SlingJspPageContext, to explain why it's here and mention that the
> wrapped Page Context's attributes take precedence by design.


I’ve added this info in 
https://github.com/apache/sling-org-apache-sling-scripting-jsp/commit/cf8343a 
<https://github.com/apache/sling-org-apache-sling-scripting-jsp/commit/cf8343a>.

Thanks,
Radu

Reply via email to