Closed (duplicate)
Project:
Commerce PayPal
Version:
7.x-1.x-dev
Component:
PayPal WPS
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
28 May 2012 at 06:26 UTC
Updated:
14 Jan 2015 at 11:46 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
das-peter commentedI'd say this is a Commerce PayPal issue.
Because of the lack of information I can only guess that you use the Commerce PayPal WPS module, which doesn't convert the amount if an unsupported currency is used, it only changes the currency code to the one set as default in the settings.
Attached patch should fix that - but I didn't really test it.
Comment #2
hmartens commentedHi Peter
Thanks for the feedback. I'm a bit scared to incorporated this patch on a live server ;) I'll have to set up a test server first to test it on. But thanks in the meantime for taking the time to respond and write a patch.
I really appreciate it and will definitely look at this patch!
Comment #3
andyg5000Same issue in WPP #1431464: Add “Use order currency (multi-currency support)” – option includes option to on/off conversion feature in settings. Whichever route is taken should be consistent across WPP/WPS.
Comment #4
rszrama commentedI've committed the other approach with some changes and made it operate the same between WPP / WPS.
fwiw, PayPal doesn't support South African Rands, so with the patch I've just committed, it should be converting those prices back to $25 for payment at PayPal.
Comment #5
hmartens commentedThank you so much Ryan! You are a star, a rockstar!
Comment #6
abelwang commentedcan somebody post the whole commerce_paypal_wps.module after patched? its so difficult to learn to patch a file. thank you!!!
Comment #7
rszrama commentedYou just need to update to 7.x-2.x. There shouldn't be any issues there.
Comment #8
hmartens commentedHi Ryan
I'm running the latest Commerce Paypal version and I see what you mean, it displays the dollar (or default currency) there regardless of what currency the customer selected. It's the safest way to do it I guess...gets around the "unsupported currency" issue ;) I'm just not to sure the customers will like it displaying dollars because they wil think their creditcard doesn't support dollar...
But for me it's fine...I've asked my client if they're ok with it (not that they have a choice :) )
Comment #9
morad7 commenteddoes this patch work fo Paypal EC too?