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

Partial Refunds show Fully Refunded #8

Open
n9yty opened this issue Mar 8, 2019 · 0 comments
Open

Partial Refunds show Fully Refunded #8

n9yty opened this issue Mar 8, 2019 · 0 comments

Comments

@n9yty
Copy link

n9yty commented Mar 8, 2019

First, this is brilliant, a great help, so thanks so much for sharing!

One small issue, I am using this to test some changes to emails sent out when refunds exist on an order, both the refund email itself and any subsequent like order complete, etc.

The header of the email says "Your order has been fully refunded", even if only a partial refund was applied.

I looked at the template code here: woocommerce/templates/emails/customer-refunded-order.php

It assumes a variable $partial_refund is set to indicate a partial refund. My guess is that when this is being called here, that value is just not calculated before calling the template? I'm not sure where exactly that is getting set, the template code just assumes it will be there. :)

If it helps, I found a little breakdown on how refunds are stored in the DB and how to determine if it is a partial or full refund.

https://stackoverflow.com/questions/37712789/how-are-partial-refunds-stored-in-the-woocommerce-database

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

No branches or pull requests

1 participant