 |
UbercartOne cart to rule them all... |
|
| Donate to Ubercart | Affiliates | ||
Broken link on Order Complete page
Submitted by deadrich on Tue, 05/13/2008 - 17:30
First: Great work with the software! I've found an error on /cart/checkout/complete After completing a sale, the user is sent to this page, which contains the following text. "Thank you for shopping at XXX. While logged in, you may continue shopping or click here to view your current order status and order history." The "click here" link does not seem to be getting the correct user ID and instead links to /user/1/orders which causes an Access Denied error. Thanks!
Yikes... can you tell me under what circumstances this goof occurs? It doesn't happen all the time, as I've been testing checkout heavily on the Livetest today and haven't run into it. This is happening after purchasing a file download, using PayPal Standard and clicking the link on PayPal that says "return to merchant". Hi! I have the same problem here. Honestly... I don't know how this is possible. After reviewing the code that generates this message, I see that it pulls the user ID straight from the logged in user. I'm not sure how it would turn up with a 1 if that's not the current user's ID. I get this error too using the Check payment method. (haven't checked any other methods) I've looked at the code and it is a little strange that it gets user id of 1, as the 'my account' link i have in a menu works fine with the correct user id. I think the issue is that the variable uc_msg_order_logged_in get set to something like the following: Thank you for shopping at [store-name]. While logged in, you may continue shopping or click here to view your current order status and order history. It seems when this variable is set you aren't using a token, and it is just set for the admin user. I emptied the variable in admin/store/settings/checkout/edit/messages and seems to do the trick as code in the uc_messages hook is fine. Ben Thanks so much for tracking this down! Looks like a silly mistake on my part. Bookmarking this post so I can work a fix into the next minor release. I've replaced this for now w/ the [order-url] token. This will go to the order view page for the order that was just placed, which is more intuitive. I left the message text the same for now but can change it later if need be.
|
|
Joined: 05/13/2008