-
Notifications
You must be signed in to change notification settings - Fork 27.4k
fix($brower): set the url even if the browser transforms it #14499
Conversation
While $location expects that $browser stores the URL unchanged, some browsers transform the URL when setting or defer the acutal update. To work around this, $browser.url() kept the unchanged URL in pendingLocation. However, it failed to update pendingLocation in all code paths, causing $browser.url() to sometimes incorrectly report previous URLs, which horribly confused $location. This fix ensures that pendingLocation is always updated if set, causing url() to report the current url. Fixes #14427
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed, please reply here (e.g.
|
1 similar comment
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed, please reply here (e.g.
|
I signed the CLA. |
CLAs look good, thanks! |
1 similar comment
CLAs look good, thanks! |
It's been two weeks ... is there anything I can do to accelerate the review process, @gkalpak? |
@petebacondarwin might be best qualified to review this change, since he introduced the |
I'll take a look next week |
@bedag-moo - really nice PR. |
While $location expects that $browser stores the URL unchanged, "some browsers" transform the URL when setting or defer the acutal update. To work around this, $browser.url() kept the unchanged URL in pendingLocation. However, it failed to update pendingLocation in all code paths, causing $browser.url() to sometimes incorrectly report previous URLs, which horribly confused $location. This fix ensures that pendingLocation is always updated if set, causing url() to report the current url. Fixes #14427 Closes #14499
Isn't there a typo in the title and now in the changelog? brower instead of browser? |
Fixed the CHANGELOG at f58d4fb |
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Fixes #14427
What is the current behavior? (You can also link to an open issue here)
See #14427
What is the new behavior (if this is a feature change)?
Features should not be affected - unless you consider inifinite digest a feature ;-)
Does this PR introduce a breaking change?
No as far as I know.
Please check if the PR fulfills these requirements
Other information:
While $location expects that $browser stores the URL unchanged, some browsers transform the URL
when setting or defer the acutal update. To work around this, $browser.url() kept the unchanged
URL in pendingLocation. However, it failed to update pendingLocation in all code paths, causing
$browser.url() to sometimes incorrectly report previous URLs, which horribly confused $location.
This fix ensures that pendingLocation is always updated if set, causing url() to report the
current url.
Fixes #14427