woocommerce / woocommerce-gateway-stripe

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'));
 	}
 }

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/admin/class-wc-stripe-admin-notices.php

Spacing +100 added lines, -100 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,9 +21,9 @@ 
 	 * @since 4.1.0
 	 */
 	public function __construct() {
-		add_action( 'admin_notices', array( $this, 'admin_notices' ) );
-		add_action( 'wp_loaded', array( $this, 'hide_notices' ) );
-		add_action( 'woocommerce_stripe_updated', array( $this, 'stripe_updated' ) );
+		add_action('admin_notices', array($this, 'admin_notices'));
+		add_action('wp_loaded', array($this, 'hide_notices'));
+		add_action('woocommerce_stripe_updated', array($this, 'stripe_updated'));
 	}
 
 	/**
@@ -32,8 +32,8 @@ 
 	 * @since 1.0.0
 	 * @version 4.0.0
 	 */
-	public function add_admin_notice( $slug, $class, $message, $dismissible = false ) {
-		$this->notices[ $slug ] = array(
+	public function add_admin_notice($slug, $class, $message, $dismissible = false) {
+		$this->notices[$slug] = array(
 			'class'       => $class,
 			'message'     => $message,
 			'dismissible' => $dismissible,
@@ -47,7 +47,7 @@ 
 	 * @version 4.0.0
 	 */
 	public function admin_notices() {
-		if ( ! current_user_can( 'manage_woocommerce' ) ) {
+		if ( ! current_user_can('manage_woocommerce')) {
 			return;
 		}
 
@@ -57,17 +57,17 @@ 
 		// All other payment methods.
 		$this->payment_methods_check_environment();
 
-		foreach ( (array) $this->notices as $notice_key => $notice ) {
-			echo '<div class="' . esc_attr( $notice['class'] ) . '" style="position:relative;">';
+		foreach ((array) $this->notices as $notice_key => $notice) {
+			echo '<div class="' . esc_attr($notice['class']) . '" style="position:relative;">';
 
-			if ( $notice['dismissible'] ) {
+			if ($notice['dismissible']) {
 				?>
-				<a href="<?php echo esc_url( wp_nonce_url( add_query_arg( 'wc-stripe-hide-notice', $notice_key ), 'wc_stripe_hide_notices_nonce', '_wc_stripe_notice_nonce' ) ); ?>" class="woocommerce-message-close notice-dismiss" style="position:relative;float:right;padding:9px 0px 9px 9px 9px;text-decoration:none;"></a>
+				<a href="<?php echo esc_url(wp_nonce_url(add_query_arg('wc-stripe-hide-notice', $notice_key), 'wc_stripe_hide_notices_nonce', '_wc_stripe_notice_nonce')); ?>" class="woocommerce-message-close notice-dismiss" style="position:relative;float:right;padding:9px 0px 9px 9px 9px;text-decoration:none;"></a>
 				<?php
 			}
 
 			echo '<p>';
-			echo wp_kses( $notice['message'], array( 'a' => array( 'href' => array(), 'target' => array() ) ) );
+			echo wp_kses($notice['message'], array('a' => array('href' => array(), 'target' => array())));
 			echo '</p></div>';
 		}
 	}
@@ -100,110 +100,110 @@ 
 	 * @version 4.0.0
 	 */
 	public function stripe_check_environment() {
-		$show_style_notice  = get_option( 'wc_stripe_show_style_notice' );
-		$show_ssl_notice    = get_option( 'wc_stripe_show_ssl_notice' );
-		$show_keys_notice   = get_option( 'wc_stripe_show_keys_notice' );
-		$show_3ds_notice    = get_option( 'wc_stripe_show_3ds_notice' );
-		$show_phpver_notice = get_option( 'wc_stripe_show_phpver_notice' );
-		$show_wcver_notice  = get_option( 'wc_stripe_show_wcver_notice' );
-		$show_curl_notice   = get_option( 'wc_stripe_show_curl_notice' );
-		$show_sca_notice    = get_option( 'wc_stripe_show_sca_notice' );
-		$options            = get_option( 'woocommerce_stripe_settings' );
-		$testmode           = ( isset( $options['testmode'] ) && 'yes' === $options['testmode'] ) ? true : false;
-		$test_pub_key       = isset( $options['test_publishable_key'] ) ? $options['test_publishable_key'] : '';
-		$test_secret_key    = isset( $options['test_secret_key'] ) ? $options['test_secret_key'] : '';
-		$live_pub_key       = isset( $options['publishable_key'] ) ? $options['publishable_key'] : '';
-		$live_secret_key    = isset( $options['secret_key'] ) ? $options['secret_key'] : '';
-		$three_d_secure     = isset( $options['three_d_secure'] ) && 'yes' === $options['three_d_secure'];
-
-		if ( isset( $options['enabled'] ) && 'yes' === $options['enabled'] ) {
-			if ( empty( $show_3ds_notice ) && $three_d_secure ) {
+		$show_style_notice  = get_option('wc_stripe_show_style_notice');
+		$show_ssl_notice    = get_option('wc_stripe_show_ssl_notice');
+		$show_keys_notice   = get_option('wc_stripe_show_keys_notice');
+		$show_3ds_notice    = get_option('wc_stripe_show_3ds_notice');
+		$show_phpver_notice = get_option('wc_stripe_show_phpver_notice');
+		$show_wcver_notice  = get_option('wc_stripe_show_wcver_notice');
+		$show_curl_notice   = get_option('wc_stripe_show_curl_notice');
+		$show_sca_notice    = get_option('wc_stripe_show_sca_notice');
+		$options            = get_option('woocommerce_stripe_settings');
+		$testmode           = (isset($options['testmode']) && 'yes' === $options['testmode']) ? true : false;
+		$test_pub_key       = isset($options['test_publishable_key']) ? $options['test_publishable_key'] : '';
+		$test_secret_key    = isset($options['test_secret_key']) ? $options['test_secret_key'] : '';
+		$live_pub_key       = isset($options['publishable_key']) ? $options['publishable_key'] : '';
+		$live_secret_key    = isset($options['secret_key']) ? $options['secret_key'] : '';
+		$three_d_secure     = isset($options['three_d_secure']) && 'yes' === $options['three_d_secure'];
+
+		if (isset($options['enabled']) && 'yes' === $options['enabled']) {
+			if (empty($show_3ds_notice) && $three_d_secure) {
 				$url = 'https://stripe.com/docs/payments/3d-secure#three-ds-radar';
 
 				/* translators: 1) A URL that explains Stripe Radar. */
-				$message = __( 'WooCommerce Stripe - We see that you had the "Require 3D secure when applicable" setting turned on. This setting is not available here anymore, because it is now replaced by Stripe Radar. You can learn more about it <a href="%s" target="_blank">here</a>.', 'woocommerce-gateway-stripe' );
+				$message = __('WooCommerce Stripe - We see that you had the "Require 3D secure when applicable" setting turned on. This setting is not available here anymore, because it is now replaced by Stripe Radar. You can learn more about it <a href="%s" target="_blank">here</a>.', 'woocommerce-gateway-stripe');
 
-				$this->add_admin_notice( '3ds', 'notice notice-warning', sprintf( $message, $url ), true );
+				$this->add_admin_notice('3ds', 'notice notice-warning', sprintf($message, $url), true);
 			}
 
-			if ( empty( $show_style_notice ) ) {
+			if (empty($show_style_notice)) {
 				/* translators: 1) int version 2) int version */
-				$message = __( 'WooCommerce Stripe - We recently made changes to Stripe that may impact the appearance of your checkout. If your checkout has changed unexpectedly, please follow these <a href="https://docs.woocommerce.com/document/stripe/#styling" target="_blank">instructions</a> to fix.', 'woocommerce-gateway-stripe' );
+				$message = __('WooCommerce Stripe - We recently made changes to Stripe that may impact the appearance of your checkout. If your checkout has changed unexpectedly, please follow these <a href="https://docs.woocommerce.com/document/stripe/#styling" target="_blank">instructions</a> to fix.', 'woocommerce-gateway-stripe');
 
-				$this->add_admin_notice( 'style', 'notice notice-warning', $message, true );
+				$this->add_admin_notice('style', 'notice notice-warning', $message, true);
 
 				return;
 			}
 
-			if ( empty( $show_phpver_notice ) ) {
-				if ( version_compare( phpversion(), WC_STRIPE_MIN_PHP_VER, '<' ) ) {
+			if (empty($show_phpver_notice)) {
+				if (version_compare(phpversion(), WC_STRIPE_MIN_PHP_VER, '<')) {
 					/* translators: 1) int version 2) int version */
-					$message = __( 'WooCommerce Stripe - The minimum PHP version required for this plugin is %1$s. You are running %2$s.', 'woocommerce-gateway-stripe' );
+					$message = __('WooCommerce Stripe - The minimum PHP version required for this plugin is %1$s. You are running %2$s.', 'woocommerce-gateway-stripe');
 
-					$this->add_admin_notice( 'phpver', 'error', sprintf( $message, WC_STRIPE_MIN_PHP_VER, phpversion() ), true );
+					$this->add_admin_notice('phpver', 'error', sprintf($message, WC_STRIPE_MIN_PHP_VER, phpversion()), true);
 
 					return;
 				}
 			}
 
-			if ( empty( $show_wcver_notice ) ) {
-				if ( version_compare( WC_VERSION, WC_STRIPE_MIN_WC_VER, '<' ) ) {
+			if (empty($show_wcver_notice)) {
+				if (version_compare(WC_VERSION, WC_STRIPE_MIN_WC_VER, '<')) {
 					/* translators: 1) int version 2) int version */
-					$message = __( 'WooCommerce Stripe - The minimum WooCommerce version required for this plugin is %1$s. You are running %2$s.', 'woocommerce-gateway-stripe' );
+					$message = __('WooCommerce Stripe - The minimum WooCommerce version required for this plugin is %1$s. You are running %2$s.', 'woocommerce-gateway-stripe');
 
-					$this->add_admin_notice( 'wcver', 'notice notice-warning', sprintf( $message, WC_STRIPE_MIN_WC_VER, WC_VERSION ), true );
+					$this->add_admin_notice('wcver', 'notice notice-warning', sprintf($message, WC_STRIPE_MIN_WC_VER, WC_VERSION), true);
 
 					return;
 				}
 			}
 
-			if ( empty( $show_curl_notice ) ) {
-				if ( ! function_exists( 'curl_init' ) ) {
-					$this->add_admin_notice( 'curl', 'notice notice-warning', __( 'WooCommerce Stripe - cURL is not installed.', 'woocommerce-gateway-stripe' ), true );
+			if (empty($show_curl_notice)) {
+				if ( ! function_exists('curl_init')) {
+					$this->add_admin_notice('curl', 'notice notice-warning', __('WooCommerce Stripe - cURL is not installed.', 'woocommerce-gateway-stripe'), true);
 				}
 			}
 
-			if ( empty( $show_keys_notice ) ) {
+			if (empty($show_keys_notice)) {
 				$secret = WC_Stripe_API::get_secret_key();
 
-				if ( empty( $secret ) && ! ( isset( $_GET['page'], $_GET['section'] ) && 'wc-settings' === $_GET['page'] && 'stripe' === $_GET['section'] ) ) {
+				if (empty($secret) && ! (isset($_GET['page'], $_GET['section']) && 'wc-settings' === $_GET['page'] && 'stripe' === $_GET['section'])) {
 					$setting_link = $this->get_setting_link();
 					/* translators: 1) link */
-					$this->add_admin_notice( 'keys', 'notice notice-warning', sprintf( __( 'Stripe is almost ready. To get started, <a href="%s">set your Stripe account keys</a>.', 'woocommerce-gateway-stripe' ), $setting_link ), true );
+					$this->add_admin_notice('keys', 'notice notice-warning', sprintf(__('Stripe is almost ready. To get started, <a href="%s">set your Stripe account keys</a>.', 'woocommerce-gateway-stripe'), $setting_link), true);
 				}
 
 				// Check if keys are entered properly per live/test mode.
-				if ( $testmode ) {
+				if ($testmode) {
 					if (
-						! empty( $test_pub_key ) && ! preg_match( '/^pk_test_/', $test_pub_key )
-						|| ( ! empty( $test_secret_key ) && ! preg_match( '/^sk_test_/', $test_secret_key )
-						&& ! empty( $test_secret_key ) && ! preg_match( '/^rk_test_/', $test_secret_key ) ) ) {
+						! empty($test_pub_key) && ! preg_match('/^pk_test_/', $test_pub_key)
+						|| ( ! empty($test_secret_key) && ! preg_match('/^sk_test_/', $test_secret_key)
+						&& ! empty($test_secret_key) && ! preg_match('/^rk_test_/', $test_secret_key)) ) {
 						$setting_link = $this->get_setting_link();
 						/* translators: 1) link */
-						$this->add_admin_notice( 'keys', 'notice notice-error', sprintf( __( 'Stripe is in test mode however your test keys may not be valid. Test keys start with pk_test and sk_test or rk_test. Please go to your settings and, <a href="%s">set your Stripe account keys</a>.', 'woocommerce-gateway-stripe' ), $setting_link ), true );
+						$this->add_admin_notice('keys', 'notice notice-error', sprintf(__('Stripe is in test mode however your test keys may not be valid. Test keys start with pk_test and sk_test or rk_test. Please go to your settings and, <a href="%s">set your Stripe account keys</a>.', 'woocommerce-gateway-stripe'), $setting_link), true);
 					}
 				} else {
 					if (
-						! empty( $live_pub_key ) && ! preg_match( '/^pk_live_/', $live_pub_key )
-						|| ( ! empty( $live_secret_key ) && ! preg_match( '/^sk_live_/', $live_secret_key )
-						&& ! empty( $live_secret_key ) && ! preg_match( '/^rk_live_/', $live_secret_key ) ) ) {
+						! empty($live_pub_key) && ! preg_match('/^pk_live_/', $live_pub_key)
+						|| ( ! empty($live_secret_key) && ! preg_match('/^sk_live_/', $live_secret_key)
+						&& ! empty($live_secret_key) && ! preg_match('/^rk_live_/', $live_secret_key)) ) {
 						$setting_link = $this->get_setting_link();
 						/* translators: 1) link */
-						$this->add_admin_notice( 'keys', 'notice notice-error', sprintf( __( 'Stripe is in live mode however your test keys may not be valid. Live keys start with pk_live and sk_live or rk_live. Please go to your settings and, <a href="%s">set your Stripe account keys</a>.', 'woocommerce-gateway-stripe' ), $setting_link ), true );
+						$this->add_admin_notice('keys', 'notice notice-error', sprintf(__('Stripe is in live mode however your test keys may not be valid. Live keys start with pk_live and sk_live or rk_live. Please go to your settings and, <a href="%s">set your Stripe account keys</a>.', 'woocommerce-gateway-stripe'), $setting_link), true);
 					}
 				}
 			}
 
-			if ( empty( $show_ssl_notice ) ) {
+			if (empty($show_ssl_notice)) {
 				// Show message if enabled and FORCE SSL is disabled and WordpressHTTPS plugin is not detected.
-				if ( ! wc_checkout_is_https() ) {
+				if ( ! wc_checkout_is_https()) {
 					/* translators: 1) link */
-					$this->add_admin_notice( 'ssl', 'notice notice-warning', sprintf( __( 'Stripe is enabled, but a SSL certificate is not detected. Your checkout may not be secure! Please ensure your server has a valid <a href="%1$s" target="_blank">SSL certificate</a>', 'woocommerce-gateway-stripe' ), 'https://en.wikipedia.org/wiki/Transport_Layer_Security' ), true );
+					$this->add_admin_notice('ssl', 'notice notice-warning', sprintf(__('Stripe is enabled, but a SSL certificate is not detected. Your checkout may not be secure! Please ensure your server has a valid <a href="%1$s" target="_blank">SSL certificate</a>', 'woocommerce-gateway-stripe'), 'https://en.wikipedia.org/wiki/Transport_Layer_Security'), true);
 				}
 			}
 
-			if ( empty( $show_sca_notice ) ) {
-				$this->add_admin_notice( 'sca', 'notice notice-success', sprintf( __( 'Stripe is now ready for Strong Customer Authentication (SCA) and 3D Secure 2! <a href="%1$s" target="_blank">Read about SCA</a>', 'woocommerce-gateway-stripe' ), 'https://woocommerce.com/posts/introducing-strong-customer-authentication-sca/' ), true );
+			if (empty($show_sca_notice)) {
+				$this->add_admin_notice('sca', 'notice notice-success', sprintf(__('Stripe is now ready for Strong Customer Authentication (SCA) and 3D Secure 2! <a href="%1$s" target="_blank">Read about SCA</a>', 'woocommerce-gateway-stripe'), 'https://woocommerce.com/posts/introducing-strong-customer-authentication-sca/'), true);
 			}
 		}
 	}
@@ -216,17 +216,17 @@ 
 	public function payment_methods_check_environment() {
 		$payment_methods = $this->get_payment_methods();
 
-		foreach ( $payment_methods as $method => $class ) {
-			$show_notice = get_option( 'wc_stripe_show_' . strtolower( $method ) . '_notice' );
+		foreach ($payment_methods as $method => $class) {
+			$show_notice = get_option('wc_stripe_show_' . strtolower($method) . '_notice');
 			$gateway     = new $class();
 
-			if ( 'yes' !== $gateway->enabled || 'no' === $show_notice ) {
+			if ('yes' !== $gateway->enabled || 'no' === $show_notice) {
 				continue;
 			}
 
-			if ( ! in_array( get_woocommerce_currency(), $gateway->get_supported_currency() ) ) {
+			if ( ! in_array(get_woocommerce_currency(), $gateway->get_supported_currency())) {
 				/* translators: %1$s Payment method, %2$s List of supported currencies */
-				$this->add_admin_notice( $method, 'notice notice-error', sprintf( __( '%1$s is enabled - it requires store currency to be set to %2$s', 'woocommerce-gateway-stripe' ), $method, implode( ', ', $gateway->get_supported_currency() ) ), true );
+				$this->add_admin_notice($method, 'notice notice-error', sprintf(__('%1$s is enabled - it requires store currency to be set to %2$s', 'woocommerce-gateway-stripe'), $method, implode(', ', $gateway->get_supported_currency())), true);
 			}
 		}
 	}
@@ -238,68 +238,68 @@ 
 	 * @version 4.0.0
 	 */
 	public function hide_notices() {
-		if ( isset( $_GET['wc-stripe-hide-notice'] ) && isset( $_GET['_wc_stripe_notice_nonce'] ) ) {
-			if ( ! wp_verify_nonce( $_GET['_wc_stripe_notice_nonce'], 'wc_stripe_hide_notices_nonce' ) ) {
-				wp_die( __( 'Action failed. Please refresh the page and retry.', 'woocommerce-gateway-stripe' ) );
+		if (isset($_GET['wc-stripe-hide-notice']) && isset($_GET['_wc_stripe_notice_nonce'])) {
+			if ( ! wp_verify_nonce($_GET['_wc_stripe_notice_nonce'], 'wc_stripe_hide_notices_nonce')) {
+				wp_die(__('Action failed. Please refresh the page and retry.', 'woocommerce-gateway-stripe'));
 			}
 
-			if ( ! current_user_can( 'manage_woocommerce' ) ) {
-				wp_die( __( 'Cheatin&#8217; huh?', 'woocommerce-gateway-stripe' ) );
+			if ( ! current_user_can('manage_woocommerce')) {
+				wp_die(__('Cheatin&#8217; huh?', 'woocommerce-gateway-stripe'));
 			}
 
-			$notice = wc_clean( $_GET['wc-stripe-hide-notice'] );
+			$notice = wc_clean($_GET['wc-stripe-hide-notice']);
 
-			switch ( $notice ) {
+			switch ($notice) {
 				case 'style':
-					update_option( 'wc_stripe_show_style_notice', 'no' );
+					update_option('wc_stripe_show_style_notice', 'no');
 					break;
 				case 'phpver':
-					update_option( 'wc_stripe_show_phpver_notice', 'no' );
+					update_option('wc_stripe_show_phpver_notice', 'no');
 					break;
 				case 'wcver':
-					update_option( 'wc_stripe_show_wcver_notice', 'no' );
+					update_option('wc_stripe_show_wcver_notice', 'no');
 					break;
 				case 'curl':
-					update_option( 'wc_stripe_show_curl_notice', 'no' );
+					update_option('wc_stripe_show_curl_notice', 'no');
 					break;
 				case 'ssl':
-					update_option( 'wc_stripe_show_ssl_notice', 'no' );
+					update_option('wc_stripe_show_ssl_notice', 'no');
 					break;
 				case 'keys':
-					update_option( 'wc_stripe_show_keys_notice', 'no' );
+					update_option('wc_stripe_show_keys_notice', 'no');
 					break;
 				case '3ds':
-					update_option( 'wc_stripe_show_3ds_notice', 'no' );
+					update_option('wc_stripe_show_3ds_notice', 'no');
 					break;
 				case 'Alipay':
-					update_option( 'wc_stripe_show_alipay_notice', 'no' );
+					update_option('wc_stripe_show_alipay_notice', 'no');
 					break;
 				case 'Bancontact':
-					update_option( 'wc_stripe_show_bancontact_notice', 'no' );
+					update_option('wc_stripe_show_bancontact_notice', 'no');
 					break;
 				case 'EPS':
-					update_option( 'wc_stripe_show_eps_notice', 'no' );
+					update_option('wc_stripe_show_eps_notice', 'no');
 					break;
 				case 'Giropay':
-					update_option( 'wc_stripe_show_giropay_notice', 'no' );
+					update_option('wc_stripe_show_giropay_notice', 'no');
 					break;
 				case 'iDeal':
-					update_option( 'wc_stripe_show_ideal_notice', 'no' );
+					update_option('wc_stripe_show_ideal_notice', 'no');
 					break;
 				case 'Multibanco':
-					update_option( 'wc_stripe_show_multibanco_notice', 'no' );
+					update_option('wc_stripe_show_multibanco_notice', 'no');
 					break;
 				case 'P24':
-					update_option( 'wc_stripe_show_p24_notice', 'no' );
+					update_option('wc_stripe_show_p24_notice', 'no');
 					break;
 				case 'SEPA':
-					update_option( 'wc_stripe_show_sepa_notice', 'no' );
+					update_option('wc_stripe_show_sepa_notice', 'no');
 					break;
 				case 'SOFORT':
-					update_option( 'wc_stripe_show_sofort_notice', 'no' );
+					update_option('wc_stripe_show_sofort_notice', 'no');
 					break;
 				case 'sca':
-					update_option( 'wc_stripe_show_sca_notice', 'no' );
+					update_option('wc_stripe_show_sca_notice', 'no');
 					break;
 			}
 		}
@@ -313,11 +313,11 @@ 
 	 * @return string Setting link
 	 */
 	public function get_setting_link() {
-		$use_id_as_section = function_exists( 'WC' ) ? version_compare( WC()->version, '2.6', '>=' ) : false;
+		$use_id_as_section = function_exists('WC') ? version_compare(WC()->version, '2.6', '>=') : false;
 
-		$section_slug = $use_id_as_section ? 'stripe' : strtolower( 'WC_Gateway_Stripe' );
+		$section_slug = $use_id_as_section ? 'stripe' : strtolower('WC_Gateway_Stripe');
 
-		return admin_url( 'admin.php?page=wc-settings&tab=checkout&section=' . $section_slug );
+		return admin_url('admin.php?page=wc-settings&tab=checkout&section=' . $section_slug);
 	}
 
 	/**
@@ -326,16 +326,16 @@ 
 	 * @since 4.3.0
 	 */
 	public function stripe_updated() {
-		$previous_version = get_option( 'wc_stripe_version' );
+		$previous_version = get_option('wc_stripe_version');
 
 		// Only show the style notice if the plugin was installed and older than 4.1.4.
-		if ( empty( $previous_version ) || version_compare( $previous_version, '4.1.4', 'ge' ) ) {
-			update_option( 'wc_stripe_show_style_notice', 'no' );
+		if (empty($previous_version) || version_compare($previous_version, '4.1.4', 'ge')) {
+			update_option('wc_stripe_show_style_notice', 'no');
 		}
 
 		// Only show the SCA notice on pre-4.3.0 installs.
-		if ( empty( $previous_version ) || version_compare( $previous_version, '4.3.0', 'ge' ) ) {
-			update_option( 'wc_stripe_show_sca_notice', 'no' );
+		if (empty($previous_version) || version_compare($previous_version, '4.3.0', 'ge')) {
+			update_option('wc_stripe_show_sca_notice', 'no');
 		}
 	}
 }

includes/compat/class-wc-stripe-email-failed-preorder-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.
 }
 
@@ -21,20 +21,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_preorder_sca_authentication';
-		$this->title          = __( 'Pre-order Payment Action Needed', 'woocommerce-gateway-stripe' );
-		$this->description    = __( 'This is an order notification sent to the customer once a pre-order is complete, but additional payment steps are required.', 'woocommerce-gateway-stripe' );
+		$this->title          = __('Pre-order Payment Action Needed', 'woocommerce-gateway-stripe');
+		$this->description    = __('This is an order notification sent to the customer once a pre-order is complete, but additional payment steps are required.', 'woocommerce-gateway-stripe');
 		$this->customer_email = true;
 
 		$this->template_html  = 'emails/failed-preorder-authentication.php';
 		$this->template_plain = 'emails/plain/failed-preorder-authentication.php';
-		$this->template_base  = plugin_dir_path( WC_STRIPE_MAIN_FILE ) . 'templates/';
+		$this->template_base  = plugin_dir_path(WC_STRIPE_MAIN_FILE) . 'templates/';
 
 		// Use the "authentication required" hook to add the correct, later 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['WC_Pre_Orders_Email_Pre_Order_Available'] ) ) {
+		if (isset($email_classes['WC_Pre_Orders_Email_Pre_Order_Available'])) {
 			$this->original_email = $email_classes['WC_Pre_Orders_Email_Pre_Order_Available'];
 		}
 
@@ -48,13 +48,13 @@ 
 	 *
 	 * @param WC_Order $order The order whose payment is failing.
 	 */
-	public function trigger( $order ) {
-		if ( class_exists( 'WC_Pre_Orders_Order' ) && WC_Pre_Orders_Order::order_contains_pre_order( $order->get_id() ) ) {
-			if ( isset( $this->original_email ) ) {
-				remove_action( 'wc_pre_order_status_completed_notification', array( $this->original_email, 'trigger' ), 10, 2 );
+	public function trigger($order) {
+		if (class_exists('WC_Pre_Orders_Order') && WC_Pre_Orders_Order::order_contains_pre_order($order->get_id())) {
+			if (isset($this->original_email)) {
+				remove_action('wc_pre_order_status_completed_notification', array($this->original_email, 'trigger'), 10, 2);
 			}
 
-			add_action( 'wc_pre_orders_pre_order_completed', array( $this, 'send_email' ), 10, 2 );
+			add_action('wc_pre_orders_pre_order_completed', array($this, 'send_email'), 10, 2);
 		}
 	}
 
@@ -64,14 +64,14 @@ 
 	 * @param WC_Order $order The order that is being paid.
 	 * @param string   $message The message, which should be added to the email.
 	 */
-	public function send_email( $order, $message ) {
+	public function send_email($order, $message) {
 		$this->custom_message = $message;
 
-		parent::trigger( $order );
+		parent::trigger($order);
 
 		// Restore the action of the original email for other bulk actions.
-		if ( isset( $this->original_email ) ) {
-			add_action( 'wc_pre_order_status_completed_notification', array( $this->original_email, 'trigger' ), 10, 2 );
+		if (isset($this->original_email)) {
+			add_action('wc_pre_order_status_completed_notification', array($this->original_email, 'trigger'), 10, 2);
 		}
 	}
 
@@ -81,7 +81,7 @@ 
 	 * @return string
 	 */
 	public function get_default_subject() {
-		return __( 'Payment authorization needed for pre-order {order_number}', 'woocommerce-gateway-stripe' );
+		return __('Payment authorization needed for pre-order {order_number}', 'woocommerce-gateway-stripe');
 	}
 
 	/**
@@ -90,7 +90,7 @@ 
 	 * @return string
 	 */
 	public function get_default_heading() {
-		return __( 'Payment authorization needed for pre-order {order_number}', 'woocommerce-gateway-stripe' );
+		return __('Payment authorization needed for pre-order {order_number}', 'woocommerce-gateway-stripe');
 	}
 
 	/**

templates/emails/failed-preorder-authentication.php

Spacing +15 added lines, -15 removed lines

@@ -1,13 +1,13 @@ 
 <?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); ?>
 
 <?php
-$pre_wc_30 = version_compare( WC_VERSION, '3.0', '<' );
+$pre_wc_30 = version_compare(WC_VERSION, '3.0', '<');
 $billing_email = $pre_wc_30 ? $order->billing_email : $order->get_billing_email();
 $billing_phone = $pre_wc_30 ? $order->billing_phone : $order->get_billing_phone();
 
@@ -16,19 +16,19 @@ 
 	echo wp_kses(
 		sprintf(
 			// translators: %s is a link to the payment re-authentication URL.
-			_x( 'Your pre-order is now available, but payment cannot be completed automatically. %s', 'In failed SCA authentication for a pre-order.', 'woocommerce-gateway-stripe' ),
-			'<a href="' . esc_url( $authorization_url ) . '">' . esc_html__( 'Authorize the payment now &raquo;', 'woocommerce-gateway-stripe' ) . '</a>'
+			_x('Your pre-order is now available, but payment cannot be completed automatically. %s', 'In failed SCA authentication for a pre-order.', 'woocommerce-gateway-stripe'),
+			'<a href="' . esc_url($authorization_url) . '">' . esc_html__('Authorize the payment now &raquo;', 'woocommerce-gateway-stripe') . '</a>'
 		),
-		array( 'a' => array( 'href' => true ) )
+		array('a' => array('href' => true))
 	);
 ?></p>
 
-<?php if ( $email->get_custom_message() ) : ?>
-	<blockquote><?php echo wpautop( wptexturize( $email->get_custom_message() ) ); ?></blockquote>
+<?php if ($email->get_custom_message()) : ?>
+	<blockquote><?php echo wpautop(wptexturize($email->get_custom_message())); ?></blockquote>
 <?php endif; ?>
 
 <?php
-do_action( 'woocommerce_email_before_order_table', $order, false, $plain_text, $email );
+do_action('woocommerce_email_before_order_table', $order, false, $plain_text, $email);
 
 /*
  * @hooked WC_Emails::order_details() Shows the order details table.
@@ -36,28 +36,28 @@ 
  * @hooked WC_Structured_Data::output_structured_data() Outputs structured data.
  * @since 2.5.0
  */
-do_action( 'woocommerce_email_order_details', $order, $sent_to_admin, $plain_text, $email );
+do_action('woocommerce_email_order_details', $order, $sent_to_admin, $plain_text, $email);
 
 /*
  * @hooked WC_Emails::order_meta() Shows order meta data.
  */
-do_action( 'woocommerce_email_order_meta', $order, $sent_to_admin, $plain_text, $email );
+do_action('woocommerce_email_order_meta', $order, $sent_to_admin, $plain_text, $email);
 
-do_action( 'woocommerce_email_after_order_table', $order, false, $plain_text, $email );
+do_action('woocommerce_email_after_order_table', $order, false, $plain_text, $email);
 
 /*
  * @hooked WC_Emails::customer_details() Shows customer details
  * @hooked WC_Emails::email_address() Shows email address
  */
-do_action( 'woocommerce_email_customer_details', $order, $sent_to_admin, $plain_text, $email );
+do_action('woocommerce_email_customer_details', $order, $sent_to_admin, $plain_text, $email);
 
 ?>
 <p>
-<?php esc_html_e( 'Thanks for shopping with us.', 'woocommerce-gateway-stripe' ); ?>
+<?php esc_html_e('Thanks for shopping with us.', 'woocommerce-gateway-stripe'); ?>
 </p>
 <?php
 
 /*
  * @hooked WC_Emails::email_footer() Output the email footer
  */
-do_action( 'woocommerce_email_footer', $email );
+do_action('woocommerce_email_footer', $email);