Jump to content
thirty bees forum
  • 0

PayPal - Order confirmation page skipped, goes to order history


Janvier

Question

12 answers to this question

Recommended Posts

  • 0

Try to edit file /modules/paypal/controllers/front/expresscheckout.php and change line

Tools::redirectLink($this->context->link->getPageLink('order-confirmation', true, '&id_cart='.$cart->id.'&id_module='.$this->module->id.'&key='.$customer->secure_key));

with

Tools::redirectLink($this->context->link->getPageLink('order-confirmation', true, null, [ 'id_cart' => $cart->id, 'id_module' => $this->module->id, 'key' => $customer->secure_key ]));

Link to comment
Share on other sites

  • 0

There's another bug in hookPaymentReturn. Edit file paypal.php, find method hookPaymentReturn and change the first line from

if (!$this->active || !isset($params['order']) || !$params['order'] instanceof Order) {

to

if (!$this->active || !isset($params['objOrder']) || !$params['objOrder'] instanceof Order) {

Note that all these fixes are already part of the paypal v6.0.0, see here

What is the reason paypal 6.0 is not used yet?

Link to comment
Share on other sites

  • 0

@datakick thank you for the information. This is really a great message and a first step for a solution for this persistent problem. Thank you thousand times!

When i change /modules/paypal/controllers/front/expresscheckout.php it works and i get a simple confirmation site (without order ID or something). When i change paypal.php additionally, i get a 500 Error without decrypted error code :(

Link to comment
Share on other sites

  • 0

Frankly I don't know. I remember there was a whole thread (https://forum.thirtybees.com/topic/1095/paypal-6-0-0-beta-the-final-push-for-victory) dedicated to paypal v6, but that seems to be deleted since. So that knowledge is gone forever.

I could find only one bug on github regarding v6 (redundant api calls on every page). While that's unfortunate, it's not something that should prevent using that version of the module. But I'm sure there is the reason nobody is using it. But it would be great if we have those reasons tracked as github issues, otherwise we can never fix this.

Here we need merchants help. Please test v6, if you can, and let us know what does not work. So we can finally resolve this paypal hell.

Link to comment
Share on other sites

  • 0

There are 2 sites in Cache:

https://webcache.googleusercontent.com/search?q=cache:BkN0CQt2f3EJ:https://forum.thirtybees.com/topic/1095/paypal-6-0-0-beta-1-the-final-push-for-victory/34+&cd=13&hl=de&ct=clnk&gl=de

https://webcache.googleusercontent.com/search?q=cache:tn9YOV1z3MAJ:https://forum.thirtybees.com/topic/1095/paypal-6-0-0-beta-the-final-push-for-victory/74%3Fpage%3D5+![cd=12&hl=de&ct=clnk&gl=de]

Link to comment
Share on other sites

  • 0
On 1/12/2019 at 12:54 AM, datakick said:

Frankly I don't know. I remember there was a whole thread (https://forum.thirtybees.com/topic/1095/paypal-6-0-0-beta-the-final-push-for-victory) dedicated to paypal v6, but that seems to be deleted since. So that knowledge is gone forever.

I could find only one bug on github regarding v6 (redundant api calls on every page). While that's unfortunate, it's not something that should prevent using that version of the module. But I'm sure there is the reason nobody is using it. But it would be great if we have those reasons tracked as github issues, otherwise we can never fix this.

Here we need merchants help. Please test v6, if you can, and let us know what does not work. So we can finally resolve this paypal hell.

I actually did use the v6 beta for a couple weeks last year and I don't remember having any issues, but decided to use the stable version, because, well, it's the stable version. lol

Edited by Purity
Link to comment
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
×
×
  • Create New...