woocommerce / woocommerce-gateway-stripe

includes/class-wc-stripe-api.php

Doc Comments -1 removed lines

@@ -23,7 +23,6 @@
 
 	/**
 	 * Set secret API Key.
-	 * @param string $key
 	 */
 	public static function set_secret_key( $secret_key ) {
 		self::$secret_key = $secret_key;

Spacing +48 added lines, -48 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -26,7 +26,7 @@ 
 	 * Set secret API Key.
 	 * @param string $key
 	 */
-	public static function set_secret_key( $secret_key ) {
+	public static function set_secret_key($secret_key) {
 		self::$secret_key = $secret_key;
 	}
 
@@ -35,11 +35,11 @@ 
 	 * @return string
 	 */
 	public static function get_secret_key() {
-		if ( ! self::$secret_key ) {
-			$options = get_option( 'woocommerce_stripe_settings' );
+		if ( ! self::$secret_key) {
+			$options = get_option('woocommerce_stripe_settings');
 
-			if ( isset( $options['testmode'], $options['secret_key'], $options['test_secret_key'] ) ) {
-				self::set_secret_key( 'yes' === $options['testmode'] ? $options['test_secret_key'] : $options['secret_key'] );
+			if (isset($options['testmode'], $options['secret_key'], $options['test_secret_key'])) {
+				self::set_secret_key('yes' === $options['testmode'] ? $options['test_secret_key'] : $options['secret_key']);
 			}
 		}
 		return self::$secret_key;
@@ -81,10 +81,10 @@ 
 		return apply_filters(
 			'woocommerce_stripe_request_headers',
 			array(
-				'Authorization'              => 'Basic ' . base64_encode( self::get_secret_key() . ':' ),
+				'Authorization'              => 'Basic ' . base64_encode(self::get_secret_key() . ':'),
 				'Stripe-Version'             => self::STRIPE_API_VERSION,
 				'User-Agent'                 => $app_info['name'] . '/' . $app_info['version'] . ' (' . $app_info['url'] . ')',
-				'X-Stripe-Client-User-Agent' => json_encode( $user_agent ),
+				'X-Stripe-Client-User-Agent' => json_encode($user_agent),
 			)
 		);
 	}
@@ -101,16 +101,16 @@ 
 	 * @return stdClass|array
 	 * @throws WC_Stripe_Exception
 	 */
-	public static function request( $request, $api = 'charges', $method = 'POST', $with_headers = false ) {
-		WC_Stripe_Logger::log( "{$api} request: " . print_r( $request, true ) );
+	public static function request($request, $api = 'charges', $method = 'POST', $with_headers = false) {
+		WC_Stripe_Logger::log("{$api} request: " . print_r($request, true));
 
 		$headers         = self::get_headers();
 		$idempotency_key = '';
 
-		if ( 'charges' === $api && 'POST' === $method ) {
-			$customer        = ! empty( $request['customer'] ) ? $request['customer'] : '';
-			$source          = ! empty( $request['source'] ) ? $request['source'] : $customer;
-			$idempotency_key = apply_filters( 'wc_stripe_idempotency_key', $request['metadata']['order_id'] . '-' . $source, $request );
+		if ('charges' === $api && 'POST' === $method) {
+			$customer        = ! empty($request['customer']) ? $request['customer'] : '';
+			$source          = ! empty($request['source']) ? $request['source'] : $customer;
+			$idempotency_key = apply_filters('wc_stripe_idempotency_key', $request['metadata']['order_id'] . '-' . $source, $request);
 
 			$headers['Idempotency-Key'] = $idempotency_key;
 		}
@@ -120,14 +120,14 @@ 
 			array(
 				'method'  => $method,
 				'headers' => $headers,
-				'body'    => apply_filters( 'woocommerce_stripe_request_body', $request, $api ),
+				'body'    => apply_filters('woocommerce_stripe_request_body', $request, $api),
 				'timeout' => 70,
 			)
 		);
 
-		if ( is_wp_error( $response ) || empty( $response['body'] ) ) {
+		if (is_wp_error($response) || empty($response['body'])) {
 			WC_Stripe_Logger::log(
-				'Error Response: ' . print_r( $response, true ) . PHP_EOL . PHP_EOL . 'Failed request: ' . print_r(
+				'Error Response: ' . print_r($response, true) . PHP_EOL . PHP_EOL . 'Failed request: ' . print_r(
 					array(
 						'api'             => $api,
 						'request'         => $request,
@@ -137,17 +137,17 @@ 
 				)
 			);
 
-			throw new WC_Stripe_Exception( print_r( $response, true ), __( 'There was a problem connecting to the Stripe API endpoint.', 'woocommerce-gateway-stripe' ) );
+			throw new WC_Stripe_Exception(print_r($response, true), __('There was a problem connecting to the Stripe API endpoint.', 'woocommerce-gateway-stripe'));
 		}
 
-		if ( $with_headers ) {
+		if ($with_headers) {
 			return array(
-				'headers' => wp_remote_retrieve_headers( $response ),
-				'body'    => json_decode( $response['body'] ),
+				'headers' => wp_remote_retrieve_headers($response),
+				'body'    => json_decode($response['body']),
 			);
 		}
 
-		return json_decode( $response['body'] );
+		return json_decode($response['body']);
 	}
 
 	/**
@@ -157,8 +157,8 @@ 
 	 * @version 4.0.0
 	 * @param string $api
 	 */
-	public static function retrieve( $api ) {
-		WC_Stripe_Logger::log( "{$api}" );
+	public static function retrieve($api) {
+		WC_Stripe_Logger::log("{$api}");
 
 		$response = wp_safe_remote_get(
 			self::ENDPOINT . $api,
@@ -169,12 +169,12 @@ 
 			)
 		);
 
-		if ( is_wp_error( $response ) || empty( $response['body'] ) ) {
-			WC_Stripe_Logger::log( 'Error Response: ' . print_r( $response, true ) );
-			return new WP_Error( 'stripe_error', __( 'There was a problem connecting to the Stripe API endpoint.', 'woocommerce-gateway-stripe' ) );
+		if (is_wp_error($response) || empty($response['body'])) {
+			WC_Stripe_Logger::log('Error Response: ' . print_r($response, true));
+			return new WP_Error('stripe_error', __('There was a problem connecting to the Stripe API endpoint.', 'woocommerce-gateway-stripe'));
 		}
 
-		return json_decode( $response['body'] );
+		return json_decode($response['body']);
 	}
 
 	/**
@@ -193,9 +193,9 @@ 
 	 *
 	 * @return stdClass|array The response
 	 */
-	public static function request_with_level3_data( $request, $api, $level3_data, $order ) {
+	public static function request_with_level3_data($request, $api, $level3_data, $order) {
 		// Do not add level3 data it's the array is empty.
-		if ( empty( $level3_data ) ) {
+		if (empty($level3_data)) {
 			return self::request(
 				$request,
 				$api
@@ -204,7 +204,7 @@ 
 
 		// If there's a transient indicating that level3 data was not accepted by
 		// Stripe in the past for this account, do not try to add level3 data.
-		if ( get_transient( 'wc_stripe_level3_not_allowed' ) ) {
+		if (get_transient('wc_stripe_level3_not_allowed')) {
 			return self::request(
 				$request,
 				$api
@@ -220,40 +220,40 @@ 
 		);
 
 		$is_level3_param_not_allowed = (
-			isset( $result->error )
-			&& isset( $result->error->code )
+			isset($result->error)
+			&& isset($result->error->code)
 			&& 'parameter_unknown' === $result->error->code
-			&& isset( $result->error->param )
+			&& isset($result->error->param)
 			&& 'level3' === $result->error->param
 		);
 
 		$is_level_3data_incorrect = (
-			isset( $result->error )
-			&& isset( $result->error->type )
+			isset($result->error)
+			&& isset($result->error->type)
 			&& 'invalid_request_error' === $result->error->type
 		);
 
-		if ( $is_level3_param_not_allowed ) {
+		if ($is_level3_param_not_allowed) {
 			// Set a transient so that future requests do not add level 3 data.
 			// Transient is set to expire in 3 months, can be manually removed if needed.
-			set_transient( 'wc_stripe_level3_not_allowed', true, 3 * MONTH_IN_SECONDS );
-		} else if ( $is_level_3data_incorrect ) {
+			set_transient('wc_stripe_level3_not_allowed', true, 3 * MONTH_IN_SECONDS);
+		} else if ($is_level_3data_incorrect) {
 			// Log the issue so we could debug it.
 			WC_Stripe_Logger::log(
 				'Level3 data sum incorrect: ' . PHP_EOL
-				. print_r( $result->error->message, true ) . PHP_EOL
-				. print_r( 'Order line items: ', true ) . PHP_EOL
-				. print_r( $order->get_items(), true ) . PHP_EOL
-				. print_r( 'Order shipping amount: ', true ) . PHP_EOL
-				. print_r( $order->get_shipping_total(), true ) . PHP_EOL
-				. print_r( 'Order currency: ', true ) . PHP_EOL
-				. print_r( $order->get_currency(), true )
+				. print_r($result->error->message, true) . PHP_EOL
+				. print_r('Order line items: ', true) . PHP_EOL
+				. print_r($order->get_items(), true) . PHP_EOL
+				. print_r('Order shipping amount: ', true) . PHP_EOL
+				. print_r($order->get_shipping_total(), true) . PHP_EOL
+				. print_r('Order currency: ', true) . PHP_EOL
+				. print_r($order->get_currency(), true)
 			);
 		}
 
 		// Make the request again without level 3 data.
-		if ( $is_level3_param_not_allowed || $is_level_3data_incorrect ) {
-			unset( $request['level3'] );
+		if ($is_level3_param_not_allowed || $is_level_3data_incorrect) {
+			unset($request['level3']);
 			return WC_Stripe_API::request(
 				$request,
 				$api

includes/payment-methods/class-wc-gateway-stripe-alipay.php

Doc Comments +1 added lines, -1 removed lines

@@ -289,7 +289,7 @@
 	 *
 	 * @param int  $order_id Reference.
 	 * @param bool $retry Should we retry on fail.
-	 * @param bool $force_save_source Force payment source to be saved.
+	 * @param bool $force_save_save Force payment source to be saved.
 	 *
 	 * @throws Exception If payment will not be accepted.
 	 *

Spacing +63 added lines, -63 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -57,9 +57,9 @@ 
 	 */
 	public function __construct() {
 		$this->id           = 'stripe_alipay';
-		$this->method_title = __( 'Stripe Alipay', 'woocommerce-gateway-stripe' );
+		$this->method_title = __('Stripe Alipay', 'woocommerce-gateway-stripe');
 		/* translators: link */
-		$this->method_description = sprintf( __( 'All other general Stripe settings can be adjusted <a href="%s">here</a>.', 'woocommerce-gateway-stripe' ), admin_url( 'admin.php?page=wc-settings&tab=checkout&section=stripe' ) );
+		$this->method_description = sprintf(__('All other general Stripe settings can be adjusted <a href="%s">here</a>.', 'woocommerce-gateway-stripe'), admin_url('admin.php?page=wc-settings&tab=checkout&section=stripe'));
 		$this->supports           = array(
 			'products',
 			'refunds',
@@ -71,23 +71,23 @@ 
 		// Load the settings.
 		$this->init_settings();
 
-		$main_settings              = get_option( 'woocommerce_stripe_settings' );
-		$this->title                = $this->get_option( 'title' );
-		$this->description          = $this->get_option( 'description' );
-		$this->enabled              = $this->get_option( 'enabled' );
-		$this->testmode             = ( ! empty( $main_settings['testmode'] ) && 'yes' === $main_settings['testmode'] ) ? true : false;
-		$this->saved_cards          = ( ! empty( $main_settings['saved_cards'] ) && 'yes' === $main_settings['saved_cards'] ) ? true : false;
-		$this->publishable_key      = ! empty( $main_settings['publishable_key'] ) ? $main_settings['publishable_key'] : '';
-		$this->secret_key           = ! empty( $main_settings['secret_key'] ) ? $main_settings['secret_key'] : '';
-		$this->statement_descriptor = ! empty( $main_settings['statement_descriptor'] ) ? $main_settings['statement_descriptor'] : '';
-
-		if ( $this->testmode ) {
-			$this->publishable_key = ! empty( $main_settings['test_publishable_key'] ) ? $main_settings['test_publishable_key'] : '';
-			$this->secret_key      = ! empty( $main_settings['test_secret_key'] ) ? $main_settings['test_secret_key'] : '';
+		$main_settings              = get_option('woocommerce_stripe_settings');
+		$this->title                = $this->get_option('title');
+		$this->description          = $this->get_option('description');
+		$this->enabled              = $this->get_option('enabled');
+		$this->testmode             = ( ! empty($main_settings['testmode']) && 'yes' === $main_settings['testmode']) ? true : false;
+		$this->saved_cards          = ( ! empty($main_settings['saved_cards']) && 'yes' === $main_settings['saved_cards']) ? true : false;
+		$this->publishable_key      = ! empty($main_settings['publishable_key']) ? $main_settings['publishable_key'] : '';
+		$this->secret_key           = ! empty($main_settings['secret_key']) ? $main_settings['secret_key'] : '';
+		$this->statement_descriptor = ! empty($main_settings['statement_descriptor']) ? $main_settings['statement_descriptor'] : '';
+
+		if ($this->testmode) {
+			$this->publishable_key = ! empty($main_settings['test_publishable_key']) ? $main_settings['test_publishable_key'] : '';
+			$this->secret_key      = ! empty($main_settings['test_secret_key']) ? $main_settings['test_secret_key'] : '';
 		}
 
-		add_action( 'woocommerce_update_options_payment_gateways_' . $this->id, array( $this, 'process_admin_options' ) );
-		add_action( 'wp_enqueue_scripts', array( $this, 'payment_scripts' ) );
+		add_action('woocommerce_update_options_payment_gateways_' . $this->id, array($this, 'process_admin_options'));
+		add_action('wp_enqueue_scripts', array($this, 'payment_scripts'));
 	}
 
 	/**
@@ -122,7 +122,7 @@ 
 	 * @return bool
 	 */
 	public function is_available() {
-		if ( ! in_array( get_woocommerce_currency(), $this->get_supported_currency() ) ) {
+		if ( ! in_array(get_woocommerce_currency(), $this->get_supported_currency())) {
 			return false;
 		}
 
@@ -141,9 +141,9 @@ 
 
 		$icons_str = '';
 
-		$icons_str .= isset( $icons['alipay'] ) ? $icons['alipay'] : '';
+		$icons_str .= isset($icons['alipay']) ? $icons['alipay'] : '';
 
-		return apply_filters( 'woocommerce_gateway_icon', $icons_str, $this->id );
+		return apply_filters('woocommerce_gateway_icon', $icons_str, $this->id);
 	}
 
 	/**
@@ -153,19 +153,19 @@ 
 	 * @version 4.0.0
 	 */
 	public function payment_scripts() {
-		if ( ! is_cart() && ! is_checkout() && ! isset( $_GET['pay_for_order'] ) && ! is_add_payment_method_page() ) {
+		if ( ! is_cart() && ! is_checkout() && ! isset($_GET['pay_for_order']) && ! is_add_payment_method_page()) {
 			return;
 		}
 
-		wp_enqueue_style( 'stripe_styles' );
-		wp_enqueue_script( 'woocommerce_stripe' );
+		wp_enqueue_style('stripe_styles');
+		wp_enqueue_script('woocommerce_stripe');
 	}
 
 	/**
 	 * Initialize Gateway Settings Form Fields.
 	 */
 	public function init_form_fields() {
-		$this->form_fields = require( WC_STRIPE_PLUGIN_PATH . '/includes/admin/stripe-alipay-settings.php' );
+		$this->form_fields = require(WC_STRIPE_PLUGIN_PATH . '/includes/admin/stripe-alipay-settings.php');
 	}
 
 	/**
@@ -178,13 +178,13 @@ 
 		$description = $this->get_description();
 
 		// If paying from order, we need to get total from order not cart.
-		if ( isset( $_GET['pay_for_order'] ) && ! empty( $_GET['key'] ) ) {
-			$order = wc_get_order( wc_clean( $wp->query_vars['order-pay'] ) );
+		if (isset($_GET['pay_for_order']) && ! empty($_GET['key'])) {
+			$order = wc_get_order(wc_clean($wp->query_vars['order-pay']));
 			$total = $order->get_total();
 		}
 
-		if ( is_add_payment_method_page() ) {
-			$pay_button_text = __( 'Add Payment', 'woocommerce-gateway-stripe' );
+		if (is_add_payment_method_page()) {
+			$pay_button_text = __('Add Payment', 'woocommerce-gateway-stripe');
 			$total           = '';
 		} else {
 			$pay_button_text = '';
@@ -192,11 +192,11 @@ 
 
 		echo '<div
 			id="stripe-alipay-payment-data"
-			data-amount="' . esc_attr( WC_Stripe_Helper::get_stripe_amount( $total ) ) . '"
-			data-currency="' . esc_attr( strtolower( get_woocommerce_currency() ) ) . '">';
+			data-amount="' . esc_attr(WC_Stripe_Helper::get_stripe_amount($total)) . '"
+			data-currency="' . esc_attr(strtolower(get_woocommerce_currency())) . '">';
 
-		if ( $description ) {
-			echo apply_filters( 'wc_stripe_description', wpautop( wp_kses_post( $description ) ), $this->id );
+		if ($description) {
+			echo apply_filters('wc_stripe_description', wpautop(wp_kses_post($description)), $this->id);
 		}
 
 		echo '</div>';
@@ -210,23 +210,23 @@ 
 	 * @param object $order
 	 * @return mixed
 	 */
-	public function create_source( $order ) {
+	public function create_source($order) {
 		$currency              = $order->get_currency();
-		$return_url            = $this->get_stripe_return_url( $order );
+		$return_url            = $this->get_stripe_return_url($order);
 		$post_data             = array();
-		$post_data['amount']   = WC_Stripe_Helper::get_stripe_amount( $order->get_total(), $currency );
-		$post_data['currency'] = strtolower( $currency );
+		$post_data['amount']   = WC_Stripe_Helper::get_stripe_amount($order->get_total(), $currency);
+		$post_data['currency'] = strtolower($currency);
 		$post_data['type']     = 'alipay';
-		$post_data['owner']    = $this->get_owner_details( $order );
-		$post_data['redirect'] = array( 'return_url' => $return_url );
+		$post_data['owner']    = $this->get_owner_details($order);
+		$post_data['redirect'] = array('return_url' => $return_url);
 
-		if ( ! empty( $this->statement_descriptor ) ) {
-			$post_data['statement_descriptor'] = WC_Stripe_Helper::clean_statement_descriptor( $this->statement_descriptor );
+		if ( ! empty($this->statement_descriptor)) {
+			$post_data['statement_descriptor'] = WC_Stripe_Helper::clean_statement_descriptor($this->statement_descriptor);
 		}
 
-		WC_Stripe_Logger::log( 'Info: Begin creating Alipay source' );
+		WC_Stripe_Logger::log('Info: Begin creating Alipay source');
 
-		return WC_Stripe_API::request( apply_filters( 'wc_stripe_alipay_source', $post_data, $order ), 'sources' );
+		return WC_Stripe_API::request(apply_filters('wc_stripe_alipay_source', $post_data, $order), 'sources');
 	}
 
 	/**
@@ -240,49 +240,49 @@ 
 	 *
 	 * @return array|void
 	 */
-	public function process_payment( $order_id, $retry = true, $force_save_save = false ) {
+	public function process_payment($order_id, $retry = true, $force_save_save = false) {
 		try {
-			$order = wc_get_order( $order_id );
+			$order = wc_get_order($order_id);
 
 			// This will throw exception if not valid.
-			$this->validate_minimum_order_amount( $order );
+			$this->validate_minimum_order_amount($order);
 
 			// This comes from the create account checkbox in the checkout page.
-			$create_account = ! empty( $_POST['createaccount'] ) ? true : false;
+			$create_account = ! empty($_POST['createaccount']) ? true : false;
 
-			if ( $create_account ) {
+			if ($create_account) {
 				$new_customer_id     = $order->get_customer_id();
-				$new_stripe_customer = new WC_Stripe_Customer( $new_customer_id );
+				$new_stripe_customer = new WC_Stripe_Customer($new_customer_id);
 				$new_stripe_customer->create_customer();
 			}
 
-			$response = $this->create_source( $order );
+			$response = $this->create_source($order);
 
-			if ( ! empty( $response->error ) ) {
-				$order->add_order_note( $response->error->message );
+			if ( ! empty($response->error)) {
+				$order->add_order_note($response->error->message);
 
-				throw new WC_Stripe_Exception( print_r( $response, true ), $response->error->message );
+				throw new WC_Stripe_Exception(print_r($response, true), $response->error->message);
 			}
 
-			$order->update_meta_data( '_stripe_source_id', $response->id );
+			$order->update_meta_data('_stripe_source_id', $response->id);
 			$order->save();
 
-			WC_Stripe_Logger::log( 'Info: Redirecting to Alipay...' );
+			WC_Stripe_Logger::log('Info: Redirecting to Alipay...');
 
 			return array(
 				'result'   => 'success',
-				'redirect' => esc_url_raw( $response->redirect->url ),
+				'redirect' => esc_url_raw($response->redirect->url),
 			);
-		} 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());
 
-			do_action( 'wc_gateway_stripe_process_payment_error', $e, $order );
+			do_action('wc_gateway_stripe_process_payment_error', $e, $order);
 
-			$statuses = array( 'pending', 'failed' );
+			$statuses = array('pending', 'failed');
 
-			if ( $order->has_status( $statuses ) ) {
-				$this->send_failed_order_email( $order_id );
+			if ($order->has_status($statuses)) {
+				$this->send_failed_order_email($order_id);
 			}
 
 			return array(

includes/class-wc-stripe-sepa-payment-token.php

Spacing +9 added lines, -9 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
@@ -35,10 +35,10 @@ 
 	 * @param  string $deprecated Deprecated since WooCommerce 3.0
 	 * @return string
 	 */
-	public function get_display_name( $deprecated = '' ) {
+	public function get_display_name($deprecated = '') {
 		$display = sprintf(
 			/* translators: last 4 digits of IBAN account */
-			__( 'SEPA IBAN ending in %s', 'woocommerce-gateway-stripe' ),
+			__('SEPA IBAN ending in %s', 'woocommerce-gateway-stripe'),
 			$this->get_last4()
 		);
 
@@ -66,11 +66,11 @@ 
 	 * @return boolean True if the passed data is valid
 	 */
 	public function validate() {
-		if ( false === parent::validate() ) {
+		if (false === parent::validate()) {
 			return false;
 		}
 
-		if ( ! $this->get_last4( 'edit' ) ) {
+		if ( ! $this->get_last4('edit')) {
 			return false;
 		}
 
@@ -85,8 +85,8 @@ 
 	 * @param  string $context
 	 * @return string Last 4 digits
 	 */
-	public function get_last4( $context = 'view' ) {
-		return $this->get_prop( 'last4', $context );
+	public function get_last4($context = 'view') {
+		return $this->get_prop('last4', $context);
 	}
 
 	/**
@@ -95,7 +95,7 @@ 
 	 * @version 4.0.0
 	 * @param string $last4
 	 */
-	public function set_last4( $last4 ) {
-		$this->set_prop( 'last4', $last4 );
+	public function set_last4($last4) {
+		$this->set_prop('last4', $last4);
 	}
 }

tests/phpunit/test-wc-stripe.php

Spacing +21 added lines, -21 removed lines

@@ -2,12 +2,12 @@ 
 
 class WC_Stripe_Test extends WP_UnitTestCase {
 	public function test_constants_defined() {
-		$this->assertTrue( defined( 'WC_STRIPE_VERSION' ) );
-		$this->assertTrue( defined( 'WC_STRIPE_MIN_PHP_VER' ) );
-		$this->assertTrue( defined( 'WC_STRIPE_MIN_WC_VER' ) );
-		$this->assertTrue( defined( 'WC_STRIPE_MAIN_FILE' ) );
-		$this->assertTrue( defined( 'WC_STRIPE_PLUGIN_URL' ) );
-		$this->assertTrue( defined( 'WC_STRIPE_PLUGIN_PATH' ) );
+		$this->assertTrue(defined('WC_STRIPE_VERSION'));
+		$this->assertTrue(defined('WC_STRIPE_MIN_PHP_VER'));
+		$this->assertTrue(defined('WC_STRIPE_MIN_WC_VER'));
+		$this->assertTrue(defined('WC_STRIPE_MAIN_FILE'));
+		$this->assertTrue(defined('WC_STRIPE_PLUGIN_URL'));
+		$this->assertTrue(defined('WC_STRIPE_PLUGIN_PATH'));
 	}
 
 	/**
@@ -15,11 +15,11 @@ 
 	 * This test will see if we're indeed converting the price correctly.
 	 */
 	public function test_price_conversion_before_send_to_stripe() {
-		$this->assertEquals( 10050, WC_Stripe_Helper::get_stripe_amount( 100.50, 'USD' ) );
-		$this->assertEquals( 10050, WC_Stripe_Helper::get_stripe_amount( 10050, 'JPY' ) );
-		$this->assertEquals( 100, WC_Stripe_Helper::get_stripe_amount( 100.50, 'JPY' ) );
-		$this->assertEquals( 10050, WC_Stripe_Helper::get_stripe_amount( 100.50 ) );
-		$this->assertInternalType( 'int', WC_Stripe_Helper::get_stripe_amount( 100.50, 'USD' ) );
+		$this->assertEquals(10050, WC_Stripe_Helper::get_stripe_amount(100.50, 'USD'));
+		$this->assertEquals(10050, WC_Stripe_Helper::get_stripe_amount(10050, 'JPY'));
+		$this->assertEquals(100, WC_Stripe_Helper::get_stripe_amount(100.50, 'JPY'));
+		$this->assertEquals(10050, WC_Stripe_Helper::get_stripe_amount(100.50));
+		$this->assertInternalType('int', WC_Stripe_Helper::get_stripe_amount(100.50, 'USD'));
 	}
 
 	/**
@@ -33,37 +33,37 @@ 
 		$balance_fee1->net = 10000;
 		$balance_fee1->currency = 'USD';
 
-		$this->assertEquals( 105.00, WC_Stripe_Helper::format_balance_fee( $balance_fee1, 'fee' ) );
+		$this->assertEquals(105.00, WC_Stripe_Helper::format_balance_fee($balance_fee1, 'fee'));
 
 		$balance_fee2 = new stdClass();
 		$balance_fee2->fee = 10500;
 		$balance_fee2->net = 10000;
 		$balance_fee2->currency = 'JPY';
 
-		$this->assertEquals( 10500, WC_Stripe_Helper::format_balance_fee( $balance_fee2, 'fee' ) );
+		$this->assertEquals(10500, WC_Stripe_Helper::format_balance_fee($balance_fee2, 'fee'));
 
 		$balance_fee3 = new stdClass();
 		$balance_fee3->fee = 10500;
 		$balance_fee3->net = 10000;
 		$balance_fee3->currency = 'USD';
 
-		$this->assertEquals( 100.00, WC_Stripe_Helper::format_balance_fee( $balance_fee3, 'net' ) );
+		$this->assertEquals(100.00, WC_Stripe_Helper::format_balance_fee($balance_fee3, 'net'));
 
 		$balance_fee4 = new stdClass();
 		$balance_fee4->fee = 10500;
 		$balance_fee4->net = 10000;
 		$balance_fee4->currency = 'JPY';
 
-		$this->assertEquals( 10000, WC_Stripe_Helper::format_balance_fee( $balance_fee4, 'net' ) );
+		$this->assertEquals(10000, WC_Stripe_Helper::format_balance_fee($balance_fee4, 'net'));
 
 		$balance_fee5 = new stdClass();
 		$balance_fee5->fee = 10500;
 		$balance_fee5->net = 10000;
 		$balance_fee5->currency = 'USD';
 
-		$this->assertEquals( 105.00, WC_Stripe_Helper::format_balance_fee( $balance_fee5 ) );
+		$this->assertEquals(105.00, WC_Stripe_Helper::format_balance_fee($balance_fee5));
 
-		$this->assertInternalType( 'string', WC_Stripe_Helper::format_balance_fee( $balance_fee5 ) );
+		$this->assertInternalType('string', WC_Stripe_Helper::format_balance_fee($balance_fee5));
 	}
 
 	/**
@@ -76,27 +76,27 @@ 
 			'expected' => 'Tests Store',
 		);
 
-		$this->assertEquals( $statement_descriptor1['expected'], WC_Stripe_Helper::clean_statement_descriptor( $statement_descriptor1['actual'] ) );
+		$this->assertEquals($statement_descriptor1['expected'], WC_Stripe_Helper::clean_statement_descriptor($statement_descriptor1['actual']));
 
 		$statement_descriptor2 = array(
 			'actual'   => 'Test\'s Store > Driving Course Range',
 			'expected' => 'Tests Store  Driving C',
 		);
 
-		$this->assertEquals( $statement_descriptor2['expected'], WC_Stripe_Helper::clean_statement_descriptor( $statement_descriptor2['actual'] ) );
+		$this->assertEquals($statement_descriptor2['expected'], WC_Stripe_Helper::clean_statement_descriptor($statement_descriptor2['actual']));
 
 		$statement_descriptor3 = array(
 			'actual'   => 'Test\'s Store < Driving Course Range',
 			'expected' => 'Tests Store  Driving C',
 		);
 
-		$this->assertEquals( $statement_descriptor3['expected'], WC_Stripe_Helper::clean_statement_descriptor( $statement_descriptor3['actual'] ) );
+		$this->assertEquals($statement_descriptor3['expected'], WC_Stripe_Helper::clean_statement_descriptor($statement_descriptor3['actual']));
 
 		$statement_descriptor4 = array(
 			'actual'   => 'Test\'s Store " Driving Course Range',
 			'expected' => 'Tests Store  Driving C',
 		);
 
-		$this->assertEquals( $statement_descriptor4['expected'], WC_Stripe_Helper::clean_statement_descriptor( $statement_descriptor4['actual'] ) );
+		$this->assertEquals($statement_descriptor4['expected'], WC_Stripe_Helper::clean_statement_descriptor($statement_descriptor4['actual']));
 	}
 }

includes/class-wc-stripe-customer.php

Doc Comments -1 removed lines

@@ -213,7 +213,6 @@
 	/**
 	 * Get a customers saved sources using their Stripe ID.
 	 *
-	 * @param  string $customer_id
 	 * @return array
 	 */
 	public function get_sources() {

Spacing +109 added lines, -109 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -32,10 +32,10 @@ 
 	 * Constructor
 	 * @param int $user_id The WP user ID
 	 */
-	public function __construct( $user_id = 0 ) {
-		if ( $user_id ) {
-			$this->set_user_id( $user_id );
-			$this->set_id( $this->get_id_from_meta( $user_id ) );
+	public function __construct($user_id = 0) {
+		if ($user_id) {
+			$this->set_user_id($user_id);
+			$this->set_id($this->get_id_from_meta($user_id));
 		}
 	}
 
@@ -51,15 +51,15 @@ 
 	 * Set Stripe customer ID.
 	 * @param [type] $id [description]
 	 */
-	public function set_id( $id ) {
+	public function set_id($id) {
 		// Backwards compat for customer ID stored in array format. (Pre 3.0)
-		if ( is_array( $id ) && isset( $id['customer_id'] ) ) {
+		if (is_array($id) && isset($id['customer_id'])) {
 			$id = $id['customer_id'];
 
-			$this->update_id_in_meta( $id );
+			$this->update_id_in_meta($id);
 		}
 
-		$this->id = wc_clean( $id );
+		$this->id = wc_clean($id);
 	}
 
 	/**
@@ -67,15 +67,15 @@ 
 	 * @return int
 	 */
 	public function get_user_id() {
-		return absint( $this->user_id );
+		return absint($this->user_id);
 	}
 
 	/**
 	 * Set User ID used by WordPress.
 	 * @param int $user_id
 	 */
-	public function set_user_id( $user_id ) {
-		$this->user_id = absint( $user_id );
+	public function set_user_id($user_id) {
+		$this->user_id = absint($user_id);
 	}
 
 	/**
@@ -83,13 +83,13 @@ 
 	 * @return WP_User
 	 */
 	protected function get_user() {
-		return $this->get_user_id() ? get_user_by( 'id', $this->get_user_id() ) : false;
+		return $this->get_user_id() ? get_user_by('id', $this->get_user_id()) : false;
 	}
 
 	/**
 	 * Store data from the Stripe API about this customer
 	 */
-	public function set_customer_data( $data ) {
+	public function set_customer_data($data) {
 		$this->customer_data = $data;
 	}
 
@@ -99,58 +99,58 @@ 
 	 * @param  array $args Additional arguments (optional).
 	 * @return array
 	 */
-	protected function generate_customer_request( $args = array() ) {
-		$billing_email = isset( $_POST['billing_email'] ) ? filter_var( $_POST['billing_email'], FILTER_SANITIZE_EMAIL ) : '';
+	protected function generate_customer_request($args = array()) {
+		$billing_email = isset($_POST['billing_email']) ? filter_var($_POST['billing_email'], FILTER_SANITIZE_EMAIL) : '';
 		$user          = $this->get_user();
 
-		if ( $user ) {
-			$billing_first_name = get_user_meta( $user->ID, 'billing_first_name', true );
-			$billing_last_name  = get_user_meta( $user->ID, 'billing_last_name', true );
+		if ($user) {
+			$billing_first_name = get_user_meta($user->ID, 'billing_first_name', true);
+			$billing_last_name  = get_user_meta($user->ID, 'billing_last_name', true);
 
 			// If billing first name does not exists try the user first name.
-			if ( empty( $billing_first_name ) ) {
-				$billing_first_name = get_user_meta( $user->ID, 'first_name', true );
+			if (empty($billing_first_name)) {
+				$billing_first_name = get_user_meta($user->ID, 'first_name', true);
 			}
 
 			// If billing last name does not exists try the user last name.
-			if ( empty( $billing_last_name ) ) {
-				$billing_last_name = get_user_meta( $user->ID, 'last_name', true );
+			if (empty($billing_last_name)) {
+				$billing_last_name = get_user_meta($user->ID, 'last_name', true);
 			}
 
 			// translators: %1$s First name, %2$s Second name, %3$s Username.
-			$description = sprintf( __( 'Name: %1$s %2$s, Username: %s', 'woocommerce-gateway-stripe' ), $billing_first_name, $billing_last_name, $user->user_login );
+			$description = sprintf(__('Name: %1$s %2$s, Username: %s', 'woocommerce-gateway-stripe'), $billing_first_name, $billing_last_name, $user->user_login);
 
 			$defaults = array(
 				'email'       => $user->user_email,
 				'description' => $description,
 			);
 
-			$billing_full_name = trim( $billing_first_name . ' ' . $billing_last_name );
-			if ( ! empty( $billing_full_name ) ) {
+			$billing_full_name = trim($billing_first_name . ' ' . $billing_last_name);
+			if ( ! empty($billing_full_name)) {
 				$defaults['name'] = $billing_full_name;
 			}
 		} else {
-			$billing_first_name = isset( $_POST['billing_first_name'] ) ? filter_var( wp_unslash( $_POST['billing_first_name'] ), FILTER_SANITIZE_STRING ) : ''; // phpcs:ignore WordPress.Security.NonceVerification
-			$billing_last_name  = isset( $_POST['billing_last_name'] ) ? filter_var( wp_unslash( $_POST['billing_last_name'] ), FILTER_SANITIZE_STRING ) : ''; // phpcs:ignore WordPress.Security.NonceVerification
+			$billing_first_name = isset($_POST['billing_first_name']) ? filter_var(wp_unslash($_POST['billing_first_name']), FILTER_SANITIZE_STRING) : ''; // phpcs:ignore WordPress.Security.NonceVerification
+			$billing_last_name  = isset($_POST['billing_last_name']) ? filter_var(wp_unslash($_POST['billing_last_name']), FILTER_SANITIZE_STRING) : ''; // phpcs:ignore WordPress.Security.NonceVerification
 
 			// translators: %1$s First name, %2$s Second name.
-			$description = sprintf( __( 'Name: %1$s %2$s, Guest', 'woocommerce-gateway-stripe' ), $billing_first_name, $billing_last_name );
+			$description = sprintf(__('Name: %1$s %2$s, Guest', 'woocommerce-gateway-stripe'), $billing_first_name, $billing_last_name);
 
 			$defaults = array(
 				'email'       => $billing_email,
 				'description' => $description,
 			);
 
-			$billing_full_name = trim( $billing_first_name . ' ' . $billing_last_name );
-			if ( ! empty( $billing_full_name ) ) {
+			$billing_full_name = trim($billing_first_name . ' ' . $billing_last_name);
+			if ( ! empty($billing_full_name)) {
 				$defaults['name'] = $billing_full_name;
 			}
 		}
 
 		$metadata             = array();
-		$defaults['metadata'] = apply_filters( 'wc_stripe_customer_metadata', $metadata, $user );
+		$defaults['metadata'] = apply_filters('wc_stripe_customer_metadata', $metadata, $user);
 
-		return wp_parse_args( $args, $defaults );
+		return wp_parse_args($args, $defaults);
 	}
 
 	/**
@@ -158,23 +158,23 @@ 
 	 * @param array $args
 	 * @return WP_Error|int
 	 */
-	public function create_customer( $args = array() ) {
-		$args     = $this->generate_customer_request( $args );
-		$response = WC_Stripe_API::request( apply_filters( 'wc_stripe_create_customer_args', $args ), 'customers' );
+	public function create_customer($args = array()) {
+		$args     = $this->generate_customer_request($args);
+		$response = WC_Stripe_API::request(apply_filters('wc_stripe_create_customer_args', $args), 'customers');
 
-		if ( ! empty( $response->error ) ) {
-			throw new WC_Stripe_Exception( print_r( $response, true ), $response->error->message );
+		if ( ! empty($response->error)) {
+			throw new WC_Stripe_Exception(print_r($response, true), $response->error->message);
 		}
 
-		$this->set_id( $response->id );
+		$this->set_id($response->id);
 		$this->clear_cache();
-		$this->set_customer_data( $response );
+		$this->set_customer_data($response);
 
-		if ( $this->get_user_id() ) {
-			$this->update_id_in_meta( $response->id );
+		if ($this->get_user_id()) {
+			$this->update_id_in_meta($response->id);
 		}
 
-		do_action( 'woocommerce_stripe_add_customer', $args, $response );
+		do_action('woocommerce_stripe_add_customer', $args, $response);
 
 		return $response->id;
 	}
@@ -189,30 +189,30 @@ 
 	 *
 	 * @throws WC_Stripe_Exception
 	 */
-	public function update_customer( $args = array(), $is_retry = false ) {
-		if ( empty( $this->get_id() ) ) {
-			throw new WC_Stripe_Exception( 'id_required_to_update_user', __( 'Attempting to update a Stripe customer without a customer ID.', 'woocommerce-gateway-stripe' ) );
+	public function update_customer($args = array(), $is_retry = false) {
+		if (empty($this->get_id())) {
+			throw new WC_Stripe_Exception('id_required_to_update_user', __('Attempting to update a Stripe customer without a customer ID.', 'woocommerce-gateway-stripe'));
 		}
 
-		$args     = $this->generate_customer_request( $args );
-		$args     = apply_filters( 'wc_stripe_update_customer_args', $args );
-		$response = WC_Stripe_API::request( $args, 'customers/' . $this->get_id() );
+		$args     = $this->generate_customer_request($args);
+		$args     = apply_filters('wc_stripe_update_customer_args', $args);
+		$response = WC_Stripe_API::request($args, 'customers/' . $this->get_id());
 
-		if ( ! empty( $response->error ) ) {
-			if ( $this->is_no_such_customer_error( $response->error ) && ! $is_retry ) {
+		if ( ! empty($response->error)) {
+			if ($this->is_no_such_customer_error($response->error) && ! $is_retry) {
 				// This can happen when switching the main Stripe account or importing users from another site.
 				// If not already retrying, recreate the customer and then try updating it again.
 				$this->recreate_customer();
-				return $this->update_customer( $args, true );
+				return $this->update_customer($args, true);
 			}
 
-			throw new WC_Stripe_Exception( print_r( $response, true ), $response->error->message );
+			throw new WC_Stripe_Exception(print_r($response, true), $response->error->message);
 		}
 
 		$this->clear_cache();
-		$this->set_customer_data( $response );
+		$this->set_customer_data($response);
 
-		do_action( 'woocommerce_stripe_update_customer', $args, $response );
+		do_action('woocommerce_stripe_update_customer', $args, $response);
 
 		return $this->get_id();
 	}
@@ -224,11 +224,11 @@ 
 	 * @since 4.1.2
 	 * @param array $error
 	 */
-	public function is_no_such_customer_error( $error ) {
+	public function is_no_such_customer_error($error) {
 		return (
 			$error &&
 			'invalid_request_error' === $error->type &&
-			preg_match( '/No such customer/i', $error->message )
+			preg_match('/No such customer/i', $error->message)
 		);
 	}
 
@@ -237,9 +237,9 @@ 
 	 * @param string $source_id
 	 * @return WP_Error|int
 	 */
-	public function add_source( $source_id ) {
-		if ( ! $this->get_id() ) {
-			$this->set_id( $this->create_customer() );
+	public function add_source($source_id) {
+		if ( ! $this->get_id()) {
+			$this->set_id($this->create_customer());
 		}
 
 		$response = WC_Stripe_API::request(
@@ -251,62 +251,62 @@ 
 
 		$wc_token = false;
 
-		if ( ! empty( $response->error ) ) {
+		if ( ! empty($response->error)) {
 			// It is possible the WC user once was linked to a customer on Stripe
 			// but no longer exists. Instead of failing, lets try to create a
 			// new customer.
-			if ( $this->is_no_such_customer_error( $response->error ) ) {
+			if ($this->is_no_such_customer_error($response->error)) {
 				$this->recreate_customer();
-				return $this->add_source( $source_id );
+				return $this->add_source($source_id);
 			} else {
 				return $response;
 			}
-		} elseif ( empty( $response->id ) ) {
-			return new WP_Error( 'error', __( 'Unable to add payment source.', 'woocommerce-gateway-stripe' ) );
+		} elseif (empty($response->id)) {
+			return new WP_Error('error', __('Unable to add payment source.', 'woocommerce-gateway-stripe'));
 		}
 
 		// Add token to WooCommerce.
-		if ( $this->get_user_id() && class_exists( 'WC_Payment_Token_CC' ) ) {
-			if ( ! empty( $response->type ) ) {
-				switch ( $response->type ) {
+		if ($this->get_user_id() && class_exists('WC_Payment_Token_CC')) {
+			if ( ! empty($response->type)) {
+				switch ($response->type) {
 					case 'alipay':
 						break;
 					case 'sepa_debit':
 						$wc_token = new WC_Payment_Token_SEPA();
-						$wc_token->set_token( $response->id );
-						$wc_token->set_gateway_id( 'stripe_sepa' );
-						$wc_token->set_last4( $response->sepa_debit->last4 );
+						$wc_token->set_token($response->id);
+						$wc_token->set_gateway_id('stripe_sepa');
+						$wc_token->set_last4($response->sepa_debit->last4);
 						break;
 					default:
-						if ( 'source' === $response->object && 'card' === $response->type ) {
+						if ('source' === $response->object && 'card' === $response->type) {
 							$wc_token = new WC_Payment_Token_CC();
-							$wc_token->set_token( $response->id );
-							$wc_token->set_gateway_id( 'stripe' );
-							$wc_token->set_card_type( strtolower( $response->card->brand ) );
-							$wc_token->set_last4( $response->card->last4 );
-							$wc_token->set_expiry_month( $response->card->exp_month );
-							$wc_token->set_expiry_year( $response->card->exp_year );
+							$wc_token->set_token($response->id);
+							$wc_token->set_gateway_id('stripe');
+							$wc_token->set_card_type(strtolower($response->card->brand));
+							$wc_token->set_last4($response->card->last4);
+							$wc_token->set_expiry_month($response->card->exp_month);
+							$wc_token->set_expiry_year($response->card->exp_year);
 						}
 						break;
 				}
 			} else {
 				// Legacy.
 				$wc_token = new WC_Payment_Token_CC();
-				$wc_token->set_token( $response->id );
-				$wc_token->set_gateway_id( 'stripe' );
-				$wc_token->set_card_type( strtolower( $response->brand ) );
-				$wc_token->set_last4( $response->last4 );
-				$wc_token->set_expiry_month( $response->exp_month );
-				$wc_token->set_expiry_year( $response->exp_year );
+				$wc_token->set_token($response->id);
+				$wc_token->set_gateway_id('stripe');
+				$wc_token->set_card_type(strtolower($response->brand));
+				$wc_token->set_last4($response->last4);
+				$wc_token->set_expiry_month($response->exp_month);
+				$wc_token->set_expiry_year($response->exp_year);
 			}
 
-			$wc_token->set_user_id( $this->get_user_id() );
+			$wc_token->set_user_id($this->get_user_id());
 			$wc_token->save();
 		}
 
 		$this->clear_cache();
 
-		do_action( 'woocommerce_stripe_add_source', $this->get_id(), $wc_token, $response, $source_id );
+		do_action('woocommerce_stripe_add_source', $this->get_id(), $wc_token, $response, $source_id);
 
 		return $response->id;
 	}
@@ -318,13 +318,13 @@ 
 	 * @return array
 	 */
 	public function get_sources() {
-		if ( ! $this->get_id() ) {
+		if ( ! $this->get_id()) {
 			return array();
 		}
 
-		$sources = get_transient( 'stripe_sources_' . $this->get_id() );
+		$sources = get_transient('stripe_sources_' . $this->get_id());
 
-		if ( false === $sources ) {
+		if (false === $sources) {
 			$response = WC_Stripe_API::request(
 				array(
 					'limit' => 100,
@@ -333,35 +333,35 @@ 
 				'GET'
 			);
 
-			if ( ! empty( $response->error ) ) {
+			if ( ! empty($response->error)) {
 				return array();
 			}
 
-			if ( is_array( $response->data ) ) {
+			if (is_array($response->data)) {
 				$sources = $response->data;
 			}
 
-			set_transient( 'stripe_sources_' . $this->get_id(), $sources, DAY_IN_SECONDS );
+			set_transient('stripe_sources_' . $this->get_id(), $sources, DAY_IN_SECONDS);
 		}
 
-		return empty( $sources ) ? array() : $sources;
+		return empty($sources) ? array() : $sources;
 	}
 
 	/**
 	 * Delete a source from stripe.
 	 * @param string $source_id
 	 */
-	public function delete_source( $source_id ) {
-		if ( ! $this->get_id() ) {
+	public function delete_source($source_id) {
+		if ( ! $this->get_id()) {
 			return false;
 		}
 
-		$response = WC_Stripe_API::request( array(), 'customers/' . $this->get_id() . '/sources/' . sanitize_text_field( $source_id ), 'DELETE' );
+		$response = WC_Stripe_API::request(array(), 'customers/' . $this->get_id() . '/sources/' . sanitize_text_field($source_id), 'DELETE');
 
 		$this->clear_cache();
 
-		if ( empty( $response->error ) ) {
-			do_action( 'wc_stripe_delete_source', $this->get_id(), $response );
+		if (empty($response->error)) {
+			do_action('wc_stripe_delete_source', $this->get_id(), $response);
 
 			return true;
 		}
@@ -373,10 +373,10 @@ 
 	 * Set default source in Stripe
 	 * @param string $source_id
 	 */
-	public function set_default_source( $source_id ) {
+	public function set_default_source($source_id) {
 		$response = WC_Stripe_API::request(
 			array(
-				'default_source' => sanitize_text_field( $source_id ),
+				'default_source' => sanitize_text_field($source_id),
 			),
 			'customers/' . $this->get_id(),
 			'POST'
@@ -384,8 +384,8 @@ 
 
 		$this->clear_cache();
 
-		if ( empty( $response->error ) ) {
-			do_action( 'wc_stripe_set_default_source', $this->get_id(), $response );
+		if (empty($response->error)) {
+			do_action('wc_stripe_set_default_source', $this->get_id(), $response);
 
 			return true;
 		}
@@ -397,8 +397,8 @@ 
 	 * Deletes caches for this users cards.
 	 */
 	public function clear_cache() {
-		delete_transient( 'stripe_sources_' . $this->get_id() );
-		delete_transient( 'stripe_customer_' . $this->get_id() );
+		delete_transient('stripe_sources_' . $this->get_id());
+		delete_transient('stripe_customer_' . $this->get_id());
 		$this->customer_data = array();
 	}
 
@@ -408,8 +408,8 @@ 
 	 * @param  int $user_id The ID of the WordPress user.
 	 * @return string|bool  Either the Stripe ID or false.
 	 */
-	public function get_id_from_meta( $user_id ) {
-		return get_user_option( '_stripe_customer_id', $user_id );
+	public function get_id_from_meta($user_id) {
+		return get_user_option('_stripe_customer_id', $user_id);
 	}
 
 	/**
@@ -417,15 +417,15 @@ 
 	 *
 	 * @param string $id The Stripe customer ID.
 	 */
-	public function update_id_in_meta( $id ) {
-		update_user_option( $this->get_user_id(), '_stripe_customer_id', $id, false );
+	public function update_id_in_meta($id) {
+		update_user_option($this->get_user_id(), '_stripe_customer_id', $id, false);
 	}
 
 	/**
 	 * Deletes the user ID from the meta table with the right key.
 	 */
 	public function delete_id_from_meta() {
-		delete_user_option( $this->get_user_id(), '_stripe_customer_id', false );
+		delete_user_option($this->get_user_id(), '_stripe_customer_id', false);
 	}
 
 	/**

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

Doc Comments +2 added lines, -2 removed lines

@@ -138,7 +138,7 @@ 
 	 * @param float $amount
 	 * @param mixed $renewal_order
 	 * @param bool $retry Should we retry the process?
-	 * @param object $previous_error
+	 * @param boolean $previous_error
 	 */
 	public function process_subscription_payment( $amount = 0.0, $renewal_order, $retry = true, $previous_error ) {
 		try {
@@ -241,7 +241,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 );

Spacing +155 added lines, -155 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_sepa_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_sepa_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'));
 		}
 	}
 
@@ -40,8 +40,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;
 		}
 
@@ -53,8 +53,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)));
 	}
 
 	/**
@@ -64,7 +64,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']));
 	}
 
 	/**
@@ -74,20 +74,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),
 				)
 			);
 		}
@@ -100,19 +100,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);
 					}
 				}
 			}
@@ -125,24 +125,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);
 		}
 	}
 
@@ -151,16 +151,16 @@ 
 	 * @param  int $order_id
 	 * @return array
 	 */
-	public function process_payment( $order_id, $retry = true, $force_save_source = false, $previous_error = 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) {
+		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 );
+			return parent::process_payment($order_id, $retry, true, $previous_error);
 		} else {
-			return parent::process_payment( $order_id, $retry, $force_save_source, $previous_error );
+			return parent::process_payment($order_id, $retry, $force_save_source, $previous_error);
 		}
 	}
 
@@ -170,27 +170,27 @@ 
 	 * @since 4.0.4
 	 * @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);
 
-			if ( empty( $prepared_source->source ) ) {
-				$localized_message = __( 'Payment processing failed. Please retry.', 'woocommerce-gateway-stripe' );
-				throw new WC_Stripe_Exception( print_r( $prepared_source, true ), $localized_message );
+			if (empty($prepared_source->source)) {
+				$localized_message = __('Payment processing failed. Please retry.', 'woocommerce-gateway-stripe');
+				throw new WC_Stripe_Exception(print_r($prepared_source, true), $localized_message);
 			}
 
-			$this->save_source_to_order( $subscription, $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());
 		}
 	}
 
@@ -200,8 +200,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);
 	}
 
 	/**
@@ -215,11 +215,11 @@ 
 	 * @param bool $retry Should we retry the process?
 	 * @param object $previous_error
 	 */
-	public function process_subscription_payment( $amount = 0.0, $renewal_order, $retry = true, $previous_error ) {
+	public function process_subscription_payment($amount = 0.0, $renewal_order, $retry = true, $previous_error) {
 		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
@@ -228,82 +228,82 @@ 
 
 			$order_id = $renewal_order->get_id();
 
-			$this->ensure_subscription_has_customer_id( $order_id );
+			$this->ensure_subscription_has_customer_id($order_id);
 
 			// 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 = '';
 			}
 
-			$request            = $this->generate_payment_request( $renewal_order, $prepared_source );
+			$request            = $this->generate_payment_request($renewal_order, $prepared_source);
 			$request['capture'] = 'true';
-			$request['amount']  = WC_Stripe_Helper::get_stripe_amount( $amount, $request['currency'] );
-			$response           = WC_Stripe_API::request( $request );
+			$request['amount']  = WC_Stripe_Helper::get_stripe_amount($amount, $request['currency']);
+			$response           = WC_Stripe_API::request($request);
 
-			if ( ! empty( $response->error ) ) {
+			if ( ! empty($response->error)) {
 				// 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);
 			}
 
-			do_action( 'wc_gateway_stripe_process_payment', $response, $renewal_order );
+			do_action('wc_gateway_stripe_process_payment', $response, $renewal_order);
 
-			$this->process_response( $response, $renewal_order );
-		} catch ( WC_Stripe_Exception $e ) {
-			WC_Stripe_Logger::log( 'Error: ' . $e->getMessage() );
+			$this->process_response($response, $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');
 		}
 	}
 
@@ -311,21 +311,21 @@ 
 	 * 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' );
-		$this->delete_renewal_meta( $resubscribe_order );
+		delete_post_meta($resubscribe_order->get_id(), '_stripe_card_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);
 
 		return $renewal_order;
 	}
@@ -339,9 +339,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));
 	}
 
 	/**
@@ -353,21 +353,21 @@ 
 	 * @param WC_Subscription $subscription An instance of a subscription object
 	 * @return array
 	 */
-	public function add_subscription_payment_meta( $payment_meta, $subscription ) {
-		$source_id = get_post_meta( $subscription->get_id(), '_stripe_source_id', true );
+	public function add_subscription_payment_meta($payment_meta, $subscription) {
+		$source_id = get_post_meta($subscription->get_id(), '_stripe_source_id', true);
 
 		// For BW compat will remove in future.
-		if ( empty( $source_id ) ) {
-			$source_id = get_post_meta( $subscription->get_id(), '_stripe_card_id', true );
+		if (empty($source_id)) {
+			$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', $source_id );
+			update_post_meta($subscription->get_id(), '_stripe_source_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->get_id(), '_stripe_customer_id', true ),
+					'value' => get_post_meta($subscription->get_id(), '_stripe_customer_id', true),
 					'label' => 'Stripe Customer ID',
 				),
 				'_stripe_source_id'   => array(
@@ -388,26 +388,26 @@ 
 	 * @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_' ) )
-				&& ( ! empty( $payment_meta['post_meta']['_stripe_source_id']['value'] )
-				&& 0 !== strpos( $payment_meta['post_meta']['_stripe_source_id']['value'], 'src_' ) ) ) {
+				( ! empty($payment_meta['post_meta']['_stripe_source_id']['value'])
+				&& 0 !== strpos($payment_meta['post_meta']['_stripe_source_id']['value'], 'card_'))
+				&& ( ! empty($payment_meta['post_meta']['_stripe_source_id']['value'])
+				&& 0 !== strpos($payment_meta['post_meta']['_stripe_source_id']['value'], 'src_')) ) {
 
-				throw new Exception( __( 'Invalid source ID. A valid source "Stripe Source ID" must begin with "src_" or "card_".', 'woocommerce-gateway-stripe' ) );
+				throw new Exception(__('Invalid source ID. A valid source "Stripe Source ID" must begin with "src_" or "card_".', 'woocommerce-gateway-stripe'));
 			}
 		}
 	}
@@ -420,67 +420,67 @@ 
 	 * @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 ) {
-			foreach ( $sources as $source ) {
-				if ( $source->id === $stripe_source_id ) {
-					if ( $source->sepa_debit ) {
+		if ($sources) {
+			foreach ($sources as $source) {
+				if ($source->id === $stripe_source_id) {
+					if ($source->sepa_debit) {
 						/* translators: 1) last 4 digits of SEPA Direct Debit */
-						$payment_method_to_display = sprintf( __( 'Via SEPA Direct Debit ending in %1$s', 'woocommerce-gateway-stripe' ), $source->sepa_debit->last4 );
+						$payment_method_to_display = sprintf(__('Via SEPA Direct Debit ending in %1$s', 'woocommerce-gateway-stripe'), $source->sepa_debit->last4);
 					}
 
 					break;

includes/deprecated/class-wc-stripe-apple-pay.php

Spacing +5 added lines, -5 removed lines

@@ -1,12 +1,12 @@ 
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 /**
  * DEPRECATED DO NOT USE!!
  */
-if ( ! class_exists( 'WC_Stripe_Apple_Pay' ) ) {
+if ( ! class_exists('WC_Stripe_Apple_Pay')) {
 	class WC_Stripe_Apple_Pay {
 		/**
 		 * This Instance.
@@ -20,15 +20,15 @@ 
 		}
 
 		public static function instance() {
-			WC_Stripe_Logger::log( 'DEPRECATED! WC_Stripe_Apple_Pay class has been hard deprecated. Please remove any code that references this class or instance. This class will be removed by version 4.2' );
+			WC_Stripe_Logger::log('DEPRECATED! WC_Stripe_Apple_Pay class has been hard deprecated. Please remove any code that references this class or instance. This class will be removed by version 4.2');
 			return self::$_this;
 		}
 
-		public function __get( $var ) {
+		public function __get($var) {
 			return null;
 		}
 
-		public function __call( $name, $arguments ) {
+		public function __call($name, $arguments) {
 			return null;
 		}
 	}

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

Doc Comments +2 added lines, -2 removed lines

@@ -138,7 +138,7 @@ 
 	 * @param float $amount
 	 * @param mixed $renewal_order
 	 * @param bool $retry Should we retry the process?
-	 * @param object $previous_error
+	 * @param boolean $previous_error
 	 */
 	public function process_subscription_payment( $amount = 0.0, $renewal_order, $retry = true, $previous_error ) {
 		try {
@@ -241,7 +241,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 );

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 = 0.0, $renewal_order, $retry = true, $previous_error ) {
+	public function process_subscription_payment($amount = 0.0, $renewal_order, $retry = true, $previous_error) {
 		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;
 		}
 	}
 }

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 = array(), $customer_id, $gateway_id ) {
-		if ( is_user_logged_in() && class_exists( 'WC_Payment_Token_CC' ) ) {
+	public function woocommerce_get_customer_payment_tokens($tokens = array(), $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());
 		}
 	}
 }