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

[REF] replace deprecated calls with buildOptions #19578

Merged
merged 1 commit into from
Feb 13, 2021

Conversation

MegaphoneJon
Copy link
Contributor

Overview

I'm replacing two calls to deprecated functions with their modern equivalent.

Before

CRM_Contribute_PseudoConstant::paymentInstrument()

After

CRM_Contribute_BAO_Contribution::buildOptions()

@civibot
Copy link

civibot bot commented Feb 10, 2021

(Standard links)

@civibot civibot bot added the master label Feb 10, 2021
@demeritcowboy
Copy link
Contributor

This looks good. Name/label and is_active seem to work out the same.

To save microseconds a future PR could maybe improve how it gets checkPaymentID in the first place, instead of getting all the options and then doing array_search.

@eileenmcnaughton eileenmcnaughton merged commit ceb0736 into civicrm:master Feb 13, 2021
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.

3 participants