CustomerQuery::get_customers() - Code Metrics - Inspection of "Try to get name from consumer bank details as fall..." - wp-pay-gateways/mollie - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — develop ( 8028c8...517b56 )

by Reüel

created 2020-03-06 22:14 UTC

CustomerQuery::get_customers() A

↳ Parent: CustomerQuery

Complexity

Conditions	2
Paths	2

Size

Total Lines	26
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	2
eloc	9
c	1
b	0
f	0
nc	2
nop	0
dl	0
loc	26
rs	9.9666

<?php
/**
 * Mollie customer query.
 *
 * @author    Pronamic <[email protected]>
 * @copyright 2005-2020 Pronamic
 * @license   GPL-3.0-or-later
 * @package   Pronamic\WordPress\Pay
 */

namespace Pronamic\WordPress\Pay\Gateways\Mollie;

/**
 * Title: Mollie customer query
 * Description:
 * Copyright: 2005-2020 Pronamic
 * Company: Pronamic
 *
 * @author  Remco Tolsma
 * @version 3.0.0
 * @since   3.0.0
 */
class CustomerQuery {
	/**
	 * Construct customer query.
	 *
	 * @param array $args Query arguments.
	 */
	public function __construct( $args = array() ) {
		$this->args = \wp_parse_args(

			$args,
			array(
				'user_id'         => null,
				'organization_id' => null,
			)
		);
	}

	/**
	 * Get customers.
	 *
	 * @return array
	 */
	public function get_customers() {
		global $wpdb;

		$where = '1 = 1';

		if ( array_key_exists( 'user_id', $this->args ) ) {
			$where .= $wpdb->prepare( ' AND mollie_customer_user.user_id = %d', $this->args['user_id'] );
		}

		$query = "
			SELECT
				mollie_customer.mollie_id,
				mollie_customer.test_mode,
				mollie_customer.name,
				mollie_customer.email
			FROM
				$wpdb->pronamic_pay_mollie_customer_users AS mollie_customer_user
					INNER JOIN
				$wpdb->pronamic_pay_mollie_customers AS mollie_customer
						ON mollie_customer_user.customer_id = mollie_customer.id
			WHERE
				 $where
			;
		";

		return $wpdb->get_results( $query );
	}
}


1			<?php
2			/**
3			* Mollie customer query.
4			*
5			* @author Pronamic <[email protected]>
6			* @copyright 2005-2020 Pronamic
7			* @license GPL-3.0-or-later
8			* @package Pronamic\WordPress\Pay
9			*/
10
11			namespace Pronamic\WordPress\Pay\Gateways\Mollie;
12
13			/**
14			* Title: Mollie customer query
15			* Description:
16			* Copyright: 2005-2020 Pronamic
17			* Company: Pronamic
18			*
19			* @author Remco Tolsma
20			* @version 3.0.0
21			* @since 3.0.0
22			*/
23			class CustomerQuery {
24			/**
25			* Construct customer query.
26			*
27			* @param array $args Query arguments.
28			*/
29			public function __construct( $args = array() ) {
30			$this->args = \wp_parse_args(
			0 ignored issues – show Bug Best Practice introduced 2020-02-12 14:45 UTC by Report Bug Copy Issue Report The property `args` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history...
31			$args,
32			array(
33			'user_id' => null,
34			'organization_id' => null,
35			)
36			);
37			}
38
39			/**
40			* Get customers.
41			*
42			* @return array
43			*/
44			public function get_customers() {
45			global $wpdb;
46
47			$where = '1 = 1';
48
49			if ( array_key_exists( 'user_id', $this->args ) ) {
50			$where .= $wpdb->prepare( ' AND mollie_customer_user.user_id = %d', $this->args['user_id'] );
51			}
52
53			$query = "
54			SELECT
55			mollie_customer.mollie_id,
56			mollie_customer.test_mode,
57			mollie_customer.name,
58			mollie_customer.email
59			FROM
60			$wpdb->pronamic_pay_mollie_customer_users AS mollie_customer_user
61			INNER JOIN
62			$wpdb->pronamic_pay_mollie_customers AS mollie_customer
63			ON mollie_customer_user.customer_id = mollie_customer.id
64			WHERE
65			$where
66			;
67			";
68
69			return $wpdb->get_results( $query );
70			}
71			}
72

wp-pay-gateways / mollie

Push — develop ( 8028c8...517b56 )

CustomerQuery::get_customers() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like