Closed (fixed)
Project:
Commerce Popup Cart
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
9 Jun 2016 at 08:12 UTC
Updated:
3 Jan 2017 at 17:24 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
rei commentedComment #3
manish.shah commentedHi Rei,
I have fixed the issue and attached patch for same. Kindly review and let me know if you find anything.
-Manish
Comment #4
rei commentedHi, the patch #3 fix this problem
thank you for fast response
Comment #5
manish.shah commentedComment #7
3cwebdev commentedCommitted to dev.
Comment #8
rei commentedhi, this is also has not commited to the dev
Comment #10
3cwebdev commentedCommitted to DEV.
Comment #11
pontus.froden commentedThanks this patch also solves the problem with selling giftcards through commerce_gc. The popup cart said the shopping card was empty when there only was a giftcard in the basket.
Haven't found any issues with it yet.
Comment #12
pontus.froden commentedJust discovered one error with this. If you apply a coupon to the cart then it will include that line when it show number of products in cart.
If you have 2 items in cart and go to checkout and add a coupon the popup will say it is 3 items in cart.
Will fix a patch as soon as I have the time.
Comment #13
3cwebdev commented@pontus.froding - I just tested this on a live install with the latest version of this module and the item count works correctly for me. I added an item to the cart and the cart pop showed a count of 1 item. I then added a coupon and the the count stayed at 1. Can you provide the exact steps to reproduce the issue?
Comment #14
pontus.froden commentedMaybe it is because my coupon is a giftcard from commerce_gc? Forgot to say that before.
Haven't looked into it anymore yet but the issue happens if I go to checkout, add the giftcard and then either refresh the checkout page or go to another page for some more shopping ...
I would say it's not a big issue, most of the customers would probably not refresh the checkout page or do some more shopping after they have added a giftcard.
Comment #15
pontus.froden commentedHere is a printscreen of the cart
Comment #16
pontus.froden commentedHere is the patch.
First I was thinking about just adding the line item type "giftcard_use" and "product_discount" in a list to exclude but then we would have the problem again if someone adds another line item type. So I made a setting for it in admin config.
Product is choosen by default, if you need to include another one then you just choose it in the Popup Cart Config.
Comment #18
3cwebdev commentedPatch committed to dev.
Comment #19
3cwebdev commented