[FEATURE ember-eager-url-update] link-to should generate the url if it eagerly updates it, not use the href because it includes rootURL. Fixes #1244

[jayphelps]

link-to used to let the transition update the URL, but #4122 made link-to request a URL update with the router, providing its own URL based off the href cp.

Unfortunately, the href includes the rootURL, if one is used, which the router, nor the Location classes are expecting it to already have, so they prepend it again.

This PR makes link-to always generate the URL, just like the transition does.

Fixes #1244

[jayphelps]

@pixelhandler

[rwjblue]

👍

@machty - When you have a moment, could you +1/-1 since this is related to ember-eager-url-update?

[jayphelps]

@rjackson The commit should actually be [FEATURE ember-eager-url-update] instead of BUGFIX since this is canary online still, right?

[rwjblue]

@jayphelps - Yes, good call. This has not made it into a beta, so FEATURE ember-eager-url-update is the right way to go.

[jayphelps]

@rjackson okay, I'm mobile right now so will amend later tonight.

[jayphelps]

@rjackson @machty amended and now includes a passing test. lmk

[pixelhandler]

@jayphelps @rjackson @machty - I updated the example (busted) app with the fix for the link-to url here: http://pixelhandler.github.io/ember-data-extensions/blog-fix/ looks like Jay's fix solved the issue. Busted rootURL example app: http://pixelhandler.github.io/ember-data-extensions/blog/