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

Finish conversion of static function to non-static #23042

Merged
merged 1 commit into from
Apr 1, 2022

Conversation

eileenmcnaughton
Copy link
Contributor

Overview

Finish conversion of static function to non-static

Before

processSecondaryContribution accepts $form as a paramter - the form variable === $this

After

Ditch the $form var - use $this

Technical Details

The processSecondaryContribution function is no longer static but the class still passes itself
to the function as the form parameter. This updates calls to form to calls to this.

Also for clarity the function is made private

I did do a quick universe search to confirm no calls to it - even thought they
would have been clearly unsupported

Comments

@civibot
Copy link

civibot bot commented Mar 26, 2022

(Standard links)

@civibot civibot bot added the master label Mar 26, 2022
The processSecondaryContribution function is no longer static but the class still passes itself
to the function as the form parameter. This updates calls to form to calls to this.

Also for clarity the function is made private

I did do a quick universe search to confirm no calls to it - even thought they
would have been clearly unsupported
@colemanw
Copy link
Member

colemanw commented Apr 1, 2022

Makes sense. Merging based on code-review & passing tests.

@colemanw colemanw merged commit e37fd5d into civicrm:master Apr 1, 2022
@eileenmcnaughton eileenmcnaughton deleted the sane branch April 1, 2022 20:43
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