Pronamic\WordPress\Pay\Gateways\Mollie\CLI::wp_cli_customers_synchronize()

Complexity

Conditions	11
Paths	2

Size

Total Lines	137
Code Lines	84

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

<?php
	public function wp_cli_customers_synchronize( $args, $assoc_args ) {

The parameter $args is not used and could be removed.

The parameter $assoc_args is not used and could be removed.

		global $post;
		global $wpdb;

		$query = new \WP_Query(
			array(
				'post_type'   => 'pronamic_gateway',
				'post_status' => 'publish',
				'nopaging'    => true,
				'meta_query'  => array(
					array(
						'key'   => '_pronamic_gateway_id',
						'value' => 'mollie',
					),
					array(
						'key'     => '_pronamic_gateway_mollie_api_key',
						'compare' => 'EXISTS',
					),
				),
			)
		);

		if ( $query->have_posts() ) {
			while ( $query->have_posts() ) {
				$query->the_post();

				$api_key = get_post_meta( $post->ID, '_pronamic_gateway_mollie_api_key', true );

				\WP_CLI::log( $post->post_title );
				\WP_CLI::log( $api_key );
				\WP_CLI::log( '' );

				$client = new Client( $api_key );

It seems like $api_key can also be of type false; however, parameter $api_key of Pronamic\WordPress\Pay\Gateways\Mollie\Client::__construct() does only seem to accept string, maybe add an additional type check?

				$urls = array(
					'https://api.mollie.com/v2/customers?limit=250',
				);

				while ( ! empty( $urls ) ) {
					$url = array_shift( $urls );

					\WP_CLI::log( $url );

					$response = $client->send_request( $url );

					if ( isset( $response->count ) ) {
						\WP_CLI::log(
							\sprintf(
								'Found %d customer(s).',
								$response->count
							)
						);
					}

					if ( isset( $response->_embedded->customers ) ) {
						\WP_CLI\Utils\format_items(

The function format_items was not found. Maybe you did not declare it correctly or list all dependencies?

							'table',
							$response->_embedded->customers,
							array(
								'id',
								'mode',
								'name',
								'email',
								'locale',
							)
						);

						foreach ( $response->_embedded->customers as $object ) {
							$id = $wpdb->get_var( $wpdb->prepare( "SELECT id FROM $wpdb->pronamic_pay_mollie_customers WHERE mollie_id = %s", $object->id ) );

							$data = array(
								'test_mode' => ( 'test' === $object->mode ),
								'email'     => $object->email,
								'name'      => $object->name,
							);

							$format = array(
								'test_mode' => '%d',
								'email'     => '%s',
								'name'      => '%s',
							);

							if ( null === $id ) {
								$data['mollie_id']   = $object->id;
								$foramt['mollie_id'] = '%s';

								$result = $wpdb->insert(
									$wpdb->pronamic_pay_mollie_customers,
									$data,
									$format
								);

								if ( false === $result ) {
									\WP_CLI::error(
										sprintf(
											'Database error: %s.',
											$wpdb->last_error
										)
									);
								}
							} else {
								$result = $wpdb->update(
									$wpdb->pronamic_pay_mollie_customers,
									$data,
									array(
										'id' => $id,
									),
									$format,
									array(
										'id' => '%d'
									)
								);

								if ( false === $result ) {
									\WP_CLI::error(
										sprintf(
											'Database error: %s.',
											$wpdb->last_error
										)
									);
								}

								$id = $wpdb->insert_id;

The assignment to $id is dead and can be removed.

							}
						}
					}

					if ( isset( $response->_links->next->href ) ) {
						$urls[] = $response->_links->next->href;
					}
				}
			}

			\wp_reset_postdata();
		}

		\WP_CLI::error( 'Command not fully implemented yet.' );
	}