dev/mail#62 - Mailing Error when civicrm_mailing_group has duplicate …

[jitendrapurohit]

…entries of recipient groups

Overview

Mailing Error when civicrm_mailing_group has duplicate entries of recipient groups

Before

Details on Gitlab - https://lab.civicrm.org/dev/mail/-/issues/62

After

max length of group_concat error is avoided by removing duplicate ids in it. The mailing is loaded and sent correctly.

Technical Details

The group_concat is still an issue if number of groups are more. But maybe, its too rare to include more than >250 groups in a single mailing? Assuming each group id is 2-3 char long with , separating them in group_concat.

Comments

Gitlab - https://lab.civicrm.org/dev/mail/-/issues/62

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[seamuslee001]

Change makes sense and looks fine to me merging