Re: svn commit: r946100 - in /ofbiz/trunk/applications/order/widget/ordermgr: CommonScreens.xml QuoteScreens.xml

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

Re: svn commit: r946100 - in /ofbiz/trunk/applications/order/widget/ordermgr: CommonScreens.xml QuoteScreens.xml

Anil Patel-3
I think this commit and r946105, 946108 should be applied to 10.04 as well.
These commits have no functional changes. They fix permission check and decorator screen location issues.

Thanks and Regards
Anil Patel
HotWax Media Inc
Find us on the web at www.hotwaxmedia.com or Google Keyword "ofbiz"

On May 19, 2010, at 6:04 AM, [hidden email] wrote:

> Author: mor
> Date: Wed May 19 10:04:17 2010
> New Revision: 946100
>
> URL: http://svn.apache.org/viewvc?rev=946100&view=rev
> Log:
> Removed an unnecessary decorator screen CommonQuotePriceDecorator related to Order Manager Quotes. The only difference between this and screen
> CommonQuoteDecorator is that the the former has a permission check for ORDERMGR__QUOTE_PRICE where as latter has a check for permission
> ORDERMGR_VIEW. The permission ORDERMGR_QUOTE_PRICE is though already checked at menu level and hence is not required explicity
> at the decorator level.
>
> Modified:
>    ofbiz/trunk/applications/order/widget/ordermgr/CommonScreens.xml
>    ofbiz/trunk/applications/order/widget/ordermgr/QuoteScreens.xml

Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r946100 - in /ofbiz/trunk/applications/order/widget/ordermgr: CommonScreens.xml QuoteScreens.xml

Vikas Mayur-3
Hi Anil,

I was a bit unsure but based on your suggestion I have migrated all of  
the code to 10.04 branch.

Regards
Vikas

On May 19, 2010, at 10:31 PM, Anil Patel wrote:

> I think this commit and r946105, 946108 should be applied to 10.04  
> as well.
> These commits have no functional changes. They fix permission check  
> and decorator screen location issues.
>
> Thanks and Regards
> Anil Patel
> HotWax Media Inc
> Find us on the web at www.hotwaxmedia.com or Google Keyword "ofbiz"
>
> On May 19, 2010, at 6:04 AM, [hidden email] wrote:
>
>> Author: mor
>> Date: Wed May 19 10:04:17 2010
>> New Revision: 946100
>>
>> URL: http://svn.apache.org/viewvc?rev=946100&view=rev
>> Log:
>> Removed an unnecessary decorator screen CommonQuotePriceDecorator  
>> related to Order Manager Quotes. The only difference between this  
>> and screen
>> CommonQuoteDecorator is that the the former has a permission check  
>> for ORDERMGR__QUOTE_PRICE where as latter has a check for permission
>> ORDERMGR_VIEW. The permission ORDERMGR_QUOTE_PRICE is though  
>> already checked at menu level and hence is not required explicity
>> at the decorator level.
>>
>> Modified:
>>   ofbiz/trunk/applications/order/widget/ordermgr/CommonScreens.xml
>>   ofbiz/trunk/applications/order/widget/ordermgr/QuoteScreens.xml
>