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] move code into the function #22288

Merged
merged 1 commit into from
Dec 22, 2021
Merged

Conversation

eileenmcnaughton
Copy link
Contributor

@eileenmcnaughton eileenmcnaughton commented Dec 21, 2021

Overview

[REF] move code into the function

Before

Membership processing starts in the main function & continues in doMembershipProcessing

After

It's all in doMembershipProcessing

Technical Details

Within the if there is some code & then a function call. There doesn't seem
to be much reason for some of the code to be in the function & some
outside - this simplifies

Comments

Within the if there is some code & then a function call. There doesn't seem
to be much reason for some of the code to be in the function & some
outside - this simplifies
@civibot
Copy link

civibot bot commented Dec 21, 2021

(Standard links)

@civibot civibot bot added the master label Dec 21, 2021
@colemanw colemanw merged commit 0839808 into civicrm:master Dec 22, 2021
@colemanw colemanw deleted the process branch December 22, 2021 02:17
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