From 805540a1d02ec6ce61e21c66349dbb0fb3403e69 Mon Sep 17 00:00:00 2001 From: Sai Harsha Vardhan <56996463+sai-harsha-vardhan@users.noreply.github.com> Date: Wed, 14 Aug 2024 15:57:07 +0530 Subject: [PATCH] fix(router): allow payments update for requires_payment_method and requires_confirmation intent status only (#5616) --- .../router/src/core/payments/operations/payment_update.rs | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/crates/router/src/core/payments/operations/payment_update.rs b/crates/router/src/core/payments/operations/payment_update.rs index b2ad13ee0d..e7045116f2 100644 --- a/crates/router/src/core/payments/operations/payment_update.rs +++ b/crates/router/src/core/payments/operations/payment_update.rs @@ -95,13 +95,11 @@ impl GetTracker, api::PaymentsRequest> for Pa .and_then(|pmd| pmd.payment_method_data.clone()), )?; - helpers::validate_payment_status_against_not_allowed_statuses( + helpers::validate_payment_status_against_allowed_statuses( &payment_intent.status, &[ - storage_enums::IntentStatus::Failed, - storage_enums::IntentStatus::Succeeded, - storage_enums::IntentStatus::PartiallyCaptured, - storage_enums::IntentStatus::RequiresCapture, + storage_enums::IntentStatus::RequiresPaymentMethod, + storage_enums::IntentStatus::RequiresConfirmation, ], "update", )?;