The attached patch is a first attempt at a version of uc_sagepayserver for Ubercart 3 on Drupal 7. It works for me on simulator and test - although I haven't tried a live payment with it yet. Please try it for yourselves and report on any issues that arise - I'm sure there will be some - consider this to be largely untested and use at your own risk.
Apply the patch against the dev version of uc_sagepayserver, and use -p1 to allow for my directory names.
In other words, with the patch in the uc_sagepayserver directory, from this directory issue this command
patch -p1 < sagepayserver_uc3.patch

Colin

CommentFileSizeAuthor
#3 sagepayserver_u3_v2.patch20.37 KBaireworth
sagepayserver_uc3.patch18.68 KBaireworth

Comments

johnmcc’s picture

johnmcc commented

Nice work, thanks for this. I'll check it out and report back.

aireworth’s picture

aireworth commented

I just noticed it currently fails for non shipping products, because, if its not set on the checkout form, the delivery country defaults to US, and then Sagepay expects a delivery state. I don't know if this is a Ubercart (possibly) or or uc_sagepayserver problem, but currently you need to set a delivery address (to get a delievery country)

Colin

aireworth’s picture

aireworth commented
StatusFileSize
new sagepayserver_u3_v2.patch20.37 KB

Here's a modified version of the patch that works with non shippable products as well.

nyleve101’s picture

nyleve101 commented

Hi,

will a stable release of the D7 version be realeased?

Thanks for the work done so far!

Hope all is well,

Eve

zincdesign’s picture

zincdesign commented

I have been using the patch in #3 for about 6 months now and it's been fine apart from it doesn't seem to be passing on the successURL and failureURL to sagepay. Can anyone give me a pointer where I can change this. I'm not much of a coder.

aireworth’s picture

aireworth commented

Does the U2 D6 version pass these urls? I only made changes in the code to make it work in D7 - I didn't change much (or anything, as I recall) that passed between the module and Sagepay. I not a confident coder in these areas myself, but I'll have a look over the weekend.

leo pitt’s picture

leo pitt commented
Status: Active » Needs review

Thanks for the work on the patch aireworth. I have applied it to 6.x-1.5 and created a 7.x-1.x dev release.

aireworth’s picture

aireworth commented

Excellent - HTH.