Completed
Push — master ( 023f04...fecf0e )
by Roy
12:15
created
includes/abstracts/abstract-wc-stripe-payment-gateway.php 1 patch
Spacing   +220 added lines, -221 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
 
@@ -21,7 +21,7 @@  discard block
 block discarded – undo
21 21
 	 * @since 4.0.5
22 22
 	 * @param array $error
23 23
 	 */
24
-	public function is_retryable_error( $error ) {
24
+	public function is_retryable_error($error) {
25 25
 		return (
26 26
 			'invalid_request_error' === $error->type ||
27 27
 			'idempotency_error' === $error->type ||
@@ -35,11 +35,11 @@  discard block
 block discarded – undo
35 35
 	 * Check if this gateway is enabled
36 36
 	 */
37 37
 	public function is_available() {
38
-		if ( 'yes' === $this->enabled ) {
39
-			if ( ! $this->testmode && is_checkout() && ! is_ssl() ) {
38
+		if ('yes' === $this->enabled) {
39
+			if ( ! $this->testmode && is_checkout() && ! is_ssl()) {
40 40
 				return false;
41 41
 			}
42
-			if ( ! $this->secret_key || ! $this->publishable_key ) {
42
+			if ( ! $this->secret_key || ! $this->publishable_key) {
43 43
 				return false;
44 44
 			}
45 45
 			return true;
@@ -54,8 +54,8 @@  discard block
 block discarded – undo
54 54
 	 * @since 4.0.0
55 55
 	 * @version 4.0.0
56 56
 	 */
57
-	public function add_admin_notice( $slug, $class, $message ) {
58
-		$this->notices[ $slug ] = array(
57
+	public function add_admin_notice($slug, $class, $message) {
58
+		$this->notices[$slug] = array(
59 59
 			'class'   => $class,
60 60
 			'message' => $message,
61 61
 		);
@@ -68,8 +68,8 @@  discard block
 block discarded – undo
68 68
 	 * @version 4.0.0
69 69
 	 */
70 70
 	public function remove_admin_notice() {
71
-		if ( did_action( 'woocommerce_update_options' ) ) {
72
-			remove_action( 'admin_notices', array( $this, 'check_environment' ) );
71
+		if (did_action('woocommerce_update_options')) {
72
+			remove_action('admin_notices', array($this, 'check_environment'));
73 73
 		}
74 74
 	}
75 75
 
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
 	 * @return array
82 82
 	 */
83 83
 	public function payment_icons() {
84
-		return apply_filters( 'wc_stripe_payment_icons', array(
84
+		return apply_filters('wc_stripe_payment_icons', array(
85 85
 			'visa'       => '<i class="stripe-pf stripe-pf-visa stripe-pf-right" alt="Visa" aria-hidden="true"></i>',
86 86
 			'amex'       => '<i class="stripe-pf stripe-pf-american-express stripe-pf-right" alt="Amex" aria-hidden="true"></i>',
87 87
 			'mastercard' => '<i class="stripe-pf stripe-pf-mastercard stripe-pf-right" alt="Mastercard" aria-hidden="true"></i>',
@@ -98,7 +98,7 @@  discard block
 block discarded – undo
98 98
 			'eps'        => '<i class="stripe-pf stripe-pf-eps stripe-pf-right" alt="EPS" aria-hidden="true"></i>',
99 99
 			'sofort'     => '<i class="stripe-pf stripe-pf-sofort stripe-pf-right" alt="SOFORT" aria-hidden="true"></i>',
100 100
 			'sepa'       => '<i class="stripe-pf stripe-pf-sepa stripe-pf-right" alt="SEPA" aria-hidden="true"></i>',
101
-		) );
101
+		));
102 102
 	}
103 103
 
104 104
 	/**
@@ -109,10 +109,10 @@  discard block
 block discarded – undo
109 109
 	 * @version 4.0.0
110 110
 	 * @param object $order
111 111
 	 */
112
-	public function validate_minimum_order_amount( $order ) {
113
-		if ( $order->get_total() * 100 < WC_Stripe_Helper::get_minimum_amount() ) {
112
+	public function validate_minimum_order_amount($order) {
113
+		if ($order->get_total() * 100 < WC_Stripe_Helper::get_minimum_amount()) {
114 114
 			/* translators: 1) dollar amount */
115
-			throw new WC_Stripe_Exception( 'Did not meet minimum amount', sprintf( __( 'Sorry, the minimum allowed order total is %1$s to use this payment method.', 'woocommerce-gateway-stripe' ), wc_price( WC_Stripe_Helper::get_minimum_amount() / 100 ) ) );
115
+			throw new WC_Stripe_Exception('Did not meet minimum amount', sprintf(__('Sorry, the minimum allowed order total is %1$s to use this payment method.', 'woocommerce-gateway-stripe'), wc_price(WC_Stripe_Helper::get_minimum_amount() / 100)));
116 116
 		}
117 117
 	}
118 118
 
@@ -122,14 +122,14 @@  discard block
 block discarded – undo
122 122
 	 * @since 4.0.0
123 123
 	 * @version 4.0.0
124 124
 	 */
125
-	public function get_transaction_url( $order ) {
126
-		if ( $this->testmode ) {
125
+	public function get_transaction_url($order) {
126
+		if ($this->testmode) {
127 127
 			$this->view_transaction_url = 'https://dashboard.stripe.com/test/payments/%s';
128 128
 		} else {
129 129
 			$this->view_transaction_url = 'https://dashboard.stripe.com/payments/%s';
130 130
 		}
131 131
 
132
-		return parent::get_transaction_url( $order );
132
+		return parent::get_transaction_url($order);
133 133
 	}
134 134
 
135 135
 	/**
@@ -138,15 +138,15 @@  discard block
 block discarded – undo
138 138
 	 * @since 4.0.0
139 139
 	 * @version 4.0.0
140 140
 	 */
141
-	public function get_stripe_customer_id( $order ) {
142
-		$customer = get_user_meta( WC_Stripe_Helper::is_pre_30() ? $order->customer_user : $order->get_customer_id(), '_stripe_customer_id', true );
141
+	public function get_stripe_customer_id($order) {
142
+		$customer = get_user_meta(WC_Stripe_Helper::is_pre_30() ? $order->customer_user : $order->get_customer_id(), '_stripe_customer_id', true);
143 143
 
144
-		if ( empty( $customer ) ) {
144
+		if (empty($customer)) {
145 145
 			// Try to get it via the order.
146
-			if ( WC_Stripe_Helper::is_pre_30() ) {
147
-				return get_post_meta( $order->id, '_stripe_customer_id', true );
146
+			if (WC_Stripe_Helper::is_pre_30()) {
147
+				return get_post_meta($order->id, '_stripe_customer_id', true);
148 148
 			} else {
149
-				return $order->get_meta( '_stripe_customer_id', true );
149
+				return $order->get_meta('_stripe_customer_id', true);
150 150
 			}
151 151
 		} else {
152 152
 			return $customer;
@@ -163,9 +163,9 @@  discard block
 block discarded – undo
163 163
 	 * @param object $order
164 164
 	 * @param int $id Stripe session id.
165 165
 	 */
166
-	public function get_stripe_return_url( $order = null, $id = null ) {
167
-		if ( is_object( $order ) ) {
168
-			if ( empty( $id ) ) {
166
+	public function get_stripe_return_url($order = null, $id = null) {
167
+		if (is_object($order)) {
168
+			if (empty($id)) {
169 169
 				$id = uniqid();
170 170
 			}
171 171
 
@@ -176,10 +176,10 @@  discard block
 block discarded – undo
176 176
 				'order_id'       => $order_id,
177 177
 			);
178 178
 
179
-			return esc_url_raw( add_query_arg( $args, $this->get_return_url( $order ) ) );
179
+			return esc_url_raw(add_query_arg($args, $this->get_return_url($order)));
180 180
 		}
181 181
 
182
-		return esc_url_raw( add_query_arg( array( 'utm_nooverride' => '1' ), $this->get_return_url() ) );
182
+		return esc_url_raw(add_query_arg(array('utm_nooverride' => '1'), $this->get_return_url()));
183 183
 	}
184 184
 
185 185
 	/**
@@ -187,8 +187,8 @@  discard block
 block discarded – undo
187 187
 	 * @param  int  $order_id
188 188
 	 * @return boolean
189 189
 	 */
190
-	public function has_subscription( $order_id ) {
191
-		return ( function_exists( 'wcs_order_contains_subscription' ) && ( wcs_order_contains_subscription( $order_id ) || wcs_is_subscription( $order_id ) || wcs_order_contains_renewal( $order_id ) ) );
190
+	public function has_subscription($order_id) {
191
+		return (function_exists('wcs_order_contains_subscription') && (wcs_order_contains_subscription($order_id) || wcs_is_subscription($order_id) || wcs_order_contains_renewal($order_id)));
192 192
 	}
193 193
 
194 194
 	/**
@@ -200,34 +200,33 @@  discard block
 block discarded – undo
200 200
 	 * @param  object $prepared_source
201 201
 	 * @return array()
202 202
 	 */
203
-	public function generate_payment_request( $order, $prepared_source ) {
204
-		$settings                          = get_option( 'woocommerce_stripe_settings', array() );
205
-		$statement_descriptor              = ! empty( $settings['statement_descriptor'] ) ? str_replace( "'", '', $settings['statement_descriptor'] ) : '';
206
-		$capture                           = ! empty( $settings['capture'] ) && 'yes' === $settings['capture'] ? true : false;
203
+	public function generate_payment_request($order, $prepared_source) {
204
+		$settings                          = get_option('woocommerce_stripe_settings', array());
205
+		$statement_descriptor              = ! empty($settings['statement_descriptor']) ? str_replace("'", '', $settings['statement_descriptor']) : '';
206
+		$capture                           = ! empty($settings['capture']) && 'yes' === $settings['capture'] ? true : false;
207 207
 		$post_data                         = array();
208
-		$post_data['currency']             = strtolower( WC_Stripe_Helper::is_pre_30() ? $order->get_order_currency() : $order->get_currency() );
209
-		$post_data['amount']               = WC_Stripe_Helper::get_stripe_amount( $order->get_total(), $post_data['currency'] );
208
+		$post_data['currency']             = strtolower(WC_Stripe_Helper::is_pre_30() ? $order->get_order_currency() : $order->get_currency());
209
+		$post_data['amount']               = WC_Stripe_Helper::get_stripe_amount($order->get_total(), $post_data['currency']);
210 210
 		/* translators: 1) blog name 2) order number */
211
-		$post_data['description']          = sprintf( __( '%1$s - Order %2$s', 'woocommerce-gateway-stripe' ), wp_specialchars_decode( get_bloginfo( 'name' ), ENT_QUOTES ), $order->get_order_number() );
211
+		$post_data['description']          = sprintf(__('%1$s - Order %2$s', 'woocommerce-gateway-stripe'), wp_specialchars_decode(get_bloginfo('name'), ENT_QUOTES), $order->get_order_number());
212 212
 		$billing_email      = WC_Stripe_Helper::is_pre_30() ? $order->billing_email : $order->get_billing_email();
213 213
 		$billing_first_name = WC_Stripe_Helper::is_pre_30() ? $order->billing_first_name : $order->get_billing_first_name();
214 214
 		$billing_last_name  = WC_Stripe_Helper::is_pre_30() ? $order->billing_last_name : $order->get_billing_last_name();
215 215
 
216
-		if ( ! empty( $billing_email ) && apply_filters( 'wc_stripe_send_stripe_receipt', false ) ) {
216
+		if ( ! empty($billing_email) && apply_filters('wc_stripe_send_stripe_receipt', false)) {
217 217
 			$post_data['receipt_email'] = $billing_email;
218 218
 		}
219 219
 
220
-		switch ( WC_Stripe_Helper::is_pre_30() ? $order->payment_method : $order->get_payment_method() ) {
221
-			case 'stripe':
222
-				if ( ! empty( $statement_descriptor ) ) {
223
-					$post_data['statement_descriptor'] = WC_Stripe_Helper::clean_statement_descriptor( $statement_descriptor );
220
+		switch (WC_Stripe_Helper::is_pre_30() ? $order->payment_method : $order->get_payment_method()) {
221
+			case 'stripe' : if ( ! empty($statement_descriptor)) {
222
+					$post_data['statement_descriptor'] = WC_Stripe_Helper::clean_statement_descriptor($statement_descriptor);
224 223
 				}
225 224
 
226 225
 				$post_data['capture'] = $capture ? 'true' : 'false';
227 226
 				break;
228 227
 			case 'stripe_sepa':
229
-				if ( ! empty( $statement_descriptor ) ) {
230
-					$post_data['statement_descriptor'] = WC_Stripe_Helper::clean_statement_descriptor( $statement_descriptor );
228
+				if ( ! empty($statement_descriptor)) {
229
+					$post_data['statement_descriptor'] = WC_Stripe_Helper::clean_statement_descriptor($statement_descriptor);
231 230
 				}
232 231
 				break;
233 232
 		}
@@ -235,25 +234,25 @@  discard block
 block discarded – undo
235 234
 		$post_data['expand[]'] = 'balance_transaction';
236 235
 
237 236
 		$metadata = array(
238
-			__( 'customer_name', 'woocommerce-gateway-stripe' ) => sanitize_text_field( $billing_first_name ) . ' ' . sanitize_text_field( $billing_last_name ),
239
-			__( 'customer_email', 'woocommerce-gateway-stripe' ) => sanitize_email( $billing_email ),
237
+			__('customer_name', 'woocommerce-gateway-stripe') => sanitize_text_field($billing_first_name) . ' ' . sanitize_text_field($billing_last_name),
238
+			__('customer_email', 'woocommerce-gateway-stripe') => sanitize_email($billing_email),
240 239
 			'order_id' => WC_Stripe_Helper::is_pre_30() ? $order->id : $order->get_id(),
241 240
 		);
242 241
 
243
-		if ( $this->has_subscription( WC_Stripe_Helper::is_pre_30() ? $order->id : $order->get_id() ) ) {
242
+		if ($this->has_subscription(WC_Stripe_Helper::is_pre_30() ? $order->id : $order->get_id())) {
244 243
 			$metadata += array(
245 244
 				'payment_type' => 'recurring',
246
-				'site_url'     => esc_url( get_site_url() ),
245
+				'site_url'     => esc_url(get_site_url()),
247 246
 			);
248 247
 		}
249 248
 
250
-		$post_data['metadata'] = apply_filters( 'wc_stripe_payment_metadata', $metadata, $order, $prepared_source );
249
+		$post_data['metadata'] = apply_filters('wc_stripe_payment_metadata', $metadata, $order, $prepared_source);
251 250
 
252
-		if ( $prepared_source->customer ) {
251
+		if ($prepared_source->customer) {
253 252
 			$post_data['customer'] = $prepared_source->customer;
254 253
 		}
255 254
 
256
-		if ( $prepared_source->source ) {
255
+		if ($prepared_source->source) {
257 256
 			$post_data['source'] = $prepared_source->source;
258 257
 		}
259 258
 
@@ -265,79 +264,79 @@  discard block
 block discarded – undo
265 264
 		 * @param WC_Order $order
266 265
 		 * @param object $source
267 266
 		 */
268
-		return apply_filters( 'wc_stripe_generate_payment_request', $post_data, $order, $prepared_source );
267
+		return apply_filters('wc_stripe_generate_payment_request', $post_data, $order, $prepared_source);
269 268
 	}
270 269
 
271 270
 	/**
272 271
 	 * Store extra meta data for an order from a Stripe Response.
273 272
 	 */
274
-	public function process_response( $response, $order ) {
275
-		WC_Stripe_Logger::log( 'Processing response: ' . print_r( $response, true ) );
273
+	public function process_response($response, $order) {
274
+		WC_Stripe_Logger::log('Processing response: ' . print_r($response, true));
276 275
 
277 276
 		$order_id = WC_Stripe_Helper::is_pre_30() ? $order->id : $order->get_id();
278 277
 
279
-		$captured = ( isset( $response->captured ) && $response->captured ) ? 'yes' : 'no';
278
+		$captured = (isset($response->captured) && $response->captured) ? 'yes' : 'no';
280 279
 
281 280
 		// Store charge data.
282
-		WC_Stripe_Helper::is_pre_30() ? update_post_meta( $order_id, '_stripe_charge_captured', $captured ) : $order->update_meta_data( '_stripe_charge_captured', $captured );
281
+		WC_Stripe_Helper::is_pre_30() ? update_post_meta($order_id, '_stripe_charge_captured', $captured) : $order->update_meta_data('_stripe_charge_captured', $captured);
283 282
 
284 283
 		// Store other data such as fees.
285
-		if ( isset( $response->balance_transaction ) && isset( $response->balance_transaction->fee ) ) {
284
+		if (isset($response->balance_transaction) && isset($response->balance_transaction->fee)) {
286 285
 			// Fees and Net needs to both come from Stripe to be accurate as the returned
287 286
 			// values are in the local currency of the Stripe account, not from WC.
288
-			$fee = ! empty( $response->balance_transaction->fee ) ? WC_Stripe_Helper::format_balance_fee( $response->balance_transaction, 'fee' ) : 0;
289
-			$net = ! empty( $response->balance_transaction->net ) ? WC_Stripe_Helper::format_balance_fee( $response->balance_transaction, 'net' ) : 0;
290
-			WC_Stripe_Helper::is_pre_30() ? update_post_meta( $order_id, self::META_NAME_FEE, $fee ) : $order->update_meta_data( self::META_NAME_FEE, $fee );
291
-			WC_Stripe_Helper::is_pre_30() ? update_post_meta( $order_id, self::META_NAME_NET, $net ) : $order->update_meta_data( self::META_NAME_NET, $net );
287
+			$fee = ! empty($response->balance_transaction->fee) ? WC_Stripe_Helper::format_balance_fee($response->balance_transaction, 'fee') : 0;
288
+			$net = ! empty($response->balance_transaction->net) ? WC_Stripe_Helper::format_balance_fee($response->balance_transaction, 'net') : 0;
289
+			WC_Stripe_Helper::is_pre_30() ? update_post_meta($order_id, self::META_NAME_FEE, $fee) : $order->update_meta_data(self::META_NAME_FEE, $fee);
290
+			WC_Stripe_Helper::is_pre_30() ? update_post_meta($order_id, self::META_NAME_NET, $net) : $order->update_meta_data(self::META_NAME_NET, $net);
292 291
 		}
293 292
 
294
-		if ( 'yes' === $captured ) {
293
+		if ('yes' === $captured) {
295 294
 			/**
296 295
 			 * Charge can be captured but in a pending state. Payment methods
297 296
 			 * that are asynchronous may take couple days to clear. Webhook will
298 297
 			 * take care of the status changes.
299 298
 			 */
300
-			if ( 'pending' === $response->status ) {
301
-				$order_stock_reduced = WC_Stripe_Helper::is_pre_30() ? get_post_meta( $order_id, '_order_stock_reduced', true ) : $order->get_meta( '_order_stock_reduced', true );
299
+			if ('pending' === $response->status) {
300
+				$order_stock_reduced = WC_Stripe_Helper::is_pre_30() ? get_post_meta($order_id, '_order_stock_reduced', true) : $order->get_meta('_order_stock_reduced', true);
302 301
 
303
-				if ( ! $order_stock_reduced ) {
304
-					WC_Stripe_Helper::is_pre_30() ? $order->reduce_order_stock() : wc_reduce_stock_levels( $order_id );
302
+				if ( ! $order_stock_reduced) {
303
+					WC_Stripe_Helper::is_pre_30() ? $order->reduce_order_stock() : wc_reduce_stock_levels($order_id);
305 304
 				}
306 305
 
307
-				WC_Stripe_Helper::is_pre_30() ? update_post_meta( $order_id, '_transaction_id', $response->id ) : $order->set_transaction_id( $response->id );
306
+				WC_Stripe_Helper::is_pre_30() ? update_post_meta($order_id, '_transaction_id', $response->id) : $order->set_transaction_id($response->id);
308 307
 				/* translators: transaction id */
309
-				$order->update_status( 'on-hold', sprintf( __( 'Stripe charge awaiting payment: %s.', 'woocommerce-gateway-stripe' ), $response->id ) );
308
+				$order->update_status('on-hold', sprintf(__('Stripe charge awaiting payment: %s.', 'woocommerce-gateway-stripe'), $response->id));
310 309
 			}
311 310
 
312
-			if ( 'succeeded' === $response->status ) {
313
-				$order->payment_complete( $response->id );
311
+			if ('succeeded' === $response->status) {
312
+				$order->payment_complete($response->id);
314 313
 
315 314
 				/* translators: transaction id */
316
-				$message = sprintf( __( 'Stripe charge complete (Charge ID: %s)', 'woocommerce-gateway-stripe' ), $response->id );
317
-				$order->add_order_note( $message );
315
+				$message = sprintf(__('Stripe charge complete (Charge ID: %s)', 'woocommerce-gateway-stripe'), $response->id);
316
+				$order->add_order_note($message);
318 317
 			}
319 318
 
320
-			if ( 'failed' === $response->status ) {
321
-				$localized_message = __( 'Payment processing failed. Please retry.', 'woocommerce-gateway-stripe' );
322
-				$order->add_order_note( $localized_message );
323
-				throw new WC_Stripe_Exception( print_r( $response, true ), $localized_message );
319
+			if ('failed' === $response->status) {
320
+				$localized_message = __('Payment processing failed. Please retry.', 'woocommerce-gateway-stripe');
321
+				$order->add_order_note($localized_message);
322
+				throw new WC_Stripe_Exception(print_r($response, true), $localized_message);
324 323
 			}
325 324
 		} else {
326
-			WC_Stripe_Helper::is_pre_30() ? update_post_meta( $order_id, '_transaction_id', $response->id ) : $order->set_transaction_id( $response->id );
325
+			WC_Stripe_Helper::is_pre_30() ? update_post_meta($order_id, '_transaction_id', $response->id) : $order->set_transaction_id($response->id);
327 326
 
328
-			if ( $order->has_status( array( 'pending', 'failed' ) ) ) {
329
-				WC_Stripe_Helper::is_pre_30() ? $order->reduce_order_stock() : wc_reduce_stock_levels( $order_id );
327
+			if ($order->has_status(array('pending', 'failed'))) {
328
+				WC_Stripe_Helper::is_pre_30() ? $order->reduce_order_stock() : wc_reduce_stock_levels($order_id);
330 329
 			}
331 330
 
332 331
 			/* translators: transaction id */
333
-			$order->update_status( 'on-hold', sprintf( __( 'Stripe charge authorized (Charge ID: %s). Process order to take payment, or cancel to remove the pre-authorization.', 'woocommerce-gateway-stripe' ), $response->id ) );
332
+			$order->update_status('on-hold', sprintf(__('Stripe charge authorized (Charge ID: %s). Process order to take payment, or cancel to remove the pre-authorization.', 'woocommerce-gateway-stripe'), $response->id));
334 333
 		}
335 334
 
336
-		if ( is_callable( array( $order, 'save' ) ) ) {
335
+		if (is_callable(array($order, 'save'))) {
337 336
 			$order->save();
338 337
 		}
339 338
 
340
-		do_action( 'wc_gateway_stripe_process_response', $response, $order );
339
+		do_action('wc_gateway_stripe_process_response', $response, $order);
341 340
 
342 341
 		return $response;
343 342
 	}
@@ -350,10 +349,10 @@  discard block
 block discarded – undo
350 349
 	 * @param int $order_id
351 350
 	 * @return null
352 351
 	 */
353
-	public function send_failed_order_email( $order_id ) {
352
+	public function send_failed_order_email($order_id) {
354 353
 		$emails = WC()->mailer()->get_emails();
355
-		if ( ! empty( $emails ) && ! empty( $order_id ) ) {
356
-			$emails['WC_Email_Failed_Order']->trigger( $order_id );
354
+		if ( ! empty($emails) && ! empty($order_id)) {
355
+			$emails['WC_Email_Failed_Order']->trigger($order_id);
357 356
 		}
358 357
 	}
359 358
 
@@ -365,7 +364,7 @@  discard block
 block discarded – undo
365 364
 	 * @param object $order
366 365
 	 * @return object $details
367 366
 	 */
368
-	public function get_owner_details( $order ) {
367
+	public function get_owner_details($order) {
369 368
 		$billing_first_name = WC_Stripe_Helper::is_pre_30() ? $order->billing_first_name : $order->get_billing_first_name();
370 369
 		$billing_last_name  = WC_Stripe_Helper::is_pre_30() ? $order->billing_last_name : $order->get_billing_last_name();
371 370
 
@@ -376,8 +375,8 @@  discard block
 block discarded – undo
376 375
 
377 376
 		$phone                             = WC_Stripe_Helper::is_pre_30() ? $order->billing_phone : $order->get_billing_phone();
378 377
 
379
-		if ( ! empty( $phone ) ) {
380
-			$details['phone']              = $phone;
378
+		if ( ! empty($phone)) {
379
+			$details['phone'] = $phone;
381 380
 		}
382 381
 
383 382
 		$details['address']['line1']       = WC_Stripe_Helper::is_pre_30() ? $order->billing_address_1 : $order->get_billing_address_1();
@@ -387,7 +386,7 @@  discard block
 block discarded – undo
387 386
 		$details['address']['postal_code'] = WC_Stripe_Helper::is_pre_30() ? $order->billing_postcode : $order->get_billing_postcode();
388 387
 		$details['address']['country']     = WC_Stripe_Helper::is_pre_30() ? $order->billing_country : $order->get_billing_country();
389 388
 
390
-		return (object) apply_filters( 'wc_stripe_owner_details', $details, $order );
389
+		return (object) apply_filters('wc_stripe_owner_details', $details, $order);
391 390
 	}
392 391
 
393 392
 	/**
@@ -396,15 +395,15 @@  discard block
 block discarded – undo
396 395
 	 * @since 4.0.3
397 396
 	 * @param string $source_id The source ID to get source object for.
398 397
 	 */
399
-	public function get_source_object( $source_id = '' ) {
400
-		if ( empty( $source_id ) ) {
398
+	public function get_source_object($source_id = '') {
399
+		if (empty($source_id)) {
401 400
 			return '';
402 401
 		}
403 402
 
404
-		$source_object = WC_Stripe_API::retrieve( 'sources/' . $source_id );
403
+		$source_object = WC_Stripe_API::retrieve('sources/' . $source_id);
405 404
 
406
-		if ( ! empty( $source_object->error ) ) {
407
-			throw new WC_Stripe_Exception( print_r( $source_object, true ), $source_object->error->message );
405
+		if ( ! empty($source_object->error)) {
406
+			throw new WC_Stripe_Exception(print_r($source_object, true), $source_object->error->message);
408 407
 		}
409 408
 
410 409
 		return $source_object;
@@ -417,11 +416,11 @@  discard block
 block discarded – undo
417 416
 	 * @param object $source_object
418 417
 	 * @return bool
419 418
 	 */
420
-	public function is_3ds_required( $source_object ) {
419
+	public function is_3ds_required($source_object) {
421 420
 		return (
422
-			$source_object && ! empty( $source_object->card ) ) &&
423
-			( 'card' === $source_object->type && 'required' === $source_object->card->three_d_secure ||
424
-			( $this->three_d_secure && 'optional' === $source_object->card->three_d_secure )
421
+			$source_object && ! empty($source_object->card) ) &&
422
+			('card' === $source_object->type && 'required' === $source_object->card->three_d_secure ||
423
+			($this->three_d_secure && 'optional' === $source_object->card->three_d_secure)
425 424
 		);
426 425
 	}
427 426
 
@@ -432,8 +431,8 @@  discard block
 block discarded – undo
432 431
 	 * @param object $source_object
433 432
 	 * @return bool
434 433
 	 */
435
-	public function is_3ds_card( $source_object ) {
436
-		return ( $source_object && 'three_d_secure' === $source_object->type );
434
+	public function is_3ds_card($source_object) {
435
+		return ($source_object && 'three_d_secure' === $source_object->type);
437 436
 	}
438 437
 
439 438
 	/**
@@ -443,8 +442,8 @@  discard block
 block discarded – undo
443 442
 	 * @param object $source_object
444 443
 	 * @return bool
445 444
 	 */
446
-	public function is_prepaid_card( $source_object ) {
447
-		return ( $source_object && 'token' === $source_object->object && 'prepaid' === $source_object->card->funding );
445
+	public function is_prepaid_card($source_object) {
446
+		return ($source_object && 'token' === $source_object->object && 'prepaid' === $source_object->card->funding);
448 447
 	}
449 448
 
450 449
 	/**
@@ -457,22 +456,22 @@  discard block
 block discarded – undo
457 456
 	 * @param string $return_url
458 457
 	 * @return mixed
459 458
 	 */
460
-	public function create_3ds_source( $order, $source_object, $return_url = '' ) {
459
+	public function create_3ds_source($order, $source_object, $return_url = '') {
461 460
 		$currency                    = WC_Stripe_Helper::is_pre_30() ? $order->get_order_currency() : $order->get_currency();
462 461
 		$order_id                    = WC_Stripe_Helper::is_pre_30() ? $order->id : $order->get_id();
463
-		$return_url                  = empty( $return_url ) ? $this->get_stripe_return_url( $order ) : $return_url;
462
+		$return_url                  = empty($return_url) ? $this->get_stripe_return_url($order) : $return_url;
464 463
 
465 464
 		$post_data                   = array();
466
-		$post_data['amount']         = WC_Stripe_Helper::get_stripe_amount( $order->get_total(), $currency );
467
-		$post_data['currency']       = strtolower( $currency );
465
+		$post_data['amount']         = WC_Stripe_Helper::get_stripe_amount($order->get_total(), $currency);
466
+		$post_data['currency']       = strtolower($currency);
468 467
 		$post_data['type']           = 'three_d_secure';
469
-		$post_data['owner']          = $this->get_owner_details( $order );
470
-		$post_data['three_d_secure'] = array( 'card' => $source_object->id );
471
-		$post_data['redirect']       = array( 'return_url' => $return_url );
468
+		$post_data['owner']          = $this->get_owner_details($order);
469
+		$post_data['three_d_secure'] = array('card' => $source_object->id);
470
+		$post_data['redirect']       = array('return_url' => $return_url);
472 471
 
473
-		WC_Stripe_Logger::log( 'Info: Begin creating 3DS source...' );
472
+		WC_Stripe_Logger::log('Info: Begin creating 3DS source...');
474 473
 
475
-		return WC_Stripe_API::request( apply_filters( 'wc_stripe_3ds_source', $post_data, $order ), 'sources' );
474
+		return WC_Stripe_API::request(apply_filters('wc_stripe_3ds_source', $post_data, $order), 'sources');
476 475
 	}
477 476
 
478 477
 	/**
@@ -488,56 +487,56 @@  discard block
 block discarded – undo
488 487
 	 * @throws Exception When card was not added or for and invalid card.
489 488
 	 * @return object
490 489
 	 */
491
-	public function prepare_source( $user_id, $force_save_source = false ) {
492
-		$customer           = new WC_Stripe_Customer( $user_id );
490
+	public function prepare_source($user_id, $force_save_source = false) {
491
+		$customer           = new WC_Stripe_Customer($user_id);
493 492
 		$set_customer       = true;
494
-		$force_save_source  = apply_filters( 'wc_stripe_force_save_source', $force_save_source, $customer );
493
+		$force_save_source  = apply_filters('wc_stripe_force_save_source', $force_save_source, $customer);
495 494
 		$source_object      = '';
496 495
 		$source_id          = '';
497 496
 		$wc_token_id        = false;
498
-		$payment_method     = isset( $_POST['payment_method'] ) ? wc_clean( $_POST['payment_method'] ) : 'stripe';
497
+		$payment_method     = isset($_POST['payment_method']) ? wc_clean($_POST['payment_method']) : 'stripe';
499 498
 
500 499
 		// New CC info was entered and we have a new source to process.
501
-		if ( ! empty( $_POST['stripe_source'] ) ) {
502
-			$source_object = self::get_source_object( wc_clean( $_POST['stripe_source'] ) );
500
+		if ( ! empty($_POST['stripe_source'])) {
501
+			$source_object = self::get_source_object(wc_clean($_POST['stripe_source']));
503 502
 			$source_id     = $source_object->id;
504 503
 
505 504
 			// This checks to see if customer opted to save the payment method to file.
506
-			$maybe_saved_card = isset( $_POST[ 'wc-' . $payment_method . '-new-payment-method' ] ) && ! empty( $_POST[ 'wc-' . $payment_method . '-new-payment-method' ] );
505
+			$maybe_saved_card = isset($_POST['wc-' . $payment_method . '-new-payment-method']) && ! empty($_POST['wc-' . $payment_method . '-new-payment-method']);
507 506
 
508 507
 			/**
509 508
 			 * This is true if the user wants to store the card to their account.
510 509
 			 * Criteria to save to file is they are logged in, they opted to save or product requirements and the source is
511 510
 			 * actually reusable. Either that or force_save_source is true.
512 511
 			 */
513
-			if ( ( $user_id && $this->saved_cards && $maybe_saved_card && 'reusable' === $source_object->usage ) || $force_save_source ) {
514
-				$response = $customer->add_source( $source_object->id );
512
+			if (($user_id && $this->saved_cards && $maybe_saved_card && 'reusable' === $source_object->usage) || $force_save_source) {
513
+				$response = $customer->add_source($source_object->id);
515 514
 
516
-				if ( ! empty( $response->error ) ) {
517
-					throw new WC_Stripe_Exception( print_r( $response, true ), $response->error->message );
515
+				if ( ! empty($response->error)) {
516
+					throw new WC_Stripe_Exception(print_r($response, true), $response->error->message);
518 517
 				}
519 518
 			}
520
-		} elseif ( isset( $_POST[ 'wc-' . $payment_method . '-payment-token' ] ) && 'new' !== $_POST[ 'wc-' . $payment_method . '-payment-token' ] ) {
519
+		} elseif (isset($_POST['wc-' . $payment_method . '-payment-token']) && 'new' !== $_POST['wc-' . $payment_method . '-payment-token']) {
521 520
 			// Use an existing token, and then process the payment.
522
-			$wc_token_id = wc_clean( $_POST[ 'wc-' . $payment_method . '-payment-token' ] );
523
-			$wc_token    = WC_Payment_Tokens::get( $wc_token_id );
521
+			$wc_token_id = wc_clean($_POST['wc-' . $payment_method . '-payment-token']);
522
+			$wc_token    = WC_Payment_Tokens::get($wc_token_id);
524 523
 
525
-			if ( ! $wc_token || $wc_token->get_user_id() !== get_current_user_id() ) {
526
-				WC()->session->set( 'refresh_totals', true );
527
-				throw new WC_Stripe_Exception( 'Invalid payment method', __( 'Invalid payment method. Please input a new card number.', 'woocommerce-gateway-stripe' ) );
524
+			if ( ! $wc_token || $wc_token->get_user_id() !== get_current_user_id()) {
525
+				WC()->session->set('refresh_totals', true);
526
+				throw new WC_Stripe_Exception('Invalid payment method', __('Invalid payment method. Please input a new card number.', 'woocommerce-gateway-stripe'));
528 527
 			}
529 528
 
530 529
 			$source_id = $wc_token->get_token();
531
-		} elseif ( isset( $_POST['stripe_token'] ) && 'new' !== $_POST['stripe_token'] ) {
532
-			$stripe_token     = wc_clean( $_POST['stripe_token'] );
533
-			$maybe_saved_card = isset( $_POST[ 'wc-' . $payment_method . '-new-payment-method' ] ) && ! empty( $_POST[ 'wc-' . $payment_method . '-new-payment-method' ] );
530
+		} elseif (isset($_POST['stripe_token']) && 'new' !== $_POST['stripe_token']) {
531
+			$stripe_token     = wc_clean($_POST['stripe_token']);
532
+			$maybe_saved_card = isset($_POST['wc-' . $payment_method . '-new-payment-method']) && ! empty($_POST['wc-' . $payment_method . '-new-payment-method']);
534 533
 
535 534
 			// This is true if the user wants to store the card to their account.
536
-			if ( ( $user_id && $this->saved_cards && $maybe_saved_card ) || $force_save_source ) {
537
-				$response = $customer->add_source( $stripe_token );
535
+			if (($user_id && $this->saved_cards && $maybe_saved_card) || $force_save_source) {
536
+				$response = $customer->add_source($stripe_token);
538 537
 
539
-				if ( ! empty( $response->error ) ) {
540
-					throw new WC_Stripe_Exception( print_r( $response, true ), $response->error->message );
538
+				if ( ! empty($response->error)) {
539
+					throw new WC_Stripe_Exception(print_r($response, true), $response->error->message);
541 540
 				}
542 541
 			} else {
543 542
 				$set_customer = false;
@@ -545,14 +544,14 @@  discard block
 block discarded – undo
545 544
 			}
546 545
 		}
547 546
 
548
-		if ( ! $set_customer ) {
547
+		if ( ! $set_customer) {
549 548
 			$customer_id = false;
550 549
 		} else {
551 550
 			$customer_id = $customer->get_id() ? $customer->get_id() : false;
552 551
 		}
553 552
 
554
-		if ( empty( $source_object ) ) {
555
-			$source_object = self::get_source_object( $source_id );
553
+		if (empty($source_object)) {
554
+			$source_object = self::get_source_object($source_id);
556 555
 		}
557 556
 
558 557
 		return (object) array(
@@ -576,37 +575,37 @@  discard block
 block discarded – undo
576 575
 	 * @param object $order
577 576
 	 * @return object
578 577
 	 */
579
-	public function prepare_order_source( $order = null ) {
578
+	public function prepare_order_source($order = null) {
580 579
 		$stripe_customer = new WC_Stripe_Customer();
581 580
 		$stripe_source   = false;
582 581
 		$token_id        = false;
583 582
 
584
-		if ( $order ) {
583
+		if ($order) {
585 584
 			$order_id = WC_Stripe_Helper::is_pre_30() ? $order->id : $order->get_id();
586 585
 
587
-			$stripe_customer_id = get_post_meta( $order_id, '_stripe_customer_id', true );
586
+			$stripe_customer_id = get_post_meta($order_id, '_stripe_customer_id', true);
588 587
 
589
-			if ( $stripe_customer_id ) {
590
-				$stripe_customer->set_id( $stripe_customer_id );
588
+			if ($stripe_customer_id) {
589
+				$stripe_customer->set_id($stripe_customer_id);
591 590
 			}
592 591
 
593
-			$source_id = WC_Stripe_Helper::is_pre_30() ? get_post_meta( $order_id, '_stripe_source_id', true ) : $order->get_meta( '_stripe_source_id', true );
592
+			$source_id = WC_Stripe_Helper::is_pre_30() ? get_post_meta($order_id, '_stripe_source_id', true) : $order->get_meta('_stripe_source_id', true);
594 593
 
595 594
 			// Since 4.0.0, we changed card to source so we need to account for that.
596
-			if ( empty( $source_id ) ) {
597
-				$source_id = WC_Stripe_Helper::is_pre_30() ? get_post_meta( $order_id, '_stripe_card_id', true ) : $order->get_meta( '_stripe_card_id', true );
595
+			if (empty($source_id)) {
596
+				$source_id = WC_Stripe_Helper::is_pre_30() ? get_post_meta($order_id, '_stripe_card_id', true) : $order->get_meta('_stripe_card_id', true);
598 597
 
599 598
 				// Take this opportunity to update the key name.
600
-				WC_Stripe_Helper::is_pre_30() ? update_post_meta( $order_id, '_stripe_source_id', $source_id ) : $order->update_meta_data( '_stripe_source_id', $source_id );
599
+				WC_Stripe_Helper::is_pre_30() ? update_post_meta($order_id, '_stripe_source_id', $source_id) : $order->update_meta_data('_stripe_source_id', $source_id);
601 600
 
602
-				if ( is_callable( array( $order, 'save' ) ) ) {
601
+				if (is_callable(array($order, 'save'))) {
603 602
 					$order->save();
604 603
 				}
605 604
 			}
606 605
 
607
-			if ( $source_id ) {
606
+			if ($source_id) {
608 607
 				$stripe_source = $source_id;
609
-			} elseif ( apply_filters( 'wc_stripe_use_default_customer_source', true ) ) {
608
+			} elseif (apply_filters('wc_stripe_use_default_customer_source', true)) {
610 609
 				/*
611 610
 				 * We can attempt to charge the customer's default source
612 611
 				 * by sending empty source id.
@@ -630,27 +629,27 @@  discard block
 block discarded – undo
630 629
 	 * @param WC_Order $order For to which the source applies.
631 630
 	 * @param stdClass $source Source information.
632 631
 	 */
633
-	public function save_source_to_order( $order, $source ) {
632
+	public function save_source_to_order($order, $source) {
634 633
 		$order_id = WC_Stripe_Helper::is_pre_30() ? $order->id : $order->get_id();
635 634
 
636 635
 		// Store source in the order.
637
-		if ( $source->customer ) {
638
-			if ( WC_Stripe_Helper::is_pre_30() ) {
639
-				update_post_meta( $order_id, '_stripe_customer_id', $source->customer );
636
+		if ($source->customer) {
637
+			if (WC_Stripe_Helper::is_pre_30()) {
638
+				update_post_meta($order_id, '_stripe_customer_id', $source->customer);
640 639
 			} else {
641
-				$order->update_meta_data( '_stripe_customer_id', $source->customer );
640
+				$order->update_meta_data('_stripe_customer_id', $source->customer);
642 641
 			}
643 642
 		}
644 643
 
645
-		if ( $source->source ) {
646
-			if ( WC_Stripe_Helper::is_pre_30() ) {
647
-				update_post_meta( $order_id, '_stripe_source_id', $source->source );
644
+		if ($source->source) {
645
+			if (WC_Stripe_Helper::is_pre_30()) {
646
+				update_post_meta($order_id, '_stripe_source_id', $source->source);
648 647
 			} else {
649
-				$order->update_meta_data( '_stripe_source_id', $source->source );
648
+				$order->update_meta_data('_stripe_source_id', $source->source);
650 649
 			}
651 650
 		}
652 651
 
653
-		if ( is_callable( array( $order, 'save' ) ) ) {
652
+		if (is_callable(array($order, 'save'))) {
654 653
 			$order->save();
655 654
 		}
656 655
 	}
@@ -664,35 +663,35 @@  discard block
 block discarded – undo
664 663
 	 * @param object $order The order object
665 664
 	 * @param int $balance_transaction_id
666 665
 	 */
667
-	public function update_fees( $order, $balance_transaction_id ) {
666
+	public function update_fees($order, $balance_transaction_id) {
668 667
 		$order_id = WC_Stripe_Helper::is_pre_30() ? $order->id : $order->get_id();
669 668
 
670
-		$balance_transaction = WC_Stripe_API::retrieve( 'balance/history/' . $balance_transaction_id );
669
+		$balance_transaction = WC_Stripe_API::retrieve('balance/history/' . $balance_transaction_id);
671 670
 
672
-		if ( empty( $balance_transaction->error ) ) {
673
-			if ( isset( $balance_transaction ) && isset( $balance_transaction->fee ) ) {
671
+		if (empty($balance_transaction->error)) {
672
+			if (isset($balance_transaction) && isset($balance_transaction->fee)) {
674 673
 				// Fees and Net needs to both come from Stripe to be accurate as the returned
675 674
 				// values are in the local currency of the Stripe account, not from WC.
676
-				$fee_refund = ! empty( $balance_transaction->fee ) ? WC_Stripe_Helper::format_balance_fee( $balance_transaction, 'fee' ) : 0;
677
-				$net_refund = ! empty( $balance_transaction->net ) ? WC_Stripe_Helper::format_balance_fee( $balance_transaction, 'net' ) : 0;
675
+				$fee_refund = ! empty($balance_transaction->fee) ? WC_Stripe_Helper::format_balance_fee($balance_transaction, 'fee') : 0;
676
+				$net_refund = ! empty($balance_transaction->net) ? WC_Stripe_Helper::format_balance_fee($balance_transaction, 'net') : 0;
678 677
 
679 678
 				// Current data fee & net.
680
-				$fee_current = WC_Stripe_Helper::is_pre_30() ? get_post_meta( $order_id, self::META_NAME_FEE, true ) : $order->get_meta( self::META_NAME_FEE, true );
681
-				$net_current = WC_Stripe_Helper::is_pre_30() ? get_post_meta( $order_id, self::META_NAME_NET, true ) : $order->get_meta( self::META_NAME_NET, true );
679
+				$fee_current = WC_Stripe_Helper::is_pre_30() ? get_post_meta($order_id, self::META_NAME_FEE, true) : $order->get_meta(self::META_NAME_FEE, true);
680
+				$net_current = WC_Stripe_Helper::is_pre_30() ? get_post_meta($order_id, self::META_NAME_NET, true) : $order->get_meta(self::META_NAME_NET, true);
682 681
 
683 682
 				// Calculation.
684 683
 				$fee = (float) $fee_current + (float) $fee_refund;
685 684
 				$net = (float) $net_current + (float) $net_refund;
686 685
 
687
-				WC_Stripe_Helper::is_pre_30() ? update_post_meta( $order_id, self::META_NAME_FEE, $fee ) : $order->update_meta_data( self::META_NAME_FEE, $fee );
688
-				WC_Stripe_Helper::is_pre_30() ? update_post_meta( $order_id, self::META_NAME_NET, $net ) : $order->update_meta_data( self::META_NAME_NET, $net );
686
+				WC_Stripe_Helper::is_pre_30() ? update_post_meta($order_id, self::META_NAME_FEE, $fee) : $order->update_meta_data(self::META_NAME_FEE, $fee);
687
+				WC_Stripe_Helper::is_pre_30() ? update_post_meta($order_id, self::META_NAME_NET, $net) : $order->update_meta_data(self::META_NAME_NET, $net);
689 688
 
690
-				if ( is_callable( array( $order, 'save' ) ) ) {
689
+				if (is_callable(array($order, 'save'))) {
691 690
 					$order->save();
692 691
 				}
693 692
 			}
694 693
 		} else {
695
-			WC_Stripe_Logger::log( "Unable to update fees/net meta for order: {$order_id}" );
694
+			WC_Stripe_Logger::log("Unable to update fees/net meta for order: {$order_id}");
696 695
 		}
697 696
 	}
698 697
 
@@ -705,33 +704,33 @@  discard block
 block discarded – undo
705 704
 	 * @param  float $amount
706 705
 	 * @return bool
707 706
 	 */
708
-	public function process_refund( $order_id, $amount = null, $reason = '' ) {
709
-		$order = wc_get_order( $order_id );
707
+	public function process_refund($order_id, $amount = null, $reason = '') {
708
+		$order = wc_get_order($order_id);
710 709
 
711
-		if ( ! $order || ! $order->get_transaction_id() ) {
710
+		if ( ! $order || ! $order->get_transaction_id()) {
712 711
 			return false;
713 712
 		}
714 713
 
715 714
 		$request = array();
716 715
 
717
-		if ( WC_Stripe_Helper::is_pre_30() ) {
718
-			$order_currency = get_post_meta( $order_id, '_order_currency', true );
719
-			$captured       = get_post_meta( $order_id, '_stripe_charge_captured', true );
716
+		if (WC_Stripe_Helper::is_pre_30()) {
717
+			$order_currency = get_post_meta($order_id, '_order_currency', true);
718
+			$captured       = get_post_meta($order_id, '_stripe_charge_captured', true);
720 719
 		} else {
721 720
 			$order_currency = $order->get_currency();
722
-			$captured       = $order->get_meta( '_stripe_charge_captured', true );
721
+			$captured       = $order->get_meta('_stripe_charge_captured', true);
723 722
 		}
724 723
 
725
-		if ( ! is_null( $amount ) ) {
726
-			$request['amount'] = WC_Stripe_Helper::get_stripe_amount( $amount, $order_currency );
724
+		if ( ! is_null($amount)) {
725
+			$request['amount'] = WC_Stripe_Helper::get_stripe_amount($amount, $order_currency);
727 726
 		}
728 727
 
729 728
 		// If order is only authorized, don't pass amount.
730
-		if ( 'yes' !== $captured ) {
731
-			unset( $request['amount'] );
729
+		if ('yes' !== $captured) {
730
+			unset($request['amount']);
732 731
 		}
733 732
 
734
-		if ( $reason ) {
733
+		if ($reason) {
735 734
 			$request['metadata'] = array(
736 735
 				'reason' => $reason,
737 736
 			);
@@ -739,35 +738,35 @@  discard block
 block discarded – undo
739 738
 
740 739
 		$request['charge'] = $order->get_transaction_id();
741 740
 
742
-		WC_Stripe_Logger::log( "Info: Beginning refund for order {$order->get_transaction_id()} for the amount of {$amount}" );
741
+		WC_Stripe_Logger::log("Info: Beginning refund for order {$order->get_transaction_id()} for the amount of {$amount}");
743 742
 
744
-		$request = apply_filters( 'wc_stripe_refund_request', $request, $order );
743
+		$request = apply_filters('wc_stripe_refund_request', $request, $order);
745 744
 
746
-		$response = WC_Stripe_API::request( $request, 'refunds' );
745
+		$response = WC_Stripe_API::request($request, 'refunds');
747 746
 
748
-		if ( ! empty( $response->error ) ) {
749
-			WC_Stripe_Logger::log( 'Error: ' . $response->error->message );
747
+		if ( ! empty($response->error)) {
748
+			WC_Stripe_Logger::log('Error: ' . $response->error->message);
750 749
 
751 750
 			return $response;
752 751
 
753
-		} elseif ( ! empty( $response->id ) ) {
754
-			WC_Stripe_Helper::is_pre_30() ? update_post_meta( $order_id, '_stripe_refund_id', $response->id ) : $order->update_meta_data( '_stripe_refund_id', $response->id );
752
+		} elseif ( ! empty($response->id)) {
753
+			WC_Stripe_Helper::is_pre_30() ? update_post_meta($order_id, '_stripe_refund_id', $response->id) : $order->update_meta_data('_stripe_refund_id', $response->id);
755 754
 
756
-			$amount = wc_price( $response->amount / 100 );
755
+			$amount = wc_price($response->amount / 100);
757 756
 
758
-			if ( in_array( strtolower( $order->get_currency() ), WC_Stripe_Helper::no_decimal_currencies() ) ) {
759
-				$amount = wc_price( $response->amount );
757
+			if (in_array(strtolower($order->get_currency()), WC_Stripe_Helper::no_decimal_currencies())) {
758
+				$amount = wc_price($response->amount);
760 759
 			}
761 760
 
762
-			if ( isset( $response->balance_transaction ) ) {
763
-				$this->update_fees( $order, $response->balance_transaction );
761
+			if (isset($response->balance_transaction)) {
762
+				$this->update_fees($order, $response->balance_transaction);
764 763
 			}
765 764
 
766 765
 			/* translators: 1) dollar amount 2) transaction id 3) refund message */
767
-			$refund_message = ( isset( $captured ) && 'yes' === $captured ) ? sprintf( __( 'Refunded %1$s - Refund ID: %2$s - Reason: %3$s', 'woocommerce-gateway-stripe' ), $amount, $response->id, $reason ) : __( 'Pre-Authorization Released', 'woocommerce-gateway-stripe' );
766
+			$refund_message = (isset($captured) && 'yes' === $captured) ? sprintf(__('Refunded %1$s - Refund ID: %2$s - Reason: %3$s', 'woocommerce-gateway-stripe'), $amount, $response->id, $reason) : __('Pre-Authorization Released', 'woocommerce-gateway-stripe');
768 767
 
769
-			$order->add_order_note( $refund_message );
770
-			WC_Stripe_Logger::log( 'Success: ' . html_entity_decode( strip_tags( $refund_message ) ) );
768
+			$order->add_order_note($refund_message);
769
+			WC_Stripe_Logger::log('Success: ' . html_entity_decode(strip_tags($refund_message)));
771 770
 
772 771
 			return true;
773 772
 		}
@@ -782,44 +781,44 @@  discard block
 block discarded – undo
782 781
 	 */
783 782
 	public function add_payment_method() {
784 783
 		$error     = false;
785
-		$error_msg = __( 'There was a problem adding the card.', 'woocommerce-gateway-stripe' );
784
+		$error_msg = __('There was a problem adding the card.', 'woocommerce-gateway-stripe');
786 785
 		$source_id = '';
787 786
 
788
-		if ( empty( $_POST['stripe_source'] ) && empty( $_POST['stripe_token'] ) || ! is_user_logged_in() ) {
787
+		if (empty($_POST['stripe_source']) && empty($_POST['stripe_token']) || ! is_user_logged_in()) {
789 788
 			$error = true;
790 789
 		}
791 790
 
792
-		$stripe_customer = new WC_Stripe_Customer( get_current_user_id() );
791
+		$stripe_customer = new WC_Stripe_Customer(get_current_user_id());
793 792
 
794
-		$source = ! empty( $_POST['stripe_source'] ) ? wc_clean( $_POST['stripe_source'] ) : '';
793
+		$source = ! empty($_POST['stripe_source']) ? wc_clean($_POST['stripe_source']) : '';
795 794
 
796
-		$source_object = WC_Stripe_API::retrieve( 'sources/' . $source );
795
+		$source_object = WC_Stripe_API::retrieve('sources/' . $source);
797 796
 
798
-		if ( isset( $source_object ) ) {
799
-			if ( ! empty( $source_object->error ) ) {
797
+		if (isset($source_object)) {
798
+			if ( ! empty($source_object->error)) {
800 799
 				$error = true;
801 800
 			}
802 801
 
803 802
 			$source_id = $source_object->id;
804
-		} elseif ( isset( $_POST['stripe_token'] ) ) {
805
-			$source_id = wc_clean( $_POST['stripe_token'] );
803
+		} elseif (isset($_POST['stripe_token'])) {
804
+			$source_id = wc_clean($_POST['stripe_token']);
806 805
 		}
807 806
 
808
-		$response = $stripe_customer->add_source( $source_id );
807
+		$response = $stripe_customer->add_source($source_id);
809 808
 
810
-		if ( ! $response || is_wp_error( $response ) || ! empty( $response->error ) ) {
809
+		if ( ! $response || is_wp_error($response) || ! empty($response->error)) {
811 810
 			$error = true;
812 811
 		}
813 812
 
814
-		if ( $error ) {
815
-			wc_add_notice( $error_msg, 'error' );
816
-			WC_Stripe_Logger::log( 'Add payment method Error: ' . $error_msg );
813
+		if ($error) {
814
+			wc_add_notice($error_msg, 'error');
815
+			WC_Stripe_Logger::log('Add payment method Error: ' . $error_msg);
817 816
 			return;
818 817
 		}
819 818
 
820 819
 		return array(
821 820
 			'result'   => 'success',
822
-			'redirect' => wc_get_endpoint_url( 'payment-methods' ),
821
+			'redirect' => wc_get_endpoint_url('payment-methods'),
823 822
 		);
824 823
 	}
825 824
 
@@ -836,10 +835,10 @@  discard block
 block discarded – undo
836 835
 		 * Stripe expects Norwegian to only be passed NO.
837 836
 		 * But WP has different dialects.
838 837
 		 */
839
-		if ( 'NO' === substr( $locale, 3, 2 ) ) {
838
+		if ('NO' === substr($locale, 3, 2)) {
840 839
 			$locale = 'no';
841 840
 		} else {
842
-			$locale = substr( get_locale(), 0, 2 );
841
+			$locale = substr(get_locale(), 0, 2);
843 842
 		}
844 843
 
845 844
 		return $locale;
@@ -853,9 +852,9 @@  discard block
 block discarded – undo
853 852
 	 * @param string $idempotency_key
854 853
 	 * @param array $request
855 854
 	 */
856
-	public function change_idempotency_key( $idempotency_key, $request ) {
857
-		$customer = ! empty( $request['customer'] ) ? $request['customer'] : '';
858
-		$source   = ! empty( $request['source'] ) ? $request['source'] : $customer;
855
+	public function change_idempotency_key($idempotency_key, $request) {
856
+		$customer = ! empty($request['customer']) ? $request['customer'] : '';
857
+		$source   = ! empty($request['source']) ? $request['source'] : $customer;
859 858
 		$count    = $this->retry_interval;
860 859
 
861 860
 		return $request['metadata']['order_id'] . '-' . $count . '-' . $source;
@@ -869,8 +868,8 @@  discard block
 block discarded – undo
869 868
 	 * @since 4.0.6
870 869
 	 * @param array $headers
871 870
 	 */
872
-	public function is_original_request( $headers ) {
873
-		if ( $headers['original-request'] === $headers['request-id'] ) {
871
+	public function is_original_request($headers) {
872
+		if ($headers['original-request'] === $headers['request-id']) {
874 873
 			return true;
875 874
 		}
876 875
 
Please login to merge, or discard this patch.
includes/class-wc-stripe-webhook-handler.php 1 patch
Spacing   +174 added lines, -174 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
 
@@ -32,9 +32,9 @@  discard block
 block discarded – undo
32 32
 	 */
33 33
 	public function __construct() {
34 34
 		$this->retry_interval = 2;
35
-		$stripe_settings      = get_option( 'woocommerce_stripe_settings', array() );
36
-		$this->testmode       = ( ! empty( $stripe_settings['testmode'] ) && 'yes' === $stripe_settings['testmode'] ) ? true : false;
37
-		add_action( 'woocommerce_api_wc_stripe', array( $this, 'check_for_webhook' ) );
35
+		$stripe_settings      = get_option('woocommerce_stripe_settings', array());
36
+		$this->testmode       = ( ! empty($stripe_settings['testmode']) && 'yes' === $stripe_settings['testmode']) ? true : false;
37
+		add_action('woocommerce_api_wc_stripe', array($this, 'check_for_webhook'));
38 38
 	}
39 39
 
40 40
 	/**
@@ -44,24 +44,24 @@  discard block
 block discarded – undo
44 44
 	 * @version 4.0.0
45 45
 	 */
46 46
 	public function check_for_webhook() {
47
-		if ( ( 'POST' !== $_SERVER['REQUEST_METHOD'] )
48
-			|| ! isset( $_GET['wc-api'] )
49
-			|| ( 'wc_stripe' !== $_GET['wc-api'] )
47
+		if (('POST' !== $_SERVER['REQUEST_METHOD'])
48
+			|| ! isset($_GET['wc-api'])
49
+			|| ('wc_stripe' !== $_GET['wc-api'])
50 50
 		) {
51 51
 			return;
52 52
 		}
53 53
 
54
-		$request_body    = file_get_contents( 'php://input' );
55
-		$request_headers = array_change_key_case( $this->get_request_headers(), CASE_UPPER );
54
+		$request_body    = file_get_contents('php://input');
55
+		$request_headers = array_change_key_case($this->get_request_headers(), CASE_UPPER);
56 56
 
57 57
 		// Validate it to make sure it is legit.
58
-		if ( $this->is_valid_request( $request_headers, $request_body ) ) {
59
-			$this->process_webhook( $request_body );
60
-			status_header( 200 );
58
+		if ($this->is_valid_request($request_headers, $request_body)) {
59
+			$this->process_webhook($request_body);
60
+			status_header(200);
61 61
 			exit;
62 62
 		} else {
63
-			WC_Stripe_Logger::log( 'Incoming webhook failed validation: ' . print_r( $request_body, true ) );
64
-			status_header( 400 );
63
+			WC_Stripe_Logger::log('Incoming webhook failed validation: ' . print_r($request_body, true));
64
+			status_header(400);
65 65
 			exit;
66 66
 		}
67 67
 	}
@@ -76,12 +76,12 @@  discard block
 block discarded – undo
76 76
 	 * @param string $request_body The request body from Stripe.
77 77
 	 * @return bool
78 78
 	 */
79
-	public function is_valid_request( $request_headers = null, $request_body = null ) {
80
-		if ( null === $request_headers || null === $request_body ) {
79
+	public function is_valid_request($request_headers = null, $request_body = null) {
80
+		if (null === $request_headers || null === $request_body) {
81 81
 			return false;
82 82
 		}
83 83
 
84
-		if ( ! empty( $request_headers['USER-AGENT'] ) && ! preg_match( '/Stripe/', $request_headers['USER-AGENT'] ) ) {
84
+		if ( ! empty($request_headers['USER-AGENT']) && ! preg_match('/Stripe/', $request_headers['USER-AGENT'])) {
85 85
 			return false;
86 86
 		}
87 87
 
@@ -97,11 +97,11 @@  discard block
 block discarded – undo
97 97
 	 * @version 4.0.0
98 98
 	 */
99 99
 	public function get_request_headers() {
100
-		if ( ! function_exists( 'getallheaders' ) ) {
100
+		if ( ! function_exists('getallheaders')) {
101 101
 			$headers = [];
102
-			foreach ( $_SERVER as $name => $value ) {
103
-				if ( 'HTTP_' === substr( $name, 0, 5 ) ) {
104
-					$headers[ str_replace( ' ', '-', ucwords( strtolower( str_replace( '_', ' ', substr( $name, 5 ) ) ) ) ) ] = $value;
102
+			foreach ($_SERVER as $name => $value) {
103
+				if ('HTTP_' === substr($name, 0, 5)) {
104
+					$headers[str_replace(' ', '-', ucwords(strtolower(str_replace('_', ' ', substr($name, 5)))))] = $value;
105 105
 				}
106 106
 			}
107 107
 
@@ -120,30 +120,30 @@  discard block
 block discarded – undo
120 120
 	 * @param object $notification
121 121
 	 * @param bool $retry
122 122
 	 */
123
-	public function process_webhook_payment( $notification, $retry = true ) {
123
+	public function process_webhook_payment($notification, $retry = true) {
124 124
 		// The following 2 payment methods are synchronous so does not need to be handle via webhook.
125
-		if ( 'card' === $notification->data->object->type || 'sepa_debit' === $notification->data->object->type ) {
125
+		if ('card' === $notification->data->object->type || 'sepa_debit' === $notification->data->object->type) {
126 126
 			return;
127 127
 		}
128 128
 
129
-		$order = WC_Stripe_Helper::get_order_by_source_id( $notification->data->object->id );
129
+		$order = WC_Stripe_Helper::get_order_by_source_id($notification->data->object->id);
130 130
 
131
-		if ( ! $order ) {
132
-			WC_Stripe_Logger::log( 'Could not find order via source ID: ' . $notification->data->object->id );
131
+		if ( ! $order) {
132
+			WC_Stripe_Logger::log('Could not find order via source ID: ' . $notification->data->object->id);
133 133
 			return;
134 134
 		}
135 135
 
136 136
 		$order_id  = WC_Stripe_Helper::is_pre_30() ? $order->id : $order->get_id();
137 137
 		$source_id = $notification->data->object->id;
138 138
 
139
-		$is_pending_receiver = ( 'receiver' === $notification->data->object->flow );
139
+		$is_pending_receiver = ('receiver' === $notification->data->object->flow);
140 140
 
141 141
 		try {
142
-			if ( 'processing' === $order->get_status() || 'completed' === $order->get_status() ) {
142
+			if ('processing' === $order->get_status() || 'completed' === $order->get_status()) {
143 143
 				return;
144 144
 			}
145 145
 
146
-			if ( 'on-hold' === $order->get_status() && ! $is_pending_receiver ) {
146
+			if ('on-hold' === $order->get_status() && ! $is_pending_receiver) {
147 147
 				return;
148 148
 			}
149 149
 
@@ -151,95 +151,95 @@  discard block
 block discarded – undo
151 151
 			$response = null;
152 152
 
153 153
 			// This will throw exception if not valid.
154
-			$this->validate_minimum_order_amount( $order );
154
+			$this->validate_minimum_order_amount($order);
155 155
 
156
-			WC_Stripe_Logger::log( "Info: (Webhook) Begin processing payment for order $order_id for the amount of {$order->get_total()}" );
156
+			WC_Stripe_Logger::log("Info: (Webhook) Begin processing payment for order $order_id for the amount of {$order->get_total()}");
157 157
 
158 158
 			// Prep source object.
159 159
 			$source_object           = new stdClass();
160 160
 			$source_object->token_id = '';
161
-			$source_object->customer = $this->get_stripe_customer_id( $order );
161
+			$source_object->customer = $this->get_stripe_customer_id($order);
162 162
 			$source_object->source   = $source_id;
163 163
 
164 164
 			// Make the request.
165
-			$response = WC_Stripe_API::request( $this->generate_payment_request( $order, $source_object ), 'charges', 'POST', true );
165
+			$response = WC_Stripe_API::request($this->generate_payment_request($order, $source_object), 'charges', 'POST', true);
166 166
 			$headers  = $response['headers'];
167 167
 			$response = $response['body'];
168 168
 
169
-			if ( ! empty( $response->error ) ) {
169
+			if ( ! empty($response->error)) {
170 170
 				// We want to retry.
171
-				if ( $this->is_retryable_error( $response->error ) ) {
172
-					if ( $retry ) {
171
+				if ($this->is_retryable_error($response->error)) {
172
+					if ($retry) {
173 173
 						// Don't do anymore retries after this.
174
-						if ( 5 <= $this->retry_interval ) {
174
+						if (5 <= $this->retry_interval) {
175 175
 
176
-							return $this->process_webhook_payment( $notification, false );
176
+							return $this->process_webhook_payment($notification, false);
177 177
 						}
178 178
 
179
-						sleep( $this->retry_interval );
179
+						sleep($this->retry_interval);
180 180
 
181 181
 						$this->retry_interval++;
182
-						return $this->process_webhook_payment( $notification, true );
182
+						return $this->process_webhook_payment($notification, true);
183 183
 					} else {
184
-						$localized_message = __( 'On going requests error and retries exhausted.', 'woocommerce-gateway-stripe' );
185
-						$order->add_order_note( $localized_message );
186
-						throw new WC_Stripe_Exception( print_r( $response, true ), $localized_message );
184
+						$localized_message = __('On going requests error and retries exhausted.', 'woocommerce-gateway-stripe');
185
+						$order->add_order_note($localized_message);
186
+						throw new WC_Stripe_Exception(print_r($response, true), $localized_message);
187 187
 					}
188 188
 				}
189 189
 
190 190
 				// Customer param wrong? The user may have been deleted on stripe's end. Remove customer_id. Can be retried without.
191
-				if ( preg_match( '/No such customer/i', $response->error->message ) && $retry ) {
192
-					if ( WC_Stripe_Helper::is_pre_30() ) {
193
-						delete_user_meta( $order->customer_user, '_stripe_customer_id' );
194
-						delete_post_meta( $order_id, '_stripe_customer_id' );
191
+				if (preg_match('/No such customer/i', $response->error->message) && $retry) {
192
+					if (WC_Stripe_Helper::is_pre_30()) {
193
+						delete_user_meta($order->customer_user, '_stripe_customer_id');
194
+						delete_post_meta($order_id, '_stripe_customer_id');
195 195
 					} else {
196
-						delete_user_meta( $order->get_customer_id(), '_stripe_customer_id' );
197
-						$order->delete_meta_data( '_stripe_customer_id' );
196
+						delete_user_meta($order->get_customer_id(), '_stripe_customer_id');
197
+						$order->delete_meta_data('_stripe_customer_id');
198 198
 						$order->save();
199 199
 					}
200 200
 
201
-					return $this->process_webhook_payment( $notification, false );
201
+					return $this->process_webhook_payment($notification, false);
202 202
 
203
-				} elseif ( preg_match( '/No such token/i', $response->error->message ) && $source_object->token_id ) {
203
+				} elseif (preg_match('/No such token/i', $response->error->message) && $source_object->token_id) {
204 204
 					// Source param wrong? The CARD may have been deleted on stripe's end. Remove token and show message.
205
-					$wc_token = WC_Payment_Tokens::get( $source_object->token_id );
205
+					$wc_token = WC_Payment_Tokens::get($source_object->token_id);
206 206
 					$wc_token->delete();
207
-					$message = __( 'This card is no longer available and has been removed.', 'woocommerce-gateway-stripe' );
208
-					$order->add_order_note( $message );
209
-					throw new WC_Stripe_Exception( print_r( $response, true ), $message );
207
+					$message = __('This card is no longer available and has been removed.', 'woocommerce-gateway-stripe');
208
+					$order->add_order_note($message);
209
+					throw new WC_Stripe_Exception(print_r($response, true), $message);
210 210
 				}
211 211
 
212 212
 				$localized_messages = WC_Stripe_Helper::get_localized_messages();
213 213
 
214
-				if ( 'card_error' === $response->error->type ) {
215
-					$localized_message = isset( $localized_messages[ $response->error->code ] ) ? $localized_messages[ $response->error->code ] : $response->error->message;
214
+				if ('card_error' === $response->error->type) {
215
+					$localized_message = isset($localized_messages[$response->error->code]) ? $localized_messages[$response->error->code] : $response->error->message;
216 216
 				} else {
217
-					$localized_message = isset( $localized_messages[ $response->error->type ] ) ? $localized_messages[ $response->error->type ] : $response->error->message;
217
+					$localized_message = isset($localized_messages[$response->error->type]) ? $localized_messages[$response->error->type] : $response->error->message;
218 218
 				}
219 219
 
220
-				$order->add_order_note( $localized_message );
220
+				$order->add_order_note($localized_message);
221 221
 
222
-				throw new WC_Stripe_Exception( print_r( $response, true ), $localized_message );
222
+				throw new WC_Stripe_Exception(print_r($response, true), $localized_message);
223 223
 			}
224 224
 
225 225
 			// To prevent double processing the order on WC side.
226
-			if ( ! $this->is_original_request( $headers ) ) {
226
+			if ( ! $this->is_original_request($headers)) {
227 227
 				return;
228 228
 			}
229 229
 
230
-			do_action( 'wc_gateway_stripe_process_webhook_payment', $response, $order );
230
+			do_action('wc_gateway_stripe_process_webhook_payment', $response, $order);
231 231
 
232
-			$this->process_response( $response, $order );
232
+			$this->process_response($response, $order);
233 233
 
234
-		} catch ( WC_Stripe_Exception $e ) {
235
-			WC_Stripe_Logger::log( 'Error: ' . $e->getMessage() );
234
+		} catch (WC_Stripe_Exception $e) {
235
+			WC_Stripe_Logger::log('Error: ' . $e->getMessage());
236 236
 
237
-			do_action( 'wc_gateway_stripe_process_webhook_payment_error', $e, $order );
237
+			do_action('wc_gateway_stripe_process_webhook_payment_error', $e, $order);
238 238
 
239
-			$statuses = array( 'pending', 'failed' );
239
+			$statuses = array('pending', 'failed');
240 240
 
241
-			if ( $order->has_status( $statuses ) ) {
242
-				$this->send_failed_order_email( $order_id );
241
+			if ($order->has_status($statuses)) {
242
+				$this->send_failed_order_email($order_id);
243 243
 			}
244 244
 		}
245 245
 	}
@@ -252,21 +252,21 @@  discard block
 block discarded – undo
252 252
 	 * @since 4.0.0
253 253
 	 * @param object $notification
254 254
 	 */
255
-	public function process_webhook_dispute( $notification ) {
256
-		$order = WC_Stripe_Helper::get_order_by_charge_id( $notification->data->object->charge );
255
+	public function process_webhook_dispute($notification) {
256
+		$order = WC_Stripe_Helper::get_order_by_charge_id($notification->data->object->charge);
257 257
 
258
-		if ( ! $order ) {
259
-			WC_Stripe_Logger::log( 'Could not find order via charge ID: ' . $notification->data->object->charge );
258
+		if ( ! $order) {
259
+			WC_Stripe_Logger::log('Could not find order via charge ID: ' . $notification->data->object->charge);
260 260
 			return;
261 261
 		}
262 262
 
263 263
 		/* translators: 1) The URL to the order. */
264
-		$order->update_status( 'on-hold', sprintf( __( 'A dispute was created for this order. Response is needed. Please go to your <a href="%s" title="Stripe Dashboard" target="_blank">Stripe Dashboard</a> to review this dispute.', 'woocommerce-gateway-stripe' ), $this->get_transaction_url( $order ) ) );
264
+		$order->update_status('on-hold', sprintf(__('A dispute was created for this order. Response is needed. Please go to your <a href="%s" title="Stripe Dashboard" target="_blank">Stripe Dashboard</a> to review this dispute.', 'woocommerce-gateway-stripe'), $this->get_transaction_url($order)));
265 265
 
266
-		do_action( 'wc_gateway_stripe_process_webhook_payment_error', $order, $notification );
266
+		do_action('wc_gateway_stripe_process_webhook_payment_error', $order, $notification);
267 267
 
268 268
 		$order_id = WC_Stripe_Helper::is_pre_30() ? $order->id : $order->get_id();
269
-		$this->send_failed_order_email( $order_id );
269
+		$this->send_failed_order_email($order_id);
270 270
 	}
271 271
 
272 272
 	/**
@@ -277,41 +277,41 @@  discard block
 block discarded – undo
277 277
 	 * @version 4.0.0
278 278
 	 * @param object $notification
279 279
 	 */
280
-	public function process_webhook_capture( $notification ) {
281
-		$order = WC_Stripe_Helper::get_order_by_charge_id( $notification->data->object->id );
280
+	public function process_webhook_capture($notification) {
281
+		$order = WC_Stripe_Helper::get_order_by_charge_id($notification->data->object->id);
282 282
 
283
-		if ( ! $order ) {
284
-			WC_Stripe_Logger::log( 'Could not find order via charge ID: ' . $notification->data->object->id );
283
+		if ( ! $order) {
284
+			WC_Stripe_Logger::log('Could not find order via charge ID: ' . $notification->data->object->id);
285 285
 			return;
286 286
 		}
287 287
 
288 288
 		$order_id = WC_Stripe_Helper::is_pre_30() ? $order->id : $order->get_id();
289 289
 
290
-		if ( 'stripe' === ( WC_Stripe_Helper::is_pre_30() ? $order->payment_method : $order->get_payment_method() ) ) {
291
-			$charge   = WC_Stripe_Helper::is_pre_30() ? get_post_meta( $order_id, '_transaction_id', true ) : $order->get_transaction_id();
292
-			$captured = WC_Stripe_Helper::is_pre_30() ? get_post_meta( $order_id, '_stripe_charge_captured', true ) : $order->get_meta( '_stripe_charge_captured', true );
290
+		if ('stripe' === (WC_Stripe_Helper::is_pre_30() ? $order->payment_method : $order->get_payment_method())) {
291
+			$charge   = WC_Stripe_Helper::is_pre_30() ? get_post_meta($order_id, '_transaction_id', true) : $order->get_transaction_id();
292
+			$captured = WC_Stripe_Helper::is_pre_30() ? get_post_meta($order_id, '_stripe_charge_captured', true) : $order->get_meta('_stripe_charge_captured', true);
293 293
 
294
-			if ( $charge && 'no' === $captured ) {
295
-				WC_Stripe_Helper::is_pre_30() ? update_post_meta( $order_id, '_stripe_charge_captured', 'yes' ) : $order->update_meta_data( '_stripe_charge_captured', 'yes' );
294
+			if ($charge && 'no' === $captured) {
295
+				WC_Stripe_Helper::is_pre_30() ? update_post_meta($order_id, '_stripe_charge_captured', 'yes') : $order->update_meta_data('_stripe_charge_captured', 'yes');
296 296
 
297 297
 				// Store other data such as fees
298
-				WC_Stripe_Helper::is_pre_30() ? update_post_meta( $order_id, '_transaction_id', $notification->data->object->id ) : $order->set_transaction_id( $notification->data->object->id );
298
+				WC_Stripe_Helper::is_pre_30() ? update_post_meta($order_id, '_transaction_id', $notification->data->object->id) : $order->set_transaction_id($notification->data->object->id);
299 299
 
300
-				if ( isset( $notification->data->object->balance_transaction ) ) {
301
-					$this->update_fees( $order, $notification->data->object->balance_transaction );
300
+				if (isset($notification->data->object->balance_transaction)) {
301
+					$this->update_fees($order, $notification->data->object->balance_transaction);
302 302
 				}
303 303
 
304
-				if ( is_callable( array( $order, 'save' ) ) ) {
304
+				if (is_callable(array($order, 'save'))) {
305 305
 					$order->save();
306 306
 				}
307 307
 
308 308
 				/* translators: transaction id */
309
-				$order->update_status( $order->needs_processing() ? 'processing' : 'completed', sprintf( __( 'Stripe charge complete (Charge ID: %s)', 'woocommerce-gateway-stripe' ), $notification->data->object->id ) );
309
+				$order->update_status($order->needs_processing() ? 'processing' : 'completed', sprintf(__('Stripe charge complete (Charge ID: %s)', 'woocommerce-gateway-stripe'), $notification->data->object->id));
310 310
 
311 311
 				// Check and see if capture is partial.
312
-				if ( $this->is_partial_capture( $notification ) ) {
313
-					$order->set_total( $this->get_partial_amount_to_charge( $notification ) );
314
-					$order->add_order_note( __( 'This charge was partially captured via Stripe Dashboard', 'woocommerce-gateway-stripe' ) );
312
+				if ($this->is_partial_capture($notification)) {
313
+					$order->set_total($this->get_partial_amount_to_charge($notification));
314
+					$order->add_order_note(__('This charge was partially captured via Stripe Dashboard', 'woocommerce-gateway-stripe'));
315 315
 					$order->save();
316 316
 				}
317 317
 			}
@@ -326,38 +326,38 @@  discard block
 block discarded – undo
326 326
 	 * @version 4.0.0
327 327
 	 * @param object $notification
328 328
 	 */
329
-	public function process_webhook_charge_succeeded( $notification ) {
329
+	public function process_webhook_charge_succeeded($notification) {
330 330
 		// The following payment methods are synchronous so does not need to be handle via webhook.
331
-		if ( ( isset( $notification->data->object->source->type ) && 'card' === $notification->data->object->source->type ) || ( isset( $notification->data->object->source->type ) && 'three_d_secure' === $notification->data->object->source->type ) ) {
331
+		if ((isset($notification->data->object->source->type) && 'card' === $notification->data->object->source->type) || (isset($notification->data->object->source->type) && 'three_d_secure' === $notification->data->object->source->type)) {
332 332
 			return;
333 333
 		}
334 334
 
335
-		$order = WC_Stripe_Helper::get_order_by_charge_id( $notification->data->object->id );
335
+		$order = WC_Stripe_Helper::get_order_by_charge_id($notification->data->object->id);
336 336
 
337
-		if ( ! $order ) {
338
-			WC_Stripe_Logger::log( 'Could not find order via charge ID: ' . $notification->data->object->id );
337
+		if ( ! $order) {
338
+			WC_Stripe_Logger::log('Could not find order via charge ID: ' . $notification->data->object->id);
339 339
 			return;
340 340
 		}
341 341
 
342 342
 		$order_id = WC_Stripe_Helper::is_pre_30() ? $order->id : $order->get_id();
343 343
 
344
-		if ( 'on-hold' !== $order->get_status() ) {
344
+		if ('on-hold' !== $order->get_status()) {
345 345
 			return;
346 346
 		}
347 347
 
348 348
 		// Store other data such as fees
349
-		WC_Stripe_Helper::is_pre_30() ? update_post_meta( $order_id, '_transaction_id', $notification->data->object->id ) : $order->set_transaction_id( $notification->data->object->id );
349
+		WC_Stripe_Helper::is_pre_30() ? update_post_meta($order_id, '_transaction_id', $notification->data->object->id) : $order->set_transaction_id($notification->data->object->id);
350 350
 
351
-		if ( isset( $notification->data->object->balance_transaction ) ) {
352
-			$this->update_fees( $order, $notification->data->object->balance_transaction );
351
+		if (isset($notification->data->object->balance_transaction)) {
352
+			$this->update_fees($order, $notification->data->object->balance_transaction);
353 353
 		}
354 354
 
355
-		if ( is_callable( array( $order, 'save' ) ) ) {
355
+		if (is_callable(array($order, 'save'))) {
356 356
 			$order->save();
357 357
 		}
358 358
 
359 359
 		/* translators: transaction id */
360
-		$order->update_status( $order->needs_processing() ? 'processing' : 'completed', sprintf( __( 'Stripe charge complete (Charge ID: %s)', 'woocommerce-gateway-stripe' ), $notification->data->object->id ) );
360
+		$order->update_status($order->needs_processing() ? 'processing' : 'completed', sprintf(__('Stripe charge complete (Charge ID: %s)', 'woocommerce-gateway-stripe'), $notification->data->object->id));
361 361
 	}
362 362
 
363 363
 	/**
@@ -368,23 +368,23 @@  discard block
 block discarded – undo
368 368
 	 * @version 4.0.0
369 369
 	 * @param object $notification
370 370
 	 */
371
-	public function process_webhook_charge_failed( $notification ) {
372
-		$order = WC_Stripe_Helper::get_order_by_charge_id( $notification->data->object->id );
371
+	public function process_webhook_charge_failed($notification) {
372
+		$order = WC_Stripe_Helper::get_order_by_charge_id($notification->data->object->id);
373 373
 
374
-		if ( ! $order ) {
375
-			WC_Stripe_Logger::log( 'Could not find order via charge ID: ' . $notification->data->object->id );
374
+		if ( ! $order) {
375
+			WC_Stripe_Logger::log('Could not find order via charge ID: ' . $notification->data->object->id);
376 376
 			return;
377 377
 		}
378 378
 
379 379
 		$order_id = WC_Stripe_Helper::is_pre_30() ? $order->id : $order->get_id();
380 380
 
381
-		if ( 'on-hold' !== $order->get_status() ) {
381
+		if ('on-hold' !== $order->get_status()) {
382 382
 			return;
383 383
 		}
384 384
 
385
-		$order->update_status( 'failed', __( 'This payment failed to clear.', 'woocommerce-gateway-stripe' ) );
385
+		$order->update_status('failed', __('This payment failed to clear.', 'woocommerce-gateway-stripe'));
386 386
 
387
-		do_action( 'wc_gateway_stripe_process_webhook_payment_error', $order, $notification );
387
+		do_action('wc_gateway_stripe_process_webhook_payment_error', $order, $notification);
388 388
 	}
389 389
 
390 390
 	/**
@@ -395,23 +395,23 @@  discard block
 block discarded – undo
395 395
 	 * @version 4.0.0
396 396
 	 * @param object $notification
397 397
 	 */
398
-	public function process_webhook_source_canceled( $notification ) {
399
-		$order = WC_Stripe_Helper::get_order_by_charge_id( $notification->data->object->id );
398
+	public function process_webhook_source_canceled($notification) {
399
+		$order = WC_Stripe_Helper::get_order_by_charge_id($notification->data->object->id);
400 400
 
401
-		if ( ! $order ) {
402
-			WC_Stripe_Logger::log( 'Could not find order via charge ID: ' . $notification->data->object->id );
401
+		if ( ! $order) {
402
+			WC_Stripe_Logger::log('Could not find order via charge ID: ' . $notification->data->object->id);
403 403
 			return;
404 404
 		}
405 405
 
406 406
 		$order_id = WC_Stripe_Helper::is_pre_30() ? $order->id : $order->get_id();
407 407
 
408
-		if ( 'on-hold' !== $order->get_status() || 'cancelled' !== $order->get_status() ) {
408
+		if ('on-hold' !== $order->get_status() || 'cancelled' !== $order->get_status()) {
409 409
 			return;
410 410
 		}
411 411
 
412
-		$order->update_status( 'cancelled', __( 'This payment has cancelled.', 'woocommerce-gateway-stripe' ) );
412
+		$order->update_status('cancelled', __('This payment has cancelled.', 'woocommerce-gateway-stripe'));
413 413
 
414
-		do_action( 'wc_gateway_stripe_process_webhook_payment_error', $order, $notification );
414
+		do_action('wc_gateway_stripe_process_webhook_payment_error', $order, $notification);
415 415
 	}
416 416
 
417 417
 	/**
@@ -422,42 +422,42 @@  discard block
 block discarded – undo
422 422
 	 * @version 4.0.0
423 423
 	 * @param object $notification
424 424
 	 */
425
-	public function process_webhook_refund( $notification ) {
426
-		$order = WC_Stripe_Helper::get_order_by_charge_id( $notification->data->object->id );
425
+	public function process_webhook_refund($notification) {
426
+		$order = WC_Stripe_Helper::get_order_by_charge_id($notification->data->object->id);
427 427
 
428
-		if ( ! $order ) {
429
-			WC_Stripe_Logger::log( 'Could not find order via charge ID: ' . $notification->data->object->id );
428
+		if ( ! $order) {
429
+			WC_Stripe_Logger::log('Could not find order via charge ID: ' . $notification->data->object->id);
430 430
 			return;
431 431
 		}
432 432
 
433 433
 		$order_id = WC_Stripe_Helper::is_pre_30() ? $order->id : $order->get_id();
434 434
 
435
-		if ( 'stripe' === ( WC_Stripe_Helper::is_pre_30() ? $order->payment_method : $order->get_payment_method() ) ) {
436
-			$charge    = WC_Stripe_Helper::is_pre_30() ? get_post_meta( $order_id, '_transaction_id', true ) : $order->get_transaction_id();
437
-			$captured  = WC_Stripe_Helper::is_pre_30() ? get_post_meta( $order_id, '_stripe_charge_captured', true ) : $order->get_meta( '_stripe_charge_captured', true );
438
-			$refund_id = WC_Stripe_Helper::is_pre_30() ? get_post_meta( $order_id, '_stripe_refund_id', true ) : $order->get_meta( '_stripe_refund_id', true );
435
+		if ('stripe' === (WC_Stripe_Helper::is_pre_30() ? $order->payment_method : $order->get_payment_method())) {
436
+			$charge    = WC_Stripe_Helper::is_pre_30() ? get_post_meta($order_id, '_transaction_id', true) : $order->get_transaction_id();
437
+			$captured  = WC_Stripe_Helper::is_pre_30() ? get_post_meta($order_id, '_stripe_charge_captured', true) : $order->get_meta('_stripe_charge_captured', true);
438
+			$refund_id = WC_Stripe_Helper::is_pre_30() ? get_post_meta($order_id, '_stripe_refund_id', true) : $order->get_meta('_stripe_refund_id', true);
439 439
 
440 440
 			// If the refund ID matches, don't continue to prevent double refunding.
441
-			if ( $notification->data->object->refunds->data[0]->id === $refund_id ) {
441
+			if ($notification->data->object->refunds->data[0]->id === $refund_id) {
442 442
 				return;
443 443
 			}
444 444
 
445 445
 			// Only refund captured charge.
446
-			if ( $charge ) {
447
-				$reason = ( isset( $captured ) && 'yes' === $captured ) ? __( 'Refunded via Stripe Dashboard', 'woocommerce-gateway-stripe' ) : __( 'Pre-Authorization Released via Stripe Dashboard', 'woocommerce-gateway-stripe' );
446
+			if ($charge) {
447
+				$reason = (isset($captured) && 'yes' === $captured) ? __('Refunded via Stripe Dashboard', 'woocommerce-gateway-stripe') : __('Pre-Authorization Released via Stripe Dashboard', 'woocommerce-gateway-stripe');
448 448
 
449 449
 				// Create the refund.
450
-				$refund = wc_create_refund( array(
450
+				$refund = wc_create_refund(array(
451 451
 					'order_id'       => $order_id,
452
-					'amount'         => $this->get_refund_amount( $notification ),
452
+					'amount'         => $this->get_refund_amount($notification),
453 453
 					'reason'         => $reason,
454
-				) );
454
+				));
455 455
 
456
-				if ( is_wp_error( $refund ) ) {
457
-					WC_Stripe_Logger::log( $refund->get_error_message() );
456
+				if (is_wp_error($refund)) {
457
+					WC_Stripe_Logger::log($refund->get_error_message());
458 458
 				}
459 459
 
460
-				$order->add_order_note( $reason );
460
+				$order->add_order_note($reason);
461 461
 			}
462 462
 		}
463 463
 	}
@@ -468,21 +468,21 @@  discard block
 block discarded – undo
468 468
 	 * @since 4.0.6
469 469
 	 * @param object $notification
470 470
 	 */
471
-	public function process_review_opened( $notification ) {
472
-		$order = WC_Stripe_Helper::get_order_by_charge_id( $notification->data->object->charge );
471
+	public function process_review_opened($notification) {
472
+		$order = WC_Stripe_Helper::get_order_by_charge_id($notification->data->object->charge);
473 473
 
474
-		if ( ! $order ) {
475
-			WC_Stripe_Logger::log( 'Could not find order via charge ID: ' . $notification->data->object->charge );
474
+		if ( ! $order) {
475
+			WC_Stripe_Logger::log('Could not find order via charge ID: ' . $notification->data->object->charge);
476 476
 			return;
477 477
 		}
478 478
 
479 479
 		/* translators: 1) The URL to the order. 2) The reason type. */
480
-		$message = sprintf( __( 'A review has been opened for this order. Action is needed. Please go to your <a href="%1$s" title="Stripe Dashboard" target="_blank">Stripe Dashboard</a> to review the issue. Reason: (%2$s)', 'woocommerce-gateway-stripe' ), $this->get_transaction_url( $order ), $notification->data->object->reason );
480
+		$message = sprintf(__('A review has been opened for this order. Action is needed. Please go to your <a href="%1$s" title="Stripe Dashboard" target="_blank">Stripe Dashboard</a> to review the issue. Reason: (%2$s)', 'woocommerce-gateway-stripe'), $this->get_transaction_url($order), $notification->data->object->reason);
481 481
 
482
-		if ( apply_filters( 'wc_stripe_webhook_review_change_order_status', true, $order, $notification ) ) {
483
-			$order->update_status( 'on-hold', $message );
482
+		if (apply_filters('wc_stripe_webhook_review_change_order_status', true, $order, $notification)) {
483
+			$order->update_status('on-hold', $message);
484 484
 		} else {
485
-			$order->add_order_note( $message );
485
+			$order->add_order_note($message);
486 486
 		}
487 487
 	}
488 488
 
@@ -492,25 +492,25 @@  discard block
 block discarded – undo
492 492
 	 * @since 4.0.6
493 493
 	 * @param object $notification
494 494
 	 */
495
-	public function process_review_closed( $notification ) {
496
-		$order = WC_Stripe_Helper::get_order_by_charge_id( $notification->data->object->charge );
495
+	public function process_review_closed($notification) {
496
+		$order = WC_Stripe_Helper::get_order_by_charge_id($notification->data->object->charge);
497 497
 
498
-		if ( ! $order ) {
499
-			WC_Stripe_Logger::log( 'Could not find order via charge ID: ' . $notification->data->object->charge );
498
+		if ( ! $order) {
499
+			WC_Stripe_Logger::log('Could not find order via charge ID: ' . $notification->data->object->charge);
500 500
 			return;
501 501
 		}
502 502
 
503 503
 		/* translators: 1) The reason type. */
504
-		$message = sprintf( __( 'The opened review for this order is now closed. Reason: (%s)', 'woocommerce-gateway-stripe' ), $notification->data->object->reason );
504
+		$message = sprintf(__('The opened review for this order is now closed. Reason: (%s)', 'woocommerce-gateway-stripe'), $notification->data->object->reason);
505 505
 
506
-		if ( 'on-hold' === $order->get_status() ) {
507
-			if ( apply_filters( 'wc_stripe_webhook_review_change_order_status', true, $order, $notification ) ) {
508
-				$order->update_status( 'processing', $message );
506
+		if ('on-hold' === $order->get_status()) {
507
+			if (apply_filters('wc_stripe_webhook_review_change_order_status', true, $order, $notification)) {
508
+				$order->update_status('processing', $message);
509 509
 			} else {
510
-				$order->add_order_note( $message );
510
+				$order->add_order_note($message);
511 511
 			}
512 512
 		} else {
513
-			$order->add_order_note( $message );
513
+			$order->add_order_note($message);
514 514
 		}
515 515
 	}
516 516
 
@@ -521,7 +521,7 @@  discard block
 block discarded – undo
521 521
 	 * @version 4.0.0
522 522
 	 * @param object $notification
523 523
 	 */
524
-	public function is_partial_capture( $notification ) {
524
+	public function is_partial_capture($notification) {
525 525
 		return 0 < $notification->data->object->amount_refunded;
526 526
 	}
527 527
 
@@ -532,11 +532,11 @@  discard block
 block discarded – undo
532 532
 	 * @version 4.0.0
533 533
 	 * @param object $notification
534 534
 	 */
535
-	public function get_refund_amount( $notification ) {
536
-		if ( $this->is_partial_capture( $notification ) ) {
535
+	public function get_refund_amount($notification) {
536
+		if ($this->is_partial_capture($notification)) {
537 537
 			$amount = $notification->data->object->amount_refunded / 100;
538 538
 
539
-			if ( in_array( strtolower( $notification->data->object->currency ), WC_Stripe_Helper::no_decimal_currencies() ) ) {
539
+			if (in_array(strtolower($notification->data->object->currency), WC_Stripe_Helper::no_decimal_currencies())) {
540 540
 				$amount = $notification->data->object->amount_refunded;
541 541
 			}
542 542
 
@@ -553,12 +553,12 @@  discard block
 block discarded – undo
553 553
 	 * @version 4.0.0
554 554
 	 * @param object $notification
555 555
 	 */
556
-	public function get_partial_amount_to_charge( $notification ) {
557
-		if ( $this->is_partial_capture( $notification ) ) {
558
-			$amount = ( $notification->data->object->amount - $notification->data->object->amount_refunded ) / 100;
556
+	public function get_partial_amount_to_charge($notification) {
557
+		if ($this->is_partial_capture($notification)) {
558
+			$amount = ($notification->data->object->amount - $notification->data->object->amount_refunded) / 100;
559 559
 
560
-			if ( in_array( strtolower( $notification->data->object->currency ), WC_Stripe_Helper::no_decimal_currencies() ) ) {
561
-				$amount = ( $notification->data->object->amount - $notification->data->object->amount_refunded );
560
+			if (in_array(strtolower($notification->data->object->currency), WC_Stripe_Helper::no_decimal_currencies())) {
561
+				$amount = ($notification->data->object->amount - $notification->data->object->amount_refunded);
562 562
 			}
563 563
 
564 564
 			return $amount;
@@ -574,44 +574,44 @@  discard block
 block discarded – undo
574 574
 	 * @version 4.0.0
575 575
 	 * @param string $request_body
576 576
 	 */
577
-	public function process_webhook( $request_body ) {
578
-		$notification = json_decode( $request_body );
577
+	public function process_webhook($request_body) {
578
+		$notification = json_decode($request_body);
579 579
 
580
-		switch ( $notification->type ) {
580
+		switch ($notification->type) {
581 581
 			case 'source.chargeable':
582
-				$this->process_webhook_payment( $notification );
582
+				$this->process_webhook_payment($notification);
583 583
 				break;
584 584
 
585 585
 			case 'source.canceled':
586
-				$this->process_webhook_source_canceled( $notification );
586
+				$this->process_webhook_source_canceled($notification);
587 587
 				break;
588 588
 
589 589
 			case 'charge.succeeded':
590
-				$this->process_webhook_charge_succeeded( $notification );
590
+				$this->process_webhook_charge_succeeded($notification);
591 591
 				break;
592 592
 
593 593
 			case 'charge.failed':
594
-				$this->process_webhook_charge_failed( $notification );
594
+				$this->process_webhook_charge_failed($notification);
595 595
 				break;
596 596
 
597 597
 			case 'charge.captured':
598
-				$this->process_webhook_capture( $notification );
598
+				$this->process_webhook_capture($notification);
599 599
 				break;
600 600
 
601 601
 			case 'charge.dispute.created':
602
-				$this->process_webhook_dispute( $notification );
602
+				$this->process_webhook_dispute($notification);
603 603
 				break;
604 604
 
605 605
 			case 'charge.refunded':
606
-				$this->process_webhook_refund( $notification );
606
+				$this->process_webhook_refund($notification);
607 607
 				break;
608 608
 
609 609
 			case 'review.opened':
610
-				$this->process_review_opened( $notification );
610
+				$this->process_review_opened($notification);
611 611
 				break;
612 612
 
613 613
 			case 'review.closed':
614
-				$this->process_review_closed( $notification );
614
+				$this->process_review_closed($notification);
615 615
 				break;
616 616
 
617 617
 		}
Please login to merge, or discard this patch.
includes/class-wc-stripe-api.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
 
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
26 26
 	 * Set secret API Key.
27 27
 	 * @param string $key
28 28
 	 */
29
-	public static function set_secret_key( $secret_key ) {
29
+	public static function set_secret_key($secret_key) {
30 30
 		self::$secret_key = $secret_key;
31 31
 	}
32 32
 
@@ -35,11 +35,11 @@  discard block
 block discarded – undo
35 35
 	 * @return string
36 36
 	 */
37 37
 	public static function get_secret_key() {
38
-		if ( ! self::$secret_key ) {
39
-			$options = get_option( 'woocommerce_stripe_settings' );
38
+		if ( ! self::$secret_key) {
39
+			$options = get_option('woocommerce_stripe_settings');
40 40
 
41
-			if ( isset( $options['testmode'], $options['secret_key'], $options['test_secret_key'] ) ) {
42
-				self::set_secret_key( 'yes' === $options['testmode'] ? $options['test_secret_key'] : $options['secret_key'] );
41
+			if (isset($options['testmode'], $options['secret_key'], $options['test_secret_key'])) {
42
+				self::set_secret_key('yes' === $options['testmode'] ? $options['test_secret_key'] : $options['secret_key']);
43 43
 			}
44 44
 		}
45 45
 		return self::$secret_key;
@@ -78,12 +78,12 @@  discard block
 block discarded – undo
78 78
 		$user_agent = self::get_user_agent();
79 79
 		$app_info   = $user_agent['application'];
80 80
 
81
-		return apply_filters( 'woocommerce_stripe_request_headers', array(
82
-			'Authorization'              => 'Basic ' . base64_encode( self::get_secret_key() . ':' ),
81
+		return apply_filters('woocommerce_stripe_request_headers', array(
82
+			'Authorization'              => 'Basic ' . base64_encode(self::get_secret_key() . ':'),
83 83
 			'Stripe-Version'             => self::STRIPE_API_VERSION,
84 84
 			'User-Agent'                 => $app_info['name'] . '/' . $app_info['version'] . ' (' . $app_info['url'] . ')',
85
-			'X-Stripe-Client-User-Agent' => json_encode( $user_agent ),
86
-		) );
85
+			'X-Stripe-Client-User-Agent' => json_encode($user_agent),
86
+		));
87 87
 	}
88 88
 
89 89
 	/**
@@ -96,16 +96,16 @@  discard block
 block discarded – undo
96 96
 	 * @param bool $with_headers To get the response with headers.
97 97
 	 * @return array|WP_Error
98 98
 	 */
99
-	public static function request( $request, $api = 'charges', $method = 'POST', $with_headers = false ) {
100
-		WC_Stripe_Logger::log( "{$api} request: " . print_r( $request, true ) );
99
+	public static function request($request, $api = 'charges', $method = 'POST', $with_headers = false) {
100
+		WC_Stripe_Logger::log("{$api} request: " . print_r($request, true));
101 101
 
102 102
 		$headers = self::get_headers();
103 103
 
104
-		if ( 'charges' === $api && 'POST' === $method ) {
105
-			$customer = ! empty( $request['customer'] ) ? $request['customer'] : '';
106
-			$source   = ! empty( $request['source'] ) ? $request['source'] : $customer;
104
+		if ('charges' === $api && 'POST' === $method) {
105
+			$customer = ! empty($request['customer']) ? $request['customer'] : '';
106
+			$source   = ! empty($request['source']) ? $request['source'] : $customer;
107 107
 
108
-			$headers['Idempotency-Key'] = apply_filters( 'wc_stripe_idempotency_key', $request['metadata']['order_id'] . '-' . $source, $request );
108
+			$headers['Idempotency-Key'] = apply_filters('wc_stripe_idempotency_key', $request['metadata']['order_id'] . '-' . $source, $request);
109 109
 		}
110 110
 
111 111
 		$response = wp_safe_remote_post(
@@ -113,21 +113,21 @@  discard block
 block discarded – undo
113 113
 			array(
114 114
 				'method'  => $method,
115 115
 				'headers' => $headers,
116
-				'body'    => apply_filters( 'woocommerce_stripe_request_body', $request, $api ),
116
+				'body'    => apply_filters('woocommerce_stripe_request_body', $request, $api),
117 117
 				'timeout' => 70,
118 118
 			)
119 119
 		);
120 120
 
121
-		if ( is_wp_error( $response ) || empty( $response['body'] ) ) {
122
-			WC_Stripe_Logger::log( 'Error Response: ' . print_r( $response, true ) );
123
-			throw new WC_Stripe_Exception( print_r( $response, true ), __( 'There was a problem connecting to the Stripe API endpoint.', 'woocommerce-gateway-stripe' ) );
121
+		if (is_wp_error($response) || empty($response['body'])) {
122
+			WC_Stripe_Logger::log('Error Response: ' . print_r($response, true));
123
+			throw new WC_Stripe_Exception(print_r($response, true), __('There was a problem connecting to the Stripe API endpoint.', 'woocommerce-gateway-stripe'));
124 124
 		}
125 125
 
126
-		if ( $with_headers ) {
127
-			return array( 'headers' => wp_remote_retrieve_headers( $response ), 'body' => json_decode( $response['body'] ) );
126
+		if ($with_headers) {
127
+			return array('headers' => wp_remote_retrieve_headers($response), 'body' => json_decode($response['body']));
128 128
 		}
129 129
 
130
-		return json_decode( $response['body'] );
130
+		return json_decode($response['body']);
131 131
 	}
132 132
 
133 133
 	/**
@@ -137,8 +137,8 @@  discard block
 block discarded – undo
137 137
 	 * @version 4.0.0
138 138
 	 * @param string $api
139 139
 	 */
140
-	public static function retrieve( $api ) {
141
-		WC_Stripe_Logger::log( "{$api}" );
140
+	public static function retrieve($api) {
141
+		WC_Stripe_Logger::log("{$api}");
142 142
 
143 143
 		$response = wp_safe_remote_get(
144 144
 			self::ENDPOINT . $api,
@@ -149,11 +149,11 @@  discard block
 block discarded – undo
149 149
 			)
150 150
 		);
151 151
 
152
-		if ( is_wp_error( $response ) || empty( $response['body'] ) ) {
153
-			WC_Stripe_Logger::log( 'Error Response: ' . print_r( $response, true ) );
154
-			return new WP_Error( 'stripe_error', __( 'There was a problem connecting to the Stripe API endpoint.', 'woocommerce-gateway-stripe' ) );
152
+		if (is_wp_error($response) || empty($response['body'])) {
153
+			WC_Stripe_Logger::log('Error Response: ' . print_r($response, true));
154
+			return new WP_Error('stripe_error', __('There was a problem connecting to the Stripe API endpoint.', 'woocommerce-gateway-stripe'));
155 155
 		}
156 156
 
157
-		return json_decode( $response['body'] );
157
+		return json_decode($response['body']);
158 158
 	}
159 159
 }
Please login to merge, or discard this patch.
includes/class-wc-stripe-order-handler.php 1 patch
Spacing   +97 added lines, -97 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,12 +23,12 @@  discard block
 block discarded – undo
23 23
 
24 24
 		$this->retry_interval = 1;
25 25
 
26
-		add_action( 'wp', array( $this, 'maybe_process_redirect_order' ) );
27
-		add_action( 'woocommerce_order_status_on-hold_to_processing', array( $this, 'capture_payment' ) );
28
-		add_action( 'woocommerce_order_status_on-hold_to_completed', array( $this, 'capture_payment' ) );
29
-		add_action( 'woocommerce_order_status_on-hold_to_cancelled', array( $this, 'cancel_payment' ) );
30
-		add_action( 'woocommerce_order_status_on-hold_to_refunded', array( $this, 'cancel_payment' ) );
31
-		add_action( 'wc_ajax_wc_stripe_validate_checkout', array( $this, 'validate_checkout' ) );
26
+		add_action('wp', array($this, 'maybe_process_redirect_order'));
27
+		add_action('woocommerce_order_status_on-hold_to_processing', array($this, 'capture_payment'));
28
+		add_action('woocommerce_order_status_on-hold_to_completed', array($this, 'capture_payment'));
29
+		add_action('woocommerce_order_status_on-hold_to_cancelled', array($this, 'cancel_payment'));
30
+		add_action('woocommerce_order_status_on-hold_to_refunded', array($this, 'cancel_payment'));
31
+		add_action('wc_ajax_wc_stripe_validate_checkout', array($this, 'validate_checkout'));
32 32
 	}
33 33
 
34 34
 	/**
@@ -49,25 +49,25 @@  discard block
 block discarded – undo
49 49
 	 * @since 4.0.0
50 50
 	 * @version 4.0.0
51 51
 	 */
52
-	public function process_redirect_payment( $order_id, $retry = true ) {
52
+	public function process_redirect_payment($order_id, $retry = true) {
53 53
 		try {
54
-			$source = wc_clean( $_GET['source'] );
54
+			$source = wc_clean($_GET['source']);
55 55
 
56
-			if ( empty( $source ) ) {
56
+			if (empty($source)) {
57 57
 				return;
58 58
 			}
59 59
 
60
-			if ( empty( $order_id ) ) {
60
+			if (empty($order_id)) {
61 61
 				return;
62 62
 			}
63 63
 
64
-			$order = wc_get_order( $order_id );
64
+			$order = wc_get_order($order_id);
65 65
 
66
-			if ( ! is_object( $order ) ) {
66
+			if ( ! is_object($order)) {
67 67
 				return;
68 68
 			}
69 69
 
70
-			if ( 'processing' === $order->get_status() || 'completed' === $order->get_status() || 'on-hold' === $order->get_status() ) {
70
+			if ('processing' === $order->get_status() || 'completed' === $order->get_status() || 'on-hold' === $order->get_status()) {
71 71
 				return;
72 72
 			}
73 73
 
@@ -75,129 +75,129 @@  discard block
 block discarded – undo
75 75
 			$response = null;
76 76
 
77 77
 			// This will throw exception if not valid.
78
-			$this->validate_minimum_order_amount( $order );
78
+			$this->validate_minimum_order_amount($order);
79 79
 
80
-			WC_Stripe_Logger::log( "Info: (Redirect) Begin processing payment for order $order_id for the amount of {$order->get_total()}" );
80
+			WC_Stripe_Logger::log("Info: (Redirect) Begin processing payment for order $order_id for the amount of {$order->get_total()}");
81 81
 
82 82
 			/**
83 83
 			 * First check if the source is chargeable at this time. If not,
84 84
 			 * webhook will take care of it later.
85 85
 			 */
86
-			$source_info = WC_Stripe_API::retrieve( 'sources/' . $source );
86
+			$source_info = WC_Stripe_API::retrieve('sources/' . $source);
87 87
 
88
-			if ( ! empty( $source_info->error ) ) {
89
-				throw new WC_Stripe_Exception( print_r( $source_info, true ), $source_info->error->message );
88
+			if ( ! empty($source_info->error)) {
89
+				throw new WC_Stripe_Exception(print_r($source_info, true), $source_info->error->message);
90 90
 			}
91 91
 
92
-			if ( 'failed' === $source_info->status || 'canceled' === $source_info->status ) {
93
-				throw new WC_Stripe_Exception( print_r( $source_info, true ), __( 'Unable to process this payment, please try again or use alternative method.', 'woocommerce-gateway-stripe' ) );
92
+			if ('failed' === $source_info->status || 'canceled' === $source_info->status) {
93
+				throw new WC_Stripe_Exception(print_r($source_info, true), __('Unable to process this payment, please try again or use alternative method.', 'woocommerce-gateway-stripe'));
94 94
 			}
95 95
 
96 96
 			// If already consumed, then ignore request.
97
-			if ( 'consumed' === $source_info->status ) {
97
+			if ('consumed' === $source_info->status) {
98 98
 				return;
99 99
 			}
100 100
 
101 101
 			// If not chargeable, then ignore request.
102
-			if ( 'chargeable' !== $source_info->status ) {
102
+			if ('chargeable' !== $source_info->status) {
103 103
 				return;
104 104
 			}
105 105
 
106 106
 			// Prep source object.
107 107
 			$source_object           = new stdClass();
108 108
 			$source_object->token_id = '';
109
-			$source_object->customer = $this->get_stripe_customer_id( $order );
109
+			$source_object->customer = $this->get_stripe_customer_id($order);
110 110
 			$source_object->source   = $source_info->id;
111 111
 
112 112
 			/* If we're doing a retry and source is chargeable, we need to pass
113 113
 			 * a different idempotency key and retry for success.
114 114
 			 */
115
-			if ( 1 < $this->retry_interval && 'chargeable' === $source_info->status ) {
116
-				add_filter( 'wc_stripe_idempotency_key', array( $this, 'change_idempotency_key' ), 10, 2 );
115
+			if (1 < $this->retry_interval && 'chargeable' === $source_info->status) {
116
+				add_filter('wc_stripe_idempotency_key', array($this, 'change_idempotency_key'), 10, 2);
117 117
 			}
118 118
 
119 119
 			// Make the request.
120
-			$response = WC_Stripe_API::request( $this->generate_payment_request( $order, $source_object ), 'charges', 'POST', true );
120
+			$response = WC_Stripe_API::request($this->generate_payment_request($order, $source_object), 'charges', 'POST', true);
121 121
 			$headers  = $response['headers'];
122 122
 			$response = $response['body'];
123 123
 
124
-			if ( ! empty( $response->error ) ) {
124
+			if ( ! empty($response->error)) {
125 125
 				// Customer param wrong? The user may have been deleted on stripe's end. Remove customer_id. Can be retried without.
126
-				if ( preg_match( '/No such customer/i', $response->error->message ) && $retry ) {
127
-					if ( WC_Stripe_Helper::is_pre_30() ) {
128
-						delete_user_meta( $order->customer_user, '_stripe_customer_id' );
129
-						delete_post_meta( $order_id, '_stripe_customer_id' );
126
+				if (preg_match('/No such customer/i', $response->error->message) && $retry) {
127
+					if (WC_Stripe_Helper::is_pre_30()) {
128
+						delete_user_meta($order->customer_user, '_stripe_customer_id');
129
+						delete_post_meta($order_id, '_stripe_customer_id');
130 130
 					} else {
131
-						delete_user_meta( $order->get_customer_id(), '_stripe_customer_id' );
132
-						$order->delete_meta_data( '_stripe_customer_id' );
131
+						delete_user_meta($order->get_customer_id(), '_stripe_customer_id');
132
+						$order->delete_meta_data('_stripe_customer_id');
133 133
 						$order->save();
134 134
 					}
135 135
 
136
-					return $this->process_redirect_payment( $order_id, false );
136
+					return $this->process_redirect_payment($order_id, false);
137 137
 
138
-				} elseif ( preg_match( '/No such token/i', $response->error->message ) && $source_object->token_id ) {
138
+				} elseif (preg_match('/No such token/i', $response->error->message) && $source_object->token_id) {
139 139
 					// Source param wrong? The CARD may have been deleted on stripe's end. Remove token and show message.
140 140
 
141
-					$wc_token = WC_Payment_Tokens::get( $source_object->token_id );
141
+					$wc_token = WC_Payment_Tokens::get($source_object->token_id);
142 142
 					$wc_token->delete();
143
-					$message = __( 'This card is no longer available and has been removed.', 'woocommerce-gateway-stripe' );
144
-					$order->add_order_note( $message );
145
-					throw new WC_Stripe_Exception( print_r( $response, true ), $message );
143
+					$message = __('This card is no longer available and has been removed.', 'woocommerce-gateway-stripe');
144
+					$order->add_order_note($message);
145
+					throw new WC_Stripe_Exception(print_r($response, true), $message);
146 146
 				}
147 147
 
148 148
 				// We want to retry.
149
-				if ( $this->is_retryable_error( $response->error ) ) {
150
-					if ( $retry ) {
149
+				if ($this->is_retryable_error($response->error)) {
150
+					if ($retry) {
151 151
 						// Don't do anymore retries after this.
152
-						if ( 5 <= $this->retry_interval ) {
153
-							return $this->process_redirect_payment( $order_id, false );
152
+						if (5 <= $this->retry_interval) {
153
+							return $this->process_redirect_payment($order_id, false);
154 154
 						}
155 155
 
156
-						sleep( $this->retry_interval );
156
+						sleep($this->retry_interval);
157 157
 
158 158
 						$this->retry_interval++;
159
-						return $this->process_redirect_payment( $order_id, true );
159
+						return $this->process_redirect_payment($order_id, true);
160 160
 					} else {
161
-						$localized_message = __( 'On going requests error and retries exhausted.', 'woocommerce-gateway-stripe' );
162
-						$order->add_order_note( $localized_message );
163
-						throw new WC_Stripe_Exception( print_r( $response, true ), $localized_message );
161
+						$localized_message = __('On going requests error and retries exhausted.', 'woocommerce-gateway-stripe');
162
+						$order->add_order_note($localized_message);
163
+						throw new WC_Stripe_Exception(print_r($response, true), $localized_message);
164 164
 					}
165 165
 				}
166 166
 
167 167
 				$localized_messages = WC_Stripe_Helper::get_localized_messages();
168 168
 
169
-				if ( 'card_error' === $response->error->type ) {
170
-					$message = isset( $localized_messages[ $response->error->code ] ) ? $localized_messages[ $response->error->code ] : $response->error->message;
169
+				if ('card_error' === $response->error->type) {
170
+					$message = isset($localized_messages[$response->error->code]) ? $localized_messages[$response->error->code] : $response->error->message;
171 171
 				} else {
172
-					$message = isset( $localized_messages[ $response->error->type ] ) ? $localized_messages[ $response->error->type ] : $response->error->message;
172
+					$message = isset($localized_messages[$response->error->type]) ? $localized_messages[$response->error->type] : $response->error->message;
173 173
 				}
174 174
 
175
-				throw new WC_Stripe_Exception( print_r( $response, true ), $message );
175
+				throw new WC_Stripe_Exception(print_r($response, true), $message);
176 176
 			}
177 177
 
178 178
 			// To prevent double processing the order on WC side.
179
-			if ( ! $this->is_original_request( $headers ) ) {
179
+			if ( ! $this->is_original_request($headers)) {
180 180
 				return;
181 181
 			}
182 182
 
183
-			do_action( 'wc_gateway_stripe_process_redirect_payment', $response, $order );
183
+			do_action('wc_gateway_stripe_process_redirect_payment', $response, $order);
184 184
 
185
-			$this->process_response( $response, $order );
185
+			$this->process_response($response, $order);
186 186
 
187
-		} catch ( WC_Stripe_Exception $e ) {
188
-			WC_Stripe_Logger::log( 'Error: ' . $e->getMessage() );
187
+		} catch (WC_Stripe_Exception $e) {
188
+			WC_Stripe_Logger::log('Error: ' . $e->getMessage());
189 189
 
190
-			do_action( 'wc_gateway_stripe_process_redirect_payment_error', $e, $order );
190
+			do_action('wc_gateway_stripe_process_redirect_payment_error', $e, $order);
191 191
 
192 192
 			/* translators: error message */
193
-			$order->update_status( 'failed', sprintf( __( 'Stripe payment failed: %s', 'woocommerce-gateway-stripe' ), $e->getLocalizedMessage() ) );
193
+			$order->update_status('failed', sprintf(__('Stripe payment failed: %s', 'woocommerce-gateway-stripe'), $e->getLocalizedMessage()));
194 194
 
195
-			if ( $order->has_status( array( 'pending', 'failed' ) ) ) {
196
-				$this->send_failed_order_email( $order_id );
195
+			if ($order->has_status(array('pending', 'failed'))) {
196
+				$this->send_failed_order_email($order_id);
197 197
 			}
198 198
 
199
-			wc_add_notice( $e->getLocalizedMessage(), 'error' );
200
-			wp_safe_redirect( wc_get_checkout_url() );
199
+			wc_add_notice($e->getLocalizedMessage(), 'error');
200
+			wp_safe_redirect(wc_get_checkout_url());
201 201
 			exit;
202 202
 		}
203 203
 	}
@@ -209,13 +209,13 @@  discard block
 block discarded – undo
209 209
 	 * @version 4.0.0
210 210
 	 */
211 211
 	public function maybe_process_redirect_order() {
212
-		if ( ! is_order_received_page() || empty( $_GET['client_secret'] ) || empty( $_GET['source'] ) ) {
212
+		if ( ! is_order_received_page() || empty($_GET['client_secret']) || empty($_GET['source'])) {
213 213
 			return;
214 214
 		}
215 215
 
216
-		$order_id = wc_clean( $_GET['order_id'] );
216
+		$order_id = wc_clean($_GET['order_id']);
217 217
 
218
-		$this->process_redirect_payment( $order_id );
218
+		$this->process_redirect_payment($order_id);
219 219
 	}
220 220
 
221 221
 	/**
@@ -225,52 +225,52 @@  discard block
 block discarded – undo
225 225
 	 * @version 4.0.0
226 226
 	 * @param  int $order_id
227 227
 	 */
228
-	public function capture_payment( $order_id ) {
229
-		$order = wc_get_order( $order_id );
228
+	public function capture_payment($order_id) {
229
+		$order = wc_get_order($order_id);
230 230
 
231
-		if ( 'stripe' === ( WC_Stripe_Helper::is_pre_30() ? $order->payment_method : $order->get_payment_method() ) ) {
232
-			$charge   = WC_Stripe_Helper::is_pre_30() ? get_post_meta( $order_id, '_transaction_id', true ) : $order->get_transaction_id();
233
-			$captured = WC_Stripe_Helper::is_pre_30() ? get_post_meta( $order_id, '_stripe_charge_captured', true ) : $order->get_meta( '_stripe_charge_captured', true );
231
+		if ('stripe' === (WC_Stripe_Helper::is_pre_30() ? $order->payment_method : $order->get_payment_method())) {
232
+			$charge   = WC_Stripe_Helper::is_pre_30() ? get_post_meta($order_id, '_transaction_id', true) : $order->get_transaction_id();
233
+			$captured = WC_Stripe_Helper::is_pre_30() ? get_post_meta($order_id, '_stripe_charge_captured', true) : $order->get_meta('_stripe_charge_captured', true);
234 234
 
235
-			if ( $charge && 'no' === $captured ) {
235
+			if ($charge && 'no' === $captured) {
236 236
 				$order_total = $order->get_total();
237 237
 
238
-				if ( 0 < $order->get_total_refunded() ) {
238
+				if (0 < $order->get_total_refunded()) {
239 239
 					$order_total = $order_total - $order->get_total_refunded();
240 240
 				}
241 241
 
242
-				$result = WC_Stripe_API::request( array(
243
-					'amount'   => WC_Stripe_Helper::get_stripe_amount( $order_total ),
242
+				$result = WC_Stripe_API::request(array(
243
+					'amount'   => WC_Stripe_Helper::get_stripe_amount($order_total),
244 244
 					'expand[]' => 'balance_transaction',
245
-				), 'charges/' . $charge . '/capture' );
245
+				), 'charges/' . $charge . '/capture');
246 246
 
247
-				if ( ! empty( $result->error ) ) {
247
+				if ( ! empty($result->error)) {
248 248
 					/* translators: error message */
249
-					$order->update_status( 'failed', sprintf( __( 'Unable to capture charge! %s', 'woocommerce-gateway-stripe' ), $result->error->message ) );
249
+					$order->update_status('failed', sprintf(__('Unable to capture charge! %s', 'woocommerce-gateway-stripe'), $result->error->message));
250 250
 				} else {
251 251
 					/* translators: transaction id */
252
-					$order->add_order_note( sprintf( __( 'Stripe charge complete (Charge ID: %s)', 'woocommerce-gateway-stripe' ), $result->id ) );
253
-					WC_Stripe_Helper::is_pre_30() ? update_post_meta( $order_id, '_stripe_charge_captured', 'yes' ) : $order->update_meta_data( '_stripe_charge_captured', 'yes' );
252
+					$order->add_order_note(sprintf(__('Stripe charge complete (Charge ID: %s)', 'woocommerce-gateway-stripe'), $result->id));
253
+					WC_Stripe_Helper::is_pre_30() ? update_post_meta($order_id, '_stripe_charge_captured', 'yes') : $order->update_meta_data('_stripe_charge_captured', 'yes');
254 254
 
255 255
 					// Store other data such as fees
256
-					WC_Stripe_Helper::is_pre_30() ? update_post_meta( $order_id, '_transaction_id', $result->id ) : $order->set_transaction_id( $result->id );
256
+					WC_Stripe_Helper::is_pre_30() ? update_post_meta($order_id, '_transaction_id', $result->id) : $order->set_transaction_id($result->id);
257 257
 
258
-					if ( isset( $result->balance_transaction ) && isset( $result->balance_transaction->fee ) ) {
258
+					if (isset($result->balance_transaction) && isset($result->balance_transaction->fee)) {
259 259
 						// Fees and Net needs to both come from Stripe to be accurate as the returned
260 260
 						// values are in the local currency of the Stripe account, not from WC.
261
-						$fee = ! empty( $result->balance_transaction->fee ) ? WC_Stripe_Helper::format_balance_fee( $result->balance_transaction, 'fee' ) : 0;
262
-						$net = ! empty( $result->balance_transaction->net ) ? WC_Stripe_Helper::format_balance_fee( $result->balance_transaction, 'net' ) : 0;
263
-						WC_Stripe_Helper::is_pre_30() ? update_post_meta( $order_id, parent::META_NAME_FEE, $fee ) : $order->update_meta_data( parent::META_NAME_FEE, $fee );
264
-						WC_Stripe_Helper::is_pre_30() ? update_post_meta( $order_id, parent::META_NAME_NET, $net ) : $order->update_meta_data( parent::META_NAME_NET, $net );
261
+						$fee = ! empty($result->balance_transaction->fee) ? WC_Stripe_Helper::format_balance_fee($result->balance_transaction, 'fee') : 0;
262
+						$net = ! empty($result->balance_transaction->net) ? WC_Stripe_Helper::format_balance_fee($result->balance_transaction, 'net') : 0;
263
+						WC_Stripe_Helper::is_pre_30() ? update_post_meta($order_id, parent::META_NAME_FEE, $fee) : $order->update_meta_data(parent::META_NAME_FEE, $fee);
264
+						WC_Stripe_Helper::is_pre_30() ? update_post_meta($order_id, parent::META_NAME_NET, $net) : $order->update_meta_data(parent::META_NAME_NET, $net);
265 265
 					}
266 266
 
267
-					if ( is_callable( array( $order, 'save' ) ) ) {
267
+					if (is_callable(array($order, 'save'))) {
268 268
 						$order->save();
269 269
 					}
270 270
 				}
271 271
 
272 272
 				// This hook fires when admin manually changes order status to processing or completed.
273
-				do_action( 'woocommerce_stripe_process_manual_capture', $order, $result );
273
+				do_action('woocommerce_stripe_process_manual_capture', $order, $result);
274 274
 			}
275 275
 		}
276 276
 	}
@@ -282,14 +282,14 @@  discard block
 block discarded – undo
282 282
 	 * @version 4.0.0
283 283
 	 * @param  int $order_id
284 284
 	 */
285
-	public function cancel_payment( $order_id ) {
286
-		$order = wc_get_order( $order_id );
285
+	public function cancel_payment($order_id) {
286
+		$order = wc_get_order($order_id);
287 287
 
288
-		if ( 'stripe' === ( WC_Stripe_Helper::is_pre_30() ? $order->payment_method : $order->get_payment_method() ) ) {
289
-			$this->process_refund( $order_id );
288
+		if ('stripe' === (WC_Stripe_Helper::is_pre_30() ? $order->payment_method : $order->get_payment_method())) {
289
+			$this->process_refund($order_id);
290 290
 
291 291
 			// This hook fires when admin manually changes order status to cancel.
292
-			do_action( 'woocommerce_stripe_process_manual_cancel', $order );
292
+			do_action('woocommerce_stripe_process_manual_cancel', $order);
293 293
 		}
294 294
 	}
295 295
 
@@ -300,8 +300,8 @@  discard block
 block discarded – undo
300 300
 	 * @version 4.0.0
301 301
 	 */
302 302
 	public function validate_checkout() {
303
-		if ( ! wp_verify_nonce( $_POST['nonce'], '_wc_stripe_nonce' ) ) {
304
-			wp_die( __( 'Cheatin&#8217; huh?', 'woocommerce-gateway-stripe' ) );
303
+		if ( ! wp_verify_nonce($_POST['nonce'], '_wc_stripe_nonce')) {
304
+			wp_die(__('Cheatin&#8217; huh?', 'woocommerce-gateway-stripe'));
305 305
 		}
306 306
 
307 307
 		/*
@@ -309,7 +309,7 @@  discard block
 block discarded – undo
309 309
 		 * i.e. wp_send_json( 'success' ); // On successful validation.
310 310
 		 * i.e. For errors follow WC https://github.com/woocommerce/woocommerce/blob/master/includes/class-wc-checkout.php#L918-L938
311 311
 		 */
312
-		do_action( 'wc_stripe_validate_modal_checkout_action', $_POST['required_fields'], $_POST['all_fields'] );
312
+		do_action('wc_stripe_validate_modal_checkout_action', $_POST['required_fields'], $_POST['all_fields']);
313 313
 	}
314 314
 }
315 315
 
Please login to merge, or discard this patch.