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

fix($location): LocationHashbangUrl() undefined error #11302

Conversation

tsuyoshizawa
Copy link
Contributor

Reproduce unit code and fix code for LocationHashbangUrl() undefined error 1.2 rc3 #4776

I have also caused the scenario reporting below.
#4776 (comment)

@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.

@tsuyoshizawa
Copy link
Contributor Author

CLA Signed

@googlebot
Copy link

CLAs look good, thanks!

@tsuyoshizawa tsuyoshizawa changed the title Fix LocationHashbangUrl() undefined error fix($location): LocationHashbangUrl() undefined error Mar 12, 2015
@petebacondarwin petebacondarwin added this to the 1.4.x - jaracimrman-existence milestone May 5, 2015
@petebacondarwin petebacondarwin self-assigned this May 5, 2015
@petebacondarwin
Copy link
Contributor

Good stuff @tsuyoshizawa - I will merge this.

petebacondarwin pushed a commit that referenced this pull request Jun 20, 2015
Previously, if you navigated outside of the current base URL angular
crashed with a `Cannot call method 'charAt' of undefined` error.

Closes #11302
Closes #4776
netman92 pushed a commit to netman92/angular.js that referenced this pull request Aug 8, 2015
Previously, if you navigated outside of the current base URL angular
crashed with a `Cannot call method 'charAt' of undefined` error.

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

Successfully merging this pull request may close these issues.

3 participants