Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

fix($location): strip off empty hash segments when comparing #10748

Closed
wants to merge 1 commit into from

Conversation

tleruitte
Copy link
Contributor

Backported from e93710f

The url is the same whether or not there is an empty # marker at the end.
This prevents unwanted calls to update the browser, since the browser is
automatically applying an empty hash if necessary to prevent page reloads.

Closes #9635

@googlebot
Copy link

Thanks for your pull request.

It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA) at https://cla.developers.google.com/.

If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check the information on your CLA or see this help article on setting the email on your git commits.

Once you've done that, please reply here to let us know. If you signed the CLA as a corporation, please let us know the company's name.

@tleruitte
Copy link
Contributor Author

I've already signed the CLA as a corporation. The corporation name is 'Whatever SA'. Should I resend the CLA to cla-submissions@google.com?

Backported from e93710f

The url is the same whether or not there is an empty `#` marker at the end.
This prevents unwanted calls to update the browser, since the browser is
automatically applying an empty hash if necessary to prevent page reloads.

Closes angular#9635
@tleruitte
Copy link
Contributor Author

So the tests fail on travis because of a timeout, but everything works fine when I run the tests locally. Is there anything I can do to fix that?

@googlebot
Copy link

CLAs look good, thanks!

petebacondarwin pushed a commit that referenced this pull request Feb 4, 2015
Backported from e93710f

The url is the same whether or not there is an empty `#` marker at the end.
This prevents unwanted calls to update the browser, since the browser is
automatically applying an empty hash if necessary to prevent page reloads.

Closes #9635
Closes #10748
@petebacondarwin
Copy link
Contributor

Landed in 1.2.x as e81b2f7.
Not sure when the next release will be...

ggershoni pushed a commit to ggershoni/angular.js that referenced this pull request Sep 29, 2015
Backported from 45f5dad

The url is the same whether or not there is an empty `#` marker at the end.
This prevents unwanted calls to update the browser, since the browser is
automatically applying an empty hash if necessary to prevent page reloads.

Closes angular#9635
Closes angular#10748
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants