@sapui5/sap.ushell 1.140.0 → 1.141.0

package/src/main/js/sap/ushell/components/shell/SideNavigation/modules/GenericNavigationListProvider.js

@@ -1,11 +1,17 @@
 // Copyright (c) 2009-2025 SAP SE, All Rights Reserved
 
 sap.ui.define([
+    "sap/base/Log",
+    "sap/tnt/NavigationList",
     "sap/ushell/Config",
-    "sap/tnt/NavigationList"
+    "sap/ushell/modules/NavigationMenu/ListProvider",
+    "sap/ushell/utils"
 ], (
+    Log,
+    NavigationList,
     Config,
-    NavigationList
+    ListProvider,
+    ushellUtils
 ) => {
     "use strict";
     /**
@@ -38,13 +44,15 @@ sap.ui.define([
      * }
      * </code>
      *
+     * @augments sap.ushell.modules.NavigationMenu.ListProvider
+     *
      * @since 1.134.0
      * @private
      */
-    class GenericNavigationListProvider {
+    class GenericNavigationListProvider extends ListProvider {
         /**
          * The side navigation API object passed to the constructor.
-         * @type {object}
+         * @type {sap.ushell.modules.NavigationMenu.ListProviderAPI}
          */
         #oSideNavAPI;
 
@@ -60,18 +68,48 @@ sap.ui.define([
          */
         #oMyHome;
 
+        /**
+         * Instance of the RecentActivity module.
+         * @type {object}
+         */
+        #oRecentActivity;
+
+        /**
+         * Instance of the Favorites module.
+         * @type {object}
+         */
+        #oFavorites;
+
+        /**
+         * Instance of the Spaces module.
+         * @type {object}
+         */
+        #oSpaces;
+
+        #oInitDeferred = Promise.withResolvers();
+
         /**
          * Creates a NavigationList for the item aggregation of the SideNavigation.
          *
-         * @param {object} oSideNavAPI The side navigation API object.
+         * @param {sap.ushell.modules.NavigationMenu.ListProviderAPI} oSideNavAPI The side navigation API object.
          *
          * @since 1.134.0
          * @private
          */
         constructor (oSideNavAPI) {
+            super(oSideNavAPI);
+
             this.#oSideNavAPI = oSideNavAPI;
             this.#oRootItem = new NavigationList();
-            this.#buildSideNavigation();
+
+            this.#buildSideNavigation().finally(this.#oInitDeferred.resolve);
+        }
+
+        /**
+         * @returns {Promise} Resolves once the init is done.
+         */
+        #initialized () {
+            return this.#oInitDeferred.promise;
         }
 
         /**
@@ -82,6 +120,8 @@ sap.ui.define([
         * @since 1.134.0
         */
         async getRootItem () {
+            await this.#initialized();
+
             return this.#oRootItem;
         }
 
@@ -91,10 +131,11 @@ sap.ui.define([
         * @since 1.134.0
         */
         async findSelectedKey () {
-            let sSelectedKey;
-            sSelectedKey = await this.#oMyHome?.findSelectedKey();
+            await this.#initialized();
+
+            let sSelectedKey = await this.#oMyHome?.findSelectedKey();
             if (!sSelectedKey) {
-                sSelectedKey = (await this.pSpaces).findSelectedKey();
+                sSelectedKey = await this.#oSpaces?.findSelectedKey();
             }
             return sSelectedKey;
         }
@@ -106,60 +147,74 @@ sap.ui.define([
         * @private
         * @since 1.134.0
         */
-        #buildSideNavigation () {
+        async #buildSideNavigation () {
             this.oChildItems = {};
 
-            const fnAddItemsToAggregation = () => {
-                this.#oRootItem.addAggregation("items", this.oChildItems.myHome);
-                this.#oRootItem.addAggregation("items", this.oChildItems.recentActivity);
-                this.#oRootItem.addAggregation("items", this.oChildItems.favorites);
-                this.#oRootItem.addAggregation("items", this.oChildItems.spaces);
-            };
-
             if (Config.last("/core/spaces/myHome/userEnabled")) {
-                sap.ui.require(["sap/ushell/components/shell/SideNavigation/modules/MyHome"], (MyHome) => {
+                try {
+                    const [MyHome] = await ushellUtils.requireAsync(["sap/ushell/components/shell/SideNavigation/modules/MyHome"]);
+
                     this.#oMyHome = new MyHome(this.#oSideNavAPI);
-                    this.#oMyHome.getRootItem().then((oItem) => {
-                        this.oChildItems.myHome = oItem;
-                        fnAddItemsToAggregation();
-                    });
-                });
+                    const oRootItem = await this.#oMyHome.getRootItem();
+                    this.oChildItems.myHome = oRootItem;
+                    this.#addItemsToAggregation();
+                } catch (oError) {
+                    Log.error("Error loading MyHome module:", oError, "SideNavigation");
+                }
             }
 
             if (this.#oSideNavAPI.getConfigValue("recentActivity.enabled")) {
-                sap.ui.require(["sap/ushell/components/shell/SideNavigation/modules/RecentActivity"], (RecentActivity) => {
-                    const oRecentActivity = new RecentActivity(this.#oSideNavAPI);
-                    oRecentActivity.getRootItem().then((oItem) => {
-                        this.oChildItems.recentActivity = oItem;
-                        fnAddItemsToAggregation();
-                    });
-                });
+                try {
+                    const [RecentActivity] = await ushellUtils.requireAsync(["sap/ushell/components/shell/SideNavigation/modules/RecentActivity"]);
+
+                    this.#oRecentActivity = new RecentActivity(this.#oSideNavAPI);
+                    const oRootItem = await this.#oRecentActivity.getRootItem();
+                    this.oChildItems.recentActivity = oRootItem;
+                    this.#addItemsToAggregation();
+                } catch (oError) {
+                    Log.error("Error loading RecentActivity module:", oError, "SideNavigation");
+                }
             }
 
             if (this.#oSideNavAPI.getConfigValue("favorites.enabled")) {
-                sap.ui.require(["sap/ushell/components/shell/SideNavigation/modules/Favorites"], (Favorites) => {
-                    const oFavorites = new Favorites(this.#oSideNavAPI);
-                    oFavorites.getRootItem().then((oItem) => {
-                        this.oChildItems.favorites = oItem;
-                        fnAddItemsToAggregation();
-                    });
-                });
+                try {
+                    const [Favorites] = await ushellUtils.requireAsync(["sap/ushell/components/shell/SideNavigation/modules/Favorites"]);
+
+                    this.#oFavorites = new Favorites(this.#oSideNavAPI);
+                    const oRootItem = await this.#oFavorites.getRootItem();
+                    this.oChildItems.favorites = oRootItem;
+                    this.#addItemsToAggregation();
+                } catch (oError) {
+                    Log.error("Error loading Favorites module:", oError, "SideNavigation");
+                }
             }
 
-            this.pSpaces = new Promise((resolve) => {
-                if (this.#oSideNavAPI.getConfigValue("spaces.enabled")) {
-                    sap.ui.require(["sap/ushell/components/shell/SideNavigation/modules/Spaces"], (Spaces) => {
-                        const oSpaces = new Spaces(this.#oSideNavAPI);
-                        oSpaces.getRootItem().then((oItem) => {
-                            this.oChildItems.spaces = oItem;
-                            fnAddItemsToAggregation();
-                        });
-                        resolve(oSpaces);
-                    });
+            if (this.#oSideNavAPI.getConfigValue("spaces.enabled")) {
+                try {
+                    const [Spaces] = await ushellUtils.requireAsync(["sap/ushell/components/shell/SideNavigation/modules/Spaces"]);
+
+                    this.#oSpaces = new Spaces(this.#oSideNavAPI);
+                    const oRootItem = await this.#oSpaces.getRootItem();
+                    this.oChildItems.spaces = oRootItem;
+                    this.#addItemsToAggregation();
+                } catch (oError) {
+                    Log.error("Error loading Spaces module:", oError, "SideNavigation");
                 }
-            });
+            }
+        }
+
+        /**
+         * Adds the child items to the root item's aggregation.
+         *
+         * @private
+         */
+        #addItemsToAggregation () {
+            this.#oRootItem.addAggregation("items", this.oChildItems.myHome);
+            this.#oRootItem.addAggregation("items", this.oChildItems.recentActivity);
+            this.#oRootItem.addAggregation("items", this.oChildItems.favorites);
+            this.#oRootItem.addAggregation("items", this.oChildItems.spaces);
         }
     }
 
     return GenericNavigationListProvider;
-}, false);
+});

package/src/main/js/sap/ushell/components/shell/SideNavigation/modules/MyHome.js

@@ -20,7 +20,7 @@ sap.ui.define([
     /**
      * The menu entry "My Home" for the SideNavigation.
      *
-     * @property {object} oSideNavAPI Holds the side navigation API object passed to the constructor.
+     * @property {sap.ushell.modules.NavigationMenu.ListProviderAPI} oSideNavAPI Holds the side navigation API object passed to the constructor.
      * @property {sap.tnt.NavigationListItem} oRootItem The root navigation list item configured with the MyHome.
      * @since 1.134.0
      * @private
@@ -29,7 +29,7 @@ sap.ui.define([
         /**
          * Creates a new menu entry "My Home" for the SideNavigation.
          *
-         * @param {object} oSideNavAPI The side navigation API object.
+         * @param {sap.ushell.modules.NavigationMenu.ListProviderAPI} oSideNavAPI The side navigation API object.
          *
          * @since 1.134.0
          * @private

package/src/main/js/sap/ushell/components/shell/SideNavigation/modules/RecentActivity.js

@@ -22,7 +22,7 @@ sap.ui.define([
     /**
      * The menu entry "Recently Used Apps" with the apps for the SideNavigation.
      *
-     * @property {object} oSideNavAPI Holds the configuration object passed to the constructor.
+     * @property {sap.ushell.modules.NavigationMenu.ListProviderAPI} oSideNavAPI Holds the configuration object passed to the constructor.
      * @property {sap.tnt.NavigationListItem} oRootItem The root navigation list item configured with the apps.
      * @property {Promise<sap.tnt.NavigationListItem>} oItemReady A promise that resolves once the recents model is prepared and ready.
      *
@@ -33,7 +33,7 @@ sap.ui.define([
         /**
         * Creates a new menu entry "Recently Used Apps" with the apps for the SideNavigation.
         *
-        * @param {object} oSideNavAPI The side navigation API object.
+        * @param {sap.ushell.modules.NavigationMenu.ListProviderAPI} oSideNavAPI The side navigation API object.
         *
         * @since 1.134.0
         * @private

package/src/main/js/sap/ushell/components/shell/SideNavigation/modules/Spaces.js

@@ -13,6 +13,7 @@ sap.ui.define([
     "sap/ushell/Config",
     "sap/ushell/Container",
     "sap/ushell/components/shell/SideNavigation/modules/NavigationHelper",
+    "sap/ushell/services/AppConfiguration",
     "sap/ushell/utils/UrlParsing"
 ], (
     ObjectPath,
@@ -27,6 +28,7 @@ sap.ui.define([
     Config,
     Container,
     NavigationHelper,
+    AppConfiguration,
     UrlParsing
 ) => {
     "use strict";
@@ -83,7 +85,7 @@ sap.ui.define([
         /**
          * Creates a new menu entry "My Spaces" with spaces and pages for the SideNavigation.
          *
-         * @param {object} oSideNavAPI The side navigation API object.
+         * @param {sap.ushell.modules.NavigationMenu.ListProviderAPI} oSideNavAPI The side navigation API object.
          *
          * @since 1.134.0
          * @private
@@ -177,7 +179,10 @@ sap.ui.define([
             const oHashParts = UrlParsing.parseShellHash(sHash);
             const sSpaceId = ObjectPath.get("params.spaceId.0", oHashParts);
             const sPageId = ObjectPath.get("params.pageId.0", oHashParts);
-            const sAppIdHint = ObjectPath.get("params.sap-ui-app-id-hint.0", oHashParts);
+
+            // first use hint from the hint from url, second the actual resolved app id (which might be different)
+            const sAppIdHint = ObjectPath.get("params.sap-ui-app-id-hint.0", oHashParts)
+                || AppConfiguration.getCurrentApplication()?.appId;
 
             oRootItem = oRootItem || await this.getRootItem();
             if (oHashParts.semanticObject === "Shell" && oHashParts.action === "home") {
@@ -234,18 +239,24 @@ sap.ui.define([
         #hasSpaceIdAndPageId (oSideNavigationEntry, sSpaceId, sPageId, sAppIdHint) {
             const aParameters = ObjectPath.get("target.parameters", oSideNavigationEntry) || [];
 
-            const bSpaceIdFound = aParameters.some((oParameter) => {
+            const bSpaceIdMatched = aParameters.some((oParameter) => {
                 return oParameter.name === "spaceId" && oParameter.value === sSpaceId;
             });
-            const bPageIdFound = aParameters.some((oParameter) => {
+            const bPageIdMatched = aParameters.some((oParameter) => {
                 return oParameter.name === "pageId" && oParameter.value === sPageId;
             });
-            const bAppIdHintFound = aParameters.some((oParameter) => {
+            const bAppIdHintMatched = aParameters.some((oParameter) => {
                 return oParameter.name === "sap-ui-app-id-hint" && oParameter.value === sAppIdHint;
             });
-            const bNoPageAndNoApp = !sPageId && !sAppIdHint;
 
-            return bSpaceIdFound && (bPageIdFound || bAppIdHintFound || bNoPageAndNoApp);
+            if (!sSpaceId) {
+                return bPageIdMatched || bAppIdHintMatched;
+            }
+
+            // when no pageId and no appId is provided, match only by spaceId
+            const bNoPageIdAndNoAppIdProvided = !sPageId && !sAppIdHint;
+
+            return bSpaceIdMatched && (bPageIdMatched || bAppIdHintMatched || bNoPageIdAndNoAppIdProvided);
         }
 
         /**

package/src/main/js/sap/ushell/components/shell/SideNavigation/modules/SpacesNavigationListProvider.js

@@ -4,12 +4,14 @@ sap.ui.define([
     "sap/tnt/NavigationList",
     "sap/ui/model/Filter",
     "sap/ui/model/FilterOperator",
-    "sap/ushell/components/shell/SideNavigation/modules/Spaces"
+    "sap/ushell/components/shell/SideNavigation/modules/Spaces",
+    "sap/ushell/modules/NavigationMenu/ListProvider"
 ], (
     NavigationList,
     Filter,
     FilterOperator,
-    Spaces
+    Spaces,
+    ListProvider
 ) => {
     "use strict";
 
@@ -36,19 +38,23 @@ sap.ui.define([
      *
      * @property {sap.tnt.NavigationList} oRootItem The root navigation list that will be populated with items from the spaces model.
      *
+     * @augments sap.ushell.modules.NavigationMenu.ListProvider
+     *
      * @since 1.134.0
      * @private
      */
-    class SpacesNavigationListProvider {
+    class SpacesNavigationListProvider extends ListProvider {
         /**
         * Creates a new menu with spaces and pages for the SideNavigation.
         *
-        * @param {object} oSideNavAPI The side navigation API object.
+        * @param {sap.ushell.modules.NavigationMenu.ListProviderAPI} oSideNavAPI The side navigation API object.
         *
         * @since 1.134.0
         * @private
         */
         constructor (oSideNavAPI) {
+            super(oSideNavAPI);
+
             this.oRootItem = new NavigationList();
             this.oSpaces = new Spaces(oSideNavAPI);
             this.pSpaces = this.oSpaces.getRootItem().then((oItem) => {
@@ -91,4 +97,4 @@ sap.ui.define([
     }
 
     return SpacesNavigationListProvider;
-}, false);
+});

package/src/main/js/sap/ushell/components/shell/UserActionsMenu/Component.js

@@ -14,7 +14,7 @@ sap.ui.define([
     return UIComponent.extend("sap.ushell.components.shell.UserActionsMenu.Component", {
 
         metadata: {
-            version: "1.140.0",
+            version: "1.141.0",
             library: "sap.ushell",
             dependencies: {
                 libs: ["sap.m"]

package/src/main/js/sap/ushell/components/shell/UserImage/Component.js

@@ -39,7 +39,7 @@ sap.ui.define([
     return UIComponent.extend("sap.ushell.components.shell.UserImage.Component", {
 
         metadata: {
-            version: "1.140.0",
+            version: "1.141.0",
             library: "sap.ushell.components.shell.UserImage",
             dependencies: {
                 libs: {

package/src/main/js/sap/ushell/components/tiles/action/ActionTile.controller.js

@@ -5,8 +5,9 @@ sap.ui.define([
     "sap/ushell/components/tiles/utils",
     "sap/ui/model/json/JSONModel",
     "sap/ui/thirdparty/jquery",
-    "sap/base/Log"
-], (Controller, tileUtils, JSONModel, jQuery, Log) => {
+    "sap/base/Log",
+    "sap/ushell/utils/LaunchpadError"
+], (Controller, tileUtils, JSONModel, jQuery, Log, LaunchpadError) => {
     "use strict";
 
     return Controller.extend("sap.ushell.components.tiles.action.ActionTile", {
@@ -67,24 +68,18 @@ sap.ui.define([
             const oTileModel = oModel.getProperty("/tileModel");
             const oTileApi = oConfigurationView.getViewData().chip;
 
-            // error handler
-            function logErrorAndReject (oError) {
-                Log.warning(oError, null, "sap.ushell.components.tiles.action.ActionTile.controller");
-                oDeferred.reject(oError);
-            }
-
             // If the input fields icon, semantic object and action are failing the input validations, then through an error message requesting the
             // user to enter/correct those fields
             const bReject = tileUtils.checkTMInputOnSaveConfig(oConfigurationView);
             if (bReject) {
-                oDeferred.reject("mandatory_fields_missing");
+                oDeferred.reject(new Error("mandatory_fields_missing"));
                 return oDeferred.promise();
             }
             // Before saving the model data, check if Mapping signature table contains duplicate parameter names
             // in this case the save will fail and all the data will be lost as this is the designer behavior.
             const oBundle = tileUtils.getResourceBundle();
             if (tileUtils.tableHasDuplicateParameterNames(oModel.getProperty("/config/rows"))) {
-                oDeferred.reject(oBundle.getText("configuration.signature.uniqueParamMessage.text"));
+                oDeferred.reject(new Error(oBundle.getText("configuration.signature.uniqueParamMessage.text")));
             } else if (!tileUtils.tableHasInvalidSapPrelaunchOperationValue(oModel.getProperty("/config/rows"))) {
                 const sSapPrelaunchOperationsTemplate = JSON.stringify([{
                     type: "split",
@@ -95,7 +90,7 @@ sap.ui.define([
                     source: ["p4", "p5"],
                     target: "p6"
                 }]);
-                oDeferred.reject(oBundle.getText("configuration.signature.invalidSapPrelaunchOperationsMessage.text", [sSapPrelaunchOperationsTemplate]));
+                oDeferred.reject(new Error(oBundle.getText("configuration.signature.invalidSapPrelaunchOperationsMessage.text", [sSapPrelaunchOperationsTemplate])));
             } else { // only if the data is valid proceed with the save operation
                 // Decide according to special flag if the setting in form factor are default
                 // if so , the configuration should not be saved - this is crucial for the backend checks
@@ -157,10 +152,23 @@ sap.ui.define([
                         () => {
                             Log.debug("property bag 'tileProperties' saved successfully");
                             oDeferred.resolve();
-                        }, logErrorAndReject // error handler
+                        // fnFailure
+                        }, (mErrorMessages, mErrorInfo) => {
+                            const sFirstFailedBag = Object.keys(mErrorMessages)[0];
+
+                            const oError = new LaunchpadError(`Bag save failed: ${mErrorMessages[sFirstFailedBag]}`, {
+                                info: mErrorInfo[sFirstFailedBag],
+                                otherMessages: mErrorMessages,
+                                otherInfo: mErrorInfo
+                            });
+                            Log.error("property bag 'tileProperties' save failed", oError, "sap.ushell.components.tiles.action.ActionTile.controller");
+                            oDeferred.reject(oError);
+                        }
                     );
-                }, logErrorAndReject // error handler
-                );
+                }, (sErrorMessage) => { // error handler
+                    Log.error(`property bag 'tileProperties' save failed: ${sErrorMessage}`, null, "sap.ushell.components.tiles.action.ActionTile.controller");
+                    oDeferred.reject(new Error(sErrorMessage));
+                });
             }
             return oDeferred.promise();
         },

package/src/main/js/sap/ushell/components/tiles/applauncher/StaticTile.controller.js

@@ -12,7 +12,8 @@ sap.ui.define([
     "sap/ui/thirdparty/jquery",
     "sap/ushell/utils/UrlParsing",
     "sap/ushell/EventHub",
-    "sap/ushell/Container"
+    "sap/ushell/Container",
+    "sap/ushell/utils/LaunchpadError"
 ], (
     Controller,
     tileUtils,
@@ -25,7 +26,8 @@ sap.ui.define([
     jQuery,
     UrlParsing,
     EventHub,
-    Container
+    Container,
+    LaunchpadError
 ) => {
     "use strict";
 
@@ -259,10 +261,6 @@ sap.ui.define([
             tilePropertiesBag.setText("display_info_text", oConfigToSave.display_info_text);
             tilePropertiesBag.setText("display_search_keywords", oConfigToSave.display_search_keywords);
 
-            function logErrorAndReject (oError) {
-                Log.error(oError, null, "sap.ushell.components.tiles.applauncher.StaticTile.controller");
-            }
-
             // saving the relevant properties
             tilePropertiesBag.save(
                 // success handler
@@ -278,7 +276,17 @@ sap.ui.define([
                     // call to refresh model which (due to the binding) will refresh the tile
                     this.getView().getModel().refresh();
                 },
-                logErrorAndReject // error handler
+                // fnFailure
+                (mErrorMessages, mErrorInfo) => {
+                    const sFirstFailedBag = Object.keys(mErrorMessages)[0];
+
+                    const oError = new LaunchpadError(`Bag save failed: ${mErrorMessages[sFirstFailedBag]}`, {
+                        info: mErrorInfo[sFirstFailedBag],
+                        otherMessages: mErrorMessages,
+                        otherInfo: mErrorInfo
+                    });
+                    Log.error("Bag save failed", oError, "sap.ushell.components.tiles.applauncher.StaticTile.controller");
+                }
             );
         },
 
@@ -309,7 +317,7 @@ sap.ui.define([
                 bReject = tileUtils.checkTileActions(oConfigurationView);
             }
             if (bReject) {
-                oDeferred.reject("mandatory_fields_missing");
+                oDeferred.reject(new Error("mandatory_fields_missing"));
                 return oDeferred.promise();
             }
             // overwrite target URL in case of semantic object navigation
@@ -329,9 +337,9 @@ sap.ui.define([
             // forward populating of tile navigation actions array into the bag, to Utils
             tileUtils.populateTileNavigationActionsBag(tileNavigationActionsBag, aTileNavigationActions);
 
-            function logErrorAndReject (oError, oErrorInfo) {
-                Log.error(oError, null, "sap.ushell.components.tiles.applauncher.StaticTile.controller");
-                oDeferred.reject(oError, oErrorInfo);
+            function logErrorAndReject (sErrorMessage, oErrorInfo) {
+                Log.error(`property bag 'tileProperties' save failed: ${sErrorMessage}`, null, "sap.ushell.components.tiles.applauncher.StaticTile.controller");
+                oDeferred.reject(new LaunchpadError(sErrorMessage, oErrorInfo));
             }
 
             // use configuration contract to write parameter values
@@ -380,7 +388,18 @@ sap.ui.define([
                                 oDeferred.resolve();
                             }
                         },
-                        logErrorAndReject // error handler
+                        // fnFailure
+                        (mErrorMessages, mErrorInfo) => {
+                            const sFirstFailedBag = Object.keys(mErrorMessages)[0];
+
+                            const oError = new LaunchpadError(`Bag save failed: ${mErrorMessages[sFirstFailedBag]}`, {
+                                info: mErrorInfo[sFirstFailedBag],
+                                otherMessages: mErrorMessages,
+                                otherInfo: mErrorInfo
+                            });
+                            Log.error("Bag save failed", oError, "sap.ushell.components.tiles.applauncher.StaticTile.controller");
+                            oDeferred.reject(oError);
+                        }
                     );
 
                     tileNavigationActionsBag.save(
@@ -388,7 +407,18 @@ sap.ui.define([
                         () => {
                             Log.debug("property bag 'navigationProperties' saved successfully");
                         },
-                        logErrorAndReject // error handler
+                        // fnFailure
+                        (mErrorMessages, mErrorInfo) => {
+                            const sFirstFailedBag = Object.keys(mErrorMessages)[0];
+
+                            const oError = new LaunchpadError(`Bag save failed: ${mErrorMessages[sFirstFailedBag]}`, {
+                                info: mErrorInfo[sFirstFailedBag],
+                                otherMessages: mErrorMessages,
+                                otherInfo: mErrorInfo
+                            });
+                            Log.error("Bag save failed", oError, "sap.ushell.components.tiles.applauncher.StaticTile.controller");
+                            oDeferred.reject(oError);
+                        }
                     );
                 },
                 logErrorAndReject // error handler