On May 20, 2012, at 9:51 AM, Jacopo Cappellato wrote:

> 
> On May 20, 2012, at 2:07 AM, Adam Heath wrote:
> 
>>> 
>>> I have replaced the usage of the static instance of the BeansWrapper with a 
>>> newly created object.
>> 
>> You didn't go far enough.  There are enough other getDefaultInstance() calls 
>> in our code that need to be fixed.
>> 
>> I'll correct those, because I also need to have a central location to 
>> properly configure the BeansWrapper, based on the ftl breakage.
> 
> Thank you Adam, you are completely right abut this and my commit rev. 1340631 
> should have fixed them all; however I would appreciate as I have made private 
> and final a couple of fields in FreeMarkerWorker.
> I hope this will make your work easier.

To summarize, now we should have two instances of beans wrapper objects in 
OFBiz:
* FreeMarkerWorker.defaultOfbizWrapper
* HtmlWidget.specialBeansWrapper

Jacopo

> 
> Kind regards,
> 
> Jacopo

Reply via email to