@oracle/oraclejet 18.0.1 → 18.0.3

package/dist/js/libs/oj/debug/ojtable.js

@@ -2047,26 +2047,16 @@ var __oj_table_metadata =
    * @private
    */
   Table.prototype._clearDataWaitingState = function (ignoreStatusMessage) {
-    if (this._pendingFetchStale) {
-      this._pendingFetchStale = false;
-      this._queueTask(
-        function () {
-          this._getLayoutManager().notifyTableUpdate(Table._UPDATE._DATA_REFRESH);
-          this._beforeDataRefresh();
-          return this._invokeDataFetchRows(this._pendingFetchOptions);
-        }.bind(this)
-      );
-    } else {
-      if (!ignoreStatusMessage) {
-        this._hideStatusMessage();
-      }
-      this._dataFetching = false;
-      this._pendingFetchStale = false;
+    if (!ignoreStatusMessage) {
+      this._hideStatusMessage();
+    }
+    this._dataFetching = false;
+    if (!this._hasRefreshInQueue) {
       this._pendingFetchOptions = null;
-      if (this._dataResolveFunc) {
-        this._dataResolveFunc();
-        this._dataResolveFunc = null;
-      }
+    }
+    if (this._dataResolveFunc) {
+      this._dataResolveFunc();
+      this._dataResolveFunc = null;
     }
   };
 
@@ -2221,8 +2211,6 @@ var __oj_table_metadata =
    */
   Table.prototype._clearAllComponentBusyStates = function () {
     this._clearBusyStateStack();
-    // clear any pending fetch state to ensure clearing data waiting state below does not queue addition fetches
-    this._pendingFetchStale = false;
     this._clearDataWaitingState();
     this._clearFocusoutBusyState();
     this._clearIdleRenderBusyState();
@@ -2239,6 +2227,7 @@ var __oj_table_metadata =
     // cleanup needed for both, 'destroy()' and 'ReleaseResources()' calls
     this._animateOnFetch = null;
     this._isEditPending = null;
+    this._deferredEditInfo = null;
     this._focusoutEditKey = null;
     this._skipScrollOnFocus = null;
     this._active = null;
@@ -2566,17 +2555,15 @@ var __oj_table_metadata =
     this._clearIdleCallback();
 
     // update edit state based on row removals before rows are removed to ensure row context is available
-    if (this._hasEditableRow()) {
-      // exit edit mode if editable row is deleted - treat as cancelling the edit
-      this._setTableEditable(false, true);
-    }
+    var beforeFetchPromise = this._hasEditableRow()
+      ? this._setTableEditable(false, true)
+      : Promise.resolve();
 
     // ensure we register 'refresh' update here
     this._getLayoutManager().notifyTableUpdate(Table._UPDATE._REFRESH);
-    if (initFetch) {
-      return this._initFetch();
-    }
-    return this._invokeDataFetchRows();
+    return beforeFetchPromise.then(() => {
+      return initFetch ? this._initFetch() : this._invokeDataFetchRows();
+    });
   };
 
   /**
@@ -4486,21 +4473,34 @@ var __oj_table_metadata =
    * @private
    */
   Table.prototype._processFetchSort = function (result) {
-    try {
-      var fetchParameters = result.fetchParameters;
-      var sortCriteria = fetchParameters[Table._CONST_SORTCRITERIA];
-      if (sortCriteria != null && sortCriteria.length > 0) {
-        var sortAttribute = sortCriteria[0].attribute;
-        var sortDirection = sortCriteria[0].direction === Table._COLUMN_SORT_ORDER._ASCENDING;
-        this._refreshSortTableHeaderColumn(sortAttribute, sortDirection);
-        // set the current row
-        this._setCurrentRow(this.options.currentRow);
-      } else {
-        this._clearSortedHeaderColumn();
-      }
-    } catch (e) {
-      Logger.error(e);
-    }
+    return new Promise(
+      function (resolve) {
+        try {
+          var fetchParameters = result.fetchParameters;
+          var sortCriteria = fetchParameters[Table._CONST_SORTCRITERIA];
+          if (sortCriteria != null && sortCriteria.length > 0) {
+            var sortAttribute = sortCriteria[0].attribute;
+            var sortDirection = sortCriteria[0].direction === Table._COLUMN_SORT_ORDER._ASCENDING;
+            this._refreshSortTableHeaderColumn(sortAttribute, sortDirection);
+            // set the current row
+            var currentStatus = this._setCurrentRow(this.options.currentRow);
+            if (currentStatus instanceof Promise) {
+              currentStatus.then(() => {
+                resolve();
+              });
+            } else {
+              resolve();
+            }
+          } else {
+            this._clearSortedHeaderColumn();
+            resolve();
+          }
+        } catch (e) {
+          Logger.error(e);
+          resolve();
+        }
+      }.bind(this)
+    );
   };
 
   /**
@@ -4962,7 +4962,7 @@ var __oj_table_metadata =
     // eslint-disable-next-line no-unused-vars
     this._domScrollerErrorFunc = function (reason) {
       // only clear data waiting state on abort if fetch is stale (mutation / refresh events)
-      if (controller && controller.signal.aborted && this._pendingFetchStale) {
+      if (controller && controller.signal.aborted && this._hasRefreshInQueue) {
         this._clearDataWaitingState();
       }
       this._clearScrollBuffer();
@@ -5955,11 +5955,12 @@ var __oj_table_metadata =
                 ).then(
                   function () {
                     this._clearDataWaitingState();
-                    this._processFetchSort(value);
-                    if (this._isLoadMoreOnScroll()) {
-                      this._registerDomScroller();
-                    }
-                    resolve(result);
+                    this._processFetchSort(value).then(() => {
+                      if (this._isLoadMoreOnScroll()) {
+                        this._registerDomScroller();
+                      }
+                      resolve(result);
+                    });
                   }.bind(this),
                   // eslint-disable-next-line no-unused-vars
                   function (reason) {
@@ -6039,15 +6040,14 @@ var __oj_table_metadata =
       header: sortCriteria[0][Table._CONST_ATTRIBUTE],
       direction: sortCriteria[0].direction
     });
-    this._beforeDataRefresh(true);
     // show the Fetching Data... message
     this._showStatusMessage();
-    this._queueTask(
-      function () {
-        this._getLayoutManager().notifyTableUpdate(Table._UPDATE._DATA_SORT);
+    this._queueTask(() => {
+      this._getLayoutManager().notifyTableUpdate(Table._UPDATE._DATA_SORT);
+      return this._beforeDataRefresh(true).then(() => {
         return this._initFetch({ sortCriteria: sortCriteria }, true);
-      }.bind(this)
-    );
+      });
+    });
   };
 
   /**
@@ -12754,6 +12754,22 @@ var __oj_table_metadata =
     }
   };
 
+  /**
+   * Helper method to handle mutation events while a pending fetch is already in progress
+   */
+  Table.prototype._handleStaleDataMutation = function (options) {
+    if (this._controller) {
+      this._controller.abort(DataCollectionUtils.getAbortReason(this.OuterWrapper));
+    }
+    this._hasRefreshInQueue = true;
+    this._queueTask(() => {
+      this._getLayoutManager().notifyTableUpdate(Table._UPDATE._DATA_REFRESH);
+      return this._beforeDataRefresh().then(() => {
+        return this._invokeDataFetchRows(options);
+      });
+    });
+  };
+
   /**
    * Callback handler for refresh in the datasource. Refresh entire
    * table body DOM and refresh the table dimensions.
@@ -12768,11 +12784,7 @@ var __oj_table_metadata =
       }
       // if already handling a fetch, mark pending fetch result as stale (to ensure new fetch is triggered), abort, and return
       if (this._dataFetching) {
-        if (this._controller) {
-          this._controller.abort(DataCollectionUtils.getAbortReason(this.OuterWrapper));
-        }
-        this._pendingFetchStale = true;
-        this._pendingFetchOptions = null;
+        this._handleStaleDataMutation();
         return;
       }
       if (event.detail && event.detail.disregardAfterKey !== undefined) {
@@ -12781,24 +12793,24 @@ var __oj_table_metadata =
             this._getLayoutManager().notifyTableUpdate(Table._UPDATE._ROWS_REMOVED);
             // reset active row to ensure active row index is not stale after refresh
             this._clearActiveRow(null, true);
-            this._removeRowsAfterLastValidRow(event.detail.disregardAfterKey);
-            if (!this._hasMoreToFetch()) {
-              this._registerDomScroller();
-            }
-            this._animateOnFetch = true;
+            this._removeRowsAfterLastValidRow(event.detail.disregardAfterKey).then(() => {
+              if (!this._hasMoreToFetch()) {
+                this._registerDomScroller();
+              }
+              this._animateOnFetch = true;
+            });
           }.bind(this)
         );
       } else {
         this._hasRefreshInQueue = true;
-        this._queueTask(
-          function () {
-            this._getLayoutManager().notifyTableUpdate(Table._UPDATE._DATA_REFRESH);
-            // reset active row to ensure active row index is not stale after refresh
-            this._clearActiveRow(null, true);
-            this._beforeDataRefresh();
+        this._queueTask(() => {
+          this._getLayoutManager().notifyTableUpdate(Table._UPDATE._DATA_REFRESH);
+          // reset active row to ensure active row index is not stale after refresh
+          this._clearActiveRow(null, true);
+          return this._beforeDataRefresh().then(() => {
             return this._invokeDataFetchRows();
-          }.bind(this)
-        );
+          });
+        });
       }
     } catch (e) {
       Logger.error(e);
@@ -12817,13 +12829,12 @@ var __oj_table_metadata =
     if (this._lastSelectedRowIdxArray != null) {
       this._lastSelectedRowIdxArray = [];
     }
-    // update edit state based on row removals before rows are removed to ensure row context is available
-    if (this._hasEditableRow()) {
-      // exit edit mode if editable row is deleted - treat as cancelling the edit
-      this._setTableEditable(false, true);
-    }
     // reset selection validation state if data refreshed or changed
     this._initialSelectionStateValidated = false;
+
+    // update edit state based on row removals before rows are removed to ensure row context is available
+    // exit edit mode if editable row is deleted - treat as cancelling the edit
+    return this._hasEditableRow() ? this._setTableEditable(false, true) : Promise.resolve();
   };
 
   /**
@@ -12832,24 +12843,33 @@ var __oj_table_metadata =
    * @private
    */
   Table.prototype._removeRowsAfterLastValidRow = function (lastValidKey) {
-    if (this._getCurrentScrollPosition().y > 0) {
-      this._bufferScrollerForLastRow(lastValidKey);
-    }
-    var lastValidItemIdx = this._getRowIdxForRowKey(lastValidKey);
-    if (lastValidItemIdx >= 0) {
-      // update edit state based on row removals before rows are removed to ensure row context is available
-      if (this._hasEditableRow()) {
-        if (this._getEditableRowIdx() > lastValidItemIdx) {
+    return new Promise(
+      function (resolve) {
+        if (this._getCurrentScrollPosition().y > 0) {
+          this._bufferScrollerForLastRow(lastValidKey);
+        }
+        var lastValidItemIdx = this._getRowIdxForRowKey(lastValidKey);
+        if (lastValidItemIdx >= 0) {
+          // update edit state based on row removals before rows are removed to ensure row context is available
           // exit edit mode if editable row is deleted - treat as cancelling the edit
-          this._setTableEditable(false, true);
+          var updateEditable =
+            this._hasEditableRow() && this._getEditableRowIdx() > lastValidItemIdx
+              ? this._setTableEditable(false, true)
+              : Promise.resolve();
+
+          updateEditable.then(() => {
+            var tableBodyRows = this._getTableBodyRows();
+            var tableBodyRowsCount = tableBodyRows.length;
+            for (var i = tableBodyRowsCount - 1; i > lastValidItemIdx; i--) {
+              this._removeTableBodyRow(i);
+            }
+            resolve();
+          });
+        } else {
+          resolve();
         }
-      }
-      var tableBodyRows = this._getTableBodyRows();
-      var tableBodyRowsCount = tableBodyRows.length;
-      for (var i = tableBodyRowsCount - 1; i > lastValidItemIdx; i--) {
-        this._removeTableBodyRow(i);
-      }
-    }
+      }.bind(this)
+    );
   };
 
   /**
@@ -12886,14 +12906,15 @@ var __oj_table_metadata =
    * @private
    */
   Table.prototype._handleDataRowMutate = function (event) {
-    if (this._dataFetching) {
-      if (this._controller) {
-        this._controller.abort(DataCollectionUtils.getAbortReason(this.OuterWrapper));
+    try {
+      // if a refresh is already in the queue, but a fetch hasn't started, just return
+      if (this._hasRefreshInQueue) {
+        return;
+      }
+      if (this._dataFetching) {
+        this._handleStaleDataMutation(this._pendingFetchOptions);
+        return;
       }
-      this._pendingFetchStale = true;
-      return;
-    }
-    try{
       var self = this;
       this._queueTask(() => {
         return self._handleDataRowRemove(event.detail.remove, event.detail.add).then((resetFocus) => {
@@ -12991,15 +13012,6 @@ var __oj_table_metadata =
         }
       }
 
-      // update edit state based on row removals before rows are removed to ensure row context is available
-      if (self._hasEditableRow()) {
-        var editRowKey = self._getEditableRowKey();
-        if (DataCollectionUtils.containsKey([...eventDetail[Table._CONST_KEYS]], editRowKey)) {
-          // exit edit mode if editable row is deleted - treat as cancelling the edit
-          self._setTableEditable(false, true);
-        }
-      }
-
       // eslint-disable-next-line no-inner-declarations
       function _removeRows() {
         var ii;
@@ -13081,26 +13093,33 @@ var __oj_table_metadata =
         return Promise.resolve();
       }
 
-      if (removeAll) {
-        self._removeAllTableBodyRows();
-        return _afterRemoveRows().then(function () {
-          return Promise.resolve(resetFocus);
-        });
-      }
       return new Promise(function (resolve) {
-        if (self._IsCustomElement()) {
-          return self
-            ._animateVisibleRows(removedTableBodyRows, rowIdxArray, 'remove')
-            .then(function () {
+        // update edit state based on row removals before rows are removed to ensure row context is available
+        // exit edit mode if editable row is deleted - treat as cancelling the edit
+        var updateEditable =
+          self._hasEditableRow() &&
+          DataCollectionUtils.containsKey(eventDetailKeys, self._getEditableRowKey())
+            ? self._setTableEditable(false, true)
+            : Promise.resolve();
+
+        updateEditable.then(() => {
+          if (removeAll) {
+            self._removeAllTableBodyRows();
+            _afterRemoveRows().then(() => {
+              resolve(resetFocus);
+            });
+          } else {
+            var handleAnimation = self._IsCustomElement()
+              ? self._animateVisibleRows(removedTableBodyRows, rowIdxArray, 'remove')
+              : Promise.resolve();
+
+            handleAnimation.then(() => {
               _removeRows();
-              return _afterRemoveRows().then(function () {
+              _afterRemoveRows().then(() => {
                 resolve(resetFocus);
               });
             });
-        }
-        _removeRows();
-        return _afterRemoveRows().then(function () {
-          resolve(resetFocus);
+          }
         });
       });
     }
@@ -17501,7 +17520,9 @@ var __oj_table_metadata =
         if (isClearEditMode) {
           if (this._focusoutEditKey != null) {
             // only clear edit mode if it was enabled prior to the focusout trigger AND the editRow has not changed
-            this._setTableEditable(false, false, 0, true, event);
+            this._queueTask(() => {
+              return this._setTableEditable(false, false, 0, true, event);
+            });
           } else {
             // otherwise ensure focus remains in the Table if it has not yet been regained and an editrow update is pending
             table.focus();
@@ -17702,7 +17723,7 @@ var __oj_table_metadata =
       var keyboardCode1 = this._getKeyboardKeys()[0];
 
       if (
-        this._isEditPending ||
+        this._isPendingEdit() ||
         (!DataCollectionUtils.isEscapeKeyEvent(keyboardCode1) &&
           !DataCollectionUtils.isEnterKeyEvent(keyboardCode1) &&
           !DataCollectionUtils.isF2KeyEvent(keyboardCode1) &&
@@ -17811,7 +17832,7 @@ var __oj_table_metadata =
       if (
         DataCollectionUtils.isEventClickthroughDisabled(event, this._getTable()) ||
         (DataCollectionUtils.isFromDefaultSelector(event) && !isShift) ||
-        this._isEditPending
+        this._isPendingEdit()
       ) {
         return;
       }
@@ -17897,7 +17918,7 @@ var __oj_table_metadata =
     'mousedown .oj-table-column-header-cell': function (event) {
       if (
         DataCollectionUtils.isEventClickthroughDisabled(event, this._getTable()) ||
-        this._isEditPending
+        this._isPendingEdit()
       ) {
         return;
       }
@@ -17914,7 +17935,7 @@ var __oj_table_metadata =
     'mousedown .oj-table-footer-cell': function (event) {
       if (
         DataCollectionUtils.isEventClickthroughDisabled(event, this._getTable()) ||
-        this._isEditPending
+        this._isPendingEdit()
       ) {
         return;
       }
@@ -17982,7 +18003,7 @@ var __oj_table_metadata =
     'mousedown .oj-table-data-cell': function (event) {
       if (
         DataCollectionUtils.isEventClickthroughDisabled(event, this._getTable()) ||
-        this._isEditPending
+        this._isPendingEdit()
       ) {
         return;
       }
@@ -18011,7 +18032,7 @@ var __oj_table_metadata =
      * invoke a sort on the column data when the mouse clicks the sort icon
      */
     'click .oj-table-sort-icon-container': function (event) {
-      if (this._isEditPending) {
+      if (this._isPendingEdit()) {
         return;
       }
       var columnIdx = this._getElementColumnIdx(event.target);
@@ -18040,7 +18061,7 @@ var __oj_table_metadata =
     'click .oj-table-body-row': function (event) {
       if (
         DataCollectionUtils.isEventClickthroughDisabled(event, this._getTable()) ||
-        this._isEditPending ||
+        this._isPendingEdit() ||
         !this._shouldAllowSelection()
       ) {
         return;
@@ -18145,7 +18166,13 @@ var __oj_table_metadata =
             columnIdx = this._getElementColumnIdx(event.target);
           }
           const rowIdx = this._getElementRowIdx(event.target);
-          this._setEditRow({ rowIndex: rowIdx }, columnIdx);
+          if (rowIdx != null) {
+            const rowKey = this._getRowKeyForRowIdx(rowIdx);
+            this._queueTask(() => {
+              return this._setEditRow();
+            });
+            this._deferredEditInfo = { editRow: { rowKey }, columnIdx };
+          }
         }
         this._currentPointerTarget = null;
         this._previousPointerTarget = null;
@@ -18157,7 +18184,7 @@ var __oj_table_metadata =
     'contextmenu .oj-table-data-cell': function (event) {
       if (
         DataCollectionUtils.isEventClickthroughDisabled(event, this._getTable()) ||
-        this._isEditPending
+        this._isPendingEdit()
       ) {
         return;
       }
@@ -18173,7 +18200,7 @@ var __oj_table_metadata =
     'click .oj-table-column-header-cell': function (event) {
       if (
         DataCollectionUtils.isEventClickthroughDisabled(event, this._getTable()) ||
-        this._isEditPending ||
+        this._isPendingEdit() ||
         !this._shouldAllowSelection()
       ) {
         return;
@@ -18222,7 +18249,7 @@ var __oj_table_metadata =
     'click .oj-table-footer-cell': function (event) {
       if (
         DataCollectionUtils.isEventClickthroughDisabled(event, this._getTable()) ||
-        this._isEditPending ||
+        this._isPendingEdit() ||
         !this._shouldAllowSelection()
       ) {
         return;
@@ -18267,7 +18294,7 @@ var __oj_table_metadata =
     'click .oj-table-add-row-placeholder': function (event) {
       if (
         DataCollectionUtils.isEventClickthroughDisabled(event, this._getTable()) ||
-        this._isEditPending
+        this._isPendingEdit()
       ) {
         return;
       }
@@ -18278,7 +18305,7 @@ var __oj_table_metadata =
      * Set dragstart handler for column DnD.
      */
     'dragstart .oj-table-column-header-cell': function (event) {
-      if (this._isEditPending) {
+      if (this._isPendingEdit()) {
         return undefined;
       }
       var eventTarget = this._getEventTargetElement(event);
@@ -18327,7 +18354,7 @@ var __oj_table_metadata =
      * handle the dragstart event on rows and invoke event callback.
      */
     'dragstart .oj-table-body-row': function (event) {
-      if (this._isEditPending) {
+      if (this._isPendingEdit()) {
         return undefined;
       }
       return this._getTableDndContext().handleRowDragStart(event);
@@ -18533,7 +18560,9 @@ var __oj_table_metadata =
             return function () {
               var eventTarget = this._getEventTargetElement(event);
               var columnIdx = this._getElementColumnIdx(eventTarget);
-              this._setTableEditable(true, false, columnIdx, true, event);
+              this._queueTask(() => {
+                return this._setTableEditable(true, false, columnIdx, true, event);
+              });
               event.preventDefault();
             }.bind(this);
           }.bind(this)(evt)
@@ -19411,36 +19440,30 @@ var __oj_table_metadata =
           (updatedCurrentRow == null ||
             !oj$1.KeyUtils.equals(existingCurrentRowKey, updatedCurrentRow.rowKey))
         ) {
-          var updateEditable = this._setTableEditable(false, false, 0, true, event);
-          if (updateEditable instanceof Promise) {
-            return updateEditable.then(
-              function (value) {
-                if (value === false) {
-                  this._resetCurrentRow(existingCurrentRow, existingCurrentRowIdx, event);
-                  return Table._CURRENT_ROW_STATUS._IGNORED;
-                }
-                var existingBodyRow = this._getTableBodyRow(existingCurrentRowIdx);
-                if (existingBodyRow != null) {
-                  existingBodyRow.classList.remove(Table.CSS_CLASSES._TABLE_DATA_CURRENT_ROW_CLASS);
-                }
-                this._currentRow = updatedCurrentRow;
-
-                this._updateCurrentRow(isClearCurrentRow, localRowIndex, event);
-                if (event == null) {
-                  // only scroll to the new current row and highlight it if the Table has focus
-                  this._setActiveRow(localRowIndex, event, false, !this._hasFocus());
-                }
-                return Table._CURRENT_ROW_STATUS._UPDATED;
-              }.bind(this),
-              function () {
+          return this._setTableEditable(false, false, 0, true, event).then(
+            (value) => {
+              if (value === false) {
                 this._resetCurrentRow(existingCurrentRow, existingCurrentRowIdx, event);
-                return Table._CURRENT_ROW_STATUS._VETOED;
-              }.bind(this)
-            );
-          } else if (updateEditable === false) {
-            this._resetCurrentRow(existingCurrentRow, existingCurrentRowIdx, event);
-            return Table._CURRENT_ROW_STATUS._VETOED;
-          }
+                return Table._CURRENT_ROW_STATUS._IGNORED;
+              }
+              var existingBodyRow = this._getTableBodyRow(existingCurrentRowIdx);
+              if (existingBodyRow != null) {
+                existingBodyRow.classList.remove(Table.CSS_CLASSES._TABLE_DATA_CURRENT_ROW_CLASS);
+              }
+              this._currentRow = updatedCurrentRow;
+
+              this._updateCurrentRow(isClearCurrentRow, localRowIndex, event);
+              if (event == null) {
+                // only scroll to the new current row and highlight it if the Table has focus
+                this._setActiveRow(localRowIndex, event, false, !this._hasFocus());
+              }
+              return Table._CURRENT_ROW_STATUS._UPDATED;
+            },
+            () => {
+              this._resetCurrentRow(existingCurrentRow, existingCurrentRowIdx, event);
+              return Table._CURRENT_ROW_STATUS._VETOED;
+            }
+          );
         }
         var existingBodyRow = this._getTableBodyRow(existingCurrentRowIdx);
         if (existingBodyRow != null) {
@@ -19754,16 +19777,19 @@ var __oj_table_metadata =
         // update the current row if we are making a row active
         if (active.type === Table.ACTIVE_ELEMENT_TYPES._DATA_ROW) {
           if (updateCurrent) {
-            const updated = this._setCurrentRow({ rowKey: active.key }, event);
-            if (updated instanceof Promise) {
-              return updated.then((value) => {
-                if (value !== Table._CURRENT_ROW_STATUS._UPDATED) {
-                  return false;
-                }
-                this._updateActive(true, active, skipScroll, skipFocus, scrollStuckRow);
-                return true;
+            const currentStatus = this._setCurrentRow({ rowKey: active.key }, event);
+            if (currentStatus instanceof Promise) {
+              this._queueTask(() => {
+                return currentStatus.then((value) => {
+                  if (value !== Table._CURRENT_ROW_STATUS._UPDATED) {
+                    return false;
+                  }
+                  this._updateActive(true, active, skipScroll, skipFocus, scrollStuckRow);
+                  return true;
+                });
               });
-            } else if (updated !== Table._CURRENT_ROW_STATUS._UPDATED) {
+              return true;
+            } else if (currentStatus !== Table._CURRENT_ROW_STATUS._UPDATED) {
               return false;
             }
           }
@@ -19778,17 +19804,20 @@ var __oj_table_metadata =
       // update the current row if we are clearing an active row
       if (this._hasActiveRow()) {
         if (updateCurrent) {
-          const updated = this._setCurrentRow(null, event);
-          if (updated instanceof Promise) {
-            return updated.then((value) => {
-              if (value !== Table._CURRENT_ROW_STATUS._UPDATED) {
-                return false;
-              }
-              this._unhighlightActive();
-              this._active = null;
-              return true;
+          const currentStatus = this._setCurrentRow(null, event);
+          if (currentStatus instanceof Promise) {
+            this._queueTask(() => {
+              return currentStatus.then((value) => {
+                if (value !== Table._CURRENT_ROW_STATUS._UPDATED) {
+                  return false;
+                }
+                this._unhighlightActive();
+                this._active = null;
+                return true;
+              });
             });
-          } else if (updated !== Table._CURRENT_ROW_STATUS._UPDATED) {
+            return true;
+          } else if (currentStatus !== Table._CURRENT_ROW_STATUS._UPDATED) {
             return false;
           }
         }
@@ -19980,9 +20009,10 @@ var __oj_table_metadata =
       }
     }
     if (this._scrollPosition != null) {
-      var scrollPosRowKey = this._scrollPosition.rowKey != null
-        ? this._scrollPosition.rowKey
-        : this._getRowKeyForRowIdx(this._scrollPosition.rowIndex);
+      var scrollPosRowKey =
+        this._scrollPosition.rowKey != null
+          ? this._scrollPosition.rowKey
+          : this._getRowKeyForRowIdx(this._scrollPosition.rowIndex);
       if (scrollPosRowKey != null) {
         eventDetail[Table._CONST_KEYS].forEach((key) => {
           if (oj$1.KeyUtils.equals(key, scrollPosRowKey)) {
@@ -20810,69 +20840,134 @@ var __oj_table_metadata =
    * Sets a row to be editable
    * @private
    */
-  Table.prototype._setEditRow = function (editRow, columnIdx = 0) {
-    if (!this._isTableEditMode()) {
-      return;
-    }
-    if (this._isEditPending !== true) {
-      this._delayed_edit = null;
-      var rowKey = editRow.rowKey;
-      var rowIndex = editRow.rowIndex;
-      if (rowKey != null || rowIndex > -1) {
-        // an editable row has to be current also
-        var changed = this._setCurrentRow({ rowKey: rowKey, rowIndex: rowIndex });
-        if (changed instanceof Promise) {
-          changed.then(
-            function (value) {
-              if (value === Table._CURRENT_ROW_STATUS._UPDATED) {
-                this._setEditableHelper(columnIdx);
-              } else if (value === Table._CURRENT_ROW_STATUS._IGNORED) {
-                this._scrollToRowHelper(editRow, columnIdx);
+  Table.prototype._setEditRow = function () {
+    return new Promise(
+      function (resolve) {
+        if (!this._isTableEditMode()) {
+          resolve();
+        }
+        this._isEditPending = true;
+
+        if (this._deferredEditInfo != null) {
+          var editRow = this._deferredEditInfo.editRow;
+          var columnIdx = this._deferredEditInfo.columnIdx || 0;
+          this._deferredEditInfo = null;
+
+          var rowKey = editRow.rowKey;
+          var rowIndex = editRow.rowIndex;
+          if (rowKey != null || rowIndex > -1) {
+            // an editable row has to be current also
+            var currentStatus = this._setCurrentRow({ rowKey: rowKey, rowIndex: rowIndex });
+            if (currentStatus instanceof Promise) {
+              currentStatus.then(
+                function (value) {
+                  if (value === Table._CURRENT_ROW_STATUS._UPDATED) {
+                    this._setTableEditable(true, false, columnIdx, true, null).then(
+                      () => {
+                        this._isEditPending = false;
+                        resolve();
+                      },
+                      () => {
+                        this._isEditPending = false;
+                        resolve();
+                      }
+                    );
+                  } else if (value === Table._CURRENT_ROW_STATUS._IGNORED) {
+                    this._scrollToRowHelper(editRow, columnIdx, resolve);
+                  }
+                }.bind(this)
+              );
+            } else if (currentStatus === Table._CURRENT_ROW_STATUS._UPDATED) {
+              this._setTableEditable(true, false, columnIdx, true, null).then(
+                () => {
+                  this._isEditPending = false;
+                  resolve();
+                },
+                () => {
+                  this._isEditPending = false;
+                  resolve();
+                }
+              );
+            } else if (currentStatus === Table._CURRENT_ROW_STATUS._IGNORED) {
+              this._scrollToRowHelper(editRow, columnIdx, resolve);
+            }
+          } else {
+            this._setTableEditable(false, false, 0, true, null).then(
+              () => {
+                this._isEditPending = false;
+                resolve();
+              },
+              () => {
+                this._isEditPending = false;
+                resolve();
               }
-            }.bind(this)
-          );
-        } else if (changed === Table._CURRENT_ROW_STATUS._UPDATED) {
-          this._setEditableHelper(columnIdx);
-        } else if (changed === Table._CURRENT_ROW_STATUS._IGNORED) {
-          this._scrollToRowHelper(editRow, columnIdx);
+            );
+          }
+        } else {
+          this._isEditPending = false;
+          resolve();
         }
-      } else {
-        this._setTableEditable(false, false, 0, true, null);
-      }
-    } else {
-      this._delayed_edit = { editRow: editRow, columnIdx: columnIdx };
-    }
+      }.bind(this)
+    );
   };
 
-  /**
-   * Helper method to set table editable
-   * @private
-   */
-  Table.prototype._setEditableHelper = function (columnIdx) {
-    if (this._delayed_edit != null) {
-      return;
-    }
-    this._setTableEditable(true, false, columnIdx, true, null);
+  Table.prototype._isPendingEdit = function () {
+    return this._isEditPending || this._deferredEditInfo != null;
   };
 
   /**
    * Helper method to scroll to row and invoke _setEditRow
    * @private
    */
-  Table.prototype._scrollToRowHelper = function (editRow, columnIdx) {
+  Table.prototype._scrollToRowHelper = function (editRow, columnIdx, resolve) {
     // try to scroll to editRow.rowKey
-    const _scrollTop = this._scrollTop;
-    if (!this._syncScrollPosition({ rowKey: editRow.rowKey, rowIndex: editRow.rowIndex })) {
-      // this will be invoked after scroll and fetch is done
-      this._editRowCallback = function () {
-        // try again after scroll and fetch
-        if (this._scrollTop !== _scrollTop) {
-          this._setEditRow(editRow, columnIdx);
-        }
-      }.bind(this);
+    const scrollTop = this._scrollTop;
+    if (editRow.rowKey != null) {
+      var validationPromise = this._validateKeyForScroll(editRow.rowKey);
+      if (validationPromise != null) {
+        validationPromise.then(
+          (valid) => {
+            if (valid) {
+              this._setupEditRowCallback(editRow, columnIdx, scrollTop);
+              this._syncScrollPosition({ rowKey: editRow.rowKey }, true);
+            }
+            this._isEditPending = false;
+            resolve();
+          },
+          () => {
+            this._isEditPending = false;
+            resolve();
+          }
+        );
+      } else {
+        this._isEditPending = false;
+        resolve();
+      }
+    } else {
+      if (!this._syncScrollPosition({ rowIndex: editRow.rowIndex })) {
+        this._setupEditRowCallback(editRow, columnIdx, scrollTop);
+      }
+      this._isEditPending = false;
+      resolve();
     }
   };
 
+  /**
+   * Helper method to setup a scroll callback for honoring an edit row setting.
+   */
+  Table.prototype._setupEditRowCallback = function (editRow, columnIdx, scrollTop) {
+    // this will be invoked after scroll and fetch is done
+    this._editRowCallback = function () {
+      // try again after scroll and fetch
+      if (this._scrollTop !== scrollTop && this._deferredEditInfo == null) {
+        this._queueTask(() => {
+          return this._setEditRow();
+        });
+        this._deferredEditInfo = { editRow, columnIdx };
+      }
+    }.bind(this);
+  };
+
   /**
    * Returns the row from editRowObject
    * @private
@@ -20905,29 +21000,17 @@ var __oj_table_metadata =
    */
   Table.prototype._setAdjacentRowEditable = function (columnIdx, origColumnIdx, isNext, event) {
     var editableRowIdx = this._getEditableRowIdx();
-    var updateEditable = this._setTableEditable(false, false, columnIdx, isNext, event);
-    if (updateEditable instanceof Promise) {
-      updateEditable.then(
-        function () {
-          this._queueTask(
-            function () {
-              this._handleAdjacentEditEndSuccessful(columnIdx, editableRowIdx, isNext, event);
-            }.bind(this)
-          );
-        }.bind(this),
-        function () {
-          this._queueTask(
-            function () {
-              this._handleEditEndRejected(origColumnIdx, editableRowIdx, !isNext);
-            }.bind(this)
-          );
-        }.bind(this)
-      );
-    } else if (updateEditable === false) {
-      this._handleEditEndRejected(origColumnIdx, editableRowIdx, !isNext);
-    } else {
-      this._handleAdjacentEditEndSuccessful(columnIdx, editableRowIdx, isNext, event);
-    }
+    return this._setTableEditable(false, false, columnIdx, isNext, event).then(
+      (updateEditable) => {
+        if (updateEditable === false) {
+          return this._handleEditEndRejected(origColumnIdx, editableRowIdx, !isNext);
+        }
+        return this._handleAdjacentEditEndSuccessful(columnIdx, editableRowIdx, isNext, event);
+      },
+      () => {
+        return this._handleEditEndRejected(origColumnIdx, editableRowIdx, !isNext);
+      }
+    );
   };
 
   /**
@@ -20939,10 +21022,10 @@ var __oj_table_metadata =
     var newRowIdx = this._findAdjacentEditableRow(rowIdx, tableBodyRows, isNext);
     if (newRowIdx !== null) {
       this._setActiveRow(newRowIdx, event, true, false, true);
-      this._setTableEditable(true, false, columnIdx, isNext, event);
-    } else {
-      this._getTable().focus();
+      return this._setTableEditable(true, false, columnIdx, isNext, event);
     }
+    this._getTable().focus();
+    return Promise.resolve();
   };
 
   /**
@@ -21005,240 +21088,284 @@ var __oj_table_metadata =
     forwardSearch,
     event
   ) {
-    if (!this._isTableEditMode() || this._isEditPending) {
-      return undefined;
-    }
-    var isLegacyDataSource = this._getData() instanceof oj$1.TableDataSourceAdapter;
-    var currentRow = this._getCurrentRow();
-    if (currentRow != null) {
-      const rowElementToBeEdited = this._getRowToBeEdited(currentRow);
-      if (
-        rowElementToBeEdited &&
-        rowElementToBeEdited[Table._DATA_OJ_EDITABLE] !== Table._CONST_OFF
-      ) {
-        var rowKey = currentRow.rowKey;
-        var rowIdx = this._getRowIdxForRowKey(rowKey);
-        var tableBodyRow;
-        var rowContext;
-        var updateEditMode;
-        var newEditRowValue;
-        var editAcceptPromiseArray = [];
-        let updatedItemPromise;
-
-        // save the old editable row info
-        var prevEditableRowKey = this._getEditableRowKey();
+    return new Promise(
+      function (resolve) {
+        if (!this._isTableEditMode()) {
+          resolve();
+          return;
+        }
 
-        try {
-          if (editable && !this._hasEditableRow()) {
-            // fire the beforeEdit event if there are no existing editable rows
-            // and we are starting edit on a row
-            tableBodyRow = this._getTableBodyRow(rowIdx);
-            rowContext = this._getRendererContextObject(tableBodyRow, {
-              row: {
-                key: rowKey,
-                index: currentRow.rowIndex
-              },
-              isCurrentRow: true
-            });
-            rowContext.item = tableBodyRow[Table._ROW_ITEM_EXPANDO];
-            updateEditMode = this._trigger('beforeRowEdit', event, {
-              accept: function (acceptPromise) {
-                editAcceptPromiseArray.push(acceptPromise);
-              },
-              rowContext: rowContext
-            });
-            if (updateEditMode) {
-              newEditRowValue = { rowKey: rowKey, rowIndex: rowIdx };
+        var isLegacyDataSource = this._getData() instanceof oj$1.TableDataSourceAdapter;
+        var currentRow = this._getCurrentRow();
+        if (currentRow != null) {
+          this._isEditPending = true;
+          const rowElementToBeEdited = this._getRowToBeEdited(currentRow);
+          if (
+            rowElementToBeEdited &&
+            rowElementToBeEdited[Table._DATA_OJ_EDITABLE] !== Table._CONST_OFF
+          ) {
+            var rowKey = currentRow.rowKey;
+            var rowIdx = this._getRowIdxForRowKey(rowKey);
+            var tableBodyRow;
+            var rowContext;
+            var updateEditMode;
+            var newEditRowValue;
+            var editAcceptPromiseArray = [];
+            let updatedItemPromise;
+
+            // save the old editable row info
+            var prevEditableRowKey = this._getEditableRowKey();
+
+            try {
+              if (editable && !this._hasEditableRow()) {
+                // fire the beforeEdit event if there are no existing editable rows
+                // and we are starting edit on a row
+                tableBodyRow = this._getTableBodyRow(rowIdx);
+                rowContext = this._getRendererContextObject(tableBodyRow, {
+                  row: {
+                    key: rowKey,
+                    index: currentRow.rowIndex
+                  },
+                  isCurrentRow: true
+                });
+                rowContext.item = tableBodyRow[Table._ROW_ITEM_EXPANDO];
+                updateEditMode = this._trigger('beforeRowEdit', event, {
+                  accept: function (acceptPromise) {
+                    editAcceptPromiseArray.push(acceptPromise);
+                  },
+                  rowContext: rowContext
+                });
+                if (updateEditMode) {
+                  newEditRowValue = { rowKey: rowKey, rowIndex: rowIdx };
+                }
+              } else if (!editable && this._hasEditableRow()) {
+                // only trigger the beforeRowEditEnd if we are actually ending an edit
+                // fire on the edited row
+                tableBodyRow = this._getTableBodyRow(this._getEditableRowIdx());
+                rowKey = this._getRowKeyForRowIdx(this._getEditableRowIdx());
+                rowContext = this._getRendererContextObject(tableBodyRow, {
+                  row: {
+                    key: rowKey,
+                    index: this._getDataSourceRowIndexForRowKey(rowKey)
+                  },
+                  isCurrentRow: true
+                });
+                rowContext.item = tableBodyRow[Table._ROW_ITEM_EXPANDO];
+                updateEditMode = this._trigger('beforeRowEditEnd', event, {
+                  accept: function (acceptPromise) {
+                    editAcceptPromiseArray.push(acceptPromise);
+                  },
+                  setUpdatedItem: function (updatedPromise) {
+                    updatedItemPromise = updatedPromise;
+                  },
+                  cancelEdit: cancelled,
+                  rowContext: rowContext
+                });
+
+                if (updateEditMode) {
+                  newEditRowValue = { rowKey: null, rowIndex: -1 };
+                }
+              } else {
+                // No updates so just exit
+                this._isEditPending = false;
+                resolve();
+                return;
+              }
+            } catch (err) {
+              this._isEditPending = false;
+              resolve(false);
+              return;
             }
-          } else if (!editable && this._hasEditableRow()) {
-            // only trigger the beforeRowEditEnd if we are actually ending an edit
-            // fire on the edited row
-            tableBodyRow = this._getTableBodyRow(this._getEditableRowIdx());
-            rowKey = this._getRowKeyForRowIdx(this._getEditableRowIdx());
-            rowContext = this._getRendererContextObject(tableBodyRow, {
-              row: {
-                key: rowKey,
-                index: this._getDataSourceRowIndexForRowKey(rowKey)
-              },
-              isCurrentRow: true
-            });
-            rowContext.item = tableBodyRow[Table._ROW_ITEM_EXPANDO];
-            updateEditMode = this._trigger('beforeRowEditEnd', event, {
-              accept: function (acceptPromise) {
-                editAcceptPromiseArray.push(acceptPromise);
-              },
-              setUpdatedItem: function (updatedPromise) {
-                updatedItemPromise = updatedPromise;
-              },
-              cancelEdit: cancelled,
-              rowContext: rowContext
-            });
+            if (!updateEditMode) {
+              this._syncActiveElement();
+              this._isEditPending = false;
+              resolve(false);
+              return;
+            }
+            var table = this._getTable();
 
-            if (updateEditMode) {
-              newEditRowValue = { rowKey: null, rowIndex: -1 };
+            // shift focus to the overall table if the target row contains focus - otherwise, the overall
+            // focusout handler may kick in, or tooltips or other ihteractive elements may cause issues
+            if (tableBodyRow.contains(document.activeElement)) {
+              // flag used to skip scrolling 'focused' things while the Table
+              // is actively updating focus due to editing requirements
+              this._isSkipScrollOnFocus = !cancelled;
+              table.focus();
             }
-          } else {
-            // No updates so just exit
-            return undefined;
-          }
-        } catch (err) {
-          return false;
-        }
-        if (!updateEditMode) {
-          this._syncActiveElement();
-          return false;
-        }
-        var table = this._getTable();
 
-        // shift focus to the overall table if the target row contains focus - otherwise, the overall
-        // focusout handler may kick in, or tooltips or other ihteractive elements may cause issues
-        if (tableBodyRow.contains(document.activeElement)) {
-          // flag used to skip scrolling 'focused' things while the Table
-          // is actively updating focus due to editing requirements
-          this._isSkipScrollOnFocus = !cancelled;
-          table.focus();
-        }
+            var _handleAsyncEdit = function (updatedItemObj) {
+              this._removeSkeletonRow(tableBodyRow);
+              // recalculate the row index as it could have changed due to other tasks
+              var editableRowIndex = this._getRowIdxForRowKey(rowKey);
+              if (editable && rowIdx !== editableRowIndex) {
+                newEditRowValue.rowIndex = editableRowIndex;
+              }
+              // update editRow option
+              this._fireEditRowChangeEvent(newEditRowValue, event);
 
-        var _handleAsyncEdit = function (updatedItemObj) {
-          this._removeSkeletonRow(tableBodyRow);
-          // recalculate the row index as it could have changed due to other tasks
-          var editableRowIndex = this._getRowIdxForRowKey(rowKey);
-          if (editable && rowIdx !== editableRowIndex) {
-            newEditRowValue.rowIndex = editableRowIndex;
-          }
-          // update editRow option
-          this._fireEditRowChangeEvent(newEditRowValue, event);
-
-          if (editable) {
-            // set the editable row index
-            this._setTableActionableMode(false, true);
-            this._setEditableRowIdx(editableRowIndex);
-            table.setAttribute(Table.DOM_ATTR._ARIA_LABELLEDBY, ''); // @HTMLUpdateOK
-            // re-render the newly editable row
-            return this._refreshRow(editableRowIndex, false, !isLegacyDataSource).then(
-              function () {
-                this._setCellInRowFocus(editableRowIndex, columnIdx, forwardSearch);
-                // clear out the old editable row
-                return this._refreshPrevEditableRow(prevEditableRowKey, updatedItemObj, false);
-              }.bind(this)
-            );
-          }
-          this._focusEditCell = null;
-          this._setEditableRowIdx(null);
+              if (editable) {
+                // set the editable row index
+                this._setTableActionableMode(false, true);
+                this._setEditableRowIdx(editableRowIndex);
+                table.setAttribute(Table.DOM_ATTR._ARIA_LABELLEDBY, ''); // @HTMLUpdateOK
+                // re-render the newly editable row
+                return this._refreshRow(editableRowIndex, false, !isLegacyDataSource).then(() => {
+                  this._setCellInRowFocus(editableRowIndex, columnIdx, forwardSearch);
+                  // clear out the old editable row
+                  return this._refreshPrevEditableRow(prevEditableRowKey, updatedItemObj, false);
+                });
+              }
+              this._focusEditCell = null;
+              this._setEditableRowIdx(null);
 
-          // clear out the old editable row
-          return this._refreshPrevEditableRow(prevEditableRowKey, updatedItemObj, cancelled);
-        }.bind(this);
+              // clear out the old editable row
+              return this._refreshPrevEditableRow(prevEditableRowKey, updatedItemObj, cancelled);
+            }.bind(this);
 
-        // handle asynchronous editing if an accept promise was provided
-        if (editAcceptPromiseArray.length !== 0) {
-          this._prepareAsyncRowEdit(tableBodyRow);
-          return this._queueTask(
-            function () {
-              this._editPromise = Promise.all(editAcceptPromiseArray).then(
-                function () {
+            // handle asynchronous editing if an accept promise was provided
+            if (editAcceptPromiseArray.length !== 0) {
+              this._prepareAsyncRowEdit(tableBodyRow);
+              Promise.all(editAcceptPromiseArray).then(
+                () => {
                   if (updatedItemPromise != null) {
-                    return updatedItemPromise.then(
-                      function (updatedItemObj) {
-                        return _handleAsyncEdit(updatedItemObj);
+                    updatedItemPromise.then(
+                      (updatedItemObj) => {
+                        _handleAsyncEdit(updatedItemObj).then(
+                          () => {
+                            this._isEditPending = false;
+                            resolve();
+                          },
+                          () => {
+                            this._isEditPending = false;
+                            resolve();
+                          }
+                        );
+                      },
+                      () => {
+                        _handleAsyncEdit().then(
+                          () => {
+                            this._isEditPending = false;
+                            resolve();
+                          },
+                          () => {
+                            this._isEditPending = false;
+                            resolve();
+                          }
+                        );
+                      }
+                    );
+                  } else {
+                    _handleAsyncEdit().then(
+                      () => {
+                        this._isEditPending = false;
+                        resolve();
                       },
-                      function () {
-                        return _handleAsyncEdit();
+                      () => {
+                        this._isEditPending = false;
+                        resolve();
                       }
                     );
                   }
-                  return _handleAsyncEdit();
                 },
-                function () {
+                () => {
                   this._removeSkeletonRow(tableBodyRow);
                   if (editable && prevEditableRowKey == null) {
                     this._syncActiveElement();
                   }
-                  this._clearEditPending();
-                  return false;
-                }.bind(this)
+                  this._isSkipScrollOnFocus = false;
+                  this._isEditPending = false;
+                  resolve(false);
+                }
               );
-              return this._editPromise;
-            }.bind(this)
-          );
-        }
+              return;
+            }
 
-        // check if an updated item promise was provided for otherwise synchronous editing
-        if (updatedItemPromise != null) {
-          this._prepareAsyncRowEdit(tableBodyRow);
-          return this._queueTask(
-            function () {
-              this._editPromise = updatedItemPromise.then(
-                function (updatedItemObj) {
-                  return _handleAsyncEdit(updatedItemObj);
+            // check if an updated item promise was provided for otherwise synchronous editing
+            if (updatedItemPromise != null) {
+              this._prepareAsyncRowEdit(tableBodyRow);
+              updatedItemPromise.then(
+                (updatedItemObj) => {
+                  _handleAsyncEdit(updatedItemObj).then(
+                    () => {
+                      this._isEditPending = false;
+                      resolve();
+                    },
+                    () => {
+                      this._isEditPending = false;
+                      resolve();
+                    }
+                  );
                 },
-                function () {
-                  return _handleAsyncEdit();
+                () => {
+                  _handleAsyncEdit().then(
+                    () => {
+                      this._isEditPending = false;
+                      resolve();
+                    },
+                    () => {
+                      this._isEditPending = false;
+                      resolve();
+                    }
+                  );
                 }
               );
-              return this._editPromise;
-            }.bind(this)
-          );
-        }
+              return;
+            }
 
-        // otherwise, handle fully synchronous editing
-        if (editable) {
-          // set the editable row index
-          this._setTableActionableMode(false, true);
-          this._setEditableRowIdx(rowIdx);
-          table.setAttribute(Table.DOM_ATTR._ARIA_LABELLEDBY, ''); // @HTMLUpdateOK
-          // re-render the newly editable row
-          this._queueTask(
-            function () {
+            // otherwise, handle fully synchronous editing
+            if (editable) {
+              // set the editable row index
+              this._setTableActionableMode(false, true);
+              this._setEditableRowIdx(rowIdx);
+              table.setAttribute(Table.DOM_ATTR._ARIA_LABELLEDBY, ''); // @HTMLUpdateOK
+              // re-render the newly editable row
               // update editRow option
               this._fireEditRowChangeEvent(newEditRowValue, event);
               // recalculate the row index as it could have changed due to other tasks
               var editableRowIndex = this._getRowIdxForRowKey(rowKey);
-              return this._refreshRow(editableRowIndex, false, !isLegacyDataSource).then(
-                function () {
+              this._refreshRow(editableRowIndex, false, !isLegacyDataSource).then(
+                () => {
                   // set focus on the column in the row
                   this._setCellInRowFocus(editableRowIndex, columnIdx, forwardSearch);
-                }.bind(this)
+                  this._refreshPrevEditableRow(prevEditableRowKey, null, cancelled).then(() => {
+                    this._isEditPending = false;
+                    resolve();
+                  });
+                },
+                () => {
+                  this._isEditPending = false;
+                  resolve();
+                }
               );
-            }.bind(this)
-          );
-        } else {
-          this._focusEditCell = null;
-          this._setEditableRowIdx(null);
-          this._queueTask(
-            function () {
-              // update editRow option
-              this._fireEditRowChangeEvent(newEditRowValue, event);
-            }.bind(this)
-          );
+              return;
+            }
+            this._focusEditCell = null;
+            this._setEditableRowIdx(null);
+            // update editRow option
+            this._fireEditRowChangeEvent(newEditRowValue, event);
+            this._refreshPrevEditableRow(prevEditableRowKey, null, cancelled).then(
+              () => {
+                this._isEditPending = false;
+                resolve();
+              },
+              () => {
+                this._isEditPending = false;
+                resolve();
+              }
+            );
+            return;
+          }
         }
-        this._queueTask(
-          function () {
-            return this._refreshPrevEditableRow(prevEditableRowKey, null, cancelled);
-          }.bind(this)
-        );
-      }
-    }
-    return undefined;
+        resolve();
+      }.bind(this)
+    );
   };
 
   Table.prototype._prepareAsyncRowEdit = function (tableBodyRow) {
-    this._isEditPending = true;
     this._insertSkeletonRow(tableBodyRow);
     this._syncScrollPosition();
   };
 
-  Table.prototype._clearEditPending = function () {
-    this._isEditPending = false;
-    this._isSkipScrollOnFocus = false;
-    this._editPromise = null;
-    if (this._delayed_edit != null) {
-      this._queueTask(() => {
-        this._setEditRow(this._delayed_edit.editRow, this._delayed_edit.columnIdx);
-      });
-    }
-  };
-
   Table.prototype._refreshPrevEditableRow = function (prevEditableRowKey, updatedItemObj, cancelled) {
     // make sure the row still exists before we refresh it
     const prevEditableRowIdx = this._getRowIdxForRowKey(prevEditableRowKey);
@@ -21249,11 +21376,12 @@ var __oj_table_metadata =
       }
       // re-render the previously editable row, which will be read-only now
       return this._refreshRow(prevEditableRowIdx, false, cancelled, rowItem).then(() => {
-        this._clearEditPending();
+        this._skipScrollOnFocus = false;
       });
     }
-    this._clearEditPending();
-    return Promise.resolve();
+    return Promise.resolve().then(() => {
+      this._skipScrollOnFocus = false;
+    });
   };
 
   /**
@@ -21606,13 +21734,17 @@ var __oj_table_metadata =
         rowElementTabIndex === tabbableElementsInRowCount - 1 &&
         !event[Table._KEYBOARD_CODES._MODIFIER_SHIFT]
       ) {
-        // last tabbable element in row so go to the next row
-        this._setAdjacentRowEditable(-1, maxIndex, true, event);
+        this._queueTask(() => {
+          // last tabbable element in row so go to the next row
+          return this._setAdjacentRowEditable(-1, maxIndex, true, event);
+        });
         event.preventDefault();
         event.stopPropagation();
       } else if (rowElementTabIndex === 0 && event[Table._KEYBOARD_CODES._MODIFIER_SHIFT]) {
-        // first tabbable element in row and Shift+Tab so go to the previous row
-        this._setAdjacentRowEditable(maxIndex, 0, false, event);
+        this._queueTask(() => {
+          // first tabbable element in row and Shift+Tab so go to the previous row
+          return this._setAdjacentRowEditable(maxIndex, 0, false, event);
+        });
         event.preventDefault();
         event.stopPropagation();
       }
@@ -21737,17 +21869,14 @@ var __oj_table_metadata =
       var currentRowIdx = currentRow.rowIndex != null ? currentRow.rowIndex : -1;
       if (currentRowIdx >= 0) {
         if (this._isTableEditMode()) {
-          if (!this._hasEditableRow()) {
-            this._setTableEditable(true, false, -1, true, event);
-            return;
-          }
-          var columnIdx = this._getElementColumnIdx(event.target);
-
-          if (!event[Table._KEYBOARD_CODES._MODIFIER_SHIFT]) {
-            this._setAdjacentRowEditable(columnIdx, columnIdx, true, event);
-          } else {
-            this._setAdjacentRowEditable(columnIdx, columnIdx, false, event);
-          }
+          this._queueTask(() => {
+            if (!this._hasEditableRow()) {
+              return this._setTableEditable(true, false, -1, true, event);
+            }
+            var columnIdx = this._getElementColumnIdx(event.target);
+            var isNext = !event[Table._KEYBOARD_CODES._MODIFIER_SHIFT];
+            return this._setAdjacentRowEditable(columnIdx, columnIdx, isNext, event);
+          });
         } else {
           this._fireActionEvent(currentRowIdx, event, false);
           this._setTableActionableMode(true);
@@ -21805,14 +21934,16 @@ var __oj_table_metadata =
     if (!this._isStickyLayoutEnabled()) {
       if (this._isTableEditMode()) {
         // pressing F2 toggles between editable modes.
-        if (!this._hasEditableRow()) {
-          this._setTableEditable(true, false, 0, true, event);
+        this._queueTask(() => {
           if (!this._hasEditableRow()) {
-            this._toggleTableActionableMode();
+            return this._setTableEditable(true, false, 0, true, event).then(() => {
+              if (!this._hasEditableRow()) {
+                this._toggleTableActionableMode();
+              }
+            });
           }
-        } else {
-          this._setTableEditable(false, false, 0, true, event);
-        }
+          return this._setTableEditable(false, false, 0, true, event);
+        });
       } else {
         this._toggleTableActionableMode();
       }
@@ -21831,7 +21962,9 @@ var __oj_table_metadata =
     if (this._hasEditableRow()) {
       event.preventDefault();
       event.stopPropagation();
-      this._setTableEditable(false, true, 0, true, event);
+      this._queueTask(() => {
+        return this._setTableEditable(false, true, 0, true, event);
+      });
     } else if (this._isTableActionableMode()) {
       event.preventDefault();
       event.stopPropagation();
@@ -26626,19 +26759,20 @@ var __oj_table_metadata =
       }
     }
     if (requiresRefresh) {
-      this._queueTask(
-        function () {
-          if (requiresHeaderRefresh) {
-            // clearing the rendered flag triggers a header render as part of _refresh()
-            this._renderedTableHeaderColumns = false;
-          }
-          if (requiresDataRefresh) {
-            this._beforeDataRefresh();
-          }
+      this._queueTask(() => {
+        if (requiresHeaderRefresh) {
+          // clearing the rendered flag triggers a header render as part of _refresh()
+          this._renderedTableHeaderColumns = false;
+        }
+        var beforeRefreshPromise = requiresDataRefresh
+          ? this._beforeDataRefresh()
+          : Promise.resolve();
+
+        return beforeRefreshPromise.then(() => {
           // _refresh clears the layout manager, so do not register 'refresh' update here
           return this._refresh();
-        }.bind(this)
-      );
+        });
+      });
     }
     this._superApply(arguments);
   };
@@ -26652,44 +26786,43 @@ var __oj_table_metadata =
       // update row index/key values in the 'selection' option
       this._syncRangeSelection(value);
       this._superApply(arguments);
-      this._queueTask(
-        function () {
-          // after applying the new selection value, sync the selection state
-          this._selectionSet = true;
-          this._validateInitialSelectionState(true);
-        }.bind(this)
-      );
+      this._queueTask(() => {
+        // after applying the new selection value, sync the selection state
+        this._selectionSet = true;
+        this._validateInitialSelectionState(true);
+      });
     } else if (key === 'selected') {
       this._superApply(arguments);
-      this._queueTask(
-        function () {
-          // after applying the new selected value, sync the selection state
-          this._selectionSet = false;
-          this._validateInitialSelectionState(true);
-        }.bind(this)
-      );
+      this._queueTask(() => {
+        // after applying the new selected value, sync the selection state
+        this._selectionSet = false;
+        this._validateInitialSelectionState(true);
+      });
     } else if (key === 'currentRow') {
       this._superApply(arguments);
-      this._queueTask(
-        function () {
-          this._setCurrentRow(value, null, true);
-        }.bind(this)
-      ).catch(function () {});
+      this._queueTask(() => {
+        var currentStatus = this._setCurrentRow(value, null, true);
+        if (currentStatus instanceof Promise) {
+          return currentStatus;
+        }
+        return Promise.resolve();
+      }).catch(function () {});
     } else if (key === 'scrollPosition') {
       this._superApply(arguments);
-      this._queueTask(
-        function () {
-          // syncScrollPosition will update with proper value
-          this._syncScrollPosition(value);
-        }.bind(this)
-      );
+      this._queueTask(() => {
+        // syncScrollPosition will update with proper value
+        this._syncScrollPosition(value);
+      });
     } else if (key === 'editRow') {
-      if (value != null && (value.rowKey != null || value.rowIndex > -1)) {
-        // clear the pending focusout editkey value if a row is becoming editable
-        this._focusoutEditKey = null;
-      }
-      // setEditRow will update with all props within value
-      this._setEditRow(value);
+      this._queueTask(() => {
+        if (value != null && (value.rowKey != null || value.rowIndex > -1)) {
+          // clear the pending focusout editkey value if a row is becoming editable
+          this._focusoutEditKey = null;
+        }
+        // setEditRow will update with all props within value
+        return this._setEditRow();
+      });
+      this._deferredEditInfo = { editRow: value };
     } else if (key === 'scrollPolicyOptions' && flags != null) {
       this._superApply(arguments);
       if (this._isStickyLayoutEnabled()) {
@@ -26704,18 +26837,14 @@ var __oj_table_metadata =
       }
     } else if (key === 'addRowDisplay') {
       this._superApply(arguments);
-      this._queueTask(
-        function () {
-          return this._refreshAddRowDisplay();
-        }.bind(this)
-      );
+      this._queueTask(() => {
+        return this._refreshAddRowDisplay();
+      });
     } else if (key === 'selectAllControl') {
       this._superApply(arguments);
-      this._queueTask(
-        function () {
-          return this._refreshTableHeader();
-        }.bind(this)
-      );
+      this._queueTask(() => {
+        return this._refreshTableHeader();
+      });
     } else {
       this._superApply(arguments);
     }