Janvier Posted January 11, 2019 Share Posted January 11, 2019 Is there a way to have it redirect to /order-confirmation for correct conversion tracking? Link to comment Share on other sites More sharing options...
0 DRMasterChief Posted January 11, 2019 Share Posted January 11, 2019 Maybe you are interested in this (but still no solution): https://forum.thirtybees.com/topic/2391/paypal-return-url Link to comment Share on other sites More sharing options...
0 datakick Posted January 11, 2019 Share 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 ])); Link to comment Share on other sites More sharing options...
0 Janvier Posted January 11, 2019 Author Share 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? Link to comment Share on other sites More sharing options...
0 datakick Posted January 11, 2019 Share 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? Link to comment Share on other sites More sharing options...
0 DRMasterChief Posted January 11, 2019 Share 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 :( Link to comment Share on other sites More sharing options...
0 datakick Posted January 11, 2019 Share 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 Link to comment Share on other sites More sharing options...
0 DRMasterChief Posted January 11, 2019 Share 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.... Link to comment Share on other sites More sharing options...
0 Purity Posted January 12, 2019 Share 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? Link to comment Share on other sites More sharing options...
0 datakick Posted January 12, 2019 Share 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. Link to comment Share on other sites More sharing options...
0 DRMasterChief Posted January 12, 2019 Share 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] Link to comment Share on other sites More sharing options...
0 Purity Posted January 19, 2019 Share 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 Link to comment Share on other sites More sharing options...
0 DRMasterChief Posted January 19, 2019 Share 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 Link to comment Share on other sites More sharing options...
Question
Janvier
Is there a way to have it redirect to /order-confirmation for correct conversion tracking?
Link to comment
Share on other sites
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