Completed
Pull Request — master (#1042)
by Radoslav
02:32
created
includes/class-wc-stripe-intent-controller.php 1 patch
Spacing   +29 added lines, -29 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;
4 4
 }
5 5
 
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
 	 * @since 4.2.0
24 24
 	 */
25 25
 	public function __construct() {
26
-		add_action( 'wc_ajax_wc_stripe_verify_intent', array( $this, 'verify_intent' ) );
26
+		add_action('wc_ajax_wc_stripe_verify_intent', array($this, 'verify_intent'));
27 27
 	}
28 28
 
29 29
 	/**
@@ -33,8 +33,8 @@  discard block
 block discarded – undo
33 33
 	 * @return WC_Gateway_Stripe
34 34
 	 */
35 35
 	protected function get_gateway() {
36
-		if ( ! isset( $this->gateway ) ) {
37
-			if ( class_exists( 'WC_Subscriptions_Order' ) && function_exists( 'wcs_create_renewal_order' ) ) {
36
+		if ( ! isset($this->gateway)) {
37
+			if (class_exists('WC_Subscriptions_Order') && function_exists('wcs_create_renewal_order')) {
38 38
 				$class_name = 'WC_Stripe_Subs_Compat';
39 39
 			} else {
40 40
 				$class_name = 'WC_Gateway_Stripe';
@@ -54,21 +54,21 @@  discard block
 block discarded – undo
54 54
 	 * @return WC_Order
55 55
 	 */
56 56
 	protected function get_order_from_request() {
57
-		if ( ! isset( $_GET['nonce'] ) || ! wp_verify_nonce( sanitize_key( $_GET['nonce'] ), 'wc_stripe_confirm_pi' ) ) {
58
-			throw new WC_Stripe_Exception( 'missing-nonce', __( 'CSRF verification failed.', 'woocommerce-gateway-stripe' ) );
57
+		if ( ! isset($_GET['nonce']) || ! wp_verify_nonce(sanitize_key($_GET['nonce']), 'wc_stripe_confirm_pi')) {
58
+			throw new WC_Stripe_Exception('missing-nonce', __('CSRF verification failed.', 'woocommerce-gateway-stripe'));
59 59
 		}
60 60
 
61 61
 		// Load the order ID.
62 62
 		$order_id = null;
63
-		if ( isset( $_GET['order'] ) && absint( $_GET['order'] ) ) {
64
-			$order_id = absint( $_GET['order'] );
63
+		if (isset($_GET['order']) && absint($_GET['order'])) {
64
+			$order_id = absint($_GET['order']);
65 65
 		}
66 66
 
67 67
 		// Retrieve the order.
68
-		$order = wc_get_order( $order_id );
68
+		$order = wc_get_order($order_id);
69 69
 
70
-		if ( ! $order ) {
71
-			throw new WC_Stripe_Exception( 'missing-order', __( 'Missing order ID for payment confirmation', 'woocommerce-gateway-stripe' ) );
70
+		if ( ! $order) {
71
+			throw new WC_Stripe_Exception('missing-order', __('Missing order ID for payment confirmation', 'woocommerce-gateway-stripe'));
72 72
 		}
73 73
 
74 74
 		return $order;
@@ -86,32 +86,32 @@  discard block
 block discarded – undo
86 86
 
87 87
 		try {
88 88
 			$order = $this->get_order_from_request();
89
-		} catch ( WC_Stripe_Exception $e ) {
89
+		} catch (WC_Stripe_Exception $e) {
90 90
 			/* translators: Error message text */
91
-			$message = sprintf( __( 'Payment verification error: %s', 'woocommerce-gateway-stripe' ), $e->getLocalizedMessage() );
92
-			wc_add_notice( esc_html( $message ), 'error' );
91
+			$message = sprintf(__('Payment verification error: %s', 'woocommerce-gateway-stripe'), $e->getLocalizedMessage());
92
+			wc_add_notice(esc_html($message), 'error');
93 93
 
94 94
 			$redirect_url = $woocommerce->cart->is_empty()
95
-				? get_permalink( WC_Stripe_Helper::is_wc_lt( '3.0' ) ? woocommerce_get_page_id( 'shop' ) : wc_get_page_id( 'shop' ) )
95
+				? get_permalink(WC_Stripe_Helper::is_wc_lt('3.0') ? woocommerce_get_page_id('shop') : wc_get_page_id('shop'))
96 96
 				: wc_get_checkout_url();
97 97
 
98
-			$this->handle_error( $e, $redirect_url );
98
+			$this->handle_error($e, $redirect_url);
99 99
 		}
100 100
 
101 101
 		try {
102
-			$gateway->verify_intent_after_checkout( $order );
102
+			$gateway->verify_intent_after_checkout($order);
103 103
 
104
-			if ( ! isset( $_GET['is_ajax'] ) ) {
105
-				$redirect_url = isset( $_GET['redirect_to'] ) // wpcs: csrf ok.
106
-					? esc_url_raw( wp_unslash( $_GET['redirect_to'] ) ) // wpcs: csrf ok.
107
-					: $gateway->get_return_url( $order );
104
+			if ( ! isset($_GET['is_ajax'])) {
105
+				$redirect_url = isset($_GET['redirect_to']) // wpcs: csrf ok.
106
+					? esc_url_raw(wp_unslash($_GET['redirect_to'])) // wpcs: csrf ok.
107
+					: $gateway->get_return_url($order);
108 108
 
109
-				wp_safe_redirect( $redirect_url );
109
+				wp_safe_redirect($redirect_url);
110 110
 			}
111 111
 
112 112
 			exit;
113
-		} catch ( WC_Stripe_Exception $e ) {
114
-			$this->handle_error( $e, $gateway->get_return_url( $order ) );
113
+		} catch (WC_Stripe_Exception $e) {
114
+			$this->handle_error($e, $gateway->get_return_url($order));
115 115
 		}
116 116
 	}
117 117
 
@@ -122,17 +122,17 @@  discard block
 block discarded – undo
122 122
 	 * @param WC_Stripe_Exception $e           The exception that was thrown.
123 123
 	 * @param string              $redirect_url An URL to use if a redirect is needed.
124 124
 	 */
125
-	protected function handle_error( $e, $redirect_url ) {
125
+	protected function handle_error($e, $redirect_url) {
126 126
 		// Log the exception before redirecting.
127
-		$message = sprintf( 'PaymentIntent verification exception: %s', $e->getLocalizedMessage() );
128
-		WC_Stripe_Logger::log( $message );
127
+		$message = sprintf('PaymentIntent verification exception: %s', $e->getLocalizedMessage());
128
+		WC_Stripe_Logger::log($message);
129 129
 
130 130
 		// `is_ajax` is only used for PI error reporting, a response is not expected.
131
-		if ( isset( $_GET['is_ajax'] ) ) {
131
+		if (isset($_GET['is_ajax'])) {
132 132
 			exit;
133 133
 		}
134 134
 
135
-		wp_safe_redirect( $redirect_url );
135
+		wp_safe_redirect($redirect_url);
136 136
 		exit;
137 137
 	}
138 138
 }
Please login to merge, or discard this patch.
includes/class-wc-stripe-helper.php 1 patch
Spacing   +105 added lines, -105 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;
4 4
 }
5 5
 
@@ -22,14 +22,14 @@  discard block
 block discarded – undo
22 22
 	 * @param object $order
23 23
 	 * @return string $currency
24 24
 	 */
25
-	public static function get_stripe_currency( $order = null ) {
26
-		if ( is_null( $order ) ) {
25
+	public static function get_stripe_currency($order = null) {
26
+		if (is_null($order)) {
27 27
 			return false;
28 28
 		}
29 29
 
30
-		$order_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
30
+		$order_id = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
31 31
 
32
-		return WC_Stripe_Helper::is_wc_lt( '3.0' ) ? get_post_meta( $order_id, self::META_NAME_STRIPE_CURRENCY, true ) : $order->get_meta( self::META_NAME_STRIPE_CURRENCY, true );
32
+		return WC_Stripe_Helper::is_wc_lt('3.0') ? get_post_meta($order_id, self::META_NAME_STRIPE_CURRENCY, true) : $order->get_meta(self::META_NAME_STRIPE_CURRENCY, true);
33 33
 	}
34 34
 
35 35
 	/**
@@ -39,14 +39,14 @@  discard block
 block discarded – undo
39 39
 	 * @param object $order
40 40
 	 * @param string $currency
41 41
 	 */
42
-	public static function update_stripe_currency( $order = null, $currency ) {
43
-		if ( is_null( $order ) ) {
42
+	public static function update_stripe_currency($order = null, $currency) {
43
+		if (is_null($order)) {
44 44
 			return false;
45 45
 		}
46 46
 
47
-		$order_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
47
+		$order_id = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
48 48
 
49
-		WC_Stripe_Helper::is_wc_lt( '3.0' ) ? update_post_meta( $order_id, self::META_NAME_STRIPE_CURRENCY, $currency ) : $order->update_meta_data( self::META_NAME_STRIPE_CURRENCY, $currency );
49
+		WC_Stripe_Helper::is_wc_lt('3.0') ? update_post_meta($order_id, self::META_NAME_STRIPE_CURRENCY, $currency) : $order->update_meta_data(self::META_NAME_STRIPE_CURRENCY, $currency);
50 50
 	}
51 51
 
52 52
 	/**
@@ -56,22 +56,22 @@  discard block
 block discarded – undo
56 56
 	 * @param object $order
57 57
 	 * @return string $amount
58 58
 	 */
59
-	public static function get_stripe_fee( $order = null ) {
60
-		if ( is_null( $order ) ) {
59
+	public static function get_stripe_fee($order = null) {
60
+		if (is_null($order)) {
61 61
 			return false;
62 62
 		}
63 63
 
64
-		$order_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
64
+		$order_id = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
65 65
 
66
-		$amount = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? get_post_meta( $order_id, self::META_NAME_FEE, true ) : $order->get_meta( self::META_NAME_FEE, true );
66
+		$amount = WC_Stripe_Helper::is_wc_lt('3.0') ? get_post_meta($order_id, self::META_NAME_FEE, true) : $order->get_meta(self::META_NAME_FEE, true);
67 67
 
68 68
 		// If not found let's check for legacy name.
69
-		if ( empty( $amount ) ) {
70
-			$amount = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? get_post_meta( $order_id, self::LEGACY_META_NAME_FEE, true ) : $order->get_meta( self::LEGACY_META_NAME_FEE, true );
69
+		if (empty($amount)) {
70
+			$amount = WC_Stripe_Helper::is_wc_lt('3.0') ? get_post_meta($order_id, self::LEGACY_META_NAME_FEE, true) : $order->get_meta(self::LEGACY_META_NAME_FEE, true);
71 71
 
72 72
 			// If found update to new name.
73
-			if ( $amount ) {
74
-				self::update_stripe_fee( $order, $amount );
73
+			if ($amount) {
74
+				self::update_stripe_fee($order, $amount);
75 75
 			}
76 76
 		}
77 77
 
@@ -85,14 +85,14 @@  discard block
 block discarded – undo
85 85
 	 * @param object $order
86 86
 	 * @param float $amount
87 87
 	 */
88
-	public static function update_stripe_fee( $order = null, $amount = 0.0 ) {
89
-		if ( is_null( $order ) ) {
88
+	public static function update_stripe_fee($order = null, $amount = 0.0) {
89
+		if (is_null($order)) {
90 90
 			return false;
91 91
 		}
92 92
 
93
-		$order_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
93
+		$order_id = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
94 94
 
95
-		WC_Stripe_Helper::is_wc_lt( '3.0' ) ? update_post_meta( $order_id, self::META_NAME_FEE, $amount ) : $order->update_meta_data( self::META_NAME_FEE, $amount );
95
+		WC_Stripe_Helper::is_wc_lt('3.0') ? update_post_meta($order_id, self::META_NAME_FEE, $amount) : $order->update_meta_data(self::META_NAME_FEE, $amount);
96 96
 	}
97 97
 
98 98
 	/**
@@ -101,15 +101,15 @@  discard block
 block discarded – undo
101 101
 	 * @since 4.1.0
102 102
 	 * @param object $order
103 103
 	 */
104
-	public static function delete_stripe_fee( $order = null ) {
105
-		if ( is_null( $order ) ) {
104
+	public static function delete_stripe_fee($order = null) {
105
+		if (is_null($order)) {
106 106
 			return false;
107 107
 		}
108 108
 
109
-		$order_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
109
+		$order_id = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
110 110
 
111
-		delete_post_meta( $order_id, self::META_NAME_FEE );
112
-		delete_post_meta( $order_id, self::LEGACY_META_NAME_FEE );
111
+		delete_post_meta($order_id, self::META_NAME_FEE);
112
+		delete_post_meta($order_id, self::LEGACY_META_NAME_FEE);
113 113
 	}
114 114
 
115 115
 	/**
@@ -119,22 +119,22 @@  discard block
 block discarded – undo
119 119
 	 * @param object $order
120 120
 	 * @return string $amount
121 121
 	 */
122
-	public static function get_stripe_net( $order = null ) {
123
-		if ( is_null( $order ) ) {
122
+	public static function get_stripe_net($order = null) {
123
+		if (is_null($order)) {
124 124
 			return false;
125 125
 		}
126 126
 
127
-		$order_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
127
+		$order_id = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
128 128
 
129
-		$amount = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? get_post_meta( $order_id, self::META_NAME_NET, true ) : $order->get_meta( self::META_NAME_NET, true );
129
+		$amount = WC_Stripe_Helper::is_wc_lt('3.0') ? get_post_meta($order_id, self::META_NAME_NET, true) : $order->get_meta(self::META_NAME_NET, true);
130 130
 
131 131
 		// If not found let's check for legacy name.
132
-		if ( empty( $amount ) ) {
133
-			$amount = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? get_post_meta( $order_id, self::LEGACY_META_NAME_NET, true ) : $order->get_meta( self::LEGACY_META_NAME_NET, true );
132
+		if (empty($amount)) {
133
+			$amount = WC_Stripe_Helper::is_wc_lt('3.0') ? get_post_meta($order_id, self::LEGACY_META_NAME_NET, true) : $order->get_meta(self::LEGACY_META_NAME_NET, true);
134 134
 
135 135
 			// If found update to new name.
136
-			if ( $amount ) {
137
-				self::update_stripe_net( $order, $amount );
136
+			if ($amount) {
137
+				self::update_stripe_net($order, $amount);
138 138
 			}
139 139
 		}
140 140
 
@@ -148,14 +148,14 @@  discard block
 block discarded – undo
148 148
 	 * @param object $order
149 149
 	 * @param float $amount
150 150
 	 */
151
-	public static function update_stripe_net( $order = null, $amount = 0.0 ) {
152
-		if ( is_null( $order ) ) {
151
+	public static function update_stripe_net($order = null, $amount = 0.0) {
152
+		if (is_null($order)) {
153 153
 			return false;
154 154
 		}
155 155
 
156
-		$order_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
156
+		$order_id = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
157 157
 
158
-		WC_Stripe_Helper::is_wc_lt( '3.0' ) ? update_post_meta( $order_id, self::META_NAME_NET, $amount ) : $order->update_meta_data( self::META_NAME_NET, $amount );
158
+		WC_Stripe_Helper::is_wc_lt('3.0') ? update_post_meta($order_id, self::META_NAME_NET, $amount) : $order->update_meta_data(self::META_NAME_NET, $amount);
159 159
 	}
160 160
 
161 161
 	/**
@@ -164,15 +164,15 @@  discard block
 block discarded – undo
164 164
 	 * @since 4.1.0
165 165
 	 * @param object $order
166 166
 	 */
167
-	public static function delete_stripe_net( $order = null ) {
168
-		if ( is_null( $order ) ) {
167
+	public static function delete_stripe_net($order = null) {
168
+		if (is_null($order)) {
169 169
 			return false;
170 170
 		}
171 171
 
172
-		$order_id = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $order->id : $order->get_id();
172
+		$order_id = WC_Stripe_Helper::is_wc_lt('3.0') ? $order->id : $order->get_id();
173 173
 
174
-		delete_post_meta( $order_id, self::META_NAME_NET );
175
-		delete_post_meta( $order_id, self::LEGACY_META_NAME_NET );
174
+		delete_post_meta($order_id, self::META_NAME_NET);
175
+		delete_post_meta($order_id, self::LEGACY_META_NAME_NET);
176 176
 	}
177 177
 
178 178
 	/**
@@ -183,15 +183,15 @@  discard block
 block discarded – undo
183 183
 	 *
184 184
 	 * @return float|int
185 185
 	 */
186
-	public static function get_stripe_amount( $total, $currency = '' ) {
187
-		if ( ! $currency ) {
186
+	public static function get_stripe_amount($total, $currency = '') {
187
+		if ( ! $currency) {
188 188
 			$currency = get_woocommerce_currency();
189 189
 		}
190 190
 
191
-		if ( in_array( strtolower( $currency ), self::no_decimal_currencies() ) ) {
192
-			return absint( $total );
191
+		if (in_array(strtolower($currency), self::no_decimal_currencies())) {
192
+			return absint($total);
193 193
 		} else {
194
-			return absint( wc_format_decimal( ( (float) $total * 100 ), wc_get_price_decimals() ) ); // In cents.
194
+			return absint(wc_format_decimal(((float) $total * 100), wc_get_price_decimals())); // In cents.
195 195
 		}
196 196
 	}
197 197
 
@@ -206,24 +206,24 @@  discard block
 block discarded – undo
206 206
 		return apply_filters(
207 207
 			'wc_stripe_localized_messages',
208 208
 			array(
209
-				'invalid_number'           => __( 'The card number is not a valid credit card number.', 'woocommerce-gateway-stripe' ),
210
-				'invalid_expiry_month'     => __( 'The card\'s expiration month is invalid.', 'woocommerce-gateway-stripe' ),
211
-				'invalid_expiry_year'      => __( 'The card\'s expiration year is invalid.', 'woocommerce-gateway-stripe' ),
212
-				'invalid_cvc'              => __( 'The card\'s security code is invalid.', 'woocommerce-gateway-stripe' ),
213
-				'incorrect_number'         => __( 'The card number is incorrect.', 'woocommerce-gateway-stripe' ),
214
-				'incomplete_number'        => __( 'The card number is incomplete.', 'woocommerce-gateway-stripe' ),
215
-				'incomplete_cvc'           => __( 'The card\'s security code is incomplete.', 'woocommerce-gateway-stripe' ),
216
-				'incomplete_expiry'        => __( 'The card\'s expiration date is incomplete.', 'woocommerce-gateway-stripe' ),
217
-				'expired_card'             => __( 'The card has expired.', 'woocommerce-gateway-stripe' ),
218
-				'incorrect_cvc'            => __( 'The card\'s security code is incorrect.', 'woocommerce-gateway-stripe' ),
219
-				'incorrect_zip'            => __( 'The card\'s zip code failed validation.', 'woocommerce-gateway-stripe' ),
220
-				'invalid_expiry_year_past' => __( 'The card\'s expiration year is in the past', 'woocommerce-gateway-stripe' ),
221
-				'card_declined'            => __( 'The card was declined.', 'woocommerce-gateway-stripe' ),
222
-				'missing'                  => __( 'There is no card on a customer that is being charged.', 'woocommerce-gateway-stripe' ),
223
-				'processing_error'         => __( 'An error occurred while processing the card.', 'woocommerce-gateway-stripe' ),
224
-				'invalid_request_error'    => __( 'Unable to process this payment, please try again or use alternative method.', 'woocommerce-gateway-stripe' ),
225
-				'invalid_sofort_country'   => __( 'The billing country is not accepted by SOFORT. Please try another country.', 'woocommerce-gateway-stripe' ),
226
-				'email_invalid'            => __( 'Invalid email address, please correct and try again.', 'woocommerce-gateway-stripe' ),
209
+				'invalid_number'           => __('The card number is not a valid credit card number.', 'woocommerce-gateway-stripe'),
210
+				'invalid_expiry_month'     => __('The card\'s expiration month is invalid.', 'woocommerce-gateway-stripe'),
211
+				'invalid_expiry_year'      => __('The card\'s expiration year is invalid.', 'woocommerce-gateway-stripe'),
212
+				'invalid_cvc'              => __('The card\'s security code is invalid.', 'woocommerce-gateway-stripe'),
213
+				'incorrect_number'         => __('The card number is incorrect.', 'woocommerce-gateway-stripe'),
214
+				'incomplete_number'        => __('The card number is incomplete.', 'woocommerce-gateway-stripe'),
215
+				'incomplete_cvc'           => __('The card\'s security code is incomplete.', 'woocommerce-gateway-stripe'),
216
+				'incomplete_expiry'        => __('The card\'s expiration date is incomplete.', 'woocommerce-gateway-stripe'),
217
+				'expired_card'             => __('The card has expired.', 'woocommerce-gateway-stripe'),
218
+				'incorrect_cvc'            => __('The card\'s security code is incorrect.', 'woocommerce-gateway-stripe'),
219
+				'incorrect_zip'            => __('The card\'s zip code failed validation.', 'woocommerce-gateway-stripe'),
220
+				'invalid_expiry_year_past' => __('The card\'s expiration year is in the past', 'woocommerce-gateway-stripe'),
221
+				'card_declined'            => __('The card was declined.', 'woocommerce-gateway-stripe'),
222
+				'missing'                  => __('There is no card on a customer that is being charged.', 'woocommerce-gateway-stripe'),
223
+				'processing_error'         => __('An error occurred while processing the card.', 'woocommerce-gateway-stripe'),
224
+				'invalid_request_error'    => __('Unable to process this payment, please try again or use alternative method.', 'woocommerce-gateway-stripe'),
225
+				'invalid_sofort_country'   => __('The billing country is not accepted by SOFORT. Please try another country.', 'woocommerce-gateway-stripe'),
226
+				'email_invalid'            => __('Invalid email address, please correct and try again.', 'woocommerce-gateway-stripe'),
227 227
 			)
228 228
 		);
229 229
 	}
@@ -264,24 +264,24 @@  discard block
 block discarded – undo
264 264
 	 * @param string $type Type of number to format
265 265
 	 * @return string
266 266
 	 */
267
-	public static function format_balance_fee( $balance_transaction, $type = 'fee' ) {
268
-		if ( ! is_object( $balance_transaction ) ) {
267
+	public static function format_balance_fee($balance_transaction, $type = 'fee') {
268
+		if ( ! is_object($balance_transaction)) {
269 269
 			return;
270 270
 		}
271 271
 
272
-		if ( in_array( strtolower( $balance_transaction->currency ), self::no_decimal_currencies() ) ) {
273
-			if ( 'fee' === $type ) {
272
+		if (in_array(strtolower($balance_transaction->currency), self::no_decimal_currencies())) {
273
+			if ('fee' === $type) {
274 274
 				return $balance_transaction->fee;
275 275
 			}
276 276
 
277 277
 			return $balance_transaction->net;
278 278
 		}
279 279
 
280
-		if ( 'fee' === $type ) {
281
-			return number_format( $balance_transaction->fee / 100, 2, '.', '' );
280
+		if ('fee' === $type) {
281
+			return number_format($balance_transaction->fee / 100, 2, '.', '');
282 282
 		}
283 283
 
284
-		return number_format( $balance_transaction->net / 100, 2, '.', '' );
284
+		return number_format($balance_transaction->net / 100, 2, '.', '');
285 285
 	}
286 286
 
287 287
 	/**
@@ -289,7 +289,7 @@  discard block
 block discarded – undo
289 289
 	 */
290 290
 	public static function get_minimum_amount() {
291 291
 		// Check order amount
292
-		switch ( get_woocommerce_currency() ) {
292
+		switch (get_woocommerce_currency()) {
293 293
 			case 'USD':
294 294
 			case 'CAD':
295 295
 			case 'EUR':
@@ -334,14 +334,14 @@  discard block
 block discarded – undo
334 334
 	 * @param string $method The payment method to get the settings from.
335 335
 	 * @param string $setting The name of the setting to get.
336 336
 	 */
337
-	public static function get_settings( $method = null, $setting = null ) {
338
-		$all_settings = null === $method ? get_option( 'woocommerce_stripe_settings', array() ) : get_option( 'woocommerce_stripe_' . $method . '_settings', array() );
337
+	public static function get_settings($method = null, $setting = null) {
338
+		$all_settings = null === $method ? get_option('woocommerce_stripe_settings', array()) : get_option('woocommerce_stripe_' . $method . '_settings', array());
339 339
 
340
-		if ( null === $setting ) {
340
+		if (null === $setting) {
341 341
 			return $all_settings;
342 342
 		}
343 343
 
344
-		return isset( $all_settings[ $setting ] ) ? $all_settings[ $setting ] : '';
344
+		return isset($all_settings[$setting]) ? $all_settings[$setting] : '';
345 345
 	}
346 346
 
347 347
 	/**
@@ -351,7 +351,7 @@  discard block
 block discarded – undo
351 351
 	 * @return bool
352 352
 	 */
353 353
 	public static function is_pre_orders_exists() {
354
-		return class_exists( 'WC_Pre_Orders_Order' );
354
+		return class_exists('WC_Pre_Orders_Order');
355 355
 	}
356 356
 
357 357
 	/**
@@ -363,9 +363,9 @@  discard block
 block discarded – undo
363 363
 	 * @return bool
364 364
 	 */
365 365
 	public static function is_pre_30() {
366
-		error_log( 'is_pre_30() function has been deprecated since 4.1.11. Please use is_wc_lt( $version ) instead.' );
366
+		error_log('is_pre_30() function has been deprecated since 4.1.11. Please use is_wc_lt( $version ) instead.');
367 367
 
368
-		return self::is_wc_lt( '3.0' );
368
+		return self::is_wc_lt('3.0');
369 369
 	}
370 370
 
371 371
 	/**
@@ -375,8 +375,8 @@  discard block
 block discarded – undo
375 375
 	 * @param string $version Version to check against.
376 376
 	 * @return bool
377 377
 	 */
378
-	public static function is_wc_lt( $version ) {
379
-		return version_compare( WC_VERSION, $version, '<' );
378
+	public static function is_wc_lt($version) {
379
+		return version_compare(WC_VERSION, $version, '<');
380 380
 	}
381 381
 
382 382
 	/**
@@ -389,7 +389,7 @@  discard block
 block discarded – undo
389 389
 	 * @return string
390 390
 	 */
391 391
 	public static function get_webhook_url() {
392
-		return add_query_arg( 'wc-api', 'wc_stripe', trailingslashit( get_home_url() ) );
392
+		return add_query_arg('wc-api', 'wc_stripe', trailingslashit(get_home_url()));
393 393
 	}
394 394
 
395 395
 	/**
@@ -399,13 +399,13 @@  discard block
 block discarded – undo
399 399
 	 * @version 4.0.0
400 400
 	 * @param string $source_id
401 401
 	 */
402
-	public static function get_order_by_source_id( $source_id ) {
402
+	public static function get_order_by_source_id($source_id) {
403 403
 		global $wpdb;
404 404
 
405
-		$order_id = $wpdb->get_var( $wpdb->prepare( "SELECT DISTINCT ID FROM $wpdb->posts as posts LEFT JOIN $wpdb->postmeta as meta ON posts.ID = meta.post_id WHERE meta.meta_value = %s AND meta.meta_key = %s", $source_id, '_stripe_source_id' ) );
405
+		$order_id = $wpdb->get_var($wpdb->prepare("SELECT DISTINCT ID FROM $wpdb->posts as posts LEFT JOIN $wpdb->postmeta as meta ON posts.ID = meta.post_id WHERE meta.meta_value = %s AND meta.meta_key = %s", $source_id, '_stripe_source_id'));
406 406
 
407
-		if ( ! empty( $order_id ) ) {
408
-			return wc_get_order( $order_id );
407
+		if ( ! empty($order_id)) {
408
+			return wc_get_order($order_id);
409 409
 		}
410 410
 
411 411
 		return false;
@@ -418,17 +418,17 @@  discard block
 block discarded – undo
418 418
 	 * @since 4.1.16 Return false if charge_id is empty.
419 419
 	 * @param string $charge_id
420 420
 	 */
421
-	public static function get_order_by_charge_id( $charge_id ) {
421
+	public static function get_order_by_charge_id($charge_id) {
422 422
 		global $wpdb;
423 423
 
424
-		if ( empty( $charge_id ) ) {
424
+		if (empty($charge_id)) {
425 425
 			return false;
426 426
 		}
427 427
 
428
-		$order_id = $wpdb->get_var( $wpdb->prepare( "SELECT DISTINCT ID FROM $wpdb->posts as posts LEFT JOIN $wpdb->postmeta as meta ON posts.ID = meta.post_id WHERE meta.meta_value = %s AND meta.meta_key = %s", $charge_id, '_transaction_id' ) );
428
+		$order_id = $wpdb->get_var($wpdb->prepare("SELECT DISTINCT ID FROM $wpdb->posts as posts LEFT JOIN $wpdb->postmeta as meta ON posts.ID = meta.post_id WHERE meta.meta_value = %s AND meta.meta_key = %s", $charge_id, '_transaction_id'));
429 429
 
430
-		if ( ! empty( $order_id ) ) {
431
-			return wc_get_order( $order_id );
430
+		if ( ! empty($order_id)) {
431
+			return wc_get_order($order_id);
432 432
 		}
433 433
 
434 434
 		return false;
@@ -441,13 +441,13 @@  discard block
 block discarded – undo
441 441
 	 * @param string $intent_id The ID of the intent.
442 442
 	 * @return WC_Order|bool Either an order or false when not found.
443 443
 	 */
444
-	public static function get_order_by_intent_id( $intent_id ) {
444
+	public static function get_order_by_intent_id($intent_id) {
445 445
 		global $wpdb;
446 446
 
447
-		$order_id = $wpdb->get_var( $wpdb->prepare( "SELECT DISTINCT ID FROM $wpdb->posts as posts LEFT JOIN $wpdb->postmeta as meta ON posts.ID = meta.post_id WHERE meta.meta_value = %s AND meta.meta_key = %s", $intent_id, '_stripe_intent_id' ) );
447
+		$order_id = $wpdb->get_var($wpdb->prepare("SELECT DISTINCT ID FROM $wpdb->posts as posts LEFT JOIN $wpdb->postmeta as meta ON posts.ID = meta.post_id WHERE meta.meta_value = %s AND meta.meta_key = %s", $intent_id, '_stripe_intent_id'));
448 448
 
449
-		if ( ! empty( $order_id ) ) {
450
-			return wc_get_order( $order_id );
449
+		if ( ! empty($order_id)) {
450
+			return wc_get_order($order_id);
451 451
 		}
452 452
 
453 453
 		return false;
@@ -460,13 +460,13 @@  discard block
 block discarded – undo
460 460
 	 * @param string $intent_id The ID of the intent.
461 461
 	 * @return WC_Order|bool Either an order or false when not found.
462 462
 	 */
463
-	public static function get_order_by_setup_intent_id( $intent_id ) {
463
+	public static function get_order_by_setup_intent_id($intent_id) {
464 464
 		global $wpdb;
465 465
 
466
-		$order_id = $wpdb->get_var( $wpdb->prepare( "SELECT DISTINCT ID FROM $wpdb->posts as posts LEFT JOIN $wpdb->postmeta as meta ON posts.ID = meta.post_id WHERE meta.meta_value = %s AND meta.meta_key = %s", $intent_id, '_stripe_setup_intent' ) );
466
+		$order_id = $wpdb->get_var($wpdb->prepare("SELECT DISTINCT ID FROM $wpdb->posts as posts LEFT JOIN $wpdb->postmeta as meta ON posts.ID = meta.post_id WHERE meta.meta_value = %s AND meta.meta_key = %s", $intent_id, '_stripe_setup_intent'));
467 467
 
468
-		if ( ! empty( $order_id ) ) {
469
-			return wc_get_order( $order_id );
468
+		if ( ! empty($order_id)) {
469
+			return wc_get_order($order_id);
470 470
 		}
471 471
 
472 472
 		return false;
@@ -482,13 +482,13 @@  discard block
 block discarded – undo
482 482
 	 * @param string $statement_descriptor
483 483
 	 * @return string $statement_descriptor Sanitized statement descriptor
484 484
 	 */
485
-	public static function clean_statement_descriptor( $statement_descriptor = '' ) {
486
-		$disallowed_characters = array( '<', '>', '"', "'" );
485
+	public static function clean_statement_descriptor($statement_descriptor = '') {
486
+		$disallowed_characters = array('<', '>', '"', "'");
487 487
 
488 488
 		// Remove special characters.
489
-		$statement_descriptor = str_replace( $disallowed_characters, '', $statement_descriptor );
489
+		$statement_descriptor = str_replace($disallowed_characters, '', $statement_descriptor);
490 490
 
491
-		$statement_descriptor = substr( trim( $statement_descriptor ), 0, 22 );
491
+		$statement_descriptor = substr(trim($statement_descriptor), 0, 22);
492 492
 
493 493
 		return $statement_descriptor;
494 494
 	}
Please login to merge, or discard this patch.
tests/phpunit/test-wc-stripe-sub-initial.php 1 patch
Spacing   +44 added lines, -44 removed lines patch added patch discarded remove patch
@@ -34,9 +34,9 @@  discard block
 block discarded – undo
34 34
 	public function setUp() {
35 35
 		parent::setUp();
36 36
 
37
-		$this->wc_stripe_subs_compat = $this->getMockBuilder( 'WC_Stripe_Subs_Compat' )
37
+		$this->wc_stripe_subs_compat = $this->getMockBuilder('WC_Stripe_Subs_Compat')
38 38
 			->disableOriginalConstructor()
39
-			->setMethods( array( 'prepare_source', 'has_subscription' ) )
39
+			->setMethods(array('prepare_source', 'has_subscription'))
40 40
 			->getMock();
41 41
 
42 42
 		// Mocked in order to get metadata[payment_type] = recurring in the HTTP request.
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
 	 */
55 55
 	public function tearDown() {
56 56
 		parent::tearDown();
57
-		delete_option( 'woocommerce_stripe_settings' );
57
+		delete_option('woocommerce_stripe_settings');
58 58
 	}
59 59
 
60 60
 	/**
@@ -69,27 +69,27 @@  discard block
 block discarded – undo
69 69
 	 */
70 70
 	public function test_initial_intent_parameters() {
71 71
 		$initial_order        = WC_Helper_Order::create_order();
72
-		$order_id             = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $initial_order->id : $initial_order->get_id();
73
-		$stripe_amount        = WC_Stripe_Helper::get_stripe_amount( $initial_order->get_total() );
74
-		$currency             = strtolower( WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $initial_order->get_order_currency() : $initial_order->get_currency() );
72
+		$order_id             = WC_Stripe_Helper::is_wc_lt('3.0') ? $initial_order->id : $initial_order->get_id();
73
+		$stripe_amount        = WC_Stripe_Helper::get_stripe_amount($initial_order->get_total());
74
+		$currency             = strtolower(WC_Stripe_Helper::is_wc_lt('3.0') ? $initial_order->get_order_currency() : $initial_order->get_currency());
75 75
 		$customer             = 'cus_123abc';
76 76
 		$source               = 'src_123abc';
77
-		$statement_descriptor = WC_Stripe_Helper::clean_statement_descriptor( $this->statement_descriptor );
77
+		$statement_descriptor = WC_Stripe_Helper::clean_statement_descriptor($this->statement_descriptor);
78 78
 		$intents_api_endpoint = 'https://api.stripe.com/v1/payment_intents';
79 79
 		$urls_used            = array();
80 80
 
81
-		if ( WC_Stripe_Helper::is_wc_lt( '3.0' ) ) {
81
+		if (WC_Stripe_Helper::is_wc_lt('3.0')) {
82 82
 			$initial_order->payment_method = 'stripe';
83
-			update_post_meta( $order_id, '_payment_method', 'stripe' ); // for `wc_get_order()`.
83
+			update_post_meta($order_id, '_payment_method', 'stripe'); // for `wc_get_order()`.
84 84
 		} else {
85
-			$initial_order->set_payment_method( 'stripe' );
85
+			$initial_order->set_payment_method('stripe');
86 86
 			$initial_order->save();
87 87
 		}
88 88
 
89 89
 		// Arrange: Mock prepare_source() so that we have a customer and source.
90 90
 		$this->wc_stripe_subs_compat
91
-			->expects( $this->any() )
92
-			->method( 'prepare_source' )
91
+			->expects($this->any())
92
+			->method('prepare_source')
93 93
 			->will(
94 94
 				$this->returnValue(
95 95
 					(object) array(
@@ -103,11 +103,11 @@  discard block
 block discarded – undo
103 103
 
104 104
 		// Emulate a subscription.
105 105
 		$this->wc_stripe_subs_compat
106
-			->expects( $this->any() )
107
-			->method( 'has_subscription' )
108
-			->will( $this->returnValue( true ) );
106
+			->expects($this->any())
107
+			->method('has_subscription')
108
+			->will($this->returnValue(true));
109 109
 
110
-		$pre_http_request_response_callback = function( $preempt, $request_args, $url ) use (
110
+		$pre_http_request_response_callback = function($preempt, $request_args, $url) use (
111 111
 			$stripe_amount,
112 112
 			$currency,
113 113
 			$customer,
@@ -118,9 +118,9 @@  discard block
 block discarded – undo
118 118
 			&$urls_used
119 119
 		) {
120 120
 			// Add all urls to array so we can later make assertions about which endpoints were used.
121
-			array_push( $urls_used, $url );
121
+			array_push($urls_used, $url);
122 122
 			// Continue without mocking the request if it's not the endpoint we care about.
123
-			if ( 0 !== strpos( $url, $intents_api_endpoint ) ) {
123
+			if (0 !== strpos($url, $intents_api_endpoint)) {
124 124
 				return false;
125 125
 			}
126 126
 
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
 			$response = array(
129 129
 				'headers'  => array(),
130 130
 				// Too bad we aren't dynamically setting things 'cus_123abc' when using this file.
131
-				'body'     => file_get_contents( 'tests/phpunit/dummy-data/subscription_signup_response_success.json' ),
131
+				'body'     => file_get_contents('tests/phpunit/dummy-data/subscription_signup_response_success.json'),
132 132
 				'response' => array(
133 133
 					'code'    => 200,
134 134
 					'message' => 'OK',
@@ -138,17 +138,17 @@  discard block
 block discarded – undo
138 138
 			);
139 139
 
140 140
 			// Respond with a successfull intent for confirmations.
141
-			if ( $url !== $intents_api_endpoint ) {
142
-				$response['body'] = str_replace( 'requires_confirmation', 'succeeded', $response['body'] );
141
+			if ($url !== $intents_api_endpoint) {
142
+				$response['body'] = str_replace('requires_confirmation', 'succeeded', $response['body']);
143 143
 				return $response;
144 144
 			}
145 145
 
146 146
 			// Assert: the request method is POST.
147
-			$this->assertArrayHasKey( 'method', $request_args );
148
-			$this->assertSame( 'POST', $request_args['method'] );
147
+			$this->assertArrayHasKey('method', $request_args);
148
+			$this->assertSame('POST', $request_args['method']);
149 149
 
150 150
 			// Assert: the request has a body.
151
-			$this->assertArrayHasKey( 'body', $request_args );
151
+			$this->assertArrayHasKey('body', $request_args);
152 152
 
153 153
 			// Assert: the request body contains these values.
154 154
 			$expected_request_body_values = array(
@@ -158,11 +158,11 @@  discard block
 block discarded – undo
158 158
 				'statement_descriptor' => $statement_descriptor,
159 159
 				'customer'             => $customer,
160 160
 				'setup_future_usage'   => 'off_session',
161
-				'payment_method_types' => array( 'card' ),
161
+				'payment_method_types' => array('card'),
162 162
 			);
163
-			foreach ( $expected_request_body_values as $key => $value ) {
164
-				$this->assertArrayHasKey( $key, $request_args['body'] );
165
-				$this->assertSame( $value, $request_args['body'][ $key ] );
163
+			foreach ($expected_request_body_values as $key => $value) {
164
+				$this->assertArrayHasKey($key, $request_args['body']);
165
+				$this->assertSame($value, $request_args['body'][$key]);
166 166
 			}
167 167
 
168 168
 			// Assert: the request body contains these keys, without checking for their value.
@@ -170,48 +170,48 @@  discard block
 block discarded – undo
170 170
 				'description',
171 171
 				'capture_method',
172 172
 			);
173
-			foreach ( $expected_request_body_keys as $key ) {
174
-				$this->assertArrayHasKey( $key, $request_args['body'] );
173
+			foreach ($expected_request_body_keys as $key) {
174
+				$this->assertArrayHasKey($key, $request_args['body']);
175 175
 			}
176 176
 
177 177
 			// Assert: the body metadata contains the order ID.
178
-			$this->assertSame( $order_id, absint( $request_args['body']['metadata']['order_id'] ) );
178
+			$this->assertSame($order_id, absint($request_args['body']['metadata']['order_id']));
179 179
 
180 180
 			// // Assert: the body metadata has these keys, without checking for their value.
181 181
 			$expected_metadata_keys = array(
182 182
 				'customer_name',
183 183
 				'customer_email',
184 184
 			);
185
-			foreach ( $expected_metadata_keys as $key ) {
186
-				$this->assertArrayHasKey( $key, $request_args['body']['metadata'] );
185
+			foreach ($expected_metadata_keys as $key) {
186
+				$this->assertArrayHasKey($key, $request_args['body']['metadata']);
187 187
 			}
188 188
 
189 189
 			// Return dummy content as the response.
190 190
 			return $response;
191 191
 		};
192
-		add_filter( 'pre_http_request', $pre_http_request_response_callback, 10, 3 );
192
+		add_filter('pre_http_request', $pre_http_request_response_callback, 10, 3);
193 193
 
194 194
 		// Act: call process_subscription_payment().
195 195
 		// We need to use `wc_stripe_subs_compat` here because we mocked this class earlier.
196
-		$result = $this->wc_stripe_subs_compat->process_payment( $order_id );
196
+		$result = $this->wc_stripe_subs_compat->process_payment($order_id);
197 197
 
198 198
 		// Assert: nothing was returned.
199
-		$this->assertEquals( $result['result'], 'success' );
200
-		$this->assertArrayHasKey( 'redirect', $result );
199
+		$this->assertEquals($result['result'], 'success');
200
+		$this->assertArrayHasKey('redirect', $result);
201 201
 
202
-		$order      = wc_get_order( $order_id );
202
+		$order      = wc_get_order($order_id);
203 203
 		$order_data = (
204
-			WC_Stripe_Helper::is_wc_lt( '3.0' )
205
-				? get_post_meta( $order_id, '_stripe_intent_id', true )
206
-				: $order->get_meta( '_stripe_intent_id' )
204
+			WC_Stripe_Helper::is_wc_lt('3.0')
205
+				? get_post_meta($order_id, '_stripe_intent_id', true)
206
+				: $order->get_meta('_stripe_intent_id')
207 207
 		);
208 208
 
209
-		$this->assertEquals( $order_data, 'pi_123abc' );
209
+		$this->assertEquals($order_data, 'pi_123abc');
210 210
 
211 211
 		// Assert: called payment intents.
212
-		$this->assertTrue( in_array( $intents_api_endpoint, $urls_used, true ) );
212
+		$this->assertTrue(in_array($intents_api_endpoint, $urls_used, true));
213 213
 
214 214
 		// Clean up.
215
-		remove_filter( 'pre_http_request', array( $this, 'pre_http_request_response_success' ) );
215
+		remove_filter('pre_http_request', array($this, 'pre_http_request_response_success'));
216 216
 	}
217 217
 }
Please login to merge, or discard this patch.
tests/phpunit/test-wc-stripe-sub-renewal.php 1 patch
Spacing   +78 added lines, -78 removed lines patch added patch discarded remove patch
@@ -34,17 +34,17 @@  discard block
 block discarded – undo
34 34
 	public function setUp() {
35 35
 		parent::setUp();
36 36
 
37
-		$this->wc_stripe_subs_compat = $this->getMockBuilder( 'WC_Stripe_Subs_Compat' )
37
+		$this->wc_stripe_subs_compat = $this->getMockBuilder('WC_Stripe_Subs_Compat')
38 38
 			->disableOriginalConstructor()
39
-			->setMethods( array( 'prepare_order_source', 'has_subscription' ) )
39
+			->setMethods(array('prepare_order_source', 'has_subscription'))
40 40
 			->getMock();
41 41
 
42 42
 		// Mocked in order to get metadata[payment_type] = recurring in the HTTP request.
43 43
 		$this->wc_stripe_subs_compat
44
-			->expects( $this->any() )
45
-			->method( 'has_subscription' )
44
+			->expects($this->any())
45
+			->method('has_subscription')
46 46
 			->will(
47
-				$this->returnValue( true )
47
+				$this->returnValue(true)
48 48
 			);
49 49
 
50 50
 		$this->statement_descriptor = 'This is a statement descriptor.';
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
 	public function tearDown() {
63 63
 		parent::tearDown();
64 64
 
65
-		delete_option( 'woocommerce_stripe_settings' );
65
+		delete_option('woocommerce_stripe_settings');
66 66
 	}
67 67
 
68 68
 	/**
@@ -79,11 +79,11 @@  discard block
 block discarded – undo
79 79
 		// Arrange: Some variables we'll use later.
80 80
 		$renewal_order                 = WC_Helper_Order::create_order();
81 81
 		$amount                        = 20; // WC Subs sends an amount to be used, instead of using the order amount.
82
-		$stripe_amount                 = WC_Stripe_Helper::get_stripe_amount( $amount );
83
-		$currency                      = strtolower( WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $renewal_order->get_order_currency() : $renewal_order->get_currency() );
82
+		$stripe_amount                 = WC_Stripe_Helper::get_stripe_amount($amount);
83
+		$currency                      = strtolower(WC_Stripe_Helper::is_wc_lt('3.0') ? $renewal_order->get_order_currency() : $renewal_order->get_currency());
84 84
 		$customer                      = 'cus_123abc';
85 85
 		$source                        = 'src_123abc';
86
-		$statement_descriptor          = WC_Stripe_Helper::clean_statement_descriptor( $this->statement_descriptor );
86
+		$statement_descriptor          = WC_Stripe_Helper::clean_statement_descriptor($this->statement_descriptor);
87 87
 		$should_retry                  = false;
88 88
 		$previous_error                = false;
89 89
 		$payments_intents_api_endpoint = 'https://api.stripe.com/v1/payment_intents';
@@ -91,16 +91,16 @@  discard block
 block discarded – undo
91 91
 
92 92
 		// Arrange: Set payment method to stripe, and not stripe_sepa, for example.
93 93
 		// This needed for testing the statement_descriptor.
94
-		if ( WC_Stripe_Helper::is_wc_lt( '3.0' ) ) {
94
+		if (WC_Stripe_Helper::is_wc_lt('3.0')) {
95 95
 			$renewal_order->payment_method = 'stripe';
96 96
 		} else {
97
-			$renewal_order->set_payment_method( 'stripe' );
97
+			$renewal_order->set_payment_method('stripe');
98 98
 		}
99 99
 
100 100
 		// Arrange: Mock prepare_order_source() so that we have a customer and source.
101 101
 		$this->wc_stripe_subs_compat
102
-			->expects( $this->any() )
103
-			->method( 'prepare_order_source' )
102
+			->expects($this->any())
103
+			->method('prepare_order_source')
104 104
 			->will(
105 105
 				$this->returnValue(
106 106
 					(object) array(
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
 
115 115
 		// Arrange: Add filter that will return a mocked HTTP response for the payment_intent call.
116 116
 		// Note: There are assertions in the callback function.
117
-		$pre_http_request_response_callback = function( $preempt, $request_args, $url ) use (
117
+		$pre_http_request_response_callback = function($preempt, $request_args, $url) use (
118 118
 			$renewal_order,
119 119
 			$stripe_amount,
120 120
 			$currency,
@@ -125,35 +125,35 @@  discard block
 block discarded – undo
125 125
 			&$urls_used
126 126
 		) {
127 127
 			// Add all urls to array so we can later make assertions about which endpoints were used.
128
-			array_push( $urls_used, $url );
128
+			array_push($urls_used, $url);
129 129
 
130 130
 			// Continue without mocking the request if it's not the endpoint we care about.
131
-			if ( $payments_intents_api_endpoint !== $url ) {
131
+			if ($payments_intents_api_endpoint !== $url) {
132 132
 				return false;
133 133
 			}
134 134
 
135 135
 			// Assert: the request method is POST.
136
-			$this->assertArrayHasKey( 'method', $request_args );
137
-			$this->assertSame( 'POST', $request_args['method'] );
136
+			$this->assertArrayHasKey('method', $request_args);
137
+			$this->assertSame('POST', $request_args['method']);
138 138
 
139 139
 			// Assert: the request has a body.
140
-			$this->assertArrayHasKey( 'body', $request_args );
140
+			$this->assertArrayHasKey('body', $request_args);
141 141
 
142 142
 			// Assert: the request body contains these values.
143 143
 			$expected_request_body_values = array(
144 144
 				'source'               => $source,
145 145
 				'amount'               => $stripe_amount,
146 146
 				'currency'             => $currency,
147
-				'payment_method_types' => array( 'card' ),
147
+				'payment_method_types' => array('card'),
148 148
 				'customer'             => $customer,
149 149
 				'off_session'          => 'true',
150 150
 				'confirm'              => 'true',
151 151
 				'confirmation_method'  => 'automatic',
152 152
 				'statement_descriptor' => $statement_descriptor,
153 153
 			);
154
-			foreach ( $expected_request_body_values as $key => $value ) {
155
-				$this->assertArrayHasKey( $key, $request_args['body'] );
156
-				$this->assertSame( $value, $request_args['body'][ $key ] );
154
+			foreach ($expected_request_body_values as $key => $value) {
155
+				$this->assertArrayHasKey($key, $request_args['body']);
156
+				$this->assertSame($value, $request_args['body'][$key]);
157 157
 			}
158 158
 
159 159
 			// Assert: the request body contains these keys, without checking for their value.
@@ -161,19 +161,19 @@  discard block
 block discarded – undo
161 161
 				'description',
162 162
 				'metadata',
163 163
 			);
164
-			foreach ( $expected_request_body_keys as $key ) {
165
-				$this->assertArrayHasKey( $key, $request_args['body'] );
164
+			foreach ($expected_request_body_keys as $key) {
165
+				$this->assertArrayHasKey($key, $request_args['body']);
166 166
 			}
167 167
 
168 168
 			// Assert: the body metadata has these values.
169
-			$order_id                 = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $renewal_order->id : (string) $renewal_order->get_id();
169
+			$order_id                 = WC_Stripe_Helper::is_wc_lt('3.0') ? $renewal_order->id : (string) $renewal_order->get_id();
170 170
 			$expected_metadata_values = array(
171 171
 				'order_id'     => $order_id,
172 172
 				'payment_type' => 'recurring',
173 173
 			);
174
-			foreach ( $expected_metadata_values as $key => $value ) {
175
-				$this->assertArrayHasKey( $key, $request_args['body']['metadata'] );
176
-				$this->assertSame( $value, $request_args['body']['metadata'][ $key ] );
174
+			foreach ($expected_metadata_values as $key => $value) {
175
+				$this->assertArrayHasKey($key, $request_args['body']['metadata']);
176
+				$this->assertSame($value, $request_args['body']['metadata'][$key]);
177 177
 			}
178 178
 
179 179
 			// Assert: the body metadata has these keys, without checking for their value.
@@ -182,8 +182,8 @@  discard block
 block discarded – undo
182 182
 				'customer_email',
183 183
 				'site_url',
184 184
 			);
185
-			foreach ( $expected_metadata_keys as $key ) {
186
-				$this->assertArrayHasKey( $key, $request_args['body']['metadata'] );
185
+			foreach ($expected_metadata_keys as $key) {
186
+				$this->assertArrayHasKey($key, $request_args['body']['metadata']);
187 187
 			}
188 188
 
189 189
 			// Assert: the request body does not contains these keys.
@@ -192,15 +192,15 @@  discard block
 block discarded – undo
192 192
 				'capture_method', // The default ('automatic') is what we want in this case, so we leave it off.
193 193
 				'expand[]',
194 194
 			);
195
-			foreach ( $expected_missing_request_body_keys as $key ) {
196
-				$this->assertArrayNotHasKey( $key, $request_args['body'] );
195
+			foreach ($expected_missing_request_body_keys as $key) {
196
+				$this->assertArrayNotHasKey($key, $request_args['body']);
197 197
 			}
198 198
 
199 199
 			// Arrange: return dummy content as the response.
200 200
 			return array(
201 201
 				'headers'  => array(),
202 202
 				// Too bad we aren't dynamically setting things 'cus_123abc' when using this file.
203
-				'body'     => file_get_contents( 'tests/phpunit/dummy-data/subscription_renewal_response_success.json' ),
203
+				'body'     => file_get_contents('tests/phpunit/dummy-data/subscription_renewal_response_success.json'),
204 204
 				'response' => array(
205 205
 					'code'    => 200,
206 206
 					'message' => 'OK',
@@ -210,55 +210,55 @@  discard block
 block discarded – undo
210 210
 			);
211 211
 		};
212 212
 
213
-		add_filter( 'pre_http_request', $pre_http_request_response_callback, 10, 3 );
213
+		add_filter('pre_http_request', $pre_http_request_response_callback, 10, 3);
214 214
 
215 215
 		// Arrange: Make sure to check that an action we care about was called
216 216
 		// by hooking into it.
217 217
 		$mock_action_process_payment = new MockAction();
218 218
 		add_action(
219 219
 			'wc_gateway_stripe_process_payment',
220
-			[ &$mock_action_process_payment, 'action' ]
220
+			[&$mock_action_process_payment, 'action']
221 221
 		);
222 222
 
223 223
 		// Act: call process_subscription_payment().
224 224
 		// We need to use `wc_stripe_subs_compat` here because we mocked this class earlier.
225
-		$result = $this->wc_stripe_subs_compat->process_subscription_payment( 20, $renewal_order, $should_retry, $previous_error );
225
+		$result = $this->wc_stripe_subs_compat->process_subscription_payment(20, $renewal_order, $should_retry, $previous_error);
226 226
 
227 227
 		// Assert: nothing was returned.
228
-		$this->assertEquals( $result, null );
228
+		$this->assertEquals($result, null);
229 229
 
230 230
 		// Assert that we saved the payment intent to the order.
231
-		$order_id   = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $renewal_order->id : $renewal_order->get_id();
232
-		$order      = wc_get_order( $order_id );
231
+		$order_id   = WC_Stripe_Helper::is_wc_lt('3.0') ? $renewal_order->id : $renewal_order->get_id();
232
+		$order      = wc_get_order($order_id);
233 233
 		$order_data = (
234
-			WC_Stripe_Helper::is_wc_lt( '3.0' )
235
-				? get_post_meta( $order_id, '_stripe_intent_id', true )
236
-				: $order->get_meta( '_stripe_intent_id' )
234
+			WC_Stripe_Helper::is_wc_lt('3.0')
235
+				? get_post_meta($order_id, '_stripe_intent_id', true)
236
+				: $order->get_meta('_stripe_intent_id')
237 237
 		);
238
-		$this->assertEquals( $order_data, 'pi_123abc' );
238
+		$this->assertEquals($order_data, 'pi_123abc');
239 239
 
240 240
 		// Transaction ID was saved to order.
241 241
 		$order_transaction_id = (
242
-			WC_Stripe_Helper::is_wc_lt( '3.0' )
243
-				? get_post_meta( $order_id, '_transaction_id', true )
242
+			WC_Stripe_Helper::is_wc_lt('3.0')
243
+				? get_post_meta($order_id, '_transaction_id', true)
244 244
 				: $order->get_transaction_id()
245 245
 		);
246
-		$this->assertEquals( $order_transaction_id, 'ch_123abc' );
246
+		$this->assertEquals($order_transaction_id, 'ch_123abc');
247 247
 
248 248
 		// Assert: the order was marked as processing (this is done in process_response()).
249
-		$this->assertEquals( $order->get_status(), 'processing' );
249
+		$this->assertEquals($order->get_status(), 'processing');
250 250
 
251 251
 		// Assert: called payment intents.
252
-		$this->assertTrue( in_array( $payments_intents_api_endpoint, $urls_used ) );
252
+		$this->assertTrue(in_array($payments_intents_api_endpoint, $urls_used));
253 253
 
254 254
 		// Assert: Our hook was called once.
255
-		$this->assertEquals( 1, $mock_action_process_payment->get_call_count() );
255
+		$this->assertEquals(1, $mock_action_process_payment->get_call_count());
256 256
 
257 257
 		// Assert: Only our hook was called.
258
-		$this->assertEquals( array( 'wc_gateway_stripe_process_payment' ), $mock_action_process_payment->get_tags() );
258
+		$this->assertEquals(array('wc_gateway_stripe_process_payment'), $mock_action_process_payment->get_tags());
259 259
 
260 260
 		// Clean up.
261
-		remove_filter( 'pre_http_request', array( $this, 'pre_http_request_response_success' ) );
261
+		remove_filter('pre_http_request', array($this, 'pre_http_request_response_success'));
262 262
 	}
263 263
 
264 264
 	/**
@@ -275,8 +275,8 @@  discard block
 block discarded – undo
275 275
 		// Arrange: Some variables we'll use later.
276 276
 		$renewal_order                 = WC_Helper_Order::create_order();
277 277
 		$amount                        = 20;
278
-		$stripe_amount                 = WC_Stripe_Helper::get_stripe_amount( $amount );
279
-		$currency                      = strtolower( WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $renewal_order->get_order_currency() : $renewal_order->get_currency() );
278
+		$stripe_amount                 = WC_Stripe_Helper::get_stripe_amount($amount);
279
+		$currency                      = strtolower(WC_Stripe_Helper::is_wc_lt('3.0') ? $renewal_order->get_order_currency() : $renewal_order->get_currency());
280 280
 		$customer                      = 'cus_123abc';
281 281
 		$source                        = 'src_123abc';
282 282
 		$should_retry                  = false;
@@ -286,8 +286,8 @@  discard block
 block discarded – undo
286 286
 
287 287
 		// Arrange: Mock prepare_order_source() so that we have a customer and source.
288 288
 		$this->wc_stripe_subs_compat
289
-			->expects( $this->any() )
290
-			->method( 'prepare_order_source' )
289
+			->expects($this->any())
290
+			->method('prepare_order_source')
291 291
 			->will(
292 292
 				$this->returnValue(
293 293
 					(object) array(
@@ -300,7 +300,7 @@  discard block
 block discarded – undo
300 300
 			);
301 301
 
302 302
 		// Arrange: Add filter that will return a mocked HTTP response for the payment_intent call.
303
-		$pre_http_request_response_callback = function( $preempt, $request_args, $url ) use (
303
+		$pre_http_request_response_callback = function($preempt, $request_args, $url) use (
304 304
 			$renewal_order,
305 305
 			$stripe_amount,
306 306
 			$currency,
@@ -310,10 +310,10 @@  discard block
 block discarded – undo
310 310
 			&$urls_used
311 311
 		) {
312 312
 			// Add all urls to array so we can later make assertions about which endpoints were used.
313
-			array_push( $urls_used, $url );
313
+			array_push($urls_used, $url);
314 314
 
315 315
 			// Continue without mocking the request if it's not the endpoint we care about.
316
-			if ( $payments_intents_api_endpoint !== $url ) {
316
+			if ($payments_intents_api_endpoint !== $url) {
317 317
 				return false;
318 318
 			}
319 319
 
@@ -321,7 +321,7 @@  discard block
 block discarded – undo
321 321
 			return array(
322 322
 				'headers'  => array(),
323 323
 				// Too bad we aren't dynamically setting things 'cus_123abc' when using this file.
324
-				'body'     => file_get_contents( 'tests/phpunit/dummy-data/subscription_renewal_response_authentication_required.json' ),
324
+				'body'     => file_get_contents('tests/phpunit/dummy-data/subscription_renewal_response_authentication_required.json'),
325 325
 				'response' => array(
326 326
 					'code'    => 402,
327 327
 					'message' => 'Payment Required',
@@ -330,56 +330,56 @@  discard block
 block discarded – undo
330 330
 				'filename' => null,
331 331
 			);
332 332
 		};
333
-		add_filter( 'pre_http_request', $pre_http_request_response_callback, 10, 3 );
333
+		add_filter('pre_http_request', $pre_http_request_response_callback, 10, 3);
334 334
 
335 335
 		// Arrange: Make sure to check that an action we care about was called
336 336
 		// by hooking into it.
337 337
 		$mock_action_process_payment = new MockAction();
338 338
 		add_action(
339 339
 			'wc_gateway_stripe_process_payment_authentication_required',
340
-			[ &$mock_action_process_payment, 'action' ]
340
+			[&$mock_action_process_payment, 'action']
341 341
 		);
342 342
 
343 343
 		// Act: call process_subscription_payment().
344 344
 		// We need to use `wc_stripe_subs_compat` here because we mocked this class earlier.
345
-		$result = $this->wc_stripe_subs_compat->process_subscription_payment( 20, $renewal_order, $should_retry, $previous_error );
345
+		$result = $this->wc_stripe_subs_compat->process_subscription_payment(20, $renewal_order, $should_retry, $previous_error);
346 346
 
347 347
 		// Assert: nothing was returned.
348
-		$this->assertEquals( $result, null );
348
+		$this->assertEquals($result, null);
349 349
 
350 350
 		// Assert that we saved the payment intent to the order.
351
-		$order_id             = WC_Stripe_Helper::is_wc_lt( '3.0' ) ? $renewal_order->id : $renewal_order->get_id();
352
-		$order                = wc_get_order( $order_id );
351
+		$order_id             = WC_Stripe_Helper::is_wc_lt('3.0') ? $renewal_order->id : $renewal_order->get_id();
352
+		$order                = wc_get_order($order_id);
353 353
 		$order_data           = (
354
-			WC_Stripe_Helper::is_wc_lt( '3.0' )
355
-				? get_post_meta( $order_id, '_stripe_intent_id', true )
356
-				: $order->get_meta( '_stripe_intent_id' )
354
+			WC_Stripe_Helper::is_wc_lt('3.0')
355
+				? get_post_meta($order_id, '_stripe_intent_id', true)
356
+				: $order->get_meta('_stripe_intent_id')
357 357
 		);
358 358
 		$order_transaction_id = (
359
-			WC_Stripe_Helper::is_wc_lt( '3.0' )
360
-				? get_post_meta( $order_id, '_transaction_id', true )
359
+			WC_Stripe_Helper::is_wc_lt('3.0')
360
+				? get_post_meta($order_id, '_transaction_id', true)
361 361
 				: $order->get_transaction_id()
362 362
 		);
363 363
 
364 364
 		// Intent was saved to order even though there was an error in the response body.
365
-		$this->assertEquals( $order_data, 'pi_123abc' );
365
+		$this->assertEquals($order_data, 'pi_123abc');
366 366
 
367 367
 		// Transaction ID was saved to order.
368
-		$this->assertEquals( $order_transaction_id, 'ch_123abc' );
368
+		$this->assertEquals($order_transaction_id, 'ch_123abc');
369 369
 
370 370
 		// Assert: the order was marked as failed.
371
-		$this->assertEquals( $order->get_status(), 'failed' );
371
+		$this->assertEquals($order->get_status(), 'failed');
372 372
 
373 373
 		// Assert: called payment intents.
374
-		$this->assertTrue( in_array( $payments_intents_api_endpoint, $urls_used ) );
374
+		$this->assertTrue(in_array($payments_intents_api_endpoint, $urls_used));
375 375
 
376 376
 		// Assert: Our hook was called once.
377
-		$this->assertEquals( 1, $mock_action_process_payment->get_call_count() );
377
+		$this->assertEquals(1, $mock_action_process_payment->get_call_count());
378 378
 
379 379
 		// Assert: Only our hook was called.
380
-		$this->assertEquals( array( 'wc_gateway_stripe_process_payment_authentication_required' ), $mock_action_process_payment->get_tags() );
380
+		$this->assertEquals(array('wc_gateway_stripe_process_payment_authentication_required'), $mock_action_process_payment->get_tags());
381 381
 
382 382
 		// Clean up.
383
-		remove_filter( 'pre_http_request', array( $this, 'pre_http_request_response_success' ) );
383
+		remove_filter('pre_http_request', array($this, 'pre_http_request_response_success'));
384 384
 	}
385 385
 }
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-renewal-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
 
@@ -14,20 +14,20 @@  discard block
 block discarded – undo
14 14
 	 *
15 15
 	 * @param WC_Email[] $email_classes All existing instances of WooCommerce emails.
16 16
 	 */
17
-	public function __construct( $email_classes = array() ) {
17
+	public function __construct($email_classes = array()) {
18 18
 		$this->id             = 'failed_renewal_authentication';
19
-		$this->title          = __( 'Failed Subscription Renewal SCA Authentication', 'woocommerce-gateway-stripe' );
20
-		$this->description    = __( 'Sent to a customer when a renewal fails because the transaction requires an SCA verification. The email contains renewal order information and payment links.', 'woocommerce-gateway-stripe' );
19
+		$this->title          = __('Failed Subscription Renewal SCA Authentication', 'woocommerce-gateway-stripe');
20
+		$this->description    = __('Sent to a customer when a renewal fails because the transaction requires an SCA verification. The email contains renewal order information and payment links.', 'woocommerce-gateway-stripe');
21 21
 		$this->customer_email = true;
22 22
 
23 23
 		$this->template_html  = 'emails/failed-renewal-authentication.php';
24 24
 		$this->template_plain = 'emails/plain/failed-renewal-authentication.php';
25
-		$this->template_base  = plugin_dir_path( WC_STRIPE_MAIN_FILE ) . 'templates/';
25
+		$this->template_base  = plugin_dir_path(WC_STRIPE_MAIN_FILE) . 'templates/';
26 26
 
27 27
 		// Triggers the email at the correct hook.
28
-		add_action( 'wc_gateway_stripe_process_payment_authentication_required', array( $this, 'trigger' ) );
28
+		add_action('wc_gateway_stripe_process_payment_authentication_required', array($this, 'trigger'));
29 29
 
30
-		if ( isset( $email_classes['WCS_Email_Customer_Renewal_Invoice'] ) ) {
30
+		if (isset($email_classes['WCS_Email_Customer_Renewal_Invoice'])) {
31 31
 			$this->original_email = $email_classes['WCS_Email_Customer_Renewal_Invoice'];
32 32
 		}
33 33
 
@@ -40,18 +40,18 @@  discard block
 block discarded – undo
40 40
 	 *
41 41
 	 * @param WC_Order $order The order that is being paid.
42 42
 	 */
43
-	public function trigger( $order ) {
44
-		if ( function_exists( 'wcs_order_contains_subscription' ) && ( wcs_order_contains_subscription( $order->get_id() ) || wcs_is_subscription( $order->get_id() ) || wcs_order_contains_renewal( $order->get_id() ) ) ) {
45
-			parent::trigger( $order );
43
+	public function trigger($order) {
44
+		if (function_exists('wcs_order_contains_subscription') && (wcs_order_contains_subscription($order->get_id()) || wcs_is_subscription($order->get_id()) || wcs_order_contains_renewal($order->get_id()))) {
45
+			parent::trigger($order);
46 46
 
47 47
 			// Prevent the renewal email from WooCommerce Subscriptions from being sent.
48
-			if ( isset( $this->original_email ) ) {
49
-				remove_action( 'woocommerce_generated_manual_renewal_order_renewal_notification', array( $this->original_email, 'trigger' ) );
50
-				remove_action( 'woocommerce_order_status_failed_renewal_notification', array( $this->original_email, 'trigger' ) );
48
+			if (isset($this->original_email)) {
49
+				remove_action('woocommerce_generated_manual_renewal_order_renewal_notification', array($this->original_email, 'trigger'));
50
+				remove_action('woocommerce_order_status_failed_renewal_notification', array($this->original_email, 'trigger'));
51 51
 			}
52 52
 
53 53
 			// Prevent the retry email from WooCommerce Subscriptions from being sent.
54
-			add_filter( 'wcs_get_retry_rule_raw', array( $this, 'prevent_retry_notification_email' ), 100, 3 );
54
+			add_filter('wcs_get_retry_rule_raw', array($this, 'prevent_retry_notification_email'), 100, 3);
55 55
 		}
56 56
 	}
57 57
 
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
 	 * @return string
62 62
 	 */
63 63
 	public function get_default_subject() {
64
-		return __( 'Payment authorization needed for renewal of {site_title} order {order_number}', 'woocommerce-gateway-stripe' );
64
+		return __('Payment authorization needed for renewal of {site_title} order {order_number}', 'woocommerce-gateway-stripe');
65 65
 	}
66 66
 
67 67
 	/**
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
 	 * @return string
71 71
 	 */
72 72
 	public function get_default_heading() {
73
-		return __( 'Payment authorization needed for renewal of order {order_number}', 'woocommerce-gateway-stripe' );
73
+		return __('Payment authorization needed for renewal of order {order_number}', 'woocommerce-gateway-stripe');
74 74
 	}
75 75
 
76 76
 	/**
@@ -81,8 +81,8 @@  discard block
 block discarded – undo
81 81
 	 * @param int   $order_id     The ID of the order that needs payment.
82 82
 	 * @return array
83 83
 	 */
84
-	public function prevent_retry_notification_email( $rule_array, $retry_number, $order_id ) {
85
-		if ( wcs_get_objects_property( $this->object, 'id' ) === $order_id ) {
84
+	public function prevent_retry_notification_email($rule_array, $retry_number, $order_id) {
85
+		if (wcs_get_objects_property($this->object, 'id') === $order_id) {
86 86
 			$rule_array['email_template_customer'] = '';
87 87
 		}
88 88
 
Please login to merge, or discard this patch.
woocommerce-gateway-stripe.php 1 patch
Spacing   +89 added lines, -89 removed lines patch added patch discarded remove patch
@@ -15,7 +15,7 @@  discard block
 block discarded – undo
15 15
  *
16 16
  */
17 17
 
18
-if ( ! defined( 'ABSPATH' ) ) {
18
+if ( ! defined('ABSPATH')) {
19 19
 	exit;
20 20
 }
21 21
 
@@ -29,29 +29,29 @@  discard block
 block discarded – undo
29 29
  */
30 30
 function woocommerce_stripe_missing_wc_notice() {
31 31
 	/* translators: 1. URL link. */
32
-	echo '<div class="error"><p><strong>' . sprintf( esc_html__( 'Stripe requires WooCommerce to be installed and active. You can download %s here.', 'woocommerce-gateway-stripe' ), '<a href="https://woocommerce.com/" target="_blank">WooCommerce</a>' ) . '</strong></p></div>';
32
+	echo '<div class="error"><p><strong>' . sprintf(esc_html__('Stripe requires WooCommerce to be installed and active. You can download %s here.', 'woocommerce-gateway-stripe'), '<a href="https://woocommerce.com/" target="_blank">WooCommerce</a>') . '</strong></p></div>';
33 33
 }
34 34
 
35
-add_action( 'plugins_loaded', 'woocommerce_gateway_stripe_init' );
35
+add_action('plugins_loaded', 'woocommerce_gateway_stripe_init');
36 36
 
37 37
 function woocommerce_gateway_stripe_init() {
38
-	load_plugin_textdomain( 'woocommerce-gateway-stripe', false, plugin_basename( dirname( __FILE__ ) ) . '/languages' );
38
+	load_plugin_textdomain('woocommerce-gateway-stripe', false, plugin_basename(dirname(__FILE__)) . '/languages');
39 39
 
40
-	if ( ! class_exists( 'WooCommerce' ) ) {
41
-		add_action( 'admin_notices', 'woocommerce_stripe_missing_wc_notice' );
40
+	if ( ! class_exists('WooCommerce')) {
41
+		add_action('admin_notices', 'woocommerce_stripe_missing_wc_notice');
42 42
 		return;
43 43
 	}
44 44
 
45
-	if ( ! class_exists( 'WC_Stripe' ) ) :
45
+	if ( ! class_exists('WC_Stripe')) :
46 46
 		/**
47 47
 		 * Required minimums and constants
48 48
 		 */
49
-		define( 'WC_STRIPE_VERSION', '4.2.5' );
50
-		define( 'WC_STRIPE_MIN_PHP_VER', '5.6.0' );
51
-		define( 'WC_STRIPE_MIN_WC_VER', '2.6.0' );
52
-		define( 'WC_STRIPE_MAIN_FILE', __FILE__ );
53
-		define( 'WC_STRIPE_PLUGIN_URL', untrailingslashit( plugins_url( basename( plugin_dir_path( __FILE__ ) ), basename( __FILE__ ) ) ) );
54
-		define( 'WC_STRIPE_PLUGIN_PATH', untrailingslashit( plugin_dir_path( __FILE__ ) ) );
49
+		define('WC_STRIPE_VERSION', '4.2.5');
50
+		define('WC_STRIPE_MIN_PHP_VER', '5.6.0');
51
+		define('WC_STRIPE_MIN_WC_VER', '2.6.0');
52
+		define('WC_STRIPE_MAIN_FILE', __FILE__);
53
+		define('WC_STRIPE_PLUGIN_URL', untrailingslashit(plugins_url(basename(plugin_dir_path(__FILE__)), basename(__FILE__))));
54
+		define('WC_STRIPE_PLUGIN_PATH', untrailingslashit(plugin_dir_path(__FILE__)));
55 55
 
56 56
 		class WC_Stripe {
57 57
 
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
 			 * @return Singleton The *Singleton* instance.
67 67
 			 */
68 68
 			public static function get_instance() {
69
-				if ( null === self::$instance ) {
69
+				if (null === self::$instance) {
70 70
 					self::$instance = new self();
71 71
 				}
72 72
 				return self::$instance;
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
 			 * *Singleton* via the `new` operator from outside of this class.
94 94
 			 */
95 95
 			private function __construct() {
96
-				add_action( 'admin_init', array( $this, 'install' ) );
96
+				add_action('admin_init', array($this, 'install'));
97 97
 				$this->init();
98 98
 			}
99 99
 
@@ -104,52 +104,52 @@  discard block
 block discarded – undo
104 104
 			 * @version 4.0.0
105 105
 			 */
106 106
 			public function init() {
107
-				if ( is_admin() ) {
108
-					require_once dirname( __FILE__ ) . '/includes/admin/class-wc-stripe-privacy.php';
107
+				if (is_admin()) {
108
+					require_once dirname(__FILE__) . '/includes/admin/class-wc-stripe-privacy.php';
109 109
 				}
110 110
 
111
-				require_once dirname( __FILE__ ) . '/includes/class-wc-stripe-exception.php';
112
-				require_once dirname( __FILE__ ) . '/includes/class-wc-stripe-logger.php';
113
-				require_once dirname( __FILE__ ) . '/includes/class-wc-stripe-helper.php';
114
-				include_once dirname( __FILE__ ) . '/includes/class-wc-stripe-api.php';
115
-				require_once dirname( __FILE__ ) . '/includes/abstracts/abstract-wc-stripe-payment-gateway.php';
116
-				require_once dirname( __FILE__ ) . '/includes/class-wc-stripe-webhook-handler.php';
117
-				require_once dirname( __FILE__ ) . '/includes/class-wc-stripe-sepa-payment-token.php';
118
-				require_once dirname( __FILE__ ) . '/includes/class-wc-stripe-apple-pay-registration.php';
119
-				require_once dirname( __FILE__ ) . '/includes/compat/class-wc-stripe-pre-orders-compat.php';
120
-				require_once dirname( __FILE__ ) . '/includes/class-wc-gateway-stripe.php';
121
-				require_once dirname( __FILE__ ) . '/includes/payment-methods/class-wc-gateway-stripe-bancontact.php';
122
-				require_once dirname( __FILE__ ) . '/includes/payment-methods/class-wc-gateway-stripe-sofort.php';
123
-				require_once dirname( __FILE__ ) . '/includes/payment-methods/class-wc-gateway-stripe-giropay.php';
124
-				require_once dirname( __FILE__ ) . '/includes/payment-methods/class-wc-gateway-stripe-eps.php';
125
-				require_once dirname( __FILE__ ) . '/includes/payment-methods/class-wc-gateway-stripe-ideal.php';
126
-				require_once dirname( __FILE__ ) . '/includes/payment-methods/class-wc-gateway-stripe-p24.php';
127
-				require_once dirname( __FILE__ ) . '/includes/payment-methods/class-wc-gateway-stripe-alipay.php';
128
-				require_once dirname( __FILE__ ) . '/includes/payment-methods/class-wc-gateway-stripe-sepa.php';
129
-				require_once dirname( __FILE__ ) . '/includes/payment-methods/class-wc-gateway-stripe-multibanco.php';
130
-				require_once dirname( __FILE__ ) . '/includes/payment-methods/class-wc-stripe-payment-request.php';
131
-				require_once dirname( __FILE__ ) . '/includes/compat/class-wc-stripe-subs-compat.php';
132
-				require_once dirname( __FILE__ ) . '/includes/compat/class-wc-stripe-sepa-subs-compat.php';
133
-				require_once dirname( __FILE__ ) . '/includes/class-wc-stripe-order-handler.php';
134
-				require_once dirname( __FILE__ ) . '/includes/class-wc-stripe-payment-tokens.php';
135
-				require_once dirname( __FILE__ ) . '/includes/class-wc-stripe-customer.php';
136
-				require_once dirname( __FILE__ ) . '/includes/class-wc-stripe-intent-controller.php';
137
-
138
-				if ( is_admin() ) {
139
-					require_once dirname( __FILE__ ) . '/includes/admin/class-wc-stripe-admin-notices.php';
111
+				require_once dirname(__FILE__) . '/includes/class-wc-stripe-exception.php';
112
+				require_once dirname(__FILE__) . '/includes/class-wc-stripe-logger.php';
113
+				require_once dirname(__FILE__) . '/includes/class-wc-stripe-helper.php';
114
+				include_once dirname(__FILE__) . '/includes/class-wc-stripe-api.php';
115
+				require_once dirname(__FILE__) . '/includes/abstracts/abstract-wc-stripe-payment-gateway.php';
116
+				require_once dirname(__FILE__) . '/includes/class-wc-stripe-webhook-handler.php';
117
+				require_once dirname(__FILE__) . '/includes/class-wc-stripe-sepa-payment-token.php';
118
+				require_once dirname(__FILE__) . '/includes/class-wc-stripe-apple-pay-registration.php';
119
+				require_once dirname(__FILE__) . '/includes/compat/class-wc-stripe-pre-orders-compat.php';
120
+				require_once dirname(__FILE__) . '/includes/class-wc-gateway-stripe.php';
121
+				require_once dirname(__FILE__) . '/includes/payment-methods/class-wc-gateway-stripe-bancontact.php';
122
+				require_once dirname(__FILE__) . '/includes/payment-methods/class-wc-gateway-stripe-sofort.php';
123
+				require_once dirname(__FILE__) . '/includes/payment-methods/class-wc-gateway-stripe-giropay.php';
124
+				require_once dirname(__FILE__) . '/includes/payment-methods/class-wc-gateway-stripe-eps.php';
125
+				require_once dirname(__FILE__) . '/includes/payment-methods/class-wc-gateway-stripe-ideal.php';
126
+				require_once dirname(__FILE__) . '/includes/payment-methods/class-wc-gateway-stripe-p24.php';
127
+				require_once dirname(__FILE__) . '/includes/payment-methods/class-wc-gateway-stripe-alipay.php';
128
+				require_once dirname(__FILE__) . '/includes/payment-methods/class-wc-gateway-stripe-sepa.php';
129
+				require_once dirname(__FILE__) . '/includes/payment-methods/class-wc-gateway-stripe-multibanco.php';
130
+				require_once dirname(__FILE__) . '/includes/payment-methods/class-wc-stripe-payment-request.php';
131
+				require_once dirname(__FILE__) . '/includes/compat/class-wc-stripe-subs-compat.php';
132
+				require_once dirname(__FILE__) . '/includes/compat/class-wc-stripe-sepa-subs-compat.php';
133
+				require_once dirname(__FILE__) . '/includes/class-wc-stripe-order-handler.php';
134
+				require_once dirname(__FILE__) . '/includes/class-wc-stripe-payment-tokens.php';
135
+				require_once dirname(__FILE__) . '/includes/class-wc-stripe-customer.php';
136
+				require_once dirname(__FILE__) . '/includes/class-wc-stripe-intent-controller.php';
137
+
138
+				if (is_admin()) {
139
+					require_once dirname(__FILE__) . '/includes/admin/class-wc-stripe-admin-notices.php';
140 140
 				}
141 141
 
142 142
 				// REMOVE IN THE FUTURE.
143
-				require_once dirname( __FILE__ ) . '/includes/deprecated/class-wc-stripe-apple-pay.php';
143
+				require_once dirname(__FILE__) . '/includes/deprecated/class-wc-stripe-apple-pay.php';
144 144
 
145
-				add_filter( 'woocommerce_payment_gateways', array( $this, 'add_gateways' ) );
146
-				add_filter( 'plugin_action_links_' . plugin_basename( __FILE__ ), array( $this, 'plugin_action_links' ) );
145
+				add_filter('woocommerce_payment_gateways', array($this, 'add_gateways'));
146
+				add_filter('plugin_action_links_' . plugin_basename(__FILE__), array($this, 'plugin_action_links'));
147 147
 
148 148
 				// Modify emails emails.
149
-				add_filter( 'woocommerce_email_classes', array( $this, 'add_emails' ), 20 );
149
+				add_filter('woocommerce_email_classes', array($this, 'add_emails'), 20);
150 150
 
151
-				if ( version_compare( WC_VERSION, '3.4', '<' ) ) {
152
-					add_filter( 'woocommerce_get_sections_checkout', array( $this, 'filter_gateway_order_admin' ) );
151
+				if (version_compare(WC_VERSION, '3.4', '<')) {
152
+					add_filter('woocommerce_get_sections_checkout', array($this, 'filter_gateway_order_admin'));
153 153
 				}
154 154
 			}
155 155
 
@@ -160,8 +160,8 @@  discard block
 block discarded – undo
160 160
 			 * @version 4.0.0
161 161
 			 */
162 162
 			public function update_plugin_version() {
163
-				delete_option( 'wc_stripe_version' );
164
-				update_option( 'wc_stripe_version', WC_STRIPE_VERSION );
163
+				delete_option('wc_stripe_version');
164
+				update_option('wc_stripe_version', WC_STRIPE_VERSION);
165 165
 			}
166 166
 
167 167
 			/**
@@ -171,15 +171,15 @@  discard block
 block discarded – undo
171 171
 			 * @version 3.1.0
172 172
 			 */
173 173
 			public function install() {
174
-				if ( ! is_plugin_active( plugin_basename( __FILE__ ) ) ) {
174
+				if ( ! is_plugin_active(plugin_basename(__FILE__))) {
175 175
 					return;
176 176
 				}
177 177
 
178
-				if ( ! defined( 'IFRAME_REQUEST' ) && ( WC_STRIPE_VERSION !== get_option( 'wc_stripe_version' ) ) ) {
179
-					do_action( 'woocommerce_stripe_updated' );
178
+				if ( ! defined('IFRAME_REQUEST') && (WC_STRIPE_VERSION !== get_option('wc_stripe_version'))) {
179
+					do_action('woocommerce_stripe_updated');
180 180
 
181
-					if ( ! defined( 'WC_STRIPE_INSTALLING' ) ) {
182
-						define( 'WC_STRIPE_INSTALLING', true );
181
+					if ( ! defined('WC_STRIPE_INSTALLING')) {
182
+						define('WC_STRIPE_INSTALLING', true);
183 183
 					}
184 184
 
185 185
 					$this->update_plugin_version();
@@ -192,13 +192,13 @@  discard block
 block discarded – undo
192 192
 			 * @since 1.0.0
193 193
 			 * @version 4.0.0
194 194
 			 */
195
-			public function plugin_action_links( $links ) {
195
+			public function plugin_action_links($links) {
196 196
 				$plugin_links = array(
197
-					'<a href="admin.php?page=wc-settings&tab=checkout&section=stripe">' . esc_html__( 'Settings', 'woocommerce-gateway-stripe' ) . '</a>',
198
-					'<a href="https://docs.woocommerce.com/document/stripe/">' . esc_html__( 'Docs', 'woocommerce-gateway-stripe' ) . '</a>',
199
-					'<a href="https://woocommerce.com/my-account/create-a-ticket?broken=primary&select=18627">' . esc_html__( 'Support', 'woocommerce-gateway-stripe' ) . '</a>',
197
+					'<a href="admin.php?page=wc-settings&tab=checkout&section=stripe">' . esc_html__('Settings', 'woocommerce-gateway-stripe') . '</a>',
198
+					'<a href="https://docs.woocommerce.com/document/stripe/">' . esc_html__('Docs', 'woocommerce-gateway-stripe') . '</a>',
199
+					'<a href="https://woocommerce.com/my-account/create-a-ticket?broken=primary&select=18627">' . esc_html__('Support', 'woocommerce-gateway-stripe') . '</a>',
200 200
 				);
201
-				return array_merge( $plugin_links, $links );
201
+				return array_merge($plugin_links, $links);
202 202
 			}
203 203
 
204 204
 			/**
@@ -207,8 +207,8 @@  discard block
 block discarded – undo
207 207
 			 * @since 1.0.0
208 208
 			 * @version 4.0.0
209 209
 			 */
210
-			public function add_gateways( $methods ) {
211
-				if ( class_exists( 'WC_Subscriptions_Order' ) && function_exists( 'wcs_create_renewal_order' ) ) {
210
+			public function add_gateways($methods) {
211
+				if (class_exists('WC_Subscriptions_Order') && function_exists('wcs_create_renewal_order')) {
212 212
 					$methods[] = 'WC_Stripe_Subs_Compat';
213 213
 					$methods[] = 'WC_Stripe_Sepa_Subs_Compat';
214 214
 				} else {
@@ -234,28 +234,28 @@  discard block
 block discarded – undo
234 234
 			 * @since 4.0.0
235 235
 			 * @version 4.0.0
236 236
 			 */
237
-			public function filter_gateway_order_admin( $sections ) {
238
-				unset( $sections['stripe'] );
239
-				unset( $sections['stripe_bancontact'] );
240
-				unset( $sections['stripe_sofort'] );
241
-				unset( $sections['stripe_giropay'] );
242
-				unset( $sections['stripe_eps'] );
243
-				unset( $sections['stripe_ideal'] );
244
-				unset( $sections['stripe_p24'] );
245
-				unset( $sections['stripe_alipay'] );
246
-				unset( $sections['stripe_sepa'] );
247
-				unset( $sections['stripe_multibanco'] );
237
+			public function filter_gateway_order_admin($sections) {
238
+				unset($sections['stripe']);
239
+				unset($sections['stripe_bancontact']);
240
+				unset($sections['stripe_sofort']);
241
+				unset($sections['stripe_giropay']);
242
+				unset($sections['stripe_eps']);
243
+				unset($sections['stripe_ideal']);
244
+				unset($sections['stripe_p24']);
245
+				unset($sections['stripe_alipay']);
246
+				unset($sections['stripe_sepa']);
247
+				unset($sections['stripe_multibanco']);
248 248
 
249 249
 				$sections['stripe']            = 'Stripe';
250
-				$sections['stripe_bancontact'] = __( 'Stripe Bancontact', 'woocommerce-gateway-stripe' );
251
-				$sections['stripe_sofort']     = __( 'Stripe SOFORT', 'woocommerce-gateway-stripe' );
252
-				$sections['stripe_giropay']    = __( 'Stripe Giropay', 'woocommerce-gateway-stripe' );
253
-				$sections['stripe_eps']        = __( 'Stripe EPS', 'woocommerce-gateway-stripe' );
254
-				$sections['stripe_ideal']      = __( 'Stripe iDeal', 'woocommerce-gateway-stripe' );
255
-				$sections['stripe_p24']        = __( 'Stripe P24', 'woocommerce-gateway-stripe' );
256
-				$sections['stripe_alipay']     = __( 'Stripe Alipay', 'woocommerce-gateway-stripe' );
257
-				$sections['stripe_sepa']       = __( 'Stripe SEPA Direct Debit', 'woocommerce-gateway-stripe' );
258
-				$sections['stripe_multibanco'] = __( 'Stripe Multibanco', 'woocommerce-gateway-stripe' );
250
+				$sections['stripe_bancontact'] = __('Stripe Bancontact', 'woocommerce-gateway-stripe');
251
+				$sections['stripe_sofort']     = __('Stripe SOFORT', 'woocommerce-gateway-stripe');
252
+				$sections['stripe_giropay']    = __('Stripe Giropay', 'woocommerce-gateway-stripe');
253
+				$sections['stripe_eps']        = __('Stripe EPS', 'woocommerce-gateway-stripe');
254
+				$sections['stripe_ideal']      = __('Stripe iDeal', 'woocommerce-gateway-stripe');
255
+				$sections['stripe_p24']        = __('Stripe P24', 'woocommerce-gateway-stripe');
256
+				$sections['stripe_alipay']     = __('Stripe Alipay', 'woocommerce-gateway-stripe');
257
+				$sections['stripe_sepa']       = __('Stripe SEPA Direct Debit', 'woocommerce-gateway-stripe');
258
+				$sections['stripe_multibanco'] = __('Stripe Multibanco', 'woocommerce-gateway-stripe');
259 259
 
260 260
 				return $sections;
261 261
 			}
@@ -266,14 +266,14 @@  discard block
 block discarded – undo
266 266
 			 * @param WC_Email[] $email_classes All existing emails.
267 267
 			 * @return WC_Email[]
268 268
 			 */
269
-			public function add_emails( $email_classes ) {
269
+			public function add_emails($email_classes) {
270 270
 				require_once WC_STRIPE_PLUGIN_PATH . '/includes/compat/class-wc-stripe-email-failed-authentication.php';
271 271
 				require_once WC_STRIPE_PLUGIN_PATH . '/includes/compat/class-wc-stripe-email-failed-renewal-authentication.php';
272 272
 				require_once WC_STRIPE_PLUGIN_PATH . '/includes/compat/class-wc-stripe-email-failed-preorder-authentication.php';
273 273
 
274 274
 				// Add all emails, generated by the gateway.
275
-				$email_classes['WC_Stripe_Email_Failed_Renewal_Authentication']  = new WC_Stripe_Email_Failed_Renewal_Authentication( $email_classes );
276
-				$email_classes['WC_Stripe_Email_Failed_Preorder_Authentication'] = new WC_Stripe_Email_Failed_Preorder_Authentication( $email_classes );
275
+				$email_classes['WC_Stripe_Email_Failed_Renewal_Authentication']  = new WC_Stripe_Email_Failed_Renewal_Authentication($email_classes);
276
+				$email_classes['WC_Stripe_Email_Failed_Preorder_Authentication'] = new WC_Stripe_Email_Failed_Preorder_Authentication($email_classes);
277 277
 
278 278
 				return $email_classes;
279 279
 			}
Please login to merge, or discard this patch.