You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by Christian Geisert <ch...@isu-gmbh.de> on 2013/07/03 12:07:04 UTC

Re: svn commit: r1497817 - in /ofbiz/trunk/applications/product: servicedef/services_facility.xml src/org/ofbiz/product/inventory/InventoryServices.java webapp/facility/WEB-INF/actions/facility/ViewFacilityInventoryByProduct.gro ovy

Hmmh, I'm not sure about this one -  I think changing an api is bad 
(even if it's probably not widely used), but within a minor release it's 
even worser. I'll change the (wrong) field name on the form for the 
release branches.

Christian

Am 28.06.2013 22:33, schrieb Jacques Le Roux:
> Should we not backport those fixes?
>
> Jacques
>
> From: <ch...@apache.org>
>> Author: chrisg
>> Date: Fri Jun 28 15:35:58 2013
>> New Revision: 1497817
>>
>> URL: http://svn.apache.org/r1497817
>> Log:
>> Fixed problem with default price on inventory report (parameter name typo)
>>
>> Modified:
>>     ofbiz/trunk/applications/product/servicedef/services_facility.xml
>>     ofbiz/trunk/applications/product/src/org/ofbiz/product/inventory/InventoryServices.java
>>     ofbiz/trunk/applications/product/webapp/facility/WEB-INF/actions/facility/ViewFacilityInventoryByProduct.groovy

Re: svn commit: r1497817 - in /ofbiz/trunk/applications/product: servicedef/services_facility.xml src/org/ofbiz/product/inventory/InventoryServices.java webapp/facility/WEB-INF/actions/facility/ViewFacilityInventoryByProduct.gro ovy

Posted by Jacques Le Roux <ja...@les7arts.com>.
OK, thanks Christian

Jacques

From: "Christian Geisert" <ch...@isu-gmbh.de>
> Hmmh, I'm not sure about this one -  I think changing an api is bad 
> (even if it's probably not widely used), but within a minor release it's 
> even worser. I'll change the (wrong) field name on the form for the 
> release branches.
> 
> Christian
> 
> Am 28.06.2013 22:33, schrieb Jacques Le Roux:
>> Should we not backport those fixes?
>>
>> Jacques
>>
>> From: <ch...@apache.org>
>>> Author: chrisg
>>> Date: Fri Jun 28 15:35:58 2013
>>> New Revision: 1497817
>>>
>>> URL: http://svn.apache.org/r1497817
>>> Log:
>>> Fixed problem with default price on inventory report (parameter name typo)
>>>
>>> Modified:
>>>     ofbiz/trunk/applications/product/servicedef/services_facility.xml
>>>     ofbiz/trunk/applications/product/src/org/ofbiz/product/inventory/InventoryServices.java
>>>     ofbiz/trunk/applications/product/webapp/facility/WEB-INF/actions/facility/ViewFacilityInventoryByProduct.groovy