WordImpress / Give

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

Spacing +98 added lines, -100 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -80,108 +80,106 @@ 
 	 */
 	public function get_data() {
 
-		$totals             = $this->get_stored_data( 'give_temp_recount_all_stats' );
-		$payment_items      = $this->get_stored_data( 'give_temp_payment_items' );
-		$processed_payments = $this->get_stored_data( 'give_temp_processed_payments' );
-		$accepted_statuses  = apply_filters( 'give_recount_accepted_statuses', array( 'publish' ) );
+		$totals             = $this->get_stored_data('give_temp_recount_all_stats');
+		$payment_items      = $this->get_stored_data('give_temp_payment_items');
+		$processed_payments = $this->get_stored_data('give_temp_processed_payments');
+		$accepted_statuses  = apply_filters('give_recount_accepted_statuses', array('publish'));
 
-		if ( false === $totals ) {
+		if (false === $totals) {
 			$totals = array();
 		}
 
-		if ( false === $payment_items ) {
+		if (false === $payment_items) {
 			$payment_items = array();
 		}
 
-		if ( false === $processed_payments ) {
+		if (false === $processed_payments) {
 			$processed_payments = array();
 		}
 
-		$all_forms = $this->get_stored_data( 'give_temp_form_ids' );
+		$all_forms = $this->get_stored_data('give_temp_form_ids');
 
-		$payments = $this->get_stored_data( 'give_temp_all_payments_data' );
+		$payments = $this->get_stored_data('give_temp_all_payments_data');
 
-		if ( empty( $payments ) ) {
-			$args = apply_filters( 'give_recount_form_stats_args', array(
+		if (empty($payments)) {
+			$args = apply_filters('give_recount_form_stats_args', array(
 				'give_forms' => $all_forms,
 				'number'     => $this->per_step,
 				'status'     => $accepted_statuses,
 				'paged'      => $this->step,
 				'output'     => 'give_payments',
-			) );
+			));
 
-			$payments_query = new Give_Payments_Query( $args );
+			$payments_query = new Give_Payments_Query($args);
 			$payments       = $payments_query->get_payments();
 		}
 
-		if ( ! empty( $payments ) ) {
+		if ( ! empty($payments)) {
 
 			//Loop through payments
-			foreach ( $payments as $payment ) {
+			foreach ($payments as $payment) {
 
-				$payment_id = ( ! empty( $payment['ID'] ) ? absint( $payment['ID'] ) : ( ! empty( $payment->ID ) ? absint( $payment->ID ) : false ) );
-				$payment = new Give_Payment( $payment_id );
+				$payment_id = ( ! empty($payment['ID']) ? absint($payment['ID']) : ( ! empty($payment->ID) ? absint($payment->ID) : false));
+				$payment = new Give_Payment($payment_id);
 
 				// Prevent payments that have all ready been retrieved from a previous sales log from counting again.
-				if ( in_array( $payment->ID, $processed_payments ) ) {
+				if (in_array($payment->ID, $processed_payments)) {
 					continue;
 				}
 
 				// Verify accepted status.
-				if ( ! in_array( $payment->post_status, $accepted_statuses ) ) {
+				if ( ! in_array($payment->post_status, $accepted_statuses)) {
 					$processed_payments[] = $payment->ID;
 					continue;
 				}
 
-				$payment_item = $payment_items[ $payment->ID ];
+				$payment_item = $payment_items[$payment->ID];
 
-				$form_id = isset( $payment_item['id'] ) ? $payment_item['id'] : '';
+				$form_id = isset($payment_item['id']) ? $payment_item['id'] : '';
 
 				// Must have a form ID.
-				if ( empty( $form_id ) ) {
+				if (empty($form_id)) {
 					continue;
 				}
 
 				// Form ID must be within $all_forms array to be validated.
-				if ( ! in_array( $form_id, $all_forms ) ) {
+				if ( ! in_array($form_id, $all_forms)) {
 					continue;
 				}
 
 				// Set Sales count
-				$totals[ $form_id ]['sales'] = isset( $totals[ $form_id ]['sales'] ) ?
-					++ $totals[ $form_id ]['sales'] :
-					1;
+				$totals[$form_id]['sales'] = isset($totals[$form_id]['sales']) ?
+					++$totals[$form_id]['sales'] : 1;
 
 				// Set Total Earnings
-				$totals[ $form_id ]['earnings'] = isset( $totals[ $form_id ]['earnings'] ) ?
-					( $totals[ $form_id ]['earnings'] + $payment_item['price'] ) :
-					$payment_item['price'];
+				$totals[$form_id]['earnings'] = isset($totals[$form_id]['earnings']) ?
+					($totals[$form_id]['earnings'] + $payment_item['price']) : $payment_item['price'];
 
 				$processed_payments[] = $payment->ID;
 			}
 
 			// Get the list of form ids which does not contain any payment record.
-			$remaining_form_ids = array_diff( $all_forms, array_keys( $totals ) );
-			foreach ( $remaining_form_ids as $form_id ) {
+			$remaining_form_ids = array_diff($all_forms, array_keys($totals));
+			foreach ($remaining_form_ids as $form_id) {
 				//If array key doesn't exist, create it
-				if ( ! array_key_exists( $form_id, $totals ) ) {
-					$totals[ $form_id ] = array(
+				if ( ! array_key_exists($form_id, $totals)) {
+					$totals[$form_id] = array(
 						'sales'    => (int) 0,
 						'earnings' => (float) 0,
 					);
 				}
 			}
 
-			$this->store_data( 'give_temp_processed_payments', $processed_payments );
-			$this->store_data( 'give_temp_recount_all_stats', $totals );
+			$this->store_data('give_temp_processed_payments', $processed_payments);
+			$this->store_data('give_temp_recount_all_stats', $totals);
 
 			return true;
 		}
 
 
-		foreach ( $totals as $key => $stats ) {
-			give_update_meta( $key, '_give_form_sales', $stats['sales'] );
-			give_update_meta( $key, '_give_form_earnings', give_sanitize_amount_for_db( $stats['earnings'] ) );
+		foreach ($totals as $key => $stats) {
+			give_update_meta($key, '_give_form_sales', $stats['sales']);
+			give_update_meta($key, '_give_form_earnings', give_sanitize_amount_for_db($stats['earnings']));
 		}
 
 		return false;
@@ -196,20 +194,20 @@ 
 	 */
 	public function get_percentage_complete() {
 
-		$total = $this->get_stored_data( 'give_recount_all_total' );
+		$total = $this->get_stored_data('give_recount_all_total');
 
-		if ( false === $total ) {
+		if (false === $total) {
 			$this->pre_fetch();
-			$total = $this->get_stored_data( 'give_recount_all_total' );
+			$total = $this->get_stored_data('give_recount_all_total');
 		}
 
 		$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;
 		}
 
@@ -223,8 +221,8 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
-		$this->form_id = isset( $request['form_id'] ) ? sanitize_text_field( $request['form_id'] ) : false;
+	public function set_properties($request) {
+		$this->form_id = isset($request['form_id']) ? sanitize_text_field($request['form_id']) : false;
 	}
 
 	/**
@@ -235,25 +233,25 @@ 
 	 */
 	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 {
-			$this->delete_data( 'give_recount_total_' . $this->form_id );
-			$this->delete_data( 'give_recount_all_total' );
-			$this->delete_data( 'give_temp_recount_all_stats' );
-			$this->delete_data( 'give_temp_payment_items' );
-			$this->delete_data( 'give_temp_form_ids' );
-			$this->delete_data( 'give_temp_processed_payments' );
+			$this->delete_data('give_recount_total_'.$this->form_id);
+			$this->delete_data('give_recount_all_total');
+			$this->delete_data('give_temp_recount_all_stats');
+			$this->delete_data('give_temp_payment_items');
+			$this->delete_data('give_temp_form_ids');
+			$this->delete_data('give_temp_processed_payments');
 			$this->done    = true;
-			$this->message = esc_html__( 'Donation form income amounts and donation counts stats successfully recounted.', 'give' );
+			$this->message = esc_html__('Donation form income amounts and donation counts stats successfully recounted.', 'give');
 
 			return false;
 		}
@@ -263,10 +261,10 @@ 
 	 * Set headers.
 	 */
 	public function headers() {
-		ignore_user_abort( true );
+		ignore_user_abort(true);
 
-		if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-			set_time_limit( 0 );
+		if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+			set_time_limit(0);
 		}
 	}
 
@@ -293,70 +291,70 @@ 
 	 */
 	public function pre_fetch() {
 
-		if ( 1 == $this->step ) {
-			$this->delete_data( 'give_temp_recount_all_total' );
-			$this->delete_data( 'give_temp_recount_all_stats' );
-			$this->delete_data( 'give_temp_payment_items' );
-			$this->delete_data( 'give_temp_processed_payments' );
-			$this->delete_data( 'give_temp_all_payments_data' );
+		if (1 == $this->step) {
+			$this->delete_data('give_temp_recount_all_total');
+			$this->delete_data('give_temp_recount_all_stats');
+			$this->delete_data('give_temp_payment_items');
+			$this->delete_data('give_temp_processed_payments');
+			$this->delete_data('give_temp_all_payments_data');
 		}
 
-		$accepted_statuses = apply_filters( 'give_recount_accepted_statuses', array( 'publish' ) );
-		$total             = $this->get_stored_data( 'give_temp_recount_all_total' );
+		$accepted_statuses = apply_filters('give_recount_accepted_statuses', array('publish'));
+		$total             = $this->get_stored_data('give_temp_recount_all_total');
 
-		if ( false === $total ) {
+		if (false === $total) {
 
-			$payment_items = $this->get_stored_data( 'give_temp_payment_items' );
+			$payment_items = $this->get_stored_data('give_temp_payment_items');
 
-			if ( false === $payment_items ) {
+			if (false === $payment_items) {
 				$payment_items = array();
-				$this->store_data( 'give_temp_payment_items', $payment_items );
+				$this->store_data('give_temp_payment_items', $payment_items);
 			}
 
 			$args = array(
 				'post_status'    => 'publish',
 				'post_type'      => 'give_forms',
-				'posts_per_page' => - 1,
+				'posts_per_page' => -1,
 				'fields'         => 'ids',
 			);
 
-			$all_forms = get_posts( $args );
+			$all_forms = get_posts($args);
 
-			$this->store_data( 'give_temp_form_ids', $all_forms );
+			$this->store_data('give_temp_form_ids', $all_forms);
 
-			$args = apply_filters( 'give_recount_form_stats_total_args', array(
+			$args = apply_filters('give_recount_form_stats_total_args', array(
 				'give_forms' => $all_forms,
 				'number'     => $this->per_step,
 				'status'     => $accepted_statuses,
 				'page'       => $this->step,
 				'output'     => 'payments',
-			) );
+			));
 
-			$payments_query = new Give_Payments_Query( $args );
+			$payments_query = new Give_Payments_Query($args);
 			$payments       = $payments_query->get_payments();
 
-			$total = wp_count_posts( 'give_payment' )->publish;
+			$total = wp_count_posts('give_payment')->publish;
 
-			$this->store_data( 'give_temp_all_payments_data', $payments );
+			$this->store_data('give_temp_all_payments_data', $payments);
 
-			if ( $payments ) {
+			if ($payments) {
 
-				foreach ( $payments as $payment ) {
+				foreach ($payments as $payment) {
 
 					$form_id = $payment->form_id;
 
 					//If for some reason somehow the form_ID isn't set check payment meta
-					if ( empty( $payment->form_id ) ) {
+					if (empty($payment->form_id)) {
 						$payment_meta = $payment->get_meta();
-						$form_id      = isset( $payment_meta['form_id'] ) ? $payment_meta['form_id'] : 0;
+						$form_id      = isset($payment_meta['form_id']) ? $payment_meta['form_id'] : 0;
 					}
 
-					if ( ! in_array( $payment->post_status, $accepted_statuses ) ) {
+					if ( ! in_array($payment->post_status, $accepted_statuses)) {
 						continue;
 					}
 
-					if ( ! array_key_exists( $payment->ID, $payment_items ) ) {
-						$payment_items[ $payment->ID ] = array(
+					if ( ! array_key_exists($payment->ID, $payment_items)) {
+						$payment_items[$payment->ID] = array(
 							'id'         => $form_id,
 							'payment_id' => $payment->ID,
 							'price'      => $payment->total,
@@ -364,8 +362,8 @@ 
 					}
 				}
 			}
-			$this->store_data( 'give_temp_payment_items', $payment_items );
-			$this->store_data( 'give_recount_all_total', $total );
+			$this->store_data('give_temp_payment_items', $payment_items);
+			$this->store_data('give_recount_all_total', $total);
 		}
 
 	}
@@ -379,17 +377,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 ) );
-		if ( empty( $value ) ) {
+		$value = $wpdb->get_var($wpdb->prepare("SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key));
+		if (empty($value)) {
 			return false;
 		}
 
-		$maybe_json = json_decode( $value );
+		$maybe_json = json_decode($value);
 
-		if ( ! is_null( $maybe_json ) ) {
-			$value = json_decode( $value, true );
+		if ( ! is_null($maybe_json)) {
+			$value = json_decode($value, true);
 		}
 
 		return $value;
@@ -405,10 +403,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,
@@ -422,7 +420,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -434,9 +432,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/admin/tools/data/class-give-tools-recount-single-donor-stats.php

Spacing +74 added lines, -74 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -53,40 +53,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 = give_donation_amount( $payment->ID );
-						$step_total     += $payment_amount;
+					if (apply_filters('give_donor_recount_should_increase_value', true, $payment)) {
+						$payment_amount = give_donation_amount($payment->ID);
+						$step_total += $payment_amount;
 					}
 
 				}
@@ -94,8 +94,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;
 		}
@@ -112,16 +112,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;
 		}
 
@@ -135,8 +135,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;
 	}
 
 	/**
@@ -147,63 +147,63 @@ 
 	 */
 	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;
 		}
 	}
 
 	public function headers() {
-		ignore_user_abort( true );
+		ignore_user_abort(true);
 
-		if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-			set_time_limit( 0 );
+		if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+			set_time_limit(0);
 		}
 	}
 
@@ -230,26 +230,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(
@@ -259,11 +259,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);
 		}
 	}
 
@@ -276,17 +276,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;
@@ -302,10 +302,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,
@@ -319,7 +319,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -331,9 +331,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/admin/tools/data/class-give-tools-delete-test-transactions.php

Spacing +48 added lines, -48 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -53,19 +53,19 @@ 
 	 * @return array|bool $data The data for the CSV file
 	 */
 	public function get_data() {
-		$items = $this->get_stored_data( 'give_temp_delete_test_ids' );
+		$items = $this->get_stored_data('give_temp_delete_test_ids');
 
-		if ( ! is_array( $items ) ) {
+		if ( ! is_array($items)) {
 			return false;
 		}
 
-		$offset     = ( $this->step - 1 ) * $this->per_step;
-		$step_items = array_slice( $items, $offset, $this->per_step );
+		$offset     = ($this->step - 1) * $this->per_step;
+		$step_items = array_slice($items, $offset, $this->per_step);
 
-		if ( $step_items ) {
-			foreach ( $step_items as $item ) {
+		if ($step_items) {
+			foreach ($step_items as $item) {
 				// Delete the main payment.
-				give_delete_donation( absint( $item['id'] ) );
+				give_delete_donation(absint($item['id']));
 			}
 			return true;
 		}
@@ -81,16 +81,16 @@ 
 	 */
 	public function get_percentage_complete() {
 
-		$items = $this->get_stored_data( 'give_temp_delete_test_ids', false );
-		$total = count( $items );
+		$items = $this->get_stored_data('give_temp_delete_test_ids', false);
+		$total = count($items);
 
 		$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;
 		}
 
@@ -104,7 +104,7 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 	}
 
 	/**
@@ -115,29 +115,29 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( __( 'You do not have permission to delete test transactions.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(__('You do not have permission to delete test transactions.', 'give'), __('Error', 'give'), array('response' => 403));
 		}
 
 		$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 ) );
-			Give_Cache::delete( Give_Cache::get_key( 'give_estimated_monthly_stats' ) );
+			update_option('give_earnings_total', give_get_total_earnings(true));
+			Give_Cache::delete(Give_Cache::get_key('give_estimated_monthly_stats'));
 
-			$this->delete_data( 'give_temp_delete_test_ids' );
+			$this->delete_data('give_temp_delete_test_ids');
 
 			// Reset the sequential order numbers
-			if ( give_get_option( 'enable_sequential' ) ) {
-				delete_option( 'give_last_payment_number' );
+			if (give_get_option('enable_sequential')) {
+				delete_option('give_last_payment_number');
 			}
 
 			$this->done    = true;
-			$this->message = __( 'Test transactions successfully deleted.', 'give' );
+			$this->message = __('Test transactions successfully deleted.', 'give');
 
 			return false;
 		}
@@ -147,10 +147,10 @@ 
 	 * Headers
 	 */
 	public function headers() {
-		ignore_user_abort( true );
+		ignore_user_abort(true);
 
-		if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-			set_time_limit( 0 );
+		if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+			set_time_limit(0);
 		}
 	}
 
@@ -174,27 +174,27 @@ 
 	 */
 	public function pre_fetch() {
 
-		if ( $this->step == 1 ) {
-			$this->delete_data( 'give_temp_delete_test_ids' );
+		if ($this->step == 1) {
+			$this->delete_data('give_temp_delete_test_ids');
 		}
 
-		$items = get_option( 'give_temp_delete_test_ids', false );
+		$items = get_option('give_temp_delete_test_ids', false);
 
-		if ( false === $items ) {
+		if (false === $items) {
 			$items = array();
 
-			$args = apply_filters( 'give_tools_reset_stats_total_args', array(
+			$args = apply_filters('give_tools_reset_stats_total_args', array(
 				'post_status' => 'any',
-				'number'      => - 1,
+				'number'      => -1,
 				'meta_key'    => '_give_payment_mode',
 				'meta_value'  => 'test'
-			) );
+			));
 
-			$posts    = new Give_Payments_Query( $args );
+			$posts    = new Give_Payments_Query($args);
 			$payments = $posts->get_payments();
 
 			/* @var Give_Payment $payment */
-			foreach ( $payments as $payment ) {
+			foreach ($payments as $payment) {
 				$items[] = array(
 					'id'   => (int) $payment->ID,
 					'type' => 'give_payment',
@@ -203,9 +203,9 @@ 
 			
 			// Allow filtering of items to remove with an unassociative array for each item.
 			// The array contains the unique ID of the item, and a 'type' for you to use in the execution of the get_data method.
-			$items = apply_filters( 'give_delete_test_items', $items );
+			$items = apply_filters('give_delete_test_items', $items);
 
-			$this->store_data( 'give_temp_delete_test_ids', $items );
+			$this->store_data('give_temp_delete_test_ids', $items);
 		}
 
 	}
@@ -219,17 +219,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;
@@ -245,10 +245,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,
@@ -262,7 +262,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -274,9 +274,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/admin/tools/data/class-give-tools-recount-donor-stats.php

Spacing +38 added lines, -38 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -57,32 +57,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(
@@ -93,29 +93,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 += give_donation_amount( $payment->ID );
+							if (apply_filters('give_donor_recount_should_increase_value', true, $payment)) {
+								$purchase_value += give_donation_amount($payment->ID);
 							}
 
-							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++;
 							}
 						}
 
@@ -123,7 +123,7 @@ 
 					}
 				}
 
-				$payment_ids = implode( ',', $payment_ids );
+				$payment_ids = implode(',', $payment_ids);
 
 				$donor_update_data = array(
 					'purchase_count' => $purchase_count,
@@ -131,8 +131,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().
 
@@ -152,21 +152,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;
 		}
 
@@ -180,7 +180,7 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 	}
 
 	/**
@@ -191,21 +191,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;
 		}
@@ -215,10 +215,10 @@ 
 	 * Headers
 	 */
 	public function headers() {
-		ignore_user_abort( true );
+		ignore_user_abort(true);
 
-		if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-			set_time_limit( 0 );
+		if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+			set_time_limit(0);
 		}
 	}
 

includes/admin/tools/data/class-give-tools-recount-income.php

Spacing +48 added lines, -48 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -53,49 +53,49 @@ 
 	 */
 	public function get_data() {
 
-		if ( $this->step == 1 ) {
-			$this->delete_data( 'give_temp_recount_earnings' );
+		if ($this->step == 1) {
+			$this->delete_data('give_temp_recount_earnings');
 		}
 
-		$total = get_option( 'give_temp_recount_earnings', false );
+		$total = get_option('give_temp_recount_earnings', false);
 
-		if ( false === $total ) {
+		if (false === $total) {
 			$total = (float) 0;
-			$this->store_data( 'give_temp_recount_earnings', $total );
+			$this->store_data('give_temp_recount_earnings', $total);
 		}
 
-		$accepted_statuses = apply_filters( 'give_recount_accepted_statuses', array( 'publish' ) );
+		$accepted_statuses = apply_filters('give_recount_accepted_statuses', array('publish'));
 
-		$args = apply_filters( 'give_recount_earnings_args', array(
+		$args = apply_filters('give_recount_earnings_args', array(
 			'number' => $this->per_step,
 			'page'   => $this->step,
 			'status' => $accepted_statuses,
 			'fields' => 'ids'
-		) );
+		));
 
-		$payments = give_get_payments( $args );
+		$payments = give_get_payments($args);
 
-		if ( ! empty( $payments ) ) {
+		if ( ! empty($payments)) {
 
-			foreach ( $payments as $payment ) {
+			foreach ($payments as $payment) {
 
-				$total += give_donation_amount( $payment );
+				$total += give_donation_amount($payment);
 
 			}
 
-			if ( $total < 0 ) {
+			if ($total < 0) {
 				$totals = 0;
 			}
 
-			$total = round( $total, give_get_price_decimals() );
+			$total = round($total, give_get_price_decimals());
 
-			$this->store_data( 'give_temp_recount_earnings', $total );
+			$this->store_data('give_temp_recount_earnings', $total);
 
 			return true;
 
 		}
 
-		update_option( 'give_earnings_total', $total );
+		update_option('give_earnings_total', $total);
 
 		return false;
 
@@ -109,25 +109,25 @@ 
 	 */
 	public function get_percentage_complete() {
 
-		$total = $this->get_stored_data( 'give_recount_earnings_total' );
+		$total = $this->get_stored_data('give_recount_earnings_total');
 
-		if ( false === $total ) {
-			$args = apply_filters( 'give_recount_earnings_total_args', array() );
+		if (false === $total) {
+			$args = apply_filters('give_recount_earnings_total_args', array());
 
-			$counts = give_count_payments( $args );
-			$total  = absint( $counts->publish );
-			$total  = apply_filters( 'give_recount_store_earnings_total', $total );
+			$counts = give_count_payments($args);
+			$total  = absint($counts->publish);
+			$total  = apply_filters('give_recount_store_earnings_total', $total);
 
-			$this->store_data( 'give_recount_earnings_total', $total );
+			$this->store_data('give_recount_earnings_total', $total);
 		}
 
 		$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;
 		}
 
@@ -141,7 +141,7 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 	}
 
 	/**
@@ -152,21 +152,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( '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 {
-			$this->delete_data( 'give_recount_earnings_total' );
-			$this->delete_data( 'give_temp_recount_earnings' );
+			$this->delete_data('give_recount_earnings_total');
+			$this->delete_data('give_temp_recount_earnings');
 			$this->done    = true;
-			$this->message = esc_html__( 'Income stats have been successfully recounted.', 'give' );
+			$this->message = esc_html__('Income stats have been successfully recounted.', 'give');
 
 			return false;
 		}
@@ -176,10 +176,10 @@ 
 	 * Headers.
 	 */
 	public function headers() {
-		ignore_user_abort( true );
+		ignore_user_abort(true);
 
-		if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-			set_time_limit( 0 );
+		if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+			set_time_limit(0);
 		}
 	}
 
@@ -207,17 +207,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;
@@ -233,10 +233,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,
@@ -250,7 +250,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -262,9 +262,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/admin/upgrades/upgrade-functions.php

Spacing +224 added lines, -224 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,59 +25,59 @@ 
  */
 function give_do_automatic_upgrades() {
 	$did_upgrade  = false;
-	$give_version = preg_replace( '/[^0-9.].*/', '', get_option( 'give_version' ) );
+	$give_version = preg_replace('/[^0-9.].*/', '', get_option('give_version'));
 
-	if ( ! $give_version ) {
+	if ( ! $give_version) {
 		// 1.0 is the first version to use this option so we must add it.
 		$give_version = '1.0';
 	}
 
-	switch ( true ) {
+	switch (true) {
 
-		case version_compare( $give_version, '1.6', '<' ) :
+		case version_compare($give_version, '1.6', '<') :
 			give_v16_upgrades();
 			$did_upgrade = true;
 
-		case version_compare( $give_version, '1.7', '<' ) :
+		case version_compare($give_version, '1.7', '<') :
 			give_v17_upgrades();
 			$did_upgrade = true;
 
-		case version_compare( $give_version, '1.8', '<' ) :
+		case version_compare($give_version, '1.8', '<') :
 			give_v18_upgrades();
 			$did_upgrade = true;
 
-		case version_compare( $give_version, '1.8.7', '<' ) :
+		case version_compare($give_version, '1.8.7', '<') :
 			give_v187_upgrades();
 			$did_upgrade = true;
 
-		case version_compare( $give_version, '1.8.8', '<' ) :
+		case version_compare($give_version, '1.8.8', '<') :
 			give_v188_upgrades();
 			$did_upgrade = true;
 
-		case version_compare( $give_version, '1.8.9', '<' ) :
+		case version_compare($give_version, '1.8.9', '<') :
 			give_v189_upgrades();
 			$did_upgrade = true;
 
-		case version_compare( $give_version, '1.8.12', '<' ) :
+		case version_compare($give_version, '1.8.12', '<') :
 			give_v1812_upgrades();
 			$did_upgrade = true;
 
-		case version_compare( $give_version, '1.8.13', '<' ) :
+		case version_compare($give_version, '1.8.13', '<') :
 			give_v1813_upgrades();
 			$did_upgrade = true;
 
-		case version_compare( $give_version, '1.8.17', '<' ) :
+		case version_compare($give_version, '1.8.17', '<') :
 			give_v1817_upgrades();
 			$did_upgrade = true;
 	}
 
-	if ( $did_upgrade ) {
-		update_option( 'give_version', preg_replace( '/[^0-9.].*/', '', GIVE_VERSION ) );
+	if ($did_upgrade) {
+		update_option('give_version', preg_replace('/[^0-9.].*/', '', GIVE_VERSION));
 	}
 }
 
-add_action( 'admin_init', 'give_do_automatic_upgrades' );
-add_action( 'give_upgrades', 'give_do_automatic_upgrades' );
+add_action('admin_init', 'give_do_automatic_upgrades');
+add_action('give_upgrades', 'give_do_automatic_upgrades');
 
 /**
  * Display Upgrade Notices
@@ -89,7 +89,7 @@ 
  *
  * @return void
  */
-function give_show_upgrade_notices( $give_updates ) {
+function give_show_upgrade_notices($give_updates) {
 	// v1.3.2 Upgrades
 	$give_updates->register(
 		array(
@@ -155,21 +155,21 @@ 
 	);
 
 	// v1.8.17 Upgrades for donations.
-	$give_updates->register( array(
+	$give_updates->register(array(
 		'id'       => 'v1817_update_donation_iranian_currency_code',
 		'version'  => '1.8.17',
 		'callback' => 'give_v1817_update_donation_iranian_currency_code',
-	) );
+	));
 
 	// v1.8.17 Upgrades for cleanup of user roles.
-	$give_updates->register( array(
+	$give_updates->register(array(
 		'id'       => 'v1817_cleanup_user_roles',
 		'version'  => '1.8.17',
 		'callback' => 'give_v1817_cleanup_user_roles',
-	) );
+	));
 }
 
-add_action( 'give_register_updates', 'give_show_upgrade_notices' );
+add_action('give_register_updates', 'give_show_upgrade_notices');
 
 /**
  * Triggers all upgrade functions
@@ -181,29 +181,29 @@ 
  */
 function give_trigger_upgrades() {
 
-	if ( ! current_user_can( 'manage_give_settings' ) ) {
-		wp_die( esc_html__( 'You do not have permission to do Give upgrades.', 'give' ), esc_html__( 'Error', 'give' ), array(
+	if ( ! current_user_can('manage_give_settings')) {
+		wp_die(esc_html__('You do not have permission to do Give upgrades.', 'give'), esc_html__('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	$give_version = get_option( 'give_version' );
+	$give_version = get_option('give_version');
 
-	if ( ! $give_version ) {
+	if ( ! $give_version) {
 		// 1.0 is the first version to use this option so we must add it.
 		$give_version = '1.0';
-		add_option( 'give_version', $give_version );
+		add_option('give_version', $give_version);
 	}
 
-	update_option( 'give_version', GIVE_VERSION );
-	delete_option( 'give_doing_upgrade' );
+	update_option('give_version', GIVE_VERSION);
+	delete_option('give_doing_upgrade');
 
-	if ( DOING_AJAX ) {
-		die( 'complete' );
+	if (DOING_AJAX) {
+		die('complete');
 	} // End if().
 }
 
-add_action( 'wp_ajax_give_trigger_upgrades', 'give_trigger_upgrades' );
+add_action('wp_ajax_give_trigger_upgrades', 'give_trigger_upgrades');
 
 
 /**
@@ -219,24 +219,24 @@ 
 	/* @var Give_Updates $give_updates */
 	$give_updates = Give_Updates::get_instance();
 
-	if ( ! current_user_can( 'manage_give_settings' ) ) {
-		wp_die( esc_html__( 'You do not have permission to do Give upgrades.', 'give' ), esc_html__( 'Error', 'give' ), array(
+	if ( ! current_user_can('manage_give_settings')) {
+		wp_die(esc_html__('You do not have permission to do Give upgrades.', 'give'), esc_html__('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	ignore_user_abort( true );
+	ignore_user_abort(true);
 
-	if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-		@set_time_limit( 0 );
+	if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+		@set_time_limit(0);
 	}
 
 	// UPDATE DB METAKEYS.
 	$sql   = "UPDATE $wpdb->postmeta SET meta_key = '_give_payment_customer_id' WHERE meta_key = '_give_payment_donor_id'";
-	$query = $wpdb->query( $sql );
+	$query = $wpdb->query($sql);
 
 	$give_updates->percentage = 100;
-	give_set_upgrade_complete( 'upgrade_give_payment_customer_id' );
+	give_set_upgrade_complete('upgrade_give_payment_customer_id');
 }
 
 
@@ -260,24 +260,24 @@ 
 	$where  .= "AND ( p.post_status = 'abandoned' )";
 	$where  .= "AND ( m.meta_key = '_give_payment_gateway' AND m.meta_value = 'offline' )";
 
-	$sql            = $select . $join . $where;
-	$found_payments = $wpdb->get_col( $sql );
+	$sql            = $select.$join.$where;
+	$found_payments = $wpdb->get_col($sql);
 
-	foreach ( $found_payments as $payment ) {
+	foreach ($found_payments as $payment) {
 
 		// Only change ones marked abandoned since our release last week because the admin may have marked some abandoned themselves.
-		$modified_time = get_post_modified_time( 'U', false, $payment );
+		$modified_time = get_post_modified_time('U', false, $payment);
 
 		// 1450124863 =  12/10/2015 20:42:25.
-		if ( $modified_time >= 1450124863 ) {
+		if ($modified_time >= 1450124863) {
 
-			give_update_payment_status( $payment, 'pending' );
+			give_update_payment_status($payment, 'pending');
 
 		}
 	}
 
 	$give_updates->percentage = 100;
-	give_set_upgrade_complete( 'upgrade_give_offline_status' );
+	give_set_upgrade_complete('upgrade_give_offline_status');
 }
 
 
@@ -290,17 +290,17 @@ 
  */
 function give_v152_cleanup_users() {
 
-	$give_version = get_option( 'give_version' );
+	$give_version = get_option('give_version');
 
-	if ( ! $give_version ) {
+	if ( ! $give_version) {
 		// 1.0 is the first version to use this option so we must add it.
 		$give_version = '1.0';
 	}
 
-	$give_version = preg_replace( '/[^0-9.].*/', '', $give_version );
+	$give_version = preg_replace('/[^0-9.].*/', '', $give_version);
 
 	// v1.5.2 Upgrades
-	if ( version_compare( $give_version, '1.5.2', '<' ) || ! give_has_upgrade_completed( 'upgrade_give_user_caps_cleanup' ) ) {
+	if (version_compare($give_version, '1.5.2', '<') || ! give_has_upgrade_completed('upgrade_give_user_caps_cleanup')) {
 
 		// Delete all caps with "ss".
 		// Also delete all unused "campaign" roles.
@@ -347,9 +347,9 @@ 
 		);
 
 		global $wp_roles;
-		foreach ( $delete_caps as $cap ) {
-			foreach ( array_keys( $wp_roles->roles ) as $role ) {
-				$wp_roles->remove_cap( $role, $cap );
+		foreach ($delete_caps as $cap) {
+			foreach (array_keys($wp_roles->roles) as $role) {
+				$wp_roles->remove_cap($role, $cap);
 			}
 		}
 
@@ -359,15 +359,15 @@ 
 		$roles->add_caps();
 
 		// The Update Ran.
-		update_option( 'give_version', preg_replace( '/[^0-9.].*/', '', GIVE_VERSION ) );
-		give_set_upgrade_complete( 'upgrade_give_user_caps_cleanup' );
-		delete_option( 'give_doing_upgrade' );
+		update_option('give_version', preg_replace('/[^0-9.].*/', '', GIVE_VERSION));
+		give_set_upgrade_complete('upgrade_give_user_caps_cleanup');
+		delete_option('give_doing_upgrade');
 
 	}// End if().
 
 }
 
-add_action( 'admin_init', 'give_v152_cleanup_users' );
+add_action('admin_init', 'give_v152_cleanup_users');
 
 /**
  * 1.6 Upgrade routine to create the customer meta table.
@@ -410,53 +410,53 @@ 
 
 	// Get addons license key.
 	$addons = array();
-	foreach ( $give_options as $key => $value ) {
-		if ( false !== strpos( $key, '_license_key' ) ) {
-			$addons[ $key ] = $value;
+	foreach ($give_options as $key => $value) {
+		if (false !== strpos($key, '_license_key')) {
+			$addons[$key] = $value;
 		}
 	}
 
 	// Bailout: We do not have any addon license data to upgrade.
-	if ( empty( $addons ) ) {
+	if (empty($addons)) {
 		return false;
 	}
 
-	foreach ( $addons as $key => $addon_license ) {
+	foreach ($addons as $key => $addon_license) {
 
 		// Get addon shortname.
-		$shortname = str_replace( '_license_key', '', $key );
+		$shortname = str_replace('_license_key', '', $key);
 
 		// Addon license option name.
-		$addon_license_option_name = $shortname . '_license_active';
+		$addon_license_option_name = $shortname.'_license_active';
 
 		// bailout if license is empty.
-		if ( empty( $addon_license ) ) {
-			delete_option( $addon_license_option_name );
+		if (empty($addon_license)) {
+			delete_option($addon_license_option_name);
 			continue;
 		}
 
 		// Get addon name.
 		$addon_name       = array();
-		$addon_name_parts = explode( '_', str_replace( 'give_', '', $shortname ) );
-		foreach ( $addon_name_parts as $name_part ) {
+		$addon_name_parts = explode('_', str_replace('give_', '', $shortname));
+		foreach ($addon_name_parts as $name_part) {
 
 			// Fix addon name
-			switch ( $name_part ) {
+			switch ($name_part) {
 				case 'authorizenet' :
 					$name_part = 'authorize.net';
 					break;
 			}
 
-			$addon_name[] = ucfirst( $name_part );
+			$addon_name[] = ucfirst($name_part);
 		}
 
-		$addon_name = implode( ' ', $addon_name );
+		$addon_name = implode(' ', $addon_name);
 
 		// Data to send to the API.
 		$api_params = array(
 			'edd_action' => 'activate_license', // never change from "edd_" to "give_"!
 			'license'    => $addon_license,
-			'item_name'  => urlencode( $addon_name ),
+			'item_name'  => urlencode($addon_name),
 			'url'        => home_url(),
 		);
 
@@ -471,17 +471,17 @@ 
 		);
 
 		// Make sure there are no errors.
-		if ( is_wp_error( $response ) ) {
-			delete_option( $addon_license_option_name );
+		if (is_wp_error($response)) {
+			delete_option($addon_license_option_name);
 			continue;
 		}
 
 		// Tell WordPress to look for updates.
-		set_site_transient( 'update_plugins', null );
+		set_site_transient('update_plugins', null);
 
 		// Decode license data.
-		$license_data = json_decode( wp_remote_retrieve_body( $response ) );
-		update_option( $addon_license_option_name, $license_data );
+		$license_data = json_decode(wp_remote_retrieve_body($response));
+		update_option($addon_license_option_name, $license_data);
 	}// End foreach().
 }
 
@@ -511,9 +511,9 @@ 
 	);
 
 	global $wp_roles;
-	foreach ( $delete_caps as $cap ) {
-		foreach ( array_keys( $wp_roles->roles ) as $role ) {
-			$wp_roles->remove_cap( $role, $cap );
+	foreach ($delete_caps as $cap) {
+		foreach (array_keys($wp_roles->roles) as $role) {
+			$wp_roles->remove_cap($role, $cap);
 		}
 	}
 
@@ -547,7 +547,7 @@ 
 function give_v18_upgrades_core_setting() {
 	// Core settings which changes from checkbox to radio.
 	$core_setting_names = array_merge(
-		array_keys( give_v18_renamed_core_settings() ),
+		array_keys(give_v18_renamed_core_settings()),
 		array(
 			'uninstall_on_delete',
 			'scripts_footer',
@@ -559,48 +559,48 @@ 
 	);
 
 	// Bailout: If not any setting define.
-	if ( $give_settings = get_option( 'give_settings' ) ) {
+	if ($give_settings = get_option('give_settings')) {
 
 		$setting_changed = false;
 
 		// Loop: check each setting field.
-		foreach ( $core_setting_names as $setting_name ) {
+		foreach ($core_setting_names as $setting_name) {
 			// New setting name.
-			$new_setting_name = preg_replace( '/^(enable_|disable_)/', '', $setting_name );
+			$new_setting_name = preg_replace('/^(enable_|disable_)/', '', $setting_name);
 
 			// Continue: If setting already set.
 			if (
-				array_key_exists( $new_setting_name, $give_settings )
-				&& in_array( $give_settings[ $new_setting_name ], array( 'enabled', 'disabled' ) )
+				array_key_exists($new_setting_name, $give_settings)
+				&& in_array($give_settings[$new_setting_name], array('enabled', 'disabled'))
 			) {
 				continue;
 			}
 
 			// Set checkbox value to radio value.
-			$give_settings[ $setting_name ] = ( ! empty( $give_settings[ $setting_name ] ) && 'on' === $give_settings[ $setting_name ] ? 'enabled' : 'disabled' );
+			$give_settings[$setting_name] = ( ! empty($give_settings[$setting_name]) && 'on' === $give_settings[$setting_name] ? 'enabled' : 'disabled');
 
 			// @see https://github.com/WordImpress/Give/issues/1063.
-			if ( false !== strpos( $setting_name, 'disable_' ) ) {
+			if (false !== strpos($setting_name, 'disable_')) {
 
-				$give_settings[ $new_setting_name ] = ( give_is_setting_enabled( $give_settings[ $setting_name ] ) ? 'disabled' : 'enabled' );
-			} elseif ( false !== strpos( $setting_name, 'enable_' ) ) {
+				$give_settings[$new_setting_name] = (give_is_setting_enabled($give_settings[$setting_name]) ? 'disabled' : 'enabled');
+			} elseif (false !== strpos($setting_name, 'enable_')) {
 
-				$give_settings[ $new_setting_name ] = ( give_is_setting_enabled( $give_settings[ $setting_name ] ) ? 'enabled' : 'disabled' );
+				$give_settings[$new_setting_name] = (give_is_setting_enabled($give_settings[$setting_name]) ? 'enabled' : 'disabled');
 			}
 
 			// Tell bot to update core setting to db.
-			if ( ! $setting_changed ) {
+			if ( ! $setting_changed) {
 				$setting_changed = true;
 			}
 		}
 
 		// Update setting only if they changed.
-		if ( $setting_changed ) {
-			update_option( 'give_settings', $give_settings );
+		if ($setting_changed) {
+			update_option('give_settings', $give_settings);
 		}
 	}// End if().
 
-	give_set_upgrade_complete( 'v18_upgrades_core_setting' );
+	give_set_upgrade_complete('v18_upgrades_core_setting');
 }
 
 /**
@@ -614,7 +614,7 @@ 
 	$give_updates = Give_Updates::get_instance();
 
 	// form query
-	$forms = new WP_Query( array(
+	$forms = new WP_Query(array(
 			'paged'          => $give_updates->step,
 			'status'         => 'any',
 			'order'          => 'ASC',
@@ -623,41 +623,41 @@ 
 		)
 	);
 
-	if ( $forms->have_posts() ) {
-		$give_updates->set_percentage( $forms->found_posts, ( $give_updates->step * 20 ) );
+	if ($forms->have_posts()) {
+		$give_updates->set_percentage($forms->found_posts, ($give_updates->step * 20));
 
-		while ( $forms->have_posts() ) {
+		while ($forms->have_posts()) {
 			$forms->the_post();
 
 			// Form content.
 			// Note in version 1.8 display content setting split into display content and content placement setting.
 			// You can delete _give_content_option in future.
-			$show_content = give_get_meta( get_the_ID(), '_give_content_option', true );
-			if ( $show_content && ! give_get_meta( get_the_ID(), '_give_display_content', true ) ) {
-				$field_value = ( 'none' !== $show_content ? 'enabled' : 'disabled' );
-				give_update_meta( get_the_ID(), '_give_display_content', $field_value );
+			$show_content = give_get_meta(get_the_ID(), '_give_content_option', true);
+			if ($show_content && ! give_get_meta(get_the_ID(), '_give_display_content', true)) {
+				$field_value = ('none' !== $show_content ? 'enabled' : 'disabled');
+				give_update_meta(get_the_ID(), '_give_display_content', $field_value);
 
-				$field_value = ( 'none' !== $show_content ? $show_content : 'give_pre_form' );
-				give_update_meta( get_the_ID(), '_give_content_placement', $field_value );
+				$field_value = ('none' !== $show_content ? $show_content : 'give_pre_form');
+				give_update_meta(get_the_ID(), '_give_content_placement', $field_value);
 			}
 
 			// "Disable" Guest Donation. Checkbox.
 			// See: https://github.com/WordImpress/Give/issues/1470.
-			$guest_donation        = give_get_meta( get_the_ID(), '_give_logged_in_only', true );
-			$guest_donation_newval = ( in_array( $guest_donation, array( 'yes', 'on' ) ) ? 'disabled' : 'enabled' );
-			give_update_meta( get_the_ID(), '_give_logged_in_only', $guest_donation_newval );
+			$guest_donation        = give_get_meta(get_the_ID(), '_give_logged_in_only', true);
+			$guest_donation_newval = (in_array($guest_donation, array('yes', 'on')) ? 'disabled' : 'enabled');
+			give_update_meta(get_the_ID(), '_give_logged_in_only', $guest_donation_newval);
 
 			// Offline Donations.
 			// See: https://github.com/WordImpress/Give/issues/1579.
-			$offline_donation = give_get_meta( get_the_ID(), '_give_customize_offline_donations', true );
-			if ( 'no' === $offline_donation ) {
+			$offline_donation = give_get_meta(get_the_ID(), '_give_customize_offline_donations', true);
+			if ('no' === $offline_donation) {
 				$offline_donation_newval = 'global';
-			} elseif ( 'yes' === $offline_donation ) {
+			} elseif ('yes' === $offline_donation) {
 				$offline_donation_newval = 'enabled';
 			} else {
 				$offline_donation_newval = 'disabled';
 			}
-			give_update_meta( get_the_ID(), '_give_customize_offline_donations', $offline_donation_newval );
+			give_update_meta(get_the_ID(), '_give_customize_offline_donations', $offline_donation_newval);
 
 			// Convert yes/no setting field to enabled/disabled.
 			$form_radio_settings = array(
@@ -677,15 +677,15 @@ 
 				'_give_offline_donation_enable_billing_fields_single',
 			);
 
-			foreach ( $form_radio_settings as $meta_key ) {
+			foreach ($form_radio_settings as $meta_key) {
 				// Get value.
-				$field_value = give_get_meta( get_the_ID(), $meta_key, true );
+				$field_value = give_get_meta(get_the_ID(), $meta_key, true);
 
 				// Convert meta value only if it is in yes/no/none.
-				if ( in_array( $field_value, array( 'yes', 'on', 'no', 'none' ) ) ) {
+				if (in_array($field_value, array('yes', 'on', 'no', 'none'))) {
 
-					$field_value = ( in_array( $field_value, array( 'yes', 'on' ) ) ? 'enabled' : 'disabled' );
-					give_update_meta( get_the_ID(), $meta_key, $field_value );
+					$field_value = (in_array($field_value, array('yes', 'on')) ? 'enabled' : 'disabled');
+					give_update_meta(get_the_ID(), $meta_key, $field_value);
 				}
 			}
 		}// End while().
@@ -694,7 +694,7 @@ 
 
 	} else {
 		// No more forms found, finish up.
-		give_set_upgrade_complete( 'v18_upgrades_form_metadata' );
+		give_set_upgrade_complete('v18_upgrades_form_metadata');
 	}
 }
 
@@ -761,7 +761,7 @@ 
 				'%_transient_give_stats_%',
 				'give_cache%',
 				'%_transient_give_add_ons_feed%',
-				'%_transient__give_ajax_works' .
+				'%_transient__give_ajax_works'.
 				'%_transient_give_total_api_keys%',
 				'%_transient_give_i18n_give_promo_hide%',
 				'%_transient_give_contributors%',
@@ -788,24 +788,24 @@ 
 		ARRAY_A
 	);
 
-	if ( ! empty( $user_apikey_options ) ) {
-		foreach ( $user_apikey_options as $user ) {
-			$cached_options[] = '_transient_' . md5( 'give_api_user_' . $user['meta_key'] );
-			$cached_options[] = '_transient_' . md5( 'give_api_user_public_key' . $user['user_id'] );
-			$cached_options[] = '_transient_' . md5( 'give_api_user_secret_key' . $user['user_id'] );
+	if ( ! empty($user_apikey_options)) {
+		foreach ($user_apikey_options as $user) {
+			$cached_options[] = '_transient_'.md5('give_api_user_'.$user['meta_key']);
+			$cached_options[] = '_transient_'.md5('give_api_user_public_key'.$user['user_id']);
+			$cached_options[] = '_transient_'.md5('give_api_user_secret_key'.$user['user_id']);
 		}
 	}
 
-	if ( ! empty( $cached_options ) ) {
-		foreach ( $cached_options as $option ) {
-			switch ( true ) {
-				case ( false !== strpos( $option, 'transient' ) ):
-					$option = str_replace( '_transient_', '', $option );
-					delete_transient( $option );
+	if ( ! empty($cached_options)) {
+		foreach ($cached_options as $option) {
+			switch (true) {
+				case (false !== strpos($option, 'transient')):
+					$option = str_replace('_transient_', '', $option);
+					delete_transient($option);
 					break;
 
 				default:
-					delete_option( $option );
+					delete_option($option);
 			}
 		}
 	}
@@ -823,7 +823,7 @@ 
 	global $wp_roles;
 
 	// Get the role object.
-	$give_worker = get_role( 'give_worker' );
+	$give_worker = get_role('give_worker');
 
 	// A list of capabilities to add for give workers.
 	$caps_to_add = array(
@@ -831,9 +831,9 @@ 
 		'edit_pages',
 	);
 
-	foreach ( $caps_to_add as $cap ) {
+	foreach ($caps_to_add as $cap) {
 		// Add the capability.
-		$give_worker->add_cap( $cap );
+		$give_worker->add_cap($cap);
 	}
 
 }
@@ -850,7 +850,7 @@ 
 	$give_updates = Give_Updates::get_instance();
 
 	// form query.
-	$donation_forms = new WP_Query( array(
+	$donation_forms = new WP_Query(array(
 			'paged'          => $give_updates->step,
 			'status'         => 'any',
 			'order'          => 'ASC',
@@ -859,10 +859,10 @@ 
 		)
 	);
 
-	if ( $donation_forms->have_posts() ) {
-		$give_updates->set_percentage( $donation_forms->found_posts, ( $give_updates->step * 20 ) );
+	if ($donation_forms->have_posts()) {
+		$give_updates->set_percentage($donation_forms->found_posts, ($give_updates->step * 20));
 
-		while ( $donation_forms->have_posts() ) {
+		while ($donation_forms->have_posts()) {
 			$donation_forms->the_post();
 			$form_id = get_the_ID();
 
@@ -870,41 +870,41 @@ 
 			update_post_meta(
 				$form_id,
 				'_give_set_price',
-				give_sanitize_amount( get_post_meta( $form_id, '_give_set_price', true ) )
+				give_sanitize_amount(get_post_meta($form_id, '_give_set_price', true))
 			);
 
 			// Remove formatting from _give_custom_amount_minimum.
 			update_post_meta(
 				$form_id,
 				'_give_custom_amount_minimum',
-				give_sanitize_amount( get_post_meta( $form_id, '_give_custom_amount_minimum', true ) )
+				give_sanitize_amount(get_post_meta($form_id, '_give_custom_amount_minimum', true))
 			);
 
 			// Bailout.
-			if ( 'set' === get_post_meta( $form_id, '_give_price_option', true ) ) {
+			if ('set' === get_post_meta($form_id, '_give_price_option', true)) {
 				continue;
 			}
 
-			$donation_levels = get_post_meta( $form_id, '_give_donation_levels', true );
+			$donation_levels = get_post_meta($form_id, '_give_donation_levels', true);
 
-			if ( ! empty( $donation_levels ) ) {
+			if ( ! empty($donation_levels)) {
 
-				foreach ( $donation_levels as $index => $donation_level ) {
-					if ( isset( $donation_level['_give_amount'] ) ) {
-						$donation_levels[ $index ]['_give_amount'] = give_sanitize_amount( $donation_level['_give_amount'] );
+				foreach ($donation_levels as $index => $donation_level) {
+					if (isset($donation_level['_give_amount'])) {
+						$donation_levels[$index]['_give_amount'] = give_sanitize_amount($donation_level['_give_amount']);
 					}
 				}
 
-				update_post_meta( $form_id, '_give_donation_levels', $donation_levels );
+				update_post_meta($form_id, '_give_donation_levels', $donation_levels);
 
-				$donation_levels_amounts = wp_list_pluck( $donation_levels, '_give_amount' );
+				$donation_levels_amounts = wp_list_pluck($donation_levels, '_give_amount');
 
-				$min_amount = min( $donation_levels_amounts );
-				$max_amount = max( $donation_levels_amounts );
+				$min_amount = min($donation_levels_amounts);
+				$max_amount = max($donation_levels_amounts);
 
 				// Set Minimum and Maximum amount for Multi Level Donation Forms
-				give_update_meta( $form_id, '_give_levels_minimum_amount', $min_amount ? give_sanitize_amount( $min_amount ) : 0 );
-				give_update_meta( $form_id, '_give_levels_maximum_amount', $max_amount ? give_sanitize_amount( $max_amount ) : 0 );
+				give_update_meta($form_id, '_give_levels_minimum_amount', $min_amount ? give_sanitize_amount($min_amount) : 0);
+				give_update_meta($form_id, '_give_levels_maximum_amount', $max_amount ? give_sanitize_amount($max_amount) : 0);
 			}
 
 		}
@@ -913,7 +913,7 @@ 
 		wp_reset_postdata();
 	} else {
 		// The Update Ran.
-		give_set_upgrade_complete( 'v189_upgrades_levels_post_meta' );
+		give_set_upgrade_complete('v189_upgrades_levels_post_meta');
 	}
 
 }
@@ -969,22 +969,22 @@ 
 	$give_settings        = give_get_settings();
 	$give_setting_updated = false;
 
-	if ( $give_settings['thousands_separator'] === $give_settings['decimal_separator'] ) {
+	if ($give_settings['thousands_separator'] === $give_settings['decimal_separator']) {
 		$give_settings['number_decimals']   = 0;
 		$give_settings['decimal_separator'] = '';
 		$give_setting_updated               = true;
 
-	} elseif ( empty( $give_settings['decimal_separator'] ) ) {
+	} elseif (empty($give_settings['decimal_separator'])) {
 		$give_settings['number_decimals'] = 0;
 		$give_setting_updated             = true;
 
-	} elseif ( 6 < absint( $give_settings['number_decimals'] ) ) {
+	} elseif (6 < absint($give_settings['number_decimals'])) {
 		$give_settings['number_decimals'] = 5;
 		$give_setting_updated             = true;
 	}
 
-	if ( $give_setting_updated ) {
-		update_option( 'give_settings', $give_settings );
+	if ($give_setting_updated) {
+		update_option('give_settings', $give_settings);
 	}
 }
 
@@ -1003,70 +1003,70 @@ 
 	$give_updates = Give_Updates::get_instance();
 
 	// form query.
-	$donation_forms = new WP_Query( array(
+	$donation_forms = new WP_Query(array(
 			'paged'          => $give_updates->step,
 			'status'         => 'any',
 			'order'          => 'ASC',
-			'post_type'      => array( 'give_forms', 'give_payment' ),
+			'post_type'      => array('give_forms', 'give_payment'),
 			'posts_per_page' => 20,
 		)
 	);
 
-	if ( $donation_forms->have_posts() ) {
-		$give_updates->set_percentage( $donation_forms->found_posts, ( $give_updates->step * 20 ) );
+	if ($donation_forms->have_posts()) {
+		$give_updates->set_percentage($donation_forms->found_posts, ($give_updates->step * 20));
 
-		while ( $donation_forms->have_posts() ) {
+		while ($donation_forms->have_posts()) {
 			$donation_forms->the_post();
 			global $post;
 
-			$meta = get_post_meta( $post->ID );
+			$meta = get_post_meta($post->ID);
 
-			switch ( $post->post_type ) {
+			switch ($post->post_type) {
 				case 'give_forms':
 					// _give_set_price.
-					if ( ! empty( $meta['_give_set_price'][0] ) ) {
-						update_post_meta( $post->ID, '_give_set_price', give_sanitize_amount_for_db( $meta['_give_set_price'][0] ) );
+					if ( ! empty($meta['_give_set_price'][0])) {
+						update_post_meta($post->ID, '_give_set_price', give_sanitize_amount_for_db($meta['_give_set_price'][0]));
 					}
 
 					// _give_custom_amount_minimum.
-					if ( ! empty( $meta['_give_custom_amount_minimum'][0] ) ) {
-						update_post_meta( $post->ID, '_give_custom_amount_minimum', give_sanitize_amount_for_db( $meta['_give_custom_amount_minimum'][0] ) );
+					if ( ! empty($meta['_give_custom_amount_minimum'][0])) {
+						update_post_meta($post->ID, '_give_custom_amount_minimum', give_sanitize_amount_for_db($meta['_give_custom_amount_minimum'][0]));
 					}
 
 					// _give_levels_minimum_amount.
-					if ( ! empty( $meta['_give_levels_minimum_amount'][0] ) ) {
-						update_post_meta( $post->ID, '_give_levels_minimum_amount', give_sanitize_amount_for_db( $meta['_give_levels_minimum_amount'][0] ) );
+					if ( ! empty($meta['_give_levels_minimum_amount'][0])) {
+						update_post_meta($post->ID, '_give_levels_minimum_amount', give_sanitize_amount_for_db($meta['_give_levels_minimum_amount'][0]));
 					}
 
 					// _give_levels_maximum_amount.
-					if ( ! empty( $meta['_give_levels_maximum_amount'][0] ) ) {
-						update_post_meta( $post->ID, '_give_levels_maximum_amount', give_sanitize_amount_for_db( $meta['_give_levels_maximum_amount'][0] ) );
+					if ( ! empty($meta['_give_levels_maximum_amount'][0])) {
+						update_post_meta($post->ID, '_give_levels_maximum_amount', give_sanitize_amount_for_db($meta['_give_levels_maximum_amount'][0]));
 					}
 
 					// _give_set_goal.
-					if ( ! empty( $meta['_give_set_goal'][0] ) ) {
-						update_post_meta( $post->ID, '_give_set_goal', give_sanitize_amount_for_db( $meta['_give_set_goal'][0] ) );
+					if ( ! empty($meta['_give_set_goal'][0])) {
+						update_post_meta($post->ID, '_give_set_goal', give_sanitize_amount_for_db($meta['_give_set_goal'][0]));
 					}
 
 					// _give_form_earnings.
-					if ( ! empty( $meta['_give_form_earnings'][0] ) ) {
-						update_post_meta( $post->ID, '_give_form_earnings', give_sanitize_amount_for_db( $meta['_give_form_earnings'][0] ) );
+					if ( ! empty($meta['_give_form_earnings'][0])) {
+						update_post_meta($post->ID, '_give_form_earnings', give_sanitize_amount_for_db($meta['_give_form_earnings'][0]));
 					}
 
 					// _give_custom_amount_minimum.
-					if ( ! empty( $meta['_give_donation_levels'][0] ) ) {
-						$donation_levels = unserialize( $meta['_give_donation_levels'][0] );
+					if ( ! empty($meta['_give_donation_levels'][0])) {
+						$donation_levels = unserialize($meta['_give_donation_levels'][0]);
 
-						foreach ( $donation_levels as $index => $level ) {
-							if ( empty( $level['_give_amount'] ) ) {
+						foreach ($donation_levels as $index => $level) {
+							if (empty($level['_give_amount'])) {
 								continue;
 							}
 
-							$donation_levels[ $index ]['_give_amount'] = give_sanitize_amount_for_db( $level['_give_amount'] );
+							$donation_levels[$index]['_give_amount'] = give_sanitize_amount_for_db($level['_give_amount']);
 						}
 
 						$meta['_give_donation_levels'] = $donation_levels;
-						update_post_meta( $post->ID, '_give_donation_levels', $meta['_give_donation_levels'] );
+						update_post_meta($post->ID, '_give_donation_levels', $meta['_give_donation_levels']);
 					}
 
 
@@ -1074,8 +1074,8 @@ 
 
 				case 'give_payment':
 					// _give_payment_total.
-					if ( ! empty( $meta['_give_payment_total'][0] ) ) {
-						update_post_meta( $post->ID, '_give_payment_total', give_sanitize_amount_for_db( $meta['_give_payment_total'][0] ) );
+					if ( ! empty($meta['_give_payment_total'][0])) {
+						update_post_meta($post->ID, '_give_payment_total', give_sanitize_amount_for_db($meta['_give_payment_total'][0]));
 					}
 
 					break;
@@ -1086,7 +1086,7 @@ 
 		wp_reset_postdata();
 	} else {
 		// The Update Ran.
-		give_set_upgrade_complete( 'v1812_update_amount_values' );
+		give_set_upgrade_complete('v1812_update_amount_values');
 	}
 }
 
@@ -1106,22 +1106,22 @@ 
 	$offset       = 1 === $give_updates->step ? 0 : $give_updates->step * 20;
 
 	// form query.
-	$donors = Give()->donors->get_donors( array(
+	$donors = Give()->donors->get_donors(array(
 			'number' => 20,
 			'offset' => $offset,
 		)
 	);
 
-	if ( ! empty( $donors ) ) {
-		$give_updates->set_percentage( Give()->donors->count(), ( $give_updates->step * 20 ) );
+	if ( ! empty($donors)) {
+		$give_updates->set_percentage(Give()->donors->count(), ($give_updates->step * 20));
 
 		/* @var Object $donor */
-		foreach ( $donors as $donor ) {
-			Give()->donors->update( $donor->id, array( 'purchase_value' => give_sanitize_amount_for_db( $donor->purchase_value ) ) );
+		foreach ($donors as $donor) {
+			Give()->donors->update($donor->id, array('purchase_value' => give_sanitize_amount_for_db($donor->purchase_value)));
 		}
 	} else {
 		// The Update Ran.
-		give_set_upgrade_complete( 'v1812_update_donor_purchase_values' );
+		give_set_upgrade_complete('v1812_update_donor_purchase_values');
 	}
 }
 
@@ -1136,25 +1136,25 @@ 
 	$offset       = 1 === $give_updates->step ? 0 : $give_updates->step * 20;
 
 	// Fetch all the existing donors.
-	$donors = Give()->donors->get_donors( array(
+	$donors = Give()->donors->get_donors(array(
 			'number' => 20,
 			'offset' => $offset,
 		)
 	);
 
-	if ( ! empty( $donors ) ) {
-		$give_updates->set_percentage( Give()->donors->count(), ( $give_updates->step * 20 ) );
+	if ( ! empty($donors)) {
+		$give_updates->set_percentage(Give()->donors->count(), ($give_updates->step * 20));
 
 		/* @var Object $donor */
-		foreach ( $donors as $donor ) {
+		foreach ($donors as $donor) {
 			$user_id = $donor->user_id;
 
 			// Proceed, if donor is attached with user.
-			if ( $user_id ) {
-				$user = get_userdata( $user_id );
+			if ($user_id) {
+				$user = get_userdata($user_id);
 
 				// Update user role, if user has subscriber role.
-				if ( is_array( $user->roles ) && in_array( 'subscriber', $user->roles ) ) {
+				if (is_array($user->roles) && in_array('subscriber', $user->roles)) {
 					wp_update_user(
 						array(
 							'ID'   => $user_id,
@@ -1166,7 +1166,7 @@ 
 		}
 	} else {
 		// The Update Ran.
-		give_set_upgrade_complete( 'v1813_update_donor_user_roles' );
+		give_set_upgrade_complete('v1813_update_donor_user_roles');
 	}
 }
 
@@ -1180,33 +1180,33 @@ 
 	$give_updates = Give_Updates::get_instance();
 
 	// form query.
-	$payments = new WP_Query( array(
+	$payments = new WP_Query(array(
 			'paged'          => $give_updates->step,
 			'status'         => 'any',
 			'order'          => 'ASC',
-			'post_type'      => array( 'give_payment' ),
+			'post_type'      => array('give_payment'),
 			'posts_per_page' => 20,
 		)
 	);
 
-	if ( $payments->have_posts() ) {
-		$give_updates->set_percentage( $payments->found_posts, ( $give_updates->step * 20 ) );
+	if ($payments->have_posts()) {
+		$give_updates->set_percentage($payments->found_posts, ($give_updates->step * 20));
 
-		while( $payments->have_posts() ) {
+		while ($payments->have_posts()) {
 			$payments->the_post();
 
-			$payment_meta = give_get_payment_meta( get_the_ID() );
+			$payment_meta = give_get_payment_meta(get_the_ID());
 
-			if ( 'RIAL' === $payment_meta['currency'] ) {
+			if ('RIAL' === $payment_meta['currency']) {
 				$payment_meta['currency'] = 'IRR';
-				give_update_meta( get_the_ID(), '_give_payment_meta', $payment_meta );
+				give_update_meta(get_the_ID(), '_give_payment_meta', $payment_meta);
 			}
 
 		}
 
-	}else{
+	} else {
 		// The Update Ran.
-		give_set_upgrade_complete( 'v1817_update_donation_iranian_currency_code' );
+		give_set_upgrade_complete('v1817_update_donation_iranian_currency_code');
 	}
 }
 
@@ -1217,7 +1217,7 @@ 
  */
 function give_v1813_upgrades() {
 	// Update admin setting.
-	give_update_option( 'donor_default_user_role', 'give_donor' );
+	give_update_option('donor_default_user_role', 'give_donor');
 
 	// Update Give roles.
 	$roles = new Give_Roles();
@@ -1234,9 +1234,9 @@ 
 function give_v1817_upgrades() {
 	$give_settings = give_get_settings();
 
-	if ( 'RIAL' === $give_settings['currency'] ) {
+	if ('RIAL' === $give_settings['currency']) {
 		$give_settings['currency'] = 'IRR';
-		update_option( 'give_settings', $give_settings );
+		update_option('give_settings', $give_settings);
 	}
 }
 
@@ -1249,7 +1249,7 @@ 
 
 	global $wp_roles;
 
-	if( ! ( $wp_roles instanceof  WP_Roles ) ) {
+	if ( ! ($wp_roles instanceof  WP_Roles)) {
 		return;
 	}
 
@@ -1273,15 +1273,15 @@ 
 		),
 	);
 
-	foreach ( $add_caps as $role => $caps ) {
-		foreach( $caps as $cap ) {
-			$wp_roles->add_cap( $role, $cap );
+	foreach ($add_caps as $role => $caps) {
+		foreach ($caps as $cap) {
+			$wp_roles->add_cap($role, $cap);
 		}
 	}
 
-	foreach ( $remove_caps as $role => $caps ) {
-		foreach( $caps as $cap ) {
-			$wp_roles->remove_cap( $role, $cap );
+	foreach ($remove_caps as $role => $caps) {
+		foreach ($caps as $cap) {
+			$wp_roles->remove_cap($role, $cap);
 		}
 	}
 
@@ -1306,6 +1306,6 @@ 
 	$roles->add_roles();
 	$roles->add_caps();
 
-	give_set_upgrade_complete( 'v1817_cleanup_user_roles' );
+	give_set_upgrade_complete('v1817_cleanup_user_roles');
 
 }
\ No newline at end of file

Braces +1 added lines, -1 removed lines

@@ -1204,7 +1204,7 @@
 
 		}
 
-	}else{
+	} else{
 		// The Update Ran.
 		give_set_upgrade_complete( 'v1817_update_donation_iranian_currency_code' );
 	}

includes/admin/class-admin-settings.php

Indentation +5 added lines, -5 removed lines

@@ -109,8 +109,8 @@ 
 
 			// Show error message if Akismet not configured and Admin try to save 'enabled' option.
 			if ( isset( $_POST['akismet_spam_protection'] )
-			     && give_is_setting_enabled( $_POST['akismet_spam_protection'] )
-			     && ! give_check_akismet_key()
+				 && give_is_setting_enabled( $_POST['akismet_spam_protection'] )
+				 && ! give_check_akismet_key()
 			) {
 				self::add_error( 'give-akismet-protection', __( 'Please properly configure Akismet to enable SPAM protection.', 'give' ) );
 
@@ -792,9 +792,9 @@ 
 						<td class="give-docs-link" colspan="2">
 							<?php
 							echo '<p class="give-docs-link"><a href="' . esc_url( $value['url'] )
-							     . '" target="_blank">'
-							     . sprintf( esc_html__( 'Need Help? See docs on "%s"', 'give' ), $value['title'] )
-							     . '<span class="dashicons dashicons-editor-help"></span></a></p>';
+								 . '" target="_blank">'
+								 . sprintf( esc_html__( 'Need Help? See docs on "%s"', 'give' ), $value['title'] )
+								 . '<span class="dashicons dashicons-editor-help"></span></a></p>';
 							?>
 						</td>
 						</tr><?php

Spacing +220 added lines, -220 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! class_exists( 'Give_Admin_Settings' ) ) :
+if ( ! class_exists('Give_Admin_Settings')) :
 
 	/**
 	 * Give_Admin_Settings Class.
@@ -72,7 +72,7 @@ 
 			 *
 			 * @param array $settings Array of settings class object.
 			 */
-			self::$settings = apply_filters( self::$setting_filter_prefix . '_get_settings_pages', array() );
+			self::$settings = apply_filters(self::$setting_filter_prefix.'_get_settings_pages', array());
 
 			return self::$settings;
 		}
@@ -86,7 +86,7 @@ 
 		 * @return bool
 		 */
 		public static function verify_nonce() {
-			if ( empty( $_REQUEST['_give-save-settings'] ) || ! wp_verify_nonce( $_REQUEST['_give-save-settings'], 'give-save-settings' ) ) {
+			if (empty($_REQUEST['_give-save-settings']) || ! wp_verify_nonce($_REQUEST['_give-save-settings'], 'give-save-settings')) {
 				return false;
 			}
 
@@ -102,17 +102,17 @@ 
 		public static function save() {
 			$current_tab = give_get_current_setting_tab();
 
-			if ( ! self::verify_nonce() ) {
-				echo '<div class="notice error"><p>' . __( 'Action failed. Please refresh the page and retry.', 'give' ) . '</p></div>';
+			if ( ! self::verify_nonce()) {
+				echo '<div class="notice error"><p>'.__('Action failed. Please refresh the page and retry.', 'give').'</p></div>';
 				die();
 			}
 
 			// Show error message if Akismet not configured and Admin try to save 'enabled' option.
-			if ( isset( $_POST['akismet_spam_protection'] )
-			     && give_is_setting_enabled( $_POST['akismet_spam_protection'] )
+			if (isset($_POST['akismet_spam_protection'])
+			     && give_is_setting_enabled($_POST['akismet_spam_protection'])
 			     && ! give_check_akismet_key()
 			) {
-				self::add_error( 'give-akismet-protection', __( 'Please properly configure Akismet to enable SPAM protection.', 'give' ) );
+				self::add_error('give-akismet-protection', __('Please properly configure Akismet to enable SPAM protection.', 'give'));
 
 				return;
 			}
@@ -126,9 +126,9 @@ 
 			 *
 			 * @since 1.8
 			 */
-			do_action( self::$setting_filter_prefix . '_save_' . $current_tab );
+			do_action(self::$setting_filter_prefix.'_save_'.$current_tab);
 
-			self::add_message( 'give-setting-updated', __( 'Your settings have been saved.', 'give' ) );
+			self::add_message('give-setting-updated', __('Your settings have been saved.', 'give'));
 
 			/**
 			 * Trigger Action.
@@ -139,7 +139,7 @@ 
 			 *
 			 * @since 1.8
 			 */
-			do_action( self::$setting_filter_prefix . '_saved' );
+			do_action(self::$setting_filter_prefix.'_saved');
 		}
 
 		/**
@@ -152,8 +152,8 @@ 
 		 *
 		 * @return void
 		 */
-		public static function add_message( $code, $message ) {
-			self::$messages[ $code ] = $message;
+		public static function add_message($code, $message) {
+			self::$messages[$code] = $message;
 		}
 
 		/**
@@ -166,8 +166,8 @@ 
 		 *
 		 * @return void
 		 */
-		public static function add_error( $code, $message ) {
-			self::$errors[ $code ] = $message;
+		public static function add_error($code, $message) {
+			self::$errors[$code] = $message;
 		}
 
 		/**
@@ -180,18 +180,18 @@ 
 			$notice_html = '';
 			$classes     = 'give-notice settings-error notice is-dismissible';
 
-			self::$errors   = apply_filters( self::$setting_filter_prefix . '_error_notices', self::$errors );
-			self::$messages = apply_filters( self::$setting_filter_prefix . '_update_notices', self::$messages );
+			self::$errors   = apply_filters(self::$setting_filter_prefix.'_error_notices', self::$errors);
+			self::$messages = apply_filters(self::$setting_filter_prefix.'_update_notices', self::$messages);
 
-			if ( 0 < count( self::$errors ) ) {
-				foreach ( self::$errors as $code => $message ) {
-					$notice_html .= '<div id="setting-error-' . $code . '" class="' . $classes . ' error"><p><strong>' . $message . '</strong></p></div>';
+			if (0 < count(self::$errors)) {
+				foreach (self::$errors as $code => $message) {
+					$notice_html .= '<div id="setting-error-'.$code.'" class="'.$classes.' error"><p><strong>'.$message.'</strong></p></div>';
 				}
 			}
 
-			if ( 0 < count( self::$messages ) ) {
-				foreach ( self::$messages as $code => $message ) {
-					$notice_html .= '<div id="setting-error-' . $code . '" class="' . $classes . ' updated"><p><strong>' . $message . '</strong></p></div>';
+			if (0 < count(self::$messages)) {
+				foreach (self::$messages as $code => $message) {
+					$notice_html .= '<div id="setting-error-'.$code.'" class="'.$classes.' updated"><p><strong>'.$message.'</strong></p></div>';
 				}
 			}
 
@@ -211,7 +211,7 @@ 
 			self::$setting_filter_prefix = give_get_current_setting_page();
 
 			// Bailout: Exit if setting page is not defined.
-			if ( empty( self::$setting_filter_prefix ) ) {
+			if (empty(self::$setting_filter_prefix)) {
 				return false;
 			}
 
@@ -224,7 +224,7 @@ 
 			 *
 			 * @since 1.8
 			 */
-			do_action( self::$setting_filter_prefix . '_start' );
+			do_action(self::$setting_filter_prefix.'_start');
 
 			$current_tab = give_get_current_setting_tab();
 
@@ -234,9 +234,9 @@ 
 			/* @var object $current_setting_obj */
 			$current_setting_obj = new StdClass;
 
-			foreach ( $all_setting as $setting ) {
+			foreach ($all_setting as $setting) {
 				if (
-					method_exists( $setting, 'get_id' ) &&
+					method_exists($setting, 'get_id') &&
 					$current_tab === $setting->get_id()
 				) {
 					$current_setting_obj = $setting;
@@ -245,7 +245,7 @@ 
 			}
 
 			// Save settings if data has been posted.
-			if ( ! empty( $_POST ) ) {
+			if ( ! empty($_POST)) {
 				self::save();
 			}
 
@@ -258,7 +258,7 @@ 
 			 *
 			 * @since 1.8
 			 */
-			$tabs = apply_filters( self::$setting_filter_prefix . '_tabs_array', array() );
+			$tabs = apply_filters(self::$setting_filter_prefix.'_tabs_array', array());
 
 			include 'views/html-admin-settings.php';
 
@@ -276,25 +276,25 @@ 
 		 *
 		 * @return string|bool
 		 */
-		public static function get_option( $option_name = '', $field_id = '', $default = false ) {
+		public static function get_option($option_name = '', $field_id = '', $default = false) {
 			// Bailout.
-			if ( empty( $option_name ) && empty( $field_id ) ) {
+			if (empty($option_name) && empty($field_id)) {
 				return false;
 			}
 
-			if ( ! empty( $field_id ) && ! empty( $option_name ) ) {
+			if ( ! empty($field_id) && ! empty($option_name)) {
 				// Get field value if any.
-				$option_value = get_option( $option_name );
+				$option_value = get_option($option_name);
 
-				$option_value = ( is_array( $option_value ) && array_key_exists( $field_id, $option_value ) )
-					? $option_value[ $field_id ]
+				$option_value = (is_array($option_value) && array_key_exists($field_id, $option_value))
+					? $option_value[$field_id]
 					: $default;
 			} else {
 				// If option name is empty but not field name then this means, setting is direct store to option table under there field name.
 				$option_name = ! $option_name ? $field_id : $option_name;
 
 				// Get option value if any.
-				$option_value = get_option( $option_name, $default );
+				$option_value = get_option($option_name, $default);
 			}
 
 			return $option_value;
@@ -312,7 +312,7 @@ 
 		 *
 		 * @return void
 		 */
-		public static function output_fields( $options, $option_name = '' ) {
+		public static function output_fields($options, $option_name = '') {
 			$current_tab = give_get_current_setting_tab();
 
 			// Field Default values.
@@ -325,52 +325,52 @@ 
 				'table_html' => true,
 			);
 
-			foreach ( $options as $value ) {
-				if ( ! isset( $value['type'] ) ) {
+			foreach ($options as $value) {
+				if ( ! isset($value['type'])) {
 					continue;
 				}
 
 				// Set title.
-				$defaults['title'] = isset( $value['name'] ) ? $value['name'] : '';
+				$defaults['title'] = isset($value['name']) ? $value['name'] : '';
 
 				// Set default setting.
-				$value = wp_parse_args( $value, $defaults );
+				$value = wp_parse_args($value, $defaults);
 
 				// Colorpicker field.
-				$value['class'] = ( 'colorpicker' === $value['type'] ? trim( $value['class'] ) . ' give-colorpicker' : $value['class'] );
-				$value['type']  = ( 'colorpicker' === $value['type'] ? 'text' : $value['type'] );
+				$value['class'] = ('colorpicker' === $value['type'] ? trim($value['class']).' give-colorpicker' : $value['class']);
+				$value['type']  = ('colorpicker' === $value['type'] ? 'text' : $value['type']);
 
 
 				// Custom attribute handling.
 				$custom_attributes = array();
 
-				if ( ! empty( $value['attributes'] ) && is_array( $value['attributes'] ) ) {
-					foreach ( $value['attributes'] as $attribute => $attribute_value ) {
-						$custom_attributes[] = esc_attr( $attribute ) . '="' . esc_attr( $attribute_value ) . '"';
+				if ( ! empty($value['attributes']) && is_array($value['attributes'])) {
+					foreach ($value['attributes'] as $attribute => $attribute_value) {
+						$custom_attributes[] = esc_attr($attribute).'="'.esc_attr($attribute_value).'"';
 					}
 				}
 
 				// Description handling.
-				$description          = self::get_field_description( $value );
+				$description = self::get_field_description($value);
 
 				// Switch based on type.
-				switch ( $value['type'] ) {
+				switch ($value['type']) {
 
 					// Section Titles
 					case 'title':
-						if ( ! empty( $value['title'] ) ) {
-							echo '<div class="give-setting-tab-header give-setting-tab-header-' . $current_tab . '"><h2>' . self::get_field_title( $value ) . '</h2><hr></div>';
+						if ( ! empty($value['title'])) {
+							echo '<div class="give-setting-tab-header give-setting-tab-header-'.$current_tab.'"><h2>'.self::get_field_title($value).'</h2><hr></div>';
 						}
 
-						if ( ! empty( $value['desc'] ) ) {
-							echo wpautop( wptexturize( wp_kses_post( $value['desc'] ) ) );
+						if ( ! empty($value['desc'])) {
+							echo wpautop(wptexturize(wp_kses_post($value['desc'])));
 						}
 
-						if ( $value['table_html'] ) {
-							echo '<table class="form-table give-setting-tab-body give-setting-tab-body-' . $current_tab . '">' . "\n\n";
+						if ($value['table_html']) {
+							echo '<table class="form-table give-setting-tab-body give-setting-tab-body-'.$current_tab.'">'."\n\n";
 						}
 
-						if ( ! empty( $value['id'] ) ) {
+						if ( ! empty($value['id'])) {
 
 							/**
 							 * Trigger Action.
@@ -379,14 +379,14 @@ 
 							 *
 							 * @since 1.8
 							 */
-							do_action( 'give_settings_' . sanitize_title( $value['id'] ) );
+							do_action('give_settings_'.sanitize_title($value['id']));
 						}
 
 						break;
 
 					// Section Ends.
 					case 'sectionend':
-						if ( ! empty( $value['id'] ) ) {
+						if ( ! empty($value['id'])) {
 
 							/**
 							 * Trigger Action.
@@ -395,14 +395,14 @@ 
 							 *
 							 * @since 1.8
 							 */
-							do_action( 'give_settings_' . sanitize_title( $value['id'] ) . '_end' );
+							do_action('give_settings_'.sanitize_title($value['id']).'_end');
 						}
 
-						if ( $value['table_html'] ) {
+						if ($value['table_html']) {
 							echo '</table>';
 						}
 
-						if ( ! empty( $value['id'] ) ) {
+						if ( ! empty($value['id'])) {
 
 							/**
 							 * Trigger Action.
@@ -411,7 +411,7 @@ 
 							 *
 							 * @since 1.8
 							 */
-							do_action( 'give_settings_' . sanitize_title( $value['id'] ) . '_after' );
+							do_action('give_settings_'.sanitize_title($value['id']).'_after');
 						}
 
 						break;
@@ -419,27 +419,27 @@ 
 					// Standard text inputs and subtypes like 'number'.
 					case 'colorpicker':
 					case 'hidden' :
-						$value['wrapper_class'] = empty( $value['wrapper_class'] ) ? 'give-hidden' : trim( $value['wrapper_class'] ) . ' give-hidden';
+						$value['wrapper_class'] = empty($value['wrapper_class']) ? 'give-hidden' : trim($value['wrapper_class']).' give-hidden';
 					case 'text':
 					case 'email':
 					case 'number':
 					case 'password' :
 						$type = $value['type'];
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 						?>
-						<tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+						<tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
 							<th scope="row" class="titledesc">
-								<label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+								<label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
 							</th>
-							<td class="give-forminp give-forminp-<?php echo sanitize_title( $value['type'] ) ?>">
+							<td class="give-forminp give-forminp-<?php echo sanitize_title($value['type']) ?>">
 								<input
-										name="<?php echo esc_attr( $value['id'] ); ?>"
-										id="<?php echo esc_attr( $value['id'] ); ?>"
-										type="<?php echo esc_attr( $type ); ?>"
-										style="<?php echo esc_attr( $value['css'] ); ?>"
-										value="<?php echo esc_attr( $option_value ); ?>"
-										class="give-input-field<?php echo( empty( $value['class'] ) ? '' : ' ' . esc_attr( $value['class'] ) ); ?>"
-									<?php echo implode( ' ', $custom_attributes ); ?>
+										name="<?php echo esc_attr($value['id']); ?>"
+										id="<?php echo esc_attr($value['id']); ?>"
+										type="<?php echo esc_attr($type); ?>"
+										style="<?php echo esc_attr($value['css']); ?>"
+										value="<?php echo esc_attr($option_value); ?>"
+										class="give-input-field<?php echo(empty($value['class']) ? '' : ' '.esc_attr($value['class'])); ?>"
+									<?php echo implode(' ', $custom_attributes); ?>
 								/> <?php echo $description; ?>
 							</td>
 						</tr>
@@ -449,23 +449,23 @@ 
 					// Textarea.
 					case 'textarea':
 
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 
 						?>
-					<tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+					<tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
 						<th scope="row" class="titledesc">
-							<label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+							<label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
 						</th>
-						<td class="give-forminp give-forminp-<?php echo sanitize_title( $value['type'] ) ?>">
+						<td class="give-forminp give-forminp-<?php echo sanitize_title($value['type']) ?>">
 								<textarea
-										name="<?php echo esc_attr( $value['id'] ); ?>"
-										id="<?php echo esc_attr( $value['id'] ); ?>"
-										style="<?php echo esc_attr( $value['css'] ); ?>"
-										class="<?php echo esc_attr( $value['class'] ); ?>"
+										name="<?php echo esc_attr($value['id']); ?>"
+										id="<?php echo esc_attr($value['id']); ?>"
+										style="<?php echo esc_attr($value['css']); ?>"
+										class="<?php echo esc_attr($value['class']); ?>"
 										rows="10"
 										cols="60"
-									<?php echo implode( ' ', $custom_attributes ); ?>
-								><?php echo esc_textarea( $option_value ); ?></textarea>
+									<?php echo implode(' ', $custom_attributes); ?>
+								><?php echo esc_textarea($option_value); ?></textarea>
 							<?php echo $description; ?>
 						</td>
 						</tr><?php
@@ -475,35 +475,35 @@ 
 					case 'select' :
 					case 'multiselect' :
 
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 
 						?>
-					<tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+					<tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
 						<th scope="row" class="titledesc">
-							<label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+							<label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
 						</th>
-						<td class="give-forminp give-forminp-<?php echo sanitize_title( $value['type'] ) ?>">
+						<td class="give-forminp give-forminp-<?php echo sanitize_title($value['type']) ?>">
 							<select
-									name="<?php echo esc_attr( $value['id'] ); ?><?php if ( $value['type'] == 'multiselect' ) {
+									name="<?php echo esc_attr($value['id']); ?><?php if ($value['type'] == 'multiselect') {
 										echo '[]';
 									} ?>"
-									id="<?php echo esc_attr( $value['id'] ); ?>"
-									style="<?php echo esc_attr( $value['css'] ); ?>"
-									class="<?php echo esc_attr( $value['class'] ); ?>"
-								<?php echo implode( ' ', $custom_attributes ); ?>
-								<?php echo ( 'multiselect' == $value['type'] ) ? 'multiple="multiple"' : ''; ?>
+									id="<?php echo esc_attr($value['id']); ?>"
+									style="<?php echo esc_attr($value['css']); ?>"
+									class="<?php echo esc_attr($value['class']); ?>"
+								<?php echo implode(' ', $custom_attributes); ?>
+								<?php echo ('multiselect' == $value['type']) ? 'multiple="multiple"' : ''; ?>
 							>
 
 								<?php
-								if ( ! empty( $value['options'] ) ) {
-									foreach ( $value['options'] as $key => $val ) {
+								if ( ! empty($value['options'])) {
+									foreach ($value['options'] as $key => $val) {
 										?>
-										<option value="<?php echo esc_attr( $key ); ?>" <?php
+										<option value="<?php echo esc_attr($key); ?>" <?php
 
-										if ( is_array( $option_value ) ) {
-											selected( in_array( $key, $option_value ), true );
+										if (is_array($option_value)) {
+											selected(in_array($key, $option_value), true);
 										} else {
-											selected( $option_value, $key );
+											selected($option_value, $key);
 										}
 
 										?>><?php echo $val ?></option>
@@ -519,28 +519,28 @@ 
 
 					// Radio inputs.
 					case 'radio_inline' :
-						$value['class'] = empty( $value['class'] ) ? 'give-radio-inline' : $value['class'] . ' give-radio-inline';
+						$value['class'] = empty($value['class']) ? 'give-radio-inline' : $value['class'].' give-radio-inline';
 					case 'radio' :
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 						?>
-					<tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+					<tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
 						<th scope="row" class="titledesc">
-							<label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+							<label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
 						</th>
-						<td class="give-forminp give-forminp-<?php echo sanitize_title( $value['type'] ) ?> <?php echo( ! empty( $value['class'] ) ? $value['class'] : '' ); ?>">
+						<td class="give-forminp give-forminp-<?php echo sanitize_title($value['type']) ?> <?php echo( ! empty($value['class']) ? $value['class'] : ''); ?>">
 							<fieldset>
 								<ul>
 									<?php
-									foreach ( $value['options'] as $key => $val ) {
+									foreach ($value['options'] as $key => $val) {
 										?>
 										<li>
 											<label><input
-														name="<?php echo esc_attr( $value['id'] ); ?>"
+														name="<?php echo esc_attr($value['id']); ?>"
 														value="<?php echo $key; ?>"
 														type="radio"
-														style="<?php echo esc_attr( $value['css'] ); ?>"
-													<?php echo implode( ' ', $custom_attributes ); ?>
-													<?php checked( $key, $option_value ); ?>
+														style="<?php echo esc_attr($value['css']); ?>"
+													<?php echo implode(' ', $custom_attributes); ?>
+													<?php checked($key, $option_value); ?>
 												/> <?php echo $val ?></label>
 										</li>
 										<?php
@@ -554,21 +554,21 @@ 
 
 					// Checkbox input.
 					case 'checkbox' :
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 						?>
-						<tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+						<tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
 							<th scope="row" class="titledesc">
-								<label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+								<label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
 							</th>
 							<td class="give-forminp">
 								<input
-										name="<?php echo esc_attr( $value['id'] ); ?>"
-										id="<?php echo esc_attr( $value['id'] ); ?>"
+										name="<?php echo esc_attr($value['id']); ?>"
+										id="<?php echo esc_attr($value['id']); ?>"
 										type="checkbox"
-										class="<?php echo esc_attr( isset( $value['class'] ) ? $value['class'] : '' ); ?>"
+										class="<?php echo esc_attr(isset($value['class']) ? $value['class'] : ''); ?>"
 										value="1"
-									<?php checked( $option_value, 'on' ); ?>
-									<?php echo implode( ' ', $custom_attributes ); ?>
+									<?php checked($option_value, 'on'); ?>
+									<?php echo implode(' ', $custom_attributes); ?>
 								/>
 								<?php echo $description; ?>
 							</td>
@@ -578,28 +578,28 @@ 
 
 					// Multi Checkbox input.
 					case 'multicheck' :
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
-						$option_value = is_array( $option_value ) ? $option_value : array();
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
+						$option_value = is_array($option_value) ? $option_value : array();
 						?>
-						<tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+						<tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
 							<th scope="row" class="titledesc">
-								<label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+								<label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
 							</th>
-							<td class="give-forminp give-forminp-<?php echo sanitize_title( $value['type'] ) ?> <?php echo( ! empty( $value['class'] ) ? $value['class'] : '' ); ?>">
+							<td class="give-forminp give-forminp-<?php echo sanitize_title($value['type']) ?> <?php echo( ! empty($value['class']) ? $value['class'] : ''); ?>">
 								<fieldset>
 									<ul>
 										<?php
-										foreach ( $value['options'] as $key => $val ) {
+										foreach ($value['options'] as $key => $val) {
 											?>
 											<li>
 												<label>
 													<input
-															name="<?php echo esc_attr( $value['id'] ); ?>[]"
+															name="<?php echo esc_attr($value['id']); ?>[]"
 															value="<?php echo $key; ?>"
 															type="checkbox"
-															style="<?php echo esc_attr( $value['css'] ); ?>"
-														<?php echo implode( ' ', $custom_attributes ); ?>
-														<?php if ( in_array( $key, $option_value ) ) {
+															style="<?php echo esc_attr($value['css']); ?>"
+														<?php echo implode(' ', $custom_attributes); ?>
+														<?php if (in_array($key, $option_value)) {
 															echo 'checked="checked"';
 														} ?>
 													/> <?php echo $val ?>
@@ -618,30 +618,30 @@ 
 					// File input field.
 					case 'file' :
 					case 'media' :
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
-						$button_label = esc_html__( sprintf( 'Add or Upload %s', ( 'file' === $value['type'] ? 'File' : 'Image' ) ), 'give' );
-						$fvalue       = empty( $value['fvalue'] ) ? 'url' : $value['fvalue'];
-
-						$allow_media_preview_tags = array( 'jpg', 'jpeg', 'png', 'gif', 'ico' );
-						$preview_image_src        = $option_value ? ( 'id' === $fvalue ? wp_get_attachment_url( $option_value ) : $option_value ) : '#';
-						$preview_image_extension  = $preview_image_src ? pathinfo( $preview_image_src, PATHINFO_EXTENSION ) : '';
-						$is_show_preview          = in_array( $preview_image_extension, $allow_media_preview_tags );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
+						$button_label = esc_html__(sprintf('Add or Upload %s', ('file' === $value['type'] ? 'File' : 'Image')), 'give');
+						$fvalue       = empty($value['fvalue']) ? 'url' : $value['fvalue'];
+
+						$allow_media_preview_tags = array('jpg', 'jpeg', 'png', 'gif', 'ico');
+						$preview_image_src        = $option_value ? ('id' === $fvalue ? wp_get_attachment_url($option_value) : $option_value) : '#';
+						$preview_image_extension  = $preview_image_src ? pathinfo($preview_image_src, PATHINFO_EXTENSION) : '';
+						$is_show_preview          = in_array($preview_image_extension, $allow_media_preview_tags);
 						?>
-						<tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+						<tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
 							<th scope="row" class="titledesc">
-								<label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+								<label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
 							</th>
 							<td class="give-forminp">
 								<div class="give-field-wrap">
 									<label for="<?php echo $value['id'] ?>">
 										<input
-												name="<?php echo esc_attr( $value['id'] ); ?>"
-												id="<?php echo esc_attr( $value['id'] ); ?>"
+												name="<?php echo esc_attr($value['id']); ?>"
+												id="<?php echo esc_attr($value['id']); ?>"
 												type="text"
-												class="give-input-field<?php echo esc_attr( isset( $value['class'] ) ? ' ' . $value['class'] : '' ); ?>"
+												class="give-input-field<?php echo esc_attr(isset($value['class']) ? ' '.$value['class'] : ''); ?>"
 												value="<?php echo $option_value; ?>"
-												style="<?php echo esc_attr( $value['css'] ); ?>"
-											<?php echo implode( ' ', $custom_attributes ); ?>
+												style="<?php echo esc_attr($value['css']); ?>"
+											<?php echo implode(' ', $custom_attributes); ?>
 										/>&nbsp;&nbsp;&nbsp;&nbsp;<input class="give-upload-button button" type="button" data-fvalue="<?php echo $fvalue; ?>" data-field-type="<?php echo $value['type']; ?>" value="<?php echo $button_label; ?>">
 										<?php echo $description ?>
 										<div class="give-image-thumb<?php echo ! $option_value || ! $is_show_preview ? ' give-hidden' : ''; ?>">
@@ -658,17 +658,17 @@ 
 					// WordPress Editor.
 					case 'wysiwyg' :
 						// Get option value.
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 
 						// Get editor settings.
-						$editor_settings = ! empty( $value['options'] ) ? $value['options'] : array();
+						$editor_settings = ! empty($value['options']) ? $value['options'] : array();
 						?>
-					<tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+					<tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
 						<th scope="row" class="titledesc">
-							<label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+							<label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
 						</th>
 						<td class="give-forminp">
-							<?php wp_editor( $option_value, $value['id'], $editor_settings ); ?>
+							<?php wp_editor($option_value, $value['id'], $editor_settings); ?>
 							<?php echo $description; ?>
 						</td>
 						</tr><?php
@@ -677,9 +677,9 @@ 
 					// Custom: System setting field.
 					case 'system_info' :
 						?>
-					<tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+					<tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
 						<th scope="row" class="titledesc">
-							<label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+							<label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
 						</th>
 						<td class="give-forminp">
 							<?php give_system_info_callback(); ?>
@@ -690,14 +690,14 @@ 
 
 					// Custom: Default gateways setting field.
 					case 'default_gateway' :
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 						?>
-					<tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+					<tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
 						<th scope="row" class="titledesc">
-							<label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+							<label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
 						</th>
 						<td class="give-forminp">
-							<?php give_default_gateway_callback( $value, $option_value ); ?>
+							<?php give_default_gateway_callback($value, $option_value); ?>
 							<?php echo $description; ?>
 						</td>
 						</tr><?php
@@ -705,14 +705,14 @@ 
 
 					// Custom: Enable gateways setting field.
 					case 'enabled_gateways' :
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
 						?>
-					<tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+					<tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
 						<th scope="row" class="titledesc">
-							<label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+							<label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
 						</th>
 						<td class="give-forminp">
-							<?php give_enabled_gateways_callback( $value, $option_value ); ?>
+							<?php give_enabled_gateways_callback($value, $option_value); ?>
 							<?php echo $description; ?>
 						</td>
 						</tr><?php
@@ -721,9 +721,9 @@ 
 					// Custom: Email preview buttons field.
 					case 'email_preview_buttons' :
 						?>
-					<tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+					<tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
 						<th scope="row" class="titledesc">
-							<label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+							<label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
 						</th>
 						<td class="give-forminp">
 							<?php give_email_preview_buttons_callback(); ?>
@@ -740,22 +740,22 @@ 
 
 					// Custom: Gateway API key.
 					case 'api_key' :
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
-						$type         = ! empty( $option_value ) ? 'password' : 'text';
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
+						$type         = ! empty($option_value) ? 'password' : 'text';
 						?>
-					<tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+					<tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
 						<th scope="row" class="titledesc">
-							<label for="<?php echo esc_attr( $value['id'] ); ?>"><?php echo self::get_field_title( $value ); ?></label>
+							<label for="<?php echo esc_attr($value['id']); ?>"><?php echo self::get_field_title($value); ?></label>
 						</th>
-						<td class="give-forminp give-forminp-<?php echo sanitize_title( $value['type'] ) ?>">
+						<td class="give-forminp give-forminp-<?php echo sanitize_title($value['type']) ?>">
 							<input
-									name="<?php echo esc_attr( $value['id'] ); ?>"
-									id="<?php echo esc_attr( $value['id'] ); ?>"
-									type="<?php echo esc_attr( $type ); ?>"
-									style="<?php echo esc_attr( $value['css'] ); ?>"
-									value="<?php echo esc_attr( trim( $option_value ) ); ?>"
-									class="give-input-field<?php echo( empty( $value['class'] ) ? '' : ' ' . esc_attr( $value['class'] ) ); ?>"
-								<?php echo implode( ' ', $custom_attributes ); ?>
+									name="<?php echo esc_attr($value['id']); ?>"
+									id="<?php echo esc_attr($value['id']); ?>"
+									type="<?php echo esc_attr($type); ?>"
+									style="<?php echo esc_attr($value['css']); ?>"
+									value="<?php echo esc_attr(trim($option_value)); ?>"
+									class="give-input-field<?php echo(empty($value['class']) ? '' : ' '.esc_attr($value['class'])); ?>"
+								<?php echo implode(' ', $custom_attributes); ?>
 							/> <?php echo $description; ?>
 						</td>
 						</tr><?php
@@ -772,7 +772,7 @@ 
 						 *
 						 * @since 1.0
 						 */
-						do_action( "give_logs_view_{$current_section}" );
+						do_action("give_logs_view_{$current_section}");
 
 						echo $description;
 						break;
@@ -780,7 +780,7 @@ 
 					// Custom: Data field.
 					case 'data' :
 
-						include GIVE_PLUGIN_DIR . 'includes/admin/tools/views/html-admin-page-data.php';
+						include GIVE_PLUGIN_DIR.'includes/admin/tools/views/html-admin-page-data.php';
 
 						echo $description;
 						break;
@@ -788,12 +788,12 @@ 
 					// Custom: Give Docs Link field type.
 					case 'give_docs_link' :
 						?>
-					<tr valign="top" <?php echo ! empty( $value['wrapper_class'] ) ? 'class="' . $value['wrapper_class'] . '"' : '' ?>>
+					<tr valign="top" <?php echo ! empty($value['wrapper_class']) ? 'class="'.$value['wrapper_class'].'"' : '' ?>>
 						<td class="give-docs-link" colspan="2">
 							<?php
-							echo '<p class="give-docs-link"><a href="' . esc_url( $value['url'] )
+							echo '<p class="give-docs-link"><a href="'.esc_url($value['url'])
 							     . '" target="_blank">'
-							     . sprintf( esc_html__( 'Need Help? See docs on "%s"', 'give' ), $value['title'] )
+							     . sprintf(esc_html__('Need Help? See docs on "%s"', 'give'), $value['title'])
 							     . '<span class="dashicons dashicons-editor-help"></span></a></p>';
 							?>
 						</td>
@@ -804,8 +804,8 @@ 
 					// You can add or handle your custom field action.
 					default:
 						// Get option value.
-						$option_value = self::get_option( $option_name, $value['id'], $value['default'] );
-						do_action( 'give_admin_field_' . $value['type'], $value, $option_value );
+						$option_value = self::get_option($option_name, $value['id'], $value['default']);
+						do_action('give_admin_field_'.$value['type'], $value, $option_value);
 						break;
 				}
 			}
@@ -821,15 +821,15 @@ 
 		 *
 		 * @return string The HTML description of the field.
 		 */
-		public static function get_field_description( $value ) {
+		public static function get_field_description($value) {
 			$description = '';
 
 			// Support for both 'description' and 'desc' args.
-			$description_key = isset( $value['description'] ) ? 'description' : 'desc';
-			$value           = ( isset( $value[ $description_key ] ) && ! empty( $value[ $description_key ] ) ) ? $value[ $description_key ] : '';
+			$description_key = isset($value['description']) ? 'description' : 'desc';
+			$value           = (isset($value[$description_key]) && ! empty($value[$description_key])) ? $value[$description_key] : '';
 
-			if ( ! empty( $value ) ) {
-				$description = '<p class="give-field-description">' . wp_kses_post( $value ) . '</p>';
+			if ( ! empty($value)) {
+				$description = '<p class="give-field-description">'.wp_kses_post($value).'</p>';
 			}
 
 			return $description;
@@ -846,11 +846,11 @@ 
 		 *
 		 * @return array The description and tip as a 2 element array
 		 */
-		public static function get_field_title( $value ) {
-			$title = esc_html( $value['title'] );
+		public static function get_field_title($value) {
+			$title = esc_html($value['title']);
 
 			// If html tag detected then allow them to print.
-			if ( strip_tags( $title ) ) {
+			if (strip_tags($title)) {
 				$title = $value['title'];
 			}
 
@@ -869,8 +869,8 @@ 
 		 *
 		 * @return bool
 		 */
-		public static function save_fields( $options, $option_name = '' ) {
-			if ( empty( $_POST ) ) {
+		public static function save_fields($options, $option_name = '') {
+			if (empty($_POST)) {
 				return false;
 			}
 
@@ -878,37 +878,37 @@ 
 			$update_options = array();
 
 			// Loop options and get values to save.
-			foreach ( $options as $option ) {
-				if ( ! isset( $option['id'] ) || ! isset( $option['type'] ) ) {
+			foreach ($options as $option) {
+				if ( ! isset($option['id']) || ! isset($option['type'])) {
 					continue;
 				}
 
 				// Get posted value.
-				if ( strstr( $option['id'], '[' ) ) {
-					parse_str( $option['id'], $option_name_array );
-					$field_option_name = current( array_keys( $option_name_array ) );
-					$setting_name      = key( $option_name_array[ $field_option_name ] );
-					$raw_value         = isset( $_POST[ $field_option_name ][ $setting_name ] ) ? wp_unslash( $_POST[ $field_option_name ][ $setting_name ] ) : null;
+				if (strstr($option['id'], '[')) {
+					parse_str($option['id'], $option_name_array);
+					$field_option_name = current(array_keys($option_name_array));
+					$setting_name      = key($option_name_array[$field_option_name]);
+					$raw_value         = isset($_POST[$field_option_name][$setting_name]) ? wp_unslash($_POST[$field_option_name][$setting_name]) : null;
 				} else {
 					$field_option_name = $option['id'];
 					$setting_name      = '';
-					$raw_value         = isset( $_POST[ $option['id'] ] ) ? wp_unslash( $_POST[ $option['id'] ] ) : null;
+					$raw_value         = isset($_POST[$option['id']]) ? wp_unslash($_POST[$option['id']]) : null;
 				}
 
 				// Format the value based on option type.
-				switch ( $option['type'] ) {
+				switch ($option['type']) {
 					case 'checkbox' :
-						$value = is_null( $raw_value ) ? '' : 'on';
+						$value = is_null($raw_value) ? '' : 'on';
 						break;
 					case 'wysiwyg'  :
 					case 'textarea' :
-						$value = wp_kses_post( trim( $raw_value ) );
+						$value = wp_kses_post(trim($raw_value));
 						break;
 					case 'multiselect' :
-						$value = array_filter( array_map( 'give_clean', (array) $raw_value ) );
+						$value = array_filter(array_map('give_clean', (array) $raw_value));
 						break;
 					default :
-						$value = give_clean( $raw_value );
+						$value = give_clean($raw_value);
 						break;
 				}
 
@@ -917,37 +917,37 @@ 
 				 *
 				 * @since 1.8
 				 */
-				$value = apply_filters( 'give_admin_settings_sanitize_option', $value, $option, $raw_value );
+				$value = apply_filters('give_admin_settings_sanitize_option', $value, $option, $raw_value);
 
 				/**
 				 * Sanitize the value of an option by option name.
 				 *
 				 * @since 1.8
 				 */
-				$value = apply_filters( "give_admin_settings_sanitize_option_{$field_option_name}", $value, $option, $raw_value );
+				$value = apply_filters("give_admin_settings_sanitize_option_{$field_option_name}", $value, $option, $raw_value);
 
-				if ( is_null( $value ) ) {
+				if (is_null($value)) {
 					continue;
 				}
 
 				// Check if option is an array and handle that differently to single values.
-				if ( $field_option_name && $setting_name ) {
-					if ( ! isset( $update_options[ $field_option_name ] ) ) {
-						$update_options[ $field_option_name ] = get_option( $field_option_name, array() );
+				if ($field_option_name && $setting_name) {
+					if ( ! isset($update_options[$field_option_name])) {
+						$update_options[$field_option_name] = get_option($field_option_name, array());
 					}
-					if ( ! is_array( $update_options[ $field_option_name ] ) ) {
-						$update_options[ $field_option_name ] = array();
+					if ( ! is_array($update_options[$field_option_name])) {
+						$update_options[$field_option_name] = array();
 					}
-					$update_options[ $field_option_name ][ $setting_name ] = $value;
+					$update_options[$field_option_name][$setting_name] = $value;
 				} else {
-					$update_options[ $field_option_name ] = $value;
+					$update_options[$field_option_name] = $value;
 				}
 			}
 
 			// Save all options in our array or there own option name i.e. option id.
-			if ( empty( $option_name ) ) {
-				foreach ( $update_options as $name => $value ) {
-					update_option( $name, $value );
+			if (empty($option_name)) {
+				foreach ($update_options as $name => $value) {
+					update_option($name, $value);
 
 					/**
 					 * Trigger action.
@@ -956,13 +956,13 @@ 
 					 *
 					 * @since 1.8
 					 */
-					do_action( "give_save_option_{$name}", $value, $name );
+					do_action("give_save_option_{$name}", $value, $name);
 				}
 			} else {
-				$old_options    = ( $old_options = get_option( $option_name ) ) ? $old_options : array();
-				$update_options = array_merge( $old_options, $update_options );
+				$old_options    = ($old_options = get_option($option_name)) ? $old_options : array();
+				$update_options = array_merge($old_options, $update_options);
 
-				update_option( $option_name, $update_options );
+				update_option($option_name, $update_options);
 
 				/**
 				 * Trigger action.
@@ -971,7 +971,7 @@ 
 				 *
 				 * @since 1.8
 				 */
-				do_action( "give_save_settings_{$option_name}", $update_options, $option_name );
+				do_action("give_save_settings_{$option_name}", $update_options, $option_name);
 			}
 
 			return true;

includes/admin/donors/donors.php

Spacing +170 added lines, -170 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,9 +24,9 @@ 
  */
 function give_donors_page() {
 	$default_views  = give_donor_views();
-	$requested_view = isset( $_GET['view'] ) ? sanitize_text_field( $_GET['view'] ) : 'donors';
-	if ( array_key_exists( $requested_view, $default_views ) && function_exists( $default_views[ $requested_view ] ) ) {
-		give_render_donor_view( $requested_view, $default_views );
+	$requested_view = isset($_GET['view']) ? sanitize_text_field($_GET['view']) : 'donors';
+	if (array_key_exists($requested_view, $default_views) && function_exists($default_views[$requested_view])) {
+		give_render_donor_view($requested_view, $default_views);
 	} else {
 		give_donors_list();
 	}
@@ -42,7 +42,7 @@ 
 
 	$views = array();
 
-	return apply_filters( 'give_donor_views', $views );
+	return apply_filters('give_donor_views', $views);
 
 }
 
@@ -56,7 +56,7 @@ 
 
 	$tabs = array();
 
-	return apply_filters( 'give_donor_tabs', $tabs );
+	return apply_filters('give_donor_tabs', $tabs);
 
 }
 
@@ -67,7 +67,7 @@ 
  * @return void
  */
 function give_donors_list() {
-	include dirname( __FILE__ ) . '/class-donor-table.php';
+	include dirname(__FILE__).'/class-donor-table.php';
 
 	$donors_table = new Give_Donor_List_Table();
 	$donors_table->prepare_items();
@@ -80,12 +80,12 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_donors_table_top' );
+		do_action('give_donors_table_top');
 		?>
 
 		<hr class="wp-header-end">
-		<form id="give-donors-search-filter" method="get" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors' ); ?>">
-			<?php $donors_table->search_box( __( 'Search Donors', 'give' ), 'give-donors' ); ?>
+		<form id="give-donors-search-filter" method="get" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors'); ?>">
+			<?php $donors_table->search_box(__('Search Donors', 'give'), 'give-donors'); ?>
 			<input type="hidden" name="post_type" value="give_forms" />
 			<input type="hidden" name="page" value="give-donors" />
 			<input type="hidden" name="view" value="donors" />
@@ -102,7 +102,7 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_donors_table_bottom' );
+		do_action('give_donors_table_bottom');
 		?>
 	</div>
 	<?php
@@ -118,33 +118,33 @@ 
  *
  * @return void
  */
-function give_render_donor_view( $view, $callbacks ) {
+function give_render_donor_view($view, $callbacks) {
 
 	$render = true;
 
-	$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 ( ! current_user_can( $donor_view_role ) ) {
-		give_set_error( 'give-no-access', __( 'You are not permitted to view this data.', 'give' ) );
+	if ( ! current_user_can($donor_view_role)) {
+		give_set_error('give-no-access', __('You are not permitted to view this data.', 'give'));
 		$render = false;
 	}
 
-	if ( ! isset( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
-		give_set_error( 'give-invalid_donor', __( 'Invalid Donor ID.', 'give' ) );
+	if ( ! isset($_GET['id']) || ! is_numeric($_GET['id'])) {
+		give_set_error('give-invalid_donor', __('Invalid Donor ID.', 'give'));
 		$render = false;
 	}
 
 	$donor_id          = (int) $_GET['id'];
-	$reconnect_user_id = ! empty( $_GET['user_id'] ) ? (int) $_GET['user_id'] : '';
-	$donor             = new Give_Donor( $donor_id );
+	$reconnect_user_id = ! empty($_GET['user_id']) ? (int) $_GET['user_id'] : '';
+	$donor             = new Give_Donor($donor_id);
 
 	// Reconnect User with Donor profile.
-	if( $reconnect_user_id ) {
-		give_connect_user_donor_profile( $donor, array( 'user_id' => $reconnect_user_id ), array() );
+	if ($reconnect_user_id) {
+		give_connect_user_donor_profile($donor, array('user_id' => $reconnect_user_id), array());
 	}
 
-	if ( empty( $donor->id ) ) {
-		give_set_error( 'give-invalid_donor', __( 'Invalid Donor ID.', 'give' ) );
+	if (empty($donor->id)) {
+		give_set_error('give-invalid_donor', __('Invalid Donor ID.', 'give'));
 		$render = false;
 	}
 
@@ -153,9 +153,9 @@ 
 
 	<div class='wrap'>
 
-		<?php if ( give_get_errors() ) : ?>
+		<?php if (give_get_errors()) : ?>
 			<div class="error settings-error">
-				<?php Give()->notices->render_frontend_notices( 0 ); ?>
+				<?php Give()->notices->render_frontend_notices(0); ?>
 			</div>
 		<?php endif; ?>
 
@@ -163,7 +163,7 @@ 
 			<?php
 			printf(
 			/* translators: %s: donor number */
-				esc_html__( 'Donor %s', 'give' ),
+				esc_html__('Donor %s', 'give'),
 				$donor_id
 			);
 			?>
@@ -171,26 +171,26 @@ 
 
 		<hr class="wp-header-end">
 		
-		<?php if ( $donor && $render ) : ?>
+		<?php if ($donor && $render) : ?>
 
 			<h2 class="nav-tab-wrapper">
 				<?php
-				foreach ( $donor_tabs as $key => $tab ) :
+				foreach ($donor_tabs as $key => $tab) :
 					$active = $key === $view ? true : false;
 					$class = $active ? 'nav-tab nav-tab-active' : 'nav-tab';
 					printf(
-						'<a href="%1$s" class="%2$s"><span class="dashicons %3$s"></span>%4$s</a>' . "\n",
-						esc_url( admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=' . $key . '&id=' . $donor->id ) ),
-						esc_attr( $class ),
-						sanitize_html_class( $tab['dashicon'] ),
-						esc_html( $tab['title'] )
+						'<a href="%1$s" class="%2$s"><span class="dashicons %3$s"></span>%4$s</a>'."\n",
+						esc_url(admin_url('edit.php?post_type=give_forms&page=give-donors&view='.$key.'&id='.$donor->id)),
+						esc_attr($class),
+						sanitize_html_class($tab['dashicon']),
+						esc_html($tab['title'])
 					);
 				endforeach;
 				?>
 			</h2>
 
 			<div id="give-donor-card-wrapper">
-				<?php $callbacks[ $view ]( $donor ) ?>
+				<?php $callbacks[$view]($donor) ?>
 			</div>
 
 		<?php endif; ?>
@@ -210,9 +210,9 @@ 
  *
  * @return void
  */
-function give_donor_view( $donor ) {
+function give_donor_view($donor) {
 
-	$donor_edit_role   = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
 	/**
 	 * Fires in donor profile screen, above the donor card.
@@ -221,32 +221,32 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_card_top', $donor );
+	do_action('give_donor_card_top', $donor);
 	?>
 
 	<div id="donor-summary" class="info-wrapper donor-section postbox">
 
-		<form id="edit-donor-info" method="post" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ); ?>">
+		<form id="edit-donor-info" method="post" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id); ?>">
 
 			<div class="donor-info">
 
 				<div class="donor-bio-header clearfix">
 
 					<div class="avatar-wrap left" id="donor-avatar">
-						<?php echo get_avatar( $donor->email ); ?>
+						<?php echo get_avatar($donor->email); ?>
 					</div>
 
 					<div id="donor-name-wrap" class="left">
 						<span class="donor-id">#<?php echo $donor->id; ?></span>
-						<span class="donor-name info-item edit-item"><input size="15" data-key="name" name="customerinfo[name]" type="text" value="<?php echo esc_attr( $donor->name ); ?>" placeholder="<?php _e( 'Donor Name', 'give' ); ?>" /></span>
+						<span class="donor-name info-item edit-item"><input size="15" data-key="name" name="customerinfo[name]" type="text" value="<?php echo esc_attr($donor->name); ?>" placeholder="<?php _e('Donor Name', 'give'); ?>" /></span>
 						<span class="donor-name info-item editable"><span data-key="name"><?php echo $donor->name; ?></span></span>
 					</div>
 					<p class="donor-since info-item">
-						<?php _e( 'Donor since', 'give' ); ?>
-						<?php echo date_i18n( give_date_format(), strtotime( $donor->date_created ) ) ?>
+						<?php _e('Donor since', 'give'); ?>
+						<?php echo date_i18n(give_date_format(), strtotime($donor->date_created)) ?>
 					</p>
-					<?php if ( current_user_can( $donor_edit_role ) ) : ?>
-						<a href="#" id="edit-donor" class="button info-item editable donor-edit-link"><?php _e( 'Edit Donor', 'give' ); ?></a>
+					<?php if (current_user_can($donor_edit_role)) : ?>
+						<a href="#" id="edit-donor" class="button info-item editable donor-edit-link"><?php _e('Edit Donor', 'give'); ?></a>
 					<?php endif; ?>
 				</div>
 				<!-- /donor-bio-header -->
@@ -256,7 +256,7 @@ 
 					<table class="widefat">
 						<tbody>
 						<tr class="alternate">
-							<th scope="col"><label for="tablecell"><?php _e( 'User:', 'give' ); ?></label></th>
+							<th scope="col"><label for="tablecell"><?php _e('User:', 'give'); ?></label></th>
 							<td>
 								<span class="donor-user-id info-item edit-item">
 									<?php
@@ -273,48 +273,48 @@ 
 										'data'  => $data_atts,
 									);
 
-									if ( ! empty( $user_id ) ) {
-										$userdata           = get_userdata( $user_id );
+									if ( ! empty($user_id)) {
+										$userdata = get_userdata($user_id);
 										$user_args['selected'] = $user_id;
 									}
 
-									echo Give()->html->ajax_user_search( $user_args );
+									echo Give()->html->ajax_user_search($user_args);
 									?>
 								</span>
 
 								<span class="donor-user-id info-item editable">
-									<?php if ( ! empty( $userdata ) ) { ?>
-										<span data-key="user_id">#<?php echo $donor->user_id . ' - ' . $userdata->display_name; ?></span>
+									<?php if ( ! empty($userdata)) { ?>
+										<span data-key="user_id">#<?php echo $donor->user_id.' - '.$userdata->display_name; ?></span>
 									<?php } else { ?>
-										<span data-key="user_id"><?php _e( 'None', 'give' ); ?></span>
+										<span data-key="user_id"><?php _e('None', 'give'); ?></span>
 									<?php } ?>
-									<?php if ( current_user_can( $donor_edit_role ) && intval( $donor->user_id ) > 0 ) { ?>
+									<?php if (current_user_can($donor_edit_role) && intval($donor->user_id) > 0) { ?>
 										<span class="disconnect-user">
  											-
- 											<a id="disconnect-donor" href="#disconnect" aria-label="<?php _e( 'Disconnects the current user ID from this donor record.', 'give' ); ?>">
- 												<?php _e( 'Disconnect User', 'give' ); ?>
+ 											<a id="disconnect-donor" href="#disconnect" aria-label="<?php _e('Disconnects the current user ID from this donor record.', 'give'); ?>">
+ 												<?php _e('Disconnect User', 'give'); ?>
 											</a>
  										</span>
 										<span class="view-user-profile">
  											|
- 											<a id="view-user-profile" href="<?php echo 'user-edit.php?user_id=' . $donor->user_id; ?>" aria-label="<?php _e( 'View User Profile of current user ID.', 'give' ); ?>">
- 												<?php _e( 'View User Profile', 'give' ); ?>
+ 											<a id="view-user-profile" href="<?php echo 'user-edit.php?user_id='.$donor->user_id; ?>" aria-label="<?php _e('View User Profile of current user ID.', 'give'); ?>">
+ 												<?php _e('View User Profile', 'give'); ?>
 											</a>
  										</span>
 									<?php } ?>
 								</span>
 							</td>
 						</tr>
-						<?php if ( isset( $donor->user_id ) && $donor->user_id > 0 ) : ?>
+						<?php if (isset($donor->user_id) && $donor->user_id > 0) : ?>
 
 							<tr>
-								<th scope="col"><?php _e( 'Address:', 'give' ); ?></th>
+								<th scope="col"><?php _e('Address:', 'give'); ?></th>
 								<td class="row-title">
 
 									<div class="donor-address-wrapper">
 
 										<?php
-										$address  = get_user_meta( $donor->user_id, '_give_user_address', true );
+										$address  = get_user_meta($donor->user_id, '_give_user_address', true);
 										$defaults = array(
 											'line1'   => '',
 											'line2'   => '',
@@ -324,10 +324,10 @@ 
 											'zip'     => '',
 										);
 
-										$address = wp_parse_args( $address, $defaults );
+										$address = wp_parse_args($address, $defaults);
 										?>
 
-										<?php if ( ! empty( $address ) ) { ?>
+										<?php if ( ! empty($address)) { ?>
 											<span class="donor-address info-item editable">
 												<span class="info-item" data-key="line1"><?php echo $address['line1']; ?></span>
 												<span class="info-item" data-key="line2"><?php echo $address['line2']; ?></span>
@@ -344,8 +344,8 @@ 
 
 										// For State.
 										$selected_state = give_get_state();
-										$states         = give_get_states( $selected_country );
-										$selected_state = ( isset( $address['state'] ) ? $address['state'] : $selected_state );
+										$states         = give_get_states($selected_country);
+										$selected_state = (isset($address['state']) ? $address['state'] : $selected_state);
 
 										// Get the country list that does not have any states init.
 										$no_states_country = give_no_states_country_list();
@@ -353,32 +353,32 @@ 
 										<span class="donor-address info-item edit-item">
 											<select data-key="country" name="customerinfo[country]" id="billing_country" class="billing_country give-select edit-item">
 												<?php
-												foreach ( $countries as $country_code => $country ) {
-													echo '<option value="' . esc_attr( $country_code ) . '"' . selected( $country_code, $selected_country, false ) . '>' . $country . '</option>';
+												foreach ($countries as $country_code => $country) {
+													echo '<option value="'.esc_attr($country_code).'"'.selected($country_code, $selected_country, false).'>'.$country.'</option>';
 												}
 												?>
 											</select>
-											<input class="info-item" type="text" data-key="line1" name="customerinfo[line1]" placeholder="<?php _e( 'Address 1', 'give' ); ?>" value="<?php echo $address['line1']; ?>" />
-											<input class="info-item" type="text" data-key="line2" name="customerinfo[line2]" placeholder="<?php _e( 'Address 2', 'give' ); ?>" value="<?php echo $address['line2']; ?>" />
-											<input class="info-item" type="text" data-key="city" name="customerinfo[city]" placeholder="<?php _e( 'City', 'give' ); ?>" value="<?php echo $address['city']; ?>" />
+											<input class="info-item" type="text" data-key="line1" name="customerinfo[line1]" placeholder="<?php _e('Address 1', 'give'); ?>" value="<?php echo $address['line1']; ?>" />
+											<input class="info-item" type="text" data-key="line2" name="customerinfo[line2]" placeholder="<?php _e('Address 2', 'give'); ?>" value="<?php echo $address['line2']; ?>" />
+											<input class="info-item" type="text" data-key="city" name="customerinfo[city]" placeholder="<?php _e('City', 'give'); ?>" value="<?php echo $address['city']; ?>" />
 											<?php
-											if ( ! empty( $states ) ) {
+											if ( ! empty($states)) {
 												?>
 												<select data-key="state" name="customerinfo[state]" id="card_state" class="card_state give-select info-item">
 													<?php
-													foreach ( $states as $state_code => $state ) {
-														echo '<option value="' . $state_code . '"' . selected( $state_code, $selected_state, false ) . '>' . $state . '</option>';
+													foreach ($states as $state_code => $state) {
+														echo '<option value="'.$state_code.'"'.selected($state_code, $selected_state, false).'>'.$state.'</option>';
 													}
 													?>
 												</select>
 												<?php
 											} else {
 												?>
-												<input type="text" size="6" data-key="state" name="customerinfo[state]" id="card_state" class="card_state give-input info-item <?php echo ( ! empty( $selected_country ) && array_key_exists( $selected_country, $no_states_country ) ? 'give-hidden' : '' ); ?>" placeholder="<?php _e( 'State / Province / County', 'give' ); ?>" />
+												<input type="text" size="6" data-key="state" name="customerinfo[state]" id="card_state" class="card_state give-input info-item <?php echo ( ! empty($selected_country) && array_key_exists($selected_country, $no_states_country) ? 'give-hidden' : ''); ?>" placeholder="<?php _e('State / Province / County', 'give'); ?>" />
 												<?php
 											}
 											?>
-											<input class="info-item" type="text" data-key="zip" name="customerinfo[zip]" placeholder="<?php _e( 'Zip / Postal Code', 'give' ); ?>" value="<?php echo $address['zip']; ?>" />
+											<input class="info-item" type="text" data-key="zip" name="customerinfo[zip]" placeholder="<?php _e('Zip / Postal Code', 'give'); ?>" value="<?php echo $address['zip']; ?>" />
 										</span>
 
 									</div>
@@ -394,10 +394,10 @@ 
 
 			<span id="donor-edit-actions" class="edit-item">
 				<input type="hidden" data-key="id" name="customerinfo[id]" value="<?php echo $donor->id; ?>" />
-				<?php wp_nonce_field( 'edit-donor', '_wpnonce', false, true ); ?>
+				<?php wp_nonce_field('edit-donor', '_wpnonce', false, true); ?>
 				<input type="hidden" name="give_action" value="edit-donor" />
-				<input type="submit" id="give-edit-donor-save" class="button-secondary" value="<?php _e( 'Update Donor', 'give' ); ?>" />
-				<a id="give-edit-donor-cancel" href="" class="delete"><?php _e( 'Cancel', 'give' ); ?></a>
+				<input type="submit" id="give-edit-donor-save" class="button-secondary" value="<?php _e('Update Donor', 'give'); ?>" />
+				<a id="give-edit-donor-cancel" href="" class="delete"><?php _e('Cancel', 'give'); ?></a>
 			</span>
 
 		</form>
@@ -412,24 +412,24 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_before_stats', $donor );
+	do_action('give_donor_before_stats', $donor);
 	?>
 
 	<div id="donor-stats-wrapper" class="donor-section postbox clear">
 		<ul>
 			<li>
-				<a href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&donor=' . absint( $donor->id ) ); ?>">
+				<a href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-payment-history&donor='.absint($donor->id)); ?>">
 					<span class="dashicons dashicons-heart"></span>
 					<?php
 					// Completed Donations.
-					$completed_donations_text = sprintf( _n( '%d Completed Donation', '%d Completed Donations', $donor->purchase_count, 'give' ), $donor->purchase_count );
-					echo apply_filters( 'give_donor_completed_donations', $completed_donations_text, $donor );
+					$completed_donations_text = sprintf(_n('%d Completed Donation', '%d Completed Donations', $donor->purchase_count, 'give'), $donor->purchase_count);
+					echo apply_filters('give_donor_completed_donations', $completed_donations_text, $donor);
 					?>
 				</a>
 			</li>
 			<li>
 				<span class="dashicons dashicons-chart-area"></span>
-				<?php echo give_currency_filter( give_format_amount( $donor->get_total_donation_amount(), array( 'sanitize' => false ) ) ); ?> <?php _e( 'Lifetime Donations', 'give' ); ?>
+				<?php echo give_currency_filter(give_format_amount($donor->get_total_donation_amount(), array('sanitize' => false))); ?> <?php _e('Lifetime Donations', 'give'); ?>
 			</li>
 			<?php
 			/**
@@ -441,7 +441,7 @@ 
 			 *
 			 * @param object $donor The donor object being displayed.
 			 */
-			do_action( 'give_donor_stats_list', $donor );
+			do_action('give_donor_stats_list', $donor);
 			?>
 		</ul>
 	</div>
@@ -454,7 +454,7 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_before_tables_wrapper', $donor );
+	do_action('give_donor_before_tables_wrapper', $donor);
 	?>
 
 	<div id="donor-tables-wrapper" class="donor-section">
@@ -467,46 +467,46 @@ 
 		 *
 		 * @param object $donor The donor object being displayed.
 		 */
-		do_action( 'give_donor_before_tables', $donor );
+		do_action('give_donor_before_tables', $donor);
 		?>
 
-		<h3><?php _e( 'Donor Emails', 'give' ); ?></h3>
+		<h3><?php _e('Donor Emails', 'give'); ?></h3>
 
 		<table class="wp-list-table widefat striped emails">
 			<thead>
 			<tr>
-				<th><?php _e( 'Email', 'give' ); ?></th>
-				<th><?php _e( 'Actions', 'give' ); ?></th>
+				<th><?php _e('Email', 'give'); ?></th>
+				<th><?php _e('Actions', 'give'); ?></th>
 			</tr>
 			</thead>
 
 			<tbody>
-			<?php if ( ! empty( $donor->emails ) ) { ?>
+			<?php if ( ! empty($donor->emails)) { ?>
 
-				<?php foreach ( $donor->emails as $key => $email ) : ?>
+				<?php foreach ($donor->emails as $key => $email) : ?>
 					<tr data-key="<?php echo $key; ?>">
 						<td>
 							<?php echo $email; ?>
-							<?php if ( 'primary' === $key ) : ?>
+							<?php if ('primary' === $key) : ?>
 								<span class="dashicons dashicons-star-filled primary-email-icon"></span>
 							<?php endif; ?>
 						</td>
 						<td>
-							<?php if ( 'primary' !== $key ) : ?>
+							<?php if ('primary' !== $key) : ?>
 								<?php
-								$base_url    = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id );
-								$promote_url = wp_nonce_url( add_query_arg( array(
-									'email' => rawurlencode( $email ),
+								$base_url    = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id);
+								$promote_url = wp_nonce_url(add_query_arg(array(
+									'email' => rawurlencode($email),
 									'give_action' => 'set_donor_primary_email',
-								), $base_url ), 'give-set-donor-primary-email' );
-								$remove_url  = wp_nonce_url( add_query_arg( array(
-									'email' => rawurlencode( $email ),
+								), $base_url), 'give-set-donor-primary-email');
+								$remove_url = wp_nonce_url(add_query_arg(array(
+									'email' => rawurlencode($email),
 									'give_action' => 'remove_donor_email',
-								), $base_url ), 'give-remove-donor-email' );
+								), $base_url), 'give-remove-donor-email');
 								?>
-								<a href="<?php echo $promote_url; ?>"><?php _e( 'Make Primary', 'give' ); ?></a>
+								<a href="<?php echo $promote_url; ?>"><?php _e('Make Primary', 'give'); ?></a>
 								&nbsp;|&nbsp;
-								<a href="<?php echo $remove_url; ?>" class="delete"><?php _e( 'Remove', 'give' ); ?></a>
+								<a href="<?php echo $remove_url; ?>" class="delete"><?php _e('Remove', 'give'); ?></a>
 							<?php endif; ?>
 						</td>
 					</tr>
@@ -516,59 +516,59 @@ 
 					<td colspan="2" class="add-donor-email-td">
 						<div class="add-donor-email-wrapper">
 							<input type="hidden" name="donor-id" value="<?php echo $donor->id; ?>" />
-							<?php wp_nonce_field( 'give_add_donor_email', 'add_email_nonce', false, true ); ?>
-							<input type="email" name="additional-email" value="" placeholder="<?php _e( 'Email Address', 'give' ); ?>" />&nbsp;
-							<input type="checkbox" name="make-additional-primary" value="1" id="make-additional-primary" />&nbsp;<label for="make-additional-primary"><?php _e( 'Make Primary', 'give' ); ?></label>
-							<button class="button-secondary give-add-donor-email" id="add-donor-email"><?php _e( 'Add Email', 'give' ); ?></button>
+							<?php wp_nonce_field('give_add_donor_email', 'add_email_nonce', false, true); ?>
+							<input type="email" name="additional-email" value="" placeholder="<?php _e('Email Address', 'give'); ?>" />&nbsp;
+							<input type="checkbox" name="make-additional-primary" value="1" id="make-additional-primary" />&nbsp;<label for="make-additional-primary"><?php _e('Make Primary', 'give'); ?></label>
+							<button class="button-secondary give-add-donor-email" id="add-donor-email"><?php _e('Add Email', 'give'); ?></button>
 							<span class="spinner"></span>
 						</div>
 						<div class="notice-wrap"></div>
 					</td>
 				</tr>
 			<?php } else { ?>
-				<tr><td colspan="2"><?php _e( 'No Emails Found', 'give' ); ?></td></tr>
+				<tr><td colspan="2"><?php _e('No Emails Found', 'give'); ?></td></tr>
 			<?php }// End if().
 			?>
 			</tbody>
 		</table>
 
-		<h3><?php _e( 'Recent Donations', 'give' ); ?></h3>
+		<h3><?php _e('Recent Donations', 'give'); ?></h3>
 		<?php
-		$payment_ids = explode( ',', $donor->payment_ids );
-		$payments    = give_get_payments( array(
+		$payment_ids = explode(',', $donor->payment_ids);
+		$payments    = give_get_payments(array(
 			'post__in' => $payment_ids,
-		) );
-		$payments    = array_slice( $payments, 0, 10 );
+		));
+		$payments    = array_slice($payments, 0, 10);
 		?>
 		<table class="wp-list-table widefat striped payments">
 			<thead>
 			<tr>
-				<th scope="col"><?php _e( 'ID', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Amount', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Date', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Status', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Actions', 'give' ); ?></th>
+				<th scope="col"><?php _e('ID', 'give'); ?></th>
+				<th scope="col"><?php _e('Amount', 'give'); ?></th>
+				<th scope="col"><?php _e('Date', 'give'); ?></th>
+				<th scope="col"><?php _e('Status', 'give'); ?></th>
+				<th scope="col"><?php _e('Actions', 'give'); ?></th>
 			</tr>
 			</thead>
 			<tbody>
-			<?php if ( ! empty( $payments ) ) { ?>
-				<?php foreach ( $payments as $payment ) : ?>
+			<?php if ( ! empty($payments)) { ?>
+				<?php foreach ($payments as $payment) : ?>
 					<tr>
 						<td><?php echo $payment->ID; ?></td>
-						<td><?php echo give_donation_amount( $payment->ID, 'donor' ); ?></td>
-						<td><?php echo date_i18n( give_date_format(), strtotime( $payment->post_date ) ); ?></td>
-						<td><?php echo give_get_payment_status( $payment, true ); ?></td>
+						<td><?php echo give_donation_amount($payment->ID, 'donor'); ?></td>
+						<td><?php echo date_i18n(give_date_format(), strtotime($payment->post_date)); ?></td>
+						<td><?php echo give_get_payment_status($payment, true); ?></td>
 						<td>
 							<?php
 							printf(
 								'<a href="%1$s" aria-label="%2$s">%3$s</a>',
-								admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id=' . $payment->ID ),
+								admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id='.$payment->ID),
 								sprintf(
 								/* translators: %s: Donation ID */
-									esc_attr__( 'View Donation %s.', 'give' ),
+									esc_attr__('View Donation %s.', 'give'),
 									$payment->ID
 								),
-								__( 'View Donation', 'give' )
+								__('View Donation', 'give')
 							);
 							?>
 
@@ -583,47 +583,47 @@ 
 							 * @param object $donor The donor object being displayed.
 							 * @param object $payment  The payment object being displayed.
 							 */
-							do_action( 'give_donor_recent_purchases_actions', $donor, $payment );
+							do_action('give_donor_recent_purchases_actions', $donor, $payment);
 							?>
 						</td>
 					</tr>
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<tr>
-					<td colspan="5"><?php _e( 'No donations found.', 'give' ); ?></td>
+					<td colspan="5"><?php _e('No donations found.', 'give'); ?></td>
 				</tr>
 			<?php }// End if().
 			?>
 			</tbody>
 		</table>
 
-		<h3><?php _e( 'Completed Forms', 'give' ); ?></h3>
+		<h3><?php _e('Completed Forms', 'give'); ?></h3>
 		<?php
-		$donations = give_get_users_completed_donations( $donor->email );
+		$donations = give_get_users_completed_donations($donor->email);
 		?>
 		<table class="wp-list-table widefat striped donations">
 			<thead>
 			<tr>
-				<th scope="col"><?php _e( 'Form', 'give' ); ?></th>
-				<th scope="col" width="120px"><?php _e( 'Actions', 'give' ); ?></th>
+				<th scope="col"><?php _e('Form', 'give'); ?></th>
+				<th scope="col" width="120px"><?php _e('Actions', 'give'); ?></th>
 			</tr>
 			</thead>
 			<tbody>
-			<?php if ( ! empty( $donations ) ) { ?>
-				<?php foreach ( $donations as $donation ) : ?>
+			<?php if ( ! empty($donations)) { ?>
+				<?php foreach ($donations as $donation) : ?>
 					<tr>
 						<td><?php echo $donation->post_title; ?></td>
 						<td>
 							<?php
 							printf(
 								'<a href="%1$s" aria-label="%2$s">%3$s</a>',
-								esc_url( admin_url( 'post.php?action=edit&post=' . $donation->ID ) ),
+								esc_url(admin_url('post.php?action=edit&post='.$donation->ID)),
 								sprintf(
 								/* translators: %s: form name */
-									esc_attr__( 'View Form %s.', 'give' ),
+									esc_attr__('View Form %s.', 'give'),
 									$donation->post_title
 								),
-								__( 'View Form', 'give' )
+								__('View Form', 'give')
 							);
 							?>
 						</td>
@@ -631,7 +631,7 @@ 
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<tr>
-					<td colspan="2"><?php _e( 'No completed donations found.', 'give' ); ?></td>
+					<td colspan="2"><?php _e('No completed donations found.', 'give'); ?></td>
 				</tr>
 			<?php } ?>
 			</tbody>
@@ -645,7 +645,7 @@ 
 		 *
 		 * @param object $donor The donor object being displayed.
 		 */
-		do_action( 'give_donor_after_tables', $donor );
+		do_action('give_donor_after_tables', $donor);
 		?>
 
 	</div>
@@ -658,7 +658,7 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_card_bottom', $donor );
+	do_action('give_donor_card_bottom', $donor);
 
 }
 
@@ -671,30 +671,30 @@ 
  *
  * @return void
  */
-function give_donor_notes_view( $donor ) {
+function give_donor_notes_view($donor) {
 
-	$paged          = isset( $_GET['paged'] ) && is_numeric( $_GET['paged'] ) ? $_GET['paged'] : 1;
-	$paged          = absint( $paged );
+	$paged          = isset($_GET['paged']) && is_numeric($_GET['paged']) ? $_GET['paged'] : 1;
+	$paged          = absint($paged);
 	$note_count     = $donor->get_notes_count();
-	$per_page       = apply_filters( 'give_donor_notes_per_page', 20 );
-	$total_pages    = ceil( $note_count / $per_page );
-	$donor_notes = $donor->get_notes( $per_page, $paged );
+	$per_page       = apply_filters('give_donor_notes_per_page', 20);
+	$total_pages    = ceil($note_count / $per_page);
+	$donor_notes = $donor->get_notes($per_page, $paged);
 	?>
 
 	<div id="donor-notes-wrapper">
 		<div class="donor-notes-header">
-			<?php echo get_avatar( $donor->email, 30 ); ?> <span><?php echo $donor->name; ?></span>
+			<?php echo get_avatar($donor->email, 30); ?> <span><?php echo $donor->name; ?></span>
 		</div>
-		<h3><?php _e( 'Notes', 'give' ); ?></h3>
+		<h3><?php _e('Notes', 'give'); ?></h3>
 
-		<?php if ( 1 == $paged ) : ?>
+		<?php if (1 == $paged) : ?>
 			<div style="display: block; margin-bottom: 55px;">
-				<form id="give-add-donor-note" method="post" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=notes&id=' . $donor->id ); ?>">
+				<form id="give-add-donor-note" method="post" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=notes&id='.$donor->id); ?>">
 					<textarea id="donor-note" name="donor_note" class="donor-note-input" rows="10"></textarea>
 					<br />
 					<input type="hidden" id="donor-id" name="customer_id" value="<?php echo $donor->id; ?>" />
 					<input type="hidden" name="give_action" value="add-donor-note" />
-					<?php wp_nonce_field( 'add-donor-note', 'add_donor_note_nonce', true, true ); ?>
+					<?php wp_nonce_field('add-donor-note', 'add_donor_note_nonce', true, true); ?>
 					<input id="add-donor-note" class="right button-primary" type="submit" value="Add Note" />
 				</form>
 			</div>
@@ -709,26 +709,26 @@ 
 			'show_all' => true,
 		);
 
-		echo paginate_links( $pagination_args );
+		echo paginate_links($pagination_args);
 		?>
 
 		<div id="give-donor-notes" class="postbox">
-			<?php if ( count( $donor_notes ) > 0 ) { ?>
-				<?php foreach ( $donor_notes as $key => $note ) : ?>
+			<?php if (count($donor_notes) > 0) { ?>
+				<?php foreach ($donor_notes as $key => $note) : ?>
 					<div class="donor-note-wrapper dashboard-comment-wrap comment-item">
 					<span class="note-content-wrap">
-						<?php echo stripslashes( $note ); ?>
+						<?php echo stripslashes($note); ?>
 					</span>
 					</div>
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<div class="give-no-donor-notes">
-					<?php _e( 'No donor notes found.', 'give' ); ?>
+					<?php _e('No donor notes found.', 'give'); ?>
 				</div>
 			<?php } ?>
 		</div>
 
-		<?php echo paginate_links( $pagination_args ); ?>
+		<?php echo paginate_links($pagination_args); ?>
 
 	</div>
 
@@ -744,9 +744,9 @@ 
  *
  * @return void
  */
-function give_donor_delete_view( $donor ) {
+function give_donor_delete_view($donor) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
 	/**
 	 * Fires in donor delete screen, above the content.
@@ -755,15 +755,15 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_delete_top', $donor );
+	do_action('give_donor_delete_top', $donor);
 	?>
 
 	<div class="info-wrapper donor-section">
 
-		<form id="delete-donor" method="post" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=delete&id=' . $donor->id ); ?>">
+		<form id="delete-donor" method="post" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=delete&id='.$donor->id); ?>">
 
 			<div class="donor-notes-header">
-				<?php echo get_avatar( $donor->email, 30 ); ?> <span><?php echo $donor->name; ?></span>
+				<?php echo get_avatar($donor->email, 30); ?> <span><?php echo $donor->name; ?></span>
 			</div>
 
 
@@ -771,20 +771,20 @@ 
 
 				<span class="delete-donor-options">
 					<p>
-						<?php echo Give()->html->checkbox( array(
+						<?php echo Give()->html->checkbox(array(
 							'name' => 'give-donor-delete-confirm',
-						) ); ?>
-						<label for="give-donor-delete-confirm"><?php _e( 'Are you sure you want to delete this donor?', 'give' ); ?></label>
+						)); ?>
+						<label for="give-donor-delete-confirm"><?php _e('Are you sure you want to delete this donor?', 'give'); ?></label>
 					</p>
 
 					<p>
-						<?php echo Give()->html->checkbox( array(
+						<?php echo Give()->html->checkbox(array(
 							'name'    => 'give-donor-delete-records',
 							'options' => array(
 								'disabled' => true,
 							),
-						) ); ?>
-						<label for="give-donor-delete-records"><?php _e( 'Delete all associated donations and records?', 'give' ); ?></label>
+						)); ?>
+						<label for="give-donor-delete-records"><?php _e('Delete all associated donations and records?', 'give'); ?></label>
 					</p>
 
 					<?php
@@ -797,16 +797,16 @@ 
 					 *
 					 * @param object $donor The donor object being displayed.
 					 */
-					do_action( 'give_donor_delete_inputs', $donor );
+					do_action('give_donor_delete_inputs', $donor);
 					?>
 				</span>
 
 				<span id="donor-edit-actions">
 					<input type="hidden" name="customer_id" value="<?php echo $donor->id; ?>" />
-					<?php wp_nonce_field( 'delete-donor', '_wpnonce', false, true ); ?>
+					<?php wp_nonce_field('delete-donor', '_wpnonce', false, true); ?>
 					<input type="hidden" name="give_action" value="delete-donor" />
-					<input type="submit" disabled="disabled" id="give-delete-donor" class="button-primary" value="<?php _e( 'Delete Donor', 'give' ); ?>" />
-					<a id="give-delete-donor-cancel" href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ); ?>" class="delete"><?php _e( 'Cancel', 'give' ); ?></a>
+					<input type="submit" disabled="disabled" id="give-delete-donor" class="button-primary" value="<?php _e('Delete Donor', 'give'); ?>" />
+					<a id="give-delete-donor-cancel" href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id); ?>" class="delete"><?php _e('Cancel', 'give'); ?></a>
 				</span>
 
 			</div>
@@ -822,5 +822,5 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_delete_bottom', $donor );
+	do_action('give_donor_delete_bottom', $donor);
 }

includes/admin/plugins.php

Spacing +22 added lines, -23 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,19 +23,19 @@ 
  *
  * @return array An array of updated action links.
  */
-function give_plugin_action_links( $actions ) {
+function give_plugin_action_links($actions) {
 	$new_actions = array(
 		'settings' => sprintf(
 			'<a href="%1$s">%2$s</a>',
-			admin_url( 'edit.php?post_type=give_forms&page=give-settings' ),
-			__( 'Settings', 'give' )
+			admin_url('edit.php?post_type=give_forms&page=give-settings'),
+			__('Settings', 'give')
 		),
 	);
 
-	return array_merge( $new_actions, $actions );
+	return array_merge($new_actions, $actions);
 }
 
-add_filter( 'plugin_action_links_' . GIVE_PLUGIN_BASENAME, 'give_plugin_action_links' );
+add_filter('plugin_action_links_'.GIVE_PLUGIN_BASENAME, 'give_plugin_action_links');
 
 
 /**
@@ -48,38 +48,38 @@ 
  *
  * @return array
  */
-function give_plugin_row_meta( $plugin_meta, $plugin_file ) {
-	if ( $plugin_file != GIVE_PLUGIN_BASENAME ) {
+function give_plugin_row_meta($plugin_meta, $plugin_file) {
+	if ($plugin_file != GIVE_PLUGIN_BASENAME) {
 		return $plugin_meta;
 	}
 
 	$new_meta_links = array(
 		sprintf(
 			'<a href="%1$s" target="_blank">%2$s</a>',
-			esc_url( add_query_arg( array(
+			esc_url(add_query_arg(array(
 					'utm_source'   => 'plugins-page',
 					'utm_medium'   => 'plugin-row',
 					'utm_campaign' => 'admin',
-				), 'https://givewp.com/documentation/' )
+				), 'https://givewp.com/documentation/')
 			),
-			__( 'Documentation', 'give' )
+			__('Documentation', 'give')
 		),
 		sprintf(
 			'<a href="%1$s" target="_blank">%2$s</a>',
-			esc_url( add_query_arg( array(
+			esc_url(add_query_arg(array(
 					'utm_source'   => 'plugins-page',
 					'utm_medium'   => 'plugin-row',
 					'utm_campaign' => 'admin',
-				), 'https://givewp.com/addons/' )
+				), 'https://givewp.com/addons/')
 			),
-			__( 'Add-ons', 'give' )
+			__('Add-ons', 'give')
 		),
 	);
 
-	return array_merge( $plugin_meta, $new_meta_links );
+	return array_merge($plugin_meta, $new_meta_links);
 }
 
-add_filter( 'plugin_row_meta', 'give_plugin_row_meta', 10, 2 );
+add_filter('plugin_row_meta', 'give_plugin_row_meta', 10, 2);
 
 
 /**
@@ -97,19 +97,18 @@ 
 	$title = '';
 	global $submenu, $plugin_page;
 
-	foreach ( array_keys( $submenu ) as $parent ) {
-		if( 'edit.php?post_type=give_forms' !== $parent ) {
+	foreach (array_keys($submenu) as $parent) {
+		if ('edit.php?post_type=give_forms' !== $parent) {
 			continue;
 		}
 
-		foreach ( $submenu[ $parent ] as $submenu_array ) {
-			if( $plugin_page !== $submenu_array[2] ){
+		foreach ($submenu[$parent] as $submenu_array) {
+			if ($plugin_page !== $submenu_array[2]) {
 				continue;
 			}
 
-			$title = isset( $submenu_array[0] ) ?
-				$submenu_array[0] :
-				$submenu_array[3];
+			$title = isset($submenu_array[0]) ?
+				$submenu_array[0] : $submenu_array[3];
 		}
 	}