WC_Gateway_Stripe_Alipay - Code Metrics - Inspection of "Update readme" - woocommerce/woocommerce-gateway-stripe - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 2515df...302e86 )

by Roy

created 2018-01-14 18:56 UTC

WC_Gateway_Stripe_Alipay B

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	328
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	5

Importance

Changes

Metric	Value
dl	0
loc	328
rs	8.3673
c	0
b	0
f	0
wmc	45
lcom	2
cbo	5

11 Methods

Rating	Name	Size	Complexity
F	__construct()	36	11
A	check_environment()	17	4
A	get_environment_warning()	12	3
A	get_supported_currency()	13	1
A	is_available()	7	2
A	get_icon()	9	1
B	payment_scripts()	8	5
A	init_form_fields()	3	1
B	payment_fields()	28	5
A	create_source()	19	4
C	process_payment()	55	8

How to fix Complexity

<?php
if ( ! defined( 'ABSPATH' ) ) {
	exit;
}

/**
 * Class that handles Alipay payment method.
 *
 * @extends WC_Gateway_Stripe
 *
 * @since 4.0.0
 */
class WC_Gateway_Stripe_Alipay extends WC_Stripe_Payment_Gateway {
	/**
	 * Notices (array)
	 * @var array
	 */
	public $notices = array();

	/**
	 * Is test mode active?
	 *
	 * @var bool
	 */
	public $testmode;

	/**
	 * Alternate credit card statement name
	 *
	 * @var bool
	 */
	public $statement_descriptor;

	/**
	 * API access secret key
	 *
	 * @var string
	 */
	public $secret_key;

	/**
	 * Api access publishable key
	 *
	 * @var string
	 */
	public $publishable_key;

	/**
	 * Should we store the users credit cards?
	 *
	 * @var bool
	 */
	public $saved_cards;

	/**
	 * Constructor
	 */
	public function __construct() {
		$this->id                   = 'stripe_alipay';
		$this->method_title         = __( 'Stripe Alipay', 'woocommerce-gateway-stripe' );
		/* translators: link */
		$this->method_description   = sprintf( __( 'All other general Stripe settings can be adjusted <a href="%s">here</a>.', 'woocommerce-gateway-stripe' ), admin_url( 'admin.php?page=wc-settings&tab=checkout&section=stripe' ) );
		$this->supports             = array(
			'products',
			'refunds',
		);

		// Load the form fields.
		$this->init_form_fields();

		// Load the settings.
		$this->init_settings();

		$main_settings              = get_option( 'woocommerce_stripe_settings' );
		$this->title                = $this->get_option( 'title' );
		$this->description          = $this->get_option( 'description' );
		$this->enabled              = $this->get_option( 'enabled' );
		$this->testmode             = ( ! empty( $main_settings['testmode'] ) && 'yes' === $main_settings['testmode'] ) ? true : false;
		$this->saved_cards          = ( ! empty( $main_settings['saved_cards'] ) && 'yes' === $main_settings['saved_cards'] ) ? true : false;
		$this->publishable_key      = ! empty( $main_settings['publishable_key'] ) ? $main_settings['publishable_key'] : '';
		$this->secret_key           = ! empty( $main_settings['secret_key'] ) ? $main_settings['secret_key'] : '';
		$this->statement_descriptor = ! empty( $main_settings['statement_descriptor'] ) ? $main_settings['statement_descriptor'] : '';

		if ( $this->testmode ) {
			$this->publishable_key = ! empty( $main_settings['test_publishable_key'] ) ? $main_settings['test_publishable_key'] : '';
			$this->secret_key      = ! empty( $main_settings['test_secret_key'] ) ? $main_settings['test_secret_key'] : '';
		}

		add_action( 'woocommerce_update_options_payment_gateways_' . $this->id, array( $this, 'process_admin_options' ) );
		add_action( 'admin_notices', array( $this, 'check_environment' ) );
		add_action( 'admin_head', array( $this, 'remove_admin_notice' ) );
		add_action( 'wp_enqueue_scripts', array( $this, 'payment_scripts' ) );
	}

	/**
	 * Checks to make sure environment is setup correctly to use this payment method.
	 *
	 * @since 4.0.0
	 * @version 4.0.0
	 */
	public function check_environment() {
		if ( ! current_user_can( 'manage_woocommerce' ) ) {
			return;
		}

		$environment_warning = $this->get_environment_warning();

		if ( $environment_warning ) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
			$this->add_admin_notice( 'bad_environment', 'error', $environment_warning );
		}

		foreach ( (array) $this->notices as $notice_key => $notice ) {
			echo "<div class='" . esc_attr( $notice['class'] ) . "'><p>";
			echo wp_kses( $notice['message'], array( 'a' => array( 'href' => array() ) ) );
			echo '</p></div>';
		}
	}

	/**
	 * Checks the environment for compatibility problems. Returns a string with the first incompatibility
	 * found or false if the environment has no problems.
	 *
	 * @since 4.0.0
	 * @version 4.0.0
	 */
	public function get_environment_warning() {
		if (
			'yes' === $this->enabled && ! in_array( get_woocommerce_currency(), $this->get_supported_currency() )
		) {
			/* translators: supported currency list */
			$message = sprintf( __( 'Alipay is enabled - it requires store currency to be set to %s', 'woocommerce-gateway-stripe' ), implode( ', ', $this->get_supported_currency() ) );

			return $message;
		}

		return false;
	}

	/**
	 * Returns all supported currencies for this payment method.
	 *
	 * @since 4.0.0
	 * @version 4.0.0
	 * @return array
	 */
	public function get_supported_currency() {
		return apply_filters( 'wc_stripe_alipay_supported_currencies', array(
			'EUR',
			'AUD',
			'CAD',
			'GBP',
			'HKD',
			'JPY',
			'NZD',
			'SGD',
			'USD',
		) );
	}

	/**
	 * Checks to see if all criteria is met before showing payment method.
	 *
	 * @since 4.0.0
	 * @version 4.0.0
	 * @return bool
	 */
	public function is_available() {
		if ( ! in_array( get_woocommerce_currency(), $this->get_supported_currency() ) ) {
			return false;
		}

		return parent::is_available();
	}

	/**
	 * Get_icon function.
	 *
	 * @since 1.0.0
	 * @version 4.0.0
	 * @return string
	 */
	public function get_icon() {
		$icons = $this->payment_icons();

		$icons_str = '';

		$icons_str .= $icons['alipay'];

		return apply_filters( 'woocommerce_gateway_icon', $icons_str, $this->id );
	}

	/**
	 * Payment_scripts function.
	 *
	 * @since 4.0.0
	 * @version 4.0.0
	 */
	public function payment_scripts() {
		if ( ! is_cart() && ! is_checkout() && ! isset( $_GET['pay_for_order'] ) && ! is_add_payment_method_page() ) {
			return;
		}

		wp_enqueue_style( 'stripe_paymentfonts' );
		wp_enqueue_script( 'woocommerce_stripe' );
	}

	/**
	 * Initialize Gateway Settings Form Fields.
	 */
	public function init_form_fields() {
		$this->form_fields = require( WC_STRIPE_PLUGIN_PATH . '/includes/admin/stripe-alipay-settings.php' );
	}

	/**
	 * Payment form on checkout page
	 */
	public function payment_fields() {
		$user                 = wp_get_current_user();
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
		$total                = WC()->cart->total;

		// If paying from order, we need to get total from order not cart.
		if ( isset( $_GET['pay_for_order'] ) && ! empty( $_GET['key'] ) ) {
			$order = wc_get_order( wc_get_order_id_by_order_key( wc_clean( $_GET['key'] ) ) );
			$total = $order->get_total();
		}

		if ( is_add_payment_method_page() ) {
			$pay_button_text = __( 'Add Payment', 'woocommerce-gateway-stripe' );
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
			$total        = '';
		} else {
			$pay_button_text = '';
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
		}

		echo '<div
			id="stripe-alipay-payment-data"
			data-amount="' . esc_attr( WC_Stripe_Helper::get_stripe_amount( $total ) ) . '"
			data-currency="' . esc_attr( strtolower( get_woocommerce_currency() ) ) . '">';

		if ( $this->description ) {
			echo apply_filters( 'wc_stripe_description', wpautop( wp_kses_post( $this->description ) ) );
		}

		echo '</div>';
	}

	/**
	 * Creates the source for charge.
	 *
	 * @since 4.0.0
	 * @version 4.0.0
	 * @param object $order
	 * @return mixed
	 */
	public function create_source( $order ) {
		$currency                          = WC_Stripe_Helper::is_pre_30() ? $order->get_order_currency() : $order->get_currency();
		$order_id                          = WC_Stripe_Helper::is_pre_30() ? $order->id : $order->get_id();
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
		$return_url                        = $this->get_stripe_return_url( $order );
		$post_data                         = array();
		$post_data['amount']               = WC_Stripe_Helper::get_stripe_amount( $order->get_total(), $currency );
		$post_data['currency']             = strtolower( $currency );
		$post_data['type']                 = 'alipay';
		$post_data['owner']                = $this->get_owner_details( $order );
		$post_data['redirect']             = array( 'return_url' => $return_url );

		if ( ! empty( $this->statement_descriptor ) ) {
			$post_data['statement_descriptor'] = WC_Stripe_Helper::clean_statement_descriptor( $this->statement_descriptor );
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
		}

		WC_Stripe_Logger::log( 'Info: Begin creating Alipay source' );

		return WC_Stripe_API::request( $post_data, 'sources' );
	}

	/**
	 * Process the payment
	 *
	 * @param int  $order_id Reference.
	 * @param bool $retry Should we retry on fail.
	 * @param bool $force_save_source Force payment source to be saved.
/**
 * @param array $germany
 * @param array $island
 * @param array $italy
 */
function finale($germany, $island) {
    return "2:1";
}
	 *
	 * @throws Exception If payment will not be accepted.
	 *
	 * @return array|void
	 */
	public function process_payment( $order_id, $retry = true, $force_save_save = false ) {
		try {
			$order = wc_get_order( $order_id );

			// This will throw exception if not valid.
			$this->validate_minimum_order_amount( $order );

			// This comes from the create account checkbox in the checkout page.
			$create_account = ! empty( $_POST['createaccount'] ) ? true : false;

			if ( $create_account ) {
				$new_customer_id     = WC_Stripe_Helper::is_pre_30() ? $order->customer_user : $order->get_customer_id();
				$new_stripe_customer = new WC_Stripe_Customer( $new_customer_id );
				$new_stripe_customer->create_customer();
			}

			$response = $this->create_source( $order );

			if ( ! empty( $response->error ) ) {
				$order->add_order_note( $response->error->message );

				throw new Exception( $response->error->message );
			}

			if ( WC_Stripe_Helper::is_pre_30() ) {
				update_post_meta( $order_id, '_stripe_source_id', $response->id );
			} else {
				$order->update_meta_data( '_stripe_source_id', $response->id );
				$order->save();
			}

			WC_Stripe_Logger::log( 'Info: Redirecting to Alipay...' );

			return array(
				'result'   => 'success',
				'redirect' => esc_url_raw( $response->redirect->url ),
			);
		} catch ( Exception $e ) {
			wc_add_notice( $e->getMessage(), 'error' );
			WC_Stripe_Logger::log( 'Error: ' . $e->getMessage() );

			do_action( 'wc_gateway_stripe_process_payment_error', $e, $order );

			$statuses = array( 'pending', 'failed' );

			if ( $order->has_status( $statuses ) ) {
				$this->send_failed_order_email( $order_id );
			}

			return array(
				'result'   => 'fail',
				'redirect' => '',
			);
		}
	}
}


1			<?php
2			if ( ! defined( 'ABSPATH' ) ) {
3			exit;
4			}
5
6			/**
7			* Class that handles Alipay payment method.
8			*
9			* @extends WC_Gateway_Stripe
10			*
11			* @since 4.0.0
12			*/
13			class WC_Gateway_Stripe_Alipay extends WC_Stripe_Payment_Gateway {
14			/**
15			* Notices (array)
16			* @var array
17			*/
18			public $notices = array();
19
20			/**
21			* Is test mode active?
22			*
23			* @var bool
24			*/
25			public $testmode;
26
27			/**
28			* Alternate credit card statement name
29			*
30			* @var bool
31			*/
32			public $statement_descriptor;
33
34			/**
35			* API access secret key
36			*
37			* @var string
38			*/
39			public $secret_key;
40
41			/**
42			* Api access publishable key
43			*
44			* @var string
45			*/
46			public $publishable_key;
47
48			/**
49			* Should we store the users credit cards?
50			*
51			* @var bool
52			*/
53			public $saved_cards;
54
55			/**
56			* Constructor
57			*/
58			public function __construct() {
59			$this->id = 'stripe_alipay';
60			$this->method_title = __( 'Stripe Alipay', 'woocommerce-gateway-stripe' );
61			/* translators: link */
62			$this->method_description = sprintf( __( 'All other general Stripe settings can be adjusted <a href="%s">here</a>.', 'woocommerce-gateway-stripe' ), admin_url( 'admin.php?page=wc-settings&tab=checkout&section=stripe' ) );
63			$this->supports = array(
64			'products',
65			'refunds',
66			);
67
68			// Load the form fields.
69			$this->init_form_fields();
70
71			// Load the settings.
72			$this->init_settings();
73
74			$main_settings = get_option( 'woocommerce_stripe_settings' );
75			$this->title = $this->get_option( 'title' );
76			$this->description = $this->get_option( 'description' );
77			$this->enabled = $this->get_option( 'enabled' );
78			$this->testmode = ( ! empty( $main_settings['testmode'] ) && 'yes' === $main_settings['testmode'] ) ? true : false;
79			$this->saved_cards = ( ! empty( $main_settings['saved_cards'] ) && 'yes' === $main_settings['saved_cards'] ) ? true : false;
80			$this->publishable_key = ! empty( $main_settings['publishable_key'] ) ? $main_settings['publishable_key'] : '';
81			$this->secret_key = ! empty( $main_settings['secret_key'] ) ? $main_settings['secret_key'] : '';
82			$this->statement_descriptor = ! empty( $main_settings['statement_descriptor'] ) ? $main_settings['statement_descriptor'] : '';
83
84			if ( $this->testmode ) {
85			$this->publishable_key = ! empty( $main_settings['test_publishable_key'] ) ? $main_settings['test_publishable_key'] : '';
86			$this->secret_key = ! empty( $main_settings['test_secret_key'] ) ? $main_settings['test_secret_key'] : '';
87			}
88
89			add_action( 'woocommerce_update_options_payment_gateways_' . $this->id, array( $this, 'process_admin_options' ) );
90			add_action( 'admin_notices', array( $this, 'check_environment' ) );
91			add_action( 'admin_head', array( $this, 'remove_admin_notice' ) );
92			add_action( 'wp_enqueue_scripts', array( $this, 'payment_scripts' ) );
93			}
94
95			/**
96			* Checks to make sure environment is setup correctly to use this payment method.
97			*
98			* @since 4.0.0
99			* @version 4.0.0
100			*/
101			public function check_environment() {
102			if ( ! current_user_can( 'manage_woocommerce' ) ) {
103			return;
104			}
105
106			$environment_warning = $this->get_environment_warning();
107
108			if ( $environment_warning ) {
			0 ignored issues – show Bug Best Practice introduced 2018-01-07 16:56 UTC by Report Bug Copy Issue Report The expression `$environment_warning` of type `string\|false` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== false` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
109			$this->add_admin_notice( 'bad_environment', 'error', $environment_warning );
110			}
111
112			foreach ( (array) $this->notices as $notice_key => $notice ) {
113			echo "<div class='" . esc_attr( $notice['class'] ) . "'><p>";
114			echo wp_kses( $notice['message'], array( 'a' => array( 'href' => array() ) ) );
115			echo '</p></div>';
116			}
117			}
118
119			/**
120			* Checks the environment for compatibility problems. Returns a string with the first incompatibility
121			* found or false if the environment has no problems.
122			*
123			* @since 4.0.0
124			* @version 4.0.0
125			*/
126			public function get_environment_warning() {
127			if (
128			'yes' === $this->enabled && ! in_array( get_woocommerce_currency(), $this->get_supported_currency() )
129			) {
130			/* translators: supported currency list */
131			$message = sprintf( __( 'Alipay is enabled - it requires store currency to be set to %s', 'woocommerce-gateway-stripe' ), implode( ', ', $this->get_supported_currency() ) );
132
133			return $message;
134			}
135
136			return false;
137			}
138
139			/**
140			* Returns all supported currencies for this payment method.
141			*
142			* @since 4.0.0
143			* @version 4.0.0
144			* @return array
145			*/
146			public function get_supported_currency() {
147			return apply_filters( 'wc_stripe_alipay_supported_currencies', array(
148			'EUR',
149			'AUD',
150			'CAD',
151			'GBP',
152			'HKD',
153			'JPY',
154			'NZD',
155			'SGD',
156			'USD',
157			) );
158			}
159
160			/**
161			* Checks to see if all criteria is met before showing payment method.
162			*
163			* @since 4.0.0
164			* @version 4.0.0
165			* @return bool
166			*/
167			public function is_available() {
168			if ( ! in_array( get_woocommerce_currency(), $this->get_supported_currency() ) ) {
169			return false;
170			}
171
172			return parent::is_available();
173			}
174
175			/**
176			* Get_icon function.
177			*
178			* @since 1.0.0
179			* @version 4.0.0
180			* @return string
181			*/
182			public function get_icon() {
183			$icons = $this->payment_icons();
184
185			$icons_str = '';
186
187			$icons_str .= $icons['alipay'];
188
189			return apply_filters( 'woocommerce_gateway_icon', $icons_str, $this->id );
190			}
191
192			/**
193			* Payment_scripts function.
194			*
195			* @since 4.0.0
196			* @version 4.0.0
197			*/
198			public function payment_scripts() {
199			if ( ! is_cart() && ! is_checkout() && ! isset( $_GET['pay_for_order'] ) && ! is_add_payment_method_page() ) {
200			return;
201			}
202
203			wp_enqueue_style( 'stripe_paymentfonts' );
204			wp_enqueue_script( 'woocommerce_stripe' );
205			}
206
207			/**
208			* Initialize Gateway Settings Form Fields.
209			*/
210			public function init_form_fields() {
211			$this->form_fields = require( WC_STRIPE_PLUGIN_PATH . '/includes/admin/stripe-alipay-settings.php' );
212			}
213
214			/**
215			* Payment form on checkout page
216			*/
217			public function payment_fields() {
218			$user = wp_get_current_user();
			0 ignored issues – show Unused Code introduced 2017-07-27 08:27 UTC by Report Bug Copy Issue Report `$user` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
219			$total = WC()->cart->total;
220
221			// If paying from order, we need to get total from order not cart.
222			if ( isset( $_GET['pay_for_order'] ) && ! empty( $_GET['key'] ) ) {
223			$order = wc_get_order( wc_get_order_id_by_order_key( wc_clean( $_GET['key'] ) ) );
224			$total = $order->get_total();
225			}
226
227			if ( is_add_payment_method_page() ) {
228			$pay_button_text = __( 'Add Payment', 'woocommerce-gateway-stripe' );
			0 ignored issues – show Unused Code introduced 2017-07-27 08:27 UTC by Report Bug Copy Issue Report `$pay_button_text` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
229			$total = '';
230			} else {
231			$pay_button_text = '';
			0 ignored issues – show Unused Code introduced 2017-07-27 08:27 UTC by Report Bug Copy Issue Report `$pay_button_text` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
232			}
233
234			echo '<div
235			id="stripe-alipay-payment-data"
236			data-amount="' . esc_attr( WC_Stripe_Helper::get_stripe_amount( $total ) ) . '"
237			data-currency="' . esc_attr( strtolower( get_woocommerce_currency() ) ) . '">';
238
239			if ( $this->description ) {
240			echo apply_filters( 'wc_stripe_description', wpautop( wp_kses_post( $this->description ) ) );
241			}
242
243			echo '</div>';
244			}
245
246			/**
247			* Creates the source for charge.
248			*
249			* @since 4.0.0
250			* @version 4.0.0
251			* @param object $order
252			* @return mixed
253			*/
254			public function create_source( $order ) {
255			$currency = WC_Stripe_Helper::is_pre_30() ? $order->get_order_currency() : $order->get_currency();
256			$order_id = WC_Stripe_Helper::is_pre_30() ? $order->id : $order->get_id();
			0 ignored issues – show Unused Code introduced 2018-01-07 16:56 UTC by Report Bug Copy Issue Report `$order_id` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
257			$return_url = $this->get_stripe_return_url( $order );
258			$post_data = array();
259			$post_data['amount'] = WC_Stripe_Helper::get_stripe_amount( $order->get_total(), $currency );
260			$post_data['currency'] = strtolower( $currency );
261			$post_data['type'] = 'alipay';
262			$post_data['owner'] = $this->get_owner_details( $order );
263			$post_data['redirect'] = array( 'return_url' => $return_url );
264
265			if ( ! empty( $this->statement_descriptor ) ) {
266			$post_data['statement_descriptor'] = WC_Stripe_Helper::clean_statement_descriptor( $this->statement_descriptor );
			0 ignored issues – show Documentation introduced 2018-01-07 16:56 UTC by Report Bug Copy Issue Report `$this->statement_descriptor` is of type `boolean`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
267			}
268
269			WC_Stripe_Logger::log( 'Info: Begin creating Alipay source' );
270
271			return WC_Stripe_API::request( $post_data, 'sources' );
272			}
273
274			/**
275			* Process the payment
276			*
277			* @param int $order_id Reference.
278			* @param bool $retry Should we retry on fail.
279			* @param bool $force_save_source Force payment source to be saved.
			0 ignored issues – show Bug introduced 2018-01-07 16:56 UTC by Report Bug Copy Issue Report There is no parameter named `$force_save_source`. Was it maybe removed? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. Consider the following example. The parameter `$italy` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $island * @param array $italy */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was removed, but the annotation was not. Loading history...
280			*
281			* @throws Exception If payment will not be accepted.
282			*
283			* @return array\|void
284			*/
285			public function process_payment( $order_id, $retry = true, $force_save_save = false ) {
286			try {
287			$order = wc_get_order( $order_id );
288
289			// This will throw exception if not valid.
290			$this->validate_minimum_order_amount( $order );
291
292			// This comes from the create account checkbox in the checkout page.
293			$create_account = ! empty( $_POST['createaccount'] ) ? true : false;
294
295			if ( $create_account ) {
296			$new_customer_id = WC_Stripe_Helper::is_pre_30() ? $order->customer_user : $order->get_customer_id();
297			$new_stripe_customer = new WC_Stripe_Customer( $new_customer_id );
298			$new_stripe_customer->create_customer();
299			}
300
301			$response = $this->create_source( $order );
302
303			if ( ! empty( $response->error ) ) {
304			$order->add_order_note( $response->error->message );
305
306			throw new Exception( $response->error->message );
307			}
308
309			if ( WC_Stripe_Helper::is_pre_30() ) {
310			update_post_meta( $order_id, '_stripe_source_id', $response->id );
311			} else {
312			$order->update_meta_data( '_stripe_source_id', $response->id );
313			$order->save();
314			}
315
316			WC_Stripe_Logger::log( 'Info: Redirecting to Alipay...' );
317
318			return array(
319			'result' => 'success',
320			'redirect' => esc_url_raw( $response->redirect->url ),
321			);
322			} catch ( Exception $e ) {
323			wc_add_notice( $e->getMessage(), 'error' );
324			WC_Stripe_Logger::log( 'Error: ' . $e->getMessage() );
325
326			do_action( 'wc_gateway_stripe_process_payment_error', $e, $order );
327
328			$statuses = array( 'pending', 'failed' );
329
330			if ( $order->has_status( $statuses ) ) {
331			$this->send_failed_order_email( $order_id );
332			}
333
334			return array(
335			'result' => 'fail',
336			'redirect' => '',
337			);
338			}
339			}
340			}
341

woocommerce / woocommerce-gateway-stripe

Push — master ( 2515df...302e86 )

WC_Gateway_Stripe_Alipay B

Complexity

Size/Duplication

Coupling/Cohesion

Importance

11 Methods

How to fix Complexity

Complex Class

Duplication Side-by-Side

Filter issues like