Shopping cart remains full after processing order through PayPal?

This is a question for Ubercart, not the theme.

subscribing

Hi, I was wondering if this is still an issue?

Same problem here

Drupal: 6.16
Ubercart: 2.2
Paypal is the only method of payment
I do not have Paypal Express Checkout
Orders are completed successfully: please see Picture 1
The shopping cart remains full: please see Picture 2

Any ideas on how to solve this problem?

Thank you in advance

Has anyone found a good solution for this?

I am having a similar problem with paypal when a user doesn't redirect back to the store. I added text the the checkout review pane to instruct customer to redirect back to the store, but I know that everyone doesn't read the messages. I also set up the redirect in the paypal sandbox account but it's only redirecting when you use a paypal account.

I am using
ubercart 2.2
drupal 6.16
uc_node_checkout 2.0
uc_pictured_cart_block 1.0
uc_restict_qty 1.1

any help would be greatly appreciated

I can confirm this problem still exists in UC 2.3 for 6.x, but this only happens if the user doesn't click the "Return to store" link on the PayPal site.

This can be a major issue, because the user might not think their order went through when they return to the website (users never read what sites like PayPal are telling them...), and they might make a double payment.

It looks like this might have been mentioned in the following forum post:
http://www.ubercart.org/forum/support/2078/paypal_ipn_cart_not_empty

Also mentioned here:
http://www.ubercart.org/forum/support/14646/paypal_standard_users_closes...

I'm actively developing a patch for this issue. The problem is not technically with IPN handling for uc_paypal – it calls uc_complete_sale when the IPN status is "Completed" – but, unfortunately, uc_complete_sale appears to expect that it's being called from the same PHP session as the one the customer is using. For IPN, that isn't true, so it's not able to locate the user's cart in order to clear it.

For logged-in users, that's an easy fix because the cart ID is just their user ID. Unfortunately, users who complete checkout before getting an account are assigned a completely random string for their cart ID, so during the IPN we will need some way to locate their cart. I may have to pass the cart ID along to PayPal as extra data, and then handle it on the other side.

The attached patch appears to correct the issue. The cart ID is now passed as part of the invoice ID, in place of what was originally the timestamp. I'm not sure what benefit the original logic of passing a timestamp had; if that logic is again required in the future, we can always append it after the cart ID.

@Island Usurper, @Andy_Lowe, or another UberCart maintainer: perhaps you can enlighten me on why we were appending a timestamp to the invoice ID passed to PayPal?

Anyway, the cart ID is set in the session during the IPN handling so that uc_complete_sale() can clear it out properly. I also noticed that uc_complete_sale() was not passing FALSE in the call to uc_cart_get_id(), which meant that in the case of the original IPN code, a new cart was created and then immediately emptied, which didn't make much sense. It was a small thing, but counter-intuitive and inefficient nonetheless, so I've patched this as well.

It Works!!! Thank you very much for this Guy and I really hope this gets committed.

I need this too - did it get added to a build yet? Thanks!

does anyone know if this was committed?

It has not been committed as of yet. Subscribing to this. Would love to get TR, Lyle or Andy to look at this. IPN has always needed a bit of help, and we've seen this issue before but never narrowed it down. Good work!

Making this a current Version issue since the code is the same even in current -dev.

The patch was applied cleanly, the logic looks good in it, to me anyway. Testing on production, will let you know if I see any side effects.

I honestly don't remember why time() was part of the invoice ID. Possibly it was a check to make sure the IPN wasn't from some old, out of date order, but there doesn't seem to be any code to do that.

Thanks, and committed.

I am experiencing the same issue with 2CheckOut.

Could they be related or is this just with PayPal? Do you recommend applying the above patch on a production site?

Thanks...

I am using Ubercart 2.4 and the changes to the patch above in #11 are not committed to this release as indicated in #20 above (actually that comment refers to 2.3, but shouldn't the changes still be in effect for 2.4?)

I am setting up a webstore and was testing the Paypal functionality when I ran into this issue and found this post. Could anyone provide feedback on this issue?

Thanks!

Seems you're right, the issue should actually have been listed as a 2.4 issue since I don't believe the fix made it in time. You should download the latest dev version instead.

spam cleanup