@sapui5/sap.ushell 1.138.1 → 1.139.0

package/src/main/js/sap/ushell/components/shell/Settings/notifications/NotificationsSetting.controller.js

@@ -8,7 +8,7 @@ sap.ui.define([
     "sap/ushell/resources",
     "sap/base/util/deepClone",
     "sap/ushell/Container"
-], function (
+], (
     Controller,
     JSONModel,
     Config,
@@ -16,37 +16,36 @@ sap.ui.define([
     resources,
     deepClone,
     Container
-) {
+) => {
     "use strict";
 
     return Controller.extend("sap.ushell.components.shell.Settings.notifications.NotificationsSetting", {
         onInit: function () {
             this._oModel = new JSONModel({
-                    rows: [],
-                    originalRows: []
-                });
-            var oView = this.getView(),
-                oResponseData;
+                rows: [],
+                originalRows: []
+            });
+            const oView = this.getView();
+            let oResponseData;
 
             oView.setBusy(true);
             oView.setModel(resources.i18nModel, "i18n");
 
             Container.getServiceAsync("NotificationsV2")
-                .then(function (Service) {
+                .then((Service) => {
                     this._oNotificationService = Service;
                     this._initializeFlags();
                     Service.readSettings()
-                        .then(function (oResult) {
+                        .then((oResult) => {
                             oResponseData = JSON.parse(oResult);
                             this._oModel.setProperty("/rows", oResponseData.value);
                             this._oModel.setProperty("/originalRows", deepClone(oResponseData.value));
-                        }.bind(this))
-                        .finally(function () {
+                        })
+                        .finally(() => {
                             oView.setBusy(false);
                             oView.setModel(this._oModel);
-                        }.bind(this));
-                }.bind(this));
-
+                        });
+                });
         },
 
         /**
@@ -56,8 +55,8 @@ sap.ui.define([
          * @private
          */
         onAfterRendering: function () {
-            var oFlags = this._oModel.getProperty("/flags"),
-                aRows = this._oModel.getProperty("/rows");
+            const oFlags = this._oModel.getProperty("/flags");
+            const aRows = this._oModel.getProperty("/rows");
 
             // After controller initialization there might be a case in which the data still wasn't fetched from the backend.
             // In this case aRows and oFlags could be undefined, hence we can't set originalRows and originalFlags yet.
@@ -76,8 +75,8 @@ sap.ui.define([
          * @private
          */
         onCancel: function () {
-            var oOriginalFlags = this._oModel.getProperty("/originalFlags"),
-                oOriginalRows = this._oModel.getProperty("/originalRows");
+            const oOriginalFlags = this._oModel.getProperty("/originalFlags");
+            const oOriginalRows = this._oModel.getProperty("/originalRows");
 
             this._oModel.setProperty("/flags/highPriorityBannerEnabled", oOriginalFlags.highPriorityBannerEnabled);
             this._oModel.setProperty("/rows", deepClone(oOriginalRows));
@@ -93,14 +92,14 @@ sap.ui.define([
          * @private
          */
         onSave: function () {
-            var aRows = this._oModel.getProperty("/rows"),
-                aOriginalRows = this._oModel.getProperty("/originalRows"),
-                oRow,
-                oOriginalRow,
-                aSettingsToSave = [],
-                aSaveDeferres = [];
-
-            for (var iIndex = 0; iIndex < aRows.length; iIndex++) {
+            const aRows = this._oModel.getProperty("/rows");
+            const aOriginalRows = this._oModel.getProperty("/originalRows");
+            let oRow;
+            let oOriginalRow;
+            const aSettingsToSave = [];
+            const aSaveDeferres = [];
+
+            for (let iIndex = 0; iIndex < aRows.length; iIndex++) {
                 oRow = aRows[iIndex];
                 oOriginalRow = aOriginalRows[iIndex];
                 if (this._notficationSettingsDiffer(oRow, oOriginalRow)) {
@@ -112,9 +111,9 @@ sap.ui.define([
                 this._handleFlagsSave();
             }
 
-            aSettingsToSave.forEach(function (setting) {
+            aSettingsToSave.forEach((setting) => {
                 aSaveDeferres.push(this._oNotificationService.saveSettingsEntry(setting));
-            }.bind(this));
+            });
             return Promise.all(aSaveDeferres).catch((error) => {
                 Log.error(
                     "Failed to save the notification settings in the notifivation service.", error,
@@ -123,7 +122,6 @@ sap.ui.define([
             });
         },
 
-
         /**
          * Checks if flags were changed by the user.
          *
@@ -142,7 +140,7 @@ sap.ui.define([
          * @private
          */
         _handleFlagsSave: function () {
-            var bHighPrioEnabled = this._oModel.getProperty("/flags/highPriorityBannerEnabled");
+            const bHighPrioEnabled = this._oModel.getProperty("/flags/highPriorityBannerEnabled");
             this._oModel.setProperty("/originalFlags/highPriorityBannerEnabled", bHighPrioEnabled);
 
             this._oNotificationService.setUserSettingsFlags({
@@ -173,8 +171,8 @@ sap.ui.define([
          * @private
          */
         onSelectMobile: function (event) {
-            var sPath = event.getSource().getBindingContext().getPath();
-            this._oModel.setProperty(sPath + "/DoNotDeliverMob", !event.getParameter("selected"));
+            const sPath = event.getSource().getBindingContext().getPath();
+            this._oModel.setProperty(`${sPath}/DoNotDeliverMob`, !event.getParameter("selected"));
         },
 
         /**
@@ -184,8 +182,8 @@ sap.ui.define([
          * @private
          */
         onSelectEmail: function (event) {
-            var sPath = event.getSource().getBindingContext().getPath();
-            this._oModel.setProperty(sPath + "/DoNotDeliverEmail", !event.getParameter("selected"));
+            const sPath = event.getSource().getBindingContext().getPath();
+            this._oModel.setProperty(`${sPath}/DoNotDeliverEmail`, !event.getParameter("selected"));
         },
 
         /**
@@ -195,11 +193,11 @@ sap.ui.define([
          * @private
          */
         onSelectHighPriority: function (event) {
-            var sPath = event.getSource().getBindingContext().getPath();
+            const sPath = event.getSource().getBindingContext().getPath();
             if (event.getParameter("selected")) {
-                this._oModel.setProperty(sPath + "/PriorityDefault", "HIGH");
+                this._oModel.setProperty(`${sPath}/PriorityDefault`, "HIGH");
             } else {
-                this._oModel.setProperty(sPath + "/PriorityDefault", "");
+                this._oModel.setProperty(`${sPath}/PriorityDefault`, "");
             }
         },
 
@@ -210,12 +208,12 @@ sap.ui.define([
          * @private
          */
         onEnableSwitchChange: function (event) {
-            var bNewState = event.getParameter("state"),
-                sPath = event.getSource().getBindingContext().getPath();
-            this._oModel.setProperty(sPath + "/DoNotDeliver", !bNewState);
-            this._oModel.setProperty(sPath + "/DoNotDeliverMob", true);
-            this._oModel.setProperty(sPath + "/DoNotDeliverEmail", true);
-            this._oModel.setProperty(sPath + "/PriorityDefault", "");
+            const bNewState = event.getParameter("state");
+            const sPath = event.getSource().getBindingContext().getPath();
+            this._oModel.setProperty(`${sPath}/DoNotDeliver`, !bNewState);
+            this._oModel.setProperty(`${sPath}/DoNotDeliverMob`, true);
+            this._oModel.setProperty(`${sPath}/DoNotDeliverEmail`, true);
+            this._oModel.setProperty(`${sPath}/PriorityDefault`, "");
         },
 
         /**
@@ -225,19 +223,19 @@ sap.ui.define([
          * @private
          */
         _initializeFlags: function () {
-            var oSwitchBarDataPromise = this._oNotificationService.getUserSettingsFlags(),
-                bMobilePushEnabled = this._oNotificationService._getNotificationSettingsMobileSupport(),
-                bEmailPushEnabled = this._oNotificationService._getNotificationSettingsEmailSupport();
+            const oSwitchBarDataPromise = this._oNotificationService.getUserSettingsFlags();
+            const bMobilePushEnabled = this._oNotificationService._getNotificationSettingsMobileSupport();
+            const bEmailPushEnabled = this._oNotificationService._getNotificationSettingsEmailSupport();
 
             oSwitchBarDataPromise
-                .then(function (oSwitchBarData) {
+                .then((oSwitchBarData) => {
                     this._oModel.setProperty("/flags", {});
                     this._oModel.setProperty("/flags/highPriorityBannerEnabled", oSwitchBarData.highPriorityBannerEnabled);
                     this._oModel.setProperty("/flags/mobileNotificationsEnabled", bMobilePushEnabled);
                     this._oModel.setProperty("/flags/emailNotificationsEnabled", bEmailPushEnabled);
                     this._oModel.setProperty("/originalFlags", {});
                     this._oModel.setProperty("/originalFlags/highPriorityBannerEnabled", oSwitchBarData.highPriorityBannerEnabled);
-                }.bind(this));
+                });
         }
     });
 });

package/src/main/js/sap/ushell/components/shell/Settings/search/Search.controller.js

@@ -1,6 +1,6 @@
 // Copyright (c) 2009-2025 SAP SE, All Rights Reserved
 
-sap.ui.define(["sap/ui/core/mvc/View", "sap/ui/core/mvc/Controller"], function (View, Controller) {
+sap.ui.define(["sap/ui/core/mvc/View", "sap/ui/core/mvc/Controller"], (View, Controller) => {
     "use strict";
 
     return Controller.extend(
@@ -11,14 +11,14 @@ sap.ui.define(["sap/ui/core/mvc/View", "sap/ui/core/mvc/Controller"], function (
             },
 
             _loadContent: function () {
-                var oVBox = this.getView().byId("searchContent");
+                const oVBox = this.getView().byId("searchContent");
 
                 View.create({
                     id: "searchPrefsDialogView",
                     viewName: "module:sap/esh/search/ui/userpref/SearchPrefsDialog.view"
-                  }).then(function (oView) {
+                }).then((oView) => {
                     oVBox.addItem(oView);
-                  });
+                });
             },
 
             onCancel: function () {

package/src/main/js/sap/ushell/components/shell/Settings/search/SearchEntry.js

@@ -5,26 +5,26 @@ sap.ui.define([
     "sap/ui/core/Lib",
     "sap/ui/core/mvc/XMLView",
     "sap/ushell/resources"
-], function (
+], (
     Log,
     Library,
     XMLView,
     resources
-) {
+) => {
     "use strict";
 
-    var searchPrefsModelPromise = null;
+    let searchPrefsModelPromise = null;
 
     function getSearchPrefsModel () {
         if (searchPrefsModelPromise) {
             return searchPrefsModelPromise;
         }
-        searchPrefsModelPromise = Library.load("sap.esh.search.ui").then(function () {
-            return new Promise(function (resolve, reject) {
+        searchPrefsModelPromise = Library.load("sap.esh.search.ui").then(() => {
+            return new Promise((resolve, reject) => {
                 sap.ui.require([
                     "sap/esh/search/ui/userpref/SearchPrefsModel",
                     "sap/esh/search/ui/SearchShellHelperAndModuleLoader"
-                ], function (SearchPrefsModel, SearchShellHelperAndModuleLoader) {
+                ], (SearchPrefsModel, SearchShellHelperAndModuleLoader) => {
                     resolve(new SearchPrefsModel());
                 });
             });
@@ -33,9 +33,9 @@ sap.ui.define([
     }
 
     function getEntry () {
-        return getSearchPrefsModel().then(function (model) {
-            var oViewInstance;
-            var oEntry = {
+        return getSearchPrefsModel().then((model) => {
+            let oViewInstance;
+            const oEntry = {
                 id: "search",
                 entryHelpID: "search",
                 title: resources.i18n.getText("searchSetting"),
@@ -44,14 +44,14 @@ sap.ui.define([
                 icon: "sap-icon://search",
                 isActive: model.isSearchPrefsActive.bind(model),
                 valueArgument: function () {
-                    return model.isMultiProvider().then(function (isMultiProvider) {
+                    return model.isMultiProvider().then((isMultiProvider) => {
                         if (isMultiProvider) {
                             return {
                                 value: 1,
                                 displayText: ""
                             };
                         }
-                        return model.isPersonalizedSearchActive().then(function (status) {
+                        return model.isPersonalizedSearchActive().then((status) => {
                             return {
                                 value: 1,
                                 displayText: status ? resources.i18n.getText("sp.persSearchOn") : resources.i18n.getText("sp.persSearchOff")
@@ -62,14 +62,14 @@ sap.ui.define([
                 contentFunc: function () {
                     return model
                         .asyncInit()
-                        .then(function () {
+                        .then(() => {
                             return XMLView.create({
                                 id: "searchView",
                                 viewName:
                                     "sap.ushell.components.shell.Settings.search.Search"
                             });
                         })
-                        .then(function (oView) {
+                        .then((oView) => {
                             oView.setModel(model);
                             oViewInstance = oView;
                             return oView;

package/src/main/js/sap/ushell/components/shell/Settings/spaces/SpacesEntry.js

@@ -7,26 +7,26 @@ sap.ui.define([
     "sap/ushell/Config",
     "sap/ushell/utils",
     "sap/ushell/Container"
-], function (
+], (
     Log,
     View,
     resources,
     Config,
     utils,
     Container
-) {
+) => {
     "use strict";
 
     function _getShowMyHome () {
-        return Container.getServiceAsync("UserInfo").then(function (oUserInfo) {
-            var bShow = oUserInfo.getUser().getShowMyHome();
+        return Container.getServiceAsync("UserInfo").then((oUserInfo) => {
+            const bShow = oUserInfo.getUser().getShowMyHome();
             return resources.i18n.getText(bShow ? "settingsMyHomeShown" : "settingsMyHomeHidden");
         });
     }
 
     // Show "My Home shown/hidden" subtitle only when both Spaces and MyHome are enabled
     function _getSubtitleCallback () {
-        var bShowSubtitle = Config.last("/core/spaces/enabled") && Config.last("/core/spaces/myHome/enabled") && !Config.last("/core/homeApp/enabled");
+        const bShowSubtitle = Config.last("/core/spaces/enabled") && Config.last("/core/spaces/myHome/enabled") && !Config.last("/core/homeApp/enabled");
         return bShowSubtitle ? _getShowMyHome : null;
     }
 
@@ -44,8 +44,8 @@ sap.ui.define([
                 }
 
                 // "Show My Home" checkbox
-                var bMyHomeEnabled = Config.last("/core/spaces/myHome/enabled");
-                var bHomeAppEnabled = Config.last("/core/homeApp/enabled");
+                const bMyHomeEnabled = Config.last("/core/spaces/myHome/enabled");
+                const bHomeAppEnabled = Config.last("/core/homeApp/enabled");
                 if (bMyHomeEnabled && !bHomeAppEnabled) { // Home App hides ushell My Home
                     return true;
                 }
@@ -54,7 +54,7 @@ sap.ui.define([
             return false;
         },
         getEntry: function () {
-            var oViewInstance;
+            let oViewInstance;
             return {
                 id: "spacesEntry",
                 entryHelpID: "spaces",
@@ -69,8 +69,8 @@ sap.ui.define([
                         // ensure that the config was updated with the user setting
                         utils.getHideEmptySpacesEnabled()
                     ])
-                        .then(function (aResults) {
-                            var oUserInfoService = aResults[0];
+                        .then((aResults) => {
+                            const oUserInfoService = aResults[0];
                             return View.create({
                                 id: "UserSettingsSpacesSettingsView",
                                 viewName: "sap.ushell.components.shell.Settings.spaces.SpacesSetting",
@@ -80,7 +80,7 @@ sap.ui.define([
                                 }
                             });
                         })
-                        .then(function (oView) {
+                        .then((oView) => {
                             oViewInstance = oView;
                             return oViewInstance;
                         });
@@ -91,7 +91,6 @@ sap.ui.define([
                     }
                     Log.warning("Save operation for user account settings was not executed, because the spaces view was not initialized");
                     return Promise.resolve();
-
                 },
                 onCancel: function () {
                     if (oViewInstance) {
@@ -111,5 +110,4 @@ sap.ui.define([
             };
         }
     };
-
 });

package/src/main/js/sap/ushell/components/shell/Settings/spaces/SpacesSetting.controller.js

@@ -8,7 +8,7 @@ sap.ui.define([
     "sap/ushell/resources",
     "sap/ushell/utils",
     "sap/base/Log"
-], function (
+], (
     Controller,
     JSONModel,
     EventHub,
@@ -16,26 +16,26 @@ sap.ui.define([
     resources,
     utils,
     Log
-) {
+) => {
     "use strict";
 
     return Controller.extend("sap.ushell.components.shell.Settings.spaces.SpacesSetting", {
         onInit: function () {
-            var oViewData = this.getView().getViewData();
+            const oViewData = this.getView().getViewData();
             this.oUserInfoService = oViewData.UserInfo;
-            var oUser = this.oUserInfoService.getUser();
+            const oUser = this.oUserInfoService.getUser();
 
-            var bSpacesEnabled = Config.last("/core/spaces/enabled");
-            var bHomeAppEnabled = Config.last("/core/homeApp/enabled");
-            var bMyHomeEnabled = Config.last("/core/spaces/myHome/enabled");
+            const bSpacesEnabled = Config.last("/core/spaces/enabled");
+            const bHomeAppEnabled = Config.last("/core/homeApp/enabled");
+            const bMyHomeEnabled = Config.last("/core/spaces/myHome/enabled");
 
-            var bShowMyHomeVisible = bSpacesEnabled && bMyHomeEnabled && !bHomeAppEnabled;
-            var bShowMyHomeSelected = oUser.getShowMyHome();
-            var sImportFlag = oUser.getImportBookmarksFlag();
-            var bMyHomeImportVisible = sImportFlag !== "done" && sImportFlag !== "not_required";
+            const bShowMyHomeVisible = bSpacesEnabled && bMyHomeEnabled && !bHomeAppEnabled;
+            const bShowMyHomeSelected = oUser.getShowMyHome();
+            const sImportFlag = oUser.getImportBookmarksFlag();
+            const bMyHomeImportVisible = sImportFlag !== "done" && sImportFlag !== "not_required";
 
             // If the FLP is in classic mode, do not show My Home and Import settigs - we do not know the values of other flags.
-            var oViewModel = new JSONModel({
+            const oViewModel = new JSONModel({
                 spaces: {
                     visible: Config.last("/core/spaces/configurable"),
                     selected: bSpacesEnabled
@@ -56,12 +56,11 @@ sap.ui.define([
             this.getView().setModel(oViewModel);
             this.getView().setModel(resources.i18nModel, "i18n");
 
-
             this.oImportBookmarksFlagListener = EventHub.on("importBookmarksFlag").do(this._setMyHomeImportSelected.bind(this));
         },
 
         _setMyHomeImportSelected: function (value) {
-            var oViewModel = this.getView().getModel();
+            const oViewModel = this.getView().getModel();
             oViewModel.setProperty("/showMyHomeImport/selected", value);
         },
 
@@ -70,25 +69,25 @@ sap.ui.define([
         },
 
         onSave: function (fnUpdateUserPreferences) {
-            var bRestart = false;
-            var bUpdate = false;
-            var aUpdatePromises = [];
+            let bRestart = false;
+            let bUpdate = false;
+            const aUpdatePromises = [];
 
-            var oModel = this.getView().getModel();
-            var oUserInfoService = this.oUserInfoService;
-            var oUser = oUserInfoService.getUser();
+            const oModel = this.getView().getModel();
+            const oUserInfoService = this.oUserInfoService;
+            const oUser = oUserInfoService.getUser();
 
-            var bOldSpacesEnabled = Config.last("/core/spaces/enabled");
-            var bNewSpacesEnabled = oModel.getProperty("/spaces/selected");
+            const bOldSpacesEnabled = Config.last("/core/spaces/enabled");
+            const bNewSpacesEnabled = oModel.getProperty("/spaces/selected");
 
-            var bOldHideEmptySpaces = Config.last("/core/spaces/hideEmptySpaces/userEnabled");
-            var bNewHideEmptySpaces = oModel.getProperty("/hideEmptySpaces/selected");
+            const bOldHideEmptySpaces = Config.last("/core/spaces/hideEmptySpaces/userEnabled");
+            const bNewHideEmptySpaces = oModel.getProperty("/hideEmptySpaces/selected");
 
-            var bOldShowMyHome = oUser.getShowMyHome();
-            var bNewShowMyHome = oModel.getProperty("/showMyHome/selected");
+            const bOldShowMyHome = oUser.getShowMyHome();
+            const bNewShowMyHome = oModel.getProperty("/showMyHome/selected");
 
-            var bOldShowImportMessage = oUser.getImportBookmarksFlag() === "pending";
-            var bNewShowImportMessage = oModel.getProperty("/showMyHomeImport/selected");
+            const bOldShowImportMessage = oUser.getImportBookmarksFlag() === "pending";
+            const bNewShowImportMessage = oModel.getProperty("/showMyHomeImport/selected");
 
             // Set and persist changed user preferences
             if (bOldSpacesEnabled !== bNewSpacesEnabled) {
@@ -105,7 +104,7 @@ sap.ui.define([
                 bRestart = true;
             }
             if (bOldShowImportMessage !== bNewShowImportMessage) {
-                var bImportFlag = bNewShowImportMessage ? "pending" : "dismissed";
+                const bImportFlag = bNewShowImportMessage ? "pending" : "dismissed";
                 oUser.setImportBookmarksFlag(bImportFlag);
                 bUpdate = true;
 
@@ -115,8 +114,8 @@ sap.ui.define([
             }
 
             if (bOldHideEmptySpaces !== bNewHideEmptySpaces) {
-                var oHideEmptySpacesPromise = utils.setHideEmptySpacesEnabled(bNewHideEmptySpaces)
-                    .catch(function (sErrorMessage) {
+                const oHideEmptySpacesPromise = utils.setHideEmptySpacesEnabled(bNewHideEmptySpaces)
+                    .catch((sErrorMessage) => {
                         oModel.setProperty("/hideEmptySpaces/selected", bOldHideEmptySpaces);
                         return Promise.reject(sErrorMessage);
                     });
@@ -129,15 +128,15 @@ sap.ui.define([
             }
             Log.debug("[000] SpacesSetting.controller.save");
             if (bUpdate) {
-                var oUserInfoPromise = new Promise(function (resolve, reject) {
+                const oUserInfoPromise = new Promise((resolve, reject) => {
                     fnUpdateUserPreferences()
-                        .then(function () {
+                        .then(() => {
                             oUser.resetChangedProperty("spacesEnabled");
                             oUser.resetChangedProperty("showMyHome");
                             oUser.resetChangedProperty("importBookmarks");
                             resolve();
                         })
-                        .catch(function (sErrorMessage) {
+                        .catch((sErrorMessage) => {
                             if (
                                 !sErrorMessage.includes("SPACES_ENABLEMENT")
                                 && !sErrorMessage.includes("MYHOME_ENABLEMENT")
@@ -148,13 +147,13 @@ sap.ui.define([
                                 oUser.resetChangedProperty("importBookmarks");
                                 resolve();
                             } else {
-                            oModel.setProperty("/spaces/selected", bOldSpacesEnabled);
-                            oModel.setProperty("/showMyHome/selected", bOldShowMyHome);
+                                oModel.setProperty("/spaces/selected", bOldSpacesEnabled);
+                                oModel.setProperty("/showMyHome/selected", bOldShowMyHome);
 
-                            oUser.resetChangedProperty("spacesEnabled");
-                            oUser.resetChangedProperty("showMyHome");
-                            oUser.resetChangedProperty("importBookmarks");
-                            reject(sErrorMessage);
+                                oUser.resetChangedProperty("spacesEnabled");
+                                oUser.resetChangedProperty("showMyHome");
+                                oUser.resetChangedProperty("importBookmarks");
+                                reject(sErrorMessage);
                             }
                         });
                 });
@@ -162,7 +161,7 @@ sap.ui.define([
             }
 
             return Promise.all(aUpdatePromises)
-                .then(function () {
+                .then(() => {
                     return {
                         refresh: bRestart,
                         noHash: true // Inform Controller, about reloading without Hash, to ensure we are not trying to Start the "MyHome" space by accident
@@ -171,7 +170,7 @@ sap.ui.define([
         },
 
         onCancel: function () {
-            var oModel = this.getView().getModel();
+            const oModel = this.getView().getModel();
             oModel.setProperty("/spaces/selected", Config.last("/core/spaces/enabled"));
             oModel.setProperty("/showMyHome/selected", this.oUserInfoService.getUser().getShowMyHome());
             oModel.setProperty("/hideEmptySpaces/selected", Config.last("/core/spaces/hideEmptySpaces/userEnabled"));

package/src/main/js/sap/ushell/components/shell/Settings/userAccount/UserAccountEntry.js

@@ -6,28 +6,28 @@ sap.ui.define([
     "sap/ushell/Config",
     "sap/ushell/resources",
     "sap/ushell/Container"
-], function (
+], (
     XMLView,
     Log,
     Config,
     resources,
     Container
-) {
+) => {
     "use strict";
 
     function getEntry () {
-        var oShellConfig = Container.getRendererInternal("fiori2").getShellConfig(),
-            bUseSelector = oShellConfig.enableUserImgConsent;
+        const oShellConfig = Container.getRendererInternal("fiori2").getShellConfig();
+        const bUseSelector = oShellConfig.enableUserImgConsent;
 
-        var sViewId = bUseSelector ? "userAccountSelector" : "userAccountSetting";
-        var sComponentNamespace = bUseSelector
+        const sViewId = bUseSelector ? "userAccountSelector" : "userAccountSetting";
+        const sComponentNamespace = bUseSelector
             ? "sap.ushell.components.shell.Settings.userAccount.UserAccountSelector"
             : "sap.ushell.components.shell.Settings.userAccount.UserAccountSetting";
 
-        var sIcon = Config.last("/core/shell/model/userImage/account") || "sap-icon://account";
-        var oViewInstance;
+        const sIcon = Config.last("/core/shell/model/userImage/account") || "sap-icon://account";
+        let oViewInstance;
 
-        var oEntry = {
+        const oEntry = {
             id: "userAccountEntry",
             entryHelpID: "userAccountEntry",
             title: resources.i18n.getText("UserAccountFld"),
@@ -35,12 +35,12 @@ sap.ui.define([
             contentResult: null,
             icon: sIcon,
             valueArgument: Container.getUser().getFullName(),
-            provideEmptyWrapper: true, //to hide the header and add custom header instead
+            provideEmptyWrapper: true, // to hide the header and add custom header instead
             contentFunc: function () {
                 return XMLView.create({
                     id: sViewId,
                     viewName: sComponentNamespace
-                }).then(function (oView) {
+                }).then((oView) => {
                     oViewInstance = oView;
                     return oView;
                 });
@@ -51,7 +51,6 @@ sap.ui.define([
                 }
                 Log.warning("Save operation for user account settings was not executed, because the userAccount view was not initialized");
                 return Promise.resolve();
-
             },
             onCancel: function () {
                 if (oViewInstance) {
@@ -67,10 +66,7 @@ sap.ui.define([
         return oEntry;
     }
 
-
-
     return {
         getEntry: getEntry
     };
-
 });