@sapui5/sap.ushell_abap 1.139.0 → 1.140.0

package/src/main/js/sap/ushell_abap/components/TCodeNavigation.js

@@ -0,0 +1,150 @@
+// Copyright (c) 2009-2025 SAP SE, All Rights Reserved
+
+sap.ui.define([
+    "sap/ushell/Container",
+    "sap/base/Log",
+    "sap/ushell/api/performance/Extension",
+    // navigation sources
+    "sap/ushell/api/performance/NavigationSource"
+], (Container, Log, Extension, NavigationSource) => {
+    "use strict";
+
+    /**
+     * @typedef {object} sap.ushell_abap.components.TCodeNavigation.NavigationResult
+     *
+     * @property {boolean} successful Indicates whether the navigation was successful. //TODO Remove as not needed due to resolved or rejected promise
+     * @property {string} messagecode The message code associated with the navigation result.
+     * @private
+     */
+
+    /**
+     * Message codes for navigation results.
+     * @enum {string} sap.ushell_abap.components.TCodeNavigation.MessageCode
+     * @property {string} NAV_SUCCESS Indicates that the navigation was successful.
+     * @property {string} NO_INBOUND_FOUND Indicates that no matching inbound was found for the transaction code or a mandatory parameter is missing.
+     * @property {string} UNKNOWN_ERROR Indicates an unknown error occurred during navigation.
+     * @private
+     * @ui5-restricted sap.esh.search.ui
+     */
+    const MessageCode = {
+        NAV_SUCCESS: "NAV_SUCCESS",
+        NO_INBOUND_FOUND: "NO_INBOUND_FOUND",
+        UNKNOWN_ERROR: "UNKNOWN_ERROR"
+    };
+
+    // Performance extension for tracking navigation sources
+    const oExtension = new Extension("TCodeNavigation");
+
+    /**
+     * Finds the first inbound that matches the given transaction code and doesn't have mandatory parameters.
+     * @param {object} oNavigationData The navigation data containing inbounds.
+     * @param {string} sTCode The transaction code to match.
+     * @returns {object|null} The matched inbound or null if no match is found.
+     * @private
+     */
+    function _getValidInbound (oNavigationData, sTCode) {
+        if (!oNavigationData?.inbounds) {
+            Log.error("No navigation data found");
+            return null;
+        }
+
+        // Find all inbounds with matching transaction code
+        const aMatchedInbounds = oNavigationData.inbounds.filter((oInbound) =>
+            oInbound.resolutionResult?.appInfo?.["abap.transaction"] === sTCode
+        );
+        if (aMatchedInbounds.length === 0) {
+            return null;
+        }
+
+        // Return the first matched inbound where all mandatory parameters have default values
+        const oValidInbound = aMatchedInbounds.find((oInbound) => {
+            const oParams = oInbound.signature?.parameters;
+            if (!oParams) {
+                return true; // No parameters, consider valid
+            }
+            // All required parameters must have a default value
+            return Object.values(oParams).every((oValue) =>
+                !oValue.required || (oValue.defaultValue && oValue.defaultValue.value)
+            );
+        });
+
+        if (!oValidInbound) {
+            throw Object.assign(
+                new Error(`Mandatory parameter is missing for transaction ${sTCode}`),
+                { code: MessageCode.NO_INBOUND_FOUND }
+            );
+        }
+
+        return oValidInbound;
+    }
+
+    /**
+     * TCodeNavigation module for navigating by Transaction (TA) Code.
+     * @alias sap.ushell_abap.components.TCodeNavigation
+     * @namespace
+     * @class
+     * @description This module provides functionality to navigate to an application intent based on a transaction code.
+     * It checks if the provided transaction code matches any inbound's appInfo parameter `abap.transaction`.
+     * If a match is found, it extracts the semantic object, action, and parameters from the inbound
+     * and uses the Navigation service to perform the navigation.
+     * @private
+     * @ui5-restricted sap.esh.search.ui  // TODO add technical name of esearch component / team name
+     * @since 1.140.0
+     */
+    class TCodeNavigation {
+    }
+
+    /**
+     * Navigates to an app intent if the given sTCode matches an inbound's appInfo parameter abap.transaction.
+     * If a matching inbound is found, it extracts the semantic object, action, and parameters
+     * and uses the Navigation service to perform the navigation.
+     * @param {string} sTCode The transaction code to search for, can also be an App ID.
+     * @param {boolean} bExplace Indicates whether to open the navigation in a new tab.
+     * @returns {Promise<NavigationResult>} Resolves with a NavigationResult object containing a successful flag and a message code. Otherwise, it rejects with an error.
+     * @throws {Error} E.g., if no matching inbound is found or if a mandatory parameter is missing.
+     * @private
+     */
+    TCodeNavigation.navigateByTCode = async (sTCode, bExplace) => {
+        try {
+            const NavDataProvider = await Container.getServiceAsync("NavigationDataProvider");
+            const oNavigationData = await NavDataProvider.getNavigationData();
+
+            const oValidInbound = _getValidInbound(oNavigationData, sTCode);
+
+            if (oValidInbound) {
+                // Extract semantic object, action, and parameters from the matched inbound for navigation
+                const { semanticObject: sSemanticObject, action: sAction } = oValidInbound;
+                const oParams = {};
+                if (bExplace) {
+                    oParams["sap-ushell-navmode"] = "explace";
+                }
+
+                // Log the navigation source for performance tracking
+                oExtension.addNavigationSource(NavigationSource.SearchDirectLaunch);
+
+                const oNavService = await Container.getServiceAsync("Navigation");
+                await oNavService.navigate({
+                    target: {
+                        semanticObject: sSemanticObject,
+                        action: sAction
+                    },
+                    params: oParams
+                });
+
+                return { successful: true, messagecode: MessageCode.NAV_SUCCESS };
+            }
+            throw Object.assign(
+                new Error(`No matching inbound found for transaction code: ${sTCode}`),
+                { code: MessageCode.NO_INBOUND_FOUND }
+            );
+        } catch (oError) {
+            Log.error("Error during TCode navigation", oError);
+            const oErrorResult = new Error("Error during TCode navigation.");
+            oErrorResult.successful = false;
+            oErrorResult.messagecode = oError.code || MessageCode.UNKNOWN_ERROR;
+            throw oErrorResult;
+        }
+    };
+
+    return TCodeNavigation;
+});

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

@@ -23,7 +23,7 @@ sap.ui.define([
     const ushellAbapLib = Library.init({
         name: "sap.ushell_abap",
         apiVersion: 2,
-        version: "1.139.0",
+        version: "1.140.0",
         dependencies: ["sap.ui.core", "sap.m"],
         noLibraryCSS: true,
         extensions: {

package/src/main/js/sap/ushell_abap/pbServices/ui2/AllCatalogs.js

@@ -38,7 +38,7 @@ sap.ui.define([
      * @class
      * @since 1.7.0
      */
-    const AllCatalogs = function (oFactory, sPageId) {
+    function AllCatalogs (oFactory, sPageId) {
         let aCatalogs = [];
         let bIsStub = true;
         const that = this;
@@ -180,7 +180,7 @@ sap.ui.define([
             throw new SrvcError("Missing page ID", "AllCatalogs");
         }
         Log.debug(`Created: ${this}`, null, "AllCatalogs");
-    };
+    }
 
     return AllCatalogs;
 });

package/src/main/js/sap/ushell_abap/pbServices/ui2/Bag.js

@@ -39,7 +39,7 @@ sap.ui.define([
      * @class
      * @since 1.3.0
      */
-    const Bag = function (oFactory, oAlterEgo, oParentBag, fnChangeListener) {
+    function Bag (oFactory, oAlterEgo, oParentBag, fnChangeListener) {
         let oPropertiesByName;
         let bResetting = false;
         let bSaving = false;
@@ -121,8 +121,6 @@ sap.ui.define([
          * @private
          */
         function getNames (aNames, bTranslatable) {
-            let oNameSet;
-
             if (aNames.length === 0) { // no parent names --> no duplicates expected, run faster code
                 oPropertiesByName.keys().forEach((sKey) => {
                     if (bTranslatable === (oPropertiesByName.get(sKey).translatable === "X")) {
@@ -133,7 +131,7 @@ sap.ui.define([
             }
 
             // parent names present --> expect duplicates and avoid them with slower code
-            oNameSet = new Utils.Map();
+            const oNameSet = new Utils.Map();
             aNames.forEach((sKey) => {
                 oNameSet.put(sKey);
             });
@@ -462,9 +460,7 @@ sap.ui.define([
          * @since 1.17.1
          */
         this.resetProperty = function (sPropertyName) {
-            let oProperty;
-
-            oProperty = oPropertiesByName.get(sPropertyName);
+            const oProperty = oPropertiesByName.get(sPropertyName);
             if (oProperty) {
                 oProperty.$currentValue = undefined; // undefined value is marker to delete property on save
                 if (!Object.prototype.hasOwnProperty.call(oProperty, "value")) { // no persisted value
@@ -708,7 +704,7 @@ sap.ui.define([
 
         fillPropertyMap(oAlterEgo);
         Log.debug(`Created: ${this}`, null, "sap.ushell_abap.pbServices.ui2.Bag");
-    };
+    }
 
     return Bag;
 });

package/src/main/js/sap/ushell_abap/pbServices/ui2/Catalog.js

@@ -49,7 +49,7 @@ sap.ui.define([
      * @since 1.2.0
      * @private
      */
-    const Catalog = function (oFactory, vCatalogData) {
+    function Catalog (oFactory, vCatalogData) {
         let oAlterEgo; // catalog's representation with all relations removed
         let oCatalogPage;
         let aChips = [];
@@ -123,7 +123,7 @@ sap.ui.define([
 
         /**
          * Gets this catalog's remote base URL.
-         * @returns {string}
+         * @returns {string} url to the remote catalog service
          * @private
          */
         this.getRemoteBaseUrl = function () {
@@ -135,7 +135,7 @@ sap.ui.define([
         /**
          * Gets the remote catalog service.
          *
-         * @returns {sap.ushell_abap.pbServices.ui2.RemoteCatalogService}
+         * @returns {sap.ushell_abap.pbServices.ui2.RemoteCatalogService} remote catalog service
          * @private
          */
         this.getRemoteCatalogService = function () {
@@ -186,7 +186,7 @@ sap.ui.define([
          * If the catalog is a remote catalog, an attempt to load its chips is made.
          * In this case, if the loading fails, the catalog is fully initialized ( with an empty chips collection ) and
          * returned as 2nd argument of the reject handler (note that it is no longer a stub!)
-         * @param {boolean} bRefresh
+         * @param {boolean} bRefresh boolean indicating whether the catalog should be refreshed
          * @param {function} fnSuccess
          *   no-args success handler
          * @param {function(string, object=)} [fnFailure]
@@ -387,13 +387,18 @@ sap.ui.define([
         this.clone = function (sNewDomainId, sNewTitle, fnSuccess, fnFailure) {
             let oCatalogData;
             /**
-                 * keeps calls to failure handler consistent with failure handler calls
-                 * in sap.ushell_abap.pbServices.ui2.Factory#createNewCatalog
-                 */
-            const fnFailureWrapper = function (sErrorMessage, oMaybeErrorInformation) {
+             * keeps calls to failure handler consistent with failure handler calls
+             * in sap.ushell_abap.pbServices.ui2.Factory#createNewCatalog
+             *
+             * @param {string} sErrorMessage the error message
+             * @param {Error|object|undefined} oMaybeErrorInformation an optional object containing the complete error information as
+             *
+             * @returns {function} failure handler
+             */
+            function fnFailureWrapper (sErrorMessage, oMaybeErrorInformation) {
                 return fnFailure(sErrorMessage, undefined,
                     oMaybeErrorInformation);
-            };
+            }
 
             if (oAlterEgo && oAlterEgo.type === "REMOTE") {
                 oCatalogData = this.getCatalogData();
@@ -435,13 +440,12 @@ sap.ui.define([
          */
         this.getCatalogData = function () {
             /* jslint nomen:true */
-            let oCatalogData;
 
             if (!oAlterEgo) {
                 return undefined; // only ID is known!
             }
 
-            oCatalogData = JSON.parse(JSON.stringify(oAlterEgo));
+            const oCatalogData = JSON.parse(JSON.stringify(oAlterEgo));
             delete oCatalogData.__metadata;
             return oCatalogData;
         };
@@ -890,14 +894,12 @@ sap.ui.define([
          * @see #getCatalogData
          */
         this.update = function (oCatalogData, fnSuccess, fnFailure) {
-            let oNewAlterEgo;
-
             checkAlterEgo();
             if (Object.hasOwnProperty.call(oCatalogData, "__metadata")) {
                 // ODataWrapper#update() relies on __metadata!
                 throw new SrvcError("Unsupported __metadata update", "Catalog");
             }
-            oNewAlterEgo = JSON.parse(JSON.stringify(oAlterEgo));
+            const oNewAlterEgo = JSON.parse(JSON.stringify(oAlterEgo));
             Object.keys(oCatalogData).forEach((sName) => {
                 oNewAlterEgo[sName] = oCatalogData[sName];
             });
@@ -932,7 +934,7 @@ sap.ui.define([
             throw new SrvcError("Missing ID", "Catalog");
         }
         Log.debug(`Created: ${this}`, null, "Catalog");
-    };
+    }
 
     return Catalog;
 });

package/src/main/js/sap/ushell_abap/pbServices/ui2/Chip.js

@@ -65,7 +65,7 @@ sap.ui.define([
      * @class
      * @since 1.2.0
      */
-    const Chip = function (oAlterEgo, oFactory) {
+    function Chip (oAlterEgo, oFactory) {
         const that = this;
         let oBags;
         /**
@@ -307,10 +307,9 @@ sap.ui.define([
             let oContract;
             let fnInitializer;
             let sName;
-            let mRequestedContracts;
 
             checkStub();
-            mRequestedContracts = oDefinition.contracts;
+            const mRequestedContracts = oDefinition.contracts;
             if (mRequestedContracts) {
                 for (sName in mRequestedContracts) {
                     if (Object.prototype.hasOwnProperty.call(mRequestedContracts, sName)) {
@@ -530,17 +529,13 @@ sap.ui.define([
          * @see #isStub()
          */
         this.getImplementationAsSapui5 = function (oApi) {
-            let oData;
-            let oImplementation;
-            let sBaseChipId;
-
             Log.error("Deprecated API call of 'Chip.getImplementationAsSapui5'. Please use 'getImplementationAsSapui5Async' instead",
                 null,
                 "sap.ushell_abap.pbServices.ui2.Chip"
             );
 
             checkStub();
-            oData = {
+            const oData = {
                 /**
                  * @namespace
                  * The namespace for the instance specific CHIP API, which allows you to
@@ -548,10 +543,10 @@ sap.ui.define([
                  */
                 chip: oApi
             };
-            oImplementation = oDefinition.implementation.sapui5;
+            const oImplementation = oDefinition.implementation.sapui5;
 
             // In case the chip is used in FLP wave 2 or later context we load it from the standard path
-            sBaseChipId = this.getBaseChipId();
+            const sBaseChipId = this.getBaseChipId();
             if ((sBaseChipId !== "X-SAP-UI2-CHIP:/UI2/STATIC_APPLAUNCHER") && (sBaseChipId !== "X-SAP-UI2-CHIP:/UI2/DYNAMIC_APPLAUNCHER")) {
                 if (oImplementation.$VirtualNamespace) {
                     oImplementation.$absolutePath = this.toAbsoluteUrl(oImplementation.basePath);
@@ -748,8 +743,8 @@ sap.ui.define([
                 }:${oChipInstance.getId()}`;
 
             /**
-             * @param {string} sActualId
-             * @returns {boolean}
+             * @param {string} sActualId the actual CHIP ID to compare with the expected one
+             * @returns {boolean} the result of the comparison
              * @private
              */
             function matchesExpectation (sActualId) {
@@ -1002,9 +997,9 @@ sap.ui.define([
          * Updates the given parameter map from the given JSON string. All parameters that actually were
          * defined in the CHIP definition XML are accepted. All others will raise a warning to the log.
          *
-         * @param {object<string,string>} mParameters
+         * @param {Object<string,string>} mParameters
          *   the parameter map to fill
-         * @param {object<string,string>|string} vConfigurationUpdates
+         * @param {Object<string,string>|string} vConfigurationUpdates
          *   the configuration updates as parameter map or as JSON string
          *   <p>If one parameter value is <code>undefined</code> (which can only happen by supplying a
          *   map) then this property is removed from <code>mParameters</code>.
@@ -1123,7 +1118,7 @@ sap.ui.define([
         }
         initBags(oAlterEgo.ChipBags && oAlterEgo.ChipBags.results);
         Log.debug(`Created: ${this}`, null, "Chip");
-    };
+    }
 
     // "public" methods (static) ------------------------------------------------
 

package/src/main/js/sap/ushell_abap/pbServices/ui2/ChipDefinition.js

@@ -43,7 +43,6 @@ sap.ui.define([
      */
     function getAttribute (oDomElement, sName) {
         let oAttribute;
-        let aAttributes;
         let i;
         let n;
         if (typeof oDomElement.getAttributeNS === "function") {
@@ -51,7 +50,7 @@ sap.ui.define([
         }
         // ugly workaround for IE's XMLHttpRequest which doesn't know getAttributeNS, but knows the
         // attribute's namespace
-        aAttributes = oDomElement.attributes;
+        const aAttributes = oDomElement.attributes;
         for (i = 0, n = aAttributes.length; i < n; i += 1) {
             oAttribute = aAttributes[i];
             if (!oAttribute.namespaceURI && getLocalName(oAttribute) === sName) {
@@ -83,7 +82,7 @@ sap.ui.define([
      *  this used for calling the handlers from <code>mChildHandlers</code>
      * @param {object} oDomNode
      *   the DOM node
-     * @param {object<string,function(object)>} mChildHandlers
+     * @param {Object<string,function(object)>} mChildHandlers
      *   maps the child name to a handler function. The handler function's parameter is the DOM node.
      */
     function visit (oThat, oDomNode, mChildHandlers) {
@@ -117,7 +116,7 @@ sap.ui.define([
      * @see sap.ushell_abap.pbServices.ui2.Chip
      * @since 1.2.0
      */
-    const ChipDefinition = function (oXml) {
+    function ChipDefinition (oXml) {
         const that = this;
 
         if (oXml instanceof ChipDefinition) {
@@ -211,7 +210,7 @@ sap.ui.define([
 
         // paranoid mode to avoid memory leaks
         oXml = null;
-    };
+    }
 
     return ChipDefinition;
 });

package/src/main/js/sap/ushell_abap/pbServices/ui2/ChipInstance.js

@@ -46,7 +46,7 @@ sap.ui.define([
      * @see sap.ushell_abap.pbServices.ui2.PageBuildingService
      * @since 1.2.0
      */
-    const ChipInstance = function (oFactory, oAlterEgo, oChip, oPage) {
+    function ChipInstance (oFactory, oAlterEgo, oChip, oPage) {
         let oApi;
         let oBags;
         let oConfiguration;
@@ -697,10 +697,9 @@ sap.ui.define([
          */
         this.load = function (fnSuccess, fnFailure) {
             // TODO disallow multiple calls?
-            let fnLoad;
 
             fnFailure = fnFailure || oFactory.getPageBuildingService().getDefaultErrorHandler();
-            fnLoad = initialize.bind(null, fnSuccess, fnFailure);
+            const fnLoad = initialize.bind(null, fnSuccess, fnFailure);
 
             if (oChip.isStub()) {
                 oChip.load(fnLoad, fnFailure);
@@ -929,7 +928,7 @@ sap.ui.define([
          * <code>a</code> back to "foo", but our user will never see this again, because the system
          * persisted both <code>a</code> and <code>b</code> in PERS.
          *
-         * @param {object<string,string>} mConfigurationUpdates
+         * @param {Object<string,string>} mConfigurationUpdates
          *   The configuration updates. The values must be strings. You can however set a value to
          *   <code>undefined</code>. This removes it from the list of updated property and effectively
          *   resets it to the default value.
@@ -995,7 +994,7 @@ sap.ui.define([
 
         delete oAlterEgo.Chip;
         Log.debug(`Created: ${this}`, null, "ChipInstance");
-    };
+    }
 
     /**
      * Clones and loads the new CHIP instance.

package/src/main/js/sap/ushell_abap/pbServices/ui2/Error.js

@@ -25,7 +25,7 @@ sap.ui.define([
      * @class
      * @since 1.2.0
      */
-    const SrvcError = function (sMessage, sComponent, bLogError) {
+    function SrvcError (sMessage, sComponent, bLogError) {
         // see also redundant declaration in utils.js which has to be in sync
         const oError = new Error(sMessage); // reuse Error constructor to benefit from it (e.g. stack)
 
@@ -38,7 +38,7 @@ sap.ui.define([
         }
 
         return oError;
-    };
+    }
 
     return SrvcError;
 });

package/src/main/js/sap/ushell_abap/pbServices/ui2/Factory.js

@@ -56,14 +56,18 @@ sap.ui.define([
      * @class
      * @since 1.2.0
      */
-    const Factory = function (oPbs) {
+    function Factory (oPbs) {
         const mChips = {}; // cache for CHIPs
         const mCatalogs = {}; // cache for catalogs
         const mChipDefinitions = {}; // cache for CHIP definitions
-        let oRemoteCatalogServicesByBaseUrl; // mapping from base URL to remote catalog service
         const that = this;
 
-        // BEWARE: constructor code below!
+        // constructor code -------------------------------------------------------
+        const oRemoteCatalogServicesByBaseUrl = new Utils.Map();
+        if (!oPbs) {
+            throw new SrvcError("Missing page building service", "Factory");
+        }
+        Log.debug(`Created: ${this}`, null, "Factory");
 
         // "private" methods ---------------------------------------------------------
 
@@ -373,9 +377,6 @@ sap.ui.define([
          * @since 1.2.0
          */
         this.createChipInstance = function (oRawChipInstance, fnSuccess, fnFailure, oPage) {
-            let oChip;
-            let oChipInstance;
-
             // ensure at least a "null object" CHIP that knows its IDs
             oRawChipInstance.Chip = oRawChipInstance.Chip || { $proxy: true };
             oRawChipInstance.Chip.id = oRawChipInstance.Chip.id || oRawChipInstance.chipId;
@@ -387,8 +388,8 @@ sap.ui.define([
                 this.createCatalog(oRawChipInstance.RemoteCatalog);
             }
 
-            oChip = this.createChip(oRawChipInstance.Chip);
-            oChipInstance = new ChipInstance(this, oRawChipInstance, oChip, oPage);
+            const oChip = this.createChip(oRawChipInstance.Chip);
+            const oChipInstance = new ChipInstance(this, oRawChipInstance, oChip, oPage);
             if (fnSuccess) {
                 oChipInstance.load(fnSuccess.bind(null, oChipInstance), fnFailure);
             }
@@ -499,8 +500,7 @@ sap.ui.define([
          * @since 1.2.0
          */
         this.createPage = function (sPageId, fnSuccess, fnFailure, bPartially) {
-            let oPage;
-            oPage = new Page(this, sPageId);
+            const oPage = new Page(this, sPageId);
             if (fnSuccess) {
                 oPage.load(fnSuccess.bind(null, oPage), fnFailure, bPartially);
             }
@@ -529,8 +529,7 @@ sap.ui.define([
          * @since 1.11.0
          */
         this.createPageSet = function (sPageSetId, fnSuccess, fnFailure) {
-            let oPageSet;
-            oPageSet = new PageSet(this, sPageSetId);
+            const oPageSet = new PageSet(this, sPageSetId);
             if (fnSuccess) {
                 oPageSet.load(fnSuccess.bind(null, oPageSet), fnFailure);
             }
@@ -601,14 +600,7 @@ sap.ui.define([
             aResult.push("})");
             return aResult.join("");
         };
-
-        // constructor code -------------------------------------------------------
-        oRemoteCatalogServicesByBaseUrl = new Utils.Map();
-        if (!oPbs) {
-            throw new SrvcError("Missing page building service", "Factory");
-        }
-        Log.debug(`Created: ${this}`, null, "Factory");
-    };
+    }
 
     // public factory function ***************************************************
 

package/src/main/js/sap/ushell_abap/pbServices/ui2/ODataService.js

@@ -34,8 +34,12 @@ sap.ui.define([
      * @public
      * @since 1.19.0
      */
-    const ODataService = function (oODataWrapper, fnDefaultFailure) {
-        let sBaseUrl;
+    function ODataService (oODataWrapper, fnDefaultFailure) {
+        if (!oODataWrapper) {
+            throw new SrvcError("Missing OData wrapper", "ODataService");
+        }
+
+        const sBaseUrl = oODataWrapper.getBaseUrl();
 
         // BEWARE: constructor code below!
 
@@ -104,15 +108,13 @@ sap.ui.define([
          * @see sap.ushell_abap.pbServices.ui2.ODataWrapper#read
          */
         this.refreshCsrfToken = function (fnSuccess, fnFailure) {
-            let oTokenData;
-
             if (typeof fnSuccess !== "function") {
                 throw new SrvcError("Success callback is not a function",
                     "ODataService");
             }
             fnFailure = fnFailure || fnDefaultFailure;
 
-            oTokenData = mCsrfTokens.get(sBaseUrl);
+            const oTokenData = mCsrfTokens.get(sBaseUrl);
 
             if (!oTokenData.fetchingInProgress) {
                 // fetch token
@@ -169,13 +171,12 @@ sap.ui.define([
          * @see #refreshCsrfToken
          */
         this.setCsrfToken = function (sNewCsrfToken) {
-            let oTokenData;
             // sNewCsrfToken must be a string OR undefined
             if (sNewCsrfToken && typeof sNewCsrfToken !== "string") {
                 throw new SrvcError(`Invalid CSRF token: ${sNewCsrfToken}`,
                     "ODataService");
             }
-            oTokenData = mCsrfTokens.get(sBaseUrl);
+            const oTokenData = mCsrfTokens.get(sBaseUrl);
             oTokenData.token = sNewCsrfToken;
         };
 
@@ -223,11 +224,7 @@ sap.ui.define([
         };
 
         // constructor code -------------------------------------------------------
-        if (!oODataWrapper) {
-            throw new SrvcError("Missing OData wrapper", "ODataService");
-        }
 
-        sBaseUrl = oODataWrapper.getBaseUrl();
         if (!mCsrfTokens.get(sBaseUrl)) {
             mCsrfTokens.put(sBaseUrl, {
                 // token: undefined,
@@ -238,7 +235,7 @@ sap.ui.define([
         }
 
         this.setDefaultErrorHandler(fnDefaultFailure);
-    };
+    }
 
     /**
      * for test purposes only