[Resolved] Replacing existing email with ‘cred_notification_recipient’

[Author]

Posts

[tinaH]

This relates to an older topic I created before with Beda: https://toolset.com/forums/topic/does-cred_notification_recipients-apply-to-scheduled-notifications/

My code works but it ADDS an additional recipient. I want it to replace the existing recipient.
Reading through the conversation my understanding is that the filter is supposed to replace ... not add.

This is what I have:

function b_invoice_mail($recipients, $notification, $form_id, $post_id) {
    if (5803 == $form_id) {
        //applies to all invoice form notifications
        $exhibitor_id = get_post_meta($post_id, '_wpcf_belongs_exhibitor_id', true);
        if (!empty($exhibitor_id)) {
            $email = get_post_meta($exhibitor_id, 'wpcf-email', true);
        } else {
            $email = get_post_meta($post_id, 'wpcf-book-exhib-mail', true);
        }
        if (!empty($email)) {
            $recipients[] = array(
                'to'        =>  'to',
                'address'   => $email,
                'name'      =>  '',
                'lastname'  =>  ''
            );
        }//else use form email settings
    }
    //this hook requires 'return $recipient' here, else disabling notifications for all forms
    //plain return; does not work
    return $recipients;
}
add_filter('cred_notification_recipients', 'b_invoice_mail', 10, 4);

[Nigel]

Hi Tina

Your code is adding a new 'to' recipient rather than overwriting the existing one.

If you dump $recipients to your debug log you'll see that it looks something like this:

Array
(
    [0] => Array
        (
            [to] => to
            [address] => nigel.a@onthegosystems.com
            [name] => 
            [lastname] => 
        )

)

If you want to replace an existing recipient you should replace $recipients[0] rather than adding another array to it.

[tinaH]

Super