Track charge completed via webhooks in order note (#4322)

* track charge completed via webhooks in order note

* add changelog

* piggyback webhook flag with response data instead of adding a new function paaram

* fix mocks

Author: Mayisha

changelog.txt

@@ -1,5 +1,8 @@
 *** Changelog ***
 
+= 9.5.3 - xxxx-xx-xx =
+* Tweak - Track charge completed via webhooks in order notes
+
 = 9.5.2 - 2025-05-22 =
 * Add - Implement custom database cache for persistent caching with in-memory optimization.
 * Update - Remove feature that flags 401s and proactively blocks subsequent API calls until the store has reauthenticated.

includes/abstracts/abstract-wc-stripe-payment-gateway.php

@@ -611,6 +611,9 @@ public function process_response( $response, $order ) {
 
 					/* translators: transaction id */
 					$message = sprintf( __( 'Stripe charge complete (Charge ID: %s)', 'woocommerce-gateway-stripe' ), $response->id );
+					if ( isset( $response->is_webhook_response ) ) {
+						$message .= ' (via webhook)';
+					}
 					$order->add_order_note( $message );
 				}
 			}

includes/class-wc-stripe-webhook-handler.php

@@ -357,6 +357,7 @@ public function process_webhook_payment( $notification, $retry = true ) {
 
 			do_action( 'wc_gateway_stripe_process_webhook_payment', $response, $order );
 
+			$response->is_webhook_response = true;
 			$this->process_response( $response, $order );
 
 		} catch ( WC_Stripe_Exception $e ) {
@@ -509,7 +510,7 @@ public function process_webhook_capture( $notification ) {
 					$order->payment_complete( $notification->data->object->id );
 
 					/* translators: transaction id */
-					$order->add_order_note( sprintf( __( 'Stripe charge complete (Charge ID: %s)', 'woocommerce-gateway-stripe' ), $notification->data->object->id ) );
+					$order->add_order_note( sprintf( __( 'Stripe charge complete (Charge ID: %s) (via webhook)', 'woocommerce-gateway-stripe' ), $notification->data->object->id ) );
 				}
 
 				if ( is_callable( [ $order, 'save' ] ) ) {
@@ -589,7 +590,7 @@ public function process_webhook_charge_succeeded( $notification ) {
 			$order->payment_complete( $charge->id );
 
 			/* translators: transaction id */
-			$order->add_order_note( sprintf( __( 'Stripe charge complete (Charge ID: %s)', 'woocommerce-gateway-stripe' ), $charge->id ) );
+			$order->add_order_note( sprintf( __( 'Stripe charge complete (Charge ID: %s) (via webhook)', 'woocommerce-gateway-stripe' ), $charge->id ) );
 		}
 
 		if ( is_callable( [ $order, 'save' ] ) ) {
@@ -1048,6 +1049,7 @@ public function process_payment_intent( $notification ) {
 
 					do_action( 'wc_gateway_stripe_process_payment', $charge, $order );
 
+					$charge->is_webhook_response = true;
 					$this->process_response( $charge, $order );
 				} else {
 					WC_Stripe_Logger::log( "Processing $notification->type ($intent->id) asynchronously for order $order_id." );
@@ -1235,6 +1237,7 @@ protected function handle_deferred_payment_intent_succeeded( $order, $intent_id
 		WC_Stripe_Logger::log( "Processing Stripe PaymentIntent {$intent_id} for order {$order->get_id()} via deferred webhook." );
 
 		do_action( 'wc_gateway_stripe_process_payment', $charge, $order );
+		$charge->is_webhook_response = true;
 		$this->process_response( $charge, $order );
 	}
 

readme.txt

@@ -110,11 +110,8 @@ If you get stuck, you can ask for help in the [Plugin Forum](https://wordpress.o
 
 == Changelog ==
 
-= 9.5.2 - 2025-05-22 =
-* Add - Implement custom database cache for persistent caching with in-memory optimization.
-* Update - Remove feature that flags 401s and proactively blocks subsequent API calls until the store has reauthenticated.
-* Fix - Disable payment settings sync when we receive unsupported payment method configurations.
-* Fix - Ensure that we use current Stripe API keys after settings updates
-* Fix - Fix initial enabled payment methods migration to the Stripe Payment Methods Configuration API
+= 9.5.3 - xxxx-xx-xx =
+
+* Tweak - Track charge completed via webhooks in order notes
 
 [See changelog for full details across versions](https://raw.githubusercontent.com/woocommerce/woocommerce-gateway-stripe/trunk/changelog.txt).

tests/phpunit/test-wc-stripe-webhook-handler.php

@@ -185,13 +185,17 @@ public function test_process_of_successful_payment_intent_deferred_webhook() {
 		// Expect the get latest charge from intent to be called.
 		$this->mock_webhook_handler->expects( $this->once() )
 			->method( 'get_latest_charge_from_intent' )
-			->willReturn( self::MOCK_PAYMENT_INTENT['charges']['data'][0] );
+			->willReturn( (object) self::MOCK_PAYMENT_INTENT['charges']['data'][0] );
 
 		// Expect the process response to be called with the charge and order.
+		$charge_param = (object) array_merge(
+			self::MOCK_PAYMENT_INTENT['charges']['data'][0],
+			[ 'is_webhook_response' => true ]
+		);
 		$this->mock_webhook_handler->expects( $this->once() )
 			->method( 'process_response' )
 			->with(
-				self::MOCK_PAYMENT_INTENT['charges']['data'][0],
+				$charge_param,
 				$this->callback(
 					function ( $passed_order ) use ( $order ) {
 						return $passed_order instanceof WC_Order && $order->get_id() === $passed_order->get_id();
@@ -422,6 +426,9 @@ public function test_process_payment_intent(
 			[ &$mock_action_process_payment_intent_incomplete, 'action' ]
 		);
 
+		$this->mock_webhook_handler->method( 'get_latest_charge_from_intent' )
+			->willReturn( (object) self::MOCK_PAYMENT_INTENT['charges']['data'][0] );
+
 		$order = WC_Helper_Order::create_order();
 		$order->set_status( $order_status );
 		if ( $order_locked ) {