WordImpress / Give

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

Spacing +56 added lines, -56 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -53,8 +53,8 @@ 
 	/**
 	 * Constructor.
 	 */
-	public function __construct( $_step = 1 ) {
-		parent::__construct( $_step );
+	public function __construct($_step = 1) {
+		parent::__construct($_step);
 
 		$this->is_writable = true;
 	}
@@ -69,43 +69,43 @@ 
 	 */
 	public function get_data() {
 
-		$accepted_statuses = apply_filters( 'give_recount_accepted_statuses', array( 'publish' ) );
+		$accepted_statuses = apply_filters('give_recount_accepted_statuses', array('publish'));
 
-		if ( $this->step == 1 ) {
-			$this->delete_data( 'give_temp_recount_form_stats' );
+		if ($this->step == 1) {
+			$this->delete_data('give_temp_recount_form_stats');
 		}
 
-		$totals = $this->get_stored_data( 'give_temp_recount_form_stats' );
+		$totals = $this->get_stored_data('give_temp_recount_form_stats');
 
-		if ( false === $totals ) {
+		if (false === $totals) {
 			$totals = array(
 				'earnings' => (float) 0,
 				'sales'    => 0,
 			);
-			$this->store_data( 'give_temp_recount_form_stats', $totals );
+			$this->store_data('give_temp_recount_form_stats', $totals);
 		}
 
-		$args = apply_filters( 'give_recount_form_stats_args', array(
+		$args = apply_filters('give_recount_form_stats_args', array(
 			'give_forms' => $this->form_id,
 			'number'     => $this->per_step,
 			'status'     => $accepted_statuses,
 			'paged'      => $this->step,
 			'fields'     => 'ids',
-		) );
+		));
 
-		$payments = new Give_Payments_Query( $args );
+		$payments = new Give_Payments_Query($args);
 		$payments = $payments->get_payments();
 
-		if ( $payments ) {
-			foreach ( $payments as $payment ) {
+		if ($payments) {
+			foreach ($payments as $payment) {
 
 				// Ensure acceptable status only.
-				if ( ! in_array( $payment->post_status, $accepted_statuses ) ) {
+				if ( ! in_array($payment->post_status, $accepted_statuses)) {
 					continue;
 				}
 
 				// Ensure only payments for this form are counted.
-				if ( $payment->form_id != $this->form_id ) {
+				if ($payment->form_id != $this->form_id) {
 					continue;
 				}
 
@@ -116,25 +116,25 @@ 
 				 */
 				$earning_amount = apply_filters(
 					'give_donation_amount',
-					give_format_amount( $payment->total, array( 'donation_id' => $payment->ID ) ),
+					give_format_amount($payment->total, array('donation_id' => $payment->ID)),
 					$payment->total,
 					$payment->ID,
-					array( 'type' => 'stats', 'currency' => false, 'amount' => false )
+					array('type' => 'stats', 'currency' => false, 'amount' => false)
 				);
 
-				$totals['sales'] ++;
-				$totals['earnings'] += (float) give_maybe_sanitize_amount( $earning_amount );
+				$totals['sales']++;
+				$totals['earnings'] += (float) give_maybe_sanitize_amount($earning_amount);
 
 			}
 
-			$this->store_data( 'give_temp_recount_form_stats', $totals );
+			$this->store_data('give_temp_recount_form_stats', $totals);
 
 			return true;
 		}
 
 
-		give_update_meta( $this->form_id, '_give_form_sales', $totals['sales'] );
-		give_update_meta( $this->form_id, '_give_form_earnings', give_sanitize_amount_for_db( $totals['earnings'] ) );
+		give_update_meta($this->form_id, '_give_form_sales', $totals['sales']);
+		give_update_meta($this->form_id, '_give_form_earnings', give_sanitize_amount_for_db($totals['earnings']));
 
 		return false;
 	}
@@ -146,35 +146,35 @@ 
 	 * @return int
 	 */
 	public function get_percentage_complete() {
-		if ( $this->step == 1 ) {
-			$this->delete_data( 'give_recount_total_' . $this->form_id );
+		if ($this->step == 1) {
+			$this->delete_data('give_recount_total_'.$this->form_id);
 		}
 
-		$accepted_statuses = apply_filters( 'give_recount_accepted_statuses', array( 'publish' ) );
-		$total             = $this->get_stored_data( 'give_recount_total_' . $this->form_id );
+		$accepted_statuses = apply_filters('give_recount_accepted_statuses', array('publish'));
+		$total             = $this->get_stored_data('give_recount_total_'.$this->form_id);
 
-		if ( false === $total ) {
+		if (false === $total) {
 			$total = 0;
-			$args  = apply_filters( 'give_recount_form_stats_total_args', array(
+			$args  = apply_filters('give_recount_form_stats_total_args', array(
 				'give_forms' => $this->form_id,
-				'number'     => - 1,
+				'number'     => -1,
 				'status'     => $accepted_statuses,
 				'fields'     => 'ids',
-			) );
+			));
 
-			$payments = new Give_Payments_Query( $args );
-			$total    = count( $payments->get_payments() );
-			$this->store_data( 'give_recount_total_' . $this->form_id, $total );
+			$payments = new Give_Payments_Query($args);
+			$total    = count($payments->get_payments());
+			$this->store_data('give_recount_total_'.$this->form_id, $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;
 		}
 
@@ -188,8 +188,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;
 	}
 
 	/**
@@ -200,21 +200,21 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( esc_html__( 'You do not have permission to recount stats.', 'give' ), esc_html__( 'Error', 'give' ), array( '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_temp_recount_form_stats' );
+			$this->delete_data('give_recount_total_'.$this->form_id);
+			$this->delete_data('give_temp_recount_form_stats');
 			$this->done    = true;
-			$this->message = sprintf( esc_html__( 'Donation counts and income amount statistics successfully recounted for "%s".', 'give' ), get_the_title( $this->form_id ) );
+			$this->message = sprintf(esc_html__('Donation counts and income amount statistics successfully recounted for "%s".', 'give'), get_the_title($this->form_id));
 
 			return false;
 		}
@@ -248,17 +248,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;
@@ -274,10 +274,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,
@@ -291,7 +291,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -303,9 +303,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));
 	}
 
 }
\ No newline at end of file

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

Spacing +100 added lines, -102 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -73,8 +73,8 @@ 
 	/**
 	 * Constructor.
 	 */
-	public function __construct( $_step = 1 ) {
-		parent::__construct( $_step );
+	public function __construct($_step = 1) {
+		parent::__construct($_step);
 
 		$this->is_writable = true;
 	}
@@ -89,108 +89,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;
@@ -205,20 +203,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;
 		}
 
@@ -232,8 +230,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;
 	}
 
 	/**
@@ -244,25 +242,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;
 		}
@@ -298,69 +296,69 @@ 
 	 */
 	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 ) ) {
+					if ( ! array_key_exists($payment->ID, $payment_items)) {
 
 						/**
 						 * Filter the payment amount.
@@ -369,22 +367,22 @@ 
 						 */
 						$payment_total = apply_filters(
 							'give_donation_amount',
-							give_format_amount( $payment->total, array( 'donation_id' => $payment->ID ) ),
+							give_format_amount($payment->total, array('donation_id' => $payment->ID)),
 							$payment->total,
 							$payment->ID,
-							array( 'type' => 'stats', 'currency' => false, 'amount' => false )
+							array('type' => 'stats', 'currency' => false, 'amount' => false)
 						);
 
-						$payment_items[ $payment->ID ] = array(
+						$payment_items[$payment->ID] = array(
 							'id'         => $form_id,
 							'payment_id' => $payment->ID,
-							'price'      => (float) give_maybe_sanitize_amount( $payment_total ),
+							'price'      => (float) give_maybe_sanitize_amount($payment_total),
 						);
 					}
 				}
 			}
-			$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);
 		}
 
 	}
@@ -398,17 +396,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;
@@ -424,10 +422,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,
@@ -441,7 +439,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -453,9 +451,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/class-settings-export.php

Spacing +12 added lines, -12 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_Export' ) ) :
+if ( ! class_exists('Give_Settings_Export')) :
 
 	/**
 	 * Give_Settings_Export.
@@ -34,19 +34,19 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'export';
-			$this->label = __( 'Export', 'give' );
+			$this->label = __('Export', 'give');
 
 			parent::__construct();
 
-			add_action( 'give_admin_field_tools_export', array( 'Give_Settings_Export', 'render_export_field' ), 10, 2 );
+			add_action('give_admin_field_tools_export', array('Give_Settings_Export', 'render_export_field'), 10, 2);
 
 			// Do not use main donor for this tab.
-			if( give_get_current_setting_tab() === $this->id ) {
-				add_action( 'give-tools_open_form', '__return_empty_string' );
-				add_action( 'give-tools_close_form', '__return_empty_string' );
+			if (give_get_current_setting_tab() === $this->id) {
+				add_action('give-tools_open_form', '__return_empty_string');
+				add_action('give-tools_close_form', '__return_empty_string');
 
 
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/class-give-export-donations.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/tools/export/class-give-export-donations.php';
 			}
 		}
 
@@ -64,7 +64,7 @@ 
 			 * @param  array $settings
 			 */
 			$settings = apply_filters(
-				'give_get_settings_' . $this->id,
+				'give_get_settings_'.$this->id,
 				array(
 					array(
 						'id'   => 'give_tools_export',
@@ -73,7 +73,7 @@ 
 					),
 					array(
 						'id'   => 'export',
-						'name' => __( 'Export', 'give' ),
+						'name' => __('Export', 'give'),
 						'type' => 'tools_export',
 					),
 					array(
@@ -97,8 +97,8 @@ 
 		 * @param $field
 		 * @param $option_value
 		 */
-		public static function render_export_field( $field, $option_value ) {
-			include_once( 'views/html-admin-page-exports.php' );
+		public static function render_export_field($field, $option_value) {
+			include_once('views/html-admin-page-exports.php');
 		}
 	}
 

includes/admin/tools/export/export-actions.php

Spacing +24 added lines, -24 removed lines

@@ -10,7 +10,7 @@ 
  * @license     https://opensource.org/licenses/gpl-license GNU Public License
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -22,13 +22,13 @@ 
  */
 function give_process_batch_export_form() {
 
-	if ( ! wp_verify_nonce( $_REQUEST['nonce'], 'give-batch-export' ) ) {
-		wp_die( esc_html__( 'Nonce verification failed.', 'give' ), esc_html__( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($_REQUEST['nonce'], 'give-batch-export')) {
+		wp_die(esc_html__('Nonce verification failed.', 'give'), esc_html__('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/class-batch-export.php';
+	require_once GIVE_PLUGIN_DIR.'includes/admin/tools/export/class-batch-export.php';
 
 	/**
 	 * Fires before batch export.
@@ -37,14 +37,14 @@ 
 	 *
 	 * @param string $class Export class.
 	 */
-	do_action( 'give_batch_export_class_include', $_REQUEST['class'] );
+	do_action('give_batch_export_class_include', $_REQUEST['class']);
 
 	$export = new $_REQUEST['class'];
 	$export->export();
 
 }
 
-add_action( 'give_form_batch_export', 'give_process_batch_export_form' );
+add_action('give_form_batch_export', 'give_process_batch_export_form');
 
 /**
  * Exports earnings for a specified time period.
@@ -55,14 +55,14 @@ 
  * @return void
  */
 function give_export_earnings() {
-	require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/class-export-earnings.php';
+	require_once GIVE_PLUGIN_DIR.'includes/admin/tools/export/class-export-earnings.php';
 
 	$earnings_export = new Give_Earnings_Export();
 
 	$earnings_export->export();
 }
 
-add_action( 'give_earnings_export', 'give_export_earnings' );
+add_action('give_earnings_export', 'give_export_earnings');
 
 /**
  * Exports Give's core settings.
@@ -73,14 +73,14 @@ 
  * @return void
  */
 function give_core_settings_export() {
-	require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/class-core-settings-export.php';
+	require_once GIVE_PLUGIN_DIR.'includes/admin/tools/export/class-core-settings-export.php';
 
 	$core_settings = new Give_Core_Settings_Export();
 
 	$core_settings->export();
 }
 
-add_action( 'give_core_settings_export', 'give_core_settings_export' );
+add_action('give_core_settings_export', 'give_core_settings_export');
 
 
 /**
@@ -90,7 +90,7 @@ 
  * @return void
  */
 function give_register_batch_exporters() {
-	if ( is_admin() ) {
+	if (is_admin()) {
 		/**
 		 * Fires in the admin, while plugins loaded.
 		 *
@@ -100,11 +100,11 @@ 
 		 *
 		 * @param string $class Export class.
 		 */
-		do_action( 'give_register_batch_exporter' );
+		do_action('give_register_batch_exporter');
 	}
 }
 
-add_action( 'plugins_loaded', 'give_register_batch_exporters' );
+add_action('plugins_loaded', 'give_register_batch_exporters');
 
 /**
  * Register the donors batch exporter.
@@ -112,10 +112,10 @@ 
  * @since  1.5.2
  */
 function give_register_donors_batch_export() {
-	add_action( 'give_batch_export_class_include', 'give_include_donors_batch_processor', 10, 1 );
+	add_action('give_batch_export_class_include', 'give_include_donors_batch_processor', 10, 1);
 }
 
-add_action( 'give_register_batch_exporter', 'give_register_donors_batch_export', 10 );
+add_action('give_register_batch_exporter', 'give_register_donors_batch_export', 10);
 
 /**
  * Loads the donors batch process if needed.
@@ -126,10 +126,10 @@ 
  *
  * @return void
  */
-function give_include_donors_batch_processor( $class ) {
+function give_include_donors_batch_processor($class) {
 
-	if ( 'Give_Batch_Donors_Export' === $class ) {
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/class-batch-export-donors.php';
+	if ('Give_Batch_Donors_Export' === $class) {
+		require_once GIVE_PLUGIN_DIR.'includes/admin/tools/export/class-batch-export-donors.php';
 	}
 
 }
@@ -140,10 +140,10 @@ 
  * @since  1.5
  */
 function give_register_forms_batch_export() {
-	add_action( 'give_batch_export_class_include', 'give_include_forms_batch_processor', 10, 1 );
+	add_action('give_batch_export_class_include', 'give_include_forms_batch_processor', 10, 1);
 }
 
-add_action( 'give_register_batch_exporter', 'give_register_forms_batch_export', 10 );
+add_action('give_register_batch_exporter', 'give_register_forms_batch_export', 10);
 
 /**
  * Loads the file downloads batch process if needed
@@ -154,10 +154,10 @@ 
  *
  * @return void
  */
-function give_include_forms_batch_processor( $class ) {
+function give_include_forms_batch_processor($class) {
 
-	if ( 'Give_Batch_Forms_Export' === $class ) {
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/class-batch-export-forms.php';
+	if ('Give_Batch_Forms_Export' === $class) {
+		require_once GIVE_PLUGIN_DIR.'includes/admin/tools/export/class-batch-export-forms.php';
 	}
 
 }

includes/admin/tools/export/class-give-export-donations.php

Spacing +47 added lines, -47 removed lines

@@ -9,11 +9,11 @@ 
  * @since       2.1
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Export_Donations' ) ) {
+if ( ! class_exists('Give_Export_Donations')) {
 
 	/**
 	 * Give_Settings_CSV.
@@ -57,7 +57,7 @@ 
 		 * @return static
 		 */
 		public static function get_instance() {
-			if ( null === static::$instance ) {
+			if (null === static::$instance) {
 				self::$instance = new static();
 			}
 
@@ -84,21 +84,21 @@ 
 		 * @return void
 		 */
 		private function setup_hooks() {
-			if ( ! $this->is_donations_export_page() ) {
+			if ( ! $this->is_donations_export_page()) {
 				return;
 			}
 
 			// Do not render main export tools page.
-			remove_action( 'give_admin_field_tools_export', array(
+			remove_action('give_admin_field_tools_export', array(
 				'Give_Settings_Export',
 				'render_export_field'
-			), 10 );
+			), 10);
 
 			// Render donation export page
-			add_action( 'give_admin_field_tools_export', array( $this, 'render_page' ) );
+			add_action('give_admin_field_tools_export', array($this, 'render_page'));
 
 			// Print the HTML.
-			add_action( 'give_tools_export_donations_form_start', array( $this, 'html' ) );
+			add_action('give_tools_export_donations_form_start', array($this, 'html'));
 		}
 
 		/**
@@ -110,7 +110,7 @@ 
 		 *
 		 * @return array args for Taxonomy
 		 */
-		function give_forms_taxonomy_dropdown( $args ) {
+		function give_forms_taxonomy_dropdown($args) {
 			$args['number'] = 30;
 
 			return $args;
@@ -128,19 +128,19 @@ 
 					<tbody>
 					<tr class="top">
 						<td colspan="2">
-							<h2 id="give-export-title"><?php _e( 'Export Donation History and Custom Fields to CSV', 'give' ) ?></h2>
-							<p class="give-field-description"><?php _e( 'Download an export of donors for specific donation forms with the option to include custom fields.', 'give' ) ?></p>
+							<h2 id="give-export-title"><?php _e('Export Donation History and Custom Fields to CSV', 'give') ?></h2>
+							<p class="give-field-description"><?php _e('Download an export of donors for specific donation forms with the option to include custom fields.', 'give') ?></p>
 						</td>
 					</tr>
 
 					<?php
-					if ( give_is_setting_enabled( give_get_option( 'categories' ) ) ) {
-						add_filter( 'give_forms_category_dropdown', array( $this, 'give_forms_taxonomy_dropdown' ) );
+					if (give_is_setting_enabled(give_get_option('categories'))) {
+						add_filter('give_forms_category_dropdown', array($this, 'give_forms_taxonomy_dropdown'));
 						?>
 						<tr>
 							<td scope="row" class="row-title">
 								<label
-									for="give_forms_categories"><?php _e( 'Filter by Categories:', 'give' ); ?></label>
+									for="give_forms_categories"><?php _e('Filter by Categories:', 'give'); ?></label>
 							</td>
 							<td class="give-field-wrap">
 								<div class="give-clearfix">
@@ -155,8 +155,8 @@ 
 											'multiple'        => true,
 											'selected'        => array(),
 											'show_option_all' => false,
-											'placeholder'     => __( 'Choose one or more from categories', 'give' ),
-											'data'            => array( 'search-type' => 'categories' ),
+											'placeholder'     => __('Choose one or more from categories', 'give'),
+											'data'            => array('search-type' => 'categories'),
 										)
 									);
 									?>
@@ -164,16 +164,16 @@ 
 							</td>
 						</tr>
 						<?php
-						remove_filter( 'give_forms_category_dropdown', array( $this, 'give_forms_taxonomy_dropdown' ) );
+						remove_filter('give_forms_category_dropdown', array($this, 'give_forms_taxonomy_dropdown'));
 					}
 
-					if ( give_is_setting_enabled( give_get_option( 'tags' ) ) ) {
-						add_filter( 'give_forms_tag_dropdown', array( $this, 'give_forms_taxonomy_dropdown' ) );
+					if (give_is_setting_enabled(give_get_option('tags'))) {
+						add_filter('give_forms_tag_dropdown', array($this, 'give_forms_taxonomy_dropdown'));
 						?>
 						<tr>
 							<td scope="row" class="row-title">
 								<label
-									for="give_forms_tags"><?php _e( 'Filter by Tags:', 'give' ); ?></label>
+									for="give_forms_tags"><?php _e('Filter by Tags:', 'give'); ?></label>
 							</td>
 							<td class="give-field-wrap">
 								<div class="give-clearfix">
@@ -188,8 +188,8 @@ 
 											'multiple'        => true,
 											'selected'        => array(),
 											'show_option_all' => false,
-											'placeholder'     => __( 'Choose one or more from tags', 'give' ),
-											'data'            => array( 'search-type' => 'tags' ),
+											'placeholder'     => __('Choose one or more from tags', 'give'),
+											'data'            => array('search-type' => 'tags'),
 										)
 									);
 									?>
@@ -197,14 +197,14 @@ 
 							</td>
 						</tr>
 						<?php
-						remove_filter( 'give_forms_tag_dropdown', array( $this, 'give_forms_taxonomy_dropdown' ) );
+						remove_filter('give_forms_tag_dropdown', array($this, 'give_forms_taxonomy_dropdown'));
 					}
 					?>
 
 					<tr class="give-export-donation-form">
 						<td scope="row" class="row-title">
 							<label
-								for="give_payment_form_select"><?php _e( 'Filter by Donation Form:', 'give' ); ?></label>
+								for="give_payment_form_select"><?php _e('Filter by Donation Form:', 'give'); ?></label>
 						</td>
 						<td class="give-field-wrap">
 							<div class="give-clearfix">
@@ -213,10 +213,10 @@ 
 									'name'        => 'forms',
 									'id'          => 'give-payment-form-select',
 									'chosen'      => true,
-									'placeholder' => __( 'All Forms', 'give' ),
-									'data'            => array( 'no-form' => __( 'No donation forms found', 'give' ), ),
+									'placeholder' => __('All Forms', 'give'),
+									'data'            => array('no-form' => __('No donation forms found', 'give'),),
 								);
-								echo Give()->html->forms_dropdown( $args );
+								echo Give()->html->forms_dropdown($args);
 								?>
 
 								<input type="hidden" name="form_ids" class="form_ids" />
@@ -226,7 +226,7 @@ 
 
 					<tr>
 						<td scope="row" class="row-title">
-							<label for="give-payment-export-start"><?php _e( 'Filter by Date:', 'give' ); ?></label>
+							<label for="give-payment-export-start"><?php _e('Filter by Date:', 'give'); ?></label>
 						</td>
 						<td class="give-field-wrap">
 							<div class="give-clearfix">
@@ -234,16 +234,16 @@ 
 								$args = array(
 									'id'          => 'give-payment-export-start',
 									'name'        => 'start',
-									'placeholder' => __( 'Start date', 'give' ),
+									'placeholder' => __('Start date', 'give'),
 								);
-								echo Give()->html->date_field( $args ); ?>
+								echo Give()->html->date_field($args); ?>
 								<?php
 								$args = array(
 									'id'          => 'give-payment-export-end',
 									'name'        => 'end',
-									'placeholder' => __( 'End date', 'give' ),
+									'placeholder' => __('End date', 'give'),
 								);
-								echo Give()->html->date_field( $args ); ?>
+								echo Give()->html->date_field($args); ?>
 							</div>
 						</td>
 					</tr>
@@ -251,16 +251,16 @@ 
 					<tr>
 						<td scope="row" class="row-title">
 							<label
-								for="give-export-donations-status"><?php _e( 'Filter by Status:', 'give' ); ?></label>
+								for="give-export-donations-status"><?php _e('Filter by Status:', 'give'); ?></label>
 						</td>
 						<td>
 							<div class="give-clearfix">
 								<select name="status" id="give-export-donations-status">
-									<option value="any"><?php _e( 'All Statuses', 'give' ); ?></option>
+									<option value="any"><?php _e('All Statuses', 'give'); ?></option>
 									<?php
 									$statuses = give_get_payment_statuses();
-									foreach ( $statuses as $status => $label ) {
-										echo '<option value="' . $status . '">' . $label . '</option>';
+									foreach ($statuses as $status => $label) {
+										echo '<option value="'.$status.'">'.$label.'</option>';
 									}
 									?>
 								</select>
@@ -274,16 +274,16 @@ 
 					 *
 					 * @since 2.1
 					 */
-					do_action( 'give_export_donation_fields' );
+					do_action('give_export_donation_fields');
 					?>
 
 					<tr class="end">
 						<td>
 						</td>
 						<td>
-							<?php wp_nonce_field( 'give_ajax_export', 'give_ajax_export' ); ?>
+							<?php wp_nonce_field('give_ajax_export', 'give_ajax_export'); ?>
 							<input type="hidden" name="give-export-class" value="Give_Export_Donations_CSV"/>
-							<input type="submit" value="<?php _e( 'Generate CSV', 'give' ) ?>" class="give-export-donation-button button button-primary">
+							<input type="submit" value="<?php _e('Generate CSV', 'give') ?>" class="give-export-donation-button button button-primary">
 							<div class="add-notices"></div>
 						</td>
 					</tr>
@@ -304,11 +304,11 @@ 
 			 *
 			 * @since 2.1
 			 */
-			do_action( 'give_tools_export_donations_main_before' );
+			do_action('give_tools_export_donations_main_before');
 			?>
 			<div id="poststuff">
 				<div class="postbox">
-					<h1 class="give-export-h1" align="center"><?php _e( 'Export Donations', 'give' ); ?></h1>
+					<h1 class="give-export-h1" align="center"><?php _e('Export Donations', 'give'); ?></h1>
 					<div class="inside give-tools-setting-page-export give-export_donations">
 						<?php
 						/**
@@ -316,7 +316,7 @@ 
 						 *
 						 * @since 2.1
 						 */
-						do_action( 'give_tools_export_donations_form_before_start' );
+						do_action('give_tools_export_donations_form_before_start');
 						?>
 						<form method="post" id="give-export_donations-form"
 						      class="give-export-form tools-setting-page-export tools-setting-page-export"
@@ -328,7 +328,7 @@ 
 							 *
 							 * @since 2.1
 							 */
-							do_action( 'give_tools_export_donations_form_start' );
+							do_action('give_tools_export_donations_form_start');
 							?>
 
 							<?php
@@ -337,7 +337,7 @@ 
 							 *
 							 * @since 2.1
 							 */
-							do_action( 'give_tools_export_donations_form_end' );
+							do_action('give_tools_export_donations_form_end');
 							?>
 						</form>
 						<?php
@@ -346,7 +346,7 @@ 
 						 *
 						 * @since 2.1
 						 */
-						do_action( 'give_tools_iexport_donations_form_after_end' );
+						do_action('give_tools_iexport_donations_form_after_end');
 						?>
 					</div><!-- .inside -->
 				</div><!-- .postbox -->
@@ -357,7 +357,7 @@ 
 			 *
 			 * @since 2.1
 			 */
-			do_action( 'give_tools_export_donations_main_after' );
+			do_action('give_tools_export_donations_main_after');
 		}
 
 		/**
@@ -368,7 +368,7 @@ 
 		 * @return bool
 		 */
 		private function is_donations_export_page() {
-			return 'export' === give_get_current_setting_tab() && isset( $_GET['type'] ) && $this->exporter_type === give_clean( $_GET['type'] );
+			return 'export' === give_get_current_setting_tab() && isset($_GET['type']) && $this->exporter_type === give_clean($_GET['type']);
 		}
 	}
 

includes/admin/tools/export/class-export.php

Spacing +25 added lines, -25 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -37,7 +37,7 @@ 
 	 * @return bool Whether we can export or not
 	 */
 	public function can_export() {
-		return (bool) apply_filters( 'give_export_capability', current_user_can( 'export_give_reports' ) );
+		return (bool) apply_filters('give_export_capability', current_user_can('export_give_reports'));
 	}
 
 	/**
@@ -60,12 +60,12 @@ 
 		 *
 		 * @return $file_name string file name
 		 */
-		$file_name = apply_filters( 'give_export_filename', 'give-export-' . $this->export_type . '-' . date( 'm-d-Y' ), $this->export_type );
+		$file_name = apply_filters('give_export_filename', 'give-export-'.$this->export_type.'-'.date('m-d-Y'), $this->export_type);
 
 		nocache_headers();
-		header( 'Content-Type: text/csv; charset=utf-8' );
-		header( 'Content-Disposition: attachment; filename=' . $file_name . '.csv' );
-		header( "Expires: 0" );
+		header('Content-Type: text/csv; charset=utf-8');
+		header('Content-Disposition: attachment; filename='.$file_name.'.csv');
+		header("Expires: 0");
 	}
 
 	/**
@@ -77,8 +77,8 @@ 
 	 */
 	public function csv_cols() {
 		$cols = array(
-			'id'   => __( 'ID', 'give' ),
-			'date' => __( 'Date', 'give' )
+			'id'   => __('ID', 'give'),
+			'date' => __('Date', 'give')
 		);
 
 		return $cols;
@@ -94,7 +94,7 @@ 
 	public function get_csv_cols() {
 		$cols = $this->csv_cols();
 
-		return apply_filters( "give_export_csv_cols_{$this->export_type}", $cols );
+		return apply_filters("give_export_csv_cols_{$this->export_type}", $cols);
 	}
 
 	/**
@@ -108,10 +108,10 @@ 
 	public function csv_cols_out() {
 		$cols = $this->get_csv_cols();
 		$i    = 1;
-		foreach ( $cols as $col_id => $column ) {
-			echo '"' . addslashes( $column ) . '"';
-			echo $i == count( $cols ) ? '' : ',';
-			$i ++;
+		foreach ($cols as $col_id => $column) {
+			echo '"'.addslashes($column).'"';
+			echo $i == count($cols) ? '' : ',';
+			$i++;
 		}
 		echo "\r\n";
 	}
@@ -128,16 +128,16 @@ 
 		$data = array(
 			0 => array(
 				'id'   => '',
-				'data' => date( 'F j, Y' )
+				'data' => date('F j, Y')
 			),
 			1 => array(
 				'id'   => '',
-				'data' => date( 'F j, Y' )
+				'data' => date('F j, Y')
 			)
 		);
 
-		$data = apply_filters( 'give_export_get_data', $data );
-		$data = apply_filters( "give_export_get_data_{$this->export_type}", $data );
+		$data = apply_filters('give_export_get_data', $data);
+		$data = apply_filters("give_export_get_data_{$this->export_type}", $data);
 
 		return $data;
 	}
@@ -155,14 +155,14 @@ 
 		$cols = $this->get_csv_cols();
 
 		// Output each row
-		foreach ( $data as $row ) {
+		foreach ($data as $row) {
 			$i = 1;
-			foreach ( $row as $col_id => $column ) {
+			foreach ($row as $col_id => $column) {
 				// Make sure the column is valid
-				if ( array_key_exists( $col_id, $cols ) ) {
-					echo '"' . addslashes( $column ) . '"';
-					echo $i == count( $cols ) ? '' : ',';
-					$i ++;
+				if (array_key_exists($col_id, $cols)) {
+					echo '"'.addslashes($column).'"';
+					echo $i == count($cols) ? '' : ',';
+					$i++;
 				}
 			}
 			echo "\r\n";
@@ -181,8 +181,8 @@ 
 	 * @return void
 	 */
 	public function export() {
-		if ( ! $this->can_export() ) {
-			wp_die( __( 'You do not have permission to export data.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(__('You do not have permission to export data.', 'give'), __('Error', 'give'), array('response' => 403));
 		}
 
 		// Set headers

includes/admin/tools/export/class-core-settings-export.php

Spacing +15 added lines, -15 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -42,9 +42,9 @@ 
 		give_ignore_user_abort();
 
 		nocache_headers();
-		header( 'Content-Type: application/json; charset=utf-8' );
-		header( 'Content-Disposition: attachment; filename=' . apply_filters( 'give_core_settings_export_filename', 'give-export-' . $this->export_type . '-' . date( 'n' ) . '-' . date( 'Y' ) ) . '.json' );
-		header( 'Expires: 0' );
+		header('Content-Type: application/json; charset=utf-8');
+		header('Content-Disposition: attachment; filename='.apply_filters('give_core_settings_export_filename', 'give-export-'.$this->export_type.'-'.date('n').'-'.date('Y')).'.json');
+		header('Expires: 0');
 	}
 
 	/**
@@ -54,18 +54,18 @@ 
 	 * @since 1.8.17
 	 */
 	public function json_core_settings_export() {
-		$settings_excludes = isset( $_POST['settings_export_excludes'] ) ? give_clean( $_POST['settings_export_excludes'] ) : array();
-		$give_settings     = get_option( 'give_settings' );
+		$settings_excludes = isset($_POST['settings_export_excludes']) ? give_clean($_POST['settings_export_excludes']) : array();
+		$give_settings     = get_option('give_settings');
 
-		if ( is_array( $settings_excludes ) && ! empty( $settings_excludes ) ) {
-			foreach ( $settings_excludes as $key => $value ) {
-				if ( give_is_setting_enabled( $value ) ) {
-					unset( $give_settings[ $key ] );
+		if (is_array($settings_excludes) && ! empty($settings_excludes)) {
+			foreach ($settings_excludes as $key => $value) {
+				if (give_is_setting_enabled($value)) {
+					unset($give_settings[$key]);
 				}
 			}
 		}
 
-		echo wp_json_encode( $give_settings );
+		echo wp_json_encode($give_settings);
 	}
 
 	/**
@@ -75,11 +75,11 @@ 
 	 * @since  1.8.17
 	 */
 	public function export() {
-		if ( ! $this->can_export() ) {
+		if ( ! $this->can_export()) {
 			wp_die(
-				esc_html__( 'You do not have permission to export data.', 'give' ),
-				esc_html__( 'Error', 'give' ),
-				array( 'response' => 403 )
+				esc_html__('You do not have permission to export data.', 'give'),
+				esc_html__('Error', 'give'),
+				array('response' => 403)
 			);
 		}
 

includes/admin/tools/views/html-admin-page-exports.php

Spacing +50 added lines, -50 removed lines

@@ -3,7 +3,7 @@ 
  * Admin View: Exports
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 } ?>
 
@@ -18,14 +18,14 @@ 
 				 *
 				 * @since 1.0
 				 */
-				do_action( 'give_tools_tab_export_content_top' );
+				do_action('give_tools_tab_export_content_top');
 				?>
 
 				<table class="widefat export-options-table give-table striped">
 					<thead>
 					<tr>
-						<th scope="col"><?php _e( 'Export Type', 'give' ); ?></th>
-						<th scope="col"><?php _e( 'Export Options', 'give' ); ?></th>
+						<th scope="col"><?php _e('Export Type', 'give'); ?></th>
+						<th scope="col"><?php _e('Export Options', 'give'); ?></th>
 					</tr>
 					</thead>
 					<tbody>
@@ -38,20 +38,20 @@ 
 					 *
 					 * @since 1.0
 					 */
-					do_action( 'give_tools_tab_export_table_top' );
+					do_action('give_tools_tab_export_table_top');
 					?>
 
 					<tr class="give-export-donations-history">
 						<td scope="row" class="row-title">
 							<h3>
-								<span><?php _e( 'Export Donation History', 'give' ); ?></span>
+								<span><?php _e('Export Donation History', 'give'); ?></span>
 							</h3>
-							<p><?php _e( 'Download a CSV of all donations recorded.', 'give' ); ?></p>
+							<p><?php _e('Download a CSV of all donations recorded.', 'give'); ?></p>
 						</td>
 						<td>
 							<a class="button"
-							   href="<?php echo add_query_arg( array( 'type' => 'export_donations' ) ); ?>">
-								<?php _e( 'Generate CSV', 'give' ); ?>
+							   href="<?php echo add_query_arg(array('type' => 'export_donations')); ?>">
+								<?php _e('Generate CSV', 'give'); ?>
 							</a>
 						</td>
 					</tr>
@@ -59,38 +59,38 @@ 
 					<tr class="give-export-pdf-sales-earnings">
 						<td scope="row" class="row-title">
 							<h3>
-								<span><?php _e( 'Export PDF of Donations and Income', 'give' ); ?></span>
+								<span><?php _e('Export PDF of Donations and Income', 'give'); ?></span>
 							</h3>
-							<p><?php _e( 'Download a PDF of Donations and Income reports for all forms for the current year.', 'give' ); ?></p>
+							<p><?php _e('Download a PDF of Donations and Income reports for all forms for the current year.', 'give'); ?></p>
 						</td>
 						<td>
 							<a class="button"
-							   href="<?php echo wp_nonce_url( add_query_arg( array( 'give-action' => 'generate_pdf' ) ), 'give_generate_pdf' ); ?>">
-								<?php _e( 'Generate PDF', 'give' ); ?>
+							   href="<?php echo wp_nonce_url(add_query_arg(array('give-action' => 'generate_pdf')), 'give_generate_pdf'); ?>">
+								<?php _e('Generate PDF', 'give'); ?>
 							</a>
 						</td>
 					</tr>
 					<tr class="give-export-sales-earnings">
 						<td scope="row" class="row-title">
 							<h3>
-								<span><?php _e( 'Export Income and Donation Stats', 'give' ); ?></span>
+								<span><?php _e('Export Income and Donation Stats', 'give'); ?></span>
 							</h3>
-							<p><?php _e( 'Download a CSV of income and donations over time.', 'give' ); ?></p>
+							<p><?php _e('Download a CSV of income and donations over time.', 'give'); ?></p>
 						</td>
 						<td>
 							<form method="post">
 								<?php
 								printf(
 								/* translators: 1: start date dropdown 2: end date dropdown */
-									esc_html__( '%1$s to %2$s', 'give' ),
-									Give()->html->year_dropdown( 'start_year' ) . ' ' . Give()->html->month_dropdown( 'start_month' ),
-									Give()->html->year_dropdown( 'end_year' ) . ' ' . Give()->html->month_dropdown( 'end_month' )
+									esc_html__('%1$s to %2$s', 'give'),
+									Give()->html->year_dropdown('start_year').' '.Give()->html->month_dropdown('start_month'),
+									Give()->html->year_dropdown('end_year').' '.Give()->html->month_dropdown('end_month')
 								);
 								?>
 								<input type="hidden" name="give-action"
 								       value="earnings_export"/>
 								<input type="submit"
-								       value="<?php esc_attr_e( 'Generate CSV', 'give' ); ?>"
+								       value="<?php esc_attr_e('Generate CSV', 'give'); ?>"
 								       class="button-secondary"/>
 							</form>
 						</td>
@@ -99,96 +99,96 @@ 
 					<tr class="give-export-donors">
 						<td scope="row" class="row-title">
 							<h3>
-								<span><?php _e( 'Export Donors', 'give' ); ?></span>
+								<span><?php _e('Export Donors', 'give'); ?></span>
 							</h3>
-							<p><?php _e( 'Download a CSV of donors. Column values reflect totals across all donation forms by default, or a single donation form if selected.', 'give' ); ?></p>
+							<p><?php _e('Download a CSV of donors. Column values reflect totals across all donation forms by default, or a single donation form if selected.', 'give'); ?></p>
 						</td>
 						<td>
 							<form method="post" id="give_donor_export" class="give-export-form">
 								<?php
 								// Start Date form field for donors
-								echo Give()->html->date_field( array(
+								echo Give()->html->date_field(array(
 									'id'          => 'give_donor_export_start_date',
 									'name'        => 'donor_export_start_date',
-									'placeholder' => esc_attr__( 'Start date', 'give' ),
-								) );
+									'placeholder' => esc_attr__('Start date', 'give'),
+								));
 
 								// End Date form field for donors
-								echo Give()->html->date_field( array(
+								echo Give()->html->date_field(array(
 									'id'          => 'give_donor_export_end_date',
 									'name'        => 'donor_export_end_date',
-									'placeholder' => esc_attr__( 'End date', 'give' ),
-								) );
+									'placeholder' => esc_attr__('End date', 'give'),
+								));
 
 								// Donation forms dropdown for donors export
-								echo Give()->html->forms_dropdown( array(
+								echo Give()->html->forms_dropdown(array(
 									'name'   => 'forms',
 									'id'     => 'give_donor_export_form',
 									'chosen' => true,
-								) );
+								));
 								?>
-								<input type="submit" value="<?php esc_attr_e( 'Generate CSV', 'give' ); ?>"
+								<input type="submit" value="<?php esc_attr_e('Generate CSV', 'give'); ?>"
 								       class="button-secondary"/>
 
 								<div id="export-donor-options-wrap"
 								     class="give-clearfix">
-									<p><?php _e( 'Export Columns:', 'give' ); ?></p>
+									<p><?php _e('Export Columns:', 'give'); ?></p>
 									<ul id="give-export-option-ul">
 										<li>
 											<label for="give-export-fullname">
 												<input type="checkbox" checked
 												       name="give_export_option[full_name]"
-												       id="give-export-fullname"><?php _e( 'Name', 'give' ); ?>
+												       id="give-export-fullname"><?php _e('Name', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-email">
 												<input type="checkbox" checked
 												       name="give_export_option[email]"
-												       id="give-export-email"><?php _e( 'Email', 'give' ); ?>
+												       id="give-export-email"><?php _e('Email', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-address">
 												<input type="checkbox" checked
 												       name="give_export_option[address]"
-												       id="give-export-address"><?php _e( 'Address', 'give' ); ?>
+												       id="give-export-address"><?php _e('Address', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-userid">
 												<input type="checkbox" checked
 												       name="give_export_option[userid]"
-												       id="give-export-userid"><?php _e( 'User ID', 'give' ); ?>
+												       id="give-export-userid"><?php _e('User ID', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-first-donation-date">
 												<input type="checkbox" checked
 												       name="give_export_option[donor_created_date]"
-												       id="give-export-first-donation-date"><?php _e( 'Donor Created Date', 'give' ); ?>
+												       id="give-export-first-donation-date"><?php _e('Donor Created Date', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-donation-number">
 												<input type="checkbox" checked
 												       name="give_export_option[donations]"
-												       id="give-export-donation-number"><?php _e( 'Number of Donations', 'give' ); ?>
+												       id="give-export-donation-number"><?php _e('Number of Donations', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-donation-sum">
 												<input type="checkbox" checked
 												       name="give_export_option[donation_sum]"
-												       id="give-export-donation-sum"><?php _e( 'Total Donated', 'give' ); ?>
+												       id="give-export-donation-sum"><?php _e('Total Donated', 'give'); ?>
 											</label>
 										</li>
 									</ul>
 								</div>
-								<?php wp_nonce_field( 'give_ajax_export', 'give_ajax_export' ); ?>
+								<?php wp_nonce_field('give_ajax_export', 'give_ajax_export'); ?>
 								<input type="hidden" name="give-export-class" value="Give_Batch_Donors_Export"/>
 								<input type="hidden" name="give_export_option[query_id]"
-								       value="<?php echo uniqid( 'give_' ); ?>"/>
+								       value="<?php echo uniqid('give_'); ?>"/>
 							</form>
 						</td>
 					</tr>
@@ -196,32 +196,32 @@ 
 					<tr class="give-export-core-settings">
 						<td scope="row" class="row-title">
 							<h3>
-								<span><?php _e( 'Export Give Settings', 'give' ); ?></span>
+								<span><?php _e('Export Give Settings', 'give'); ?></span>
 							</h3>
-							<p><?php _e( 'Download an export of Give\'s settings and import it in a new WordPress installation.', 'give' ); ?></p>
+							<p><?php _e('Download an export of Give\'s settings and import it in a new WordPress installation.', 'give'); ?></p>
 						</td>
 						<td>
 							<form method="post">
 								<?php
-								$export_excludes = apply_filters( 'settings_export_excludes', array() );
-								if ( ! empty( $export_excludes ) ) {
+								$export_excludes = apply_filters('settings_export_excludes', array());
+								if ( ! empty($export_excludes)) {
 									?>
-									<i class="settings-excludes-title"><?php echo __( 'Checked options from the list will not be exported.', 'give' ); ?></i>
+									<i class="settings-excludes-title"><?php echo __('Checked options from the list will not be exported.', 'give'); ?></i>
 									<ul class="settings-excludes-list">
-										<?php foreach ( $export_excludes as $option_key => $option_label ) { ?>
+										<?php foreach ($export_excludes as $option_key => $option_label) { ?>
 											<li>
 												<label for="settings_export_excludes[<?php echo $option_key ?>]">
 													<input
 														type="checkbox" checked
 														name="settings_export_excludes[<?php echo $option_key ?>]"
-														id="settings_export_excludes[<?php echo $option_key ?>]"><?php echo esc_html( $option_label ); ?>
+														id="settings_export_excludes[<?php echo $option_key ?>]"><?php echo esc_html($option_label); ?>
 												</label>
 											</li>
 										<?php } ?>
 									</ul>
 								<?php } ?>
 								<input type="hidden" name="give-action" value="core_settings_export"/>
-								<input type="submit" value="<?php esc_attr_e( 'Export JSON', 'give' ); ?>"
+								<input type="submit" value="<?php esc_attr_e('Export JSON', 'give'); ?>"
 								       class="button-secondary"/>
 							</form>
 						</td>
@@ -235,7 +235,7 @@ 
 					 *
 					 * @since 1.0
 					 */
-					do_action( 'give_tools_tab_export_table_bottom' );
+					do_action('give_tools_tab_export_table_bottom');
 					?>
 					</tbody>
 				</table>
@@ -246,7 +246,7 @@ 
 				 *
 				 * @since 1.0
 				 */
-				do_action( 'give_tools_tab_export_content_bottom' );
+				do_action('give_tools_tab_export_content_bottom');
 				?>
 
 			</div>

includes/admin/tools/views/html-admin-page-import-donations.php

Indentation +7 added lines, -7 removed lines

@@ -22,15 +22,15 @@
         <div class="postbox">
             <h1 class="give-importer-h1" align="center">
 	            <?php
-	            _e( 'Import Donations', 'give' );
+				_e( 'Import Donations', 'give' );
 
-	            if ( ! empty( $_POST['mapto'] ) && ! empty( $_GET['dry_run'] ) ) {
-		            printf(
+				if ( ! empty( $_POST['mapto'] ) && ! empty( $_GET['dry_run'] ) ) {
+					printf(
 						'<strong> %s</strong>',
-			            __( '(Dry Run)', 'give' )
-		            );
-	            }
-	            ?>
+						__( '(Dry Run)', 'give' )
+					);
+				}
+				?>
             </h1>
             <div class="inside give-tools-setting-page-import give-import-donations">
 				<?php

Spacing +11 added lines, -11 removed lines

@@ -3,11 +3,11 @@ 
  * Admin View: Import Donations
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! current_user_can( 'manage_give_settings' ) ) {
+if ( ! current_user_can('manage_give_settings')) {
 	return;
 }
 
@@ -16,18 +16,18 @@ 
  *
  * @since 1.8.13
  */
-do_action( 'give_tools_import_donations_main_before' );
+do_action('give_tools_import_donations_main_before');
 ?>
     <div id="poststuff">
         <div class="postbox">
             <h1 class="give-importer-h1" align="center">
 	            <?php
-	            _e( 'Import Donations', 'give' );
+	            _e('Import Donations', 'give');
 
-	            if ( ! empty( $_POST['mapto'] ) && ! empty( $_GET['dry_run'] ) ) {
+	            if ( ! empty($_POST['mapto']) && ! empty($_GET['dry_run'])) {
 		            printf(
 						'<strong> %s</strong>',
-			            __( '(Dry Run)', 'give' )
+			            __('(Dry Run)', 'give')
 		            );
 	            }
 	            ?>
@@ -39,7 +39,7 @@ 
 				 *
 				 * @since 1.8.14
 				 */
-				do_action( 'give_tools_import_donations_form_before_start' );
+				do_action('give_tools_import_donations_form_before_start');
 				?>
                 <form method="post" id="give-import-donations-form" class="give-import-form tools-setting-page-import tools-setting-page-import">
 
@@ -49,7 +49,7 @@ 
 					 *
 					 * @since 1.8.14
 					 */
-					do_action( 'give_tools_import_donations_form_start' );
+					do_action('give_tools_import_donations_form_start');
 					?>
 
 					<?php
@@ -58,7 +58,7 @@ 
 					 *
 					 * @since 1.8.14
 					 */
-					do_action( 'give_tools_import_donations_form_end' );
+					do_action('give_tools_import_donations_form_end');
 					?>
                 </form>
 				<?php
@@ -67,7 +67,7 @@ 
 				 *
 				 * @since 1.8.14
 				 */
-				do_action( 'give_tools_import_donations_form_after_end' );
+				do_action('give_tools_import_donations_form_after_end');
 				?>
             </div><!-- .inside -->
         </div><!-- .postbox -->
@@ -78,4 +78,4 @@ 
  *
  * @since 1.8.13
  */
-do_action( 'give_tools_import_donations_main_after' );
+do_action('give_tools_import_donations_main_after');