It would be good for site maintainers to have an indication that Mollie Payment is running in test mode so that they don't put the site online open for visitors to order stuff for free. :-)

CommentFileSizeAuthor
#4 interdiff-2565489-2-4.txt2 KBricovandevin
#4 mollie_payment-test-key-indication-2565489-4.patch1.78 KBricovandevin
#2 mollie_payment-test-key-indication-2565489-2.patch920 bytesricovandevin

Comments

ricovandevin created an issue. See original summary.

ricovandevin’s picture

ricovandevin commented
Status: Active » Needs review
StatusFileSize
new mollie_payment-test-key-indication-2565489-2.patch920 bytes

Added message by implementing hook_init().

ricovandevin’s picture

ricovandevin commented
Status: Needs review » Needs work

Maybe we should make this configurable...

ricovandevin’s picture

ricovandevin commented
Status: Needs work » Needs review
StatusFileSize
new mollie_payment-test-key-indication-2565489-4.patch1.78 KB
new interdiff-2565489-2-4.txt2 KB

Admins can now set whether or not they want to see the message about using a test API key.

ricovandevin’s picture

ricovandevin commented
Status: Needs review » Fixed

Issue fixed.

  • ricovandevin committed cc6fa11 on 7.x-1.x 
    Issue #2565489 by ricovandevin: Indicate that Mollie Payment is in test...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

BlinX’s picture

BlinX commented

Rico,
Thanks for adding this!
Please explain: what is the difference between a patch and interdiff.txt??
Note, I did not perform the patch yet, I had also a second payment method with a test Mollie API key... that resulted in the message "This site is using a test API key for Mollie." even when this second payment method was switched off!

ricovandevin’s picture

ricovandevin commented

In an interdiff you'll find the difference between the submitted patch and the previous patch. It is meant to make life easier for patch reviewers.

Please open a new issue (bug report) for the problem with the switched off Mollie test account. Thanks!