woocommerce / woocommerce-gateway-stripe

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

Spacing +60 added lines, -60 removed lines

@@ -5,7 +5,7 @@ 
  * @since 4.0.6
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -32,17 +32,17 @@ 
 	public $apple_pay_verify_notice;
 
 	public function __construct() {
-		add_action( 'init', array( $this, 'add_domain_association_rewrite_rule' ) );
-		add_filter( 'query_vars', array( $this, 'whitelist_domain_association_query_param' ), 10, 1 );
-		add_action( 'parse_request', array( $this, 'parse_domain_association_request' ), 10, 1 );
+		add_action('init', array($this, 'add_domain_association_rewrite_rule'));
+		add_filter('query_vars', array($this, 'whitelist_domain_association_query_param'), 10, 1);
+		add_action('parse_request', array($this, 'parse_domain_association_request'), 10, 1);
 
-		add_action( 'woocommerce_stripe_updated', array( $this, 'verify_domain_if_configured' ) );
-		add_action( 'add_option_woocommerce_stripe_settings', array( $this, 'verify_domain_on_new_settings' ), 10, 2 );
-		add_action( 'update_option_woocommerce_stripe_settings', array( $this, 'verify_domain_on_updated_settings' ), 10, 2 );
-		add_action( 'admin_notices', array( $this, 'admin_notices' ) );
+		add_action('woocommerce_stripe_updated', array($this, 'verify_domain_if_configured'));
+		add_action('add_option_woocommerce_stripe_settings', array($this, 'verify_domain_on_new_settings'), 10, 2);
+		add_action('update_option_woocommerce_stripe_settings', array($this, 'verify_domain_on_updated_settings'), 10, 2);
+		add_action('admin_notices', array($this, 'admin_notices'));
 
-		$this->stripe_settings         = get_option( 'woocommerce_stripe_settings', array() );
-		$this->apple_pay_domain_set    = 'yes' === $this->get_option( 'apple_pay_domain_set', 'no' );
+		$this->stripe_settings         = get_option('woocommerce_stripe_settings', array());
+		$this->apple_pay_domain_set    = 'yes' === $this->get_option('apple_pay_domain_set', 'no');
 		$this->apple_pay_verify_notice = '';
 	}
 
@@ -54,13 +54,13 @@ 
 	 * @param string default
 	 * @return string $setting_value
 	 */
-	public function get_option( $setting = '', $default = '' ) {
-		if ( empty( $this->stripe_settings ) ) {
+	public function get_option($setting = '', $default = '') {
+		if (empty($this->stripe_settings)) {
 			return $default;
 		}
 
-		if ( ! empty( $this->stripe_settings[ $setting ] ) ) {
-			return $this->stripe_settings[ $setting ];
+		if ( ! empty($this->stripe_settings[$setting])) {
+			return $this->stripe_settings[$setting];
 		}
 
 		return $default;
@@ -73,8 +73,8 @@ 
 	 * @return string Whether Apple Pay required settings are enabled.
 	 */
 	private function is_enabled() {
-		$stripe_enabled                 = 'yes' === $this->get_option( 'enabled', 'no' );
-		$payment_request_button_enabled = 'yes' === $this->get_option( 'payment_request', 'yes' );
+		$stripe_enabled                 = 'yes' === $this->get_option('enabled', 'no');
+		$payment_request_button_enabled = 'yes' === $this->get_option('payment_request', 'yes');
 
 		return $stripe_enabled && $payment_request_button_enabled;
 	}
@@ -86,8 +86,8 @@ 
 	 * @return string Secret key.
 	 */
 	private function get_secret_key() {
-		$testmode = 'yes' === $this->get_option( 'testmode', 'no' );
-		return $testmode ? $this->get_option( 'test_secret_key' ) : $this->get_option( 'secret_key' );
+		$testmode = 'yes' === $this->get_option('testmode', 'no');
+		return $testmode ? $this->get_option('test_secret_key') : $this->get_option('secret_key');
 	}
 
 	/**
@@ -97,7 +97,7 @@ 
 		$regex    = '^\.well-known\/apple-developer-merchantid-domain-association$';
 		$redirect = 'index.php?apple-developer-merchantid-domain-association=1';
 
-		add_rewrite_rule( $regex, $redirect, 'top' );
+		add_rewrite_rule($regex, $redirect, 'top');
 	}
 
 	/**
@@ -106,7 +106,7 @@ 
 	 * @param  array $query_vars - provided public query vars.
 	 * @return array Updated public query vars.
 	 */
-	public function whitelist_domain_association_query_param( $query_vars ) {
+	public function whitelist_domain_association_query_param($query_vars) {
 		$query_vars[] = 'apple-developer-merchantid-domain-association';
 		return $query_vars;
 	}
@@ -116,17 +116,17 @@ 
 	 *
 	 * @param WP WordPress environment object.
 	 */
-	public function parse_domain_association_request( $wp ) {
+	public function parse_domain_association_request($wp) {
 		if (
-			! isset( $wp->query_vars['apple-developer-merchantid-domain-association'] ) ||
+			! isset($wp->query_vars['apple-developer-merchantid-domain-association']) ||
 			'1' !== $wp->query_vars['apple-developer-merchantid-domain-association']
 		) {
 			return;
 		}
 
 		$path = WC_STRIPE_PLUGIN_PATH . '/apple-developer-merchantid-domain-association';
-		header( 'Content-Type: application/octet-stream' );
-		echo esc_html( file_get_contents( $path ) );
+		header('Content-Type: application/octet-stream');
+		echo esc_html(file_get_contents($path));
 		exit;
 	}
 
@@ -137,9 +137,9 @@ 
 	 * @version 4.5.4
 	 * @param string $secret_key
 	 */
-	private function make_domain_registration_request( $secret_key ) {
-		if ( empty( $secret_key ) ) {
-			throw new Exception( __( 'Unable to verify domain - missing secret key.', 'woocommerce-gateway-stripe' ) );
+	private function make_domain_registration_request($secret_key) {
+		if (empty($secret_key)) {
+			throw new Exception(__('Unable to verify domain - missing secret key.', 'woocommerce-gateway-stripe'));
 		}
 
 		$endpoint = 'https://api.stripe.com/v1/apple_pay/domains';
@@ -157,22 +157,22 @@ 
 			$endpoint,
 			array(
 				'headers' => $headers,
-				'body'    => http_build_query( $data ),
+				'body'    => http_build_query($data),
 			)
 		);
 
-		if ( is_wp_error( $response ) ) {
+		if (is_wp_error($response)) {
 			/* translators: error message */
-			throw new Exception( sprintf( __( 'Unable to verify domain - %s', 'woocommerce-gateway-stripe' ), $response->get_error_message() ) );
+			throw new Exception(sprintf(__('Unable to verify domain - %s', 'woocommerce-gateway-stripe'), $response->get_error_message()));
 		}
 
-		if ( 200 !== $response['response']['code'] ) {
-			$parsed_response = json_decode( $response['body'] );
+		if (200 !== $response['response']['code']) {
+			$parsed_response = json_decode($response['body']);
 
 			$this->apple_pay_verify_notice = $parsed_response->error->message;
 
 			/* translators: error message */
-			throw new Exception( sprintf( __( 'Unable to verify domain - %s', 'woocommerce-gateway-stripe' ), $parsed_response->error->message ) );
+			throw new Exception(sprintf(__('Unable to verify domain - %s', 'woocommerce-gateway-stripe'), $parsed_response->error->message));
 		}
 	}
 
@@ -186,27 +186,27 @@ 
 	 *
 	 * @return bool Whether domain verification succeeded.
 	 */
-	public function register_domain_with_apple( $secret_key ) {
+	public function register_domain_with_apple($secret_key) {
 		try {
-			$this->make_domain_registration_request( $secret_key );
+			$this->make_domain_registration_request($secret_key);
 
 			// No errors to this point, verification success!
 			$this->stripe_settings['apple_pay_domain_set'] = 'yes';
 			$this->apple_pay_domain_set                    = true;
 
-			update_option( 'woocommerce_stripe_settings', $this->stripe_settings );
+			update_option('woocommerce_stripe_settings', $this->stripe_settings);
 
-			WC_Stripe_Logger::log( 'Your domain has been verified with Apple Pay!' );
+			WC_Stripe_Logger::log('Your domain has been verified with Apple Pay!');
 
 			return true;
 
-		} catch ( Exception $e ) {
+		} catch (Exception $e) {
 			$this->stripe_settings['apple_pay_domain_set'] = 'no';
 			$this->apple_pay_domain_set                    = false;
 
-			update_option( 'woocommerce_stripe_settings', $this->stripe_settings );
+			update_option('woocommerce_stripe_settings', $this->stripe_settings);
 
-			WC_Stripe_Logger::log( 'Error: ' . $e->getMessage() );
+			WC_Stripe_Logger::log('Error: ' . $e->getMessage());
 
 			return false;
 		}
@@ -221,7 +221,7 @@ 
 	public function verify_domain_if_configured() {
 		$secret_key = $this->get_secret_key();
 
-		if ( ! $this->is_enabled() || empty( $secret_key ) ) {
+		if ( ! $this->is_enabled() || empty($secret_key)) {
 			return;
 		}
 
@@ -229,10 +229,10 @@ 
 		flush_rewrite_rules();
 
 		// Register the domain with Apple Pay.
-		$verification_complete = $this->register_domain_with_apple( $secret_key );
+		$verification_complete = $this->register_domain_with_apple($secret_key);
 
 		// Show/hide notes if necessary.
-		WC_Stripe_Inbox_Notes::notify_on_apple_pay_domain_verification( $verification_complete );
+		WC_Stripe_Inbox_Notes::notify_on_apple_pay_domain_verification($verification_complete);
 	}
 
 	/**
@@ -241,8 +241,8 @@ 
 	 * @since 4.5.4
 	 * @version 4.5.4
 	 */
-	public function verify_domain_on_new_settings( $option, $settings ) {
-		$this->verify_domain_on_updated_settings( array(), $settings );
+	public function verify_domain_on_new_settings($option, $settings) {
+		$this->verify_domain_on_updated_settings(array(), $settings);
 	}
 
 	/**
@@ -251,7 +251,7 @@ 
 	 * @since 4.5.3
 	 * @version 4.5.4
 	 */
-	public function verify_domain_on_updated_settings( $prev_settings, $settings ) {
+	public function verify_domain_on_updated_settings($prev_settings, $settings) {
 		// Grab previous state and then update cached settings.
 		$this->stripe_settings = $prev_settings;
 		$prev_secret_key       = $this->get_secret_key();
@@ -259,7 +259,7 @@ 
 		$this->stripe_settings = $settings;
 
 		// If Stripe or Payment Request Button wasn't enabled (or secret key was different) then might need to verify now.
-		if ( ! $prev_is_enabled || ( $this->get_secret_key() !== $prev_secret_key ) ) {
+		if ( ! $prev_is_enabled || ($this->get_secret_key() !== $prev_secret_key)) {
 			$this->verify_domain_if_configured();
 		}
 	}
@@ -270,16 +270,16 @@ 
 	 * @since 4.0.6
 	 */
 	public function admin_notices() {
-		if ( ! $this->is_enabled() ) {
+		if ( ! $this->is_enabled()) {
 			return;
 		}
 
-		if ( ! current_user_can( 'manage_woocommerce' ) ) {
+		if ( ! current_user_can('manage_woocommerce')) {
 			return;
 		}
 
-		$empty_notice = empty( $this->apple_pay_verify_notice );
-		if ( $empty_notice && ( $this->apple_pay_domain_set || empty( $this->secret_key ) ) ) {
+		$empty_notice = empty($this->apple_pay_verify_notice);
+		if ($empty_notice && ($this->apple_pay_domain_set || empty($this->secret_key))) {
 			return;
 		}
 
@@ -288,28 +288,28 @@ 
 		 * when setting screen is displayed. So if domain verification is not set,
 		 * something went wrong so lets notify user.
 		 */
-		$allowed_html                      = array(
+		$allowed_html = array(
 			'a' => array(
 				'href'  => array(),
 				'title' => array(),
 			),
 		);
-		$verification_failed_without_error = __( 'Apple Pay domain verification failed.', 'woocommerce-gateway-stripe' );
-		$verification_failed_with_error    = __( 'Apple Pay domain verification failed with the following error:', 'woocommerce-gateway-stripe' );
+		$verification_failed_without_error = __('Apple Pay domain verification failed.', 'woocommerce-gateway-stripe');
+		$verification_failed_with_error    = __('Apple Pay domain verification failed with the following error:', 'woocommerce-gateway-stripe');
 		$check_log_text                    = sprintf(
 			/* translators: 1) HTML anchor open tag 2) HTML anchor closing tag */
-			esc_html__( 'Please check the %1$slogs%2$s for more details on this issue. Logging must be enabled to see recorded logs.', 'woocommerce-gateway-stripe' ),
-			'<a href="' . admin_url( 'admin.php?page=wc-status&tab=logs' ) . '">',
+			esc_html__('Please check the %1$slogs%2$s for more details on this issue. Logging must be enabled to see recorded logs.', 'woocommerce-gateway-stripe'),
+			'<a href="' . admin_url('admin.php?page=wc-status&tab=logs') . '">',
 			'</a>'
 		);
 
 		?>
 		<div class="error stripe-apple-pay-message">
-			<?php if ( $empty_notice ) : ?>
-				<p><?php echo esc_html( $verification_failed_without_error ); ?></p>
+			<?php if ($empty_notice) : ?>
+				<p><?php echo esc_html($verification_failed_without_error); ?></p>
 			<?php else : ?>
-				<p><?php echo esc_html( $verification_failed_with_error ); ?></p>
-				<p><i><?php echo wp_kses( make_clickable( esc_html( $this->apple_pay_verify_notice ) ), $allowed_html ); ?></i></p>
+				<p><?php echo esc_html($verification_failed_with_error); ?></p>
+				<p><i><?php echo wp_kses(make_clickable(esc_html($this->apple_pay_verify_notice)), $allowed_html); ?></i></p>
 			<?php endif; ?>
 			<p><?php echo $check_log_text; ?></p>
 		</div>

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

Spacing +40 added lines, -40 removed lines

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

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

Spacing +35 added lines, -35 removed lines

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

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

Spacing +1 added lines, -1 removed lines

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

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

Spacing +2 added lines, -2 removed lines

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

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

Spacing +6 added lines, -6 removed lines

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

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

Spacing +9 added lines, -9 removed lines

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

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

Spacing +16 added lines, -16 removed lines

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

tests/phpunit/bootstrap.php

Spacing +6 added lines, -6 removed lines

@@ -1,8 +1,8 @@ 
 <?php
 
-$_tests_dir = getenv( 'WP_TESTS_DIR' ) ?: '/tmp/wordpress-tests-lib';
+$_tests_dir = getenv('WP_TESTS_DIR') ?: '/tmp/wordpress-tests-lib';
 
-if ( PHP_VERSION_ID >= 80000 && is_dir( $_tests_dir . '/includes/phpunit7/MockObject' ) ) {
+if (PHP_VERSION_ID >= 80000 && is_dir($_tests_dir . '/includes/phpunit7/MockObject')) {
 	// WP Core test library includes patches for PHPUnit 7 to make it compatible with PHP8.
 	require_once $_tests_dir . '/includes/phpunit7/MockObject/Builder/NamespaceMatch.php';
 	require_once $_tests_dir . '/includes/phpunit7/MockObject/Builder/ParametersMatch.php';
@@ -13,13 +13,13 @@ 
 require_once $_tests_dir . '/includes/functions.php';
 
 function _manually_load_plugin() {
-	$plugins_dir = dirname( dirname( dirname( __DIR__ ) ) );
+	$plugins_dir = dirname(dirname(dirname(__DIR__)));
 	require $plugins_dir . '/woocommerce/woocommerce.php';
 	require __DIR__ . '/setup.php';
 	require $plugins_dir . '/woocommerce-gateway-stripe/woocommerce-gateway-stripe.php';
 }
 
-tests_add_filter( 'muplugins_loaded', '_manually_load_plugin' );
+tests_add_filter('muplugins_loaded', '_manually_load_plugin');
 
 require $_tests_dir . '/includes/bootstrap.php';
 
@@ -32,6 +32,6 @@ 
 require_once __DIR__ . '/helpers/class-wc-helper-subscriptions-cart.php';
 
 // We use outdated PHPUnit version, which emits deprecation errors in PHP 7.4 (deprecated reflection APIs).
-if ( defined( 'PHP_VERSION_ID' ) && PHP_VERSION_ID >= 70400 ) {
-	error_reporting( error_reporting() ^ E_DEPRECATED ); // phpcs:ignore
+if (defined('PHP_VERSION_ID') && PHP_VERSION_ID >= 70400) {
+	error_reporting(error_reporting() ^ E_DEPRECATED); // phpcs:ignore
 }