[jira] [Commented] (OFBIZ-6329) Malfunction of configurable FTL-Template caching in DataResourceWorker.renderDataResourceAsText

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Commented] (OFBIZ-6329) Malfunction of configurable FTL-Template caching in DataResourceWorker.renderDataResourceAsText

Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-6329?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14528284#comment-14528284 ]

Martin Becker commented on OFBIZ-6329:
--------------------------------------

Sorry for that, Adrian. I'm using git locally and created the patch files with "git format-patch" as Michael stated. Therefore there are some header informations additional to the standard diff in the patch file. There should be no need for a committer to modify a patch file to get it working, so we will create the patch files with "git diff", that should be fine.

I will provide corrected patch files here soon and Adrian may check if they are usable for him (and thus for all others).

> Malfunction of configurable FTL-Template caching in DataResourceWorker.renderDataResourceAsText
> -----------------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-6329
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-6329
>             Project: OFBiz
>          Issue Type: Bug
>          Components: content
>    Affects Versions: Trunk
>            Reporter: Martin Becker
>         Attachments: OFBIZ-6329_FTL-Caching.patch, OFBIZ-6329_Non-Template-Caching.patch
>
>
> There are several problems with the current caching logic in DataResourceWorker.renderDataResourceAsText(...).
> Enabling the caching of rendered FTL-Templates from DataResources with the property disable.ftl.template.cache in content.properties enables a non-functioning block of code that handles the rendering of the cached template. And if it is deactivated (default), the FTL-Templates are still cached by the FreeMarkerWorker.
> However the correct logic for caching and using the rendered FTL-Template is already implemented in the FreeMarkerWorker and controlled by an optional useCache parameter.
> In addition there is an API call to DataResourceWorker.writeDataResourceText for non template content with a static "true" for useCache instead of using the given cache parameter value of the renderDataResourceAsText method itself, so even if the caller do not want to use caching at all, the non template text data is cached an FTL-Templates are cached also.
> I will provide a patch for those two issues in the mentioned method.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)