Re: svn commit: r911761 - /ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/Product.groovy

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

Re: svn commit: r911761 - /ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/Product.groovy

Adam Heath-2
[hidden email] wrote:

> Author: ashish
> Date: Fri Feb 19 09:47:47 2010
> New Revision: 911761
>
> URL: http://svn.apache.org/viewvc?rev=911761&view=rev
> Log:
> Fix for groovy.lang.MissingPropertyException: No such property: configproductdetailScreen for class: Product.
> Patch from Anurag (Thanks!)
>
> Modified:
>     ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/Product.groovy
>
> Modified: ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/Product.groovy
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/Product.groovy?rev=911761&r1=911760&r2=911761&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/Product.groovy (original)
> +++ ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/Product.groovy Fri Feb 19 09:47:47 2010
> @@ -93,7 +93,7 @@
>          context.metaKeywords = StringUtil.join(keywords, ", ");
>  
>          // Set the default template for aggregated product (product component configurator ui)
> -        if (product.productTypeId && "AGGREGATED".equals(product.productTypeId) && configproductdetailScreen) {
> +        if (product.productTypeId && "AGGREGATED".equals(product.productTypeId) && context.configproductdetailScreen) {
>              detailScreen = configproductdetailScreen;
>          }

Shouldn't the line inside the condition also be changed too?

Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r911761 - /ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/Product.groovy

Ashish Vijaywargiya
Either option will work. Committed in r918053.

--
Ashish

On Mon, Mar 1, 2010 at 11:59 PM, Adam Heath <[hidden email]> wrote:

> [hidden email] wrote:
>> Author: ashish
>> Date: Fri Feb 19 09:47:47 2010
>> New Revision: 911761
>>
>> URL: http://svn.apache.org/viewvc?rev=911761&view=rev
>> Log:
>> Fix for groovy.lang.MissingPropertyException: No such property: configproductdetailScreen for class: Product.
>> Patch from Anurag (Thanks!)
>>
>> Modified:
>>     ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/Product.groovy
>>
>> Modified: ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/Product.groovy
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/Product.groovy?rev=911761&r1=911760&r2=911761&view=diff
>> ==============================================================================
>> --- ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/Product.groovy (original)
>> +++ ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/Product.groovy Fri Feb 19 09:47:47 2010
>> @@ -93,7 +93,7 @@
>>          context.metaKeywords = StringUtil.join(keywords, ", ");
>>
>>          // Set the default template for aggregated product (product component configurator ui)
>> -        if (product.productTypeId && "AGGREGATED".equals(product.productTypeId) && configproductdetailScreen) {
>> +        if (product.productTypeId && "AGGREGATED".equals(product.productTypeId) && context.configproductdetailScreen) {
>>              detailScreen = configproductdetailScreen;
>>          }
>
> Shouldn't the line inside the condition also be changed too?
>
>