Category Content Wrapper

[Len Shein-3]

All,

Just a query; was there a reason the CategoryContentWrapper does not use a ‘cache’ store, in the same way the ProductContentWrapper uses ‘product.content.rendered’ cache store?

 

Len Shein

[hidden email]

 

Office: 516.742.7888 ext.225

Home Office: 732.333.4303

Cell: 917.882.8515

 

 

[Jacques Le Roux]

I can't see no reasons and I spotted it as well while working on r1705329 and sequels. As you may know I'm still in the middle of something regarding
*ContentWrapper, so I decided to not change it yet to avoid mixing issues.
BTW ProductConfigItemContentWrapper is also concerned

Jacques


Le 12/10/2015 21:57, Len Shein a écrit :

>
> All,
>
> Just a query; was there a reason the CategoryContentWrapper does not use a ‘cache’ store, in the same way the ProductContentWrapper uses
> ‘product.content.rendered’ cache store?
>
> Len Shein
>
> [hidden email] <mailto:[hidden email]>
>
> Office: 516.742.7888 ext.225
>
> Home Office: 732.333.4303
>
> Cell: 917.882.8515
>

[Jacques Le Roux]

All XSS Content issues are normally fixed after OFBIZ-6669. So I will certainly add caches there also

Jacques

Le 13/10/2015 02:06, Jacques Le Roux a écrit :

> I can't see no reasons and I spotted it as well while working on r1705329 and sequels. As you may know I'm still in the middle of something
> regarding *ContentWrapper, so I decided to not change it yet to avoid mixing issues.
> BTW ProductConfigItemContentWrapper is also concerned
>
> Jacques
>
>
> Le 12/10/2015 21:57, Len Shein a écrit :
>>
>> All,
>>
>> Just a query; was there a reason the CategoryContentWrapper does not use a ‘cache’ store, in the same way the ProductContentWrapper uses
>> ‘product.content.rendered’ cache store?
>>
>> Len Shein
>>
>> [hidden email] <mailto:[hidden email]>
>>
>> Office: 516.742.7888 ext.225
>>
>> Home Office: 732.333.4303
>>
>> Cell: 917.882.8515
>>
>
>

[Jacques Le Roux]

Done at revision: 1708326, see OFBIZ-6672

Jacques

Le 13/10/2015 09:54, Jacques Le Roux a écrit :

> All XSS Content issues are normally fixed after OFBIZ-6669. So I will certainly add caches there also
>
> Jacques
>
> Le 13/10/2015 02:06, Jacques Le Roux a écrit :
>> I can't see no reasons and I spotted it as well while working on r1705329 and sequels. As you may know I'm still in the middle of something
>> regarding *ContentWrapper, so I decided to not change it yet to avoid mixing issues.
>> BTW ProductConfigItemContentWrapper is also concerned
>>
>> Jacques
>>
>>
>> Le 12/10/2015 21:57, Len Shein a écrit :
>>>
>>> All,
>>>
>>> Just a query; was there a reason the CategoryContentWrapper does not use a ‘cache’ store, in the same way the ProductContentWrapper uses
>>> ‘product.content.rendered’ cache store?
>>>
>>> Len Shein
>>>
>>> [hidden email] <mailto:[hidden email]>
>>>
>>> Office: 516.742.7888 ext.225
>>>
>>> Home Office: 732.333.4303
>>>
>>> Cell: 917.882.8515
>>>
>>
>>
>