refactor

Author: swangi-kumari

crates/api_models/src/payments.rs

@@ -7637,6 +7637,7 @@ pub struct PaymentsCancelRequest {
     pub merchant_connector_details: Option<admin::MerchantConnectorDetailsWrap>,
 }
 
+/// Request to cancel a payment when the payment is already captured
 #[derive(Default, Debug, serde::Deserialize, serde::Serialize, Clone, ToSchema)]
 pub struct PaymentsCancelPostCaptureRequest {
     /// The identifier for the payment

crates/hyperswitch_connectors/src/connectors/worldpayvantiv/transformers.rs

@@ -79,7 +79,7 @@ pub enum OperationId {
     Auth,
     Capture,
     Void,
-    // VoidPC
+    // VoidPostCapture
     VoidPC,
     Refund,
 }
@@ -2613,7 +2613,7 @@ pub enum WorldpayvantivPaymentFlow {
     Auth,
     Capture,
     Void,
-    //VoidPC
+    //VoidPostCapture
     VoidPC,
 }
 

crates/openapi/src/routes/payments.rs

@@ -814,7 +814,7 @@ pub fn payments_cancel() {}
 
 /// Payments - Cancel Post Capture
 ///
-/// A Payment could can be cancelled when it is in one of these statuses: `requires_payment_method`, `requires_capture`, `requires_confirmation`, `requires_customer_action`.
+/// A Payment could can be cancelled when it is in one of these statuses: `succeeded`, `partially_captured`, `partially_captured_and_capturable`.
 #[utoipa::path(
     post,
     path = "/payments/{payment_id}/cancel_post_capture",
@@ -837,7 +837,7 @@ pub fn payments_cancel() {}
         ("payment_id" = String, Path, description = "The identifier for payment")
     ),
     responses(
-        (status = 200, description = "Payment canceled"),
+        (status = 200, description = "Payment canceled post capture"),
         (status = 400, description = "Missing mandatory fields", body = GenericErrorResponseOpenApi)
     ),
     tag = "Payments",

crates/router/src/core/payments.rs

@@ -6912,7 +6912,6 @@ where
             payment_data.get_payment_intent().status,
             storage_enums::IntentStatus::RequiresCapture
         ),
-
         _ => false,
     }
 }

crates/router_derive/src/macros/operation.rs

@@ -92,7 +92,6 @@ impl Conversion {
             Derives::SyncData => syn::Ident::new("PaymentsSyncData", Span::call_site()),
             Derives::Cancel => syn::Ident::new("PaymentsCancelRequest", Span::call_site()),
             Derives::CancelData => syn::Ident::new("PaymentsCancelData", Span::call_site()),
-
             Derives::ApproveData => syn::Ident::new("PaymentsApproveData", Span::call_site()),
             Derives::Reject => syn::Ident::new("PaymentsRejectRequest", Span::call_site()),
             Derives::RejectData => syn::Ident::new("PaymentsRejectData", Span::call_site()),