You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by do...@apache.org on 2010/12/09 16:14:48 UTC
svn commit: r1043999 -
/ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreAutoPreferences.java
Author: doogie
Date: Thu Dec 9 15:14:47 2010
New Revision: 1043999
URL: http://svn.apache.org/viewvc?rev=1043999&view=rev
Log:
Yet another bad bug. The name/value pair was never actually turned into
a map. findByAnd(entityName, field, value, list) was never a valid
call. However, this code still compiled, as it ended up using the
vararg variant of findByAnd. If this code had ever been run, then it
would have blown up.
broken since: 984366, initial commit
date: Wed Aug 11 10:39:10 2010 +0000
Modified:
ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreAutoPreferences.java
Modified: ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreAutoPreferences.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreAutoPreferences.java?rev=1043999&r1=1043998&r2=1043999&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreAutoPreferences.java (original)
+++ ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreAutoPreferences.java Thu Dec 9 15:14:47 2010
@@ -1269,7 +1269,8 @@ public class EbayStoreAutoPreferences {
} else if (priceType.equals("RETAIL_PRICE")) {
//ignore
} else if (priceType.equals("SELLER_COST")) {
- List<GenericValue> supplierProduct = delegator.findByAnd("SupplierProduct", "productId", SKUItem, UtilMisc.toList("availableFromDate DESC"));
+ // FIXME
+ List<GenericValue> supplierProduct = delegator.findByAnd("SupplierProduct", UtilMisc.toMap("productId", SKUItem), UtilMisc.toList("availableFromDate DESC"));
String lastPrice = supplierProduct.get(0).getString("lastPrice");
doBasePrice = Double.parseDouble(lastPrice);
} else if (priceType.equals("SECOND_CHANCE_PRICE")) {
Re: svn commit: r1043999 - /ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreAutoPreferences.java
Posted by Adam Heath <do...@brainfood.com>.
On 12/09/2010 10:26 AM, Jacopo Cappellato wrote:
>
> On Dec 9, 2010, at 4:14 PM, doogie@apache.org wrote:
>
>> Author: doogie
>> Date: Thu Dec 9 15:14:47 2010
>> New Revision: 1043999
>>
>> URL: http://svn.apache.org/viewvc?rev=1043999&view=rev
>> Log:
>> Yet another bad bug. The name/value pair was never actually turned into
>> a map. findByAnd(entityName, field, value, list) was never a valid
>> call. However, this code still compiled, as it ended up using the
>> vararg variant of findByAnd. If this code had ever been run, then it
>> would have blown up.
>>
>> broken since: 984366, initial commit
>> date: Wed Aug 11 10:39:10 2010 +0000
>>
>> Modified:
>> ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreAutoPreferences.java
>>
>> Modified: ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreAutoPreferences.java
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreAutoPreferences.java?rev=1043999&r1=1043998&r2=1043999&view=diff
>> ==============================================================================
>> --- ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreAutoPreferences.java (original)
>> +++ ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreAutoPreferences.java Thu Dec 9 15:14:47 2010
>> @@ -1269,7 +1269,8 @@ public class EbayStoreAutoPreferences {
>> } else if (priceType.equals("RETAIL_PRICE")) {
>> //ignore
>> } else if (priceType.equals("SELLER_COST")) {
>> - List<GenericValue> supplierProduct = delegator.findByAnd("SupplierProduct", "productId", SKUItem, UtilMisc.toList("availableFromDate DESC"));
>> + // FIXME
>> + List<GenericValue> supplierProduct = delegator.findByAnd("SupplierProduct", UtilMisc.toMap("productId", SKUItem), UtilMisc.toList("availableFromDate DESC"));
>
> And "-availableFromDate" should be used instead of "availableFromDate DESC".
Why? Both work.
My last 4 commits need to be discusssed. 3 of them *NEVER* worked,
even with the initial commit to ofbiz. There is no way at all that
the code was run by the original author. So why was the code there in
the first place?
Then, the one change by Scott was a findByAnd->findList, whether the
parameters were changed, but the method name was not. I am not
annoyed that that line was changed badly. Those things happen. What
is annoying is that the problem code hadn't been run by anyone in the
last 2 years.
Re: svn commit: r1043999 - /ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreAutoPreferences.java
Posted by Jacopo Cappellato <ja...@hotwaxmedia.com>.
On Dec 9, 2010, at 4:14 PM, doogie@apache.org wrote:
> Author: doogie
> Date: Thu Dec 9 15:14:47 2010
> New Revision: 1043999
>
> URL: http://svn.apache.org/viewvc?rev=1043999&view=rev
> Log:
> Yet another bad bug. The name/value pair was never actually turned into
> a map. findByAnd(entityName, field, value, list) was never a valid
> call. However, this code still compiled, as it ended up using the
> vararg variant of findByAnd. If this code had ever been run, then it
> would have blown up.
>
> broken since: 984366, initial commit
> date: Wed Aug 11 10:39:10 2010 +0000
>
> Modified:
> ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreAutoPreferences.java
>
> Modified: ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreAutoPreferences.java
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreAutoPreferences.java?rev=1043999&r1=1043998&r2=1043999&view=diff
> ==============================================================================
> --- ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreAutoPreferences.java (original)
> +++ ofbiz/trunk/specialpurpose/ebaystore/src/org/ofbiz/ebaystore/EbayStoreAutoPreferences.java Thu Dec 9 15:14:47 2010
> @@ -1269,7 +1269,8 @@ public class EbayStoreAutoPreferences {
> } else if (priceType.equals("RETAIL_PRICE")) {
> //ignore
> } else if (priceType.equals("SELLER_COST")) {
> - List<GenericValue> supplierProduct = delegator.findByAnd("SupplierProduct", "productId", SKUItem, UtilMisc.toList("availableFromDate DESC"));
> + // FIXME
> + List<GenericValue> supplierProduct = delegator.findByAnd("SupplierProduct", UtilMisc.toMap("productId", SKUItem), UtilMisc.toList("availableFromDate DESC"));
And "-availableFromDate" should be used instead of "availableFromDate DESC".
Jacopo
> String lastPrice = supplierProduct.get(0).getString("lastPrice");
> doBasePrice = Double.parseDouble(lastPrice);
> } else if (priceType.equals("SECOND_CHANCE_PRICE")) {
>
>