OrderReceiptMail should honor order BCC even if not passed to send()

Created on 17 September 2021, over 3 years ago
Updated 13 May 2025, 24 days ago

Describe your bug or feature request.

When using Resend Receipt, I noticed that the BCC from the order type is not set.

The button triggers OrderReceiptMail::send($order), which by default sets params $to and $bcc = NULL. Inside the function, the To address is loaded from the order, but BCC is left alone.

I think that the BCC should be loaded from the order type in OrderReceiptMail::send(), so that receipt sends are always consistent.

Patch attached.

Feature request
Status

Active

Version

2.0

Component

Order

Created by

🇺🇸United States rwanth New York, USA

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • 🇧🇷Brazil adinancenci

    Slight changes to the original patch. It applies for 2.36 ~ 2.40

  • 🇮🇱Israel jsacksick

    When the email is resent, it doesn't necessarily make sense to resend to the BCC (which is why it was implemented that way).
    You're free to alter the form itself if you want a different behavior. Also in the patch you're loading the order type even though this happens right after already.

Production build 0.71.5 2024