-
-
Notifications
You must be signed in to change notification settings - Fork 814
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Extract code to set isEmailReceipt in Contribution.completeOrder
This consolidates all the code in the completeOrder that determines whether or not to send an email receipt. Note that - testCompleteTransactionWithEmailReceiptInput tests that input is respected - for events the event::sendMail function will not send unless it's configured for the event - which explains our final default of TRUE (testCompleteTransactionWithParticipantRecord) - testCompleteTransaction covers the back-office case which also defaults to true - I figure the cost of reloading the value for contribution_recur is pretty cheap compared to the confusion of passing values around - testpayLater covers contribution_page with is_email_receipt
- Loading branch information
1 parent
d40ae7d
commit ee022fa
Showing
1 changed file
with
35 additions
and
25 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters