WordImpress / Give

includes/admin/emails/class-email-notification-util.php

Spacing +41 added lines, -41 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if access directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -46,7 +46,7 @@ 
 	 * @return static
 	 */
 	static function get_instance() {
-		if ( null === static::$instance ) {
+		if (null === static::$instance) {
 			self::$instance = new static();
 		}
 
@@ -64,7 +64,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function has_preview( Give_Email_Notification $email ) {
+	public static function has_preview(Give_Email_Notification $email) {
 		return $email->config['has_preview'];
 	}
 
@@ -78,7 +78,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function has_recipient_field( Give_Email_Notification $email ) {
+	public static function has_recipient_field(Give_Email_Notification $email) {
 		return $email->config['has_recipient_field'];
 	}
 
@@ -92,7 +92,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function is_notification_status_editable( Give_Email_Notification $email ) {
+	public static function is_notification_status_editable(Give_Email_Notification $email) {
 		$user_can_edit = $email->config['notification_status_editable'];
 
 		return (bool) $user_can_edit;
@@ -108,7 +108,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function is_content_type_editable( Give_Email_Notification $email ) {
+	public static function is_content_type_editable(Give_Email_Notification $email) {
 		return $email->config['content_type_editable'];
 	}
 
@@ -122,7 +122,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function is_email_preview_has_header( Give_Email_Notification $email ) {
+	public static function is_email_preview_has_header(Give_Email_Notification $email) {
 		return $email->config['has_preview_header'];
 	}
 
@@ -136,7 +136,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function is_email_preview( Give_Email_Notification $email ) {
+	public static function is_email_preview(Give_Email_Notification $email) {
 		return $email->config['has_preview'];
 	}
 
@@ -150,7 +150,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function is_show_on_emails_setting_page( Give_Email_Notification $email ){
+	public static function is_show_on_emails_setting_page(Give_Email_Notification $email) {
 		return $email->config['show_on_emails_setting_page'];
 	}
 
@@ -165,8 +165,8 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function can_use_form_email_options( Give_Email_Notification $email, $form_id = null ){
-		return give_is_setting_enabled( give_get_meta( $form_id, '_give_email_options', true ) );
+	public static function can_use_form_email_options(Give_Email_Notification $email, $form_id = null) {
+		return give_is_setting_enabled(give_get_meta($form_id, '_give_email_options', true));
 	}
 
 	/**
@@ -180,13 +180,13 @@ 
 	 *
 	 * @return string
 	 */
-	public static function is_email_notification_active( Give_Email_Notification $email, $form_id = null ) {
-		$notification_status = $email->get_notification_status( $form_id );
+	public static function is_email_notification_active(Give_Email_Notification $email, $form_id = null) {
+		$notification_status = $email->get_notification_status($form_id);
 
 
-		$notification_status = empty( $form_id )
-			? give_is_setting_enabled( $notification_status )
-			: give_is_setting_enabled( give_get_option( "{$email->config['id']}_notification", $email->config['notification_status'] ) ) && give_is_setting_enabled( $notification_status, array( 'enabled', 'global' ) );
+		$notification_status = empty($form_id)
+			? give_is_setting_enabled($notification_status)
+			: give_is_setting_enabled(give_get_option("{$email->config['id']}_notification", $email->config['notification_status'])) && give_is_setting_enabled($notification_status, array('enabled', 'global'));
 			// To check if email notification is active or not on a per-form basis, email notification must be globally active—otherwise it will be considered disabled.
 
 		/**
@@ -200,7 +200,7 @@ 
 		 *
 		 * @param bool $notification_status True if notification is enable and false when disable
 		 */
-		return apply_filters( "give_{$email->config['id']}_is_email_notification_active", $notification_status, $email, $form_id );
+		return apply_filters("give_{$email->config['id']}_is_email_notification_active", $notification_status, $email, $form_id);
 	}
 
 	/**
@@ -214,8 +214,8 @@ 
 		$is_preview = false;
 
 		if (
-			current_user_can( 'manage_give_settings' )
-			&& ! empty( $_GET['give_action'] )
+			current_user_can('manage_give_settings')
+			&& ! empty($_GET['give_action'])
 			&& 'preview_email' === $_GET['give_action']
 		) {
 			$is_preview = true;
@@ -235,8 +235,8 @@ 
 		$is_preview = false;
 
 		if (
-			current_user_can( 'manage_give_settings' )
-			&& ! empty( $_GET['give_action'] )
+			current_user_can('manage_give_settings')
+			&& ! empty($_GET['give_action'])
 			&& 'send_preview_email' === $_GET['give_action']
 		) {
 			$is_preview = true;
@@ -256,13 +256,13 @@ 
 	 *
 	 * @return string
 	 */
-	public static function get_formatted_email_type( $content_type ) {
+	public static function get_formatted_email_type($content_type) {
 		$email_contents = array(
-			'text/html'  => __( 'HTML', 'give' ),
-			'text/plain' => __( 'Plain', 'give' ),
+			'text/html'  => __('HTML', 'give'),
+			'text/plain' => __('Plain', 'give'),
 		);
 
-		return $email_contents[ $content_type ];
+		return $email_contents[$content_type];
 	}
 
 
@@ -279,41 +279,41 @@ 
 	 *
 	 * @return mixed
 	 */
-	public static function get_value( Give_Email_Notification $email, $option_name, $form_id = null, $default = false ) {
+	public static function get_value(Give_Email_Notification $email, $option_name, $form_id = null, $default = false) {
 		// If form id set then option name can be contain _give_ prefix which is only used for meta key,
 		// So make sure you are using correct option name.
-		$global_option_name = ( 0 === strpos( $option_name, '_give_' )
-			? str_replace( '_give_', '', $option_name )
-			: $option_name );
-		$option_value = give_get_option( $global_option_name, $default );
+		$global_option_name = (0 === strpos($option_name, '_give_')
+			? str_replace('_give_', '', $option_name)
+			: $option_name);
+		$option_value = give_get_option($global_option_name, $default);
 
 		if (
-			! empty( $form_id )
+			! empty($form_id)
 			&& give_is_setting_enabled(
 				give_get_meta(
 					$form_id,
-					Give_Email_Setting_Field::get_prefix( $email, $form_id ) . 'notification',
+					Give_Email_Setting_Field::get_prefix($email, $form_id).'notification',
 					true,
 					'global'
 				)
 			)
 		) {
-			$option_value = get_post_meta( $form_id, $option_name, true );
+			$option_value = get_post_meta($form_id, $option_name, true);
 
 			// Get only email field value from recipients setting.
-			if( Give_Email_Setting_Field::get_prefix( $email, $form_id ) . 'recipient' === $option_name ) {
-				$option_value = wp_list_pluck( $option_value, 'email' );
+			if (Give_Email_Setting_Field::get_prefix($email, $form_id).'recipient' === $option_name) {
+				$option_value = wp_list_pluck($option_value, 'email');
 			}
 		}
 
-		$option_value = empty( $option_value ) ? $default : $option_value;
+		$option_value = empty($option_value) ? $default : $option_value;
 
 		/**
 		 * Filter the setting value
 		 *
 		 * @since 2.0
 		 */
-		return apply_filters( 'give_email_setting_value', $option_value, $option_name, $email, $form_id, $default );
+		return apply_filters('give_email_setting_value', $option_value, $option_name, $email, $form_id, $default);
 	}
 
 
@@ -328,12 +328,12 @@ 
 	 *
 	 * @return string
 	 */
-	public static function get_email_logo( $form_id ) {
+	public static function get_email_logo($form_id) {
 
 		// Email logo tag.
-		$header_img = $form_id && give_is_setting_enabled( give_get_meta( $form_id, '_give_email_options', true ) )
-			? give_get_meta( $form_id, '_give_email_logo', true )
-			: give_get_option( 'email_logo', '' );
+		$header_img = $form_id && give_is_setting_enabled(give_get_meta($form_id, '_give_email_options', true))
+			? give_get_meta($form_id, '_give_email_logo', true)
+			: give_get_option('email_logo', '');
 
 		return $header_img;
 	}

includes/admin/settings/class-settings-license.php

Spacing +15 added lines, -15 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly.
 }
 
-if ( ! class_exists( 'Give_Settings_License' ) ) :
+if ( ! class_exists('Give_Settings_License')) :
 
 	/**
 	 * Give_Settings_License.
@@ -27,12 +27,12 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'licenses';
-			$this->label = esc_html__( 'Licenses', 'give' );
+			$this->label = esc_html__('Licenses', 'give');
 
 			parent::__construct();
 
 			// Filter to remove the license tab.
-			add_filter( 'give-settings_tabs_array', array( $this, 'remove_license_tab' ), 9999999, 1 );
+			add_filter('give-settings_tabs_array', array($this, 'remove_license_tab'), 9999999, 1);
 
 		}
 
@@ -49,7 +49,7 @@ 
 			 * Filter the licenses settings.
 			 * Backward compatibility: Please do not use this filter. This filter is deprecated in 1.8
 			 */
-			$settings = apply_filters( 'give_settings_licenses', $settings );
+			$settings = apply_filters('give_settings_licenses', $settings);
 
 			/**
 			 * Filter the settings.
@@ -58,7 +58,7 @@ 
 			 *
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, $settings );
+			$settings = apply_filters('give_get_settings_'.$this->id, $settings);
 
 			// Output.
 			return $settings;
@@ -74,13 +74,13 @@ 
 		 *
 		 * @return array
 		 */
-		public function remove_license_tab( $tabs ) {
+		public function remove_license_tab($tabs) {
 			/**
 			 * Remove the license tab if no Give licensed addon
 			 * is activated.
 			 */
-			if ( ! $this->is_show_setting_page() ) {
-				unset( $tabs['licenses'] );
+			if ( ! $this->is_show_setting_page()) {
+				unset($tabs['licenses']);
 			}
 
 			return $tabs;
@@ -97,17 +97,17 @@ 
 		 */
 		private function is_show_setting_page() {
 			$licensed_addons   = Give_License::get_licensed_addons();
-			$activated_plugins = get_option( 'active_plugins', array() );
+			$activated_plugins = get_option('active_plugins', array());
 
 			// Get list of network enabled plugin.
-			if ( is_multisite() ) {
-				$sitewide_activated_plugins = array_keys( get_site_option( 'active_sitewide_plugins', array() ) );
-				$activated_plugins = ! empty( $activated_plugins )
-					? array_merge( $sitewide_activated_plugins, $activated_plugins )
+			if (is_multisite()) {
+				$sitewide_activated_plugins = array_keys(get_site_option('active_sitewide_plugins', array()));
+				$activated_plugins = ! empty($activated_plugins)
+					? array_merge($sitewide_activated_plugins, $activated_plugins)
 					: $sitewide_activated_plugins;
 			}
 
-			return (bool) count( array_intersect( $activated_plugins, $licensed_addons ) );
+			return (bool) count(array_intersect($activated_plugins, $licensed_addons));
 		}
 	}
 

includes/admin/donors/donor-actions.php

Doc Comments +3 added lines, -3 removed lines

@@ -21,7 +21,7 @@ 
  *
  * @since 1.0
  *
- * @return array|bool $output Response messages
+ * @return false|null $output Response messages
  */
 function give_edit_donor( $args ) {
 
@@ -434,7 +434,7 @@ 
  *
  * @since  1.7
  *
- * @return bool|null
+ * @return false|null
  */
 function give_remove_donor_email() {
 	if ( empty( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
@@ -478,7 +478,7 @@ 
  *
  * @since  1.7
  *
- * @return bool|null
+ * @return false|null
  */
 function give_set_donor_primary_email() {
 	if ( empty( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {

Spacing +174 added lines, -174 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,37 +23,37 @@ 
  *
  * @return array|bool $output Response messages
  */
-function give_edit_donor( $args ) {
+function give_edit_donor($args) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$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(
+	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 ) ) {
+	if (empty($args)) {
 		return false;
 	}
 
 	// Sanitize Data.
-	$args = give_clean( $args );
+	$args = give_clean($args);
 
 	// Verify Nonce.
-	if ( ! wp_verify_nonce( $args['_wpnonce'], 'edit-donor' ) ) {
-		wp_die( esc_html__( 'Cheatin’ uh?', 'give' ), esc_html__( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($args['_wpnonce'], 'edit-donor')) {
+		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_id   = intval($donor_info['id']);
 
-	$donor = new Give_Donor( $donor_id );
+	$donor = new Give_Donor($donor_id);
 
 	// Bailout, if donor id doesn't exists.
-	if ( empty( $donor->id ) ) {
+	if (empty($donor->id)) {
 		return false;
 	}
 
@@ -69,30 +69,30 @@ 
 		'country' => '',
 	);
 
-	$donor_info = wp_parse_args( $donor_info, $defaults );
+	$donor_info = wp_parse_args($donor_info, $defaults);
 
-	if ( (int) $donor_info['user_id'] !== (int) $donor->user_id ) {
+	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'] ) ) {
+		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' ),
+					__('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 ) {
+		$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' ),
+					__('The User ID #%d does not exist. Please assign an existing user.', 'give'),
 					$donor_info['user_id']
 				)
 			);
@@ -100,29 +100,29 @@ 
 	}
 
 	// Bailout, if errors are present.
-	if ( give_get_errors() ) {
+	if (give_get_errors()) {
 		return false;
 	}
 
-	$donor->update_meta( '_give_anonymous_donor', absint( $args['give_anonymous_donor'] ) );
+	$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'] ) );
+	$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'] ) ) {
+	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['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 );
+	$donor_data = apply_filters('give_edit_donor_info', $donor_data, $donor_id);
 
 	/**
 	 * Filter the address
@@ -131,21 +131,21 @@ 
 	 *
 	 * @since 1.0
 	 */
-	$address = apply_filters( 'give_edit_donor_address', array(), $donor_id );
+	$address = apply_filters('give_edit_donor_address', array(), $donor_id);
 
-	$donor_data = give_clean( $donor_data );
-	$address    = give_clean( $address );
+	$donor_data = give_clean($donor_data);
+	$address    = give_clean($address);
 
-	$output = give_connect_user_donor_profile( $donor, $donor_data, $address );
+	$output = give_connect_user_donor_profile($donor, $donor_data, $address);
 
-	if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
-		header( 'Content-Type: application/json' );
-		echo wp_json_encode( $output );
+	if (defined('DOING_AJAX') && DOING_AJAX) {
+		header('Content-Type: application/json');
+		echo wp_json_encode($output);
 		wp_die();
 	}
 
-	if ( $output['success'] ) {
-		wp_safe_redirect( add_query_arg(
+	if ($output['success']) {
+		wp_safe_redirect(add_query_arg(
 			array(
 				'post_type'       => 'give_forms',
 				'page'            => 'give-donors',
@@ -153,15 +153,15 @@ 
 				'id'              => $donor_id,
 				'give-messages[]' => 'profile-updated'
 			),
-			esc_url( admin_url( 'edit.php' ) )
-		) );
+			esc_url(admin_url('edit.php'))
+		));
 	}
 
 	exit;
 
 }
 
-add_action( 'give_edit-donor', 'give_edit_donor', 10, 1 );
+add_action('give_edit-donor', 'give_edit_donor', 10, 1);
 
 /**
  * Save a donor note.
@@ -172,40 +172,40 @@ 
  *
  * @return int The Note ID that was saved, or 0 if nothing was saved.
  */
-function give_donor_save_note( $args ) {
+function give_donor_save_note($args) {
 
-	$donor_view_role = apply_filters( 'give_view_donors_role', 'view_give_reports' );
+	$donor_view_role = apply_filters('give_view_donors_role', 'view_give_reports');
 
-	if ( ! is_admin() || ! current_user_can( $donor_view_role ) ) {
-		wp_die( __( 'You do not have permission to edit this donor.', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! is_admin() || ! current_user_can($donor_view_role)) {
+		wp_die(__('You do not have permission to edit this donor.', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	if ( empty( $args ) ) {
+	if (empty($args)) {
 		return false;
 	}
 
-	$donor_note = trim( give_clean( $args['donor_note'] ) );
+	$donor_note = trim(give_clean($args['donor_note']));
 	$donor_id   = (int) $args['customer_id'];
 	$nonce      = $args['add_donor_note_nonce'];
 
-	if ( ! wp_verify_nonce( $nonce, 'add-donor-note' ) ) {
-		wp_die( __( 'Cheatin’ uh?', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'add-donor-note')) {
+		wp_die(__('Cheatin’ uh?', 'give'), __('Error', 'give'), array(
 			'response' => 400,
-		) );
+		));
 	}
 
-	if ( empty( $donor_note ) ) {
-		give_set_error( 'empty-donor-note', __( 'A note is required.', 'give' ) );
+	if (empty($donor_note)) {
+		give_set_error('empty-donor-note', __('A note is required.', 'give'));
 	}
 
-	if ( give_get_errors() ) {
+	if (give_get_errors()) {
 		return false;
 	}
 
-	$donor    = new Give_Donor( $donor_id );
-	$new_note = $donor->add_note( $donor_note );
+	$donor    = new Give_Donor($donor_id);
+	$new_note = $donor->add_note($donor_note);
 
 	/**
 	 * Fires before inserting donor note.
@@ -215,22 +215,22 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_pre_insert_donor_note', $donor_id, $new_note );
+	do_action('give_pre_insert_donor_note', $donor_id, $new_note);
 
-	if ( ! empty( $new_note ) && ! empty( $donor->id ) ) {
+	if ( ! empty($new_note) && ! empty($donor->id)) {
 
 		ob_start();
 		?>
 		<div class="donor-note-wrapper dashboard-comment-wrap comment-item">
 			<span class="note-content-wrap">
-				<?php echo stripslashes( $new_note ); ?>
+				<?php echo stripslashes($new_note); ?>
 			</span>
 		</div>
 		<?php
 		$output = ob_get_contents();
 		ob_end_clean();
 
-		if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
+		if (defined('DOING_AJAX') && DOING_AJAX) {
 			echo $output;
 			exit;
 		}
@@ -243,7 +243,7 @@ 
 
 }
 
-add_action( 'give_add-donor-note', 'give_donor_save_note', 10, 1 );
+add_action('give_add-donor-note', 'give_donor_save_note', 10, 1);
 
 
 /**
@@ -255,17 +255,17 @@ 
  *
  * @return bool|array If the disconnect was successful.
  */
-function give_disconnect_donor_user_id( $args ) {
+function give_disconnect_donor_user_id($args) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
-	if ( ! is_admin() || ! current_user_can( $donor_edit_role ) ) {
-		wp_die( __( 'You do not have permission to edit this donor.', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! is_admin() || ! current_user_can($donor_edit_role)) {
+		wp_die(__('You do not have permission to edit this donor.', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	if ( empty( $args ) ) {
+	if (empty($args)) {
 		return false;
 	}
 
@@ -273,14 +273,14 @@ 
 
 	$nonce = $args['_wpnonce'];
 
-	if ( ! wp_verify_nonce( $nonce, 'edit-donor' ) ) {
-		wp_die( __( 'Cheatin&#8217; uh?', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'edit-donor')) {
+		wp_die(__('Cheatin&#8217; uh?', 'give'), __('Error', 'give'), array(
 			'response' => 400,
-		) );
+		));
 	}
 
-	$donor = new Give_Donor( $donor_id );
-	if ( empty( $donor->id ) ) {
+	$donor = new Give_Donor($donor_id);
+	if (empty($donor->id)) {
 		return false;
 	}
 
@@ -294,7 +294,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_pre_donor_disconnect_user_id', $donor_id, $user_id );
+	do_action('give_pre_donor_disconnect_user_id', $donor_id, $user_id);
 
 	$output     = array();
 	$donor_args = array(
@@ -303,19 +303,19 @@ 
 
 
 	$output['success'] = true;
-	if ( ! $donor->update( $donor_args ) ) {
-		update_user_meta( $user_id, '_give_is_donor_disconnected', true );
-		update_user_meta( $user_id, '_give_disconnected_donor_id', $donor->id );
-		$donor->update_meta( '_give_disconnected_user_id', $user_id );
+	if ( ! $donor->update($donor_args)) {
+		update_user_meta($user_id, '_give_is_donor_disconnected', true);
+		update_user_meta($user_id, '_give_disconnected_donor_id', $donor->id);
+		$donor->update_meta('_give_disconnected_user_id', $user_id);
 
 		$output['success'] = true;
 
 	} else {
 		$output['success'] = false;
-		give_set_error( 'give-disconnect-user-fail', __( 'Failed to disconnect user from donor.', 'give' ) );
+		give_set_error('give-disconnect-user-fail', __('Failed to disconnect user from donor.', 'give'));
 	}
 
-	$output['redirect'] = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' ) . $donor_id;
+	$output['redirect'] = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id=').$donor_id;
 
 	/**
 	 * Fires after disconnecting user ID from a donor.
@@ -324,11 +324,11 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_post_donor_disconnect_user_id', $donor_id );
+	do_action('give_post_donor_disconnect_user_id', $donor_id);
 
-	if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
-		header( 'Content-Type: application/json' );
-		echo json_encode( $output );
+	if (defined('DOING_AJAX') && DOING_AJAX) {
+		header('Content-Type: application/json');
+		echo json_encode($output);
 		wp_die();
 	}
 
@@ -336,7 +336,7 @@ 
 
 }
 
-add_action( 'give_disconnect-userid', 'give_disconnect_donor_user_id', 10, 1 );
+add_action('give_disconnect-userid', 'give_disconnect_donor_user_id', 10, 1);
 
 /**
  * Add an email address to the donor from within the admin and log a donor note.
@@ -347,86 +347,86 @@ 
  *
  * @return mixed If DOING_AJAX echos out JSON, otherwise returns array of success (bool) and message (string).
  */
-function give_add_donor_email( $args ) {
+function give_add_donor_email($args) {
 
 	$donor_id = '';
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
-	if ( ! is_admin() || ! current_user_can( $donor_edit_role ) ) {
-		wp_die( __( 'You do not have permission to edit this donor.', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! is_admin() || ! current_user_can($donor_edit_role)) {
+		wp_die(__('You do not have permission to edit this donor.', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
 	$output = array();
-	if ( empty( $args ) || empty( $args['email'] ) || empty( $args['customer_id'] ) ) {
+	if (empty($args) || empty($args['email']) || empty($args['customer_id'])) {
 		$output['success'] = false;
-		if ( empty( $args['email'] ) ) {
-			$output['message'] = __( 'Email address is required.', 'give' );
-		} elseif ( empty( $args['customer_id'] ) ) {
-			$output['message'] = __( 'Donor ID is required.', 'give' );
+		if (empty($args['email'])) {
+			$output['message'] = __('Email address is required.', 'give');
+		} elseif (empty($args['customer_id'])) {
+			$output['message'] = __('Donor ID is required.', 'give');
 		} else {
-			$output['message'] = __( 'An error has occurred. Please try again.', 'give' );
+			$output['message'] = __('An error has occurred. Please try again.', 'give');
 		}
-	} elseif ( ! wp_verify_nonce( $args['_wpnonce'], 'give_add_donor_email' ) ) {
+	} elseif ( ! wp_verify_nonce($args['_wpnonce'], 'give_add_donor_email')) {
 		$output = array(
 			'success' => false,
-			'message' => __( 'Nonce verification failed.', 'give' ),
+			'message' => __('Nonce verification failed.', 'give'),
 		);
-	} elseif ( ! is_email( $args['email'] ) ) {
+	} elseif ( ! is_email($args['email'])) {
 		$output = array(
 			'success' => false,
-			'message' => __( 'Invalid email.', 'give' ),
+			'message' => __('Invalid email.', 'give'),
 		);
 	} else {
-		$email    = sanitize_email( $args['email'] );
+		$email    = sanitize_email($args['email']);
 		$donor_id = (int) $args['customer_id'];
 		$primary  = 'true' === $args['primary'] ? true : false;
-		$donor    = new Give_Donor( $donor_id );
-		if ( false === $donor->add_email( $email, $primary ) ) {
-			if ( in_array( $email, $donor->emails ) ) {
+		$donor    = new Give_Donor($donor_id);
+		if (false === $donor->add_email($email, $primary)) {
+			if (in_array($email, $donor->emails)) {
 				$output = array(
 					'success' => false,
-					'message' => __( 'Email already associated with this donor.', 'give' ),
+					'message' => __('Email already associated with this donor.', 'give'),
 				);
 			} else {
 				$output = array(
 					'success' => false,
-					'message' => __( 'Email address is already associated with another donor.', 'give' ),
+					'message' => __('Email address is already associated with another donor.', 'give'),
 				);
 			}
 		} else {
-			$redirect = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor_id . '&give-messages[]=email-added' );
+			$redirect = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor_id.'&give-messages[]=email-added');
 			$output   = array(
 				'success'  => true,
-				'message'  => __( 'Email successfully added to donor.', 'give' ),
+				'message'  => __('Email successfully added to donor.', 'give'),
 				'redirect' => $redirect,
 			);
 
 			$user       = wp_get_current_user();
-			$user_login = ! empty( $user->user_login ) ? $user->user_login : __( 'System', 'give' );
-			$donor_note = sprintf( __( 'Email address %1$s added by %2$s', 'give' ), $email, $user_login );
-			$donor->add_note( $donor_note );
+			$user_login = ! empty($user->user_login) ? $user->user_login : __('System', 'give');
+			$donor_note = sprintf(__('Email address %1$s added by %2$s', 'give'), $email, $user_login);
+			$donor->add_note($donor_note);
 
-			if ( $primary ) {
-				$donor_note = sprintf( __( 'Email address %1$s set as primary by %2$s', 'give' ), $email, $user_login );
-				$donor->add_note( $donor_note );
+			if ($primary) {
+				$donor_note = sprintf(__('Email address %1$s set as primary by %2$s', 'give'), $email, $user_login);
+				$donor->add_note($donor_note);
 			}
 		}
 	} // End if().
 
-	do_action( 'give_post_add_donor_email', $donor_id, $args );
+	do_action('give_post_add_donor_email', $donor_id, $args);
 
-	if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
-		header( 'Content-Type: application/json' );
-		echo json_encode( $output );
+	if (defined('DOING_AJAX') && DOING_AJAX) {
+		header('Content-Type: application/json');
+		echo json_encode($output);
 		wp_die();
 	}
 
 	return $output;
 }
 
-add_action( 'give_add_donor_email', 'give_add_donor_email', 10, 1 );
+add_action('give_add_donor_email', 'give_add_donor_email', 10, 1);
 
 
 /**
@@ -437,39 +437,39 @@ 
  * @return bool|null
  */
 function give_remove_donor_email() {
-	if ( empty( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
+	if (empty($_GET['id']) || ! is_numeric($_GET['id'])) {
 		return false;
 	}
-	if ( empty( $_GET['email'] ) || ! is_email( $_GET['email'] ) ) {
+	if (empty($_GET['email']) || ! is_email($_GET['email'])) {
 		return false;
 	}
-	if ( empty( $_GET['_wpnonce'] ) ) {
+	if (empty($_GET['_wpnonce'])) {
 		return false;
 	}
 
 	$nonce = $_GET['_wpnonce'];
-	if ( ! wp_verify_nonce( $nonce, 'give-remove-donor-email' ) ) {
-		wp_die( __( 'Nonce verification failed', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'give-remove-donor-email')) {
+		wp_die(__('Nonce verification failed', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	$donor = new Give_Donor( $_GET['id'] );
-	if ( $donor->remove_email( $_GET['email'] ) ) {
-		$url        = add_query_arg( 'give-messages[]', 'email-removed', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ) );
+	$donor = new Give_Donor($_GET['id']);
+	if ($donor->remove_email($_GET['email'])) {
+		$url        = add_query_arg('give-messages[]', 'email-removed', admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id));
 		$user       = wp_get_current_user();
-		$user_login = ! empty( $user->user_login ) ? $user->user_login : __( 'System', 'give' );
-		$donor_note = sprintf( __( 'Email address %1$s removed by %2$s', 'give' ), $_GET['email'], $user_login );
-		$donor->add_note( $donor_note );
+		$user_login = ! empty($user->user_login) ? $user->user_login : __('System', 'give');
+		$donor_note = sprintf(__('Email address %1$s removed by %2$s', 'give'), $_GET['email'], $user_login);
+		$donor->add_note($donor_note);
 	} else {
-		$url = add_query_arg( 'give-messages[]', 'email-remove-failed', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ) );
+		$url = add_query_arg('give-messages[]', 'email-remove-failed', admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id));
 	}
 
-	wp_safe_redirect( $url );
+	wp_safe_redirect($url);
 	exit;
 }
 
-add_action( 'give_remove_donor_email', 'give_remove_donor_email', 10 );
+add_action('give_remove_donor_email', 'give_remove_donor_email', 10);
 
 
 /**
@@ -481,44 +481,44 @@ 
  * @return bool|null
  */
 function give_set_donor_primary_email() {
-	if ( empty( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
+	if (empty($_GET['id']) || ! is_numeric($_GET['id'])) {
 		return false;
 	}
 
-	if ( empty( $_GET['email'] ) || ! is_email( $_GET['email'] ) ) {
+	if (empty($_GET['email']) || ! is_email($_GET['email'])) {
 		return false;
 	}
 
-	if ( empty( $_GET['_wpnonce'] ) ) {
+	if (empty($_GET['_wpnonce'])) {
 		return false;
 	}
 
 	$nonce = $_GET['_wpnonce'];
 
-	if ( ! wp_verify_nonce( $nonce, 'give-set-donor-primary-email' ) ) {
-		wp_die( __( 'Nonce verification failed', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'give-set-donor-primary-email')) {
+		wp_die(__('Nonce verification failed', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	$donor = new Give_Donor( $_GET['id'] );
+	$donor = new Give_Donor($_GET['id']);
 
-	if ( $donor->set_primary_email( $_GET['email'] ) ) {
-		$url        = add_query_arg( 'give-messages[]', 'primary-email-updated', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ) );
+	if ($donor->set_primary_email($_GET['email'])) {
+		$url        = add_query_arg('give-messages[]', 'primary-email-updated', admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id));
 		$user       = wp_get_current_user();
-		$user_login = ! empty( $user->user_login ) ? $user->user_login : __( 'System', 'give' );
-		$donor_note = sprintf( __( 'Email address %1$s set as primary by %2$s', 'give' ), $_GET['email'], $user_login );
+		$user_login = ! empty($user->user_login) ? $user->user_login : __('System', 'give');
+		$donor_note = sprintf(__('Email address %1$s set as primary by %2$s', 'give'), $_GET['email'], $user_login);
 
-		$donor->add_note( $donor_note );
+		$donor->add_note($donor_note);
 	} else {
-		$url = add_query_arg( 'give-messages[]', 'primary-email-failed', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ) );
+		$url = add_query_arg('give-messages[]', 'primary-email-failed', admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id));
 	}
 
-	wp_safe_redirect( $url );
+	wp_safe_redirect($url);
 	exit;
 }
 
-add_action( 'give_set_donor_primary_email', 'give_set_donor_primary_email', 10 );
+add_action('give_set_donor_primary_email', 'give_set_donor_primary_email', 10);
 
 
 /**
@@ -528,15 +528,15 @@ 
  *
  * @since 2.2
  */
-function give_process_donor_deletion( $args ) {
+function give_process_donor_deletion($args) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
 	// Verify user capabilities to proceed for deleting donor.
-	if ( ! is_admin() || ! current_user_can( $donor_edit_role ) ) {
+	if ( ! is_admin() || ! current_user_can($donor_edit_role)) {
 		wp_die(
-			esc_html__( 'You do not have permission to delete donors.', 'give' ),
-			esc_html__( 'Error', 'give' ),
+			esc_html__('You do not have permission to delete donors.', 'give'),
+			esc_html__('Error', 'give'),
 			array(
 				'response' => 403,
 			)
@@ -544,32 +544,32 @@ 
 	}
 
 	$nonce_action = '';
-	if ( 'delete_bulk_donor' === $args['give_action'] ) {
+	if ('delete_bulk_donor' === $args['give_action']) {
 		$nonce_action = 'bulk-donors';
-	} elseif ( 'delete_donor' === $args['give_action'] ) {
+	} elseif ('delete_donor' === $args['give_action']) {
 		$nonce_action = 'give-delete-donor';
 	}
 
 	// Verify Nonce for deleting bulk donors.
-	give_validate_nonce( $args['_wpnonce'], $nonce_action );
+	give_validate_nonce($args['_wpnonce'], $nonce_action);
 
 	$redirect_args            = array();
-	$donor_ids                = ( isset( $args['donor'] ) && is_array( $args['donor'] ) ) ? $args['donor'] : array( $args['donor_id'] );
-	$redirect_args['order']   = ! empty( $args['order'] ) ? $args['order'] : 'DESC';
-	$redirect_args['orderby'] = ! empty( $args['orderby'] ) ? $args['orderby'] : 'ID';
-	$redirect_args['s']       = ! empty( $args['s'] ) ? $args['s'] : '';
-	$delete_donor             = ! empty( $args['give-donor-delete-confirm'] ) ? give_is_setting_enabled( $args['give-donor-delete-confirm'] ) : false;
-	$delete_donation          = ! empty( $args['give-donor-delete-records'] ) ? give_is_setting_enabled( $args['give-donor-delete-records'] ) : false;
+	$donor_ids                = (isset($args['donor']) && is_array($args['donor'])) ? $args['donor'] : array($args['donor_id']);
+	$redirect_args['order']   = ! empty($args['order']) ? $args['order'] : 'DESC';
+	$redirect_args['orderby'] = ! empty($args['orderby']) ? $args['orderby'] : 'ID';
+	$redirect_args['s']       = ! empty($args['s']) ? $args['s'] : '';
+	$delete_donor             = ! empty($args['give-donor-delete-confirm']) ? give_is_setting_enabled($args['give-donor-delete-confirm']) : false;
+	$delete_donation          = ! empty($args['give-donor-delete-records']) ? give_is_setting_enabled($args['give-donor-delete-records']) : false;
 
-	if ( count( $donor_ids ) > 0 ) {
+	if (count($donor_ids) > 0) {
 
 		// Loop through the selected donors to delete.
-		foreach ( $donor_ids as $donor_id ) {
+		foreach ($donor_ids as $donor_id) {
 
-			$donor = new Give_Donor( $donor_id );
+			$donor = new Give_Donor($donor_id);
 
 			// Proceed only if valid donor id is provided.
-			if ( $donor->id > 0 ) {
+			if ($donor->id > 0) {
 
 				/**
 				 * Fires before deleting donor.
@@ -580,19 +580,19 @@ 
 				 *
 				 * @since 1.0
 				 */
-				do_action( 'give_pre_delete_donor', $donor->id, $delete_donor, $delete_donation );
+				do_action('give_pre_delete_donor', $donor->id, $delete_donor, $delete_donation);
 
 				// Proceed only, if user confirmed whether they need to delete the donor.
-				if ( $delete_donor ) {
+				if ($delete_donor) {
 
 					// Delete donor and linked donations.
-					$donor_delete_status = give_delete_donor_and_related_donation( $donor, array(
+					$donor_delete_status = give_delete_donor_and_related_donation($donor, array(
 						'delete_donation' => $delete_donation,
-					) );
+					));
 
-					if ( 1 === $donor_delete_status ) {
+					if (1 === $donor_delete_status) {
 						$redirect_args['give-messages[]'] = 'donor-deleted';
-					} elseif ( 2 === $donor_delete_status ) {
+					} elseif (2 === $donor_delete_status) {
 						$redirect_args['give-messages[]'] = 'donor-donations-deleted';
 					}
 				} else {
@@ -608,12 +608,12 @@ 
 
 	$redirect_url = add_query_arg(
 		$redirect_args,
-		admin_url( 'edit.php?post_type=give_forms&page=give-donors' )
+		admin_url('edit.php?post_type=give_forms&page=give-donors')
 	);
 
-	wp_safe_redirect( $redirect_url );
+	wp_safe_redirect($redirect_url);
 	give_die();
 
 }
-add_action( 'give_delete_donor', 'give_process_donor_deletion' );
-add_action( 'give_delete_bulk_donor', 'give_process_donor_deletion' );
+add_action('give_delete_donor', 'give_process_donor_deletion');
+add_action('give_delete_bulk_donor', 'give_process_donor_deletion');

includes/admin/donors/donor-functions.php

Doc Comments +1 added lines, -1 removed lines

@@ -162,7 +162,7 @@
 /**
  * This function is used to delete donor and related donation without redirection.
  *
- * @param int|Give_Donor $donor Donor ID or List of Donor IDs.
+ * @param Give_Donor $donor Donor ID or List of Donor IDs.
  * @param array          $args  List of arguments to handle donor and related donation deletion process.
  *
  * @type bool delete_donation Delete donor linked donations if set to true. Default is false.

Spacing +37 added lines, -37 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,7 +23,7 @@ 
  *
  * @return array        The altered list of views.
  */
-function give_register_default_donor_views( $views ) {
+function give_register_default_donor_views($views) {
 
 	$default_views = array(
 		'overview' => 'give_donor_view',
@@ -31,11 +31,11 @@ 
 		'notes'    => 'give_donor_notes_view',
 	);
 
-	return array_merge( $views, $default_views );
+	return array_merge($views, $default_views);
 
 }
 
-add_filter( 'give_donor_views', 'give_register_default_donor_views', 1, 1 );
+add_filter('give_donor_views', 'give_register_default_donor_views', 1, 1);
 
 /**
  * Register a tab for the single donor view.
@@ -46,23 +46,23 @@ 
  *
  * @return array       The altered list of tabs
  */
-function give_register_default_donor_tabs( $tabs ) {
+function give_register_default_donor_tabs($tabs) {
 
 	$default_tabs = array(
 		'overview' => array(
 			'dashicon' => 'dashicons-admin-users',
-			'title'    => __( 'Donor Profile', 'give' ),
+			'title'    => __('Donor Profile', 'give'),
 		),
 		'notes'    => array(
 			'dashicon' => 'dashicons-admin-comments',
-			'title'    => __( 'Donor Notes', 'give' ),
+			'title'    => __('Donor Notes', 'give'),
 		),
 	);
 
-	return array_merge( $tabs, $default_tabs );
+	return array_merge($tabs, $default_tabs);
 }
 
-add_filter( 'give_donor_tabs', 'give_register_default_donor_tabs', 1, 1 );
+add_filter('give_donor_tabs', 'give_register_default_donor_tabs', 1, 1);
 
 /**
  * Register the Delete icon as late as possible so it's at the bottom.
@@ -73,17 +73,17 @@ 
  *
  * @return array       The altered list of tabs, with 'delete' at the bottom.
  */
-function give_register_delete_donor_tab( $tabs ) {
+function give_register_delete_donor_tab($tabs) {
 
 	$tabs['delete'] = array(
 		'dashicon' => 'dashicons-trash',
-		'title'    => __( 'Delete Donor', 'give' ),
+		'title'    => __('Delete Donor', 'give'),
 	);
 
 	return $tabs;
 }
 
-add_filter( 'give_donor_tabs', 'give_register_delete_donor_tab', PHP_INT_MAX, 1 );
+add_filter('give_donor_tabs', 'give_register_delete_donor_tab', PHP_INT_MAX, 1);
 
 /**
  * Connect and Reconnect Donor with User profile.
@@ -98,7 +98,7 @@ 
  *
  * @return array
  */
-function give_connect_user_donor_profile( $donor, $donor_data, $address ) {
+function give_connect_user_donor_profile($donor, $donor_data, $address) {
 
 	$donor_id         = $donor->id;
 	$previous_user_id = $donor->user_id;
@@ -112,32 +112,32 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_pre_edit_donor', $donor_id, $donor_data, $address );
+	do_action('give_pre_edit_donor', $donor_id, $donor_data, $address);
 
 	$output = array();
 
-	if ( $donor->update( $donor_data ) ) {
+	if ($donor->update($donor_data)) {
 
 		// Create and Update Donor First Name and Last Name in Meta Fields.
-		$donor->update_meta( '_give_donor_first_name', $donor_data['first_name'] );
-		$donor->update_meta( '_give_donor_last_name', $donor_data['last_name'] );
-		$donor->update_meta( '_give_donor_title_prefix', $donor_data['title'] );
+		$donor->update_meta('_give_donor_first_name', $donor_data['first_name']);
+		$donor->update_meta('_give_donor_last_name', $donor_data['last_name']);
+		$donor->update_meta('_give_donor_title_prefix', $donor_data['title']);
 
 		// Fetch disconnected user id, if exists.
-		$disconnected_user_id = $donor->get_meta( '_give_disconnected_user_id', true );
+		$disconnected_user_id = $donor->get_meta('_give_disconnected_user_id', true);
 
 		// Flag User and Donor Disconnection.
-		delete_user_meta( $disconnected_user_id, '_give_is_donor_disconnected' );
+		delete_user_meta($disconnected_user_id, '_give_is_donor_disconnected');
 
 		// Check whether the disconnected user id and the reconnected user id are same or not.
 		// If both are same then delete user id store in donor meta.
-		if ( $donor_data['user_id'] === $disconnected_user_id ) {
-			delete_user_meta( $disconnected_user_id, '_give_disconnected_donor_id' );
-			$donor->delete_meta( '_give_disconnected_user_id' );
+		if ($donor_data['user_id'] === $disconnected_user_id) {
+			delete_user_meta($disconnected_user_id, '_give_disconnected_donor_id');
+			$donor->delete_meta('_give_disconnected_user_id');
 		}
 
 		$output['success']       = true;
-		$donor_data              = array_merge( $donor_data, $address );
+		$donor_data              = array_merge($donor_data, $address);
 		$output['customer_info'] = $donor_data;
 
 	} else {
@@ -154,7 +154,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_post_edit_donor', $donor_id, $donor_data );
+	do_action('give_post_edit_donor', $donor_id, $donor_data);
 
 	return $output;
 }
@@ -171,39 +171,39 @@ 
  *
  * @return int
  */
-function give_delete_donor_and_related_donation( $donor, $args = array() ) {
+function give_delete_donor_and_related_donation($donor, $args = array()) {
 
 	// Default Arguments.
 	$default_args = array(
 		'delete_donation' => false,
 	);
 
-	$args = wp_parse_args( $args, $default_args );
+	$args = wp_parse_args($args, $default_args);
 
 	// If $donor not an instance of Give_Donor then create one.
-	if ( ! $donor instanceof Give_Donor ) {
-		$donor = new Give_Donor( $donor );
+	if ( ! $donor instanceof Give_Donor) {
+		$donor = new Give_Donor($donor);
 	}
 
-	if ( $donor->id > 0 ) {
+	if ($donor->id > 0) {
 
 		// Delete Donor.
-		$donor_deleted = Give()->donors->delete( $donor->id );
+		$donor_deleted = Give()->donors->delete($donor->id);
 
 		// Fetch linked donations of a particular donor.
-		$donation_ids  = explode( ',', $donor->payment_ids );
+		$donation_ids  = explode(',', $donor->payment_ids);
 
 		// Proceed to delete related donation, if user opted and donor is deleted successfully.
-		if ( $donor_deleted && $args['delete_donation'] ) {
-			foreach ( $donation_ids as $donation_id ) {
-				give_delete_donation( $donation_id );
+		if ($donor_deleted && $args['delete_donation']) {
+			foreach ($donation_ids as $donation_id) {
+				give_delete_donation($donation_id);
 			}
 
 			return 2; // Donor and linked Donations deleted.
 
 		} else {
-			foreach ( $donation_ids as $donation_id ) {
-				give_update_payment_meta( $donation_id, '_give_payment_donor_id', 0 );
+			foreach ($donation_ids as $donation_id) {
+				give_update_payment_meta($donation_id, '_give_payment_donor_id', 0);
 			}
 		}
 

includes/admin/tools/data/class-give-tools-delete-import-donors.php

Doc Comments +1 added lines, -1 removed lines

@@ -290,7 +290,7 @@
 	 *
 	 * @since 1.8.12
 	 *
-	 * @return int
+	 * @return double
 	 */
 	public function get_percentage_complete() {
 		return ceil( ( 100 * $this->step_completed ) / $this->total_step );

Spacing +116 added lines, -116 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -134,8 +134,8 @@ 
 	 *
 	 * @param int $_step Steps.
 	 */
-	public function __construct( $_step = 1 ) {
-		parent::__construct( $_step );
+	public function __construct($_step = 1) {
+		parent::__construct($_step);
 
 		$this->is_writable = true;
 	}
@@ -154,38 +154,38 @@ 
 		$donor_ids    = array();
 
 		// Check if the ajax request if running for the first time.
-		if ( 1 === (int) $this->step ) {
+		if (1 === (int) $this->step) {
 
 			// Delete all the form ids.
-			$this->delete_option( $this->form_key );
+			$this->delete_option($this->form_key);
 
 			// Delete all the donation ids.
-			$this->delete_option( $this->donation_key );
+			$this->delete_option($this->donation_key);
 
 			// Delete all the donor ids.
-			$this->delete_option( $this->donor_key );
+			$this->delete_option($this->donor_key);
 
 			// Delete all the step and set to 'count' which if the first step in the process of deleting the donors.
-			$this->update_option( $this->step_key, 'count' );
+			$this->update_option($this->step_key, 'count');
 
 			// Delete tha page count of the step.
-			$this->update_option( $this->step_on_key, '0' );
+			$this->update_option($this->step_on_key, '0');
 		} else {
 
 			// Get the old donors list.
-			$donor_ids = $this->get_option( $this->donor_key );
+			$donor_ids = $this->get_option($this->donor_key);
 
 			// Get the old donation list.
-			$donation_ids = $this->get_option( $this->donation_key );
+			$donation_ids = $this->get_option($this->donation_key);
 		}
 
 		// Get the step and check for it if it's on the first step( 'count' ) or not.
 		$step = (int) $this->get_step();
-		if ( 1 === $step ) {
+		if (1 === $step) {
 			/**
 			 * Will add or update the donation and donor data by running wp query.
 			 */
-			$this->count( $step, $donation_ids, $donor_ids );
+			$this->count($step, $donation_ids, $donor_ids);
 		}
 	}
 
@@ -196,18 +196,18 @@ 
 	 * @param array  $donation_ids Contain the list of all the donation id's that has being add before this.
 	 * @param array  $donor_ids    Contain the list of all the donors id's that has being add before this.
 	 */
-	private function count( $step, $donation_ids = array(), $donor_ids = array() ) {
+	private function count($step, $donation_ids = array(), $donor_ids = array()) {
 
 		// Get the Page count by default it's zero.
 		$paged = (int) $this->get_step_page();
 
 		// Increased the page count by one.
-		++ $paged;
+		++$paged;
 
 		/**
 		 * Filter add to alter the argument before the wp quest run
 		 */
-		$args = apply_filters( 'give_tools_reset_stats_total_args', array(
+		$args = apply_filters('give_tools_reset_stats_total_args', array(
 			'post_type'      => 'give_payment',
 			'post_status'    => 'any',
 			'posts_per_page' => $this->per_step,
@@ -215,41 +215,41 @@ 
 			'meta_key'       => '_give_payment_import',
 			'meta_value_num' => 1,
 			'meta_compare'   => '=',
-		) );
+		));
 
 		// Reset the post data.
 		wp_reset_postdata();
 
 		// Getting the new donation.
-		$donation_posts = new WP_Query( $args );
+		$donation_posts = new WP_Query($args);
 
 		// The Loop.
-		if ( $donation_posts->have_posts() ) {
-			while ( $donation_posts->have_posts() ) {
+		if ($donation_posts->have_posts()) {
+			while ($donation_posts->have_posts()) {
 				$add_author = true;
 				$donation_posts->the_post();
 				global $post;
 				// Add the donation id in side the array.
 				$donation_ids[] = $post->ID;
 
-				$donor_id = (int) give_get_meta( $post->ID, '_give_payment_customer_id', true );
-				if ( ! empty( $donor_id ) ) {
-					$donor = new Give_Donor( $donor_id );
-					if ( ! empty( $donor->id ) ) {
-						if ( empty( $donor->user_id ) && ! empty( $donor->payment_ids ) ) {
+				$donor_id = (int) give_get_meta($post->ID, '_give_payment_customer_id', true);
+				if ( ! empty($donor_id)) {
+					$donor = new Give_Donor($donor_id);
+					if ( ! empty($donor->id)) {
+						if (empty($donor->user_id) && ! empty($donor->payment_ids)) {
 							$add_author = false;
-							$count      = (int) count( $donor->payment_ids );
-							if ( 1 === $count ) {
-								give_delete_donor_and_related_donation( $donor );
+							$count      = (int) count($donor->payment_ids);
+							if (1 === $count) {
+								give_delete_donor_and_related_donation($donor);
 							} else {
-								$donor->remove_payment( $post->ID );
+								$donor->remove_payment($post->ID);
 								$donor->decrease_donation_count();
 							}
 						}
 					}
 				}
 
-				if ( ! empty( $add_author ) ) {
+				if ( ! empty($add_author)) {
 					// Add the donor id in side the array.
 					$donor_ids[] = (int) $post->post_author;
 				}
@@ -263,24 +263,24 @@ 
 		$max_num_pages = (int) $donation_posts->max_num_pages;
 
 		// Check current page is less then max number of page or not.
-		if ( $paged < $max_num_pages ) {
+		if ($paged < $max_num_pages) {
 
 			// Update the current page variable for the next step.
-			$this->update_option( $this->step_on_key, $paged );
+			$this->update_option($this->step_on_key, $paged);
 
 			// Calculating percentage.
 			$page_remain          = $max_num_pages - $paged;
-			$this->total_step     = (int) $max_num_pages + ( $total_donation / $this->per_step ) + ( ( $page_remain * 2 ) * count( $donor_ids ) );
+			$this->total_step     = (int) $max_num_pages + ($total_donation / $this->per_step) + (($page_remain * 2) * count($donor_ids));
 			$this->step_completed = $paged;
 		} else {
-			$donation_ids_count = count( $donor_ids );
-			$this->update_option( $this->step_key, 'donation' );
-			$this->update_option( $this->step_on_key, '0' );
+			$donation_ids_count = count($donor_ids);
+			$this->update_option($this->step_key, 'donation');
+			$this->update_option($this->step_on_key, '0');
 		}
 
-		$donor_ids = array_unique( $donor_ids );
-		$this->update_option( $this->donor_key, $donor_ids );
-		$this->update_option( $this->donation_key, $donation_ids );
+		$donor_ids = array_unique($donor_ids);
+		$this->update_option($this->donor_key, $donor_ids);
+		$this->update_option($this->donation_key, $donation_ids);
 
 		wp_reset_postdata();
 	}
@@ -293,7 +293,7 @@ 
 	 * @return int
 	 */
 	public function get_percentage_complete() {
-		return ceil( ( 100 * $this->step_completed ) / $this->total_step );
+		return ceil((100 * $this->step_completed) / $this->total_step);
 	}
 
 	/**
@@ -303,10 +303,10 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
+		if ( ! $this->can_export()) {
 			wp_die(
-				esc_html__( 'You do not have permission to delete Import transactions.', 'give' ),
-				esc_html__( 'Error', 'give' ),
+				esc_html__('You do not have permission to delete Import transactions.', 'give'),
+				esc_html__('Error', 'give'),
 				array(
 					'response' => 403,
 				)
@@ -315,18 +315,18 @@ 
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
-			update_option( 'give_earnings_total', give_get_total_earnings( true ), false );
-			Give_Cache::delete( Give_Cache::get_key( 'give_estimated_monthly_stats' ) );
+			update_option('give_earnings_total', give_get_total_earnings(true), false);
+			Give_Cache::delete(Give_Cache::get_key('give_estimated_monthly_stats'));
 
-			$this->delete_option( $this->donation_key );
+			$this->delete_option($this->donation_key);
 
 			$this->done    = true;
-			$this->message = __( 'Imported donor and transactions successfully deleted.', 'give' );
+			$this->message = __('Imported donor and transactions successfully deleted.', 'give');
 
 			return false;
 		}
@@ -344,12 +344,12 @@ 
 	public function get_data() {
 
 		// Get the donation id's.
-		$donation_ids = $this->get_option( $this->donation_key );
+		$donation_ids = $this->get_option($this->donation_key);
 
 		/**
 		 * Return false id not Import donation is found.
 		 */
-		if ( empty( $donation_ids ) ) {
+		if (empty($donation_ids)) {
 			$this->is_empty   = true;
 			$this->total_step = 1;
 
@@ -360,110 +360,110 @@ 
 		$step = (int) $this->get_step();
 
 		// Get the donor ids.
-		$donor_ids = $this->get_option( $this->donor_key );
+		$donor_ids = $this->get_option($this->donor_key);
 
 		// Delete all the imported donations.
-		if ( 2 === $step ) {
+		if (2 === $step) {
 			$pass_to_donor = false;
 			$page          = (int) $this->get_step_page();
-			$page ++;
-			$count = count( $donation_ids );
+			$page++;
+			$count = count($donation_ids);
 
-			$this->total_step     = ( ( count( $donation_ids ) / $this->per_step ) * 2 ) + count( $donor_ids );
+			$this->total_step     = ((count($donation_ids) / $this->per_step) * 2) + count($donor_ids);
 			$this->step_completed = $page;
 
-			if ( $count > $this->per_step ) {
+			if ($count > $this->per_step) {
 
-				$this->update_option( $this->step_on_key, $page );
-				$donation_ids = $this->get_delete_ids( $donation_ids, $page );
-				$current_page = (int) ceil( $count / $this->per_step );
+				$this->update_option($this->step_on_key, $page);
+				$donation_ids = $this->get_delete_ids($donation_ids, $page);
+				$current_page = (int) ceil($count / $this->per_step);
 
-				if ( $page === $current_page ) {
+				if ($page === $current_page) {
 					$pass_to_donor = true;
 				}
 			} else {
 				$pass_to_donor = true;
 			}
 
-			if ( true === $pass_to_donor ) {
-				$this->update_option( $this->step_key, 'donor' );
-				$this->update_option( $this->step_on_key, '0' );
+			if (true === $pass_to_donor) {
+				$this->update_option($this->step_key, 'donor');
+				$this->update_option($this->step_on_key, '0');
 			}
 
 			// Get the old form list.
-			$form_ids = (array) $this->get_option( $this->form_key );
+			$form_ids = (array) $this->get_option($this->form_key);
 
-			foreach ( $donation_ids as $item ) {
-				$form_ids[] = give_get_meta( $item, '_give_payment_form_id', true );
+			foreach ($donation_ids as $item) {
+				$form_ids[] = give_get_meta($item, '_give_payment_form_id', true);
 
 				// Delete the main payment.
-				give_delete_donation( absint( $item ) );
+				give_delete_donation(absint($item));
 			}
 
 			// Update the new form list.
-			$this->update_option( $this->form_key, $form_ids );
+			$this->update_option($this->form_key, $form_ids);
 		} // End if().
 
 		// Delete all the donors.
-		if ( 3 === $step ) {
+		if (3 === $step) {
 
 			// Get the old form list.
-			$form_ids = (array) $this->get_option( $this->form_key );
-			if ( ! empty( $form_ids ) ) {
-				$form_ids = array_unique( $form_ids );
-				foreach ( $form_ids as $form_id ) {
-					give_recount_form_income_donation( (int) $form_id );
+			$form_ids = (array) $this->get_option($this->form_key);
+			if ( ! empty($form_ids)) {
+				$form_ids = array_unique($form_ids);
+				foreach ($form_ids as $form_id) {
+					give_recount_form_income_donation((int) $form_id);
 				}
 			}
 			// update the new form list.
-			$this->update_option( $this->form_key, array() );
+			$this->update_option($this->form_key, array());
 
 			$page  = (int) $this->get_step_page();
-			$count = count( $donor_ids );
+			$count = count($donor_ids);
 
-			$this->total_step     = ( ( count( $donation_ids ) / $this->per_step ) * 2 ) + count( $donor_ids );
-			$this->step_completed = $page + ( count( $donation_ids ) / $this->per_step );
+			$this->total_step     = ((count($donation_ids) / $this->per_step) * 2) + count($donor_ids);
+			$this->step_completed = $page + (count($donation_ids) / $this->per_step);
 
-			if ( ! empty( $donor_ids[ $page ] ) ) {
-				$args = apply_filters( 'give_tools_reset_stats_total_args', array(
+			if ( ! empty($donor_ids[$page])) {
+				$args = apply_filters('give_tools_reset_stats_total_args', array(
 					'post_status'    => 'any',
 					'posts_per_page' => 1,
-					'author'         => $donor_ids[ $page ],
-				) );
+					'author'         => $donor_ids[$page],
+				));
 
 				$donations = array();
-				$payments  = new Give_Payments_Query( $args );
+				$payments  = new Give_Payments_Query($args);
 				$payments  = $payments->get_payments();
-				if ( empty( $payments ) ) {
-					Give()->donors->delete_by_user_id( $donor_ids[ $page ] );
+				if (empty($payments)) {
+					Give()->donors->delete_by_user_id($donor_ids[$page]);
 
 					/**
 					 * If Checked then delete WP user.
 					 *
 					 * @since 1.8.14
 					 */
-					$delete_import_donors = isset( $_REQUEST['delete-import-donors'] ) ? sanitize_text_field( wp_unslash( $_REQUEST['delete-import-donors'] ) ) : '';
+					$delete_import_donors = isset($_REQUEST['delete-import-donors']) ? sanitize_text_field(wp_unslash($_REQUEST['delete-import-donors'])) : '';
 
-					if ( 'on' === (string) $delete_import_donors ) {
-						wp_delete_user( $donor_ids[ $page ] );
+					if ('on' === (string) $delete_import_donors) {
+						wp_delete_user($donor_ids[$page]);
 					}
 				} else {
-					foreach ( $payments as $payment ) {
+					foreach ($payments as $payment) {
 						$donations[] = $payment->ID;
 					}
 
-					$donor          = new Give_Donor( $donor_ids[ $page ], true );
+					$donor          = new Give_Donor($donor_ids[$page], true);
 					$data_to_update = array(
-						'purchase_count' => count( $donations ),
-						'payment_ids'    => implode( ',', $donations ),
+						'purchase_count' => count($donations),
+						'payment_ids'    => implode(',', $donations),
 					);
-					$donor->update( $data_to_update );
+					$donor->update($data_to_update);
 				}
 			} // End if().
 
-			$page ++;
-			$this->update_option( $this->step_on_key, $page );
-			if ( $count === $page ) {
+			$page++;
+			$this->update_option($this->step_on_key, $page);
+			if ($count === $page) {
 				$this->is_empty = false;
 
 				return false;
@@ -483,24 +483,24 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function get_delete_ids( $donation_ids, $page ) {
-		$index            = $page --;
-		$count            = count( $donation_ids );
+	public function get_delete_ids($donation_ids, $page) {
+		$index            = $page--;
+		$count            = count($donation_ids);
 		$temp             = 0;
 		$current_page     = 0;
 		$post_delete      = $this->per_step;
 		$page_donation_id = array();
 
-		foreach ( $donation_ids as $item ) {
-			$temp ++;
-			$page_donation_id[ $current_page ][] = $item;
-			if ( $temp === $post_delete ) {
-				$current_page ++;
+		foreach ($donation_ids as $item) {
+			$temp++;
+			$page_donation_id[$current_page][] = $item;
+			if ($temp === $post_delete) {
+				$current_page++;
 				$temp = 0;
 			}
 		}
 
-		return $page_donation_id[ $page ];
+		return $page_donation_id[$page];
 	}
 
 	/**
@@ -513,8 +513,8 @@ 
 	 *
 	 * @return mixed Returns the data from the database
 	 */
-	public function get_option( $key, $default_value = false ) {
-		return get_option( $key, $default_value );
+	public function get_option($key, $default_value = false) {
+		return get_option($key, $default_value);
 	}
 
 	/**
@@ -527,8 +527,8 @@ 
 	 *
 	 * @return void
 	 */
-	public function update_option( $key, $value ) {
-		update_option( $key, $value, false );
+	public function update_option($key, $value) {
+		update_option($key, $value, false);
 	}
 
 	/**
@@ -540,8 +540,8 @@ 
 	 *
 	 * @return void
 	 */
-	public function delete_option( $key ) {
-		delete_option( $key );
+	public function delete_option($key) {
+		delete_option($key);
 	}
 
 	/**
@@ -552,12 +552,12 @@ 
 	 * @return int|string
 	 */
 	private function get_step() {
-		$step_key = (string) $this->get_option( $this->step_key, false );
-		if ( 'count' === $step_key ) {
+		$step_key = (string) $this->get_option($this->step_key, false);
+		if ('count' === $step_key) {
 			return 1;
-		} elseif ( 'donation' === $step_key ) {
+		} elseif ('donation' === $step_key) {
 			return 2;
-		} elseif ( 'donor' === $step_key ) {
+		} elseif ('donor' === $step_key) {
 			return 3;
 		} else {
 			return $step_key;
@@ -568,6 +568,6 @@ 
 	 * Get the current $page value in the ajax.
 	 */
 	private function get_step_page() {
-		return $this->get_option( $this->step_on_key, false );
+		return $this->get_option($this->step_on_key, false);
 	}
 }

includes/class-give-comment.php

Doc Comments +2 added lines, -1 removed lines

@@ -445,7 +445,7 @@ 
 	 * @param int        $comment_id
 	 * @param WP_Comment $comment
 	 *
-	 * @return mixed
+	 * @return string
 	 */
 	public function __get_comment_author( $author, $comment_id, $comment ) {
 		if ( in_array( $comment->comment_type, $this->comment_types ) ) {
@@ -468,6 +468,7 @@ 
 	 * @access public
 	 *
 	 * @param array @comment_types
+	 * @param string[] $comment_types
 	 *
 	 * @return array
 	 */

Spacing +80 added lines, -80 removed lines

@@ -46,7 +46,7 @@ 
 	 * @return Give_Comment
 	 */
 	public static function get_instance() {
-		if ( null === static::$instance ) {
+		if (null === static::$instance) {
 			self::$instance = new static();
 			self::$instance->init();
 		}
@@ -68,14 +68,14 @@ 
 		 */
 		$this->comment_types = apply_filters(
 			'give_comment_type',
-			self::get_comment_types( array( 'payment', 'donor' ) )
+			self::get_comment_types(array('payment', 'donor'))
 		);
 
-		add_action( 'pre_get_comments', array( $this, 'hide_comments' ), 10 );
-		add_filter( 'comments_clauses', array( $this, 'hide_comments_pre_wp_41' ), 10, 1 );
-		add_filter( 'comment_feed_where', array( $this, 'hide_comments_from_feeds' ), 10, 1 );
-		add_filter( 'wp_count_comments', array( $this, 'remove_comments_from_comment_counts' ), 10, 2 );
-		add_filter( 'get_comment_author', array( $this, '__get_comment_author' ), 10, 3 );
+		add_action('pre_get_comments', array($this, 'hide_comments'), 10);
+		add_filter('comments_clauses', array($this, 'hide_comments_pre_wp_41'), 10, 1);
+		add_filter('comment_feed_where', array($this, 'hide_comments_from_feeds'), 10, 1);
+		add_filter('wp_count_comments', array($this, 'remove_comments_from_comment_counts'), 10, 2);
+		add_filter('get_comment_author', array($this, '__get_comment_author'), 10, 3);
 	}
 
 	/**
@@ -91,13 +91,13 @@ 
 	 *
 	 * @return int|WP_Error
 	 */
-	public static function add( $id, $note, $comment_type, $comment_args = array() ) {
+	public static function add($id, $note, $comment_type, $comment_args = array()) {
 		// Bailout
-		if ( empty( $id ) || empty( $note ) || empty( $comment_type ) ) {
-			return new WP_Error( 'give_invalid_required_param', __( 'This comment has invalid ID or comment text or cooment type', 'give' ) );
+		if (empty($id) || empty($note) || empty($comment_type)) {
+			return new WP_Error('give_invalid_required_param', __('This comment has invalid ID or comment text or cooment type', 'give'));
 		}
 
-		$is_existing_comment = array_key_exists( 'comment_ID', $comment_args ) && ! empty( $comment_args['comment_ID'] );
+		$is_existing_comment = array_key_exists('comment_ID', $comment_args) && ! empty($comment_args['comment_ID']);
 		$action_type         = $is_existing_comment ? 'update' : 'insert';
 
 		/**
@@ -108,7 +108,7 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( "give_pre_{$action_type}_{$comment_type}_note", $id, $note );
+		do_action("give_pre_{$action_type}_{$comment_type}_note", $id, $note);
 
 		$comment_args = wp_parse_args(
 			$comment_args,
@@ -116,8 +116,8 @@ 
 				'comment_post_ID'      => $id,
 				'comment_content'      => $note,
 				'user_id'              => is_admin() ? get_current_user_id() : 0,
-				'comment_date'         => current_time( 'mysql' ),
-				'comment_date_gmt'     => current_time( 'mysql', 1 ),
+				'comment_date'         => current_time('mysql'),
+				'comment_date_gmt'     => current_time('mysql', 1),
 				'comment_approved'     => 1,
 				'comment_parent'       => 0,
 				'comment_author'       => '',
@@ -131,30 +131,30 @@ 
 
 
 		// Check comment max length.
-		$error = wp_check_comment_data_max_lengths( $comment_args );
-		if( is_wp_error( $error ) ) {
+		$error = wp_check_comment_data_max_lengths($comment_args);
+		if (is_wp_error($error)) {
 			return $error;
 		}
 
 		// Remove moderation emails when comment posted.
-		remove_action( 'comment_post', 'wp_new_comment_notify_moderator' );
-		remove_action( 'comment_post', 'wp_new_comment_notify_postauthor' );
+		remove_action('comment_post', 'wp_new_comment_notify_moderator');
+		remove_action('comment_post', 'wp_new_comment_notify_postauthor');
 
 		// Remove comment flood check.
-		remove_action( 'check_comment_flood', 'check_comment_flood_db', 10 );
+		remove_action('check_comment_flood', 'check_comment_flood_db', 10);
 
 		$comment_id = $is_existing_comment
-			? wp_update_comment( $comment_args )
-			: wp_new_comment( $comment_args, true );
+			? wp_update_comment($comment_args)
+			: wp_new_comment($comment_args, true);
 
 		// Add moderation emails when comment posted.
-		add_action( 'comment_post', 'wp_new_comment_notify_moderator' );
-		add_action( 'comment_post', 'wp_new_comment_notify_postauthor' );
+		add_action('comment_post', 'wp_new_comment_notify_moderator');
+		add_action('comment_post', 'wp_new_comment_notify_postauthor');
 
 		// Add comment flood check.
-		add_action( 'check_comment_flood', 'check_comment_flood_db', 10, 4 );
+		add_action('check_comment_flood', 'check_comment_flood_db', 10, 4);
 
-		update_comment_meta( $comment_id, "_give_{$comment_type}_id", $id );
+		update_comment_meta($comment_id, "_give_{$comment_type}_id", $id);
 
 		/**
 		 * Fires after payment|donor comment inserted/updated.
@@ -165,7 +165,7 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( "give_{$action_type}_{$comment_type}_note", $comment_id, $id, $note );
+		do_action("give_{$action_type}_{$comment_type}_note", $comment_id, $id, $note);
 
 		return $comment_id;
 	}
@@ -185,11 +185,11 @@ 
 	 *
 	 * @return bool True on success, false otherwise.
 	 */
-	public static function delete( $comment_id, $id, $comment_type ) {
+	public static function delete($comment_id, $id, $comment_type) {
 		$ret = false;
 
 		// Bailout
-		if ( empty( $id ) || empty( $comment_id ) || empty( $comment_type ) ) {
+		if (empty($id) || empty($comment_id) || empty($comment_type)) {
 			return $ret;
 		}
 
@@ -201,9 +201,9 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( "give_pre_delete_{$comment_type}_note", $comment_id, $id );
+		do_action("give_pre_delete_{$comment_type}_note", $comment_id, $id);
 
-		$ret = wp_delete_comment( $comment_id, true );
+		$ret = wp_delete_comment($comment_id, true);
 
 		/**
 		 * Fires after donation note deleted.
@@ -214,7 +214,7 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( "give_post_delete_{$comment_type}_note", $comment_id, $id, $ret );
+		do_action("give_post_delete_{$comment_type}_note", $comment_id, $id, $ret);
 
 		return $ret;
 	}
@@ -233,25 +233,25 @@ 
 	 *
 	 * @return array
 	 */
-	public static function get( $id, $comment_type, $comment_args = array(), $search = '' ) {
+	public static function get($id, $comment_type, $comment_args = array(), $search = '') {
 		$comments = array();
 
 		// Set default meta_query value.
-		if ( ! isset( $comment_args['meta_query'] ) ) {
+		if ( ! isset($comment_args['meta_query'])) {
 			$comment_args['meta_query'] = array();
 		}
 
 		// Bailout
-		if ( empty( $id ) || empty( $comment_type ) ) {
+		if (empty($id) || empty($comment_type)) {
 			return $comments;
 		}
 
-		remove_action( 'pre_get_comments', array( self::$instance, 'hide_comments' ), 10 );
-		remove_filter( 'comments_clauses', array( self::$instance, 'hide_comments_pre_wp_41' ), 10 );
+		remove_action('pre_get_comments', array(self::$instance, 'hide_comments'), 10);
+		remove_filter('comments_clauses', array(self::$instance, 'hide_comments_pre_wp_41'), 10);
 
-		switch ( $comment_type ) {
+		switch ($comment_type) {
 			case 'payment':
-				$comment_args['meta_query'] = ! empty( $comment_args['meta_query'] )
+				$comment_args['meta_query'] = ! empty($comment_args['meta_query'])
 					? $comment_args['meta_query']
 					: array(
 						array(
@@ -260,7 +260,7 @@ 
 						)
 					);
 
-				$comments = get_comments( wp_parse_args(
+				$comments = get_comments(wp_parse_args(
 					$comment_args,
 					array(
 						'post_id' => $id,
@@ -268,11 +268,11 @@ 
 						'search'  => $search,
 						'type'    => 'give_payment_note'
 					)
-				) );
+				));
 				break;
 
 			case 'donor':
-				$comment_args['meta_query'] = ! empty( $comment_args['meta_query'] )
+				$comment_args['meta_query'] = ! empty($comment_args['meta_query'])
 					? $comment_args['meta_query']
 					: array(
 						array(
@@ -281,19 +281,19 @@ 
 						)
 					);
 
-				$comments = get_comments( wp_parse_args(
+				$comments = get_comments(wp_parse_args(
 					$comment_args,
 					array(
 						'order'  => 'ASC',
 						'search' => $search,
 						'type'   => 'give_donor_note'
 					)
-				) );
+				));
 				break;
 		}
 
-		add_action( 'pre_get_comments', array( self::$instance, 'hide_comments' ), 10, 1 );
-		add_filter( 'comments_clauses', array( self::$instance, 'hide_comments_pre_wp_41' ), 10, 1 );
+		add_action('pre_get_comments', array(self::$instance, 'hide_comments'), 10, 1);
+		add_filter('comments_clauses', array(self::$instance, 'hide_comments_pre_wp_41'), 10, 1);
 
 		return $comments;
 	}
@@ -309,14 +309,14 @@ 
 	 *
 	 * @return void
 	 */
-	public function hide_comments( $query ) {
-		if ( version_compare( floatval( get_bloginfo( 'version' ) ), '4.1', '>=' ) ) {
-			$types = isset( $query->query_vars['type__not_in'] ) ? $query->query_vars['type__not_in'] : array();
-			if ( ! is_array( $types ) ) {
-				$types = array( $types );
+	public function hide_comments($query) {
+		if (version_compare(floatval(get_bloginfo('version')), '4.1', '>=')) {
+			$types = isset($query->query_vars['type__not_in']) ? $query->query_vars['type__not_in'] : array();
+			if ( ! is_array($types)) {
+				$types = array($types);
 			}
 
-			$types = array_filter( array_merge( $types, $this->comment_types ) );
+			$types = array_filter(array_merge($types, $this->comment_types));
 
 			$query->query_vars['type__not_in'] = $types;
 		}
@@ -332,9 +332,9 @@ 
 	 *
 	 * @return array $clauses Updated comment clauses.
 	 */
-	public function hide_comments_pre_wp_41( $clauses ) {
-		if ( version_compare( floatval( get_bloginfo( 'version' ) ), '4.1', '<' ) ) {
-			foreach ( $this->comment_types as $comment_type ) {
+	public function hide_comments_pre_wp_41($clauses) {
+		if (version_compare(floatval(get_bloginfo('version')), '4.1', '<')) {
+			foreach ($this->comment_types as $comment_type) {
 				$clauses['where'] .= " AND comment_type != \"{$comment_type}\"";
 			}
 		}
@@ -352,11 +352,11 @@ 
 	 *
 	 * @return string $where
 	 */
-	public function hide_comments_from_feeds( $where ) {
+	public function hide_comments_from_feeds($where) {
 		global $wpdb;
 
-		foreach ( $this->comment_types as $comment_type ) {
-			$where .= $wpdb->prepare( ' AND comment_type!=%s', $comment_type );
+		foreach ($this->comment_types as $comment_type) {
+			$where .= $wpdb->prepare(' AND comment_type!=%s', $comment_type);
 		}
 
 		return $where;
@@ -373,30 +373,30 @@ 
 	 *
 	 * @return array|object Array of comment counts.
 	 */
-	public function remove_comments_from_comment_counts( $stats, $post_id ) {
+	public function remove_comments_from_comment_counts($stats, $post_id) {
 		global $wpdb;
 
 		$post_id = (int) $post_id;
 
-		if ( apply_filters( 'give_count_payment_notes_in_comments', false ) ) {
+		if (apply_filters('give_count_payment_notes_in_comments', false)) {
 			return $stats;
 		}
 
-		$stats = Give_Cache::get_group( "comments-{$post_id}", 'counts' );
+		$stats = Give_Cache::get_group("comments-{$post_id}", 'counts');
 
 		// Return result from cache.
-		if ( ! is_null( $stats ) ) {
+		if ( ! is_null($stats)) {
 			return $stats;
 		}
 
 		$where = 'WHERE';
 
-		foreach ( $this->comment_types as $index => $comment_type ) {
-			$where .= ( $index ? ' AND ' : ' ' ) . "comment_type != \"{$comment_type}\"";
+		foreach ($this->comment_types as $index => $comment_type) {
+			$where .= ($index ? ' AND ' : ' ')."comment_type != \"{$comment_type}\"";
 		}
 
-		if ( $post_id > 0 ) {
-			$where .= $wpdb->prepare( ' AND comment_post_ID = %d', $post_id );
+		if ($post_id > 0) {
+			$where .= $wpdb->prepare(' AND comment_post_ID = %d', $post_id);
 		}
 
 		$count = $wpdb->get_results(
@@ -417,26 +417,26 @@ 
 			'post-trashed' => 'post-trashed',
 		);
 
-		foreach ( (array) $count as $row ) {
+		foreach ((array) $count as $row) {
 			// Don't count post-trashed toward totals.
-			if ( 'post-trashed' != $row['comment_approved'] && 'trash' != $row['comment_approved'] ) {
+			if ('post-trashed' != $row['comment_approved'] && 'trash' != $row['comment_approved']) {
 				$total += $row['num_comments'];
 			}
-			if ( isset( $approved[ $row['comment_approved'] ] ) ) {
-				$stats[ $approved[ $row['comment_approved'] ] ] = $row['num_comments'];
+			if (isset($approved[$row['comment_approved']])) {
+				$stats[$approved[$row['comment_approved']]] = $row['num_comments'];
 			}
 		}
 
 		$stats['total_comments'] = $stats['all'] = $total;
-		foreach ( $approved as $key ) {
-			if ( empty( $stats[ $key ] ) ) {
-				$stats[ $key ] = 0;
+		foreach ($approved as $key) {
+			if (empty($stats[$key])) {
+				$stats[$key] = 0;
 			}
 		}
 
 		$stats = (object) $stats;
 
-		Give_Cache::set_group( "comments-{$post_id}", $stats, 'counts' );
+		Give_Cache::set_group("comments-{$post_id}", $stats, 'counts');
 
 		return $stats;
 	}
@@ -453,12 +453,12 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function __get_comment_author( $author, $comment_id, $comment ) {
-		if ( in_array( $comment->comment_type, $this->comment_types ) ) {
-			switch ( $comment->comment_type ) {
+	public function __get_comment_author($author, $comment_id, $comment) {
+		if (in_array($comment->comment_type, $this->comment_types)) {
+			switch ($comment->comment_type) {
 				case 'give_payment_note':
-					if ( get_comment_meta( $comment_id, '_give_donor_id', true ) ) {
-						$author = give_get_donor_name_by( $comment->comment_post_ID );
+					if (get_comment_meta($comment_id, '_give_donor_id', true)) {
+						$author = give_get_donor_name_by($comment->comment_post_ID);
 					}
 			}
 		}
@@ -477,9 +477,9 @@ 
 	 *
 	 * @return array
 	 */
-	public static function get_comment_types( $comment_types ) {
+	public static function get_comment_types($comment_types) {
 		$_comment_types = array();
-		foreach ( $comment_types as $comment_type ) {
+		foreach ($comment_types as $comment_type) {
 			$_comment_types[] = "give_{$comment_type}_note";
 		}
 

includes/class-give-db-meta.php

Doc Comments +2 added lines, -2 removed lines

@@ -279,7 +279,7 @@ 
 	/**
 	 * Rename query clauses for new meta table
 	 *
-	 * @param $clause
+	 * @param string $clause
 	 * @param $filter
 	 *
 	 * @return mixed
@@ -515,7 +515,7 @@ 
 	 * @since  2.0.4
 	 * @access public
 	 *
-	 * @return string
+	 * @return boolean
 	 */
 	public function get_meta_type() {
 		return $this->meta_type;

Spacing +89 added lines, -89 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -77,40 +77,40 @@ 
 		parent::__construct();
 
 		// Bailout.
-		if ( empty( $this->supports ) || ! $this->is_custom_meta_table_active() ) {
+		if (empty($this->supports) || ! $this->is_custom_meta_table_active()) {
 			return;
 		}
 
-		if ( in_array( 'add_post_metadata', $this->supports ) ) {
-			add_filter( 'add_post_metadata', array( $this, '__add_meta' ), 0, 5 );
+		if (in_array('add_post_metadata', $this->supports)) {
+			add_filter('add_post_metadata', array($this, '__add_meta'), 0, 5);
 		}
 
-		if ( in_array( 'get_post_metadata', $this->supports ) ) {
-			add_filter( 'get_post_metadata', array( $this, '__get_meta' ), 10, 4 );
+		if (in_array('get_post_metadata', $this->supports)) {
+			add_filter('get_post_metadata', array($this, '__get_meta'), 10, 4);
 		}
 
-		if ( in_array( 'update_post_metadata', $this->supports ) ) {
-			add_filter( 'update_post_metadata', array( $this, '__update_meta' ), 0, 5 );
+		if (in_array('update_post_metadata', $this->supports)) {
+			add_filter('update_post_metadata', array($this, '__update_meta'), 0, 5);
 		}
 
-		if ( in_array( 'delete_post_metadata', $this->supports ) ) {
-			add_filter( 'delete_post_metadata', array( $this, '__delete_meta' ), 0, 5 );
+		if (in_array('delete_post_metadata', $this->supports)) {
+			add_filter('delete_post_metadata', array($this, '__delete_meta'), 0, 5);
 		}
 
-		if ( in_array( 'posts_where', $this->supports ) ) {
-			add_filter( 'posts_where', array( $this, '__rename_meta_table_name_in_query' ), 99999, 2 );
+		if (in_array('posts_where', $this->supports)) {
+			add_filter('posts_where', array($this, '__rename_meta_table_name_in_query'), 99999, 2);
 		}
 
-		if ( in_array( 'posts_join', $this->supports ) ) {
-			add_filter( 'posts_join', array( $this, '__rename_meta_table_name_in_query' ), 99999, 2 );
+		if (in_array('posts_join', $this->supports)) {
+			add_filter('posts_join', array($this, '__rename_meta_table_name_in_query'), 99999, 2);
 		}
 
-		if ( in_array( 'posts_groupby', $this->supports ) ) {
-			add_filter( 'posts_groupby', array( $this, '__rename_meta_table_name_in_query' ), 99999, 2 );
+		if (in_array('posts_groupby', $this->supports)) {
+			add_filter('posts_groupby', array($this, '__rename_meta_table_name_in_query'), 99999, 2);
 		}
 
-		if ( in_array( 'posts_orderby', $this->supports ) ) {
-			add_filter( 'posts_orderby', array( $this, '__rename_meta_table_name_in_query' ), 99999, 2 );
+		if (in_array('posts_orderby', $this->supports)) {
+			add_filter('posts_orderby', array($this, '__rename_meta_table_name_in_query'), 99999, 2);
 		}
 	}
 
@@ -128,16 +128,16 @@ 
 	 * @return  mixed                 Will be an array if $single is false. Will be value of meta data field if $single
 	 *                                is true.
 	 */
-	public function get_meta( $id = 0, $meta_key = '', $single = false ) {
-		$id = $this->sanitize_id( $id );
+	public function get_meta($id = 0, $meta_key = '', $single = false) {
+		$id = $this->sanitize_id($id);
 
 		// Bailout.
-		if ( ! $this->is_valid_post_type( $id ) ) {
+		if ( ! $this->is_valid_post_type($id)) {
 			return $this->check;
 		}
 
-		if ( $this->raw_result ) {
-			if ( ! ( $value = get_metadata( $this->meta_type, $id, $meta_key, false ) ) ) {
+		if ($this->raw_result) {
+			if ( ! ($value = get_metadata($this->meta_type, $id, $meta_key, false))) {
 				$value = '';
 			}
 
@@ -145,7 +145,7 @@ 
 			$this->raw_result = false;
 
 		} else {
-			$value = get_metadata( $this->meta_type, $id, $meta_key, $single );
+			$value = get_metadata($this->meta_type, $id, $meta_key, $single);
 		}
 
 		return $value;
@@ -167,18 +167,18 @@ 
 	 *
 	 * @return  int|bool                  False for failure. True for success.
 	 */
-	public function add_meta( $id = 0, $meta_key = '', $meta_value, $unique = false ) {
-		$id = $this->sanitize_id( $id );
+	public function add_meta($id = 0, $meta_key = '', $meta_value, $unique = false) {
+		$id = $this->sanitize_id($id);
 
 		// Bailout.
-		if ( ! $this->is_valid_post_type( $id ) ) {
+		if ( ! $this->is_valid_post_type($id)) {
 			return $this->check;
 		}
 
-		$meta_id = add_metadata( $this->meta_type, $id, $meta_key, $meta_value, $unique );
+		$meta_id = add_metadata($this->meta_type, $id, $meta_key, $meta_value, $unique);
 
-		if ( $meta_id ) {
-			$this->delete_cache( $id );
+		if ($meta_id) {
+			$this->delete_cache($id);
 		}
 
 		return $meta_id;
@@ -204,18 +204,18 @@ 
 	 *
 	 * @return  int|bool                  False on failure, true if success.
 	 */
-	public function update_meta( $id = 0, $meta_key = '', $meta_value, $prev_value = '' ) {
-		$id = $this->sanitize_id( $id );
+	public function update_meta($id = 0, $meta_key = '', $meta_value, $prev_value = '') {
+		$id = $this->sanitize_id($id);
 
 		// Bailout.
-		if ( ! $this->is_valid_post_type( $id ) ) {
+		if ( ! $this->is_valid_post_type($id)) {
 			return $this->check;
 		}
 
-		$meta_id = update_metadata( $this->meta_type, $id, $meta_key, $meta_value, $prev_value );
+		$meta_id = update_metadata($this->meta_type, $id, $meta_key, $meta_value, $prev_value);
 
-		if ( $meta_id ) {
-			$this->delete_cache( $id );
+		if ($meta_id) {
+			$this->delete_cache($id);
 		}
 
 		return $meta_id;
@@ -238,18 +238,18 @@ 
 	 *
 	 * @return  bool                  False for failure. True for success.
 	 */
-	public function delete_meta( $id = 0, $meta_key = '', $meta_value = '', $delete_all = '' ) {
-		$id = $this->sanitize_id( $id );
+	public function delete_meta($id = 0, $meta_key = '', $meta_value = '', $delete_all = '') {
+		$id = $this->sanitize_id($id);
 
 		// Bailout.
-		if ( ! $this->is_valid_post_type( $id ) ) {
+		if ( ! $this->is_valid_post_type($id)) {
 			return $this->check;
 		}
 
-		$is_meta_deleted = delete_metadata( $this->meta_type, $id, $meta_key, $meta_value, $delete_all );
+		$is_meta_deleted = delete_metadata($this->meta_type, $id, $meta_key, $meta_value, $delete_all);
 
-		if ( $is_meta_deleted ) {
-			$this->delete_cache( $id );
+		if ($is_meta_deleted) {
+			$this->delete_cache($id);
 		}
 
 		return $is_meta_deleted;
@@ -266,10 +266,10 @@ 
 	 *
 	 * @return string
 	 */
-	public function __rename_meta_table_name_in_query( $clause, $wp_query ) {
+	public function __rename_meta_table_name_in_query($clause, $wp_query) {
 		// Add new table to sql query.
-		if ( $this->is_post_type_query( $wp_query ) && ! empty( $wp_query->meta_query->queries ) ) {
-			$clause = $this->__rename_meta_table_name( $clause, current_filter() );
+		if ($this->is_post_type_query($wp_query) && ! empty($wp_query->meta_query->queries)) {
+			$clause = $this->__rename_meta_table_name($clause, current_filter());
 		}
 
 		return $clause;
@@ -284,42 +284,42 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function __rename_meta_table_name( $clause, $filter ) {
+	public function __rename_meta_table_name($clause, $filter) {
 		global $wpdb;
 
-		$clause = str_replace( "{$wpdb->postmeta}.post_id", "{$this->table_name}.{$this->meta_type}_id", $clause );
-		$clause = str_replace( $wpdb->postmeta, $this->table_name, $clause );
+		$clause = str_replace("{$wpdb->postmeta}.post_id", "{$this->table_name}.{$this->meta_type}_id", $clause);
+		$clause = str_replace($wpdb->postmeta, $this->table_name, $clause);
 
-		switch ( $filter ) {
+		switch ($filter) {
 			case 'posts_join':
-				$joins = array( 'INNER JOIN', 'LEFT JOIN' );
+				$joins = array('INNER JOIN', 'LEFT JOIN');
 
-				foreach ( $joins as $join ) {
-					if ( false !== strpos( $clause, $join ) ) {
-						$clause = explode( $join, $clause );
+				foreach ($joins as $join) {
+					if (false !== strpos($clause, $join)) {
+						$clause = explode($join, $clause);
 
-						foreach ( $clause as $key => $clause_part ) {
-							if ( empty( $clause_part ) ) {
+						foreach ($clause as $key => $clause_part) {
+							if (empty($clause_part)) {
 								continue;
 							}
 
-							preg_match( '/' . $wpdb->prefix . 'give_' . $this->meta_type . 'meta AS (.*) ON/', $clause_part, $alias_table_name );
+							preg_match('/'.$wpdb->prefix.'give_'.$this->meta_type.'meta AS (.*) ON/', $clause_part, $alias_table_name);
 
-							if ( isset( $alias_table_name[1] ) ) {
-								$clause[ $key ] = str_replace( "{$alias_table_name[1]}.post_id", "{$alias_table_name[1]}.{$this->meta_type}_id", $clause_part );
+							if (isset($alias_table_name[1])) {
+								$clause[$key] = str_replace("{$alias_table_name[1]}.post_id", "{$alias_table_name[1]}.{$this->meta_type}_id", $clause_part);
 							}
 						}
 
-						$clause = implode( "{$join} ", $clause );
+						$clause = implode("{$join} ", $clause);
 					}
 				}
 				break;
 
 			case 'posts_where':
-				$clause = str_replace( array( 'mt2.post_id', 'mt1.post_id' ), array(
+				$clause = str_replace(array('mt2.post_id', 'mt1.post_id'), array(
 					"mt2.{$this->meta_type}_id",
 					"mt1.{$this->meta_type}_id"
-				), $clause );
+				), $clause);
 				break;
 		}
 
@@ -337,19 +337,19 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function is_post_type_query( $wp_query ) {
+	protected function is_post_type_query($wp_query) {
 		$status = false;
 
 		// Check if it is payment query.
-		if ( ! empty( $wp_query->query['post_type'] ) ) {
+		if ( ! empty($wp_query->query['post_type'])) {
 			if (
-				is_string( $wp_query->query['post_type'] ) &&
+				is_string($wp_query->query['post_type']) &&
 				$this->post_type === $wp_query->query['post_type']
 			) {
 				$status = true;
 			} elseif (
-				is_array( $wp_query->query['post_type'] ) &&
-				in_array( $this->post_type, $wp_query->query['post_type'] )
+				is_array($wp_query->query['post_type']) &&
+				in_array($this->post_type, $wp_query->query['post_type'])
 			) {
 				$status = true;
 			}
@@ -368,8 +368,8 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function is_valid_post_type( $ID ) {
-		return $ID && ( $this->post_type === get_post_type( $ID ) );
+	protected function is_valid_post_type($ID) {
+		return $ID && ($this->post_type === get_post_type($ID));
 	}
 
 	/**
@@ -395,8 +395,8 @@ 
 	 *
 	 * @return void
 	 */
-	private function delete_cache( $id, $meta_type = '' ) {
-		$meta_type = empty( $meta_type ) ? $this->meta_type : $meta_type;
+	private function delete_cache($id, $meta_type = '') {
+		$meta_type = empty($meta_type) ? $this->meta_type : $meta_type;
 
 		$group = array(
 			'payment'  => 'give-donations', // Backward compatibility
@@ -405,8 +405,8 @@ 
 			'customer' => 'give-donors', // Backward compatibility for pre upgrade in 2.0
 		);
 
-		if ( array_key_exists( $meta_type, $group ) ) {
-			Give_Cache::delete_group( $id, $group[ $meta_type ] );
+		if (array_key_exists($meta_type, $group)) {
+			Give_Cache::delete_group($id, $group[$meta_type]);
 		}
 	}
 
@@ -421,8 +421,8 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function __call( $name, $arguments ) {
-		switch ( $name ) {
+	public function __call($name, $arguments) {
+		switch ($name) {
 			case '__add_meta':
 				$this->check = $arguments[0];
 				$id          = $arguments[1];
@@ -431,11 +431,11 @@ 
 				$unique      = $arguments[4];
 
 				// Bailout.
-				if ( ! $this->is_valid_post_type( $id ) ) {
+				if ( ! $this->is_valid_post_type($id)) {
 					return $this->check;
 				}
 
-				return $this->add_meta( $id, $meta_key, $meta_value, $unique );
+				return $this->add_meta($id, $meta_key, $meta_value, $unique);
 
 			case '__get_meta':
 				$this->check = $arguments[0];
@@ -444,13 +444,13 @@ 
 				$single      = $arguments[3];
 
 				// Bailout.
-				if ( ! $this->is_valid_post_type( $id ) ) {
+				if ( ! $this->is_valid_post_type($id)) {
 					return $this->check;
 				}
 
 				$this->raw_result = true;
 
-				return $this->get_meta( $id, $meta_key, $single );
+				return $this->get_meta($id, $meta_key, $single);
 
 			case '__update_meta':
 				$this->check = $arguments[0];
@@ -459,11 +459,11 @@ 
 				$meta_value  = $arguments[3];
 
 				// Bailout.
-				if ( ! $this->is_valid_post_type( $id ) ) {
+				if ( ! $this->is_valid_post_type($id)) {
 					return $this->check;
 				}
 
-				return $this->update_meta( $id, $meta_key, $meta_value );
+				return $this->update_meta($id, $meta_key, $meta_value);
 
 			case '__delete_meta':
 				$this->check = $arguments[0];
@@ -473,11 +473,11 @@ 
 				$delete_all  = $arguments[3];
 
 				// Bailout.
-				if ( ! $this->is_valid_post_type( $id ) ) {
+				if ( ! $this->is_valid_post_type($id)) {
 					return $this->check;
 				}
 
-				return $this->delete_meta( $id, $meta_key, $meta_value, $delete_all );
+				return $this->delete_meta($id, $meta_key, $meta_value, $delete_all);
 		}
 	}
 
@@ -502,10 +502,10 @@ 
 			KEY meta_key (meta_key({$this->min_index_length}))
 			) {$charset_collate};";
 
-		require_once( ABSPATH . 'wp-admin/includes/upgrade.php' );
-		dbDelta( $sql );
+		require_once(ABSPATH.'wp-admin/includes/upgrade.php');
+		dbDelta($sql);
 
-		update_option( $this->table_name . '_db_version', $this->version, false );
+		update_option($this->table_name.'_db_version', $this->version, false);
 	}
 
 
@@ -531,12 +531,12 @@ 
 	 *
 	 * @return  bool  False for failure. True for success.
 	 */
-	public function delete_all_meta( $id = 0 ) {
+	public function delete_all_meta($id = 0) {
 		global $wpdb;
-		$status = $wpdb->delete( $this->table_name, array( "{$this->meta_type}_id" => $id ), array( '%d' ) );
+		$status = $wpdb->delete($this->table_name, array("{$this->meta_type}_id" => $id), array('%d'));
 
-		if ( $status ) {
-			$this->delete_cache( $id, $this->meta_type );
+		if ($status) {
+			$this->delete_cache($id, $this->meta_type);
 		}
 
 		return $status;

includes/class-give-donor.php

Doc Comments +1 added lines, -1 removed lines

@@ -1052,7 +1052,7 @@
 	 * @access public
 	 *
 	 * @param  string $meta_key   Metadata name. Default is empty.
-	 * @param  mixed  $meta_value Optional. Metadata value. Default is empty.
+	 * @param  string  $meta_value Optional. Metadata value. Default is empty.
 	 *
 	 * @return bool               False for failure. True for success.
 	 */

Spacing +273 added lines, -273 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -149,29 +149,29 @@ 
 	 * @param int|bool $_id_or_email
 	 * @param bool     $by_user_id
 	 */
-	public function __construct( $_id_or_email = false, $by_user_id = false ) {
+	public function __construct($_id_or_email = false, $by_user_id = false) {
 
 		$this->db = Give()->donors;
 
-		if ( false === $_id_or_email || ( is_numeric( $_id_or_email ) && (int) $_id_or_email !== absint( $_id_or_email ) ) ) {
+		if (false === $_id_or_email || (is_numeric($_id_or_email) && (int) $_id_or_email !== absint($_id_or_email))) {
 			return false;
 		}
 
-		$by_user_id = is_bool( $by_user_id ) ? $by_user_id : false;
+		$by_user_id = is_bool($by_user_id) ? $by_user_id : false;
 
-		if ( is_numeric( $_id_or_email ) ) {
+		if (is_numeric($_id_or_email)) {
 			$field = $by_user_id ? 'user_id' : 'id';
 		} else {
 			$field = 'email';
 		}
 
-		$donor = $this->db->get_donor_by( $field, $_id_or_email );
+		$donor = $this->db->get_donor_by($field, $_id_or_email);
 
-		if ( empty( $donor ) || ! is_object( $donor ) ) {
+		if (empty($donor) || ! is_object($donor)) {
 			return false;
 		}
 
-		$this->setup_donor( $donor );
+		$this->setup_donor($donor);
 
 	}
 
@@ -187,19 +187,19 @@ 
 	 *
 	 * @return bool             If the setup was successful or not.
 	 */
-	private function setup_donor( $donor ) {
+	private function setup_donor($donor) {
 
-		if ( ! is_object( $donor ) ) {
+		if ( ! is_object($donor)) {
 			return false;
 		}
 
 		// Get cached donors.
-		$donor_vars = Give_Cache::get_group( $donor->id, 'give-donors' );
+		$donor_vars = Give_Cache::get_group($donor->id, 'give-donors');
 
-		if ( is_null( $donor_vars ) ) {
-			foreach ( $donor as $key => $value ) {
+		if (is_null($donor_vars)) {
+			foreach ($donor as $key => $value) {
 
-				switch ( $key ) {
+				switch ($key) {
 
 					case 'notes':
 						$this->$key = $this->get_notes();
@@ -213,20 +213,20 @@ 
 			}
 
 			// Get donor's all email including primary email.
-			$this->emails = (array) $this->get_meta( 'additional_email', false );
-			$this->emails = array( 'primary' => $this->email ) + $this->emails;
+			$this->emails = (array) $this->get_meta('additional_email', false);
+			$this->emails = array('primary' => $this->email) + $this->emails;
 
 			$this->setup_address();
 
-			Give_Cache::set_group( $donor->id, get_object_vars( $this ), 'give-donors' );
+			Give_Cache::set_group($donor->id, get_object_vars($this), 'give-donors');
 		} else {
-			foreach ( $donor_vars as $donor_var => $value ) {
+			foreach ($donor_vars as $donor_var => $value) {
 				$this->$donor_var = $value;
 			}
 		}
 
 		// Donor ID and email are the only things that are necessary, make sure they exist.
-		if ( ! empty( $this->id ) && ! empty( $this->email ) ) {
+		if ( ! empty($this->id) && ! empty($this->email)) {
 			return true;
 		}
 
@@ -245,25 +245,25 @@ 
 		global $wpdb;
 		$meta_type = Give()->donor_meta->meta_type;
 
-		$addresses = $wpdb->get_results( $wpdb->prepare( "
+		$addresses = $wpdb->get_results($wpdb->prepare("
 				SELECT meta_key, meta_value FROM {$wpdb->donormeta}
 				WHERE meta_key
 				LIKE '%%%s%%'
 				AND {$meta_type}_id=%d
-				", 'give_donor_address', $this->id ), ARRAY_N );
+				", 'give_donor_address', $this->id), ARRAY_N);
 
-		if ( empty( $addresses ) ) {
+		if (empty($addresses)) {
 			return $this->address;
 		}
 
-		foreach ( $addresses as $address ) {
-			$address[0] = str_replace( '_give_donor_address_', '', $address[0] );
-			$address[0] = explode( '_', $address[0] );
+		foreach ($addresses as $address) {
+			$address[0] = str_replace('_give_donor_address_', '', $address[0]);
+			$address[0] = explode('_', $address[0]);
 
-			if ( 3 === count( $address[0] ) ) {
-				$this->address[ $address[0][0] ][ $address[0][2] ][ $address[0][1] ] = $address[1];
+			if (3 === count($address[0])) {
+				$this->address[$address[0][0]][$address[0][2]][$address[0][1]] = $address[1];
 			} else {
-				$this->address[ $address[0][0] ][ $address[0][1] ] = $address[1];
+				$this->address[$address[0][0]][$address[0][1]] = $address[1];
 			}
 		}
 	}
@@ -279,7 +279,7 @@ 
 	 *
 	 * @return array The donor's address, if any
 	 */
-	public function get_donor_address( $args = array() ) {
+	public function get_donor_address($args = array()) {
 		$args = wp_parse_args(
 			$args,
 			array(
@@ -297,24 +297,24 @@ 
 		);
 
 		// Backward compatibility.
-		if ( ! give_has_upgrade_completed( 'v20_upgrades_user_address' ) ) {
+		if ( ! give_has_upgrade_completed('v20_upgrades_user_address')) {
 
 			// Backward compatibility for user id param.
-			return wp_parse_args( (array) get_user_meta( $this->user_id, '_give_user_address', true ), $default_address );
+			return wp_parse_args((array) get_user_meta($this->user_id, '_give_user_address', true), $default_address);
 
 		}
 
-		if ( ! $this->id || empty( $this->address ) || ! array_key_exists( $args['address_type'], $this->address ) ) {
+		if ( ! $this->id || empty($this->address) || ! array_key_exists($args['address_type'], $this->address)) {
 			return $default_address;
 		}
 
-		switch ( true ) {
-			case is_string( end( $this->address[ $args['address_type'] ] ) ):
-				$address = wp_parse_args( $this->address[ $args['address_type'] ], $default_address );
+		switch (true) {
+			case is_string(end($this->address[$args['address_type']])):
+				$address = wp_parse_args($this->address[$args['address_type']], $default_address);
 				break;
 
-			case is_array( end( $this->address[ $args['address_type'] ] ) ):
-				$address = wp_parse_args( array_shift( $this->address[ $args['address_type'] ] ), $default_address );
+			case is_array(end($this->address[$args['address_type']])):
+				$address = wp_parse_args(array_shift($this->address[$args['address_type']]), $default_address);
 				break;
 		}
 
@@ -331,16 +331,16 @@ 
 	 *
 	 * @return mixed|\WP_Error
 	 */
-	public function __get( $key ) {
+	public function __get($key) {
 
-		if ( method_exists( $this, 'get_' . $key ) ) {
+		if (method_exists($this, 'get_'.$key)) {
 
-			return call_user_func( array( $this, 'get_' . $key ) );
+			return call_user_func(array($this, 'get_'.$key));
 
 		} else {
 
 			/* translators: %s: property key */
-			return new WP_Error( 'give-donor-invalid-property', sprintf( esc_html__( 'Can\'t get property %s.', 'give' ), $key ) );
+			return new WP_Error('give-donor-invalid-property', sprintf(esc_html__('Can\'t get property %s.', 'give'), $key));
 
 		}
 
@@ -356,9 +356,9 @@ 
 	 *
 	 * @return bool|int    False if not a valid creation, donor ID if user is found or valid creation.
 	 */
-	public function create( $data = array() ) {
+	public function create($data = array()) {
 
-		if ( $this->id != 0 || empty( $data ) ) {
+		if ($this->id != 0 || empty($data)) {
 			return false;
 		}
 
@@ -366,15 +366,15 @@ 
 			'payment_ids' => '',
 		);
 
-		$args = wp_parse_args( $data, $defaults );
-		$args = $this->sanitize_columns( $args );
+		$args = wp_parse_args($data, $defaults);
+		$args = $this->sanitize_columns($args);
 
-		if ( empty( $args['email'] ) || ! is_email( $args['email'] ) ) {
+		if (empty($args['email']) || ! is_email($args['email'])) {
 			return false;
 		}
 
-		if ( ! empty( $args['payment_ids'] ) && is_array( $args['payment_ids'] ) ) {
-			$args['payment_ids'] = implode( ',', array_unique( array_values( $args['payment_ids'] ) ) );
+		if ( ! empty($args['payment_ids']) && is_array($args['payment_ids'])) {
+			$args['payment_ids'] = implode(',', array_unique(array_values($args['payment_ids'])));
 		}
 
 		/**
@@ -384,18 +384,18 @@ 
 		 *
 		 * @param array $args Donor attributes.
 		 */
-		do_action( 'give_donor_pre_create', $args );
+		do_action('give_donor_pre_create', $args);
 
 		$created = false;
 
 		// The DB class 'add' implies an update if the donor being asked to be created already exists
-		if ( $this->db->add( $data ) ) {
+		if ($this->db->add($data)) {
 
 			// We've successfully added/updated the donor, reset the class vars with the new data
-			$donor = $this->db->get_donor_by( 'email', $args['email'] );
+			$donor = $this->db->get_donor_by('email', $args['email']);
 
 			// Setup the donor data with the values from DB
-			$this->setup_donor( $donor );
+			$this->setup_donor($donor);
 
 			$created = $this->id;
 		}
@@ -408,7 +408,7 @@ 
 		 * @param bool|int $created False if not a valid creation, donor ID if user is found or valid creation.
 		 * @param array    $args    Customer attributes.
 		 */
-		do_action( 'give_donor_post_create', $created, $args );
+		do_action('give_donor_post_create', $created, $args);
 
 		return $created;
 
@@ -424,13 +424,13 @@ 
 	 *
 	 * @return bool        If the update was successful or not.
 	 */
-	public function update( $data = array() ) {
+	public function update($data = array()) {
 
-		if ( empty( $data ) ) {
+		if (empty($data)) {
 			return false;
 		}
 
-		$data = $this->sanitize_columns( $data );
+		$data = $this->sanitize_columns($data);
 
 		/**
 		 * Fires before updating donors.
@@ -440,15 +440,15 @@ 
 		 * @param int   $donor_id Donor id.
 		 * @param array $data     Donor attributes.
 		 */
-		do_action( 'give_donor_pre_update', $this->id, $data );
+		do_action('give_donor_pre_update', $this->id, $data);
 
 		$updated = false;
 
-		if ( $this->db->update( $this->id, $data ) ) {
+		if ($this->db->update($this->id, $data)) {
 
-			$donor = $this->db->get_donor_by( 'id', $this->id );
+			$donor = $this->db->get_donor_by('id', $this->id);
 
-			$this->setup_donor( $donor );
+			$this->setup_donor($donor);
 
 			$updated = true;
 		}
@@ -462,7 +462,7 @@ 
 		 * @param int   $donor_id Donor id.
 		 * @param array $data     Donor attributes.
 		 */
-		do_action( 'give_donor_post_update', $updated, $this->id, $data );
+		do_action('give_donor_post_update', $updated, $this->id, $data);
 
 		return $updated;
 	}
@@ -480,27 +480,27 @@ 
 	 *
 	 * @return bool            If the attachment was successfully.
 	 */
-	public function attach_payment( $payment_id = 0, $update_stats = true ) {
+	public function attach_payment($payment_id = 0, $update_stats = true) {
 
-		if ( empty( $payment_id ) ) {
+		if (empty($payment_id)) {
 			return false;
 		}
 
-		if ( empty( $this->payment_ids ) ) {
+		if (empty($this->payment_ids)) {
 
 			$new_payment_ids = $payment_id;
 
 		} else {
 
-			$payment_ids = array_map( 'absint', explode( ',', $this->payment_ids ) );
+			$payment_ids = array_map('absint', explode(',', $this->payment_ids));
 
-			if ( in_array( $payment_id, $payment_ids ) ) {
+			if (in_array($payment_id, $payment_ids)) {
 				$update_stats = false;
 			}
 
 			$payment_ids[] = $payment_id;
 
-			$new_payment_ids = implode( ',', array_unique( array_values( $payment_ids ) ) );
+			$new_payment_ids = implode(',', array_unique(array_values($payment_ids)));
 
 		}
 
@@ -512,20 +512,20 @@ 
 		 * @param int $payment_id Payment id.
 		 * @param int $donor_id   Donor id.
 		 */
-		do_action( 'give_donor_pre_attach_payment', $payment_id, $this->id );
+		do_action('give_donor_pre_attach_payment', $payment_id, $this->id);
 
-		$payment_added = $this->update( array( 'payment_ids' => $new_payment_ids ) );
+		$payment_added = $this->update(array('payment_ids' => $new_payment_ids));
 
-		if ( $payment_added ) {
+		if ($payment_added) {
 
 			$this->payment_ids = $new_payment_ids;
 
 			// We added this payment successfully, increment the stats
-			if ( $update_stats ) {
-				$payment_amount = give_donation_amount( $payment_id, array( 'type' => 'stats' ) );
+			if ($update_stats) {
+				$payment_amount = give_donation_amount($payment_id, array('type' => 'stats'));
 
-				if ( ! empty( $payment_amount ) ) {
-					$this->increase_value( $payment_amount );
+				if ( ! empty($payment_amount)) {
+					$this->increase_value($payment_amount);
 				}
 
 				$this->increase_purchase_count();
@@ -541,7 +541,7 @@ 
 		 * @param int  $payment_id    Payment id.
 		 * @param int  $donor_id      Donor id.
 		 */
-		do_action( 'give_donor_post_attach_payment', $payment_added, $payment_id, $this->id );
+		do_action('give_donor_post_attach_payment', $payment_added, $payment_id, $this->id);
 
 		return $payment_added;
 	}
@@ -559,33 +559,33 @@ 
 	 *
 	 * @return boolean               If the removal was successful.
 	 */
-	public function remove_payment( $payment_id = 0, $update_stats = true ) {
+	public function remove_payment($payment_id = 0, $update_stats = true) {
 
-		if ( empty( $payment_id ) ) {
+		if (empty($payment_id)) {
 			return false;
 		}
 
-		$payment = new Give_Payment( $payment_id );
+		$payment = new Give_Payment($payment_id);
 
-		if ( 'publish' !== $payment->status && 'revoked' !== $payment->status ) {
+		if ('publish' !== $payment->status && 'revoked' !== $payment->status) {
 			$update_stats = false;
 		}
 
 		$new_payment_ids = '';
 
-		if ( ! empty( $this->payment_ids ) ) {
+		if ( ! empty($this->payment_ids)) {
 
-			$payment_ids = array_map( 'absint', explode( ',', $this->payment_ids ) );
+			$payment_ids = array_map('absint', explode(',', $this->payment_ids));
 
-			$pos = array_search( $payment_id, $payment_ids );
-			if ( false === $pos ) {
+			$pos = array_search($payment_id, $payment_ids);
+			if (false === $pos) {
 				return false;
 			}
 
-			unset( $payment_ids[ $pos ] );
-			$payment_ids = array_filter( $payment_ids );
+			unset($payment_ids[$pos]);
+			$payment_ids = array_filter($payment_ids);
 
-			$new_payment_ids = implode( ',', array_unique( array_values( $payment_ids ) ) );
+			$new_payment_ids = implode(',', array_unique(array_values($payment_ids)));
 
 		}
 
@@ -597,20 +597,20 @@ 
 		 * @param int $payment_id Payment id.
 		 * @param int $donor_id   Customer id.
 		 */
-		do_action( 'give_donor_pre_remove_payment', $payment_id, $this->id );
+		do_action('give_donor_pre_remove_payment', $payment_id, $this->id);
 
-		$payment_removed = $this->update( array( 'payment_ids' => $new_payment_ids ) );
+		$payment_removed = $this->update(array('payment_ids' => $new_payment_ids));
 
-		if ( $payment_removed ) {
+		if ($payment_removed) {
 
 			$this->payment_ids = $new_payment_ids;
 
-			if ( $update_stats ) {
+			if ($update_stats) {
 				// We removed this payment successfully, decrement the stats
-				$payment_amount = give_donation_amount( $payment_id );
+				$payment_amount = give_donation_amount($payment_id);
 
-				if ( ! empty( $payment_amount ) ) {
-					$this->decrease_value( $payment_amount );
+				if ( ! empty($payment_amount)) {
+					$this->decrease_value($payment_amount);
 				}
 
 				$this->decrease_donation_count();
@@ -626,7 +626,7 @@ 
 		 * @param int  $payment_id      Payment id.
 		 * @param int  $donor_id        Donor id.
 		 */
-		do_action( 'give_donor_post_remove_payment', $payment_removed, $payment_id, $this->id );
+		do_action('give_donor_post_remove_payment', $payment_removed, $payment_id, $this->id);
 
 		return $payment_removed;
 
@@ -642,10 +642,10 @@ 
 	 *
 	 * @return int        The donation count.
 	 */
-	public function increase_purchase_count( $count = 1 ) {
+	public function increase_purchase_count($count = 1) {
 
 		// Make sure it's numeric and not negative.
-		if ( ! is_numeric( $count ) || $count != absint( $count ) ) {
+		if ( ! is_numeric($count) || $count != absint($count)) {
 			return false;
 		}
 
@@ -659,9 +659,9 @@ 
 		 * @param int $count    The number to increase by.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_pre_increase_donation_count', $count, $this->id );
+		do_action('give_donor_pre_increase_donation_count', $count, $this->id);
 
-		if ( $this->update( array( 'purchase_count' => $new_total ) ) ) {
+		if ($this->update(array('purchase_count' => $new_total))) {
 			$this->purchase_count = $new_total;
 		}
 
@@ -674,7 +674,7 @@ 
 		 * @param int $count          The number increased by.
 		 * @param int $donor_id       Donor id.
 		 */
-		do_action( 'give_donor_post_increase_donation_count', $this->purchase_count, $count, $this->id );
+		do_action('give_donor_post_increase_donation_count', $this->purchase_count, $count, $this->id);
 
 		return $this->purchase_count;
 	}
@@ -689,16 +689,16 @@ 
 	 *
 	 * @return mixed      If successful, the new count, otherwise false.
 	 */
-	public function decrease_donation_count( $count = 1 ) {
+	public function decrease_donation_count($count = 1) {
 
 		// Make sure it's numeric and not negative
-		if ( ! is_numeric( $count ) || $count != absint( $count ) ) {
+		if ( ! is_numeric($count) || $count != absint($count)) {
 			return false;
 		}
 
 		$new_total = (int) $this->purchase_count - (int) $count;
 
-		if ( $new_total < 0 ) {
+		if ($new_total < 0) {
 			$new_total = 0;
 		}
 
@@ -710,9 +710,9 @@ 
 		 * @param int $count    The number to decrease by.
 		 * @param int $donor_id Customer id.
 		 */
-		do_action( 'give_donor_pre_decrease_donation_count', $count, $this->id );
+		do_action('give_donor_pre_decrease_donation_count', $count, $this->id);
 
-		if ( $this->update( array( 'purchase_count' => $new_total ) ) ) {
+		if ($this->update(array('purchase_count' => $new_total))) {
 			$this->purchase_count = $new_total;
 		}
 
@@ -725,7 +725,7 @@ 
 		 * @param int $count          The number decreased by.
 		 * @param int $donor_id       Donor id.
 		 */
-		do_action( 'give_donor_post_decrease_donation_count', $this->purchase_count, $count, $this->id );
+		do_action('give_donor_post_decrease_donation_count', $this->purchase_count, $count, $this->id);
 
 		return $this->purchase_count;
 	}
@@ -740,9 +740,9 @@ 
 	 *
 	 * @return mixed        If successful, the new value, otherwise false.
 	 */
-	public function increase_value( $value = 0.00 ) {
+	public function increase_value($value = 0.00) {
 
-		$new_value = floatval( $this->purchase_value ) + $value;
+		$new_value = floatval($this->purchase_value) + $value;
 
 		/**
 		 * Fires before increasing donor lifetime value.
@@ -752,9 +752,9 @@ 
 		 * @param float $value    The value to increase by.
 		 * @param int   $donor_id Customer id.
 		 */
-		do_action( 'give_donor_pre_increase_value', $value, $this->id );
+		do_action('give_donor_pre_increase_value', $value, $this->id);
 
-		if ( $this->update( array( 'purchase_value' => $new_value ) ) ) {
+		if ($this->update(array('purchase_value' => $new_value))) {
 			$this->purchase_value = $new_value;
 		}
 
@@ -767,7 +767,7 @@ 
 		 * @param float $value          The value increased by.
 		 * @param int   $donor_id       Donor id.
 		 */
-		do_action( 'give_donor_post_increase_value', $this->purchase_value, $value, $this->id );
+		do_action('give_donor_post_increase_value', $this->purchase_value, $value, $this->id);
 
 		return $this->purchase_value;
 	}
@@ -782,11 +782,11 @@ 
 	 *
 	 * @return mixed        If successful, the new value, otherwise false.
 	 */
-	public function decrease_value( $value = 0.00 ) {
+	public function decrease_value($value = 0.00) {
 
-		$new_value = floatval( $this->purchase_value ) - $value;
+		$new_value = floatval($this->purchase_value) - $value;
 
-		if ( $new_value < 0 ) {
+		if ($new_value < 0) {
 			$new_value = 0.00;
 		}
 
@@ -798,9 +798,9 @@ 
 		 * @param float $value    The value to decrease by.
 		 * @param int   $donor_id Donor id.
 		 */
-		do_action( 'give_donor_pre_decrease_value', $value, $this->id );
+		do_action('give_donor_pre_decrease_value', $value, $this->id);
 
-		if ( $this->update( array( 'purchase_value' => $new_value ) ) ) {
+		if ($this->update(array('purchase_value' => $new_value))) {
 			$this->purchase_value = $new_value;
 		}
 
@@ -813,7 +813,7 @@ 
 		 * @param float $value          The value decreased by.
 		 * @param int   $donor_id       Donor id.
 		 */
-		do_action( 'give_donor_post_decrease_value', $this->purchase_value, $value, $this->id );
+		do_action('give_donor_post_decrease_value', $this->purchase_value, $value, $this->id);
 
 		return $this->purchase_value;
 	}
@@ -833,7 +833,7 @@ 
 	 *
 	 * @return mixed              If successful, the new donation stat value, otherwise false.
 	 */
-	public function update_donation_value( $curr_amount, $new_amount ) {
+	public function update_donation_value($curr_amount, $new_amount) {
 		/**
 		 * Payment total difference value can be:
 		 *  zero   (in case amount not change)
@@ -843,15 +843,15 @@ 
 		$payment_total_diff = $new_amount - $curr_amount;
 
 		// We do not need to update donation stat if donation did not change.
-		if ( ! $payment_total_diff ) {
+		if ( ! $payment_total_diff) {
 			return false;
 		}
 
-		if ( $payment_total_diff > 0 ) {
-			$this->increase_value( $payment_total_diff );
+		if ($payment_total_diff > 0) {
+			$this->increase_value($payment_total_diff);
 		} else {
 			// Pass payment total difference as +ve value to decrease amount from user lifetime stat.
-			$this->decrease_value( - $payment_total_diff );
+			$this->decrease_value( -$payment_total_diff );
 		}
 
 		return $this->purchase_value;
@@ -868,15 +868,15 @@ 
 	 *
 	 * @return array       The notes requested.
 	 */
-	public function get_notes( $length = 20, $paged = 1 ) {
+	public function get_notes($length = 20, $paged = 1) {
 
-		$length = is_numeric( $length ) ? $length : 20;
-		$offset = is_numeric( $paged ) && $paged != 1 ? ( ( absint( $paged ) - 1 ) * $length ) : 0;
+		$length = is_numeric($length) ? $length : 20;
+		$offset = is_numeric($paged) && $paged != 1 ? ((absint($paged) - 1) * $length) : 0;
 
 		$all_notes   = $this->get_raw_notes();
-		$notes_array = array_reverse( array_filter( explode( "\n\n", $all_notes ) ) );
+		$notes_array = array_reverse(array_filter(explode("\n\n", $all_notes)));
 
-		$desired_notes = array_slice( $notes_array, $offset, $length );
+		$desired_notes = array_slice($notes_array, $offset, $length);
 
 		return $desired_notes;
 
@@ -893,9 +893,9 @@ 
 	public function get_notes_count() {
 
 		$all_notes   = $this->get_raw_notes();
-		$notes_array = array_reverse( array_filter( explode( "\n\n", $all_notes ) ) );
+		$notes_array = array_reverse(array_filter(explode("\n\n", $all_notes)));
 
-		return count( $notes_array );
+		return count($notes_array);
 
 	}
 
@@ -908,7 +908,7 @@ 
 	 *
 	 * @return string|float
 	 */
-	public function get_total_donation_amount( $args = array() ) {
+	public function get_total_donation_amount($args = array()) {
 
 		/**
 		 * Filter total donation amount.
@@ -919,7 +919,7 @@ 
 		 * @param integer      $donor_id       Donor ID.
 		 * @param array        $args           Pass additional data.
 		 */
-		return apply_filters( 'give_get_total_donation_amount', $this->purchase_value, $this->id, $args );
+		return apply_filters('give_get_total_donation_amount', $this->purchase_value, $this->id, $args);
 	}
 
 	/**
@@ -932,22 +932,22 @@ 
 	 *
 	 * @return string|boolean The new note if added successfully, false otherwise.
 	 */
-	public function add_note( $note = '' ) {
+	public function add_note($note = '') {
 
-		$note = trim( $note );
-		if ( empty( $note ) ) {
+		$note = trim($note);
+		if (empty($note)) {
 			return false;
 		}
 
 		$notes = $this->get_raw_notes();
 
-		if ( empty( $notes ) ) {
+		if (empty($notes)) {
 			$notes = '';
 		}
 
-		$note_string = date_i18n( 'F j, Y H:i:s', current_time( 'timestamp' ) ) . ' - ' . $note;
-		$new_note    = apply_filters( 'give_customer_add_note_string', $note_string );
-		$notes       .= "\n\n" . $new_note;
+		$note_string = date_i18n('F j, Y H:i:s', current_time('timestamp')).' - '.$note;
+		$new_note    = apply_filters('give_customer_add_note_string', $note_string);
+		$notes .= "\n\n".$new_note;
 
 		/**
 		 * Fires before donor note is added.
@@ -957,11 +957,11 @@ 
 		 * @param string $new_note New note to add.
 		 * @param int    $donor_id Donor id.
 		 */
-		do_action( 'give_donor_pre_add_note', $new_note, $this->id );
+		do_action('give_donor_pre_add_note', $new_note, $this->id);
 
-		$updated = $this->update( array( 'notes' => $notes ) );
+		$updated = $this->update(array('notes' => $notes));
 
-		if ( $updated ) {
+		if ($updated) {
 			$this->notes = $this->get_notes();
 		}
 
@@ -974,7 +974,7 @@ 
 		 * @param string $new_note    New note added.
 		 * @param int    $donor_id    Donor id.
 		 */
-		do_action( 'give_donor_post_add_note', $this->notes, $new_note, $this->id );
+		do_action('give_donor_post_add_note', $this->notes, $new_note, $this->id);
 
 		// Return the formatted note, so we can test, as well as update any displays
 		return $new_note;
@@ -991,7 +991,7 @@ 
 	 */
 	private function get_raw_notes() {
 
-		$all_notes = $this->db->get_column( 'notes', $this->id );
+		$all_notes = $this->db->get_column('notes', $this->id);
 
 		return $all_notes;
 
@@ -1009,8 +1009,8 @@ 
 	 * @return mixed            Will be an array if $single is false. Will be value of meta data field if $single is
 	 *                          true.
 	 */
-	public function get_meta( $meta_key = '', $single = true ) {
-		return Give()->donor_meta->get_meta( $this->id, $meta_key, $single );
+	public function get_meta($meta_key = '', $single = true) {
+		return Give()->donor_meta->get_meta($this->id, $meta_key, $single);
 	}
 
 	/**
@@ -1025,8 +1025,8 @@ 
 	 *
 	 * @return bool               False for failure. True for success.
 	 */
-	public function add_meta( $meta_key = '', $meta_value, $unique = false ) {
-		return Give()->donor_meta->add_meta( $this->id, $meta_key, $meta_value, $unique );
+	public function add_meta($meta_key = '', $meta_value, $unique = false) {
+		return Give()->donor_meta->add_meta($this->id, $meta_key, $meta_value, $unique);
 	}
 
 	/**
@@ -1041,8 +1041,8 @@ 
 	 *
 	 * @return bool               False on failure, true if success.
 	 */
-	public function update_meta( $meta_key = '', $meta_value, $prev_value = '' ) {
-		return Give()->donor_meta->update_meta( $this->id, $meta_key, $meta_value, $prev_value );
+	public function update_meta($meta_key = '', $meta_value, $prev_value = '') {
+		return Give()->donor_meta->update_meta($this->id, $meta_key, $meta_value, $prev_value);
 	}
 
 	/**
@@ -1056,8 +1056,8 @@ 
 	 *
 	 * @return bool               False for failure. True for success.
 	 */
-	public function delete_meta( $meta_key = '', $meta_value = '' ) {
-		return Give()->donor_meta->delete_meta( $this->id, $meta_key, $meta_value );
+	public function delete_meta($meta_key = '', $meta_value = '') {
+		return Give()->donor_meta->delete_meta($this->id, $meta_key, $meta_value);
 	}
 
 	/**
@@ -1070,51 +1070,51 @@ 
 	 *
 	 * @return array       The sanitized data, based off column defaults.
 	 */
-	private function sanitize_columns( $data ) {
+	private function sanitize_columns($data) {
 
 		$columns        = $this->db->get_columns();
 		$default_values = $this->db->get_column_defaults();
 
-		foreach ( $columns as $key => $type ) {
+		foreach ($columns as $key => $type) {
 
 			// Only sanitize data that we were provided
-			if ( ! array_key_exists( $key, $data ) ) {
+			if ( ! array_key_exists($key, $data)) {
 				continue;
 			}
 
-			switch ( $type ) {
+			switch ($type) {
 
 				case '%s':
-					if ( 'email' == $key ) {
-						$data[ $key ] = sanitize_email( $data[ $key ] );
-					} elseif ( 'notes' == $key ) {
-						$data[ $key ] = strip_tags( $data[ $key ] );
+					if ('email' == $key) {
+						$data[$key] = sanitize_email($data[$key]);
+					} elseif ('notes' == $key) {
+						$data[$key] = strip_tags($data[$key]);
 					} else {
-						$data[ $key ] = sanitize_text_field( $data[ $key ] );
+						$data[$key] = sanitize_text_field($data[$key]);
 					}
 					break;
 
 				case '%d':
-					if ( ! is_numeric( $data[ $key ] ) || (int) $data[ $key ] !== absint( $data[ $key ] ) ) {
-						$data[ $key ] = $default_values[ $key ];
+					if ( ! is_numeric($data[$key]) || (int) $data[$key] !== absint($data[$key])) {
+						$data[$key] = $default_values[$key];
 					} else {
-						$data[ $key ] = absint( $data[ $key ] );
+						$data[$key] = absint($data[$key]);
 					}
 					break;
 
 				case '%f':
 					// Convert what was given to a float
-					$value = floatval( $data[ $key ] );
+					$value = floatval($data[$key]);
 
-					if ( ! is_float( $value ) ) {
-						$data[ $key ] = $default_values[ $key ];
+					if ( ! is_float($value)) {
+						$data[$key] = $default_values[$key];
 					} else {
-						$data[ $key ] = $value;
+						$data[$key] = $value;
 					}
 					break;
 
 				default:
-					$data[ $key ] = sanitize_text_field( $data[ $key ] );
+					$data[$key] = sanitize_text_field($data[$key]);
 					break;
 
 			}
@@ -1134,33 +1134,33 @@ 
 	 *
 	 * @return bool            If the email was added successfully
 	 */
-	public function add_email( $email = '', $primary = false ) {
-		if ( ! is_email( $email ) ) {
+	public function add_email($email = '', $primary = false) {
+		if ( ! is_email($email)) {
 			return false;
 		}
-		$existing = new Give_Donor( $email );
+		$existing = new Give_Donor($email);
 
-		if ( $existing->id > 0 ) {
+		if ($existing->id > 0) {
 			// Email address already belongs to another donor
 			return false;
 		}
 
-		if ( email_exists( $email ) ) {
-			$user = get_user_by( 'email', $email );
-			if ( $user->ID != $this->user_id ) {
+		if (email_exists($email)) {
+			$user = get_user_by('email', $email);
+			if ($user->ID != $this->user_id) {
 				return false;
 			}
 		}
 
-		do_action( 'give_donor_pre_add_email', $email, $this->id, $this );
+		do_action('give_donor_pre_add_email', $email, $this->id, $this);
 
 		// Add is used to ensure duplicate emails are not added
-		$ret = (bool) $this->add_meta( 'additional_email', $email );
+		$ret = (bool) $this->add_meta('additional_email', $email);
 
-		do_action( 'give_donor_post_add_email', $email, $this->id, $this );
+		do_action('give_donor_post_add_email', $email, $this->id, $this);
 
-		if ( $ret && true === $primary ) {
-			$this->set_primary_email( $email );
+		if ($ret && true === $primary) {
+			$this->set_primary_email($email);
 		}
 
 		return $ret;
@@ -1176,16 +1176,16 @@ 
 	 *
 	 * @return bool          If the email was removed successfully.
 	 */
-	public function remove_email( $email = '' ) {
-		if ( ! is_email( $email ) ) {
+	public function remove_email($email = '') {
+		if ( ! is_email($email)) {
 			return false;
 		}
 
-		do_action( 'give_donor_pre_remove_email', $email, $this->id, $this );
+		do_action('give_donor_pre_remove_email', $email, $this->id, $this);
 
-		$ret = (bool) $this->delete_meta( 'additional_email', $email );
+		$ret = (bool) $this->delete_meta('additional_email', $email);
 
-		do_action( 'give_donor_post_remove_email', $email, $this->id, $this );
+		do_action('give_donor_post_remove_email', $email, $this->id, $this);
 
 		return $ret;
 	}
@@ -1202,16 +1202,16 @@ 
 	 *
 	 * @return bool                      If the email was set as primary successfully.
 	 */
-	public function set_primary_email( $new_primary_email = '' ) {
-		if ( ! is_email( $new_primary_email ) ) {
+	public function set_primary_email($new_primary_email = '') {
+		if ( ! is_email($new_primary_email)) {
 			return false;
 		}
 
-		do_action( 'give_donor_pre_set_primary_email', $new_primary_email, $this->id, $this );
+		do_action('give_donor_pre_set_primary_email', $new_primary_email, $this->id, $this);
 
-		$existing = new Give_Donor( $new_primary_email );
+		$existing = new Give_Donor($new_primary_email);
 
-		if ( $existing->id > 0 && (int) $existing->id !== (int) $this->id ) {
+		if ($existing->id > 0 && (int) $existing->id !== (int) $this->id) {
 			// This email belongs to another donor.
 			return false;
 		}
@@ -1219,21 +1219,21 @@ 
 		$old_email = $this->email;
 
 		// Update donor record with new email.
-		$update = $this->update( array( 'email' => $new_primary_email ) );
+		$update = $this->update(array('email' => $new_primary_email));
 
 		// Remove new primary from list of additional emails.
-		$remove = $this->remove_email( $new_primary_email );
+		$remove = $this->remove_email($new_primary_email);
 
 		// Add old email to additional emails list.
-		$add = $this->add_email( $old_email );
+		$add = $this->add_email($old_email);
 
 		$ret = $update && $remove && $add;
 
-		if ( $ret ) {
+		if ($ret) {
 			$this->email = $new_primary_email;
 		}
 
-		do_action( 'give_donor_post_set_primary_email', $new_primary_email, $this->id, $this );
+		do_action('give_donor_post_set_primary_email', $new_primary_email, $this->id, $this);
 
 		return $ret;
 	}
@@ -1248,17 +1248,17 @@ 
 	 *
 	 * @return bool
 	 */
-	private function is_valid_address( $address ) {
+	private function is_valid_address($address) {
 		$is_valid_address = true;
 
 		// Address ready to process even if only one value set.
-		foreach ( $address as $address_type => $value ) {
+		foreach ($address as $address_type => $value) {
 			// @todo: Handle state field validation on basis of country.
-			if ( in_array( $address_type, array( 'line2', 'state' ) ) ) {
+			if (in_array($address_type, array('line2', 'state'))) {
 				continue;
 			}
 
-			if ( empty( $value ) ) {
+			if (empty($value)) {
 				$is_valid_address = false;
 				break;
 			}
@@ -1285,57 +1285,57 @@ 
 	 *
 	 * @return bool
 	 */
-	public function add_address( $address_type, $address ) {
+	public function add_address($address_type, $address) {
 		// Bailout.
-		if ( empty( $address_type ) || ! $this->is_valid_address( $address ) || ! $this->id ) {
+		if (empty($address_type) || ! $this->is_valid_address($address) || ! $this->id) {
 			return false;
 		}
 
 		// Check if multiple address exist or not and set params.
 		$multi_address_id = null;
-		if ( $is_multi_address = ( false !== strpos( $address_type, '[]' ) ) ) {
-			$address_type = $is_multi_address ? str_replace( '[]', '', $address_type ) : $address_type;
-		} elseif ( $is_multi_address = ( false !== strpos( $address_type, '_' ) ) ) {
-			$multi_address_id = $is_multi_address ? array_pop( explode( '_', $address_type ) ) : $address_type;
+		if ($is_multi_address = (false !== strpos($address_type, '[]'))) {
+			$address_type = $is_multi_address ? str_replace('[]', '', $address_type) : $address_type;
+		} elseif ($is_multi_address = (false !== strpos($address_type, '_'))) {
+			$multi_address_id = $is_multi_address ? array_pop(explode('_', $address_type)) : $address_type;
 
-			$address_type = $is_multi_address ? array_shift( explode( '_', $address_type ) ) : $address_type;
+			$address_type = $is_multi_address ? array_shift(explode('_', $address_type)) : $address_type;
 		}
 
 		// Bailout: do not save duplicate orders
-		if ( $this->does_address_exist( $address_type, $address ) ) {
+		if ($this->does_address_exist($address_type, $address)) {
 			return false;
 		}
 
 		// Set default address.
-		$address = wp_parse_args( $address, array(
+		$address = wp_parse_args($address, array(
 			'line1'   => '',
 			'line2'   => '',
 			'city'    => '',
 			'state'   => '',
 			'country' => '',
 			'zip'     => '',
-		) );
+		));
 
 		// Set meta key prefix.
 		global $wpdb;
 		$meta_key_prefix = "_give_donor_address_{$address_type}_{address_name}";
 		$meta_type       = Give()->donor_meta->meta_type;
 
-		if ( $is_multi_address ) {
-			if ( is_null( $multi_address_id ) ) {
+		if ($is_multi_address) {
+			if (is_null($multi_address_id)) {
 				// Get latest address key to set multi address id.
-				$multi_address_id = $wpdb->get_var( $wpdb->prepare( "
+				$multi_address_id = $wpdb->get_var($wpdb->prepare("
 						SELECT meta_key FROM {$wpdb->donormeta}
 						WHERE meta_key
 						LIKE '%%%s%%'
 						AND {$meta_type}_id=%d
 						ORDER BY meta_id DESC
 						LIMIT 1
-						", "_give_donor_address_{$address_type}_line1", $this->id ) );
+						", "_give_donor_address_{$address_type}_line1", $this->id));
 
-				if ( ! empty( $multi_address_id ) ) {
-					$multi_address_id = absint( substr( strrchr( $multi_address_id, '_' ), 1 ) );
-					$multi_address_id ++;
+				if ( ! empty($multi_address_id)) {
+					$multi_address_id = absint(substr(strrchr($multi_address_id, '_'), 1));
+					$multi_address_id++;
 				} else {
 					$multi_address_id = 0;
 				}
@@ -1345,9 +1345,9 @@ 
 		}
 
 		// Save donor address.
-		foreach ( $address as $type => $value ) {
-			$meta_key = str_replace( '{address_name}', $type, $meta_key_prefix );
-			Give()->donor_meta->update_meta( $this->id, $meta_key, $value );
+		foreach ($address as $type => $value) {
+			$meta_key = str_replace('{address_name}', $type, $meta_key_prefix);
+			Give()->donor_meta->update_meta($this->id, $meta_key, $value);
 		}
 
 		$this->setup_address();
@@ -1366,31 +1366,31 @@ 
 	 *
 	 * @return bool
 	 */
-	public function remove_address( $address_id ) {
+	public function remove_address($address_id) {
 		global $wpdb;
 
 		// Get address type.
-		$is_multi_address = false !== strpos( $address_id, '_' ) ? true : false;
+		$is_multi_address = false !== strpos($address_id, '_') ? true : false;
 
-		$address_type = false !== strpos( $address_id, '_' ) ? array_shift( explode( '_', $address_id ) ) : $address_id;
+		$address_type = false !== strpos($address_id, '_') ? array_shift(explode('_', $address_id)) : $address_id;
 
-		$address_count = false !== strpos( $address_id, '_' ) ? array_pop( explode( '_', $address_id ) ) : null;
+		$address_count = false !== strpos($address_id, '_') ? array_pop(explode('_', $address_id)) : null;
 
 		// Set meta key prefix.
 		$meta_key_prefix = "_give_donor_address_{$address_type}_%";
-		if ( $is_multi_address && is_numeric( $address_count ) ) {
+		if ($is_multi_address && is_numeric($address_count)) {
 			$meta_key_prefix .= "_{$address_count}";
 		}
 
 		$meta_type = Give()->donor_meta->meta_type;
 
 		// Process query.
-		$row_affected = $wpdb->query( $wpdb->prepare( "
+		$row_affected = $wpdb->query($wpdb->prepare("
 				DELETE FROM {$wpdb->donormeta}
 				WHERE meta_key
 				LIKE '%s'
 				AND {$meta_type}_id=%d
-				", $meta_key_prefix, $this->id ) );
+				", $meta_key_prefix, $this->id));
 
 		$this->setup_address();
 
@@ -1409,39 +1409,39 @@ 
 	 *
 	 * @return bool
 	 */
-	public function update_address( $address_id, $address ) {
+	public function update_address($address_id, $address) {
 		global $wpdb;
 
 		// Get address type.
-		$is_multi_address = false !== strpos( $address_id, '_' ) ? true : false;
+		$is_multi_address = false !== strpos($address_id, '_') ? true : false;
 
-		$address_type = false !== strpos( $address_id, '_' ) ? array_shift( explode( '_', $address_id ) ) : $address_id;
+		$address_type = false !== strpos($address_id, '_') ? array_shift(explode('_', $address_id)) : $address_id;
 
-		$address_count = false !== strpos( $address_id, '_' ) ? array_pop( explode( '_', $address_id ) ) : null;
+		$address_count = false !== strpos($address_id, '_') ? array_pop(explode('_', $address_id)) : null;
 
 		// Set meta key prefix.
 		$meta_key_prefix = "_give_donor_address_{$address_type}_%";
-		if ( $is_multi_address && is_numeric( $address_count ) ) {
+		if ($is_multi_address && is_numeric($address_count)) {
 			$meta_key_prefix .= "_{$address_count}";
 		}
 
 		$meta_type = Give()->donor_meta->meta_type;
 
 		// Process query.
-		$row_affected = $wpdb->get_results( $wpdb->prepare( "
+		$row_affected = $wpdb->get_results($wpdb->prepare("
 				SELECT meta_key FROM {$wpdb->donormeta}
 				WHERE meta_key
 				LIKE '%s'
 				AND {$meta_type}_id=%d
-				", $meta_key_prefix, $this->id ) );
+				", $meta_key_prefix, $this->id));
 
 		// Return result.
-		if ( ! count( $row_affected ) ) {
+		if ( ! count($row_affected)) {
 			return false;
 		}
 
 		// Update address.
-		if ( ! $this->add_address( $address_id, $address ) ) {
+		if ( ! $this->add_address($address_id, $address)) {
 			return false;
 		}
 
@@ -1460,39 +1460,39 @@ 
 	 *
 	 * @return bool|null
 	 */
-	public function does_address_exist( $current_address_type, $current_address ) {
+	public function does_address_exist($current_address_type, $current_address) {
 		$status = false;
 
 		// Bailout.
-		if ( empty( $current_address_type ) || empty( $current_address ) ) {
+		if (empty($current_address_type) || empty($current_address)) {
 			return null;
 		}
 
 		// Bailout.
-		if ( empty( $this->address ) || empty( $this->address[ $current_address_type ] ) ) {
+		if (empty($this->address) || empty($this->address[$current_address_type])) {
 			return $status;
 		}
 
 		// Get address.
-		$address = $this->address[ $current_address_type ];
+		$address = $this->address[$current_address_type];
 
-		switch ( true ) {
+		switch (true) {
 
 			// Single address.
-			case is_string( end( $address ) ) :
-				$status = $this->is_address_match( $current_address, $address );
+			case is_string(end($address)) :
+				$status = $this->is_address_match($current_address, $address);
 				break;
 
 			// Multi address.
-			case is_array( end( $address ) ):
+			case is_array(end($address)):
 				// Compare address.
-				foreach ( $address as $saved_address ) {
-					if ( empty( $saved_address ) ) {
+				foreach ($address as $saved_address) {
+					if (empty($saved_address)) {
 						continue;
 					}
 
 					// Exit loop immediately if address exist.
-					if ( $status = $this->is_address_match( $current_address, $saved_address ) ) {
+					if ($status = $this->is_address_match($current_address, $saved_address)) {
 						break;
 					}
 				}
@@ -1513,10 +1513,10 @@ 
 	 *
 	 * @return bool
 	 */
-	private function is_address_match( $address_1, $address_2 ) {
-		$result = array_diff( $address_1, $address_2 );
+	private function is_address_match($address_1, $address_2) {
+		$result = array_diff($address_1, $address_2);
 
-		return empty( $result );
+		return empty($result);
 	}
 
 	/**
@@ -1527,23 +1527,23 @@ 
 	 * @since   2.0
 	 * @return  object
 	 */
-	public function split_donor_name( $id ) {
+	public function split_donor_name($id) {
 		$first_name = $last_name = '';
-		$donor      = new Give_Donor( $id );
+		$donor      = new Give_Donor($id);
 
-		$split_donor_name = explode( ' ', $donor->name, 2 );
+		$split_donor_name = explode(' ', $donor->name, 2);
 
 		// Check for existence of first name after split of donor name.
-		if ( is_array( $split_donor_name ) && ! empty( $split_donor_name[0] ) ) {
+		if (is_array($split_donor_name) && ! empty($split_donor_name[0])) {
 			$first_name = $split_donor_name[0];
 		}
 
 		// Check for existence of last name after split of donor name.
-		if ( is_array( $split_donor_name ) && ! empty( $split_donor_name[1] ) ) {
+		if (is_array($split_donor_name) && ! empty($split_donor_name[1])) {
 			$last_name = $split_donor_name[1];
 		}
 
-		return (object) array( 'first_name' => $first_name, 'last_name' => $last_name );
+		return (object) array('first_name' => $first_name, 'last_name' => $last_name);
 	}
 
 	/**
@@ -1553,9 +1553,9 @@ 
 	 * @return  string
 	 */
 	public function get_first_name() {
-		$first_name = $this->get_meta( '_give_donor_first_name' );
-		if ( ! $first_name ) {
-			$first_name = $this->split_donor_name( $this->id )->first_name;
+		$first_name = $this->get_meta('_give_donor_first_name');
+		if ( ! $first_name) {
+			$first_name = $this->split_donor_name($this->id)->first_name;
 		}
 
 		return $first_name;
@@ -1568,15 +1568,15 @@ 
 	 * @return  string
 	 */
 	public function get_last_name() {
-		$first_name = $this->get_meta( '_give_donor_first_name' );
-		$last_name  = $this->get_meta( '_give_donor_last_name' );
+		$first_name = $this->get_meta('_give_donor_first_name');
+		$last_name  = $this->get_meta('_give_donor_last_name');
 
 		// This condition will prevent unnecessary splitting of donor name to fetch last name.
-		if ( ! $first_name && ! $last_name ) {
-			$last_name = $this->split_donor_name( $this->id )->last_name;
+		if ( ! $first_name && ! $last_name) {
+			$last_name = $this->split_donor_name($this->id)->last_name;
 		}
 
-		return ( $last_name ) ? $last_name : '';
+		return ($last_name) ? $last_name : '';
 	}
 
 	/**
@@ -1587,7 +1587,7 @@ 
 	 * @return  string $company_name Donor Company Name
 	 */
 	public function get_company_name() {
-		$company_name = $this->get_meta( '_give_donor_company' );
+		$company_name = $this->get_meta('_give_donor_company');
 
 		return $company_name;
 	}
@@ -1601,9 +1601,9 @@ 
 	 */
 	public function get_last_donation() {
 
-		$payments = array_unique( array_values( explode( ',', $this->payment_ids ) ) );
+		$payments = array_unique(array_values(explode(',', $this->payment_ids)));
 
-		return end( $payments );
+		return end($payments);
 
 	}
 
@@ -1616,18 +1616,18 @@ 
 	 *
 	 * @return string The date of the last donation.
 	 */
-	public function get_last_donation_date( $formatted = false ) {
+	public function get_last_donation_date($formatted = false) {
 		$completed_data = '';
 
 		// Return if donation id is invalid.
-		if( ! ( $last_donation = absint( $this->get_last_donation() ) ) ) {
+		if ( ! ($last_donation = absint($this->get_last_donation()))) {
 			return $completed_data;
 		}
 
-		$completed_data = give_get_payment_completed_date( $last_donation );
+		$completed_data = give_get_payment_completed_date($last_donation);
 
-		if ( $formatted ) {
-			return date_i18n( give_date_format(), strtotime( $completed_data ) );
+		if ($formatted) {
+			return date_i18n(give_date_format(), strtotime($completed_data));
 		}
 
 		return $completed_data;
@@ -1643,10 +1643,10 @@ 
 	 */
 	public function get_donor_initals() {
 
-		$first_name_initial = mb_substr( $this->get_first_name(), 0, 1, 'utf-8' );
-		$last_name_initial  = mb_substr( $this->get_last_name(), 0, 1, 'utf-8' );
+		$first_name_initial = mb_substr($this->get_first_name(), 0, 1, 'utf-8');
+		$last_name_initial  = mb_substr($this->get_last_name(), 0, 1, 'utf-8');
 
-		return apply_filters( 'get_donor_initals', $first_name_initial . $last_name_initial );
+		return apply_filters('get_donor_initals', $first_name_initial.$last_name_initial);
 
 	}
 

includes/donors/frontend-donor-functions.php

Doc Comments +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
  *
  * @since 2.2.0
  *
- * @param string|int $id_or_email
+ * @param string $id_or_email
  *
  * @return bool
  */

Spacing +42 added lines, -42 removed lines

@@ -21,15 +21,15 @@ 
  *
  * @return string HTML output.
  */
-function give_get_donor_avatar( $donor, $size = 60 ) {
+function give_get_donor_avatar($donor, $size = 60) {
 	ob_start();
 	?>
 	<div class="give-donor__image">
 		<?php
 		// Check if gravatar exists.
-		if ( give_validate_gravatar( $donor->email ) ) {
+		if (give_validate_gravatar($donor->email)) {
 			// Return avatar.
-			echo get_avatar( $donor->email, $size );
+			echo get_avatar($donor->email, $size);
 		} else {
 			// No gravatar = output initials.
 			echo $donor->get_donor_initals();
@@ -37,7 +37,7 @@ 
 	</div>
 	<?php
 
-	return apply_filters( 'give_get_donor_avatar', ob_get_clean() );
+	return apply_filters('give_get_donor_avatar', ob_get_clean());
 
 }
 
@@ -50,51 +50,51 @@ 
  *
  * @return bool
  */
-function give_validate_gravatar( $id_or_email ) {
+function give_validate_gravatar($id_or_email) {
 
 	//id or email code borrowed from wp-includes/pluggable.php
 	$email = '';
-	if ( is_numeric( $id_or_email ) ) {
+	if (is_numeric($id_or_email)) {
 		$id   = (int) $id_or_email;
-		$user = get_userdata( $id );
-		if ( $user ) {
+		$user = get_userdata($id);
+		if ($user) {
 			$email = $user->user_email;
 		}
-	} elseif ( is_object( $id_or_email ) ) {
+	} elseif (is_object($id_or_email)) {
 		// No avatar for pingbacks or trackbacks
-		$allowed_comment_types = apply_filters( 'get_avatar_comment_types', array( 'comment' ) );
-		if ( ! empty( $id_or_email->comment_type ) && ! in_array( $id_or_email->comment_type, (array) $allowed_comment_types ) ) {
+		$allowed_comment_types = apply_filters('get_avatar_comment_types', array('comment'));
+		if ( ! empty($id_or_email->comment_type) && ! in_array($id_or_email->comment_type, (array) $allowed_comment_types)) {
 			return false;
 		}
 
-		if ( ! empty( $id_or_email->user_id ) ) {
+		if ( ! empty($id_or_email->user_id)) {
 			$id   = (int) $id_or_email->user_id;
-			$user = get_userdata( $id );
-			if ( $user ) {
+			$user = get_userdata($id);
+			if ($user) {
 				$email = $user->user_email;
 			}
-		} elseif ( ! empty( $id_or_email->comment_author_email ) ) {
+		} elseif ( ! empty($id_or_email->comment_author_email)) {
 			$email = $id_or_email->comment_author_email;
 		}
 	} else {
 		$email = $id_or_email;
 	}
 
-	$hashkey = md5( strtolower( trim( $email ) ) );
-	$uri     = 'http://www.gravatar.com/avatar/' . $hashkey . '?d=404';
+	$hashkey = md5(strtolower(trim($email)));
+	$uri     = 'http://www.gravatar.com/avatar/'.$hashkey.'?d=404';
 
-	$data = wp_cache_get( $hashkey );
-	if ( false === $data ) {
-		$response = wp_remote_head( $uri );
-		if ( is_wp_error( $response ) ) {
+	$data = wp_cache_get($hashkey);
+	if (false === $data) {
+		$response = wp_remote_head($uri);
+		if (is_wp_error($response)) {
 			$data = 'not200';
 		} else {
 			$data = $response['response']['code'];
 		}
-		wp_cache_set( $hashkey, $data, $group = '', $expire = 60 * 5 );
+		wp_cache_set($hashkey, $data, $group = '', $expire = 60 * 5);
 
 	}
-	if ( $data == '200' ) {
+	if ($data == '200') {
 		return true;
 	} else {
 		return false;
@@ -114,18 +114,18 @@ 
  *
  * @return int The new note ID
  */
-function give_insert_donor_donation_comment( $donation_id, $donor, $note, $comment_args = array() ) {
+function give_insert_donor_donation_comment($donation_id, $donor, $note, $comment_args = array()) {
 	$comment_args = wp_parse_args(
 		$comment_args,
 		array(
 			'comment_approved' => 0,
-			'comment_parent'   => give_get_payment_form_id( $donation_id )
+			'comment_parent'   => give_get_payment_form_id($donation_id)
 		)
 	);
 
-	$comment_id = Give_Comment::add( $donation_id, $note, 'payment', $comment_args );
+	$comment_id = Give_Comment::add($donation_id, $note, 'payment', $comment_args);
 
-	update_comment_meta( $comment_id, '_give_donor_id', $donor );
+	update_comment_meta($comment_id, '_give_donor_id', $donor);
 
 	return $comment_id;
 }
@@ -144,7 +144,7 @@ 
  *
  * @return WP_Comment|array
  */
-function give_get_donor_donation_comment( $donation_id, $donor_id, $search = '' ) {
+function give_get_donor_donation_comment($donation_id, $donor_id, $search = '') {
 	$comments = Give_Comment::get(
 		$donation_id,
 		'payment',
@@ -160,7 +160,7 @@ 
 		$search
 	);
 
-	return ( ! empty( $comments ) ? current( $comments ) : array() );
+	return ( ! empty($comments) ? current($comments) : array());
 }
 
 /**
@@ -176,9 +176,9 @@ 
  *
  * @return int
  */
-function give_get_donor_donation_comment_id( $donation_id, $donor_id, $search = '' ) {
+function give_get_donor_donation_comment_id($donation_id, $donor_id, $search = '') {
 	/* @var WP_Comment|array $comment */
-	$comment    = give_get_donor_donation_comment( $donation_id, $donor_id, $search );
+	$comment    = give_get_donor_donation_comment($donation_id, $donor_id, $search);
 	$comment_id = $comment instanceof WP_Comment ? $comment->comment_ID : 0;
 
 	return $comment_id;
@@ -197,7 +197,7 @@ 
  *
  * @return array
  */
-function give_get_donor_donation_comments( $donor_id, $comment_args = array(), $search = '' ) {
+function give_get_donor_donation_comments($donor_id, $comment_args = array(), $search = '') {
 	$comments = Give_Comment::get(
 		$donor_id,
 		'payment',
@@ -205,7 +205,7 @@ 
 		$search
 	);
 
-	return ( ! empty( $comments ) ? $comments : array() );
+	return ( ! empty($comments) ? $comments : array());
 }
 
 
@@ -219,19 +219,19 @@ 
  *
  * @return string
  */
-function give_get_donor_donation_comment_html( $comment, $payment_id = 0 ) {
+function give_get_donor_donation_comment_html($comment, $payment_id = 0) {
 
-	if ( is_numeric( $comment ) ) {
-		$comment = get_comment( $comment );
+	if (is_numeric($comment)) {
+		$comment = get_comment($comment);
 	}
 
-	$date_format = give_date_format() . ', ' . get_option( 'time_format' );
+	$date_format = give_date_format().', '.get_option('time_format');
 
 	$comment_html = sprintf(
 		'<div class="give-payment-note" id="give-payment-note-%s"><p><strong>%s</strong>&nbsp;&ndash;&nbsp;<span style="color:#aaa;font-style:italic;">%s</span><br/>%s</p></div>',
 		$comment->comment_ID,
-		get_comment_author( $comment->comment_ID ),
-		date_i18n( $date_format, strtotime( $comment->comment_date ) ),
+		get_comment_author($comment->comment_ID),
+		date_i18n($date_format, strtotime($comment->comment_date)),
 		$comment->comment_content
 	);
 
@@ -250,7 +250,7 @@ 
  *
  * @return WP_Comment/array
  */
-function give_get_donor_latest_comment( $donor_id, $form_id = 0 ) {
+function give_get_donor_latest_comment($donor_id, $form_id = 0) {
 	$comment_args = array(
 		'post_id'    => 0,
 		'orderby'    => 'comment_ID',
@@ -270,11 +270,11 @@ 
 	);
 
 	// Get donor donation comment for specific form.
-	if ( $form_id ) {
+	if ($form_id) {
 		$comment_args['parent'] = $form_id;
 	}
 
-	$comment = current( give_get_donor_donation_comments( $donor_id, $comment_args ) );
+	$comment = current(give_get_donor_donation_comments($donor_id, $comment_args));
 
 	return $comment;
 }