Inspection of "Don’t require test publishable key when using live" - woocommerce/woocommerce-gateway-stripe - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#467)

by Caleb

created 2018-01-18 16:32 UTC

Status

Spacing +9 added lines, -9 removed lines patch added patch discarded remove patch

@@ -1,6 +1,6 @@  discard block
 block discarded – undo
 <?php
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
@@ -35,10 +35,10 @@  discard block
 block discarded – undo
 	 * @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 @@  discard block
 block discarded – undo
 	 * @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 @@  discard block
 block discarded – undo
 	 * @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 @@  discard block
 block discarded – undo
 	 * @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);
 	}
 }

Please login to merge, or discard this patch.

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

Spacing +57 added lines, -57 removed lines patch added patch discarded remove patch

@@ -1,5 +1,5 @@  discard block
 block discarded – undo
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,11 +21,11 @@  discard block
 block discarded – undo
 	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 @@  discard block
 block discarded – undo
 	 * @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';
 		}
 
@@ -62,67 +62,67 @@  discard block
 block discarded – undo
 	 * @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;
 						}
 					}
 				}
@@ -141,13 +141,13 @@  discard block
 block discarded – undo
 	 * @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())) {
 			return $item;
 		}
 
 		$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;
 	}
@@ -158,10 +158,10 @@  discard block
 block discarded – undo
 	 * @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());
 		}
 	}
 
@@ -171,12 +171,12 @@  discard block
 block discarded – undo
 	 * @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());
 		}
 	}
 }

Please login to merge, or discard this patch.

includes/class-wc-stripe-logger.php 1 patch

Spacing +11 added lines, -11 removed lines patch added patch discarded remove patch

@@ -1,5 +1,5 @@  discard block
 block discarded – undo
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
@@ -20,23 +20,23 @@  discard block
 block discarded – undo
 	 * @since 4.0.0
 	 * @version 4.0.0
 	 */
-	public static function log( $message, $start_time = null, $end_time = null ) {
-		if ( empty( self::$logger ) ) {
+	public static function log($message, $start_time = null, $end_time = null) {
+		if (empty(self::$logger)) {
 			self::$logger = new WC_Logger();
 		}
 
-		$settings = get_option( 'woocommerce_stripe_settings' );
+		$settings = get_option('woocommerce_stripe_settings');
 
-		if ( empty( $settings ) || isset( $settings['logging'] ) && 'yes' !== $settings['logging'] ) {
+		if (empty($settings) || isset($settings['logging']) && 'yes' !== $settings['logging']) {
 			return;
 		}
 
-		if ( ! is_null( $start_time ) ) {
+		if ( ! is_null($start_time)) {
 
-			$formatted_start_time = date_i18n( get_option( 'date_format' ) . ' g:ia', $start_time );
-			$end_time             = is_null( $end_time ) ? current_time( 'timestamp' ) : $end_time;
-			$formatted_end_time   = date_i18n( get_option( 'date_format' ) . ' g:ia', $end_time );
-			$elapsed_time         = round( abs( $end_time - $start_time ) / 60, 2 );
+			$formatted_start_time = date_i18n(get_option('date_format') . ' g:ia', $start_time);
+			$end_time             = is_null($end_time) ? current_time('timestamp') : $end_time;
+			$formatted_end_time   = date_i18n(get_option('date_format') . ' g:ia', $end_time);
+			$elapsed_time         = round(abs($end_time - $start_time) / 60, 2);
 
 			$log_entry  = '====Start Log ' . $formatted_start_time . '====' . "\n" . $message . "\n";
 			$log_entry .= '====End Log ' . $formatted_end_time . ' (' . $elapsed_time . ')====' . "\n\n";
@@ -47,6 +47,6 @@  discard block
 block discarded – undo
 
 		}
 
-		self::$logger->add( self::WC_LOG_FILENAME, $log_entry );
+		self::$logger->add(self::WC_LOG_FILENAME, $log_entry);
 	}
 }

Please login to merge, or discard this patch.

includes/admin/stripe-bitcoin-settings.php 1 patch

Spacing +15 added lines, -15 removed lines patch added patch discarded remove patch

@@ -1,50 +1,50 @@
 block discarded – undo
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 $webhook_url = WC_Stripe_Helper::get_webhook_url();
 
-return apply_filters( 'wc_stripe_bitcoin_settings',
+return apply_filters('wc_stripe_bitcoin_settings',
 	array(
 		'geo_target' => array(
-			'description' => __( 'Relevant Payer Geography: Global', 'woocommerce-gateway-stripe' ),
+			'description' => __('Relevant Payer Geography: Global', 'woocommerce-gateway-stripe'),
 			'type'        => 'title',
 		),
 		'guide' => array(
-			'description' => __( '<a href="https://stripe.com/payments/payment-methods-guide#bitcoin" target="_blank">Payment Method Guide</a>', 'woocommerce-gateway-stripe' ),
+			'description' => __('<a href="https://stripe.com/payments/payment-methods-guide#bitcoin" target="_blank">Payment Method Guide</a>', 'woocommerce-gateway-stripe'),
 			'type'        => 'title',
 		),
 		'activation' => array(
-			'description' => __( 'Must be activated from your Stripe Dashboard Settings <a href="https://dashboard.stripe.com/account/payments/settings" target="_blank">here</a>', 'woocommerce-gateway-stripe' ),
+			'description' => __('Must be activated from your Stripe Dashboard Settings <a href="https://dashboard.stripe.com/account/payments/settings" target="_blank">here</a>', 'woocommerce-gateway-stripe'),
 			'type'   => 'title',
 		),
 		'enabled' => array(
-			'title'       => __( 'Enable/Disable', 'woocommerce-gateway-stripe' ),
-			'label'       => __( 'Enable Stripe Bitcoin', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Enable/Disable', 'woocommerce-gateway-stripe'),
+			'label'       => __('Enable Stripe Bitcoin', 'woocommerce-gateway-stripe'),
 			'type'        => 'checkbox',
 			'description' => '',
 			'default'     => 'no',
 		),
 		'title' => array(
-			'title'       => __( 'Title', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Title', 'woocommerce-gateway-stripe'),
 			'type'        => 'text',
-			'description' => __( 'This controls the title which the user sees during checkout.', 'woocommerce-gateway-stripe' ),
-			'default'     => __( 'Bitcoin', 'woocommerce-gateway-stripe' ),
+			'description' => __('This controls the title which the user sees during checkout.', 'woocommerce-gateway-stripe'),
+			'default'     => __('Bitcoin', 'woocommerce-gateway-stripe'),
 			'desc_tip'    => true,
 		),
 		'description' => array(
-			'title'       => __( 'Description', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Description', 'woocommerce-gateway-stripe'),
 			'type'        => 'text',
-			'description' => __( 'This controls the description which the user sees during checkout.', 'woocommerce-gateway-stripe' ),
-			'default'     => __( 'Bitcoin payment information will be provided when you place the order.', 'woocommerce-gateway-stripe' ),
+			'description' => __('This controls the description which the user sees during checkout.', 'woocommerce-gateway-stripe'),
+			'default'     => __('Bitcoin payment information will be provided when you place the order.', 'woocommerce-gateway-stripe'),
 			'desc_tip'    => true,
 		),
 		'webhook' => array(
-			'title'       => __( 'Webhook Endpoints', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Webhook Endpoints', 'woocommerce-gateway-stripe'),
 			'type'        => 'title',
 			/* translators: webhook URL */
-			'description' => sprintf( __( 'You must add the webhook endpoint <strong style="background-color:#ddd;">&nbsp;&nbsp;%s&nbsp;&nbsp;</strong> to your Stripe Account Settings <a href="https://dashboard.stripe.com/account/webhooks" target="_blank">Here</a> so you can receive notifications on the charge statuses.', 'woocommerce-gateway-stripe' ), $webhook_url ),
+			'description' => sprintf(__('You must add the webhook endpoint <strong style="background-color:#ddd;">&nbsp;&nbsp;%s&nbsp;&nbsp;</strong> to your Stripe Account Settings <a href="https://dashboard.stripe.com/account/webhooks" target="_blank">Here</a> so you can receive notifications on the charge statuses.', 'woocommerce-gateway-stripe'), $webhook_url),
 		),
 	)
 );

Please login to merge, or discard this patch.

includes/admin/stripe-ideal-settings.php 1 patch

Spacing +15 added lines, -15 removed lines patch added patch discarded remove patch

@@ -1,50 +1,50 @@
 block discarded – undo
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 $webhook_url = WC_Stripe_Helper::get_webhook_url();
 
-return apply_filters( 'wc_stripe_ideal_settings',
+return apply_filters('wc_stripe_ideal_settings',
 	array(
 		'geo_target' => array(
-			'description' => __( 'Relevant Payer Geography: The Netherlands', 'woocommerce-gateway-stripe' ),
+			'description' => __('Relevant Payer Geography: The Netherlands', 'woocommerce-gateway-stripe'),
 			'type'        => 'title',
 		),
 		'guide' => array(
-			'description' => __( '<a href="https://stripe.com/payments/payment-methods-guide#ideal" target="_blank">Payment Method Guide</a>', 'woocommerce-gateway-stripe' ),
+			'description' => __('<a href="https://stripe.com/payments/payment-methods-guide#ideal" target="_blank">Payment Method Guide</a>', 'woocommerce-gateway-stripe'),
 			'type'        => 'title',
 		),
 		'activation' => array(
-			'description' => __( 'Must be activated from your Stripe Dashboard Settings <a href="https://dashboard.stripe.com/account/payments/settings" target="_blank">here</a>', 'woocommerce-gateway-stripe' ),
+			'description' => __('Must be activated from your Stripe Dashboard Settings <a href="https://dashboard.stripe.com/account/payments/settings" target="_blank">here</a>', 'woocommerce-gateway-stripe'),
 			'type'   => 'title',
 		),
 		'enabled' => array(
-			'title'       => __( 'Enable/Disable', 'woocommerce-gateway-stripe' ),
-			'label'       => __( 'Enable Stripe iDeal', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Enable/Disable', 'woocommerce-gateway-stripe'),
+			'label'       => __('Enable Stripe iDeal', 'woocommerce-gateway-stripe'),
 			'type'        => 'checkbox',
 			'description' => '',
 			'default'     => 'no',
 		),
 		'title' => array(
-			'title'       => __( 'Title', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Title', 'woocommerce-gateway-stripe'),
 			'type'        => 'text',
-			'description' => __( 'This controls the title which the user sees during checkout.', 'woocommerce-gateway-stripe' ),
-			'default'     => __( 'iDeal', 'woocommerce-gateway-stripe' ),
+			'description' => __('This controls the title which the user sees during checkout.', 'woocommerce-gateway-stripe'),
+			'default'     => __('iDeal', 'woocommerce-gateway-stripe'),
 			'desc_tip'    => true,
 		),
 		'description' => array(
-			'title'       => __( 'Description', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Description', 'woocommerce-gateway-stripe'),
 			'type'        => 'text',
-			'description' => __( 'This controls the description which the user sees during checkout.', 'woocommerce-gateway-stripe' ),
-			'default'     => __( 'You will be redirected to iDeal.', 'woocommerce-gateway-stripe' ),
+			'description' => __('This controls the description which the user sees during checkout.', 'woocommerce-gateway-stripe'),
+			'default'     => __('You will be redirected to iDeal.', 'woocommerce-gateway-stripe'),
 			'desc_tip'    => true,
 		),
 		'webhook' => array(
-			'title'       => __( 'Webhook Endpoints', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Webhook Endpoints', 'woocommerce-gateway-stripe'),
 			'type'        => 'title',
 			/* translators: webhook URL */
-			'description' => sprintf( __( 'You must add the webhook endpoint <strong style="background-color:#ddd;">&nbsp;&nbsp;%s&nbsp;&nbsp;</strong> to your Stripe Account Settings <a href="https://dashboard.stripe.com/account/webhooks" target="_blank">Here</a> so you can receive notifications on the charge statuses.', 'woocommerce-gateway-stripe' ), $webhook_url ),
+			'description' => sprintf(__('You must add the webhook endpoint <strong style="background-color:#ddd;">&nbsp;&nbsp;%s&nbsp;&nbsp;</strong> to your Stripe Account Settings <a href="https://dashboard.stripe.com/account/webhooks" target="_blank">Here</a> so you can receive notifications on the charge statuses.', 'woocommerce-gateway-stripe'), $webhook_url),
 		),
 	)
 );

Please login to merge, or discard this patch.

includes/admin/stripe-sofort-settings.php 1 patch

Spacing +15 added lines, -15 removed lines patch added patch discarded remove patch

@@ -1,50 +1,50 @@
 block discarded – undo
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 $webhook_url = WC_Stripe_Helper::get_webhook_url();
 
-return apply_filters( 'wc_stripe_sofort_settings',
+return apply_filters('wc_stripe_sofort_settings',
 	array(
 		'geo_target' => array(
-			'description' => __( 'Relevant Payer Geography: Germany, Austria', 'woocommerce-gateway-stripe' ),
+			'description' => __('Relevant Payer Geography: Germany, Austria', 'woocommerce-gateway-stripe'),
 			'type'        => 'title',
 		),
 		'guide' => array(
-			'description' => __( '<a href="https://stripe.com/payments/payment-methods-guide#sofort" target="_blank">Payment Method Guide</a>', 'woocommerce-gateway-stripe' ),
+			'description' => __('<a href="https://stripe.com/payments/payment-methods-guide#sofort" target="_blank">Payment Method Guide</a>', 'woocommerce-gateway-stripe'),
 			'type'        => 'title',
 		),
 		'activation' => array(
-			'description' => __( 'Must be activated from your Stripe Dashboard Settings <a href="https://dashboard.stripe.com/account/payments/settings" target="_blank">here</a>', 'woocommerce-gateway-stripe' ),
+			'description' => __('Must be activated from your Stripe Dashboard Settings <a href="https://dashboard.stripe.com/account/payments/settings" target="_blank">here</a>', 'woocommerce-gateway-stripe'),
 			'type'   => 'title',
 		),
 		'enabled' => array(
-			'title'       => __( 'Enable/Disable', 'woocommerce-gateway-stripe' ),
-			'label'       => __( 'Enable Stripe SOFORT', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Enable/Disable', 'woocommerce-gateway-stripe'),
+			'label'       => __('Enable Stripe SOFORT', 'woocommerce-gateway-stripe'),
 			'type'        => 'checkbox',
 			'description' => '',
 			'default'     => 'no',
 		),
 		'title' => array(
-			'title'       => __( 'Title', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Title', 'woocommerce-gateway-stripe'),
 			'type'        => 'text',
-			'description' => __( 'This controls the title which the user sees during checkout.', 'woocommerce-gateway-stripe' ),
-			'default'     => __( 'SOFORT', 'woocommerce-gateway-stripe' ),
+			'description' => __('This controls the title which the user sees during checkout.', 'woocommerce-gateway-stripe'),
+			'default'     => __('SOFORT', 'woocommerce-gateway-stripe'),
 			'desc_tip'    => true,
 		),
 		'description' => array(
-			'title'       => __( 'Description', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Description', 'woocommerce-gateway-stripe'),
 			'type'        => 'text',
-			'description' => __( 'This controls the description which the user sees during checkout.', 'woocommerce-gateway-stripe' ),
-			'default'     => __( 'You will be redirected to SOFORT.', 'woocommerce-gateway-stripe' ),
+			'description' => __('This controls the description which the user sees during checkout.', 'woocommerce-gateway-stripe'),
+			'default'     => __('You will be redirected to SOFORT.', 'woocommerce-gateway-stripe'),
 			'desc_tip'    => true,
 		),
 		'webhook' => array(
-			'title'       => __( 'Webhook Endpoints', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Webhook Endpoints', 'woocommerce-gateway-stripe'),
 			'type'        => 'title',
 			/* translators: webhook URL */
-			'description' => sprintf( __( 'You must add the webhook endpoint <strong style="background-color:#ddd;">&nbsp;&nbsp;%s&nbsp;&nbsp;</strong> to your Stripe Account Settings <a href="https://dashboard.stripe.com/account/webhooks" target="_blank">Here</a> so you can receive notifications on the charge statuses.', 'woocommerce-gateway-stripe' ), $webhook_url ),
+			'description' => sprintf(__('You must add the webhook endpoint <strong style="background-color:#ddd;">&nbsp;&nbsp;%s&nbsp;&nbsp;</strong> to your Stripe Account Settings <a href="https://dashboard.stripe.com/account/webhooks" target="_blank">Here</a> so you can receive notifications on the charge statuses.', 'woocommerce-gateway-stripe'), $webhook_url),
 		),
 	)
 );

Please login to merge, or discard this patch.

includes/admin/stripe-giropay-settings.php 1 patch

Spacing +15 added lines, -15 removed lines patch added patch discarded remove patch

@@ -1,50 +1,50 @@
 block discarded – undo
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 $webhook_url = WC_Stripe_Helper::get_webhook_url();
 
-return apply_filters( 'wc_stripe_giropay_settings',
+return apply_filters('wc_stripe_giropay_settings',
 	array(
 		'geo_target' => array(
-			'description' => __( 'Relevant Payer Geography: Germany', 'woocommerce-gateway-stripe' ),
+			'description' => __('Relevant Payer Geography: Germany', 'woocommerce-gateway-stripe'),
 			'type'        => 'title',
 		),
 		'guide' => array(
-			'description' => __( '<a href="https://stripe.com/payments/payment-methods-guide#giropay" target="_blank">Payment Method Guide</a>', 'woocommerce-gateway-stripe' ),
+			'description' => __('<a href="https://stripe.com/payments/payment-methods-guide#giropay" target="_blank">Payment Method Guide</a>', 'woocommerce-gateway-stripe'),
 			'type'        => 'title',
 		),
 		'activation' => array(
-			'description' => __( 'Must be activated from your Stripe Dashboard Settings <a href="https://dashboard.stripe.com/account/payments/settings" target="_blank">here</a>', 'woocommerce-gateway-stripe' ),
+			'description' => __('Must be activated from your Stripe Dashboard Settings <a href="https://dashboard.stripe.com/account/payments/settings" target="_blank">here</a>', 'woocommerce-gateway-stripe'),
 			'type'   => 'title',
 		),
 		'enabled' => array(
-			'title'       => __( 'Enable/Disable', 'woocommerce-gateway-stripe' ),
-			'label'       => __( 'Enable Stripe Giropay', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Enable/Disable', 'woocommerce-gateway-stripe'),
+			'label'       => __('Enable Stripe Giropay', 'woocommerce-gateway-stripe'),
 			'type'        => 'checkbox',
 			'description' => '',
 			'default'     => 'no',
 		),
 		'title' => array(
-			'title'       => __( 'Title', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Title', 'woocommerce-gateway-stripe'),
 			'type'        => 'text',
-			'description' => __( 'This controls the title which the user sees during checkout.', 'woocommerce-gateway-stripe' ),
-			'default'     => __( 'Giropay', 'woocommerce-gateway-stripe' ),
+			'description' => __('This controls the title which the user sees during checkout.', 'woocommerce-gateway-stripe'),
+			'default'     => __('Giropay', 'woocommerce-gateway-stripe'),
 			'desc_tip'    => true,
 		),
 		'description' => array(
-			'title'       => __( 'Description', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Description', 'woocommerce-gateway-stripe'),
 			'type'        => 'text',
-			'description' => __( 'This controls the description which the user sees during checkout.', 'woocommerce-gateway-stripe' ),
-			'default'     => __( 'You will be redirected to Giropay.', 'woocommerce-gateway-stripe' ),
+			'description' => __('This controls the description which the user sees during checkout.', 'woocommerce-gateway-stripe'),
+			'default'     => __('You will be redirected to Giropay.', 'woocommerce-gateway-stripe'),
 			'desc_tip'    => true,
 		),
 		'webhook' => array(
-			'title'       => __( 'Webhook Endpoints', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Webhook Endpoints', 'woocommerce-gateway-stripe'),
 			'type'        => 'title',
 			/* translators: webhook URL */
-			'description' => sprintf( __( 'You must add the webhook endpoint <strong style="background-color:#ddd;">&nbsp;&nbsp;%s&nbsp;&nbsp;</strong> to your Stripe Account Settings <a href="https://dashboard.stripe.com/account/webhooks" target="_blank">Here</a> so you can receive notifications on the charge statuses.', 'woocommerce-gateway-stripe' ), $webhook_url ),
+			'description' => sprintf(__('You must add the webhook endpoint <strong style="background-color:#ddd;">&nbsp;&nbsp;%s&nbsp;&nbsp;</strong> to your Stripe Account Settings <a href="https://dashboard.stripe.com/account/webhooks" target="_blank">Here</a> so you can receive notifications on the charge statuses.', 'woocommerce-gateway-stripe'), $webhook_url),
 		),
 	)
 );

Please login to merge, or discard this patch.

includes/admin/stripe-sepa-settings.php 1 patch

Spacing +15 added lines, -15 removed lines patch added patch discarded remove patch

@@ -1,50 +1,50 @@
 block discarded – undo
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 $webhook_url = WC_Stripe_Helper::get_webhook_url();
 
-return apply_filters( 'wc_stripe_sepa_settings',
+return apply_filters('wc_stripe_sepa_settings',
 	array(
 		'geo_target' => array(
-			'description' => __( 'Relevant Payer Geography: France, Germany, Spain, Belgium, Netherlands, Luxembourg, Italy, Portugal, Austria, Ireland', 'woocommerce-gateway-stripe' ),
+			'description' => __('Relevant Payer Geography: France, Germany, Spain, Belgium, Netherlands, Luxembourg, Italy, Portugal, Austria, Ireland', 'woocommerce-gateway-stripe'),
 			'type'        => 'title',
 		),
 		'guide' => array(
-			'description' => __( '<a href="https://stripe.com/payments/payment-methods-guide#sepa-direct-debit" target="_blank">Payment Method Guide</a>', 'woocommerce-gateway-stripe' ),
+			'description' => __('<a href="https://stripe.com/payments/payment-methods-guide#sepa-direct-debit" target="_blank">Payment Method Guide</a>', 'woocommerce-gateway-stripe'),
 			'type'        => 'title',
 		),
 		'activation' => array(
-			'description' => __( 'Must be activated from your Stripe Dashboard Settings <a href="https://dashboard.stripe.com/account/payments/settings" target="_blank">here</a>', 'woocommerce-gateway-stripe' ),
+			'description' => __('Must be activated from your Stripe Dashboard Settings <a href="https://dashboard.stripe.com/account/payments/settings" target="_blank">here</a>', 'woocommerce-gateway-stripe'),
 			'type'   => 'title',
 		),
 		'enabled' => array(
-			'title'       => __( 'Enable/Disable', 'woocommerce-gateway-stripe' ),
-			'label'       => __( 'Enable Stripe SEPA Direct Debit', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Enable/Disable', 'woocommerce-gateway-stripe'),
+			'label'       => __('Enable Stripe SEPA Direct Debit', 'woocommerce-gateway-stripe'),
 			'type'        => 'checkbox',
 			'description' => '',
 			'default'     => 'no',
 		),
 		'title' => array(
-			'title'       => __( 'Title', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Title', 'woocommerce-gateway-stripe'),
 			'type'        => 'text',
-			'description' => __( 'This controls the title which the user sees during checkout.', 'woocommerce-gateway-stripe' ),
-			'default'     => __( 'SEPA Direct Debit', 'woocommerce-gateway-stripe' ),
+			'description' => __('This controls the title which the user sees during checkout.', 'woocommerce-gateway-stripe'),
+			'default'     => __('SEPA Direct Debit', 'woocommerce-gateway-stripe'),
 			'desc_tip'    => true,
 		),
 		'description' => array(
-			'title'       => __( 'Description', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Description', 'woocommerce-gateway-stripe'),
 			'type'        => 'text',
-			'description' => __( 'This controls the description which the user sees during checkout.', 'woocommerce-gateway-stripe' ),
-			'default'     => __( 'Mandate Information.', 'woocommerce-gateway-stripe' ),
+			'description' => __('This controls the description which the user sees during checkout.', 'woocommerce-gateway-stripe'),
+			'default'     => __('Mandate Information.', 'woocommerce-gateway-stripe'),
 			'desc_tip'    => true,
 		),
 		'webhook' => array(
-			'title'       => __( 'Webhook Endpoints', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Webhook Endpoints', 'woocommerce-gateway-stripe'),
 			'type'        => 'title',
 			/* translators: webhook URL */
-			'description' => sprintf( __( 'You must add the webhook endpoint <strong style="background-color:#ddd;">&nbsp;&nbsp;%s&nbsp;&nbsp;</strong> to your Stripe Account Settings <a href="https://dashboard.stripe.com/account/webhooks" target="_blank">Here</a> so you can receive notifications on the charge statuses.', 'woocommerce-gateway-stripe' ), $webhook_url ),
+			'description' => sprintf(__('You must add the webhook endpoint <strong style="background-color:#ddd;">&nbsp;&nbsp;%s&nbsp;&nbsp;</strong> to your Stripe Account Settings <a href="https://dashboard.stripe.com/account/webhooks" target="_blank">Here</a> so you can receive notifications on the charge statuses.', 'woocommerce-gateway-stripe'), $webhook_url),
 		),
 	)
 );

Please login to merge, or discard this patch.

includes/admin/stripe-bancontact-settings.php 1 patch

Spacing +15 added lines, -15 removed lines patch added patch discarded remove patch

@@ -1,50 +1,50 @@
 block discarded – undo
 <?php
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 $webhook_url = WC_Stripe_Helper::get_webhook_url();
 
-return apply_filters( 'wc_stripe_bancontact_settings',
+return apply_filters('wc_stripe_bancontact_settings',
 	array(
 		'geo_target' => array(
-			'description' => __( 'Relevant Payer Geography: Belgium', 'woocommerce-gateway-stripe' ),
+			'description' => __('Relevant Payer Geography: Belgium', 'woocommerce-gateway-stripe'),
 			'type'        => 'title',
 		),
 		'guide' => array(
-			'description' => __( '<a href="https://stripe.com/payments/payment-methods-guide#bancontact" target="_blank">Payment Method Guide</a>', 'woocommerce-gateway-stripe' ),
+			'description' => __('<a href="https://stripe.com/payments/payment-methods-guide#bancontact" target="_blank">Payment Method Guide</a>', 'woocommerce-gateway-stripe'),
 			'type'        => 'title',
 		),
 		'activation' => array(
-			'description' => __( 'Must be activated from your Stripe Dashboard Settings <a href="https://dashboard.stripe.com/account/payments/settings" target="_blank">here</a>', 'woocommerce-gateway-stripe' ),
+			'description' => __('Must be activated from your Stripe Dashboard Settings <a href="https://dashboard.stripe.com/account/payments/settings" target="_blank">here</a>', 'woocommerce-gateway-stripe'),
 			'type'   => 'title',
 		),
 		'enabled' => array(
-			'title'       => __( 'Enable/Disable', 'woocommerce-gateway-stripe' ),
-			'label'       => __( 'Enable Stripe Bancontact', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Enable/Disable', 'woocommerce-gateway-stripe'),
+			'label'       => __('Enable Stripe Bancontact', 'woocommerce-gateway-stripe'),
 			'type'        => 'checkbox',
 			'description' => '',
 			'default'     => 'no',
 		),
 		'title' => array(
-			'title'       => __( 'Title', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Title', 'woocommerce-gateway-stripe'),
 			'type'        => 'text',
-			'description' => __( 'This controls the title which the user sees during checkout.', 'woocommerce-gateway-stripe' ),
-			'default'     => __( 'Bancontact', 'woocommerce-gateway-stripe' ),
+			'description' => __('This controls the title which the user sees during checkout.', 'woocommerce-gateway-stripe'),
+			'default'     => __('Bancontact', 'woocommerce-gateway-stripe'),
 			'desc_tip'    => true,
 		),
 		'description' => array(
-			'title'       => __( 'Description', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Description', 'woocommerce-gateway-stripe'),
 			'type'        => 'text',
-			'description' => __( 'This controls the description which the user sees during checkout.', 'woocommerce-gateway-stripe' ),
-			'default'     => __( 'You will be redirected to Bancontact.', 'woocommerce-gateway-stripe' ),
+			'description' => __('This controls the description which the user sees during checkout.', 'woocommerce-gateway-stripe'),
+			'default'     => __('You will be redirected to Bancontact.', 'woocommerce-gateway-stripe'),
 			'desc_tip'    => true,
 		),
 		'webhook' => array(
-			'title'       => __( 'Webhook Endpoints', 'woocommerce-gateway-stripe' ),
+			'title'       => __('Webhook Endpoints', 'woocommerce-gateway-stripe'),
 			'type'        => 'title',
 			/* translators: webhook URL */
-			'description' => sprintf( __( 'You must add the webhook endpoint <strong style="background-color:#ddd;">&nbsp;&nbsp;%s&nbsp;&nbsp;</strong> to your Stripe Account Settings <a href="https://dashboard.stripe.com/account/webhooks" target="_blank">Here</a> so you can receive notifications on the charge statuses.', 'woocommerce-gateway-stripe' ), $webhook_url ),
+			'description' => sprintf(__('You must add the webhook endpoint <strong style="background-color:#ddd;">&nbsp;&nbsp;%s&nbsp;&nbsp;</strong> to your Stripe Account Settings <a href="https://dashboard.stripe.com/account/webhooks" target="_blank">Here</a> so you can receive notifications on the charge statuses.', 'woocommerce-gateway-stripe'), $webhook_url),
 		),
 	)
 );

Please login to merge, or discard this patch.

		@@ -1,5 +1,5 @@ discard block
		block discarded – undo
1	1	<?php
2		-if ( ! defined( 'ABSPATH' ) ) {
	2	+if ( ! defined('ABSPATH')) {
3	3	exit;
4	4	}
5	5
		@@ -21,11 +21,11 @@ discard block
		block discarded – undo
21	21	public function __construct() {
22	22	self::$_this = $this;
23	23
24		- add_filter( 'woocommerce_get_customer_payment_tokens', array( $this, 'woocommerce_get_customer_payment_tokens' ), 10, 3 );
25		- add_filter( 'woocommerce_payment_methods_list_item', array( $this, 'get_account_saved_payment_methods_list_item_sepa' ), 10, 2 );
26		- add_filter( 'woocommerce_get_credit_card_type_label', array( $this, 'normalize_sepa_label' ) );
27		- add_action( 'woocommerce_payment_token_deleted', array( $this, 'woocommerce_payment_token_deleted' ), 10, 2 );
28		- add_action( 'woocommerce_payment_token_set_default', array( $this, 'woocommerce_payment_token_set_default' ) );
	24	+ add_filter('woocommerce_get_customer_payment_tokens', array($this, 'woocommerce_get_customer_payment_tokens'), 10, 3);
	25	+ add_filter('woocommerce_payment_methods_list_item', array($this, 'get_account_saved_payment_methods_list_item_sepa'), 10, 2);
	26	+ add_filter('woocommerce_get_credit_card_type_label', array($this, 'normalize_sepa_label'));
	27	+ add_action('woocommerce_payment_token_deleted', array($this, 'woocommerce_payment_token_deleted'), 10, 2);
	28	+ add_action('woocommerce_payment_token_set_default', array($this, 'woocommerce_payment_token_set_default'));
29	29	}
30	30
31	31	/**
		@@ -46,8 +46,8 @@ discard block
		block discarded – undo
46	46	* @param string $label
47	47	* @return string $label
48	48	*/
49		- public function normalize_sepa_label( $label ) {
50		- if ( 'sepa iban' === strtolower( $label ) ) {
	49	+ public function normalize_sepa_label($label) {
	50	+ if ('sepa iban' === strtolower($label)) {
51	51	return 'SEPA IBAN';
52	52	}
53	53
		@@ -62,67 +62,67 @@ discard block
		block discarded – undo
62	62	* @param array $tokens
63	63	* @return array
64	64	*/
65		- public function woocommerce_get_customer_payment_tokens( $tokens = array(), $customer_id, $gateway_id ) {
66		- if ( is_user_logged_in() && class_exists( 'WC_Payment_Token_CC' ) ) {
	65	+ public function woocommerce_get_customer_payment_tokens($tokens = array(), $customer_id, $gateway_id) {
	66	+ if (is_user_logged_in() && class_exists('WC_Payment_Token_CC')) {
67	67	$stored_tokens = array();
68	68
69		- foreach ( $tokens as $token ) {
	69	+ foreach ($tokens as $token) {
70	70	$stored_tokens[] = $token->get_token();
71	71	}
72	72
73		- if ( 'stripe' === $gateway_id ) {
74		- $stripe_customer = new WC_Stripe_Customer( $customer_id );
	73	+ if ('stripe' === $gateway_id) {
	74	+ $stripe_customer = new WC_Stripe_Customer($customer_id);
75	75	$stripe_sources = $stripe_customer->get_sources();
76	76
77		- foreach ( $stripe_sources as $source ) {
78		- if ( isset( $source->type ) && 'card' === $source->type ) {
79		- if ( ! in_array( $source->id, $stored_tokens ) ) {
	77	+ foreach ($stripe_sources as $source) {
	78	+ if (isset($source->type) && 'card' === $source->type) {
	79	+ if ( ! in_array($source->id, $stored_tokens)) {
80	80	$token = new WC_Payment_Token_CC();
81		- $token->set_token( $source->id );
82		- $token->set_gateway_id( 'stripe' );
83		-
84		- if ( 'source' === $source->object && 'card' === $source->type ) {
85		- $token->set_card_type( strtolower( $source->card->brand ) );
86		- $token->set_last4( $source->card->last4 );
87		- $token->set_expiry_month( $source->card->exp_month );
88		- $token->set_expiry_year( $source->card->exp_year );
	81	+ $token->set_token($source->id);
	82	+ $token->set_gateway_id('stripe');
	83	+
	84	+ if ('source' === $source->object && 'card' === $source->type) {
	85	+ $token->set_card_type(strtolower($source->card->brand));
	86	+ $token->set_last4($source->card->last4);
	87	+ $token->set_expiry_month($source->card->exp_month);
	88	+ $token->set_expiry_year($source->card->exp_year);
89	89	}
90	90
91		- $token->set_user_id( $customer_id );
	91	+ $token->set_user_id($customer_id);
92	92	$token->save();
93		- $tokens[ $token->get_id() ] = $token;
	93	+ $tokens[$token->get_id()] = $token;
94	94	}
95	95	} else {
96		- if ( ! in_array( $source->id, $stored_tokens ) && 'card' === $source->object ) {
	96	+ if ( ! in_array($source->id, $stored_tokens) && 'card' === $source->object) {
97	97	$token = new WC_Payment_Token_CC();
98		- $token->set_token( $source->id );
99		- $token->set_gateway_id( 'stripe' );
100		- $token->set_card_type( strtolower( $source->brand ) );
101		- $token->set_last4( $source->last4 );
102		- $token->set_expiry_month( $source->exp_month );
103		- $token->set_expiry_year( $source->exp_year );
104		- $token->set_user_id( $customer_id );
	98	+ $token->set_token($source->id);
	99	+ $token->set_gateway_id('stripe');
	100	+ $token->set_card_type(strtolower($source->brand));
	101	+ $token->set_last4($source->last4);
	102	+ $token->set_expiry_month($source->exp_month);
	103	+ $token->set_expiry_year($source->exp_year);
	104	+ $token->set_user_id($customer_id);
105	105	$token->save();
106		- $tokens[ $token->get_id() ] = $token;
	106	+ $tokens[$token->get_id()] = $token;
107	107	}
108	108	}
109	109	}
110	110	}
111	111
112		- if ( 'stripe_sepa' === $gateway_id ) {
113		- $stripe_customer = new WC_Stripe_Customer( $customer_id );
	112	+ if ('stripe_sepa' === $gateway_id) {
	113	+ $stripe_customer = new WC_Stripe_Customer($customer_id);
114	114	$stripe_sources = $stripe_customer->get_sources();
115	115
116		- foreach ( $stripe_sources as $source ) {
117		- if ( isset( $source->type ) && 'sepa_debit' === $source->type ) {
118		- if ( ! in_array( $source->id, $stored_tokens ) ) {
	116	+ foreach ($stripe_sources as $source) {
	117	+ if (isset($source->type) && 'sepa_debit' === $source->type) {
	118	+ if ( ! in_array($source->id, $stored_tokens)) {
119	119	$token = new WC_Payment_Token_SEPA();
120		- $token->set_token( $source->id );
121		- $token->set_gateway_id( 'stripe_sepa' );
122		- $token->set_last4( $source->sepa_debit->last4 );
123		- $token->set_user_id( $customer_id );
	120	+ $token->set_token($source->id);
	121	+ $token->set_gateway_id('stripe_sepa');
	122	+ $token->set_last4($source->sepa_debit->last4);
	123	+ $token->set_user_id($customer_id);
124	124	$token->save();
125		- $tokens[ $token->get_id() ] = $token;
	125	+ $tokens[$token->get_id()] = $token;
126	126	}
127	127	}
128	128	}
		@@ -141,13 +141,13 @@ discard block
		block discarded – undo
141	141	* @param WC_Payment_Token $payment_token The payment token associated with this method entry
142	142	* @return array Filtered item
143	143	*/
144		- public function get_account_saved_payment_methods_list_item_sepa( $item, $payment_token ) {
145		- if ( 'sepa' !== strtolower( $payment_token->get_type() ) ) {
	144	+ public function get_account_saved_payment_methods_list_item_sepa($item, $payment_token) {
	145	+ if ('sepa' !== strtolower($payment_token->get_type())) {
146	146	return $item;
147	147	}
148	148
149	149	$item['method']['last4'] = $payment_token->get_last4();
150		- $item['method']['brand'] = esc_html__( 'SEPA IBAN', 'woocommerce-gateway-stripe' );
	150	+ $item['method']['brand'] = esc_html__('SEPA IBAN', 'woocommerce-gateway-stripe');
151	151
152	152	return $item;
153	153	}
		@@ -158,10 +158,10 @@ discard block
		block discarded – undo
158	158	* @since 3.1.0
159	159	* @version 4.0.0
160	160	*/
161		- public function woocommerce_payment_token_deleted( $token_id, $token ) {
162		- if ( 'stripe' === $token->get_gateway_id() \|\| 'stripe_sepa' === $token->get_gateway_id() ) {
163		- $stripe_customer = new WC_Stripe_Customer( get_current_user_id() );
164		- $stripe_customer->delete_source( $token->get_token() );
	161	+ public function woocommerce_payment_token_deleted($token_id, $token) {
	162	+ if ('stripe' === $token->get_gateway_id() \|\| 'stripe_sepa' === $token->get_gateway_id()) {
	163	+ $stripe_customer = new WC_Stripe_Customer(get_current_user_id());
	164	+ $stripe_customer->delete_source($token->get_token());
165	165	}
166	166	}
167	167
		@@ -171,12 +171,12 @@ discard block
		block discarded – undo
171	171	* @since 3.1.0
172	172	* @version 4.0.0
173	173	*/
174		- public function woocommerce_payment_token_set_default( $token_id ) {
175		- $token = WC_Payment_Tokens::get( $token_id );
	174	+ public function woocommerce_payment_token_set_default($token_id) {
	175	+ $token = WC_Payment_Tokens::get($token_id);
176	176
177		- if ( 'stripe' === $token->get_gateway_id() \|\| 'stripe_sepa' === $token->get_gateway_id() ) {
178		- $stripe_customer = new WC_Stripe_Customer( get_current_user_id() );
179		- $stripe_customer->set_default_source( $token->get_token() );
	177	+ if ('stripe' === $token->get_gateway_id() \|\| 'stripe_sepa' === $token->get_gateway_id()) {
	178	+ $stripe_customer = new WC_Stripe_Customer(get_current_user_id());
	179	+ $stripe_customer->set_default_source($token->get_token());
180	180	}
181	181	}
182	182	}

		@@ -1,6 +1,6 @@ discard block
		block discarded – undo
1	1	<?php
2	2
3		-if ( ! defined( 'ABSPATH' ) ) {
	3	+if ( ! defined('ABSPATH')) {
4	4	exit; // Exit if accessed directly
5	5	}
6	6
		@@ -35,10 +35,10 @@ discard block
		block discarded – undo
35	35	* @param string $deprecated Deprecated since WooCommerce 3.0
36	36	* @return string
37	37	*/
38		- public function get_display_name( $deprecated = '' ) {
	38	+ public function get_display_name($deprecated = '') {
39	39	$display = sprintf(
40	40	/* translators: last 4 digits of IBAN account */
41		- __( 'SEPA IBAN ending in %s', 'woocommerce-gateway-stripe' ),
	41	+ __('SEPA IBAN ending in %s', 'woocommerce-gateway-stripe'),
42	42	$this->get_last4()
43	43	);
44	44
		@@ -66,11 +66,11 @@ discard block
		block discarded – undo
66	66	* @return boolean True if the passed data is valid
67	67	*/
68	68	public function validate() {
69		- if ( false === parent::validate() ) {
	69	+ if (false === parent::validate()) {
70	70	return false;
71	71	}
72	72
73		- if ( ! $this->get_last4( 'edit' ) ) {
	73	+ if ( ! $this->get_last4('edit')) {
74	74	return false;
75	75	}
76	76
		@@ -85,8 +85,8 @@ discard block
		block discarded – undo
85	85	* @param string $context
86	86	* @return string Last 4 digits
87	87	*/
88		- public function get_last4( $context = 'view' ) {
89		- return $this->get_prop( 'last4', $context );
	88	+ public function get_last4($context = 'view') {
	89	+ return $this->get_prop('last4', $context);
90	90	}
91	91
92	92	/**
		@@ -95,7 +95,7 @@ discard block
		block discarded – undo
95	95	* @version 4.0.0
96	96	* @param string $last4
97	97	*/
98		- public function set_last4( $last4 ) {
99		- $this->set_prop( 'last4', $last4 );
	98	+ public function set_last4($last4) {
	99	+ $this->set_prop('last4', $last4);
100	100	}
101	101	}

		@@ -1,50 +1,50 @@
		block discarded – undo
1	1	<?php
2		-if ( ! defined( 'ABSPATH' ) ) {
	2	+if ( ! defined('ABSPATH')) {
3	3	exit;
4	4	}
5	5
6	6	$webhook_url = WC_Stripe_Helper::get_webhook_url();
7	7
8		-return apply_filters( 'wc_stripe_bitcoin_settings',
	8	+return apply_filters('wc_stripe_bitcoin_settings',
9	9	array(
10	10	'geo_target' => array(
11		- 'description' => __( 'Relevant Payer Geography: Global', 'woocommerce-gateway-stripe' ),
	11	+ 'description' => __('Relevant Payer Geography: Global', 'woocommerce-gateway-stripe'),
12	12	'type' => 'title',
13	13	),
14	14	'guide' => array(
15		- 'description' => __( '<a href="https://stripe.com/payments/payment-methods-guide#bitcoin" target="_blank">Payment Method Guide</a>', 'woocommerce-gateway-stripe' ),
	15	+ 'description' => __('<a href="https://stripe.com/payments/payment-methods-guide#bitcoin" target="_blank">Payment Method Guide</a>', 'woocommerce-gateway-stripe'),
16	16	'type' => 'title',
17	17	),
18	18	'activation' => array(
19		- 'description' => __( 'Must be activated from your Stripe Dashboard Settings <a href="https://dashboard.stripe.com/account/payments/settings" target="_blank">here</a>', 'woocommerce-gateway-stripe' ),
	19	+ 'description' => __('Must be activated from your Stripe Dashboard Settings <a href="https://dashboard.stripe.com/account/payments/settings" target="_blank">here</a>', 'woocommerce-gateway-stripe'),
20	20	'type' => 'title',
21	21	),
22	22	'enabled' => array(
23		- 'title' => __( 'Enable/Disable', 'woocommerce-gateway-stripe' ),
24		- 'label' => __( 'Enable Stripe Bitcoin', 'woocommerce-gateway-stripe' ),
	23	+ 'title' => __('Enable/Disable', 'woocommerce-gateway-stripe'),
	24	+ 'label' => __('Enable Stripe Bitcoin', 'woocommerce-gateway-stripe'),
25	25	'type' => 'checkbox',
26	26	'description' => '',
27	27	'default' => 'no',
28	28	),
29	29	'title' => array(
30		- 'title' => __( 'Title', 'woocommerce-gateway-stripe' ),
	30	+ 'title' => __('Title', 'woocommerce-gateway-stripe'),
31	31	'type' => 'text',
32		- 'description' => __( 'This controls the title which the user sees during checkout.', 'woocommerce-gateway-stripe' ),
33		- 'default' => __( 'Bitcoin', 'woocommerce-gateway-stripe' ),
	32	+ 'description' => __('This controls the title which the user sees during checkout.', 'woocommerce-gateway-stripe'),
	33	+ 'default' => __('Bitcoin', 'woocommerce-gateway-stripe'),
34	34	'desc_tip' => true,
35	35	),
36	36	'description' => array(
37		- 'title' => __( 'Description', 'woocommerce-gateway-stripe' ),
	37	+ 'title' => __('Description', 'woocommerce-gateway-stripe'),
38	38	'type' => 'text',
39		- 'description' => __( 'This controls the description which the user sees during checkout.', 'woocommerce-gateway-stripe' ),
40		- 'default' => __( 'Bitcoin payment information will be provided when you place the order.', 'woocommerce-gateway-stripe' ),
	39	+ 'description' => __('This controls the description which the user sees during checkout.', 'woocommerce-gateway-stripe'),
	40	+ 'default' => __('Bitcoin payment information will be provided when you place the order.', 'woocommerce-gateway-stripe'),
41	41	'desc_tip' => true,
42	42	),
43	43	'webhook' => array(
44		- 'title' => __( 'Webhook Endpoints', 'woocommerce-gateway-stripe' ),
	44	+ 'title' => __('Webhook Endpoints', 'woocommerce-gateway-stripe'),
45	45	'type' => 'title',
46	46	/* translators: webhook URL */
47		- 'description' => sprintf( __( 'You must add the webhook endpoint <strong style="background-color:#ddd;">  %s  </strong> to your Stripe Account Settings <a href="https://dashboard.stripe.com/account/webhooks" target="_blank">Here</a> so you can receive notifications on the charge statuses.', 'woocommerce-gateway-stripe' ), $webhook_url ),
	47	+ 'description' => sprintf(__('You must add the webhook endpoint <strong style="background-color:#ddd;">  %s  </strong> to your Stripe Account Settings <a href="https://dashboard.stripe.com/account/webhooks" target="_blank">Here</a> so you can receive notifications on the charge statuses.', 'woocommerce-gateway-stripe'), $webhook_url),
48	48	),
49	49	)
50	50	);

woocommerce / woocommerce-gateway-stripe

Pull Request — master (#467)

Status

Category

Spacing +9 added lines, -9 removed lines patch added patch discarded remove patch

Spacing +57 added lines, -57 removed lines patch added patch discarded remove patch

Spacing +11 added lines, -11 removed lines patch added patch discarded remove patch

Spacing +15 added lines, -15 removed lines patch added patch discarded remove patch

Spacing +15 added lines, -15 removed lines patch added patch discarded remove patch

Spacing +15 added lines, -15 removed lines patch added patch discarded remove patch

Spacing +15 added lines, -15 removed lines patch added patch discarded remove patch

Spacing +15 added lines, -15 removed lines patch added patch discarded remove patch

Spacing +15 added lines, -15 removed lines patch added patch discarded remove patch