You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by "Jacopo Cappellato (JIRA)" <ji...@apache.org> on 2008/06/02 07:29:44 UTC
[jira] Closed: (OFBIZ-1816) Conversion from Writer to Appendable
for all rendering classes
[ https://issues.apache.org/jira/browse/OFBIZ-1816?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Jacopo Cappellato closed OFBIZ-1816.
------------------------------------
Resolution: Fixed
Thanks for your review, Adrian.
Committed in rev. 662328
> Conversion from Writer to Appendable for all rendering classes
> --------------------------------------------------------------
>
> Key: OFBIZ-1816
> URL: https://issues.apache.org/jira/browse/OFBIZ-1816
> Project: OFBiz
> Issue Type: Improvement
> Components: content, framework
> Reporter: Jacopo Cappellato
> Priority: Minor
> Attachments: writer2appendable.patch
>
>
> Based on a recent discussion in the dev list, and especially on the comments from David Jones and the work done by Adrian Crum, I'm submitting here for your review a patch to refactor all the widget rendering methods and some of the content and Freemarker rendering classes to use Appendable instead of Writer.
> What do you think?
> There is just one place where the code is not very good; in the FreeMarkerWorker.renderTemplate (at line 197) I've added a casting from Appendable to Writer:
> Environment env = template.createProcessingEnvironment(context, (Writer)outWriter);
> This would cause an issue if in the future we will not pass a Writer to that method.
> I'm testing this work right now; if there are no objections I am going to commit this in the next few days.
> Jacopo
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.