includes/admin/donors/donor-actions.php ➔ give_edit_donor()

Complexity

Conditions	16
Paths	186

Size

Total Lines

137

Duplication

Lines	10
Ratio	7.3 %

Importance

Changes

0

<?php
function give_edit_donor( $args ) {

	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );

	if ( ! is_admin() || ! current_user_can( $donor_edit_role ) ) {
		wp_die( esc_html__( 'You do not have permission to edit this donor.', 'give' ), esc_html__( 'Error', 'give' ), array(
			'response' => 403,
		) );
	}

	if ( empty( $args ) ) {
		return false;
	}

	// Sanitize Data.
	$args = give_clean( $args );

	// Verify Nonce.
	if ( ! wp_verify_nonce( $args['_wpnonce'], 'edit-donor' ) ) {

This code seems to be duplicated across your project.

		wp_die( esc_html__( 'Cheatin’ uh?', 'give' ), esc_html__( 'Error', 'give' ), array(
			'response' => 400,
		) );
	}

	$donor_info = $args['donor_info'];
	$donor_id   = intval( $donor_info['id'] );

	$donor = new Give_Donor( $donor_id );

	// Bailout, if donor id doesn't exists.
	if ( empty( $donor->id ) ) {
		return false;
	}

	$defaults = array(
		'title'   => '',
		'name'    => '',
		'user_id' => 0,
		'line1'   => '',
		'line2'   => '',
		'city'    => '',
		'zip'     => '',
		'state'   => '',
		'country' => '',
	);

	$donor_info = wp_parse_args( $donor_info, $defaults );

	if ( (int) $donor_info['user_id'] !== (int) $donor->user_id ) {

		// Make sure we don't already have this user attached to a donor.
		if ( ! empty( $donor_info['user_id'] ) && false !== Give()->donors->get_donor_by( 'user_id', $donor_info['user_id'] ) ) {
			give_set_error(
				'give-invalid-donor-user_id',
				sprintf(
					/* translators: %d User ID */
					__( 'The User ID #%d is already associated with a different donor.', 'give' ),
					$donor_info['user_id']
				)
			);
		}

		// Make sure it's actually a user.
		$user = get_user_by( 'id', $donor_info['user_id'] );
		if ( ! empty( $donor_info['user_id'] ) && false === $user ) {
			give_set_error(
				'give-invalid-user_id',
				sprintf(
					/* translators: %d User ID */
					__( 'The User ID #%d does not exist. Please assign an existing user.', 'give' ),
					$donor_info['user_id']
				)
			);
		}
	}

	// Bailout, if errors are present.
	if ( give_get_errors() ) {
		return false;
	}

	$donor->update_meta( '_give_anonymous_donor', absint( $args['give_anonymous_donor'] ) );

	// Save company name in when admin update donor company name from dashboard.
	$donor->update_meta( '_give_donor_company', sanitize_text_field( $args['give_donor_company'] ) );

	// If First name of donor is empty, then fetch the current first name of donor.
	if ( empty( $donor_info['first_name'] ) ) {
		$donor_info['first_name'] = $donor->get_first_name();
	}

	// Sanitize the inputs.
	$donor_data               = array();
	$donor_data['name']       = trim( "{$donor_info['first_name']} {$donor_info['last_name']}" );
	$donor_data['first_name'] = $donor_info['first_name'];
	$donor_data['last_name']  = $donor_info['last_name'];
	$donor_data['title']      = $donor_info['title'];
	$donor_data['user_id']    = $donor_info['user_id'];

	$donor_data = apply_filters( 'give_edit_donor_info', $donor_data, $donor_id );

	/**
	 * Filter the address
	 *
	 * @todo unnecessary filter because we are not storing donor address to user.
	 *
	 * @since 1.0
	 */
	$address = apply_filters( 'give_edit_donor_address', array(), $donor_id );

	$donor_data = give_clean( $donor_data );
	$address    = give_clean( $address );

	$output = give_connect_user_donor_profile( $donor, $donor_data, $address );

It seems like $donor_data defined by give_clean($donor_data) on line 136 can also be of type string; however, give_connect_user_donor_profile() does only seem to accept array, maybe add an additional type check?

It seems like $address defined by give_clean($address) on line 137 can also be of type string; however, give_connect_user_donor_profile() does only seem to accept array, maybe add an additional type check?

	if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {

This code seems to be duplicated across your project.

		header( 'Content-Type: application/json' );
		echo wp_json_encode( $output );
		wp_die();
	}

	if ( $output['success'] ) {
		wp_safe_redirect( add_query_arg(
			array(
				'post_type'       => 'give_forms',
				'page'            => 'give-donors',
				'view'            => 'overview',
				'id'              => $donor_id,
				'give-messages[]' => 'profile-updated'
			),
			esc_url( admin_url( 'edit.php' ) )
		) );
	}

	exit;

}