@sapui5/sap.ushell 1.120.8 → 1.121.0

package/src/main/js/sap/ushell/services/CrossApplicationNavigation.js

@@ -6,7 +6,7 @@
  * It exposes interfaces to perform a hash change and/or trigger an external navigation.
  *
  * @deprecated since 1.120 Please use {@link sap.ushell.services.Navigation} instead
- * @version 1.120.8
+ * @version 1.121.0
  */
 sap.ui.define([
     "sap/ushell/services/AppConfiguration",
@@ -54,12 +54,9 @@ sap.ui.define([
      * Allows navigating to "external" targets outside of the currently running app (but still in scope of the current Fiori launchpad)
      * or to create links to such external targets.
      *
-     * To use the CrossApplicationNavigation service you can retrieve an instance via ushell's Container:
-     * <pre>
-     * sap.ushell.Container.getServiceAsync("CrossApplicationNavigation").then(function (CrossApplicationNavigationService) {
-     *   // Use the CrossApplicationNavigation service
-     * });
-     * </pre>
+     * <b>Note:</b> To retrieve a valid instance of this service, it is necessary to call
+     * <code>sap.ushell.Container.getServiceAsync("ClientSideTargetResolution")</code>. For details, see
+     * {@link sap.ushell.Container#getServiceAsync}.
      *
      * The CrossApplicationNavigation service currently provides platform independent functionality.
      *
@@ -109,10 +106,10 @@ sap.ui.define([
      * @param {string} sParameters The parameters.
      * @param {object} oServiceConf Service Configuration.
      *
+     * @augments sap.ushell.services.Service
      * @hideconstructor
      *
-     * @see sap.ushell.services.Container#getServiceAsync
-     *
+     * @see sap.ushell.Container#getServiceAsync
      * @since 1.15.0
      * @deprecated since 1.120 Please use {@link sap.ushell.services.Navigation} instead
      * @public
@@ -545,7 +542,6 @@ sap.ui.define([
          * @since 1.15.0
          * @public
          * @deprecated since 1.98. Please use {@link #hrefForExternalAsync} instead.
-         * @alias sap.ushell.services.CrossApplicationNavigation#hrefForExternal
          */
         this.hrefForExternal = function (oArgs, oComponent, bAsync) {
             // Check oComponent and bAsync as oComponent is optional
@@ -591,14 +587,14 @@ sap.ui.define([
 
             oArgsClone = this._injectInnerAppRoute(oArgsClone, oExtraction.innerAppRoute);
 
-            var ShellNavigation = sap.ushell.Container.getService("ShellNavigation"); // LEGACY API (deprecated)
+            var ShellNavigationInternal = sap.ushell.Container.getService("ShellNavigationInternal"); // LEGACY API (deprecated)
 
-            if (!ShellNavigation) {
+            if (!ShellNavigationInternal) {
                 Log.debug("Shell not available, no Cross App Navigation");
                 return "";
             }
 
-            return ShellNavigation.hrefForExternalSync(oArgsClone, undefined, oComponent);
+            return ShellNavigationInternal.hrefForExternalSync(oArgsClone, undefined, oComponent);
         };
 
         /**
@@ -675,7 +671,6 @@ sap.ui.define([
          *
          * @since 1.94.0
          * @public
-         * @alias sap.ushell.services.CrossApplicationNavigation#hrefForExternalAsync
          */
         this.hrefForExternalAsync = function (oArgs, oComponent) {
             // clone because _extractInnerAppRoute may change the original structure
@@ -706,11 +701,11 @@ sap.ui.define([
                     oArgsClone = amendTargetWithSapUshellEncTestParameter(oArgsClone);
                     oArgsClone = this._injectInnerAppRoute(oArgsClone, oExtraction.innerAppRoute);
 
-                    return sap.ushell.Container.getServiceAsync("ShellNavigation");
+                    return sap.ushell.Container.getServiceAsync("ShellNavigationInternal");
                 }.bind(this))
-                .then(function (ShellNavigation) {
+                .then(function (ShellNavigationInternal) {
                     return new Promise(function (resolve, reject) {
-                        ShellNavigation.hrefForExternal(oArgsClone, undefined, oComponent)
+                        ShellNavigationInternal.hrefForExternal(oArgsClone, undefined, oComponent)
                             .then(resolve)
                             .catch(reject);
                     });
@@ -726,9 +721,9 @@ sap.ui.define([
          * in a hash, this function replaces it into a long url with all parameters expanded
          *
          * @param {string} sHashFragment an (internal format) shell hash
-         * @returns {jQuery.Promise} promise the success handler of the resolve promise get an expanded shell hash as first argument
+         * @returns {jQuery.Promise} Resolves the expanded shell hash as first argument.
+         *
          * @public
-         * @alias sap.ushell.services.CrossApplicationNavigation#expandCompactHash
          */
         this.expandCompactHash = function (sHashFragment) {
             var oDeferred = new jQuery.Deferred();
@@ -754,7 +749,6 @@ sap.ui.define([
          * @returns {Promise<undefined>} A promise which resolves once the back navigation was triggered
          *
          * @public
-         * @alias sap.ushell.services.CrossApplicationNavigation#backToPreviousApp
          */
         this.backToPreviousApp = function () {
             return this.isInitialNavigationAsync()
@@ -774,8 +768,8 @@ sap.ui.define([
          * If no argument is provided it will navigate back 1 step.
          *
          * @param {int} iSteps positive integer representing the steps to go back in the history
+         *
          * @public
-         * @alias sap.ushell.services.CrossApplicationNavigation#historyBack
          */
         this.historyBack = function (iSteps) {
             var iActualStepsBack = -1;
@@ -798,10 +792,10 @@ sap.ui.define([
          * without a navigation from another app or FLP home.
          *
          * @returns {boolean} Whether the initial navigation occurred.
+         *
+         * @since 1.36.0
          * @public
          * @deprecated since 1.94. Please use {@link #isInitialNavigationAsync} instead.
-         * @alias sap.ushell.services.CrossApplicationNavigation#isInitialNavigation
-         * @since 1.36.0
          */
         this.isInitialNavigation = function () {
             Log.error("Deprecated API call of 'sap.ushell.CrossApplicationNavigation.isInitialNavigation'. Please use 'isInitialNavigationAsync' instead",
@@ -810,25 +804,23 @@ sap.ui.define([
             );
 
             var oContainer = ObjectPath.get("sap.ushell.Container");
-            var oShellNavigation = oContainer
+            var oShellNavigationInternal = oContainer
                 && typeof oContainer.getService === "function"
-                && oContainer.getService("ShellNavigation");
+                && oContainer.getService("ShellNavigationInternal");
 
-            if (!oShellNavigation) {
+            if (!oShellNavigationInternal) {
                 Log.debug(
-                    "ShellNavigation service not available",
+                    "ShellNavigationInternal service not available",
                     "This will be treated as the initial navigation",
                     "sap.ushell.services.CrossApplicationNavigation"
                 );
                 return true;
             }
 
-            var bIsInitialNavigation = oShellNavigation.isInitialNavigation();
+            var bIsInitialNavigation = oShellNavigationInternal.isInitialNavigation();
 
-            /*
-             * An undefined value indicates that the ShellNavigation service did not initialize the ShellNavigationHashChanger yet.
-             * Hence this is the first navigation in case asked at this point in time.
-             */
+            // An undefined value indicates that the ShellNavigationInternal service did not initialize the ShellNavigationHashChanger yet.
+            // Hence this is the first navigation in case asked at this point in time.
             if (typeof bIsInitialNavigation === "undefined") {
                 return true;
             }
@@ -842,19 +834,17 @@ sap.ui.define([
          * without a previous navigation to another app, to the FLP home, or another target that adds an entry in the browser history.
          *
          * @returns {Promise<boolean>} This promise resolves with a boolean indicating if the current navigation is considered initial
-         * @public
-         * @alias sap.ushell.services.CrossApplicationNavigation#isInitialNavigationAsync
+         *
          * @since 1.94.0
+         * @public
          */
         this.isInitialNavigationAsync = function () {
-            return sap.ushell.Container.getServiceAsync("ShellNavigation")
-                .then(function (ShellNavigation) {
-                    var bIsInitialNavigation = ShellNavigation.isInitialNavigation();
-
-                    /*
-                    * An undefined value indicates that the ShellNavigation service did not initialize the ShellNavigationHashChanger yet.
-                    * Hence this is the first navigation in case asked at this point in time.
-                    */
+            return sap.ushell.Container.getServiceAsync("ShellNavigationInternal")
+                .then(function (ShellNavigationInternal) {
+                    var bIsInitialNavigation = ShellNavigationInternal.isInitialNavigation();
+
+                    // An undefined value indicates that the ShellNavigationInternal service did not initialize the ShellNavigationHashChanger yet.
+                    // Hence this is the first navigation in case asked at this point in time.
                     if (typeof bIsInitialNavigation === "undefined") {
                         return true;
                     }
@@ -863,7 +853,7 @@ sap.ui.define([
                 })
                 .catch(function () {
                     Log.debug(
-                        "ShellNavigation service not available",
+                        "ShellNavigationInternal service not available",
                         "This will be treated as the initial navigation",
                         "sap.ushell.services.CrossApplicationNavigation"
                     );
@@ -964,7 +954,6 @@ sap.ui.define([
          *
          * @since 1.15.0
          * @public
-         * @alias sap.ushell.services.CrossApplicationNavigation#toExternal
          */
         this.toExternal = function (oArgs, oComponent) {
             var bWriteHistory = oArgs.writeHistory;
@@ -977,21 +966,21 @@ sap.ui.define([
             var oExtraction = this._extractInnerAppRoute(oArgsClone);
 
             var vIntentNoAppRoute = oExtraction.intent;
-            var oShellNavigationService;
+            var oShellNavigationInternal;
             var oCurrentApplication;
 
             return Promise.all([
                 sap.ushell.Container.getServiceAsync("AppLifeCycle"),
-                sap.ushell.Container.getServiceAsync("ShellNavigation")
+                sap.ushell.Container.getServiceAsync("ShellNavigationInternal")
             ])
                 .then(function (aServices) {
                     var oAppLifeCycleService = aServices[0];
-                    oShellNavigationService = aServices[1];
+                    oShellNavigationInternal = aServices[1];
                     oCurrentApplication = oAppLifeCycleService.getCurrentApplication();
                     return oCurrentApplication && oCurrentApplication.getIntent();
                 })
                 .then(function (oIntent) {
-                    this._checkIfAppNeedsToBeReloaded(oNavTarget, oCurrentApplication, oIntent, oShellNavigationService.hashChanger);
+                    this._checkIfAppNeedsToBeReloaded(oNavTarget, oCurrentApplication, oIntent, oShellNavigationInternal.hashChanger);
                     return oUtils.addXAppStateFromParameterAsync(
                         vIntentNoAppRoute,
                         "sap-xapp-state-data" /* parameter containing data */
@@ -1014,7 +1003,7 @@ sap.ui.define([
 
                     oArgsClone = this._injectInnerAppRoute(oArgsClone, oExtraction.innerAppRoute);
 
-                    return oShellNavigationService.toExternal(oArgsClone, oComponent, bWriteHistory);
+                    return oShellNavigationInternal.toExternal(oArgsClone, oComponent, bWriteHistory);
                 }.bind(this))
                 .catch(function (oError) {
                     Log.error("CrossAppNavigation.toExternal failed", oError, "sap.ushell.services.CrossApplicationNavigation");
@@ -1032,7 +1021,6 @@ sap.ui.define([
          *
          * @since 1.106
          * @private
-         * @alias sap.ushell.services.CrossApplicationNavigation#_checkIfAppNeedsToBeReloaded
          */
         this._checkIfAppNeedsToBeReloaded = function (navTarget, currentApplication, currentApplicationIntent, shellNavigationHashChanger) {
             if (!currentApplication || !currentApplicationIntent || !navTarget || !navTarget.target) {
@@ -1101,10 +1089,10 @@ sap.ui.define([
          *   Note that sAppHash shall not exceed 512 bytes when serialized as UTF-8.
          * @returns {string} A string which can be put into the link tag, containing the current shell navigation target
          *   and the specified application specific hash suffix
+         *
          * @since 1.15.0
          * @public
          * @deprecated since 1.94. Please use {@link #hrefForAppSpecificHashAsync} instead.
-         * @alias sap.ushell.services.CrossApplicationNavigation#hrefForAppSpecificHash
          */
         this.hrefForAppSpecificHash = function (sAppHash) {
             Log.error("Deprecated API call of 'sap.ushell.CrossApplicationNavigation.hrefForAppSpecificHash'. Please use 'hrefForAppSpecificHashAsync' instead",
@@ -1114,9 +1102,9 @@ sap.ui.define([
 
             var oContainer = ObjectPath.get("sap.ushell.Container");
             if (oContainer && typeof oContainer.getService === "function") {
-                var oShellNavigation = oContainer.getService("ShellNavigation");
-                if (oShellNavigation) {
-                    return oShellNavigation.hrefForAppSpecificHash(sAppHash);
+                var oShellNavigationInternal = oContainer.getService("ShellNavigationInternal");
+                if (oShellNavigationInternal) {
+                    return oShellNavigationInternal.hrefForAppSpecificHash(sAppHash);
                 }
             }
             Log.debug("Shell not available, no Cross App Navigation; fallback to app-specific part only");
@@ -1137,14 +1125,14 @@ sap.ui.define([
          *   Note that sAppHash shall not exceed 512 bytes when serialized as UTF-8.
          * @returns {Promise<string>} A promise which resolves a string which can be put into the link tag, containing the current
          *   shell navigation target and the specified application specific hash suffix
+         *
          * @since 1.94.0
          * @public
-         * @alias sap.ushell.services.CrossApplicationNavigation#hrefForAppSpecificHashAsync
          */
         this.hrefForAppSpecificHashAsync = function (sAppHash) {
-            return sap.ushell.Container.getServiceAsync("ShellNavigation")
-                .then(function (ShellNavigation) {
-                    return ShellNavigation.hrefForAppSpecificHash(sAppHash);
+            return sap.ushell.Container.getServiceAsync("ShellNavigationInternal")
+                .then(function (ShellNavigationInternal) {
+                    return ShellNavigationInternal.hrefForAppSpecificHash(sAppHash);
                 })
                 .catch(function () {
                     Log.debug("Shell not available, no Cross App Navigation; fallback to app-specific part only");
@@ -1165,8 +1153,7 @@ sap.ui.define([
          *
          * @param {string} sSemanticObject Semantic object.
          * @param {object} [mParameters] @see sap.ushell.services.CrossApplicationNavigation#getSemanticObjectLinks for description.
-         * @returns {jQuery.Deferred} When a relevant link object exists,
-         *   it will return a promise that resolves to an object of the following form:
+         * @returns {jQuery.Promise} Resolves to an object of the following form when a relevant link object exists:
          *   <pre>
          *   {
          *     intent: "#AnObject-Action?A=B&C=e&C=j",
@@ -1177,9 +1164,9 @@ sap.ui.define([
          *   }
          *   </pre>
          *   Otherwise, the returned promise will resolve to null when no relevant link object exists.
-         * @public
+         *
          * @since 1.48
-         * @alias sap.ushell.services.CrossApplicationNavigation#getPrimaryIntent
+         * @public
          */
         this.getPrimaryIntent = function (sSemanticObject, mParameters) {
             var oQuery = {};
@@ -1251,8 +1238,7 @@ sap.ui.define([
          * @param {object} [oComponent] SAP UI5 Component invoking the service
          * @param {string} [sAppStateKey] application state key to add to the generated links, SAP internal usage only
          * @param {boolean} [bCompactIntents] whether the returned intents should be returned in compact format. Defaults to false.
-         * @returns {jQuery.Promise} A <code>jQuery.Deferred</code> object's promise which is resolved with
-         *   an array of link objects containing (at least) the following properties:
+         * @returns {jQuery.Promise} Resolves with an array of link objects containing (at least) the following properties:
          * <pre>
          *   {
          *     intent: "#AnObject-action?A=B&C=e",
@@ -1276,7 +1262,6 @@ sap.ui.define([
          * @deprecated since 1.38. Please use {@link #getLinks} instead.
          * @since 1.19.0
          * @public
-         * @alias sap.ushell.services.CrossApplicationNavigation#getSemanticObjectLinks
          */
         this.getSemanticObjectLinks = function (sSemanticObject, mParameters, bIgnoreFormFactor, oComponent, sAppStateKey, bCompactIntents) {
             var oDeferred = new jQuery.Deferred();
@@ -1432,7 +1417,7 @@ sap.ui.define([
          * @param {boolean} [vArgs.compactIntents=false] Optional, whether intents should be returned in compact format. Defaults to false.
          * @param {string[]} [vArgs.tags] Optional, if specified, only returns links that match inbound with certain tags.
          *
-         * @returns {jQuery.Deferred.promise} A promise that resolves with an array of link objects containing (at least) the following properties:
+         * @returns {jQuery.Promise} Resolves with an array of link objects containing (at least) the following properties:
          *  <pre>
          *   {
          *     intent: "#AnObject-Action?A=B&C=e&C=j",
@@ -1485,10 +1470,8 @@ sap.ui.define([
          */
         this.getLinks = function (vArgs) {
             var aExpandedIntents;
-            /*
-             * the invokeUnfoldingArrayArguments does not want [oArg1, oArg2, oArg3], but [ [oArg1], [oArg2], [oArg3] ], because the
-             * logic in that method is based on positional parameters - however we have only one argument (the oArg object in this case).
-             */
+            // the invokeUnfoldingArrayArguments does not want [oArg1, oArg2, oArg3], but [ [oArg1], [oArg2], [oArg3] ], because the
+            // logic in that method is based on positional parameters - however we have only one argument (the oArg object in this case).
             aExpandedIntents = utils.invokeUnfoldingArrayArguments(this._getLinks.bind(this), [vArgs]);
 
             return aExpandedIntents;
@@ -1555,13 +1538,13 @@ sap.ui.define([
         /**
          * Returns a list of semantic objects of the intents the current user can navigate to.
          *
-         * @returns {jQuery.Deferred.promise} A promise that resolves with an array of strings representing the semantic objects
+         * @returns {jQuery.Promise} Resolves with an array of strings representing the semantic objects
          *   of the intents the current user can navigate to, or rejects with an error message.
          *   The returned array will not contain duplicates.
          *   <p>NOTE: the caller should not rely on the specific order the semantic objects appear in the returned array.</p>
-         * @public
-         * @alias sap.ushell.services.CrossApplicationNavigation#getDistinctSemanticObjects
+         *
          * @since 1.38.0
+         * @public
          */
         this.getDistinctSemanticObjects = function () {
             var oDeferred = new jQuery.Deferred();
@@ -1582,8 +1565,7 @@ sap.ui.define([
          *
          * @param {string[]} aIntents the intents (such as <code>["#AnObject-action?A=B&c=e"]</code>) to be checked
          * @param {object} [oComponent] the root component of the application
-         * @returns {jQuery.Promise} A <code>jQuery.Deferred</code> object's promise which is resolved with a map containing
-         *   the intents from <code>aIntents</code> as keys.
+         * @returns {jQuery.Promise} Resolves with a map containing the intents from <code>aIntents</code> as keys.
          *   The map values are objects with a property <code>supported</code> of type <code>boolean</code>.<br/>
          *   Example:
          *     <pre>
@@ -1609,10 +1591,10 @@ sap.ui.define([
          *     // request failed or other error
          *   });
          * </pre>
-         * @deprecated since 1.31. Please use {@link #isNavigationSupported} instead. Note that this has a slightly different response format.
+         *
          * @since 1.19.1
          * @public
-         * @alias sap.ushell.services.CrossApplicationNavigation#isIntentSupported
+         * @deprecated since 1.31. Please use {@link #isNavigationSupported} instead. Note that this has a slightly different response format.
          */
         this.isIntentSupported = function (aIntents, oComponent) {
             var oDeferred = new jQuery.Deferred();
@@ -1687,8 +1669,7 @@ sap.ui.define([
          *   }
          * </pre>
          * @param {object} [oComponent] The root component of the application
-         * @returns {jQuery.Promise} Promise of a <code>jQuery.Deferred</code> object that resolves to an array
-         *   of objects indicating whether the intent is supported or not. Each object has a property
+         * @returns {jQuery.Promise} Resolves to an array of objects indicating whether the intent is supported or not. Each object has a property
          *   <code>supported</code> of type <code>boolean</code>.
          *
          *   Example:
@@ -1742,7 +1723,6 @@ sap.ui.define([
          *
          * @since 1.32
          * @public
-         * @alias sap.ushell.services.CrossApplicationNavigation#isNavigationSupported
          */
         this.isNavigationSupported = function (aIntents, oComponent) {
             var oDeferred = new jQuery.Deferred();
@@ -1831,7 +1811,7 @@ sap.ui.define([
          *     -- componentData.startupParameters, componentData.config, componentData["sap-xapp-state"].
          * @param {object} [oOwnerComponent] If specified, the created component will be called within the context of the oOwnerComponent
          *   (via oOwnerComponent.runAsOwner(fn))
-         * @returns {jQuery.Promise} promise (component instance)
+         * @returns {jQuery.Promise} Resolves the component instance.
          * @since 1.32.0
          * @private
          */
@@ -1898,7 +1878,7 @@ sap.ui.define([
          *   Note that the `componentData` member is cloned for use by this method, and the following properties are unconditionally set by
          *   this method and should not be passed in `componentData`:
          *     -- componentData.startupParameters, componentData.config, componentData["sap-xapp-state"].
-         * @returns {Promise<object>} promise (component data)
+         * @returns {Promise<object>} Resolves the component data.
          * @since 1.88.0
          * @private
          */
@@ -2063,10 +2043,11 @@ sap.ui.define([
          * Get the app state object that was used for the current cross application navigation
          *
          * @param {object} oAppComponent - UI5 component, key will be extracted from component data
-         * @returns {jQuery.Promise} promise object returning the app state object.
+         * @returns {jQuery.Promise} Resolves the app state object.
          *   Note that this is an unmodifiable container and its data must be copied into a writable container!
          * @since 1.28
-         * @ignore SAP Internal usage only, beware! internally public, cannot be changed, but not part of the public documentation
+         * @private
+         * @ui5-restricted SAP Internal usage only, beware! internally public, cannot be changed, but not part of the public documentation
          */
         this.getStartupAppState = function (oAppComponent) {
             this._checkComponent(oAppComponent);
@@ -2093,10 +2074,11 @@ sap.ui.define([
          *
          * @param {object} oAppComponent - UI5 component, key will be extracted from component data
          * @param {object} sAppStateKey - the application state key. SAP internal usage only.
-         * @returns {jQuery.Promise} promise object returning the app state object.
+         * @returns {jQuery.Promise} Resolves the app state object.
          *   Note that this is an unmodifiable container and its data must be copied into a writable container!
          * @since 1.28
-         * @ignore SAP Internal usage only, beware! internally public, cannot be changed, but not part of the public documentation
+         * @private
+         * @ui5-restricted SAP Internal usage only, beware! internally public, cannot be changed, but not part of the public documentation
          */
         this.getAppState = function (oAppComponent, sAppStateKey) {
             // see stakeholders in SFIN etc.
@@ -2128,10 +2110,9 @@ sap.ui.define([
          * A lookup for a cross user app state will be performed.
          *
          * @param {object} sAppStateKeyOrArray - the application state key, or an array, see below. SAP internal usage only
-         * @returns {jQuery.Promise} promise object returning the data of an AppState object, or an empty <code>{}</code> javascript
+         * @returns {jQuery.Promise} Resolves the data of an AppState object, or an empty <code>{}</code> javascript
          *   object if the key could not be resolved or an error occurred!
          * @since 1.32
-         * @ignore SAP Internal usage only, beware! internally public, cannot be changed, but not part of the public documentation
          *
          * This is interface exposed to platforms who need a serializable form of the application state data
          *
@@ -2140,7 +2121,8 @@ sap.ui.define([
          *   the result of the response will an corresponding array of array
          *   <code>[[{asdata1}],[{asdata2}]</code>
          *
-         * @private internal usage(exposure to WebDynpro ABAP)
+         * @ui5-restricted SAP Internal usage only, beware! internally public, cannot be changed, but not part of the public documentation and exposure to WebDynpro ABAP
+         * @private
          */
         this.getAppStateData = function (sAppStateKeyOrArray) {
             return utils.invokeUnfoldingArrayArguments(this._getAppStateData.bind(this),
@@ -2152,10 +2134,9 @@ sap.ui.define([
          * A lookup for a cross user app state will be performed.
          *
          * @param {object} sAppStateKey - the application state key, or an array, see below. SAP internal usage only.
-         * @returns {jQuery.Promise} promise object returning the data of an AppState object, or an empty <code>{}</code> javascript
+         * @returns {jQuery.Promise} Resolves the data of an AppState object, or an empty <code>{}</code> javascript
          *   object if the key could not be resolved or an error occurred!
          * @since 1.32
-         * @ignore SAP Internal usage only, beware! internally public, cannot be changed, but not part of the public documentation.
          *
          * This is interface exposed to platforms who need a serializable form of the application state data
          *
@@ -2165,6 +2146,7 @@ sap.ui.define([
          *   <code>[[{asdata1}],[{asdata2}]</code>
          *
          * @private
+         * @ui5-restricted SAP Internal usage only, beware! internally public, cannot be changed, but not part of the public documentation.
          */
         this._getAppStateData = function (sAppStateKey) {
             var oDeferred = new jQuery.Deferred();
@@ -2192,11 +2174,11 @@ sap.ui.define([
         /**
          * persist multiple app states (in future potentially batched in a single roundtrip)
          *
-         * @param {Array} aAppStates Array of application States
-         * @returns {jQuery.Promise} a jQuery.Deferred. Returns a promise,
-         *   in case of success an array of individual save promise objects is returned as argument
+         * @param {object[]} aAppStates Array of application States
+         * @returns {jQuery.Promise} Resolves an array of individual save promise objects is returned as argument
          *   in case of a reject, individual responses are not available
-         * @private see remarks in getAppState
+         * @private
+         * @ui5-restricted SAP Internal usage only, beware! internally public, cannot be changed, but not part of the public documentation
          */
         this.saveMultipleAppStates = function (aAppStates) {
             var aResult = [];
@@ -2212,7 +2194,7 @@ sap.ui.define([
             return oDeferred.promise();
         };
 
-        /*
+        /**
          * Process navigation parameters to allow "shellHash" together with parameters section, something that is currently not allowed.
          * This functionality is enabled only upon flag that is added to the navigation parameters. e.g.:
          * <pre>
@@ -2281,7 +2263,7 @@ sap.ui.define([
          * @param {int} iPersistencyMethod The chosen persistency method
          * @param {object} oPersistencySettings The additional settings PersistencySettings
          *
-         * @returns {jQuery.Promise<string>} A promise which resolves a new key
+         * @returns {jQuery.Promise<string>}Resolves a new key.
          */
         this.makeStatePersistent = function (sKey, iPersistencyMethod, oPersistencySettings) {
             var oDeferred = new jQuery.Deferred();
@@ -2303,7 +2285,7 @@ sap.ui.define([
          *
          * @param {string} sHashFragment The formatted URL hash fragment in internal format (as obtained by the SAPUI5 hasher service) not as given in <code>location.hash</code>)!
          *   Example: <code>#SemanticObject-action?P1=V1&P2=A%20B%20C</code>
-         * @returns {jQuery.Promise} A jQuery.Promise.
+         * @returns {jQuery.Promise} Resolves the intent.
          * @protected
          */
         this.resolveIntent = function (sHashFragment) {

package/src/main/js/sap/ushell/services/DarkModeSupport.js

@@ -5,20 +5,20 @@
  *
  * The service provide dark mode support functionality
  *
- * @version 1.120.8
+ * @version 1.121.0
  */
 sap.ui.define([
     "sap/base/Log",
-    "sap/ui/core/Configuration",
     "sap/ui/core/Core",
     "sap/ui/Device",
+    "sap/ui/core/Theming",
     "sap/ushell/Config",
     "sap/ushell/EventHub"
 ], function (
     Log,
-    Configuration,
     Core,
     Device,
+    Theming,
     Config,
     EventHub
 ) {
@@ -33,7 +33,7 @@ sap.ui.define([
      *
      * <b>Note:</b> To retrieve a valid instance of this service, it is necessary to call
      * <code>sap.ushell.Container.getServiceAsync("DarkModeSupport")</code>. For details, see
-     * {@link sap.ushell.services.Container#getServiceAsync}.
+     * {@link sap.ushell.Container#getServiceAsync}.
      *
      * @hideconstructor
      *
@@ -51,7 +51,7 @@ sap.ui.define([
     }
 
     /**
-     * @name sap.ushell.services.DarkModeSupport.Mode
+     * @alias sap.ushell.services.DarkModeSupport.Mode
      * @since 1.72.0
      * @private
      */
@@ -228,7 +228,7 @@ sap.ui.define([
     };
 
     DarkModeSupport.prototype._getCurrentTheme = function () {
-        return Configuration.getTheme();
+        return Theming.getTheme();
     };
 
     DarkModeSupport.prototype._findSupportedThemePair = function (sCurrentTheme) {

package/src/main/js/sap/ushell/services/EndUserFeedback.js

@@ -10,28 +10,30 @@ sap.ui.define([
     "use strict";
 
     /**
-     * This method MUST be called by the Unified Shell's container only, others MUST call <code>sap.ushell.Container.getServiceAsync("EndUserFeedback").then(function (EndUserFeedback) {});</code>.
-     * Constructs a new instance of the end user feedback service.
-     *
      * @alias sap.ushell.services.EndUserFeedback
      * @class
-     * @classdesc The Unified Shell's end user feedback service.
-     * This service is deprecated and does nothing.
+     * @classdesc The Unified Shell's end user feedback service. This service is deprecated and does nothing.
      * End user feedback functionality is not part of the ushell library.
      *
-     * @public
-     * @see sap.ushell.services.Container#getServiceAsync
+     * <b>Note:</b> To retrieve a valid instance of this service, it is necessary to call
+     * <code>sap.ushell.Container.getServiceAsync("DarkModeSupport")</code>. For details, see
+     * {@link sap.ushell.Container#getServiceAsync}.
+     *
+     * @augments sap.ushell.services.Service
+     * @hideconstructor
+     *
+     * @see sap.ushell.Container#getServiceAsync
      * @since 1.25.1
+     * @public
      * @deprecated since 1.93
      */
     function EndUserFeedback () {
         /**
          * Sends a feedback.
-         *
-         * @public
          * @returns {Promise} Empty promise
-         * @alias sap.ushell.services.EndUserFeedback#sendFeedback
+         *
          * @since 1.25.1
+         * @public
          * @deprecated since 1.93
          */
         this.sendFeedback = function () {
@@ -40,9 +42,9 @@ sap.ui.define([
 
         /**
          * @returns {string} Empty string
-         * @public
-         * @alias sap.ushell.services.EndUserFeedback#getLegalText
+         *
          * @since 1.25.1
+         * @public
          * @deprecated since 1.93
          */
         this.getLegalText = function () {
@@ -52,10 +54,10 @@ sap.ui.define([
         /**
          * The service is deprecated. The function always returns a negative answer.
          *
-         * @returns {object} Rejected promise
-         * @public
-         * @alias sap.ushell.services.EndUserFeedback#isEnabled
+         * @returns {Promise} Rejected promise.
+         *
          * @since 1.25.1
+         * @public
          * @deprecated since 1.93
          */
         this.isEnabled = function () {