myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Martin Koci (JIRA)" <...@myfaces.apache.org>
Subject [jira] Commented: (MYFACES-2527) Support for decorator design pattern: RenderKit(s)
Date Sun, 31 Jan 2010 13:11:34 GMT

    [ https://issues.apache.org/jira/browse/MYFACES-2527?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12806819#action_12806819
] 

Martin Koci commented on MYFACES-2527:
--------------------------------------

Possible patch:

org.apache.myfaces.config.impl.digester.DigesterFacesConfigDispenserImpl.feed(FacesConfig)
- do not merge renderkit if render-kit-class is provided.

org.apache.myfaces.config.FacesConfigurator.configureRenderKits() - do not use class.newInstance()
but method getApplicationObject (which is already used for other decorations)


> Support for decorator design pattern: RenderKit(s)
> --------------------------------------------------
>
>                 Key: MYFACES-2527
>                 URL: https://issues.apache.org/jira/browse/MYFACES-2527
>             Project: MyFaces Core
>          Issue Type: Bug
>          Components: JSR-314
>            Reporter: Martin Koci
>
> Spec. "11.4.6 Delegating Implementation Support" - "The runtime must support the decorator
design pattern .. for .. RenderKit". This is especially true for HTML_BASIC, following example
should work:
> faces-config.xml
> <render-kit>
>     <render-kit-class>com.foo.render.RenderKitImpl</render-kit-class>
>     <render-kit-id>HTML_BASIC</render-kit-id>
> </render-kit>
> RenderKitImpl:
> class RenderKitImpl extends RenderKit implements FacesWrapper<RenderKit> {
>   public RenderKitImpl(RenderKit wrapped) {
>     super();
>     this.wrapped = wrapped;
>   }
>   // method delegation here ...
> }

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message