Active
Project:
Drupal Commerce Extra Price Formatters
Version:
7.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
12 Dec 2012 at 14:52 UTC
Updated:
14 Aug 2014 at 07:51 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
vasikehere is patch that deletes the value for the checkbox, si it can be changed.
Comment #2
ikos commentedThanks for reporting! Committed to latest dev.
Comment #4
joerch commentedHi!
I tested the 7.x-1.1+7-dev but it's not working here. Not calculating discount and tax.
Comment #5
heyyo commentednot working also for multicurrency. It displays default currency even if "Show calculated price" enabled.
Comment #6
heyyo commentedComment #7
philyHi,
It's not working here neither: it always shows calculated price (inc. VAT) whatever the state of the checkbox.
The website I'm designing needs to display prices excluding VAT if products pages, cart block/form even if the checkout process includes it.
Thanks for any help.
Comment #8
tajindersingh commentedThis doesn't work at all. 'Show calculated price' marked for 'Price with Prefix / Suffix' doesn't take affect ever.
Similar thing (the one removed in patch) exists for other two formatters in this module i.e. '#value' => TRUE
It seems the values are not getting saved or are if they are then the saved values don't have any effect on the prices displayed.
This field should better be replaced with below, for all of the filters:
Comment #9
tajindersingh commentedUpdating priority as this bug makes the module unusable.
Comment #10
anybodyStill no change here? Is there an active maintainer?
Comment #11
anybodyThe patch works great for me, the value is being saved correctly (@ #8 I think you forgot to save the whole form after setting the value in the subform).
Anyway from the code perspective the patch is correct, but the logic should be applied to ALL formatters provided by this module. The patch only solves the problem for "commerce_price_prefix_suffix".
Furthermore this kind of code bug requires a new STABLE release from my point of view.