Skip to content

Bug fix in report, to include contributions with mixed financial types

Martin requested to merge github/fork/PeaceWorksTechnologySolutions/master into master

I'm not sure if this applies to civi across the board or if this site has a strange usage. But in this site's case, they have online event payments using a price set that includes both event fees (non-tax-deduct) and donations (tax-deduct). The contribution objects in the db are created with the financial type in that table being event fee, which caused this report to skip over them (fixed now with this commit). The line item objects in the db have the correct financial types and otherwise civi and receipting appears to be correct.

This fix is rather important in this case, as without it the organization would likely miss receipting some donations.

Merge request reports