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] Extract code to get the from address for a recurring contribution. #21357

Merged
merged 1 commit into from
Sep 6, 2021

Conversation

eileenmcnaughton
Copy link
Contributor

@eileenmcnaughton eileenmcnaughton commented Sep 3, 2021

Overview

[REF] Extract code to get the from address for a recurring contribution.

Before

Long code block

After

spiffing new function

Technical Details

This makes the retrieval generally available, since is it part of the data model,
rather than tied to the form. It could possibly be a token, or at least retrievable on
the recurring edit workflow template - but that is beyond the scope of this

Note that in some cases this could result in an extra query but in others it might make the query quicker. I don't think an extra query in a form post process is a biggie (deep in the BAO it could be)

This parameter is specifically covered in the test I pre-added but I updated the test so that it was using the more 'complicted' variant - ie retrieving from a contribution page

Comments

@seamuslee001 are you able to MOP this - I think it's pretty straight forward & will make code conversations easier if we get rid of this noise

@civibot
Copy link

civibot bot commented Sep 3, 2021

(Standard links)

This makes the retrieval generally available, since is it part of the data model,
rather than tied to the form. It could possibly be a token, or at least retrievable on
the recurring edit workflow template - but that is beyond the scope of this
@colemanw
Copy link
Member

colemanw commented Sep 6, 2021

Refactor makes sense to me.

@colemanw colemanw merged commit de82140 into civicrm:master Sep 6, 2021
@colemanw colemanw deleted the recur_from branch September 6, 2021 02:24
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