woocommerce/woocommerce-paypal-payments
2
0
Fork 0
mirror of https://github.com/woocommerce/woocommerce-paypal-payments.git synced 2025-09-06 13:44:42 +08:00
Code Issues Projects Packages Wiki Activity Actions

Throw exception when capture payment returns false

Browse source
This commit is contained in:
dinamiko 2022-03-08 12:00:09 +01:00
parent 54f84aac26
commit 0291f34309
1 changed files with 6 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

10
modules/ppcp-vaulting/src/PaymentTokenChecker.php
View file

@ -116,8 +116,8 @@ class PaymentTokenChecker {
if ( $tokens ) { if ( $tokens ) {
try { try {
$this->capture_authorized_payment( $wc_order ); $this->capture_authorized_payment( $wc_order );
} catch ( NotFoundException $exception ) { } catch ( Exception $exception ) {
$this->logger->warning( "It was not possible to capture the payment for order: #{$order_id}" ); $this->logger->error( $exception->getMessage() );
} }
return; return;
@ -139,11 +139,13 @@ class PaymentTokenChecker {
* Captures authorized payments for the given WC order. * Captures authorized payments for the given WC order.
* *
* @param WC_Order $wc_order The WC order. * @param WC_Order $wc_order The WC order.
* @throws NotFoundException When there is a problem capturing the payment. * @throws Exception When there is a problem capturing the payment.
*/ */
private function capture_authorized_payment( WC_Order $wc_order ): void { private function capture_authorized_payment( WC_Order $wc_order ): void {
if ( $this->settings->has( 'intent' ) && strtoupper( (string) $this->settings->get( 'intent' ) ) === 'CAPTURE' ) { if ( $this->settings->has( 'intent' ) && strtoupper( (string) $this->settings->get( 'intent' ) ) === 'CAPTURE' ) {
$this->authorized_payments_processor->capture_authorized_payment( $wc_order ); if ( ! $this->authorized_payments_processor->capture_authorized_payment( $wc_order ) ) {
throw new Exception( "Could not capture payment for order: #{$wc_order->get_id()}" );
}
} }
} }