@oracle/oraclejet 16.0.5 → 16.1.0

package/dist/js/libs/oj/debug/ojgauge-toolkit.js

@@ -933,31 +933,7 @@ define(['exports', 'ojs/ojdvt-toolkit', 'ojs/ojdvt-axis'], function (exports, dv
       this.UpdateAriaAttributes();
 
       if (!this._bStaticRendering && !this.Options['readOnly']) {
-        container.setAriaRole('slider');
-        container.setAriaProperty('valuemin', this.Options['min']);
-        container.setAriaProperty('valuemax', this.Options['max']);
-        var value = DvtGaugeRenderer.getFormattedMetricLabel(this.Options['value'], this);
-        container.setAriaProperty('valuenow', value);
-        container.setAriaProperty('valuetext', value);
-        var tooltip = DvtGaugeRenderer.getTooltipString(this);
-
-        if (dvt.Agent.isTouchDevice()) {
-          this._container.setAriaProperty('live', 'assertive');
-          if (value !== tooltip) {
-            tooltip = value + dvt.AriaUtils.ARIA_LABEL_DESC_DELIMITER + tooltip;
-          }
-        }
-        if (value !== tooltip) {
-          container.setAriaProperty('label', tooltip);
-        }
-
-        // Generate a unique id to force screen readers to update for each render (and potential value change)
-        this._renderCount = this._renderCount != null ? this._renderCount + 1 : 0;
-        var ariaId = this.getId() + '_' + this._renderCount;
-        container.setId(ariaId);
-
-        // Set as the active element that's read by the screen reader.
-        this.getCtx().setActiveElement(container);
+        this.setAriaAttributes();
       }
 
       // . To support ADF action attribute
@@ -971,6 +947,23 @@ define(['exports', 'ojs/ojdvt-toolkit', 'ojs/ojdvt-axis'], function (exports, dv
       }
     }
 
+    setAriaAttributes() {
+      var parentContainer = this.getCtx().getContainer();
+      parentContainer.setAttribute('role', 'slider');
+      parentContainer.setAttribute('aria-valuemin', this.Options['min']);
+      parentContainer.setAttribute('aria-valuemax', this.Options['max']);
+      parentContainer.setAttribute('aria-valuenow', this.Options['value']);
+
+      var valueText = DvtGaugeRenderer.getFormattedMetricLabel(this.Options['value'], this);
+      if (Number(valueText) === this.Options['value']) valueText = this.Options['value'].toString();
+
+      parentContainer.setAttribute('aria-valuetext', valueText);
+
+      if (this.Options['disabled']) {
+        parentContainer.setAttribute('aria-disabled', true);
+      }
+    }
+
     /**
      * Creates and returns a logical object for this gauge.
      * @return {dvt.SimpleObjPeer}
@@ -1150,21 +1143,6 @@ define(['exports', 'ojs/ojdvt-toolkit', 'ojs/ojdvt-axis'], function (exports, dv
      */
     renderUpdate() {
       this.render();
-      this.UpdateAriaLiveValue(this._container);
-    }
-
-    /**
-     * Helper function for updating the live values that are read out while editing for accessibility
-     * @param {dvt.Container} container The slider container we are using.
-     * @param {number} value The value of the gauge
-     * @protected
-     */
-    UpdateAriaLiveValue(container, value) {
-      value = DvtGaugeRenderer.getFormattedMetricLabel(value ? value : this.Options['value'], this);
-      container.setAriaProperty('valuenow', value);
-      container.setAriaProperty('valuetext', value);
-
-      if (dvt.Agent.isTouchDevice()) container.setAriaProperty('label', value);
     }
 
     /**
@@ -1349,25 +1327,19 @@ define(['exports', 'ojs/ojdvt-toolkit', 'ojs/ojdvt-axis'], function (exports, dv
         var tooltip = DvtGaugeRenderer.getTooltipString(this);
         if (this.IsParentRoot()) {
           var translations = this.Options.translations;
+          this.getCtx().setAriaLabel(
+            dvt.ResourceUtils.format(translations.labelAndValue, [
+              translations.labelDataVisualization,
+              dvt.Displayable.generateAriaLabel(
+                dvt.AriaUtils.processAriaLabel(this.GetComponentDescription()),
+                tooltip ? [tooltip] : null
+              )
+            ])
+          );
           if (!this.IsInteractive()) {
             this.getCtx().setAriaRole('img');
-            this.getCtx().setAriaLabel(
-              dvt.ResourceUtils.format(translations.labelAndValue, [
-                translations.labelDataVisualization,
-                dvt.Displayable.generateAriaLabel(
-                  dvt.AriaUtils.processAriaLabel(this.GetComponentDescription()),
-                  tooltip ? [tooltip] : null
-                )
-              ])
-            );
           } else {
-            this.getCtx().setAriaRole('application');
-            this.getCtx().setAriaLabel(
-              dvt.ResourceUtils.format(translations.labelAndValue, [
-                translations.labelDataVisualization,
-                dvt.AriaUtils.processAriaLabel(this.GetComponentDescription())
-              ])
-            );
+            this.getCtx().setAriaRole('slider');
           }
         } else if (!this.IsInteractive()) {
           this.setAriaRole('img');
@@ -2587,7 +2559,6 @@ define(['exports', 'ojs/ojdvt-toolkit', 'ojs/ojdvt-axis'], function (exports, dv
           hoverContainer.setClipPath(hoverClip);
         }
       }
-      this.UpdateAriaLiveValue(container, value);
     }
 
     /**

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

@@ -3416,6 +3416,10 @@ var __oj_status_meter_gauge_metadata =
    * <p>title will no longer be required to make your rating gauge accessible. Use aria-label attribute instead for text to be used by screen readers, and datatip or tooltip for text to be shown on hover or focus.
    * The tooltip attribute is for readonly rating gauge only. If aria-label is not present for readonly rating gauge, tooltip will be used as aria-label. Note the datatip should be used for interactive gauges only and it takes a function.
    * </p>
+   * <h5>role change</h5>
+   * <p>The role of interactive oj-rating-gauge has been changed from 'application' to 'slider'. The screenreader behavior will be consistent with that of role slider.
+   * Slider related aria attributes (aria-valuenow, aria-valuemin, aria-valuemax, aria-valuetext) will be populated according to element value, min and max attribute.
+   * </p>
    * @ojfragment migrationDoc
    * @memberof oj.ojRatingGauge
    */
@@ -3696,9 +3700,9 @@ var __oj_status_meter_gauge_metadata =
    * <p>
    * indicator-size attribute will no longer support values greater than 1 in oj-c-meter-bar and oj-c-meter-circle.
    * </p>
-   * <h5>min height for vertical meter bar</h5>
-   * <p>
-   * The vertical oj-c-meter-bar now has a default min-height which is theme driven.
+   * <h5>role change</h5>
+   * <p>The role of interactive oj-status-meter-gauge has been changed from 'application' to 'slider'. The screenreader behavior will be consistent with that of role slider.
+   * Slider related aria attributes (aria-valuenow, aria-valuemin, aria-valuemax, aria-valuetext) will be populated according to element value, min and max attribute.
    * </p>
    * @ojfragment migrationDoc
    * @memberof oj.ojStatusMeterGauge

package/dist/js/libs/oj/debug/ojlegend-toolkit.js

@@ -2034,7 +2034,11 @@ define(['exports', 'ojs/ojdvt-toolkit'], function (exports, dvt) { 'use strict';
         marker.setStyle().setClassName();
       }
 
-      if ((hideAndShow != 'none' && hideAndShow != 'off') || item['shortDesc'] != null) {
+      if (
+        (hideAndShow != 'none' && hideAndShow != 'off') ||
+        item['shortDesc'] != null ||
+        item['drilling'] == 'on'
+      ) {
         itemRect.setAriaRole('img');
         peer.updateAriaLabel();
       }

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

@@ -5363,7 +5363,7 @@ var __oj_list_view_metadata =
           tempFocusDiv.setAttribute('tabindex', '-1');
           container.appendChild(tempFocusDiv);
         }
-        tempFocusDiv.focus();
+        tempFocusDiv.focus({ preventScroll: true });
       },
 
       /**

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

@@ -3984,6 +3984,7 @@ define(['jquery', 'ojs/ojjquery-hammer', 'ojs/ojpopupcore', 'ojs/ojoption', 'ojs
         var element = this.element;
         var status = oj.ZOrderUtils.getStatus(element);
         if (status === oj.ZOrderUtils.STATUS.OPEN) {
+          ojcustomelementUtils.CustomElementUtils.cleanComponentBindings(element[0]);
           element.remove();
         }
       },

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

@@ -1579,7 +1579,11 @@ define(['ojs/ojcore-base', 'jquery', 'knockout', 'ojs/ojanimation', 'ojs/ojconte
   };
 
   MessagesViewModel.prototype._surrogateRemoveHandler = function () {
-    this._closeOverlay();
+    var composite = $(this._composite);
+    if (oj.ZOrderUtils.getStatus(composite) === oj.ZOrderUtils.STATUS.OPEN) {
+      ojcustomelementUtils.CustomElementUtils.cleanComponentBindings(composite[0]);
+      this._closeOverlay();
+    }
   };
 
   MessagesViewModel.prototype._createObservables = function () {

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

@@ -2347,6 +2347,7 @@ define(['ojs/ojpopupcore', 'ojs/ojcore-base', 'jquery', 'ojs/ojcontext', 'ojs/oj
         var element = this.element;
         var status = oj.ZOrderUtils.getStatus(element);
         if (status === oj.ZOrderUtils.STATUS.OPEN) {
+          ojcustomelementUtils.CustomElementUtils.cleanComponentBindings(element[0]);
           element.remove();
         }
       },

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

@@ -1171,11 +1171,11 @@ define(['exports', 'ojs/ojcore-base', 'jquery', 'ojs/ojcomponentcore', 'ojs/ojlo
    * Key to store the current operation status of the popup.
    * @const
    * @private
-   * @type {string}
+   * @type {Symbol}
    * @see ZOrderUtils.getStatus
    * @see ZOrderUtils.setStatus
    */
-  ZOrderUtils._STATUS_DATA = 'oj-popup-status';
+  ZOrderUtils._STATUS_DATA = Symbol('PopupStatus');
 
   /**
    * Returns the current operation status of the target popup element.
@@ -1190,9 +1190,12 @@ define(['exports', 'ojs/ojcore-base', 'jquery', 'ojs/ojcomponentcore', 'ojs/ojlo
     }
 
     /** @type {?} */
-    var status = popup.data(ZOrderUtils._STATUS_DATA);
-    if (isNaN(status)) {
-      return ZOrderUtils.STATUS.UNKNOWN;
+    var status = ZOrderUtils.STATUS.UNKNOWN;
+    if (popup.length) {
+      status = popup[0][ZOrderUtils._STATUS_DATA];
+      if (isNaN(status)) {
+        status = ZOrderUtils.STATUS.UNKNOWN;
+      }
     }
     return status;
   };
@@ -1208,9 +1211,10 @@ define(['exports', 'ojs/ojcore-base', 'jquery', 'ojs/ojcomponentcore', 'ojs/ojlo
       // eslint-disable-next-line no-param-reassign
       popup = $(popup);
     }
+    var popupDom = popup[0];
 
     if (status >= ZOrderUtils.STATUS.UNKNOWN && status <= ZOrderUtils.STATUS.CLOSE) {
-      popup.data(ZOrderUtils._STATUS_DATA, status);
+      popupDom[ZOrderUtils._STATUS_DATA] = status;
     }
   };
 
@@ -1318,7 +1322,7 @@ define(['exports', 'ojs/ojcore-base', 'jquery', 'ojs/ojcomponentcore', 'ojs/ojlo
     popup.appendTo(layer); // @HTMLUpdateOK
 
     // link the popup to the layer @see ZOrderUtils.getOpenPopupLayer
-    popup.data(ZOrderUtils._LAYER_ID_DATA, layer.attr('id'));
+    popupDom[ZOrderUtils._LAYER_ID_DATA] = layer.attr('id');
 
     layer.appendTo(ancestorLayer); // @HTMLUpdateOK
     Components.subtreeAttached(popupDom);
@@ -1481,11 +1485,13 @@ define(['exports', 'ojs/ojcore-base', 'jquery', 'ojs/ojcomponentcore', 'ojs/ojlo
   ZOrderUtils.getOpenPopupLayer = function (popup) {
     /** @type {?} */
     var layer = popup.parent();
+    var popupDom = popup[0];
+
     if (!layer || layer.length === 0) {
       // the open popup has been detached from the layer before it was closed
       // use the backup pointer for better cleanup
       /** @type {?} */
-      var layerId = popup.data(ZOrderUtils._LAYER_ID_DATA);
+      var layerId = popupDom[ZOrderUtils._LAYER_ID_DATA];
       layer = $(document.getElementById(layerId));
     }
 
@@ -1511,9 +1517,10 @@ define(['exports', 'ojs/ojcore-base', 'jquery', 'ojs/ojcomponentcore', 'ojs/ojlo
 
     layer.removeData(ZOrderUtils._EVENTS_DATA);
     layer.removeData(ZOrderUtils._MODALITY_DATA);
-    popup.removeData(ZOrderUtils._LAYER_ID_DATA);
 
     var popupDom = popup[0];
+    delete popupDom[ZOrderUtils._LAYER_ID_DATA];
+
     Components.subtreeDetached(popupDom);
     var originatingSubtreeExists = ZOrderUtils._removeSurrogate(layer);
 
@@ -2283,10 +2290,10 @@ define(['exports', 'ojs/ojcore-base', 'jquery', 'ojs/ojcomponentcore', 'ojs/ojlo
    *
    * @const
    * @private
-   * @type {string}
+   * @type {Symbol}
    * @see ZOrderUtils.getOpenPopupLayer
    */
-  ZOrderUtils._LAYER_ID_DATA = 'oj-popup-layer-id';
+  ZOrderUtils._LAYER_ID_DATA = Symbol('PopupLayerId');
 
   /**
    * The attribute name assigned to the popup layer for open dialogs that have a
@@ -4038,7 +4045,7 @@ define(['exports', 'ojs/ojcore-base', 'jquery', 'ojs/ojcomponentcore', 'ojs/ojlo
       link.on('keydown keyup keypress', PopupSkipLink._keyHandler);
     }
 
-    sibling.data(PopupSkipLink._SKIPLINK_ATTR, link);
+    sibling[0][PopupSkipLink._SKIPLINK_ATTR] = link;
   };
 
   /**
@@ -4074,8 +4081,8 @@ define(['exports', 'ojs/ojcore-base', 'jquery', 'ojs/ojcomponentcore', 'ojs/ojlo
     delete this._callback;
 
     if (sibling) {
-      var link = sibling.data(PopupSkipLink._SKIPLINK_ATTR);
-      sibling.removeData(PopupSkipLink._SKIPLINK_ATTR);
+      var link = sibling[0][PopupSkipLink._SKIPLINK_ATTR];
+      delete sibling[0][PopupSkipLink._SKIPLINK_ATTR];
       if (link) {
         link.off('click keydown keyup keypress');
         link.remove();
@@ -4095,7 +4102,7 @@ define(['exports', 'ojs/ojcore-base', 'jquery', 'ojs/ojcomponentcore', 'ojs/ojlo
     /** @type {jQuery} */
     var link;
     if (sibling) {
-      link = sibling.data(PopupSkipLink._SKIPLINK_ATTR);
+      link = sibling[0][PopupSkipLink._SKIPLINK_ATTR];
     }
     return link;
   };
@@ -4105,9 +4112,9 @@ define(['exports', 'ojs/ojcore-base', 'jquery', 'ojs/ojcomponentcore', 'ojs/ojlo
    * reference for the associated skip link.
    * @const
    * @private
-   * @type {string}
+   * @type {Symbol}
    */
-  PopupSkipLink._SKIPLINK_ATTR = 'oj-skiplink';
+  PopupSkipLink._SKIPLINK_ATTR = Symbol('ojSkipLink');
 
   /**
    * Coordinate communications between an event being fulfilled and one or more promises

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

@@ -5039,6 +5039,15 @@ define(['exports', 'ojs/ojeditablevalue', 'ojs/ojpopupcore', 'ojs/ojinputtext',
         var processSetOptions = this._processSetOptions.pop();
 
         const finishProcessingSetOptions = () => {
+          // JET-66038 - REGRESSION FOR EXPENSETYPEID FIELD 'GETINPUTELEM' ERROR
+          // When this function is executed in a microtask, it's possible that the element has been
+          // removed from the DOM and its resources released since the microtask was queued.  If so,
+          // simply return without doing anything; otherwise we may throw an NPE because we've
+          // already destroyed our internal structures.
+          if (this._bReleasedResources) {
+            return;
+          }
+
           // JET-34601 - SELECT SINGLE- CHANGING DISABLED PROPERTY GIVES A GLOWING EFFECT
           // if we need to refresh, do that before setting a new value
           if (processSetOptions.forRefresh) {
@@ -5166,11 +5175,25 @@ define(['exports', 'ojs/ojeditablevalue', 'ojs/ojpopupcore', 'ojs/ojinputtext',
       // which properties are set by the preact. Then we can use the flags as well as the latest value to determine
       // how to process the value and valueItem correctly.
       const elemState = ojcustomelementUtils.CustomElementUtils.getElementState(this.OuterWrapper);
+      // JET-66502 - Dynamic UI Defect - oj-dyn form not working with JET 16 for select single LOVs
+      // We only need to enter this block if we're currently processing a value or valueItem
+      // being set.  Otherwise, if multiple properties are being set at the same time, properties
+      // handled here after the value or valueItem may not be processed correctly because we
+      // return early from this method.
       if (
         elemState.getBindingProviderType() === 'preact' &&
         (key === 'value' || key === this._GetValueItemPropertyName())
       ) {
         const processValueAndValueItem = () => {
+          // JET-66038 - REGRESSION FOR EXPENSETYPEID FIELD 'GETINPUTELEM' ERROR
+          // When this function is executed in a microtask, it's possible that the element has been
+          // removed from the DOM and its resources released since the microtask was queued.  If so,
+          // simply return without doing anything; otherwise we may throw an NPE because we've
+          // already destroyed our internal structures.
+          if (this._bReleasedResources) {
+            return;
+          }
+
           // Get the latest value and valueItem
           const _valueItem = this.options[this._GetValueItemPropertyName()];
           const _value = this.options.value;
@@ -5260,6 +5283,14 @@ define(['exports', 'ojs/ojeditablevalue', 'ojs/ojpopupcore', 'ojs/ojinputtext',
             this._processingSetOptionsMicroTask = true;
             window.queueMicrotask(processValueAndValueItem);
           }
+          // JET-66502 - Dynamic UI Defect - oj-dyn form not working with JET 16 for select
+          // single LOVs
+          // Because we return early here based on the instance flags for processing a value or
+          // valueItem, we will also return early from here when we try to process any other
+          // properties set at the same time as the value or valueItem and processed after one
+          // of them.  This is why we only conditionally enter this whole block of code when
+          // we're currently processing a value or valueItem.  We skip this block of code for
+          // any other property.
           return;
         }
       }

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

@@ -966,6 +966,7 @@ var __oj_sunburst_metadata =
     }
   },
   "methods": {
+    "getAutomation": {},
     "getContextByNode": {},
     "getNode": {},
     "getProperty": {},
@@ -2156,6 +2157,16 @@ var __oj_sunburst_node_metadata =
       return this._component.getAutomation().getNode(subIdPath);
     },
 
+    /**
+     * Returns the sunburst automation component used by webdriver.
+     * @memberof oj.ojSunburst
+     * @instance
+     * @ojhidden
+     */
+    getAutomation: function () {
+      return this._component.getAutomation();
+    },
+
     /**
      * {@ojinclude "name":"nodeContextDoc"}
      * @param {!Element} node - {@ojinclude "name":"nodeContextParam"}