ember-source 4.2.0-alpha.6 → 4.3.0-alpha.2

package/dist/header/license.js

@@ -5,5 +5,5 @@
  *            Portions Copyright 2008-2011 Apple Inc. All rights reserved.
  * @license   Licensed under MIT license
  *            See https://raw.github.com/emberjs/ember.js/master/LICENSE
- * @version   4.2.0-alpha.6
+ * @version   4.3.0-alpha.2
  */

package/dist/header/loader.js

@@ -81,7 +81,6 @@ var define, require;
     return internalRequire(name, null);
   };
 
-  // eslint-disable-next-line no-unused-vars
   define = function (name, deps, callback) {
     registry[name] = { deps: deps, callback: callback };
   };

package/dist/packages/@ember/-internals/bootstrap/index.js

@@ -2,9 +2,7 @@ import require from 'require';
 
 (function bootstrap() {
   // Bootstrap Node module
-  // eslint-disable-next-line no-undef
   if (typeof module === 'object' && typeof module.require === 'function') {
-    // tslint:disable-next-line: no-require-imports
     module.exports = require('ember').default;
   }
 })();

package/dist/packages/@ember/-internals/container/index.js

@@ -811,6 +811,7 @@ class Registry {
       url: 'https://deprecations.emberjs.com/v4.x#toc_implicit-injections',
       for: 'ember-source',
       since: {
+        available: '4.0.0',
         enabled: '4.0.0'
       }
     });

package/dist/packages/@ember/-internals/glimmer/index.js

@@ -2,11 +2,11 @@ import { templateFactory, programCompilationContext } from '@glimmer/opcode-comp
 export { templateFactory as template, templateCacheCounters } from '@glimmer/opcode-compiler';
 import { setOwner, getOwner } from '@ember/-internals/owner';
 import { guidFor, symbol, enumerableSymbol, getDebugName, isProxy, isEmberArray, isObject, uuid } from '@ember/-internals/utils';
-import { assert, warn, debugFreeze, deprecate } from '@ember/debug';
+import { assert, warn, debugFreeze, inspect, deprecate } from '@ember/debug';
 import { setComponentTemplate, setInternalComponentManager, setInternalHelperManager, getInternalHelperManager, helperCapabilities, setHelperManager, capabilityFlagsFrom, setInternalModifierManager, getComponentTemplate, getInternalComponentManager, componentCapabilities, modifierCapabilities, setComponentManager } from '@glimmer/manager';
 import { createConstRef, isConstRef, valueForRef, isUpdatableRef, updateRef, childRefFor, childRefFromParts, createComputeRef, createPrimitiveRef, createUnboundRef, isInvokableRef, createInvokableRef, createReadOnlyRef, createDebugAliasRef, UNDEFINED_REFERENCE } from '@glimmer/reference';
 import { untrack, consumeTag, createCache, getValue, tagFor, beginUntrackFrame, endUntrackFrame, valueForTag, beginTrackFrame, endTrackFrame, validateTag, createTag, dirtyTag, CONSTANT_TAG, isTracking, setTrackingTransactionEnv, createUpdatableTag, CURRENT_TAG } from '@glimmer/validator';
-import { tracked, get, PROPERTY_DID_CHANGE, tagForObject, objectAt, tagForProperty, _getProp, _setProp, set } from '@ember/-internals/metal';
+import { tracked, get, PROPERTY_DID_CHANGE, tagForObject, objectAt, tagForProperty, set, _getProp, _setProp } from '@ember/-internals/metal';
 import { action } from '@ember/object';
 import { hasDOM } from '@ember/-internals/browser-environment';
 import { isSimpleClick, clearElementView, clearViewElement, getViewElement, MUTABLE_CELL, addChildView, setElementView, setViewElement, ActionSupport, ChildViewsSupport, ClassNamesSupport, CoreView, ViewMixin, ViewStateSupport, constructStyleDeprecationMessage, ActionManager, getViewId } from '@ember/-internals/views';
@@ -16,9 +16,9 @@ import { service } from '@ember/service';
 import { DEBUG } from '@glimmer/env';
 import { dasherize } from '@ember/string';
 import { registerDestructor, associateDestroyableChild, destroy } from '@glimmer/destroyable';
-import { join, _backburner, schedule, _getCurrentRunLoop } from '@ember/runloop';
+import { join, schedule, _backburner, _getCurrentRunLoop } from '@ember/runloop';
 import { _WeakSet, EMPTY_ARRAY, unwrapTemplate, dict } from '@glimmer/util';
-import { reifyPositional, normalizeProperty, EMPTY_ARGS, createCapturedArgs, curry, EMPTY_POSITIONAL, array, concat, fn, get as get$1, hash, on, TEMPLATE_ONLY_COMPONENT_MANAGER, templateOnlyComponent, clientBuilder, DOMChanges, DOMTreeConstruction, inTransaction, renderMain, runtimeContext, rehydrationBuilder } from '@glimmer/runtime';
+import { reifyPositional, normalizeProperty, EMPTY_ARGS, createCapturedArgs, curry, EMPTY_POSITIONAL, array, concat, fn, get as get$1, hash, on, templateOnlyComponent, TEMPLATE_ONLY_COMPONENT_MANAGER, clientBuilder, DOMChanges, DOMTreeConstruction, inTransaction, renderMain, runtimeContext, rehydrationBuilder } from '@glimmer/runtime';
 export { DOMChanges, DOMTreeConstruction, isSerializationFirstNode } from '@glimmer/runtime';
 import { TargetActionSupport, FrameworkObject, _contentFor, isArray } from '@ember/-internals/runtime';
 import { ENV } from '@ember/-internals/environment';
@@ -975,9 +975,15 @@ class LinkTo extends InternalComponent {
       try {
         return routing.generateURL(route, models, query);
       } catch (e) {
-        // tslint:disable-next-line:max-line-length
-        e.message = `While generating link to route "${route}": ${e.message}`;
-        throw e;
+        let details = e instanceof Error ? e.message : inspect(e);
+        let message = `While generating link to route "${route}": ${details}`;
+
+        if (e instanceof Error) {
+          e.message = message;
+          throw e;
+        } else {
+          throw message;
+        }
       }
     } else {
       return routing.generateURL(route, models, query);
@@ -1023,8 +1029,7 @@ class LinkTo extends InternalComponent {
       transition: undefined
     };
     flaggedInstrument('interaction.link-to', payload, () => {
-      assert('[BUG] route can only be missing if isLoading is true', isPresent(route)); // TODO: is the signature wrong? this.query is definitely NOT a QueryParam!
-
+      assert('[BUG] route can only be missing if isLoading is true', isPresent(route));
       payload.transition = routing.transitionTo(route, models, query, replace);
     });
   }
@@ -1054,8 +1059,7 @@ class LinkTo extends InternalComponent {
     } else {
       return EMPTY_ARRAY$1;
     }
-  } // TODO: this should probably be Record<string, unknown> or something
-
+  }
 
   get query() {
     if ('query' in this.args.named) {
@@ -1155,8 +1159,7 @@ class LinkTo extends InternalComponent {
         query,
         routing
       } = this;
-      assert('[BUG] route can only be missing if isLoading is true', isPresent(route)); // TODO: is the signature wrong? this.query is definitely NOT a QueryParam!
-
+      assert('[BUG] route can only be missing if isLoading is true', isPresent(route));
       return routing.isActiveForRoute(models, query, route, state);
     }
   }
@@ -1491,8 +1494,8 @@ function installAttributeBinding(component, rootRef, parsed, operations) {
       elementId = component.elementId;
     }
 
-    elementId = createPrimitiveRef(elementId);
-    operations.setAttribute('id', elementId, true, null);
+    let elementIdRef = createPrimitiveRef(elementId);
+    operations.setAttribute('id', elementIdRef, true, null);
     return;
   }
 
@@ -1962,7 +1965,6 @@ function makeClosureAction(context, target, action$$1, processArgs, debugKey) {
     self = context;
     fn$$1 = action$$1;
   } else {
-    // tslint:disable-next-line:max-line-length
     assert(`An action could not be made for \`${debugKey || action$$1}\` in ${target}. Please confirm that you are using either a quoted action name (i.e. \`(action '${debugKey || 'myAction'}')\`) or a function available in ${target}.`, false);
   }
 
@@ -2048,7 +2050,6 @@ const EMBER_VIEW_REF = createPrimitiveRef('ember-view');
 
 function aliasIdToElementId(args, props) {
   if (args.named.has('id')) {
-    // tslint:disable-next-line:max-line-length
     assert(`You cannot invoke a component with both 'id' and 'elementId' at the same time.`, !args.named.has('elementId'));
     props.elementId = props.id;
   }
@@ -3141,7 +3142,7 @@ const Component = CoreView.extend(ChildViewsSupport, ViewStateSupport, ClassName
 
     if (DEBUG && eventDispatcher && this.renderer._isInteractive && this.tagName === '') {
       let eventNames = [];
-      let events = eventDispatcher.finalEventNameMapping; // tslint:disable-next-line:forin
+      let events = eventDispatcher.finalEventNameMapping;
 
       for (let key in events) {
         let methodName = events[key];
@@ -3152,8 +3153,7 @@ const Component = CoreView.extend(ChildViewsSupport, ViewStateSupport, ClassName
       } // If in a tagless component, assert that no event handlers are defined
 
 
-      assert( // tslint:disable-next-line:max-line-length
-      `You can not define \`${eventNames}\` function(s) to handle DOM event in the \`${this}\` tagless component since it doesn't have any DOM element.`, !eventNames.length);
+      assert(`You can not define \`${eventNames}\` function(s) to handle DOM event in the \`${this}\` tagless component since it doesn't have any DOM element.`, !eventNames.length);
     }
   },
 
@@ -3546,7 +3546,7 @@ class ClassicHelperManager {
   }
 
   getDebugName(definition) {
-    return getDebugName(definition.class['prototype']);
+    return getDebugName((definition.class || definition)['prototype']);
   }
 
 }
@@ -4516,6 +4516,7 @@ const VM_DEPRECATION_OVERRIDES = [{
   until: '4.4.0',
   for: 'ember-source',
   since: {
+    available: '3.28.0',
     enabled: '3.28.0'
   }
 }];
@@ -5744,13 +5745,11 @@ class DynamicScope {
   }
 
   get(key) {
-    // tslint:disable-next-line:max-line-length
     assert(`Using \`-get-dynamic-scope\` is only supported for \`outletState\` (you used \`${key}\`).`, key === 'outletState');
     return this.outletState;
   }
 
   set(key, value) {
-    // tslint:disable-next-line:max-line-length
     assert(`Using \`-with-dynamic-scope\` is only supported for \`outletState\` (you used \`${key}\`).`, key === 'outletState');
     this.outletState = value;
     return value;
@@ -5773,6 +5772,7 @@ function errorLoopTransaction(fn$$1) {
           // Noop the function so that we won't keep calling it and causing
           // infinite looping failures;
           fn$$1 = () => {
+            // eslint-disable-next-line no-console
             console.warn('Attempted to rerender, but the Ember application has had an unrecoverable error occur during render. You should reload the application after fixing the cause of the error.');
           };
         }

package/dist/packages/@ember/-internals/meta/lib/meta.js

@@ -36,9 +36,16 @@ export const UNDEFINED = symbol('undefined');
 let currentListenerVersion = 1;
 export class Meta {
   // DEBUG
+
+  /** @internal */
   constructor(obj) {
+    /** @internal */
     this._listenersVersion = 1;
+    /** @internal */
+
     this._inheritedEnd = -1;
+    /** @internal */
+
     this._flattenedVersion = 0;
 
     if (DEBUG) {
@@ -59,6 +66,8 @@ export class Meta {
     this.proto = obj.constructor === undefined ? undefined : obj.constructor.prototype;
     this._listeners = undefined;
   }
+  /** @internal */
+
 
   get parent() {
     let parent = this._parent;
@@ -74,26 +83,38 @@ export class Meta {
   setInitializing() {
     this._isInit = true;
   }
+  /** @internal */
+
 
   unsetInitializing() {
     this._isInit = false;
   }
+  /** @internal */
+
 
   isInitializing() {
     return this._isInit;
   }
+  /** @internal */
+
 
   isPrototypeMeta(obj) {
     return this.proto === this.source && this.source === obj;
   }
+  /** @internal */
+
 
   _getOrCreateOwnMap(key) {
     return this[key] || (this[key] = Object.create(null));
   }
+  /** @internal */
+
 
   _getOrCreateOwnSet(key) {
     return this[key] || (this[key] = new Set());
   }
+  /** @internal */
+
 
   _findInheritedMap(key, subkey) {
     let pointer = this;
@@ -112,6 +133,8 @@ export class Meta {
       pointer = pointer.parent;
     }
   }
+  /** @internal */
+
 
   _hasInInheritedSet(key, value) {
     let pointer = this;
@@ -128,28 +151,38 @@ export class Meta {
 
     return false;
   }
+  /** @internal */
+
 
   valueFor(key) {
     let values = this._values;
     return values !== undefined ? values[key] : undefined;
   }
+  /** @internal */
+
 
   setValueFor(key, value) {
     let values = this._getOrCreateOwnMap('_values');
 
     values[key] = value;
   }
+  /** @internal */
+
 
   revisionFor(key) {
     let revisions = this._revisions;
     return revisions !== undefined ? revisions[key] : undefined;
   }
+  /** @internal */
+
 
   setRevisionFor(key, revision) {
     let revisions = this._getOrCreateOwnMap('_revisions');
 
     revisions[key] = revision;
   }
+  /** @internal */
+
 
   writableLazyChainsFor(key) {
     if (DEBUG) {
@@ -166,6 +199,8 @@ export class Meta {
 
     return chains;
   }
+  /** @internal */
+
 
   readableLazyChainsFor(key) {
     if (DEBUG) {
@@ -180,6 +215,8 @@ export class Meta {
 
     return undefined;
   }
+  /** @internal */
+
 
   addMixin(mixin) {
     assert(isDestroyed(this.source) ? `Cannot add mixins of \`${toString(mixin)}\` on \`${toString(this.source)}\` call addMixin after it has been destroyed.` : '', !isDestroyed(this.source));
@@ -188,10 +225,14 @@ export class Meta {
 
     set.add(mixin);
   }
+  /** @internal */
+
 
   hasMixin(mixin) {
     return this._hasInInheritedSet('_mixins', mixin);
   }
+  /** @internal */
+
 
   forEachMixins(fn) {
     let pointer = this;
@@ -214,22 +255,30 @@ export class Meta {
       pointer = pointer.parent;
     }
   }
+  /** @internal */
+
 
   writeDescriptors(subkey, value) {
     assert(isDestroyed(this.source) ? `Cannot update descriptors for \`${subkey}\` on \`${toString(this.source)}\` after it has been destroyed.` : '', !isDestroyed(this.source));
     let map = this._descriptors || (this._descriptors = new Map());
     map.set(subkey, value);
   }
+  /** @internal */
+
 
   peekDescriptors(subkey) {
     let possibleDesc = this._findInheritedMap('_descriptors', subkey);
 
     return possibleDesc === UNDEFINED ? undefined : possibleDesc;
   }
+  /** @internal */
+
 
   removeDescriptors(subkey) {
     this.writeDescriptors(subkey, UNDEFINED);
   }
+  /** @internal */
+
 
   forEachDescriptors(fn) {
     let pointer = this;
@@ -254,6 +303,8 @@ export class Meta {
       pointer = pointer.parent;
     }
   }
+  /** @internal */
+
 
   addToListeners(eventName, target, method, once, sync) {
     if (DEBUG) {
@@ -266,6 +317,8 @@ export class Meta {
     /* ADD */
     , sync);
   }
+  /** @internal */
+
 
   removeFromListeners(eventName, target, method) {
     if (DEBUG) {
@@ -311,8 +364,8 @@ export class Meta {
       && listener.kind !== 2
       /* REMOVE */
       ) {
-          listeners.splice(i, 1);
-        } else {
+        listeners.splice(i, 1);
+      } else {
         assert(`You attempted to add an observer for the same method on '${event.split(':')[0]}' twice to ${target} as both sync and async. Observers must be either sync or async, they cannot be both. This is likely a mistake, you should either remove the code that added the observer a second time, or update it to always be sync or async. The method was ${method}.`, !(listener.kind === 0
         /* ADD */
         && kind === 0
@@ -416,6 +469,8 @@ export class Meta {
 
     return this._listeners;
   }
+  /** @internal */
+
 
   matchingListeners(eventName) {
     let listeners = this.flattenedListeners();
@@ -450,6 +505,8 @@ export class Meta {
 
     return result;
   }
+  /** @internal */
+
 
   observerEvents() {
     let listeners = this.flattenedListeners();

package/dist/packages/@ember/-internals/metal/index.js

@@ -759,7 +759,6 @@ function getChainTags(chainTags, obj, path, tagMeta, meta$$1) {
           descriptor = currentMeta !== null ? currentMeta.peekDescriptors(segment) : undefined; // If the key is an alias, we need to bootstrap it
 
           if (descriptor !== undefined && typeof descriptor.altKey === 'string') {
-            // tslint:disable-next-line: no-unused-expression
             item[segment];
           }
         }
@@ -781,7 +780,6 @@ function getChainTags(chainTags, obj, path, tagMeta, meta$$1) {
       // bootstrap the alias. This is because aliases, unlike other CPs, should
       // always be in sync with the aliased value.
       if (CHAIN_PASS_THROUGH.has(descriptor)) {
-        // tslint:disable-next-line: no-unused-expression
         current[segment];
       }
 
@@ -1585,11 +1583,15 @@ class ComputedDecoratorImpl extends Function {
   } // TODO: Remove this when we can provide alternatives in the ecosystem to
   // addons such as ember-macro-helpers that use it.
 
+  /** @internal */
+
 
   get _getter() {
     return descriptorForDecorator(this)._getter;
   } // TODO: Refactor this, this is an internal API only
 
+  /** @internal */
+
 
   set enumerable(value) {
     descriptorForDecorator(this).enumerable = value;
@@ -1633,6 +1635,8 @@ function getCachedValueFor(obj, key) {
 
   if (meta$$1) {
     return meta$$1.valueFor(key);
+  } else {
+    return undefined;
   }
 }
 
@@ -1766,45 +1770,7 @@ if (DEBUG) {
       return Reflect.get(content, keyName, obj);
     }
   };
-} // ..........................................................
-// GET AND SET
-//
-// If we are on a platform that supports accessors we can use those.
-// Otherwise simulate accessors by looking up the property directly on the
-// object.
-
-/**
-  Gets the value of a property on an object. If the property is computed,
-  the function will be invoked. If the property is not defined but the
-  object implements the `unknownProperty` method then that will be invoked.
-
-  ```javascript
-  import { get } from '@ember/object';
-  get(obj, "name");
-  ```
-
-  If you plan to run on IE8 and older browsers then you should use this
-  method anytime you want to retrieve a property on an object that you don't
-  know for sure is private. (Properties beginning with an underscore '_'
-  are considered private.)
-
-  On all newer browsers, you only need to use this method to retrieve
-  properties if the property might not be defined on the object and you want
-  to respect the `unknownProperty` handler. Otherwise you can ignore this
-  method.
-
-  Note that if the object itself is `undefined`, this method will throw
-  an error.
-
-  @method get
-  @for @ember/object
-  @static
-  @param {Object} obj The object to retrieve from.
-  @param {String} keyName The property key to retrieve
-  @return {Object} the property value or `null`.
-  @public
-*/
-
+}
 
 function get(obj, keyName) {
   assert(`Get must be called with two arguments; an object and a property key`, arguments.length === 2);
@@ -2083,7 +2049,6 @@ class AliasedProperty extends ComputedDescriptor {
 }
 
 function AliasedProperty_readOnlySet(obj, keyName) {
-  // eslint-disable-line no-unused-vars
   throw new EmberError(`Cannot set read-only property '${keyName}' on object: ${inspect(obj)}`);
 }
 
@@ -2399,6 +2364,7 @@ if (DEBUG) {
   Libraries.prototype.logVersions = function () {
     let libs = this._registry;
     let nameLengths = libs.map(item => get(item, 'name.length'));
+    assert('nameLengths is number array', nameLengths instanceof Array && nameLengths.every(n => typeof n === 'number'));
     let maxNameLength = Math.max.apply(null, nameLengths);
     debug('-------------------------------');
 
@@ -2415,39 +2381,6 @@ if (DEBUG) {
 const LIBRARIES = new Libraries();
 LIBRARIES.registerCoreLibrary('Ember', VERSION);
 
-/**
- @module @ember/object
-*/
-
-/**
-  To get multiple properties at once, call `getProperties`
-  with an object followed by a list of strings or an array:
-
-  ```javascript
-  import { getProperties } from '@ember/object';
-
-  getProperties(record, 'firstName', 'lastName', 'zipCode');
-  // { firstName: 'John', lastName: 'Doe', zipCode: '10011' }
-  ```
-
-  is equivalent to:
-
-  ```javascript
-  import { getProperties } from '@ember/object';
-
-  getProperties(record, ['firstName', 'lastName', 'zipCode']);
-  // { firstName: 'John', lastName: 'Doe', zipCode: '10011' }
-  ```
-
-  @method getProperties
-  @static
-  @for @ember/object
-  @param {Object} obj
-  @param {String...|Array} list of keys to get
-  @return {Object}
-  @public
-*/
-
 function getProperties(obj, keys) {
   let ret = {};
   let propertyNames = arguments;
@@ -3076,6 +3009,7 @@ const MIXINS = new _WeakSet();
 */
 
 class Mixin {
+  /** @internal */
   constructor(mixins, properties) {
     MIXINS.add(this);
     this.properties = extractAccessors(properties);
@@ -3116,6 +3050,8 @@ class Mixin {
   } // returns the mixins currently applied to the specified object
   // TODO: Make `mixin`
 
+  /** @internal */
+
 
   static mixins(obj) {
     let meta$$1 = peekMeta(obj);
@@ -3137,6 +3073,7 @@ class Mixin {
     @method reopen
     @param arguments*
     @private
+    @internal
   */
 
 
@@ -3161,6 +3098,7 @@ class Mixin {
     @param obj
     @return applied object
     @private
+    @internal
   */
 
 
@@ -3172,6 +3110,8 @@ class Mixin {
     // _hideKeys disables enumerablity when applying the mixin. This is a hack, and we should stop mutating the array prototype by default 😫
     return applyMixin(obj, [this], _hideKeys);
   }
+  /** @internal */
+
 
   applyPartial(obj) {
     return applyMixin(obj, [this]);
@@ -3181,6 +3121,7 @@ class Mixin {
     @param obj
     @return {Boolean}
     @private
+    @internal
   */
 
 
@@ -3201,16 +3142,22 @@ class Mixin {
 
     return meta$$1.hasMixin(this);
   }
+  /** @internal */
+
 
   without(...args) {
     let ret = new Mixin([this]);
     ret._without = args;
     return ret;
   }
+  /** @internal */
+
 
   keys() {
     return _keys(this);
   }
+  /** @internal */
+
 
   toString() {
     return '(unknown mixin)';
@@ -3292,11 +3239,39 @@ function _keys(mixin, ret = new Set(), seen = new Set()) {
 
   return ret;
 }
+/**
+  Specify a method that observes property changes.
+
+  ```javascript
+  import EmberObject from '@ember/object';
+  import { observer } from '@ember/object';
+
+  export default EmberObject.extend({
+    valueObserver: observer('value', function() {
+      // Executes whenever the "value" property changes
+    })
+  });
+  ```
+
+  Also available as `Function.prototype.observes` if prototype extensions are
+  enabled.
+
+  @method observer
+  @for @ember/object
+  @param {String} propertyNames*
+  @param {Function} func
+  @return func
+  @public
+  @static
+*/
+
 
 function observer(...args) {
   let funcOrDef = args.pop();
   assert('observer must be provided a function or an observer definition', typeof funcOrDef === 'function' || typeof funcOrDef === 'object' && funcOrDef !== null);
-  let func, dependentKeys, sync;
+  let func;
+  let dependentKeys;
+  let sync;
 
   if (typeof funcOrDef === 'function') {
     func = funcOrDef;