Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.I am working on this now. A preliminary version should be committed within a few days.
| Comment | File | Size | Author |
|---|---|---|---|
| #11 | 2643682-2checkout.patch | 5.32 KB | longwave |
| |||
Comments
Comment #3
TR CreditAttribution: TR commentedHere's the initial port. I'd REALLY like someone who uses 2Checkout to help me test this. I'll probably commit this as-is then cleanup problems in a separate commit - there's already too much here review.
Comment #5
TR CreditAttribution: TR commentedStill looking for people with 2Checkout experience to help test this. The above patch and others have been committed, so all you have to do is install the latest 8.x-4.x of Ubercart, configure 2Checkout with your account settings, and make some test transactions.
Comment #6
longwaveMade some further fixes. I have tested this as far as possible on the sandbox, but as the documentation mentions, "the 'x_receipt_link_url’ parameter will only function on Live accounts. This option will not work on the Sandbox environment." - which makes this somewhat difficult to test fully.
Comment #7
TR CreditAttribution: TR commentedLooks good. Is there a reason to default to USA instead of the store country?
Comment #9
longwaveThe notification controller definitely needs some more work, but I committed #6 with a change from 'USA' to '' for the default country, as 2Checkout will ask for the country if you leave it blank (and this case only happens if the user has not installed any countries in Ubercart, which is unlikely).
Comment #10
TR CreditAttribution: TR commentedI don't have a live account with 2checkout, but this works as far as I can test it. Perhaps we just mark this as fixed and deal with any issues as they are reported?
Comment #11
longwaveNo idea if this works properly, but here are some more fixes for the notification controller.
Comment #13
longwaveCommitted, this will have to do for now.