The Back to site button loses its path occasionally and links to "/undefined". As I cannot reliably recreate this issue I just took a shot in the dark with the fix in the patch and it would be nice to see if anyone can confirm the fix.

CommentFileSizeAuthor
#4 2550367.navbar.undefined.4.patch1.22 KBeshta
navbar-back_to_site_undefined-1.patch549 bytesTyler the Creator

Comments

Tyler Pepper created an issue. See original summary.

Tyler the Creator’s picture

Tyler the Creator commented
Related issues: +#2442997: "Back to site" 403s when losing state (gets confused by r4032login)

mqanneh queued navbar-back_to_site_undefined-1.patch for re-testing.

eshta’s picture

eshta commented
StatusFileSize
new 2550367.navbar.undefined.4.patch1.22 KB

So I'm not able to reproduce this reliably either but have seen it. I decided to take a look at the D8 code as I have not seen the problem there in the last several releases. At one point the code changed to utilize window.location for setting the url to be used as the back link. This was done to preserve language settings, etc., however, in our situation I think it would also work for this given that window.location will always be properly populated. One more reason to do a full sweep of JS changes from D8....

  • eshta committed 090a522 on 7.x-1.x 
    Issue #2550367 by eshta: Use full window location for back to site...
eshta’s picture

eshta commented
Status: Needs review » Fixed

I've gone ahead and committed the change and seeing that this is intermittent and hard to reproduce I'll close this ticket for now.... If this crops up again let's re-open and try again hopefully with steps to reproduce.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.