SelectBox: Skip custom item creating if customItemCreateEvent is '' and component loses focus (T1269852)

Co-authored-by: marker dao ® <[email protected]>

Author: marker-dao

packages/devextreme/js/__internal/ui/m_select_box.ts

@@ -55,7 +55,7 @@ const SelectBox = (DropDownList as any).inherit({
           this._resetCaretPosition(true);
         }
 
-        parent.tab && parent.tab.apply(this, arguments);
+        parent.tab?.apply(this, arguments);
 
         if (!popupHasFocusableElements) {
           this._cancelSearchIfNeed();
@@ -85,18 +85,18 @@ const SelectBox = (DropDownList as any).inherit({
       },
       rightArrow() {
         searchIfNeeded();
-        parent.rightArrow && parent.rightArrow.apply(this, arguments);
+        parent.rightArrow?.apply(this, arguments);
       },
       home() {
         searchIfNeeded();
-        parent.home && parent.home.apply(this, arguments);
+        parent.home?.apply(this, arguments);
       },
       end() {
         searchIfNeeded();
-        parent.end && parent.end.apply(this, arguments);
+        parent.end?.apply(this, arguments);
       },
       escape() {
-        const result = parent.escape && parent.escape.apply(this, arguments);
+        const result = parent.escape?.apply(this, arguments);
         this._cancelEditing();
 
         return result ?? true;
@@ -126,7 +126,7 @@ const SelectBox = (DropDownList as any).inherit({
             return isOpened;
           }
 
-          if (parent.enter && parent.enter.apply(this, arguments)) {
+          if (parent.enter?.apply(this, arguments)) {
             return isOpened;
           }
         }
@@ -269,7 +269,7 @@ const SelectBox = (DropDownList as any).inherit({
   },
 
   _scrollToSelectedItem() {
-    this._list && this._list.scrollToItem(this._list.option('selectedItem'));
+    this._list?.scrollToItem(this._list.option('selectedItem'));
   },
 
   _listContentReadyHandler() {
@@ -534,16 +534,25 @@ const SelectBox = (DropDownList as any).inherit({
       return;
     }
 
-    this._loadItemDeferred && this._loadItemDeferred.always(() => {
+    this._loadItemDeferred?.always(() => {
       const {
         acceptCustomValue,
         text,
         selectedItem: initialSelectedItem,
+        customItemCreateEvent,
       } = this.option();
 
       if (acceptCustomValue) {
         if (!saveEditingValue && !this._isValueChanging) {
-          this._updateField(initialSelectedItem ?? this._createCustomItem(text));
+          let initialItem = null;
+
+          if (isDefined(initialSelectedItem)) {
+            initialItem = initialSelectedItem;
+          } else if (customItemCreateEvent !== '') {
+            initialItem = this._createCustomItem(text);
+          }
+
+          this._updateField(initialItem);
           this._clearFilter();
         }
         return;
@@ -664,7 +673,7 @@ const SelectBox = (DropDownList as any).inherit({
   _fieldRenderData() {
     const $listFocused = this._list && this.option('opened') && $(this._list.option('focusedElement'));
 
-    if ($listFocused && $listFocused.length) {
+    if ($listFocused?.length) {
       return this._list._getItemData($listFocused);
     }
 

packages/devextreme/testing/tests/DevExpress.ui.widgets.editors/selectBox.tests.js

@@ -2380,6 +2380,24 @@ QUnit.module('editing', moduleSetup, () => {
         assert.equal(onCustomItemCreating.callCount, 0, 'action has not been called');
     });
 
+    QUnit.test('onCustomItemCreating should not be called when customItemCreateEvent is equal to empty string and component loses focus (T1269852)', function(assert) {
+        const onCustomItemCreating = sinon.stub();
+
+        const $selectBox = $('#selectBox').dxSelectBox({
+            acceptCustomValue: true,
+            customItemCreateEvent: '',
+            onCustomItemCreating: onCustomItemCreating,
+        });
+
+        const $input = $selectBox.find(toSelector(TEXTEDITOR_INPUT_CLASS));
+        const keyboard = keyboardMock($input);
+
+        keyboard.type('t');
+        $input.trigger('focusout');
+
+        assert.strictEqual(onCustomItemCreating.callCount, 0, 'action has not been called');
+    });
+
     QUnit.test('onCustomItemCreating should not be called more then once even when there is value change handler call inside of event handler (T893205)', function(assert) {
         let handlerCallCount = 0;