[Relay/Snippets] Test group-based snippets in 'Before sending mail from worker' behaviors from relay

[jstanden]

Source: https://twitter.com/mattmartincouk/status/973008952618348545

[iammattmartin]

To make this a bit more serious, it turns out when replying via the API, the same thing happens.

We can only send mail via the GUI to have this data properly populated.

[iammattmartin]

To add more to this, when you send a first response via the API (i.e first is outbound), the data IS populated. It just appears to be on non-initial emails.

[iammattmartin]

Further info.

We set a custom field on a group called "Public", which will show the matching groups in our customer portal.

We have a bot that will set the referenced signature where it gets the group snippets but I have determined that even the bot logic cannot see the group which is why the group snippet data is probably blank.

(Added {{group_custom_27}} to output the data and it is blank when message sent from worker via the API, relay and mobile interface but is populated when via the GUI).

[jstanden]

Looking into this now.

[jstanden]

The issue here appears to be that if you don't explicitly provide a group_id or bucket_id on a reply from the API, we aren't defaulting them. The UI always set them because they're in the form. With the relay or mobile UI you didn't even have the option.

I tweaked the code to default to the ticket's current values when not provided.
cerb-625.patch.txt

[jstanden]

Fixed in 8.3.3