Needs work
Project:
Commerce Bank Transfer
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
17 Aug 2011 at 21:38 UTC
Updated:
18 May 2013 at 21:13 UTC
Jump to comment: Most recent file
Comments
Comment #1
bladeduComment #2
derjochenmeyer commentedGreat patch eddib. I just added a seperator.
Comment #3
xibun commentedpatch rerolled to match latest dev version - including:
- clean-up of empty lines formatting
- removed bugfix #1304086: payment method undefined which was already committed to dev version separately
- updating new data to also uses !empty() check instead of <> ''
what is in this patch? (needs mention since it is not following common drupal practice 1 fix per issue)
- configurable separator
- additional data field "Account address"
- additional data field "Message for bank transfer form"
- structured message field that can contain tokens (order, user, date)
- allow tokens in bank transfer message
- theme output with div and span
to do:
- IMO "Account address" is not necessary as international banking standard is to ignore this data anyway
- please explain why have "Message for bank transfer form" - how is this different from payment instructions?
- explain logic/code line 128 in patch (is this a bug?)
- untangle patch into separated issues so we can see interest and discussion for each one of them
Comment #4
IckZ commentedHey xibun!
Nice patch (althoug i had to patch it manually.. patch -i had problems with it)
Do you also have this notice at the checkout process after submitting?
cheers?
edit:
I could "solve" this Problem by adding:
to the End of this function function commerce_bank_transfer_submit_form() over the return.. patch found here: http://drupal.org/node/1230666
Comment #5
hadsie commentedAs per the comment #3 I've started to break out some individual patches / issues based on this patch:
* Support for tokens - #1676822: Token support for payment instructions field
* Improved theme support (not based on the patch here, but instead allows the output to be more easily overridden) - #1998080: Use hook_theme to theme bank details