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

fix($location): consider baseHref in relative link for legacy browsers #8233

Closed
wants to merge 6 commits into from
Closed

fix($location): consider baseHref in relative link for legacy browsers #8233

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
25da859
fix($location): consider baseHref in realtive link for legacy browsers
thenikso Jul 17, 2014
df05668
Merge branch 'master' of https://github.com/angular/angular.js into h…
thenikso Jul 29, 2014
dfd2959
Use cached value of baseHref instead of calculating it
thenikso Jul 29, 2014
b6d00f7
Add test for html5mode fallback with base path
thenikso Jul 29, 2014
5b99a00
Remove unneccessary setup and cleanup to legacy browser html5mode sup…
thenikso Aug 20, 2014
b65fd3c
Augment html5mode spec for legacy browser base support to include non…
thenikso Aug 20, 2014
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions src/ng/location.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -668,6 +668,13 @@ function $LocationProvider(){
if (href.indexOf('://') < 0) { // Ignore absolute URLs
var prefix = '#' + hashPrefix;
if (href[0] == '/') {
// Account for base href already present in appBase
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

// use baseHref variable is fine

if (!baseHref && href.indexOf(baseHref) === 0) {
href = href.substr(baseHref.length);
if (!href || href[0] != '/') {
href = '/' + href;
}
}

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks! I've updated accordingly

if ($browser.baseHref() != '' && href.indexOf($browser.baseHref()) === 0) {
href = href.substr($browser.baseHref().length);
if (href == '' || href[0] != '/') {
href = '/' + href;
}
}
// absolute path - replace old path
absHref = appBase + prefix + href;
} else if (href[0] == '#') {
Expand Down