Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • M mjwshared
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 2
    • Issues 2
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 1
    • Merge requests 1
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Extensions
  • mjwshared
  • Merge requests
  • !29

Failed payments are treated as successful

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged JonGold requested to merge JonGold/mjwshared:throw-error-exceptions into master Mar 03, 2022
  • Overview 2
  • Commits 1
  • Pipelines 0
  • Changes 1

Steps to replicate

  • Install D9, webform, webform_civicrm, mjwshared and authnet.
  • Create any form that can accept a contribution (set in test mode).
  • Submit a card with an expiration date in the past.

Expected Behavior

  • Card is declined, contribution isn't recorded, user receives an on-screen error.

Actual behavior

  • Card is declined, contribution is recorded as "Complete" in CiviCRM, user receives a message that their transaction is complete.

I traced this down to the "@fixme - Throw an exception" in the handleError() method. I made what I believe are the necessary changes in authnet, mjwshared, and stripe. I can confirm that this fixes my scenario, but I wanted to get feedback on whether I was missing something obvious, because it seems like the sort of thing that you would've done if it was straightforward.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: throw-error-exceptions