Civi shows duplicate PayPal donations
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	AMJohnson
	Tue, Jun 21, 4:32 PM

Description

We've noticed some new records where the Civi record is showing two separate contributions for one donation made via PayPal.

The transaction numbers for the two transactions are the same, so we were thinking Civi is possibly just duplicating the transaction? Each of the second transactions in civi say "dup-[transactionid]" which we were thinking possibly dup = duplicate?

There are possibly many more, but the few we've spotted so far are:

ticket #	CID	Date of transaction	Paypal Transaction #
1131032	CID 55865495	06/20/2022	2N835366VC341811Y
1130804	CID 55851768	06/19/2022	1WA27673UX1838908
1130815	CID 55857356	06/19/2022	49A63104CP268790R

Details

	Project	Branch	Lines +/-	Subject
	wikimedia/fundraising/tools	master	+8 -2	PayPal audit: search one-time donations under both tags

Customize query in gerrit

Event Timeline

AMJohnson created this task.Tue, Jun 21, 4:32 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptTue, Jun 21, 4:32 PM

krobinson added a subscriber: RKumar_WMF.Tue, Jun 21, 4:41 PM

This is donor-facing, as @krobinson noted they are receiving extra thank you emails.

XenoRyet moved this task from Triage to Current Sprint on the Fundraising-Backlog board.Tue, Jun 21, 7:48 PM

XenoRyet added a project: Fundraising Sprint Localhost Hospitality Studies.Tue, Jun 21, 8:31 PM

Ejegg claimed this task.Wed, Jun 22, 7:51 PM

Ejegg moved this task from Backlog to Doing on the Fundraising Sprint Localhost Hospitality Studies board.

Looks like the audit parser is incorrectly tagging some donations as legacy 'paypal' instead of 'paypal_ec'. Since we search for existing transactions based on the processor tag plus the processor-side id, this incorrect processor tag makes it look like a new donations.

We can try to fix the heuristic that the audit parser uses to decide which integration the payment came from, or we can add a special case to the import's duplicate donation handling to discard them in the import.

In any case, once we move all the existing paypal donations to braintree, the whole paypal/paypal_ec confusion will be a thing of the past.

Change 807612 had a related patch set uploaded (by Ejegg; author: Ejegg):

[wikimedia/fundraising/tools@master] PayPal audit: search one-time donations under both tags

https://gerrit.wikimedia.org/r/807612

gerritbot added a project: Patch-For-Review.Wed, Jun 22, 8:11 PM

Ejegg moved this task from Doing to Review on the Fundraising Sprint Localhost Hospitality Studies board.Wed, Jun 22, 8:12 PM

Change 807612 merged by jenkins-bot:

[wikimedia/fundraising/tools@master] PayPal audit: search one-time donations under both tags

https://gerrit.wikimedia.org/r/807612

Maintenance_bot removed a project: Patch-For-Review.Thu, Jun 23, 6:30 PM

Ejegg moved this task from Review to Done on the Fundraising Sprint Localhost Hospitality Studies board.Thu, Jun 23, 7:22 PM

XenoRyet closed this task as Resolved.Tue, Jul 5, 7:17 PM