You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@myfaces.apache.org by "Martin Marinschek (JIRA)" <de...@myfaces.apache.org> on 2006/02/06 13:38:04 UTC

[jira] Commented: (MYFACES-1034) Allow customisable AddResource implementations

    [ http://issues.apache.org/jira/browse/MYFACES-1034?page=comments#action_12365269 ] 

Martin Marinschek commented on MYFACES-1034:
--------------------------------------------

Hi Peter,

sorry to have to cause you work again - I just tried to commit your patches, and that didn't work with the current changes occurring to AddResource. If you provide me with a patch again today, I'll commit it right away - promised.

regards,

Martin

> Allow customisable AddResource implementations
> ----------------------------------------------
>
>          Key: MYFACES-1034
>          URL: http://issues.apache.org/jira/browse/MYFACES-1034
>      Project: MyFaces
>         Type: Improvement
>   Components: Tomahawk
>     Versions: Nightly
>     Reporter: Peter Mahoney
>     Assignee: Martin Marinschek
>  Attachments: AddResourcePatch.zip, DefaultAddResource.java
>
> I use an alternative version of the AddResource. class 
> The two areas which I have customised are the getResourceUri() method (which rewrites the URL to call resources through a forwarding controller servlet) and writeWithFullHeader() (which has different rules as to where insertions will take place when there is no <head> or <body> tags, as the applications only output HTML fragments).
> Currently to do this a custom MyFaces has to be built.
> Ideally, AddResource would be an interface. An AddResourceFactory class would provide the getInstance() method (or equivalent). This method would check for a context parameter specifying the implementation class to use. If not specified, an instance of DefaultAddResource would be used. DefaultAddResource would be non-final and would therefore be subclassed to provide the customisations described above.
> Would this be a suitable change? If so, I would be happy to implement the change and supply a patch.
> Thanks

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira