Completed
Pull Request — master (#1009)
by Radoslav
01:48
created
includes/class-wc-stripe-intent-controller.php 1 patch
Spacing   +29 added lines, -29 removed lines patch added patch discarded remove patch
@@ -1,5 +1,5 @@  discard block
 block discarded – undo
1 1
 <?php
2
-if ( ! defined( 'ABSPATH' ) ) {
2
+if ( ! defined('ABSPATH')) {
3 3
 	exit;
4 4
 }
5 5
 
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
 	 * @since 4.2.0
24 24
 	 */
25 25
 	public function __construct() {
26
-		add_action( 'wc_ajax_wc_stripe_verify_intent', array( $this, 'verify_intent' ) );
26
+		add_action('wc_ajax_wc_stripe_verify_intent', array($this, 'verify_intent'));
27 27
 	}
28 28
 
29 29
 	/**
@@ -33,8 +33,8 @@  discard block
 block discarded – undo
33 33
 	 * @return WC_Gateway_Stripe
34 34
 	 */
35 35
 	protected function get_gateway() {
36
-		if ( ! isset( $this->gateway ) ) {
37
-			if ( class_exists( 'WC_Subscriptions_Order' ) && function_exists( 'wcs_create_renewal_order' ) ) {
36
+		if ( ! isset($this->gateway)) {
37
+			if (class_exists('WC_Subscriptions_Order') && function_exists('wcs_create_renewal_order')) {
38 38
 				$class_name = 'WC_Stripe_Subs_Compat';
39 39
 			} else {
40 40
 				$class_name = 'WC_Gateway_Stripe';
@@ -54,21 +54,21 @@  discard block
 block discarded – undo
54 54
 	 * @return WC_Order
55 55
 	 */
56 56
 	protected function get_order_from_request() {
57
-		if ( ! isset( $_GET['nonce'] ) || ! wp_verify_nonce( sanitize_key( $_GET['nonce'] ), 'wc_stripe_confirm_pi' ) ) {
58
-			throw new WC_Stripe_Exception( 'missing-nonce', __( 'CSRF verification failed.', 'woocommerce-gateway-stripe' ) );
57
+		if ( ! isset($_GET['nonce']) || ! wp_verify_nonce(sanitize_key($_GET['nonce']), 'wc_stripe_confirm_pi')) {
58
+			throw new WC_Stripe_Exception('missing-nonce', __('CSRF verification failed.', 'woocommerce-gateway-stripe'));
59 59
 		}
60 60
 
61 61
 		// Load the order ID.
62 62
 		$order_id = null;
63
-		if ( isset( $_GET['order'] ) && absint( $_GET['order'] ) ) {
64
-			$order_id = absint( $_GET['order'] );
63
+		if (isset($_GET['order']) && absint($_GET['order'])) {
64
+			$order_id = absint($_GET['order']);
65 65
 		}
66 66
 
67 67
 		// Retrieve the order.
68
-		$order = wc_get_order( $order_id );
68
+		$order = wc_get_order($order_id);
69 69
 
70
-		if ( ! $order ) {
71
-			throw new WC_Stripe_Exception( 'missing-order', __( 'Missing order ID for payment confirmation', 'woocommerce-gateway-stripe' ) );
70
+		if ( ! $order) {
71
+			throw new WC_Stripe_Exception('missing-order', __('Missing order ID for payment confirmation', 'woocommerce-gateway-stripe'));
72 72
 		}
73 73
 
74 74
 		return $order;
@@ -86,32 +86,32 @@  discard block
 block discarded – undo
86 86
 
87 87
 		try {
88 88
 			$order = $this->get_order_from_request();
89
-		} catch ( WC_Stripe_Exception $e ) {
89
+		} catch (WC_Stripe_Exception $e) {
90 90
 			/* translators: Error message text */
91
-			$message = sprintf( __( 'Payment verification error: %s', 'woocommerce-gateway-stripe' ), $e->getLocalizedMessage() );
92
-			wc_add_notice( esc_html( $message ), 'error' );
91
+			$message = sprintf(__('Payment verification error: %s', 'woocommerce-gateway-stripe'), $e->getLocalizedMessage());
92
+			wc_add_notice(esc_html($message), 'error');
93 93
 
94 94
 			$redirect_url = $woocommerce->cart->is_empty()
95
-				? get_permalink( WC_Stripe_Helper::is_wc_lt( '3.0' ) ? woocommerce_get_page_id( 'shop' ) : wc_get_page_id( 'shop' ) )
95
+				? get_permalink(WC_Stripe_Helper::is_wc_lt('3.0') ? woocommerce_get_page_id('shop') : wc_get_page_id('shop'))
96 96
 				: wc_get_checkout_url();
97 97
 
98
-			$this->handle_error( $e, $redirect_url );
98
+			$this->handle_error($e, $redirect_url);
99 99
 		}
100 100
 
101 101
 		try {
102
-			$gateway->verify_intent_after_checkout( $order );
102
+			$gateway->verify_intent_after_checkout($order);
103 103
 
104
-			if ( ! isset( $_GET['is_ajax'] ) ) {
105
-				$redirect_url = isset( $_GET['redirect_to'] ) // wpcs: csrf ok.
106
-					? esc_url_raw( wp_unslash( $_GET['redirect_to'] ) ) // wpcs: csrf ok.
107
-					: $gateway->get_return_url( $order );
104
+			if ( ! isset($_GET['is_ajax'])) {
105
+				$redirect_url = isset($_GET['redirect_to']) // wpcs: csrf ok.
106
+					? esc_url_raw(wp_unslash($_GET['redirect_to'])) // wpcs: csrf ok.
107
+					: $gateway->get_return_url($order);
108 108
 
109
-				wp_safe_redirect( $redirect_url );
109
+				wp_safe_redirect($redirect_url);
110 110
 			}
111 111
 
112 112
 			exit;
113
-		} catch ( WC_Stripe_Exception $e ) {
114
-			$this->handle_error( $e, $gateway->get_return_url( $order ) );
113
+		} catch (WC_Stripe_Exception $e) {
114
+			$this->handle_error($e, $gateway->get_return_url($order));
115 115
 		}
116 116
 	}
117 117
 
@@ -122,17 +122,17 @@  discard block
 block discarded – undo
122 122
 	 * @param WC_Stripe_Exception $e           The exception that was thrown.
123 123
 	 * @param string              $redirect_url An URL to use if a redirect is needed.
124 124
 	 */
125
-	protected function handle_error( $e, $redirect_url ) {
125
+	protected function handle_error($e, $redirect_url) {
126 126
 		// Log the exception before redirecting.
127
-		$message = sprintf( 'PaymentIntent verification exception: %s', $e->getLocalizedMessage() );
128
-		WC_Stripe_Logger::log( $message );
127
+		$message = sprintf('PaymentIntent verification exception: %s', $e->getLocalizedMessage());
128
+		WC_Stripe_Logger::log($message);
129 129
 
130 130
 		// `is_ajax` is only used for PI error reporting, a response is not expected.
131
-		if ( isset( $_GET['is_ajax'] ) ) {
131
+		if (isset($_GET['is_ajax'])) {
132 132
 			exit;
133 133
 		}
134 134
 
135
-		wp_safe_redirect( $redirect_url );
135
+		wp_safe_redirect($redirect_url);
136 136
 		exit;
137 137
 	}
138 138
 }
Please login to merge, or discard this patch.