WordImpress / Give

includes/class-give-db-payment-meta.php

Spacing +3 added lines, -3 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -50,7 +50,7 @@ 
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
-		$wpdb->paymentmeta = $this->table_name = $wpdb->prefix . 'give_paymentmeta';
+		$wpdb->paymentmeta = $this->table_name = $wpdb->prefix.'give_paymentmeta';
 		$this->primary_key = 'meta_id';
 		$this->version     = '1.0';
 
@@ -84,6 +84,6 @@ 
 	 * @return bool
 	 */
 	protected function is_custom_meta_table_active() {
-		return give_has_upgrade_completed( 'v20_move_metadata_into_new_table' );
+		return give_has_upgrade_completed('v20_move_metadata_into_new_table');
 	}
 }

includes/class-give-db-logs-meta.php

Spacing +7 added lines, -7 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -50,7 +50,7 @@ 
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
-		$wpdb->logmeta     = $this->table_name = $wpdb->prefix . 'give_logmeta';
+		$wpdb->logmeta     = $this->table_name = $wpdb->prefix.'give_logmeta';
 		$this->primary_key = 'meta_id';
 		$this->version     = '1.0';
 
@@ -86,18 +86,18 @@ 
 	 *
 	 * @return bool
 	 */
-	public function delete_row( $log_id = 0 ) {
+	public function delete_row($log_id = 0) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Row ID must be positive integer
-		$log_id = absint( $log_id );
+		$log_id = absint($log_id);
 
-		if ( empty( $log_id ) ) {
+		if (empty($log_id)) {
 			return false;
 		}
 
-		if ( false === $wpdb->query( $wpdb->prepare( "DELETE FROM $this->table_name WHERE log_id = %d", $log_id ) ) ) {
+		if (false === $wpdb->query($wpdb->prepare("DELETE FROM $this->table_name WHERE log_id = %d", $log_id))) {
 			return false;
 		}
 
@@ -114,7 +114,7 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function is_valid_post_type( $ID ) {
+	protected function is_valid_post_type($ID) {
 		return $ID && true;
 	}
 }

includes/admin/tools/data/class-give-tools-recount-donor-stats.php

Spacing +37 added lines, -37 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -48,8 +48,8 @@ 
 	/**
 	 * Constructor.
 	 */
-	public function __construct( $_step = 1 ) {
-		parent::__construct( $_step );
+	public function __construct($_step = 1) {
+		parent::__construct($_step);
 
 		$this->is_writable = true;
 	}
@@ -66,32 +66,32 @@ 
 
 		$args = array(
 			'number'  => $this->per_step,
-			'offset'  => $this->per_step * ( $this->step - 1 ),
+			'offset'  => $this->per_step * ($this->step - 1),
 			'orderby' => 'id',
 			'order'   => 'DESC',
 		);
 
-		$donors = Give()->donors->get_donors( $args );
+		$donors = Give()->donors->get_donors($args);
 
-		if ( $donors ) {
+		if ($donors) {
 
-			$allowed_payment_status = apply_filters( 'give_recount_donors_donation_statuses', give_get_payment_status_keys() );
+			$allowed_payment_status = apply_filters('give_recount_donors_donation_statuses', give_get_payment_status_keys());
 
-			foreach ( $donors as $donor ) {
+			foreach ($donors as $donor) {
 
-				$attached_payment_ids = explode( ',', $donor->payment_ids );
+				$attached_payment_ids = explode(',', $donor->payment_ids);
 
 				$attached_args = array(
 					'post__in' => $attached_payment_ids,
-					'number'   => - 1,
+					'number'   => -1,
 					'status'   => $allowed_payment_status,
 				);
 
-				$attached_payments = (array) give_get_payments( $attached_args );
+				$attached_payments = (array) give_get_payments($attached_args);
 
 				$unattached_args = array(
 					'post__not_in' => $attached_payment_ids,
-					'number'       => - 1,
+					'number'       => -1,
 					'status'       => $allowed_payment_status,
 					'meta_query'   => array(
 						array(
@@ -102,29 +102,29 @@ 
 					),
 				);
 
-				$unattached_payments = give_get_payments( $unattached_args );
+				$unattached_payments = give_get_payments($unattached_args);
 
-				$payments = array_merge( $attached_payments, $unattached_payments );
+				$payments = array_merge($attached_payments, $unattached_payments);
 
 				$purchase_value = 0.00;
 				$purchase_count = 0;
 				$payment_ids    = array();
 
-				if ( $payments ) {
+				if ($payments) {
 
-					foreach ( $payments as $payment ) {
+					foreach ($payments as $payment) {
 
 						$should_process_payment = 'publish' == $payment->post_status ? true : false;
-						$should_process_payment = apply_filters( 'give_donor_recount_should_process_donation', $should_process_payment, $payment );
+						$should_process_payment = apply_filters('give_donor_recount_should_process_donation', $should_process_payment, $payment);
 
-						if ( true === $should_process_payment ) {
+						if (true === $should_process_payment) {
 
-							if ( apply_filters( 'give_donor_recount_should_increase_value', true, $payment ) ) {
-								$purchase_value += (float) give_donation_amount( $payment->ID, array( 'type' => 'stats' ) );
+							if (apply_filters('give_donor_recount_should_increase_value', true, $payment)) {
+								$purchase_value += (float) give_donation_amount($payment->ID, array('type' => 'stats'));
 							}
 
-							if ( apply_filters( 'give_donor_recount_should_increase_count', true, $payment ) ) {
-								$purchase_count ++;
+							if (apply_filters('give_donor_recount_should_increase_count', true, $payment)) {
+								$purchase_count++;
 							}
 						}
 
@@ -132,7 +132,7 @@ 
 					}
 				}
 
-				$payment_ids = implode( ',', $payment_ids );
+				$payment_ids = implode(',', $payment_ids);
 
 				$donor_update_data = array(
 					'purchase_count' => $purchase_count,
@@ -140,8 +140,8 @@ 
 					'payment_ids'    => $payment_ids,
 				);
 
-				$donor_instance = new Give_Donor( $donor->id );
-				$donor_instance->update( $donor_update_data );
+				$donor_instance = new Give_Donor($donor->id);
+				$donor_instance->update($donor_update_data);
 
 			}// End foreach().
 
@@ -161,21 +161,21 @@ 
 	public function get_percentage_complete() {
 
 		$args = array(
-			'number'  => - 1,
+			'number'  => -1,
 			'orderby' => 'id',
 			'order'   => 'DESC',
 		);
 
-		$donors = Give()->donors->get_donors( $args );
-		$total     = count( $donors );
+		$donors = Give()->donors->get_donors($args);
+		$total = count($donors);
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( $this->per_step * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = (($this->per_step * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -189,7 +189,7 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 	}
 
 	/**
@@ -200,21 +200,21 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( esc_html__( 'You do not have permission to recount stats.', 'give' ), esc_html__( 'Error', 'give' ), array(
+		if ( ! $this->can_export()) {
+			wp_die(esc_html__('You do not have permission to recount stats.', 'give'), esc_html__('Error', 'give'), array(
 				'response' => 403,
-			) );
+			));
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
 			$this->done    = true;
-			$this->message = esc_html__( 'Donor stats have been successfully recounted.', 'give' );
+			$this->message = esc_html__('Donor stats have been successfully recounted.', 'give');
 
 			return false;
 		}

includes/admin/tools/data/class-give-tools-recount-single-donor-stats.php

Spacing +73 added lines, -73 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -46,8 +46,8 @@ 
 	/**
 	 * Constructor.
 	 */
-	public function __construct( $_step = 1 ) {
-		parent::__construct( $_step );
+	public function __construct($_step = 1) {
+		parent::__construct($_step);
 
 		$this->is_writable = true;
 	}
@@ -62,40 +62,40 @@ 
 	 */
 	public function get_data() {
 
-		$donor    = new Give_Donor( $this->customer_id );
-		$payments = $this->get_stored_data( 'give_recount_donor_payments_' . $donor->id );
+		$donor    = new Give_Donor($this->customer_id);
+		$payments = $this->get_stored_data('give_recount_donor_payments_'.$donor->id);
 
-		$offset     = ( $this->step - 1 ) * $this->per_step;
-		$step_items = array_slice( $payments, $offset, $this->per_step );
+		$offset     = ($this->step - 1) * $this->per_step;
+		$step_items = array_slice($payments, $offset, $this->per_step);
 
-		if ( count( $step_items ) > 0 ) {
-			$pending_total = (float) $this->get_stored_data( 'give_stats_donor_pending_total' . $donor->id );
+		if (count($step_items) > 0) {
+			$pending_total = (float) $this->get_stored_data('give_stats_donor_pending_total'.$donor->id);
 			$step_total    = 0;
 
-			$found_payment_ids = $this->get_stored_data( 'give_stats_found_payments_' . $donor->id );
+			$found_payment_ids = $this->get_stored_data('give_stats_found_payments_'.$donor->id);
 
-			foreach ( $step_items as $payment ) {
-				$payment = get_post( $payment->ID );
+			foreach ($step_items as $payment) {
+				$payment = get_post($payment->ID);
 
-				if ( is_null( $payment ) || is_wp_error( $payment ) || 'give_payment' !== $payment->post_type ) {
+				if (is_null($payment) || is_wp_error($payment) || 'give_payment' !== $payment->post_type) {
 
-					$missing_payments   = $this->get_stored_data( 'give_stats_missing_payments' . $donor->id );
+					$missing_payments   = $this->get_stored_data('give_stats_missing_payments'.$donor->id);
 					$missing_payments[] = $payment->ID;
-					$this->store_data( 'give_stats_missing_payments' . $donor->id, $missing_payments );
+					$this->store_data('give_stats_missing_payments'.$donor->id, $missing_payments);
 
 					continue;
 				}
 
 				$should_process_payment = 'publish' == $payment->post_status ? true : false;
-				$should_process_payment = apply_filters( 'give_donor_recount_should_process_donation', $should_process_payment, $payment );
+				$should_process_payment = apply_filters('give_donor_recount_should_process_donation', $should_process_payment, $payment);
 
-				if ( true === $should_process_payment ) {
+				if (true === $should_process_payment) {
 
 					$found_payment_ids[] = $payment->ID;
 
-					if ( apply_filters( 'give_donor_recount_should_increase_value', true, $payment ) ) {
-						$payment_amount = (float) give_donation_amount( $payment->ID, array( 'type' => 'stats' ) );
-						$step_total     += $payment_amount;
+					if (apply_filters('give_donor_recount_should_increase_value', true, $payment)) {
+						$payment_amount = (float) give_donation_amount($payment->ID, array('type' => 'stats'));
+						$step_total += $payment_amount;
 					}
 
 				}
@@ -103,8 +103,8 @@ 
 			}
 
 			$updated_total = $pending_total + $step_total;
-			$this->store_data( 'give_stats_donor_pending_total' . $donor->id, $updated_total );
-			$this->store_data( 'give_stats_found_payments_' . $donor->id, $found_payment_ids );
+			$this->store_data('give_stats_donor_pending_total'.$donor->id, $updated_total);
+			$this->store_data('give_stats_found_payments_'.$donor->id, $found_payment_ids);
 
 			return true;
 		}
@@ -121,16 +121,16 @@ 
 	 */
 	public function get_percentage_complete() {
 
-		$payments = $this->get_stored_data( 'give_recount_donor_payments_' . $this->customer_id );
-		$total    = count( $payments );
+		$payments = $this->get_stored_data('give_recount_donor_payments_'.$this->customer_id);
+		$total    = count($payments);
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( $this->per_step * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = (($this->per_step * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -144,8 +144,8 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
-		$this->customer_id = isset( $request['customer_id'] ) ? sanitize_text_field( $request['customer_id'] ) : false;
+	public function set_properties($request) {
+		$this->customer_id = isset($request['customer_id']) ? sanitize_text_field($request['customer_id']) : false;
 	}
 
 	/**
@@ -156,53 +156,53 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( esc_html__( 'You do not have permission to recount stats.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(esc_html__('You do not have permission to recount stats.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
-			$donor       = new Give_Donor( $this->customer_id );
-			$payment_ids = get_option( 'give_stats_found_payments_' . $donor->id, array() );
-			$this->delete_data( 'give_stats_found_payments_' . $donor->id );
+			$donor       = new Give_Donor($this->customer_id);
+			$payment_ids = get_option('give_stats_found_payments_'.$donor->id, array());
+			$this->delete_data('give_stats_found_payments_'.$donor->id);
 
-			$removed_payments = array_unique( get_option( 'give_stats_missing_payments' . $donor->id, array() ) );
+			$removed_payments = array_unique(get_option('give_stats_missing_payments'.$donor->id, array()));
 
 			// Find non-existing payments (deleted) and total up the donation count
 			$purchase_count = 0;
-			foreach ( $payment_ids as $key => $payment_id ) {
-				if ( in_array( $payment_id, $removed_payments ) ) {
-					unset( $payment_ids[ $key ] );
+			foreach ($payment_ids as $key => $payment_id) {
+				if (in_array($payment_id, $removed_payments)) {
+					unset($payment_ids[$key]);
 					continue;
 				}
 
-				$payment = get_post( $payment_id );
-				if ( apply_filters( 'give_donor_recount_should_increase_count', true, $payment ) ) {
-					$purchase_count ++;
+				$payment = get_post($payment_id);
+				if (apply_filters('give_donor_recount_should_increase_count', true, $payment)) {
+					$purchase_count++;
 				}
 			}
 
-			$this->delete_data( 'give_stats_missing_payments' . $donor->id );
+			$this->delete_data('give_stats_missing_payments'.$donor->id);
 
-			$pending_total = $this->get_stored_data( 'give_stats_donor_pending_total' . $donor->id );
-			$this->delete_data( 'give_stats_donor_pending_total' . $donor->id );
-			$this->delete_data( 'give_recount_donor_stats_' . $donor->id );
-			$this->delete_data( 'give_recount_donor_payments_' . $this->customer_id );
+			$pending_total = $this->get_stored_data('give_stats_donor_pending_total'.$donor->id);
+			$this->delete_data('give_stats_donor_pending_total'.$donor->id);
+			$this->delete_data('give_recount_donor_stats_'.$donor->id);
+			$this->delete_data('give_recount_donor_payments_'.$this->customer_id);
 
-			$payment_ids = implode( ',', $payment_ids );
-			$donor->update( array(
+			$payment_ids = implode(',', $payment_ids);
+			$donor->update(array(
 				'payment_ids'    => $payment_ids,
 				'purchase_count' => $purchase_count,
 				'purchase_value' => $pending_total,
-			) );
+			));
 
 			$this->done    = true;
-			$this->message = esc_html__( 'Donor stats have been successfully recounted.', 'give' );
+			$this->message = esc_html__('Donor stats have been successfully recounted.', 'give');
 
 			return false;
 		}
@@ -235,26 +235,26 @@ 
 	 * @return void
 	 */
 	public function pre_fetch() {
-		if ( $this->step === 1 ) {
-			$allowed_payment_status = apply_filters( 'give_recount_donors_donation_statuses', give_get_payment_status_keys() );
+		if ($this->step === 1) {
+			$allowed_payment_status = apply_filters('give_recount_donors_donation_statuses', give_get_payment_status_keys());
 
 			// Before we start, let's zero out the customer's data
-			$donor = new Give_Donor( $this->customer_id );
-			$donor->update( array( 'purchase_value' => give_format_amount( 0, array( 'sanitize' => false ) ), 'purchase_count' => 0 ) );
+			$donor = new Give_Donor($this->customer_id);
+			$donor->update(array('purchase_value' => give_format_amount(0, array('sanitize' => false)), 'purchase_count' => 0));
 
-			$attached_payment_ids = explode( ',', $donor->payment_ids );
+			$attached_payment_ids = explode(',', $donor->payment_ids);
 
 			$attached_args = array(
 				'post__in' => $attached_payment_ids,
-				'number'   => - 1,
+				'number'   => -1,
 				'status'   => $allowed_payment_status,
 			);
 
-			$attached_payments = give_get_payments( $attached_args );
+			$attached_payments = give_get_payments($attached_args);
 
 			$unattached_args = array(
 				'post__not_in' => $attached_payment_ids,
-				'number'       => - 1,
+				'number'       => -1,
 				'status'       => $allowed_payment_status,
 				'meta_query'   => array(
 					array(
@@ -264,11 +264,11 @@ 
 				),
 			);
 
-			$unattached_payments = give_get_payments( $unattached_args );
+			$unattached_payments = give_get_payments($unattached_args);
 
-			$payments = array_merge( $attached_payments, $unattached_payments );
+			$payments = array_merge($attached_payments, $unattached_payments);
 
-			$this->store_data( 'give_recount_donor_payments_' . $donor->id, $payments );
+			$this->store_data('give_recount_donor_payments_'.$donor->id, $payments);
 		}
 	}
 
@@ -281,17 +281,17 @@ 
 	 *
 	 * @return mixed       Returns the data from the database
 	 */
-	private function get_stored_data( $key ) {
+	private function get_stored_data($key) {
 		global $wpdb;
-		$value = $wpdb->get_var( $wpdb->prepare( "SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key ) );
+		$value = $wpdb->get_var($wpdb->prepare("SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key));
 
-		if ( empty( $value ) ) {
+		if (empty($value)) {
 			return false;
 		}
 
-		$maybe_json = json_decode( $value );
-		if ( ! is_null( $maybe_json ) ) {
-			$value = json_decode( $value, true );
+		$maybe_json = json_decode($value);
+		if ( ! is_null($maybe_json)) {
+			$value = json_decode($value, true);
 		}
 
 		return $value;
@@ -307,10 +307,10 @@ 
 	 *
 	 * @return void
 	 */
-	private function store_data( $key, $value ) {
+	private function store_data($key, $value) {
 		global $wpdb;
 
-		$value = is_array( $value ) ? wp_json_encode( $value ) : esc_attr( $value );
+		$value = is_array($value) ? wp_json_encode($value) : esc_attr($value);
 
 		$data = array(
 			'option_name'  => $key,
@@ -324,7 +324,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -336,9 +336,9 @@ 
 	 *
 	 * @return void
 	 */
-	private function delete_data( $key ) {
+	private function delete_data($key) {
 		global $wpdb;
-		$wpdb->delete( $wpdb->options, array( 'option_name' => $key ) );
+		$wpdb->delete($wpdb->options, array('option_name' => $key));
 	}
 
 }

includes/template-functions.php

Spacing +89 added lines, -89 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,7 +21,7 @@ 
  * @return string
  */
 function give_get_templates_dir() {
-	return GIVE_PLUGIN_DIR . 'templates';
+	return GIVE_PLUGIN_DIR.'templates';
 }
 
 /**
@@ -31,7 +31,7 @@ 
  * @return string
  */
 function give_get_templates_url() {
-	return GIVE_PLUGIN_URL . 'templates';
+	return GIVE_PLUGIN_URL.'templates';
 }
 
 /**
@@ -44,24 +44,24 @@ 
  * @param string $template_path Template file path. Default is empty.
  * @param string $default_path  Default path. Default is empty.
  */
-function give_get_template( $template_name, $args = array(), $template_path = '', $default_path = '' ) {
-	if ( ! empty( $args ) && is_array( $args ) ) {
-		extract( $args );
+function give_get_template($template_name, $args = array(), $template_path = '', $default_path = '') {
+	if ( ! empty($args) && is_array($args)) {
+		extract($args);
 	}
 
 	$template_names = "{$template_name}.php";
 
-	$located = give_get_locate_template( $template_names, $template_path, $default_path );
+	$located = give_get_locate_template($template_names, $template_path, $default_path);
 
-	if ( ! file_exists( $located ) ) {
+	if ( ! file_exists($located)) {
 		/* translators: %s: the template */
-		Give()->notices->print_frontend_notice( sprintf( __( 'The %s template was not found.', 'give' ), $located ), true );
+		Give()->notices->print_frontend_notice(sprintf(__('The %s template was not found.', 'give'), $located), true);
 
 		return;
 	}
 
 	// Allow 3rd party plugin filter template file from their plugin.
-	$located = apply_filters( 'give_get_template', $located, $template_name, $args, $template_path, $default_path );
+	$located = apply_filters('give_get_template', $located, $template_name, $args, $template_path, $default_path);
 
 	/**
 	 * Fires in give template, before the file is included.
@@ -75,9 +75,9 @@ 
 	 * @param string $located       Template file filter by 3rd party plugin.
 	 * @param array  $args          Passed arguments.
 	 */
-	do_action( 'give_before_template_part', $template_name, $template_path, $located, $args );
+	do_action('give_before_template_part', $template_name, $template_path, $located, $args);
 
-	include( $located );
+	include($located);
 
 	/**
 	 * Fires in give template, after the file is included.
@@ -91,7 +91,7 @@ 
 	 * @param string $located       Template file filter by 3rd party plugin.
 	 * @param array  $args          Passed arguments.
 	 */
-	do_action( 'give_after_template_part', $template_name, $template_path, $located, $args );
+	do_action('give_after_template_part', $template_name, $template_path, $located, $args);
 }
 
 /**
@@ -107,7 +107,7 @@ 
  *
  * @return string
  */
-function give_get_template_part( $slug, $name = null, $load = true ) {
+function give_get_template_part($slug, $name = null, $load = true) {
 
 	/**
 	 * Fires in give template part, before the template part is retrieved.
@@ -119,20 +119,20 @@ 
 	 * @param string $slug Template part file slug {slug}.php.
 	 * @param string $name Template part file name {slug}-{name}.php.
 	 */
-	do_action( "get_template_part_{$slug}", $slug, $name );
+	do_action("get_template_part_{$slug}", $slug, $name);
 
 	// Setup possible parts
 	$templates = array();
-	if ( isset( $name ) ) {
-		$templates[] = $slug . '-' . $name . '.php';
+	if (isset($name)) {
+		$templates[] = $slug.'-'.$name.'.php';
 	}
-	$templates[] = $slug . '.php';
+	$templates[] = $slug.'.php';
 
 	// Allow template parts to be filtered
-	$templates = apply_filters( 'give_get_template_part', $templates, $slug, $name );
+	$templates = apply_filters('give_get_template_part', $templates, $slug, $name);
 
 	// Return the part that is found
-	return give_locate_template( $templates, $load, false );
+	return give_locate_template($templates, $load, false);
 }
 
 /**
@@ -153,37 +153,37 @@ 
  *
  * @return string The template filename if one is located.
  */
-function give_locate_template( $template_names, $load = false, $require_once = true ) {
+function give_locate_template($template_names, $load = false, $require_once = true) {
 	// No file found yet
 	$located = false;
 
 	// Try to find a template file
-	foreach ( (array) $template_names as $template_name ) {
+	foreach ((array) $template_names as $template_name) {
 
 		// Continue if template is empty
-		if ( empty( $template_name ) ) {
+		if (empty($template_name)) {
 			continue;
 		}
 
 		// Trim off any slashes from the template name
-		$template_name = ltrim( $template_name, '/' );
+		$template_name = ltrim($template_name, '/');
 
 		// try locating this template file by looping through the template paths
-		foreach ( give_get_theme_template_paths() as $template_path ) {
+		foreach (give_get_theme_template_paths() as $template_path) {
 
-			if ( file_exists( $template_path . $template_name ) ) {
-				$located = $template_path . $template_name;
+			if (file_exists($template_path.$template_name)) {
+				$located = $template_path.$template_name;
 				break;
 			}
 		}
 
-		if ( $located ) {
+		if ($located) {
 			break;
 		}
 	}
 
-	if ( ( true == $load ) && ! empty( $located ) ) {
-		load_template( $located, $require_once );
+	if ((true == $load) && ! empty($located)) {
+		load_template($located, $require_once);
 	}
 
 	return $located;
@@ -207,26 +207,26 @@ 
  *
  * @return string
  */
-function give_get_locate_template( $template_name, $template_path = '', $default_path = '' ) {
-	if ( ! $template_path ) {
-		$template_path = give_get_theme_template_dir_name() . '/';
+function give_get_locate_template($template_name, $template_path = '', $default_path = '') {
+	if ( ! $template_path) {
+		$template_path = give_get_theme_template_dir_name().'/';
 	}
 
-	if ( ! $default_path ) {
-		$default_path = GIVE_PLUGIN_DIR . 'templates/';
+	if ( ! $default_path) {
+		$default_path = GIVE_PLUGIN_DIR.'templates/';
 	}
 
 	// Look within passed path within the theme - this is priority.
 	$template = locate_template(
 		array(
-			trailingslashit( $template_path ) . $template_name,
+			trailingslashit($template_path).$template_name,
 			$template_name,
 		)
 	);
 
 	// Get default template/
-	if ( ! $template ) {
-		$template = $default_path . $template_name;
+	if ( ! $template) {
+		$template = $default_path.$template_name;
 	}
 
 	/**
@@ -234,7 +234,7 @@ 
 	 *
 	 * @since 2.0.3
 	 */
-	return apply_filters( 'give_get_locate_template', $template, $template_name, $template_path );
+	return apply_filters('give_get_locate_template', $template, $template_name, $template_path);
 }
 
 /**
@@ -248,17 +248,17 @@ 
 	$template_dir = give_get_theme_template_dir_name();
 
 	$file_paths = array(
-		1   => trailingslashit( get_stylesheet_directory() ) . $template_dir,
-		10  => trailingslashit( get_template_directory() ) . $template_dir,
+		1   => trailingslashit(get_stylesheet_directory()).$template_dir,
+		10  => trailingslashit(get_template_directory()).$template_dir,
 		100 => give_get_templates_dir(),
 	);
 
-	$file_paths = apply_filters( 'give_template_paths', $file_paths );
+	$file_paths = apply_filters('give_template_paths', $file_paths);
 
 	// sort the file paths based on priority
-	ksort( $file_paths, SORT_NUMERIC );
+	ksort($file_paths, SORT_NUMERIC);
 
-	return array_map( 'trailingslashit', $file_paths );
+	return array_map('trailingslashit', $file_paths);
 }
 
 /**
@@ -270,7 +270,7 @@ 
  * @return string
  */
 function give_get_theme_template_dir_name() {
-	return trailingslashit( apply_filters( 'give_templates_dir', 'give' ) );
+	return trailingslashit(apply_filters('give_templates_dir', 'give'));
 }
 
 /**
@@ -280,10 +280,10 @@ 
  * @return void
  */
 function give_version_in_header() {
-	echo '<meta name="generator" content="Give v' . GIVE_VERSION . '" />' . "\n";
+	echo '<meta name="generator" content="Give v'.GIVE_VERSION.'" />'."\n";
 }
 
-add_action( 'wp_head', 'give_version_in_header' );
+add_action('wp_head', 'give_version_in_header');
 
 /**
  * Determines if we're currently on the Donations History page.
@@ -293,9 +293,9 @@ 
  */
 function give_is_donation_history_page() {
 
-	$ret = is_page( give_get_option( 'history_page' ) );
+	$ret = is_page(give_get_option('history_page'));
 
-	return apply_filters( 'give_is_donation_history_page', $ret );
+	return apply_filters('give_is_donation_history_page', $ret);
 }
 
 /**
@@ -307,25 +307,25 @@ 
  *
  * @return array Modified array of classes
  */
-function give_add_body_classes( $class ) {
+function give_add_body_classes($class) {
 	$classes = (array) $class;
 
-	if ( give_is_success_page() ) {
+	if (give_is_success_page()) {
 		$classes[] = 'give-success';
 		$classes[] = 'give-page';
 	}
 
-	if ( give_is_failed_transaction_page() ) {
+	if (give_is_failed_transaction_page()) {
 		$classes[] = 'give-failed-transaction';
 		$classes[] = 'give-page';
 	}
 
-	if ( give_is_donation_history_page() ) {
+	if (give_is_donation_history_page()) {
 		$classes[] = 'give-donation-history';
 		$classes[] = 'give-page';
 	}
 
-	if ( give_is_test_mode() ) {
+	if (give_is_test_mode()) {
 		$classes[] = 'give-test-mode';
 		$classes[] = 'give-page';
 	}
@@ -334,7 +334,7 @@ 
 	/* @var WP_Theme $current_theme */
 	$current_theme = wp_get_theme();
 
-	switch ( $current_theme->get_template() ) {
+	switch ($current_theme->get_template()) {
 
 		case 'Divi':
 			$classes[] = 'give-divi';
@@ -351,10 +351,10 @@ 
 
 	}
 
-	return array_unique( $classes );
+	return array_unique($classes);
 }
 
-add_filter( 'body_class', 'give_add_body_classes' );
+add_filter('body_class', 'give_add_body_classes');
 
 
 /**
@@ -370,22 +370,22 @@ 
  *
  * @return array
  */
-function give_add_post_class( $classes, $class = '', $post_id = '' ) {
-	if ( ! $post_id || 'give_forms' !== get_post_type( $post_id ) ) {
+function give_add_post_class($classes, $class = '', $post_id = '') {
+	if ( ! $post_id || 'give_forms' !== get_post_type($post_id)) {
 		return $classes;
 	}
 
 	//@TODO: Add classes for custom taxonomy and form configurations (multi vs single donations, etc).
 
-	if ( false !== ( $key = array_search( 'hentry', $classes ) ) ) {
-		unset( $classes[ $key ] );
+	if (false !== ($key = array_search('hentry', $classes))) {
+		unset($classes[$key]);
 	}
 
 	return $classes;
 }
 
 
-add_filter( 'post_class', 'give_add_post_class', 20, 3 );
+add_filter('post_class', 'give_add_post_class', 20, 3);
 
 /**
  * Get the placeholder image URL for forms etc
@@ -395,74 +395,74 @@ 
  */
 function give_get_placeholder_img_src() {
 
-	$placeholder_url = '//placehold.it/600x600&text=' . urlencode( esc_attr__( 'Give Placeholder Image', 'give' ) );
+	$placeholder_url = '//placehold.it/600x600&text='.urlencode(esc_attr__('Give Placeholder Image', 'give'));
 
-	return apply_filters( 'give_placeholder_img_src', $placeholder_url );
+	return apply_filters('give_placeholder_img_src', $placeholder_url);
 }
 
 
 /**
  * Global
  */
-if ( ! function_exists( 'give_output_content_wrapper' ) ) {
+if ( ! function_exists('give_output_content_wrapper')) {
 
 	/**
 	 * Output the start of the page wrapper.
 	 */
 	function give_output_content_wrapper() {
-		give_get_template_part( 'global/wrapper-start' );
+		give_get_template_part('global/wrapper-start');
 	}
 }
-if ( ! function_exists( 'give_output_content_wrapper_end' ) ) {
+if ( ! function_exists('give_output_content_wrapper_end')) {
 
 	/**
 	 * Output the end of the page wrapper.
 	 */
 	function give_output_content_wrapper_end() {
-		give_get_template_part( 'global/wrapper-end' );
+		give_get_template_part('global/wrapper-end');
 	}
 }
 
 /**
  * Single Give Form
  */
-if ( ! function_exists( 'give_left_sidebar_pre_wrap' ) ) {
+if ( ! function_exists('give_left_sidebar_pre_wrap')) {
 	function give_left_sidebar_pre_wrap() {
-		echo apply_filters( 'give_left_sidebar_pre_wrap', '<div id="give-sidebar-left" class="give-sidebar give-single-form-sidebar-left">' );
+		echo apply_filters('give_left_sidebar_pre_wrap', '<div id="give-sidebar-left" class="give-sidebar give-single-form-sidebar-left">');
 	}
 }
 
-if ( ! function_exists( 'give_left_sidebar_post_wrap' ) ) {
+if ( ! function_exists('give_left_sidebar_post_wrap')) {
 	function give_left_sidebar_post_wrap() {
-		echo apply_filters( 'give_left_sidebar_post_wrap', '</div>' );
+		echo apply_filters('give_left_sidebar_post_wrap', '</div>');
 	}
 }
 
-if ( ! function_exists( 'give_get_forms_sidebar' ) ) {
+if ( ! function_exists('give_get_forms_sidebar')) {
 	function give_get_forms_sidebar() {
-		give_get_template_part( 'single-give-form/sidebar' );
+		give_get_template_part('single-give-form/sidebar');
 	}
 }
 
-if ( ! function_exists( 'give_show_form_images' ) ) {
+if ( ! function_exists('give_show_form_images')) {
 
 	/**
 	 * Output the donation form featured image.
 	 */
 	function give_show_form_images() {
-		if ( give_is_setting_enabled( give_get_option( 'form_featured_img' ) ) ) {
-			give_get_template_part( 'single-give-form/featured-image' );
+		if (give_is_setting_enabled(give_get_option('form_featured_img'))) {
+			give_get_template_part('single-give-form/featured-image');
 		}
 	}
 }
 
-if ( ! function_exists( 'give_template_single_title' ) ) {
+if ( ! function_exists('give_template_single_title')) {
 
 	/**
 	 * Output the form title.
 	 */
 	function give_template_single_title() {
-		give_get_template_part( 'single-give-form/title' );
+		give_get_template_part('single-give-form/title');
 	}
 }
 
@@ -470,7 +470,7 @@ 
  * Conditional Functions
  */
 
-if ( ! function_exists( 'is_give_form' ) ) {
+if ( ! function_exists('is_give_form')) {
 
 	/**
 	 * is_give_form
@@ -482,11 +482,11 @@ 
 	 * @return bool
 	 */
 	function is_give_form() {
-		return is_singular( array( 'give_form' ) );
+		return is_singular(array('give_form'));
 	}
 }
 
-if ( ! function_exists( 'is_give_category' ) ) {
+if ( ! function_exists('is_give_category')) {
 
 	/**
 	 * is_give_category
@@ -501,12 +501,12 @@ 
 	 *
 	 * @return bool
 	 */
-	function is_give_category( $term = '' ) {
-		return is_tax( 'give_forms_category', $term );
+	function is_give_category($term = '') {
+		return is_tax('give_forms_category', $term);
 	}
 }
 
-if ( ! function_exists( 'is_give_tag' ) ) {
+if ( ! function_exists('is_give_tag')) {
 
 	/**
 	 * is_give_tag
@@ -521,12 +521,12 @@ 
 	 *
 	 * @return bool
 	 */
-	function is_give_tag( $term = '' ) {
-		return is_tax( 'give_forms_tag', $term );
+	function is_give_tag($term = '') {
+		return is_tax('give_forms_tag', $term);
 	}
 }
 
-if ( ! function_exists( 'is_give_taxonomy' ) ) {
+if ( ! function_exists('is_give_taxonomy')) {
 
 	/**
 	 * is_give_taxonomy
@@ -538,6 +538,6 @@ 
 	 * @return bool
 	 */
 	function is_give_taxonomy() {
-		return is_tax( get_object_taxonomies( 'give_form' ) );
+		return is_tax(get_object_taxonomies('give_form'));
 	}
 }

includes/class-give-background-updater.php

Indentation +2 added lines, -2 removed lines

@@ -202,8 +202,8 @@
 		// 2. Processing percentage greater then 100%
 		if( (
 			101 < $resume_update['total_percentage'] ) ||
-		    ( $give_updates->get_total_db_update_count() < $resume_update['update'] ) ||
-		    ! in_array( $resume_update['update_info']['id'], $give_updates->get_update_ids() )
+			( $give_updates->get_total_db_update_count() < $resume_update['update'] ) ||
+			! in_array( $resume_update['update_info']['id'], $give_updates->get_update_ids() )
 		) {
 			if( ! $this->is_paused_process() ){
 				$give_updates->__pause_db_update(true);

Spacing +64 added lines, -64 removed lines

@@ -11,7 +11,7 @@ 
  * @category Class
  * @author   WordImpress
  */
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -55,7 +55,7 @@ 
 	 * @return bool
 	 */
 	public function has_queue() {
-		return ( ! parent::is_queue_empty() );
+		return ( ! parent::is_queue_empty());
 	}
 
 
@@ -71,12 +71,12 @@ 
 	 */
 	protected function lock_process() {
 		// Check if admin want to pause upgrade.
-		if( get_option('give_pause_upgrade') ) {
+		if (get_option('give_pause_upgrade')) {
 			self::flush_cache();
 
-			delete_option( 'give_paused_batches' );
+			delete_option('give_paused_batches');
 
-			Give_Updates::get_instance()->__pause_db_update( true );
+			Give_Updates::get_instance()->__pause_db_update(true);
 
 			delete_option('give_pause_upgrade');
 
@@ -85,7 +85,7 @@ 
 			 *
 			 * @since 2.0.1
 			 */
-			do_action( 'give_pause_db_upgrade', Give_Updates::get_instance() );
+			do_action('give_pause_db_upgrade', Give_Updates::get_instance());
 
 			wp_die();
 		}
@@ -93,10 +93,10 @@ 
 
 		$this->start_time = time(); // Set start time of current process.
 
-		$lock_duration = ( property_exists( $this, 'queue_lock_time' ) ) ? $this->queue_lock_time : 60; // 1 minute
-		$lock_duration = apply_filters( $this->identifier . '_queue_lock_time', $lock_duration );
+		$lock_duration = (property_exists($this, 'queue_lock_time')) ? $this->queue_lock_time : 60; // 1 minute
+		$lock_duration = apply_filters($this->identifier.'_queue_lock_time', $lock_duration);
 
-		set_site_transient( $this->identifier . '_process_lock', microtime(), $lock_duration );
+		set_site_transient($this->identifier.'_process_lock', microtime(), $lock_duration);
 	}
 
 	/**
@@ -106,12 +106,12 @@ 
 	 * and data exists in the queue.
 	 */
 	public function handle_cron_healthcheck() {
-		if ( $this->is_process_running() || $this->is_paused_process()  ) {
+		if ($this->is_process_running() || $this->is_paused_process()) {
 			// Background process already running.
 			return;
 		}
 
-		if ( $this->is_queue_empty() ) {
+		if ($this->is_queue_empty()) {
 			// No data to process.
 			$this->clear_scheduled_event();
 
@@ -125,8 +125,8 @@ 
 	 * Schedule fallback event.
 	 */
 	protected function schedule_event() {
-		if ( ! wp_next_scheduled( $this->cron_hook_identifier ) && ! $this->is_paused_process() ) {
-			wp_schedule_event( time() + 10, $this->cron_interval_identifier, $this->cron_hook_identifier );
+		if ( ! wp_next_scheduled($this->cron_hook_identifier) && ! $this->is_paused_process()) {
+			wp_schedule_event(time() + 10, $this->cron_interval_identifier, $this->cron_hook_identifier);
 		}
 	}
 
@@ -142,13 +142,13 @@ 
 	 *
 	 * @return mixed
 	 */
-	protected function task( $update ) {
+	protected function task($update) {
 		// Pause upgrade immediately if admin pausing upgrades.
-		if( $this->is_paused_process() ) {
+		if ($this->is_paused_process()) {
 			wp_die();
 		}
 
-		if ( empty( $update ) ) {
+		if (empty($update)) {
 			return false;
 		}
 
@@ -165,7 +165,7 @@ 
 				'update_info'      => $update,
 				'step'             => 1,
 				'update'           => 1,
-				'heading'          => sprintf( 'Update %s of {update_count}', 1 ),
+				'heading'          => sprintf('Update %s of {update_count}', 1),
 				'percentage'       => $give_updates->percentage,
 				'total_percentage' => 0,
 			)
@@ -174,22 +174,22 @@ 
 		// Continuously skip update if previous update does not complete yet.
 		if (
 			$resume_update['update_info']['id'] !== $update['id'] &&
-			! give_has_upgrade_completed( $resume_update['update_info']['id'] )
+			! give_has_upgrade_completed($resume_update['update_info']['id'])
 		) {
 			return $update;
 		}
 
 		// Set params.
 		$resume_update['update_info'] = $update;
-		$give_updates->step           = absint( $resume_update['step'] );
-		$give_updates->update         = absint( $resume_update['update'] );
-		$is_parent_update_completed   = $give_updates->is_parent_updates_completed( $update );
+		$give_updates->step           = absint($resume_update['step']);
+		$give_updates->update         = absint($resume_update['update']);
+		$is_parent_update_completed   = $give_updates->is_parent_updates_completed($update);
 
 
 		// Skip update if dependency update does not complete yet.
-		if ( empty( $is_parent_update_completed ) ) {
+		if (empty($is_parent_update_completed)) {
 			// @todo: set error when you have only one update with invalid dependency
-			if ( ! is_null( $is_parent_update_completed ) ) {
+			if ( ! is_null($is_parent_update_completed)) {
 				return $update;
 			}
 
@@ -200,23 +200,23 @@ 
 		// Pause upgrade immediately if found following:
 		// 1. Running update number greater then total update count
 		// 2. Processing percentage greater then 100%
-		if( (
+		if ((
 			101 < $resume_update['total_percentage'] ) ||
-		    ( $give_updates->get_total_db_update_count() < $resume_update['update'] ) ||
-		    ! in_array( $resume_update['update_info']['id'], $give_updates->get_update_ids() )
+		    ($give_updates->get_total_db_update_count() < $resume_update['update']) ||
+		    ! in_array($resume_update['update_info']['id'], $give_updates->get_update_ids())
 		) {
-			if( ! $this->is_paused_process() ){
+			if ( ! $this->is_paused_process()) {
 				$give_updates->__pause_db_update(true);
 			}
 
-			update_option( 'give_upgrade_error', 1 );
+			update_option('give_upgrade_error', 1);
 
-			$log_data = 'Update Task' . "\n";
+			$log_data = 'Update Task'."\n";
 			$log_data .= "Total update count: {$give_updates->get_total_db_update_count()}\n";
-			$log_data .= 'Update IDs: ' . print_r( $give_updates->get_update_ids() , true );
-			$log_data .= 'Update: ' . print_r( $resume_update , true );
+			$log_data .= 'Update IDs: '.print_r($give_updates->get_update_ids(), true);
+			$log_data .= 'Update: '.print_r($resume_update, true);
 
-			Give()->logs->add( 'Update Error', $log_data, 0, 'update' );
+			Give()->logs->add('Update Error', $log_data, 0, 'update');
 
 			wp_die();
 		}
@@ -224,25 +224,25 @@ 
 		// Disable cache.
 		Give_Cache::disable();
 
-		try{
+		try {
 			// Run update.
-			if ( is_array( $update['callback'] ) ) {
+			if (is_array($update['callback'])) {
 				$update['callback'][0]->$update['callback'][1]();
 			} else {
 				$update['callback']();
 			}
-		} catch ( Exception $e ){
+		} catch (Exception $e) {
 
-			if( ! $this->is_paused_process() ){
+			if ( ! $this->is_paused_process()) {
 				$give_updates->__pause_db_update(true);
 			}
 
-			$log_data = 'Update Task' . "\n";
-			$log_data .= print_r( $resume_update, true ) . "\n\n";
+			$log_data = 'Update Task'."\n";
+			$log_data .= print_r($resume_update, true)."\n\n";
 			$log_data .= "Error\n {$e->getMessage()}";
 
-			Give()->logs->add( 'Update Error', $log_data, 0, 'update' );
-			update_option( 'give_upgrade_error', 1 );
+			Give()->logs->add('Update Error', $log_data, 0, 'update');
+			update_option('give_upgrade_error', 1);
 
 			wp_die();
 		}
@@ -250,21 +250,21 @@ 
 		// Set update info.
 		$doing_upgrade_args = array(
 			'update_info'      => $update,
-			'step'             => ++ $give_updates->step,
+			'step'             => ++$give_updates->step,
 			'update'           => $give_updates->update,
-			'heading'          => sprintf( 'Update %s of %s', $give_updates->update, get_option( 'give_db_update_count' ) ),
+			'heading'          => sprintf('Update %s of %s', $give_updates->update, get_option('give_db_update_count')),
 			'percentage'       => $give_updates->percentage,
 			'total_percentage' => $give_updates->get_db_update_processing_percentage(),
 		);
 
 		// Cache upgrade.
-		update_option( 'give_doing_upgrade', $doing_upgrade_args );
+		update_option('give_doing_upgrade', $doing_upgrade_args);
 
 		// Enable cache.
 		Give_Cache::enable();
 
 		// Check if current update completed or not.
-		if ( give_has_upgrade_completed( $update['id'] ) ) {
+		if (give_has_upgrade_completed($update['id'])) {
 			return false;
 		}
 
@@ -278,23 +278,23 @@ 
 	 * performed, or, call parent::complete().
 	 */
 	public function complete() {
-		if ( $this->is_paused_process() ) {
+		if ($this->is_paused_process()) {
 			return false;
 		}
 
 		parent::complete();
 
-		delete_option( 'give_pause_upgrade' );
-		delete_option( 'give_upgrade_error' );
-		delete_option( 'give_db_update_count' );
-		delete_option( 'give_doing_upgrade' );
-		add_option( 'give_show_db_upgrade_complete_notice', 1, '', 'no' );
+		delete_option('give_pause_upgrade');
+		delete_option('give_upgrade_error');
+		delete_option('give_db_update_count');
+		delete_option('give_doing_upgrade');
+		add_option('give_show_db_upgrade_complete_notice', 1, '', 'no');
 
 		// Flush cache.
 		Give_Cache::get_instance()->flush_cache();
 
-		if ( $cache_keys = Give_Cache::get_options_like( '' ) ) {
-			Give_Cache::delete( $cache_keys );
+		if ($cache_keys = Give_Cache::get_options_like('')) {
+			Give_Cache::delete($cache_keys);
 		}
 	}
 
@@ -304,19 +304,19 @@ 
 	 * @return int
 	 */
 	protected function get_memory_limit() {
-		if ( function_exists( 'ini_get' ) ) {
-			$memory_limit = ini_get( 'memory_limit' );
+		if (function_exists('ini_get')) {
+			$memory_limit = ini_get('memory_limit');
 		} else {
 			// Sensible default.
 			$memory_limit = '128M';
 		}
 
-		if ( ! $memory_limit || '-1' === $memory_limit ) {
+		if ( ! $memory_limit || '-1' === $memory_limit) {
 			// Unlimited, set to 32GB.
 			$memory_limit = '32000M';
 		}
 
-		return intval( $memory_limit ) * 1024 * 1024;
+		return intval($memory_limit) * 1024 * 1024;
 	}
 
 	/**
@@ -329,17 +329,17 @@ 
 		// Don't lock up other requests while processing
 		session_write_close();
 
-		if ( $this->is_process_running() || $this->is_paused_process() ) {
+		if ($this->is_process_running() || $this->is_paused_process()) {
 			// Background process already running.
 			wp_die();
 		}
 
-		if ( $this->is_queue_empty() ) {
+		if ($this->is_queue_empty()) {
 			// No data to process.
 			wp_die();
 		}
 
-		check_ajax_referer( $this->identifier, 'nonce' );
+		check_ajax_referer($this->identifier, 'nonce');
 
 		$this->handle();
 
@@ -354,13 +354,13 @@ 
 	 * @access public
 	 * @return bool
 	 */
-	public function is_paused_process(){
+	public function is_paused_process() {
 		// Delete cache.
-		wp_cache_delete( 'give_paused_batches', 'options' );
+		wp_cache_delete('give_paused_batches', 'options');
 
 		$paused_batches = get_option('give_paused_batches');
 
-		return ! empty( $paused_batches );
+		return ! empty($paused_batches);
 	}
 
 
@@ -406,8 +406,8 @@ 
 		);
 
 
-		foreach ( $options as $option ) {
-			wp_cache_delete( $option, 'options' );
+		foreach ($options as $option) {
+			wp_cache_delete($option, 'options');
 		}
 	}
 }

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
@@ -513,7 +513,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,39 +284,39 @@ 
 	 *
 	 * @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( "mt2.{$this->meta_type}_id", "mt1.{$this->meta_type}_id" ), $clause );
+				$clause = str_replace(array('mt2.post_id', 'mt1.post_id'), array("mt2.{$this->meta_type}_id", "mt1.{$this->meta_type}_id"), $clause);
 				break;
 		}
 
@@ -334,19 +334,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;
 			}
@@ -365,8 +365,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));
 	}
 
 	/**
@@ -392,8 +392,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(
 			// 'form'    => 'give-forms',
@@ -403,8 +403,8 @@ 
 			// 'log'     => 'give-logs',
 		);
 
-		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]);
 		}
 	}
 
@@ -419,8 +419,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];
@@ -429,11 +429,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];
@@ -442,13 +442,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];
@@ -457,11 +457,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];
@@ -471,11 +471,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);
 		}
 	}
 
@@ -500,10 +500,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 );
+		update_option($this->table_name.'_db_version', $this->version);
 	}
 
 
@@ -515,7 +515,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_meta_type(){
+	public function get_meta_type() {
 		return $this->meta_type;
 	}
 
@@ -529,12 +529,12 @@ 
 	 *
 	 * @return  bool  False for failure. True for success.
 	 */
-	public function delete_all_meta( $id = 0 ) {
+	public function delete_all_meta($id = 0) {
 		global $wpdb;
-		$status = $wpdb->delete( $this->table_name, array( "{$this->meta_type}_id" => $id ), array( '%d' ) );
+		$status = $wpdb->delete($this->table_name, array("{$this->meta_type}_id" => $id), array('%d'));
 
-		if ( $status ) {
-			$this->delete_cache( $id, $this->meta_type );
+		if ($status) {
+			$this->delete_cache($id, $this->meta_type);
 		}
 
 		return $status;

includes/class-give-db.php

Spacing +61 added lines, -61 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -72,8 +72,8 @@ 
 	 * @access public
 	 */
 	public function __construct() {
-		if( is_multisite() ) {
-			add_action( 'switch_blog', array( $this, 'handle_switch_blog' ), 10, 2 );
+		if (is_multisite()) {
+			add_action('switch_blog', array($this, 'handle_switch_blog'), 10, 2);
 		}
 	}
 
@@ -111,16 +111,16 @@ 
 	 *
 	 * @return object
 	 */
-	public function get( $row_id ) {
+	public function get($row_id) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Bailout.
-		if ( empty( $row_id ) ) {
+		if (empty($row_id)) {
 			return null;
 		}
 
-		return $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id ) );
+		return $wpdb->get_row($wpdb->prepare("SELECT * FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id));
 	}
 
 	/**
@@ -134,18 +134,18 @@ 
 	 *
 	 * @return object
 	 */
-	public function get_by( $column, $row_id ) {
+	public function get_by($column, $row_id) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Bailout.
-		if ( empty( $column ) || empty( $row_id ) ) {
+		if (empty($column) || empty($row_id)) {
 			return null;
 		}
 
-		$column = esc_sql( $column );
+		$column = esc_sql($column);
 
-		return $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $this->table_name WHERE $column = %s LIMIT 1;", $row_id ) );
+		return $wpdb->get_row($wpdb->prepare("SELECT * FROM $this->table_name WHERE $column = %s LIMIT 1;", $row_id));
 	}
 
 	/**
@@ -159,18 +159,18 @@ 
 	 *
 	 * @return string      Column value.
 	 */
-	public function get_column( $column, $row_id ) {
+	public function get_column($column, $row_id) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Bailout.
-		if ( empty( $column ) || empty( $row_id ) ) {
+		if (empty($column) || empty($row_id)) {
 			return null;
 		}
 
-		$column = esc_sql( $column );
+		$column = esc_sql($column);
 
-		return $wpdb->get_var( $wpdb->prepare( "SELECT $column FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id ) );
+		return $wpdb->get_var($wpdb->prepare("SELECT $column FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id));
 	}
 
 	/**
@@ -185,19 +185,19 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_column_by( $column, $column_where, $column_value ) {
+	public function get_column_by($column, $column_where, $column_value) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Bailout.
-		if ( empty( $column ) || empty( $column_where ) || empty( $column_value ) ) {
+		if (empty($column) || empty($column_where) || empty($column_value)) {
 			return null;
 		}
 
-		$column_where = esc_sql( $column_where );
-		$column       = esc_sql( $column );
+		$column_where = esc_sql($column_where);
+		$column       = esc_sql($column);
 
-		return $wpdb->get_var( $wpdb->prepare( "SELECT $column FROM $this->table_name WHERE $column_where = %s LIMIT 1;", $column_value ) );
+		return $wpdb->get_var($wpdb->prepare("SELECT $column FROM $this->table_name WHERE $column_where = %s LIMIT 1;", $column_value));
 	}
 
 	/**
@@ -211,12 +211,12 @@ 
 	 *
 	 * @return int
 	 */
-	public function insert( $data, $type = '' ) {
+	public function insert($data, $type = '') {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Set default values.
-		$data = wp_parse_args( $data, $this->get_column_defaults() );
+		$data = wp_parse_args($data, $this->get_column_defaults());
 
 		/**
 		 * Fires before inserting data to the database.
@@ -225,7 +225,7 @@ 
 		 *
 		 * @param array $data
 		 */
-		do_action( "give_pre_insert_{$type}", $data );
+		do_action("give_pre_insert_{$type}", $data);
 
 		// Initialise column format array
 		$column_formats = $this->get_columns();
@@ -234,13 +234,13 @@ 
 		// $data = array_change_key_case( $data );
 
 		// White list columns
-		$data = array_intersect_key( $data, $column_formats );
+		$data = array_intersect_key($data, $column_formats);
 
 		// Reorder $column_formats to match the order of columns given in $data
-		$data_keys      = array_keys( $data );
-		$column_formats = array_merge( array_flip( $data_keys ), $column_formats );
+		$data_keys      = array_keys($data);
+		$column_formats = array_merge(array_flip($data_keys), $column_formats);
 
-		$wpdb->insert( $this->table_name, $data, $column_formats );
+		$wpdb->insert($this->table_name, $data, $column_formats);
 
 		/**
 		 * Fires after inserting data to the database.
@@ -250,7 +250,7 @@ 
 		 * @param int   $insert_id
 		 * @param array $data
 		 */
-		do_action( "give_post_insert_{$type}", $wpdb->insert_id, $data );
+		do_action("give_post_insert_{$type}", $wpdb->insert_id, $data);
 
 		return $wpdb->insert_id;
 	}
@@ -267,18 +267,18 @@ 
 	 *
 	 * @return bool
 	 */
-	public function update( $row_id, $data = array(), $where = '' ) {
+	public function update($row_id, $data = array(), $where = '') {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Row ID must be positive integer
-		$row_id = absint( $row_id );
+		$row_id = absint($row_id);
 
-		if ( empty( $row_id ) ) {
+		if (empty($row_id)) {
 			return false;
 		}
 
-		if ( empty( $where ) ) {
+		if (empty($where)) {
 			$where = $this->primary_key;
 		}
 
@@ -286,16 +286,16 @@ 
 		$column_formats = $this->get_columns();
 
 		// Force fields to lower case
-		$data = array_change_key_case( $data );
+		$data = array_change_key_case($data);
 
 		// White list columns
-		$data = array_intersect_key( $data, $column_formats );
+		$data = array_intersect_key($data, $column_formats);
 
 		// Reorder $column_formats to match the order of columns given in $data
-		$data_keys      = array_keys( $data );
-		$column_formats = array_merge( array_flip( $data_keys ), $column_formats );
+		$data_keys      = array_keys($data);
+		$column_formats = array_merge(array_flip($data_keys), $column_formats);
 
-		if ( false === $wpdb->update( $this->table_name, $data, array( $where => $row_id ), $column_formats ) ) {
+		if (false === $wpdb->update($this->table_name, $data, array($where => $row_id), $column_formats)) {
 			return false;
 		}
 
@@ -312,18 +312,18 @@ 
 	 *
 	 * @return bool
 	 */
-	public function delete( $row_id = 0 ) {
+	public function delete($row_id = 0) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Row ID must be positive integer
-		$row_id = absint( $row_id );
+		$row_id = absint($row_id);
 
-		if ( empty( $row_id ) ) {
+		if (empty($row_id)) {
 			return false;
 		}
 
-		if ( false === $wpdb->query( $wpdb->prepare( "DELETE FROM $this->table_name WHERE $this->primary_key = %d", $row_id ) ) ) {
+		if (false === $wpdb->query($wpdb->prepare("DELETE FROM $this->table_name WHERE $this->primary_key = %d", $row_id))) {
 			return false;
 		}
 
@@ -340,13 +340,13 @@ 
 	 *
 	 * @return bool          If the table name exists.
 	 */
-	public function table_exists( $table ) {
+	public function table_exists($table) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
-		$table = sanitize_text_field( $table );
+		$table = sanitize_text_field($table);
 
-		return $wpdb->get_var( $wpdb->prepare( "SHOW TABLES LIKE '%s'", $table ) ) === $table;
+		return $wpdb->get_var($wpdb->prepare("SHOW TABLES LIKE '%s'", $table)) === $table;
 	}
 
 	/**
@@ -360,16 +360,16 @@ 
 	 *
 	 * @return bool
 	 */
-	public function does_column_exist( $column_name ) {
+	public function does_column_exist($column_name) {
 
 		global $wpdb;
 
-		$column = $wpdb->get_results( $wpdb->prepare(
+		$column = $wpdb->get_results($wpdb->prepare(
 			"SELECT * FROM INFORMATION_SCHEMA.COLUMNS WHERE TABLE_SCHEMA = %s AND TABLE_NAME = %s AND COLUMN_NAME = %s ",
 			DB_NAME, $this->table_name, $column_name
-		) );
+		));
 
-		if ( ! empty( $column ) ) {
+		if ( ! empty($column)) {
 			return true;
 		}
 
@@ -385,7 +385,7 @@ 
 	 * @return bool Returns if the customers table was installed and upgrade routine run.
 	 */
 	public function installed() {
-		return $this->table_exists( $this->table_name );
+		return $this->table_exists($this->table_name);
 	}
 
 	/**
@@ -395,8 +395,8 @@ 
 	 * @access public
 	 */
 	public function register_table() {
-		$current_version = get_option( $this->table_name . '_db_version' );
-		if ( ! $current_version || version_compare( $current_version, $this->version, '<' ) ) {
+		$current_version = get_option($this->table_name.'_db_version');
+		if ( ! $current_version || version_compare($current_version, $this->version, '<')) {
 			$this->create_table();
 		}
 	}
@@ -421,23 +421,23 @@ 
 	 *
 	 * @return int|bool                The normalized log ID or false if it's found to not be valid.
 	 */
-	public function sanitize_id( $id ) {
-		if ( ! is_numeric( $id ) ) {
+	public function sanitize_id($id) {
+		if ( ! is_numeric($id)) {
 			return false;
 		}
 
 		$id = (int) $id;
 
 		// We were given a non positive number.
-		if ( absint( $id ) !== $id ) {
+		if (absint($id) !== $id) {
 			return false;
 		}
 
-		if ( empty( $id ) ) {
+		if (empty($id)) {
 			return false;
 		}
 
-		return absint( $id );
+		return absint($id);
 
 	}
 
@@ -451,23 +451,23 @@ 
 	 * @param $new_blog_id
 	 * @param $prev_blog_id
 	 */
-	public function handle_switch_blog( $new_blog_id, $prev_blog_id ) {
+	public function handle_switch_blog($new_blog_id, $prev_blog_id) {
 		global $wpdb;
 
 		// Bailout.
-		if ( $new_blog_id === $prev_blog_id ) {
+		if ($new_blog_id === $prev_blog_id) {
 			return;
 		}
 
 
 		$this->table_name = str_replace(
-			1 != $prev_blog_id ? $wpdb->get_blog_prefix( $prev_blog_id ) : $wpdb->base_prefix,
-			1 != $new_blog_id ? $wpdb->get_blog_prefix( $new_blog_id ) : $wpdb->base_prefix,
+			1 != $prev_blog_id ? $wpdb->get_blog_prefix($prev_blog_id) : $wpdb->base_prefix,
+			1 != $new_blog_id ? $wpdb->get_blog_prefix($new_blog_id) : $wpdb->base_prefix,
 			$this->table_name
 		);
 
-		if ( $this instanceof Give_DB_Meta ) {
-			$wpdb->{$this->get_meta_type() . 'meta'} = $this->table_name;
+		if ($this instanceof Give_DB_Meta) {
+			$wpdb->{$this->get_meta_type().'meta'} = $this->table_name;
 		}
 
 	}

includes/admin/class-addon-activation-banner.php

Doc Comments +1 added lines, -1 removed lines

@@ -505,7 +505,7 @@
 	 *
 	 * @param string $main_file Plugin Main File.
 	 *
-	 * @return bool|mixed|string
+	 * @return string
 	 */
 	public function get_plugin_folder_name( $main_file ) {
 		// Remove plugin file and get the Add-on's folder name only.

Spacing +104 added lines, -104 removed lines

@@ -7,7 +7,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -35,13 +35,13 @@ 
 	 *                               'testing'           => false, // (required) Never leave as "true" in production!!!
 	 *                               }
 	 */
-	function __construct( $_banner_details ) {
+	function __construct($_banner_details) {
 		global $give_addons, $pagenow;
 
 		// Append add-on information to the global variable.
 		$give_addons[] = $_banner_details;
 
-		if ( 'plugins.php' === $pagenow ) {
+		if ('plugins.php' === $pagenow) {
 
 			// Get the current user.
 			$current_user  = wp_get_current_user();
@@ -54,9 +54,9 @@ 
 			$this->add_addon_activate_meta();
 
 			// Check if notice callback is already hooked.
-			if ( ! $this->is_banner_notice_hooked() ) {
+			if ( ! $this->is_banner_notice_hooked()) {
 				// If multiple add-on are activated then show activation banner in tab view.
-				add_action( 'admin_notices', array( $this, 'addon_activation_banner_notices' ), 10 );
+				add_action('admin_notices', array($this, 'addon_activation_banner_notices'), 10);
 			}
 		}
 	}
@@ -69,8 +69,8 @@ 
 	 *
 	 * @return string
 	 */
-	public static function get_banner_user_meta_key( $addon_banner_key ) {
-		$addon_slug = sanitize_text_field( $addon_banner_key );
+	public static function get_banner_user_meta_key($addon_banner_key) {
+		$addon_slug = sanitize_text_field($addon_banner_key);
 
 		return "give_addon_{$addon_slug}_active_by_user";
 	}
@@ -85,24 +85,24 @@ 
 	 */
 	public function init() {
 		// Get the current page to add the notice to
-		add_action( 'current_screen', array( $this, 'give_addon_notice_ignore' ) );
+		add_action('current_screen', array($this, 'give_addon_notice_ignore'));
 
 		// Get the Give add-ons.
 		$give_addons = $this->get_plugin_file_names();
 
-		if ( ! empty( $give_addons ) ) {
+		if ( ! empty($give_addons)) {
 
 			// Go through each of the add-on and hook deactivate action.
-			foreach ( $give_addons as $addon_name => $give_addon ) {
+			foreach ($give_addons as $addon_name => $give_addon) {
 
 				// Testing?
-				if ( true === $give_addon['testing'] ) {
-					$nag_meta_key = 'give_addon_activation_ignore_' . $addon_name;
-					delete_user_meta( $this->user_id, $nag_meta_key );
+				if (true === $give_addon['testing']) {
+					$nag_meta_key = 'give_addon_activation_ignore_'.$addon_name;
+					delete_user_meta($this->user_id, $nag_meta_key);
 				}
 
 				// Add deactivate hook.
-				add_action( 'deactivate_' . $give_addon['plugin_main_file'], array( $this, 'remove_addon_activate_meta' ) );
+				add_action('deactivate_'.$give_addon['plugin_main_file'], array($this, 'remove_addon_activate_meta'));
 			}
 		}
 	}
@@ -118,29 +118,29 @@ 
 		global $give_addons;
 
 		// Get recently activated plugins.
-		$active_plugins = get_option( 'active_plugins' );
+		$active_plugins = get_option('active_plugins');
 
 		$file_names = array();
 
-		if ( empty( $give_addons ) ) {
+		if (empty($give_addons)) {
 			return $file_names;
 		}
 
 		// Go through each addon and get the plugin file url.
-		foreach ( $give_addons as $give_addon ) {
+		foreach ($give_addons as $give_addon) {
 			$file_name = '';
-			$file_path = explode( '/plugins/', $give_addon['file'] );
-			if ( $file_path ) {
-				$file_path = array_pop( $file_path );
-				$file_name = current( explode( '/', $file_path ) );
+			$file_path = explode('/plugins/', $give_addon['file']);
+			if ($file_path) {
+				$file_path = array_pop($file_path);
+				$file_name = current(explode('/', $file_path));
 			}
 
-			if ( ! empty( $file_name ) ) {
-				foreach ( $active_plugins as $plugin ) {
-					if ( false !== strpos( $plugin, $file_name ) ) {
-						$add_on_key                     = sanitize_title( $give_addon['name'] );
+			if ( ! empty($file_name)) {
+				foreach ($active_plugins as $plugin) {
+					if (false !== strpos($plugin, $file_name)) {
+						$add_on_key                     = sanitize_title($give_addon['name']);
 						$give_addon['plugin_main_file'] = $plugin; // Include plugin file.
-						$file_names[ $add_on_key ]      = $give_addon;
+						$file_names[$add_on_key]      = $give_addon;
 						break;
 					}
 				}
@@ -160,16 +160,16 @@ 
 		// Get all activated add-ons.
 		$give_addons = $this->get_plugin_file_names();
 
-		if ( ! empty( $give_addons ) ) {
+		if ( ! empty($give_addons)) {
 
 			// Go through rach add-ons and add meta data.
-			foreach ( $give_addons as $banner_addon_name => $addon ) {
+			foreach ($give_addons as $banner_addon_name => $addon) {
 
 				// User meta key.
-				$user_id = __give_get_active_by_user_meta( $banner_addon_name );
+				$user_id = __give_get_active_by_user_meta($banner_addon_name);
 
-				if ( ! $user_id ) {
-					update_option( self::get_banner_user_meta_key( $banner_addon_name ), $this->user_id, '' );
+				if ( ! $user_id) {
+					update_option(self::get_banner_user_meta_key($banner_addon_name), $this->user_id, '');
 				}
 			}
 		}
@@ -186,14 +186,14 @@ 
 		global $wp_filter;
 		$notice_already_hooked = false;
 
-		if ( isset( $wp_filter['admin_notices']->callbacks[10] ) ) {
+		if (isset($wp_filter['admin_notices']->callbacks[10])) {
 			// Get all of the hooks.
-			$admin_notice_callbacks = array_keys( $wp_filter['admin_notices']->callbacks[10] );
+			$admin_notice_callbacks = array_keys($wp_filter['admin_notices']->callbacks[10]);
 
-			if ( ! empty( $admin_notice_callbacks ) ) {
-				foreach ( $admin_notice_callbacks as $key ) {
+			if ( ! empty($admin_notice_callbacks)) {
+				foreach ($admin_notice_callbacks as $key) {
 					//If the key is found in your string, set $found to true
-					if ( false !== strpos( $key, 'addon_activation_banner_notices' ) ) {
+					if (false !== strpos($key, 'addon_activation_banner_notices')) {
 						$notice_already_hooked = true;
 					}
 				}
@@ -212,7 +212,7 @@ 
 		global $pagenow, $give_addons;
 
 		// Bailout.
-		if ( 'plugins.php' !== $pagenow ) {
+		if ('plugins.php' !== $pagenow) {
 			return false;
 		}
 
@@ -224,55 +224,55 @@ 
 		$latest_addon     = array();
 
 		// Get the plugin folder name, because many give-addon not sending proper plugin_file.
-		if ( ! empty( $recent_activated ) ) {
-			foreach ( $recent_activated as $recent_addon ) {
+		if ( ! empty($recent_activated)) {
+			foreach ($recent_activated as $recent_addon) {
 				// Get the add-on folder name.
-				$latest_addon[] = substr( $recent_addon, 0, strpos( $recent_addon, '/' ) );
+				$latest_addon[] = substr($recent_addon, 0, strpos($recent_addon, '/'));
 			}
 		}
 
 		// Go through each of the give add-on.
-		foreach ( $give_addons as $addon ) {
-			$addon_sanitized_name = sanitize_title( $addon['name'] );
+		foreach ($give_addons as $addon) {
+			$addon_sanitized_name = sanitize_title($addon['name']);
 
 			// Get the add-on dismiss status.
-			$add_on_state = get_user_meta( $this->user_id, "give_addon_activation_ignore_{$addon_sanitized_name}", true );
+			$add_on_state = get_user_meta($this->user_id, "give_addon_activation_ignore_{$addon_sanitized_name}", true);
 
 			// Get the option key.
-			$activate_by_user = (int) __give_get_active_by_user_meta( $addon_sanitized_name );
+			$activate_by_user = (int) __give_get_active_by_user_meta($addon_sanitized_name);
 
 			// Remove plugin file and get the Add-on's folder name only.
-			$file_path = $this->get_plugin_folder_name( $addon['file'] );
+			$file_path = $this->get_plugin_folder_name($addon['file']);
 
 			// If add-on were never dismissed.
-			if ( 'true' !== $add_on_state && $this->user_id === $activate_by_user ) {
-				if ( ! empty( $latest_addon ) && ( in_array( $file_path, $latest_addon, true ) || empty( $latest_addon ) ) ) {
+			if ('true' !== $add_on_state && $this->user_id === $activate_by_user) {
+				if ( ! empty($latest_addon) && (in_array($file_path, $latest_addon, true) || empty($latest_addon))) {
 					$addon_to_display[] = $addon;
 				}
 			}
 		}
 
-		if ( ! empty( $addon_to_display ) ) {
+		if ( ! empty($addon_to_display)) {
 			ob_start();
 
 			// Output inline styles here because there's no reason
 			// to enqueued them after the alert is dismissed.
 			$this->print_css_js();
 			?>
-			<div class="<?php echo ( 1 !== count( $addon_to_display ) ) ? 'give-alert-tab-wrapper' : ''; ?> updated give-addon-alert give-notice">
+			<div class="<?php echo (1 !== count($addon_to_display)) ? 'give-alert-tab-wrapper' : ''; ?> updated give-addon-alert give-notice">
 				<?php
 				// If multiple add-on are activated.
-				if ( 1 !== count( $addon_to_display ) ) {
+				if (1 !== count($addon_to_display)) {
 					?>
 					<div class="give-vertical-tab">
 						<div class="give-addon-tab-list">
 							<ul class="give-alert-addon-list">
 								<?php
 								$is_first = true;
-								foreach ( $addon_to_display as $banner ) {
+								foreach ($addon_to_display as $banner) {
 									?>
-									<li class="give-tab-list<?php echo ( true === $is_first ) ? ' active' : ''; ?>" id="give-addon-<?php echo esc_attr( basename( $banner['file'], '.php' ) ); ?>">
-										<a href="#"><?php echo esc_html( $banner['name'] ); ?></a>
+									<li class="give-tab-list<?php echo (true === $is_first) ? ' active' : ''; ?>" id="give-addon-<?php echo esc_attr(basename($banner['file'], '.php')); ?>">
+										<a href="#"><?php echo esc_html($banner['name']); ?></a>
 									</li>
 									<?php
 									$is_first = false;
@@ -282,11 +282,11 @@ 
 							</ul>
 						</div>
 						<div class="give-right-side-block">
-							<?php foreach ( $addon_to_display as $banner ) : ?>
-								<div class="give-tab-details <?php echo ( true === $is_first ) ? ' active' : ''; ?> " id="give-addon-<?php echo esc_attr( basename( $banner['file'], '.php' ) ); ?>">
+							<?php foreach ($addon_to_display as $banner) : ?>
+								<div class="give-tab-details <?php echo (true === $is_first) ? ' active' : ''; ?> " id="give-addon-<?php echo esc_attr(basename($banner['file'], '.php')); ?>">
 									<?php
 										// Render single add banner.
-										$this->render_single_addon_banner( $banner, false );
+										$this->render_single_addon_banner($banner, false);
 										$is_first = false;
 									?>
 								</div>
@@ -295,7 +295,7 @@ 
 					</div>
 					<?php
 				} else {
-					$this->render_single_addon_banner( $addon_to_display[0], true );
+					$this->render_single_addon_banner($addon_to_display[0], true);
 				}
 				?>
 			</div>
@@ -303,12 +303,12 @@ 
 			$notice_html = ob_get_clean();
 
 			// Register notice.
-			Give()->notices->register_notice( array(
+			Give()->notices->register_notice(array(
 				'id'               => 'give_add_on_activation_notice',
 				'type'             => 'updated',
 				'description_html' => $notice_html,
 				'show'             => true,
-			) );
+			));
 		}
 	}
 
@@ -320,7 +320,7 @@ 
 	 * @param array $banner_arr Banner options.
 	 * @param bool  $is_single  Is single.
 	 */
-	private function render_single_addon_banner( $banner_arr, $is_single ) {
+	private function render_single_addon_banner($banner_arr, $is_single) {
 		// Get all give add-on.
 		$give_addons = give_get_plugins();
 
@@ -328,80 +328,80 @@ 
 		$plugin_file = $banner_arr['file'];
 
 		// Get the plugin main file.
-		foreach ( $give_addons as $main_file => $addon ) {
+		foreach ($give_addons as $main_file => $addon) {
 			// Plugin should be activated.
-			if ( ! is_plugin_active( $main_file ) ) {
+			if ( ! is_plugin_active($main_file)) {
 				continue;
 			}
 
 			if (
-				isset( $banner_arr['name'] )
+				isset($banner_arr['name'])
 				&& 'add-on' === $addon['Type']
-				&& $this->get_plugin_folder_name( $main_file ) === $this->get_plugin_folder_name( $plugin_file )
+				&& $this->get_plugin_folder_name($main_file) === $this->get_plugin_folder_name($plugin_file)
 			) {
-				$plugin_file = WP_PLUGIN_DIR . '/' . $main_file;
+				$plugin_file = WP_PLUGIN_DIR.'/'.$main_file;
 				break;
 			}
 		}
 
 		// Create dismiss URL.
 		$dismiss_url = $is_single
-			? admin_url( 'plugins.php?give_addon_activation_ignore=1&give_addon=' . sanitize_title( $banner_arr['name'] ) )
-			: admin_url( 'plugins.php?give_addon_activation_ignore=1&give_addon=all' );
+			? admin_url('plugins.php?give_addon_activation_ignore=1&give_addon='.sanitize_title($banner_arr['name']))
+			: admin_url('plugins.php?give_addon_activation_ignore=1&give_addon=all');
 
 		// Get the add-on details.
-		$plugin_data = get_plugin_data( $plugin_file );
+		$plugin_data = get_plugin_data($plugin_file);
 		?>
-		<img src="<?php echo esc_url( GIVE_PLUGIN_URL . 'assets/dist/images/give-icon-full-circle.svg' ); ?>" class="give-logo" />
+		<img src="<?php echo esc_url(GIVE_PLUGIN_URL.'assets/dist/images/give-icon-full-circle.svg'); ?>" class="give-logo" />
 		<div class="give-alert-message">
 			<h3>
 				<?php
 				printf(
 					/* translators: %s: Add-on name */
 					'%s<span>%s</span>',
-					__( 'New Give Add-on Activated: ', 'give' ),
-					esc_html( $banner_arr['name'] )
+					__('New Give Add-on Activated: ', 'give'),
+					esc_html($banner_arr['name'])
 				);
 				?>
 			</h3>
-			<a href="<?php echo esc_url( $dismiss_url ); ?>" class="dismiss">
+			<a href="<?php echo esc_url($dismiss_url); ?>" class="dismiss">
 				<span class="dashicons dashicons-dismiss"></span>
 			</a>
 			<div class="alert-actions">
 				<?php
 				//Point them to your settings page.
-				if ( ! empty( $plugin_data['Description'] ) ) {
+				if ( ! empty($plugin_data['Description'])) {
 					?>
 					<span class="give-addon-description">
-					<em><?php echo esc_html( strip_tags( $plugin_data['Description'] ) ); ?></em></span><br />
+					<em><?php echo esc_html(strip_tags($plugin_data['Description'])); ?></em></span><br />
 					<?php
 				}
-				if ( isset( $banner_arr['settings_url'] ) ) {
+				if (isset($banner_arr['settings_url'])) {
 					printf(
 						'<a href="%s"><span class="dashicons dashicons-admin-settings"></span>%s</a>',
-						esc_url( $banner_arr['settings_url'] ),
-						esc_html__( 'Go to Settings', 'give' )
+						esc_url($banner_arr['settings_url']),
+						esc_html__('Go to Settings', 'give')
 					);
 				}
 				// Show them how to configure the Addon.
-				if ( isset( $banner_arr['documentation_url'] ) ) {
+				if (isset($banner_arr['documentation_url'])) {
 					printf(
 						'<a href="%s" target="_blank"><span class="dashicons dashicons-media-text"></span>%s</a>',
-						esc_url( $banner_arr['documentation_url'] ),
+						esc_url($banner_arr['documentation_url']),
 						sprintf(
 							/* translators: %s: Add-on name */
-							esc_html__( 'Documentation: %s Add-on', 'give' ),
-							esc_html( $banner_arr['name'] )
+							esc_html__('Documentation: %s Add-on', 'give'),
+							esc_html($banner_arr['name'])
 						)
 					);
 				}
 
 				//Let them signup for plugin updates
-				if ( isset( $banner_arr['support_url'] ) ) {
+				if (isset($banner_arr['support_url'])) {
 					printf(
 						'<a href="%s" target="_blank"><span class="dashicons dashicons-sos"></span>%s</a>',
-						esc_url( $banner_arr['support_url'] ),
-						esc_html__( 'Get Support', 'give' )
+						esc_url($banner_arr['support_url']),
+						esc_html__('Get Support', 'give')
 					);
 				}
 				?>
@@ -424,30 +424,30 @@ 
 		 * See here: http://codex.wordpress.org/Function_Reference/add_user_meta
 		 */
 		if (
-			isset( $_GET['give_addon'], $_GET['give_addon_activation_ignore'] )
+			isset($_GET['give_addon'], $_GET['give_addon_activation_ignore'])
 			&& '1' === $_GET['give_addon_activation_ignore']
 		) {
 			// Get the value of the 'give_addon' query string.
-			$addon_query_arg    = sanitize_text_field( wp_unslash( $_GET['give_addon'] ) );
+			$addon_query_arg    = sanitize_text_field(wp_unslash($_GET['give_addon']));
 			$deactivated_addons = array();
 
 			// If All add-on requested to dismiss.
-			if ( 'all' === $addon_query_arg ) {
+			if ('all' === $addon_query_arg) {
 				// Get all activated add-ons.
 				$give_addons = $this->get_plugin_file_names();
 
-				if ( ! empty( $give_addons ) ) {
-					$deactivated_addons = array_keys( $give_addons );
+				if ( ! empty($give_addons)) {
+					$deactivated_addons = array_keys($give_addons);
 				}
 			} else {
 				// Store the addon to deactivate.
 				$deactivated_addons[] = $addon_query_arg;
 			}
 
-			if ( ! empty( $deactivated_addons ) ) {
-				foreach ( $deactivated_addons as $addon ) {
+			if ( ! empty($deactivated_addons)) {
+				foreach ($deactivated_addons as $addon) {
 					// Record it user meta.
-					add_user_meta( $this->user_id, "give_addon_activation_ignore_{$addon}", 'true', true );
+					add_user_meta($this->user_id, "give_addon_activation_ignore_{$addon}", 'true', true);
 				}
 			}
 		}
@@ -462,26 +462,26 @@ 
 	 */
 	public function remove_addon_activate_meta() {
 		// Get the hook name and then grab the plugin file from it.
-		$plugin_file = str_replace( 'deactivate_', '', current_action() );
+		$plugin_file = str_replace('deactivate_', '', current_action());
 
 		// Get all activated add-ons.
 		$give_addons = $this->get_plugin_file_names();
 
-		if ( ! empty( $give_addons ) ) {
-			foreach ( $give_addons as $banner_addon_name => $addon ) {
-				if ( $plugin_file === $addon['plugin_main_file'] ) {
+		if ( ! empty($give_addons)) {
+			foreach ($give_addons as $banner_addon_name => $addon) {
+				if ($plugin_file === $addon['plugin_main_file']) {
 
 					// Get the user meta key.
-					$user_id = (int) __give_get_active_by_user_meta( $banner_addon_name );
+					$user_id = (int) __give_get_active_by_user_meta($banner_addon_name);
 
-					if ( $user_id ) {
+					if ($user_id) {
 						// Get user meta for this add-on.
 						$nag_meta_key = "give_addon_activation_ignore_{$banner_addon_name}";
 
 						// Delete plugin activation option key.
-						delete_option( self::get_banner_user_meta_key( $banner_addon_name ) );
+						delete_option(self::get_banner_user_meta_key($banner_addon_name));
 						// Delete user meta of plugin activation.
-						delete_user_meta( $user_id, $nag_meta_key );
+						delete_user_meta($user_id, $nag_meta_key);
 					}
 				}
 			}
@@ -508,11 +508,11 @@ 
 	 *
 	 * @return bool|mixed|string
 	 */
-	public function get_plugin_folder_name( $main_file ) {
+	public function get_plugin_folder_name($main_file) {
 		// Remove plugin file and get the Add-on's folder name only.
-		$file_path       = explode( '/plugins/', $main_file );
-		$addon_file_path = array_pop( $file_path );
-		$addon_file_path = substr( $addon_file_path, 0, strpos( $addon_file_path, '/' ) );
+		$file_path       = explode('/plugins/', $main_file);
+		$addon_file_path = array_pop($file_path);
+		$addon_file_path = substr($addon_file_path, 0, strpos($addon_file_path, '/'));
 
 		return $addon_file_path;
 	}