@sapui5/sap.ushell 1.140.0 → 1.141.0

package/src/main/js/sap/ushell/ApplicationType/urlTemplateResolution.js

@@ -2,7 +2,6 @@
 sap.ui.define([
     "sap/base/Log",
     "sap/base/i18n/Localization",
-    "sap/ui/thirdparty/hasher",
     "sap/base/util/ObjectPath",
     "sap/ushell/utils",
     "sap/ushell/User",
@@ -15,11 +14,12 @@ sap.ui.define([
     "sap/ushell/Container",
     "sap/ushell/ApplicationType/guiResolution",
     "sap/ushell/utils/UrlParsing",
-    "sap/ushell/ApplicationType/systemAlias"
+    "sap/ushell/ApplicationType/systemAlias",
+    "sap/ushell/appIntegration/ApplicationContainerCache",
+    "sap/ushell/URLTemplateProcessor"
 ], (
     Log,
     Localization,
-    hasher,
     ObjectPath,
     ushellUtils,
     User,
@@ -32,39 +32,31 @@ sap.ui.define([
     Container,
     guiResolution,
     oURLParsing,
-    SystemAlias
+    SystemAlias,
+    ApplicationContainerCache,
+    URLTemplateProcessor
 ) => {
     "use strict";
 
-    const oUrlParams = (new URI(document.URL)).search(true);
-    const sIframeURLDomainForTests = oUrlParams["iframe-url"];
-
-    // eslint-disable-next-line prefer-const
-    let oExportedAPIs;
-
     // extracts the inner app route from the browser hash
-    function getInnerAppRoute (oTemplateContext) {
+    function getInnerAppRoute (oTemplateContext, sFullHash) {
         const oTemplatePayload = oTemplateContext.payload;
-        let sInnerAppRoute;
-        const sHashFragment = hasher.getHash() || oExportedAPIs.getBrowserHash();
-        const indexOfInnerRoute = sHashFragment.indexOf("&/");
+        const iIndexOfInnerRoute = sFullHash.indexOf("&/");
+        const bIsJam = oTemplateContext.siteAppSection?.["sap.integration"]?.urlTemplateId === "urltemplate.jam";
         let iOffset = 1; // to avoid the starting "&";
-        let bIsJam = false;
 
-        if (oTemplateContext.siteAppSection?.["sap.integration"]?.urlTemplateId === "urltemplate.jam") {
-            bIsJam = true;
-        }
-        if (indexOfInnerRoute > 0) {
-            if (oTemplatePayload && oTemplatePayload.capabilities && oTemplatePayload.capabilities.appFrameworkId === "UI5") {
+        let sInnerAppRoute;
+        if (iIndexOfInnerRoute > 0) {
+            if (oTemplatePayload?.capabilities?.appFrameworkId === "UI5") {
                 iOffset = 2; // to avoid the starting "&/"
             }
-            sInnerAppRoute = sHashFragment.substring(indexOfInnerRoute + iOffset);
+            sInnerAppRoute = sFullHash.substring(iIndexOfInnerRoute + iOffset);
             try {
-                if (!bIsJam && sInnerAppRoute && sInnerAppRoute.length > 0) {
+                if (!bIsJam && sInnerAppRoute?.length > 0) {
                     sInnerAppRoute = decodeURIComponent(sInnerAppRoute);
                 }
-            } catch (e) {
-                Log.warning("inner route should be double encoded", e, "sap.ushell.ApplicationType.getInnerAppRoute");
+            } catch (oError) {
+                Log.warning("inner route should be double encoded", oError, "sap.ushell.ApplicationType.getInnerAppRoute");
             }
         }
         return sInnerAppRoute;
@@ -82,12 +74,11 @@ sap.ui.define([
         return sMode;
     }
 
-    function getInnerAppState () {
-        const sHash = hasher && hasher.getHash();
+    function getInnerAppState (sFullHash) {
         let sKey = "";
 
-        if (sHash && sHash.length > 0 && sHash.indexOf("sap-iapp-state=") > 0) {
-            const aParams = /(?:sap-iapp-state=)([^&/\\]+)/.exec(sHash);
+        if (sFullHash.length > 0 && sFullHash.indexOf("sap-iapp-state=") > 0) {
+            const aParams = /(?:sap-iapp-state=)([^&/\\]+)/.exec(sFullHash);
             if (aParams && aParams.length === 2) {
                 sKey = aParams[1];
             }
@@ -96,57 +87,54 @@ sap.ui.define([
         return sKey;
     }
 
-    function createEnv () {
-        return new Promise((fnResolve) => {
-            Promise.all([
-                Container.getServiceAsync("UserInfo"),
-                Container.getServiceAsync("PluginManager"),
-                ushellUtils.getUi5Version()
-            ]).then((aResults) => {
-                const oUserInfoService = aResults[0];
-                const oPluginsService = aResults[1];
-                const sUi5Version = aResults[2];
-
-                const oUser = oUserInfoService.getUser();
-                const sContentDensity = oUser.getContentDensity() || (document.body.classList.contains("sapUiSizeCompact") ? "compact" : "cozy");
-                let sTheme = oUser.getTheme();
-                if (sTheme.indexOf("sap_") !== 0) {
-                    const sThemeFormat = User.prototype.constants.themeFormat.THEME_NAME_PLUS_URL;
-                    sTheme = oUser.getTheme(sThemeFormat);
-                }
+    function createEnv (sFullHash) {
+        return Promise.all([
+            Container.getServiceAsync("UserInfo"),
+            Container.getServiceAsync("PluginManager"),
+            ushellUtils.getUi5Version()
+        ]).then((aResults) => {
+            const oUserInfoService = aResults[0];
+            const oPluginsService = aResults[1];
+            const sUi5Version = aResults[2];
+
+            const oUser = oUserInfoService.getUser();
+            const sContentDensity = oUser.getContentDensity() || (document.body.classList.contains("sapUiSizeCompact") ? "compact" : "cozy");
+            let sTheme = oUser.getTheme();
+            if (sTheme.indexOf("sap_") !== 0) {
+                const sThemeFormat = User.prototype.constants.themeFormat.THEME_NAME_PLUS_URL;
+                sTheme = oUser.getTheme(sThemeFormat);
+            }
 
-                const sLanguage = Localization.getLanguage();
-                const sLogonLanguage = Localization.getSAPLogonLanguage();
+            const sLanguage = Localization.getLanguage();
+            const sLogonLanguage = Localization.getSAPLogonLanguage();
 
-                const themeServiceRoot = `${window.location.protocol}//${window.location.host // host
-                }/comsapuitheming.runtime/themeroot/v1`; // route to theme service
-                let sessionTimeout = 0;
-                if (Config.last("/core/shell/sessionTimeoutIntervalInMinutes") > 0) {
-                    sessionTimeout = Config.last("/core/shell/sessionTimeoutIntervalInMinutes");
-                }
+            const themeServiceRoot = `${window.location.protocol}//${window.location.host}/comsapuitheming.runtime/themeroot/v1`;
+            let sessionTimeout = 0;
+            if (Config.last("/core/shell/sessionTimeoutIntervalInMinutes") > 0) {
+                sessionTimeout = Config.last("/core/shell/sessionTimeoutIntervalInMinutes");
+            }
 
-                let debugMode = false;
-                if (window["sap-ui-debug"] !== false && window["sap-ui-debug"] !== undefined) {
-                    debugMode = window["sap-ui-debug"];
-                }
+            let debugMode = false;
+            if (window["sap-ui-debug"] !== false && window["sap-ui-debug"] !== undefined) {
+                debugMode = window["sap-ui-debug"];
+            }
 
-                const bEnablePersonalization = Config.last("/core/shell/enablePersonalization");
-
-                fnResolve({
-                    language: sLanguage,
-                    logonLanguage: sLogonLanguage,
-                    theme: sTheme,
-                    themeServiceRoot: themeServiceRoot,
-                    isDebugMode: debugMode,
-                    ui5Version: sUi5Version,
-                    contentDensity: sContentDensity,
-                    sapPlugins: oPluginsService._getNamesOfPluginsWithAgents(),
-                    innerAppState: getInnerAppState(),
-                    sessionTimeout: sessionTimeout,
-                    historyDirection: History.getInstance().getDirection() || "",
-                    enableShellPersonalization: bEnablePersonalization
-                });
-            });
+            const bEnablePersonalization = Config.last("/core/shell/enablePersonalization");
+
+            return {
+                language: sLanguage,
+                logonLanguage: sLogonLanguage,
+                theme: sTheme,
+                themeServiceRoot: themeServiceRoot,
+                isDebugMode: debugMode,
+                ui5Version: sUi5Version,
+                contentDensity: sContentDensity,
+                sapPlugins: oPluginsService._getNamesOfPluginsWithAgents(),
+                innerAppState: getInnerAppState(sFullHash),
+                sessionTimeout: sessionTimeout,
+                historyDirection: History.getInstance().getDirection() || "",
+                enableShellPersonalization: bEnablePersonalization
+            };
         });
     }
 
@@ -168,8 +156,8 @@ sap.ui.define([
             });
 
             // guiResolution
-            const aUnneccessaryParameters = guiResolution.getUnnecessaryWebguiParameters(oRuntime.startupParameter, oInbound || {});
-            guiResolution.removeObjectKey(oRuntime.startupParameter, aUnneccessaryParameters);
+            const aUnnecessaryParameters = guiResolution.getUnnecessaryWebguiParameters(oRuntime.startupParameter, oInbound || {});
+            guiResolution.removeObjectKey(oRuntime.startupParameter, aUnnecessaryParameters);
 
             const oEffectiveParametersToAppend = guiResolution.getWebguiNonBusinessParameters(oRuntime.startupParameter);
             guiResolution.removeObjectKey(oRuntime.startupParameter, Object.keys(oEffectiveParametersToAppend));
@@ -183,7 +171,7 @@ sap.ui.define([
         }
     }
 
-    // Special temporary code to fix for SAPIT internal sap site until they will start using SWZ external
+    // Special temporary code to fix for SAP IT internal sap site until they will start using SWZ external
     // subaccount content provider (then, this code should be removed).
     // Please contact Alon Barnes
     function handleSAPITTempSolution (sURLTemplateId, oTemplatePayload, oTemplateParams) {
@@ -198,7 +186,7 @@ sap.ui.define([
                 };
                 return oPayload;
             }
-        } catch (e) {
+        } catch {
             // ignore - will return the original payload to continue as before
         }
         return oTemplatePayload;
@@ -297,14 +285,14 @@ sap.ui.define([
                     }
 
                     fnResolve(sUrlCompacted);
-                }).fail((sError) => {
-                    fnReject(sError);
+                }).fail((oError) => {
+                    fnReject(oError);
                 });
             });
         });
     }
 
-    function isTransformationEnabled (URLTemplateProcessor, oTransformation, oTemplateContext) {
+    function isTransformationEnabled (oTransformation, oTemplateContext) {
         if (typeof oTransformation.enabled === "boolean") {
             return oTransformation.enabled;
         }
@@ -327,11 +315,11 @@ sap.ui.define([
             oTransformData.oResolvedParameters.enabled : false);
     }
 
-    function handleURLTransformation (URLTemplateProcessor, sUrl, oCapabilities, oTemplateContext) {
+    function handleURLTransformation (sUrl, oCapabilities, oTemplateContext) {
         return new Promise((fnResolve) => {
             const oTransformation = oCapabilities.urlTransformation || { enabled: false };
 
-            if (isTransformationEnabled(URLTemplateProcessor, oTransformation, oTemplateContext)) {
+            if (isTransformationEnabled(oTransformation, oTemplateContext)) {
                 const oIframeURI = new URI(sUrl);
                 const oFirstTransformation = oTransformation.transformations[0];
                 const oService = oFirstTransformation.service.uri;
@@ -359,8 +347,8 @@ sap.ui.define([
                     queryParams: oTransformData.oResolvedParameters
                 }).toString();
 
-                sap.ui.require(["sap/ui/thirdparty/datajs"], (OData) => {
-                    OData.read({
+                sap.ui.require(["sap/ui/thirdparty/datajs"], (datajs) => {
+                    datajs.read({
                         requestUri: sServiceUrl,
                         headers: {
                             "Cache-Control": "no-cache, no-store, must-revalidate",
@@ -404,10 +392,10 @@ sap.ui.define([
                         fnResolve(sUrl);
                     },
                     // Fail handler
-                    (oMessage) => {
+                    (oDataJsError) => {
                         Log.error(
                             "URL Transformation Failed",
-                            JSON.stringify(oMessage),
+                            JSON.stringify(oDataJsError.message),
                             "sap.ushell.ApplicationType"
                         );
                         fnResolve(sUrl);
@@ -444,7 +432,7 @@ sap.ui.define([
 
                 Log.error(
                     "App-defined navigation mode was ignored",
-                    "Application requests to be opened inplace, but the template's navigation mode doesn't allow!",
+                    "Application requests to be opened inplace, but the template's navigation mode doesn't allow it.",
                     "sap.ushell.ApplicationType"
                 );
 
@@ -460,17 +448,20 @@ sap.ui.define([
 
                 Log.error(
                     "App-defined navigation mode was ignored",
-                    "Application requests to be opened explace, but the template's navigation mode doesn't allow!",
+                    "Application requests to be opened explace, but the template's navigation mode doesn't allow it.",
                     "sap.ushell.ApplicationType"
                 );
 
                 break;
             default: // Fallback to URL template-defined navigation mode.
-                Log.error(
-                    "App-defined navigation mode is not valid!",
-                    "Fallback to URL Template's capability",
-                    "sap.ushell.ApplicationType"
-                );
+                if (sAppExternalNavMode) {
+                    Log.error(
+                        "App-defined navigation mode is not valid",
+                        sAppExternalNavMode,
+                        "Fallback to URL Template's capability",
+                        "sap.ushell.ApplicationType"
+                    );
+                }
 
                 if (["embedded", /* Legacy Modes: */ "inplace", "newWindowThenEmbedded"].includes(sTemplateNavigationMode)) {
                     return "embedded";
@@ -536,232 +527,216 @@ sap.ui.define([
         return oAppCapabilities;
     }
 
-    function generateURLTemplateResolutionResult (oMatchingTarget, sBaseUrl, fnExternalSystemAliasResolver) {
-        return new Promise((fnResolveGlobal) => {
-            sap.ui.require(["sap/ushell/URLTemplateProcessor"], (URLTemplateProcessor) => {
-                const oInbound = oMatchingTarget.inbound;
-                const oTemplateContext = oInbound.templateContext;
-                const oCapabilities = oTemplateContext.payload.capabilities || {};
-
-                // Although the URL templates in WZ (e.g. JAM teamplate for WZ Advanced Homepage -> workzone-home) state "newWindow"
-                // they need to run "embedded" until the URL templates are fixed.
-                // TODO: Remove code when URL templates are fixed!
-                if (sap?.cf?.config?.siteId !== undefined && oTemplateContext?.siteAppSection["sap.integration"]?.navMode === "inplace") {
-                    Log.warning("URL template's navigationMode capability is ignored! navigationMode is forced to 'embedded'.");
-                    oCapabilities.navigationMode = "embedded";
-                }
+    async function generateURLTemplateResolutionResult (oMatchingTarget, sBaseUrl, fnExternalSystemAliasResolver) {
+        const oInbound = oMatchingTarget.inbound;
+        const oTemplateContext = oInbound.templateContext;
+        const oCapabilities = oTemplateContext.payload.capabilities || {};
+        let sFullHash = oMatchingTarget.fullHash;
+        let sReplaceHashBeforeOpenAppInplace;
+
+        // Although the URL templates in WZ (e.g. JAM template for WZ Advanced Homepage -> workzone-home) state "newWindow"
+        // they need to run "embedded" until the URL templates are fixed.
+        // TODO: Remove code when URL templates are fixed!
+        if (sap?.cf?.config?.siteId !== undefined && oTemplateContext?.siteAppSection["sap.integration"]?.navMode === "inplace") {
+            Log.warning("URL template's navigationMode capability is ignored! navigationMode is forced to 'embedded'.");
+            oCapabilities.navigationMode = "embedded";
+        }
 
-                if (oMatchingTarget.mappedIntentParamsPlusSimpleDefaults &&
-                    oMatchingTarget.mappedIntentParamsPlusSimpleDefaults.hasOwnProperty("sap-ushell-innerAppRoute")) {
-                    let sTopHash = hasher.getHash();
-                    if (oMatchingTarget.mappedIntentParamsPlusSimpleDefaults["sap-ushell-innerAppRoute"].length > 0 &&
-                        sTopHash.indexOf("&/") === -1) {
-                        sTopHash += `&/${oMatchingTarget.mappedIntentParamsPlusSimpleDefaults["sap-ushell-innerAppRoute"]}`;
-                        hasher.replaceHash(sTopHash);
-                    }
-                }
-                // "mappedIntentParamsPlusSimpleDefaults" are a result of URLPasing.parseShellHash(). With that, parameters with "no" value are expressed
-                // as array with empty string, e.g. {sap-system: [""]}.
-                // However, that is not handled correctly in the URLTemplateProcessor (as an array with empty string is handled as "a value", not as undefined/empty).
-                oURLParsing.removeParametersWithEmptyValue(oMatchingTarget.mappedIntentParamsPlusSimpleDefaults, ["sap-system"]);
-
-                /*
-                 * Attention!
-                 * The names in this object must be kept stable. They might
-                 * appear at any time in any template at runtime. Also, choose a name
-                 * that can be read by a user. E.g., defaultParameterNames is good,
-                 * mappedDefaultedParamNames is bad.
-                 */
-                const oRuntime = {
-                    // the inner app route
-                    innerAppRoute: getInnerAppRoute(oTemplateContext) || oMatchingTarget.parsedIntent.appSpecificRoute,
-                    // the target navigation mode
-                    targetNavMode: getTargetNavigationMode(oMatchingTarget),
-                    // the names of default parameters among the startupParameters
-                    defaultParameterNames: oMatchingTarget.mappedDefaultedParamNames,
-                    /*
-                     * the parameters (defaults + inent parameters) that must be passed
-                     * to the application in order to start it
-                     */
-                    startupParameter: oMatchingTarget.mappedIntentParamsPlusSimpleDefaults,
-                    // remote application information (for the scube scenario)
-                    remoteApplication: {
-                        remoteSO: undefined,
-                        remoteAction: undefined
-                    }
-                };
+        if (
+            oMatchingTarget.mappedIntentParamsPlusSimpleDefaults
+            && oMatchingTarget.mappedIntentParamsPlusSimpleDefaults.hasOwnProperty("sap-ushell-innerAppRoute")
+        ) {
+            if (
+                oMatchingTarget.mappedIntentParamsPlusSimpleDefaults["sap-ushell-innerAppRoute"].length > 0
+                && sFullHash.indexOf("&/") === -1
+            ) {
+                sFullHash += `&/${oMatchingTarget.mappedIntentParamsPlusSimpleDefaults["sap-ushell-innerAppRoute"]}`;
+                sReplaceHashBeforeOpenAppInplace = sFullHash;
+            }
+        }
+        // "mappedIntentParamsPlusSimpleDefaults" are a result of URLParsing.parseShellHash(). With that, parameters with "no" value are expressed
+        // as array with empty string, e.g. {sap-system: [""]}.
+        // However, that is not handled correctly in the URLTemplateProcessor (as an array with empty string is handled as "a value", not as undefined/empty).
+        oURLParsing.removeParametersWithEmptyValue(oMatchingTarget.mappedIntentParamsPlusSimpleDefaults, ["sap-system"]);
+
+        /*
+         * Attention!
+         * The names in this object must be kept stable. They might
+         * appear at any time in any template at runtime. Also, choose a name
+         * that can be read by a user. E.g., defaultParameterNames is good,
+         * mappedDefaultedParamNames is bad.
+         */
+        const oRuntime = {
+            // the inner app route
+            innerAppRoute: getInnerAppRoute(oTemplateContext, sFullHash) || oMatchingTarget.parsedIntent.appSpecificRoute,
+            // the target navigation mode
+            targetNavMode: getTargetNavigationMode(oMatchingTarget),
+            // the names of default parameters among the startupParameters
+            defaultParameterNames: oMatchingTarget.mappedDefaultedParamNames,
+            /*
+             * the parameters (defaults + intent parameters) that must be passed
+             * to the application in order to start it
+             */
+            startupParameter: oMatchingTarget.mappedIntentParamsPlusSimpleDefaults,
+            // remote application information (for the scube scenario)
+            remoteApplication: {
+                remoteSO: undefined,
+                remoteAction: undefined
+            }
+        };
 
-                // eslint-disable-next-line complexity
-                createEnv().then((oEnv) => {
-                    // the runtime environment, containing data from the current state of the FLP
-                    oRuntime.env = oEnv;
+        const oEnv = await createEnv(sFullHash);
 
-                    const oTemplateParams = ObjectPath.get(["sap.integration", "urlTemplateParams", "query"], oTemplateContext.siteAppSection) || {};
-                    if (oTemplateParams.hasOwnProperty("sap-cssurl")) {
-                        oRuntime.env.themeServiceRoot = undefined;
-                        oRuntime.env.theme = undefined;
-                    }
+        // the runtime environment, containing data from the current state of the FLP
+        oRuntime.env = oEnv;
 
-                    if (oCapabilities.appFrameworkId === "UI5" && oRuntime.startupParameter) {
-                        for (const key in oRuntime.startupParameter) {
-                            if (key !== "sap-ushell-innerAppRoute") {
-                                oRuntime.startupParameter[key][0] = encodeURIComponent(oRuntime.startupParameter[key][0]);
-                            }
-                            if (key === "sap-shell-so") {
-                                oRuntime.remoteApplication.remoteSO = oRuntime.startupParameter[key][0];
-                            }
-                            if (key === "sap-shell-action") {
-                                oRuntime.remoteApplication.remoteAction = oRuntime.startupParameter[key][0];
-                            }
-                        }
-                        if (oMatchingTarget.mappedDefaultedParamNames && oMatchingTarget.mappedDefaultedParamNames.length > 0) {
-                            const tmpMappedDefaultedParamNames = oMatchingTarget.mappedDefaultedParamNames.filter((paramName) => {
-                                return paramName !== "sap-shell-so" && paramName !== "sap-shell-action" && paramName !== "sap-system";
-                            });
-                            oRuntime.startupParameter["sap-ushell-defaultedParameterNames"] = [JSON.stringify(tmpMappedDefaultedParamNames)];
-                        }
-                        delete oRuntime.startupParameter["sap-shell-so"];
-                        delete oRuntime.startupParameter["sap-shell-action"];
-                    }
+        const oTemplateParams = ObjectPath.get(["sap.integration", "urlTemplateParams", "query"], oTemplateContext.siteAppSection) || {};
+        if (oTemplateParams.hasOwnProperty("sap-cssurl")) {
+            oRuntime.env.themeServiceRoot = undefined;
+            oRuntime.env.theme = undefined;
+        }
 
-                    // this is a hot fix made for an escelation opened by
-                    // Lockheed Martin 885662/2021 about sap workzone, which
-                    // is the only quick way to solve the issue. A more
-                    // deep process to solve the issue in a proper way will
-                    // be done via a BLI as the issue is very complex
-                    // please do not remove the usage of "window.location.hash"
-                    const sJamSearchPref = "/universal_search/search";
-                    const iJamSearchPos = oRuntime.innerAppRoute && oRuntime.innerAppRoute.indexOf(sJamSearchPref);
-                    let JamInnerRoute;
-                    if (iJamSearchPos === 0) {
-                        JamInnerRoute = oRuntime.innerAppRoute.substring(1);
-                        oRuntime.innerAppRoute = "/JAMSEARCHPATHH?JAMSEARCHVALUEE=VALUEE";
-                    }
+        if (oCapabilities.appFrameworkId === "UI5" && oRuntime.startupParameter) {
+            for (const sKey in oRuntime.startupParameter) {
+                if (sKey !== "sap-ushell-innerAppRoute") {
+                    oRuntime.startupParameter[sKey][0] = encodeURIComponent(oRuntime.startupParameter[sKey][0]);
+                }
+                if (sKey === "sap-shell-so") {
+                    oRuntime.remoteApplication.remoteSO = oRuntime.startupParameter[sKey][0];
+                }
+                if (sKey === "sap-shell-action") {
+                    oRuntime.remoteApplication.remoteAction = oRuntime.startupParameter[sKey][0];
+                }
+            }
+            if (oMatchingTarget.mappedDefaultedParamNames && oMatchingTarget.mappedDefaultedParamNames.length > 0) {
+                const tmpMappedDefaultedParamNames = oMatchingTarget.mappedDefaultedParamNames.filter((paramName) => {
+                    return paramName !== "sap-shell-so" && paramName !== "sap-shell-action" && paramName !== "sap-system";
+                });
+                oRuntime.startupParameter["sap-ushell-defaultedParameterNames"] = [JSON.stringify(tmpMappedDefaultedParamNames)];
+            }
+            delete oRuntime.startupParameter["sap-shell-so"];
+            delete oRuntime.startupParameter["sap-shell-action"];
+        }
 
-                    // Special temporary code to fix issues when openning web gui application
-                    // this code will be removed once the URL template in CF will be fixed.
-                    // Please contact Alon Barnes
-                    const sURLTemplateId = ushellUtils.getMember(oTemplateContext.siteAppSection, "sap|integration.urlTemplateId");
-                    let oTemplatePayload = oTemplateContext.payload;
-                    if (sURLTemplateId === "urltemplate.gui") {
-                        oTemplatePayload = deepClone(oTemplateContext.payload, 20);
-                        handleTempWebGuiBugTemp(oTemplatePayload, oRuntime, oInbound);
-                    }
+        // this is a hot fix made for an escalation opened by
+        // Lockheed Martin 885662/2021 about sap workzone, which
+        // is the only quick way to solve the issue. A more
+        // deep process to solve the issue in a proper way will
+        // be done via a BLI as the issue is very complex
+        const sJamSearchPref = "/universal_search/search";
+        const iJamSearchPos = oRuntime.innerAppRoute && oRuntime.innerAppRoute.indexOf(sJamSearchPref);
+        let JamInnerRoute;
+        if (iJamSearchPos === 0) {
+            JamInnerRoute = oRuntime.innerAppRoute.substring(1);
+            oRuntime.innerAppRoute = "/JAMSEARCHPATHH?JAMSEARCHVALUEE=VALUEE";
+        }
 
-                    // See comment in function for this temporary special case
-                    oTemplatePayload = handleSAPITTempSolution(sURLTemplateId, oTemplatePayload, oTemplateParams);
+        // Special temporary code to fix issues when opening web gui application
+        // this code will be removed once the URL template in CF will be fixed.
+        // Please contact Alon Barnes
+        const sURLTemplateId = ushellUtils.getMember(oTemplateContext.siteAppSection, "sap|integration.urlTemplateId");
+        let oTemplatePayload = oTemplateContext.payload;
+        if (sURLTemplateId === "urltemplate.gui") {
+            oTemplatePayload = deepClone(oTemplateContext.payload, 20);
+            handleTempWebGuiBugTemp(oTemplatePayload, oRuntime, oInbound);
+        }
 
-                    let sURL = URLTemplateProcessor.expand(
-                        oTemplatePayload,
-                        oTemplateContext.site,
-                        oRuntime,
-                        oTemplateContext.siteAppSection,
-                        "startupParameter"
-                    );
+        // See comment in function for this temporary special case
+        oTemplatePayload = handleSAPITTempSolution(sURLTemplateId, oTemplatePayload, oTemplateParams);
 
-                    if (iJamSearchPos === 0) {
-                        sURL = sURL.replace("/JAMSEARCHPATHH?JAMSEARCHVALUEE=VALUEE", `/${JamInnerRoute}`);
-                    }
+        let sURL = URLTemplateProcessor.expand(
+            oTemplatePayload,
+            oTemplateContext.site,
+            oRuntime,
+            oTemplateContext.siteAppSection,
+            "startupParameter",
+            sFullHash
+        );
 
-                    if (oRuntime.env.theme === undefined) {
-                        sURL = sURL.replace("&sap-theme=", "");
-                    }
+        if (iJamSearchPos === 0) {
+            sURL = sURL.replace("/JAMSEARCHPATHH?JAMSEARCHVALUEE=VALUEE", `/${JamInnerRoute}`);
+        }
 
-                    // temporary bug fix until URITemplate.js will be fixed.
-                    // for ui5 apps, the hash added to the URL in the template processing is encoded twice
-                    // and there for it does not match the hash of FLP (browser url). we need to replace the
-                    // hash in the URL with the correct hash of FLP which is encoded only once.
-                    // currently, there is no way to do that in the template itself.
-                    if (oCapabilities.appFrameworkId === "UI5" && oExportedAPIs.getBrowserHash().length > 1) {
-                        sURL = sURL.split("#")[0] + oExportedAPIs.getBrowserHash();
-                        Log.debug(`- created URL with fixed hash: ${sURL}`, "sap.ushell.ApplicationType");
-                    }
+        if (oRuntime.env.theme === undefined) {
+            sURL = sURL.replace("&sap-theme=", "");
+        }
 
-                    // returns a function that instantiates an attribute, like appId of a template
-                    function fnInstantiateTemplate (sTemplate) {
-                        const oPayloadClone = deepClone(oTemplateContext.payload, 20);
-                        oPayloadClone.urlTemplate = sTemplate;
-                        return URLTemplateProcessor.expand(
-                            oPayloadClone,
-                            oTemplateContext.site,
-                            oRuntime,
-                            oTemplateContext.siteAppSection,
-                            "startupParameter"
-                        );
-                    }
-                    const sSytemAliasOrDestination = (oTemplateContext.siteAppSection["sap.app"] &&
-                        oTemplateContext.siteAppSection["sap.app"].destination) ||
-                    oTemplateContext.siteAppSection.destination || "";
-                    const sContentProviderId = oTemplateContext?.siteAppSection["sap.app"]?.contentProviderId || "";
-                    const sSystemAlias = SystemAlias.getSystemAliasInProvider(sSytemAliasOrDestination, sContentProviderId);
-
-                    const oResult = {
-                        applicationType: "URL",
-                        text: oInbound.title,
-                        appCapabilities: createAppCapabilities(
-                            oCapabilities,
-                            oTemplateContext.siteAppSection,
-                            fnInstantiateTemplate
-                        ),
-                        url: sURL,
-                        extendedInfo: oApplicationTypeUtils.getExtendedInfo(oMatchingTarget, oTemplateContext.siteAppSection, oTemplateContext.site),
-                        contentProviderId: oInbound.contentProviderId || "",
-                        systemAlias: sSystemAlias
-                    };
-                    oApplicationTypeUtils.addIframeCacheHintToURL(oResult, oResult.appCapabilities.appFrameworkId);
-                    oApplicationTypeUtils.addKeepAliveToURLTemplateResult(oResult);
-                    addSpacesModeToURLTemplateResult(oResult);
-                    _addLanguageToURLTemplateResult(oResult, oTemplateContext.siteAppSection, oRuntime);
-
-                    const oPromisePostTemplateProcessing = new Promise((fnResolve) => {
-                        Container.getServiceAsync("URLTemplate").then((URLTemplate) => {
-                            sap.ui.require(["sap/ushell/appIntegration/ApplicationContainerCache"], (ApplicationContainerCache) => {
-                                // todo: [FLPCOREANDUX-10024] should this really be necessary?
-                                const bForNewIframe = !ApplicationContainerCache.findFreeContainerByUrl(oResult.url);
-                                URLTemplate.handlePostTemplateProcessing(oResult.url, oTemplateContext.siteAppSection, bForNewIframe).then(fnResolve);
-                            });
-                        });
-                    });
-
-                    oPromisePostTemplateProcessing.then((sURLNew) => {
-                        // special case for selenium tests that use different iframe domain
-                        if (sIframeURLDomainForTests && sURLNew.indexOf("ui5appruntime.html") > 0) {
-                            const arrUrlParts = sURLNew.split("?");
-                            arrUrlParts[0] = sIframeURLDomainForTests;
-                            sURLNew = arrUrlParts.join("?");
-                        }
-                        oResult.url = sURLNew;
-                        handleURLTransformation(URLTemplateProcessor, oResult.url, oCapabilities, oTemplateContext).then((sTransformedURL) => {
-                            oResult.url = sTransformedURL;
-                            // GUI url should not be compacted as it is not compacted in ABAP FLP
-                            if (oResult.url && typeof oResult.url === "string" && oResult.url.indexOf("sap-iframe-hint=GUI") > 0) {
-                                fnResolveGlobal(oResult);
-                            } else {
-                                compactURLParameters(oResult.url, oRuntime.targetNavMode, oCapabilities)
-                                    .then((sCompactURL) => {
-                                        oResult.url = sCompactURL;
-                                        fnResolveGlobal(oResult);
-                                    }, () => {
-                                        fnResolveGlobal(oResult);
-                                    });
-                            }
-                        });
-                    });
-                });
-            });
-        });
-    }
+        // temporary bug fix until URITemplate.js will be fixed.
+        // for ui5 apps, the hash added to the URL in the template processing is encoded twice
+        // and there for it does not match the hash of FLP (browser url). we need to replace the
+        // hash in the URL with the correct hash of FLP which is encoded only once.
+        // currently, there is no way to do that in the template itself.
+        if (oCapabilities.appFrameworkId === "UI5" && sFullHash.length > 1) {
+            sURL = sURL.split("#")[0] + sFullHash;
+            Log.debug(`- created URL with fixed hash: ${sURL}`, "sap.ushell.ApplicationType");
+        }
+
+        // returns a function that instantiates an attribute, like appId of a template
+        function fnInstantiateTemplate (sTemplate) {
+            const oPayloadClone = deepClone(oTemplateContext.payload, 20);
+            oPayloadClone.urlTemplate = sTemplate;
+            return URLTemplateProcessor.expand(
+                oPayloadClone,
+                oTemplateContext.site,
+                oRuntime,
+                oTemplateContext.siteAppSection,
+                "startupParameter",
+                sFullHash
+            );
+        }
+        const sSystemAliasOrDestination = (
+            oTemplateContext.siteAppSection["sap.app"]
+            && oTemplateContext.siteAppSection["sap.app"].destination
+        ) || oTemplateContext.siteAppSection.destination || "";
+        const sContentProviderId = oTemplateContext?.siteAppSection["sap.app"]?.contentProviderId || "";
+        const sSystemAlias = SystemAlias.getSystemAliasInProvider(sSystemAliasOrDestination, sContentProviderId);
+
+        const oResult = {
+            applicationType: "URL",
+            text: oInbound.title,
+            appCapabilities: createAppCapabilities(
+                oCapabilities,
+                oTemplateContext.siteAppSection,
+                fnInstantiateTemplate
+            ),
+            url: sURL,
+            extendedInfo: oApplicationTypeUtils.getExtendedInfo(oMatchingTarget, oTemplateContext.siteAppSection, oTemplateContext.site),
+            contentProviderId: oInbound.contentProviderId || "",
+            systemAlias: sSystemAlias,
+            replaceHashBeforeOpenAppInplace: sReplaceHashBeforeOpenAppInplace
+        };
+        oApplicationTypeUtils.addIframeCacheHintToURL(oResult, oResult.appCapabilities.appFrameworkId);
+        oApplicationTypeUtils.addKeepAliveToURLTemplateResult(oResult);
+        addSpacesModeToURLTemplateResult(oResult);
+        _addLanguageToURLTemplateResult(oResult, oTemplateContext.siteAppSection, oRuntime);
+
+        const URLTemplate = await Container.getServiceAsync("URLTemplate");
+
+        // CF Platform skips auth processing when the container is already available
+        const bForNewIframe = !ApplicationContainerCache.findFreeContainerByUrl(oResult.url);
+        oResult.url = await URLTemplate.handlePostTemplateProcessing(oResult.url, oTemplateContext.siteAppSection, bForNewIframe);
+
+        const sTransformedURL = await handleURLTransformation(oResult.url, oCapabilities, oTemplateContext);
+        oResult.url = sTransformedURL;
+
+        // GUI url should not be compacted as it is not compacted in ABAP FLP
+        if (oResult.url && typeof oResult.url === "string" && oResult.url.indexOf("sap-iframe-hint=GUI") > 0) {
+            return oResult;
+        }
+
+        try {
+            const sCompactURL = await compactURLParameters(oResult.url, oRuntime.targetNavMode, oCapabilities);
+            oResult.url = sCompactURL;
+        } catch { /* fail silently */ }
 
-    function getBrowserHash () {
-        return window.location.hash;
+        return oResult;
     }
 
-    oExportedAPIs = {
+    const oExportedAPIs = {
         generateURLTemplateResolutionResult: generateURLTemplateResolutionResult,
         handleURLTransformation: handleURLTransformation,
 
         // for testing
-        getBrowserHash: getBrowserHash,
         _createEnv: createEnv,
         _addLanguageToURLTemplateResult: _addLanguageToURLTemplateResult,
         compactURLParameters: compactURLParameters