Skip to content
Snippets Groups Projects
Unverified Commit d5080251 authored by JonGold's avatar JonGold
Browse files

fixes #3661: Don't check API permissions in CRM_Mailing_BAO_Mailing::getGroupNames()

parent 084274a5
Branches
Tags
No related merge requests found
......@@ -1413,7 +1413,12 @@ ORDER BY civicrm_email.is_bulkmail DESC
return [];
}
$mailingGroups = \Civi\Api4\MailingGroup::get()
/*
This bypasses permissions to maintain compatibility with the SQL it replaced. This should ideally not bypass
permissions in the future, but it's called by some extensions during mail processing, when cron isn't necessarily
called with a logged-in user.
*/
$mailingGroups = \Civi\Api4\MailingGroup::get(FALSE)
->addSelect('group.title', 'group.frontend_title')
->addJoin('Group AS group', 'LEFT', ['entity_id', '=', 'group.id'])
->addWhere('mailing_id', '=', $this->id)
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment