30knees Posted October 26 Share Posted October 26 Is it correct that I would need to update classes/module/PaymentModule.php to enable {invoice_number} to be used in emails, eg bankwire module? https://github.com/thirtybees/thirtybees/blob/main/classes/module/PaymentModule.php And then adding here: $params = [ '{voucher_amount}' => Tools::displayPrice($voucher->reduction_amount, $this->context->currency, false), '{voucher_num}' => $voucher->code, '{firstname}' => $this->context->customer->firstname, '{lastname}' => $this->context->customer->lastname, '{id_order}' => $order->reference, '{order_name}' => $order->getUniqReference(), ]; '{invoice_number}' => $order->invoice_number, Link to comment Share on other sites More sharing options...
0 datakick Posted October 26 Share Posted October 26 This class handles sending of order_conf and voucher emails. So if you want to extend those email templates, then it's the right place to modify. It has no impact on order status emails, like bankwire, though. Link to comment Share on other sites More sharing options...
0 30knees Posted October 26 Author Share Posted October 26 Thank you, I was looking for the bankwire status email. Where do I look for that? Is this something that you would be fine with having in the core? Then I could do a PR. Link to comment Share on other sites More sharing options...
0 datakick Posted October 26 Share Posted October 26 OrderHistory https://github.com/thirtybees/thirtybees/blob/0913a2fa58cfd7db1e1f9ac33b99389777edba78/classes/order/OrderHistory.php#L480 Sure, we would gladly accept PR. Extra parameters passed to email templates are fine, as long as it doesn't cause errors or performance issues. 1 Link to comment Share on other sites More sharing options...
Question
30knees
Is it correct that I would need to update classes/module/PaymentModule.php to enable {invoice_number} to be used in emails, eg bankwire module?
https://github.com/thirtybees/thirtybees/blob/main/classes/module/PaymentModule.php
And then adding here:
'{invoice_number}' => $order->invoice_number,
Link to comment
Share on other sites
3 answers to this question
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now