Remove unreachable customGroup section in online event receipt
authoreileen <emcnaughton@wikimedia.org>
Sun, 29 Nov 2020 03:20:14 +0000 (16:20 +1300)
committereileen <emcnaughton@wikimedia.org>
Sun, 29 Nov 2020 03:20:14 +0000 (16:20 +1300)
commitffc37ca8b24922bc390a3f9e432f81ef2b129a71
tree093c4d5454034a7d9dae72fca857752e8977a3d0
parent5435c94d224625b0f9c52c5ef35ae0f84ea665d8
Remove unreachable customGroup section in online event receipt

After digging into https://github.com/civicrm/civicrm-core/pull/18697 I came to the
conclusion that currently both the online & offline event receipts have
sections for custom profiles and custom data (marked as customGroup).

The profiles are only assigned from the online flow and the data only from the offline
flow so in each receipt template one is unused.

I agree with the discussion on https://lab.civicrm.org/dev/core/-/issues/2076
that suggests that since every event has (potentially) assigned profiles
we could use those in both scenarios and never use the generic 'customGroup'.
The situation is less clear for contributions (where there is no profile
if there is no contribution page). Thus, I am inclined
to remove customGroup from the offline template too, after fixing to
include the profiles. However, there are additional
steps to get to the point where that is doable - at this stage I think
we CAN remove it from the online template because I cannot find anywhere
in the code where customGroup is assigned in that flow, nor logically
think of any reason other than copy and paste for it
xml/templates/message_templates/event_online_receipt_html.tpl
xml/templates/message_templates/event_online_receipt_text.tpl