Duplicate charges_charge_id_unique

[tin-robot]

SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '6005686372' for key 'charges_charge_id_unique'

I just got an interesting error recently. Any idea why this would happen? Wouldn't each charge be unique?

[gnikyt]

Hmmm... yes it should be unique from Shopify. Trying to think how this would trigger... I wonder if someone tried to accept a charge, Shopify failed or was down for some reason, then they tried to accept it again? By that time it would've been in the database possibly. Do you have any other information that might lead to how it would happen?

Edit: If thats the case, maybe I should do a findOrCreate on the charge ID to elevate this.

[tin-robot]

No idea. It was just sent to Sentry.

[gnikyt]

I think this is the issue. I'm going to modify it to do a find or create.

…

________________________________ From: Cory Fail <notifications@github.com> Sent: Tuesday, January 29, 2019 2:21:38 PM To: ohmybrew/laravel-shopify Cc: Tyler King; Comment Subject: Re: [ohmybrew/laravel-shopify] Duplicate charges_charge_id_unique (#175) No idea. It was just sent to Sentry. — You are receiving this because you commented. Reply to this email directly, view it on GitHub<#175 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ACTuOkTzPb_HrsVPmq6k-aTNG7Z51HU5ks5vIIoqgaJpZM4aWSAP>.

[gnikyt]

Release 5.2.0 includes a fix to the charge creation which does firstOrNew based on charge_id and shop_id only. So in the case of the above situation, a second go-around, it will allow it to go through and save. This should solve the edge case issue :) ill close for now, but if it pops up again in the future, please let me know.