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.

CommentFileSizeAuthor
#2 3236416-2.patch840 bytesadinancenci
commerce_order-get_bcc_if_null-1.patch868 bytesrwanth

Comments

rwanth created an issue. See original summary.

adinancenci’s picture

adinancenci commented
StatusFileSize
new 3236416-2.patch840 bytes

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

jsacksick’s picture

jsacksick commented
Status: Active » Closed (won't fix)

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.