Validation error during recurring charge, in field: processor_contact_id. Message: Input should be a valid UUID, invalid character: expected an optional prefix of urn:uuid: followed by [0-9a-fA-F-], found .
248169602.1
248170334.1
248169925.1
248140922.1
248169660.1
248139545.1
248138563.1
248133159.1
Description
Description
Details
Details
Related Changes in Gerrit:
Customize query in gerrit
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | BUG REPORT | AnnWF | T431327 Prevent to assign order id to processor_contact_id | ||
| Resolved | AnnWF | T432484 Add catch to adyen audit to not set processor_contact_id when its a gravy adyen transaction |
Event Timeline
Comment Actions
Change #1309698 had a related patch set uploaded (by Wfan; author: Wfan):
[wikimedia/fundraising/SmashPig@master] set the processor_contact_id to null if not uuid
Comment Actions
Change #1309698 merged by jenkins-bot:
[wikimedia/fundraising/SmashPig@master] set the processor_contact_id to null if not uuid
Comment Actions
select entity_id, processor_contact_id from civicrm_contribution_recur cr JOIN civicrm_contribution_recur_smashpig crs ON crs.entity_id = cr.id WHERE cr.payment_processor_id = 19 AND crs.processor_contact_id IS NOT NULL AND crs.processor_contact_id NOT REGEXP '^[0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{12}$' AND cr.cancel_date IS NULL;
+-----------+----------------------+
| entity_id | processor_contact_id |
+-----------+----------------------+
| 2819316 | 248133922.1 |
| 2819319 | 248140664.1 |
| 2819325 | 248170032.1 |
| 2819366 | 248397423.1 |
| 2820095 | 248169661.1 |
| 2820096 | 248169670.1 |
| 2820098 | 248170072.1 |
| 2820196 | 248420030.1 |
| 2820480 | 248169984.1 |
| 2820485 | 248169794.1 |
| 2820486 | 248169854.1 |
| 2827412 | 248649937.1 |
| 2827413 | 248648953.1 |
| 2829405 | 248648126.1 |
| 2829406 | 248648452.1 |
| 2829407 | 248648776.1 |
| 2829408 | 248649536.1 |
| 2829409 | 248650233.1 |
| 2829410 | 248650894.1 |
| 2829411 | 248650993.1 |
| 2842709 | 249038625.1 |
| 2842712 | 249061167.1 |
| 2843532 | 249164367.1 |
| 2843909 | 249115985.1 |
| 2843910 | 249161769.2 |
| 2845119 | 249162142.1 |
| 2852620 | 249240955.1 |
| 2854229 | 249355336.1 |
| 2854230 | 249373870.1 |
| 2854689 | 249460156.1 |
| 2854690 | 249459943.1 |
| 2854691 | 249460544.1 |
| 2854692 | 249461174.1 |
| 2854693 | 249461404.1 |
| 2856387 | 249458583.1 |
| 2856388 | 249458970.1 |
| 2856389 | 249459004.1 |
| 2856390 | 249459420.1 |
| 2856391 | 249459692.1 |
| 2856392 | 249460700.1 |
| 2856393 | 249461266.1 |
| 2856394 | 249461502.1 |
| 2862305 | 249571057.1 |
+-----------+----------------------+
then I cleaned up:
UPDATE civicrm_contribution_recur_smashpig SET processor_contact_id = NULL WHERE entity_id IN (xxx,xxxx);
Comment Actions
We hunted down that they are getting into civi in the first place from the adyen audit, the adyen adyen transactions use this column while gravy adyen doesnt. The adyen audit wins over the gravy audit so they are coming in from that. To plug the hole we can add a catch to exclude gravy adyen from adding this field in the audit
Comment Actions
https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/SmashPig/+/819220 some history reference of why we have assigned processor_contact_id as order_id to get tokenization info from adyen if not there.
Comment Actions
Change #1312011 had a related patch set uploaded (by Wfan; author: Wfan):
[wikimedia/fundraising/crm@master] Do not assign processor_contact_id from order id if not need to tokenize
Comment Actions
Change #1314123 had a related patch set uploaded (by Ejegg; author: Ejegg):
[wikimedia/fundraising/SmashPig@master] Only assign valid UUIDs to gravy buyer_id
Comment Actions
Change #1314127 had a related patch set uploaded (by Ejegg; author: Ejegg):
[wikimedia/fundraising/SmashPig@master] Fix validator mutation for buyer_id and names
Comment Actions
Change #1314123 abandoned by Ejegg:
[wikimedia/fundraising/SmashPig@master] Only assign valid UUIDs to gravy buyer_id
Reason:
Fixing WFan's filter instead: I103d17e77f8367882c19d842593a8afaccb06cb8
Comment Actions
Change #1314127 merged by jenkins-bot:
[wikimedia/fundraising/SmashPig@master] Fix validator mutation for buyer_id and names
Comment Actions
Change #1312011 abandoned by Wfan:
[wikimedia/fundraising/crm@master] Do not assign processor_contact_id from order id if not need to tokenize
Reason:
https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/SmashPig/+/1309698 catch it before send for process
Comment Actions
Change #1312011 restored by Wfan:
[wikimedia/fundraising/crm@master] Do not assign processor_contact_id from order id if not need to tokenize