Completed
Pull Request — master (#1383)
by
unknown
01:41
created
includes/class-wc-stripe-sepa-payment-token.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -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
 }
Please login to merge, or discard this patch.
tests/phpunit/test-wc-stripe.php 1 patch
Spacing   +21 added lines, -21 removed lines patch added patch discarded remove patch
@@ -2,12 +2,12 @@  discard block
 block discarded – undo
2 2
 
3 3
 class WC_Stripe_Test extends WP_UnitTestCase {
4 4
 	public function test_constants_defined() {
5
-		$this->assertTrue( defined( 'WC_STRIPE_VERSION' ) );
6
-		$this->assertTrue( defined( 'WC_STRIPE_MIN_PHP_VER' ) );
7
-		$this->assertTrue( defined( 'WC_STRIPE_MIN_WC_VER' ) );
8
-		$this->assertTrue( defined( 'WC_STRIPE_MAIN_FILE' ) );
9
-		$this->assertTrue( defined( 'WC_STRIPE_PLUGIN_URL' ) );
10
-		$this->assertTrue( defined( 'WC_STRIPE_PLUGIN_PATH' ) );
5
+		$this->assertTrue(defined('WC_STRIPE_VERSION'));
6
+		$this->assertTrue(defined('WC_STRIPE_MIN_PHP_VER'));
7
+		$this->assertTrue(defined('WC_STRIPE_MIN_WC_VER'));
8
+		$this->assertTrue(defined('WC_STRIPE_MAIN_FILE'));
9
+		$this->assertTrue(defined('WC_STRIPE_PLUGIN_URL'));
10
+		$this->assertTrue(defined('WC_STRIPE_PLUGIN_PATH'));
11 11
 	}
12 12
 
13 13
 	/**
@@ -15,11 +15,11 @@  discard block
 block discarded – undo
15 15
 	 * This test will see if we're indeed converting the price correctly.
16 16
 	 */
17 17
 	public function test_price_conversion_before_send_to_stripe() {
18
-		$this->assertEquals( 10050, WC_Stripe_Helper::get_stripe_amount( 100.50, 'USD' ) );
19
-		$this->assertEquals( 10050, WC_Stripe_Helper::get_stripe_amount( 10050, 'JPY' ) );
20
-		$this->assertEquals( 100, WC_Stripe_Helper::get_stripe_amount( 100.50, 'JPY' ) );
21
-		$this->assertEquals( 10050, WC_Stripe_Helper::get_stripe_amount( 100.50 ) );
22
-		$this->assertInternalType( 'int', WC_Stripe_Helper::get_stripe_amount( 100.50, 'USD' ) );
18
+		$this->assertEquals(10050, WC_Stripe_Helper::get_stripe_amount(100.50, 'USD'));
19
+		$this->assertEquals(10050, WC_Stripe_Helper::get_stripe_amount(10050, 'JPY'));
20
+		$this->assertEquals(100, WC_Stripe_Helper::get_stripe_amount(100.50, 'JPY'));
21
+		$this->assertEquals(10050, WC_Stripe_Helper::get_stripe_amount(100.50));
22
+		$this->assertInternalType('int', WC_Stripe_Helper::get_stripe_amount(100.50, 'USD'));
23 23
 	}
24 24
 
25 25
 	/**
@@ -33,37 +33,37 @@  discard block
 block discarded – undo
33 33
 		$balance_fee1->net = 10000;
34 34
 		$balance_fee1->currency = 'USD';
35 35
 
36
-		$this->assertEquals( 105.00, WC_Stripe_Helper::format_balance_fee( $balance_fee1, 'fee' ) );
36
+		$this->assertEquals(105.00, WC_Stripe_Helper::format_balance_fee($balance_fee1, 'fee'));
37 37
 
38 38
 		$balance_fee2 = new stdClass();
39 39
 		$balance_fee2->fee = 10500;
40 40
 		$balance_fee2->net = 10000;
41 41
 		$balance_fee2->currency = 'JPY';
42 42
 
43
-		$this->assertEquals( 10500, WC_Stripe_Helper::format_balance_fee( $balance_fee2, 'fee' ) );
43
+		$this->assertEquals(10500, WC_Stripe_Helper::format_balance_fee($balance_fee2, 'fee'));
44 44
 
45 45
 		$balance_fee3 = new stdClass();
46 46
 		$balance_fee3->fee = 10500;
47 47
 		$balance_fee3->net = 10000;
48 48
 		$balance_fee3->currency = 'USD';
49 49
 
50
-		$this->assertEquals( 100.00, WC_Stripe_Helper::format_balance_fee( $balance_fee3, 'net' ) );
50
+		$this->assertEquals(100.00, WC_Stripe_Helper::format_balance_fee($balance_fee3, 'net'));
51 51
 
52 52
 		$balance_fee4 = new stdClass();
53 53
 		$balance_fee4->fee = 10500;
54 54
 		$balance_fee4->net = 10000;
55 55
 		$balance_fee4->currency = 'JPY';
56 56
 
57
-		$this->assertEquals( 10000, WC_Stripe_Helper::format_balance_fee( $balance_fee4, 'net' ) );
57
+		$this->assertEquals(10000, WC_Stripe_Helper::format_balance_fee($balance_fee4, 'net'));
58 58
 
59 59
 		$balance_fee5 = new stdClass();
60 60
 		$balance_fee5->fee = 10500;
61 61
 		$balance_fee5->net = 10000;
62 62
 		$balance_fee5->currency = 'USD';
63 63
 
64
-		$this->assertEquals( 105.00, WC_Stripe_Helper::format_balance_fee( $balance_fee5 ) );
64
+		$this->assertEquals(105.00, WC_Stripe_Helper::format_balance_fee($balance_fee5));
65 65
 
66
-		$this->assertInternalType( 'string', WC_Stripe_Helper::format_balance_fee( $balance_fee5 ) );
66
+		$this->assertInternalType('string', WC_Stripe_Helper::format_balance_fee($balance_fee5));
67 67
 	}
68 68
 
69 69
 	/**
@@ -76,27 +76,27 @@  discard block
 block discarded – undo
76 76
 			'expected' => 'Tests Store',
77 77
 		);
78 78
 
79
-		$this->assertEquals( $statement_descriptor1['expected'], WC_Stripe_Helper::clean_statement_descriptor( $statement_descriptor1['actual'] ) );
79
+		$this->assertEquals($statement_descriptor1['expected'], WC_Stripe_Helper::clean_statement_descriptor($statement_descriptor1['actual']));
80 80
 
81 81
 		$statement_descriptor2 = array(
82 82
 			'actual'   => 'Test\'s Store > Driving Course Range',
83 83
 			'expected' => 'Tests Store  Driving C',
84 84
 		);
85 85
 
86
-		$this->assertEquals( $statement_descriptor2['expected'], WC_Stripe_Helper::clean_statement_descriptor( $statement_descriptor2['actual'] ) );
86
+		$this->assertEquals($statement_descriptor2['expected'], WC_Stripe_Helper::clean_statement_descriptor($statement_descriptor2['actual']));
87 87
 
88 88
 		$statement_descriptor3 = array(
89 89
 			'actual'   => 'Test\'s Store < Driving Course Range',
90 90
 			'expected' => 'Tests Store  Driving C',
91 91
 		);
92 92
 
93
-		$this->assertEquals( $statement_descriptor3['expected'], WC_Stripe_Helper::clean_statement_descriptor( $statement_descriptor3['actual'] ) );
93
+		$this->assertEquals($statement_descriptor3['expected'], WC_Stripe_Helper::clean_statement_descriptor($statement_descriptor3['actual']));
94 94
 
95 95
 		$statement_descriptor4 = array(
96 96
 			'actual'   => 'Test\'s Store " Driving Course Range',
97 97
 			'expected' => 'Tests Store  Driving C',
98 98
 		);
99 99
 
100
-		$this->assertEquals( $statement_descriptor4['expected'], WC_Stripe_Helper::clean_statement_descriptor( $statement_descriptor4['actual'] ) );
100
+		$this->assertEquals($statement_descriptor4['expected'], WC_Stripe_Helper::clean_statement_descriptor($statement_descriptor4['actual']));
101 101
 	}
102 102
 }
Please login to merge, or discard this patch.
includes/deprecated/class-wc-stripe-apple-pay.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -1,12 +1,12 @@  discard block
 block discarded – undo
1 1
 <?php
2
-if ( ! defined( 'ABSPATH' ) ) {
2
+if ( ! defined('ABSPATH')) {
3 3
 	exit;
4 4
 }
5 5
 
6 6
 /**
7 7
  * DEPRECATED DO NOT USE!!
8 8
  */
9
-if ( ! class_exists( 'WC_Stripe_Apple_Pay' ) ) {
9
+if ( ! class_exists('WC_Stripe_Apple_Pay')) {
10 10
 	class WC_Stripe_Apple_Pay {
11 11
 		/**
12 12
 		 * This Instance.
@@ -20,15 +20,15 @@  discard block
 block discarded – undo
20 20
 		}
21 21
 
22 22
 		public static function instance() {
23
-			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' );
23
+			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');
24 24
 			return self::$_this;
25 25
 		}
26 26
 
27
-		public function __get( $var ) {
27
+		public function __get($var) {
28 28
 			return null;
29 29
 		}
30 30
 
31
-		public function __call( $name, $arguments ) {
31
+		public function __call($name, $arguments) {
32 32
 			return null;
33 33
 		}
34 34
 	}
Please login to merge, or discard this patch.
includes/class-wc-stripe-exception.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -7,7 +7,7 @@  discard block
 block discarded – undo
7 7
  * @since 4.0.2
8 8
  */
9 9
 
10
-if ( ! defined( 'ABSPATH' ) ) {
10
+if ( ! defined('ABSPATH')) {
11 11
 	exit;
12 12
 }
13 13
 
@@ -23,9 +23,9 @@  discard block
 block discarded – undo
23 23
 	 * @param string $error_message Full response
24 24
 	 * @param string $localized_message user-friendly translated error message
25 25
 	 */
26
-	public function __construct( $error_message = '', $localized_message = '' ) {
26
+	public function __construct($error_message = '', $localized_message = '') {
27 27
 		$this->localized_message = $localized_message;
28
-		parent::__construct( $error_message );
28
+		parent::__construct($error_message);
29 29
 	}
30 30
 
31 31
 	/**
Please login to merge, or discard this patch.
uninstall.php 1 patch
Spacing   +28 added lines, -28 removed lines patch added patch discarded remove patch
@@ -1,10 +1,10 @@  discard block
 block discarded – undo
1 1
 <?php
2
-if ( ! defined( 'ABSPATH' ) ) {
2
+if ( ! defined('ABSPATH')) {
3 3
 	exit; // Exit if accessed directly
4 4
 }
5 5
 
6 6
 // if uninstall not called from WordPress exit
7
-if ( ! defined( 'WP_UNINSTALL_PLUGIN' ) ) {
7
+if ( ! defined('WP_UNINSTALL_PLUGIN')) {
8 8
 	exit;
9 9
 }
10 10
 
@@ -13,31 +13,31 @@  discard block
 block discarded – undo
13 13
  * wp-config.php. This is to prevent data loss when deleting the plugin from the backend
14 14
  * and to ensure only the site owner can perform this action.
15 15
  */
16
-if ( defined( 'WC_REMOVE_ALL_DATA' ) && true === WC_REMOVE_ALL_DATA ) {
16
+if (defined('WC_REMOVE_ALL_DATA') && true === WC_REMOVE_ALL_DATA) {
17 17
 	// Delete options.
18
-	delete_option( 'woocommerce_stripe_settings' );
19
-	delete_option( 'wc_stripe_show_styles_notice' );
20
-	delete_option( 'wc_stripe_show_request_api_notice' );
21
-	delete_option( 'wc_stripe_show_apple_pay_notice' );
22
-	delete_option( 'wc_stripe_show_ssl_notice' );
23
-	delete_option( 'wc_stripe_show_keys_notice' );
24
-	delete_option( 'wc_stripe_show_alipay_notice' );
25
-	delete_option( 'wc_stripe_show_bancontact_notice' );
26
-	delete_option( 'wc_stripe_show_bitcoin_notice' );
27
-	delete_option( 'wc_stripe_show_eps_notice' );
28
-	delete_option( 'wc_stripe_show_giropay_notice' );
29
-	delete_option( 'wc_stripe_show_ideal_notice' );
30
-	delete_option( 'wc_stripe_show_multibanco_notice' );
31
-	delete_option( 'wc_stripe_show_p24_notice' );
32
-	delete_option( 'wc_stripe_show_sepa_notice' );
33
-	delete_option( 'wc_stripe_show_sofort_notice' );
34
-	delete_option( 'wc_stripe_version' );
35
-	delete_option( 'woocommerce_stripe_bancontact_settings' );
36
-	delete_option( 'woocommerce_stripe_alipay_settings' );
37
-	delete_option( 'woocommerce_stripe_bitcoin_settings' );
38
-	delete_option( 'woocommerce_stripe_ideal_settings' );
39
-	delete_option( 'woocommerce_stripe_p24_settings' );
40
-	delete_option( 'woocommerce_stripe_giropay_settings' );
41
-	delete_option( 'woocommerce_stripe_sepa_settings' );
42
-	delete_option( 'woocommerce_stripe_sofort_settings' );
18
+	delete_option('woocommerce_stripe_settings');
19
+	delete_option('wc_stripe_show_styles_notice');
20
+	delete_option('wc_stripe_show_request_api_notice');
21
+	delete_option('wc_stripe_show_apple_pay_notice');
22
+	delete_option('wc_stripe_show_ssl_notice');
23
+	delete_option('wc_stripe_show_keys_notice');
24
+	delete_option('wc_stripe_show_alipay_notice');
25
+	delete_option('wc_stripe_show_bancontact_notice');
26
+	delete_option('wc_stripe_show_bitcoin_notice');
27
+	delete_option('wc_stripe_show_eps_notice');
28
+	delete_option('wc_stripe_show_giropay_notice');
29
+	delete_option('wc_stripe_show_ideal_notice');
30
+	delete_option('wc_stripe_show_multibanco_notice');
31
+	delete_option('wc_stripe_show_p24_notice');
32
+	delete_option('wc_stripe_show_sepa_notice');
33
+	delete_option('wc_stripe_show_sofort_notice');
34
+	delete_option('wc_stripe_version');
35
+	delete_option('woocommerce_stripe_bancontact_settings');
36
+	delete_option('woocommerce_stripe_alipay_settings');
37
+	delete_option('woocommerce_stripe_bitcoin_settings');
38
+	delete_option('woocommerce_stripe_ideal_settings');
39
+	delete_option('woocommerce_stripe_p24_settings');
40
+	delete_option('woocommerce_stripe_giropay_settings');
41
+	delete_option('woocommerce_stripe_sepa_settings');
42
+	delete_option('woocommerce_stripe_sofort_settings');
43 43
 }
Please login to merge, or discard this patch.
templates/emails/plain/failed-renewal-authentication.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -1,17 +1,17 @@
 block discarded – undo
1 1
 <?php
2
-if ( ! defined( 'ABSPATH' ) ) {
2
+if ( ! defined('ABSPATH')) {
3 3
 	exit; // Exit if accessed directly
4 4
 }
5 5
 
6 6
 echo $email_heading . "\n\n";
7 7
 
8 8
 // translators: %1$s: name of the blog, %2$s: link to checkout payment url, note: no full stop due to url at the end
9
-printf( esc_html_x( 'The automatic payment to renew your subscription with %1$s has failed. To reactivate the subscription, please login and authorize the renewal from your account page: %2$s', 'In failed renewal authentication email', 'woocommerce-gateway-stripe' ), esc_html( get_bloginfo( 'name' ) ), esc_attr( $authorization_url ) );
9
+printf(esc_html_x('The automatic payment to renew your subscription with %1$s has failed. To reactivate the subscription, please login and authorize the renewal from your account page: %2$s', 'In failed renewal authentication email', 'woocommerce-gateway-stripe'), esc_html(get_bloginfo('name')), esc_attr($authorization_url));
10 10
 
11 11
 echo "\n\n=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=\n";
12 12
 
13
-do_action( 'woocommerce_subscriptions_email_order_details', $order, $sent_to_admin, $plain_text, $email );
13
+do_action('woocommerce_subscriptions_email_order_details', $order, $sent_to_admin, $plain_text, $email);
14 14
 
15 15
 echo "\n=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=\n\n";
16 16
 
17
-echo apply_filters( 'woocommerce_email_footer_text', get_option( 'woocommerce_email_footer_text' ) );
17
+echo apply_filters('woocommerce_email_footer_text', get_option('woocommerce_email_footer_text'));
Please login to merge, or discard this patch.
templates/emails/failed-renewal-authentication.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -1,17 +1,17 @@
 block discarded – undo
1 1
 <?php
2
-if ( ! defined( 'ABSPATH' ) ) {
2
+if ( ! defined('ABSPATH')) {
3 3
 	exit; // Exit if accessed directly
4 4
 }
5 5
 ?>
6 6
 
7
-<?php do_action( 'woocommerce_email_header', $email_heading, $email ); ?>
7
+<?php do_action('woocommerce_email_header', $email_heading, $email); ?>
8 8
 
9 9
 <p>
10 10
 	<?php
11 11
 	// translators: %1$s: name of the blog, %2$s: link to payment re-authentication URL, note: no full stop due to url at the end
12
-	echo wp_kses( sprintf( _x( 'The automatic payment to renew your subscription with %1$s has failed. To reactivate the subscription, please login and authorize the renewal from your account page: %2$s', 'In failed renewal authentication email', 'woocommerce-gateway-stripe' ), esc_html( get_bloginfo( 'name' ) ), '<a href="' . esc_url( $authorization_url ) . '">' . esc_html__( 'Authorize the payment &raquo;', 'woocommerce-gateway-stripe' ) . '</a>' ), array( 'a' => array( 'href' => true ) ) ); ?>
12
+	echo wp_kses(sprintf(_x('The automatic payment to renew your subscription with %1$s has failed. To reactivate the subscription, please login and authorize the renewal from your account page: %2$s', 'In failed renewal authentication email', 'woocommerce-gateway-stripe'), esc_html(get_bloginfo('name')), '<a href="' . esc_url($authorization_url) . '">' . esc_html__('Authorize the payment &raquo;', 'woocommerce-gateway-stripe') . '</a>'), array('a' => array('href' => true))); ?>
13 13
 </p>
14 14
 
15
-<?php do_action( 'woocommerce_subscriptions_email_order_details', $order, $sent_to_admin, $plain_text, $email ); ?>
15
+<?php do_action('woocommerce_subscriptions_email_order_details', $order, $sent_to_admin, $plain_text, $email); ?>
16 16
 
17
-<?php do_action( 'woocommerce_email_footer', $email ); ?>
17
+<?php do_action('woocommerce_email_footer', $email); ?>
Please login to merge, or discard this patch.
includes/compat/class-wc-stripe-email-failed-authentication.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -1,5 +1,5 @@  discard block
 block discarded – undo
1 1
 <?php
2
-if ( ! defined( 'ABSPATH' ) ) {
2
+if ( ! defined('ABSPATH')) {
3 3
 	exit; // Exit if accessed directly.
4 4
 }
5 5
 
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 				'email_heading'     => $this->get_heading(),
31 31
 				'sent_to_admin'     => false,
32 32
 				'plain_text'        => false,
33
-				'authorization_url' => $this->get_authorization_url( $this->object ),
33
+				'authorization_url' => $this->get_authorization_url($this->object),
34 34
 				'email'             => $this,
35 35
 			),
36 36
 			'',
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
 				'email_heading'     => $this->get_heading(),
54 54
 				'sent_to_admin'     => false,
55 55
 				'plain_text'        => true,
56
-				'authorization_url' => $this->get_authorization_url( $this->object ),
56
+				'authorization_url' => $this->get_authorization_url($this->object),
57 57
 				'email'             => $this,
58 58
 			),
59 59
 			'',
@@ -68,8 +68,8 @@  discard block
 block discarded – undo
68 68
 	 * @param WC_Order $order The order whose payment needs authentication.
69 69
 	 * @return string
70 70
 	 */
71
-	public function get_authorization_url( $order ) {
72
-		return add_query_arg( 'wc-stripe-confirmation', 1, $order->get_checkout_payment_url( false ) );
71
+	public function get_authorization_url($order) {
72
+		return add_query_arg('wc-stripe-confirmation', 1, $order->get_checkout_payment_url(false));
73 73
 	}
74 74
 
75 75
 	/**
@@ -81,9 +81,9 @@  discard block
 block discarded – undo
81 81
 
82 82
 		$this->form_fields = array(
83 83
 			'enabled'    => array(
84
-				'title'   => _x( 'Enable/Disable', 'an email notification', 'woocommerce-gateway-stripe' ),
84
+				'title'   => _x('Enable/Disable', 'an email notification', 'woocommerce-gateway-stripe'),
85 85
 				'type'    => 'checkbox',
86
-				'label'   => __( 'Enable this email notification', 'woocommerce-gateway-stripe' ),
86
+				'label'   => __('Enable this email notification', 'woocommerce-gateway-stripe'),
87 87
 				'default' => 'yes',
88 88
 			),
89 89
 
@@ -98,29 +98,29 @@  discard block
 block discarded – undo
98 98
 	 *
99 99
 	 * @param WC_Order $order The renewal order whose payment failed.
100 100
 	 */
101
-	public function trigger( $order ) {
102
-		if ( ! $this->is_enabled() ) {
101
+	public function trigger($order) {
102
+		if ( ! $this->is_enabled()) {
103 103
 			return;
104 104
 		}
105 105
 
106 106
 		$this->object = $order;
107 107
 
108
-		if ( method_exists( $order, 'get_billing_email' ) ) {
108
+		if (method_exists($order, 'get_billing_email')) {
109 109
 			$this->recipient = $order->get_billing_email();
110 110
 		} else {
111 111
 			$this->recipient = $order->billing_email;
112 112
 		}
113 113
 
114 114
 		$this->find['order_date'] = '{order_date}';
115
-		if ( function_exists( 'wc_format_datetime' ) ) { // WC 3.0+
116
-			$this->replace['order_date'] = wc_format_datetime( $order->get_date_created() );
115
+		if (function_exists('wc_format_datetime')) { // WC 3.0+
116
+			$this->replace['order_date'] = wc_format_datetime($order->get_date_created());
117 117
 		} else { // WC < 3.0
118
-			$this->replace['order_date'] = $order->date_created->date_i18n( wc_date_format() );
118
+			$this->replace['order_date'] = $order->date_created->date_i18n(wc_date_format());
119 119
 		}
120 120
 
121 121
 		$this->find['order_number']    = '{order_number}';
122 122
 		$this->replace['order_number'] = $order->get_order_number();
123 123
 
124
-		$this->send( $this->get_recipient(), $this->get_subject(), $this->get_content(), $this->get_headers(), $this->get_attachments() );
124
+		$this->send($this->get_recipient(), $this->get_subject(), $this->get_content(), $this->get_headers(), $this->get_attachments());
125 125
 	}
126 126
 }
Please login to merge, or discard this patch.
includes/compat/class-wc-stripe-email-failed-preorder-authentication.php 1 patch
Spacing   +18 added lines, -18 removed lines patch added patch discarded remove patch
@@ -1,5 +1,5 @@  discard block
 block discarded – undo
1 1
 <?php
2
-if ( ! defined( 'ABSPATH' ) ) {
2
+if ( ! defined('ABSPATH')) {
3 3
 	exit; // Exit if accessed directly.
4 4
 }
5 5
 
@@ -21,20 +21,20 @@  discard block
 block discarded – undo
21 21
 	 *
22 22
 	 * @param WC_Email[] $email_classes All existing instances of WooCommerce emails.
23 23
 	 */
24
-	public function __construct( $email_classes = array() ) {
24
+	public function __construct($email_classes = array()) {
25 25
 		$this->id             = 'failed_preorder_sca_authentication';
26
-		$this->title          = __( 'Pre-order Payment Action Needed', 'woocommerce-gateway-stripe' );
27
-		$this->description    = __( 'This is an order notification sent to the customer once a pre-order is complete, but additional payment steps are required.', 'woocommerce-gateway-stripe' );
26
+		$this->title          = __('Pre-order Payment Action Needed', 'woocommerce-gateway-stripe');
27
+		$this->description    = __('This is an order notification sent to the customer once a pre-order is complete, but additional payment steps are required.', 'woocommerce-gateway-stripe');
28 28
 		$this->customer_email = true;
29 29
 
30 30
 		$this->template_html  = 'emails/failed-preorder-authentication.php';
31 31
 		$this->template_plain = 'emails/plain/failed-preorder-authentication.php';
32
-		$this->template_base  = plugin_dir_path( WC_STRIPE_MAIN_FILE ) . 'templates/';
32
+		$this->template_base  = plugin_dir_path(WC_STRIPE_MAIN_FILE) . 'templates/';
33 33
 
34 34
 		// Use the "authentication required" hook to add the correct, later hook.
35
-		add_action( 'wc_gateway_stripe_process_payment_authentication_required', array( $this, 'trigger' ) );
35
+		add_action('wc_gateway_stripe_process_payment_authentication_required', array($this, 'trigger'));
36 36
 
37
-		if ( isset( $email_classes['WC_Pre_Orders_Email_Pre_Order_Available'] ) ) {
37
+		if (isset($email_classes['WC_Pre_Orders_Email_Pre_Order_Available'])) {
38 38
 			$this->original_email = $email_classes['WC_Pre_Orders_Email_Pre_Order_Available'];
39 39
 		}
40 40
 
@@ -48,13 +48,13 @@  discard block
 block discarded – undo
48 48
 	 *
49 49
 	 * @param WC_Order $order The order whose payment is failing.
50 50
 	 */
51
-	public function trigger( $order ) {
52
-		if ( class_exists( 'WC_Pre_Orders_Order' ) && WC_Pre_Orders_Order::order_contains_pre_order( $order->get_id() ) ) {
53
-			if ( isset( $this->original_email ) ) {
54
-				remove_action( 'wc_pre_order_status_completed_notification', array( $this->original_email, 'trigger' ), 10, 2 );
51
+	public function trigger($order) {
52
+		if (class_exists('WC_Pre_Orders_Order') && WC_Pre_Orders_Order::order_contains_pre_order($order->get_id())) {
53
+			if (isset($this->original_email)) {
54
+				remove_action('wc_pre_order_status_completed_notification', array($this->original_email, 'trigger'), 10, 2);
55 55
 			}
56 56
 
57
-			add_action( 'wc_pre_orders_pre_order_completed', array( $this, 'send_email' ), 10, 2 );
57
+			add_action('wc_pre_orders_pre_order_completed', array($this, 'send_email'), 10, 2);
58 58
 		}
59 59
 	}
60 60
 
@@ -64,14 +64,14 @@  discard block
 block discarded – undo
64 64
 	 * @param WC_Order $order The order that is being paid.
65 65
 	 * @param string   $message The message, which should be added to the email.
66 66
 	 */
67
-	public function send_email( $order, $message ) {
67
+	public function send_email($order, $message) {
68 68
 		$this->custom_message = $message;
69 69
 
70
-		parent::trigger( $order );
70
+		parent::trigger($order);
71 71
 
72 72
 		// Restore the action of the original email for other bulk actions.
73
-		if ( isset( $this->original_email ) ) {
74
-			add_action( 'wc_pre_order_status_completed_notification', array( $this->original_email, 'trigger' ), 10, 2 );
73
+		if (isset($this->original_email)) {
74
+			add_action('wc_pre_order_status_completed_notification', array($this->original_email, 'trigger'), 10, 2);
75 75
 		}
76 76
 	}
77 77
 
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
 	 * @return string
82 82
 	 */
83 83
 	public function get_default_subject() {
84
-		return __( 'Payment authorization needed for pre-order {order_number}', 'woocommerce-gateway-stripe' );
84
+		return __('Payment authorization needed for pre-order {order_number}', 'woocommerce-gateway-stripe');
85 85
 	}
86 86
 
87 87
 	/**
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
 	 * @return string
91 91
 	 */
92 92
 	public function get_default_heading() {
93
-		return __( 'Payment authorization needed for pre-order {order_number}', 'woocommerce-gateway-stripe' );
93
+		return __('Payment authorization needed for pre-order {order_number}', 'woocommerce-gateway-stripe');
94 94
 	}
95 95
 
96 96
 	/**
Please login to merge, or discard this patch.