neo.mjs 8.43.1 → 9.0.0

package/src/table/Container.mjs

@@ -232,6 +232,29 @@ class Container extends BaseContainer {
         }
     }
 
+    /**
+     * Triggered after the store config got changed
+     * @param {Number} value
+     * @param {Number} oldValue
+     * @protected
+     */
+    afterSetStore(value, oldValue) {
+        let me        = this,
+            listeners = {
+                filter: me.onStoreFilter,
+                load  : me.onStoreLoad,
+                scope : me
+            };
+
+        value   ?.on(listeners);
+        oldValue?.un(listeners);
+
+        // in case we dynamically change the store, the view needs to get the new reference
+        if (me.view) {
+            me.view.store = value
+        }
+    }
+
     /**
      * Triggered after the useCustomScrollbars config got changed
      * @param {Boolean} value
@@ -293,35 +316,13 @@ class Container extends BaseContainer {
 
     /**
      * Triggered before the store config gets changed.
-     * @param {Neo.data.Store} value
-     * @param {Neo.data.Store} oldValue
+     * @param {Object|Neo.data.Store|null} value
+     * @param {Neo.data.Store}             oldValue
      * @protected
      */
     beforeSetStore(value, oldValue) {
-        oldValue?.destroy();
-
         if (value) {
-            let me = this,
-
-            listeners = {
-                filter      : me.onStoreFilter,
-                load        : me.onStoreLoad,
-                recordChange: me.onStoreRecordChange,
-                scope       : me
-            };
-
-            if (value instanceof Store) {
-                value.on(listeners)
-            } else {
-                value = ClassSystemUtil.beforeSetInstance(value, Store, {
-                    listeners
-                })
-            }
-
-            // in case we dynamically change the store, the view needs to get the new reference
-            if (me.view) {
-                me.view.store = value
-            }
+            value = ClassSystemUtil.beforeSetInstance(value, Store)
         }
 
         return value
@@ -404,19 +405,6 @@ class Container extends BaseContainer {
         return columns
     }
 
-    /**
-     *
-     */
-    createViewData() {
-        let me = this;
-
-        me.view.createViewData();
-
-        if (me.useCustomScrollbars && me.scrollbarsCssApplied === false) {
-            me.applyCustomScrollbarsCss()
-        }
-    }
-
     /**
      * @override
      * @returns {*}
@@ -458,50 +446,29 @@ class Container extends BaseContainer {
 
         me.store.sort(opts);
         me.removeSortingCss(opts.property);
-        me.onStoreLoad(me.store.items)
+        opts.direction && me.view.onStoreLoad()
     }
 
     /**
      *
      */
     onStoreFilter() {
-        this.onStoreLoad(this.store.items)
+        this.onStoreLoad()
     }
 
     /**
-     * @param {Object[]} data
      * @protected
      */
-    onStoreLoad(data) {
+    onStoreLoad() {
         let me = this;
 
-        if (me.rendered) {
-            me.createViewData();
-
-            if (me.store.sorters.length < 1) {
-                me.removeSortingCss()
-            }
-        } else {
-            me.on('rendered', () => {
-                me.timeout(50).then(() => {
-                    me.createViewData()
-                })
-            }, me, {once: true})
+        if (me.useCustomScrollbars && me.scrollbarsCssApplied === false) {
+            me.applyCustomScrollbarsCss()
         }
-    }
 
-    /**
-     * Gets triggered after changing the value of a record field.
-     * E.g. myRecord.foo = 'bar';
-     * @param {Object} opts
-     * @param {String} opts.field The name of the field which got changed
-     * @param {Neo.data.Model} opts.model The model instance of the changed record
-     * @param {*} opts.oldValue
-     * @param {Object} opts.record
-     * @param {*} opts.value
-     */
-    onStoreRecordChange(opts) {
-        this.view.onStoreRecordChange(opts)
+        if (me.store.sorters?.length < 1) {
+            me.removeSortingCss()
+        }
     }
 
     /**

package/src/table/View.mjs

@@ -56,9 +56,9 @@ class View extends Component {
          */
         selectedRecordField: 'annotations.selected',
         /**
-         * @member {Neo.data.Store|null} store=null
+         * @member {Neo.data.Store|null} store_=null
          */
-        store: null,
+        store_: null,
         /**
          * @member {Boolean} useRowRecordIds=true
          */
@@ -112,6 +112,25 @@ class View extends Component {
         this.rendered && value.register(this)
     }
 
+    /**
+     * Triggered after the store config got changed
+     * @param {Number} value
+     * @param {Number} oldValue
+     * @protected
+     */
+    afterSetStore(value, oldValue) {
+        let me        = this,
+            listeners = {
+                filter      : me.onStoreFilter,
+                load        : me.onStoreLoad,
+                recordChange: me.onStoreRecordChange,
+                scope       : me
+            };
+
+        oldValue?.un(listeners);
+        value   ?.on(listeners);
+    }
+
     /**
      * @param {Object} data
      * @param {String} [data.cellId]
@@ -145,6 +164,8 @@ class View extends Component {
             fieldValue = ''
         }
 
+        me.bindCallback(column.renderer, 'renderer', column.rendererScope || tableContainer, column);
+
         rendererOutput = column.renderer.call(column.rendererScope || tableContainer, {
             column,
             columnIndex,
@@ -523,6 +544,33 @@ class View extends Component {
         this.fireRowEvent(data, 'rowDoubleClick')
     }
 
+    /**
+     *
+     */
+    onStoreFilter() {
+        this.onStoreLoad()
+    }
+
+    /**
+     * @param {Object[]} data
+     * @protected
+     */
+    onStoreLoad(data) {
+        let me = this;
+
+        me.createViewData();
+
+        if (me.mounted) {
+            me.timeout(50).then(() => {
+                Neo.main.DomAccess.scrollTo({
+                    direction: 'top',
+                    id       : me.vdom.id,
+                    value    : 0
+                })
+            })
+        }
+    }
+
     /**
      * Gets triggered after changing the value of a record field.
      * E.g. myRecord.foo = 'bar';
@@ -531,7 +579,7 @@ class View extends Component {
      * @param {Neo.data.Model} opts.model The model instance of the changed record
      * @param {Object} opts.record
      */
-    onStoreRecordChange({fields, model, record}) {
+    onStoreRecordChange({fields, record}) {
         let me                     = this,
             fieldNames             = fields.map(field => field.name),
             needsUpdate            = false,

package/src/table/header/Button.mjs

@@ -197,25 +197,11 @@ class Button extends BaseButton {
      * @param {Boolean} oldValue
      * @protected
      */
-     afterSetSortable(value, oldValue) {
+    afterSetSortable(value, oldValue) {
         let me    = this,
             {cls} = me;
 
-        if (value === true) {
-            NeoArray.remove(cls, 'neo-sort-hidden');
-
-            me.addDomListeners({
-                click: me.onButtonClick,
-                scope: me
-            })
-        } else {
-            NeoArray.add(cls, 'neo-sort-hidden');
-
-            me.removeDomListeners({
-                click: me.onButtonClick,
-                scope: me
-            })
-        }
+        NeoArray.toggle(cls, 'neo-sort-hidden', !value);
 
         me.cls = cls;
         me.update()
@@ -354,9 +340,10 @@ class Button extends BaseButton {
     }
 
     /**
+     * @param {Object} data
      * @protected
      */
-    onButtonClick() {
+    onClick(data) {
         let me = this,
             map;
 
@@ -374,7 +361,9 @@ class Button extends BaseButton {
             }
         }
 
-        me.isSorted = map[me.isSorted + '']
+        me.isSorted = map[me.isSorted + ''];
+
+        super.onClick(data)
     }
 
     /**

package/src/util/Function.mjs

@@ -135,10 +135,12 @@ export function intercept(target, targetMethodName, interceptFunction, scope, pr
  * @returns {Object}
  */
 export function resolveCallback(fn, scope=this) {
-    if (typeof fn === 'string') {
+    if (Neo.isString(fn)) {
         if (!scope[fn] && fn.startsWith('up.')) {
             fn = fn.slice(3);
             while (!scope[fn] && (scope = scope.parent));
+        } else {
+            scope = scope.getController?.()?.getHandlerScope(fn, null) || scope
         }
 
         fn = scope[fn]

package/src/util/VDom.mjs

@@ -393,10 +393,10 @@ class VDom extends Base {
             }
 
             if (childNodes) {
-                cn   = childNodes.map(item => VDom.getVdom(item));
-                cn   = cn.filter(item => item.removeDom !== true);
-                i    = 0;
-                len  = cn?.length || 0;
+                cn  = childNodes.map(item => VDom.getVdom(item));
+                cn  = cn.filter(item => item.removeDom !== true);
+                i   = 0;
+                len = cn?.length || 0;
 
                 for (; i < len; i++) {
                     if (vnode.childNodes) {