[ 
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

Reply via email to