spree-contrib / spree_paypal_express

allowing use of paypal express in spree, using (a modified version of) active merchant
BSD 3-Clause "New" or "Revised" License
113 stars 141 forks source link

Gateway Error: translation missing: en.the_totals_of_the_cart_item_amounts_do_not_match_order_amounts #106

Closed pavelnikolov closed 11 years ago

pavelnikolov commented 11 years ago

Hello,

I have spree 1-2-stable, and I'm using spree-paypap-express 1-2-stable branch (tried with master too but again failed). When I try to checkout an order and select PayPal express as the payment method it fails with an error: "Gateway Error: translation missing: en.the_totals_of_the_cart_item_amounts_do_not_match_order_amounts". For some reason the order total does not match cart item amounts. I spent whole weekend on that and still have no idea why this is happening.

The issue only happens if I include shipping with price > 0 (it works if I set the shipping price to 0). As soon as I change the shipping price to be a value >0 or some % it breaks. I tried with both "Shipment including VAT" true and false - same thing.

Any ideas? Is this a bug or I'm I missing something?

Pavel

pavelnikolov commented 11 years ago

Here are more details. I have 3 shipping options. 1 for Denmark, 1 for rest of Europe and one for outside of Europe. The problem is that in the spp extension it is hardcoded to always get the first one instead of the one matching the shipping country. This caused big problems for me.

bolom commented 11 years ago

I have the same issue

JDutil commented 11 years ago

Thanks for the report and details guys. I'll try taking a look into this when I get a chance. If you are able to tackle this with a pull request in the meantime before I can get to it that would be awesome, and likely get this issue resolved faster.

bolom commented 11 years ago

@jdutil pull submitted

pavelnikolov commented 11 years ago

Thanks about your effort!

JDutil commented 11 years ago

@pavelnikolov could you please:

$ bundle update spree_paypal_express

Then give my latest update a shot, which should resolve this issue for you. In master as 9dd1f47110dbe26aac045441ea36d545db743d1c but for 1-2-stable which I believe your ussing is dc6bccfad3cec99bd3384de1c59d8ddbbd1f1914

jetsgit commented 11 years ago

@jdutil was this fixed in Spree v.1.3.2? I just did a vanilla install + spree_tax_cloud and I got this same error.

Jet
JDutil commented 11 years ago

@bluehandtalking the fix for the particular error being received before is in 1-3-stable. Are you sure your Gemfile.lock is referencing the latest commit in the 1-3-stable branch? There could possibly be some incorrect values being passed around depending on how spree_tax_cloud ends up integrating with this extension, but I don't know of any reason why they shouldn't just work fine together.

jetsgit commented 11 years ago

@jdutil

Think you are right, was pulling in wrong version---that error has disppeared.

There is no problem with spree_tax_cloud at this point, as I have removed it for this test. Side note, spree_tax_cloud now properly updates tax in cart if user leaves checkout process to add or delete items in cart.

I am getting another error which I will open a separate issue on

 Jet

On 02/02/2013 12:15 PM, Jeff Dutil wrote:

@bluehandtalking https://github.com/bluehandtalking the fix for the particular error being received before is in 1-3-stable. Are you sure your Gemfile.lock is referencing the latest commit in the 1-3-stable branch? There could possibly be some incorrect values being passed around depending on how spree_tax_cloud ends up integrating with this extension, but I don't know of any reason why they shouldn't just work fine together.

— Reply to this email directly or view it on GitHub https://github.com/spree/spree_paypal_express/issues/106#issuecomment-13036545.