When an invoice is mailed through the admin forms (/admin/store/orders//invoice/mail), the From line contains the default site e-mail address rather than the store's address.

Patch: drupal_mail() in uc_order.admin.inc:uc_order_mail_invoice_form_submit() needs uc_store_email_from() as its 6th argument.

CommentFileSizeAuthor
#3 uc_order.admin_.inc_.540682.patch664 byteswimh
uc_order.admin_.inc_.patch538 byteswimh

Comments

rszrama’s picture

rszrama commented
Status: Active » Fixed

Put it in. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

wimh’s picture

wimh commented
Version: 6.x-2.0-rc3 » 6.x-2.0-rc5
Status: Closed (fixed) » Active
StatusFileSize
new uc_order.admin_.inc_.540682.patch664 bytes

In RC5 this patch didn't appear, but something else in the same line was changed. Maybe there was a merge conflict and this patch got thrown out. I've attached a new patch that applies to RC5.

Island Usurper’s picture

Island Usurper commented
Status: Active » Fixed

I think I must have applied a patch by hand right over that fix and didn't notice the difference. It's certainly my name on the commit log. :P

Thanks for the patch (again). Committed, hopefully for good.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.