Jump to content
JcMagpie

Support thread for JcM Bank Transfer Payments V1.0 Phoenix

Recommended Posts

This is a simple add-on that allows you to add Bank Transfer as a Payments option.

It’s an update to an existing add-on so all credit to original author.

It requires NO core changes so just copy over the files provided.

Once it’s installed simply go to admin and install and turn on.

Bank details are set in admin

image.thumb.png.4f84d6a23e39c7e53a051a394e0c8d99.png

Email sent to customer will show bank details to make payment

image.png.ff015a60f56d9726123831ba268dd3e7.png

If you wish the invoice to show your bank details then you need to edit your invoice.php or use the one provided (this is optional).

That’s it enjoy.

It has only been tested on Phoenix 1.0.4.0 only.

Can be seen working on this custom test site

image.thumb.png.1c224fb833de35d53fa0a47b68ece4fc.png

image.thumb.png.589ea1c86db22297c2ad9781648eb39f.png


 

Share this post


Link to post
Share on other sites

A small over site by me, the invoice.php provided shows the bank details even when another payment methd is selected. To fix this use the attached file if you used the one provided.

or if you are editing your own invoice.php then you need to add this wherver you decide to add the bank details. This wll then only show on invoice if bank transfer is selected.

<?php if ($order->info['payment_method'] === "Bank Transfer Payment") {?>
                    <div class="notice"> <b>Our Bank Details:</b><br>
                    Please use the following details to transfer your total order value:<br>
                                            osc-phoenix Ltd<br>
                                            Bank = MyBank<br>
                                            Sort Code = 00-00-00<br>
                                            Account Number = 12345678<br>
                                            For European (EU) Customers<br>
                                            SWIFTBIC = **************<br>
                                            IBAN = **********************<br>
                                            Once the <u>full amount</u> has been received and confirmed Your order will be processed.<br><br>
                                            </div>
                                            <?php }?>

 

invoice.php


 

Share this post


Link to post
Share on other sites

A little suggestion:
There is a lot of hardcoded English language in the invoice.php file
It would be nice if it was multi-language, right?


Live   : OsC 2.2, php 5.6 & UTF-8  |  Local : phoenix and oscommerce v4 test when v4 will be available

Share this post


Link to post
Share on other sites
20 minutes ago, bonbec said:

It would be nice if it was multi-language, right?

Yes would be better as mulit-language, will need to find time to update it. I just used one I had available.

Edited by JcMagpie

 

Share this post


Link to post
Share on other sites

Bravo for the reactivity, I would test this evening. Thank you.


Live   : OsC 2.2, php 5.6 & UTF-8  |  Local : phoenix and oscommerce v4 test when v4 will be available

Share this post


Link to post
Share on other sites

Only a few nights ago I created a list of current modules used on my site that would need updating and this was onre of the ones listed. Many thanks @JcMagpie Zahid. Saved me the bother of even looking at it.


REMEMBER BACKUP, BACKUP AND BACKUP

Get the latest Phoenix version here

It's easy to say to yes every question.

Share this post


Link to post
Share on other sites

found a "bug"

Please update the package with this fix:

Rename all the "payments" folders to "payment". You have an extra "s"

Edited by Owl Sauron
theres other folders besides includes

Share this post


Link to post
Share on other sites

Hi there!
First of all, I'd like to congratulate you for a ton of great contributions that you are making to this project! Excellent job! ❤️

Now the issue with this addon. On Phoenix v1.0.57 it's throwing the following error on the payment modules installation page:

Quote

Deprecated: Methods with the same name as their class will not be constructors in a future version of PHP; banktransfer has a deprecated constructor

Would you be kind enough to update this addon so it will be compatible with the newer versions? Unfortunately I'm not a coder so this is a little beyond me to fix. 😕

TIA for your help! And keep up the good work!


Patty

Share this post


Link to post
Share on other sites
On 4/14/2020 at 8:08 AM, Patty said:

Hi there!
First of all, I'd like to congratulate you for a ton of great contributions that you are making to this project! Excellent job! ❤️

Now the issue with this addon. On Phoenix v1.0.57 it's throwing the following error on the payment modules installation page:

Would you be kind enough to update this addon so it will be compatible with the newer versions? Unfortunately I'm not a coder so this is a little beyond me to fix. 😕

TIA for your help! And keep up the good work!

take a look at the few posts around this on page 14  it might work.   at least to a point. 

 

Edited by hungryfrank

you can cut up to 4 pages of your checkout by using my three add_ons

login create account in one page

Express checkout  

login pop up modal

Share this post


Link to post
Share on other sites

Thank you, but I don't see how it's relevant to this issue. Those posts are from 2005 for a completely different addon, while this is a new issue related to new code. I'm sorry, but I'm not a coder and I can't find a solution there.

But thanks for trying to help anyway. 😉


Patty

Share this post


Link to post
Share on other sites

one page 14   of that tread there is the solution that might work 

you have to change the name of the function to 

this

    function banktransfer() {

function __construct() {

to

in the module/paymennt/banktransfer.php

it is worth the try

 

Edited by hungryfrank

you can cut up to 4 pages of your checkout by using my three add_ons

login create account in one page

Express checkout  

login pop up modal

Share this post


Link to post
Share on other sites
49 minutes ago, Patty said:

Thank you, but I don't see how it's relevant to this issue. Those posts are from 2005 for a completely different addon, while this is a new issue related to new code. I'm sorry, but I'm not a coder and I can't find a solution there.

But thanks for trying to help anyway. 😉

 


you can cut up to 4 pages of your checkout by using my three add_ons

login create account in one page

Express checkout  

login pop up modal

Share this post


Link to post
Share on other sites

Oh, ok! Thank you!

But I think it's a lit bit more complicated than that. I was comparing this file with a v1.0.5.7 moneyorder file and there's more code that needs to be updated. I'm not very familiar with Phoenix yet to adventure in doing it myself. 


Patty

Share this post


Link to post
Share on other sites

Has anyone updated this addon to work with the latest Phoenix 1.7.0.6. I have had a try but still get lots of undefined errors. May have to try again.


REMEMBER BACKUP, BACKUP AND BACKUP

Get the latest Phoenix version here

It's easy to say to yes every question.

Share this post


Link to post
Share on other sites

Hello JcM,

i need help with the script i would like to use.

i change in banktransfer.php

function banktransfer() {

 to

function __construct() {

it works,

but i have many warnings when I want to install the app. a lot of use of undefined constant.

after the install  can i edit the bank account and save.

from the user side i have 2 more notice. (Constant MODULE_PAYMENT_BANKTRANSFER_SORT_ORDER already defined AND Constant MODULE_PAYMENT_BANKTRANSFER_ORDER_STATUS_ID already defined)

can you help me and fix the problems?

Best Regards Frank

 

 

 

 

Share this post


Link to post
Share on other sites

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 account

Sign in

Already have an account? Sign in here.

Sign In Now

×