neo.mjs 6.17.2 → 6.18.1

package/src/draggable/toolbar/SortZone.mjs

@@ -82,10 +82,9 @@ class SortZone extends DragZone {
      * @param {Object} data
      */
     async onDragEnd(data) {
-        let me         = this,
-            owner      = me.owner,
-            itemStyles = me.itemStyles,
-            ownerStyle = owner.style || {},
+        let me                  = this,
+            {itemStyles, owner} = me,
+            ownerStyle          = owner.style || {},
             itemStyle;
 
         await me.timeout(10);
@@ -108,14 +107,14 @@ class SortZone extends DragZone {
                 });
 
                 if (index === me.startIndex) {
-                    itemStyle.visibility = null;
+                    itemStyle.visibility = null
                 }
 
-                item.style = itemStyle;
+                item.style = itemStyle
             });
 
             if (me.startIndex !== me.currentIndex) {
-                me.moveTo(me.startIndex, me.currentIndex);
+                me.moveTo(me.startIndex, me.currentIndex)
             }
 
             Object.assign(me, {
@@ -129,7 +128,7 @@ class SortZone extends DragZone {
 
             await me.timeout(30);
 
-            me.dragEnd(data); // we do not want to trigger the super class call here
+            me.dragEnd(data) // we do not want to trigger the super class call here
         }
     }
 
@@ -138,33 +137,33 @@ class SortZone extends DragZone {
      */
     onDragMove(data) {
         if (this.itemRects) { // the method can trigger before we got the client rects from the main thread
-            let me         = this,
-                moveFactor = 0.55, // we can not use 0.5, since items would jump back & forth
-                index      = me.currentIndex,
-                itemRects  = me.itemRects,
-                maxItems   = itemRects.length - 1,
-                reversed   = me.reversedLayoutDirection,
+            let me          = this,
+                moveFactor  = 0.55, // we can not use 0.5, since items would jump back & forth
+                index       = me.currentIndex,
+                {itemRects} = me,
+                maxItems    = itemRects.length - 1,
+                reversed    = me.reversedLayoutDirection,
                 delta, itemWidth;
 
             if (me.sortDirection === 'horizontal') {
                 delta     = data.clientX - me.offsetX - itemRects[index].left;
-                itemWidth = 'width';
+                itemWidth = 'width'
             } else {
                 delta     = data.clientY - me.offsetY - itemRects[index].top;
-                itemWidth = 'height';
+                itemWidth = 'height'
             }
 
             if (index > 0 && (!reversed && delta < 0 || reversed && delta > 0)) {
                 if (Math.abs(delta) > itemRects[index - 1][itemWidth] * moveFactor) {
                     me.currentIndex--;
-                    me.switchItems(index, me.currentIndex);
+                    me.switchItems(index, me.currentIndex)
                 }
             }
 
             else if (index < maxItems && (!reversed && delta > 0 || reversed && delta < 0)) {
                 if (Math.abs(delta) > itemRects[index + 1][itemWidth] * moveFactor) {
                     me.currentIndex++;
-                    me.switchItems(index, me.currentIndex);
+                    me.switchItems(index, me.currentIndex)
                 }
             }
         }
@@ -176,9 +175,9 @@ class SortZone extends DragZone {
     onDragStart(data) {
         let me         = this,
             button     = Neo.getComponent(data.path[0].id),
-            owner      = me.owner,
+            {owner}    = me,
             itemStyles = me.itemStyles = [],
-            layout     = owner.layout,
+            {layout}   = owner,
             ownerStyle = owner.style || {},
             index, indexMap, itemStyle, rect;
 
@@ -205,7 +204,7 @@ class SortZone extends DragZone {
                 itemStyles.push({
                     height: item.style?.height,
                     width : item.style?.width
-                });
+                })
             });
 
             owner.getDomRect([owner.id].concat(owner.items.map(e => e.id))).then(itemRects => {
@@ -231,7 +230,7 @@ class SortZone extends DragZone {
                         position: 'absolute',
                         top     : `${rect.top}px`,
                         width   : `${rect.width}px`
-                    });
+                    })
                 });
 
                 // we need to add a short (1 frame) delay to ensure the item has switched to an absolute position
@@ -239,8 +238,8 @@ class SortZone extends DragZone {
                     itemStyle = button.style || {};
                     itemStyle.visibility = 'hidden';
                     button.style = itemStyle;
-                }, 5);
-            });
+                }, 5)
+            })
         }
     }
 
@@ -269,11 +268,11 @@ class SortZone extends DragZone {
         if (me.sortDirection === 'horizontal') {
             rect1.width = rect2Copy.width;
             rect2.x     = rect1Copy.x + rect2Copy.width;
-            rect2.width = rect1Copy.width;
+            rect2.width = rect1Copy.width
         } else {
             rect1.height = rect2Copy.height;
             rect2.height = rect1Copy.height;
-            rect2.y      = rect1Copy.y + rect2Copy.height;
+            rect2.y      = rect1Copy.y + rect2Copy.height
         }
 
         tmp         = map[index1];
@@ -281,7 +280,7 @@ class SortZone extends DragZone {
         map[index2] = tmp;
 
         me.updateItem(index1, rect1);
-        me.updateItem(index2, rect2);
+        me.updateItem(index2, rect2)
     }
 
     /**
@@ -289,14 +288,14 @@ class SortZone extends DragZone {
      * @param {Object} rect
      */
     updateItem(index, rect) {
-        let me    = this,
-            item  = me.owner.items[me.indexMap[index]],
-            style = item.style;
+        let me      = this,
+            item    = me.owner.items[me.indexMap[index]],
+            {style} = item;
 
         style.left = `${rect.left}px`;
         style.top  = `${rect.top}px`;
 
-        item.style = style;
+        item.style = style
     }
 }
 

package/src/draggable/tree/DragZone.mjs

@@ -40,19 +40,19 @@ class DragZone extends BaseDragZone {
      */
     afterSetLeafNodesOnly(value, oldValue) {
         if (oldValue !== undefined) { // we only need to react to dynamic changes
-            let owner = this.owner,
-                store = owner.store,
+            let {owner} = this,
+                {store} = owner,
                 node;
 
             store.items.forEach(record => {
                 if (!record.isLeaf) {
                     node = owner.getVdomChild(owner.getItemId(record.id), owner.vdom);
                     node.cls = node.cls || [];
-                    NeoArray[value ? 'remove' : 'add'](node.cls, 'neo-draggable');
+                    NeoArray[value ? 'remove' : 'add'](node.cls, 'neo-draggable')
                 }
             });
 
-            owner.update();
+            owner.update()
         }
     }
 
@@ -60,7 +60,7 @@ class DragZone extends BaseDragZone {
      * @returns {Object}
      */
     getDragElementRoot() {
-        return this.dragElement.cn[0];
+        return this.dragElement.cn[0]
     }
 
     /**
@@ -69,13 +69,13 @@ class DragZone extends BaseDragZone {
      * @returns {Object|null} vdom
      */
     getItemVdom(record, index) {
-        let owner = this.owner;
+        let {owner} = this;
 
         if (!(this.leafNodesOnly && !record.isLeaf)) {
-            return owner.getVdomChild(owner.getItemId(record.id), owner.vdom);
+            return owner.getVdomChild(owner.getItemId(record.id), owner.vdom)
         }
 
-        return null;
+        return null
     }
 
     /**
@@ -91,7 +91,7 @@ class DragZone extends BaseDragZone {
                 cn : [VDomUtil.findVdomChild(me.owner.vdom, data.path[0].id).vdom]
             };
 
-            me.dragStart(data);
+            me.dragStart(data)
         }
     }
 }

package/src/filter/BooleanContainer.mjs

@@ -47,7 +47,7 @@ class BooleanContainer extends Container {
                 component: this,
                 oldValue,
                 value
-            });
+            })
         }
     }
 
@@ -85,7 +85,7 @@ class BooleanContainer extends Container {
             ...me.radioConfig
         }];
 
-        super.createItems();
+        super.createItems()
     }
 
     /**
@@ -93,7 +93,7 @@ class BooleanContainer extends Container {
      */
     onRadioChange(data) {
         if (data.value) {
-            this.value = data.component.fieldValue;
+            this.value = data.component.fieldValue
         }
     }
 }

package/src/filter/NumberContainer.mjs

@@ -61,7 +61,7 @@ class NumberContainer extends Container {
                 component: this,
                 oldValue,
                 value
-            });
+            })
         }
     }
 
@@ -76,7 +76,7 @@ class NumberContainer extends Container {
                 component: this,
                 oldValue,
                 value
-            });
+            })
         }
     }
 
@@ -100,21 +100,21 @@ class NumberContainer extends Container {
             ...me.fieldConfig
         }];
 
-        super.createItems();
+        super.createItems()
     }
 
     /**
      * @param {Object} data
      */
     onOperatorChange(data) {
-        this.operator = data.component.value;
+        this.operator = data.component.value
     }
 
     /**
      * @param {Object} data
      */
     onValueChange(data) {
-        this.value = data.component.value;
+        this.value = data.component.value
     }
 }
 

package/src/filter/ToggleOperatorsButton.mjs

@@ -37,7 +37,7 @@ class ToggleOperatorsButton extends Button {
 
         me.handler = me.onButtonClick;
 
-        me.text = me.value || me.operators[0];
+        me.text = me.value || me.operators[0]
     }
 
     /**
@@ -55,7 +55,7 @@ class ToggleOperatorsButton extends Button {
                 component: me,
                 oldValue,
                 value
-            });
+            })
         }
     }
 
@@ -72,12 +72,12 @@ class ToggleOperatorsButton extends Button {
 
             for (; i < len; i++) {
                 if (this.beforeSetEnumValue(value[i], oldValue, 'operators', Filter.operators) !== value[i]) {
-                    return oldValue;
+                    return oldValue
                 }
             }
         }
 
-        return value;
+        return value
     }
 
     /**
@@ -87,7 +87,7 @@ class ToggleOperatorsButton extends Button {
         let me    = this,
             index = (me.operators.indexOf(data.component.text) + 1) % me.operators.length;
 
-        me.value = me.operators[index];
+        me.value = me.operators[index]
     }
 }
 

package/src/form/Fieldset.mjs

@@ -94,28 +94,28 @@ class Fieldset extends FormContainer {
 
                         if (value) {
                             if (item.disabled) {
-                                me.itemsDisabledMap.push(item.id);
+                                me.itemsDisabledMap.push(item.id)
                             } else {
-                                item._disabled = true; // silent update
+                                item._disabled = true // silent update
                             }
                         } else {
                             if (!me.itemsDisabledMap.includes(item.id)) {
-                                item._disabled = false; // silent update
+                                item._disabled = false // silent update
                             }
                         }
                     }
 
-                    item.vdom.removeDom = value;
+                    item.vdom.removeDom = value
                 }
             });
 
             if (!value) {
                 // reset the disabled items map when expanding
-                me.itemsDisabledMap = [];
+                me.itemsDisabledMap = []
             }
         }
 
-        me.update();
+        me.update()
     }
 
     /**
@@ -125,14 +125,14 @@ class Fieldset extends FormContainer {
      * @protected
      */
     afterSetCollapsible(value, oldValue) {
-        let me  = this,
-            cls = me.cls;
+        let me            = this,
+            {cls, legend} = me;
 
         NeoArray[value ? 'add' : 'remove'](cls, 'neo-collapsible');
         me.cls = cls;
 
-        if (me.legend) {
-            me.legend.useIcon = value;
+        if (legend) {
+            legend.useIcon = value;
         }
 
         if (value && !me.hasLabelClickListener) {
@@ -141,7 +141,7 @@ class Fieldset extends FormContainer {
             me.addDomListeners({
                 click   : me.onLegendClick,
                 delegate: 'neo-legend'
-            });
+            })
         }
     }
 
@@ -153,7 +153,7 @@ class Fieldset extends FormContainer {
      */
     afterSetIconClsChecked(value, oldValue) {
         if (oldValue !== undefined) {
-            this.updateLegend();
+            this.updateLegend()
         }
     }
 
@@ -165,7 +165,7 @@ class Fieldset extends FormContainer {
      */
     afterSetIconClsUnchecked(value, oldValue) {
         if (oldValue !== undefined) {
-            this.updateLegend();
+            this.updateLegend()
         }
     }
 
@@ -176,7 +176,7 @@ class Fieldset extends FormContainer {
      * @protected
      */
     afterSetTitle(value, oldValue) {
-        this.updateLegend();
+        this.updateLegend()
     }
 
     /**
@@ -185,7 +185,7 @@ class Fieldset extends FormContainer {
     onConstructed() {
         super.onConstructed();
 
-        this.collapsed && this.afterSetCollapsed(true, false);
+        this.collapsed && this.afterSetCollapsed(true, false)
     }
 
     /**
@@ -195,7 +195,7 @@ class Fieldset extends FormContainer {
         let me = this;
 
         if (me.collapsible) {
-            me.collapsed = !me.collapsed;
+            me.collapsed = !me.collapsed
         }
     }
 
@@ -203,33 +203,33 @@ class Fieldset extends FormContainer {
      *
      */
     updateLegend() {
-        let me      = this,
-            iconCls = me.collapsed ? me.iconClsUnchecked : me.iconClsChecked,
-            title   = me.title;
+        let me              = this,
+            iconCls         = me.collapsed ? me.iconClsUnchecked : me.iconClsChecked,
+            {legend, title} = me;
 
         if (iconCls === '' && title === '') {
-            if (me.legend) {
-                me.legend.vdom.reomveDom = true;
+            if (legend) {
+                legend.vdom.reomveDom = true;
             }
         } else {
-            if (me.legend) {
-                me.legend.setSilent({
+            if (legend) {
+                legend.setSilent({
                     iconCls,
                     text: title
                 });
 
-                delete me.legend.vdom.reomveDom;
+                delete legend.vdom.reomveDom
             } else {
                 me.legend = me.insert(0, {
                     module: Legend,
                     iconCls,
                     text  : title,
                     ...me.legendConfig
-                });
+                })
             }
         }
 
-        me.update();
+        me.update()
     }
 }
 

package/src/form/field/Base.mjs

@@ -104,7 +104,7 @@ class Base extends Component {
      * @param {String|null} oldValue
      */
     afterSetIsTouched(value, oldValue) {
-        let cls = this.cls;
+        let {cls} = this;
 
         NeoArray.toggle(cls, 'neo-is-touched', value);
         this.cls = cls

package/src/form/field/CheckBox.mjs

@@ -189,7 +189,7 @@ class CheckBox extends Base {
             oldCls  = value ? me.iconCls : me.iconClsChecked;
 
         if (oldValue) {
-            me.clean = false;
+            me.clean = false
         }
 
         me.validate(); // silent
@@ -259,8 +259,7 @@ class CheckBox extends Base {
      */
     afterSetId(value, oldValue) {
         let me      = this,
-            vdom    = me.vdom,
-            labelEl = vdom.cn[0];
+            labelEl = me.vdom.cn[0];
 
         labelEl.cn[0].id = me.getLabelId();
         labelEl.cn[1].id = me.getInputElId();
@@ -304,8 +303,8 @@ class CheckBox extends Base {
      * @protected
      */
     afterSetLabelPosition(value, oldValue) {
-        let me  = this,
-            cls = me.cls;
+        let me    = this,
+            {cls} = me;
 
         NeoArray.remove(cls, 'neo-label-' + oldValue);
         NeoArray.add(   cls, 'neo-label-' + value);
@@ -371,7 +370,7 @@ class CheckBox extends Base {
      * @protected
      */
     afterSetUseAlertState(value, oldValue) {
-        let cls = this.cls;
+        let {cls} = this;
         NeoArray.toggle(cls, 'neo-use-alert-state', value);
         this.cls = cls
     }
@@ -398,7 +397,7 @@ class CheckBox extends Base {
             showLabel  = !!value; // hide the label, in case value === null || value === ''
 
         if (showLabel) {
-            valueLabel.innerHTML = value;
+            valueLabel.innerHTML = value
         }
 
         valueLabel.removeDom = !showLabel;
@@ -571,7 +570,7 @@ class CheckBox extends Base {
      */
     updateError(value, silent=false) {
         let me        = this,
-            cls       = me.cls,
+            {cls}     = me,
             showError = value && me.showErrorTexts,
             errorNode, errorWrapper;
 
@@ -585,9 +584,9 @@ class CheckBox extends Base {
             errorNode    = errorWrapper.cn[0];
 
             if (showError) {
-                errorNode.html = value;
+                errorNode.html = value
             } else {
-                delete errorNode.html;
+                delete errorNode.html
             }
 
             errorWrapper.removeDom = !showError;
@@ -603,7 +602,7 @@ class CheckBox extends Base {
      */
     validate(silent=true) {
         let me          = this,
-            name        = me.name,
+            {name}      = me,
             returnValue = true,
             checkBox, checkBoxes;
 
@@ -618,14 +617,14 @@ class CheckBox extends Base {
             // discuss: we could limit this to checkBoxes / radios inside the same form, IF a top level form is used
             checkBoxes = ComponentManager.find({
                 ntype: me.ntype,
-                name : me.name
+                name
             });
 
             // get the group validity state first
             for (checkBox of checkBoxes) {
                 if (checkBox.checked) {
                     returnValue = true;
-                    break;
+                    break
                 }
             }
 
@@ -641,15 +640,15 @@ class CheckBox extends Base {
             }
 
             if (!returnValue) {
-                me._error = me.errorTextGroupRequired({name});
+                me._error = me.errorTextGroupRequired({name})
             }
         } else if (me.required && !me.checked) {
             me._error = me.errorTextRequired;
-            returnValue = false;
+            returnValue = false
         }
 
         if (returnValue) {
-            me._error = null;
+            me._error = null
         }
 
         !me.clean && me.updateError(me._error, silent);

package/src/form/field/Color.mjs

@@ -74,8 +74,7 @@ class Color extends ComboBox {
     afterSetValue(value, oldValue, preventFilter=false) {
         let me             = this,
             colorIndicator = VDomUtil.findVdomChild(me.vdom, {id: me.getColorIndicatorId()})?.vdom,
-            list           = me.list,
-            record         = me.record,
+            {list, record} = me,
             selectionModel = me.list?.selectionModel;
 
         if (colorIndicator) {
@@ -83,54 +82,53 @@ class Color extends ComboBox {
         }
 
         if (record) {
-            selectionModel?.select(list.getItemId(record[me.store.keyProperty]));
+            selectionModel?.select(list.getItemId(record[me.store.keyProperty]))
         } else {
-            selectionModel?.deselectAll(true);
+            selectionModel?.deselectAll(true)
         }
 
         // the super call will trigger the vdom update
-        super.afterSetValue(value, oldValue, preventFilter);
+        super.afterSetValue(value, oldValue, preventFilter)
     }
 
     /**
      * @returns {String}
      */
     getColor() {
-        let me     = this,
-            record = me.record,
-            value  = me.value;
+        let me              = this,
+            {record, value} = me;
 
-        return record ? me.colorFormatter(me, record) : me.forceSelection ? null : value;
+        return record ? me.colorFormatter(me, record) : me.forceSelection ? null : value
     }
 
     /**
      * @returns {String}
      */
     getColorIndicatorId() {
-        return `${this.id}__color-indicator`;
+        return `${this.id}__color-indicator`
     }
 
     /**
      * @protected
      */
     onSelectPostLastItem() {
-        let list  = this.list,
-            index = list.store.getCount() - 1;
+        let {list} = this,
+            index  = list.store.getCount() - 1;
 
         list.vdom.cn[index] = list.createItem(list.store.getAt(index), index);
 
-        super.onSelectPostLastItem();
+        super.onSelectPostLastItem()
     }
 
     /**
      * @protected
      */
     onSelectPreFirstItem() {
-        let list = this.list;
+        let {list} = this;
 
         list.vdom.cn[0] = list.createItem(list.store.getAt(0), 0);
 
-        super.onSelectPreFirstItem();
+        super.onSelectPreFirstItem()
     }
 }