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

fix($location): avoid unnecessary $locationChange* events due to empty hash #16636

Closed
gkalpak wants to merge 9 commits into from
Closed

fix($location): avoid unnecessary $locationChange* events due to empty hash #16636

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
7f524c4
refactor($location): minor changes (unused deps, exported globals, un…
gkalpak Jul 19, 2018
8f28de4
refactor($location): remove unnecessary capturing group in RegExp
gkalpak Jul 19, 2018
8053bc4
refactor($browser): correctly export helper used in specs
gkalpak Jul 19, 2018
bbf6900
test($location): add assertion
gkalpak Jul 19, 2018
b085893
fix($location): avoid unnecessary `$locationChange*` events due to em…
gkalpak Jul 19, 2018
c37c2e4
fixup! fix($location): avoid unnecessary `$locationChange*` events du…
gkalpak Jul 25, 2018
cdd1f42
fixup! fix($location): avoid unnecessary `$locationChange*` events du…
gkalpak Jul 25, 2018
d736771
fixup! fix($location): avoid unnecessary `$locationChange*` events du…
gkalpak Jul 26, 2018
c3ca532
fixup! fix($location): avoid unnecessary `$locationChange*` events du…
gkalpak Jul 27, 2018
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
Prev Previous commit
Next Next commit
refactor($browser): correctly export helper used in specs 
The helper is used in `fakeWindow.location.hash`. ATM, no test is using
the `hash` getter, so there were no errors.
  • Loading branch information
@gkalpak
gkalpak committed Jul 26, 2018
commit 8053bc48537b77a6b6ea7375a9d34c87a25746bd
12 changes: 6 additions & 6 deletions src/ng/browser.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,10 @@
'use strict';
/* global stripHash: true */
/* global getHash: true, stripHash: false */

function getHash(url) {
var index = url.indexOf('#');
return index === -1 ? '' : url.substr(index);
}

/**
* ! This is a private undocumented service !
Expand Down Expand Up @@ -62,11 +67,6 @@ function Browser(window, document, $log, $sniffer, $$taskTrackerFactory) {

cacheState();

function getHash(url) {
var index = url.indexOf('#');
return index === -1 ? '' : url.substr(index);
}

/**
* @name $browser#url
*
Expand Down