I am comparing RC2 with the dev version I am using. There seems to be mixed updates.
uc_cart.module:
uc_cart_get_item:
The RC2 adds the following line to make display_price work.
$item->display_price = $product->display_price;
uc_cart_get_contents RC2 has:
$item->display_price = $product->sell_price;
while the dev version has:
$item->display_price = $product->display_price;
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | 1305634-uc_cart_get_item.patch | 2.44 KB | longwave |
| #5 | 1305634-uc_cart_get_item.patch | 2.44 KB | longwave |
| #2 | 1305634-uc_cart_get_item.patch | 2.35 KB | longwave |
Comments
Comment #1
longwaveGood catch, fix committed. uc_cart_get_item() isn't actually used in core, so this would only have affected contrib. We could reduce code duplication here, so I'm retitling this issue to deal with that.
Comment #2
longwaveComment #3
Island Usurper commentedGood fix. Committed.
Comment #4
longwaveMight as well do the same in 6.x.
Comment #5
longwaveComment #6
longwaveOops, use this one
Comment #8
longwaveTest failures are unrelated:
User mFxJ6ot1myk856JgxCR5 successfully logged in.
Raw "While logged in" found
Seems to be something to do with the random username/password not always working here?
Comment #9
longwave#6: 1305634-uc_cart_get_item.patch queued for re-testing.
Comment #10
longwaveCommitted #6