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. >
smime.p7s
Description: S/MIME cryptographic signature