Skip to content

Add financial type and payment method to receipts issued report (Option A)

DaveD requested to merge DaveD/cdntaxreceipts:report-fintype into master

This is option A for adding financial type and payment method to the receipts issued report. It looks up the CURRENT VALUE from the corresponding contribution.

Before this PR, except for contact name which is a bit inconsistent, this report uses the taxreceipt log record to get the values, so that they correspond to what was on the issued receipt, and don't change if you later edit the contribution. That's a good thing.

Since financial type and payment method aren't stored in the log, this Option A looks them up from the contribution, which means they might be different from when the tax receipt was issued if somebody edited the contribution since then. So another option B would be to start storing financial type and payment method in the log too.

Merge request reports