woocommerce / woocommerce-gateway-stripe

includes/class-wc-stripe-intent-controller.php

Spacing +29 added lines, -29 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,7 +23,7 @@ 
 	 * @since 4.2.0
 	 */
 	public function __construct() {
-		add_action( 'wc_ajax_wc_stripe_verify_intent', array( $this, 'verify_intent' ) );
+		add_action('wc_ajax_wc_stripe_verify_intent', array($this, 'verify_intent'));
 	}
 
 	/**
@@ -33,8 +33,8 @@ 
 	 * @return WC_Gateway_Stripe
 	 */
 	protected function get_gateway() {
-		if ( ! isset( $this->gateway ) ) {
-			if ( class_exists( 'WC_Subscriptions_Order' ) && function_exists( 'wcs_create_renewal_order' ) ) {
+		if ( ! isset($this->gateway)) {
+			if (class_exists('WC_Subscriptions_Order') && function_exists('wcs_create_renewal_order')) {
 				$class_name = 'WC_Stripe_Subs_Compat';
 			} else {
 				$class_name = 'WC_Gateway_Stripe';
@@ -54,21 +54,21 @@ 
 	 * @return WC_Order
 	 */
 	protected function get_order_from_request() {
-		if ( ! isset( $_GET['nonce'] ) || ! wp_verify_nonce( sanitize_key( $_GET['nonce'] ), 'wc_stripe_confirm_pi' ) ) {
-			throw new WC_Stripe_Exception( 'missing-nonce', __( 'CSRF verification failed.', 'woocommerce-gateway-stripe' ) );
+		if ( ! isset($_GET['nonce']) || ! wp_verify_nonce(sanitize_key($_GET['nonce']), 'wc_stripe_confirm_pi')) {
+			throw new WC_Stripe_Exception('missing-nonce', __('CSRF verification failed.', 'woocommerce-gateway-stripe'));
 		}
 
 		// Load the order ID.
 		$order_id = null;
-		if ( isset( $_GET['order'] ) && absint( $_GET['order'] ) ) {
-			$order_id = absint( $_GET['order'] );
+		if (isset($_GET['order']) && absint($_GET['order'])) {
+			$order_id = absint($_GET['order']);
 		}
 
 		// Retrieve the order.
-		$order = wc_get_order( $order_id );
+		$order = wc_get_order($order_id);
 
-		if ( ! $order ) {
-			throw new WC_Stripe_Exception( 'missing-order', __( 'Missing order ID for payment confirmation', 'woocommerce-gateway-stripe' ) );
+		if ( ! $order) {
+			throw new WC_Stripe_Exception('missing-order', __('Missing order ID for payment confirmation', 'woocommerce-gateway-stripe'));
 		}
 
 		return $order;
@@ -86,32 +86,32 @@ 
 
 		try {
 			$order = $this->get_order_from_request();
-		} catch ( WC_Stripe_Exception $e ) {
+		} catch (WC_Stripe_Exception $e) {
 			/* translators: Error message text */
-			$message = sprintf( __( 'Payment verification error: %s', 'woocommerce-gateway-stripe' ), $e->getLocalizedMessage() );
-			wc_add_notice( esc_html( $message ), 'error' );
+			$message = sprintf(__('Payment verification error: %s', 'woocommerce-gateway-stripe'), $e->getLocalizedMessage());
+			wc_add_notice(esc_html($message), 'error');
 
 			$redirect_url = $woocommerce->cart->is_empty()
-				? get_permalink( WC_Stripe_Helper::is_wc_lt( '3.0' ) ? woocommerce_get_page_id( 'shop' ) : wc_get_page_id( 'shop' ) )
+				? get_permalink(WC_Stripe_Helper::is_wc_lt('3.0') ? woocommerce_get_page_id('shop') : wc_get_page_id('shop'))
 				: wc_get_checkout_url();
 
-			$this->handle_error( $e, $redirect_url );
+			$this->handle_error($e, $redirect_url);
 		}
 
 		try {
-			$gateway->verify_intent_after_checkout( $order );
+			$gateway->verify_intent_after_checkout($order);
 
-			if ( ! isset( $_GET['is_ajax'] ) ) {
-				$redirect_url = isset( $_GET['redirect_to'] ) // wpcs: csrf ok.
-					? esc_url_raw( wp_unslash( $_GET['redirect_to'] ) ) // wpcs: csrf ok.
-					: $gateway->get_return_url( $order );
+			if ( ! isset($_GET['is_ajax'])) {
+				$redirect_url = isset($_GET['redirect_to']) // wpcs: csrf ok.
+					? esc_url_raw(wp_unslash($_GET['redirect_to'])) // wpcs: csrf ok.
+					: $gateway->get_return_url($order);
 
-				wp_safe_redirect( $redirect_url );
+				wp_safe_redirect($redirect_url);
 			}
 
 			exit;
-		} catch ( WC_Stripe_Exception $e ) {
-			$this->handle_error( $e, $gateway->get_return_url( $order ) );
+		} catch (WC_Stripe_Exception $e) {
+			$this->handle_error($e, $gateway->get_return_url($order));
 		}
 	}
 
@@ -122,17 +122,17 @@ 
 	 * @param WC_Stripe_Exception $e           The exception that was thrown.
 	 * @param string              $redirect_url An URL to use if a redirect is needed.
 	 */
-	protected function handle_error( $e, $redirect_url ) {
+	protected function handle_error($e, $redirect_url) {
 		// Log the exception before redirecting.
-		$message = sprintf( 'PaymentIntent verification exception: %s', $e->getLocalizedMessage() );
-		WC_Stripe_Logger::log( $message );
+		$message = sprintf('PaymentIntent verification exception: %s', $e->getLocalizedMessage());
+		WC_Stripe_Logger::log($message);
 
 		// `is_ajax` is only used for PI error reporting, a response is not expected.
-		if ( isset( $_GET['is_ajax'] ) ) {
+		if (isset($_GET['is_ajax'])) {
 			exit;
 		}
 
-		wp_safe_redirect( $redirect_url );
+		wp_safe_redirect($redirect_url);
 		exit;
 	}
 }

includes/class-wc-stripe-helper.php

Spacing +105 added lines, -105 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -22,14 +22,14 @@ 
 	 * @param object $order
 	 * @return string $currency
 	 */
-	public static function get_stripe_currency( $order = null ) {
-		if ( is_null( $order ) ) {
+	public static function get_stripe_currency($order = null) {
+		if (is_null($order)) {
 			return false;
 		}
 
-		$order_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
+		$order_id = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
 
-		return WC_Stripe_Helper::is_wc_lt( '3.0' ) ? get_post_meta( $order_id, self::META_NAME_STRIPE_CURRENCY, true ) : $order->get_meta( self::META_NAME_STRIPE_CURRENCY, true );
+		return WC_Stripe_Helper::is_wc_lt('3.0') ? get_post_meta($order_id, self::META_NAME_STRIPE_CURRENCY, true) : $order->get_meta(self::META_NAME_STRIPE_CURRENCY, true);
 	}
 
 	/**
@@ -39,14 +39,14 @@ 
 	 * @param object $order
 	 * @param string $currency
 	 */
-	public static function update_stripe_currency( $order = null, $currency ) {
-		if ( is_null( $order ) ) {
+	public static function update_stripe_currency($order = null, $currency) {
+		if (is_null($order)) {
 			return false;
 		}
 
-		$order_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
+		$order_id = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
 
-		WC_Stripe_Helper::is_wc_lt( '3.0' ) ? update_post_meta( $order_id, self::META_NAME_STRIPE_CURRENCY, $currency ) : $order->update_meta_data( self::META_NAME_STRIPE_CURRENCY, $currency );
+		WC_Stripe_Helper::is_wc_lt('3.0') ? update_post_meta($order_id, self::META_NAME_STRIPE_CURRENCY, $currency) : $order->update_meta_data(self::META_NAME_STRIPE_CURRENCY, $currency);
 	}
 
 	/**
@@ -56,22 +56,22 @@ 
 	 * @param object $order
 	 * @return string $amount
 	 */
-	public static function get_stripe_fee( $order = null ) {
-		if ( is_null( $order ) ) {
+	public static function get_stripe_fee($order = null) {
+		if (is_null($order)) {
 			return false;
 		}
 
-		$order_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
+		$order_id = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
 
-		$amount = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? get_post_meta( $order_id, self::META_NAME_FEE, true ) : $order->get_meta( self::META_NAME_FEE, true );
+		$amount = WC_Stripe_Helper::is_wc_lt('3.0') ? get_post_meta($order_id, self::META_NAME_FEE, true) : $order->get_meta(self::META_NAME_FEE, true);
 
 		// If not found let's check for legacy name.
-		if ( empty( $amount ) ) {
-			$amount = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? get_post_meta( $order_id, self::LEGACY_META_NAME_FEE, true ) : $order->get_meta( self::LEGACY_META_NAME_FEE, true );
+		if (empty($amount)) {
+			$amount = WC_Stripe_Helper::is_wc_lt('3.0') ? get_post_meta($order_id, self::LEGACY_META_NAME_FEE, true) : $order->get_meta(self::LEGACY_META_NAME_FEE, true);
 
 			// If found update to new name.
-			if ( $amount ) {
-				self::update_stripe_fee( $order, $amount );
+			if ($amount) {
+				self::update_stripe_fee($order, $amount);
 			}
 		}
 
@@ -85,14 +85,14 @@ 
 	 * @param object $order
 	 * @param float $amount
 	 */
-	public static function update_stripe_fee( $order = null, $amount = 0.0 ) {
-		if ( is_null( $order ) ) {
+	public static function update_stripe_fee($order = null, $amount = 0.0) {
+		if (is_null($order)) {
 			return false;
 		}
 
-		$order_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
+		$order_id = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
 
-		WC_Stripe_Helper::is_wc_lt( '3.0' ) ? update_post_meta( $order_id, self::META_NAME_FEE, $amount ) : $order->update_meta_data( self::META_NAME_FEE, $amount );
+		WC_Stripe_Helper::is_wc_lt('3.0') ? update_post_meta($order_id, self::META_NAME_FEE, $amount) : $order->update_meta_data(self::META_NAME_FEE, $amount);
 	}
 
 	/**
@@ -101,15 +101,15 @@ 
 	 * @since 4.1.0
 	 * @param object $order
 	 */
-	public static function delete_stripe_fee( $order = null ) {
-		if ( is_null( $order ) ) {
+	public static function delete_stripe_fee($order = null) {
+		if (is_null($order)) {
 			return false;
 		}
 
-		$order_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
+		$order_id = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
 
-		delete_post_meta( $order_id, self::META_NAME_FEE );
-		delete_post_meta( $order_id, self::LEGACY_META_NAME_FEE );
+		delete_post_meta($order_id, self::META_NAME_FEE);
+		delete_post_meta($order_id, self::LEGACY_META_NAME_FEE);
 	}
 
 	/**
@@ -119,22 +119,22 @@ 
 	 * @param object $order
 	 * @return string $amount
 	 */
-	public static function get_stripe_net( $order = null ) {
-		if ( is_null( $order ) ) {
+	public static function get_stripe_net($order = null) {
+		if (is_null($order)) {
 			return false;
 		}
 
-		$order_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
+		$order_id = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
 
-		$amount = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? get_post_meta( $order_id, self::META_NAME_NET, true ) : $order->get_meta( self::META_NAME_NET, true );
+		$amount = WC_Stripe_Helper::is_wc_lt('3.0') ? get_post_meta($order_id, self::META_NAME_NET, true) : $order->get_meta(self::META_NAME_NET, true);
 
 		// If not found let's check for legacy name.
-		if ( empty( $amount ) ) {
-			$amount = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? get_post_meta( $order_id, self::LEGACY_META_NAME_NET, true ) : $order->get_meta( self::LEGACY_META_NAME_NET, true );
+		if (empty($amount)) {
+			$amount = WC_Stripe_Helper::is_wc_lt('3.0') ? get_post_meta($order_id, self::LEGACY_META_NAME_NET, true) : $order->get_meta(self::LEGACY_META_NAME_NET, true);
 
 			// If found update to new name.
-			if ( $amount ) {
-				self::update_stripe_net( $order, $amount );
+			if ($amount) {
+				self::update_stripe_net($order, $amount);
 			}
 		}
 
@@ -148,14 +148,14 @@ 
 	 * @param object $order
 	 * @param float $amount
 	 */
-	public static function update_stripe_net( $order = null, $amount = 0.0 ) {
-		if ( is_null( $order ) ) {
+	public static function update_stripe_net($order = null, $amount = 0.0) {
+		if (is_null($order)) {
 			return false;
 		}
 
-		$order_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
+		$order_id = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
 
-		WC_Stripe_Helper::is_wc_lt( '3.0' ) ? update_post_meta( $order_id, self::META_NAME_NET, $amount ) : $order->update_meta_data( self::META_NAME_NET, $amount );
+		WC_Stripe_Helper::is_wc_lt('3.0') ? update_post_meta($order_id, self::META_NAME_NET, $amount) : $order->update_meta_data(self::META_NAME_NET, $amount);
 	}
 
 	/**
@@ -164,15 +164,15 @@ 
 	 * @since 4.1.0
 	 * @param object $order
 	 */
-	public static function delete_stripe_net( $order = null ) {
-		if ( is_null( $order ) ) {
+	public static function delete_stripe_net($order = null) {
+		if (is_null($order)) {
 			return false;
 		}
 
-		$order_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
+		$order_id = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
 
-		delete_post_meta( $order_id, self::META_NAME_NET );
-		delete_post_meta( $order_id, self::LEGACY_META_NAME_NET );
+		delete_post_meta($order_id, self::META_NAME_NET);
+		delete_post_meta($order_id, self::LEGACY_META_NAME_NET);
 	}
 
 	/**
@@ -183,15 +183,15 @@ 
 	 *
 	 * @return float|int
 	 */
-	public static function get_stripe_amount( $total, $currency = '' ) {
-		if ( ! $currency ) {
+	public static function get_stripe_amount($total, $currency = '') {
+		if ( ! $currency) {
 			$currency = get_woocommerce_currency();
 		}
 
-		if ( in_array( strtolower( $currency ), self::no_decimal_currencies() ) ) {
-			return absint( $total );
+		if (in_array(strtolower($currency), self::no_decimal_currencies())) {
+			return absint($total);
 		} else {
-			return absint( wc_format_decimal( ( (float) $total * 100 ), wc_get_price_decimals() ) ); // In cents.
+			return absint(wc_format_decimal(((float) $total * 100), wc_get_price_decimals())); // In cents.
 		}
 	}
 
@@ -206,24 +206,24 @@ 
 		return apply_filters(
 			'wc_stripe_localized_messages',
 			array(
-				'invalid_number'           => __( 'The card number is not a valid credit card number.', 'woocommerce-gateway-stripe' ),
-				'invalid_expiry_month'     => __( 'The card\'s expiration month is invalid.', 'woocommerce-gateway-stripe' ),
-				'invalid_expiry_year'      => __( 'The card\'s expiration year is invalid.', 'woocommerce-gateway-stripe' ),
-				'invalid_cvc'              => __( 'The card\'s security code is invalid.', 'woocommerce-gateway-stripe' ),
-				'incorrect_number'         => __( 'The card number is incorrect.', 'woocommerce-gateway-stripe' ),
-				'incomplete_number'        => __( 'The card number is incomplete.', 'woocommerce-gateway-stripe' ),
-				'incomplete_cvc'           => __( 'The card\'s security code is incomplete.', 'woocommerce-gateway-stripe' ),
-				'incomplete_expiry'        => __( 'The card\'s expiration date is incomplete.', 'woocommerce-gateway-stripe' ),
-				'expired_card'             => __( 'The card has expired.', 'woocommerce-gateway-stripe' ),
-				'incorrect_cvc'            => __( 'The card\'s security code is incorrect.', 'woocommerce-gateway-stripe' ),
-				'incorrect_zip'            => __( 'The card\'s zip code failed validation.', 'woocommerce-gateway-stripe' ),
-				'invalid_expiry_year_past' => __( 'The card\'s expiration year is in the past', 'woocommerce-gateway-stripe' ),
-				'card_declined'            => __( 'The card was declined.', 'woocommerce-gateway-stripe' ),
-				'missing'                  => __( 'There is no card on a customer that is being charged.', 'woocommerce-gateway-stripe' ),
-				'processing_error'         => __( 'An error occurred while processing the card.', 'woocommerce-gateway-stripe' ),
-				'invalid_request_error'    => __( 'Unable to process this payment, please try again or use alternative method.', 'woocommerce-gateway-stripe' ),
-				'invalid_sofort_country'   => __( 'The billing country is not accepted by SOFORT. Please try another country.', 'woocommerce-gateway-stripe' ),
-				'email_invalid'            => __( 'Invalid email address, please correct and try again.', 'woocommerce-gateway-stripe' ),
+				'invalid_number'           => __('The card number is not a valid credit card number.', 'woocommerce-gateway-stripe'),
+				'invalid_expiry_month'     => __('The card\'s expiration month is invalid.', 'woocommerce-gateway-stripe'),
+				'invalid_expiry_year'      => __('The card\'s expiration year is invalid.', 'woocommerce-gateway-stripe'),
+				'invalid_cvc'              => __('The card\'s security code is invalid.', 'woocommerce-gateway-stripe'),
+				'incorrect_number'         => __('The card number is incorrect.', 'woocommerce-gateway-stripe'),
+				'incomplete_number'        => __('The card number is incomplete.', 'woocommerce-gateway-stripe'),
+				'incomplete_cvc'           => __('The card\'s security code is incomplete.', 'woocommerce-gateway-stripe'),
+				'incomplete_expiry'        => __('The card\'s expiration date is incomplete.', 'woocommerce-gateway-stripe'),
+				'expired_card'             => __('The card has expired.', 'woocommerce-gateway-stripe'),
+				'incorrect_cvc'            => __('The card\'s security code is incorrect.', 'woocommerce-gateway-stripe'),
+				'incorrect_zip'            => __('The card\'s zip code failed validation.', 'woocommerce-gateway-stripe'),
+				'invalid_expiry_year_past' => __('The card\'s expiration year is in the past', 'woocommerce-gateway-stripe'),
+				'card_declined'            => __('The card was declined.', 'woocommerce-gateway-stripe'),
+				'missing'                  => __('There is no card on a customer that is being charged.', 'woocommerce-gateway-stripe'),
+				'processing_error'         => __('An error occurred while processing the card.', 'woocommerce-gateway-stripe'),
+				'invalid_request_error'    => __('Unable to process this payment, please try again or use alternative method.', 'woocommerce-gateway-stripe'),
+				'invalid_sofort_country'   => __('The billing country is not accepted by SOFORT. Please try another country.', 'woocommerce-gateway-stripe'),
+				'email_invalid'            => __('Invalid email address, please correct and try again.', 'woocommerce-gateway-stripe'),
 			)
 		);
 	}
@@ -264,24 +264,24 @@ 
 	 * @param string $type Type of number to format
 	 * @return string
 	 */
-	public static function format_balance_fee( $balance_transaction, $type = 'fee' ) {
-		if ( ! is_object( $balance_transaction ) ) {
+	public static function format_balance_fee($balance_transaction, $type = 'fee') {
+		if ( ! is_object($balance_transaction)) {
 			return;
 		}
 
-		if ( in_array( strtolower( $balance_transaction->currency ), self::no_decimal_currencies() ) ) {
-			if ( 'fee' === $type ) {
+		if (in_array(strtolower($balance_transaction->currency), self::no_decimal_currencies())) {
+			if ('fee' === $type) {
 				return $balance_transaction->fee;
 			}
 
 			return $balance_transaction->net;
 		}
 
-		if ( 'fee' === $type ) {
-			return number_format( $balance_transaction->fee / 100, 2, '.', '' );
+		if ('fee' === $type) {
+			return number_format($balance_transaction->fee / 100, 2, '.', '');
 		}
 
-		return number_format( $balance_transaction->net / 100, 2, '.', '' );
+		return number_format($balance_transaction->net / 100, 2, '.', '');
 	}
 
 	/**
@@ -289,7 +289,7 @@ 
 	 */
 	public static function get_minimum_amount() {
 		// Check order amount
-		switch ( get_woocommerce_currency() ) {
+		switch (get_woocommerce_currency()) {
 			case 'USD':
 			case 'CAD':
 			case 'EUR':
@@ -334,14 +334,14 @@ 
 	 * @param string $method The payment method to get the settings from.
 	 * @param string $setting The name of the setting to get.
 	 */
-	public static function get_settings( $method = null, $setting = null ) {
-		$all_settings = null === $method ? get_option( 'woocommerce_stripe_settings', array() ) : get_option( 'woocommerce_stripe_' . $method . '_settings', array() );
+	public static function get_settings($method = null, $setting = null) {
+		$all_settings = null === $method ? get_option('woocommerce_stripe_settings', array()) : get_option('woocommerce_stripe_' . $method . '_settings', array());
 
-		if ( null === $setting ) {
+		if (null === $setting) {
 			return $all_settings;
 		}
 
-		return isset( $all_settings[ $setting ] ) ? $all_settings[ $setting ] : '';
+		return isset($all_settings[$setting]) ? $all_settings[$setting] : '';
 	}
 
 	/**
@@ -351,7 +351,7 @@ 
 	 * @return bool
 	 */
 	public static function is_pre_orders_exists() {
-		return class_exists( 'WC_Pre_Orders_Order' );
+		return class_exists('WC_Pre_Orders_Order');
 	}
 
 	/**
@@ -363,9 +363,9 @@ 
 	 * @return bool
 	 */
 	public static function is_pre_30() {
-		error_log( 'is_pre_30() function has been deprecated since 4.1.11. Please use is_wc_lt( $version ) instead.' );
+		error_log('is_pre_30() function has been deprecated since 4.1.11. Please use is_wc_lt( $version ) instead.');
 
-		return self::is_wc_lt( '3.0' );
+		return self::is_wc_lt('3.0');
 	}
 
 	/**
@@ -375,8 +375,8 @@ 
 	 * @param string $version Version to check against.
 	 * @return bool
 	 */
-	public static function is_wc_lt( $version ) {
-		return version_compare( WC_VERSION, $version, '<' );
+	public static function is_wc_lt($version) {
+		return version_compare(WC_VERSION, $version, '<');
 	}
 
 	/**
@@ -389,7 +389,7 @@ 
 	 * @return string
 	 */
 	public static function get_webhook_url() {
-		return add_query_arg( 'wc-api', 'wc_stripe', trailingslashit( get_home_url() ) );
+		return add_query_arg('wc-api', 'wc_stripe', trailingslashit(get_home_url()));
 	}
 
 	/**
@@ -399,13 +399,13 @@ 
 	 * @version 4.0.0
 	 * @param string $source_id
 	 */
-	public static function get_order_by_source_id( $source_id ) {
+	public static function get_order_by_source_id($source_id) {
 		global $wpdb;
 
-		$order_id = $wpdb->get_var( $wpdb->prepare( "SELECT DISTINCT ID FROM $wpdb->posts as posts LEFT JOIN $wpdb->postmeta as meta ON posts.ID = meta.post_id WHERE meta.meta_value = %s AND meta.meta_key = %s", $source_id, '_stripe_source_id' ) );
+		$order_id = $wpdb->get_var($wpdb->prepare("SELECT DISTINCT ID FROM $wpdb->posts as posts LEFT JOIN $wpdb->postmeta as meta ON posts.ID = meta.post_id WHERE meta.meta_value = %s AND meta.meta_key = %s", $source_id, '_stripe_source_id'));
 
-		if ( ! empty( $order_id ) ) {
-			return wc_get_order( $order_id );
+		if ( ! empty($order_id)) {
+			return wc_get_order($order_id);
 		}
 
 		return false;
@@ -418,17 +418,17 @@ 
 	 * @since 4.1.16 Return false if charge_id is empty.
 	 * @param string $charge_id
 	 */
-	public static function get_order_by_charge_id( $charge_id ) {
+	public static function get_order_by_charge_id($charge_id) {
 		global $wpdb;
 
-		if ( empty( $charge_id ) ) {
+		if (empty($charge_id)) {
 			return false;
 		}
 
-		$order_id = $wpdb->get_var( $wpdb->prepare( "SELECT DISTINCT ID FROM $wpdb->posts as posts LEFT JOIN $wpdb->postmeta as meta ON posts.ID = meta.post_id WHERE meta.meta_value = %s AND meta.meta_key = %s", $charge_id, '_transaction_id' ) );
+		$order_id = $wpdb->get_var($wpdb->prepare("SELECT DISTINCT ID FROM $wpdb->posts as posts LEFT JOIN $wpdb->postmeta as meta ON posts.ID = meta.post_id WHERE meta.meta_value = %s AND meta.meta_key = %s", $charge_id, '_transaction_id'));
 
-		if ( ! empty( $order_id ) ) {
-			return wc_get_order( $order_id );
+		if ( ! empty($order_id)) {
+			return wc_get_order($order_id);
 		}
 
 		return false;
@@ -441,13 +441,13 @@ 
 	 * @param string $intent_id The ID of the intent.
 	 * @return WC_Order|bool Either an order or false when not found.
 	 */
-	public static function get_order_by_intent_id( $intent_id ) {
+	public static function get_order_by_intent_id($intent_id) {
 		global $wpdb;
 
-		$order_id = $wpdb->get_var( $wpdb->prepare( "SELECT DISTINCT ID FROM $wpdb->posts as posts LEFT JOIN $wpdb->postmeta as meta ON posts.ID = meta.post_id WHERE meta.meta_value = %s AND meta.meta_key = %s", $intent_id, '_stripe_intent_id' ) );
+		$order_id = $wpdb->get_var($wpdb->prepare("SELECT DISTINCT ID FROM $wpdb->posts as posts LEFT JOIN $wpdb->postmeta as meta ON posts.ID = meta.post_id WHERE meta.meta_value = %s AND meta.meta_key = %s", $intent_id, '_stripe_intent_id'));
 
-		if ( ! empty( $order_id ) ) {
-			return wc_get_order( $order_id );
+		if ( ! empty($order_id)) {
+			return wc_get_order($order_id);
 		}
 
 		return false;
@@ -460,13 +460,13 @@ 
 	 * @param string $intent_id The ID of the intent.
 	 * @return WC_Order|bool Either an order or false when not found.
 	 */
-	public static function get_order_by_setup_intent_id( $intent_id ) {
+	public static function get_order_by_setup_intent_id($intent_id) {
 		global $wpdb;
 
-		$order_id = $wpdb->get_var( $wpdb->prepare( "SELECT DISTINCT ID FROM $wpdb->posts as posts LEFT JOIN $wpdb->postmeta as meta ON posts.ID = meta.post_id WHERE meta.meta_value = %s AND meta.meta_key = %s", $intent_id, '_stripe_setup_intent' ) );
+		$order_id = $wpdb->get_var($wpdb->prepare("SELECT DISTINCT ID FROM $wpdb->posts as posts LEFT JOIN $wpdb->postmeta as meta ON posts.ID = meta.post_id WHERE meta.meta_value = %s AND meta.meta_key = %s", $intent_id, '_stripe_setup_intent'));
 
-		if ( ! empty( $order_id ) ) {
-			return wc_get_order( $order_id );
+		if ( ! empty($order_id)) {
+			return wc_get_order($order_id);
 		}
 
 		return false;
@@ -482,13 +482,13 @@ 
 	 * @param string $statement_descriptor
 	 * @return string $statement_descriptor Sanitized statement descriptor
 	 */
-	public static function clean_statement_descriptor( $statement_descriptor = '' ) {
-		$disallowed_characters = array( '<', '>', '"', "'" );
+	public static function clean_statement_descriptor($statement_descriptor = '') {
+		$disallowed_characters = array('<', '>', '"', "'");
 
 		// Remove special characters.
-		$statement_descriptor = str_replace( $disallowed_characters, '', $statement_descriptor );
+		$statement_descriptor = str_replace($disallowed_characters, '', $statement_descriptor);
 
-		$statement_descriptor = substr( trim( $statement_descriptor ), 0, 22 );
+		$statement_descriptor = substr(trim($statement_descriptor), 0, 22);
 
 		return $statement_descriptor;
 	}

tests/phpunit/test-wc-stripe-sub-initial.php

Spacing +44 added lines, -44 removed lines

@@ -34,9 +34,9 @@ 
 	public function setUp() {
 		parent::setUp();
 
-		$this->wc_stripe_subs_compat = $this->getMockBuilder( 'WC_Stripe_Subs_Compat' )
+		$this->wc_stripe_subs_compat = $this->getMockBuilder('WC_Stripe_Subs_Compat')
 			->disableOriginalConstructor()
-			->setMethods( array( 'prepare_source', 'has_subscription' ) )
+			->setMethods(array('prepare_source', 'has_subscription'))
 			->getMock();
 
 		// Mocked in order to get metadata[payment_type] = recurring in the HTTP request.
@@ -54,7 +54,7 @@ 
 	 */
 	public function tearDown() {
 		parent::tearDown();
-		delete_option( 'woocommerce_stripe_settings' );
+		delete_option('woocommerce_stripe_settings');
 	}
 
 	/**
@@ -69,27 +69,27 @@ 
 	 */
 	public function test_initial_intent_parameters() {
 		$initial_order        = WC_Helper_Order::create_order();
-		$order_id             = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $initial_order->id : $initial_order->get_id();
-		$stripe_amount        = WC_Stripe_Helper::get_stripe_amount( $initial_order->get_total() );
-		$currency             = strtolower( WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $initial_order->get_order_currency() : $initial_order->get_currency() );
+		$order_id             = WC_Stripe_Helper::is_wc_lt('3.0') ? $initial_order->id : $initial_order->get_id();
+		$stripe_amount        = WC_Stripe_Helper::get_stripe_amount($initial_order->get_total());
+		$currency             = strtolower(WC_Stripe_Helper::is_wc_lt('3.0') ? $initial_order->get_order_currency() : $initial_order->get_currency());
 		$customer             = 'cus_123abc';
 		$source               = 'src_123abc';
-		$statement_descriptor = WC_Stripe_Helper::clean_statement_descriptor( $this->statement_descriptor );
+		$statement_descriptor = WC_Stripe_Helper::clean_statement_descriptor($this->statement_descriptor);
 		$intents_api_endpoint = 'https://api.stripe.com/v1/payment_intents';
 		$urls_used            = array();
 
-		if ( WC_Stripe_Helper::is_wc_lt( '3.0' ) ) {
+		if (WC_Stripe_Helper::is_wc_lt('3.0')) {
 			$initial_order->payment_method = 'stripe';
-			update_post_meta( $order_id, '_payment_method', 'stripe' ); // for `wc_get_order()`.
+			update_post_meta($order_id, '_payment_method', 'stripe'); // for `wc_get_order()`.
 		} else {
-			$initial_order->set_payment_method( 'stripe' );
+			$initial_order->set_payment_method('stripe');
 			$initial_order->save();
 		}
 
 		// Arrange: Mock prepare_source() so that we have a customer and source.
 		$this->wc_stripe_subs_compat
-			->expects( $this->any() )
-			->method( 'prepare_source' )
+			->expects($this->any())
+			->method('prepare_source')
 			->will(
 				$this->returnValue(
 					(object) array(
@@ -103,11 +103,11 @@ 
 
 		// Emulate a subscription.
 		$this->wc_stripe_subs_compat
-			->expects( $this->any() )
-			->method( 'has_subscription' )
-			->will( $this->returnValue( true ) );
+			->expects($this->any())
+			->method('has_subscription')
+			->will($this->returnValue(true));
 
-		$pre_http_request_response_callback = function( $preempt, $request_args, $url ) use (
+		$pre_http_request_response_callback = function($preempt, $request_args, $url) use (
 			$stripe_amount,
 			$currency,
 			$customer,
@@ -118,9 +118,9 @@ 
 			&$urls_used
 		) {
 			// Add all urls to array so we can later make assertions about which endpoints were used.
-			array_push( $urls_used, $url );
+			array_push($urls_used, $url);
 			// Continue without mocking the request if it's not the endpoint we care about.
-			if ( 0 !== strpos( $url, $intents_api_endpoint ) ) {
+			if (0 !== strpos($url, $intents_api_endpoint)) {
 				return false;
 			}
 
@@ -128,7 +128,7 @@ 
 			$response = array(
 				'headers'  => array(),
 				// Too bad we aren't dynamically setting things 'cus_123abc' when using this file.
-				'body'     => file_get_contents( 'tests/phpunit/dummy-data/subscription_signup_response_success.json' ),
+				'body'     => file_get_contents('tests/phpunit/dummy-data/subscription_signup_response_success.json'),
 				'response' => array(
 					'code'    => 200,
 					'message' => 'OK',
@@ -138,17 +138,17 @@ 
 			);
 
 			// Respond with a successfull intent for confirmations.
-			if ( $url !== $intents_api_endpoint ) {
-				$response['body'] = str_replace( 'requires_confirmation', 'succeeded', $response['body'] );
+			if ($url !== $intents_api_endpoint) {
+				$response['body'] = str_replace('requires_confirmation', 'succeeded', $response['body']);
 				return $response;
 			}
 
 			// Assert: the request method is POST.
-			$this->assertArrayHasKey( 'method', $request_args );
-			$this->assertSame( 'POST', $request_args['method'] );
+			$this->assertArrayHasKey('method', $request_args);
+			$this->assertSame('POST', $request_args['method']);
 
 			// Assert: the request has a body.
-			$this->assertArrayHasKey( 'body', $request_args );
+			$this->assertArrayHasKey('body', $request_args);
 
 			// Assert: the request body contains these values.
 			$expected_request_body_values = array(
@@ -158,11 +158,11 @@ 
 				'statement_descriptor' => $statement_descriptor,
 				'customer'             => $customer,
 				'setup_future_usage'   => 'off_session',
-				'payment_method_types' => array( 'card' ),
+				'payment_method_types' => array('card'),
 			);
-			foreach ( $expected_request_body_values as $key => $value ) {
-				$this->assertArrayHasKey( $key, $request_args['body'] );
-				$this->assertSame( $value, $request_args['body'][ $key ] );
+			foreach ($expected_request_body_values as $key => $value) {
+				$this->assertArrayHasKey($key, $request_args['body']);
+				$this->assertSame($value, $request_args['body'][$key]);
 			}
 
 			// Assert: the request body contains these keys, without checking for their value.
@@ -170,48 +170,48 @@ 
 				'description',
 				'capture_method',
 			);
-			foreach ( $expected_request_body_keys as $key ) {
-				$this->assertArrayHasKey( $key, $request_args['body'] );
+			foreach ($expected_request_body_keys as $key) {
+				$this->assertArrayHasKey($key, $request_args['body']);
 			}
 
 			// Assert: the body metadata contains the order ID.
-			$this->assertSame( $order_id, absint( $request_args['body']['metadata']['order_id'] ) );
+			$this->assertSame($order_id, absint($request_args['body']['metadata']['order_id']));
 
 			// // Assert: the body metadata has these keys, without checking for their value.
 			$expected_metadata_keys = array(
 				'customer_name',
 				'customer_email',
 			);
-			foreach ( $expected_metadata_keys as $key ) {
-				$this->assertArrayHasKey( $key, $request_args['body']['metadata'] );
+			foreach ($expected_metadata_keys as $key) {
+				$this->assertArrayHasKey($key, $request_args['body']['metadata']);
 			}
 
 			// Return dummy content as the response.
 			return $response;
 		};
-		add_filter( 'pre_http_request', $pre_http_request_response_callback, 10, 3 );
+		add_filter('pre_http_request', $pre_http_request_response_callback, 10, 3);
 
 		// Act: call process_subscription_payment().
 		// We need to use `wc_stripe_subs_compat` here because we mocked this class earlier.
-		$result = $this->wc_stripe_subs_compat->process_payment( $order_id );
+		$result = $this->wc_stripe_subs_compat->process_payment($order_id);
 
 		// Assert: nothing was returned.
-		$this->assertEquals( $result['result'], 'success' );
-		$this->assertArrayHasKey( 'redirect', $result );
+		$this->assertEquals($result['result'], 'success');
+		$this->assertArrayHasKey('redirect', $result);
 
-		$order      = wc_get_order( $order_id );
+		$order      = wc_get_order($order_id);
 		$order_data = (
-			WC_Stripe_Helper::is_wc_lt( '3.0' )
-				? get_post_meta( $order_id, '_stripe_intent_id', true )
-				: $order->get_meta( '_stripe_intent_id' )
+			WC_Stripe_Helper::is_wc_lt('3.0')
+				? get_post_meta($order_id, '_stripe_intent_id', true)
+				: $order->get_meta('_stripe_intent_id')
 		);
 
-		$this->assertEquals( $order_data, 'pi_123abc' );
+		$this->assertEquals($order_data, 'pi_123abc');
 
 		// Assert: called payment intents.
-		$this->assertTrue( in_array( $intents_api_endpoint, $urls_used, true ) );
+		$this->assertTrue(in_array($intents_api_endpoint, $urls_used, true));
 
 		// Clean up.
-		remove_filter( 'pre_http_request', array( $this, 'pre_http_request_response_success' ) );
+		remove_filter('pre_http_request', array($this, 'pre_http_request_response_success'));
 	}
 }

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

Spacing +241 added lines, -241 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -39,12 +39,12 @@ 
 	 */
 	public function __construct() {
 		$this->retry_interval = 2;
-		$stripe_settings      = get_option( 'woocommerce_stripe_settings', array() );
-		$this->testmode       = ( ! empty( $stripe_settings['testmode'] ) && 'yes' === $stripe_settings['testmode'] ) ? true : false;
-		$secret_key           = ( $this->testmode ? 'test_' : '' ) . 'webhook_secret';
-		$this->secret         = ! empty( $stripe_settings[ $secret_key ] ) ? $stripe_settings[ $secret_key ] : false;
+		$stripe_settings      = get_option('woocommerce_stripe_settings', array());
+		$this->testmode       = ( ! empty($stripe_settings['testmode']) && 'yes' === $stripe_settings['testmode']) ? true : false;
+		$secret_key           = ($this->testmode ? 'test_' : '') . 'webhook_secret';
+		$this->secret         = ! empty($stripe_settings[$secret_key]) ? $stripe_settings[$secret_key] : false;
 
-		add_action( 'woocommerce_api_wc_stripe', array( $this, 'check_for_webhook' ) );
+		add_action('woocommerce_api_wc_stripe', array($this, 'check_for_webhook'));
 	}
 
 	/**
@@ -54,24 +54,24 @@ 
 	 * @version 4.0.0
 	 */
 	public function check_for_webhook() {
-		if ( ( 'POST' !== $_SERVER['REQUEST_METHOD'] )
-			|| ! isset( $_GET['wc-api'] )
-			|| ( 'wc_stripe' !== $_GET['wc-api'] )
+		if (('POST' !== $_SERVER['REQUEST_METHOD'])
+			|| ! isset($_GET['wc-api'])
+			|| ('wc_stripe' !== $_GET['wc-api'])
 		) {
 			return;
 		}
 
-		$request_body    = file_get_contents( 'php://input' );
-		$request_headers = array_change_key_case( $this->get_request_headers(), CASE_UPPER );
+		$request_body    = file_get_contents('php://input');
+		$request_headers = array_change_key_case($this->get_request_headers(), CASE_UPPER);
 
 		// Validate it to make sure it is legit.
-		if ( $this->is_valid_request( $request_headers, $request_body ) ) {
-			$this->process_webhook( $request_body );
-			status_header( 200 );
+		if ($this->is_valid_request($request_headers, $request_body)) {
+			$this->process_webhook($request_body);
+			status_header(200);
 			exit;
 		} else {
-			WC_Stripe_Logger::log( 'Incoming webhook failed validation: ' . print_r( $request_body, true ) );
-			status_header( 400 );
+			WC_Stripe_Logger::log('Incoming webhook failed validation: ' . print_r($request_body, true));
+			status_header(400);
 			exit;
 		}
 	}
@@ -85,34 +85,34 @@ 
 	 * @param string $request_body The request body from Stripe.
 	 * @return bool
 	 */
-	public function is_valid_request( $request_headers = null, $request_body = null ) {
-		if ( null === $request_headers || null === $request_body ) {
+	public function is_valid_request($request_headers = null, $request_body = null) {
+		if (null === $request_headers || null === $request_body) {
 			return false;
 		}
 
-		if ( ! empty( $request_headers['USER-AGENT'] ) && ! preg_match( '/Stripe/', $request_headers['USER-AGENT'] ) ) {
+		if ( ! empty($request_headers['USER-AGENT']) && ! preg_match('/Stripe/', $request_headers['USER-AGENT'])) {
 			return false;
 		}
 
-		if ( ! empty( $this->secret ) ) {
+		if ( ! empty($this->secret)) {
 			// Check for a valid signature.
 			$signature_format = '/^t=(?P<timestamp>\d+)(?P<signatures>(,v\d+=[a-z0-9]+){1,2})$/';
-			if ( empty( $request_headers['STRIPE-SIGNATURE'] ) || ! preg_match( $signature_format, $request_headers['STRIPE-SIGNATURE'], $matches ) ) {
+			if (empty($request_headers['STRIPE-SIGNATURE']) || ! preg_match($signature_format, $request_headers['STRIPE-SIGNATURE'], $matches)) {
 				return false;
 			}
 
 			// Verify the timestamp.
-			$timestamp = intval( $matches['timestamp'] );
-			if ( abs( $timestamp - time() ) > 5 * MINUTE_IN_SECONDS ) {
+			$timestamp = intval($matches['timestamp']);
+			if (abs($timestamp - time()) > 5 * MINUTE_IN_SECONDS) {
 				return;
 			}
 
 			// Generate the expected signature.
 			$signed_payload     = $timestamp . '.' . $request_body;
-			$expected_signature = hash_hmac( 'sha256', $signed_payload, $this->secret );
+			$expected_signature = hash_hmac('sha256', $signed_payload, $this->secret);
 
 			// Check if the expected signature is present.
-			if ( ! preg_match( '/,v\d+=' . preg_quote( $expected_signature, '/' ) . '/', $matches['signatures'] ) ) {
+			if ( ! preg_match('/,v\d+=' . preg_quote($expected_signature, '/') . '/', $matches['signatures'])) {
 				return false;
 			}
 		}
@@ -129,12 +129,12 @@ 
 	 * @version 4.0.0
 	 */
 	public function get_request_headers() {
-		if ( ! function_exists( 'getallheaders' ) ) {
+		if ( ! function_exists('getallheaders')) {
 			$headers = array();
 
-			foreach ( $_SERVER as $name => $value ) {
-				if ( 'HTTP_' === substr( $name, 0, 5 ) ) {
-					$headers[ str_replace( ' ', '-', ucwords( strtolower( str_replace( '_', ' ', substr( $name, 5 ) ) ) ) ) ] = $value;
+			foreach ($_SERVER as $name => $value) {
+				if ('HTTP_' === substr($name, 0, 5)) {
+					$headers[str_replace(' ', '-', ucwords(strtolower(str_replace('_', ' ', substr($name, 5)))))] = $value;
 				}
 			}
 
@@ -153,30 +153,30 @@ 
 	 * @param object $notification
 	 * @param bool $retry
 	 */
-	public function process_webhook_payment( $notification, $retry = true ) {
+	public function process_webhook_payment($notification, $retry = true) {
 		// The following 3 payment methods are synchronous so does not need to be handle via webhook.
-		if ( 'card' === $notification->data->object->type || 'sepa_debit' === $notification->data->object->type || 'three_d_secure' === $notification->data->object->type ) {
+		if ('card' === $notification->data->object->type || 'sepa_debit' === $notification->data->object->type || 'three_d_secure' === $notification->data->object->type) {
 			return;
 		}
 
-		$order = WC_Stripe_Helper::get_order_by_source_id( $notification->data->object->id );
+		$order = WC_Stripe_Helper::get_order_by_source_id($notification->data->object->id);
 
-		if ( ! $order ) {
-			WC_Stripe_Logger::log( 'Could not find order via source ID: ' . $notification->data->object->id );
+		if ( ! $order) {
+			WC_Stripe_Logger::log('Could not find order via source ID: ' . $notification->data->object->id);
 			return;
 		}
 
-		$order_id  = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
+		$order_id  = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
 		$source_id = $notification->data->object->id;
 
-		$is_pending_receiver = ( 'receiver' === $notification->data->object->flow );
+		$is_pending_receiver = ('receiver' === $notification->data->object->flow);
 
 		try {
-			if ( 'processing' === $order->get_status() || 'completed' === $order->get_status() ) {
+			if ('processing' === $order->get_status() || 'completed' === $order->get_status()) {
 				return;
 			}
 
-			if ( 'on-hold' === $order->get_status() && ! $is_pending_receiver ) {
+			if ('on-hold' === $order->get_status() && ! $is_pending_receiver) {
 				return;
 			}
 
@@ -184,94 +184,94 @@ 
 			$response = null;
 
 			// This will throw exception if not valid.
-			$this->validate_minimum_order_amount( $order );
+			$this->validate_minimum_order_amount($order);
 
-			WC_Stripe_Logger::log( "Info: (Webhook) Begin processing payment for order $order_id for the amount of {$order->get_total()}" );
+			WC_Stripe_Logger::log("Info: (Webhook) Begin processing payment for order $order_id for the amount of {$order->get_total()}");
 
 			// Prep source object.
 			$source_object           = new stdClass();
 			$source_object->token_id = '';
-			$source_object->customer = $this->get_stripe_customer_id( $order );
+			$source_object->customer = $this->get_stripe_customer_id($order);
 			$source_object->source   = $source_id;
 
 			// Make the request.
-			$response = WC_Stripe_API::request( $this->generate_payment_request( $order, $source_object ), 'charges', 'POST', true );
+			$response = WC_Stripe_API::request($this->generate_payment_request($order, $source_object), 'charges', 'POST', true);
 			$headers  = $response['headers'];
 			$response = $response['body'];
 
-			if ( ! empty( $response->error ) ) {
+			if ( ! empty($response->error)) {
 				// Customer param wrong? The user may have been deleted on stripe's end. Remove customer_id. Can be retried without.
-				if ( $this->is_no_such_customer_error( $response->error ) ) {
-					if ( WC_Stripe_Helper::is_wc_lt( '3.0' ) ) {
-						delete_user_meta( $order->customer_user, '_stripe_customer_id' );
-						delete_post_meta( $order_id, '_stripe_customer_id' );
+				if ($this->is_no_such_customer_error($response->error)) {
+					if (WC_Stripe_Helper::is_wc_lt('3.0')) {
+						delete_user_meta($order->customer_user, '_stripe_customer_id');
+						delete_post_meta($order_id, '_stripe_customer_id');
 					} else {
-						delete_user_meta( $order->get_customer_id(), '_stripe_customer_id' );
-						$order->delete_meta_data( '_stripe_customer_id' );
+						delete_user_meta($order->get_customer_id(), '_stripe_customer_id');
+						$order->delete_meta_data('_stripe_customer_id');
 						$order->save();
 					}
 				}
 
-				if ( $this->is_no_such_token_error( $response->error ) && $prepared_source->token_id ) {
+				if ($this->is_no_such_token_error($response->error) && $prepared_source->token_id) {
 					// Source param wrong? The CARD may have been deleted on stripe's end. Remove token and show message.
-					$wc_token = WC_Payment_Tokens::get( $prepared_source->token_id );
+					$wc_token = WC_Payment_Tokens::get($prepared_source->token_id);
 					$wc_token->delete();
-					$localized_message = __( 'This card is no longer available and has been removed.', 'woocommerce-gateway-stripe' );
-					$order->add_order_note( $localized_message );
-					throw new WC_Stripe_Exception( print_r( $response, true ), $localized_message );
+					$localized_message = __('This card is no longer available and has been removed.', 'woocommerce-gateway-stripe');
+					$order->add_order_note($localized_message);
+					throw new WC_Stripe_Exception(print_r($response, true), $localized_message);
 				}
 
 				// We want to retry.
-				if ( $this->is_retryable_error( $response->error ) ) {
-					if ( $retry ) {
+				if ($this->is_retryable_error($response->error)) {
+					if ($retry) {
 						// Don't do anymore retries after this.
-						if ( 5 <= $this->retry_interval ) {
+						if (5 <= $this->retry_interval) {
 
-							return $this->process_webhook_payment( $notification, false );
+							return $this->process_webhook_payment($notification, false);
 						}
 
-						sleep( $this->retry_interval );
+						sleep($this->retry_interval);
 
 						$this->retry_interval++;
-						return $this->process_webhook_payment( $notification, true );
+						return $this->process_webhook_payment($notification, true);
 					} else {
-						$localized_message = __( 'Sorry, we are unable to process your payment at this time. Please retry later.', 'woocommerce-gateway-stripe' );
-						$order->add_order_note( $localized_message );
-						throw new WC_Stripe_Exception( print_r( $response, true ), $localized_message );
+						$localized_message = __('Sorry, we are unable to process your payment at this time. Please retry later.', 'woocommerce-gateway-stripe');
+						$order->add_order_note($localized_message);
+						throw new WC_Stripe_Exception(print_r($response, true), $localized_message);
 					}
 				}
 
 				$localized_messages = WC_Stripe_Helper::get_localized_messages();
 
-				if ( 'card_error' === $response->error->type ) {
-					$localized_message = isset( $localized_messages[ $response->error->code ] ) ? $localized_messages[ $response->error->code ] : $response->error->message;
+				if ('card_error' === $response->error->type) {
+					$localized_message = isset($localized_messages[$response->error->code]) ? $localized_messages[$response->error->code] : $response->error->message;
 				} else {
-					$localized_message = isset( $localized_messages[ $response->error->type ] ) ? $localized_messages[ $response->error->type ] : $response->error->message;
+					$localized_message = isset($localized_messages[$response->error->type]) ? $localized_messages[$response->error->type] : $response->error->message;
 				}
 
-				$order->add_order_note( $localized_message );
+				$order->add_order_note($localized_message);
 
-				throw new WC_Stripe_Exception( print_r( $response, true ), $localized_message );
+				throw new WC_Stripe_Exception(print_r($response, true), $localized_message);
 			}
 
 			// To prevent double processing the order on WC side.
-			if ( ! $this->is_original_request( $headers ) ) {
+			if ( ! $this->is_original_request($headers)) {
 				return;
 			}
 
-			do_action( 'wc_gateway_stripe_process_webhook_payment', $response, $order );
+			do_action('wc_gateway_stripe_process_webhook_payment', $response, $order);
 
-			$this->process_response( $response, $order );
+			$this->process_response($response, $order);
 
-		} catch ( WC_Stripe_Exception $e ) {
-			WC_Stripe_Logger::log( 'Error: ' . $e->getMessage() );
+		} catch (WC_Stripe_Exception $e) {
+			WC_Stripe_Logger::log('Error: ' . $e->getMessage());
 
-			do_action( 'wc_gateway_stripe_process_webhook_payment_error', $order, $notification, $e );
+			do_action('wc_gateway_stripe_process_webhook_payment_error', $order, $notification, $e);
 
-			$statuses = array( 'pending', 'failed' );
+			$statuses = array('pending', 'failed');
 
-			if ( $order->has_status( $statuses ) ) {
-				$this->send_failed_order_email( $order_id );
+			if ($order->has_status($statuses)) {
+				$this->send_failed_order_email($order_id);
 			}
 		}
 	}
@@ -284,21 +284,21 @@ 
 	 * @since 4.0.0
 	 * @param object $notification
 	 */
-	public function process_webhook_dispute( $notification ) {
-		$order = WC_Stripe_Helper::get_order_by_charge_id( $notification->data->object->charge );
+	public function process_webhook_dispute($notification) {
+		$order = WC_Stripe_Helper::get_order_by_charge_id($notification->data->object->charge);
 
-		if ( ! $order ) {
-			WC_Stripe_Logger::log( 'Could not find order via charge ID: ' . $notification->data->object->charge );
+		if ( ! $order) {
+			WC_Stripe_Logger::log('Could not find order via charge ID: ' . $notification->data->object->charge);
 			return;
 		}
 
 		/* translators: 1) The URL to the order. */
-		$order->update_status( 'on-hold', sprintf( __( 'A dispute was created for this order. Response is needed. Please go to your <a href="%s" title="Stripe Dashboard" target="_blank">Stripe Dashboard</a> to review this dispute.', 'woocommerce-gateway-stripe' ), $this->get_transaction_url( $order ) ) );
+		$order->update_status('on-hold', sprintf(__('A dispute was created for this order. Response is needed. Please go to your <a href="%s" title="Stripe Dashboard" target="_blank">Stripe Dashboard</a> to review this dispute.', 'woocommerce-gateway-stripe'), $this->get_transaction_url($order)));
 
-		do_action( 'wc_gateway_stripe_process_webhook_payment_error', $order, $notification );
+		do_action('wc_gateway_stripe_process_webhook_payment_error', $order, $notification);
 
-		$order_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
-		$this->send_failed_order_email( $order_id );
+		$order_id = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
+		$this->send_failed_order_email($order_id);
 	}
 
 	/**
@@ -309,45 +309,45 @@ 
 	 * @version 4.0.0
 	 * @param object $notification
 	 */
-	public function process_webhook_capture( $notification ) {
-		$order = WC_Stripe_Helper::get_order_by_charge_id( $notification->data->object->id );
+	public function process_webhook_capture($notification) {
+		$order = WC_Stripe_Helper::get_order_by_charge_id($notification->data->object->id);
 
-		if ( ! $order ) {
-			WC_Stripe_Logger::log( 'Could not find order via charge ID: ' . $notification->data->object->id );
+		if ( ! $order) {
+			WC_Stripe_Logger::log('Could not find order via charge ID: ' . $notification->data->object->id);
 			return;
 		}
 
-		$order_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
+		$order_id = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
 
-		if ( 'stripe' === ( WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->payment_method : $order->get_payment_method() ) ) {
-			$charge   = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? get_post_meta( $order_id, '_transaction_id', true ) : $order->get_transaction_id();
-			$captured = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? get_post_meta( $order_id, '_stripe_charge_captured', true ) : $order->get_meta( '_stripe_charge_captured', true );
+		if ('stripe' === (WC_Stripe_Helper::is_wc_lt('3.0') ? $order->payment_method : $order->get_payment_method())) {
+			$charge   = WC_Stripe_Helper::is_wc_lt('3.0') ? get_post_meta($order_id, '_transaction_id', true) : $order->get_transaction_id();
+			$captured = WC_Stripe_Helper::is_wc_lt('3.0') ? get_post_meta($order_id, '_stripe_charge_captured', true) : $order->get_meta('_stripe_charge_captured', true);
 
-			if ( $charge && 'no' === $captured ) {
-				WC_Stripe_Helper::is_wc_lt( '3.0' ) ? update_post_meta( $order_id, '_stripe_charge_captured', 'yes' ) : $order->update_meta_data( '_stripe_charge_captured', 'yes' );
+			if ($charge && 'no' === $captured) {
+				WC_Stripe_Helper::is_wc_lt('3.0') ? update_post_meta($order_id, '_stripe_charge_captured', 'yes') : $order->update_meta_data('_stripe_charge_captured', 'yes');
 
 				// Store other data such as fees
-				WC_Stripe_Helper::is_wc_lt( '3.0' ) ? update_post_meta( $order_id, '_transaction_id', $notification->data->object->id ) : $order->set_transaction_id( $notification->data->object->id );
+				WC_Stripe_Helper::is_wc_lt('3.0') ? update_post_meta($order_id, '_transaction_id', $notification->data->object->id) : $order->set_transaction_id($notification->data->object->id);
 
-				if ( isset( $notification->data->object->balance_transaction ) ) {
-					$this->update_fees( $order, $notification->data->object->balance_transaction );
+				if (isset($notification->data->object->balance_transaction)) {
+					$this->update_fees($order, $notification->data->object->balance_transaction);
 				}
 
 				// Check and see if capture is partial.
-				if ( $this->is_partial_capture( $notification ) ) {
-					$partial_amount = $this->get_partial_amount_to_charge( $notification );
-					$order->set_total( $partial_amount );
-					$this->update_fees( $order, $notification->data->object->refunds->data[0]->balance_transaction );
+				if ($this->is_partial_capture($notification)) {
+					$partial_amount = $this->get_partial_amount_to_charge($notification);
+					$order->set_total($partial_amount);
+					$this->update_fees($order, $notification->data->object->refunds->data[0]->balance_transaction);
 					/* translators: partial captured amount */
-					$order->add_order_note( sprintf( __( 'This charge was partially captured via Stripe Dashboard in the amount of: %s', 'woocommerce-gateway-stripe' ), $partial_amount ) );
+					$order->add_order_note(sprintf(__('This charge was partially captured via Stripe Dashboard in the amount of: %s', 'woocommerce-gateway-stripe'), $partial_amount));
 				} else {
-					$order->payment_complete( $notification->data->object->id );
+					$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)', 'woocommerce-gateway-stripe'), $notification->data->object->id));
 				}
 
-				if ( is_callable( array( $order, 'save' ) ) ) {
+				if (is_callable(array($order, 'save'))) {
 					$order->save();
 				}
 			}
@@ -362,43 +362,43 @@ 
 	 * @version 4.0.0
 	 * @param object $notification
 	 */
-	public function process_webhook_charge_succeeded( $notification ) {
+	public function process_webhook_charge_succeeded($notification) {
 		// Ignore the notification for charges, created through PaymentIntents.
-		if ( isset( $notification->data->object->payment_intent ) && $notification->data->object->payment_intent ) {
+		if (isset($notification->data->object->payment_intent) && $notification->data->object->payment_intent) {
 			return;
 		}
 
 		// The following payment methods are synchronous so does not need to be handle via webhook.
-		if ( ( isset( $notification->data->object->source->type ) && 'card' === $notification->data->object->source->type ) || ( isset( $notification->data->object->source->type ) && 'three_d_secure' === $notification->data->object->source->type ) ) {
+		if ((isset($notification->data->object->source->type) && 'card' === $notification->data->object->source->type) || (isset($notification->data->object->source->type) && 'three_d_secure' === $notification->data->object->source->type)) {
 			return;
 		}
 
-		$order = WC_Stripe_Helper::get_order_by_charge_id( $notification->data->object->id );
+		$order = WC_Stripe_Helper::get_order_by_charge_id($notification->data->object->id);
 
-		if ( ! $order ) {
-			WC_Stripe_Logger::log( 'Could not find order via charge ID: ' . $notification->data->object->id );
+		if ( ! $order) {
+			WC_Stripe_Logger::log('Could not find order via charge ID: ' . $notification->data->object->id);
 			return;
 		}
 
-		$order_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
+		$order_id = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
 
-		if ( 'on-hold' !== $order->get_status() ) {
+		if ('on-hold' !== $order->get_status()) {
 			return;
 		}
 
 		// Store other data such as fees
-		WC_Stripe_Helper::is_wc_lt( '3.0' ) ? update_post_meta( $order_id, '_transaction_id', $notification->data->object->id ) : $order->set_transaction_id( $notification->data->object->id );
+		WC_Stripe_Helper::is_wc_lt('3.0') ? update_post_meta($order_id, '_transaction_id', $notification->data->object->id) : $order->set_transaction_id($notification->data->object->id);
 
-		if ( isset( $notification->data->object->balance_transaction ) ) {
-			$this->update_fees( $order, $notification->data->object->balance_transaction );
+		if (isset($notification->data->object->balance_transaction)) {
+			$this->update_fees($order, $notification->data->object->balance_transaction);
 		}
 
-		$order->payment_complete( $notification->data->object->id );
+		$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)', 'woocommerce-gateway-stripe'), $notification->data->object->id));
 
-		if ( is_callable( array( $order, 'save' ) ) ) {
+		if (is_callable(array($order, 'save'))) {
 			$order->save();
 		}
 	}
@@ -410,24 +410,24 @@ 
 	 * @since 4.1.5 Can handle any fail payments from any methods.
 	 * @param object $notification
 	 */
-	public function process_webhook_charge_failed( $notification ) {
-		$order = WC_Stripe_Helper::get_order_by_charge_id( $notification->data->object->id );
+	public function process_webhook_charge_failed($notification) {
+		$order = WC_Stripe_Helper::get_order_by_charge_id($notification->data->object->id);
 
-		if ( ! $order ) {
-			WC_Stripe_Logger::log( 'Could not find order via charge ID: ' . $notification->data->object->id );
+		if ( ! $order) {
+			WC_Stripe_Logger::log('Could not find order via charge ID: ' . $notification->data->object->id);
 			return;
 		}
 
-		$order_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
+		$order_id = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
 
 		// If order status is already in failed status don't continue.
-		if ( 'failed' === $order->get_status() ) {
+		if ('failed' === $order->get_status()) {
 			return;
 		}
 
-		$order->update_status( 'failed', __( 'This payment failed to clear.', 'woocommerce-gateway-stripe' ) );
+		$order->update_status('failed', __('This payment failed to clear.', 'woocommerce-gateway-stripe'));
 
-		do_action( 'wc_gateway_stripe_process_webhook_payment_error', $order, $notification );
+		do_action('wc_gateway_stripe_process_webhook_payment_error', $order, $notification);
 	}
 
 	/**
@@ -438,30 +438,30 @@ 
 	 * @since 4.1.15 Add check to make sure order is processed by Stripe.
 	 * @param object $notification
 	 */
-	public function process_webhook_source_canceled( $notification ) {
-		$order = WC_Stripe_Helper::get_order_by_charge_id( $notification->data->object->id );
+	public function process_webhook_source_canceled($notification) {
+		$order = WC_Stripe_Helper::get_order_by_charge_id($notification->data->object->id);
 
 		// If can't find order by charge ID, try source ID.
-		if ( ! $order ) {
-			$order = WC_Stripe_Helper::get_order_by_source_id( $notification->data->object->id );
+		if ( ! $order) {
+			$order = WC_Stripe_Helper::get_order_by_source_id($notification->data->object->id);
 
-			if ( ! $order ) {
-				WC_Stripe_Logger::log( 'Could not find order via charge/source ID: ' . $notification->data->object->id );
+			if ( ! $order) {
+				WC_Stripe_Logger::log('Could not find order via charge/source ID: ' . $notification->data->object->id);
 				return;
 			}
 		}
 
 		// Don't proceed if payment method isn't Stripe.
-		if ( 'stripe' !== $order->get_payment_method() ) {
-			WC_Stripe_Logger::log( 'Canceled webhook abort: Order was not processed by Stripe: ' . $order->get_id() );
+		if ('stripe' !== $order->get_payment_method()) {
+			WC_Stripe_Logger::log('Canceled webhook abort: Order was not processed by Stripe: ' . $order->get_id());
 			return;
 		}
 
-		if ( 'cancelled' !== $order->get_status() ) {
-			$order->update_status( 'cancelled', __( 'This payment has cancelled.', 'woocommerce-gateway-stripe' ) );
+		if ('cancelled' !== $order->get_status()) {
+			$order->update_status('cancelled', __('This payment has cancelled.', 'woocommerce-gateway-stripe'));
 		}
 
-		do_action( 'wc_gateway_stripe_process_webhook_payment_error', $order, $notification );
+		do_action('wc_gateway_stripe_process_webhook_payment_error', $order, $notification);
 	}
 
 	/**
@@ -471,59 +471,59 @@ 
 	 * @version 4.0.0
 	 * @param object $notification
 	 */
-	public function process_webhook_refund( $notification ) {
-		$order = WC_Stripe_Helper::get_order_by_charge_id( $notification->data->object->id );
+	public function process_webhook_refund($notification) {
+		$order = WC_Stripe_Helper::get_order_by_charge_id($notification->data->object->id);
 
-		if ( ! $order ) {
-			WC_Stripe_Logger::log( 'Could not find order via charge ID: ' . $notification->data->object->id );
+		if ( ! $order) {
+			WC_Stripe_Logger::log('Could not find order via charge ID: ' . $notification->data->object->id);
 			return;
 		}
 
-		$order_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
+		$order_id = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
 
-		if ( 'stripe' === ( WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->payment_method : $order->get_payment_method() ) ) {
-			$charge    = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? get_post_meta( $order_id, '_transaction_id', true ) : $order->get_transaction_id();
-			$captured  = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? get_post_meta( $order_id, '_stripe_charge_captured', true ) : $order->get_meta( '_stripe_charge_captured', true );
-			$refund_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? get_post_meta( $order_id, '_stripe_refund_id', true ) : $order->get_meta( '_stripe_refund_id', true );
+		if ('stripe' === (WC_Stripe_Helper::is_wc_lt('3.0') ? $order->payment_method : $order->get_payment_method())) {
+			$charge    = WC_Stripe_Helper::is_wc_lt('3.0') ? get_post_meta($order_id, '_transaction_id', true) : $order->get_transaction_id();
+			$captured  = WC_Stripe_Helper::is_wc_lt('3.0') ? get_post_meta($order_id, '_stripe_charge_captured', true) : $order->get_meta('_stripe_charge_captured', true);
+			$refund_id = WC_Stripe_Helper::is_wc_lt('3.0') ? get_post_meta($order_id, '_stripe_refund_id', true) : $order->get_meta('_stripe_refund_id', true);
 
 			// If the refund ID matches, don't continue to prevent double refunding.
-			if ( $notification->data->object->refunds->data[0]->id === $refund_id ) {
+			if ($notification->data->object->refunds->data[0]->id === $refund_id) {
 				return;
 			}
 
 			// Only refund captured charge.
-			if ( $charge ) {
-				$reason = ( isset( $captured ) && 'yes' === $captured ) ? __( 'Refunded via Stripe Dashboard', 'woocommerce-gateway-stripe' ) : __( 'Pre-Authorization Released via Stripe Dashboard', 'woocommerce-gateway-stripe' );
+			if ($charge) {
+				$reason = (isset($captured) && 'yes' === $captured) ? __('Refunded via Stripe Dashboard', 'woocommerce-gateway-stripe') : __('Pre-Authorization Released via Stripe Dashboard', 'woocommerce-gateway-stripe');
 
 				// Create the refund.
 				$refund = wc_create_refund(
 					array(
 						'order_id' => $order_id,
-						'amount'   => $this->get_refund_amount( $notification ),
+						'amount'   => $this->get_refund_amount($notification),
 						'reason'   => $reason,
 					)
 				);
 
-				if ( is_wp_error( $refund ) ) {
-					WC_Stripe_Logger::log( $refund->get_error_message() );
+				if (is_wp_error($refund)) {
+					WC_Stripe_Logger::log($refund->get_error_message());
 				}
 
-				WC_Stripe_Helper::is_wc_lt( '3.0' ) ? update_post_meta( $order_id, '_stripe_refund_id', $notification->data->object->refunds->data[0]->id ) : $order->update_meta_data( '_stripe_refund_id', $notification->data->object->refunds->data[0]->id );
+				WC_Stripe_Helper::is_wc_lt('3.0') ? update_post_meta($order_id, '_stripe_refund_id', $notification->data->object->refunds->data[0]->id) : $order->update_meta_data('_stripe_refund_id', $notification->data->object->refunds->data[0]->id);
 
-				$amount = wc_price( $notification->data->object->refunds->data[0]->amount / 100 );
+				$amount = wc_price($notification->data->object->refunds->data[0]->amount / 100);
 
-				if ( in_array( strtolower( WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->get_order_currency() : $order->get_currency() ), WC_Stripe_Helper::no_decimal_currencies() ) ) {
-					$amount = wc_price( $notification->data->object->refunds->data[0]->amount );
+				if (in_array(strtolower(WC_Stripe_Helper::is_wc_lt('3.0') ? $order->get_order_currency() : $order->get_currency()), WC_Stripe_Helper::no_decimal_currencies())) {
+					$amount = wc_price($notification->data->object->refunds->data[0]->amount);
 				}
 
-				if ( isset( $notification->data->object->refunds->data[0]->balance_transaction ) ) {
-					$this->update_fees( $order, $notification->data->object->refunds->data[0]->balance_transaction );
+				if (isset($notification->data->object->refunds->data[0]->balance_transaction)) {
+					$this->update_fees($order, $notification->data->object->refunds->data[0]->balance_transaction);
 				}
 
 				/* translators: 1) dollar amount 2) transaction id 3) refund message */
-				$refund_message = ( isset( $captured ) && 'yes' === $captured ) ? sprintf( __( 'Refunded %1$s - Refund ID: %2$s - %3$s', 'woocommerce-gateway-stripe' ), $amount, $notification->data->object->refunds->data[0]->id, $reason ) : __( 'Pre-Authorization Released via Stripe Dashboard', 'woocommerce-gateway-stripe' );
+				$refund_message = (isset($captured) && 'yes' === $captured) ? sprintf(__('Refunded %1$s - Refund ID: %2$s - %3$s', 'woocommerce-gateway-stripe'), $amount, $notification->data->object->refunds->data[0]->id, $reason) : __('Pre-Authorization Released via Stripe Dashboard', 'woocommerce-gateway-stripe');
 
-				$order->add_order_note( $refund_message );
+				$order->add_order_note($refund_message);
 			}
 		}
 	}
@@ -534,30 +534,30 @@ 
 	 * @since 4.0.6
 	 * @param object $notification
 	 */
-	public function process_review_opened( $notification ) {
-		if ( isset( $notification->data->object->payment_intent ) ) {
-			$order = WC_Stripe_Helper::get_order_by_intent_id( $notification->data->object->payment_intent );
+	public function process_review_opened($notification) {
+		if (isset($notification->data->object->payment_intent)) {
+			$order = WC_Stripe_Helper::get_order_by_intent_id($notification->data->object->payment_intent);
 
-			if ( ! $order ) {
-				WC_Stripe_Logger::log( '[Review Opened] Could not find order via intent ID: ' . $notification->data->object->payment_intent );
+			if ( ! $order) {
+				WC_Stripe_Logger::log('[Review Opened] Could not find order via intent ID: ' . $notification->data->object->payment_intent);
 				return;
 			}
 		} else {
-			$order = WC_Stripe_Helper::get_order_by_charge_id( $notification->data->object->charge );
+			$order = WC_Stripe_Helper::get_order_by_charge_id($notification->data->object->charge);
 
-			if ( ! $order ) {
-				WC_Stripe_Logger::log( '[Review Opened] Could not find order via charge ID: ' . $notification->data->object->charge );
+			if ( ! $order) {
+				WC_Stripe_Logger::log('[Review Opened] Could not find order via charge ID: ' . $notification->data->object->charge);
 				return;
 			}
 		}
 
 		/* translators: 1) The URL to the order. 2) The reason type. */
-		$message = sprintf( __( 'A review has been opened for this order. Action is needed. Please go to your <a href="%1$s" title="Stripe Dashboard" target="_blank">Stripe Dashboard</a> to review the issue. Reason: (%2$s)', 'woocommerce-gateway-stripe' ), $this->get_transaction_url( $order ), $notification->data->object->reason );
+		$message = sprintf(__('A review has been opened for this order. Action is needed. Please go to your <a href="%1$s" title="Stripe Dashboard" target="_blank">Stripe Dashboard</a> to review the issue. Reason: (%2$s)', 'woocommerce-gateway-stripe'), $this->get_transaction_url($order), $notification->data->object->reason);
 
-		if ( apply_filters( 'wc_stripe_webhook_review_change_order_status', true, $order, $notification ) ) {
-			$order->update_status( 'on-hold', $message );
+		if (apply_filters('wc_stripe_webhook_review_change_order_status', true, $order, $notification)) {
+			$order->update_status('on-hold', $message);
 		} else {
-			$order->add_order_note( $message );
+			$order->add_order_note($message);
 		}
 	}
 
@@ -567,34 +567,34 @@ 
 	 * @since 4.0.6
 	 * @param object $notification
 	 */
-	public function process_review_closed( $notification ) {
-		if ( isset( $notification->data->object->payment_intent ) ) {
-			$order = WC_Stripe_Helper::get_order_by_intent_id( $notification->data->object->payment_intent );
+	public function process_review_closed($notification) {
+		if (isset($notification->data->object->payment_intent)) {
+			$order = WC_Stripe_Helper::get_order_by_intent_id($notification->data->object->payment_intent);
 
-			if ( ! $order ) {
-				WC_Stripe_Logger::log( '[Review Closed] Could not find order via intent ID: ' . $notification->data->object->payment_intent );
+			if ( ! $order) {
+				WC_Stripe_Logger::log('[Review Closed] Could not find order via intent ID: ' . $notification->data->object->payment_intent);
 				return;
 			}
 		} else {
-			$order = WC_Stripe_Helper::get_order_by_charge_id( $notification->data->object->charge );
+			$order = WC_Stripe_Helper::get_order_by_charge_id($notification->data->object->charge);
 
-			if ( ! $order ) {
-				WC_Stripe_Logger::log( '[Review Closed] Could not find order via charge ID: ' . $notification->data->object->charge );
+			if ( ! $order) {
+				WC_Stripe_Logger::log('[Review Closed] Could not find order via charge ID: ' . $notification->data->object->charge);
 				return;
 			}
 		}
 
 		/* translators: 1) The reason type. */
-		$message = sprintf( __( 'The opened review for this order is now closed. Reason: (%s)', 'woocommerce-gateway-stripe' ), $notification->data->object->reason );
+		$message = sprintf(__('The opened review for this order is now closed. Reason: (%s)', 'woocommerce-gateway-stripe'), $notification->data->object->reason);
 
-		if ( 'on-hold' === $order->get_status() ) {
-			if ( apply_filters( 'wc_stripe_webhook_review_change_order_status', true, $order, $notification ) ) {
-				$order->update_status( 'processing', $message );
+		if ('on-hold' === $order->get_status()) {
+			if (apply_filters('wc_stripe_webhook_review_change_order_status', true, $order, $notification)) {
+				$order->update_status('processing', $message);
 			} else {
-				$order->add_order_note( $message );
+				$order->add_order_note($message);
 			}
 		} else {
-			$order->add_order_note( $message );
+			$order->add_order_note($message);
 		}
 	}
 
@@ -605,7 +605,7 @@ 
 	 * @version 4.0.0
 	 * @param object $notification
 	 */
-	public function is_partial_capture( $notification ) {
+	public function is_partial_capture($notification) {
 		return 0 < $notification->data->object->amount_refunded;
 	}
 
@@ -616,11 +616,11 @@ 
 	 * @version 4.0.0
 	 * @param object $notification
 	 */
-	public function get_refund_amount( $notification ) {
-		if ( $this->is_partial_capture( $notification ) ) {
+	public function get_refund_amount($notification) {
+		if ($this->is_partial_capture($notification)) {
 			$amount = $notification->data->object->refunds->data[0]->amount / 100;
 
-			if ( in_array( strtolower( $notification->data->object->currency ), WC_Stripe_Helper::no_decimal_currencies() ) ) {
+			if (in_array(strtolower($notification->data->object->currency), WC_Stripe_Helper::no_decimal_currencies())) {
 				$amount = $notification->data->object->refunds->data[0]->amount;
 			}
 
@@ -637,12 +637,12 @@ 
 	 * @version 4.0.0
 	 * @param object $notification
 	 */
-	public function get_partial_amount_to_charge( $notification ) {
-		if ( $this->is_partial_capture( $notification ) ) {
-			$amount = ( $notification->data->object->amount - $notification->data->object->amount_refunded ) / 100;
+	public function get_partial_amount_to_charge($notification) {
+		if ($this->is_partial_capture($notification)) {
+			$amount = ($notification->data->object->amount - $notification->data->object->amount_refunded) / 100;
 
-			if ( in_array( strtolower( $notification->data->object->currency ), WC_Stripe_Helper::no_decimal_currencies() ) ) {
-				$amount = ( $notification->data->object->amount - $notification->data->object->amount_refunded );
+			if (in_array(strtolower($notification->data->object->currency), WC_Stripe_Helper::no_decimal_currencies())) {
+				$amount = ($notification->data->object->amount - $notification->data->object->amount_refunded);
 			}
 
 			return $amount;
@@ -651,69 +651,69 @@ 
 		return false;
 	}
 
-	public function process_payment_intent_success( $notification ) {
+	public function process_payment_intent_success($notification) {
 		$intent = $notification->data->object;
-		$order = WC_Stripe_Helper::get_order_by_intent_id( $intent->id );
+		$order = WC_Stripe_Helper::get_order_by_intent_id($intent->id);
 
-		if ( ! $order ) {
-			WC_Stripe_Logger::log( 'Could not find order via intent ID: ' . $intent->id );
+		if ( ! $order) {
+			WC_Stripe_Logger::log('Could not find order via intent ID: ' . $intent->id);
 			return;
 		}
 
-		if ( 'pending' !== $order->get_status() && 'failed' !== $order->get_status() ) {
+		if ('pending' !== $order->get_status() && 'failed' !== $order->get_status()) {
 			return;
 		}
 
-		if ( $this->lock_order_payment( $order, $intent ) ) {
+		if ($this->lock_order_payment($order, $intent)) {
 			return;
 		}
 
-		$order_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
-		if ( 'payment_intent.succeeded' === $notification->type || 'payment_intent.amount_capturable_updated' === $notification->type ) {
-			$charge = end( $intent->charges->data );
-			WC_Stripe_Logger::log( "Stripe PaymentIntent $intent->id succeeded for order $order_id" );
+		$order_id = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
+		if ('payment_intent.succeeded' === $notification->type || 'payment_intent.amount_capturable_updated' === $notification->type) {
+			$charge = end($intent->charges->data);
+			WC_Stripe_Logger::log("Stripe PaymentIntent $intent->id succeeded for order $order_id");
 
-			do_action( 'wc_gateway_stripe_process_payment', $charge, $order );
+			do_action('wc_gateway_stripe_process_payment', $charge, $order);
 
 			// Process valid response.
-			$this->process_response( $charge, $order );
+			$this->process_response($charge, $order);
 
 		} else {
 			$error_message = $intent->last_payment_error ? $intent->last_payment_error->message : "";
 
 			/* translators: 1) The error message that was received from Stripe. */
-			$order->update_status( 'failed', sprintf( __( 'Stripe SCA authentication failed. Reason: %s', 'woocommerce-gateway-stripe' ), $error_message ) );
+			$order->update_status('failed', sprintf(__('Stripe SCA authentication failed. Reason: %s', 'woocommerce-gateway-stripe'), $error_message));
 
-			do_action( 'wc_gateway_stripe_process_webhook_payment_error', $order, $notification );
+			do_action('wc_gateway_stripe_process_webhook_payment_error', $order, $notification);
 
-			$this->send_failed_order_email( $order_id );
+			$this->send_failed_order_email($order_id);
 		}
 
-		$this->unlock_order_payment( $order );
+		$this->unlock_order_payment($order);
 	}
 
-	public function process_setup_intent( $notification ) {
+	public function process_setup_intent($notification) {
 		$intent = $notification->data->object;
-		$order = WC_Stripe_Helper::get_order_by_setup_intent_id( $intent->id );
+		$order = WC_Stripe_Helper::get_order_by_setup_intent_id($intent->id);
 
-		if ( ! $order ) {
-			WC_Stripe_Logger::log( 'Could not find order via setup intent ID: ' . $intent->id );
+		if ( ! $order) {
+			WC_Stripe_Logger::log('Could not find order via setup intent ID: ' . $intent->id);
 			return;
 		}
 
-		if ( 'pending' !== $order->get_status() && 'failed' !== $order->get_status() ) {
+		if ('pending' !== $order->get_status() && 'failed' !== $order->get_status()) {
 			return;
 		}
 
-		if ( $this->lock_order_payment( $order, $intent ) ) {
+		if ($this->lock_order_payment($order, $intent)) {
 			return;
 		}
 
-		$order_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
-		if ( 'setup_intent.succeeded' === $notification->type ) {
-			WC_Stripe_Logger::log( "Stripe SetupIntent $intent->id succeeded for order $order_id" );
-			if ( WC_Stripe_Helper::is_pre_orders_exists() && WC_Pre_Orders_Order::order_contains_pre_order( $order ) ) {
-				WC_Pre_Orders_Order::mark_order_as_pre_ordered( $order );
+		$order_id = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
+		if ('setup_intent.succeeded' === $notification->type) {
+			WC_Stripe_Logger::log("Stripe SetupIntent $intent->id succeeded for order $order_id");
+			if (WC_Stripe_Helper::is_pre_orders_exists() && WC_Pre_Orders_Order::order_contains_pre_order($order)) {
+				WC_Pre_Orders_Order::mark_order_as_pre_ordered($order);
 			} else {
 				$order->payment_complete();
 			}
@@ -721,12 +721,12 @@ 
 			$error_message = $intent->last_setup_error ? $intent->last_setup_error->message : "";
 
 			/* translators: 1) The error message that was received from Stripe. */
-			$order->update_status( 'failed', sprintf( __( 'Stripe SCA authentication failed. Reason: %s', 'woocommerce-gateway-stripe' ), $error_message ) );
+			$order->update_status('failed', sprintf(__('Stripe SCA authentication failed. Reason: %s', 'woocommerce-gateway-stripe'), $error_message));
 
-			$this->send_failed_order_email( $order_id );
+			$this->send_failed_order_email($order_id);
 		}
 
-		$this->unlock_order_payment( $order );
+		$this->unlock_order_payment($order);
 	}
 
 	/**
@@ -736,54 +736,54 @@ 
 	 * @version 4.0.0
 	 * @param string $request_body
 	 */
-	public function process_webhook( $request_body ) {
-		$notification = json_decode( $request_body );
+	public function process_webhook($request_body) {
+		$notification = json_decode($request_body);
 
-		switch ( $notification->type ) {
+		switch ($notification->type) {
 			case 'source.chargeable':
-				$this->process_webhook_payment( $notification );
+				$this->process_webhook_payment($notification);
 				break;
 
 			case 'source.canceled':
-				$this->process_webhook_source_canceled( $notification );
+				$this->process_webhook_source_canceled($notification);
 				break;
 
 			case 'charge.succeeded':
-				$this->process_webhook_charge_succeeded( $notification );
+				$this->process_webhook_charge_succeeded($notification);
 				break;
 
 			case 'charge.failed':
-				$this->process_webhook_charge_failed( $notification );
+				$this->process_webhook_charge_failed($notification);
 				break;
 
 			case 'charge.captured':
-				$this->process_webhook_capture( $notification );
+				$this->process_webhook_capture($notification);
 				break;
 
 			case 'charge.dispute.created':
-				$this->process_webhook_dispute( $notification );
+				$this->process_webhook_dispute($notification);
 				break;
 
 			case 'charge.refunded':
-				$this->process_webhook_refund( $notification );
+				$this->process_webhook_refund($notification);
 				break;
 
 			case 'review.opened':
-				$this->process_review_opened( $notification );
+				$this->process_review_opened($notification);
 				break;
 
 			case 'review.closed':
-				$this->process_review_closed( $notification );
+				$this->process_review_closed($notification);
 				break;
 
 			case 'payment_intent.succeeded':
 			case 'payment_intent.payment_failed':
 			case 'payment_intent.amount_capturable_updated':
-				$this->process_payment_intent_success( $notification );
+				$this->process_payment_intent_success($notification);
 
 			case 'setup_intent.succeeded':
 			case 'setup_intent.setup_failed':
-				$this->process_setup_intent( $notification );
+				$this->process_setup_intent($notification);
 
 		}
 	}

tests/phpunit/test-wc-stripe-sub-renewal.php

Spacing +78 added lines, -78 removed lines

@@ -34,17 +34,17 @@ 
 	public function setUp() {
 		parent::setUp();
 
-		$this->wc_stripe_subs_compat = $this->getMockBuilder( 'WC_Stripe_Subs_Compat' )
+		$this->wc_stripe_subs_compat = $this->getMockBuilder('WC_Stripe_Subs_Compat')
 			->disableOriginalConstructor()
-			->setMethods( array( 'prepare_order_source', 'has_subscription' ) )
+			->setMethods(array('prepare_order_source', 'has_subscription'))
 			->getMock();
 
 		// Mocked in order to get metadata[payment_type] = recurring in the HTTP request.
 		$this->wc_stripe_subs_compat
-			->expects( $this->any() )
-			->method( 'has_subscription' )
+			->expects($this->any())
+			->method('has_subscription')
 			->will(
-				$this->returnValue( true )
+				$this->returnValue(true)
 			);
 
 		$this->statement_descriptor = 'This is a statement descriptor.';
@@ -62,7 +62,7 @@ 
 	public function tearDown() {
 		parent::tearDown();
 
-		delete_option( 'woocommerce_stripe_settings' );
+		delete_option('woocommerce_stripe_settings');
 	}
 
 	/**
@@ -79,11 +79,11 @@ 
 		// Arrange: Some variables we'll use later.
 		$renewal_order                 = WC_Helper_Order::create_order();
 		$amount                        = 20; // WC Subs sends an amount to be used, instead of using the order amount.
-		$stripe_amount                 = WC_Stripe_Helper::get_stripe_amount( $amount );
-		$currency                      = strtolower( WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $renewal_order->get_order_currency() : $renewal_order->get_currency() );
+		$stripe_amount                 = WC_Stripe_Helper::get_stripe_amount($amount);
+		$currency                      = strtolower(WC_Stripe_Helper::is_wc_lt('3.0') ? $renewal_order->get_order_currency() : $renewal_order->get_currency());
 		$customer                      = 'cus_123abc';
 		$source                        = 'src_123abc';
-		$statement_descriptor          = WC_Stripe_Helper::clean_statement_descriptor( $this->statement_descriptor );
+		$statement_descriptor          = WC_Stripe_Helper::clean_statement_descriptor($this->statement_descriptor);
 		$should_retry                  = false;
 		$previous_error                = false;
 		$payments_intents_api_endpoint = 'https://api.stripe.com/v1/payment_intents';
@@ -91,16 +91,16 @@ 
 
 		// Arrange: Set payment method to stripe, and not stripe_sepa, for example.
 		// This needed for testing the statement_descriptor.
-		if ( WC_Stripe_Helper::is_wc_lt( '3.0' ) ) {
+		if (WC_Stripe_Helper::is_wc_lt('3.0')) {
 			$renewal_order->payment_method = 'stripe';
 		} else {
-			$renewal_order->set_payment_method( 'stripe' );
+			$renewal_order->set_payment_method('stripe');
 		}
 
 		// Arrange: Mock prepare_order_source() so that we have a customer and source.
 		$this->wc_stripe_subs_compat
-			->expects( $this->any() )
-			->method( 'prepare_order_source' )
+			->expects($this->any())
+			->method('prepare_order_source')
 			->will(
 				$this->returnValue(
 					(object) array(
@@ -114,7 +114,7 @@ 
 
 		// Arrange: Add filter that will return a mocked HTTP response for the payment_intent call.
 		// Note: There are assertions in the callback function.
-		$pre_http_request_response_callback = function( $preempt, $request_args, $url ) use (
+		$pre_http_request_response_callback = function($preempt, $request_args, $url) use (
 			$renewal_order,
 			$stripe_amount,
 			$currency,
@@ -125,35 +125,35 @@ 
 			&$urls_used
 		) {
 			// Add all urls to array so we can later make assertions about which endpoints were used.
-			array_push( $urls_used, $url );
+			array_push($urls_used, $url);
 
 			// Continue without mocking the request if it's not the endpoint we care about.
-			if ( $payments_intents_api_endpoint !== $url ) {
+			if ($payments_intents_api_endpoint !== $url) {
 				return false;
 			}
 
 			// Assert: the request method is POST.
-			$this->assertArrayHasKey( 'method', $request_args );
-			$this->assertSame( 'POST', $request_args['method'] );
+			$this->assertArrayHasKey('method', $request_args);
+			$this->assertSame('POST', $request_args['method']);
 
 			// Assert: the request has a body.
-			$this->assertArrayHasKey( 'body', $request_args );
+			$this->assertArrayHasKey('body', $request_args);
 
 			// Assert: the request body contains these values.
 			$expected_request_body_values = array(
 				'source'               => $source,
 				'amount'               => $stripe_amount,
 				'currency'             => $currency,
-				'payment_method_types' => array( 'card' ),
+				'payment_method_types' => array('card'),
 				'customer'             => $customer,
 				'off_session'          => 'true',
 				'confirm'              => 'true',
 				'confirmation_method'  => 'automatic',
 				'statement_descriptor' => $statement_descriptor,
 			);
-			foreach ( $expected_request_body_values as $key => $value ) {
-				$this->assertArrayHasKey( $key, $request_args['body'] );
-				$this->assertSame( $value, $request_args['body'][ $key ] );
+			foreach ($expected_request_body_values as $key => $value) {
+				$this->assertArrayHasKey($key, $request_args['body']);
+				$this->assertSame($value, $request_args['body'][$key]);
 			}
 
 			// Assert: the request body contains these keys, without checking for their value.
@@ -161,19 +161,19 @@ 
 				'description',
 				'metadata',
 			);
-			foreach ( $expected_request_body_keys as $key ) {
-				$this->assertArrayHasKey( $key, $request_args['body'] );
+			foreach ($expected_request_body_keys as $key) {
+				$this->assertArrayHasKey($key, $request_args['body']);
 			}
 
 			// Assert: the body metadata has these values.
-			$order_id                 = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $renewal_order->id : (string) $renewal_order->get_id();
+			$order_id                 = WC_Stripe_Helper::is_wc_lt('3.0') ? $renewal_order->id : (string) $renewal_order->get_id();
 			$expected_metadata_values = array(
 				'order_id'     => $order_id,
 				'payment_type' => 'recurring',
 			);
-			foreach ( $expected_metadata_values as $key => $value ) {
-				$this->assertArrayHasKey( $key, $request_args['body']['metadata'] );
-				$this->assertSame( $value, $request_args['body']['metadata'][ $key ] );
+			foreach ($expected_metadata_values as $key => $value) {
+				$this->assertArrayHasKey($key, $request_args['body']['metadata']);
+				$this->assertSame($value, $request_args['body']['metadata'][$key]);
 			}
 
 			// Assert: the body metadata has these keys, without checking for their value.
@@ -182,8 +182,8 @@ 
 				'customer_email',
 				'site_url',
 			);
-			foreach ( $expected_metadata_keys as $key ) {
-				$this->assertArrayHasKey( $key, $request_args['body']['metadata'] );
+			foreach ($expected_metadata_keys as $key) {
+				$this->assertArrayHasKey($key, $request_args['body']['metadata']);
 			}
 
 			// Assert: the request body does not contains these keys.
@@ -192,15 +192,15 @@ 
 				'capture_method', // The default ('automatic') is what we want in this case, so we leave it off.
 				'expand[]',
 			);
-			foreach ( $expected_missing_request_body_keys as $key ) {
-				$this->assertArrayNotHasKey( $key, $request_args['body'] );
+			foreach ($expected_missing_request_body_keys as $key) {
+				$this->assertArrayNotHasKey($key, $request_args['body']);
 			}
 
 			// Arrange: return dummy content as the response.
 			return array(
 				'headers'  => array(),
 				// Too bad we aren't dynamically setting things 'cus_123abc' when using this file.
-				'body'     => file_get_contents( 'tests/phpunit/dummy-data/subscription_renewal_response_success.json' ),
+				'body'     => file_get_contents('tests/phpunit/dummy-data/subscription_renewal_response_success.json'),
 				'response' => array(
 					'code'    => 200,
 					'message' => 'OK',
@@ -210,55 +210,55 @@ 
 			);
 		};
 
-		add_filter( 'pre_http_request', $pre_http_request_response_callback, 10, 3 );
+		add_filter('pre_http_request', $pre_http_request_response_callback, 10, 3);
 
 		// Arrange: Make sure to check that an action we care about was called
 		// by hooking into it.
 		$mock_action_process_payment = new MockAction();
 		add_action(
 			'wc_gateway_stripe_process_payment',
-			[ &$mock_action_process_payment, 'action' ]
+			[&$mock_action_process_payment, 'action']
 		);
 
 		// Act: call process_subscription_payment().
 		// We need to use `wc_stripe_subs_compat` here because we mocked this class earlier.
-		$result = $this->wc_stripe_subs_compat->process_subscription_payment( 20, $renewal_order, $should_retry, $previous_error );
+		$result = $this->wc_stripe_subs_compat->process_subscription_payment(20, $renewal_order, $should_retry, $previous_error);
 
 		// Assert: nothing was returned.
-		$this->assertEquals( $result, null );
+		$this->assertEquals($result, null);
 
 		// Assert that we saved the payment intent to the order.
-		$order_id   = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $renewal_order->id : $renewal_order->get_id();
-		$order      = wc_get_order( $order_id );
+		$order_id   = WC_Stripe_Helper::is_wc_lt('3.0') ? $renewal_order->id : $renewal_order->get_id();
+		$order      = wc_get_order($order_id);
 		$order_data = (
-			WC_Stripe_Helper::is_wc_lt( '3.0' )
-				? get_post_meta( $order_id, '_stripe_intent_id', true )
-				: $order->get_meta( '_stripe_intent_id' )
+			WC_Stripe_Helper::is_wc_lt('3.0')
+				? get_post_meta($order_id, '_stripe_intent_id', true)
+				: $order->get_meta('_stripe_intent_id')
 		);
-		$this->assertEquals( $order_data, 'pi_123abc' );
+		$this->assertEquals($order_data, 'pi_123abc');
 
 		// Transaction ID was saved to order.
 		$order_transaction_id = (
-			WC_Stripe_Helper::is_wc_lt( '3.0' )
-				? get_post_meta( $order_id, '_transaction_id', true )
+			WC_Stripe_Helper::is_wc_lt('3.0')
+				? get_post_meta($order_id, '_transaction_id', true)
 				: $order->get_transaction_id()
 		);
-		$this->assertEquals( $order_transaction_id, 'ch_123abc' );
+		$this->assertEquals($order_transaction_id, 'ch_123abc');
 
 		// Assert: the order was marked as processing (this is done in process_response()).
-		$this->assertEquals( $order->get_status(), 'processing' );
+		$this->assertEquals($order->get_status(), 'processing');
 
 		// Assert: called payment intents.
-		$this->assertTrue( in_array( $payments_intents_api_endpoint, $urls_used ) );
+		$this->assertTrue(in_array($payments_intents_api_endpoint, $urls_used));
 
 		// Assert: Our hook was called once.
-		$this->assertEquals( 1, $mock_action_process_payment->get_call_count() );
+		$this->assertEquals(1, $mock_action_process_payment->get_call_count());
 
 		// Assert: Only our hook was called.
-		$this->assertEquals( array( 'wc_gateway_stripe_process_payment' ), $mock_action_process_payment->get_tags() );
+		$this->assertEquals(array('wc_gateway_stripe_process_payment'), $mock_action_process_payment->get_tags());
 
 		// Clean up.
-		remove_filter( 'pre_http_request', array( $this, 'pre_http_request_response_success' ) );
+		remove_filter('pre_http_request', array($this, 'pre_http_request_response_success'));
 	}
 
 	/**
@@ -275,8 +275,8 @@ 
 		// Arrange: Some variables we'll use later.
 		$renewal_order                 = WC_Helper_Order::create_order();
 		$amount                        = 20;
-		$stripe_amount                 = WC_Stripe_Helper::get_stripe_amount( $amount );
-		$currency                      = strtolower( WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $renewal_order->get_order_currency() : $renewal_order->get_currency() );
+		$stripe_amount                 = WC_Stripe_Helper::get_stripe_amount($amount);
+		$currency                      = strtolower(WC_Stripe_Helper::is_wc_lt('3.0') ? $renewal_order->get_order_currency() : $renewal_order->get_currency());
 		$customer                      = 'cus_123abc';
 		$source                        = 'src_123abc';
 		$should_retry                  = false;
@@ -286,8 +286,8 @@ 
 
 		// Arrange: Mock prepare_order_source() so that we have a customer and source.
 		$this->wc_stripe_subs_compat
-			->expects( $this->any() )
-			->method( 'prepare_order_source' )
+			->expects($this->any())
+			->method('prepare_order_source')
 			->will(
 				$this->returnValue(
 					(object) array(
@@ -300,7 +300,7 @@ 
 			);
 
 		// Arrange: Add filter that will return a mocked HTTP response for the payment_intent call.
-		$pre_http_request_response_callback = function( $preempt, $request_args, $url ) use (
+		$pre_http_request_response_callback = function($preempt, $request_args, $url) use (
 			$renewal_order,
 			$stripe_amount,
 			$currency,
@@ -310,10 +310,10 @@ 
 			&$urls_used
 		) {
 			// Add all urls to array so we can later make assertions about which endpoints were used.
-			array_push( $urls_used, $url );
+			array_push($urls_used, $url);
 
 			// Continue without mocking the request if it's not the endpoint we care about.
-			if ( $payments_intents_api_endpoint !== $url ) {
+			if ($payments_intents_api_endpoint !== $url) {
 				return false;
 			}
 
@@ -321,7 +321,7 @@ 
 			return array(
 				'headers'  => array(),
 				// Too bad we aren't dynamically setting things 'cus_123abc' when using this file.
-				'body'     => file_get_contents( 'tests/phpunit/dummy-data/subscription_renewal_response_authentication_required.json' ),
+				'body'     => file_get_contents('tests/phpunit/dummy-data/subscription_renewal_response_authentication_required.json'),
 				'response' => array(
 					'code'    => 402,
 					'message' => 'Payment Required',
@@ -330,56 +330,56 @@ 
 				'filename' => null,
 			);
 		};
-		add_filter( 'pre_http_request', $pre_http_request_response_callback, 10, 3 );
+		add_filter('pre_http_request', $pre_http_request_response_callback, 10, 3);
 
 		// Arrange: Make sure to check that an action we care about was called
 		// by hooking into it.
 		$mock_action_process_payment = new MockAction();
 		add_action(
 			'wc_gateway_stripe_process_payment_authentication_required',
-			[ &$mock_action_process_payment, 'action' ]
+			[&$mock_action_process_payment, 'action']
 		);
 
 		// Act: call process_subscription_payment().
 		// We need to use `wc_stripe_subs_compat` here because we mocked this class earlier.
-		$result = $this->wc_stripe_subs_compat->process_subscription_payment( 20, $renewal_order, $should_retry, $previous_error );
+		$result = $this->wc_stripe_subs_compat->process_subscription_payment(20, $renewal_order, $should_retry, $previous_error);
 
 		// Assert: nothing was returned.
-		$this->assertEquals( $result, null );
+		$this->assertEquals($result, null);
 
 		// Assert that we saved the payment intent to the order.
-		$order_id             = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $renewal_order->id : $renewal_order->get_id();
-		$order                = wc_get_order( $order_id );
+		$order_id             = WC_Stripe_Helper::is_wc_lt('3.0') ? $renewal_order->id : $renewal_order->get_id();
+		$order                = wc_get_order($order_id);
 		$order_data           = (
-			WC_Stripe_Helper::is_wc_lt( '3.0' )
-				? get_post_meta( $order_id, '_stripe_intent_id', true )
-				: $order->get_meta( '_stripe_intent_id' )
+			WC_Stripe_Helper::is_wc_lt('3.0')
+				? get_post_meta($order_id, '_stripe_intent_id', true)
+				: $order->get_meta('_stripe_intent_id')
 		);
 		$order_transaction_id = (
-			WC_Stripe_Helper::is_wc_lt( '3.0' )
-				? get_post_meta( $order_id, '_transaction_id', true )
+			WC_Stripe_Helper::is_wc_lt('3.0')
+				? get_post_meta($order_id, '_transaction_id', true)
 				: $order->get_transaction_id()
 		);
 
 		// Intent was saved to order even though there was an error in the response body.
-		$this->assertEquals( $order_data, 'pi_123abc' );
+		$this->assertEquals($order_data, 'pi_123abc');
 
 		// Transaction ID was saved to order.
-		$this->assertEquals( $order_transaction_id, 'ch_123abc' );
+		$this->assertEquals($order_transaction_id, 'ch_123abc');
 
 		// Assert: the order was marked as failed.
-		$this->assertEquals( $order->get_status(), 'failed' );
+		$this->assertEquals($order->get_status(), 'failed');
 
 		// Assert: called payment intents.
-		$this->assertTrue( in_array( $payments_intents_api_endpoint, $urls_used ) );
+		$this->assertTrue(in_array($payments_intents_api_endpoint, $urls_used));
 
 		// Assert: Our hook was called once.
-		$this->assertEquals( 1, $mock_action_process_payment->get_call_count() );
+		$this->assertEquals(1, $mock_action_process_payment->get_call_count());
 
 		// Assert: Only our hook was called.
-		$this->assertEquals( array( 'wc_gateway_stripe_process_payment_authentication_required' ), $mock_action_process_payment->get_tags() );
+		$this->assertEquals(array('wc_gateway_stripe_process_payment_authentication_required'), $mock_action_process_payment->get_tags());
 
 		// Clean up.
-		remove_filter( 'pre_http_request', array( $this, 'pre_http_request_response_success' ) );
+		remove_filter('pre_http_request', array($this, 'pre_http_request_response_success'));
 	}
 }

templates/emails/plain/failed-renewal-authentication.php

Spacing +4 added lines, -4 removed lines

@@ -1,17 +1,17 @@
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
 echo $email_heading . "\n\n";
 
 // translators: %1$s: name of the blog, %2$s: link to checkout payment url, note: no full stop due to url at the end
-printf( esc_html_x( 'The automatic payment to renew your subscription with %1$s has failed. To reactivate the subscription, please login and authorize the renewal from your account page: %2$s', 'In failed renewal authentication email', 'woocommerce-gateway-stripe' ), esc_html( get_bloginfo( 'name' ) ), esc_attr( $authorization_url ) );
+printf(esc_html_x('The automatic payment to renew your subscription with %1$s has failed. To reactivate the subscription, please login and authorize the renewal from your account page: %2$s', 'In failed renewal authentication email', 'woocommerce-gateway-stripe'), esc_html(get_bloginfo('name')), esc_attr($authorization_url));
 
 echo "\n\n=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=\n";
 
-do_action( 'woocommerce_subscriptions_email_order_details', $order, $sent_to_admin, $plain_text, $email );
+do_action('woocommerce_subscriptions_email_order_details', $order, $sent_to_admin, $plain_text, $email);
 
 echo "\n=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=\n\n";
 
-echo apply_filters( 'woocommerce_email_footer_text', get_option( 'woocommerce_email_footer_text' ) );
+echo apply_filters('woocommerce_email_footer_text', get_option('woocommerce_email_footer_text'));

templates/emails/failed-renewal-authentication.php

Spacing +5 added lines, -5 removed lines

@@ -1,17 +1,17 @@
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 ?>
 
-<?php do_action( 'woocommerce_email_header', $email_heading, $email ); ?>
+<?php do_action('woocommerce_email_header', $email_heading, $email); ?>
 
 <p>
 	<?php
 	// translators: %1$s: name of the blog, %2$s: link to payment re-authentication URL, note: no full stop due to url at the end
-	echo wp_kses( sprintf( _x( 'The automatic payment to renew your subscription with %1$s has failed. To reactivate the subscription, please login and authorize the renewal from your account page: %2$s', 'In failed renewal authentication email', 'woocommerce-gateway-stripe' ), esc_html( get_bloginfo( 'name' ) ), '<a href="' . esc_url( $authorization_url ) . '">' . esc_html__( 'Authorize the payment &raquo;', 'woocommerce-gateway-stripe' ) . '</a>' ), array( 'a' => array( 'href' => true ) ) ); ?>
+	echo wp_kses(sprintf(_x('The automatic payment to renew your subscription with %1$s has failed. To reactivate the subscription, please login and authorize the renewal from your account page: %2$s', 'In failed renewal authentication email', 'woocommerce-gateway-stripe'), esc_html(get_bloginfo('name')), '<a href="' . esc_url($authorization_url) . '">' . esc_html__('Authorize the payment &raquo;', 'woocommerce-gateway-stripe') . '</a>'), array('a' => array('href' => true))); ?>
 </p>
 
-<?php do_action( 'woocommerce_subscriptions_email_order_details', $order, $sent_to_admin, $plain_text, $email ); ?>
+<?php do_action('woocommerce_subscriptions_email_order_details', $order, $sent_to_admin, $plain_text, $email); ?>
 
-<?php do_action( 'woocommerce_email_footer', $email ); ?>
+<?php do_action('woocommerce_email_footer', $email); ?>

includes/compat/class-wc-stripe-email-failed-authentication.php

Spacing +14 added lines, -14 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly.
 }
 
@@ -30,7 +30,7 @@ 
 				'email_heading'     => $this->get_heading(),
 				'sent_to_admin'     => false,
 				'plain_text'        => false,
-				'authorization_url' => $this->get_authorization_url( $this->object ),
+				'authorization_url' => $this->get_authorization_url($this->object),
 				'email'             => $this,
 			),
 			'',
@@ -53,7 +53,7 @@ 
 				'email_heading'     => $this->get_heading(),
 				'sent_to_admin'     => false,
 				'plain_text'        => true,
-				'authorization_url' => $this->get_authorization_url( $this->object ),
+				'authorization_url' => $this->get_authorization_url($this->object),
 				'email'             => $this,
 			),
 			'',
@@ -68,8 +68,8 @@ 
 	 * @param WC_Order $order The order whose payment needs authentication.
 	 * @return string
 	 */
-	public function get_authorization_url( $order ) {
-		return add_query_arg( 'wc-stripe-confirmation', 1, $order->get_checkout_payment_url( false ) );
+	public function get_authorization_url($order) {
+		return add_query_arg('wc-stripe-confirmation', 1, $order->get_checkout_payment_url(false));
 	}
 
 	/**
@@ -81,9 +81,9 @@ 
 
 		$this->form_fields = array(
 			'enabled'    => array(
-				'title'   => _x( 'Enable/Disable', 'an email notification', 'woocommerce-gateway-stripe' ),
+				'title'   => _x('Enable/Disable', 'an email notification', 'woocommerce-gateway-stripe'),
 				'type'    => 'checkbox',
-				'label'   => __( 'Enable this email notification', 'woocommerce-gateway-stripe' ),
+				'label'   => __('Enable this email notification', 'woocommerce-gateway-stripe'),
 				'default' => 'yes',
 			),
 
@@ -98,29 +98,29 @@ 
 	 *
 	 * @param WC_Order $order The renewal order whose payment failed.
 	 */
-	public function trigger( $order ) {
-		if ( ! $this->is_enabled() ) {
+	public function trigger($order) {
+		if ( ! $this->is_enabled()) {
 			return;
 		}
 
 		$this->object = $order;
 
-		if ( method_exists( $order, 'get_billing_email' ) ) {
+		if (method_exists($order, 'get_billing_email')) {
 			$this->recipient = $order->get_billing_email();
 		} else {
 			$this->recipient = $order->billing_email;
 		}
 
 		$this->find['order_date'] = '{order_date}';
-		if ( function_exists( 'wc_format_datetime' ) ) { // WC 3.0+
-			$this->replace['order_date'] = wc_format_datetime( $order->get_date_created() );
+		if (function_exists('wc_format_datetime')) { // WC 3.0+
+			$this->replace['order_date'] = wc_format_datetime($order->get_date_created());
 		} else { // WC < 3.0
-			$this->replace['order_date'] = $order->date_created->date_i18n( wc_date_format() );
+			$this->replace['order_date'] = $order->date_created->date_i18n(wc_date_format());
 		}
 
 		$this->find['order_number']    = '{order_number}';
 		$this->replace['order_number'] = $order->get_order_number();
 
-		$this->send( $this->get_recipient(), $this->get_subject(), $this->get_content(), $this->get_headers(), $this->get_attachments() );
+		$this->send($this->get_recipient(), $this->get_subject(), $this->get_content(), $this->get_headers(), $this->get_attachments());
 	}
 }

includes/compat/class-wc-stripe-email-failed-renewal-authentication.php

Spacing +18 added lines, -18 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly.
 }
 
@@ -14,20 +14,20 @@ 
 	 *
 	 * @param WC_Email[] $email_classes All existing instances of WooCommerce emails.
 	 */
-	public function __construct( $email_classes = array() ) {
+	public function __construct($email_classes = array()) {
 		$this->id             = 'failed_renewal_authentication';
-		$this->title          = __( 'Failed Subscription Renewal SCA Authentication', 'woocommerce-gateway-stripe' );
-		$this->description    = __( 'Sent to a customer when a renewal fails because the transaction requires an SCA verification. The email contains renewal order information and payment links.', 'woocommerce-gateway-stripe' );
+		$this->title          = __('Failed Subscription Renewal SCA Authentication', 'woocommerce-gateway-stripe');
+		$this->description    = __('Sent to a customer when a renewal fails because the transaction requires an SCA verification. The email contains renewal order information and payment links.', 'woocommerce-gateway-stripe');
 		$this->customer_email = true;
 
 		$this->template_html  = 'emails/failed-renewal-authentication.php';
 		$this->template_plain = 'emails/plain/failed-renewal-authentication.php';
-		$this->template_base  = plugin_dir_path( WC_STRIPE_MAIN_FILE ) . 'templates/';
+		$this->template_base  = plugin_dir_path(WC_STRIPE_MAIN_FILE) . 'templates/';
 
 		// Triggers the email at the correct hook.
-		add_action( 'wc_gateway_stripe_process_payment_authentication_required', array( $this, 'trigger' ) );
+		add_action('wc_gateway_stripe_process_payment_authentication_required', array($this, 'trigger'));
 
-		if ( isset( $email_classes['WCS_Email_Customer_Renewal_Invoice'] ) ) {
+		if (isset($email_classes['WCS_Email_Customer_Renewal_Invoice'])) {
 			$this->original_email = $email_classes['WCS_Email_Customer_Renewal_Invoice'];
 		}
 
@@ -40,18 +40,18 @@ 
 	 *
 	 * @param WC_Order $order The order that is being paid.
 	 */
-	public function trigger( $order ) {
-		if ( function_exists( 'wcs_order_contains_subscription' ) && ( wcs_order_contains_subscription( $order->get_id() ) || wcs_is_subscription( $order->get_id() ) || wcs_order_contains_renewal( $order->get_id() ) ) ) {
-			parent::trigger( $order );
+	public function trigger($order) {
+		if (function_exists('wcs_order_contains_subscription') && (wcs_order_contains_subscription($order->get_id()) || wcs_is_subscription($order->get_id()) || wcs_order_contains_renewal($order->get_id()))) {
+			parent::trigger($order);
 
 			// Prevent the renewal email from WooCommerce Subscriptions from being sent.
-			if ( isset( $this->original_email ) ) {
-				remove_action( 'woocommerce_generated_manual_renewal_order_renewal_notification', array( $this->original_email, 'trigger' ) );
-				remove_action( 'woocommerce_order_status_failed_renewal_notification', array( $this->original_email, 'trigger' ) );
+			if (isset($this->original_email)) {
+				remove_action('woocommerce_generated_manual_renewal_order_renewal_notification', array($this->original_email, 'trigger'));
+				remove_action('woocommerce_order_status_failed_renewal_notification', array($this->original_email, 'trigger'));
 			}
 
 			// Prevent the retry email from WooCommerce Subscriptions from being sent.
-			add_filter( 'wcs_get_retry_rule_raw', array( $this, 'prevent_retry_notification_email' ), 100, 3 );
+			add_filter('wcs_get_retry_rule_raw', array($this, 'prevent_retry_notification_email'), 100, 3);
 		}
 	}
 
@@ -61,7 +61,7 @@ 
 	 * @return string
 	 */
 	public function get_default_subject() {
-		return __( 'Payment authorization needed for renewal of {site_title} order {order_number}', 'woocommerce-gateway-stripe' );
+		return __('Payment authorization needed for renewal of {site_title} order {order_number}', 'woocommerce-gateway-stripe');
 	}
 
 	/**
@@ -70,7 +70,7 @@ 
 	 * @return string
 	 */
 	public function get_default_heading() {
-		return __( 'Payment authorization needed for renewal of order {order_number}', 'woocommerce-gateway-stripe' );
+		return __('Payment authorization needed for renewal of order {order_number}', 'woocommerce-gateway-stripe');
 	}
 
 	/**
@@ -81,8 +81,8 @@ 
 	 * @param int   $order_id     The ID of the order that needs payment.
 	 * @return array
 	 */
-	public function prevent_retry_notification_email( $rule_array, $retry_number, $order_id ) {
-		if ( wcs_get_objects_property( $this->object, 'id' ) === $order_id ) {
+	public function prevent_retry_notification_email($rule_array, $retry_number, $order_id) {
+		if (wcs_get_objects_property($this->object, 'id') === $order_id) {
 			$rule_array['email_template_customer'] = '';
 		}