Post by eddy » Thu Aug 04, 2011 4:02 pm

Just as the title says: BankTransfer details are not sent to customer via email.
Doesn't work neither on 1.5.0 neither on 1.5.1.1

Newbie

Posts

Joined
Fri Jul 15, 2011 1:41 am

Post by uksitebuilder » Thu Aug 04, 2011 4:09 pm

Does a comment get added to the order in your admin containing the bank details ?

User avatar
Guru Member

Posts

Joined
Thu Jun 09, 2011 11:37 pm
Location - United Kindgom

Post by fido-x » Thu Aug 04, 2011 4:19 pm

Let me get this straight.

You want to send a customer's bank account details in a "plain text" email that can easily be read by any packet sniffer so the WHOLE WORLD can see it? Might as well put their credit card information in there as well.

Image
Modules for OpenCart 2.3.0.2
Homepage Module [Free - since OpenCart 0.7.7]
Multistore Extensions
Store Manager Multi-Vendor/Multi-Store management tool

If you're not living on the edge ... you're taking up too much space!


User avatar
Expert Member

Posts

Joined
Sat Jun 28, 2008 1:09 am
Location - Tasmania, Australia

Post by uksitebuilder » Thu Aug 04, 2011 4:26 pm

It is standard practice to send bank details via email to your customers.

There's less information than there is on a cheque.

All they can do with the info is pay into the account.

I don't see the problem.

I believe the same thing is also happening with cheque payments, no info on who to make the cheque payable to etc is sent in the order confirmations.

User avatar
Guru Member

Posts

Joined
Thu Jun 09, 2011 11:37 pm
Location - United Kindgom

Post by fido-x » Thu Aug 04, 2011 4:32 pm

The email already contains the customer's name and address, adding the bank account details (account number, BSB, etc) provides a scammer or hacker with enough information to wipe out that customer's bank account before they've even received the confirmation email.

Image
Modules for OpenCart 2.3.0.2
Homepage Module [Free - since OpenCart 0.7.7]
Multistore Extensions
Store Manager Multi-Vendor/Multi-Store management tool

If you're not living on the edge ... you're taking up too much space!


User avatar
Expert Member

Posts

Joined
Sat Jun 28, 2008 1:09 am
Location - Tasmania, Australia

Post by uksitebuilder » Thu Aug 04, 2011 4:35 pm

It's not the customer's address we are referring to, it is the store holder's

User avatar
Guru Member

Posts

Joined
Thu Jun 09, 2011 11:37 pm
Location - United Kindgom

Post by eddy » Thu Aug 04, 2011 4:51 pm

I would like it to be so that MY bank account number (number to which customer will transfer money) would be emailed to the customer.
uksitebuilder wrote:Does a comment get added to the order in your admin containing the bank details ?
Yes
Last edited by eddy on Thu Aug 04, 2011 4:52 pm, edited 1 time in total.

Newbie

Posts

Joined
Fri Jul 15, 2011 1:41 am

Post by fido-x » Thu Aug 04, 2011 4:52 pm

uksitebuilder wrote:It's not the customer's address we are referring to, it is the store holder's
You mean bank details.

I must admit that I haven't looked at a lot of the payment modules (only those that I use myself, ie. free checkout and paypal), so I guess I've shown my ignorance on this one. :-[

Yes, uksitebuilder is right. There should be a comment in the email showing the "Bank Transfer Instructions" you entered in the admin. This should contain instructions for the customer on how to make the transfer, including your bank details (account number, BSB, etc).

Image
Modules for OpenCart 2.3.0.2
Homepage Module [Free - since OpenCart 0.7.7]
Multistore Extensions
Store Manager Multi-Vendor/Multi-Store management tool

If you're not living on the edge ... you're taking up too much space!


User avatar
Expert Member

Posts

Joined
Sat Jun 28, 2008 1:09 am
Location - Tasmania, Australia

Post by Daniel » Thu Aug 04, 2011 5:43 pm

add it to the bug tracker please.

OpenCart®
Project Owner & Developer.


User avatar
Administrator

Posts

Joined
Fri Nov 03, 2006 6:57 pm

Post by uksitebuilder » Thu Aug 04, 2011 6:10 pm

Will add it to the tracker.

Just to note, the details are sent in the plain text email the admin receives.

I think I have narrowed it down to catalog/view/theme/default/template/mail/order.php not referencing the available comment data

User avatar
Guru Member

Posts

Joined
Thu Jun 09, 2011 11:37 pm
Location - United Kindgom

Post by fido-x » Thu Aug 04, 2011 7:34 pm

Try this. In "catalog/model/checkout/order.php" (at around line 253), insert:

Code: Select all

$template->data['text_comment'] = $language->get('text_new_comment');
if ($order_info['comment']) {
    $template->data['comment'] = $order_info['comment'];
} 
Then, in "catalog/view/theme/default/template/mail/order.tpl", insert:

Code: Select all

<?php if ($comment) { ?>
<p><?php echo $text_comment; ?></p>
<p><?php echo $comment; ?></p>
<?php } ?>
before:

Code: Select all

<p><?php echo $text_footer; ?></p>
Haven't tested it, but it looks like it should work.

Image
Modules for OpenCart 2.3.0.2
Homepage Module [Free - since OpenCart 0.7.7]
Multistore Extensions
Store Manager Multi-Vendor/Multi-Store management tool

If you're not living on the edge ... you're taking up too much space!


User avatar
Expert Member

Posts

Joined
Sat Jun 28, 2008 1:09 am
Location - Tasmania, Australia

Post by eddy » Fri Aug 05, 2011 2:33 am

fido-x wrote:Try this. In "catalog/model/checkout/order.php" (at around line 253), insert:

Code: Select all

$template->data['text_comment'] = $language->get('text_new_comment');
if ($order_info['comment']) {
    $template->data['comment'] = $order_info['comment'];
} 
Then, in "catalog/view/theme/default/template/mail/order.tpl", insert:

Code: Select all

<?php if ($comment) { ?>
<p><?php echo $text_comment; ?></p>
<p><?php echo $comment; ?></p>
<?php } ?>
before:

Code: Select all

<p><?php echo $text_footer; ?></p>
Haven't tested it, but it looks like it should work.
Thanks for fast reply, but i am now getting an error:

Notice: Undefined variable: comment in /home/dragonltu/domains/galidejas.lt/public_html/shop/catalog/view/theme/galidejas/template/mail/order.tpl on line 162

Newbie

Posts

Joined
Fri Jul 15, 2011 1:41 am

Post by fido-x » Fri Aug 05, 2011 2:39 am

OK, change this bit that you added to "catalog/model/checkout/order.php" from:

Code: Select all

if ($order_info['comment']) {
    $template->data['comment'] = $order_info['comment'];
} 
to:

Code: Select all

if ($order_info['comment']) {
    $template->data['comment'] = $order_info['comment'];
} else {
    $template->data['comment'] = '';
} 

Image
Modules for OpenCart 2.3.0.2
Homepage Module [Free - since OpenCart 0.7.7]
Multistore Extensions
Store Manager Multi-Vendor/Multi-Store management tool

If you're not living on the edge ... you're taking up too much space!


User avatar
Expert Member

Posts

Joined
Sat Jun 28, 2008 1:09 am
Location - Tasmania, Australia

Post by eddy » Fri Aug 05, 2011 3:23 am

Error disappeared, but the bank details are still missing in the email.

Now, I might be wrong, but maybe your last peace of code is missing language get?

Newbie

Posts

Joined
Fri Jul 15, 2011 1:41 am

Post by fido-x » Fri Aug 05, 2011 3:37 am

Just had another look at the code in the checkout/order model and in the bank transfer module. Scrap what I told you before -- I got it wrong! I was looking at the wrong comment.

OK. In "catalog/model/checkout/order.php" replace the block of code I told you to insert at line 253, with:

Code: Select all

$template->data['comment'] = $comment; 
And, in "catalog/view/theme/default/template/mail/order.tpl", replace the code I gave you with:

Code: Select all

<?php if ($comment) { ?>
<p><?php echo $comment; ?></p>
<?php } ?>

Image
Modules for OpenCart 2.3.0.2
Homepage Module [Free - since OpenCart 0.7.7]
Multistore Extensions
Store Manager Multi-Vendor/Multi-Store management tool

If you're not living on the edge ... you're taking up too much space!


User avatar
Expert Member

Posts

Joined
Sat Jun 28, 2008 1:09 am
Location - Tasmania, Australia

Post by eddy » Fri Aug 05, 2011 3:50 am

fido-x wrote:Just had another look at the code in the checkout/order model and in the bank transfer module. Scrap what I told you before -- I got it wrong! I was looking at the wrong comment.

OK. In "catalog/model/checkout/order.php" replace the block of code I told you to insert at line 253, with:

Code: Select all

$template->data['comment'] = $comment; 
And, in "catalog/view/theme/default/template/mail/order.tpl", replace the code I gave you with:

Code: Select all

<?php if ($comment) { ?>
<p><?php echo $comment; ?></p>
<?php } ?>
Yes, this works perfectly! Thank you!

Newbie

Posts

Joined
Fri Jul 15, 2011 1:41 am

Post by fsit » Tue Aug 09, 2011 4:53 am

Thanks for the information, it solved this almost for me but maybe I lost something in the corrections or is my shop too customized but when I applied this fix, it overwrites the customer remarks in the e-mail.

So I added in catalog/model/checkout/order.php below:

Code: Select all

$template->data['comment'] = $order_info['comment'];
this new line:

Code: Select all

$template->data['comment1'] = $comment;  //extra
And in catalog/view/theme/default/template/mail/order.tpl above the $text_footer:

Code: Select all

 <?php if ($comment1) { ?>
<p><?php echo $comment1; ?></p>
<?php } ?>
Which solved my override in 1.5.0.5 with this extension: http://www.opencart.com/index.php?route ... order=DESC

New member

Posts

Joined
Mon Jul 25, 2011 12:44 am

Post by Hanis » Tue Oct 25, 2011 4:41 pm

Hi

I did this solution but when i do testing email. The comment become like this

Bank Transfer Instructions Thank you for buying with us. Please transfer the total amount to the following bank account. Maybank Acc# :xxx CIMB Acc# :xxx Kindly inform us reference number after payment. Email to xxxx or SMS to xxxxxx Your order will not ship until we receive payment. Your order will not ship until we receive payment.

The comment do not have any line.

While it should be like this

Please transfer the total amount to the following bank account.

Maybank Acc# : xxxx
CIMB Acc# : xxxxx

Kindly inform us reference number after payment. Email to xxx or SMS to xxxx

Your order will not ship until we receive payment.

Please help

Newbie

Posts

Joined
Fri Jul 15, 2011 12:01 pm

Post by Angeloop » Tue Nov 01, 2011 9:09 am

Hanis wrote:Hi

I did this solution but when i do testing email. The comment become like this

Bank Transfer Instructions Thank you for buying with us. Please transfer the total amount to the following bank account. Maybank Acc# :xxx CIMB Acc# :xxx Kindly inform us reference number after payment. Email to xxxx or SMS to xxxxxx Your order will not ship until we receive payment. Your order will not ship until we receive payment.

The comment do not have any line.

While it should be like this

Please transfer the total amount to the following bank account.

Maybank Acc# : xxxx
CIMB Acc# : xxxxx

Kindly inform us reference number after payment. Email to xxx or SMS to xxxx

Your order will not ship until we receive payment.

Please help
Hi Hanis!

Were you able to solve this? I'm having the same problem thank you.

Regards.

Newbie

Posts

Joined
Wed Mar 09, 2011 8:43 am

Post by Ed73 » Tue Nov 29, 2011 6:03 pm

@Hanis @Angeloop

Code: Select all

$template->data['comment'] = (nl2br($comment));

Newbie

Posts

Joined
Fri Sep 23, 2011 8:37 pm
Who is online

Users browsing this forum: No registered users and 14 guests