Not sure this one reached the ML Jacques
----- Message d'origine ----- De : "Jacques Le Roux" <[EMAIL PROTECTED]> À : <[email protected]> Envoyé : samedi 11 août 2007 13:10 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 > I confirm that there are no uses of this class. If nobody complain I will > remove this file soon (trunk & release) > > Jacques > > De : "Adrian Crum" <[EMAIL PROTECTED]> > > 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 > > >>>> > > >>>> > > >>>> > > >>>> > > >>>> > > >>> > > > > > > >
