Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

'testmode' parameter causes recurring transactions to fail checksum #234

Closed
brentmc79 opened this issue Dec 6, 2011 · 0 comments · Fixed by #235
Closed

'testmode' parameter causes recurring transactions to fail checksum #234

brentmc79 opened this issue Dec 6, 2011 · 0 comments · Fixed by #235

Comments

@brentmc79
Copy link
Contributor

I ran into a problem with my test suite where a small subset of my specs were failing. After a bit of debugging, I contacted the Quickpay support, as it seemed like I was following their docs correctly.

We determined that the testmode parameter (testmode=1) was causing the problem. Apparently, when a referencing transaction id is included in the post, then the testmode is determined from that transaction. The unnecessary parameter was causing a checksum failure, or at least that's what the response code was indicating.

Removing the testmode parameter only for posts that include a transaction parameter allowed my test to pass.
brentmc79 added a commit to brentmc79/active_merchant that referenced this issue Dec 6, 2011
@brentmc79
Quickpay Testmode Fix
3a9f863 
 -Including testmode=1 for recurring message types causes a
  checksum failure
 -Recurring message type include a transaction id
 -The testmode is determined from that reference transaction
 -Omit the testmode parameter for recurring message types

  refs activemerchant#234
@brentmc79 brentmc79 mentioned this issue Dec 6, 2011
Merged
brentmc79 added a commit to brentmc79/active_merchant that referenced this issue Dec 7, 2011
@brentmc79
Add test to verify new Quickpay gateway behavior
54ce542 
 -add_testmode method should NOT add a testmode parameter to the
  post hash if a transaction id is present
 -add_testmode method should add a testmode parameter equal to the
  test? method if there is no transaction id

 refs activemerchant#234
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Resolves issue #234 with the Quickpay gateway brentmc79/active_merchant
2 participants
@ntalbott @brentmc79