A redirect with an alias keeps the original url

[callumacrae]

Version

2.7.0

Reproduction link

http://jsfiddle.net/9r6xhqbp/37/

Steps to reproduce

Add the following to your router:

  {
    path: '/view',
    alias: '/view/*',
    redirect: '/analytics',
  },

Navigate to /view/blabla.

What is expected?

I'd expect to be redirected to /analytics

What is actually happening?

/view/* is now an alias for /analytics, where I would have expected it to redirect instead.

/view redirects as expected.

[callumacrae]

Oh and for anyone else experiencing this (unlikely, it seems pretty specific), you can work around it by just adding two entries to your router:

  {
    path: '/view',
    redirect: '/analytics',
  },
  {
    path: '/view/*',
    redirect: '/analytics',
  },

[posva]

Callum please, take the time for the repro and instructions 😉

[callumacrae]

I figured it would be quicker to add that block of code into any existing project using vue-router (and change the value of redirect:) than to clone a git repo and run npm install 😛 will set up a repo now

[posva]

You don't need a repo for that, use a fiddle: http://jsfiddle.net/posva/9r6xhqbp/

[callumacrae]

that's easier 😄 added it to the main issue: http://jsfiddle.net/9r6xhqbp/37/

[posva]

Thanks, as you can imagine, the workaround is to use two routes, one for every redirect

[posva]

On second thought, this is working as expected, because the point of alias is to keep the original url, so I prefer documenting this and keeping current behaviour which is more flexible

[nandin-borjigin]

I suppose redirect should take a higher priority here.

User story:

1. User navigates to /view/*
2. Router finds out /view/* is an alias and it should keep the original url while following other rules defined under /view
3. Router finds a redirect under /view
4. Router performs redirection to /analytics

Since /analytics is totally a different thing from /view, so the expected behavior of the alias(i.e. keep the original url) is out of scope here, hence /analytics in the url bar and $route.path.

ping @posva

[posva]

~~For that scenario, use multiple redirects but currently, the situation allows to keep the alias, I don't want to remove that~~

Revisiting this, I think it makes more sense for the redirect to take precedence