By the way, we can remove the FtlTransform.java file - it isn't being used and I strongly discourage
its use.
-Adrian
Jacques Le Roux wrote:
Scott, Jacopo,
I understand your concerns. Anyway if I have to revert from trunk I will revert
from release too. IMHO it's surer to revert both
than to forget to merge to release. This is why I took this decision and also
because it seems to me that these changes are safe.
About ftl cache by default. Actually it is not disable totally, only cleared
every 10 seconds by default to facilitate development.
And yes, I tested with cache enabled (related cache.properties lines commented
out). Actually I continue to test...
I was sure putting the magic letters CTR, I will get some comments ;o)
Jacques
----- Message d'origine -----
De : "Jacopo Cappellato" <[EMAIL PROTECTED]>
À : <[email protected]>
Envoyé : vendredi 10 août 2007 12:14
Objet : Re: svn commit: r564531 - in /ofbiz/branches/release4.0:
applications/content/src/org/ofbiz/content/data/
applications/content/src/org/ofbiz/content/survey/
framework/base/src/base/org/ofbiz/base/util/template/
framework/webtools/src/org/ofbiz/webtoo
Yes,
I agree with Scott here.
And about this specific commit: Jacques, did you test it with ftl cache
enabled? I'm asking you this because, by default, if I'm not wrong,cache
for ftl files is disabled...
Jacopo
Scott Gray wrote:
Hi Jacques
While I have no idea about this specific commit, I'm a big -1 for a CTR
approach when it comes to the release branch. I think if your committing a
CTR fix to the trunk then we should at least wait a week or two before
putting it into the release.
It's great to get fixes into the branch, but we need to do our utmost not to
introduce new bugs in the process.
Regards
Scott
On 10/08/07, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote:
Author: jleroux
Date: Fri Aug 10 02:52:26 2007
New Revision: 564531
URL: http://svn.apache.org/viewvc?view=rev&rev=564531
Log:
Applied fix from trunk for revision: 564530
