fix: always include payment ID in card payment redirect URL

jimmyn · jimmyn · commit 8d3f0620a373 · 2025-10-03T15:26:50.000+02:00
- Always add payment ID, order ID, and status to redirect URL after confirmPayment()
- Ensures verify_and_complete_order() can fetch payment and complete order
- Fixes race condition where order shows as unpaid for embedded card payments
- Matches PrestaShop's unified flow approach

Previously only included payment ID when saving payment method, causing
orders to show as "not paid" when user didn't check "save payment method"
diff --git a/assets/js/components/monei-cc-component.js b/assets/js/components/monei-cc-component.js
@@ -218,17 +218,14 @@ export const MoneiCCContent = ( props ) => {
 						failUrl.searchParams.set( 'status', 'FAILED' );
 						window.location.href = failUrl.toString();
 					} else {
-						let redirectUrl = paymentDetails.completeUrl;
-
-						if ( shouldSavePayment === true ) {
-							const { orderId, paymentId } = paymentDetails;
-							const url = new URL( paymentDetails.completeUrl );
-							url.searchParams.set( 'id', paymentId );
-							url.searchParams.set( 'orderId', orderId );
-							redirectUrl = url.toString();
-						}
-
-						window.location.href = redirectUrl;
+						// Always include payment ID in redirect URL for order verification
+						const { orderId, paymentId } = paymentDetails;
+						const url = new URL( paymentDetails.completeUrl );
+						url.searchParams.set( 'id', paymentId );
+						url.searchParams.set( 'orderId', orderId );
+						url.searchParams.set( 'status', result.status );
+
+						window.location.href = url.toString();
 					}
 				} catch ( error ) {
 					console.error(
