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

Total Paid in order index is always $0 #1482

Closed
engram-design opened this issue Jun 1, 2020 · 3 comments
Closed

Total Paid in order index is always $0 #1482

engram-design opened this issue Jun 1, 2020 · 3 comments

Comments

@engram-design
Copy link
Contributor

Total Paid in order index is always $0, despite it being correct when viewing the order detail screen.

Refer to:
Screen Shot 2020-06-01 at 2 41 51 pm

But is all correct:
Screen Shot 2020-06-01 at 2 44 15 pm

In addition, there are multiple "Total" options to pick from in the element index, which is confusing.

Screen Shot 2020-06-01 at 2 42 26 pm

Craft CMS 3.4.22.1
Commerce 3.1.5

@lukeholder
Copy link
Member

Looking into this now.

Renaming Totals to All Totals since it is a summary column that shows multiple totals.

Leaving the first Total as is, and the second one to Total Price. Total Price is different to Total as it also checks the minimumTotalPriceStrategy setting.

If you resave the order, does it update the total?

Label fixes: 639cbc6

@engram-design
Copy link
Contributor Author

Thanks, and re-saving the order doesn't change anything.

@lukeholder lukeholder self-assigned this Jun 1, 2020
@lukeholder
Copy link
Member

{%  set commerce = craft.app.getPlugins().getStoredPluginInfo('commerce') %}
Version: {{ commerce.version }}

Using the above the version of Commerce was being reported as dev-develop and not a reaal version number due to composer.json configuration. Making sure to be on a real version number fixed the issue due to this order query logic for totals: https://github.com/craftcms/commerce/blob/develop/src/elements/db/OrderQuery.php#L973-L983

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

No branches or pull requests

2 participants