Skip to content

Commit

Permalink
Merge pull request #14229 from rwjblue/fix-ie9-auto-location
Browse files Browse the repository at this point in the history
[BUGFIX beta] Fix issue with IE9 compatibility.
  • Loading branch information
rwjblue committed Sep 8, 2016
2 parents 945d81a + 693e188 commit aeced15
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 2 deletions.
4 changes: 2 additions & 2 deletions packages/ember-routing/lib/location/auto_location.js
Original file line number Diff line number Diff line change
Expand Up @@ -70,9 +70,9 @@ export default EmberObject.extend({
@since 1.11
@property global
@default environment.global
@default window
*/
global: environment.global,
global: environment.window,

/**
@private
Expand Down
8 changes: 8 additions & 0 deletions packages/ember-routing/tests/location/auto_location_test.js
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
import { environment } from 'ember-environment';
import { get, run, assign } from 'ember-metal';
import AutoLocation from '../../location/auto_location';
import {
Expand Down Expand Up @@ -60,6 +61,13 @@ QUnit.module('Ember.AutoLocation', {
}
});

QUnit.test('AutoLocation should have the `global`', function(assert) {
let location = AutoLocation.create();

assert.ok(location.global, 'has a global defined');
assert.strictEqual(location.global, environment.window, 'has the environments window global');
});

QUnit.test('AutoLocation should return concrete implementation\'s value for `getURL`', function() {
expect(1);

Expand Down

0 comments on commit aeced15

Please sign in to comment.