Skip to content

Commit

Permalink
Fix App.container.lookup deprecation
Browse files Browse the repository at this point in the history
Using `ApplicationInstance.container.lookup` is deprecated since Ember 2.1:

http://emberjs.com/deprecations/v2.x/#toc_ember-applicationinstance-container
  • Loading branch information
trianglegrrl committed Dec 10, 2015
1 parent a91de97 commit b0825b6
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 5 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ problem we can use the `configure()` function in an initializer:
// app/instance-initializers/configure-trackjs.js

export function initialize(application) {
const trackJs = application.container.lookup('service:trackjs');
const trackJs = application.lookup('service:trackjs');

trackJs.configure({
onError(payload, err) {
Expand Down
10 changes: 6 additions & 4 deletions app/instance-initializers/configure-trackjs.js
Original file line number Diff line number Diff line change
@@ -1,15 +1,17 @@
import Ember from 'ember';
import ErrorHandler from '../utils/error-handler';

export function initialize(application) {
let trackJs = application.container.lookup('service:trackjs');
let appVersion = application.container.lookup('application:main').get('version');
export function initialize(app) {
const instance = app.lookup ? app : app.container;

const trackJs = instance.lookup('service:trackjs');
const appVersion = instance.lookup('application:main').get('version');

trackJs.configure({
version: appVersion
});

let handler = new ErrorHandler(trackJs);
const handler = new ErrorHandler(trackJs);

Ember.onerror = handler.report.bind(handler);
}
Expand Down

0 comments on commit b0825b6

Please sign in to comment.