Pronamic\WordPress\Pay\Forms\FormProcessor::init()

Complexity

Conditions	7
Paths	7

Size

Total Lines	85
Code Lines	42

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
	public function init() {
		global $pronamic_pay_errors;

		$pronamic_pay_errors = array();

		// Nonce.
		if ( ! filter_has_var( INPUT_POST, 'pronamic_pay_nonce' ) ) {
			return;
		}

		$nonce = filter_input( INPUT_POST, 'pronamic_pay_nonce', FILTER_SANITIZE_STRING );

		if ( ! wp_verify_nonce( $nonce, 'pronamic_pay' ) ) {
			return;
		}

		// Validate.
		$valid = $this->validate();

		if ( ! $valid ) {
			return;
		}

		// Gateway.
		$id = filter_input( INPUT_POST, 'pronamic_pay_form_id', FILTER_VALIDATE_INT );

		$config_id = get_post_meta( $id, '_pronamic_payment_form_config_id', true );

		$gateway = Plugin::get_gateway( $config_id );

		if ( ! $gateway ) {
			return;
		}

		// Data.
		$data = new PaymentFormData();

		$payment = Plugin::start( $config_id, $gateway, $data );

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

		$error = $gateway->get_error();

		if ( $error instanceof WP_Error ) {

$error is always a sub-type of WP_Error.

			Plugin::render_errors( $error );

$error of type WP_Error is incompatible with the type array expected by parameter $errors of Pronamic\WordPress\Pay\Plugin::render_errors().

			exit;

Using exit here is not recommended.

		}

		// @see https://github.com/WordImpress/Give/blob/1.1/includes/payments/functions.php#L172-L178.
		// @see https://github.com/woothemes/woocommerce/blob/2.4.3/includes/wc-user-functions.php#L36-L118.
		$first_name = filter_input( INPUT_POST, 'pronamic_pay_first_name', FILTER_SANITIZE_STRING );
		$last_name  = filter_input( INPUT_POST, 'pronamic_pay_last_name', FILTER_SANITIZE_STRING );
		$email      = filter_input( INPUT_POST, 'pronamic_pay_email', FILTER_VALIDATE_EMAIL );

		$user = get_user_by( 'email', $email );

		if ( ! $user ) {
			// Make a random string for password.
			$password = wp_generate_password( 10 );

			// Make a user with the username as the email.
			$user_id = wp_insert_user(
				array(
					'user_login' => $email,
					'user_pass'  => $password,
					'user_email' => $email,
					'role'       => 'payer',
					'first_name' => $first_name,
					'last_name'  => $last_name,
				)
			);

			// User.
			$user = new WP_User( $user_id );
		}

		wp_update_post(
			array(
				'ID'          => $payment->post->ID,
				'post_author' => $user->ID,
			)
		);

		$gateway->redirect( $payment );

		exit;

Using exit here is not recommended.

	}