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

umbracoUrlAlias is not working when it contains uppercase characters #3214

Closed
dawoe opened this issue Oct 9, 2018 · 3 comments

Comments

Projects
None yet
2 participants
@dawoe
Copy link
Member

commented Oct 9, 2018

Bug summary

When you use the umbracoUrlAlias routing property this doesn't get resolved.

This is because the request path always comes in in lower case so it can't find the node to redirect too.

Specifics

When the editor fills the field with a value like "Starterkit" and then tries to call http:///Starterkit or http:///starterkit he doesn't get redirected to the page where the url alias is set.

If he however changes it to "starterkit" both urls above will work.

Steps to reproduce

  1. Setup a clean umbraco environment with the starter kit installed
  2. Add a field with alias "umbracoUrlAlias" to the Content page doctype
  3. Open a content page and put in "Starterkit" in the url alias field.
  4. Open the browser and go to http:///Starterkit or http:///starterkit

Expected result

The visitor would be redirected to the page where the alias is set

Actual result

The user is not redirected. It only works if the alias is entered in lower case

@dawoe

This comment has been minimized.

Copy link
Member Author

commented Oct 9, 2018

I already know what to fix...so expect a PR coming soon

dawoe added a commit to dawoe/Umbraco-CMS that referenced this issue Oct 9, 2018

umbraco#3214 make sure aliases are matched in lower case in xpath que…
…ry, because incoming url is always in lowercase

dawoe added a commit to dawoe/Umbraco-CMS that referenced this issue Oct 9, 2018

umbraco#3214 optimized the xapth query by using id function instead o…
…f scanning all id properties when we have a root node id

@dawoe dawoe referenced this issue Oct 9, 2018

Merged

#3214 - umbracoUrlAlias with uppercase characters #3220

2 of 2 tasks complete
@dawoe

This comment has been minimized.

Copy link
Member Author

commented Oct 9, 2018

PR has been created for this one #3220

nul800sebastiaan added a commit that referenced this issue Oct 9, 2018

@nul800sebastiaan

This comment has been minimized.

Copy link
Member

commented Oct 9, 2018

Fixed in #3220

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.