Bank deposit payment method

Contrib type: 
Module
Status: 
Complete and working

Downloads

Latest version: 
1.3
Compatibility: 
Ubercart 1.x

Adds a simple bank deposit payment method, so that customers may choose to deposit funds into a bank account you nominate.

The store administrator(s) can define the bank account details. If this payment method is selected in the checkout then the account details will be shown to the user in the checkout and order review pages.

It also creates a token called [bank-deposit-account] that will be replaced with the account details (useful for invoices). I use this code in the customer invoice:

<?php if ($order->payment_method == 'bankdeposit') { ?>
              <tr>
                <td nowrap="nowrap" valign="top">
                  <b><?php echo t('Account details:'); ?></b>
                </td>
                <td width="98%">
                  [bank-deposit-account]
                </td>
              </tr>
<?php } ?>

Update: Somehow the code for the token was missing from version 1.1, it's been reinstated in version 1.2, sorry for any inconvenience caused!

Changelog:
5.x-1.3
- Removes extraneous code and table (Thanks TR)

5.x-1.2
- Reinstated token functionality (Thanks xeeb)

5.x-1.1
- ?

5.x-1.0
- Initial release


PreviewAttachmentSize
uc_bankdeposit-5.x-1.3.tar.gz1.28 KB
xeeb's picture
Offline
Joined: 02/08/2008
Juice: 2
Re: Bank deposit payment method

Does the [bank-deposit-account] token work?

I have been trying to import the account details into invoices using many different methods, but none seem to work.

How do I implement this token?

oliver coleman's picture
Offline
Bug Finder
Joined: 01/09/2008
Juice: 212
Gah

Sorry for the inconvenience xeeb, somehow the code for the token was removed from the version I uploaded (and I couldn't find a copy of it anywhere). I've reimplemented it, tested it, and uploaded it here (now version 1.2).

TR
TR's picture
Offline
Bug FinderFAQ ModeratorGetting busy with the Ubercode.
Joined: 11/05/2007
Juice: 3413
Re: Gah

What's the purpose of the DB table you create and populate? I don't see that it does anything for you.

<tr>.
oliver coleman's picture
Offline
Bug Finder
Joined: 01/09/2008
Juice: 212
How silly

I think this must have been a hold over from when the module attempted to implement some other functionality. Was a while since I touched it (to reinstate token) and didn't pay much attention to anything else except token stuff... Thanks for picking this up.

lemandarin's picture
Offline
Joined: 03/30/2008
Juice: 8
Token inside

Cool module, but I do not manage to add token in my text with my coordinates banking. I will like to be able to indicate this:

"
Name: blabla
Account: 000-00000-00
Communication: IDUSER+DATEORDER
"

So that I can easily see from which customer the payment comes.

Thank,
Bye Tom.

krunar's picture
Offline
Joined: 02/03/2008
Juice: 54
Hi, first of all thanks for

Hi,
first of all thanks for this module, it is exactly what i was looking for.
in order to add full details off my account i have used html code in the bank deposit settings. something like this
Bank NAme:
Bank Account: XXXXXXXXXXXXXXXXXXXX
Name: XXXX YYYYY
What i would like to know, is if it possible to have these details translated.
i tried using php t() in the settings but it is ignored. Any idea how to do this?
thank you

snicers's picture
Offline
Uber DonorInternationalizationizer
Joined: 09/20/2007
Juice: 192
not working with 1.3

with 1.3 I get an error message that Übercart can't complete the order. It returns to the cart and nothing happens.

moby's picture
Offline
Joined: 10/05/2008
Juice: 188
Order ID as payment reference..

Hi,

Is there a way to include the OrderID in the bank account details area.

So that you can instruct the customer to put it as the reference with the payment.

Cheers!

Craig

ragavendra_bn's picture
Offline
Joined: 01/10/2009
Juice: 68
Re: Bank deposit payment method

Is this module for D6.x availaible.........

timoti's picture
Offline
Joined: 01/23/2009
Juice: 6
d6 version

I'm really grateful someone has developed this feature, but i'd also be really keen to know if/when this will fly for drupal 6 - any idea?

one planet, one chance

Docc's picture
Offline
Getting busy with the Ubercode.
Joined: 07/03/2008
Juice: 169
Re: d6 version

A 1 minute conversion to 6.x

AttachmentSize
uc_bankdeposit-6.x.zip 2.27 KB
xibun's picture
Offline
Joined: 05/13/2009
Juice: 105
move project to drupal.org?

Hi,

I was just wondering if I should move the uc_direct_deposit to drupal.org and came across this module. as far as I can tell both try to do the same thing!

how about a merge of the two modules?
if yes: any opinion what name we should go for? - native English speakers? (uc_direct_deposit / uc_bank_deposit / uc_bank_transfer / uc_giro_transfer)

cheers

Docc's picture
Offline
Getting busy with the Ubercode.
Joined: 07/03/2008
Juice: 169
Re: move project to drupal.org?

Yes you should move your module to drupal.org.
You can contact the owner of this module telling him about your module for 6.x so he knows about it and tkae it from there. But this module seems dead to me.

I would go for uc_bank_transfer. But it doesnt really mather as long as it works Smiling

Regards.

oliver coleman's picture
Offline
Bug Finder
Joined: 01/09/2008
Juice: 212
Merge, post on d.o

Hi xibun,

Yeah I noticed your module while looking through the contrib list and wondered why someone had written a module that presumably did the same thing as this one! I was too busy at the time to actually see if it did do something similar, though.

I don't have plans to maintain this module at present, and am completely happy for someone to merge it with another module or do whatever else they like with the code, so go for it.

Cheers,
Oliver

xibun's picture
Offline
Joined: 05/13/2009
Juice: 105
done.

please find the project now maintained on drupal.org: http://drupal.org/project/uc_bank_transfer

eric's picture
Offline
Joined: 11/25/2007
Juice: 56
Drupal 6 version

Can you believe there is another copy of this direct deposit for Drupal 6 over here
http://www.ubercart.org/contrib/6736

I have tried it, exactly the same as this one. I like the fact both are direct upgrade of this Drupal 5 direct deposit, all information entered in the field in the Drupal 5 version are retained.
Except the other one shows the version as 6.x-1.0-dev where this one shows nothing.

I would vote for this to be included in the Ubercart core - Payment Method Pack
Would make it so much more convenient.

Chun Ming (Eric) Wong
Managing Director
Portable Electronics Ltd // Seller of Android devices

xibun's picture
Offline
Joined: 05/13/2009
Juice: 105
read. think. write.

@eric please read the posts first before adding unnecessary information!

-> in comment #12 cross reference to the same module as you are referring to
-> in comment #12 proposal to merge the two modules
-> in comment #15 announcing merger completed and maintenance moved to drupal.org

finally: my personal opinion is to make the core lighter and remove all non generic modules from core and only leave a smart APIs for the individual modules to plug in. please don't abuse of this discussion thread to continue principle discussion. please have it where it belongs - for example in Ubercart 3.x Battle Plans.

eric's picture
Offline
Joined: 11/25/2007
Juice: 56
@xibun Don't be so

@xibun

Don't be so upset....I did read those and in fact, tried all the bank deposit modules, including the one your one at http://drupal.org/project/uc_bank_transfer

If possible, write more information, provide screenshot, etc on http://drupal.org/project/uc_bank_transfer
Some critical information you mentioned previously in the thread is missing from your description!!! since most ppl won't be reading through all the threads to find out...

"based on the original code I've added some features:
- added additional bank detail fields (IBAN, SWIFT, Clearing Number, Bank Branch)
- all fields are now optional and are only displayed when they contain data
- added a token [order-payment-bank-details] to display the information on the invoice"

However, your uc_bank_transfer is NOT the same as this original module, providing a field to put all the details (which is what I need). Until you have actually merged the two modules together and publish it, I don't see any problems. Whatever I have put up can be deleted later anyway.....

Plus, I did not create anything new but re-post what is already done Eye-wink

Btw, I hardly come to this site at all. Maybe once every 3-6 months.

Chun Ming (Eric) Wong
Managing Director
Portable Electronics Ltd // Seller of Android devices

xibun's picture
Offline
Joined: 05/13/2009
Juice: 105
modules are merged

I will add some information on http://drupal.org/project/uc_bank_transfer.

Since the two modules I merged had a different approach the merged module obviously cannot be the same as the bank deposit module! But the modules are merged and your requested functionality is in there. Please install it and read the instructions provided under "Administer/Store administration/Configuration/Payment settings/Payment methods/Bank Transfer settings" to see how to do it.

eric's picture
Offline
Joined: 11/25/2007
Juice: 56
Re: modules are merged

Thanks, now I understand. I didn't realize the "payment instruction" field was meant for this....I thought it was just for additional instruction but not for bank info

the default words
"Please note: bank details will not be displayed after checkout."
kinda mislead me....

Plus, the absence of the Bank payment details on the check-out page makes me think that it is not right and removed it asap, since I was in a hurry to complete my site upgrade and work on other modules............the previous bank deposit module displays the details there and I put alternative payment instructions there......because I only accept bank transfer........(Actually, I think the way the original bank deposit module works is more suitable to my needs atm....)

I have deleted the post on the bank deposit module, maybe you should "post your contribution" here as well. I think most ubercart users would be looking for the module here rather than on Drupal and might have trouble locating your bank transfer module.

Lastly, you said
"Please note: bank details will not be displayed after checkout."
So the customer won't be able to find the payment instruction after the review page?
Seems like this can be a problem if the user does not get the payment instruction by e-mail or can't really go back and check?

Chun Ming (Eric) Wong
Managing Director
Portable Electronics Ltd // Seller of Android devices

eric's picture
Offline
Joined: 11/25/2007
Juice: 56
Re: Re: modules are merged

Sorry, I still think the community deserve to know that there is an alternative bank deposit.

Especially when I think the old way module works better for me because I need to tell my customer that Paypal is available on demand but I can't put up paypal on Ubercart for two reasons. 1) there isn't a 2.x module to add Paypal fees. 2)I have two paypal account for two different country, using two different currency. Ubercart doesn't support this yet, so it is still manual atm.

So I have put it up again Sticking out tongue

No offence for your hard work. I have marked it as depreciated and put links to your new module.

Chun Ming (Eric) Wong
Managing Director
Portable Electronics Ltd // Seller of Android devices

xibun's picture
Offline
Joined: 05/13/2009
Juice: 105
Re: Re: Re: modules are merged

I have now posted the contribution here as you suggested. I thought links from the original modules would be enough, but why not.

If, but only if, the administrator is not using the token in the invoice then yes, the client cannot see order details on the invoice. In the module I already provide a template that shows how to use the token, so really easy. So why did I leave the old message in the "Payment instructions" field? Because I couldn't think of any new/other generic text people might want to put there. And therefor instead discussed the token in the footnote. BUT as the text in the field is written in bigger letters a quick "reader" might miss the details. So I will have a look in changing this and instead strongly suggest to use the token which I agree is much better user experience.

Can you please specify what functionality is missing from uc_bank_transfer? so I can extend the module. Please read the following short guidelines so you can understand how the Drupal community works: http://drupal.org/node/23789.