@sapui5/sap.ushell 1.136.3 → 1.138.0

package/src/main/js/sap/ushell/adapters/cep/NavTargetResolutionInternalAdapter.js

@@ -2,7 +2,7 @@
 
 /**
  * @fileoverview The NavTargetResolutionInternal adapter for SAP MyHome
- * @version 1.136.3
+ * @version 1.138.0
  * @since 1.120
  */
 
@@ -14,7 +14,7 @@ sap.ui.define([
     "sap/ushell/services/ClientSideTargetResolution/VirtualInbounds",
     "sap/ushell/utils/HttpClient",
     "sap/ushell/Config"
-], function (
+], (
     Localization,
     ObjectPath,
     jQuery,
@@ -22,7 +22,7 @@ sap.ui.define([
     oVirtualInbounds,
     HttpClient,
     Config
-) {
+) => {
     ("use strict");
     /** Constructs a new instance of  The NavTargetResolutionInternal adapter for SAP MyHome.
      * It calls the backend service to retrieve navigation data.
@@ -33,7 +33,7 @@ sap.ui.define([
      * @since 1.101.0
      * @private
      */
-    var NavTargetResolutionInternalAdapter = function (
+    function NavTargetResolutionInternalAdapter (
         oUnused,
         sParameter,
         oAdapterConfiguration
@@ -41,7 +41,7 @@ sap.ui.define([
         this.oAdapterConfiguration = oAdapterConfiguration;
         this._addVirtualInboundsToApplications();
         this._initializeHttpClient();
-    };
+    }
 
     /**
      * Adds virtual inbounds to applications
@@ -49,13 +49,13 @@ sap.ui.define([
      * @private
      */
     NavTargetResolutionInternalAdapter.prototype._addVirtualInboundsToApplications = function () {
-        var oApplications = ObjectPath.create(
+        const oApplications = ObjectPath.create(
             "config.applications",
             this.oAdapterConfiguration
         );
-        var aVirtualInbounds = oVirtualInbounds.getInbounds();
-        aVirtualInbounds.forEach(function (oInbound) {
-            var sIntent = oInbound.semanticObject + "-" + oInbound.action;
+        const aVirtualInbounds = oVirtualInbounds.getInbounds();
+        aVirtualInbounds.forEach((oInbound) => {
+            const sIntent = `${oInbound.semanticObject}-${oInbound.action}`;
             oApplications[sIntent] = oInbound.resolutionResult;
         });
     };
@@ -73,8 +73,8 @@ sap.ui.define([
      */
     NavTargetResolutionInternalAdapter.prototype._getIntentForService = function (sHashFragment) {
         return Container.getServiceAsync("URLParsing").then(
-            function (URLParsing) {
-                var oHashFragment = URLParsing.parseShellHash(sHashFragment);
+            (URLParsing) => {
+                const oHashFragment = URLParsing.parseShellHash(sHashFragment);
 
                 return {
                     semanticObject: oHashFragment.semanticObject,
@@ -94,13 +94,13 @@ sap.ui.define([
      * @since 1.110.0
      */
     NavTargetResolutionInternalAdapter.prototype._initializeHttpClient = function () {
-        var oHeaders = {
+        const oHeaders = {
             "Content-Type": "application/json; charset=utf-8",
             Accept: "application/json",
             "Accept-Language": Localization.getLanguageTag().toString() || ""
         };
 
-        var sBasePath = this.oAdapterConfiguration.config.navServiceUrl + "/";
+        const sBasePath = `${this.oAdapterConfiguration.config.navServiceUrl}/`;
         this.oHttpClient = new HttpClient(sBasePath, { headers: oHeaders });
     };
 
@@ -110,9 +110,9 @@ sap.ui.define([
      * @returns {fn} wrapped function
      */
     NavTargetResolutionInternalAdapter.prototype.toPromise = function (fn) {
-        var that = this;
+        const that = this;
         return function (mArg) {
-            var oDeferred = new jQuery.Deferred();
+            const oDeferred = new jQuery.Deferred();
             fn.call(that, mArg)
                 .then(oDeferred.resolve)
                 .catch(oDeferred.reject)
@@ -146,18 +146,18 @@ sap.ui.define([
         sHashFragment
     ) {
         return this._getIntentForService(sHashFragment)
-            .then(function (oIntentForService) {
-                var sResolvePath = this.oAdapterConfiguration.config.navServiceUrl + "/resolve";
+            .then((oIntentForService) => {
+                const sResolvePath = `${this.oAdapterConfiguration.config.navServiceUrl}/resolve`;
                 return this.oHttpClient.post(sResolvePath, { data: JSON.stringify(oIntentForService) })
-                    .then(function (oResponse) {
+                    .then((oResponse) => {
                         if (oResponse.status < 200 || oResponse.status >= 300) {
-                            return Promise.reject("HTTP request to navigation service failed with status: " + oResponse.status + " - " + oResponse.statusText);
+                            return Promise.reject(`HTTP request to navigation service failed with status: ${oResponse.status} - ${oResponse.statusText}`);
                         }
                         return JSON.parse(oResponse.responseText);
                     })
-                    .then(function (oResponseData) {
-                        var oData = oResponseData.value || oResponseData;
-                        var sUrl = Array.isArray(oData) ? oData[0].url : oData.url;
+                    .then((oResponseData) => {
+                        const oData = oResponseData.value || oResponseData;
+                        const sUrl = Array.isArray(oData) ? oData[0].url : oData.url;
                         return {
                             additionalInformation: "",
                             url: sUrl,
@@ -165,7 +165,7 @@ sap.ui.define([
                             navigationMode: "newWindow"
                         };
                     });
-            }.bind(this));
+            });
     };
 
     /**
@@ -185,11 +185,11 @@ sap.ui.define([
      * @since 1.121.0
      */
     NavTargetResolutionInternalAdapter.prototype.isIntentSupported = function (aIntents) {
-        var oIntents = {};
+        const oIntents = {};
         aIntents.forEach((elem, i) => {
             oIntents[elem] = { supported: true };
         });
-        var oDeferred = new jQuery.Deferred();
+        const oDeferred = new jQuery.Deferred();
         oDeferred.resolve(oIntents);
         return oDeferred.promise();
     };

package/src/main/js/sap/ushell/adapters/cep/SearchCEPAdapter.js

@@ -2,7 +2,7 @@
 /**
  * @fileOverview The Unified Shell's Search adapter for CEP applications (client-side search).
  *
- * @version 1.136.3
+ * @version 1.138.0
  */
 sap.ui.define([
     "sap/ui/core/Lib",
@@ -10,13 +10,13 @@ sap.ui.define([
     "sap/ui/model/Filter",
     "sap/ui/model/Sorter",
     "sap/ushell/Container"
-], function (
+], (
     Library,
     JSONModel,
     Filter,
     Sorter,
     Container
-) {
+) => {
     "use strict";
     /**
      * The Unified Shell's Search adapter for SAP Start (client-side search).
@@ -31,10 +31,9 @@ sap.ui.define([
      * @since 1.115.0
      * @private
      */
-    var SearchCEPAdapter = function (oSystem, sParameters, oConfig) {
-    };
+    function SearchCEPAdapter (oSystem, sParameters, oConfig) { }
 
-     /**
+    /**
      * Searching applications via the a client side search service.
      *
      * @param {object} oParameters The parameters for the search
@@ -58,13 +57,15 @@ sap.ui.define([
 
     /**
      * Search using the SearchableContent service and the jsSearch implementation
+     *
+     * @param {object} oParameters a map-like object containing the parameters.
+     * @returns {Promise<object>} the result of the search.
      */
     SearchCEPAdapter.prototype._searchWithSearchableContentAndJsSearch = function (oParameters) {
-
-        return Promise.all([this._getNormalizedSearchableContentApps(oParameters), this._loadJsSearchModule()]).then(function (aPromiseResults) {
-            var aNormalizedSearchableContentApps = aPromiseResults[0];
-            var JsSearch = aPromiseResults[1];
-            var oSearchEngine = new JsSearch({
+        return Promise.all([this._getNormalizedSearchableContentApps(oParameters), this._loadJsSearchModule()]).then((aPromiseResults) => {
+            const aNormalizedSearchableContentApps = aPromiseResults[0];
+            const JsSearch = aPromiseResults[1];
+            const oSearchEngine = new JsSearch({
                 objects: aNormalizedSearchableContentApps,
                 fields: ["title", "label", "contentProviderLabel", "description", "keywords", "fioriAppId"],
                 shouldNormalize: !String.prototype.normalize,
@@ -73,13 +74,13 @@ sap.ui.define([
                     options: [50, 49, 40, 39, 5, 4, 51]
                 }
             });
-            var oSearchOptions = {
-                searchFor: oParameters && oParameters.searchTerm || "*", // set wildcard if no search term specified
+            const oSearchOptions = {
+                searchFor: oParameters?.searchTerm || "*", // set wildcard if no search term specified
                 top: Number.MAX_SAFE_INTEGER, // apply pagination after search due to filtering
                 skip: 0
             };
-            var oSearchResult = oSearchEngine.search(oSearchOptions);
-            var aResultData = oSearchResult.results.map(function (o) {
+            const oSearchResult = oSearchEngine.search(oSearchOptions);
+            const aResultData = oSearchResult.results.map((o) => {
                 return o.object;
             });
 
@@ -88,8 +89,7 @@ sap.ui.define([
                 oParameters.skip || 0,
                 oParameters.top || 10
             );
-
-        }.bind(this));
+        });
     };
 
     SearchCEPAdapter.prototype._getPaginatedResult = function (aResultData, iSkip, iTop) {
@@ -100,42 +100,42 @@ sap.ui.define([
     };
 
     SearchCEPAdapter.prototype._getSearchableContent = function (oParameters) {
-        var oSearchableContentParameters = oParameters && {
+        const oSearchableContentParameters = oParameters && {
             includeAppsWithoutVisualizations: !!oParameters.includeAppsWithoutVisualizations,
             enableVisualizationPreview: oParameters.enableVisualizationPreview !== false
         };
 
         return Container.getServiceAsync("SearchableContent")
-            .then(function (oService) {
+            .then((oService) => {
                 return oService.getApps(oSearchableContentParameters);
             });
     };
 
     SearchCEPAdapter.prototype._getNormalizedSearchableContentApps = function (oParameters) {
-        return this._getSearchableContent(oParameters).then(function (aApps) {
+        return this._getSearchableContent(oParameters).then((aApps) => {
             return this._normalizeSearchableContentServiceResult(aApps);
-        }.bind(this));
+        });
     };
 
     SearchCEPAdapter.prototype._normalizeSearchableContentServiceResult = function (aApps) {
         if (!Array.isArray(aApps)) {
             aApps = [];
         }
-        var aResultApps = [];
-        var title;
-        var description;
-        var longDescription;
-        var label;
+        const aResultApps = [];
+        let title;
+        let description;
+        let longDescription;
+        let label;
         let fioriAppIds;
 
-        aApps.forEach(function (oApp) {
+        aApps.forEach((oApp) => {
             if (oApp.visualizations && oApp.visualizations.length > 0) {
-                oApp.visualizations.forEach(function (oVis) {
+                oApp.visualizations.forEach((oVis) => {
                     title = (oVis.vizConfig && oVis.vizConfig["sap.app"] && oVis.vizConfig["sap.app"].title) || oVis.title || oApp.title;
                     description = (oVis.vizConfig && oVis.vizConfig["sap.app"] && oVis.vizConfig["sap.app"].subTitle) || oVis.subtitle || oApp.subtitle;
                     longDescription = (oVis.vizConfig && oVis.vizConfig["sap.app"] && oVis.vizConfig["sap.app"].description) || oVis.description || oApp.description;
                     fioriAppIds = oVis?.vizConfig?.["sap.fiori"]?.registrationIds || [];
-                    label = description ? title + " - " + description : title;
+                    label = description ? `${title} - ${description}` : title;
                     aResultApps.push({
                         fioriAppId: fioriAppIds.join(" "),
                         title: title,
@@ -153,7 +153,7 @@ sap.ui.define([
             } else {
                 title = oApp.title;
                 description = oApp.subtitle;
-                label = description ? title + " - " + description : title;
+                label = description ? `${title} - ${description}` : title;
 
                 aResultApps.push({
                     title: title,
@@ -171,12 +171,11 @@ sap.ui.define([
     };
 
     SearchCEPAdapter.prototype._loadJsSearchModule = function () {
-
-        this._loadJsSearchModulePromise = new Promise(function (resolve) {
+        this._loadJsSearchModulePromise = new Promise((resolve) => {
             // always load the library first to ensure async loading
             // (even if the JsSearch module is self-contained)
-            Library.load("sap.esh.search.ui").then(function () {
-                sap.ui.require(["sap/esh/search/ui/appsearch/JsSearch"], function (JsSearch) {
+            Library.load("sap.esh.search.ui").then(() => {
+                sap.ui.require(["sap/esh/search/ui/appsearch/JsSearch"], (JsSearch) => {
                     resolve(JsSearch);
                 });
             });
@@ -195,12 +194,12 @@ sap.ui.define([
      * @private
      * @since 1.115.0
      */
-     SearchCEPAdapter.prototype._createFilters = function (vFilters) {
-        var aFilters = null;
+    SearchCEPAdapter.prototype._createFilters = function (vFilters) {
+        let aFilters = null;
         if (vFilters) {
             if (typeof vFilters === "object") {
                 if (Array.isArray(vFilters.filters)) {
-                    vFilters.filters = vFilters.filters.map(function (oFilter) {
+                    vFilters.filters = vFilters.filters.map((oFilter) => {
                         return oFilter instanceof Filter ? oFilter : new Filter(oFilter);
                     });
                 }
@@ -221,13 +220,13 @@ sap.ui.define([
      * @since 1.115.0
      */
     SearchCEPAdapter.prototype._createSorters = function (vSorter) {
-        var aSorters = null;
+        let aSorters = null;
         if (vSorter) {
             if (typeof vSorter === "object" && !Array.isArray(vSorter)) {
                 vSorter = [vSorter];
             }
             if (Array.isArray(vSorter)) {
-                aSorters = vSorter.map(function (oSorter) {
+                aSorters = vSorter.map((oSorter) => {
                     return oSorter instanceof Sorter ? oSorter : new Sorter(oSorter);
                 });
             }
@@ -253,12 +252,12 @@ sap.ui.define([
         if (!vFilters && !vSorter) {
             return aData;
         }
-        var aFilters = this._createFilters(vFilters),
-            aSorters = this._createSorters(vSorter),
-            oModel = new JSONModel();
+        const aFilters = this._createFilters(vFilters);
+        const aSorters = this._createSorters(vSorter);
+        const oModel = new JSONModel();
         oModel.setSizeLimit(Number.MAX_SAFE_INTEGER);
         oModel.setData(aData);
-        return oModel.bindList("/", null, aSorters, aFilters).getContexts().map(function (oContext) {
+        return oModel.bindList("/", null, aSorters, aFilters).getContexts().map((oContext) => {
             return oContext.getObject();
         });
     };

package/src/main/js/sap/ushell/adapters/cflp/UserDefaultParameterPersistenceAdapter.js

@@ -4,7 +4,7 @@
  * @fileOverview The Unified Shell's UserDefaultParameterPersistence adapter for the CFLP
  *               platform.
  *
- * @version 1.136.3
+ * @version 1.138.0
  */
 sap.ui.define([
     "sap/ushell/utils/HttpClient",
@@ -12,13 +12,13 @@ sap.ui.define([
     "sap/base/Log",
     "sap/base/util/ObjectPath",
     "sap/ushell/Container"
-], function (
+], (
     HttpClient,
     jQuery,
     Logger,
     ObjectPath,
     Container
-) {
+) => {
     "use strict";
 
     /**
@@ -37,7 +37,7 @@ sap.ui.define([
      * @since 1.79.0
      * @private
      */
-    var UserDefaultParameterPersistenceAdapter = function (oSystem, sParameters, oConfig) {
+    function UserDefaultParameterPersistenceAdapter (oSystem, sParameters, oConfig) {
         this.sBaseServicePath = "/sap/opu/odata/UI2/INTEROP/";
 
         this.oConfig = {};
@@ -56,7 +56,7 @@ sap.ui.define([
             data: {}
         };
         this._mSystems = {};
-    };
+    }
 
     /**
      * @typedef {object} SystemContext
@@ -88,28 +88,28 @@ sap.ui.define([
      * @private
      */
     UserDefaultParameterPersistenceAdapter.prototype.loadParameterValue = function (sParameterName, oSystemContext) {
-        var oDeferred = new jQuery.Deferred();
-        var bIsValid = this._isValidParameterName(sParameterName);
+        const oDeferred = new jQuery.Deferred();
+        const bIsValid = this._isValidParameterName(sParameterName);
 
         if (bIsValid) {
             this._getUserDefaultParameterContainer(oSystemContext)
-                .done(function (aContainerItems) {
-                    var oItem = aContainerItems.find(function (oItem) {
+                .done((aContainerItems) => {
+                    const oItem = aContainerItems.find((oItem) => {
                         return oItem.id === sParameterName;
                     });
 
                     if (oItem !== undefined) {
-                        var oResponse = JSON.parse(oItem.value);
+                        const oResponse = JSON.parse(oItem.value);
                         oDeferred.resolve(oResponse);
                     } else {
-                        oDeferred.reject("Parameter does not exist: " + sParameterName);
+                        oDeferred.reject(`Parameter does not exist: ${sParameterName}`);
                     }
                 })
-                .fail(function (oResponseData) {
+                .fail((oResponseData) => {
                     oDeferred.reject(oResponseData);
                 });
         } else {
-            oDeferred.reject("Illegal Parameter Key. Parameter must be less than 40 characters and [A-Za-z0-9.-_]+ :\"" + sParameterName + "\"");
+            oDeferred.reject(`Illegal Parameter Key. Parameter must be less than 40 characters and [A-Za-z0-9.-_]+ :"${sParameterName}"`);
         }
         return oDeferred.promise();
     };
@@ -133,7 +133,7 @@ sap.ui.define([
                 category: "P",
                 // Unsure what to do with this value.
                 // This does not seem to be required for the request.
-                //clientExpirationTime: "/Date(253373439600000)/",
+                // clientExpirationTime: "/Date(253373439600000)/",
                 component: "",
                 id: "sap.ushell.UserDefaultParameter"
             },
@@ -163,22 +163,22 @@ sap.ui.define([
      * @private
      */
     UserDefaultParameterPersistenceAdapter.prototype.saveParameterValue = function (sParameterName, oValueObject, oSystemContext) {
-        var oDeferred = new jQuery.Deferred();
-        var bIsValid = this._isValidParameterName(sParameterName);
+        const oDeferred = new jQuery.Deferred();
+        const bIsValid = this._isValidParameterName(sParameterName);
 
         if (bIsValid) {
-            var oHttpWrapper = this._getXHttpWrapper(oSystemContext);
+            const oHttpWrapper = this._getXHttpWrapper(oSystemContext);
 
             // get cache or fetch container from server
             this._getUserDefaultParameterContainer(oSystemContext)
-                .done(function (aContainerItems) {
-                    var iIndex = aContainerItems.findIndex(function (element) {
+                .done((aContainerItems) => {
+                    const iIndex = aContainerItems.findIndex((element) => {
                         return element.id === sParameterName;
                     });
                     if (iIndex > -1) {
                         aContainerItems[iIndex].value = JSON.stringify(oValueObject);
                     } else {
-                        var oUserDefaultParameter = {
+                        const oUserDefaultParameter = {
                             category: "I",
                             containerCategory: "P",
                             containerId: "sap.ushell.UserDefaultParameter",
@@ -191,10 +191,10 @@ sap.ui.define([
                     oHttpWrapper.post("PersContainers", this._createRequest(aContainerItems))
                         .then(oDeferred.resolve)
                         .catch(oDeferred.reject);
-                }.bind(this))
+                })
                 .fail(oDeferred.reject);
         } else {
-            oDeferred.reject("Illegal Parameter Key. Parameter must be less than 40 characters and [A-Za-z0-9.-_]+ :\"" + sParameterName + "\"");
+            oDeferred.reject(`Illegal Parameter Key. Parameter must be less than 40 characters and [A-Za-z0-9.-_]+ :"${sParameterName}"`);
         }
 
         return oDeferred.promise();
@@ -217,17 +217,17 @@ sap.ui.define([
      * @private
      */
     UserDefaultParameterPersistenceAdapter.prototype.deleteParameter = function (sParameterName, oSystemContext) {
-        var oDeferred = new jQuery.Deferred();
-        var bIsValid = this._isValidParameterName(sParameterName);
+        const oDeferred = new jQuery.Deferred();
+        const bIsValid = this._isValidParameterName(sParameterName);
 
         if (bIsValid) {
-            var oHttpWrapper = this._getXHttpWrapper(oSystemContext);
+            const oHttpWrapper = this._getXHttpWrapper(oSystemContext);
 
             // HttpClient returns a native promise
             this._getUserDefaultParameterContainer(oSystemContext)
-                .done(function (aContainerItems) {
+                .done((aContainerItems) => {
                     // delete
-                    var iIndex = aContainerItems.findIndex(function (element) {
+                    const iIndex = aContainerItems.findIndex((element) => {
                         return element.id === sParameterName;
                     });
                     if (iIndex > -1) {
@@ -239,10 +239,10 @@ sap.ui.define([
                     } else {
                         oDeferred.resolve();
                     }
-                }.bind(this))
+                })
                 .fail(oDeferred.reject);
         } else {
-            oDeferred.reject("Illegal Parameter Key. Parameter must be less than 40 characters and [A-Za-z0-9.-_]+ :\"" + sParameterName + "\"");
+            oDeferred.reject(`Illegal Parameter Key. Parameter must be less than 40 characters and [A-Za-z0-9.-_]+ :"${sParameterName}"`);
         }
 
         return oDeferred.promise();
@@ -260,18 +260,18 @@ sap.ui.define([
      * @private
      */
     UserDefaultParameterPersistenceAdapter.prototype.getStoredParameterNames = function (oSystemContext) {
-        var oDeferred = new jQuery.Deferred();
+        const oDeferred = new jQuery.Deferred();
 
         this._getUserDefaultParameterContainer(oSystemContext)
-            .done(function (aContainerItems) {
-                var aKeys = aContainerItems.map(function (oItem) {
+            .done((aContainerItems) => {
+                const aKeys = aContainerItems.map((oItem) => {
                     return oItem.id;
                 });
 
                 aKeys.sort();
                 oDeferred.resolve(aKeys);
             })
-            .fail(function (oError) {
+            .fail((oError) => {
                 oDeferred.reject(oError);
             });
         return oDeferred.promise();
@@ -297,7 +297,7 @@ sap.ui.define([
             this._mSystems[oSystemContext.id] = {};
         }
         if (!this._mSystems[oSystemContext.id].oHttpWrapper) {
-            var sBaseUrl = oSystemContext.getFullyQualifiedXhrUrl(this.oConfig.sBaseServicePath);
+            const sBaseUrl = oSystemContext.getFullyQualifiedXhrUrl(this.oConfig.sBaseServicePath);
 
             this._mSystems[oSystemContext.id].oHttpWrapper = new HttpClient(sBaseUrl, this.oConfig.oHttpClientConfig);
         }
@@ -319,10 +319,10 @@ sap.ui.define([
      * @private
      */
     UserDefaultParameterPersistenceAdapter.prototype._isValidParameterName = function (sParameterName) {
-        var bIsValid = !!(typeof sParameterName === "string" && sParameterName.length <= 40 && /^[A-Za-z0-9.-_]+$/.exec(sParameterName));
+        const bIsValid = !!(typeof sParameterName === "string" && sParameterName.length <= 40 && /^[A-Za-z0-9.-_]+$/.exec(sParameterName));
 
         if (!bIsValid) {
-            Logger.error("Illegal Parameter Key. Parameter must be less than 40 characters and [A-Za-z0-9.-_]+ :\"" + sParameterName + "\"");
+            Logger.error(`Illegal Parameter Key. Parameter must be less than 40 characters and [A-Za-z0-9.-_]+ :"${sParameterName}"`);
         }
         return bIsValid;
     };
@@ -345,8 +345,8 @@ sap.ui.define([
      */
     UserDefaultParameterPersistenceAdapter.prototype._getUserDefaultParameterContainer = function (oSystemContext) {
         // Using a jQuery promise for consistency
-        var oDeferred = new jQuery.Deferred();
-        var oHttpWrapper;
+        const oDeferred = new jQuery.Deferred();
+        let oHttpWrapper;
 
         try {
             // _getXHttpWrapper initializes the cache that is accessed later.
@@ -358,29 +358,29 @@ sap.ui.define([
             return oDeferred.promise();
         }
 
-        var oSystem = this._mSystems[oSystemContext.id];
+        const oSystem = this._mSystems[oSystemContext.id];
 
         if (!oSystem.oContainerCache) {
             oSystem.oContainerCache = oDeferred;
 
-            var oRequestPromise = oHttpWrapper.get("PersContainers(category='P',id='sap.ushell.UserDefaultParameter')?$expand=PersContainerItems");
+            const oRequestPromise = oHttpWrapper.get("PersContainers(category='P',id='sap.ushell.UserDefaultParameter')?$expand=PersContainerItems");
 
             // HttpClient returns a native promise
             oRequestPromise
-                .then(function (oResponseData) { //resolve
+                .then((oResponseData) => { // resolve
                     oSystem.bDataReceived = true;
 
-                    var oResponse = oResponseData.responseText ? JSON.parse(oResponseData.responseText) : {};
-                    var aContainerItems = ObjectPath.get("d.PersContainerItems.results", oResponse) || [];
+                    const oResponse = oResponseData.responseText ? JSON.parse(oResponseData.responseText) : {};
+                    const aContainerItems = ObjectPath.get("d.PersContainerItems.results", oResponse) || [];
                     oSystem.oContainerCache.resolve(aContainerItems);
                 })
-                .catch(function (oResponseData) { //reject
+                .catch((oResponseData) => { // reject
                     // not found is not an error, it simply means
                     // there's nothing to load
                     if (oResponseData.status === 404) {
                         oSystem.oContainerCache.resolve([]);
                     } else {
-                        var sErrorMessage = "Server error: " + oResponseData.statusText;
+                        const sErrorMessage = `Server error: ${oResponseData.statusText}`;
                         oSystem.oContainerCache.reject(sErrorMessage);
                     }
                 }