ember-source 4.5.0-alpha.4 → 4.5.0-beta.2

package/dist/packages/@ember/-internals/runtime/lib/system/object_proxy.js

@@ -1,81 +1,7 @@
 import FrameworkObject from './object';
 import _ProxyMixin from '../mixins/-proxy';
-/**
-  `ObjectProxy` forwards all properties not defined by the proxy itself
-  to a proxied `content` object.
 
-  ```javascript
-  import EmberObject from '@ember/object';
-  import ObjectProxy from '@ember/object/proxy';
+class ObjectProxy extends FrameworkObject {}
 
-  let exampleObject = EmberObject.create({
-    name: 'Foo'
-  });
-
-  let exampleProxy = ObjectProxy.create({
-    content: exampleObject
-  });
-
-  // Access and change existing properties
-  exampleProxy.get('name');          // 'Foo'
-  exampleProxy.set('name', 'Bar');
-  exampleObject.get('name');         // 'Bar'
-
-  // Create new 'description' property on `exampleObject`
-  exampleProxy.set('description', 'Foo is a whizboo baz');
-  exampleObject.get('description');  // 'Foo is a whizboo baz'
-  ```
-
-  While `content` is unset, setting a property to be delegated will throw an
-  Error.
-
-  ```javascript
-  import ObjectProxy from '@ember/object/proxy';
-
-  let exampleProxy = ObjectProxy.create({
-    content: null,
-    flag: null
-  });
-  exampleProxy.set('flag', true);
-  exampleProxy.get('flag');         // true
-  exampleProxy.get('foo');          // undefined
-  exampleProxy.set('foo', 'data');  // throws Error
-  ```
-
-  Delegated properties can be bound to and will change when content is updated.
-
-  Computed properties on the proxy itself can depend on delegated properties.
-
-  ```javascript
-  import { computed } from '@ember/object';
-  import ObjectProxy from '@ember/object/proxy';
-
-  ProxyWithComputedProperty = ObjectProxy.extend({
-    fullName: computed('firstName', 'lastName', function() {
-      var firstName = this.get('firstName'),
-          lastName = this.get('lastName');
-      if (firstName && lastName) {
-        return firstName + ' ' + lastName;
-      }
-      return firstName || lastName;
-    })
-  });
-
-  let exampleProxy = ProxyWithComputedProperty.create();
-
-  exampleProxy.get('fullName');  // undefined
-  exampleProxy.set('content', {
-    firstName: 'Tom', lastName: 'Dale'
-  }); // triggers property change for fullName on proxy
-
-  exampleProxy.get('fullName');  // 'Tom Dale'
-  ```
-
-  @class ObjectProxy
-  @extends EmberObject
-  @uses Ember.ProxyMixin
-  @public
-*/
-
-export default class ObjectProxy extends FrameworkObject {}
-ObjectProxy.PrototypeMixin.reopen(_ProxyMixin);
+ObjectProxy.PrototypeMixin.reopen(_ProxyMixin);
+export default ObjectProxy;

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

@@ -509,7 +509,7 @@ function inspectObject(obj, depth, seen) {
     let key = keys[i];
     assert('has key', key); // Looping over array
 
-    s += `${inspectKey(key)}: ${inspectValue(obj[key], depth, seen)}`;
+    s += `${inspectKey(String(key))}: ${inspectValue(obj[key], depth, seen)}`;
   }
 
   s += ' }';

package/dist/packages/@ember/-internals/utils/types.js

@@ -0,0 +1 @@
+export {};

package/dist/packages/@ember/-internals/views/lib/compat/attrs.js

@@ -1,2 +1 @@
-import { symbol } from '@ember/-internals/utils';
-export let MUTABLE_CELL = symbol('MUTABLE_CELL');
+export const MUTABLE_CELL = Symbol('MUTABLE_CELL');

package/dist/packages/@ember/-internals/views/lib/component_lookup.js

@@ -1,8 +1,8 @@
 import { Object as EmberObject } from '@ember/-internals/runtime';
 export default EmberObject.extend({
-  componentFor(name, owner, options) {
+  componentFor(name, owner) {
     let fullName = `component:${name}`;
-    return owner.factoryFor(fullName, options);
+    return owner.factoryFor(fullName);
   },
 
   layoutFor(name, owner, options) {

package/dist/packages/@ember/-internals/views/lib/mixins/action_support.js

@@ -4,7 +4,7 @@
 import { inspect } from '@ember/-internals/utils';
 import { Mixin, get } from '@ember/-internals/metal';
 import { assert } from '@ember/debug';
-const mixinObj = {
+const ActionSupport = Mixin.create({
   send(actionName, ...args) {
     assert(`Attempted to call .send() with the action '${actionName}' on the destroyed object '${this}'.`, !this.isDestroying && !this.isDestroyed);
     let action = this.actions && this.actions[actionName];
@@ -27,11 +27,5 @@ const mixinObj = {
     }
   }
 
-};
-/**
- @class ActionSupport
- @namespace Ember
- @private
-*/
-
-export default Mixin.create(mixinObj);
+});
+export default ActionSupport;

package/dist/packages/@ember/-internals/views/lib/mixins/child_views_support.js

@@ -1,12 +1,9 @@
-/**
-@module ember
-*/
 import { Mixin, nativeDescDecorator } from '@ember/-internals/metal';
 import { getChildViews, addChildView } from '../system/utils';
-export default Mixin.create({
+const ChildViewsSupport = Mixin.create({
   /**
     Array of child views. You should never edit this array directly.
-     @property childViews
+       @property childViews
     @type Array
     @default []
     @private
@@ -25,4 +22,5 @@ export default Mixin.create({
     addChildView(this, view);
   }
 
-});
+});
+export default ChildViewsSupport;

package/dist/packages/@ember/-internals/views/lib/mixins/class_names_support.js

@@ -4,13 +4,7 @@
 import { descriptorForProperty, Mixin } from '@ember/-internals/metal';
 import { assert } from '@ember/debug';
 const EMPTY_ARRAY = Object.freeze([]);
-/**
-  @class ClassNamesSupport
-  @namespace Ember
-  @private
-*/
-
-export default Mixin.create({
+const ClassNamesSupport = Mixin.create({
   concatenatedProperties: ['classNames', 'classNameBindings'],
 
   init() {
@@ -24,7 +18,7 @@ export default Mixin.create({
     Standard CSS class names to apply to the view's outer element. This
     property automatically inherits any class names defined by the view's
     superclasses as well.
-     @property classNames
+       @property classNames
     @type Array
     @default ['ember-view']
     @public
@@ -35,7 +29,7 @@ export default Mixin.create({
     A list of properties of the view to apply as class names. If the property
     is a string value, the value of that string will be applied as a class
     name.
-     ```javascript
+       ```javascript
     // Applies the 'high' class to the view element
     import Component from '@ember/component';
     Component.extend({
@@ -43,9 +37,9 @@ export default Mixin.create({
       priority: 'high'
     });
     ```
-     If the value of the property is a Boolean, the name of that property is
+       If the value of the property is a Boolean, the name of that property is
     added as a dasherized class name.
-     ```javascript
+       ```javascript
     // Applies the 'is-urgent' class to the view element
     import Component from '@ember/component';
     Component.extend({
@@ -53,9 +47,9 @@ export default Mixin.create({
       isUrgent: true
     });
     ```
-     If you would prefer to use a custom value instead of the dasherized
+       If you would prefer to use a custom value instead of the dasherized
     property name, you can pass a binding like this:
-     ```javascript
+       ```javascript
     // Applies the 'urgent' class to the view element
     import Component from '@ember/component';
     Component.extend({
@@ -63,9 +57,9 @@ export default Mixin.create({
       isUrgent: true
     });
     ```
-     If you would like to specify a class that should only be added when the
+       If you would like to specify a class that should only be added when the
     property is false, you can declare a binding like this:
-     ```javascript
+       ```javascript
     // Applies the 'disabled' class to the view element
     import Component from '@ember/component';
     Component.extend({
@@ -73,11 +67,12 @@ export default Mixin.create({
       isEnabled: false
     });
     ```
-     This list of properties is inherited from the component's superclasses as well.
-     @property classNameBindings
+       This list of properties is inherited from the component's superclasses as well.
+       @property classNameBindings
     @type Array
     @default []
     @public
   */
   classNameBindings: EMPTY_ARRAY
-});
+});
+export default ClassNamesSupport;

package/dist/packages/@ember/-internals/views/lib/mixins/view_state_support.js

@@ -2,7 +2,7 @@
 @module ember
 */
 import { Mixin } from '@ember/-internals/metal';
-export default Mixin.create({
+const ViewStateSupport = Mixin.create({
   _transitionTo(state) {
     let priorState = this._currentState;
     let currentState = this._currentState = this._states[state];
@@ -17,4 +17,5 @@ export default Mixin.create({
     }
   }
 
-});
+});
+export default ViewStateSupport;

package/dist/packages/@ember/-internals/views/lib/mixins/view_support.js

@@ -8,43 +8,43 @@ function K() {
   return this;
 }
 
-let mixin = {
+const ViewMixin = Mixin.create({
   /**
    A list of properties of the view to apply as attributes. If the property
    is a string value, the value of that string will be applied as the value
    for an attribute of the property's name.
-    The following example creates a tag like `<div priority="high" />`.
-    ```app/components/my-component.js
+      The following example creates a tag like `<div priority="high" />`.
+      ```app/components/my-component.js
    import Component from '@ember/component';
-    export default Component.extend({
+      export default Component.extend({
       attributeBindings: ['priority'],
       priority: 'high'
     });
    ```
-    If the value of the property is a Boolean, the attribute is treated as
+      If the value of the property is a Boolean, the attribute is treated as
    an HTML Boolean attribute. It will be present if the property is `true`
    and omitted if the property is `false`.
-    The following example creates markup like `<div visible />`.
-    ```app/components/my-component.js
+      The following example creates markup like `<div visible />`.
+      ```app/components/my-component.js
    import Component from '@ember/component';
-    export default Component.extend({
+      export default Component.extend({
       attributeBindings: ['visible'],
       visible: true
     });
    ```
-    If you would prefer to use a custom value instead of the property name,
+      If you would prefer to use a custom value instead of the property name,
    you can create the same markup as the last example with a binding like
    this:
-    ```app/components/my-component.js
+      ```app/components/my-component.js
    import Component from '@ember/component';
-    export default Component.extend({
+      export default Component.extend({
       attributeBindings: ['isVisible:visible'],
       isVisible: true
     });
    ```
-    This list of attributes is inherited from the component's superclasses,
+      This list of attributes is inherited from the component's superclasses,
    as well.
-    @property attributeBindings
+      @property attributeBindings
    @type Array
    @default []
    @public
@@ -58,7 +58,7 @@ let mixin = {
   /**
    Return the nearest ancestor that is an instance of the provided
    class or mixin.
-    @method nearestOfType
+      @method nearestOfType
    @param {Class,Mixin} klass Subclass of Ember.View (or Ember.View itself),
    or an instance of Mixin.
    @return Ember.View
@@ -76,11 +76,13 @@ let mixin = {
 
       view = view.parentView;
     }
+
+    return;
   },
 
   /**
    Return the nearest ancestor that has a given property.
-    @method nearestWithProperty
+      @method nearestWithProperty
    @param {String} property A property name
    @return Ember.View
    @deprecated use `yield` and contextual components for composition instead.
@@ -103,13 +105,13 @@ let mixin = {
    view is already in the DOM or not. If the view is in the DOM, the
    rendering process will be deferred to give bindings a chance
    to synchronize.
-    If children were added during the rendering process using `appendChild`,
+      If children were added during the rendering process using `appendChild`,
    `rerender` will remove them, because they will be added again
    if needed by the next `render`.
-    In general, if the display of your view changes, you should modify
+      In general, if the display of your view changes, you should modify
    the DOM element directly instead of manually calling `rerender`, which can
    be slow.
-    @method rerender
+      @method rerender
    @public
    */
   rerender() {
@@ -122,7 +124,7 @@ let mixin = {
 
   /**
    Returns the current DOM element for the view.
-     @property element
+       @property element
     @type DOMElement
     @public
   */
@@ -138,14 +140,14 @@ let mixin = {
 
   /**
    Appends the view's element to the specified parent element.
-    Note that this method just schedules the view to be appended; the DOM
+      Note that this method just schedules the view to be appended; the DOM
    element will not be appended to the given element until all bindings have
    finished synchronizing.
-    This is not typically a function that you will need to call directly when
+      This is not typically a function that you will need to call directly when
    building your application. If you do need to use `appendTo`, be sure that
    the target element you are providing is associated with an `Application`
    and does not have an ancestor element that is associated with an Ember view.
-    @method appendTo
+      @method appendTo
    @param {String|DOMElement} A selector, element, HTML string
    @return {Ember.View} receiver
    @private
@@ -154,14 +156,15 @@ let mixin = {
     let target;
 
     if (hasDOM) {
+      assert(`Expected a selector or instance of Element`, typeof selector === 'string' || selector instanceof Element);
       target = typeof selector === 'string' ? document.querySelector(selector) : selector;
       assert(`You tried to append to (${selector}) but that isn't in the DOM`, target);
       assert('You cannot append to an existing Ember.View.', !matches(target, '.ember-view'));
       assert('You cannot append to an existing Ember.View.', (() => {
         let node = target.parentNode;
 
-        while (node) {
-          if (node.nodeType !== 9 && matches(node, '.ember-view')) {
+        while (node instanceof Element) {
+          if (matches(node, '.ember-view')) {
             return false;
           }
 
@@ -173,8 +176,10 @@ let mixin = {
     } else {
       target = selector;
       assert(`You tried to append to a selector string (${selector}) in an environment without a DOM`, typeof target !== 'string');
-      assert(`You tried to append to a non-Element (${selector}) in an environment without a DOM`, typeof selector.appendChild === 'function');
-    }
+      assert(`You tried to append to a non-Element (${selector}) in an environment without a DOM`, typeof target.appendChild === 'function');
+    } // SAFETY: SimpleElement is supposed to be a subset of Element so this _should_ be safe.
+    // However, the types are more specific in some places which necessitates the `as`.
+
 
     this.renderer.appendTo(this, target);
     return this;
@@ -184,13 +189,13 @@ let mixin = {
    Appends the view's element to the document body. If the view does
    not have an HTML representation yet
    the element will be generated automatically.
-    If your application uses the `rootElement` property, you must append
+      If your application uses the `rootElement` property, you must append
    the view within that element. Rendering views outside of the `rootElement`
    is not supported.
-    Note that this method just schedules the view to be appended; the DOM
+      Note that this method just schedules the view to be appended; the DOM
    element will not be appended to the document body until all bindings have
    finished synchronizing.
-    @method append
+      @method append
    @return {Ember.View} receiver
    @private
    */
@@ -201,17 +206,17 @@ let mixin = {
   /**
    The HTML `id` of the view's element in the DOM. You can provide this
    value yourself but it must be unique (just as in HTML):
-    ```handlebars
+      ```handlebars
    {{my-component elementId="a-really-cool-id"}}
    ```
-    If not manually set a default value will be provided by the framework.
-    Once rendered an element's `elementId` is considered immutable and you
+      If not manually set a default value will be provided by the framework.
+      Once rendered an element's `elementId` is considered immutable and you
    should never change it. If you need to compute a dynamic value for the
    `elementId`, you should do this when the component or element is being
    instantiated:
-    ```app/components/my-component.js
+      ```app/components/my-component.js
    import Component from '@ember/component';
-    export default Component.extend({
+      export default Component.extend({
       init() {
         this._super(...arguments);
         let index = this.get('index');
@@ -219,7 +224,7 @@ let mixin = {
       }
     });
    ```
-    @property elementId
+      @property elementId
    @type String
    @public
    */
@@ -227,7 +232,7 @@ let mixin = {
 
   /**
    Called when a view is going to insert an element into the DOM.
-    @event willInsertElement
+      @event willInsertElement
    @public
    */
   willInsertElement: K,
@@ -236,9 +241,9 @@ let mixin = {
    Called when the element of the view has been inserted into the DOM.
    Override this function to do any set up that requires an element
    in the document body.
-    When a view has children, didInsertElement will be called on the
+      When a view has children, didInsertElement will be called on the
    child view(s) first and on itself afterwards.
-    @event didInsertElement
+      @event didInsertElement
    @public
    */
   didInsertElement: K,
@@ -247,7 +252,7 @@ let mixin = {
    Called when the view is about to rerender, but before anything has
    been torn down. This is a good opportunity to tear down any manual
    observers you have installed based on the DOM state
-    @event willClearRender
+      @event willClearRender
    @public
    */
   willClearRender: K,
@@ -257,7 +262,7 @@ let mixin = {
    child views). This will remove the view from any parent node, then make
    sure that the DOM element managed by the view can be released by the
    memory manager.
-    @method destroy
+      @method destroy
    @private
    */
   destroy() {
@@ -270,23 +275,23 @@ let mixin = {
    Called when the element of the view is going to be destroyed. Override
    this function to do any teardown that requires an element, like removing
    event listeners.
-    Please note: any property changes made during this event will have no
+      Please note: any property changes made during this event will have no
    effect on object observers.
-    @event willDestroyElement
+      @event willDestroyElement
    @public
    */
   willDestroyElement: K,
 
   /**
    Called after the element of the view is destroyed.
-    @event willDestroyElement
+      @event willDestroyElement
    @public
    */
   didDestroyElement: K,
 
   /**
    Called when the parentView property has changed.
-    @event parentViewDidChange
+      @event parentViewDidChange
    @private
    */
   parentViewDidChange: K,
@@ -298,13 +303,13 @@ let mixin = {
    Tag name for the view's outer element. The tag name is only used when an
    element is first created. If you change the `tagName` for an element, you
    must destroy and recreate the view element.
-    By default, the render buffer will use a `<div>` tag for views.
-    If the tagName is `''`, the view will be tagless, with no outer element.
+      By default, the render buffer will use a `<div>` tag for views.
+      If the tagName is `''`, the view will be tagless, with no outer element.
    Component properties that depend on the presence of an outer element, such
    as `classNameBindings` and `attributeBindings`, do not work with tagless
    components. Tagless components cannot implement methods to handle events,
    and their `element` property has a `null` value.
-    @property tagName
+      @property tagName
    @type String
    @default null
    @public
@@ -319,10 +324,10 @@ let mixin = {
 
   /**
    Setup a view, but do not finish waking it up.
-    * configure `childViews`
+      * configure `childViews`
    * register the view with the global views hash, which is used for event
    dispatch
-    @method init
+      @method init
    @private
    */
   init() {
@@ -344,7 +349,7 @@ let mixin = {
 
   /**
    Handle events from `EventDispatcher`
-    @method handleEvent
+      @method handleEvent
    @param eventName {String}
    @param evt {Event}
    @private
@@ -353,11 +358,5 @@ let mixin = {
     return this._currentState.handleEvent(this, eventName, evt);
   }
 
-};
-/**
- @class ViewMixin
- @namespace Ember
- @private
-*/
-
-export default Mixin.create(mixin);
+});
+export default ViewMixin;