Современный 7.0 ComboBox запускает событие «select» при каждом изменении значения - PullRequest
1 голос
/ 27 февраля 2020

В современных списках выпадающий список вызывает событие «выбор» каждый раз, когда изменяется вход. Это очень отличается от класса c. Таким образом, невозможно провести различие между пользователем, делающим выбор и прагматически устанавливающим значение поля.

Это сообщалось как ошибка на форумах Sencha:

https://www.sencha.com/forum/showthread.php?468730-Modern-6-5-2-ComboBox-fires-select-event-on-every-value-change

Ссылка выше также содержит скрипач для демонстрации проблемы.

Кто-нибудь сталкивался с этим как с проблемой, и как вы ее преодолели?

1 Ответ

1 голос
/ 27 февраля 2020

forceSelection: true поможет решить эту проблему, но не отменит ошибку в случаях, когда принудительный выбор не требуется

Редактировать:

Такое поведение связано с методом syncValue ( поиск в этом источнике - метод является частным и не имеет документации)

Я не понимаю, почему разработчик компонента решил создать запись, даже если она не существует.

Комментарий из исходного файла:

Либо пользователь набрал что-то (isInput), либо у нас было setValue для значения, которое не соответствует в хранилище, и мы не являемся forceSelection : правда. Мы создаем новую запись.

Я предлагаю исправить это поведение, используя следующее переопределение:

fiddle

Ext.define('Ext.field.SelectOverride', {
    override: 'Ext.field.Select',
    autoCreateRecord: false,
    syncValue: function() {
        var me = this,
            store = me.getStore(),
            forceSelection = me.getForceSelection(),
            valueNotFoundText = me.getValueNotFoundText(),
            is, isCleared, isInput, value, matchedRecord;


        if (me.reconcilingValue || !store || !store.isLoaded() || store.hasPendingLoad()) {
            return;
        }

        me.reconcilingValue = true;

        me.getSelection(); // make sure selection config is flushed

        is = {};
        is[me.syncMode] = true;
        value = ((isInput = is.input || is.filter)) ? me.getInputValue() : me.getValue();
        isCleared = value == null || value === '';


        if (!isCleared) {
            if (me.getMultiSelect()) {
                return me.syncMultiValues(Ext.Array.from(value));
            }

            matchedRecord = (isInput ? store.byText : store.byValue).get(value);

            if (matchedRecord) {
                if (!matchedRecord.isEntity) {

                    matchedRecord = matchedRecord[0];
                }
            }
            else if (!forceSelection) {
                matchedRecord = me.findRecordByValue(value);
            }
        }

        // Either user has typed something (isInput), or we've had a setValue
        // to a value which has no match in the store, and we are not forceSelection: true.
        // We create a new record.
        if (!isCleared && !matchedRecord && !forceSelection && me.autoCreateRecord) {
            matchedRecord = me.createEnteredRecord(value);
        }
        else {
            if (isInput || is.store) {
                if (!matchedRecord && forceSelection) {
                    me.setValue(null);
                    me.setSelection(null);

                    if (!is.filter) {
                        me.setFieldDisplay();
                    }
                }
            }            else {
                if (isCleared) {
                    if (me.mustAutoSelect()) {
                        matchedRecord = store.first();

                        if (me.getAutoSelect() === 'initial') {
                            me.setAutoSelect(false);
                        }
                    }
                    else {
                        me.setSelection(null);
                    }
                }
                else if (!matchedRecord && valueNotFoundText) {
                    me.setError(valueNotFoundText);
                }
            }
        }

        if (matchedRecord) {
            me.setSelection(matchedRecord);
        }

        me.reconcilingValue = false;
    }
});
...