Payment's currency handling was written to work with the euro. However, there are currencies that work differently (number of decimals, fractions...) that Payment should also be able to process.

CommentFileSizeAuthor
#8 payment_1446076_8.patch297 bytesxano

Comments

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
Title: Fix currency handling » Fix payment amount display

For the record: Payment works with any currency, even fictional ones, but it's the display of payment amounts that's sub-optional.

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
Title: Fix payment amount display » Add Currency as a dependency
Status: Active » Postponed

Currency will be rewritten to take care of localized currency display and it will be a dependency for Payment 7.x-2.x.

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented

Currency features we'll want to leverage:
- Localized amount display (hardcoded and using CurrencyAmountViewsHandlerField)
- Currency display using CurrencyViewsHandlerField
- The currency_amount form element as part of the payment_line_item form element.

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
Category: bug » feature
helmo’s picture

helmo commented

Maybe for the 1.x branch a hook_requirements() implementation could check if there is a currency?

In a payment_webform I was slightly puzzled by an empty currency select box.

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented

@helmo, see #1991740: Add currency as a dependency for that.

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
Version: 7.x-1.x-dev » 8.x-2.x-dev
Status: Postponed » Active
xano’s picture

xano
he/they
Primary language English
Location Southampton
commented
Status: Active » Fixed
StatusFileSize
new payment_1446076_8.patch297 bytes

I added the dependency in anticipation of #2040229: Port line item form elements, so we won't get any test failures due to outdated cached dependencies on the testbot.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.