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

Order marked paid twice #23

Closed
stenvdb opened this issue Nov 25, 2019 · 14 comments
Closed

Order marked paid twice #23

stenvdb opened this issue Nov 25, 2019 · 14 comments
Assignees
Labels

Comments

@stenvdb
Copy link

stenvdb commented Nov 25, 2019

Description

In our commerce shop we got an order that has been marked as paid twice (aka the double of the actual amount).

I have tried setting the totalPaid db column manually to the correct amount, but that doesn't show up in the CMS (caching?). Have sent an email about this to craft support, but have gotten no response since oct. 25th. So I'll try here.. The payment gateway is Mollie. Is there some kind of race condition that is not 100% bullet proof?

Steps to reproduce

Additional info

  • Craft version: 3.3.6
  • PHP version: 7.2
  • Database driver & version:
  • Plugins & versions: Commerce: 2.1.13, Mollie: 2.1.0.1
@stenvdb
Copy link
Author

stenvdb commented Nov 26, 2019

Just heard from support and this might be related to craftcms/commerce-sagepay#10 and wil get fixed in the mollie gateway. Thanks @lukeholder 🎉

@stenvdb
Copy link
Author

stenvdb commented Dec 5, 2019

Any update on when this could get fixed in the mollie plugin?

I would also like to know the correct way to fix this in the database, so that our client can see the correct paid amount in the control panel (and the invoices etc.)

@stenvdb
Copy link
Author

stenvdb commented Jan 20, 2020

Any update on this? Changing the totalPaid column does not change the amount displayed in the cp. @nfourtythree

@lukeholder lukeholder transferred this issue from craftcms/commerce Jan 23, 2020
@nfourtythree
Copy link
Contributor

Hi @stenvdb

We have pushed an update to webhooks for mollie that could solve this issue.

I posted about how to update to the pre-release and asked for feedback to see if this helped with issues.

Would be great if you could see if this solves the problems you are seeing.

Thanks!

@stenvdb
Copy link
Author

stenvdb commented Jan 27, 2020

Hi @nfourtythree , will update asap, and wil let you know if this would happen again. Thanks!

Could you help me fix the issue above though? Changing the totalPaid column in the database, does not update in the cp. Can't figure this out.. We need to do something so the double paid amount gets fixed in the invoices etc.

@stenvdb
Copy link
Author

stenvdb commented Mar 16, 2020

@nfourtythree I have tracked down the issue to 2 records in the transaction table for the specific order, both marked as status 'success'. I assume the webhook fix in commerce-mollie 2.1.1. (#22) prevents this from happening in the future.

For now (as I don't have time to update), I have set 1 of the 2 records as failed, making the totalPaid appear as it should.

As far as I'm concerned, this can be closed.

@nfourtythree
Copy link
Contributor

Hi @stenvdb

Thanks for the update!

@PieterxJan
Copy link

Hi guys,

I still have this issue in a project for a client of us. As Sten stated, there are two transaction records, created on the exact same time, with the success status.

We have the latest version (2.1.1). I can fix this issue by setting the status manually, but the invoices are incorrect when they're generated because of this issue.

This is all on a production environment. I can't seem to reproduce this on my local dev environment.

Additional info

  • Craft version: 3.4.18
  • PHP version: 7.2.30
  • Craft Commerce version: 3.1.3

@stenvdb
Copy link
Author

stenvdb commented Jun 1, 2020

Hi @nfourtythree, unfortunately as @PieterxJan stated, I too have noticed that the issue is still not resolved. This is at least for our clients, a very critical bug. Because of this bug our client thought initially they sold way more than they actually did.

Of course logging in to mollie qiuckly shows you a correct sale amount. But because of this bug, for example the Commerce revenue dashboard widgets show a completely false overview of your shop sales. Very hard to explain to our clients.

If you want I can share a database dump, or even access to the site. Hopefully it can get fixed soon 🤞 Thanks for looking into this!

@stenvdb
Copy link
Author

stenvdb commented Jun 8, 2020

@brandonkelly @angrybrad Sorry to bug you on this ticket, but it has been 2 weeks without a reply. We have several shops running on Mollie and hope to get this fixed. 🙏

@nfourtythree
Copy link
Contributor

Hi @stenvdb

Thank you for emailing support, we have sent you a reply there.

Just to keep everyone updated, this is something we are looking into and will update the issue when we have some information to share.

Will re-open this for visibility.

Thanks.

@alexthn
Copy link

alexthn commented Jun 15, 2020

Just to let you guys know, we also face this problem. For us it doesn't happen all the time but it does at least a couple of times a week.

@nfourtythree
Copy link
Contributor

Hi All

Depending on what version of Commerce you are on we have just pushed 2.2.21 and 3.1.9.

The appropriate release and version 2.1.2 should fix this issue. Please update and let us know how you get on.

Thanks for all of your help in tracking this down.

@stenvdb
Copy link
Author

stenvdb commented Jun 18, 2020

Thank you @nfourtythree! I will update our sites running on mollie today and keep an eye on it.

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

No branches or pull requests

4 participants