Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[5.x]: Mixed currencies when viewing order in CP #3891

Open
tommysvr opened this issue Feb 12, 2025 · 2 comments · May be fixed by #3893
Open

[5.x]: Mixed currencies when viewing order in CP #3891

tommysvr opened this issue Feb 12, 2025 · 2 comments · May be fixed by #3893

Comments

@tommysvr
Copy link
Contributor

What happened?

Description

An order made on a non-primary site will display certain items in the currency of the primary site.

Steps to reproduce

  1. Create two sites and stores: Site A - USD / Site B - GBP
  2. Make an order from Site B in GBP (either from the front-end or inside control panel)
  3. When viewing the order in the control panel, line items and adjustments will show as USD, but the total will show as GBP

Expected behavior

All values are shown in the currency of the store the order was placed (GBP in this case)

Actual behavior

Currencies are mixed; line items and adjustments are in the primary store currency, total is in the currency of the store the order was placed

Other things I've checked

  • Language of Site B is en-GB
  • Language & Formatting Locale set on my user is en-GB

Image

Craft CMS version

5.6.6

Craft Commerce version

5.3.2.2

PHP version

No response

Operating system and version

No response

Database type and version

No response

Image driver and version

No response

Installed plugins and versions

Copy link

linear bot commented Feb 12, 2025

@lukeholder lukeholder self-assigned this Feb 13, 2025
@lukeholder lukeholder linked a pull request Feb 13, 2025 that will close this issue
@lukeholder
Copy link
Member

Thanks for reporting. I have a PR that fixes this issue and will update here once the fix is included in a release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants