@sapui5/sap.fe.core 1.102.0 → 1.103.0

package/src/sap/fe/core/controllerextensions/InternalEditFlow.ts

@@ -1,5 +1,5 @@
 import Log from "sap/base/Log";
-import AppComponent from "sap/fe/core/AppComponent";
+import type AppComponent from "sap/fe/core/AppComponent";
 import CommonUtils from "sap/fe/core/CommonUtils";
 import BusyLocker from "sap/fe/core/controllerextensions/BusyLocker";
 import { send } from "sap/fe/core/controllerextensions/collaboration/ActivitySync";
@@ -16,18 +16,18 @@ import {
 } from "sap/fe/core/helpers/ClassSupport";
 import EditState from "sap/fe/core/helpers/EditState";
 import FELibrary from "sap/fe/core/library";
-import PageController from "sap/fe/core/PageController";
+import type PageController from "sap/fe/core/PageController";
 import Button from "sap/m/Button";
 import Dialog from "sap/m/Dialog";
 import Text from "sap/m/Text";
 import ControllerExtension from "sap/ui/core/mvc/ControllerExtension";
 import OverrideExecution from "sap/ui/core/mvc/OverrideExecution";
-import View from "sap/ui/core/mvc/View";
-import Table from "sap/ui/mdc/Table";
-import JSONModel from "sap/ui/model/json/JSONModel";
-import Model from "sap/ui/model/Model";
-import Context from "sap/ui/model/odata/v4/Context";
-import ODataListBinding from "sap/ui/model/odata/v4/ODataListBinding";
+import type View from "sap/ui/core/mvc/View";
+import type Table from "sap/ui/mdc/Table";
+import type JSONModel from "sap/ui/model/json/JSONModel";
+import type Model from "sap/ui/model/Model";
+import type Context from "sap/ui/model/odata/v4/Context";
+import type ODataListBinding from "sap/ui/model/odata/v4/ODataListBinding";
 
 const ProgrammingModel = FELibrary.ProgrammingModel,
 	DraftStatus = FELibrary.DraftStatus,
@@ -78,12 +78,12 @@ class InternalEditFlow extends ControllerExtension {
 			oResourceBundle = (this.getView().getController() as any).oResourceBundle;
 
 		BusyLocker.lock(oLockObject);
+		let aFinalContexts: any[] = [];
 		return this.syncTask()
 			.then(function () {
-				const onBeforeCreatePromise = beforeCreateCallBack
+				return beforeCreateCallBack
 					? beforeCreateCallBack({ contextPath: oListBinding && oListBinding.getPath() })
 					: Promise.resolve();
-				return onBeforeCreatePromise;
 			})
 			.then(() => {
 				const oModel = oListBinding.getModel(),
@@ -91,7 +91,7 @@ class InternalEditFlow extends ControllerExtension {
 				let sMetaPath: string;
 
 				if (oListBinding.getContext()) {
-					sMetaPath = oMetaModel.getMetaPath(oListBinding.getContext().getPath() + "/" + oListBinding.getPath());
+					sMetaPath = oMetaModel.getMetaPath(`${oListBinding.getContext().getPath()}/${oListBinding.getPath()}`);
 				} else {
 					sMetaPath = oMetaModel.getMetaPath(oListBinding.getPath());
 				}
@@ -111,7 +111,7 @@ class InternalEditFlow extends ControllerExtension {
 
 					// Remove navigation properties as we don't support deep create
 					for (const sPropertyPath in mPropertyValues) {
-						const oProperty = oMetaModel.getObject(sMetaPath + "/" + sPropertyPath);
+						const oProperty = oMetaModel.getObject(`${sMetaPath}/${sPropertyPath}`);
 						if (oProperty && oProperty.$kind !== "NavigationProperty" && mPropertyValues[sPropertyPath]) {
 							mParameters.data[sPropertyPath] = mPropertyValues[sPropertyPath];
 						}
@@ -131,6 +131,7 @@ class InternalEditFlow extends ControllerExtension {
 			})
 			.then(function (aContexts: any) {
 				// transient contexts are reliably removed once oNewContext.created() is resolved
+				aFinalContexts = aContexts;
 				return Promise.all(
 					aContexts.map(function (oNewContext: any) {
 						if (!oNewContext.bInactive) {
@@ -157,6 +158,9 @@ class InternalEditFlow extends ControllerExtension {
 			})
 			.finally(function () {
 				BusyLocker.unlock(oLockObject);
+			})
+			.then(() => {
+				return aFinalContexts;
 			});
 	}
 	@publicExtension()
@@ -284,8 +288,8 @@ class InternalEditFlow extends ControllerExtension {
 	/**
 	 * Sets the edit mode.
 	 *
-	 * @param {string} sEditMode
-	 * @param {boolean} bCreationMode CreateMode flag to identify the creation mode
+	 * @param sEditMode
+	 * @param bCreationMode CreateMode flag to identify the creation mode
 	 */
 	@publicExtension()
 	@finalExtension()
@@ -342,9 +346,8 @@ class InternalEditFlow extends ControllerExtension {
 	 * @name sap.fe.core.controllerextensions.EditFlow#syncTask
 	 * @memberof sap.fe.core.controllerextensions.EditFlow
 	 * @static
-	 * @param {Promise|Function} [vTask] Optional, a promise or function to be executed synchronously
-	 * @returns {Promise} Promise resolves once the task is completed
-	 *
+	 * @param [vTask] Optional, a promise or function to be executed synchronously
+	 * @returns Promise resolves once the task is completed
 	 * @ui5-restricted
 	 * @final
 	 */
@@ -361,7 +364,7 @@ class InternalEditFlow extends ControllerExtension {
 		}
 
 		this._pTasks = this._pTasks || Promise.resolve();
-		if (!!fnNewTask) {
+		if (fnNewTask) {
 			this._pTasks = this._pTasks.then(fnNewTask).catch(function () {
 				return Promise.resolve();
 			});
@@ -408,7 +411,7 @@ class InternalEditFlow extends ControllerExtension {
 	 * Handles the create event: shows messages and in case of a draft, updates the draft indicator.
 	 *
 	 * @memberof sap.fe.core.controllerextensions.EditFlow
-	 * @param {object} oBinding OData list binding object
+	 * @param oBinding OData list binding object
 	 */
 	@publicExtension()
 	@finalExtension()
@@ -454,10 +457,10 @@ class InternalEditFlow extends ControllerExtension {
 
 	/**
 	 * Creates a new promise to wait for an action to be executed
+	 *
 	 * @function
 	 * @name _createActionPromise
 	 * @memberof sap.fe.core.controllerextensions.EditFlow
-	 *
 	 * @returns {Function} The resolver function which can be used to externally resolve the promise
 	 */
 
@@ -480,8 +483,7 @@ class InternalEditFlow extends ControllerExtension {
 	 * @function
 	 * @name _getCurrentActionPromise
 	 * @memberof sap.fe.core.controllerextensions.EditFlow
-	 *
-	 * @returns {Promise} Returns the promise
+	 * @returns Returns the promise
 	 */
 	@publicExtension()
 	@finalExtension()
@@ -499,9 +501,9 @@ class InternalEditFlow extends ControllerExtension {
 	 * @function
 	 * @name getActionResponseDataAndKeys
 	 * @memberof sap.fe.core.controllerextensions.EditFlow
-	 * @param {string} sActionName The name of the action that is executed
-	 * @param {object} oResponse The bound action's response data or response context
-	 * @returns {object} Object with data and names of the key fields of the response
+	 * @param sActionName The name of the action that is executed
+	 * @param oResponse The bound action's response data or response context
+	 * @returns Object with data and names of the key fields of the response
 	 */
 	@publicExtension()
 	@finalExtension()
@@ -629,7 +631,6 @@ class InternalEditFlow extends ControllerExtension {
 
 	/**
 	 * @description Method to display a 'discard' popover when exiting a sticky session.
-	 *
 	 * @function
 	 * @name onBackNavigationInSession
 	 * @memberof sap.fe.core.controllerextensions.InternalEditFlow
@@ -673,7 +674,7 @@ class InternalEditFlow extends ControllerExtension {
 
 			const oMetaModel = oView.getModel().getMetaModel(),
 				sMetaPath = oContext.getPath().substring(0, oContext.getPath().indexOf("(")),
-				oStickySession = oMetaModel.getObject(sMetaPath + "@com.sap.vocabularies.Session.v1.StickySessionSupported");
+				oStickySession = oMetaModel.getObject(`${sMetaPath}@com.sap.vocabularies.Session.v1.StickySessionSupported`);
 
 			if (oStickySession && oStickySession.NewAction && oStickySession.NewAction === sCustomAction) {
 				return true;
@@ -749,15 +750,13 @@ class InternalEditFlow extends ControllerExtension {
 	}
 
 	_attachSessionTimeout(oContext: any, i18nModel: Model) {
-		const that = this;
-
-		return function fnHandleSessionTimeout() {
+		return () => {
 			try {
 				if (oContext === undefined) {
 					throw new Error("Context missing for function fnHandleSessionTimeout");
 				}
 				// remove transient messages since we will showing our own message
-				that.getMessageHandler().removeTransitionMessages();
+				this.getMessageHandler().removeTransitionMessages();
 
 				const oDialog = new Dialog({
 					title: "{sap.fe.i18n>C_EDITFLOW_OBJECT_PAGE_SESSION_EXPIRED_DIALOG_TITLE}",
@@ -766,10 +765,10 @@ class InternalEditFlow extends ControllerExtension {
 					beginButton: new Button({
 						text: "{sap.fe.i18n>C_COMMON_DIALOG_OK}",
 						type: "Emphasized",
-						press: function () {
+						press: () => {
 							// remove sticky handling after navigation since session has already been terminated
-							that.handleStickyOff();
-							that.getRoutingListener().navigateBackFromContext(oContext);
+							this.handleStickyOff();
+							this.getRoutingListener().navigateBackFromContext(oContext);
 						}
 					}),
 					afterClose: function () {
@@ -778,7 +777,7 @@ class InternalEditFlow extends ControllerExtension {
 				});
 				oDialog.addStyleClass("sapUiContentPadding");
 				oDialog.setModel(i18nModel, "sap.fe.i18n");
-				that.getView().addDependent(oDialog);
+				this.getView().addDependent(oDialog);
 				oDialog.open();
 			} catch (error) {
 				Log.info(error as any);