neo.mjs 2.3.17 → 3.0.1

package/src/component/Gallery.mjs

@@ -167,11 +167,10 @@ class Gallery extends Component {
     }}
 
     /**
-     *
      * @param {Object} config
      */
-    constructor(config) {
-        super(config);
+    construct(config) {
+        super.construct(config);
 
         let me           = this,
             domListeners = Neo.clone(me.domListeners, true),
@@ -365,7 +364,6 @@ class Gallery extends Component {
     }
 
     /**
-     *
      * @returns {*}
      */
     beforeGetItemTpl() {
@@ -457,7 +455,6 @@ class Gallery extends Component {
     }
 
     /**
-     *
      * @param {Number} [startIndex]
      * @param {Number} [amountItems]
      */
@@ -476,7 +473,6 @@ class Gallery extends Component {
     }
 
     /**
-     *
      * @param {Number} index
      * @returns {Number[]}
      */
@@ -504,7 +500,6 @@ class Gallery extends Component {
     }
 
     /**
-     *
      * @param {String} vnodeId
      * @returns {Number} itemId
      */
@@ -521,7 +516,6 @@ class Gallery extends Component {
     }
 
     /**
-     *
      * @param {Number} index
      * @returns {String}
      */
@@ -548,7 +542,6 @@ class Gallery extends Component {
     }
 
     /**
-     *
      * @param {String} id
      * @returns {String}
      */
@@ -569,7 +562,6 @@ class Gallery extends Component {
     }
 
     /**
-     *
      * @param {Object} data
      */
     onClick(data) {
@@ -607,7 +599,6 @@ class Gallery extends Component {
     }
 
     /**
-     *
      * @param {Object} data
      */
     onMouseWheel(data) {
@@ -629,7 +620,6 @@ class Gallery extends Component {
     }
 
     /**
-     *
      * @param {Array} value
      */
     onSelectionChange(value) {
@@ -752,7 +742,6 @@ class Gallery extends Component {
     }
 
     /**
-     *
      * @param {Array} items
      */
     onStoreLoad(items) {
@@ -788,7 +777,6 @@ class Gallery extends Component {
     }
 
     /**
-     *
      * @param {Number} x
      * @param {Number} y
      * @param {Number} z

package/src/component/Helix.mjs

@@ -248,11 +248,10 @@ class Helix extends Component {
     }}
 
     /**
-     *
      * @param {Object} config
      */
-    constructor(config) {
-        super(config);
+    construct(config) {
+        super.construct(config);
 
         let me           = this,
             domListeners = Neo.clone(me.domListeners, true);
@@ -359,7 +358,6 @@ class Helix extends Component {
     }
 
     /**
-     *
      * @param callback
      * @param animationTime
      * @param callbackParam
@@ -404,7 +402,6 @@ class Helix extends Component {
     }
 
     /**
-     *
      * @returns {Object}
      */
     beforeGetItemTpl() {
@@ -638,7 +635,6 @@ class Helix extends Component {
     }
 
     /**
-     *
      * @param {Number} [startIndex]
      * @param {Number} [amountItems]
      */
@@ -705,7 +701,6 @@ class Helix extends Component {
     }
 
     /**
-     *
      * @returns {String}
      */
     getCloneTransform() {
@@ -718,7 +713,6 @@ class Helix extends Component {
     }
 
     /**
-     *
      * @param {String} vnodeId
      * @returns {Number}
      */
@@ -735,7 +729,6 @@ class Helix extends Component {
     }
 
     /**
-     *
      * @param {Number|String} id
      * @returns {String}
      */
@@ -783,7 +776,6 @@ class Helix extends Component {
     }
 
     /**
-     *
      * @param {String} itemId
      */
     moveToSelectedItem(itemId) {
@@ -792,7 +784,6 @@ class Helix extends Component {
     }
 
     /**
-     *
      * @param {Object} data
      */
     onClick(data) {
@@ -816,7 +807,6 @@ class Helix extends Component {
     }
 
     /**
-     *
      * @param {Object} data
      */
     onKeyDownEnter(data) {
@@ -825,7 +815,6 @@ class Helix extends Component {
     }
 
     /**
-     *
      * @param {Object} data
      */
     onKeyDownSpace(data) {
@@ -833,7 +822,6 @@ class Helix extends Component {
     }
 
     /**
-     *
      * @param {Object} data
      */
     onMouseWheel(data) {
@@ -855,7 +843,6 @@ class Helix extends Component {
     }
 
     /**
-     *
      * @param {String[]} value
      * @param {String[]} oldValue
      */
@@ -879,7 +866,6 @@ class Helix extends Component {
     }
 
     /**
-     *
      * @param {Array} items
      */
     onStoreLoad(items) {

package/src/component/Splitter.mjs

@@ -69,11 +69,10 @@ class Splitter extends Component {
     }}
 
     /**
-     *
      * @param {Object} config
      */
-    constructor(config) {
-        super(config);
+    construct(config) {
+        super.construct(config);
 
         let me           = this,
             domListeners = me.domListeners;
@@ -142,7 +141,6 @@ class Splitter extends Component {
     }
 
     /**
-     *
      * @param data
      */
     onDragEnd(data) {
@@ -160,9 +158,7 @@ class Splitter extends Component {
 
         me.style = style;
 
-        Neo.main.DomAccess.getBoundingClientRect({
-            id: me.parentId
-        }).then(parentRect => {
+        me.getDomRect(me.parentId).then(parentRect => {
             parent  = Neo.getComponent(me.parentId);
             index   = parent.indexOf(me);
             sibling = parent.items[resizeNext ? index + 1 :index - 1];
@@ -201,7 +197,6 @@ class Splitter extends Component {
     }
 
     /**
-     *
      * @param data
      */
     onDragStart(data) {

package/src/component/mwc/Button.mjs

@@ -58,8 +58,8 @@ class Button extends Component {
     /**
      * @param {Object} config
      */
-    constructor(config) {
-        super(config);
+    construct(config) {
+        super.construct(config);
 
         if (!Neo.config.mainThreadAddons.includes('Mwc')) {
             console.error('Please include "Mwc" into the mainThreadAddons of your neo-config.json file.');

package/src/component/mwc/TextField.mjs

@@ -65,8 +65,8 @@ class TextField extends Component {
     /**
      * @param {Object} config
      */
-    constructor(config) {
-        super(config);
+    construct(config) {
+        super.construct(config);
 
         if (!Neo.config.mainThreadAddons.includes('Mwc')) {
             console.error('Please include "Mwc" into the mainThreadAddons of your neo-config.json file.');
@@ -206,7 +206,6 @@ class TextField extends Component {
     }
 
     /**
-     *
      * @param {Object} data
      * @param {Neo.component.mwc.TextField} data.component
      * @param {Object[]} data.path

package/src/component/wrapper/AmChart.mjs

@@ -1,5 +1,5 @@
 import Component from '../Base.mjs';
-import Logger    from '../../core/Logger.mjs';
+import Logger    from '../../util/Logger.mjs';
 
 /**
  * Convenience class to render an amChart

package/src/component/wrapper/MapboxGL.mjs

@@ -278,7 +278,6 @@ class MapboxGL extends Component {
     }
 
     /**
-     *
      * @param {Object[]} data
      * @returns {Object} Object matching the geojson format
      */
@@ -313,7 +312,6 @@ class MapboxGL extends Component {
     }
 
     /**
-     *
      * @param {Object} value
      * @param {Number} value.lat
      * @param {Number} value.lng
@@ -350,7 +348,6 @@ class MapboxGL extends Component {
     }
 
     /**
-     *
      * @param {Object} data
      * @param {String} data.layerId
      * @param {Object} data.options
@@ -368,7 +365,6 @@ class MapboxGL extends Component {
     }
 
     /**
-     *
      * @param {Object} data
      * @param {String} data.layerId
      * @param {String} data.key
@@ -388,7 +384,6 @@ class MapboxGL extends Component {
     }
 
     /**
-     *
      * @param {Object} data
      * @param {String} data.layerId
      * @param {String} data.key

package/src/container/Base.mjs

@@ -4,7 +4,7 @@ import LayoutCard from '../layout/Card.mjs';
 import LayoutFit  from '../layout/Fit.mjs';
 import LayoutHbox from '../layout/HBox.mjs';
 import LayoutVBox from '../layout/VBox.mjs';
-import Logger     from '../core/Logger.mjs';
+import Logger     from '../util/Logger.mjs';
 import NeoArray   from '../util/Array.mjs';
 
 /**
@@ -109,7 +109,6 @@ class Base extends Component {
     }
 
     /**
-     *
      * @param {Neo.layout.Base} value
      * @param {Neo.layout.Base} oldValue
      * @protected
@@ -171,7 +170,6 @@ class Base extends Component {
     }
 
     /**
-     *
      * @param {Object|String} value
      * @returns {Neo.layout.Base}
      * @protected
@@ -181,7 +179,6 @@ class Base extends Component {
     }
 
     /**
-     *
      * @param {*} item
      * @param {Number} index
      * @returns {Neo.component.Base|Object} Object for lazy loaded items
@@ -254,7 +251,6 @@ class Base extends Component {
     }
 
     /**
-     *
      * @protected
      */
     createItems() {
@@ -280,7 +276,6 @@ class Base extends Component {
     }
 
     /**
-     *
      * @param {Object|String|Neo.layout.Base} value
      * @protected
      * @returns {Neo.layout.Base}
@@ -456,7 +451,6 @@ class Base extends Component {
     }
 
     /**
-     *
      * @param {Object|String} config
      * @protected
      * @returns {Object} layoutConfig

package/src/container/Panel.mjs

@@ -48,11 +48,10 @@ class Panel extends Container {
     }}
 
     /**
-     *
      * @param {Object} config
      */
-    constructor(config) {
-        super(config);
+    construct(config) {
+        super.construct(config);
 
         let me = this;
 
@@ -65,7 +64,6 @@ class Panel extends Container {
     }
 
     /**
-     *
      * @param {Object} header the header config
      * @returns {Object}
      */
@@ -178,7 +176,6 @@ class Panel extends Container {
     }
 
     /**
-     *
      * @returns {Boolean}
      */
     hasHeaders() {

package/src/controller/Application.mjs

@@ -46,16 +46,15 @@ class Application extends Base {
     }}
 
     /**
-     *
      * @param {Object} config
      */
-    constructor(config) {
+    construct(config) {
         // to guarantee that the main view can access Neo.apps at any point,
         // we need to trigger its assignment at the end of the ctor.
         let mainView = config.mainView;
         delete config.mainView;
 
-        super(config);
+        super.construct(config);
 
         let me = this;
 

package/src/controller/Base.mjs

@@ -28,11 +28,10 @@ class Base extends CoreBase {
     }}
 
     /**
-     *
      * @param {Object} config
      */
-    constructor(config) {
-        super(config);
+    construct(config) {
+        super.construct(config);
 
         HashHistory.on('change', this.onHashChange, this);
     }

package/src/controller/Component.mjs

@@ -1,7 +1,7 @@
 import Base             from './Base.mjs';
 import ComponentManager from '../manager/Component.mjs';
 import DomEventManager  from '../manager/DomEvent.mjs';
-import Logger           from '../core/Logger.mjs';
+import Logger           from '../util/Logger.mjs';
 
 /**
  * @class Neo.controller.Component
@@ -36,11 +36,10 @@ class Component extends Base {
     }}
 
     /**
-     *
      * @param {Object} config
      */
-    constructor(config) {
-        super(config);
+    construct(config) {
+        super.construct(config);
 
         let me        = this,
             component = me.component,
@@ -69,7 +68,6 @@ class Component extends Base {
     }
 
     /**
-     *
      * @param {String} handlerName
      * @returns {Neo.controller.Component|null}
      */
@@ -137,7 +135,6 @@ class Component extends Base {
     onComponentConstructed() {}
 
     /**
-     *
      * @param {Neo.component.Base} [component=this.component]
      */
     parseConfig(component=this.component) {

package/src/core/Base.mjs

@@ -74,7 +74,7 @@ class Base {
      * Applies the observable mixin if needed, grants remote access if needed.
      * @param {Object} config={}
      */
-    constructor(config={}) {
+    construct(config={}) {
         let me = this;
 
         Object.defineProperties(me, {
@@ -99,6 +99,9 @@ class Base {
 
         me.getStaticConfig('observable') && me.initObservable(config);
 
+        // assign class field values prior to configs
+        config = me.setFields(config);
+
         me.initConfig(config);
 
         Object.defineProperty(me, 'configsApplied', {
@@ -148,7 +151,7 @@ class Base {
         let values = Array.isArray(staticName) ? staticName : this.getStaticConfig(staticName);
 
         if (!values.includes(value)) {
-            Neo.logError(`Supported values for ${name} are: ${values.join(', ')}`, this);
+            console.error(`Supported values for ${name} are: ${values.join(', ')}`, this);
             return oldValue;
         }
 
@@ -350,7 +353,6 @@ class Base {
     }
 
     /**
-     *
      * @param {String} className
      * @param {Object} remote
      */
@@ -377,6 +379,8 @@ class Base {
     set(values={}) {
         let me = this;
 
+        values = me.setFields(values);
+
         // If the initial config processing is still running,
         // finish this one first before dropping new values into the configSymbol.
         // see: https://github.com/neomjs/neo/issues/2201
@@ -389,6 +393,25 @@ class Base {
         me.processConfigs(true);
     }
 
+    /**
+     * We want to assign class fields first and remove them from the config object,
+     * so that afterSet(), beforeGet() and beforeSet() methods can get the new values right away
+     * @param {Object} config
+     * @returns {Object}
+     */
+    setFields(config) {
+        let configNames = this.constructor.config;
+
+        Object.entries(config).forEach(([key, value]) => {
+            if (!configNames.hasOwnProperty(key) && !Neo.hasPropertySetter(this, key)) {
+                this[key] = value;
+                delete config[key];
+            }
+        });
+
+        return config;
+    }
+
     /**
      * Sets the value of a static config by a given key
      * @param {String} key The key of a staticConfig defined inside static getStaticConfig

package/src/core/Compare.mjs

@@ -14,7 +14,6 @@ class Compare extends Base {
     }}
 
     /**
-     *
      * @param {*} item1
      * @param {*} item2
      * @returns {Boolean}

package/src/core/IdGenerator.mjs

@@ -40,10 +40,9 @@ class IdGenerator {
     }}
 
     /**
-     *
      * @param config
      */
-    constructor(config) {
+    construct(config) {
         let me = this;
 
         me.idCounter = {};
@@ -53,7 +52,6 @@ class IdGenerator {
     }
 
     /**
-     *
      * @param name
      * @returns {string}
      */
@@ -82,4 +80,4 @@ let instance = Neo.create(IdGenerator);
 
 Neo.applyToGlobalNs(instance);
 
-export default instance;
+export default instance;

package/src/core/Observable.mjs

@@ -24,7 +24,6 @@ class Observable extends Base {
     }}
 
     /**
-     *
      * @param {Object|String} name
      * @param {Object} [opts]
      * @param {Object} [scope]
@@ -93,7 +92,6 @@ class Observable extends Base {
     }
 
     /**
-     *
      * @param name
      */
     fire(name) {
@@ -166,7 +164,6 @@ class Observable extends Base {
     }
 
     /**
-     *
      * @param {Object|String} name
      * @param {String} [eventId]
      */

package/src/core/Util.mjs

@@ -29,7 +29,6 @@ class Util extends Base {
     }}
 
     /**
-     *
      * @param {Object} scope
      * @param {String[]} values
      */

package/src/core/_export.mjs

@@ -1,8 +1,7 @@
 import Base        from './Base.mjs';
 import Compare     from './Compare.mjs';
 import IdGenerator from './IdGenerator.mjs';
-import Logger      from './Logger.mjs';
 import Observable  from './Observable.mjs';
 import Util        from './Util.mjs';
 
-export {Base, Compare, IdGenerator, Logger, Observable, Util};
+export {Base, Compare, IdGenerator, Observable, Util};

package/src/data/RecordFactory.mjs

@@ -1,5 +1,5 @@
 import Base   from '../core/Base.mjs';
-import Logger from '../core/Logger.mjs';
+import Logger from '../util/Logger.mjs';
 import Model  from './Model.mjs';
 
 let instance;
@@ -34,7 +34,6 @@ class RecordFactory extends Base {
     }}
 
     /**
-     *
      * @param {Neo.data.Model} model
      * @param {Object} config
      * @returns {Object}
@@ -50,7 +49,6 @@ class RecordFactory extends Base {
     }
 
     /**
-     *
      * @param {Neo.data.Model} model
      * @returns {Object}
      */
@@ -156,7 +154,6 @@ class RecordFactory extends Base {
     }
 
     /**
-     *
      * @param {Object} record
      * @param {Boolean} trackModifiedFields
      * @returns {Boolean} true in case a change was found
@@ -183,7 +180,6 @@ class RecordFactory extends Base {
     }
 
     /**
-     *
      * @param {Object} record
      * @param {String} fieldName
      * @returns {Boolean|null} null in case the model does not use trackModifiedFields, true in case a change was found
@@ -240,7 +236,6 @@ class RecordFactory extends Base {
     }
 
     /**
-     *
      * @param {Neo.data.Model} model
      * @param {Object} record
      * @param {Object} fields