@sapui5/sap.ushell_abap 1.151.0 → 1.152.0

package/src/main/js/sap/ushell_abap/bootstrap/SAPCompanionConditionSetter.js

@@ -7,11 +7,9 @@
  */
 sap.ui.define([
     "sap/base/Log",
-    "sap/base/util/Deferred",
     "sap/ushell/bootstrap/Config"
 ], (
     Log,
-    Deferred,
     BootstrapConfig
 ) => {
     "use strict";
@@ -43,7 +41,7 @@ sap.ui.define([
         }
 
         run () {
-            const oDeferred = new Deferred();
+            const oDeferred = Promise.withResolvers();
 
             function fnKeepTrying () {
                 if (this.#counter < this.#timeoutCounter) {

package/src/main/js/sap/ushell_abap/integration/fileshares/NavTargetResolutionFileShareSupport.js

@@ -3,10 +3,12 @@
 sap.ui.define([
     "sap/base/Log",
     "sap/ui/core/Manifest",
+    "sap/ushell/base/promisify",
     "sap/ushell/Container"
 ], (
     Log,
     Manifest,
+    promisify,
     Container
 ) => {
     "use strict";
@@ -46,7 +48,7 @@ sap.ui.define([
     NavTargetResolutionFileShareSupport.getDataSource = function () {
         if (!NavTargetResolutionFileShareSupport._getDataSourcePromise) {
             NavTargetResolutionFileShareSupport._getDataSourcePromise = Container.getServiceAsync("NavTargetResolutionInternal").then((oNavTargetResolutionService) => {
-                return oNavTargetResolutionService.resolveHashFragment(FILE_SHARE_INTENT); // jQuery promise is implicitly converted to native
+                return promisify(oNavTargetResolutionService.resolveHashFragment(FILE_SHARE_INTENT));
             }).then((oResolvedHashFragment) => {
                 const sManifestUrl = oResolvedHashFragment && oResolvedHashFragment.applicationDependencies && oResolvedHashFragment.applicationDependencies.manifest;
                 if (sManifestUrl) {

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

@@ -32,7 +32,7 @@ sap.ui.define([
     const ushellAbapLib = Library.init({
         name: "sap.ushell_abap",
         apiVersion: 2,
-        version: "1.151.0",
+        version: "1.152.0",
         dependencies: [
             "sap.ui.core",
             "sap.m",

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

@@ -8,14 +8,12 @@ sap.ui.define([
     "sap/ui/core/EventBus",
     "sap/ushell_abap/base/Ui2Error",
     "sap/ushell_abap/base/Ui2Utils",
-    "sap/base/Log",
-    "sap/ui/thirdparty/jquery"
+    "sap/base/Log"
 ], (
     EventBus,
-    SrvcError,
-    Utils,
-    Log,
-    jQuery
+    Ui2Error,
+    Ui2Utils,
+    Log
 ) => {
     "use strict";
 
@@ -53,11 +51,13 @@ sap.ui.define([
         let oAlterEgo; // catalog's representation with all relations removed
         let oCatalogPage;
         let aChips = [];
-        let oReadPromise; // if existing and in state "pending", a load/refresh is running; see read
+        let oReadPromise; // shared in-flight promise; see read
+        let bReadPending = false; // true while oReadPromise is in flight
         let bIsStub = true; // TODO replace by aChips === undefined
         let oCachedRemoteErrorArgs;
         let sId;
         let oReadRegisteredChipsPromise; // see readRegisteredChips
+        let bReadRegisteredChipsPending = false; // true while oReadRegisteredChipsPromise is in flight
         let aRegisteredChipIds = [];
         const that = this;
 
@@ -71,7 +71,7 @@ sap.ui.define([
          */
         function checkStub () {
             if (bIsStub) {
-                throw new SrvcError(`${that}: page is just a stub`, "Page");
+                throw new Ui2Error(`${that}: page is just a stub`, "Page");
             }
         }
 
@@ -82,7 +82,7 @@ sap.ui.define([
          */
         function checkAlterEgo () {
             if (!oAlterEgo) {
-                throw new SrvcError(`${that}: catalog is just an ID`, "Catalog");
+                throw new Ui2Error(`${that}: catalog is just an ID`, "Catalog");
             }
         }
 
@@ -106,7 +106,9 @@ sap.ui.define([
             aChips = [];
             bIsStub = true;
             oReadPromise = undefined;
+            bReadPending = false;
             oReadRegisteredChipsPromise = undefined;
+            bReadRegisteredChipsPending = false;
             aRegisteredChipIds = [];
 
             // delete methods on deleted catalog to prevent wrong behavior
@@ -141,11 +143,10 @@ sap.ui.define([
         this.getRemoteCatalogService = function () {
             return oFactory.getRemoteCatalogService(oAlterEgo) || {
                 readChips: function (sBaseUrl, sCatalogId, aChipIds, fnSuccess, fnFailure) {
-                    Log.error(`Catalog '${oAlterEgo.id}', type '${oAlterEgo.type
-                        }' not supported`, null, "Catalog");
+                    Log.error(`Catalog '${oAlterEgo.id}', type '${oAlterEgo.type}' not supported`, null, "Catalog");
                     // fake an empty response to make the catalog valid
                     // When created with alterEgo, we're still synchronous, so make it async
-                    Utils.callHandler(fnSuccess.bind(null, { results: [] }), undefined, /* async=*/true);
+                    Ui2Utils.callHandler(fnSuccess.bind(null, { results: [] }), undefined, /* async=*/true);
                 }
             };
         };
@@ -199,7 +200,8 @@ sap.ui.define([
          * @private
          */
         this.read = function (bRefresh, fnSuccess, fnFailure) {
-            let oDeferred;
+            // only relevant when no read is already in progress
+            const oReadDeferred = Promise.withResolvers();
 
             /**
              * @param {object} oResult
@@ -208,7 +210,8 @@ sap.ui.define([
             function onCatalogLoaded (oResult) {
                 oAlterEgo = oResult;
                 initialize();
-                oDeferred.resolve();
+                bReadPending = false;
+                oReadDeferred.resolve();
             }
 
             /**
@@ -218,7 +221,8 @@ sap.ui.define([
             function onChipsLoaded (oResult) {
                 oAlterEgo.Chips = oResult;
                 initialize();
-                oDeferred.resolve();
+                bReadPending = false;
+                oReadDeferred.resolve();
             }
 
             /**
@@ -244,10 +248,12 @@ sap.ui.define([
              *   delivered by the ODataService. See fnFailure parameter of {@link sap.ushell_abap.pbServices.ui2.ODataWrapper#onError}
              *   for more details.
              */
-            function couldNotLoadRemoteChips () {
+            function couldNotLoadRemoteChips (sErrMsg, oErrDetails) {
                 initialize(); // fail gracefully
-                oCachedRemoteErrorArgs = arguments;
-                oDeferred.reject.apply(this, arguments);
+                // Store as array so getCachedRemoteFailureArguments()[0] and [1] work correctly
+                oCachedRemoteErrorArgs = [sErrMsg, oErrDetails];
+                bReadPending = false;
+                oReadDeferred.reject(oCachedRemoteErrorArgs);
             }
 
             /**
@@ -275,21 +281,29 @@ sap.ui.define([
                 }
             }
 
-            if (!oReadPromise || oReadPromise.state() !== "pending") {
-                oDeferred = new jQuery.Deferred();
-                oReadPromise = oDeferred.promise();
+            if (!bReadPending) {
+                bReadPending = true;
+                // No in-flight read — start a new one
+                oReadPromise = oReadDeferred.promise;
                 if (!oAlterEgo || (bRefresh && !oAlterEgo.remoteId)) {
                     // read the local catalog if it is unknown yet or if refreshing a non-remote catalog
-                    oFactory.getPageBuildingService().readCatalog(sId, processLocalResponse,
-                        oDeferred.reject.bind(oDeferred));
+                    oFactory.getPageBuildingService().readCatalog(
+                        sId,
+                        processLocalResponse,
+                        (sErrMsg, oErrDetails) => {
+                            bReadPending = false;
+                            oReadDeferred.reject([sErrMsg, oErrDetails]);
+                        }
+                    );
                 } else {
                     processLocalResponse.bind(this)(oAlterEgo);
                 }
             }
             fnFailure = fnFailure || oFactory.getPageBuildingService().getDefaultErrorHandler();
             // TODO this needs "sap.ushell_abap.pbServices.ui2.utils" which is never required!
-            oReadPromise.done(Utils.callHandler.bind(null, fnSuccess, fnFailure));
-            oReadPromise.fail(fnFailure);
+            oReadPromise
+                .then(() => { Ui2Utils.callHandler(fnSuccess, fnFailure); })
+                .catch((aRejectArgs) => { fnFailure(aRejectArgs[0], aRejectArgs[1]); });
         };
 
         // "public" methods ----------------------------------------------------------
@@ -334,7 +348,7 @@ sap.ui.define([
         this.addSystemToServiceUrl = function (sServiceUrl, sSystem) {
             /* jslint regexp:true */
             if (sServiceUrl.indexOf("/") !== 0 || sServiceUrl.indexOf("//") === 0) {
-                throw new SrvcError(`addSystemToServiceUrl: Invalid URL: ${sServiceUrl}`,
+                throw new Ui2Error(`addSystemToServiceUrl: Invalid URL: ${sServiceUrl}`,
                     "Catalog");
             }
             sSystem = sSystem || this.getSystemAlias();
@@ -495,7 +509,7 @@ sap.ui.define([
          */
         this.getChips = function () {
             if (bIsStub) {
-                throw new SrvcError(`${that}: catalog is just a stub`, "Catalog");
+                throw new Ui2Error(`${that}: catalog is just a stub`, "Catalog");
             }
             return aChips.slice();
         };
@@ -657,7 +671,7 @@ sap.ui.define([
          */
         this.load = function (fnSuccess, fnFailure) {
             if (!bIsStub) {
-                throw new SrvcError("Catalog is not a stub anymore", "Catalog");
+                throw new Ui2Error("Catalog is not a stub anymore", "Catalog");
             }
             this.read(false, fnSuccess, fnFailure);
         };
@@ -686,7 +700,7 @@ sap.ui.define([
                     oFactory.createChip(oRawChipData);
                 });
                 // TODO this needs "utils" which is never required!
-                Utils.callHandler(fnSuccess, fnFailure);
+                Ui2Utils.callHandler(fnSuccess, fnFailure);
             }
 
             function createRemoteChips (oRawData) {
@@ -734,26 +748,38 @@ sap.ui.define([
          */
         this.readRegisteredChips = function (fnSuccess, fnFailure) {
             function start () {
-                const oDeferred = new jQuery.Deferred();
+                const oReadDeferred = Promise.withResolvers();
                 const aChipIds = aRegisteredChipIds;
 
                 aRegisteredChipIds = [];
+                bReadRegisteredChipsPending = true;
                 if (aChipIds.length) {
-                    that.readChips(aChipIds, oDeferred.resolve.bind(oDeferred),
-                        oDeferred.reject.bind(oDeferred));
+                    that.readChips(
+                        aChipIds,
+                        () => {
+                            bReadRegisteredChipsPending = false;
+                            oReadDeferred.resolve();
+                        },
+                        (sErrMsg, oErrDetails) => {
+                            bReadRegisteredChipsPending = false;
+                            oReadDeferred.reject([sErrMsg, oErrDetails]);
+                        }
+                    );
                 } else {
-                    oDeferred.resolve();
+                    bReadRegisteredChipsPending = false;
+                    oReadDeferred.resolve();
                 }
-                return oDeferred.promise();
+                return oReadDeferred.promise;
             }
 
-            if (!oReadRegisteredChipsPromise || oReadRegisteredChipsPromise.state() !== "pending") {
+            if (!oReadRegisteredChipsPromise || !bReadRegisteredChipsPending) {
                 oReadRegisteredChipsPromise = start();
             }
             fnFailure = fnFailure || oFactory.getPageBuildingService().getDefaultErrorHandler();
             // TODO this needs "utils" which is never required!
-            oReadRegisteredChipsPromise.fail(fnFailure)
-                .done(Utils.callHandler.bind(null, fnSuccess, fnFailure));
+            oReadRegisteredChipsPromise
+                .then(() => { Ui2Utils.callHandler(fnSuccess, fnFailure); })
+                .catch((aRejectArgs) => { fnFailure(aRejectArgs[0], aRejectArgs[1]); });
         };
 
         /**
@@ -788,8 +814,8 @@ sap.ui.define([
          * @private
          */
         this.registerChip = function (oChip) {
-            if (oReadRegisteredChipsPromise && oReadRegisteredChipsPromise.state() === "pending") {
-                throw new SrvcError(
+            if (bReadRegisteredChipsPending) {
+                throw new Ui2Error(
                     "Invalid state: registerChip while readRegisteredChips pending",
                     "Catalog"
                 );
@@ -821,8 +847,8 @@ sap.ui.define([
          */
         this.remove = function (fnSuccess, fnFailure) {
             checkAlterEgo();
-            if (oReadPromise && oReadPromise.state() === "pending") {
-                throw new SrvcError(`Catalog is being refreshed: ${this}`,
+            if (bReadPending) {
+                throw new Ui2Error(`Catalog is being refreshed: ${this}`,
                     "Catalog");
             }
 
@@ -897,7 +923,7 @@ sap.ui.define([
             checkAlterEgo();
             if (Object.hasOwnProperty.call(oCatalogData, "__metadata")) {
                 // ODataWrapper#update() relies on __metadata!
-                throw new SrvcError("Unsupported __metadata update", "Catalog");
+                throw new Ui2Error("Unsupported __metadata update", "Catalog");
             }
             const oNewAlterEgo = JSON.parse(JSON.stringify(oAlterEgo));
             Object.keys(oCatalogData).forEach((sName) => {
@@ -931,7 +957,7 @@ sap.ui.define([
             sId = vCatalogData;
         }
         if (!sId) {
-            throw new SrvcError("Missing ID", "Catalog");
+            throw new Ui2Error("Missing ID", "Catalog");
         }
         Log.debug(`Created: ${this}`, null, "Catalog");
     }

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

@@ -5,6 +5,7 @@
  */
 sap.ui.define([
     "sap/base/Log",
+    "sap/ushell/base/requireAsync",
     "sap/ushell_abap/pbServices/ui2/Bag",
     "sap/ushell_abap/pbServices/ui2/ChipDefinition",
     "sap/ushell_abap/base/Ui2Error",
@@ -12,6 +13,7 @@ sap.ui.define([
     "sap/ushell/utils/DeprecationLog"
 ], (
     Log,
+    requireAsync,
     Bag,
     ChipDefinition,
     SrvcError,
@@ -558,21 +560,13 @@ sap.ui.define([
                 const ComponentContainer = sap.ui.requireSync("sap/ui/core/ComponentContainer"); // LEGACY API (deprecated)
                 const oComponentContainer = new ComponentContainer();
                 // load the content lazily to enable components using async interface
-                this.oComponentPromise = new Promise((resolve, reject) => {
-                    sap.ui.require([
-                        "sap/ui/core/Component"
-                    ], (Component) => {
-                        Component.create({
-                            name: oImplementation.componentName,
-                            componentData: oData,
-                            manifest: false
-                        })
-                            .then((oComponent) => {
-                                oComponentContainer.setComponent(oComponent);
-                            })
-                            .then(resolve)
-                            .catch(reject);
+                this.oComponentPromise = requireAsync(["sap/ui/core/Component"]).then(async ([Component]) => {
+                    const oComponent = await Component.create({
+                        name: oImplementation.componentName,
+                        componentData: oData,
+                        manifest: false
                     });
+                    oComponentContainer.setComponent(oComponent);
                 });
 
                 return oComponentContainer;
@@ -602,59 +596,54 @@ sap.ui.define([
         this.getImplementationAsSapui5Async = async function (oApi) {
             checkStub();
 
-            return new Promise((resolve, reject) => {
-                sap.ui.require([
-                    "sap/ui/core/ComponentContainer",
-                    "sap/ui/core/Component",
-                    "sap/ui/core/mvc/View"
-                ], (ComponentContainer, Component, View) => {
-                    const oData = {
-                        /**
-                         * @namespace
-                         * The namespace for the instance specific CHIP API, which allows you to
-                         * access the various contracts consumed by your CHIP instance.
-                         * @private
-                         */
-                        chip: oApi
-                    };
-                    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
-                    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);
-                            sap.ui.loader.config({ paths: JSON.parse(`{"${oImplementation.$Namespace.replace(/\./g, "/")}":"${oImplementation.$absolutePath}"}`) });
-                        } else {
-                            sap.ui.loader.config({ paths: JSON.parse(`{"${oImplementation.$Namespace.replace(/\./g, "/")}":"${oImplementation.$UrlPrefix}"}`) });
-                        }
-                    }
+            const [
+                ComponentContainer,
+                Component,
+                View
+            ] = await requireAsync([
+                "sap/ui/core/ComponentContainer",
+                "sap/ui/core/Component",
+                "sap/ui/core/mvc/View"
+            ]);
+            const oData = {
+                /**
+                 * @namespace
+                 * The namespace for the instance specific CHIP API, which allows you to
+                 * access the various contracts consumed by your CHIP instance.
+                 * @private
+                 */
+                chip: oApi
+            };
+            const oImplementation = oDefinition.implementation.sapui5;
 
-                    if (oImplementation.componentName) {
-                        // SAPUI5 component
-                        return Component.create({
-                            name: oImplementation.componentName,
-                            componentData: oData,
-                            manifest: false
-                        })
-                            .then((oComponent) => {
-                                return new ComponentContainer({
-                                    component: oComponent
-                                });
-                            })
-                            .then(resolve)
-                            .catch(reject);
-                    }
+            // In case the chip is used in FLP wave 2 or later context we load it from the standard path
+            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);
+                    sap.ui.loader.config({ paths: JSON.parse(`{"${oImplementation.$Namespace.replace(/\./g, "/")}":"${oImplementation.$absolutePath}"}`) });
+                } else {
+                    sap.ui.loader.config({ paths: JSON.parse(`{"${oImplementation.$Namespace.replace(/\./g, "/")}":"${oImplementation.$UrlPrefix}"}`) });
+                }
+            }
 
-                    // SAPUI5 MVC
-                    return View.create({
-                        type: oImplementation.$ViewType,
-                        viewName: oImplementation.$ViewName,
-                        viewData: oData
-                    })
-                        .then(resolve)
-                        .catch(reject);
+            if (oImplementation.componentName) {
+                // SAPUI5 component
+                const oComponent = await Component.create({
+                    name: oImplementation.componentName,
+                    componentData: oData,
+                    manifest: false
+                });
+                return new ComponentContainer({
+                    component: oComponent
                 });
+            }
+
+            // SAPUI5 MVC
+            return View.create({
+                type: oImplementation.$ViewType,
+                viewName: oImplementation.$ViewName,
+                viewData: oData
             });
         };