Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Patch will follow.
Comment | File | Size | Author |
---|---|---|---|
#7 | 2454683-7.order_total_area_handler.patch | 4.92 KB | rszrama |
#5 | commerce-allow_to_change_order_total_display_settings-2454683-5.patch | 4.86 KB | RoSk0 |
Comments
Comment #1
RoSk0Patch.
Comment #2
joelpittetThis kinda looks like it resolves this @todo no?
This patch looks great, I don't currently have need for this but I'm sure someone does!
Comment #3
RoSk0About TODO: it looks so, but I wasn't sure and decided to leave it as is. Should I get rid of it?
Comment #4
joelpittetIf we can find the person who wrote it that would be best, but I'd say maybe remove it and if someone says anything it can be added back.
Comment #5
RoSk0@rszrama added this in 983181f9dae9f1cdedcb960a10eaf964ac6ebc42.
New patch version without TODO comment.
Comment #6
torgosPizzaThis does exactly what we need, so thanks! Basically we have the coupon form on the Cart page from #2872455: Allow coupons to be redeemed on the cart page, but when adding a coupon, the order total currently shown does not reflect the discount. This also happens with Gift cards and other site-wide discounts: the Order Total handler for Views seems to not calculate prices by default, which seems like an odd choice, but in any case this patch allows us to show users more information about their current cart, and avoids confusion when the current price is not reflected there.
Comment #7
rszrama CreditAttribution: rszrama at Centarro commentedI've updated the patch to make code, comment, and interface changes. I changed the help text when enabling a custom label to make it clear that theme updates may be required since adding a label breaks some default CSS. Otherwise, looks good to go!
Comment #9
rszrama CreditAttribution: rszrama at Centarro commentedCommitted.