Completed
Pull Request — master (#660)
by Jon
08:35
created
includes/class-wc-stripe-webhook-handler.php 1 patch
Spacing   +183 added lines, -183 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,12 +97,12 @@  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 = array();
102 102
 
103
-			foreach ( $_SERVER as $name => $value ) {
104
-				if ( 'HTTP_' === substr( $name, 0, 5 ) ) {
105
-					$headers[ str_replace( ' ', '-', ucwords( strtolower( str_replace( '_', ' ', substr( $name, 5 ) ) ) ) ) ] = $value;
103
+			foreach ($_SERVER as $name => $value) {
104
+				if ('HTTP_' === substr($name, 0, 5)) {
105
+					$headers[str_replace(' ', '-', ucwords(strtolower(str_replace('_', ' ', substr($name, 5)))))] = $value;
106 106
 				}
107 107
 			}
108 108
 
@@ -121,30 +121,30 @@  discard block
 block discarded – undo
121 121
 	 * @param object $notification
122 122
 	 * @param bool $retry
123 123
 	 */
124
-	public function process_webhook_payment( $notification, $retry = true ) {
124
+	public function process_webhook_payment($notification, $retry = true) {
125 125
 		// The following 2 payment methods are synchronous so does not need to be handle via webhook.
126
-		if ( 'card' === $notification->data->object->type || 'sepa_debit' === $notification->data->object->type ) {
126
+		if ('card' === $notification->data->object->type || 'sepa_debit' === $notification->data->object->type) {
127 127
 			return;
128 128
 		}
129 129
 
130
-		$order = WC_Stripe_Helper::get_order_by_source_id( $notification->data->object->id );
130
+		$order = WC_Stripe_Helper::get_order_by_source_id($notification->data->object->id);
131 131
 
132
-		if ( ! $order ) {
133
-			WC_Stripe_Logger::log( 'Could not find order via source ID: ' . $notification->data->object->id );
132
+		if ( ! $order) {
133
+			WC_Stripe_Logger::log('Could not find order via source ID: ' . $notification->data->object->id);
134 134
 			return;
135 135
 		}
136 136
 
137 137
 		$order_id  = WC_Stripe_Helper::is_pre_30() ? $order->id : $order->get_id();
138 138
 		$source_id = $notification->data->object->id;
139 139
 
140
-		$is_pending_receiver = ( 'receiver' === $notification->data->object->flow );
140
+		$is_pending_receiver = ('receiver' === $notification->data->object->flow);
141 141
 
142 142
 		try {
143
-			if ( 'processing' === $order->get_status() || 'completed' === $order->get_status() ) {
143
+			if ('processing' === $order->get_status() || 'completed' === $order->get_status()) {
144 144
 				return;
145 145
 			}
146 146
 
147
-			if ( 'on-hold' === $order->get_status() && ! $is_pending_receiver ) {
147
+			if ('on-hold' === $order->get_status() && ! $is_pending_receiver) {
148 148
 				return;
149 149
 			}
150 150
 
@@ -152,94 +152,94 @@  discard block
 block discarded – undo
152 152
 			$response = null;
153 153
 
154 154
 			// This will throw exception if not valid.
155
-			$this->validate_minimum_order_amount( $order );
155
+			$this->validate_minimum_order_amount($order);
156 156
 
157
-			WC_Stripe_Logger::log( "Info: (Webhook) Begin processing payment for order $order_id for the amount of {$order->get_total()}" );
157
+			WC_Stripe_Logger::log("Info: (Webhook) Begin processing payment for order $order_id for the amount of {$order->get_total()}");
158 158
 
159 159
 			// Prep source object.
160 160
 			$source_object           = new stdClass();
161 161
 			$source_object->token_id = '';
162
-			$source_object->customer = $this->get_stripe_customer_id( $order );
162
+			$source_object->customer = $this->get_stripe_customer_id($order);
163 163
 			$source_object->source   = $source_id;
164 164
 
165 165
 			// Make the request.
166
-			$response = WC_Stripe_API::request( $this->generate_payment_request( $order, $source_object ), 'charges', 'POST', true );
166
+			$response = WC_Stripe_API::request($this->generate_payment_request($order, $source_object), 'charges', 'POST', true);
167 167
 			$headers  = $response['headers'];
168 168
 			$response = $response['body'];
169 169
 
170
-			if ( ! empty( $response->error ) ) {
170
+			if ( ! empty($response->error)) {
171 171
 				// Customer param wrong? The user may have been deleted on stripe's end. Remove customer_id. Can be retried without.
172
-				if ( $this->is_no_such_customer_error( $response->error ) ) {
173
-					if ( WC_Stripe_Helper::is_pre_30() ) {
174
-						delete_user_meta( $order->customer_user, '_stripe_customer_id' );
175
-						delete_post_meta( $order_id, '_stripe_customer_id' );
172
+				if ($this->is_no_such_customer_error($response->error)) {
173
+					if (WC_Stripe_Helper::is_pre_30()) {
174
+						delete_user_meta($order->customer_user, '_stripe_customer_id');
175
+						delete_post_meta($order_id, '_stripe_customer_id');
176 176
 					} else {
177
-						delete_user_meta( $order->get_customer_id(), '_stripe_customer_id' );
178
-						$order->delete_meta_data( '_stripe_customer_id' );
177
+						delete_user_meta($order->get_customer_id(), '_stripe_customer_id');
178
+						$order->delete_meta_data('_stripe_customer_id');
179 179
 						$order->save();
180 180
 					}
181 181
 				}
182 182
 
183
-				if ( $this->is_no_such_token_error( $response->error ) && $prepared_source->token_id ) {
183
+				if ($this->is_no_such_token_error($response->error) && $prepared_source->token_id) {
184 184
 					// Source param wrong? The CARD may have been deleted on stripe's end. Remove token and show message.
185
-					$wc_token = WC_Payment_Tokens::get( $prepared_source->token_id );
185
+					$wc_token = WC_Payment_Tokens::get($prepared_source->token_id);
186 186
 					$wc_token->delete();
187
-					$localized_message = __( 'This card is no longer available and has been removed.', 'woocommerce-gateway-stripe' );
188
-					$order->add_order_note( $localized_message );
189
-					throw new WC_Stripe_Exception( print_r( $response, true ), $localized_message );
187
+					$localized_message = __('This card is no longer available and has been removed.', 'woocommerce-gateway-stripe');
188
+					$order->add_order_note($localized_message);
189
+					throw new WC_Stripe_Exception(print_r($response, true), $localized_message);
190 190
 				}
191 191
 
192 192
 				// We want to retry.
193
-				if ( $this->is_retryable_error( $response->error ) ) {
194
-					if ( $retry ) {
193
+				if ($this->is_retryable_error($response->error)) {
194
+					if ($retry) {
195 195
 						// Don't do anymore retries after this.
196
-						if ( 5 <= $this->retry_interval ) {
196
+						if (5 <= $this->retry_interval) {
197 197
 
198
-							return $this->process_webhook_payment( $notification, false );
198
+							return $this->process_webhook_payment($notification, false);
199 199
 						}
200 200
 
201
-						sleep( $this->retry_interval );
201
+						sleep($this->retry_interval);
202 202
 
203 203
 						$this->retry_interval++;
204
-						return $this->process_webhook_payment( $notification, true );
204
+						return $this->process_webhook_payment($notification, true);
205 205
 					} else {
206
-						$localized_message = __( 'Sorry, we are unable to process your payment at this time. Please retry later.', 'woocommerce-gateway-stripe' );
207
-						$order->add_order_note( $localized_message );
208
-						throw new WC_Stripe_Exception( print_r( $response, true ), $localized_message );
206
+						$localized_message = __('Sorry, we are unable to process your payment at this time. Please retry later.', 'woocommerce-gateway-stripe');
207
+						$order->add_order_note($localized_message);
208
+						throw new WC_Stripe_Exception(print_r($response, true), $localized_message);
209 209
 					}
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', $order, $notification, $e );
237
+			do_action('wc_gateway_stripe_process_webhook_payment_error', $order, $notification, $e);
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,44 +277,44 @@  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 304
 				// Check and see if capture is partial.
305
-				if ( $this->is_partial_capture( $notification ) ) {
306
-					$partial_amount = $this->get_partial_amount_to_charge( $notification );
307
-					$order->set_total( $partial_amount );
305
+				if ($this->is_partial_capture($notification)) {
306
+					$partial_amount = $this->get_partial_amount_to_charge($notification);
307
+					$order->set_total($partial_amount);
308 308
 					/* translators: partial captured amount */
309
-					$order->add_order_note( sprintf( __( 'This charge was partially captured via Stripe Dashboard in the amount of: %s', 'woocommerce-gateway-stripe' ), $partial_amount ) );
309
+					$order->add_order_note(sprintf(__('This charge was partially captured via Stripe Dashboard in the amount of: %s', 'woocommerce-gateway-stripe'), $partial_amount));
310 310
 				} else {
311
-					$order->payment_complete( $notification->data->object->id );
311
+					$order->payment_complete($notification->data->object->id);
312 312
 
313 313
 					/* translators: transaction id */
314
-					$order->add_order_note( sprintf( __( 'Stripe charge complete (Charge ID: %s)', 'woocommerce-gateway-stripe' ), $notification->data->object->id ) );
314
+					$order->add_order_note(sprintf(__('Stripe charge complete (Charge ID: %s)', 'woocommerce-gateway-stripe'), $notification->data->object->id));
315 315
 				}
316 316
 
317
-				if ( is_callable( array( $order, 'save' ) ) ) {
317
+				if (is_callable(array($order, 'save'))) {
318 318
 					$order->save();
319 319
 				}
320 320
 			}
@@ -329,38 +329,38 @@  discard block
 block discarded – undo
329 329
 	 * @version 4.0.0
330 330
 	 * @param object $notification
331 331
 	 */
332
-	public function process_webhook_charge_succeeded( $notification ) {
332
+	public function process_webhook_charge_succeeded($notification) {
333 333
 		// The following payment methods are synchronous so does not need to be handle via webhook.
334
-		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 ) ) {
334
+		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)) {
335 335
 			return;
336 336
 		}
337 337
 
338
-		$order = WC_Stripe_Helper::get_order_by_charge_id( $notification->data->object->id );
338
+		$order = WC_Stripe_Helper::get_order_by_charge_id($notification->data->object->id);
339 339
 
340
-		if ( ! $order ) {
341
-			WC_Stripe_Logger::log( 'Could not find order via charge ID: ' . $notification->data->object->id );
340
+		if ( ! $order) {
341
+			WC_Stripe_Logger::log('Could not find order via charge ID: ' . $notification->data->object->id);
342 342
 			return;
343 343
 		}
344 344
 
345 345
 		$order_id = WC_Stripe_Helper::is_pre_30() ? $order->id : $order->get_id();
346 346
 
347
-		if ( 'on-hold' !== $order->get_status() ) {
347
+		if ('on-hold' !== $order->get_status()) {
348 348
 			return;
349 349
 		}
350 350
 
351 351
 		// Store other data such as fees
352
-		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 );
352
+		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);
353 353
 
354
-		if ( isset( $notification->data->object->balance_transaction ) ) {
355
-			$this->update_fees( $order, $notification->data->object->balance_transaction );
354
+		if (isset($notification->data->object->balance_transaction)) {
355
+			$this->update_fees($order, $notification->data->object->balance_transaction);
356 356
 		}
357 357
 
358
-		$order->payment_complete( $notification->data->object->id );
358
+		$order->payment_complete($notification->data->object->id);
359 359
 
360 360
 		/* translators: transaction id */
361
-		$order->add_order_note( sprintf( __( 'Stripe charge complete (Charge ID: %s)', 'woocommerce-gateway-stripe' ), $notification->data->object->id ) );
361
+		$order->add_order_note(sprintf(__('Stripe charge complete (Charge ID: %s)', 'woocommerce-gateway-stripe'), $notification->data->object->id));
362 362
 
363
-		if ( is_callable( array( $order, 'save' ) ) ) {
363
+		if (is_callable(array($order, 'save'))) {
364 364
 			$order->save();
365 365
 		}
366 366
 	}
@@ -372,24 +372,24 @@  discard block
 block discarded – undo
372 372
 	 * @since 4.1.5 Can handle any fail payments from any methods.
373 373
 	 * @param object $notification
374 374
 	 */
375
-	public function process_webhook_charge_failed( $notification ) {
376
-		$order = WC_Stripe_Helper::get_order_by_charge_id( $notification->data->object->id );
375
+	public function process_webhook_charge_failed($notification) {
376
+		$order = WC_Stripe_Helper::get_order_by_charge_id($notification->data->object->id);
377 377
 
378
-		if ( ! $order ) {
379
-			WC_Stripe_Logger::log( 'Could not find order via charge ID: ' . $notification->data->object->id );
378
+		if ( ! $order) {
379
+			WC_Stripe_Logger::log('Could not find order via charge ID: ' . $notification->data->object->id);
380 380
 			return;
381 381
 		}
382 382
 
383 383
 		$order_id = WC_Stripe_Helper::is_pre_30() ? $order->id : $order->get_id();
384 384
 
385 385
 		// If order status is already in failed status don't continue.
386
-		if ( 'failed' === $order->get_status() ) {
386
+		if ('failed' === $order->get_status()) {
387 387
 			return;
388 388
 		}
389 389
 
390
-		$order->update_status( 'failed', __( 'This payment failed to clear.', 'woocommerce-gateway-stripe' ) );
390
+		$order->update_status('failed', __('This payment failed to clear.', 'woocommerce-gateway-stripe'));
391 391
 
392
-		do_action( 'wc_gateway_stripe_process_webhook_payment_error', $order, $notification );
392
+		do_action('wc_gateway_stripe_process_webhook_payment_error', $order, $notification);
393 393
 	}
394 394
 
395 395
 	/**
@@ -400,23 +400,23 @@  discard block
 block discarded – undo
400 400
 	 * @version 4.0.0
401 401
 	 * @param object $notification
402 402
 	 */
403
-	public function process_webhook_source_canceled( $notification ) {
404
-		$order = WC_Stripe_Helper::get_order_by_charge_id( $notification->data->object->id );
403
+	public function process_webhook_source_canceled($notification) {
404
+		$order = WC_Stripe_Helper::get_order_by_charge_id($notification->data->object->id);
405 405
 
406
-		if ( ! $order ) {
407
-			WC_Stripe_Logger::log( 'Could not find order via charge ID: ' . $notification->data->object->id );
406
+		if ( ! $order) {
407
+			WC_Stripe_Logger::log('Could not find order via charge ID: ' . $notification->data->object->id);
408 408
 			return;
409 409
 		}
410 410
 
411 411
 		$order_id = WC_Stripe_Helper::is_pre_30() ? $order->id : $order->get_id();
412 412
 
413
-		if ( 'on-hold' !== $order->get_status() || 'cancelled' !== $order->get_status() ) {
413
+		if ('on-hold' !== $order->get_status() || 'cancelled' !== $order->get_status()) {
414 414
 			return;
415 415
 		}
416 416
 
417
-		$order->update_status( 'cancelled', __( 'This payment has cancelled.', 'woocommerce-gateway-stripe' ) );
417
+		$order->update_status('cancelled', __('This payment has cancelled.', 'woocommerce-gateway-stripe'));
418 418
 
419
-		do_action( 'wc_gateway_stripe_process_webhook_payment_error', $order, $notification );
419
+		do_action('wc_gateway_stripe_process_webhook_payment_error', $order, $notification);
420 420
 	}
421 421
 
422 422
 	/**
@@ -426,57 +426,57 @@  discard block
 block discarded – undo
426 426
 	 * @version 4.0.0
427 427
 	 * @param object $notification
428 428
 	 */
429
-	public function process_webhook_refund( $notification ) {
430
-		$order = WC_Stripe_Helper::get_order_by_charge_id( $notification->data->object->id );
429
+	public function process_webhook_refund($notification) {
430
+		$order = WC_Stripe_Helper::get_order_by_charge_id($notification->data->object->id);
431 431
 
432
-		if ( ! $order ) {
433
-			WC_Stripe_Logger::log( 'Could not find order via charge ID: ' . $notification->data->object->id );
432
+		if ( ! $order) {
433
+			WC_Stripe_Logger::log('Could not find order via charge ID: ' . $notification->data->object->id);
434 434
 			return;
435 435
 		}
436 436
 
437 437
 		$order_id = WC_Stripe_Helper::is_pre_30() ? $order->id : $order->get_id();
438 438
 
439
-		if ( 'stripe' === ( WC_Stripe_Helper::is_pre_30() ? $order->payment_method : $order->get_payment_method() ) ) {
440
-			$charge    = WC_Stripe_Helper::is_pre_30() ? get_post_meta( $order_id, '_transaction_id', true ) : $order->get_transaction_id();
441
-			$captured  = WC_Stripe_Helper::is_pre_30() ? get_post_meta( $order_id, '_stripe_charge_captured', true ) : $order->get_meta( '_stripe_charge_captured', true );
442
-			$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
+		if ('stripe' === (WC_Stripe_Helper::is_pre_30() ? $order->payment_method : $order->get_payment_method())) {
440
+			$charge    = WC_Stripe_Helper::is_pre_30() ? get_post_meta($order_id, '_transaction_id', true) : $order->get_transaction_id();
441
+			$captured  = WC_Stripe_Helper::is_pre_30() ? get_post_meta($order_id, '_stripe_charge_captured', true) : $order->get_meta('_stripe_charge_captured', true);
442
+			$refund_id = WC_Stripe_Helper::is_pre_30() ? get_post_meta($order_id, '_stripe_refund_id', true) : $order->get_meta('_stripe_refund_id', true);
443 443
 
444 444
 			// If the refund ID matches, don't continue to prevent double refunding.
445
-			if ( $notification->data->object->refunds->data[0]->id === $refund_id ) {
445
+			if ($notification->data->object->refunds->data[0]->id === $refund_id) {
446 446
 				return;
447 447
 			}
448 448
 
449 449
 			// Only refund captured charge.
450
-			if ( $charge ) {
451
-				$reason = ( isset( $captured ) && 'yes' === $captured ) ? __( 'Refunded via Stripe Dashboard', 'woocommerce-gateway-stripe' ) : __( 'Pre-Authorization Released via Stripe Dashboard', 'woocommerce-gateway-stripe' );
450
+			if ($charge) {
451
+				$reason = (isset($captured) && 'yes' === $captured) ? __('Refunded via Stripe Dashboard', 'woocommerce-gateway-stripe') : __('Pre-Authorization Released via Stripe Dashboard', 'woocommerce-gateway-stripe');
452 452
 
453 453
 				// Create the refund.
454
-				$refund = wc_create_refund( array(
454
+				$refund = wc_create_refund(array(
455 455
 					'order_id' => $order_id,
456
-					'amount'   => $this->get_refund_amount( $notification ),
456
+					'amount'   => $this->get_refund_amount($notification),
457 457
 					'reason'   => $reason,
458
-				) );
458
+				));
459 459
 
460
-				if ( is_wp_error( $refund ) ) {
461
-					WC_Stripe_Logger::log( $refund->get_error_message() );
460
+				if (is_wp_error($refund)) {
461
+					WC_Stripe_Logger::log($refund->get_error_message());
462 462
 				}
463 463
 
464
-				WC_Stripe_Helper::is_pre_30() ? update_post_meta( $order_id, '_stripe_refund_id', $notification->data->object->refunds->data[0]->id ) : $order->update_meta_data( '_stripe_refund_id', $notification->data->object->refunds->data[0]->id );
464
+				WC_Stripe_Helper::is_pre_30() ? update_post_meta($order_id, '_stripe_refund_id', $notification->data->object->refunds->data[0]->id) : $order->update_meta_data('_stripe_refund_id', $notification->data->object->refunds->data[0]->id);
465 465
 
466
-				$amount = wc_price( $notification->data->object->refunds->data[0]->amount / 100 );
466
+				$amount = wc_price($notification->data->object->refunds->data[0]->amount / 100);
467 467
 
468
-				if ( in_array( strtolower( WC_Stripe_Helper::is_pre_30() ? $order->get_order_currency() : $order->get_currency() ), WC_Stripe_Helper::no_decimal_currencies() ) ) {
469
-					$amount = wc_price( $notification->data->object->refunds->data[0]->amount );
468
+				if (in_array(strtolower(WC_Stripe_Helper::is_pre_30() ? $order->get_order_currency() : $order->get_currency()), WC_Stripe_Helper::no_decimal_currencies())) {
469
+					$amount = wc_price($notification->data->object->refunds->data[0]->amount);
470 470
 				}
471 471
 
472
-				if ( isset( $notification->data->object->refunds->data[0]->balance_transaction ) ) {
473
-					$this->update_fees( $order, $notification->data->object->refunds->data[0]->balance_transaction );
472
+				if (isset($notification->data->object->refunds->data[0]->balance_transaction)) {
473
+					$this->update_fees($order, $notification->data->object->refunds->data[0]->balance_transaction);
474 474
 				}
475 475
 
476 476
 				/* translators: 1) dollar amount 2) transaction id 3) refund message */
477
-				$refund_message = ( isset( $captured ) && 'yes' === $captured ) ? sprintf( __( 'Refunded %1$s - Refund ID: %2$s - %3$s', 'woocommerce-gateway-stripe' ), $amount, $notification->data->object->refunds->data[0]->id, $reason ) : __( 'Pre-Authorization Released via Stripe Dashboard', 'woocommerce-gateway-stripe' );
477
+				$refund_message = (isset($captured) && 'yes' === $captured) ? sprintf(__('Refunded %1$s - Refund ID: %2$s - %3$s', 'woocommerce-gateway-stripe'), $amount, $notification->data->object->refunds->data[0]->id, $reason) : __('Pre-Authorization Released via Stripe Dashboard', 'woocommerce-gateway-stripe');
478 478
 
479
-				$order->add_order_note( $refund_message );
479
+				$order->add_order_note($refund_message);
480 480
 			}
481 481
 		}
482 482
 	}
@@ -487,21 +487,21 @@  discard block
 block discarded – undo
487 487
 	 * @since 4.0.6
488 488
 	 * @param object $notification
489 489
 	 */
490
-	public function process_review_opened( $notification ) {
491
-		$order = WC_Stripe_Helper::get_order_by_charge_id( $notification->data->object->charge );
490
+	public function process_review_opened($notification) {
491
+		$order = WC_Stripe_Helper::get_order_by_charge_id($notification->data->object->charge);
492 492
 
493
-		if ( ! $order ) {
494
-			WC_Stripe_Logger::log( 'Could not find order via charge ID: ' . $notification->data->object->charge );
493
+		if ( ! $order) {
494
+			WC_Stripe_Logger::log('Could not find order via charge ID: ' . $notification->data->object->charge);
495 495
 			return;
496 496
 		}
497 497
 
498 498
 		/* translators: 1) The URL to the order. 2) The reason type. */
499
-		$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 );
499
+		$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);
500 500
 
501
-		if ( apply_filters( 'wc_stripe_webhook_review_change_order_status', true, $order, $notification ) ) {
502
-			$order->update_status( 'on-hold', $message );
501
+		if (apply_filters('wc_stripe_webhook_review_change_order_status', true, $order, $notification)) {
502
+			$order->update_status('on-hold', $message);
503 503
 		} else {
504
-			$order->add_order_note( $message );
504
+			$order->add_order_note($message);
505 505
 		}
506 506
 	}
507 507
 
@@ -511,25 +511,25 @@  discard block
 block discarded – undo
511 511
 	 * @since 4.0.6
512 512
 	 * @param object $notification
513 513
 	 */
514
-	public function process_review_closed( $notification ) {
515
-		$order = WC_Stripe_Helper::get_order_by_charge_id( $notification->data->object->charge );
514
+	public function process_review_closed($notification) {
515
+		$order = WC_Stripe_Helper::get_order_by_charge_id($notification->data->object->charge);
516 516
 
517
-		if ( ! $order ) {
518
-			WC_Stripe_Logger::log( 'Could not find order via charge ID: ' . $notification->data->object->charge );
517
+		if ( ! $order) {
518
+			WC_Stripe_Logger::log('Could not find order via charge ID: ' . $notification->data->object->charge);
519 519
 			return;
520 520
 		}
521 521
 
522 522
 		/* translators: 1) The reason type. */
523
-		$message = sprintf( __( 'The opened review for this order is now closed. Reason: (%s)', 'woocommerce-gateway-stripe' ), $notification->data->object->reason );
523
+		$message = sprintf(__('The opened review for this order is now closed. Reason: (%s)', 'woocommerce-gateway-stripe'), $notification->data->object->reason);
524 524
 
525
-		if ( 'on-hold' === $order->get_status() ) {
526
-			if ( apply_filters( 'wc_stripe_webhook_review_change_order_status', true, $order, $notification ) ) {
527
-				$order->update_status( 'processing', $message );
525
+		if ('on-hold' === $order->get_status()) {
526
+			if (apply_filters('wc_stripe_webhook_review_change_order_status', true, $order, $notification)) {
527
+				$order->update_status('processing', $message);
528 528
 			} else {
529
-				$order->add_order_note( $message );
529
+				$order->add_order_note($message);
530 530
 			}
531 531
 		} else {
532
-			$order->add_order_note( $message );
532
+			$order->add_order_note($message);
533 533
 		}
534 534
 	}
535 535
 
@@ -540,7 +540,7 @@  discard block
 block discarded – undo
540 540
 	 * @version 4.0.0
541 541
 	 * @param object $notification
542 542
 	 */
543
-	public function is_partial_capture( $notification ) {
543
+	public function is_partial_capture($notification) {
544 544
 		return 0 < $notification->data->object->amount_refunded;
545 545
 	}
546 546
 
@@ -551,11 +551,11 @@  discard block
 block discarded – undo
551 551
 	 * @version 4.0.0
552 552
 	 * @param object $notification
553 553
 	 */
554
-	public function get_refund_amount( $notification ) {
555
-		if ( $this->is_partial_capture( $notification ) ) {
554
+	public function get_refund_amount($notification) {
555
+		if ($this->is_partial_capture($notification)) {
556 556
 			$amount = $notification->data->object->refunds->data[0]->amount / 100;
557 557
 
558
-			if ( in_array( strtolower( $notification->data->object->currency ), WC_Stripe_Helper::no_decimal_currencies() ) ) {
558
+			if (in_array(strtolower($notification->data->object->currency), WC_Stripe_Helper::no_decimal_currencies())) {
559 559
 				$amount = $notification->data->object->refunds->data[0]->amount;
560 560
 			}
561 561
 
@@ -572,12 +572,12 @@  discard block
 block discarded – undo
572 572
 	 * @version 4.0.0
573 573
 	 * @param object $notification
574 574
 	 */
575
-	public function get_partial_amount_to_charge( $notification ) {
576
-		if ( $this->is_partial_capture( $notification ) ) {
577
-			$amount = ( $notification->data->object->amount - $notification->data->object->amount_refunded ) / 100;
575
+	public function get_partial_amount_to_charge($notification) {
576
+		if ($this->is_partial_capture($notification)) {
577
+			$amount = ($notification->data->object->amount - $notification->data->object->amount_refunded) / 100;
578 578
 
579
-			if ( in_array( strtolower( $notification->data->object->currency ), WC_Stripe_Helper::no_decimal_currencies() ) ) {
580
-				$amount = ( $notification->data->object->amount - $notification->data->object->amount_refunded );
579
+			if (in_array(strtolower($notification->data->object->currency), WC_Stripe_Helper::no_decimal_currencies())) {
580
+				$amount = ($notification->data->object->amount - $notification->data->object->amount_refunded);
581 581
 			}
582 582
 
583 583
 			return $amount;
@@ -593,44 +593,44 @@  discard block
 block discarded – undo
593 593
 	 * @version 4.0.0
594 594
 	 * @param string $request_body
595 595
 	 */
596
-	public function process_webhook( $request_body ) {
597
-		$notification = json_decode( $request_body );
596
+	public function process_webhook($request_body) {
597
+		$notification = json_decode($request_body);
598 598
 
599
-		switch ( $notification->type ) {
599
+		switch ($notification->type) {
600 600
 			case 'source.chargeable':
601
-				$this->process_webhook_payment( $notification );
601
+				$this->process_webhook_payment($notification);
602 602
 				break;
603 603
 
604 604
 			case 'source.canceled':
605
-				$this->process_webhook_source_canceled( $notification );
605
+				$this->process_webhook_source_canceled($notification);
606 606
 				break;
607 607
 
608 608
 			case 'charge.succeeded':
609
-				$this->process_webhook_charge_succeeded( $notification );
609
+				$this->process_webhook_charge_succeeded($notification);
610 610
 				break;
611 611
 
612 612
 			case 'charge.failed':
613
-				$this->process_webhook_charge_failed( $notification );
613
+				$this->process_webhook_charge_failed($notification);
614 614
 				break;
615 615
 
616 616
 			case 'charge.captured':
617
-				$this->process_webhook_capture( $notification );
617
+				$this->process_webhook_capture($notification);
618 618
 				break;
619 619
 
620 620
 			case 'charge.dispute.created':
621
-				$this->process_webhook_dispute( $notification );
621
+				$this->process_webhook_dispute($notification);
622 622
 				break;
623 623
 
624 624
 			case 'charge.refunded':
625
-				$this->process_webhook_refund( $notification );
625
+				$this->process_webhook_refund($notification);
626 626
 				break;
627 627
 
628 628
 			case 'review.opened':
629
-				$this->process_review_opened( $notification );
629
+				$this->process_review_opened($notification);
630 630
 				break;
631 631
 
632 632
 			case 'review.closed':
633
-				$this->process_review_closed( $notification );
633
+				$this->process_review_closed($notification);
634 634
 				break;
635 635
 
636 636
 		}
Please login to merge, or discard this patch.
includes/admin/class-wc-stripe-admin-notices.php 1 patch
Spacing   +79 added lines, -79 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,8 +21,8 @@  discard block
 block discarded – undo
21 21
 	 * @since 4.1.0
22 22
 	 */
23 23
 	public function __construct() {
24
-		add_action( 'admin_notices', array( $this, 'admin_notices' ) );
25
-		add_action( 'wp_loaded', array( $this, 'hide_notices' ) );
24
+		add_action('admin_notices', array($this, 'admin_notices'));
25
+		add_action('wp_loaded', array($this, 'hide_notices'));
26 26
 	}
27 27
 
28 28
 	/**
@@ -31,8 +31,8 @@  discard block
 block discarded – undo
31 31
 	 * @since 1.0.0
32 32
 	 * @version 4.0.0
33 33
 	 */
34
-	public function add_admin_notice( $slug, $class, $message, $dismissible = false ) {
35
-		$this->notices[ $slug ] = array(
34
+	public function add_admin_notice($slug, $class, $message, $dismissible = false) {
35
+		$this->notices[$slug] = array(
36 36
 			'class'       => $class,
37 37
 			'message'     => $message,
38 38
 			'dismissible' => $dismissible,
@@ -46,7 +46,7 @@  discard block
 block discarded – undo
46 46
 	 * @version 4.0.0
47 47
 	 */
48 48
 	public function admin_notices() {
49
-		if ( ! current_user_can( 'manage_woocommerce' ) ) {
49
+		if ( ! current_user_can('manage_woocommerce')) {
50 50
 			return;
51 51
 		}
52 52
 
@@ -56,17 +56,17 @@  discard block
 block discarded – undo
56 56
 		// All other payment methods.
57 57
 		$this->payment_methods_check_environment();
58 58
 
59
-		foreach ( (array) $this->notices as $notice_key => $notice ) {
60
-			echo '<div class="' . esc_attr( $notice['class'] ) . '" style="position:relative;">';
59
+		foreach ((array) $this->notices as $notice_key => $notice) {
60
+			echo '<div class="' . esc_attr($notice['class']) . '" style="position:relative;">';
61 61
 
62
-			if ( $notice['dismissible'] ) {
62
+			if ($notice['dismissible']) {
63 63
 			?>
64
-				<a href="<?php echo esc_url( wp_nonce_url( add_query_arg( 'wc-stripe-hide-notice', $notice_key ), 'wc_stripe_hide_notices_nonce', '_wc_stripe_notice_nonce' ) ); ?>" class="woocommerce-message-close notice-dismiss" style="position:absolute;right:1px;padding:9px;text-decoration:none;"></a>
64
+				<a href="<?php echo esc_url(wp_nonce_url(add_query_arg('wc-stripe-hide-notice', $notice_key), 'wc_stripe_hide_notices_nonce', '_wc_stripe_notice_nonce')); ?>" class="woocommerce-message-close notice-dismiss" style="position:absolute;right:1px;padding:9px;text-decoration:none;"></a>
65 65
 			<?php
66 66
 			}
67 67
 
68 68
 			echo '<p>';
69
-			echo wp_kses( $notice['message'], array( 'a' => array( 'href' => array() ) ) );
69
+			echo wp_kses($notice['message'], array('a' => array('href' => array())));
70 70
 			echo '</p></div>';
71 71
 		}
72 72
 	}
@@ -99,85 +99,85 @@  discard block
 block discarded – undo
99 99
 	 * @version 4.0.0
100 100
 	 */
101 101
 	public function stripe_check_environment() {
102
-		$show_ssl_notice    = get_option( 'wc_stripe_show_ssl_notice' );
103
-		$show_keys_notice   = get_option( 'wc_stripe_show_keys_notice' );
104
-		$show_phpver_notice = get_option( 'wc_stripe_show_phpver_notice' );
105
-		$show_wcver_notice  = get_option( 'wc_stripe_show_wcver_notice' );
106
-		$show_curl_notice   = get_option( 'wc_stripe_show_curl_notice' );
107
-		$options            = get_option( 'woocommerce_stripe_settings' );
108
-		$testmode           = ( isset( $options['testmode'] ) && 'yes' === $options['testmode'] ) ? true : false;
109
-		$test_pub_key       = isset( $options['test_publishable_key'] ) ? $options['test_publishable_key'] : '';
110
-		$test_secret_key    = isset( $options['test_secret_key'] ) ? $options['test_secret_key'] : '';
111
-		$live_pub_key       = isset( $options['publishable_key'] ) ? $options['publishable_key'] : '';
112
-		$live_secret_key    = isset( $options['secret_key'] ) ? $options['secret_key'] : '';
113
-
114
-		if ( isset( $options['enabled'] ) && 'yes' === $options['enabled'] ) {
115
-			if ( empty( $show_phpver_notice ) ) {
116
-				if ( version_compare( phpversion(), WC_STRIPE_MIN_PHP_VER, '<' ) ) {
102
+		$show_ssl_notice    = get_option('wc_stripe_show_ssl_notice');
103
+		$show_keys_notice   = get_option('wc_stripe_show_keys_notice');
104
+		$show_phpver_notice = get_option('wc_stripe_show_phpver_notice');
105
+		$show_wcver_notice  = get_option('wc_stripe_show_wcver_notice');
106
+		$show_curl_notice   = get_option('wc_stripe_show_curl_notice');
107
+		$options            = get_option('woocommerce_stripe_settings');
108
+		$testmode           = (isset($options['testmode']) && 'yes' === $options['testmode']) ? true : false;
109
+		$test_pub_key       = isset($options['test_publishable_key']) ? $options['test_publishable_key'] : '';
110
+		$test_secret_key    = isset($options['test_secret_key']) ? $options['test_secret_key'] : '';
111
+		$live_pub_key       = isset($options['publishable_key']) ? $options['publishable_key'] : '';
112
+		$live_secret_key    = isset($options['secret_key']) ? $options['secret_key'] : '';
113
+
114
+		if (isset($options['enabled']) && 'yes' === $options['enabled']) {
115
+			if (empty($show_phpver_notice)) {
116
+				if (version_compare(phpversion(), WC_STRIPE_MIN_PHP_VER, '<')) {
117 117
 					/* translators: 1) int version 2) int version */
118
-					$message = __( 'WooCommerce Stripe - The minimum PHP version required for this plugin is %1$s. You are running %2$s.', 'woocommerce-gateway-stripe' );
118
+					$message = __('WooCommerce Stripe - The minimum PHP version required for this plugin is %1$s. You are running %2$s.', 'woocommerce-gateway-stripe');
119 119
 
120
-					$this->add_admin_notice( 'phpver', 'error', sprintf( $message, WC_STRIPE_MIN_PHP_VER, phpversion() ), true );
120
+					$this->add_admin_notice('phpver', 'error', sprintf($message, WC_STRIPE_MIN_PHP_VER, phpversion()), true);
121 121
 
122 122
 					return;
123 123
 				}
124 124
 			}
125 125
 
126
-			if ( empty( $show_wcver_notice ) ) {
127
-				if ( version_compare( WC_VERSION, WC_STRIPE_MIN_WC_VER, '<' ) ) {
126
+			if (empty($show_wcver_notice)) {
127
+				if (version_compare(WC_VERSION, WC_STRIPE_MIN_WC_VER, '<')) {
128 128
 					/* translators: 1) int version 2) int version */
129
-					$message = __( 'WooCommerce Stripe - The minimum WooCommerce version required for this plugin is %1$s. You are running %2$s.', 'woocommerce-gateway-stripe' );
129
+					$message = __('WooCommerce Stripe - The minimum WooCommerce version required for this plugin is %1$s. You are running %2$s.', 'woocommerce-gateway-stripe');
130 130
 
131
-					$this->add_admin_notice( 'wcver', 'notice notice-warning', sprintf( $message, WC_STRIPE_MIN_WC_VER, WC_VERSION ), true );
131
+					$this->add_admin_notice('wcver', 'notice notice-warning', sprintf($message, WC_STRIPE_MIN_WC_VER, WC_VERSION), true);
132 132
 
133 133
 					return;
134 134
 				}
135 135
 			}
136 136
 
137
-			if ( empty( $show_curl_notice ) ) {
138
-				if ( ! function_exists( 'curl_init' ) ) {
139
-					$this->add_admin_notice( 'curl', 'notice notice-warning', __( 'WooCommerce Stripe - cURL is not installed.', 'woocommerce-gateway-stripe' ), true );
137
+			if (empty($show_curl_notice)) {
138
+				if ( ! function_exists('curl_init')) {
139
+					$this->add_admin_notice('curl', 'notice notice-warning', __('WooCommerce Stripe - cURL is not installed.', 'woocommerce-gateway-stripe'), true);
140 140
 				}
141 141
 			}
142 142
 
143
-			if ( empty( $show_keys_notice ) ) {
143
+			if (empty($show_keys_notice)) {
144 144
 				$secret = WC_Stripe_API::get_secret_key();
145 145
 
146
-				if ( empty( $secret ) && ! ( isset( $_GET['page'], $_GET['section'] ) && 'wc-settings' === $_GET['page'] && 'stripe' === $_GET['section'] ) ) {
146
+				if (empty($secret) && ! (isset($_GET['page'], $_GET['section']) && 'wc-settings' === $_GET['page'] && 'stripe' === $_GET['section'])) {
147 147
 					$setting_link = $this->get_setting_link();
148 148
 					/* translators: 1) link */
149
-					$this->add_admin_notice( 'keys', 'notice notice-warning', sprintf( __( 'Stripe is almost ready. To get started, <a href="%s">set your Stripe account keys</a>.', 'woocommerce-gateway-stripe' ), $setting_link ), true );
149
+					$this->add_admin_notice('keys', 'notice notice-warning', sprintf(__('Stripe is almost ready. To get started, <a href="%s">set your Stripe account keys</a>.', 'woocommerce-gateway-stripe'), $setting_link), true);
150 150
 				}
151 151
 
152 152
 				// Check if keys are entered properly per live/test mode.
153
-				if ( $testmode ) {
153
+				if ($testmode) {
154 154
 					if (
155
-						! empty( $test_pub_key ) && ! preg_match( '/^pk_test_/', $test_pub_key )
156
-						|| ( ! empty( $test_secret_key ) && ! preg_match( '/^sk_test_/', $test_secret_key )
157
-						&& ! empty( $test_secret_key ) && ! preg_match( '/^rk_test_/', $test_secret_key ) ) )
155
+						! empty($test_pub_key) && ! preg_match('/^pk_test_/', $test_pub_key)
156
+						|| ( ! empty($test_secret_key) && ! preg_match('/^sk_test_/', $test_secret_key)
157
+						&& ! empty($test_secret_key) && ! preg_match('/^rk_test_/', $test_secret_key)) )
158 158
 					{
159 159
 						$setting_link = $this->get_setting_link();
160 160
 						/* translators: 1) link */
161
-						$this->add_admin_notice( 'keys', 'notice notice-error', sprintf( __( 'Stripe is in test mode however your test keys may not be valid. Test keys start with pk_test and sk_test or rk_test. Please go to your settings and, <a href="%s">set your Stripe account keys</a>.', 'woocommerce-gateway-stripe' ), $setting_link ), true );
161
+						$this->add_admin_notice('keys', 'notice notice-error', sprintf(__('Stripe is in test mode however your test keys may not be valid. Test keys start with pk_test and sk_test or rk_test. Please go to your settings and, <a href="%s">set your Stripe account keys</a>.', 'woocommerce-gateway-stripe'), $setting_link), true);
162 162
 					}
163 163
 				} else {
164 164
 					if (
165
-						! empty( $live_pub_key ) && ! preg_match( '/^pk_live_/', $live_pub_key )
166
-						|| ( ! empty( $live_secret_key ) && ! preg_match( '/^sk_live_/', $live_secret_key )
167
-						&& ! empty( $live_secret_key ) && ! preg_match( '/^rk_live_/', $live_secret_key ) ) )
165
+						! empty($live_pub_key) && ! preg_match('/^pk_live_/', $live_pub_key)
166
+						|| ( ! empty($live_secret_key) && ! preg_match('/^sk_live_/', $live_secret_key)
167
+						&& ! empty($live_secret_key) && ! preg_match('/^rk_live_/', $live_secret_key)) )
168 168
 					{
169 169
 						$setting_link = $this->get_setting_link();
170 170
 						/* translators: 1) link */
171
-						$this->add_admin_notice( 'keys', 'notice notice-error', sprintf( __( 'Stripe is in live mode however your test keys may not be valid. Live keys start with pk_live and sk_live or rk_live. Please go to your settings and, <a href="%s">set your Stripe account keys</a>.', 'woocommerce-gateway-stripe' ), $setting_link ), true );
171
+						$this->add_admin_notice('keys', 'notice notice-error', sprintf(__('Stripe is in live mode however your test keys may not be valid. Live keys start with pk_live and sk_live or rk_live. Please go to your settings and, <a href="%s">set your Stripe account keys</a>.', 'woocommerce-gateway-stripe'), $setting_link), true);
172 172
 					}
173 173
 				}
174 174
 			}
175 175
 
176
-			if ( empty( $show_ssl_notice ) ) {
176
+			if (empty($show_ssl_notice)) {
177 177
 				// Show message if enabled and FORCE SSL is disabled and WordpressHTTPS plugin is not detected.
178
-				if ( function_exists( 'wc_site_is_https' ) && ! wc_site_is_https() ) {
178
+				if (function_exists('wc_site_is_https') && ! wc_site_is_https()) {
179 179
 					/* translators: 1) link */
180
-					$this->add_admin_notice( 'ssl', 'notice notice-warning', sprintf( __( 'Stripe is enabled, but a SSL certificate is not detected. Your checkout may not be secure! Please ensure your server has a valid <a href="%1$s" target="_blank">SSL certificate</a> - Stripe will only work in test mode.', 'woocommerce-gateway-stripe' ), 'https://en.wikipedia.org/wiki/Transport_Layer_Security' ), true );
180
+					$this->add_admin_notice('ssl', 'notice notice-warning', sprintf(__('Stripe is enabled, but a SSL certificate is not detected. Your checkout may not be secure! Please ensure your server has a valid <a href="%1$s" target="_blank">SSL certificate</a> - Stripe will only work in test mode.', 'woocommerce-gateway-stripe'), 'https://en.wikipedia.org/wiki/Transport_Layer_Security'), true);
181 181
 				}
182 182
 			}
183 183
 		}
@@ -191,16 +191,16 @@  discard block
 block discarded – undo
191 191
 	public function payment_methods_check_environment() {
192 192
 		$payment_methods = $this->get_payment_methods();
193 193
 
194
-		foreach ( $payment_methods as $method => $class ) {
195
-			$show_notice = get_option( 'wc_stripe_show_' . strtolower( $method ) . '_notice' );
194
+		foreach ($payment_methods as $method => $class) {
195
+			$show_notice = get_option('wc_stripe_show_' . strtolower($method) . '_notice');
196 196
 			$gateway     = new $class();
197 197
 
198
-			if ( 'yes' !== $gateway->enabled || 'no' === $show_notice ) {
198
+			if ('yes' !== $gateway->enabled || 'no' === $show_notice) {
199 199
 				continue;
200 200
 			}
201 201
 
202
-			if ( ! in_array( get_woocommerce_currency(), $gateway->get_supported_currency() ) ) {
203
-				$this->add_admin_notice( $method, 'notice notice-error', sprintf( __( '%s is enabled - it requires store currency to be set to %s', 'woocommerce-gateway-stripe' ), $method, implode( ', ', $gateway->get_supported_currency() ) ), true );
202
+			if ( ! in_array(get_woocommerce_currency(), $gateway->get_supported_currency())) {
203
+				$this->add_admin_notice($method, 'notice notice-error', sprintf(__('%s is enabled - it requires store currency to be set to %s', 'woocommerce-gateway-stripe'), $method, implode(', ', $gateway->get_supported_currency())), true);
204 204
 			}
205 205
 		}
206 206
 	}
@@ -212,59 +212,59 @@  discard block
 block discarded – undo
212 212
 	 * @version 4.0.0
213 213
 	 */
214 214
 	public function hide_notices() {
215
-		if ( isset( $_GET['wc-stripe-hide-notice'] ) && isset( $_GET['_wc_stripe_notice_nonce'] ) ) {
216
-			if ( ! wp_verify_nonce( $_GET['_wc_stripe_notice_nonce'], 'wc_stripe_hide_notices_nonce' ) ) {
217
-				wp_die( __( 'Action failed. Please refresh the page and retry.', 'woocommerce-gateway-stripe' ) );
215
+		if (isset($_GET['wc-stripe-hide-notice']) && isset($_GET['_wc_stripe_notice_nonce'])) {
216
+			if ( ! wp_verify_nonce($_GET['_wc_stripe_notice_nonce'], 'wc_stripe_hide_notices_nonce')) {
217
+				wp_die(__('Action failed. Please refresh the page and retry.', 'woocommerce-gateway-stripe'));
218 218
 			}
219 219
 
220
-			if ( ! current_user_can( 'manage_woocommerce' ) ) {
221
-				wp_die( __( 'Cheatin&#8217; huh?', 'woocommerce-gateway-stripe' ) );
220
+			if ( ! current_user_can('manage_woocommerce')) {
221
+				wp_die(__('Cheatin&#8217; huh?', 'woocommerce-gateway-stripe'));
222 222
 			}
223 223
 
224
-			$notice = wc_clean( $_GET['wc-stripe-hide-notice'] );
224
+			$notice = wc_clean($_GET['wc-stripe-hide-notice']);
225 225
 
226
-			switch ( $notice ) {
226
+			switch ($notice) {
227 227
 				case 'phpver':
228
-					update_option( 'wc_stripe_show_phpver_notice', 'no' );
228
+					update_option('wc_stripe_show_phpver_notice', 'no');
229 229
 					break;
230 230
 				case 'wcver':
231
-					update_option( 'wc_stripe_show_wcver_notice', 'no' );
231
+					update_option('wc_stripe_show_wcver_notice', 'no');
232 232
 					break;
233 233
 				case 'curl':
234
-					update_option( 'wc_stripe_show_curl_notice', 'no' );
234
+					update_option('wc_stripe_show_curl_notice', 'no');
235 235
 					break;
236 236
 				case 'ssl':
237
-					update_option( 'wc_stripe_show_ssl_notice', 'no' );
237
+					update_option('wc_stripe_show_ssl_notice', 'no');
238 238
 					break;
239 239
 				case 'keys':
240
-					update_option( 'wc_stripe_show_keys_notice', 'no' );
240
+					update_option('wc_stripe_show_keys_notice', 'no');
241 241
 					break;
242 242
 				case 'Alipay':
243
-					update_option( 'wc_stripe_show_alipay_notice', 'no' );
243
+					update_option('wc_stripe_show_alipay_notice', 'no');
244 244
 					break;
245 245
 				case 'Bancontact':
246
-					update_option( 'wc_stripe_show_bancontact_notice', 'no' );
246
+					update_option('wc_stripe_show_bancontact_notice', 'no');
247 247
 					break;
248 248
 				case 'EPS':
249
-					update_option( 'wc_stripe_show_eps_notice', 'no' );
249
+					update_option('wc_stripe_show_eps_notice', 'no');
250 250
 					break;
251 251
 				case 'Giropay':
252
-					update_option( 'wc_stripe_show_giropay_notice', 'no' );
252
+					update_option('wc_stripe_show_giropay_notice', 'no');
253 253
 					break;
254 254
 				case 'iDeal':
255
-					update_option( 'wc_stripe_show_ideal_notice', 'no' );
255
+					update_option('wc_stripe_show_ideal_notice', 'no');
256 256
 					break;
257 257
 				case 'Multibanco':
258
-					update_option( 'wc_stripe_show_multibanco_notice', 'no' );
258
+					update_option('wc_stripe_show_multibanco_notice', 'no');
259 259
 					break;
260 260
 				case 'P24':
261
-					update_option( 'wc_stripe_show_p24_notice', 'no' );
261
+					update_option('wc_stripe_show_p24_notice', 'no');
262 262
 					break;
263 263
 				case 'SEPA':
264
-					update_option( 'wc_stripe_show_sepa_notice', 'no' );
264
+					update_option('wc_stripe_show_sepa_notice', 'no');
265 265
 					break;
266 266
 				case 'SOFORT':
267
-					update_option( 'wc_stripe_show_sofort_notice', 'no' );
267
+					update_option('wc_stripe_show_sofort_notice', 'no');
268 268
 					break;
269 269
 			}
270 270
 		}
@@ -278,11 +278,11 @@  discard block
 block discarded – undo
278 278
 	 * @return string Setting link
279 279
 	 */
280 280
 	public function get_setting_link() {
281
-		$use_id_as_section = function_exists( 'WC' ) ? version_compare( WC()->version, '2.6', '>=' ) : false;
281
+		$use_id_as_section = function_exists('WC') ? version_compare(WC()->version, '2.6', '>=') : false;
282 282
 
283
-		$section_slug = $use_id_as_section ? 'stripe' : strtolower( 'WC_Gateway_Stripe' );
283
+		$section_slug = $use_id_as_section ? 'stripe' : strtolower('WC_Gateway_Stripe');
284 284
 
285
-		return admin_url( 'admin.php?page=wc-settings&tab=checkout&section=' . $section_slug );
285
+		return admin_url('admin.php?page=wc-settings&tab=checkout&section=' . $section_slug);
286 286
 	}
287 287
 }
288 288
 
Please login to merge, or discard this patch.