spree-contrib / better_spree_paypal_express

A better Spree PayPal Express Extension.
http://guides.spreecommerce.org
BSD 3-Clause "New" or "Revised" License
110 stars 269 forks source link

Customer Reimburse fails #169

Open thecoconut opened 9 years ago

thecoconut commented 9 years ago

When I get to Customers Returns page and try to do Reimburse by clicking either button "Reimburse" or "Cancel" it comes back with Error 500 page. So I guess, basically I can't do reimburse or even just mark the order as reimbursed when I use this extension for accepting PayPal payments and later trying to reimburse it using the Original way of payment provided by the extension. Just reporting.

simontseng commented 9 years ago

which version of spree and the paypal addon are you using? I have just setup an 3.0 spree and cancel is not working working totally with error saying that cancel method is not implemented

in 2.4 cancel will give me error but the order is marked as canceled after refresh even with the errors refund is working in the 2.4 but i found it is broken in 3-0-stable due to currency_symbole_position is no longer a preference option

can you find your logs and post it here

thecoconut commented 9 years ago

Hi Simon, I use 3-0-stable version and same version of better PayPal extension. I will get the log later and post it to you.

Thanks!

Max

On 6 July 2015 at 14:54, Simon Tseng notifications@github.com wrote:

which version of spree and the paypal addon are you using? I have just setup an 3.0 spree and cancel is not working working totally with error saying that cancel method is not implemented

in 2.4 cancel will give me error but the order is marked as canceled after refresh even with the errors refund is working in the 2.4 but i found it is broken in 3-0-stable due to currency_symbole_position is no longer a preference option

can you find your logs and post it here

— Reply to this email directly or view it on GitHub https://github.com/spree-contrib/better_spree_paypal_express/issues/169#issuecomment-118707193 .

simontseng commented 9 years ago

i believe it should related to the other issues I have raised

171

170

scan commented 8 years ago

This is not an identical issue to #171 or #170, because this is about Reimburse, not order cancel. I have followed it and identified the problem with these lines in Spree. And additionally this line.

Basically Spree delegates the method credit for a reimbursement directly to the provider, which it assumes to be an ActiveMerchant-Class. Which it is not in this case, the provider is a PayPal::SDK::Merchant::API.

I have not found a solution to this yet, anyone has an idea?