@sapui5/sap.ushell_abap 1.139.0 → 1.141.0

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@sapui5/sap.ushell_abap",
-    "version": "1.139.0",
+    "version": "1.141.0",
     "description": "SAPUI5 Library sap.ushell_abap",
     "homepage": "https://sap.github.io/ui5-tooling/pages/SAPUI5/",
     "keywords": [
@@ -15,7 +15,7 @@
         "build-themes": "cross-env-shell ui5 build --exclude-task=* --include-task=buildThemes --framework-version $npm_package_version"
     },
     "devDependencies": {
-        "@ui5/cli": "^3.11.6",
+        "@ui5/cli": "^4.0.24",
         "cross-env": "^7.0.3"
     }
 }

package/src/main/js/sap/ushell_abap/.library

@@ -2,7 +2,7 @@
 <library xmlns="http://www.sap.com/sap.ui.library.xsd">
     <name>sap.ushell_abap</name>
     <vendor>SAP SE</vendor>
-    <version>1.139.0</version>
+    <version>1.141.0</version>
     <copyright>Copyright (c) 2009-2025 SAP SE, All Rights Reserved</copyright>
     <documentation>SAP library: sap.ushell_abap</documentation>
 

package/src/main/js/sap/ushell_abap/adapters/abap/AdapterContainer.js

@@ -6,14 +6,16 @@ sap.ui.define([
     "sap/ui/thirdparty/jquery",
     "sap/ui/core/format/DateFormat",
     "sap/ushell_abap/pbServices/ui2/ODataService",
-    "sap/ushell/services/_Personalization/constants"
+    "sap/ushell/services/_Personalization/constants",
+    "sap/ushell/utils/LaunchpadError"
 ], (
     ObjectPath,
     Log,
     jQuery,
     DateFormat,
     ODataService,
-    constants
+    constants,
+    LaunchpadError
 ) => {
     "use strict";
 
@@ -34,7 +36,7 @@ sap.ui.define([
         return sContainerKey.substring(sCONTAINER_KEY_PREFIX.length, sCONTAINER_KEY_PREFIX.length + 40);
     }
 
-    const AdapterContainer = function (sContainerKey, oService, oScope, sAppName) {
+    function AdapterContainer (sContainerKey, oService, oScope, sAppName) {
         this._oService = oService;
         this._oScope = oScope;
         this["sap-cache-id"] = ObjectPath.get("_oService._oConfig.services.personalization.cacheId", this);
@@ -55,7 +57,7 @@ sap.ui.define([
         };
         this._oPropertyBag = {};
         this._aOperationQueue = [];
-    };
+    }
 
     /**
      * Resets the container item values to initial ( retaining key, validity, etc!)
@@ -92,11 +94,12 @@ sap.ui.define([
             // response contains items.results (!)
             that._oJSONContainer.PersContainerItems = (that._oJSONContainer.PersContainerItems && that._oJSONContainer.PersContainerItems.results) || [];
             oDeferred.resolve(that);
-        }, (sErrorMessage) => {
+        }, (sErrorMessage, oParsedError) => {
             // this error handler used to resolve with an empty personalization container
             // but was changed to reject the promise so that the caller can react to the error
             Log.error(sErrorMessage);
-            oDeferred.reject(sErrorMessage);
+
+            oDeferred.reject(new LaunchpadError(sErrorMessage, oParsedError));
         });
         return oDeferred.promise();
     };
@@ -114,8 +117,8 @@ sap.ui.define([
 
         oDataWrapper.create(sRelativeURL, this._oJSONContainer, (/* response */) => {
             oDeferred.resolve(that);
-        }, (sErrorMessage) => {
-            oDeferred.reject(sErrorMessage);
+        }, (sErrorMessage, oParsedError) => {
+            oDeferred.reject(new LaunchpadError(sErrorMessage, oParsedError));
         });
         return oDeferred.promise();
     };
@@ -132,8 +135,8 @@ sap.ui.define([
 
         oDataWrapper.del(sRelativeURL, (response) => {
             oDeferred.resolve(that);
-        }, (sErrorMessage) => {
-            oDeferred.reject(sErrorMessage);
+        }, (sErrorMessage, oParsedError) => {
+            oDeferred.reject(new LaunchpadError(sErrorMessage, oParsedError));
         });
         this._reset();
         return oDeferred.promise();
@@ -200,8 +203,8 @@ sap.ui.define([
     };
 
     /**
-     * Locates an item for the key sItemKey,
-     * returns { index : nr,  TrueItemKey : truekey,  containerProperty : }
+     * @param {string} sItemKey - The item key.
+     * @returns {{index: int, TrueItemKey: string, containerProperty: string}} An object containing match details.
      * either trueKey xor containerProperty is set.
      * index is filled iff it is a present item
      */
@@ -276,7 +279,7 @@ sap.ui.define([
             sItemValue = this._oJSONContainer.PersContainerItems[oItemRef.index].value;
             try {
                 oItemValue = JSON.parse(sItemValue);
-            } catch (e) {
+            } catch {
                 // Workaround for
                 // GW Bug "true" => "X" and false => "" at the backend
                 // can be removed once Correction of Note 2013368 is implemented in landscape
@@ -299,6 +302,8 @@ sap.ui.define([
     /**
      * set oItemValue under sItemKey
      * returns undefined
+     * @param {string} sItemKey the item key
+     * @param {object} oItemValue the item value to be set
      */
     AdapterContainer.prototype.setItemValue = function (sItemKey, oItemValue) {
         const sItemValue = JSON.stringify(oItemValue);
@@ -332,6 +337,7 @@ sap.ui.define([
 
     /**
      * delete (1st) item with key sItemKey
+     * @param {string} sItemKey the item key
      */
     AdapterContainer.prototype.delItem = function (sItemKey) {
         const oItemRef = this._locateItem(sItemKey);

package/src/main/js/sap/ushell_abap/adapters/abap/AppStateAdapter.js

@@ -2,7 +2,7 @@
 
 /**
  * @fileOverview The Unified Shell's AppStateAdapter for the ABAP platform.
- * @version 1.139.0
+ * @version 1.141.0
  */
 sap.ui.define([
     "sap/base/util/ObjectPath",
@@ -38,7 +38,7 @@ sap.ui.define([
      *
      * @private
      */
-    const AppStateAdapter = function (oSystem, sParameters, oConfig) {
+    function AppStateAdapter (oSystem, sParameters, oConfig) {
         this._oConfig = oConfig && oConfig.config;
         const sAppStateServiceURL = `${ObjectPath.get("services.appState.baseUrl", oConfig) || "/sap/opu/odata/UI2/INTEROP"}/`;
         const oODataWrapperSettings = {
@@ -47,11 +47,11 @@ sap.ui.define([
             "sap-client": Container.getLogonSystem().getClient()
         };
         this._oWrapper = ODataWrapper.createODataWrapper(oODataWrapperSettings);
-        function fnDefaultFailure (oMessage) {
-            throw new SrvcError(oMessage, "sap.ushell_abap.adapters.abap.AppStateAdapter");
+        function fnDefaultFailure (sErrorMessage) {
+            throw new SrvcError(sErrorMessage, "sap.ushell_abap.adapters.abap.AppStateAdapter");
         }
         ODataService.call(this, this._oWrapper, fnDefaultFailure);
-    };
+    }
 
     /**
      * Save the given data sValue for the given key at the persistence layer
@@ -83,8 +83,8 @@ sap.ui.define([
         this._oWrapper.create(sRelativeUrl, oPayload, (/* response */) => {
             oDeferred.resolve();
         }, (sErrorMessage) => {
-            oDeferred.reject(sErrorMessage);
-            Log.error(sErrorMessage);
+            Log.error(`AppState save failed: ${sErrorMessage}`);
+            oDeferred.reject(new Error(sErrorMessage));
         });
 
         return oDeferred.promise();
@@ -111,12 +111,12 @@ sap.ui.define([
         this._oWrapper.read(sRelativeUrl, (response) => {
             oDeferredRead.resolve(response.id, response.value);
         }, (sErrorMessage) => {
-            Log.error(sErrorMessage);
-            oDeferredRead.reject(sErrorMessage);
+            Log.error(`AppState load failed: ${sErrorMessage}`);
+            oDeferredRead.reject(new Error(sErrorMessage));
         }, false);
         this._oWrapper.submitBatchQueue(() => { }, (sErrorMessage) => {
-            Log.error(sErrorMessage);
-            oDeferredRead.reject(sErrorMessage);
+            Log.error(`AppState load failed: ${sErrorMessage}`);
+            oDeferredRead.reject(new Error(sErrorMessage));
         });
 
         return oDeferredRead.promise();

package/src/main/js/sap/ushell_abap/adapters/abap/ClientSideTargetResolutionAdapter.js

@@ -9,7 +9,7 @@
  *     <li>provide the resolveHashFragmentFallback function, a fallback method called by ClientSideTargetResolution service.</li>
  *   </ul>
  *
- * @version 1.139.0
+ * @version 1.141.0
  */
 sap.ui.define([
     "sap/base/util/ObjectPath",
@@ -106,7 +106,7 @@ sap.ui.define([
      * @since 1.34.0
      * @private
      */
-    const ClientSideTargetResolutionAdapter = function (oSystem, sParameters, oAdapterConfig) {
+    function ClientSideTargetResolutionAdapter (oSystem, sParameters, oAdapterConfig) {
         const that = this;
         // as the method getProductVersion returns the product name, the following holds true.
         const oContainer = Container;
@@ -214,8 +214,8 @@ sap.ui.define([
                     that._storeFromFullStartupResponse(oFullStartupResult);
                     fnResolve(); // resolve the full promise
                 })
-                .catch((sMsg) => {
-                    fnReject(sMsg); // reject the full promise
+                .catch((oError) => {
+                    fnReject(oError); // reject the full promise
                 });
         };
 
@@ -271,10 +271,10 @@ sap.ui.define([
                     "sap.ushell_abap.adapters.abap.ClientSideTargetResolutionAdapter"
                 );
             }
-        }).catch((sReason) => {
+        }).catch((oError) => {
             Log.error(
                 "Initial segment promise was rejected.",
-                sReason,
+                oError,
                 "sap.ushell_abap.adapters.abap.ClientSideTargetResolutionAdapter"
             );
         });
@@ -287,7 +287,7 @@ sap.ui.define([
             );
             that._storeFromFullStartupResponse(oFullStartupResult);
         });
-    };
+    }
 
     /**
      * Checks the Resolve Link/Start Up result for Application Dependencies and if present modifies the results
@@ -450,26 +450,26 @@ sap.ui.define([
             });
 
             // must wait on the full target mappings promise to complete before returning
-            this._oNavTargetPromise.then(() => {
-                Log.debug(
-                    "Got inbounds from full start_up response",
-                    "",
-                    "sap.ushell_abap.adapters.abap.ClientSideTargetResolutionAdapter"
-                );
+            this._oNavTargetPromise
+                .then(() => {
+                    Log.debug(
+                        "Got inbounds from full start_up response",
+                        "",
+                        "sap.ushell_abap.adapters.abap.ClientSideTargetResolutionAdapter"
+                    );
 
-                // convert (once) when needed
-                this._aInbounds = this._formatDirectStart(this._aTargetMappings);
+                    // convert (once) when needed
+                    this._aInbounds = this._formatDirectStart(this._aTargetMappings);
 
-                // we don't want to check for the initial segment in the future anymore, since we've got all the inbounds now.
-                this.getInbounds = function () {
-                    // replace implementation of this method (no conversion in future calls anymore!)
-                    return Promise.resolve(this._aInbounds);
-                };
+                    // we don't want to check for the initial segment in the future anymore, since we've got all the inbounds now.
+                    this.getInbounds = function () {
+                        // replace implementation of this method (no conversion in future calls anymore!)
+                        return Promise.resolve(this._aInbounds);
+                    };
 
-                fnResolve(this._aInbounds);
-            }, (sMsg) => {
-                fnReject(sMsg);
-            });
+                    fnResolve(this._aInbounds);
+                })
+                .catch(fnReject);
         });
     };
 
@@ -528,12 +528,12 @@ sap.ui.define([
                 (sNavTargetDataResult) => {
                     const oNavTargetDataResult = JSON.parse(sNavTargetDataResult);
                     if (!oNavTargetDataResult) {
-                        fnReject("Malformed Full TM Result");
+                        fnReject(new Error("Malformed Full TM Result"));
                     }
                     fnResolve(oNavTargetDataResult);
                 },
-                (sMessage) => {
-                    fnReject(sMessage);
+                (sErrorMessage) => {
+                    fnReject(new Error(sErrorMessage));
                 }
             );
         });
@@ -871,9 +871,7 @@ sap.ui.define([
         }
 
         oODataResponse.results.forEach((oEntry) => {
-            let oParam;
             const sEntryName = oEntry.name;
-            let sEntryValue;
             let sUserDefaultValue;
 
             if (Object.prototype.hasOwnProperty.call(oRes.parameters, sEntryName)) {
@@ -886,8 +884,8 @@ sap.ui.define([
                 required: that._getObjectDefaulting(oEntry, "required", false)
             };
 
-            oParam = oRes.parameters[sEntryName];
-            sEntryValue = that._getObjectDefaulting(oEntry, "value", "");
+            const oParam = oRes.parameters[sEntryName];
+            const sEntryValue = that._getObjectDefaulting(oEntry, "value", "");
 
             if (oEntry.regexp === true) {
                 oParam.filter = {
@@ -1131,7 +1129,6 @@ sap.ui.define([
      */
     ClientSideTargetResolutionAdapter.prototype._resolveHashFragmentBE = function (sFragmentNoHash) {
         const that = this;
-        let bBatchQueueOpened;
         const sFormFactor = Utils.getFormFactor();
 
         /**
@@ -1147,7 +1144,7 @@ sap.ui.define([
             return encodeURIComponent(sUnencoded).replace(/'/g, "''");
         }
 
-        bBatchQueueOpened = this._openBatchQueueUnlessOpen(this._getODataWrapper());
+        const bBatchQueueOpened = this._openBatchQueueUnlessOpen(this._getODataWrapper());
 
         /**
          * Adds the given post parameters to a url
@@ -1207,18 +1204,21 @@ sap.ui.define([
 
                     that._compactTooLongWdaUrl(oAdjustedResult).then((oCompactedResult) => {
                         fnResolve(oCompactedResult);
-                    }).catch((sMsg) => {
-                        fnReject(`Could not resolve link '${sFragmentNoHash}' due to compactation failure${sMsg}`);
+                    }).catch((oError) => {
+                        Log.error(`Could not resolve link '${sFragmentNoHash}' due to compactation failure`, oError);
+                        fnReject(oError);
                     });
                 } else {
-                    fnReject(`Could not resolve link '${sFragmentNoHash}'`);
+                    fnReject(new Error(`Could not resolve link '${sFragmentNoHash}'`));
                 }
-            }, (sMessage) => {
-                fnReject(sMessage);
+            }, (sErrorMessage) => {
+                fnReject(new Error(sErrorMessage));
             });
 
             if (bBatchQueueOpened) {
-                that._getODataWrapper().submitBatchQueue(() => { }, fnReject);
+                that._getODataWrapper().submitBatchQueue(() => { }, (sErrorMessage) => {
+                    fnReject(new Error(sErrorMessage));
+                });
             }
         });
     };
@@ -1324,6 +1324,45 @@ sap.ui.define([
 
         // check if we have it already
         return new Promise((fnResolve, fnReject) => {
+            /*
+             * Trigger call to front-end server. This case may occur, for example, during application direct start with
+             * a system alias that is not sent by the backend. Backend only sends data about the system aliases that are
+             * mentioned in the systemAlias fields of the target mappings (or for virtual target mappings).
+             */
+            function fnRetryBufferOrUseInterop () {
+                oSystemAliasData = that._readSystemAliasFromBuffer(sSystemAlias); // note: already in ClientSideTargetResolutionAdapter format
+                if (oSystemAliasData) {
+                    Log.debug(
+                        `System alias '${sSystemAlias}' is now in buffer`,
+                        "Skipping INTEROP service call",
+                        "sap.ushell_abap.adapters.abap.ClientSideTargetResolutionAdapter"
+                    );
+                    fnResolve(oSystemAliasData);
+                } else {
+                    Log.debug(
+                        `System alias '${sSystemAlias}' still not in buffer`,
+                        "Resolving via INTEROP service",
+                        "sap.ushell_abap.adapters.abap.ClientSideTargetResolutionAdapter"
+                    );
+                    that._readSystemAliasViaInterop(sSystemAlias)
+                        .catch((oError) => {
+                            fnReject(oError);
+                        })
+                        .then((oOdataSystemAliasData) => {
+                            oSystemAliasData = that._fixSystemAlias(oOdataSystemAliasData);
+
+                            if (oSystemAliasData && oSystemAliasData.id) {
+                                that._writeSystemAliasToBuffer(oSystemAliasData);
+                                fnResolve(oSystemAliasData);
+                            } else {
+                                sMessage = "Data returned for system alias is not valid. Has no 'id'. System alias data will not be cached.";
+                                Log.warning(`ClientSideTargetResolutionAdapter: ${sMessage}`);
+                                fnResolve(oSystemAliasData);
+                            }
+                        });
+                }
+            }
+
             oSystemAliasData = this._readSystemAliasFromBuffer(sSystemAlias); // note: already in ClientSideTargetResolutionAdapter format
             if (oSystemAliasData) {
                 Log.debug(
@@ -1343,46 +1382,7 @@ sap.ui.define([
 
                 // must wait at least on the small start_up response before using INTEROP
 
-                /*
-                 * Trigger call to front-end server. This case may occur, for example, during application direct start with
-                 * a system alias that is not sent by the backend. Backend only sends data about the system aliases that are
-                 * mentioned in the systemAlias fields of the target mappings (or for virtual target mappings).
-                 */
-                const fnRetryBufferOrUseInterop = function () {
-                    oSystemAliasData = that._readSystemAliasFromBuffer(sSystemAlias); // note: already in ClientSideTargetResolutionAdapter format
-                    if (oSystemAliasData) {
-                        Log.debug(
-                            `System alias '${sSystemAlias}' is now in buffer`,
-                            "Skipping INTEROP service call",
-                            "sap.ushell_abap.adapters.abap.ClientSideTargetResolutionAdapter"
-                        );
-                        fnResolve(oSystemAliasData);
-                    } else {
-                        Log.debug(
-                            `System alias '${sSystemAlias}' still not in buffer`,
-                            "Resolving via INTEROP service",
-                            "sap.ushell_abap.adapters.abap.ClientSideTargetResolutionAdapter"
-                        );
-                        that._readSystemAliasViaInterop(sSystemAlias)
-                            .catch((sError) => {
-                                fnReject(sError);
-                            })
-                            .then((oOdataSystemAliasData) => {
-                                oSystemAliasData = that._fixSystemAlias(oOdataSystemAliasData);
-
-                                if (oSystemAliasData && oSystemAliasData.id) {
-                                    that._writeSystemAliasToBuffer(oSystemAliasData);
-                                    fnResolve(oSystemAliasData);
-                                } else {
-                                    sMessage = "Data returned for system alias is not valid. Has no 'id'. System alias data will not be cached.";
-                                    Log.warning(`ClientSideTargetResolutionAdapter: ${sMessage}`);
-                                    fnResolve(oSystemAliasData);
-                                }
-                            });
-                    }
-                };
-
-                this._oInitialSegmentPromise.then(fnRetryBufferOrUseInterop, fnRetryBufferOrUseInterop);
+                this._oInitialSegmentPromise.finally(fnRetryBufferOrUseInterop);
             }
         });
     };
@@ -1520,8 +1520,8 @@ sap.ui.define([
                 (oSystemAliasData) => {
                     fnResolve(oSystemAliasData);
                 },
-                (sMessage) => {
-                    fnReject(sMessage);
+                (sErrorMessage) => {
+                    fnReject(new Error(sErrorMessage));
                 }
             );
         });

package/src/main/js/sap/ushell_abap/adapters/abap/CommonDataModelAdapter.js

@@ -2,7 +2,7 @@
 /**
  * @fileOverview The Unified Shell's CommonDataModelAdapter for the ABAP platform, it is used to avoid 404 when no adapter is found
  *
- * @version 1.139.0
+ * @version 1.141.0
  */
 sap.ui.define([
 ], () => {

package/src/main/js/sap/ushell_abap/adapters/abap/ConfigurationDefaultsAdapter.js

@@ -2,7 +2,7 @@
 
 /**
  * @fileOverview The ConfigurationDefaults adapter for the ABAP platform.
- * @version 1.139.0
+ * @version 1.141.0
  */
 sap.ui.define([
     "sap/ushell_abap/bootstrap/evo/abap.constants",
@@ -11,7 +11,6 @@ sap.ui.define([
     "use strict";
 
     /**
-     * @returns {sap.ushell_abap.adapters.abap.ConfigurationDefaultsAdapter}
      * @private
      */
     return function () {

package/src/main/js/sap/ushell_abap/adapters/abap/ContainerAdapter.js

@@ -2,7 +2,7 @@
 /**
  * @fileOverview The Unified Shell's container adapter for the ABAP platform.
  *
- * @version 1.139.0
+ * @version 1.141.0
  */
 sap.ui.define([
     "sap/ui/thirdparty/jquery",
@@ -25,7 +25,7 @@ sap.ui.define([
     abapUtils,
     ODataWrapper,
     URI,
-    OData,
+    datajs,
     deepClone,
     ObjectPath,
     Log,
@@ -42,6 +42,8 @@ sap.ui.define([
      * Constructs a new instance of the container adapter for the ABAP platform.
      *
      * @param {sap.ushell.System} oSystem the logon system (alias, platform, base URL)
+     * @param {string} sParameter  the parameter to be passed to the container adapter
+     * @param {object} oProperties the properties to be passed to the container adapter, e.g. the configuration
      *
      * @hideconstructor
      * @see sap.ushell.services.initializeContainer
@@ -49,7 +51,7 @@ sap.ui.define([
      * @since 1.11.0
      * @private
      */
-    const ContainerAdapter = function (oSystem, sParameter, oProperties) {
+    function ContainerAdapter (oSystem, sParameter, oProperties) {
         let oUser;
         const S_LOGOFF_URL = "/sap/public/bc/icf/logoff";
 
@@ -78,7 +80,7 @@ sap.ui.define([
                 oFrame.setAttribute("src", sUrl);
 
                 function onload () {
-                    parent.postMessage(sSafeUrl, "*");
+                    window.parent.postMessage(sSafeUrl, "*");
                 }
 
                 oFrame.addEventListener("load", onload);
@@ -129,9 +131,7 @@ sap.ui.define([
         // startup result (modified!)
         this._setThemeAccessibilityFlags = function (oStartupResult) {
             if (oStartupResult.userProfile && oStartupResult.userProfile.length) {
-                let oUserProfileDataTheme;
-                let oUserProfileDataAccessibility;
-                oUserProfileDataTheme = oStartupResult.userProfile.filter((profileProperty) => {
+                const oUserProfileDataTheme = oStartupResult.userProfile.filter((profileProperty) => {
                     return profileProperty.id && profileProperty.id === "THEME";
                 })[0];
                 // Theme
@@ -145,7 +145,7 @@ sap.ui.define([
                 }
                 // AccessibilityPermitted: Note: we only register whether editing the Accessibility is allowed on the
                 // ABAP Platform, we don't test whether accessibility is available!
-                oUserProfileDataAccessibility = oStartupResult.userProfile.filter((profileProperty) => {
+                const oUserProfileDataAccessibility = oStartupResult.userProfile.filter((profileProperty) => {
                     return profileProperty.id && profileProperty.id === "ACCESSIBILITY";
                 })[0];
                 if (oUserProfileDataAccessibility && oUserProfileDataAccessibility.id) {
@@ -391,11 +391,9 @@ sap.ui.define([
                                 });
                             }
                             oDeferred.resolve();
-                        }, (sError) => { // error handler
-                            Log.error(`Reading REMOTE catalogs failed: ${
-                                sError}`, null,
-                            "sap.ushell_abap.adapters.abap.ContainerAdapter");
-                            oDeferred.reject();
+                        }, (sErrorMessage) => { // error handler
+                            Log.error("Reading REMOTE catalogs failed:", sErrorMessage, "sap.ushell_abap.adapters.abap.ContainerAdapter");
+                            oDeferred.reject(new Error(sErrorMessage));
                         });
                 })
                 .catch(oDeferred.reject);
@@ -513,13 +511,13 @@ sap.ui.define([
             }
 
             if (isUserImageEnabled() && oUser && oUser.isJamActive && oUser.isJamActive()) {
-                OData.read("/sap/bc/ui2/smi/rest_tunnel/Jam/api/v1/OData/Self?$format=json", (oResponseData) => {
+                datajs.read("/sap/bc/ui2/smi/rest_tunnel/Jam/api/v1/OData/Self?$format=json", (oResponseData) => {
                     const sJamUserId = oResponseData.results.Id;
                     const sJamUserImageUrl = `/sap/bc/ui2/smi/rest_tunnel/Jam/api/v1/OData/Members('${sJamUserId}')/ProfilePhoto/$value`;
 
                     oUser.setImage(sJamUserImageUrl);
-                }, () => {
-                    Log.error("Could not recieve JAM user data");
+                }, (oDataJsError) => {
+                    Log.error("Could not receive JAM user data");
                 });
             }
         };
@@ -540,7 +538,7 @@ sap.ui.define([
             }
             return sUrl;
         };
-    };
+    }
 
     return ContainerAdapter;
 });