woocommerce / woocommerce-gateway-stripe

includes/class-wc-stripe-payment-tokens.php

Spacing +63 added lines, -63 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,11 +21,11 @@ 
 	public function __construct() {
 		self::$_this = $this;
 
-		add_filter( 'woocommerce_get_customer_payment_tokens', array( $this, 'woocommerce_get_customer_payment_tokens' ), 10, 3 );
-		add_filter( 'woocommerce_payment_methods_list_item', array( $this, 'get_account_saved_payment_methods_list_item_sepa' ), 10, 2 );
-		add_filter( 'woocommerce_get_credit_card_type_label', array( $this, 'normalize_sepa_label' ) );
-		add_action( 'woocommerce_payment_token_deleted', array( $this, 'woocommerce_payment_token_deleted' ), 10, 2 );
-		add_action( 'woocommerce_payment_token_set_default', array( $this, 'woocommerce_payment_token_set_default' ) );
+		add_filter('woocommerce_get_customer_payment_tokens', array($this, 'woocommerce_get_customer_payment_tokens'), 10, 3);
+		add_filter('woocommerce_payment_methods_list_item', array($this, 'get_account_saved_payment_methods_list_item_sepa'), 10, 2);
+		add_filter('woocommerce_get_credit_card_type_label', array($this, 'normalize_sepa_label'));
+		add_action('woocommerce_payment_token_deleted', array($this, 'woocommerce_payment_token_deleted'), 10, 2);
+		add_action('woocommerce_payment_token_set_default', array($this, 'woocommerce_payment_token_set_default'));
 	}
 
 	/**
@@ -46,8 +46,8 @@ 
 	 * @param string $label
 	 * @return string $label
 	 */
-	public function normalize_sepa_label( $label ) {
-		if ( 'sepa iban' === strtolower( $label ) ) {
+	public function normalize_sepa_label($label) {
+		if ('sepa iban' === strtolower($label)) {
 			return 'SEPA IBAN';
 		}
 
@@ -61,19 +61,19 @@ 
 	 * @param int $customer_id
 	 * @return bool
 	 */
-	public static function customer_has_saved_methods( $customer_id ) {
-		$gateways = array( 'stripe', 'stripe_sepa' );
+	public static function customer_has_saved_methods($customer_id) {
+		$gateways = array('stripe', 'stripe_sepa');
 
-		if ( empty( $customer_id ) ) {
+		if (empty($customer_id)) {
 			return false;
 		}
 
 		$has_token = false;
 
-		foreach ( $gateways as $gateway ) {
-			$tokens = WC_Payment_Tokens::get_customer_tokens( $customer_id, $gateway );
+		foreach ($gateways as $gateway) {
+			$tokens = WC_Payment_Tokens::get_customer_tokens($customer_id, $gateway);
 
-			if ( ! empty( $tokens ) ) {
+			if ( ! empty($tokens)) {
 				$has_token = true;
 				break;
 			}
@@ -90,67 +90,67 @@ 
 	 * @param array $tokens
 	 * @return array
 	 */
-	public function woocommerce_get_customer_payment_tokens( $tokens, $customer_id, $gateway_id ) {
-		if ( is_user_logged_in() && class_exists( 'WC_Payment_Token_CC' ) ) {
+	public function woocommerce_get_customer_payment_tokens($tokens, $customer_id, $gateway_id) {
+		if (is_user_logged_in() && class_exists('WC_Payment_Token_CC')) {
 			$stored_tokens = array();
 
-			foreach ( $tokens as $token ) {
+			foreach ($tokens as $token) {
 				$stored_tokens[] = $token->get_token();
 			}
 
-			if ( 'stripe' === $gateway_id ) {
-				$stripe_customer = new WC_Stripe_Customer( $customer_id );
+			if ('stripe' === $gateway_id) {
+				$stripe_customer = new WC_Stripe_Customer($customer_id);
 				$stripe_sources  = $stripe_customer->get_sources();
 
-				foreach ( $stripe_sources as $source ) {
-					if ( isset( $source->type ) && 'card' === $source->type ) {
-						if ( ! in_array( $source->id, $stored_tokens ) ) {
+				foreach ($stripe_sources as $source) {
+					if (isset($source->type) && 'card' === $source->type) {
+						if ( ! in_array($source->id, $stored_tokens)) {
 							$token = new WC_Payment_Token_CC();
-							$token->set_token( $source->id );
-							$token->set_gateway_id( 'stripe' );
-
-							if ( 'source' === $source->object && 'card' === $source->type ) {
-								$token->set_card_type( strtolower( $source->card->brand ) );
-								$token->set_last4( $source->card->last4 );
-								$token->set_expiry_month( $source->card->exp_month );
-								$token->set_expiry_year( $source->card->exp_year );
+							$token->set_token($source->id);
+							$token->set_gateway_id('stripe');
+
+							if ('source' === $source->object && 'card' === $source->type) {
+								$token->set_card_type(strtolower($source->card->brand));
+								$token->set_last4($source->card->last4);
+								$token->set_expiry_month($source->card->exp_month);
+								$token->set_expiry_year($source->card->exp_year);
 							}
 
-							$token->set_user_id( $customer_id );
+							$token->set_user_id($customer_id);
 							$token->save();
-							$tokens[ $token->get_id() ] = $token;
+							$tokens[$token->get_id()] = $token;
 						}
 					} else {
-						if ( ! in_array( $source->id, $stored_tokens ) && 'card' === $source->object ) {
+						if ( ! in_array($source->id, $stored_tokens) && 'card' === $source->object) {
 							$token = new WC_Payment_Token_CC();
-							$token->set_token( $source->id );
-							$token->set_gateway_id( 'stripe' );
-							$token->set_card_type( strtolower( $source->brand ) );
-							$token->set_last4( $source->last4 );
-							$token->set_expiry_month( $source->exp_month );
-							$token->set_expiry_year( $source->exp_year );
-							$token->set_user_id( $customer_id );
+							$token->set_token($source->id);
+							$token->set_gateway_id('stripe');
+							$token->set_card_type(strtolower($source->brand));
+							$token->set_last4($source->last4);
+							$token->set_expiry_month($source->exp_month);
+							$token->set_expiry_year($source->exp_year);
+							$token->set_user_id($customer_id);
 							$token->save();
-							$tokens[ $token->get_id() ] = $token;
+							$tokens[$token->get_id()] = $token;
 						}
 					}
 				}
 			}
 
-			if ( 'stripe_sepa' === $gateway_id ) {
-				$stripe_customer = new WC_Stripe_Customer( $customer_id );
+			if ('stripe_sepa' === $gateway_id) {
+				$stripe_customer = new WC_Stripe_Customer($customer_id);
 				$stripe_sources  = $stripe_customer->get_sources();
 
-				foreach ( $stripe_sources as $source ) {
-					if ( isset( $source->type ) && 'sepa_debit' === $source->type ) {
-						if ( ! in_array( $source->id, $stored_tokens ) ) {
+				foreach ($stripe_sources as $source) {
+					if (isset($source->type) && 'sepa_debit' === $source->type) {
+						if ( ! in_array($source->id, $stored_tokens)) {
 							$token = new WC_Payment_Token_SEPA();
-							$token->set_token( $source->id );
-							$token->set_gateway_id( 'stripe_sepa' );
-							$token->set_last4( $source->sepa_debit->last4 );
-							$token->set_user_id( $customer_id );
+							$token->set_token($source->id);
+							$token->set_gateway_id('stripe_sepa');
+							$token->set_last4($source->sepa_debit->last4);
+							$token->set_user_id($customer_id);
 							$token->save();
-							$tokens[ $token->get_id() ] = $token;
+							$tokens[$token->get_id()] = $token;
 						}
 					}
 				}
@@ -169,10 +169,10 @@ 
 	 * @param  WC_Payment_Token $payment_token The payment token associated with this method entry
 	 * @return array                           Filtered item
 	 */
-	public function get_account_saved_payment_methods_list_item_sepa( $item, $payment_token ) {
-		if ( 'sepa' === strtolower( $payment_token->get_type() ) ) {
+	public function get_account_saved_payment_methods_list_item_sepa($item, $payment_token) {
+		if ('sepa' === strtolower($payment_token->get_type())) {
 			$item['method']['last4'] = $payment_token->get_last4();
-			$item['method']['brand'] = esc_html__( 'SEPA IBAN', 'woocommerce-gateway-stripe' );
+			$item['method']['brand'] = esc_html__('SEPA IBAN', 'woocommerce-gateway-stripe');
 		}
 
 		return $item;
@@ -184,10 +184,10 @@ 
 	 * @since 3.1.0
 	 * @version 4.0.0
 	 */
-	public function woocommerce_payment_token_deleted( $token_id, $token ) {
-		if ( 'stripe' === $token->get_gateway_id() || 'stripe_sepa' === $token->get_gateway_id() ) {
-			$stripe_customer = new WC_Stripe_Customer( get_current_user_id() );
-			$stripe_customer->delete_source( $token->get_token() );
+	public function woocommerce_payment_token_deleted($token_id, $token) {
+		if ('stripe' === $token->get_gateway_id() || 'stripe_sepa' === $token->get_gateway_id()) {
+			$stripe_customer = new WC_Stripe_Customer(get_current_user_id());
+			$stripe_customer->delete_source($token->get_token());
 		}
 	}
 
@@ -197,12 +197,12 @@ 
 	 * @since 3.1.0
 	 * @version 4.0.0
 	 */
-	public function woocommerce_payment_token_set_default( $token_id ) {
-		$token = WC_Payment_Tokens::get( $token_id );
+	public function woocommerce_payment_token_set_default($token_id) {
+		$token = WC_Payment_Tokens::get($token_id);
 
-		if ( 'stripe' === $token->get_gateway_id() || 'stripe_sepa' === $token->get_gateway_id() ) {
-			$stripe_customer = new WC_Stripe_Customer( get_current_user_id() );
-			$stripe_customer->set_default_source( $token->get_token() );
+		if ('stripe' === $token->get_gateway_id() || 'stripe_sepa' === $token->get_gateway_id()) {
+			$stripe_customer = new WC_Stripe_Customer(get_current_user_id());
+			$stripe_customer->set_default_source($token->get_token());
 		}
 	}
 }

includes/compat/class-wc-stripe-subs-compat.php

Spacing +203 added lines, -203 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -15,21 +15,21 @@ 
 	public function __construct() {
 		parent::__construct();
 
-		if ( class_exists( 'WC_Subscriptions_Order' ) ) {
-			add_action( 'woocommerce_scheduled_subscription_payment_' . $this->id, array( $this, 'scheduled_subscription_payment' ), 10, 2 );
-			add_action( 'wcs_resubscribe_order_created', array( $this, 'delete_resubscribe_meta' ), 10 );
-			add_action( 'wcs_renewal_order_created', array( $this, 'delete_renewal_meta' ), 10 );
-			add_action( 'woocommerce_subscription_failing_payment_method_updated_stripe', array( $this, 'update_failing_payment_method' ), 10, 2 );
-			add_action( 'wc_stripe_cards_payment_fields', array( $this, 'display_update_subs_payment_checkout' ) );
-			add_action( 'wc_stripe_add_payment_method_' . $this->id . '_success', array( $this, 'handle_add_payment_method_success' ), 10, 2 );
+		if (class_exists('WC_Subscriptions_Order')) {
+			add_action('woocommerce_scheduled_subscription_payment_' . $this->id, array($this, 'scheduled_subscription_payment'), 10, 2);
+			add_action('wcs_resubscribe_order_created', array($this, 'delete_resubscribe_meta'), 10);
+			add_action('wcs_renewal_order_created', array($this, 'delete_renewal_meta'), 10);
+			add_action('woocommerce_subscription_failing_payment_method_updated_stripe', array($this, 'update_failing_payment_method'), 10, 2);
+			add_action('wc_stripe_cards_payment_fields', array($this, 'display_update_subs_payment_checkout'));
+			add_action('wc_stripe_add_payment_method_' . $this->id . '_success', array($this, 'handle_add_payment_method_success'), 10, 2);
 
 			// display the credit card used for a subscription in the "My Subscriptions" table
-			add_filter( 'woocommerce_my_subscriptions_payment_method', array( $this, 'maybe_render_subscription_payment_method' ), 10, 2 );
+			add_filter('woocommerce_my_subscriptions_payment_method', array($this, 'maybe_render_subscription_payment_method'), 10, 2);
 
 			// allow store managers to manually set Stripe as the payment method on a subscription
-			add_filter( 'woocommerce_subscription_payment_meta', array( $this, 'add_subscription_payment_meta' ), 10, 2 );
-			add_filter( 'woocommerce_subscription_validate_payment_meta', array( $this, 'validate_subscription_payment_meta' ), 10, 2 );
-			add_filter( 'wc_stripe_display_save_payment_method_checkbox', array( $this, 'maybe_hide_save_checkbox' ) );
+			add_filter('woocommerce_subscription_payment_meta', array($this, 'add_subscription_payment_meta'), 10, 2);
+			add_filter('woocommerce_subscription_validate_payment_meta', array($this, 'validate_subscription_payment_meta'), 10, 2);
+			add_filter('wc_stripe_display_save_payment_method_checkbox', array($this, 'maybe_hide_save_checkbox'));
 
 			/*
 			 * WC subscriptions hooks into the "template_redirect" hook with priority 100.
@@ -37,8 +37,8 @@ 
 			 * See: https://github.com/woocommerce/woocommerce-subscriptions/blob/99a75687e109b64cbc07af6e5518458a6305f366/includes/class-wcs-cart-renewal.php#L165
 			 * If we are in the "You just need to authorize SCA" flow, we don't want that redirection to happen.
 			 */
-			add_action( 'template_redirect', array( $this, 'remove_order_pay_var' ), 99 );
-			add_action( 'template_redirect', array( $this, 'restore_order_pay_var' ), 101 );
+			add_action('template_redirect', array($this, 'remove_order_pay_var'), 99);
+			add_action('template_redirect', array($this, 'restore_order_pay_var'), 101);
 		}
 	}
 
@@ -49,8 +49,8 @@ 
 	 * @since 4.0.0
 	 * @version 4.0.0
 	 */
-	public function maybe_hide_save_checkbox( $display_tokenization ) {
-		if ( WC_Subscriptions_Cart::cart_contains_subscription() ) {
+	public function maybe_hide_save_checkbox($display_tokenization) {
+		if (WC_Subscriptions_Cart::cart_contains_subscription()) {
 			return false;
 		}
 
@@ -62,8 +62,8 @@ 
 	 * @param  int  $order_id
 	 * @return boolean
 	 */
-	public function has_subscription( $order_id ) {
-		return ( function_exists( 'wcs_order_contains_subscription' ) && ( wcs_order_contains_subscription( $order_id ) || wcs_is_subscription( $order_id ) || wcs_order_contains_renewal( $order_id ) ) );
+	public function has_subscription($order_id) {
+		return (function_exists('wcs_order_contains_subscription') && (wcs_order_contains_subscription($order_id) || wcs_is_subscription($order_id) || wcs_order_contains_renewal($order_id)));
 	}
 
 	/**
@@ -73,7 +73,7 @@ 
 	 * @return bool
 	 */
 	public function is_subs_change_payment() {
-		return ( isset( $_GET['pay_for_order'] ) && isset( $_GET['change_payment_method'] ) );
+		return (isset($_GET['pay_for_order']) && isset($_GET['change_payment_method']));
 	}
 
 	/**
@@ -83,20 +83,20 @@ 
 	 * @since 4.1.11
 	 */
 	public function display_update_subs_payment_checkout() {
-		$subs_statuses = apply_filters( 'wc_stripe_update_subs_payment_method_card_statuses', array( 'active' ) );
+		$subs_statuses = apply_filters('wc_stripe_update_subs_payment_method_card_statuses', array('active'));
 		if (
-			apply_filters( 'wc_stripe_display_update_subs_payment_method_card_checkbox', true ) &&
-			wcs_user_has_subscription( get_current_user_id(), '', $subs_statuses ) &&
+			apply_filters('wc_stripe_display_update_subs_payment_method_card_checkbox', true) &&
+			wcs_user_has_subscription(get_current_user_id(), '', $subs_statuses) &&
 			is_add_payment_method_page()
 		) {
-			$label = esc_html( apply_filters( 'wc_stripe_save_to_subs_text', __( 'Update the Payment Method used for all of my active subscriptions.', 'woocommerce-gateway-stripe' ) ) );
-			$id    = sprintf( 'wc-%1$s-update-subs-payment-method-card', $this->id );
+			$label = esc_html(apply_filters('wc_stripe_save_to_subs_text', __('Update the Payment Method used for all of my active subscriptions.', 'woocommerce-gateway-stripe')));
+			$id    = sprintf('wc-%1$s-update-subs-payment-method-card', $this->id);
 			woocommerce_form_field(
 				$id,
 				array(
 					'type'    => 'checkbox',
 					'label'   => $label,
-					'default' => apply_filters( 'wc_stripe_save_to_subs_checked', false ),
+					'default' => apply_filters('wc_stripe_save_to_subs_checked', false),
 				)
 			);
 		}
@@ -109,19 +109,19 @@ 
 	 * @param string $source_id
 	 * @param object $source_object
 	 */
-	public function handle_add_payment_method_success( $source_id, $source_object ) {
-		if ( isset( $_POST[ 'wc-' . $this->id . '-update-subs-payment-method-card' ] ) ) {
+	public function handle_add_payment_method_success($source_id, $source_object) {
+		if (isset($_POST['wc-' . $this->id . '-update-subs-payment-method-card'])) {
 			$all_subs        = wcs_get_users_subscriptions();
-			$subs_statuses   = apply_filters( 'wc_stripe_update_subs_payment_method_card_statuses', array( 'active' ) );
-			$stripe_customer = new WC_Stripe_Customer( get_current_user_id() );
-
-			if ( ! empty( $all_subs ) ) {
-				foreach ( $all_subs as $sub ) {
-					if ( $sub->has_status( $subs_statuses ) ) {
-						update_post_meta( $sub->get_id(), '_stripe_source_id', $source_id );
-						update_post_meta( $sub->get_id(), '_stripe_customer_id', $stripe_customer->get_id() );
-						update_post_meta( $sub->get_id(), '_payment_method', $this->id );
-						update_post_meta( $sub->get_id(), '_payment_method_title', $this->method_title );
+			$subs_statuses   = apply_filters('wc_stripe_update_subs_payment_method_card_statuses', array('active'));
+			$stripe_customer = new WC_Stripe_Customer(get_current_user_id());
+
+			if ( ! empty($all_subs)) {
+				foreach ($all_subs as $sub) {
+					if ($sub->has_status($subs_statuses)) {
+						update_post_meta($sub->get_id(), '_stripe_source_id', $source_id);
+						update_post_meta($sub->get_id(), '_stripe_customer_id', $stripe_customer->get_id());
+						update_post_meta($sub->get_id(), '_payment_method', $this->id);
+						update_post_meta($sub->get_id(), '_payment_method_title', $this->method_title);
 					}
 				}
 			}
@@ -135,24 +135,24 @@ 
 	 * @since 4.1.11 Remove 3DS check as it is not needed.
 	 * @param int $order_id
 	 */
-	public function change_subs_payment_method( $order_id ) {
+	public function change_subs_payment_method($order_id) {
 		try {
-			$subscription    = wc_get_order( $order_id );
-			$prepared_source = $this->prepare_source( get_current_user_id(), true );
+			$subscription    = wc_get_order($order_id);
+			$prepared_source = $this->prepare_source(get_current_user_id(), true);
 
-			$this->maybe_disallow_prepaid_card( $prepared_source );
-			$this->check_source( $prepared_source );
-			$this->save_source_to_order( $subscription, $prepared_source );
+			$this->maybe_disallow_prepaid_card($prepared_source);
+			$this->check_source($prepared_source);
+			$this->save_source_to_order($subscription, $prepared_source);
 
-			do_action( 'wc_stripe_change_subs_payment_method_success', $prepared_source->source, $prepared_source );
+			do_action('wc_stripe_change_subs_payment_method_success', $prepared_source->source, $prepared_source);
 
 			return array(
 				'result'   => 'success',
-				'redirect' => $this->get_return_url( $subscription ),
+				'redirect' => $this->get_return_url($subscription),
 			);
-		} catch ( WC_Stripe_Exception $e ) {
-			wc_add_notice( $e->getLocalizedMessage(), 'error' );
-			WC_Stripe_Logger::log( 'Error: ' . $e->getMessage() );
+		} catch (WC_Stripe_Exception $e) {
+			wc_add_notice($e->getLocalizedMessage(), 'error');
+			WC_Stripe_Logger::log('Error: ' . $e->getMessage());
 		}
 	}
 
@@ -161,16 +161,16 @@ 
 	 * @param  int $order_id
 	 * @return array
 	 */
-	public function process_payment( $order_id, $retry = true, $force_save_source = false, $previous_error = false, $use_order_source = false ) {
-		if ( $this->has_subscription( $order_id ) ) {
-			if ( $this->is_subs_change_payment() ) {
-				return $this->change_subs_payment_method( $order_id );
+	public function process_payment($order_id, $retry = true, $force_save_source = false, $previous_error = false, $use_order_source = false) {
+		if ($this->has_subscription($order_id)) {
+			if ($this->is_subs_change_payment()) {
+				return $this->change_subs_payment_method($order_id);
 			}
 
 			// Regular payment with force customer enabled
-			return parent::process_payment( $order_id, $retry, true, $previous_error, $use_order_source );
+			return parent::process_payment($order_id, $retry, true, $previous_error, $use_order_source);
 		} else {
-			return parent::process_payment( $order_id, $retry, $force_save_source, $previous_error, $use_order_source );
+			return parent::process_payment($order_id, $retry, $force_save_source, $previous_error, $use_order_source);
 		}
 	}
 
@@ -182,11 +182,11 @@ 
 	 * @param object   $prepared_source The source that is used for the payment.
 	 * @return array                    The arguments for the request.
 	 */
-	public function generate_create_intent_request( $order, $prepared_source ) {
-		$request = parent::generate_create_intent_request( $order, $prepared_source );
+	public function generate_create_intent_request($order, $prepared_source) {
+		$request = parent::generate_create_intent_request($order, $prepared_source);
 
 		// Non-subscription orders do not need any additional parameters.
-		if ( ! $this->has_subscription( $order ) ) {
+		if ( ! $this->has_subscription($order)) {
 			return $request;
 		}
 
@@ -202,8 +202,8 @@ 
 	 * @param $amount_to_charge float The amount to charge.
 	 * @param $renewal_order WC_Order A WC_Order object created to record the renewal payment.
 	 */
-	public function scheduled_subscription_payment( $amount_to_charge, $renewal_order ) {
-		$this->process_subscription_payment( $amount_to_charge, $renewal_order, true, false );
+	public function scheduled_subscription_payment($amount_to_charge, $renewal_order) {
+		$this->process_subscription_payment($amount_to_charge, $renewal_order, true, false);
 	}
 
 	/**
@@ -217,11 +217,11 @@ 
 	 * @param bool $retry Should we retry the process?
 	 * @param object $previous_error
 	 */
-	public function process_subscription_payment( $amount, $renewal_order, $retry = true, $previous_error = false ) {
+	public function process_subscription_payment($amount, $renewal_order, $retry = true, $previous_error = false) {
 		try {
-			if ( $amount * 100 < WC_Stripe_Helper::get_minimum_amount() ) {
+			if ($amount * 100 < WC_Stripe_Helper::get_minimum_amount()) {
 				/* translators: minimum amount */
-				$message = sprintf( __( 'Sorry, the minimum allowed order total is %1$s to use this payment method.', 'woocommerce-gateway-stripe' ), wc_price( WC_Stripe_Helper::get_minimum_amount() / 100 ) );
+				$message = sprintf(__('Sorry, the minimum allowed order total is %1$s to use this payment method.', 'woocommerce-gateway-stripe'), wc_price(WC_Stripe_Helper::get_minimum_amount() / 100));
 				throw new WC_Stripe_Exception(
 					'Error while processing renewal order ' . $renewal_order->get_id() . ' : ' . $message,
 					$message
@@ -230,141 +230,141 @@ 
 
 			$order_id = $renewal_order->get_id();
 
-			$this->ensure_subscription_has_customer_id( $order_id );
+			$this->ensure_subscription_has_customer_id($order_id);
 
 			// Unlike regular off-session subscription payments, early renewals are treated as on-session payments, involving the customer.
-			if ( isset( $_REQUEST['process_early_renewal'] ) ) { // wpcs: csrf ok.
-				$response = parent::process_payment( $order_id, true, false, $previous_error, true );
+			if (isset($_REQUEST['process_early_renewal'])) { // wpcs: csrf ok.
+				$response = parent::process_payment($order_id, true, false, $previous_error, true);
 
-				if( 'success' === $response['result'] && isset( $response['payment_intent_secret'] ) ) {
+				if ('success' === $response['result'] && isset($response['payment_intent_secret'])) {
 					$verification_url = add_query_arg(
 						array(
 							'order'         => $order_id,
-							'nonce'         => wp_create_nonce( 'wc_stripe_confirm_pi' ),
-							'redirect_to'   => remove_query_arg( array( 'process_early_renewal', 'subscription_id', 'wcs_nonce' ) ),
+							'nonce'         => wp_create_nonce('wc_stripe_confirm_pi'),
+							'redirect_to'   => remove_query_arg(array('process_early_renewal', 'subscription_id', 'wcs_nonce')),
 							'early_renewal' => true,
 						),
-						WC_AJAX::get_endpoint( 'wc_stripe_verify_intent' )
+						WC_AJAX::get_endpoint('wc_stripe_verify_intent')
 					);
 
-					echo wp_json_encode( array(
+					echo wp_json_encode(array(
 						'stripe_sca_required' => true,
 						'intent_secret'       => $response['payment_intent_secret'],
 						'redirect_url'        => $verification_url,
-					) );
+					));
 
 					exit;
 				}
 
 				// Hijack all other redirects in order to do the redirection in JavaScript.
-				add_action( 'wp_redirect', array( $this, 'redirect_after_early_renewal' ), 100 );
+				add_action('wp_redirect', array($this, 'redirect_after_early_renewal'), 100);
 
 				return;
 			}
 
 			// Check for an existing intent, which is associated with the order.
-			if ( $this->has_authentication_already_failed( $renewal_order ) ) {
+			if ($this->has_authentication_already_failed($renewal_order)) {
 				return;
 			}
 
 			// Get source from order
-			$prepared_source = $this->prepare_order_source( $renewal_order );
+			$prepared_source = $this->prepare_order_source($renewal_order);
 			$source_object   = $prepared_source->source_object;
 
-			if ( ! $prepared_source->customer ) {
+			if ( ! $prepared_source->customer) {
 				throw new WC_Stripe_Exception(
 					'Failed to process renewal for order ' . $renewal_order->get_id() . '. Stripe customer id is missing in the order',
-					__( 'Customer not found', 'woocommerce-gateway-stripe' )
+					__('Customer not found', 'woocommerce-gateway-stripe')
 				);
 			}
 
-			WC_Stripe_Logger::log( "Info: Begin processing subscription payment for order {$order_id} for the amount of {$amount}" );
+			WC_Stripe_Logger::log("Info: Begin processing subscription payment for order {$order_id} for the amount of {$amount}");
 
 			/* If we're doing a retry and source is chargeable, we need to pass
 			 * a different idempotency key and retry for success.
 			 */
-			if ( is_object( $source_object ) && empty( $source_object->error ) && $this->need_update_idempotency_key( $source_object, $previous_error ) ) {
-				add_filter( 'wc_stripe_idempotency_key', array( $this, 'change_idempotency_key' ), 10, 2 );
+			if (is_object($source_object) && empty($source_object->error) && $this->need_update_idempotency_key($source_object, $previous_error)) {
+				add_filter('wc_stripe_idempotency_key', array($this, 'change_idempotency_key'), 10, 2);
 			}
 
-			if ( ( $this->is_no_such_source_error( $previous_error ) || $this->is_no_linked_source_error( $previous_error ) ) && apply_filters( 'wc_stripe_use_default_customer_source', true ) ) {
+			if (($this->is_no_such_source_error($previous_error) || $this->is_no_linked_source_error($previous_error)) && apply_filters('wc_stripe_use_default_customer_source', true)) {
 				// Passing empty source will charge customer default.
 				$prepared_source->source = '';
 			}
 
-			$this->lock_order_payment( $renewal_order );
+			$this->lock_order_payment($renewal_order);
 
-			$response                   = $this->create_and_confirm_intent_for_off_session( $renewal_order, $prepared_source, $amount );
-			$is_authentication_required = $this->is_authentication_required_for_payment( $response );
+			$response                   = $this->create_and_confirm_intent_for_off_session($renewal_order, $prepared_source, $amount);
+			$is_authentication_required = $this->is_authentication_required_for_payment($response);
 
 			// It's only a failed payment if it's an error and it's not of the type 'authentication_required'.
 			// If it's 'authentication_required', then we should email the user and ask them to authenticate.
-			if ( ! empty( $response->error ) && ! $is_authentication_required ) {
+			if ( ! empty($response->error) && ! $is_authentication_required) {
 				// 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 ) {
-							return $this->process_subscription_payment( $amount, $renewal_order, false, $response->error );
+						if (5 <= $this->retry_interval) {
+							return $this->process_subscription_payment($amount, $renewal_order, false, $response->error);
 						}
 
-						sleep( $this->retry_interval );
+						sleep($this->retry_interval);
 
 						$this->retry_interval++;
 
-						return $this->process_subscription_payment( $amount, $renewal_order, true, $response->error );
+						return $this->process_subscription_payment($amount, $renewal_order, true, $response->error);
 					} else {
-						$localized_message = __( 'Sorry, we are unable to process your payment at this time. Please retry later.', 'woocommerce-gateway-stripe' );
-						$renewal_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');
+						$renewal_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;
 				}
 
-				$renewal_order->add_order_note( $localized_message );
+				$renewal_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);
 			}
 
 			// Either the charge was successfully captured, or it requires further authentication.
 
-			if ( $is_authentication_required ) {
-				do_action( 'wc_gateway_stripe_process_payment_authentication_required', $renewal_order, $response );
+			if ($is_authentication_required) {
+				do_action('wc_gateway_stripe_process_payment_authentication_required', $renewal_order, $response);
 
-				$error_message = __( 'This transaction requires authentication.', 'woocommerce-gateway-stripe' );
-				$renewal_order->add_order_note( $error_message );
+				$error_message = __('This transaction requires authentication.', 'woocommerce-gateway-stripe');
+				$renewal_order->add_order_note($error_message);
 
-				$charge = end( $response->error->payment_intent->charges->data );
+				$charge = end($response->error->payment_intent->charges->data);
 				$id = $charge->id;
 				$order_id = $renewal_order->get_id();
 
-				$renewal_order->set_transaction_id( $id );
-				$renewal_order->update_status( 'failed', sprintf( __( 'Stripe charge awaiting authentication by user: %s.', 'woocommerce-gateway-stripe' ), $id ) );
-				if ( is_callable( array( $renewal_order, 'save' ) ) ) {
+				$renewal_order->set_transaction_id($id);
+				$renewal_order->update_status('failed', sprintf(__('Stripe charge awaiting authentication by user: %s.', 'woocommerce-gateway-stripe'), $id));
+				if (is_callable(array($renewal_order, 'save'))) {
 					$renewal_order->save();
 				}
 			} else {
 				// The charge was successfully captured
-				do_action( 'wc_gateway_stripe_process_payment', $response, $renewal_order );
+				do_action('wc_gateway_stripe_process_payment', $response, $renewal_order);
 
-				$this->process_response( end( $response->charges->data ), $renewal_order );
+				$this->process_response(end($response->charges->data), $renewal_order);
 			}
 
-			$this->unlock_order_payment( $renewal_order );
-		} catch ( WC_Stripe_Exception $e ) {
-			WC_Stripe_Logger::log( 'Error: ' . $e->getMessage() );
+			$this->unlock_order_payment($renewal_order);
+		} catch (WC_Stripe_Exception $e) {
+			WC_Stripe_Logger::log('Error: ' . $e->getMessage());
 
-			do_action( 'wc_gateway_stripe_process_payment_error', $e, $renewal_order );
+			do_action('wc_gateway_stripe_process_payment_error', $e, $renewal_order);
 
 			/* translators: error message */
-			$renewal_order->update_status( 'failed' );
+			$renewal_order->update_status('failed');
 		}
 	}
 
@@ -374,24 +374,24 @@ 
 	 * @since 3.1.0
 	 * @version 4.0.0
 	 */
-	public function save_source_to_order( $order, $source ) {
-		parent::save_source_to_order( $order, $source );
+	public function save_source_to_order($order, $source) {
+		parent::save_source_to_order($order, $source);
 
 		$order_id = $order->get_id();
 
 		// Also store it on the subscriptions being purchased or paid for in the order
-		if ( function_exists( 'wcs_order_contains_subscription' ) && wcs_order_contains_subscription( $order_id ) ) {
-			$subscriptions = wcs_get_subscriptions_for_order( $order_id );
-		} elseif ( function_exists( 'wcs_order_contains_renewal' ) && wcs_order_contains_renewal( $order_id ) ) {
-			$subscriptions = wcs_get_subscriptions_for_renewal_order( $order_id );
+		if (function_exists('wcs_order_contains_subscription') && wcs_order_contains_subscription($order_id)) {
+			$subscriptions = wcs_get_subscriptions_for_order($order_id);
+		} elseif (function_exists('wcs_order_contains_renewal') && wcs_order_contains_renewal($order_id)) {
+			$subscriptions = wcs_get_subscriptions_for_renewal_order($order_id);
 		} else {
 			$subscriptions = array();
 		}
 
-		foreach ( $subscriptions as $subscription ) {
+		foreach ($subscriptions as $subscription) {
 			$subscription_id = $subscription->get_id();
-			update_post_meta( $subscription_id, '_stripe_customer_id', $source->customer );
-			update_post_meta( $subscription_id, '_stripe_source_id', $source->source );
+			update_post_meta($subscription_id, '_stripe_customer_id', $source->customer);
+			update_post_meta($subscription_id, '_stripe_source_id', $source->source);
 		}
 	}
 
@@ -399,26 +399,26 @@ 
 	 * Don't transfer Stripe customer/token meta to resubscribe orders.
 	 * @param int $resubscribe_order The order created for the customer to resubscribe to the old expired/cancelled subscription
 	 */
-	public function delete_resubscribe_meta( $resubscribe_order ) {
-		delete_post_meta( $resubscribe_order->get_id(), '_stripe_customer_id' );
-		delete_post_meta( $resubscribe_order->get_id(), '_stripe_source_id' );
+	public function delete_resubscribe_meta($resubscribe_order) {
+		delete_post_meta($resubscribe_order->get_id(), '_stripe_customer_id');
+		delete_post_meta($resubscribe_order->get_id(), '_stripe_source_id');
 		// For BW compat will remove in future
-		delete_post_meta( $resubscribe_order->get_id(), '_stripe_card_id' );
+		delete_post_meta($resubscribe_order->get_id(), '_stripe_card_id');
 		// delete payment intent ID
-		delete_post_meta( $resubscribe_order->get_id(), '_stripe_intent_id' );
-		$this->delete_renewal_meta( $resubscribe_order );
+		delete_post_meta($resubscribe_order->get_id(), '_stripe_intent_id');
+		$this->delete_renewal_meta($resubscribe_order);
 	}
 
 	/**
 	 * Don't transfer Stripe fee/ID meta to renewal orders.
 	 * @param int $resubscribe_order The order created for the customer to resubscribe to the old expired/cancelled subscription
 	 */
-	public function delete_renewal_meta( $renewal_order ) {
-		WC_Stripe_Helper::delete_stripe_fee( $renewal_order );
-		WC_Stripe_Helper::delete_stripe_net( $renewal_order );
+	public function delete_renewal_meta($renewal_order) {
+		WC_Stripe_Helper::delete_stripe_fee($renewal_order);
+		WC_Stripe_Helper::delete_stripe_net($renewal_order);
 
 		// delete payment intent ID
-		delete_post_meta( $renewal_order->get_id(), '_stripe_intent_id' );
+		delete_post_meta($renewal_order->get_id(), '_stripe_intent_id');
 
 		return $renewal_order;
 	}
@@ -432,9 +432,9 @@ 
 	 * @param WC_Order $renewal_order The order which recorded the successful payment (to make up for the failed automatic payment).
 	 * @return void
 	 */
-	public function update_failing_payment_method( $subscription, $renewal_order ) {
-		update_post_meta( $subscription->get_id(), '_stripe_customer_id', $renewal_order->get_meta( '_stripe_customer_id', true ) );
-		update_post_meta( $subscription->get_id(), '_stripe_source_id', $renewal_order->get_meta( '_stripe_source_id', true ) );
+	public function update_failing_payment_method($subscription, $renewal_order) {
+		update_post_meta($subscription->get_id(), '_stripe_customer_id', $renewal_order->get_meta('_stripe_customer_id', true));
+		update_post_meta($subscription->get_id(), '_stripe_source_id', $renewal_order->get_meta('_stripe_source_id', true));
 	}
 
 	/**
@@ -446,23 +446,23 @@ 
 	 * @param WC_Subscription $subscription An instance of a subscription object
 	 * @return array
 	 */
-	public function add_subscription_payment_meta( $payment_meta, $subscription ) {
+	public function add_subscription_payment_meta($payment_meta, $subscription) {
 		$subscription_id = $subscription->get_id();
-		$source_id       = get_post_meta( $subscription_id, '_stripe_source_id', true );
+		$source_id       = get_post_meta($subscription_id, '_stripe_source_id', true);
 
 		// For BW compat will remove in future.
-		if ( empty( $source_id ) ) {
-			$source_id = get_post_meta( $subscription_id, '_stripe_card_id', true );
+		if (empty($source_id)) {
+			$source_id = get_post_meta($subscription_id, '_stripe_card_id', true);
 
 			// Take this opportunity to update the key name.
-			update_post_meta( $subscription_id, '_stripe_source_id', $source_id );
-			delete_post_meta( $subscription_id, '_stripe_card_id', $source_id );
+			update_post_meta($subscription_id, '_stripe_source_id', $source_id);
+			delete_post_meta($subscription_id, '_stripe_card_id', $source_id);
 		}
 
-		$payment_meta[ $this->id ] = array(
+		$payment_meta[$this->id] = array(
 			'post_meta' => array(
 				'_stripe_customer_id' => array(
-					'value' => get_post_meta( $subscription_id, '_stripe_customer_id', true ),
+					'value' => get_post_meta($subscription_id, '_stripe_customer_id', true),
 					'label' => 'Stripe Customer ID',
 				),
 				'_stripe_source_id'   => array(
@@ -485,27 +485,27 @@ 
 	 * @param array $payment_meta associative array of meta data required for automatic payments
 	 * @return array
 	 */
-	public function validate_subscription_payment_meta( $payment_method_id, $payment_meta ) {
-		if ( $this->id === $payment_method_id ) {
+	public function validate_subscription_payment_meta($payment_method_id, $payment_meta) {
+		if ($this->id === $payment_method_id) {
 
-			if ( ! isset( $payment_meta['post_meta']['_stripe_customer_id']['value'] ) || empty( $payment_meta['post_meta']['_stripe_customer_id']['value'] ) ) {
+			if ( ! isset($payment_meta['post_meta']['_stripe_customer_id']['value']) || empty($payment_meta['post_meta']['_stripe_customer_id']['value'])) {
 
 				// Allow empty stripe customer id during subscription renewal. It will be added when processing payment if required.
-				if ( ! isset( $_POST['wc_order_action'] ) || 'wcs_process_renewal' !== $_POST['wc_order_action'] ) {
-					throw new Exception( __( 'A "Stripe Customer ID" value is required.', 'woocommerce-gateway-stripe' ) );
+				if ( ! isset($_POST['wc_order_action']) || 'wcs_process_renewal' !== $_POST['wc_order_action']) {
+					throw new Exception(__('A "Stripe Customer ID" value is required.', 'woocommerce-gateway-stripe'));
 				}
-			} elseif ( 0 !== strpos( $payment_meta['post_meta']['_stripe_customer_id']['value'], 'cus_' ) ) {
-				throw new Exception( __( 'Invalid customer ID. A valid "Stripe Customer ID" must begin with "cus_".', 'woocommerce-gateway-stripe' ) );
+			} elseif (0 !== strpos($payment_meta['post_meta']['_stripe_customer_id']['value'], 'cus_')) {
+				throw new Exception(__('Invalid customer ID. A valid "Stripe Customer ID" must begin with "cus_".', 'woocommerce-gateway-stripe'));
 			}
 
 			if (
-				! empty( $payment_meta['post_meta']['_stripe_source_id']['value'] ) && (
-					0 !== strpos( $payment_meta['post_meta']['_stripe_source_id']['value'], 'card_' )
-					&& 0 !== strpos( $payment_meta['post_meta']['_stripe_source_id']['value'], 'src_' )
-					&& 0 !== strpos( $payment_meta['post_meta']['_stripe_source_id']['value'], 'pm_' )
+				! empty($payment_meta['post_meta']['_stripe_source_id']['value']) && (
+					0 !== strpos($payment_meta['post_meta']['_stripe_source_id']['value'], 'card_')
+					&& 0 !== strpos($payment_meta['post_meta']['_stripe_source_id']['value'], 'src_')
+					&& 0 !== strpos($payment_meta['post_meta']['_stripe_source_id']['value'], 'pm_')
 				)
 			) {
-				throw new Exception( __( 'Invalid source ID. A valid source "Stripe Source ID" must begin with "src_", "pm_", or "card_".', 'woocommerce-gateway-stripe' ) );
+				throw new Exception(__('Invalid source ID. A valid source "Stripe Source ID" must begin with "src_", "pm_", or "card_".', 'woocommerce-gateway-stripe'));
 			}
 		}
 	}
@@ -518,75 +518,75 @@ 
 	 * @param WC_Subscription $subscription the subscription details
 	 * @return string the subscription payment method
 	 */
-	public function maybe_render_subscription_payment_method( $payment_method_to_display, $subscription ) {
+	public function maybe_render_subscription_payment_method($payment_method_to_display, $subscription) {
 		$customer_user = $subscription->get_customer_id();
 
 		// bail for other payment methods
-		if ( $subscription->get_payment_method() !== $this->id || ! $customer_user ) {
+		if ($subscription->get_payment_method() !== $this->id || ! $customer_user) {
 			return $payment_method_to_display;
 		}
 
-		$stripe_source_id = get_post_meta( $subscription->get_id(), '_stripe_source_id', true );
+		$stripe_source_id = get_post_meta($subscription->get_id(), '_stripe_source_id', true);
 
 		// For BW compat will remove in future.
-		if ( empty( $stripe_source_id ) ) {
-			$stripe_source_id = get_post_meta( $subscription->get_id(), '_stripe_card_id', true );
+		if (empty($stripe_source_id)) {
+			$stripe_source_id = get_post_meta($subscription->get_id(), '_stripe_card_id', true);
 
 			// Take this opportunity to update the key name.
-			update_post_meta( $subscription->get_id(), '_stripe_source_id', $stripe_source_id );
+			update_post_meta($subscription->get_id(), '_stripe_source_id', $stripe_source_id);
 		}
 
 		$stripe_customer    = new WC_Stripe_Customer();
-		$stripe_customer_id = get_post_meta( $subscription->get_id(), '_stripe_customer_id', true );
+		$stripe_customer_id = get_post_meta($subscription->get_id(), '_stripe_customer_id', true);
 
 		// If we couldn't find a Stripe customer linked to the subscription, fallback to the user meta data.
-		if ( ! $stripe_customer_id || ! is_string( $stripe_customer_id ) ) {
+		if ( ! $stripe_customer_id || ! is_string($stripe_customer_id)) {
 			$user_id            = $customer_user;
-			$stripe_customer_id = get_user_option( '_stripe_customer_id', $user_id );
-			$stripe_source_id   = get_user_option( '_stripe_source_id', $user_id );
+			$stripe_customer_id = get_user_option('_stripe_customer_id', $user_id);
+			$stripe_source_id   = get_user_option('_stripe_source_id', $user_id);
 
 			// For BW compat will remove in future.
-			if ( empty( $stripe_source_id ) ) {
-				$stripe_source_id = get_user_option( '_stripe_card_id', $user_id );
+			if (empty($stripe_source_id)) {
+				$stripe_source_id = get_user_option('_stripe_card_id', $user_id);
 
 				// Take this opportunity to update the key name.
-				update_user_option( $user_id, '_stripe_source_id', $stripe_source_id, false );
+				update_user_option($user_id, '_stripe_source_id', $stripe_source_id, false);
 			}
 		}
 
 		// If we couldn't find a Stripe customer linked to the account, fallback to the order meta data.
-		if ( ( ! $stripe_customer_id || ! is_string( $stripe_customer_id ) ) && false !== $subscription->order ) {
-			$stripe_customer_id = get_post_meta( $subscription->get_parent_id(), '_stripe_customer_id', true );
-			$stripe_source_id   = get_post_meta( $subscription->get_parent_id(), '_stripe_source_id', true );
+		if (( ! $stripe_customer_id || ! is_string($stripe_customer_id)) && false !== $subscription->order) {
+			$stripe_customer_id = get_post_meta($subscription->get_parent_id(), '_stripe_customer_id', true);
+			$stripe_source_id   = get_post_meta($subscription->get_parent_id(), '_stripe_source_id', true);
 
 			// For BW compat will remove in future.
-			if ( empty( $stripe_source_id ) ) {
-				$stripe_source_id = get_post_meta( $subscription->get_parent_id(), '_stripe_card_id', true );
+			if (empty($stripe_source_id)) {
+				$stripe_source_id = get_post_meta($subscription->get_parent_id(), '_stripe_card_id', true);
 
 				// Take this opportunity to update the key name.
-				update_post_meta( $subscription->get_parent_id(), '_stripe_source_id', $stripe_source_id );
+				update_post_meta($subscription->get_parent_id(), '_stripe_source_id', $stripe_source_id);
 			}
 		}
 
-		$stripe_customer->set_id( $stripe_customer_id );
+		$stripe_customer->set_id($stripe_customer_id);
 
 		$sources                   = $stripe_customer->get_sources();
-		$payment_method_to_display = __( 'N/A', 'woocommerce-gateway-stripe' );
+		$payment_method_to_display = __('N/A', 'woocommerce-gateway-stripe');
 
-		if ( $sources ) {
+		if ($sources) {
 			$card = false;
 
-			foreach ( $sources as $source ) {
-				if ( isset( $source->type ) && 'card' === $source->type ) {
+			foreach ($sources as $source) {
+				if (isset($source->type) && 'card' === $source->type) {
 					$card = $source->card;
-				} elseif ( isset( $source->object ) && 'card' === $source->object ) {
+				} elseif (isset($source->object) && 'card' === $source->object) {
 					$card = $source;
 				}
 
-				if ( $source->id === $stripe_source_id ) {
-					if ( $card ) {
+				if ($source->id === $stripe_source_id) {
+					if ($card) {
 						/* translators: 1) card brand 2) last 4 digits */
-						$payment_method_to_display = sprintf( __( 'Via %1$s card ending in %2$s', 'woocommerce-gateway-stripe' ), ( isset( $card->brand ) ? $card->brand : __( 'N/A', 'woocommerce-gateway-stripe' ) ), $card->last4 );
+						$payment_method_to_display = sprintf(__('Via %1$s card ending in %2$s', 'woocommerce-gateway-stripe'), (isset($card->brand) ? $card->brand : __('N/A', 'woocommerce-gateway-stripe')), $card->last4);
 					}
 
 					break;
@@ -603,7 +603,7 @@ 
 	 */
 	public function remove_order_pay_var() {
 		global $wp;
-		if ( isset( $_GET['wc-stripe-confirmation'] ) ) {
+		if (isset($_GET['wc-stripe-confirmation'])) {
 			$this->order_pay_var = $wp->query_vars['order-pay'];
 			$wp->query_vars['order-pay'] = null;
 		}
@@ -614,7 +614,7 @@ 
 	 */
 	public function restore_order_pay_var() {
 		global $wp;
-		if ( isset( $this->order_pay_var ) ) {
+		if (isset($this->order_pay_var)) {
 			$wp->query_vars['order-pay'] = $this->order_pay_var;
 		}
 	}
@@ -625,13 +625,13 @@ 
 	 * @param WC_Order $renewal_order The renewal order.
 	 * @return boolean
 	 */
-	public function has_authentication_already_failed( $renewal_order ) {
-		$existing_intent = $this->get_intent_from_order( $renewal_order );
+	public function has_authentication_already_failed($renewal_order) {
+		$existing_intent = $this->get_intent_from_order($renewal_order);
 
 		if (
 			! $existing_intent
 			|| 'requires_payment_method' !== $existing_intent->status
-			|| empty( $existing_intent->last_payment_error )
+			|| empty($existing_intent->last_payment_error)
 			|| 'authentication_required' !== $existing_intent->last_payment_error->code
 		) {
 			return false;
@@ -645,12 +645,12 @@ 
 		 *
 		 * @param WC_Order $renewal_order The order that is being renewed.
 		 */
-		do_action( 'wc_gateway_stripe_process_payment_authentication_required', $renewal_order );
+		do_action('wc_gateway_stripe_process_payment_authentication_required', $renewal_order);
 
 		// Fail the payment attempt (order would be currently pending because of retry rules).
-		$charge    = end( $existing_intent->charges->data );
+		$charge    = end($existing_intent->charges->data);
 		$charge_id = $charge->id;
-		$renewal_order->update_status( 'failed', sprintf( __( 'Stripe charge awaiting authentication by user: %s.', 'woocommerce-gateway-stripe' ), $charge_id ) );
+		$renewal_order->update_status('failed', sprintf(__('Stripe charge awaiting authentication by user: %s.', 'woocommerce-gateway-stripe'), $charge_id));
 
 		return true;
 	}
@@ -661,7 +661,7 @@ 
 	 * @param string $url The URL that Subscriptions attempts a redirect to.
 	 * @return void
 	 */
-	public function redirect_after_early_renewal( $url ) {
+	public function redirect_after_early_renewal($url) {
 		echo wp_json_encode(
 			array(
 				'stripe_sca_required' => false,
@@ -678,12 +678,12 @@ 
 	 * @param WC_Order $order The renewal order.
 	 * @param stdClass $intent The Payment Intent object.
 	 */
-	protected function handle_intent_verification_success( $order, $intent ) {
-		parent::handle_intent_verification_success( $order, $intent );
+	protected function handle_intent_verification_success($order, $intent) {
+		parent::handle_intent_verification_success($order, $intent);
 
-		if ( isset( $_GET['early_renewal'] ) ) { // wpcs: csrf ok.
-			wcs_update_dates_after_early_renewal( wcs_get_subscription( $order->get_meta( '_subscription_renewal' ) ), $order );
-			wc_add_notice( __( 'Your early renewal order was successful.', 'woocommerce-gateway-stripe' ), 'success' );
+		if (isset($_GET['early_renewal'])) { // wpcs: csrf ok.
+			wcs_update_dates_after_early_renewal(wcs_get_subscription($order->get_meta('_subscription_renewal')), $order);
+			wc_add_notice(__('Your early renewal order was successful.', 'woocommerce-gateway-stripe'), 'success');
 		}
 	}
 
@@ -693,12 +693,12 @@ 
 	 * @param WC_Order $order The renewal order.
 	 * @param stdClass $intent The Payment Intent object (unused).
 	 */
-	protected function handle_intent_verification_failure( $order, $intent ) {
-		if ( isset( $_GET['early_renewal'] ) ) {
-			$order->delete( true );
-			wc_add_notice( __( 'Payment authorization for the renewal order was unsuccessful, please try again.', 'woocommerce-gateway-stripe' ), 'error' );
-			$renewal_url = wcs_get_early_renewal_url( wcs_get_subscription( $order->get_meta( '_subscription_renewal' ) ) );
-			wp_redirect( $renewal_url ); exit;
+	protected function handle_intent_verification_failure($order, $intent) {
+		if (isset($_GET['early_renewal'])) {
+			$order->delete(true);
+			wc_add_notice(__('Payment authorization for the renewal order was unsuccessful, please try again.', 'woocommerce-gateway-stripe'), 'error');
+			$renewal_url = wcs_get_early_renewal_url(wcs_get_subscription($order->get_meta('_subscription_renewal')));
+			wp_redirect($renewal_url); exit;
 		}
 	}
 }

Doc Comments +1 added lines, -1 removed lines

@@ -321,7 +321,7 @@
 
 	/**
 	 * Don't transfer Stripe fee/ID meta to renewal orders.
-	 * @param int $resubscribe_order The order created for the customer to resubscribe to the old expired/cancelled subscription
+	 * @param integer $renewal_order
 	 */
 	public function delete_renewal_meta( $renewal_order ) {
 		WC_Stripe_Helper::delete_stripe_fee( $renewal_order );

includes/admin/class-wc-stripe-inbox-notes.php

Doc Comments +1 added lines

@@ -21,6 +21,7 @@
 
 	/**
 	 * Manage notes to show after Apple Pay domain verification.
+	 * @param boolean $verification_complete
 	 */
 	public static function notify_on_apple_pay_domain_verification( $verification_complete ) {
 		if ( ! class_exists( 'Automattic\WooCommerce\Admin\Notes\WC_Admin_Notes' ) ) {

Spacing +40 added lines, -40 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -18,41 +18,41 @@ 
 	const POST_SETUP_SUCCESS_ACTION = 'wc_stripe_apple_pay_post_setup_success';
 
 	public function __construct() {
-		add_action( self::POST_SETUP_SUCCESS_ACTION, array( self::class, 'create_marketing_note' ) );
+		add_action(self::POST_SETUP_SUCCESS_ACTION, array(self::class, 'create_marketing_note'));
 	}
 
 	/**
 	 * Manage notes to show after Apple Pay domain verification.
 	 */
-	public static function notify_on_apple_pay_domain_verification( $verification_complete ) {
-		if ( ! class_exists( 'Automattic\WooCommerce\Admin\Notes\WC_Admin_Notes' ) ) {
+	public static function notify_on_apple_pay_domain_verification($verification_complete) {
+		if ( ! class_exists('Automattic\WooCommerce\Admin\Notes\WC_Admin_Notes')) {
 			return;
 		}
 
-		if ( ! class_exists( 'WC_Data_Store' ) ) {
+		if ( ! class_exists('WC_Data_Store')) {
 			return;
 		}
 
-		if ( $verification_complete ) {
-			if ( self::should_show_marketing_note() && ! wp_next_scheduled( self::POST_SETUP_SUCCESS_ACTION ) ) {
-				wp_schedule_single_event( time() + DAY_IN_SECONDS, self::POST_SETUP_SUCCESS_ACTION );
+		if ($verification_complete) {
+			if (self::should_show_marketing_note() && ! wp_next_scheduled(self::POST_SETUP_SUCCESS_ACTION)) {
+				wp_schedule_single_event(time() + DAY_IN_SECONDS, self::POST_SETUP_SUCCESS_ACTION);
 			}
 
 			// If the domain verification completed after failure note was created, make sure it's marked as actioned.
 			try {
-				$data_store       = WC_Data_Store::load( 'admin-note' );
-				$failure_note_ids = $data_store->get_notes_with_name( self::FAILURE_NOTE_NAME );
-				if ( ! empty( $failure_note_ids ) ) {
-					$note_id = array_pop( $failure_note_ids );
-					$note    = WC_Admin_Notes::get_note( $note_id );
-					if ( false !== $note && WC_Admin_Note::E_WC_ADMIN_NOTE_ACTIONED !== $note->get_status() ) {
-						$note->set_status( WC_Admin_Note::E_WC_ADMIN_NOTE_ACTIONED );
+				$data_store       = WC_Data_Store::load('admin-note');
+				$failure_note_ids = $data_store->get_notes_with_name(self::FAILURE_NOTE_NAME);
+				if ( ! empty($failure_note_ids)) {
+					$note_id = array_pop($failure_note_ids);
+					$note    = WC_Admin_Notes::get_note($note_id);
+					if (false !== $note && WC_Admin_Note::E_WC_ADMIN_NOTE_ACTIONED !== $note->get_status()) {
+						$note->set_status(WC_Admin_Note::E_WC_ADMIN_NOTE_ACTIONED);
 						$note->save();
 					}
 				}
-			} catch ( Exception $e ) {}  // @codingStandardsIgnoreLine.
+			} catch (Exception $e) {}  // @codingStandardsIgnoreLine.
 		} else {
-			if ( empty( $failure_note_ids ) ) {
+			if (empty($failure_note_ids)) {
 				self::create_failure_note();
 			}
 		}
@@ -64,27 +64,27 @@ 
 	public static function should_show_marketing_note() {
 		// Display to US merchants only.
 		$base_location = wc_get_base_location();
-		if ( ! $base_location || 'US' !== $base_location['country'] ) {
+		if ( ! $base_location || 'US' !== $base_location['country']) {
 			return false;
 		}
 
 		// Make sure Apple Pay is enabled and setup is successful.
-		$stripe_settings       = get_option( 'woocommerce_stripe_settings', array() );
-		$stripe_enabled        = isset( $stripe_settings['enabled'] ) && 'yes' === $stripe_settings['enabled'];
-		$button_enabled        = isset( $stripe_settings['payment_request'] ) && 'yes' === $stripe_settings['payment_request'];
-		$verification_complete = isset( $stripe_settings['apple_pay_domain_set'] ) && 'yes' === $stripe_settings['apple_pay_domain_set'];
-		if ( ! $stripe_enabled || ! $button_enabled || ! $verification_complete ) {
+		$stripe_settings       = get_option('woocommerce_stripe_settings', array());
+		$stripe_enabled        = isset($stripe_settings['enabled']) && 'yes' === $stripe_settings['enabled'];
+		$button_enabled        = isset($stripe_settings['payment_request']) && 'yes' === $stripe_settings['payment_request'];
+		$verification_complete = isset($stripe_settings['apple_pay_domain_set']) && 'yes' === $stripe_settings['apple_pay_domain_set'];
+		if ( ! $stripe_enabled || ! $button_enabled || ! $verification_complete) {
 			return false;
 		}
 
 		// Make sure note doesn't already exist.
 		try {
-			$data_store       = WC_Data_Store::load( 'admin-note' );
-			$success_note_ids = $data_store->get_notes_with_name( self::SUCCESS_NOTE_NAME );
-			if ( ! empty( $success_note_ids ) ) {
+			$data_store       = WC_Data_Store::load('admin-note');
+			$success_note_ids = $data_store->get_notes_with_name(self::SUCCESS_NOTE_NAME);
+			if ( ! empty($success_note_ids)) {
 				return false;
 			}
-		} catch ( Exception $e ) {
+		} catch (Exception $e) {
 			return false; // If unable to check, assume it shouldn't show note.
 		}
 
@@ -96,19 +96,19 @@ 
 	 */
 	public static function create_marketing_note() {
 		// Make sure conditions for this note still hold.
-		if ( ! self::should_show_marketing_note() ) {
+		if ( ! self::should_show_marketing_note()) {
 			return;
 		}
 
 		$note = new WC_Admin_Note();
-		$note->set_title( __( 'Boost sales this holiday season with Apple Pay!', 'woocommerce-gateway-stripe' ) );
-		$note->set_content( __( 'Now that you accept Apple Pay® with Stripe, you can increase conversion rates by letting your customers know that Apple Pay is available. Here’s a marketing guide to help you get started.', 'woocommerce-gateway-stripe' ) );
-		$note->set_type( WC_Admin_Note::E_WC_ADMIN_NOTE_MARKETING );
-		$note->set_name( self::SUCCESS_NOTE_NAME );
-		$note->set_source( 'woocommerce-gateway-stripe' );
+		$note->set_title(__('Boost sales this holiday season with Apple Pay!', 'woocommerce-gateway-stripe'));
+		$note->set_content(__('Now that you accept Apple Pay® with Stripe, you can increase conversion rates by letting your customers know that Apple Pay is available. Here’s a marketing guide to help you get started.', 'woocommerce-gateway-stripe'));
+		$note->set_type(WC_Admin_Note::E_WC_ADMIN_NOTE_MARKETING);
+		$note->set_name(self::SUCCESS_NOTE_NAME);
+		$note->set_source('woocommerce-gateway-stripe');
 		$note->add_action(
 			'marketing-guide',
-			__( 'See marketing guide', 'woocommerce-gateway-stripe' ),
+			__('See marketing guide', 'woocommerce-gateway-stripe'),
 			'https://developer.apple.com/apple-pay/marketing/'
 		);
 		$note->save();
@@ -119,14 +119,14 @@ 
 	 */
 	public static function create_failure_note() {
 		$note = new WC_Admin_Note();
-		$note->set_title( __( 'Apple Pay domain verification needed', 'woocommerce-gateway-stripe' ) );
-		$note->set_content( __( 'The WooCommerce Stripe Gateway extension attempted to perform domain verification on behalf of your store, but was unable to do so. This must be resolved before Apple Pay can be offered to your customers.', 'woocommerce-gateway-stripe' ) );
-		$note->set_type( WC_Admin_Note::E_WC_ADMIN_NOTE_INFORMATIONAL );
-		$note->set_name( self::FAILURE_NOTE_NAME );
-		$note->set_source( 'woocommerce-gateway-stripe' );
+		$note->set_title(__('Apple Pay domain verification needed', 'woocommerce-gateway-stripe'));
+		$note->set_content(__('The WooCommerce Stripe Gateway extension attempted to perform domain verification on behalf of your store, but was unable to do so. This must be resolved before Apple Pay can be offered to your customers.', 'woocommerce-gateway-stripe'));
+		$note->set_type(WC_Admin_Note::E_WC_ADMIN_NOTE_INFORMATIONAL);
+		$note->set_name(self::FAILURE_NOTE_NAME);
+		$note->set_source('woocommerce-gateway-stripe');
 		$note->add_action(
 			'learn-more',
-			__( 'Learn more', 'woocommerce-gateway-stripe' ),
+			__('Learn more', 'woocommerce-gateway-stripe'),
 			'https://docs.woocommerce.com/document/stripe/#apple-pay'
 		);
 		$note->save();

tests/phpunit/helpers/class-wc-helper-order.php

Spacing +35 added lines, -35 removed lines

@@ -17,19 +17,19 @@ 
 	 *
 	 * @param int $order_id ID of the order to delete.
 	 */
-	public static function delete_order( $order_id ) {
+	public static function delete_order($order_id) {
 
-		$order = wc_get_order( $order_id );
+		$order = wc_get_order($order_id);
 
 		// Delete all products in the order.
-		foreach ( $order->get_items() as $item ) {
-			WC_Helper_Product::delete_product( $item['product_id'] );
+		foreach ($order->get_items() as $item) {
+			WC_Helper_Product::delete_product($item['product_id']);
 		}
 
 		WC_Helper_Shipping::delete_simple_flat_rate();
 
 		// Delete the order post.
-		$order->delete( true );
+		$order->delete(true);
 	}
 
 	/**
@@ -45,9 +45,9 @@ 
 	 *
 	 * @return WC_Order
 	 */
-	public static function create_order( $customer_id = 1, $total = 50, $product = null ) {
+	public static function create_order($customer_id = 1, $total = 50, $product = null) {
 
-		if ( ! is_a( $product, 'WC_Product' ) ) {
+		if ( ! is_a($product, 'WC_Product')) {
 			$product = WC_Helper_Product::create_simple_product();
 		}
 
@@ -61,7 +61,7 @@ 
 		];
 
 		$_SERVER['REMOTE_ADDR'] = '127.0.0.1'; // Required, else wc_create_order throws an exception.
-		$order                  = wc_create_order( $order_data );
+		$order                  = wc_create_order($order_data);
 
 		// Add order products.
 		$item = new WC_Order_Item_Product();
@@ -69,54 +69,54 @@ 
 			[
 				'product'  => $product,
 				'quantity' => 4,
-				'subtotal' => wc_get_price_excluding_tax( $product, [ 'qty' => 4 ] ),
-				'total'    => wc_get_price_excluding_tax( $product, [ 'qty' => 4 ] ),
+				'subtotal' => wc_get_price_excluding_tax($product, ['qty' => 4]),
+				'total'    => wc_get_price_excluding_tax($product, ['qty' => 4]),
 			]
 		);
 		$item->save();
-		$order->add_item( $item );
+		$order->add_item($item);
 
 		// Set billing address.
-		$order->set_billing_first_name( 'Jeroen' );
-		$order->set_billing_last_name( 'Sormani' );
-		$order->set_billing_company( 'WooCompany' );
-		$order->set_billing_address_1( 'WooAddress' );
-		$order->set_billing_address_2( '' );
-		$order->set_billing_city( 'WooCity' );
-		$order->set_billing_state( 'NY' );
-		$order->set_billing_postcode( '12345' );
-		$order->set_billing_country( 'US' );
-		$order->set_billing_email( 'admin@example.org' );
-		$order->set_billing_phone( '555-32123' );
+		$order->set_billing_first_name('Jeroen');
+		$order->set_billing_last_name('Sormani');
+		$order->set_billing_company('WooCompany');
+		$order->set_billing_address_1('WooAddress');
+		$order->set_billing_address_2('');
+		$order->set_billing_city('WooCity');
+		$order->set_billing_state('NY');
+		$order->set_billing_postcode('12345');
+		$order->set_billing_country('US');
+		$order->set_billing_email('admin@example.org');
+		$order->set_billing_phone('555-32123');
 
 		// Add shipping costs.
-		$shipping_taxes = WC_Tax::calc_shipping_tax( '10', WC_Tax::get_shipping_tax_rates() );
-		$rate           = new WC_Shipping_Rate( 'flat_rate_shipping', 'Flat rate shipping', '10', $shipping_taxes, 'flat_rate' );
+		$shipping_taxes = WC_Tax::calc_shipping_tax('10', WC_Tax::get_shipping_tax_rates());
+		$rate           = new WC_Shipping_Rate('flat_rate_shipping', 'Flat rate shipping', '10', $shipping_taxes, 'flat_rate');
 		$item           = new WC_Order_Item_Shipping();
 		$item->set_props(
 			[
 				'method_title' => $rate->label,
 				'method_id'    => $rate->id,
-				'total'        => wc_format_decimal( $rate->cost ),
+				'total'        => wc_format_decimal($rate->cost),
 				'taxes'        => $rate->taxes,
 			]
 		);
-		foreach ( $rate->get_meta_data() as $key => $value ) {
-			$item->add_meta_data( $key, $value, true );
+		foreach ($rate->get_meta_data() as $key => $value) {
+			$item->add_meta_data($key, $value, true);
 		}
-		$order->add_item( $item );
+		$order->add_item($item);
 
 		// Set payment gateway.
 		$payment_gateways = WC()->payment_gateways->payment_gateways();
-		$order->set_payment_method( $payment_gateways['bacs'] );
+		$order->set_payment_method($payment_gateways['bacs']);
 
 		// Set totals.
-		$order->set_shipping_total( 10 );
-		$order->set_discount_total( 0 );
-		$order->set_discount_tax( 0 );
-		$order->set_cart_tax( 0 );
-		$order->set_shipping_tax( 0 );
-		$order->set_total( $total );
+		$order->set_shipping_total(10);
+		$order->set_discount_total(0);
+		$order->set_discount_tax(0);
+		$order->set_cart_tax(0);
+		$order->set_shipping_tax(0);
+		$order->set_total($total);
 		$order->save();
 
 		return $order;

tests/phpunit/helpers/class-wc-helper-subscriptions-cart.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
 		return self::$cart_contains_subscription_result;
 	}
 
-	public static function set_cart_contains_subscription( $result ) {
+	public static function set_cart_contains_subscription($result) {
 		self::$cart_contains_subscription_result = $result;
 	}
 }

tests/phpunit/helpers/class-wc-helper-subscription.php

Spacing +2 added lines, -2 removed lines

@@ -18,11 +18,11 @@
 	 */
 	public $related_orders;
 
-	public function get_related_orders( $type ) {
+	public function get_related_orders($type) {
 		return $this->related_orders;
 	}
 
-	public function set_related_orders( $array ) {
+	public function set_related_orders($array) {
 		$this->related_orders = $array;
 	}
 }

tests/phpunit/helpers/class-wc-helper-shipping.php

Spacing +6 added lines, -6 removed lines

@@ -27,9 +27,9 @@ 
 			'cost'         => '10',
 		];
 
-		update_option( 'woocommerce_flat_rate_settings', $flat_rate_settings );
-		update_option( 'woocommerce_flat_rate', [] );
-		WC_Cache_Helper::get_transient_version( 'shipping', true );
+		update_option('woocommerce_flat_rate_settings', $flat_rate_settings);
+		update_option('woocommerce_flat_rate', []);
+		WC_Cache_Helper::get_transient_version('shipping', true);
 		WC()->shipping()->load_shipping_methods();
 	}
 
@@ -39,9 +39,9 @@ 
 	 * @since 2.3
 	 */
 	public static function delete_simple_flat_rate() {
-		delete_option( 'woocommerce_flat_rate_settings' );
-		delete_option( 'woocommerce_flat_rate' );
-		WC_Cache_Helper::get_transient_version( 'shipping', true );
+		delete_option('woocommerce_flat_rate_settings');
+		delete_option('woocommerce_flat_rate');
+		WC_Cache_Helper::get_transient_version('shipping', true);
 		WC()->shipping()->unregister_shipping_methods();
 	}
 }

tests/phpunit/helpers/class-wc-helper-token.php

Spacing +9 added lines, -9 removed lines

@@ -19,17 +19,17 @@
 	 * @param int    $user_id        ID of the token's user, defaults to get_current_user_id().
 	 * @param string $gateway        Token's Gateway ID, default to WC_Payment_Gateway_WCPay::GATEWAY_ID
 	 */
-	public static function create_token( $payment_method, $user_id = null, $gateway = WC_Payment_Gateway_WCPay::GATEWAY_ID ) {
+	public static function create_token($payment_method, $user_id = null, $gateway = WC_Payment_Gateway_WCPay::GATEWAY_ID) {
 		$token = new WC_Payment_Token_CC();
-		$token->set_token( $payment_method );
-		$token->set_gateway_id( $gateway );
-		$token->set_user_id( isset ( $user_id ) ? $user_id : get_current_user_id() );
-		$token->set_card_type( 'visa' );
-		$token->set_last4( '4242' );
-		$token->set_expiry_month( 6 );
-		$token->set_expiry_year( intval( gmdate( 'Y' ) ) + 1 );
+		$token->set_token($payment_method);
+		$token->set_gateway_id($gateway);
+		$token->set_user_id(isset ($user_id) ? $user_id : get_current_user_id());
+		$token->set_card_type('visa');
+		$token->set_last4('4242');
+		$token->set_expiry_month(6);
+		$token->set_expiry_year(intval(gmdate('Y')) + 1);
 		$token->save();
 
-		return WC_Payment_Tokens::get( $token->get_id() );
+		return WC_Payment_Tokens::get($token->get_id());
 	}
 }

tests/phpunit/helpers/class-wc-helper-subscriptions.php

Spacing +16 added lines, -16 removed lines

@@ -6,32 +6,32 @@ 
  */
 
 // Set up subscriptions mocks.
-function wcs_order_contains_subscription( $order ) {
-	if ( ! WC_Subscriptions::$wcs_order_contains_subscription ) {
+function wcs_order_contains_subscription($order) {
+	if ( ! WC_Subscriptions::$wcs_order_contains_subscription) {
 		return;
 	}
-	return call_user_func(  WC_Subscriptions::$wcs_order_contains_subscription, $order );
+	return call_user_func(WC_Subscriptions::$wcs_order_contains_subscription, $order);
 }
 
-function wcs_get_subscriptions_for_order( $order ) {
-	if ( ! WC_Subscriptions::$wcs_get_subscriptions_for_order ) {
+function wcs_get_subscriptions_for_order($order) {
+	if ( ! WC_Subscriptions::$wcs_get_subscriptions_for_order) {
 		return;
 	}
-	return call_user_func(  WC_Subscriptions::$wcs_get_subscriptions_for_order, $order );
+	return call_user_func(WC_Subscriptions::$wcs_get_subscriptions_for_order, $order);
 }
 
-function wcs_is_subscription( $order ) {
-	if ( ! WC_Subscriptions::$wcs_is_subscription ) {
+function wcs_is_subscription($order) {
+	if ( ! WC_Subscriptions::$wcs_is_subscription) {
 		return;
 	}
-	return call_user_func(  WC_Subscriptions::$wcs_is_subscription, $order );
+	return call_user_func(WC_Subscriptions::$wcs_is_subscription, $order);
 }
 
-function wcs_get_subscription( $subscription ) {
-	if ( ! WC_Subscriptions::$wcs_get_subscription ) {
+function wcs_get_subscription($subscription) {
+	if ( ! WC_Subscriptions::$wcs_get_subscription) {
 		return;
 	}
-	return call_user_func(  WC_Subscriptions::$wcs_get_subscription, $subscription );
+	return call_user_func(WC_Subscriptions::$wcs_get_subscription, $subscription);
 }
 
 /**
@@ -75,19 +75,19 @@ 
 	 */
 	public static $wcs_get_subscription = null;
 
-	public static function set_wcs_order_contains_subscription( $function ) {
+	public static function set_wcs_order_contains_subscription($function) {
 		self::$wcs_order_contains_subscription = $function;
 	}
 
-	public static function set_wcs_get_subscriptions_for_order( $function ) {
+	public static function set_wcs_get_subscriptions_for_order($function) {
 		self::$wcs_get_subscriptions_for_order = $function;
 	}
 
-	public static function set_wcs_is_subscription( $function ) {
+	public static function set_wcs_is_subscription($function) {
 		self::$wcs_is_subscription = $function;
 	}
 
-	public static function set_wcs_get_subscription( $function ) {
+	public static function set_wcs_get_subscription($function) {
 		self::$wcs_get_subscription = $function;
 	}
 }