Avoid resending Processing Order customer email when merchant wins dispute (#3674)

* Skip 'Processing Order' customer email on merchant dispute win

* Add changelog and readme entries

---------

Co-authored-by: Wesley Rosa <[email protected]>

Author: annemirasol

changelog.txt

@@ -1,6 +1,7 @@
 *** Changelog ***
 
 = 9.1.0 - xxxx-xx-xx =
+* Tweak - Avoid re-sending Processing Order customer email when merchant wins dispute.
 * Fix - Allow the saving of iDEAL tokens when SEPA is disabled.
 * Fix - Fixes the incompatibility notice in editor due missing style property when instantiating Stripe payment methods. 
 * Dev - Updates the GitHub caching action (`actions/cache`) to v4 due deprecation.

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

@@ -400,6 +400,19 @@ public function process_webhook_dispute_closed( $notification ) {
 
 			// Fail order if dispute is lost, or else revert to pre-dispute status.
 			$order_status = 'lost' === $status ? 'failed' : $this->get_stripe_order_status_before_hold( $order );
+
+			// Do not re-send "Processing Order" email to customer after a dispute win.
+			if ( 'processing' === $order_status ) {
+				$emails = WC()->mailer()->get_emails();
+				if ( isset( $emails['WC_Email_Customer_Processing_Order'] ) ) {
+					$callback = [ $emails['WC_Email_Customer_Processing_Order'], 'trigger' ];
+					remove_action(
+						'woocommerce_order_status_on-hold_to_processing_notification',
+						$callback
+					);
+				}
+			}
+
 			$order->update_status( $order_status, $message );
 		} else {
 			$order->add_order_note( $message );

readme.txt

@@ -111,6 +111,7 @@ If you get stuck, you can ask for help in the [Plugin Forum](https://wordpress.o
 == Changelog ==
 
 = 9.1.0 - xxxx-xx-xx =
+* Tweak - Avoid re-sending Processing Order customer email when merchant wins dispute.
 * Fix - Allow the saving of iDEAL tokens when SEPA is disabled.
 * Fix - Fixes the incompatibility notice in editor due missing style property when instantiating Stripe payment methods. 
 * Dev - Updates the GitHub caching action (`actions/cache`) to v4 due deprecation.