WordImpress / Give

includes/admin/upgrades/class-give-updates.php

Braces +5 added lines, -2 removed lines

@@ -615,8 +615,11 @@
 				<a href="<?php echo esc_url( add_query_arg( array( 'give-restart-db-upgrades' => 1 ), admin_url( 'edit.php?post_type=give_forms&page=give-updates' ) ) ); ?>" class="button button-primary give-restart-updater-btn">
 					<?php _e( 'Restart the updater', 'give' ); ?>
 				</a>
-			<?php else: ?>
-				<strong><?php _e( 'Database Update', 'give' ); ?></strong>
+			<?php else {
+	: ?>
+				<strong><?php _e( 'Database Update', 'give' );
+}
+?></strong>
 				&nbsp;&#8211;&nbsp;<?php _e( 'An unexpected issue occurred during the database update which caused it to stop automatically. Please contact support for assistance.', 'give' ); ?>
 				<a href="<?php echo esc_url('http://docs.givewp.com/troubleshooting-db-updates')?>" target="_blank"><?php _e( 'Read More', 'give' ); ?> &raquo;</a>
 			<?php

Spacing +246 added lines, -253 removed lines

@@ -80,34 +80,34 @@ 
 	 *
 	 * @param array $args
 	 */
-	public function register( $args ) {
+	public function register($args) {
 		$args_default = array(
 			'id'       => '',
 			'version'  => '',
 			'callback' => '',
 		);
 
-		$args = wp_parse_args( $args, $args_default );
+		$args = wp_parse_args($args, $args_default);
 
 		// You can only register database upgrade.
 		$args['type'] = 'database';
 
 		// Bailout.
 		if (
-			empty( $args['id'] ) ||
-			empty( $args['version'] ) ||
-			empty( $args['callback'] ) ||
-			! is_callable( $args['callback'] )
+			empty($args['id']) ||
+			empty($args['version']) ||
+			empty($args['callback']) ||
+			! is_callable($args['callback'])
 		) {
 			return;
 		}
 
 		// Change depend param to array.
-		if ( isset( $args['depend'] ) && is_string( $args['depend'] ) ) {
-			$args['depend'] = array( $args['depend'] );
+		if (isset($args['depend']) && is_string($args['depend'])) {
+			$args['depend'] = array($args['depend']);
 		}
 
-		$this->updates[ $args['type'] ][] = $args;
+		$this->updates[$args['type']][] = $args;
 	}
 
 	/**
@@ -118,7 +118,7 @@ 
 	 * @return static
 	 */
 	static function get_instance() {
-		if ( is_null( self::$instance ) ) {
+		if (is_null(self::$instance)) {
 			self::$instance = new self();
 		}
 
@@ -136,27 +136,27 @@ 
 		/**
 		 * Load file
 		 */
-		require_once GIVE_PLUGIN_DIR . 'includes/class-give-background-updater.php';
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/upgrade-functions.php';
+		require_once GIVE_PLUGIN_DIR.'includes/class-give-background-updater.php';
+		require_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/upgrade-functions.php';
 
 		self::$background_updater = new Give_Background_Updater();
 
 		/**
 		 * Setup hooks.
 		 */
-		add_action( 'init', array( $this, '__register_upgrade' ), 9999 );
-		add_action( 'give_set_upgrade_completed', array( $this, '__flush_resume_updates' ), 9999 );
-		add_action( 'wp_ajax_give_db_updates_info', array( $this, '__give_db_updates_info' ) );
-		add_action( 'wp_ajax_give_run_db_updates', array( $this, '__give_start_updating' ) );
-		add_action( 'admin_init', array( $this, '__redirect_admin' ) );
-		add_action( 'admin_init', array( $this, '__pause_db_update' ), - 1 );
-		add_action( 'admin_init', array( $this, '__restart_db_update' ), - 1 );
-		add_action( 'admin_notices', array( $this, '__show_notice' ) );
-		add_action( 'give_restart_db_upgrade', array( $this, '__health_background_update' ) );
-
-		if ( is_admin() ) {
-			add_action( 'admin_init', array( $this, '__change_donations_label' ), 9999 );
-			add_action( 'admin_menu', array( $this, '__register_menu' ), 9999 );
+		add_action('init', array($this, '__register_upgrade'), 9999);
+		add_action('give_set_upgrade_completed', array($this, '__flush_resume_updates'), 9999);
+		add_action('wp_ajax_give_db_updates_info', array($this, '__give_db_updates_info'));
+		add_action('wp_ajax_give_run_db_updates', array($this, '__give_start_updating'));
+		add_action('admin_init', array($this, '__redirect_admin'));
+		add_action('admin_init', array($this, '__pause_db_update'), - 1);
+		add_action('admin_init', array($this, '__restart_db_update'), - 1);
+		add_action('admin_notices', array($this, '__show_notice'));
+		add_action('give_restart_db_upgrade', array($this, '__health_background_update'));
+
+		if (is_admin()) {
+			add_action('admin_init', array($this, '__change_donations_label'), 9999);
+			add_action('admin_menu', array($this, '__register_menu'), 9999);
 		}
 	}
 
@@ -170,12 +170,12 @@ 
 		$addons         = give_get_plugins();
 		$plugin_updates = get_plugin_updates();
 
-		foreach ( $addons as $key => $info ) {
-			if ( 'active' != $info['Status'] || 'add-on' != $info['Type'] || empty( $plugin_updates[ $key ] ) ) {
+		foreach ($addons as $key => $info) {
+			if ('active' != $info['Status'] || 'add-on' != $info['Type'] || empty($plugin_updates[$key])) {
 				continue;
 			}
 
-			$this->updates['plugin'][] = array_merge( $info, (array) $plugin_updates[ $key ] );
+			$this->updates['plugin'][] = array_merge($info, (array) $plugin_updates[$key]);
 		}
 	}
 
@@ -187,7 +187,7 @@ 
 	 * @access public
 	 */
 	public function __register_upgrade() {
-		if ( ! is_admin() ) {
+		if ( ! is_admin()) {
 			return;
 		}
 
@@ -196,7 +196,7 @@ 
 		 *
 		 * @since 1.8.12
 		 */
-		do_action( 'give_register_updates', $this );
+		do_action('give_register_updates', $this);
 	}
 
 	/**
@@ -209,23 +209,22 @@ 
 		global $menu;
 
 		// Bailout.
-		if ( empty( $menu ) || ! $this->get_total_update_count() ) {
+		if (empty($menu) || ! $this->get_total_update_count()) {
 			return;
 		}
 
-		$is_update = ( $this->is_doing_updates() && ! self::$background_updater->is_paused_process() );
+		$is_update = ($this->is_doing_updates() && ! self::$background_updater->is_paused_process());
 
-		foreach ( $menu as $index => $menu_item ) {
-			if ( 'edit.php?post_type=give_forms' !== $menu_item[2] ) {
+		foreach ($menu as $index => $menu_item) {
+			if ('edit.php?post_type=give_forms' !== $menu_item[2]) {
 				continue;
 			}
 
-			$menu[ $index ][0] = sprintf(
+			$menu[$index][0] = sprintf(
 				'%1$s <span class="update-plugins"><span class="plugin-count give-update-progress-count">%2$s%3$s</span></span>',
-				__( 'Donations', 'give' ),
+				__('Donations', 'give'),
 				$is_update ?
-					$this->get_db_update_processing_percentage() :
-					$this->get_total_update_count(),
+					$this->get_db_update_processing_percentage() : $this->get_total_update_count(),
 				$is_update ? '%' : ''
 			);
 
@@ -244,41 +243,40 @@ 
 		$this->__register_plugin_addon_updates();
 
 		// Bailout.
-		if ( ! $this->get_total_update_count() ) {
+		if ( ! $this->get_total_update_count()) {
 			// Show complete update message if still on update setting page.
-			if ( isset( $_GET['page'] ) && 'give-updates' === $_GET['page'] ) {
+			if (isset($_GET['page']) && 'give-updates' === $_GET['page']) {
 				// Upgrades
 				add_submenu_page(
 					'edit.php?post_type=give_forms',
-					esc_html__( 'Give Updates Complete', 'give' ),
-					__( 'Updates', 'give' ),
+					esc_html__('Give Updates Complete', 'give'),
+					__('Updates', 'give'),
 					'manage_give_settings',
 					'give-updates',
-					array( $this, 'render_complete_page' )
+					array($this, 'render_complete_page')
 				);
 			}
 
 			return;
 		}
 
-		$is_update = ( $this->is_doing_updates() && ! self::$background_updater->is_paused_process() );
+		$is_update = ($this->is_doing_updates() && ! self::$background_updater->is_paused_process());
 
 		// Upgrades
 		add_submenu_page(
 			'edit.php?post_type=give_forms',
-			esc_html__( 'Give Updates', 'give' ),
+			esc_html__('Give Updates', 'give'),
 			sprintf(
 				'%1$s <span class="update-plugins"%2$s><span class="plugin-count give-update-progress-count">%3$s%4$s</span></span>',
-				__( 'Updates', 'give' ),
-				isset( $_GET['give-pause-db-upgrades'] ) ? ' style="display:none;"' : '',
+				__('Updates', 'give'),
+				isset($_GET['give-pause-db-upgrades']) ? ' style="display:none;"' : '',
 				$is_update ?
-					$this->get_db_update_processing_percentage() :
-					$this->get_total_update_count(),
+					$this->get_db_update_processing_percentage() : $this->get_total_update_count(),
 				$is_update ? '%' : ''
 			),
 			'manage_give_settings',
 			'give-updates',
-			array( $this, 'render_page' )
+			array($this, 'render_page')
 		);
 	}
 
@@ -293,13 +291,13 @@ 
 		// Show db upgrade completed notice.
 		if (
 			! wp_doing_ajax() &&
-			current_user_can( 'manage_give_settings' ) &&
-			get_option( 'give_show_db_upgrade_complete_notice' ) &&
-			! isset( $_GET['give-db-update-completed'] )
+			current_user_can('manage_give_settings') &&
+			get_option('give_show_db_upgrade_complete_notice') &&
+			! isset($_GET['give-db-update-completed'])
 		) {
-			delete_option( 'give_show_db_upgrade_complete_notice' );
+			delete_option('give_show_db_upgrade_complete_notice');
 
-			wp_redirect( admin_url( 'edit.php?post_type=give_forms&page=give-updates&give-db-update-completed=give_db_upgrade_completed' ) );
+			wp_redirect(admin_url('edit.php?post_type=give_forms&page=give-updates&give-db-update-completed=give_db_upgrade_completed'));
 			exit();
 		}
 	}
@@ -315,15 +313,15 @@ 
 	 *
 	 * @return bool
 	 */
-	public function __pause_db_update( $force = false ) {
+	public function __pause_db_update($force = false) {
 		// Bailout.
 		if (
 			! $force &&
 			(
 				wp_doing_ajax() ||
-				! isset( $_GET['page'] ) ||
+				! isset($_GET['page']) ||
 				'give-updates' !== $_GET['page'] ||
-				! isset( $_GET['give-pause-db-upgrades'] ) ||
+				! isset($_GET['give-pause-db-upgrades']) ||
 				self::$background_updater->is_paused_process()
 			)
 
@@ -331,13 +329,13 @@ 
 			return false;
 		}
 
-		delete_option( 'give_upgrade_error' );
+		delete_option('give_upgrade_error');
 
-		$this->__health_background_update( $this );
+		$this->__health_background_update($this);
 		$batch = self::$background_updater->get_all_batch();
 
 		// Bailout: if batch is empty
-		if ( empty( $batch->data ) ) {
+		if (empty($batch->data)) {
 			return false;
 		}
 
@@ -346,25 +344,25 @@ 
 
 		// Do not stop background process immediately if task running.
 		// @see Give_Background_Updater::lock_process
-		if ( ! $force && self::$background_updater->is_process_running() ) {
-			update_option( 'give_pause_upgrade', 1, false );
+		if ( ! $force && self::$background_updater->is_process_running()) {
+			update_option('give_pause_upgrade', 1, false);
 
 			return true;
 		}
 
-		update_option( 'give_paused_batches', $batch, false );
-		delete_option( $batch->key );
-		delete_site_transient( self::$background_updater->get_identifier() . '_process_lock' );
-		wp_clear_scheduled_hook( self::$background_updater->get_cron_identifier() );
+		update_option('give_paused_batches', $batch, false);
+		delete_option($batch->key);
+		delete_site_transient(self::$background_updater->get_identifier().'_process_lock');
+		wp_clear_scheduled_hook(self::$background_updater->get_cron_identifier());
 
-		Give()->logs->add( 'Update Pause', print_r( $batch, true ), 0, 'update' );
+		Give()->logs->add('Update Pause', print_r($batch, true), 0, 'update');
 
 		/**
 		 * Fire action when pause db updates
 		 *
 		 * @since 2.0.1
 		 */
-		do_action( 'give_pause_db_upgrade', $this );
+		do_action('give_pause_db_upgrade', $this);
 
 		return true;
 	}
@@ -381,31 +379,31 @@ 
 		// Bailout.
 		if (
 			wp_doing_ajax() ||
-			! isset( $_GET['page'] ) ||
+			! isset($_GET['page']) ||
 			'give-updates' !== $_GET['page'] ||
-			! isset( $_GET['give-restart-db-upgrades'] ) ||
+			! isset($_GET['give-restart-db-upgrades']) ||
 			! self::$background_updater->is_paused_process()
 		) {
 			return false;
 		}
 
 		Give_Background_Updater::flush_cache();
-		$batch = get_option( 'give_paused_batches' );
+		$batch = get_option('give_paused_batches');
 
-		if ( ! empty( $batch ) ) {
-			wp_cache_delete( $batch->key, 'options' );
-			update_option( $batch->key, $batch->data, false );
+		if ( ! empty($batch)) {
+			wp_cache_delete($batch->key, 'options');
+			update_option($batch->key, $batch->data, false);
 
-			delete_option( 'give_paused_batches' );
+			delete_option('give_paused_batches');
 
-			Give()->logs->add( 'Update Restart', print_r( $batch, true ), 0, 'update' );
+			Give()->logs->add('Update Restart', print_r($batch, true), 0, 'update');
 
 
 			/** Fire action when restart db updates
 			 *
 			 * @since 2.0.1
 			 */
-			do_action( 'give_restart_db_upgrade', $this );
+			do_action('give_restart_db_upgrade', $this);
 
 			self::$background_updater->dispatch();
 		}
@@ -421,53 +419,53 @@ 
 	 *
 	 * @param Give_Updates $give_updates
 	 */
-	public function __health_background_update( $give_updates ) {
-		if ( ! $this->is_doing_updates() ) {
+	public function __health_background_update($give_updates) {
+		if ( ! $this->is_doing_updates()) {
 			return;
 		}
 
 		Give_Background_Updater::flush_cache();
 
 		$batch                = Give_Updates::$background_updater->get_all_batch();
-		$batch_data_count     = count( $batch->data );
-		$all_updates          = $give_updates->get_updates( 'database', 'all' );
-		$all_update_ids       = wp_list_pluck( $all_updates, 'id' );
-		$all_batch_update_ids = ! empty( $batch->data ) ? wp_list_pluck( $batch->data, 'id' ) : array();
+		$batch_data_count     = count($batch->data);
+		$all_updates          = $give_updates->get_updates('database', 'all');
+		$all_update_ids       = wp_list_pluck($all_updates, 'id');
+		$all_batch_update_ids = ! empty($batch->data) ? wp_list_pluck($batch->data, 'id') : array();
 		$log_data             = '';
-		$doing_upgrade_args   = get_option( 'give_doing_upgrade' );
+		$doing_upgrade_args   = get_option('give_doing_upgrade');
 
-		if ( ! empty( $doing_upgrade_args ) ) {
-			$log_data .= 'Doing update:' . "\n";
-			$log_data .= print_r( $doing_upgrade_args, true ) . "\n";
+		if ( ! empty($doing_upgrade_args)) {
+			$log_data .= 'Doing update:'."\n";
+			$log_data .= print_r($doing_upgrade_args, true)."\n";
 		}
 
 		/**
 		 * Add remove upgrade from batch
 		 */
-		if ( ! empty( $batch->data ) ) {
+		if ( ! empty($batch->data)) {
 
-			foreach ( $batch->data as $index => $update ) {
-				$log_data = print_r( $update, true ) . "\n";
+			foreach ($batch->data as $index => $update) {
+				$log_data = print_r($update, true)."\n";
 
-				if ( ! is_callable( $update['callback'] ) ) {
-					$log_data .= 'Removing missing callback update: ' . "{$update['id']}\n";
-					unset( $batch->data[ $index ] );
-				} elseif ( give_has_upgrade_completed( $update['id'] ) ) {
-					$log_data .= 'Removing already completed update: ' . "{$update['id']}\n";
-					unset( $batch->data[ $index ] );
+				if ( ! is_callable($update['callback'])) {
+					$log_data .= 'Removing missing callback update: '."{$update['id']}\n";
+					unset($batch->data[$index]);
+				} elseif (give_has_upgrade_completed($update['id'])) {
+					$log_data .= 'Removing already completed update: '."{$update['id']}\n";
+					unset($batch->data[$index]);
 				}
 
-				if ( ! empty( $update['depend'] ) ) {
+				if ( ! empty($update['depend'])) {
 
-					foreach ( $update['depend'] as $depend ) {
-						if ( give_has_upgrade_completed( $depend ) ) {
-							$log_data .= 'Completed update: ' . "{$depend}\n";
+					foreach ($update['depend'] as $depend) {
+						if (give_has_upgrade_completed($depend)) {
+							$log_data .= 'Completed update: '."{$depend}\n";
 							continue;
 						}
 
-						if ( in_array( $depend, $all_update_ids ) && ! in_array( $depend, $all_batch_update_ids ) ) {
-							$log_data .= 'Adding missing update: ' . "{$depend}\n";
-							array_unshift( $batch->data, $all_updates[ array_search( $depend, $all_update_ids ) ] );
+						if (in_array($depend, $all_update_ids) && ! in_array($depend, $all_batch_update_ids)) {
+							$log_data .= 'Adding missing update: '."{$depend}\n";
+							array_unshift($batch->data, $all_updates[array_search($depend, $all_update_ids)]);
 						}
 					}
 				}
@@ -477,52 +475,52 @@ 
 		/**
 		 * Add new upgrade to batch
 		 */
-		if ( $new_updates = $this->get_updates( 'database', 'new' ) ) {
-			$all_batch_update_ids = ! empty( $batch->data ) ? wp_list_pluck( $batch->data, 'id' ) : array();
+		if ($new_updates = $this->get_updates('database', 'new')) {
+			$all_batch_update_ids = ! empty($batch->data) ? wp_list_pluck($batch->data, 'id') : array();
 
-			foreach ( $new_updates as $index => $new_update ) {
-				if ( give_has_upgrade_completed( $new_update['id'] ) || in_array( $new_update['id'], $all_batch_update_ids ) ) {
-					unset( $new_updates[ $index ] );
+			foreach ($new_updates as $index => $new_update) {
+				if (give_has_upgrade_completed($new_update['id']) || in_array($new_update['id'], $all_batch_update_ids)) {
+					unset($new_updates[$index]);
 				}
 			}
 
-			if ( ! empty( $new_updates ) ) {
-				$log_data .= 'Adding new update: ' . "\n";
-				$log_data .= print_r( $new_updates, true ) . "\n";
+			if ( ! empty($new_updates)) {
+				$log_data .= 'Adding new update: '."\n";
+				$log_data .= print_r($new_updates, true)."\n";
 
-				$batch->data = array_merge( (array) $batch->data, $new_updates );
-				update_option( 'give_db_update_count', ( absint( get_option( 'give_db_update_count' ) ) + count( $new_updates ) ), false );
+				$batch->data = array_merge((array) $batch->data, $new_updates);
+				update_option('give_db_update_count', (absint(get_option('give_db_update_count')) + count($new_updates)), false);
 			}
 		}
 
 		/**
 		 * Fix batch
 		 */
-		if ( empty( $batch->data ) ) {
+		if (empty($batch->data)) {
 			// Complete batch if do not have any data to process.
-			self::$background_updater->delete( $batch->key );
+			self::$background_updater->delete($batch->key);
 
-			if ( self::$background_updater->has_queue() ) {
-				$this->__health_background_update( $this );
+			if (self::$background_updater->has_queue()) {
+				$this->__health_background_update($this);
 			} else {
-				delete_site_transient( self::$background_updater->get_identifier() . '_process_lock' );
-				wp_clear_scheduled_hook( self::$background_updater->get_cron_identifier() );
+				delete_site_transient(self::$background_updater->get_identifier().'_process_lock');
+				wp_clear_scheduled_hook(self::$background_updater->get_cron_identifier());
 
 				self::$background_updater->complete();
 			}
 
-		} elseif ( $batch_data_count !== count( $batch->data ) ) {
+		} elseif ($batch_data_count !== count($batch->data)) {
 
-			$log_data .= 'Updating batch' . "\n";
-			$log_data .= print_r( $batch, true );
+			$log_data .= 'Updating batch'."\n";
+			$log_data .= print_r($batch, true);
 
-			if ( ! empty( $batch->key ) ) {
-				wp_cache_delete( $batch->key, 'options' );
-				update_option( $batch->key, $batch->data, false );
+			if ( ! empty($batch->key)) {
+				wp_cache_delete($batch->key, 'options');
+				update_option($batch->key, $batch->data, false);
 			} else {
 
-				foreach ( $batch->data as $data ) {
-					Give_Updates::$background_updater->push_to_queue( $data );
+				foreach ($batch->data as $data) {
+					Give_Updates::$background_updater->push_to_queue($data);
 				}
 
 				Give_Updates::$background_updater->save();
@@ -533,31 +531,31 @@ 
 		/**
 		 * Fix give_doing_upgrade option
 		 */
-		if( $fresh_new_db_count = $this->get_total_new_db_update_count( true ) ) {
-			update_option( 'give_db_update_count', $fresh_new_db_count, false );
+		if ($fresh_new_db_count = $this->get_total_new_db_update_count(true)) {
+			update_option('give_db_update_count', $fresh_new_db_count, false);
 		}
 
 		$doing_upgrade_args['update']           = 1;
-		$doing_upgrade_args['heading']          = sprintf( 'Update %s of %s', 1, $fresh_new_db_count );
-		$doing_upgrade_args['total_percentage'] = $this->get_db_update_processing_percentage( true );
+		$doing_upgrade_args['heading']          = sprintf('Update %s of %s', 1, $fresh_new_db_count);
+		$doing_upgrade_args['total_percentage'] = $this->get_db_update_processing_percentage(true);
 
 		// Remove already completed update from info.
 		if (
-			empty( $doing_upgrade_args['update_info'] )
-			|| give_has_upgrade_completed( $doing_upgrade_args['update_info']['id'] )
+			empty($doing_upgrade_args['update_info'])
+			|| give_has_upgrade_completed($doing_upgrade_args['update_info']['id'])
 		) {
-			$doing_upgrade_args['update_info'] = current( array_values( $batch->data ) );
+			$doing_upgrade_args['update_info'] = current(array_values($batch->data));
 			$doing_upgrade_args['step']        = 1;
 		}
 
 		// Check if dependency completed or not.
-		if ( isset( $doing_upgrade_args['update_info']['depend'] ) ) {
-			foreach ( $doing_upgrade_args['update_info']['depend'] as $depend ) {
-				if ( give_has_upgrade_completed( $depend ) ) {
+		if (isset($doing_upgrade_args['update_info']['depend'])) {
+			foreach ($doing_upgrade_args['update_info']['depend'] as $depend) {
+				if (give_has_upgrade_completed($depend)) {
 					continue;
 				}
 
-				$doing_upgrade_args['update_info']      = $all_updates[ array_search( $depend, $all_update_ids ) ];
+				$doing_upgrade_args['update_info']      = $all_updates[array_search($depend, $all_update_ids)];
 				$doing_upgrade_args['step']             = 1;
 				$doing_upgrade_args['percentage']       = 0;
 				$doing_upgrade_args['total_percentage'] = 0;
@@ -566,14 +564,14 @@ 
 			}
 		}
 
-		if( ! empty( $doing_upgrade_args['update_info'] ) ) {
-			update_option( 'give_doing_upgrade', $doing_upgrade_args, false );
+		if ( ! empty($doing_upgrade_args['update_info'])) {
+			update_option('give_doing_upgrade', $doing_upgrade_args, false);
 
-			$log_data .= 'Updated doing update:' . "\n";
-			$log_data .= print_r( $doing_upgrade_args, true ) . "\n";
+			$log_data .= 'Updated doing update:'."\n";
+			$log_data .= print_r($doing_upgrade_args, true)."\n";
 		}
 
-		Give()->logs->add( 'Update Health Check', $log_data, 0, 'update' );
+		Give()->logs->add('Update Health Check', $log_data, 0, 'update');
 	}
 
 
@@ -587,90 +585,90 @@ 
 		$current_screen = get_current_screen();
 
 		// Bailout.
-		if ( ! current_user_can( 'manage_give_settings' ) ) {
+		if ( ! current_user_can('manage_give_settings')) {
 			return;
 		}
 
 		// Run DB updates.
-		if ( ! empty( $_GET['give-run-db-update'] ) ) {
+		if ( ! empty($_GET['give-run-db-update'])) {
 			$this->run_db_update();
 		}
 
 
 		// Bailout.
-		if ( in_array( $current_screen->base, array( 'give_forms_page_give-updates', 'update-core' ) ) ) {
+		if (in_array($current_screen->base, array('give_forms_page_give-updates', 'update-core'))) {
 			return;
 		}
 
 		// Show notice if upgrade paused.
-		if ( self::$background_updater->is_paused_process() ) {
+		if (self::$background_updater->is_paused_process()) {
 			ob_start();
 
-			$upgrade_error = get_option( 'give_upgrade_error' );
-			if ( ! $upgrade_error ) : ?>
-				<strong><?php _e( 'Database Update', 'give' ); ?></strong>
-				&nbsp;&#8211;&nbsp;<?php _e( 'GiveWP needs to update your database to the latest version. The following process will make updates to your site\'s database. Please create a backup before proceeding.', 'give' ); ?>
+			$upgrade_error = get_option('give_upgrade_error');
+			if ( ! $upgrade_error) : ?>
+				<strong><?php _e('Database Update', 'give'); ?></strong>
+				&nbsp;&#8211;&nbsp;<?php _e('GiveWP needs to update your database to the latest version. The following process will make updates to your site\'s database. Please create a backup before proceeding.', 'give'); ?>
 				<br>
 				<br>
-				<a href="<?php echo esc_url( add_query_arg( array( 'give-restart-db-upgrades' => 1 ), admin_url( 'edit.php?post_type=give_forms&page=give-updates' ) ) ); ?>" class="button button-primary give-restart-updater-btn">
-					<?php _e( 'Restart the updater', 'give' ); ?>
+				<a href="<?php echo esc_url(add_query_arg(array('give-restart-db-upgrades' => 1), admin_url('edit.php?post_type=give_forms&page=give-updates'))); ?>" class="button button-primary give-restart-updater-btn">
+					<?php _e('Restart the updater', 'give'); ?>
 				</a>
 			<?php else: ?>
-				<strong><?php _e( 'Database Update', 'give' ); ?></strong>
-				&nbsp;&#8211;&nbsp;<?php _e( 'An unexpected issue occurred during the database update which caused it to stop automatically. Please contact support for assistance.', 'give' ); ?>
-				<a href="<?php echo esc_url('http://docs.givewp.com/troubleshooting-db-updates')?>" target="_blank"><?php _e( 'Read More', 'give' ); ?> &raquo;</a>
+				<strong><?php _e('Database Update', 'give'); ?></strong>
+				&nbsp;&#8211;&nbsp;<?php _e('An unexpected issue occurred during the database update which caused it to stop automatically. Please contact support for assistance.', 'give'); ?>
+				<a href="<?php echo esc_url('http://docs.givewp.com/troubleshooting-db-updates')?>" target="_blank"><?php _e('Read More', 'give'); ?> &raquo;</a>
 			<?php
 			endif;
 			$desc_html = ob_get_clean();
 
-			Give()->notices->register_notice( array(
+			Give()->notices->register_notice(array(
 				'id'          => 'give_upgrade_db',
 				'type'        => 'error',
 				'dismissible' => false,
 				'description' => $desc_html,
-			) );
+			));
 		}
 
 		// Bailout if doing upgrades.
-		if ( $this->is_doing_updates() ) {
+		if ($this->is_doing_updates()) {
 			return;
 		}
 
 		// Show db upgrade completed notice.
-		if ( ! empty( $_GET['give-db-update-completed'] ) ) {
-			Give()->notices->register_notice( array(
+		if ( ! empty($_GET['give-db-update-completed'])) {
+			Give()->notices->register_notice(array(
 				'id'          => 'give_db_upgrade_completed',
 				'type'        => 'updated',
-				'description' => __( 'Give database updates completed successfully. Thank you for updating to the latest version!', 'give' ),
+				'description' => __('Give database updates completed successfully. Thank you for updating to the latest version!', 'give'),
 				'show'        => true,
-			) );
+			));
 
 			// Start update.
-		} elseif ( ! empty( $_GET['give-run-db-update'] ) ) {
+		} elseif ( ! empty($_GET['give-run-db-update'])) {
 			$this->run_db_update();
 
 			// Show run the update notice.
-		} elseif ( $this->get_total_new_db_update_count() ) {
+		} elseif ($this->get_total_new_db_update_count()) {
 			ob_start();
 			?>
 			<p>
-				<strong><?php _e( 'Database Update', 'give' ); ?></strong>
-				&nbsp;&#8211;&nbsp;<?php _e( 'GiveWP needs to update your database to the latest version. The following process will make updates to your site\'s database. Please create a complete backup before proceeding.', 'give' ); ?>
+				<strong><?php _e('Database Update', 'give'); ?></strong>
+				&nbsp;&#8211;&nbsp;<?php _e('GiveWP needs to update your database to the latest version. The following process will make updates to your site\'s database. Please create a complete backup before proceeding.', 'give'); ?>
 			</p>
 			<p class="submit">
-				<a href="<?php echo esc_url( add_query_arg( array( 'give-run-db-update' => 1 ), admin_url( 'edit.php?post_type=give_forms&page=give-updates' ) ) ); ?>" class="button button-primary give-run-update-now">
-					<?php _e( 'Run the updater', 'give' ); ?>
+				<a href="<?php echo esc_url(add_query_arg(array('give-run-db-update' => 1), admin_url('edit.php?post_type=give_forms&page=give-updates'))); ?>" class="button button-primary give-run-update-now">
+					<?php _e('Run the updater', 'give'); ?>
 				</a>
 			</p>
 			<?php
 			$desc_html = ob_get_clean();
 
-			Give()->notices->register_notice( array(
+			Give()->notices->register_notice(array(
 				'id'          => 'give_upgrade_db',
 				'type'        => 'updated',
 				'dismissible' => false,
 				'description' => $desc_html,
-			) );
+			));
 		}
 	}
 
@@ -681,7 +679,7 @@ 
 	 * @access public
 	 */
 	public function render_complete_page() {
-		include_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/views/upgrades-complete.php';
+		include_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/views/upgrades-complete.php';
 	}
 
 	/**
@@ -691,7 +689,7 @@ 
 	 * @access public
 	 */
 	public function render_page() {
-		include_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/views/upgrades.php';
+		include_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/views/upgrades.php';
 	}
 
 	/**
@@ -702,26 +700,26 @@ 
 	 */
 	private function run_db_update() {
 		// Bailout.
-		if ( $this->is_doing_updates() || ! $this->get_total_new_db_update_count() ) {
+		if ($this->is_doing_updates() || ! $this->get_total_new_db_update_count()) {
 			return;
 		}
 
-		$updates = $this->get_updates( 'database', 'new' );
+		$updates = $this->get_updates('database', 'new');
 
-		foreach ( $updates as $update ) {
-			self::$background_updater->push_to_queue( $update );
+		foreach ($updates as $update) {
+			self::$background_updater->push_to_queue($update);
 		}
 
-		add_option( 'give_db_update_count', count( $updates ), '', false );
+		add_option('give_db_update_count', count($updates), '', false);
 
-		add_option( 'give_doing_upgrade', array(
+		add_option('give_doing_upgrade', array(
 			'update_info'      => $updates[0],
 			'step'             => 1,
 			'update'           => 1,
-			'heading'          => sprintf( 'Update %s of %s', 1, count( $updates ) ),
+			'heading'          => sprintf('Update %s of %s', 1, count($updates)),
 			'percentage'       => 0,
 			'total_percentage' => 0,
-		), '', false );
+		), '', false);
 
 		self::$background_updater->save()->dispatch();
 	}
@@ -735,14 +733,13 @@ 
 	 */
 	public function __flush_resume_updates() {
 		//delete_option( 'give_doing_upgrade' );
-		update_option( 'give_version', preg_replace( '/[^0-9.].*/', '', GIVE_VERSION ), false );
+		update_option('give_version', preg_replace('/[^0-9.].*/', '', GIVE_VERSION), false);
 
 		// Reset counter.
 		$this->step = $this->percentage = 0;
 
-		$this->update = ( $this->get_total_db_update_count() > $this->update ) ?
-			( $this->update + 1 ) :
-			$this->update;
+		$this->update = ($this->get_total_db_update_count() > $this->update) ?
+			($this->update + 1) : $this->update;
 	}
 
 
@@ -757,7 +754,7 @@ 
 	public function __give_start_updating() {
 		// Check permission.
 		if (
-			! current_user_can( 'manage_give_settings' ) ||
+			! current_user_can('manage_give_settings') ||
 			$this->is_doing_updates()
 		) {
 			// Run update via ajax
@@ -768,7 +765,7 @@ 
 
 		// @todo: validate nonce
 		// @todo: set http method to post
-		if ( empty( $_POST['run_db_update'] ) ) {
+		if (empty($_POST['run_db_update'])) {
 			wp_send_json_error();
 		}
 
@@ -787,34 +784,34 @@ 
 	 * @return string
 	 */
 	public function __give_db_updates_info() {
-		$update_info   = get_option( 'give_doing_upgrade' );
+		$update_info   = get_option('give_doing_upgrade');
 		$response_type = '';
 
-		if ( self::$background_updater->is_paused_process() ) {
+		if (self::$background_updater->is_paused_process()) {
 			$update_info = array(
-				'message'    => __( 'The updates have been paused.', 'give' ),
+				'message'    => __('The updates have been paused.', 'give'),
 				'heading'    => '',
 				'percentage' => 0,
 			);
 
-			if ( get_option( 'give_upgrade_error' ) ) {
-				$update_info['message'] = __( 'An unexpected issue occurred during the database update which caused it to stop automatically. Please contact support for assistance.', 'give' );
+			if (get_option('give_upgrade_error')) {
+				$update_info['message'] = __('An unexpected issue occurred during the database update which caused it to stop automatically. Please contact support for assistance.', 'give');
 			}
 
 			$response_type = 'error';
 
-		} elseif ( empty( $update_info ) || ! $this->get_total_new_db_update_count( true ) ) {
+		} elseif (empty($update_info) || ! $this->get_total_new_db_update_count(true)) {
 			$update_info   = array(
-				'message'    => __( 'Give database updates completed successfully. Thank you for updating to the latest version!', 'give' ),
-				'heading'    => __( 'Updates Completed.', 'give' ),
+				'message'    => __('Give database updates completed successfully. Thank you for updating to the latest version!', 'give'),
+				'heading'    => __('Updates Completed.', 'give'),
 				'percentage' => 0,
 			);
 			$response_type = 'success';
 
-			delete_option( 'give_show_db_upgrade_complete_notice' );
+			delete_option('give_show_db_upgrade_complete_notice');
 		}
 
-		$this->send_ajax_response( $update_info, $response_type );
+		$this->send_ajax_response($update_info, $response_type);
 	}
 
 	/**
@@ -826,7 +823,7 @@ 
 	 * @param        $data
 	 * @param string $type
 	 */
-	public function send_ajax_response( $data, $type = '' ) {
+	public function send_ajax_response($data, $type = '') {
 		$default = array(
 			'message'    => '',
 			'heading'    => '',
@@ -836,24 +833,24 @@ 
 		);
 
 		// Set data.
-		$data = wp_parse_args( $data, $default );
+		$data = wp_parse_args($data, $default);
 
 		// Enable cache.
 		Give_Cache::enable();
 
-		switch ( $type ) {
+		switch ($type) {
 			case 'success':
-				wp_send_json_success( $data );
+				wp_send_json_success($data);
 				break;
 
 			case 'error':
-				wp_send_json_error( $data );
+				wp_send_json_error($data);
 				break;
 
 			default:
-				wp_send_json( array(
+				wp_send_json(array(
 					'data' => $data,
-				) );
+				));
 				break;
 		}
 	}
@@ -867,12 +864,12 @@ 
 	 * @param $total
 	 * @param $current_total
 	 */
-	public function set_percentage( $total, $current_total ) {
+	public function set_percentage($total, $current_total) {
 		// Set percentage.
-		$this->percentage = $total ? ( ( $current_total ) / $total ) * 100 : 0;
+		$this->percentage = $total ? (($current_total) / $total) * 100 : 0;
 
 		// Verify percentage.
-		$this->percentage = ( 100 < $this->percentage ) ? 100 : $this->percentage;
+		$this->percentage = (100 < $this->percentage) ? 100 : $this->percentage;
 	}
 
 	/**
@@ -885,22 +882,22 @@ 
 	 *
 	 * @return bool|null
 	 */
-	public function is_parent_updates_completed( $update ) {
+	public function is_parent_updates_completed($update) {
 		// Bailout.
-		if ( empty( $update['depend'] ) ) {
+		if (empty($update['depend'])) {
 			return true;
 		}
 
 		// Check if dependency is valid or not.
-		if ( ! $this->has_valid_dependency( $update ) ) {
+		if ( ! $this->has_valid_dependency($update)) {
 			return null;
 		}
 
 		$is_dependency_completed = true;
 
-		foreach ( $update['depend'] as $depend ) {
+		foreach ($update['depend'] as $depend) {
 
-			if ( ! give_has_upgrade_completed( $depend ) ) {
+			if ( ! give_has_upgrade_completed($depend)) {
 				$is_dependency_completed = false;
 				break;
 			}
@@ -917,7 +914,7 @@ 
 	 * @return bool
 	 */
 	public function is_doing_updates() {
-		return (bool) get_option( 'give_doing_upgrade' );
+		return (bool) get_option('give_doing_upgrade');
 	}
 
 
@@ -931,7 +928,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public function has_valid_dependency( $update ) {
+	public function has_valid_dependency($update) {
 		$is_valid_dependency = true;
 		// $update_ids          = wp_list_pluck( $this->get_updates( 'database', 'all' ), 'id' );
 		//
@@ -957,33 +954,33 @@ 
 	 *
 	 * @return array
 	 */
-	public function get_updates( $update_type = '', $status = 'all' ) {
+	public function get_updates($update_type = '', $status = 'all') {
 		// return all updates.
-		if ( empty( $update_type ) ) {
+		if (empty($update_type)) {
 			return $this->updates;
 		}
 
 		// Get specific update.
-		$updates = ! empty( $this->updates[ $update_type ] ) ? $this->updates[ $update_type ] : array();
+		$updates = ! empty($this->updates[$update_type]) ? $this->updates[$update_type] : array();
 
 		// Bailout.
-		if ( empty( $updates ) ) {
+		if (empty($updates)) {
 			return $updates;
 		}
 
-		switch ( $status ) {
+		switch ($status) {
 			case 'new':
 				// Remove already completed updates.
-				wp_cache_delete( 'give_completed_upgrades', 'options' );
+				wp_cache_delete('give_completed_upgrades', 'options');
 				$completed_updates = give_get_completed_upgrades();
 
-				if ( ! empty( $completed_updates ) ) {
-					foreach ( $updates as $index => $update ) {
-						if ( in_array( $update['id'], $completed_updates ) ) {
-							unset( $updates[ $index ] );
+				if ( ! empty($completed_updates)) {
+					foreach ($updates as $index => $update) {
+						if (in_array($update['id'], $completed_updates)) {
+							unset($updates[$index]);
 						}
 					}
-					$updates = array_values( $updates );
+					$updates = array_values($updates);
 				}
 
 				break;
@@ -1000,7 +997,7 @@ 
 	 * @return int
 	 */
 	public function get_total_plugin_update_count() {
-		return count( $this->get_updates( 'plugin' ) );
+		return count($this->get_updates('plugin'));
 	}
 
 	/**
@@ -1015,7 +1012,7 @@ 
 		$db_update_count     = $this->get_pending_db_update_count();
 		$plugin_update_count = $this->get_total_plugin_update_count();
 
-		return ( $db_update_count + $plugin_update_count );
+		return ($db_update_count + $plugin_update_count);
 	}
 
 	/**
@@ -1027,7 +1024,7 @@ 
 	 * @return int
 	 */
 	public function get_pending_db_update_count() {
-		return count( $this->get_updates( 'database', 'new' ) );
+		return count($this->get_updates('database', 'new'));
 	}
 
 	/**
@@ -1039,7 +1036,7 @@ 
 	 * @return int
 	 */
 	public function get_total_db_update_count() {
-		return count( $this->get_updates( 'database', 'all' ) );
+		return count($this->get_updates('database', 'all'));
 	}
 
 	/**
@@ -1052,10 +1049,9 @@ 
 	 *
 	 * @return int
 	 */
-	public function get_total_new_db_update_count( $refresh = false ) {
+	public function get_total_new_db_update_count($refresh = false) {
 		$update_count = $this->is_doing_updates() && ! $refresh ?
-			get_option( 'give_db_update_count' ) :
-			$this->get_pending_db_update_count();
+			get_option('give_db_update_count') : $this->get_pending_db_update_count();
 
 		return $update_count;
 	}
@@ -1070,11 +1066,11 @@ 
 	 *
 	 * @return int
 	 */
-	public function get_running_db_update( $refresh = false ) {
+	public function get_running_db_update($refresh = false) {
 		$current_update = 1;
 
-		if ( $this->is_doing_updates() && ! $refresh ) {
-			$current_update = get_option( 'give_doing_upgrade' );
+		if ($this->is_doing_updates() && ! $refresh) {
+			$current_update = get_option('give_doing_upgrade');
 			$current_update = $current_update['update'];
 		}
 
@@ -1091,25 +1087,23 @@ 
 	 *
 	 * @return float|int
 	 */
-	public function get_db_update_processing_percentage( $refresh = false ) {
+	public function get_db_update_processing_percentage($refresh = false) {
 		// Bailout.
-		if ( ! $this->get_total_new_db_update_count( $refresh ) ) {
+		if ( ! $this->get_total_new_db_update_count($refresh)) {
 			return 0;
 		}
 
-		$resume_update            = get_option( 'give_doing_upgrade' );
-		$update_count_percentages = ( ( $this->get_running_db_update( $refresh ) - 1 ) / $this->get_total_new_db_update_count( $refresh ) ) * 100;
-		$update_percentage_share  = ( 1 / $this->get_total_new_db_update_count() ) * 100;
-		$upgrade_percentage       = ( ( $resume_update['percentage'] * $update_percentage_share ) / 100 );
+		$resume_update            = get_option('give_doing_upgrade');
+		$update_count_percentages = (($this->get_running_db_update($refresh) - 1) / $this->get_total_new_db_update_count($refresh)) * 100;
+		$update_percentage_share  = (1 / $this->get_total_new_db_update_count()) * 100;
+		$upgrade_percentage       = (($resume_update['percentage'] * $update_percentage_share) / 100);
 
 		$final_percentage = $update_count_percentages + $upgrade_percentage;
 
 		return $this->is_doing_updates() ?
-			( absint( $final_percentage ) ?
-				absint( $final_percentage ) :
-				round( $final_percentage, 2 )
-			) :
-			0;
+			(absint($final_percentage) ?
+				absint($final_percentage) : round($final_percentage, 2)
+			) : 0;
 	}
 
 
@@ -1121,8 +1115,8 @@ 
 	 * @return array
 	 */
 	public function get_update_ids() {
-		$all_updates    = $this->get_updates( 'database', 'all' );
-		$all_update_ids = wp_list_pluck( $all_updates, 'id' );
+		$all_updates    = $this->get_updates('database', 'all');
+		$all_update_ids = wp_list_pluck($all_updates, 'id');
 
 		return $all_update_ids;
 	}
@@ -1137,10 +1131,9 @@ 
 	 *
 	 * @return float|int
 	 */
-	public function get_offset( $process_item_count ) {
-		return ( 1 === $this->step ) ?
-			0 :
-			( $this->step - 1 ) * $process_item_count;
+	public function get_offset($process_item_count) {
+		return (1 === $this->step) ?
+			0 : ($this->step - 1) * $process_item_count;
 	}
 }
 

includes/admin/shortcodes/shortcode-give-receipt.php

Spacing +27 added lines, -27 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,10 +24,10 @@ 
 	 */
 	public function __construct() {
 
-		$this->shortcode['title'] = esc_html__( 'Donation Receipt', 'give' );
-		$this->shortcode['label'] = esc_html__( 'Donation Receipt', 'give' );
+		$this->shortcode['title'] = esc_html__('Donation Receipt', 'give');
+		$this->shortcode['label'] = esc_html__('Donation Receipt', 'give');
 
-		parent::__construct( 'give_receipt' );
+		parent::__construct('give_receipt');
 	}
 
 	/**
@@ -40,71 +40,71 @@ 
 		return array(
 			array(
 				'type' => 'container',
-				'html' => sprintf( '<p class="strong">%s</p>', esc_html__( 'Optional settings', 'give' ) ),
+				'html' => sprintf('<p class="strong">%s</p>', esc_html__('Optional settings', 'give')),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'price',
-				'label'   => esc_html__( 'Show Donation Amount:', 'give' ),
+				'label'   => esc_html__('Show Donation Amount:', 'give'),
 				'options' => array(
-					'false' => esc_html__( 'Hide', 'give' ),
+					'false' => esc_html__('Hide', 'give'),
 				),
-				'placeholder' => esc_html__( 'Show', 'give' )
+				'placeholder' => esc_html__('Show', 'give')
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'donor',
-				'label'   => esc_html__( 'Show Donor Name:', 'give' ),
+				'label'   => esc_html__('Show Donor Name:', 'give'),
 				'options' => array(
-					'true'  => esc_html__( 'Show', 'give' ),
-					'false' => esc_html__( 'Hide', 'give' ),
+					'true'  => esc_html__('Show', 'give'),
+					'false' => esc_html__('Hide', 'give'),
 				),
-				'placeholder' => esc_html__( 'Show', 'give' )
+				'placeholder' => esc_html__('Show', 'give')
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'date',
-				'label'   => esc_html__( 'Show Date:', 'give' ),
+				'label'   => esc_html__('Show Date:', 'give'),
 				'options' => array(
-					'false' => esc_html__( 'Hide', 'give' ),
+					'false' => esc_html__('Hide', 'give'),
 				),
-				'placeholder' => esc_html__( 'Show', 'give' ),
+				'placeholder' => esc_html__('Show', 'give'),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'payment_key',
-				'label'   => esc_html__( 'Show Payment Key:', 'give' ),
+				'label'   => esc_html__('Show Payment Key:', 'give'),
 				'options' => array(
-					'true'  => esc_html__( 'Show', 'give' ),
+					'true'  => esc_html__('Show', 'give'),
 				),
-				'placeholder' => esc_html__( 'Hide', 'give' ),
+				'placeholder' => esc_html__('Hide', 'give'),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'payment_method',
-				'label'   => esc_html__( 'Show Payment Method:', 'give' ),
+				'label'   => esc_html__('Show Payment Method:', 'give'),
 				'options' => array(
-					'false' => esc_html__( 'Hide', 'give' ),
+					'false' => esc_html__('Hide', 'give'),
 				),
-				'placeholder' => esc_html__( 'Show', 'give' ),
+				'placeholder' => esc_html__('Show', 'give'),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'payment_id',
-				'label'   => esc_html__( 'Show Payment ID:', 'give' ),
+				'label'   => esc_html__('Show Payment ID:', 'give'),
 				'options' => array(
-					'false' => esc_html__( 'Hide', 'give' ),
+					'false' => esc_html__('Hide', 'give'),
 				),
-				'placeholder' => esc_html__( 'Show', 'give' ),
+				'placeholder' => esc_html__('Show', 'give'),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'company_name',
-				'label'   => esc_html__( 'Company Name:', 'give' ),
+				'label'   => esc_html__('Company Name:', 'give'),
 				'options' => array(
-					'true' => esc_html__( 'Show', 'give' ),
+					'true' => esc_html__('Show', 'give'),
 				),
-				'placeholder' => esc_html__( 'Hide', 'give' )
+				'placeholder' => esc_html__('Hide', '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;