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

Switch to Call contribution api #20041

Merged
merged 1 commit into from
Apr 15, 2021

Conversation

eileenmcnaughton
Copy link
Contributor

@eileenmcnaughton eileenmcnaughton commented Apr 12, 2021

Overview

Switches to using the v3 contribution create api in preparation for switching to the v3 Order api

Before

Calls BAO create somewhat directly

After

Calls v3 contribution.create api. 2 params that could not be set are removed (this used to be shared code)

  • is_pay_later
    • skipLineItem

Technical Details

Comments

is_pay_later & skipLineItem could only be unset here
@civibot
Copy link

civibot bot commented Apr 12, 2021

(Standard links)

@civibot civibot bot added the master label Apr 12, 2021
@colemanw
Copy link
Member

colemanw commented Apr 15, 2021

I've read through this diff and looked at the difference between what APIv3 does vs the regular BAO. I don't see any gotchas.
Additional logic in the v3 create function includes:

  • Handle legacy param names (doesn't apply)
  • Handle financial permissions (only applies if check_permissions is passed in, and it isn't)
  • Cleans money (doesn't apply if skipCleanMoney is passed in, and it is)
  • Handles update if id is passed in (it isn't)

@colemanw colemanw merged commit 1e8d712 into civicrm:master Apr 15, 2021
@colemanw colemanw deleted the mem_cont_switch branch April 15, 2021 23:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants