selectic 3.0.5 → 3.0.6

package/dist/selectic.common.js

@@ -101,6 +101,26 @@ let closePreviousSelectic;
 let uid = 0;
 class SelecticStore {
     constructor(props = {}) {
+        /* Do not need reactivity */
+        this.requestId = 0;
+        this._uid = ++uid;
+        /* {{{ Props */
+        const defaultProps = {
+            value: null,
+            selectionIsExcluded: false,
+            disabled: false,
+            options: null,
+            childOptions: [],
+            groups: [],
+            texts: null,
+            params: {},
+            fetchCallback: null,
+            getItemsCallback: null,
+            keepOpenWithOtherSelectic: false,
+        };
+        const propsVal = assignObject(defaultProps, props);
+        this.props = vue.reactive(propsVal);
+        /* }}} */
         /* {{{ data */
         this.state = vue.reactive({
             multiple: false,
@@ -141,27 +161,6 @@ class SelecticStore {
                 automaticClose: false,
             },
         });
-        /* Do not need reactivity */
-        this.requestId = 0;
-        this._uid = ++uid;
-        /* {{{ Props */
-        const defaultProps = {
-            value: null,
-            selectionIsExcluded: false,
-            disabled: false,
-            options: null,
-            childOptions: [],
-            groups: [],
-            texts: null,
-            params: {},
-            fetchCallback: null,
-            getItemsCallback: null,
-            keepOpenWithOtherSelectic: false,
-        };
-        const propsVal = assignObject(defaultProps, props);
-        this.props = vue.reactive(propsVal);
-        /* }}} */
-        /* {{{ data */
         this.data = vue.reactive({
             labels: Object.assign({}, messages),
             itemsPerPage: 10,
@@ -275,8 +274,8 @@ class SelecticStore {
          * and ensure convertValue run with correct state */
         assignObject(this.state, {
             internalValue: this.convertTypeValue(value),
-            selectionIsExcluded: props.selectionIsExcluded,
-            disabled: props.disabled,
+            selectionIsExcluded: !!props.selectionIsExcluded,
+            disabled: !!props.disabled, /* XXX: !! is needed to copy value and not proxy reference */
         });
         this.checkHideFilter();
         if (this.props.texts) {

package/dist/selectic.esm.js

@@ -97,6 +97,26 @@ let closePreviousSelectic;
 let uid = 0;
 class SelecticStore {
     constructor(props = {}) {
+        /* Do not need reactivity */
+        this.requestId = 0;
+        this._uid = ++uid;
+        /* {{{ Props */
+        const defaultProps = {
+            value: null,
+            selectionIsExcluded: false,
+            disabled: false,
+            options: null,
+            childOptions: [],
+            groups: [],
+            texts: null,
+            params: {},
+            fetchCallback: null,
+            getItemsCallback: null,
+            keepOpenWithOtherSelectic: false,
+        };
+        const propsVal = assignObject(defaultProps, props);
+        this.props = reactive(propsVal);
+        /* }}} */
         /* {{{ data */
         this.state = reactive({
             multiple: false,
@@ -137,27 +157,6 @@ class SelecticStore {
                 automaticClose: false,
             },
         });
-        /* Do not need reactivity */
-        this.requestId = 0;
-        this._uid = ++uid;
-        /* {{{ Props */
-        const defaultProps = {
-            value: null,
-            selectionIsExcluded: false,
-            disabled: false,
-            options: null,
-            childOptions: [],
-            groups: [],
-            texts: null,
-            params: {},
-            fetchCallback: null,
-            getItemsCallback: null,
-            keepOpenWithOtherSelectic: false,
-        };
-        const propsVal = assignObject(defaultProps, props);
-        this.props = reactive(propsVal);
-        /* }}} */
-        /* {{{ data */
         this.data = reactive({
             labels: Object.assign({}, messages),
             itemsPerPage: 10,
@@ -271,8 +270,8 @@ class SelecticStore {
          * and ensure convertValue run with correct state */
         assignObject(this.state, {
             internalValue: this.convertTypeValue(value),
-            selectionIsExcluded: props.selectionIsExcluded,
-            disabled: props.disabled,
+            selectionIsExcluded: !!props.selectionIsExcluded,
+            disabled: !!props.disabled, /* XXX: !! is needed to copy value and not proxy reference */
         });
         this.checkHideFilter();
         if (this.props.texts) {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "selectic",
-    "version": "3.0.5",
+    "version": "3.0.6",
     "description": "Smart Select for VueJS 3.x",
     "main": "dist/selectic.common.js",
     "module": "dist/selectic.esm.js",

package/src/Store.tsx

@@ -411,48 +411,7 @@ export default class SelecticStore {
 
     /* {{{ data */
 
-    public state = reactive<SelecticStoreState>({
-        multiple: false,
-        disabled: false,
-        placeholder: '',
-        hideFilter: false,
-        keepFilterOpen: false,
-        allowRevert: undefined,
-        allowClearSelection: false,
-        autoSelect: true,
-        autoDisabled: true,
-        strictValue: false,
-        selectionOverflow: 'collapsed',
-
-        internalValue: null,
-        isOpen: false,
-        searchText: '',
-        selectionIsExcluded: false,
-        allOptions: [],
-        dynOptions: [],
-        filteredOptions: [],
-        selectedOptions: null,
-        totalAllOptions: Infinity,
-        totalDynOptions: Infinity,
-        totalFilteredOptions: Infinity,
-        groups: new Map(),
-        offsetItem: 0,
-        activeItemIdx: -1,
-        pageSize: 100,
-        listPosition: 'auto',
-
-        optionBehaviorOperation: 'sort',
-        optionBehaviorOrder: ['O', 'D', 'E'],
-
-        status: {
-            searching: false,
-            errorMessage: '',
-            areAllSelected: false,
-            hasChanged: false,
-            automaticChange: false,
-            automaticClose: false,
-        },
-    });
+    public state: SelecticStoreState;
     public data: Data;
 
     /* Do not need reactivity */
@@ -500,6 +459,49 @@ export default class SelecticStore {
         /* }}} */
         /* {{{ data */
 
+        this.state = reactive<SelecticStoreState>({
+            multiple: false,
+            disabled: false,
+            placeholder: '',
+            hideFilter: false,
+            keepFilterOpen: false,
+            allowRevert: undefined,
+            allowClearSelection: false,
+            autoSelect: true,
+            autoDisabled: true,
+            strictValue: false,
+            selectionOverflow: 'collapsed',
+
+            internalValue: null,
+            isOpen: false,
+            searchText: '',
+            selectionIsExcluded: false,
+            allOptions: [],
+            dynOptions: [],
+            filteredOptions: [],
+            selectedOptions: null,
+            totalAllOptions: Infinity,
+            totalDynOptions: Infinity,
+            totalFilteredOptions: Infinity,
+            groups: new Map(),
+            offsetItem: 0,
+            activeItemIdx: -1,
+            pageSize: 100,
+            listPosition: 'auto',
+
+            optionBehaviorOperation: 'sort',
+            optionBehaviorOrder: ['O', 'D', 'E'],
+
+            status: {
+                searching: false,
+                errorMessage: '',
+                areAllSelected: false,
+                hasChanged: false,
+                automaticChange: false,
+                automaticClose: false,
+            },
+        });
+
         this.data = reactive({
             labels: Object.assign({}, messages),
             itemsPerPage: 10,
@@ -649,8 +651,8 @@ export default class SelecticStore {
          * and ensure convertValue run with correct state */
         assignObject(this.state, {
             internalValue: this.convertTypeValue(value),
-            selectionIsExcluded: props.selectionIsExcluded,
-            disabled: props.disabled,
+            selectionIsExcluded: !!props.selectionIsExcluded,
+            disabled: !!props.disabled, /* XXX: !! is needed to copy value and not proxy reference */
         });
 
         this.checkHideFilter();

package/test/Store/Store_props.spec.js

@@ -223,6 +223,7 @@ tape.test('change props', (subT) => {
             const store = new Store({
                 value: 2,
                 options: getOptions(5, 'alpha'),
+                disabled: false,
                 params: {
                     autoDisabled: true,
                 },
@@ -249,6 +250,7 @@ tape.test('change props', (subT) => {
             const store = new Store({
                 value: 2,
                 options: getOptions(5, 'alpha'),
+                disabled: false,
                 params: {
                     autoDisabled: true,
                     strictValue: true,
@@ -298,6 +300,37 @@ tape.test('change props', (subT) => {
             t.end();
         });
 
+        sTest.test('should re-enable the select when data are loaded', async (t) => {
+            const store = new Store({
+                options: [],
+                disabled: false,
+                params: {
+                    autoDisabled: true,
+                },
+            });
+            await _.nextVueTick(store);
+
+            store.commit('isOpen', true);
+            await _.nextVueTick(store);
+
+            t.is(store.state.internalValue, null);
+            t.is(store.state.disabled, true);
+            t.is(store.state.isOpen, false);
+
+            store.props.options = getOptions(0, 'alpha');
+            await _.nextVueTick(store);
+
+            t.is(store.state.internalValue, null);
+            t.is(store.state.disabled, true), 'should be disabled';
+
+            store.props.options = getOptions(5, 'beta');
+            await _.nextVueTick(store);
+
+            t.is(store.state.internalValue, 0);
+            t.is(store.state.disabled, false), 'should be enabled';
+            t.end();
+        });
+
         sTest.test('should not re-enable the select if disable is set', async (t) => {
             const store = new Store({
                 options: getOptions(1, 'alpha'),

package/types/List.d.ts

@@ -1,5 +1,5 @@
 import { Vue, h } from 'vtyx';
-import Store, { OptionItem, OptionId } from './Store';
+import Store, { OptionItem } from './Store';
 export interface Props {
     store: Store;
     options?: any[];
@@ -14,31 +14,7 @@ export default class List extends Vue<Props> {
     private itemHeight;
     private groupId;
     private doNotScroll;
-    get filteredOptions(): {
-        selected: boolean;
-        disabled: boolean;
-        isGroup: boolean;
-        id: OptionId;
-        text: string;
-        title?: string | undefined;
-        group?: import("./Store").StrictOptionId | undefined;
-        className?: string | undefined;
-        style?: string | undefined;
-        icon?: string | undefined;
-        options?: {
-            id: OptionId;
-            text: string;
-            title?: string | undefined;
-            disabled?: boolean | undefined;
-            group?: import("./Store").StrictOptionId | undefined;
-            className?: string | undefined;
-            style?: string | undefined;
-            icon?: string | undefined;
-            options?: any[] | undefined;
-            data?: any;
-        }[] | undefined;
-        data?: any;
-    }[];
+    get filteredOptions(): OptionItem[];
     get isMultiple(): boolean;
     get itemsMargin(): number;
     get shortOptions(): OptionItem[];

package/types/Store.d.ts

@@ -142,141 +142,7 @@ export declare type PartialMessages = {
 export declare function changeTexts(texts: PartialMessages): void;
 export default class SelecticStore {
     props: InternalProps;
-    state: {
-        internalValue: OptionId | StrictOptionId[];
-        selectionIsExcluded: boolean;
-        multiple: boolean;
-        disabled: boolean;
-        placeholder: string;
-        hideFilter: boolean;
-        keepFilterOpen: boolean;
-        allowRevert?: boolean | undefined;
-        allowClearSelection: boolean;
-        autoSelect: boolean;
-        autoDisabled: boolean;
-        strictValue: boolean;
-        selectionOverflow: SelectionOverflow;
-        isOpen: boolean;
-        searchText: string;
-        allOptions: {
-            id: OptionId;
-            text: string;
-            title?: string | undefined;
-            disabled?: boolean | undefined;
-            group?: StrictOptionId | undefined;
-            className?: string | undefined;
-            style?: string | undefined;
-            icon?: string | undefined;
-            options?: any[] | undefined;
-            data?: any;
-        }[];
-        dynOptions: {
-            id: OptionId;
-            text: string;
-            title?: string | undefined;
-            disabled?: boolean | undefined;
-            group?: StrictOptionId | undefined;
-            className?: string | undefined;
-            style?: string | undefined;
-            icon?: string | undefined;
-            options?: any[] | undefined;
-            data?: any;
-        }[];
-        filteredOptions: {
-            selected: boolean;
-            disabled: boolean;
-            isGroup: boolean;
-            id: OptionId;
-            text: string;
-            title?: string | undefined;
-            group?: StrictOptionId | undefined;
-            className?: string | undefined;
-            style?: string | undefined;
-            icon?: string | undefined;
-            options?: {
-                id: OptionId;
-                text: string;
-                title?: string | undefined;
-                disabled?: boolean | undefined;
-                group?: StrictOptionId | undefined;
-                className?: string | undefined;
-                style?: string | undefined;
-                icon?: string | undefined;
-                options?: any[] | undefined;
-                data?: any;
-            }[] | undefined;
-            data?: any;
-        }[];
-        selectedOptions: {
-            selected: boolean;
-            disabled: boolean;
-            isGroup: boolean;
-            id: OptionId;
-            text: string;
-            title?: string | undefined;
-            group?: StrictOptionId | undefined;
-            className?: string | undefined;
-            style?: string | undefined;
-            icon?: string | undefined;
-            options?: {
-                id: OptionId;
-                text: string;
-                title?: string | undefined;
-                disabled?: boolean | undefined;
-                group?: StrictOptionId | undefined;
-                className?: string | undefined;
-                style?: string | undefined;
-                icon?: string | undefined;
-                options?: any[] | undefined;
-                data?: any;
-            }[] | undefined;
-            data?: any;
-        } | {
-            selected: boolean;
-            disabled: boolean;
-            isGroup: boolean;
-            id: OptionId;
-            text: string;
-            title?: string | undefined;
-            group?: StrictOptionId | undefined;
-            className?: string | undefined;
-            style?: string | undefined;
-            icon?: string | undefined;
-            options?: {
-                id: OptionId;
-                text: string;
-                title?: string | undefined;
-                disabled?: boolean | undefined;
-                group?: StrictOptionId | undefined;
-                className?: string | undefined;
-                style?: string | undefined;
-                icon?: string | undefined;
-                options?: any[] | undefined;
-                data?: any;
-            }[] | undefined;
-            data?: any;
-        }[] | null;
-        totalAllOptions: number;
-        totalDynOptions: number;
-        totalFilteredOptions: number;
-        groups: Map<OptionId, string>;
-        offsetItem: number;
-        activeItemIdx: number;
-        pageSize: number;
-        formatOption?: FormatCallback | undefined;
-        formatSelection?: FormatCallback | undefined;
-        optionBehaviorOperation: OptionBehaviorOperation;
-        optionBehaviorOrder: OptionBehaviorOrder[];
-        listPosition: ListPosition;
-        status: {
-            searching: boolean;
-            errorMessage: string;
-            areAllSelected: boolean;
-            hasChanged: boolean;
-            automaticChange: boolean;
-            automaticClose: boolean;
-        };
-    };
+    state: SelecticStoreState;
     data: Data;
     private requestId;
     private cacheRequest;