@oracle/oraclejet 12.1.4 → 12.1.7

package/dist/css/stable/oj-stable.css

@@ -333,7 +333,7 @@
   visibility: hidden; }
 
 .oj-theme-json {
-  font-family: '{"name":"stable","targetPlatform":"all","behavior":"redwood","jetReleaseVersion":"v12.1.4"}' !important; }
+  font-family: '{"name":"stable","targetPlatform":"all","behavior":"redwood","jetReleaseVersion":"v12.1.7"}' !important; }
 
 .oj-helper-hidden-accessible {
   border: 0;

package/dist/js/libs/oj/debug/ojcore-base.js

@@ -37,12 +37,12 @@ define(['ojs/ojlogger'], function (Logger) { 'use strict';
      * @global
      * @member {string} version JET version numberr
      */
-    version: '12.1.4',
+    version: '12.1.7',
     /**
      * @global
      * @member {string} revision JET source code revision number
      */
-    revision: '2022-07-05_18-07-33',
+    revision: '2022-09-06_20-30-55',
 
     // This function is only meant to be used outside the library, so quoting the name
     // to avoid renaming is appropriate

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

@@ -92,7 +92,23 @@ define(['exports', 'ojs/ojvcomponent', 'preact', 'jquery', 'ojs/ojanimation', 'o
                 PopupService.getInstance().triggerOnDescendents($drawerElement, PopupService.EVENT.POPUP_REFRESH);
             };
             this.getFocusables = () => {
-                return this.rootRef.current.querySelectorAll('button, [href], input, select, textarea, [tabindex]:not([tabindex="-1"]), video');
+                const defaultFocusableElements = [
+                    'button',
+                    '[href]',
+                    'input',
+                    'select',
+                    'textarea',
+                    '[tabindex]',
+                    'video'
+                ];
+                const selectorSuffix = ':not([tabindex="-1"]):not([disabled])';
+                const elementsCount = defaultFocusableElements.length;
+                let safeFocusablesSelector = '';
+                for (let i = 0; i < elementsCount; i++) {
+                    const elSelector = `${defaultFocusableElements[i]}${selectorSuffix}`;
+                    safeFocusablesSelector += i < elementsCount - 1 ? `${elSelector}, ` : `${elSelector}`;
+                }
+                return this.rootRef.current.querySelectorAll(safeFocusablesSelector);
             };
             this.isTargetDescendantOfOwnZorderLayerOrItsNextSiblings = (zorderLayer, target) => {
                 const zorderLayersBuffer = [zorderLayer];

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

@@ -1948,11 +1948,11 @@ define(['exports', 'ojs/ojcore-base', 'jquery', 'hammerjs', 'ojs/ojcontext', 'oj
 
     // use hammer for swipe
     mOptions = {
+      // ensure pinch zoom work properly
+      touchAction: 'pinch-zoom',
       recognizers: [
          [Hammer.Pan, { direction: Hammer.DIRECTION_HORIZONTAL }]
-      ],
-      // ensure native gesture like pinch zoom work properly
-      touchAction: 'auto'
+      ]
      };
 
     // workaround for Hammer with iOS 13 issue, see: https://github.com/hammerjs/hammer.js/issues/1237
@@ -1997,8 +1997,6 @@ define(['exports', 'ojs/ojcore-base', 'jquery', 'hammerjs', 'ojs/ojcontext', 'oj
           default:
         }
 
-        event.preventDefault();
-
         if (direction === null) {
           return;
         }
@@ -2075,7 +2073,6 @@ define(['exports', 'ojs/ojcore-base', 'jquery', 'hammerjs', 'ojs/ojcontext', 'oj
 
           // stop bubbling
         event.stopPropagation();
-        event.preventDefault();
       })
       .on('panend', function (event) {
           // don't do anything if start is vetoed
@@ -2134,7 +2131,6 @@ define(['exports', 'ojs/ojcore-base', 'jquery', 'hammerjs', 'ojs/ojcontext', 'oj
           // restore item position
         wrapper.addClass(OffcanvasUtils.TRANSITION_SELECTOR);
         OffcanvasUtils._setTranslationX(wrapper, 'start', '0px');
-        event.preventDefault();
       });
   };
 

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

@@ -1713,6 +1713,10 @@ var __oj_select_many_metadata =
       widget.options._dataProvider = null;
     },
 
+    isDataProviderWrapped: function (widget) {
+      return widget.options._dataProvider != null;
+    },
+
     //  - need to be able to specify the initial value of select components bound to dprv
     // traversal using depth first search
     // return an oj.Option object if found otherwise return null
@@ -3742,10 +3746,11 @@ var __oj_select_many_metadata =
           this._focus();
         }
 
-        _ComboUtils.addDataProviderEventListeners(opts.ojContext);
-
         // Readonly support
         this.applyReadonlyState();
+
+        // flag to indicate that the OjChoice instance is alive
+        this.isInitialized = true;
       },
 
       /**
@@ -4132,6 +4137,20 @@ var __oj_select_many_metadata =
 
         // Clear all active timers
         this._clearActiveTimers();
+
+        // remove jQuery data
+        this.container.removeData(this._elemNm);
+
+        // Cleanup elements reference
+        this._dropdownPositioningProxyContainer = null;
+        this.container = null;
+        this.dropdown = null;
+        this.results = null;
+        this.search = null;
+        this.selection = null;
+
+        // update the flag to indicate the current instance is destroyed
+        this.isInitialized = false;
       },
 
       /**
@@ -8901,6 +8920,11 @@ var __oj_select_many_metadata =
                 if ((tagName === 'select') && !self.ojContext._HasPlaceholderSet()) {
                   _ComboUtils.fetchFirstBlockFromDataProvider(self.container, opts, 1)
                     .then(function (data) {
+                      // check if the current instance is still alive. If it has been destroyed
+                      // while fetching the data, do not do anything here
+                      if (!self.isInitialized) {
+                        return;
+                      }
                       // since the fetch process is not synchronous, at this point
                       // we need to check if the current value is still null and we should
                       // default it to the first option only if the value is not updated
@@ -12466,12 +12490,8 @@ var __oj_select_many_metadata =
           ojeditablevalue.EditableValueUtils._GetNormalizedAsyncValidatorsFromOption,
 
       _setup: function () {
-        var opts = {};
         var multi = this.multiple;
 
-        opts.element = this.element;
-        opts.ojContext = this;
-
         // fixup valueOption(s) if placeholder is selected
         if (_ComboUtils.isValueForPlaceholder(multi, this.options.value)) {
           if (multi) {
@@ -12482,14 +12502,11 @@ var __oj_select_many_metadata =
                 _ComboUtils.getValueOptionsForPlaceholder(this, this.options.valueOption);
           }
         }
-        opts = $.extend(this.options, opts);
 
         // Fetch the option defaults from the CSS file
         this.cssOptionDefaults = ThemeUtils.parseJSONFromFontFamily('oj-combobox-option-defaults') || {};
 
-        this.combobox = multi ? new _OjMultiCombobox() : new _OjSingleCombobox();
-
-        this.combobox._init(opts);
+        this._initComboboxInstance();
         this._refreshRequired(this.options.required);
 
         // JET-43071 - messagescustom property doesn't work when initial render
@@ -12528,6 +12545,105 @@ var __oj_select_many_metadata =
         this._super();
       },
 
+      /**
+       * Instantiates the combobox instance
+       *
+       * @memberof! oj.ojCombobox
+       * @instance
+       * @private
+       */
+      _initComboboxInstance: function () {
+        const opts = $.extend(this.options, {
+          element: this.element,
+          ojContext: this
+        });
+        this.combobox = this.multiple ? new _OjMultiCombobox() : new _OjSingleCombobox();
+
+        this.combobox._init(opts);
+      },
+
+      /**
+       * Destroys the combobox instance
+       *
+       * @memberof! oj.ojCombobox
+       * @instance
+       * @private
+       */
+      _destroyComboboxInstance: function () {
+        this.combobox._destroy();
+        this.combobox = null;
+      },
+
+      /**
+       * Checks if the combobox is already instantiated
+       *
+       * @returns {boolean} indicates if we have the combobox instantiated
+       *
+       * @memberof! oj.ojCombobox
+       * @instance
+       * @private
+       */
+      _isComboboxInstantiated: function () {
+        return this.combobox != null;
+      },
+
+      /**
+       * Setup resources for the combobox
+       *
+       * @memberof! oj.ojCombobox
+       * @instance
+       * @private
+       */
+      _setupComboboxResources: function () {
+        if (!this._isComboboxInstantiated()) {
+          this._initComboboxInstance();
+        }
+        // Check if the dp is already wrapped
+        if (!_ComboUtils.isDataProviderWrapped(this)) {
+          _ComboUtils.wrapDataProviderIfNeeded(this, this.combobox ? this.combobox.opts : null);
+        }
+        _ComboUtils.addDataProviderEventListeners(this);
+      },
+
+      /**
+       * releases resources of the combobox
+       *
+       * @memberof! oj.ojCombobox
+       * @instance
+       * @private
+       */
+      _releaseComboboxResources: function () {
+        if (this._isComboboxInstantiated()) {
+          this._destroyComboboxInstance();
+        }
+        _ComboUtils.removeDataProviderEventListeners(this);
+        _ComboUtils.clearDataProviderWrapper(this);
+      },
+
+      /**
+       * Override to setup combobox resources
+       * @memberof! oj.ojCombobox
+       * @instance
+       * @protected
+       * @override
+       */
+      _SetupResources: function () {
+        this._super();
+        this._setupComboboxResources();
+      },
+
+      /**
+       * Override to release combobox resources
+       * @memberof! oj.ojCombobox
+       * @instance
+       * @protected
+       * @override
+       */
+      _ReleaseResources: function () {
+        this._super();
+        this._releaseComboboxResources();
+      },
+
       /**
        * Returns if the element is a text field element or not.
        * @memberof oj.ojCombobox
@@ -12568,8 +12684,10 @@ var __oj_select_many_metadata =
       refresh: function () {
         this._super();
 
-        this.combobox._destroy();
+        this._releaseComboboxResources();
+
         this._setup();
+        this._setupComboboxResources();
         this._SetRootAttributes();
         this._initComponentMessaging();
       },
@@ -18267,8 +18385,10 @@ var __oj_select_many_metadata =
 
         // cleanup the old HTML and setup the new HTML markups
           this._cleanup();
+          this._releaseSelectResources();
 
           this._setup();
+          this._setupSelectResources();
         // TODO: apply value in options for the selected value
 
           if (this._isNative() && this.options.value) {
@@ -18307,6 +18427,57 @@ var __oj_select_many_metadata =
           this._super();
         },
 
+      /**
+       * Setup resources for the select
+       *
+       * @memberof! oj.ojSelect
+       * @instance
+       * @private
+       */
+       _setupSelectResources: function () {
+         // Check if the dataprovider is already wrapped
+         if (!_ComboUtils.isDataProviderWrapped(this)) {
+          _ComboUtils.wrapDataProviderIfNeeded(this, this.select ? this.select.opts : null);
+         }
+        _ComboUtils.addDataProviderEventListeners(this);
+      },
+
+      /**
+       * releases resources of the select
+       *
+       * @memberof! oj.ojSelect
+       * @instance
+       * @private
+       */
+      _releaseSelectResources: function () {
+        _ComboUtils.removeDataProviderEventListeners(this);
+        _ComboUtils.clearDataProviderWrapper(this);
+      },
+
+      /**
+       * Override to setup select resources
+       * @memberof! oj.ojSelect
+       * @instance
+       * @protected
+       * @override
+       */
+      _SetupResources: function () {
+        this._super();
+        this._setupSelectResources();
+      },
+
+      /**
+       * Override to release select resources
+       * @memberof! oj.ojSelect
+       * @instance
+       * @protected
+       * @override
+       */
+      _ReleaseResources: function () {
+        this._super();
+        this._releaseSelectResources();
+      },
+
         // 19670760, dropdown popup should be closed on subtreeDetached notification.
         _NotifyDetached: function () {
           this._superApply(arguments);

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

@@ -7126,7 +7126,7 @@ var __oj_table_metadata =
       // if first row is in edit mode then fetching cell style from second row
       if (this._table._editableRowIdx === 0 && rowsCount > 1) {
         tableBodyCell = this._table._getTableBodyCell(1, columnIndex, null);
-      } else {
+      } else if (rowsCount > 0) {
         tableBodyCell = this._table._getTableBodyCell(0, columnIndex, null);
       }
       footerCell = this._table._getTableFooterCell(columnIndex);
@@ -7150,9 +7150,8 @@ var __oj_table_metadata =
           this._applyForcedColumnWidth(headerCell, forcedColumnWidth);
         }
 
-        // update column table body cell widths
+        // update width buffer row for legacy rendering
         var legacyWidthBuffer = this._table._getTableBodyLegacyWidthBuffer();
-        // check for width buffer row for legacy rendering
         if (legacyWidthBuffer != null) {
           var bufferCell = legacyWidthBuffer.childNodes[columnIndex];
           var bufferCellWidth = this._getForcedColumnWidth(bufferCell,
@@ -7160,21 +7159,24 @@ var __oj_table_metadata =
           this._applyForcedColumnWidth(bufferCell, bufferCellWidth);
         }
 
-        if (!this._table._hasRowOrCellRendererOrTemplate(columnIndex)) {
-          columnCellCompStyle = window.getComputedStyle(tableBodyCell);
-          forcedColumnCellWidth =
-            this._getForcedColumnWidth(tableBodyCell, columnCellCompStyle, colTestWidth);
-        }
-        for (var i = 0; i < rowsCount; i++) {
-          tableBodyCell = this._table._getTableBodyCell(i, columnIndex, null);
-          if (tableBodyCell != null) {
-            if (columnCellCompStyle == null) {
-              var cellCompStyle = window.getComputedStyle(tableBodyCell);
-              var forcedCellWidth =
-                this._getForcedColumnWidth(tableBodyCell, cellCompStyle, colTestWidth);
-              this._applyForcedColumnWidth(tableBodyCell, forcedCellWidth);
-            } else {
-              this._applyForcedColumnWidth(tableBodyCell, forcedColumnCellWidth);
+        // update column table body cell widths
+        if (tableBodyCell != null) {
+          if (!this._table._hasRowOrCellRendererOrTemplate(columnIndex)) {
+            columnCellCompStyle = window.getComputedStyle(tableBodyCell);
+            forcedColumnCellWidth =
+              this._getForcedColumnWidth(tableBodyCell, columnCellCompStyle, colTestWidth);
+          }
+          for (var i = 0; i < rowsCount; i++) {
+            tableBodyCell = this._table._getTableBodyCell(i, columnIndex, null);
+            if (tableBodyCell != null) {
+              if (columnCellCompStyle == null) {
+                var cellCompStyle = window.getComputedStyle(tableBodyCell);
+                var forcedCellWidth =
+                  this._getForcedColumnWidth(tableBodyCell, cellCompStyle, colTestWidth);
+                this._applyForcedColumnWidth(tableBodyCell, forcedCellWidth);
+              } else {
+                this._applyForcedColumnWidth(tableBodyCell, forcedColumnCellWidth);
+              }
             }
           }
         }

package/dist/js/libs/oj/debug_esm/ojcore-base.js

@@ -37,12 +37,12 @@ const oj = {
    * @global
    * @member {string} version JET version numberr
    */
-  version: '12.1.4',
+  version: '12.1.7',
   /**
    * @global
    * @member {string} revision JET source code revision number
    */
-  revision: '2022-07-05_18-07-33',
+  revision: '2022-09-06_20-30-55',
 
   // This function is only meant to be used outside the library, so quoting the name
   // to avoid renaming is appropriate

package/dist/js/libs/oj/debug_esm/ojdrawerpopup.js

@@ -97,7 +97,23 @@ let DrawerPopup = DrawerPopup_1 = class DrawerPopup extends Component {
             PopupService.getInstance().triggerOnDescendents($drawerElement, PopupService.EVENT.POPUP_REFRESH);
         };
         this.getFocusables = () => {
-            return this.rootRef.current.querySelectorAll('button, [href], input, select, textarea, [tabindex]:not([tabindex="-1"]), video');
+            const defaultFocusableElements = [
+                'button',
+                '[href]',
+                'input',
+                'select',
+                'textarea',
+                '[tabindex]',
+                'video'
+            ];
+            const selectorSuffix = ':not([tabindex="-1"]):not([disabled])';
+            const elementsCount = defaultFocusableElements.length;
+            let safeFocusablesSelector = '';
+            for (let i = 0; i < elementsCount; i++) {
+                const elSelector = `${defaultFocusableElements[i]}${selectorSuffix}`;
+                safeFocusablesSelector += i < elementsCount - 1 ? `${elSelector}, ` : `${elSelector}`;
+            }
+            return this.rootRef.current.querySelectorAll(safeFocusablesSelector);
         };
         this.isTargetDescendantOfOwnZorderLayerOrItsNextSiblings = (zorderLayer, target) => {
             const zorderLayersBuffer = [zorderLayer];

package/dist/js/libs/oj/debug_esm/ojoffcanvas.js

@@ -1953,11 +1953,11 @@ OffcanvasUtils.setupPanToReveal = function (_offcanvas) {
 
   // use hammer for swipe
   mOptions = {
+    // ensure pinch zoom work properly
+    touchAction: 'pinch-zoom',
     recognizers: [
        [Pan, { direction: DIRECTION_HORIZONTAL }]
-    ],
-    // ensure native gesture like pinch zoom work properly
-    touchAction: 'auto'
+    ]
    };
 
   // workaround for Hammer with iOS 13 issue, see: https://github.com/hammerjs/hammer.js/issues/1237
@@ -2002,8 +2002,6 @@ OffcanvasUtils.setupPanToReveal = function (_offcanvas) {
         default:
       }
 
-      event.preventDefault();
-
       if (direction === null) {
         return;
       }
@@ -2080,7 +2078,6 @@ OffcanvasUtils.setupPanToReveal = function (_offcanvas) {
 
         // stop bubbling
       event.stopPropagation();
-      event.preventDefault();
     })
     .on('panend', function (event) {
         // don't do anything if start is vetoed
@@ -2139,7 +2136,6 @@ OffcanvasUtils.setupPanToReveal = function (_offcanvas) {
         // restore item position
       wrapper.addClass(OffcanvasUtils.TRANSITION_SELECTOR);
       OffcanvasUtils._setTranslationX(wrapper, 'start', '0px');
-      event.preventDefault();
     });
 };