Order conf email is sent twice

[Mario Ropič]

Hello!

I have a problem, when I order, I get two order conf emails. One is filled out with real data and the other is filled with {variable} data..

The email is sent twice only when I select cash on delivery.

Can someone help me?

Sorry for my bad English, I come from Slovenia.


Regards, Mario

[Clay]

same case here.. I´d like to disable the e-mail which is sent with empty variables (I think it´s the one that´s set up in Orders > Order statuses) and only keep the other one. However, if I disable sending the email for Order confirmation, then nothing is sent at all, none of the two. I´m using OnePageCheckout module.

Any idea where I can edit the emails sent for each order status? Is there a file that can simply be modified (eg. add comment tags to the line which would send the one mail)?

Thanks!

[mento]

yeah me too i got the same problem

[tuk66]

Your Cash on delivery module is built-in one? There are many patched COD module around.

[mento]

Your Cash on delivery module is built-in one? There are many patched COD module around.

i don't use COD module. i'm using Bankwire modified module

[tuk66]

I see. Contact the developer because it looks like a bug is in the code.

[mento]

I see. Contact the developer because it looks like a bug is in the code.

but the modified module only change the logo and language only

 

im using bankwire from PS also send me twice 

Edited October 2, 2013 by mento (see edit history)

[Pez]

Sorry to revive an old thread but has anyone found a solution to this? I have the same problem, and I'm using PS 1.6.0.9, one page checkout and the official Paypal module.

[pattila01]

PS 1.3.7:

1. deactivate email sending in the valid status (order_conf),
2. /classes/PaymentModule.php => ~ 400:

if ($orderStatus->send_email AND Validate::isEmail($customer->email))
    Mail::Send(intval($order->id_lang), 'order_conf', 'Order conf', $data, $customer->email, $customer->firstname.' '.$customer->lastname, NULL, NULL, $fileAttachment);
    $this->currentOrder = intval($order->id);
return true;

delete the "if" and "return" parts:

Mail::Send(intval($order->id_lang), 'order_conf', 'Order conf', $data, $customer->email, $customer->firstname.' '.$customer->lastname, NULL, NULL, $fileAttachment);
$this->currentOrder = intval($order->id);

this solution is works in 1.3.7.

Edited February 18, 2016 by pattila01 (see edit history)