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.
Those currencies that have no decimals should not have a minor_unit defined in the commerce_commerce_currency_info array
Also delete dead currencies, like CLE (Chilean Escudos) which are no more in circulation
Comment | File | Size | Author |
---|---|---|---|
#3 | 1061330.patch | 838 bytes | mikejoconnor |
#2 | 1061330.patch | 9.39 KB | Haza |
Comments
Comment #1
Cookiz CreditAttribution: Cookiz commentedComment #2
HazaPatch attached, all minor unit have been deleted for currencies that have decimal = 0.
But, are you sure that thoses currencies doesn't use the minor unit anymore ? It could make sense to, at least, keep the name information for the minor unit. What do you think about that ?
Also removed CLE, don't know if there are some other "dead" currencies.
Comment #3
mikejoconnor CreditAttribution: mikejoconnor commentedHaza, Thanks for the work on this. I'm a bit concerned that removing this information may be a problem in the future.
So I propose that we remove the dead currency, and we remove the minor unit for Vanuatu Vatu, because it is set to None.
I think we should leave the minor currency names for all of the other currencies that do have them, even if they are not used.
Here's a new patch.
Comment #4
HazaI was thinking the same :)
Seems good for me.
Comment #5
rszrama CreditAttribution: rszrama commentedCommitted! Huzzah!
Comment #6
rszrama CreditAttribution: rszrama commented