Closed (outdated)
Project:
Commerce PayPal
Version:
7.x-2.x-dev
Component:
PayPal WPS
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
29 Apr 2013 at 09:09 UTC
Updated:
18 Aug 2025 at 21:11 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
olteanu5000 commentedAttached patch with desired functionality. Inspired by paypal wpp module and commerce_be2bill module.
Comment #2
olteanu5000 commentedComment #3
vasikeLet's move this one to the current (2.x) version of the module.
patch re-rolled.
Also re-titled, the patch includes also the Capture option for Authorized payments.
Comment #4
vasikeit seems it needs more work.
Comment #5
vasikeThere is a new patch for this - it is a re-work of the previous one.
I added also the Void option to the Pending payments.
Comment #6
vasikeit seems there some more work needed for Refund, some changes to WPS IPN process for refunds.
there is an updated patch for this.
Comment #7
vasikethere is a new patch that completes the solution for this with Rules integration:
Rules Actions available for Capture/Refund/Void.
This means it could also used with VBOs (Rules components).
Comment #8
YurkinPark commentedThanks for the patch. As i discovered, this patch can be implemented for 7.x-2.3 as well.
I found one issue after implementing this patch. Sometimes, PayPal doesn't authorize payment, and returns
instead of
This means you have not provide service/ship merchandise. Check in official documentation https://developer.paypal.com/docs/classic/express-checkout/integration-g... . My patch implements previous (7th) patch and provides tools which you can use to cover "payment under review" case.
Comment #9
YurkinPark commentedIn case we have many failed transaction before, sometimes i'm obtaining non last transaction (which is paypal transaction). Added condition to match payment method commerce_paypal_wps_check_ipn_pending_reason rule condition callback.
Comment #10
rszrama commentedNew functionality targets the -dev version, even if the patch applies cleanly on prior point releases. However, it isn't clear to me why you disabled a FALSE return value in the IPN validation function. That seems ... quite drastic to have no explanation. : P
Comment #11
ruvenshilpakar commentedHi,
I tried this patch but when i try to complete the checkout and get redirected to the paypal account page, i get an error and nothing happens. What could i be doing wrong? I used the#9 patch
Comment #12
roam2345 commentedReroll patch against 7.x-2.x and - added a comment as to why 'disabled a FALSE return value in the IPN validation function'.
Comment #13
roam2345 commentedfix incorrect path in last patch
Comment #14
ronino commentedI could only test refunds via the sandbox and it mostly works, great! When I do a partial refund first though and then want to refund the remainder, I get these errors:
I enhanced #13 to fix this.
Comment #15
fpalewacki commentedHey, I've fixed a couple of issues and typos and improve multi currency support to get the proper errors returned from PayPal if we created a transaction in a different currency.
Refund request was not sending the currency parameter so if you used different currency in commerce than in PayPal then the transaction would have different currency than the request sent to PayPal resulting in e.g. issuing the "3 EUR" refund but in USD
This ticket is kinda old, I'm on "7.x-2.7" version
Comment #16
jsuchogorskafp commentedI've fixed issue about order balance, when refund was did from paypal.
Refund from paypal got a confirmation with the e.g "0 EUR", so the order balance does not change.
Refund from platform got confirmation with correct amount and order balance changed.
Comment #17
tomtech commentedAutomatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.