Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IExplore 11: 404 when switiching from hierarchy to alphabetical and back to hierarchy #633

Closed
d0rg0ld opened this issue Oct 6, 2017 · 1 comment

Comments

@d0rg0ld
Copy link

d0rg0ld commented Oct 6, 2017

Hi,

In Internet Explorer (Not in Chrome and not in Firefox, versions see below) I encounter the following problem:

At which URL did you encounter the problem?

This happens when using the current SKOSMOS dev snapshot, but can also be reproduced in Finto for example:

http://finto.fi/pto/en/page/?uri=http://paikkatiedot.fi/def/1001002/p678

What steps will reproduce the problem?

  1. Navigate to http://finto.fi/pto/en/page/?uri=http://paikkatiedot.fi/def/1001002/p678
  2. Switch to "Alphabetical"
  3. Switch back to "Hierarchy"

What is the expected output? What do you see instead?

The browser should navigate back to http://finto.fi/pto/en/page/?uri=http://paikkatiedot.fi/def/1001002/p678 but tries to open http://finto.fi/pto/en/pto/en/page/?uri=http://paikkatiedot.fi/def/1001002/p678 instead, leading to a 404.

The malformed URL contains one vocab/lang pair ( here pto/en/ ) too much

I suspect that this happens in the context of the call "window.location.replace(encodeURI(redirectUrl));" in resource/js/docready.js:364.

What browser did you use? (eg. Firefox, Chrome, Safari, Internet explorer)

IExplore 11 (11.0.9600.18697)
Windows 7

This problem does not appear in Chrome 61.0.3163.100 and Firefox 55.0.3

@osma osma added the bug label Oct 6, 2017
@osma osma added this to the 1.10 milestone Oct 6, 2017
@osma
Copy link
Member

osma commented Oct 6, 2017

Thanks for reporting.

henriyli pushed a commit that referenced this issue Oct 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants