Bank deposit payment method

Contrib type: 
Module
Status: 
Complete and working
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
Joined: 02/08/2008
Juice: 2

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?

Joined: 01/09/2008
Juice: 152

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
Bug FinderFAQ ModeratorGetting busy with the Ubercode.
Joined: 11/05/2007
Juice: 3225

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

<tr>.
Joined: 01/09/2008
Juice: 152

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.

Joined: 03/30/2008
Juice: 8

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.

Joined: 02/03/2008
Juice: 41

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

Joined: 09/20/2007
Juice: 156

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

Joined: 10/05/2008
Juice: 69

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

Joined: 01/10/2009
Juice: 61

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

Joined: 01/23/2009
Juice: 6

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

Joined: 07/03/2008
Juice: 124

A 1 minute conversion to 6.x

AttachmentSize
uc_bankdeposit-6.x.zip 2.27 KB
Joined: 05/13/2009
Juice: 71

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

Joined: 07/03/2008
Juice: 124

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.

Joined: 01/09/2008
Juice: 152

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

Joined: 05/13/2009
Juice: 71

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

Joined: 11/25/2007
Juice: 35

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.

Joined: 05/13/2009
Juice: 71

@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.

Joined: 11/25/2007
Juice: 35

@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.

Joined: 05/13/2009
Juice: 71

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.

Joined: 11/25/2007
Juice: 35

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?

Joined: 11/25/2007
Juice: 35

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.

Joined: 05/13/2009
Juice: 71

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.