@sapui5/sap.fe.core 1.131.0 → 1.133.0

package/src/sap/fe/core/controllerextensions/editFlow/TransactionHelper.ts

@@ -51,6 +51,7 @@ import type ODataListBinding from "sap/ui/model/odata/v4/ODataListBinding";
 import type { ODataListBinding$CreateCompletedEvent } from "sap/ui/model/odata/v4/ODataListBinding";
 import type ODataMetaModel from "sap/ui/model/odata/v4/ODataMetaModel";
 import type ODataModel from "sap/ui/model/odata/v4/ODataModel";
+import type { OperationResult } from "./ODataOperation";
 
 const CreationMode = FELibrary.CreationMode;
 const ProgrammingModel = FELibrary.ProgrammingModel;
@@ -441,7 +442,18 @@ class TransactionHelper {
 			: undefined;
 
 		if (newAction) {
-			return this.performCreationWithAction(newAction, mainListBinding, parameters, appComponent, resourceModel, messageHandler);
+			const result = await this.performCreationWithAction(
+				newAction,
+				mainListBinding,
+				parameters,
+				appComponent,
+				resourceModel,
+				messageHandler
+			);
+			if (result.status === "rejected") {
+				throw result.reason;
+			}
+			return result.value.returnedContext;
 		} else {
 			return this.performCreationWithPOST(mainListBinding, parameters, appComponent, resourceModel, messageHandler, fromCopyPaste);
 		}
@@ -464,7 +476,7 @@ class TransactionHelper {
 		appComponent: AppComponent,
 		resourceModel: ResourceModel,
 		messageHandler: MessageHandler
-	): Promise<ODataV4Context | undefined> {
+	): Promise<PromiseSettledResult<OperationResult>> {
 		const headerContextPath = mainListBinding.getHeaderContext()!.getPath();
 		const metaModel = mainListBinding.getModel().getMetaModel();
 		const metaPath = metaModel.getMetaPath(headerContextPath);
@@ -477,7 +489,7 @@ class TransactionHelper {
 		const result = await this.callAction(
 			actionName,
 			{
-				contexts: mainListBinding.getModel().bindList(headerContextPath).getHeaderContext(),
+				contexts: mainListBinding.getHeaderContext(),
 				showActionParameterDialog: true,
 				label: this._getSpecificCreateActionDialogLabel(metaModel, metaPath, actionName, resourceModel),
 				bindingParameters: bindingParameters,
@@ -489,7 +501,7 @@ class TransactionHelper {
 			appComponent,
 			messageHandler
 		);
-		return result as ODataV4Context | undefined;
+		return result[0];
 	}
 
 	/**
@@ -891,7 +903,6 @@ class TransactionHelper {
 		if (!oContext) {
 			throw new Error("No context exists. Pass a meaningful context");
 		}
-		this.busyLock(appComponent);
 		const mParameters = getParameters(mInParameters);
 		const oModel = oContext.getModel();
 		const sProgrammingModel = this.getProgrammingModel(oContext);
@@ -899,6 +910,7 @@ class TransactionHelper {
 		if (sProgrammingModel !== ProgrammingModel.Draft && sProgrammingModel !== ProgrammingModel.Sticky) {
 			throw new Error("Cancel document only allowed for draft or sticky session supported services");
 		}
+		this.busyLock(appComponent);
 		try {
 			let returnedValue: ODataV4Context | undefined;
 
@@ -1039,7 +1051,6 @@ class TransactionHelper {
 	 * @param [mParameters.model] Mandatory for an unbound action: An instance of an OData V4 model
 	 * @param [mParameters.invocationGrouping] Mode how actions are to be called: 'ChangeSet' to put all action calls into one changeset, 'Isolated' to put them into separate changesets
 	 * @param [mParameters.label] A human-readable label for the action
-	 * @param [mParameters.bGetBoundContext] If specified, the action promise returns the bound context
 	 * @param mParameters.bindingParameters
 	 * @param mParameters.showActionParameterDialog
 	 * @param mParameters.entitySetName
@@ -1067,7 +1078,6 @@ class TransactionHelper {
 			skipParameterDialog?: boolean;
 			showActionParameterDialog?: boolean;
 			label?: string;
-			bGetBoundContext?: boolean;
 			invocationGrouping?: string;
 			entitySetName?: string;
 			contexts?: ODataV4Context | ODataV4Context[] | null;
@@ -1086,7 +1096,7 @@ class TransactionHelper {
 		appComponent: AppComponent,
 		messageHandler: MessageHandler,
 		disableStrictHandling?: boolean
-	): Promise<undefined | ODataV4Context | PromiseSettledResult<ODataV4Context | undefined>[]> {
+	): Promise<PromiseSettledResult<OperationResult>[]> {
 		mParameters = getParameters(mParameters);
 		let contextToProcess;
 		let oModel: ODataModel | undefined;
@@ -1120,8 +1130,8 @@ class TransactionHelper {
 			) || {};
 		const messageHandlingKey = messageHandler ? messageHandler.registerToHoldMessages() : undefined;
 		try {
-			let oResult: undefined | ODataV4Context | PromiseSettledResult<undefined | ODataV4Context>[];
-			if (contextToProcess) {
+			let oResult: PromiseSettledResult<OperationResult>[];
+			if (contextToProcess && mParameters.contexts) {
 				oResult = await operations.callBoundAction(sActionName, mParameters.contexts as ODataV4Context[], oModel, appComponent, {
 					parameterValues: mParameters.parameterValues as Record<string, string | boolean | number | MultiValueFieldItem[]>[],
 					invocationGrouping: mParameters.invocationGrouping,
@@ -1142,11 +1152,10 @@ class TransactionHelper {
 					internalModelContext: mParameters.internalModelContext,
 					operationAvailableMap: mParameters.operationAvailableMap,
 					bIsCreateAction: mParameters.bIsCreateAction,
-					bGetBoundContext: mParameters.bGetBoundContext,
 					bObjectPage: mParameters.bObjectPage,
 					messageHandler: messageHandler,
 					defaultValuesExtensionFunction: mParameters.defaultValuesExtensionFunction,
-					selectedItems: mParameters.contexts as ODataV4Context[],
+					selectedItems: Array.isArray(mParameters.contexts) ? mParameters.contexts : [mParameters.contexts],
 					disableStrictHandling,
 					groupId: mParameters.groupId,
 					ignoreETag: mParameters.ignoreETag,
@@ -1203,7 +1212,13 @@ class TransactionHelper {
 	 */
 	async _handleActionResponse(
 		messageHandler: MessageHandler,
-		mParameters: { label?: string; internalModelContext?: InternalModelContext | null; controlId?: string; parentControl?: Control },
+		mParameters: {
+			label?: string;
+			internalModelContext?: InternalModelContext | null;
+			controlId?: string;
+			parentControl?: Control;
+			contexts?: Context | Context[] | null;
+		},
 		sActionName: string,
 		messageHandlingKey?: string
 	): Promise<void | boolean | Dialog> {
@@ -1218,7 +1233,12 @@ class TransactionHelper {
 		} else {
 			control = mParameters.parentControl;
 		}
-		return messageHandler.showMessages({ sActionName: actionName, control: control, unHoldKey: messageHandlingKey });
+		return messageHandler.showMessages({
+			sActionName: actionName,
+			control: control,
+			unHoldKey: messageHandlingKey,
+			aSelectedContexts: mParameters?.contexts
+		});
 	}
 
 	/**