CiviCRM Community Forums (archive)

Hi All,

I am working on the extension that needs to alter mail parameter so I use hook_civicrm_alterMailParams. However, the extension requires the mailing job_id. I drilled down into the MailingBAO and Hook classes the solution would be.

1. Modified alterMailParams hook for passing job_id as below:

**
 * Implementation of hook_civicrm_alterMailParams( )
 */
function mailjet_civicrm_alterMailParams(&$params, $context, $jobId) {
  if(isset($jobId)){
   //do something
  }
}

2. The second one is to add  job_id as parameter before passing to the hook in Mailing BAO and unset after the hook was called

    //@erawat added $job_id
    $mailParams['job_id'] = $job_id;
    CRM_Utils_Hook::alterMailParams($mailParams, 'civimail');
    unset($mailParams['job_id']);

Which one is more practical to implement as I'd like to contribute it back to the core.

Thanks,

Erawat

The requirement on this hook I hit yesterday was to have the hook set a param like $mailParams['skip'] & have the mail not go out. In this case it's because our customers have taken fright at the fact that CiviCRM ignores 'do_not_email' for reminders which means they have to use them with extreme caution.

This doesn't seem like the only use case - I guess the issue would be what does happen - does an activity still get created?