Skip to content

Commit f9fedfc

Browse files
ember_debug: remove import Ember from js code
1 parent 5e1f2a6 commit f9fedfc

File tree

9 files changed

+42
-65
lines changed

9 files changed

+42
-65
lines changed

ember_debug/adapters/web-extension.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import BasicAdapter from './basic';
22
import { typeOf } from 'ember-debug/utils/type-check';
33

4-
import Ember from 'ember-debug/utils/ember';
4+
import { getEnv } from 'ember-debug/utils/ember';
55
import { run } from 'ember-debug/utils/ember/runloop';
66

77
const { isArray } = Array;
@@ -106,7 +106,7 @@ export default class extends BasicAdapter {
106106
// adapter later. See GH #1114.
107107
const HAS_ARRAY_PROTOTYPE_EXTENSIONS = (() => {
108108
try {
109-
return Ember.ENV.EXTEND_PROTOTYPES.Array === true;
109+
return getEnv().EXTEND_PROTOTYPES.Array === true;
110110
} catch {
111111
return false;
112112
}

ember_debug/general-debug.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
/* eslint no-empty:0 */
22
import DebugPort from './debug-port';
33

4-
import Ember from 'ember-debug/utils/ember';
4+
import { libraries } from 'ember-debug/utils/ember';
55

66
/**
77
* Class that handles gathering general information of the inspected app.
@@ -88,7 +88,7 @@ export default class extends DebugPort {
8888
*/
8989
getLibraries() {
9090
this.sendMessage('libraries', {
91-
libraries: Ember.libraries?._registry,
91+
libraries: libraries?._registry,
9292
});
9393
},
9494

ember_debug/models/profile-manager.js

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import ProfileNode from './profile-node';
2-
import Ember from 'ember-debug/utils/ember';
2+
import { VERSION } from 'ember-debug/utils/ember';
33
import { compareVersion } from 'ember-debug/utils/version';
44

55
import { later, scheduleOnce, cancel } from 'ember-debug/utils/ember/runloop';
@@ -75,8 +75,7 @@ export default class ProfileManager {
7575
this.stylesheet = insertStylesheet();
7676
// keep track of all the active highlights
7777
this.highlights = [];
78-
// eslint-disable-next-line ember/new-module-imports
79-
this.isHighlightEnabled = compareVersion(Ember?.VERSION, '3.20.0') !== -1;
78+
this.isHighlightEnabled = compareVersion(VERSION, '3.20.0') !== -1;
8079
}
8180

8281
began(timestamp, payload, now) {

ember_debug/utils/ember.js

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,14 +63,22 @@ let EmberDestroyable = emberSafeRequire('@ember/destroyable');
6363
let ObjectInternals = emberSafeRequire('@ember/object/internals');
6464
let Instrumentation = emberSafeRequire('@ember/instrumentation');
6565
let Runloop = emberSafeRequire('@ember/runloop');
66+
6667
let RSVP = emberSafeRequire('rsvp');
68+
6769
let GlimmerComponent = emberSafeRequire('@glimmer/component');
6870
let GlimmerManager = emberSafeRequire('@glimmer/manager');
6971
let GlimmerReference = emberSafeRequire('@glimmer/reference');
7072
let GlimmerRuntime = emberSafeRequire('@glimmer/runtime');
7173
let GlimmerUtil = emberSafeRequire('@glimmer/util');
7274
let GlimmerValidator = emberSafeRequire('@glimmer/validator');
7375

76+
let inspect = Debug?.inspect || InternalsUtils?.inspect;
77+
let subscribe = Instrumentation?.subscribe;
78+
let cacheFor = ObjectInternals?.cacheFor;
79+
let guidFor = ObjectInternals?.guidFor;
80+
let libraries = InternalsMetal?.libraries;
81+
7482
if (Ember) {
7583
captureRenderTree = Ember._captureRenderTree;
7684
getEnv = () => Ember.ENV;
@@ -96,6 +104,15 @@ if (Ember) {
96104
set = Ember.set;
97105
computed = Ember.computed;
98106
ActionHandler = Ember.ActionHandler;
107+
Debug = Debug ?? Ember.Debug;
108+
inspect = inspect ?? Ember.inspect;
109+
Instrumentation = Instrumentation ?? Ember.Instrumentation;
110+
subscribe = subscribe ?? Ember.subscribe;
111+
RSVP = RSVP ?? Ember.RSVP;
112+
Runloop = Runloop ?? Ember.run;
113+
cacheFor = cacheFor ?? Ember.cacheFor;
114+
guidFor = guidFor ?? Ember.guidFor;
115+
libraries = libraries ?? Ember.libraries;
99116
} else {
100117
captureRenderTree = emberSafeRequire('@ember/debug')?.captureRenderTree;
101118
getEnv = emberSafeRequire('@ember/-internals/environment')?.getENV;
@@ -160,6 +177,11 @@ export {
160177
set,
161178
captureRenderTree,
162179
getEnv,
180+
inspect,
181+
subscribe,
182+
cacheFor,
183+
guidFor,
184+
libraries,
163185
GlimmerComponent,
164186
GlimmerManager,
165187
GlimmerReference,

ember_debug/utils/ember/debug.js

Lines changed: 4 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,5 @@
1-
import Ember, { Debug, InternalsUtils } from '../ember';
1+
import { Debug, inspect as emberInspect } from '../ember';
22

3-
let module;
4-
export let inspect;
5-
6-
if (Debug) {
7-
module = Debug;
8-
inspect = Debug.inspect || InternalsUtils.inspect;
9-
} else {
10-
module = Ember.Debug;
11-
// eslint-disable-next-line ember/new-module-imports
12-
inspect = Ember.inspect;
13-
}
14-
15-
if (!inspect) {
16-
// eslint-disable-next-line ember/new-module-imports
17-
inspect = Ember.inspect;
18-
}
19-
20-
export let { registerDeprecationHandler } = module;
21-
export default module;
3+
export let inspect = emberInspect;
4+
export let { registerDeprecationHandler } = Debug;
5+
export default Debug;
Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,3 @@
1-
import Ember, { Instrumentation } from '../ember';
1+
import { subscribe as emberSubscribe } from '../ember';
22

3-
let module;
4-
5-
if (Instrumentation) {
6-
module = Instrumentation;
7-
} else {
8-
module = Ember;
9-
}
10-
11-
export let { subscribe } = module;
3+
export let subscribe = emberSubscribe;

ember_debug/utils/ember/object/internals.js

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,4 @@
1-
import Ember, { ObjectInternals } from '../../ember';
2-
3-
// eslint-disable-next-line ember/new-module-imports
4-
let cacheFor = ObjectInternals?.cacheFor ?? Ember.cacheFor;
5-
// eslint-disable-next-line ember/new-module-imports
6-
let emberGuidFor = ObjectInternals?.guidFor ?? Ember.guidFor;
1+
import { cacheFor, guidFor as emberGuidFor } from '../../ember';
72

83
// it can happen that different ember apps/iframes have the same id for different objects
94
// since the implementation is just a counter, so we add a prefix per iframe & app

ember_debug/utils/ember/runloop.js

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import Ember, { Runloop as EmberRunloop } from '../ember';
1+
import { Runloop as EmberRunloop } from '../ember';
22
import * as runloop from './own-runloop';
33

44
// it could happen that runloop is available but _backburner is not exported (dead code)
@@ -8,15 +8,9 @@ let _backburner = runloop._backburner;
88

99
const keys = ['cancel', 'debounce', 'join', 'later', 'scheduleOnce'];
1010

11-
if (EmberRunloop) {
12-
module = EmberRunloop;
13-
_backburner = EmberRunloop._backburner || EmberRunloop.backburner;
14-
} else {
15-
// eslint-disable-next-line ember/new-module-imports
16-
module = Ember?.run || module;
17-
// eslint-disable-next-line ember/new-module-imports
18-
_backburner = Ember?.run?.backburner || _backburner;
19-
}
11+
module = EmberRunloop;
12+
_backburner =
13+
EmberRunloop._backburner || EmberRunloop.backburner || _backburner;
2014

2115
if (!keys.every((k) => k in module)) {
2216
module = runloop;

ember_debug/utils/rsvp.js

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,5 @@
1-
import Ember, { RSVP as emberRSVP } from './ember';
1+
import { RSVP as emberRSVP } from './ember';
22

3-
let module;
3+
export let { Promise, all, resolve } = emberRSVP;
44

5-
if (emberRSVP) {
6-
module = emberRSVP;
7-
} else {
8-
// eslint-disable-next-line ember/new-module-imports
9-
module = Ember.RSVP;
10-
}
11-
12-
export let { Promise, all, resolve } = module;
13-
14-
export default module;
5+
export default emberRSVP;

0 commit comments

Comments
 (0)