Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.When trying to use 7.x-1.0-alpha2 version of code, found some settings are incorrect and documentation was incorrect.
Initially got issues with Payment type but no call was ever made to Worldpay. Checked Worldpay settings with Worldpay support and all correct. Issue thus with module code.
A working version exists so this issue will merge patches from that working version to hopefully create a new working version of this module.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | commerce_worldpay-2941100_2.patch | 1.21 KB | technotim2010 |
| #2 | commerce_worldpay-2941100_1.patch | 2.63 KB | sakadava |
Comments
Comment #2
sakadava commentedHi @technotim I've uploaded a patch which is actually to 7.x-1.0-alpha2. I could rework this to 7.x-dev if necessary but I'm just bunging it up here for speed.
Bear in mind I don't know whether this specifically targets your issue which may be due to configuration.
This patch addresses two issues:
- The secret key size is increased to 30 characters (our secret key was too large to fit the original 16 characters)
- Card method selection is presented only if 'Select payment method locally' is checked
These were done ages ago so it's difficult for me to remember the exact issues although clearly these were considered useful at the time. I can separate the two issues if you like.
I'm also happy to continue helping you with this by referencing my working version, in the event these do not resolve the issue.
Comment #4
aburrows commentedPatch applied and tested correctly, we should keep a check on WP API as it has vastly changed.
Comment #5
technotim2010 commentedThere are changes required to change the code and the messages for a successful payment
Specifically changing "WorldPay excepted the user's transaction. " to "Worldpay accepted the user's transaction"
and "The user canceled the payment." to "The user cancelled the payment."
Comment #7
aburrows commentedGreat stuff @TechnoTim2010 and @sakadava - I will deem this issue fixed, but any other issues please open a new issue so this can be investigated.