Janvier Posted January 11, 2019 Posted January 11, 2019 Is there a way to have it redirect to /order-confirmation for correct conversion tracking?
0 DRMasterChief Posted January 11, 2019 Posted January 11, 2019 Maybe you are interested in this (but still no solution): https://forum.thirtybees.com/topic/2391/paypal-return-url
0 datakick Posted January 11, 2019 Posted January 11, 2019 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 ]));
0 Janvier Posted January 11, 2019 Author Posted January 11, 2019 Thank you, it works! But the page has no useful content for the user. This is what I have : https://i.imgur.com/X4t7ZTx.jpg How can I make it a little nicer and useful?
0 datakick Posted January 11, 2019 Posted January 11, 2019 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?
0 DRMasterChief Posted January 11, 2019 Posted January 11, 2019 @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 :(
0 datakick Posted January 11, 2019 Posted January 11, 2019 Oh, it's also necessary to change the next line $order = $params['order']; to $order = $params['objOrder']; Basically, the beginning of the hookPaymentReturn method should look like the highlighted code here
0 DRMasterChief Posted January 11, 2019 Posted January 11, 2019 Ah yes, already seen this and it works (copied from the Github file). Can we implement the order no. into the confirmation in any way ? Think this would be enough for most of the customers. Thank you so much :) I am on the best way to finish my tb test shop....
0 Guest Posted January 12, 2019 Posted January 12, 2019 @datakick said in PayPal - Order confirmation page skipped, goes to order history: What is the reason paypal 6.0 is not used yet? I was under the impression we were all advised not to use the 6.0 beta because it makes strange calls on every page, and other erratic behavior which I don't remember. Is this not the case?
0 datakick Posted January 12, 2019 Posted January 12, 2019 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.
0 DRMasterChief Posted January 12, 2019 Posted January 12, 2019 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]
0 Guest Posted January 19, 2019 Posted January 19, 2019 (edited) 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 January 19, 2019 by Purity
0 DRMasterChief Posted January 19, 2019 Posted January 19, 2019 Hi, thank you for this information. Now you are using the stable version - which one is this? And are there the same problems as described before (from theme opener)? thank you
Question
Janvier
Is there a way to have it redirect to /order-confirmation for correct conversion tracking?
12 answers to this question
Recommended Posts
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 accountSign in
Already have an account? Sign in here.
Sign In Now