You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@myfaces.apache.org by "Paul Nicolucci (JIRA)" <de...@myfaces.apache.org> on 2013/08/06 04:53:47 UTC
[jira] [Created] (MYFACES-3745)
org.apache.myfaces.view.facelets.impl.DefaultFaceletContext constructors
are inconsistent
Paul Nicolucci created MYFACES-3745:
---------------------------------------
Summary: org.apache.myfaces.view.facelets.impl.DefaultFaceletContext constructors are inconsistent
Key: MYFACES-3745
URL: https://issues.apache.org/jira/browse/MYFACES-3745
Project: MyFaces Core
Issue Type: Bug
Components: JSR-314
Affects Versions: 2.1.12, 2.0.18
Reporter: Paul Nicolucci
Priority: Minor
In MYFACES-3246 the following constructor was updated:
DefaultFaceletContext(FacesContext faces, AbstractFacelet facelet, FaceletCompositionContext mctx)
It no longer does the following:
//Set FACELET_CONTEXT_KEY on FacesContext attribute map, to
//reflect the current facelet context instance
faces.getAttributes().put(FaceletContext.FACELET_CONTEXT_KEY, this);
Setting the FACELET_CONTEXT_KEY is now done where ever the DefaultFaceletContext is created for instance in DefaultFacelet.
DefaultFaceletContext ctxWrapper = new DefaultFaceletContext((DefaultFaceletContext)ctx, this, false);
ctx.getFacesContext().getAttributes().put(FaceletContext.FACELET_CONTEXT_KEY, ctxWrapper);
However, the other constructor which is actually being called above still sets the FACELET_CONTEXT_KEY and so it is set in the constructor and then set again directly after creation.
//Update FACELET_CONTEXT_KEY on FacesContext attribute map, to
//reflect the current facelet context instance
ctx.getFacesContext().getAttributes().put(FaceletContext.FACELET_CONTEXT_KEY, this);
I think this was just an oversight when fixing this bug. But I think we should clean this up and I'll provide the trivial patch that can be applied.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira