I didn't say the fix was incorrect, just that it broke something which 
previously worked.  The demo estimates have a null productStoreShipMethId so no 
results are returned.  Possibly the code should remove that condition if no 
results are returned and try again without it.

Regards
Scott

On 17/04/2010, at 6:27 PM, Ean Schuessler wrote:

> I'm fairly sure this fix is correct. We need to look at what the demo store 
> is doing. Matching with an OR doesn't make sense to me. If you've specified 
> an exact productStoreShipMethId, why should you receive anything other than 
> the corresponding entity? 
> 
> ----- "Scott Gray" wrote: 
>> This commit broke the demo system, no cost estimates are returned during 
>> checkout any more. 
> 

Attachment: smime.p7s
Description: S/MIME cryptographic signature

Reply via email to