WordImpress / Give

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

Doc Comments +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
 	 * @since 1.5
 	 * @global object $wpdb Used to query the database using the WordPress
 	 *   Database API
-	 * @return array $data The data for the CSV file
+	 * @return boolean $data The data for the CSV file
 	 */
 	public function get_data() {
 		global $give_logs, $wpdb;

Spacing +99 added lines, -99 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -55,26 +55,26 @@ 
 	public function get_data() {
 		global $give_logs, $wpdb;
 
-		$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');
 
-		$args = apply_filters( 'give_recount_form_stats_args', array(
+		$args = apply_filters('give_recount_form_stats_args', array(
 			'post_parent__in' => $all_forms,
 			'post_type'       => 'give_log',
 			'posts_per_page'  => $this->per_step,
@@ -82,73 +82,73 @@ 
 			'paged'           => $this->step,
 			'log_type'        => 'sale',
 			'fields'          => 'ids',
-		) );
+		));
 
-		$log_ids = $give_logs->get_connected_logs( $args, 'sale' );
+		$log_ids = $give_logs->get_connected_logs($args, 'sale');
 
-		if ( $log_ids ) {
-			$log_ids = implode( ',', $log_ids );
+		if ($log_ids) {
+			$log_ids = implode(',', $log_ids);
 
-			$payment_ids = $wpdb->get_col( "SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)" );
-			unset( $log_ids );
+			$payment_ids = $wpdb->get_col("SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)");
+			unset($log_ids);
 
-			$payment_ids = implode( ',', $payment_ids );
-			$payments    = $wpdb->get_results( "SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (" . $payment_ids . ")" );
-			unset( $payment_ids );
+			$payment_ids = implode(',', $payment_ids);
+			$payments    = $wpdb->get_results("SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (".$payment_ids.")");
+			unset($payment_ids);
 
 			//Loop through payments
-			foreach ( $payments as $payment ) {
+			foreach ($payments as $payment) {
 
 				// 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;
 				}
 
 				//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,
 					);
 				}
 
-				$totals[ $form_id ]['sales'] ++;
-				$totals[ $form_id ]['earnings'] += $payment_item['price'];
+				$totals[$form_id]['sales']++;
+				$totals[$form_id]['earnings'] += $payment_item['price'];
 
 				$processed_payments[] = $payment->ID;
 
 			}
 
-			$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', $stats['earnings'] );
+		foreach ($totals as $key => $stats) {
+			give_update_meta($key, '_give_form_sales', $stats['sales']);
+			give_update_meta($key, '_give_form_earnings', $stats['earnings']);
 		}
 
 		return false;
@@ -163,20 +163,20 @@ 
 	 */
 	public function get_percentage_complete() {
 
-		$total = $this->get_stored_data( 'give_recount_all_total', false );
+		$total = $this->get_stored_data('give_recount_all_total', false);
 
-		if ( false === $total ) {
+		if (false === $total) {
 			$this->pre_fetch();
-			$total = $this->get_stored_data( 'give_recount_all_total', 0 );
+			$total = $this->get_stored_data('give_recount_all_total', 0);
 		}
 
 		$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;
 		}
 
@@ -190,8 +190,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;
 	}
 
 	/**
@@ -202,34 +202,34 @@ 
 	 */
 	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_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_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;
 		}
 	}
 
 	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);
 		}
 	}
 
@@ -258,76 +258,76 @@ 
 
 		global $give_logs, $wpdb;
 
-		if ( $this->step == 1 ) {
-			$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' );
+		if ($this->step == 1) {
+			$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');
 		}
 
-		$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) {
 			$total         = 0;
-			$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);
 			}
 
-			$all_forms = $this->get_stored_data( 'give_temp_form_ids' );
+			$all_forms = $this->get_stored_data('give_temp_form_ids');
 
-			if ( false === $all_forms ) {
+			if (false === $all_forms) {
 				$args = array(
 					'post_status'    => 'any',
 					'post_type'      => 'give_forms',
-					'posts_per_page' => - 1,
+					'posts_per_page' => -1,
 					'fields'         => 'ids',
 				);
 
-				$all_forms = get_posts( $args );
-				$this->store_data( 'give_temp_form_ids', $all_forms );
+				$all_forms = get_posts($args);
+				$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(
 				'post_parent__in' => $all_forms,
 				'post_type'       => 'give_log',
 				'post_status'     => 'publish',
 				'log_type'        => 'sale',
 				'fields'          => 'ids',
 				'nopaging'        => true,
-			) );
+			));
 
-			$all_logs = $give_logs->get_connected_logs( $args, 'sale' );
+			$all_logs = $give_logs->get_connected_logs($args, 'sale');
 
-			if ( $all_logs ) {
-				$log_ids     = implode( ',', $all_logs );
-				$payment_ids = $wpdb->get_col( "SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)" );
-				unset( $log_ids );
+			if ($all_logs) {
+				$log_ids     = implode(',', $all_logs);
+				$payment_ids = $wpdb->get_col("SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)");
+				unset($log_ids);
 
-				$payment_ids = implode( ',', $payment_ids );
-				$payments    = $wpdb->get_results( "SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (" . $payment_ids . ")" );
-				unset( $payment_ids );
+				$payment_ids = implode(',', $payment_ids);
+				$payments    = $wpdb->get_results("SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (".$payment_ids.")");
+				unset($payment_ids);
 
-				foreach ( $payments as $payment ) {
+				foreach ($payments as $payment) {
 
-					$payment = new Give_Payment( $payment->ID );
+					$payment = new Give_Payment($payment->ID);
 					$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
@@ -336,11 +336,11 @@ 
 
 				}
 
-				$total = count( $all_logs );
+				$total = count($all_logs);
 			}
 
-			$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);
 		}
 
 	}
@@ -354,11 +354,11 @@ 
 	 *
 	 * @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));
 
-		return empty( $value ) ? false : maybe_unserialize( $value );
+		return empty($value) ? false : maybe_unserialize($value);
 	}
 
 	/**
@@ -371,10 +371,10 @@ 
 	 *
 	 * @return void
 	 */
-	private function store_data( $key, $value ) {
+	private function store_data($key, $value) {
 		global $wpdb;
 
-		$value = maybe_serialize( $value );
+		$value = maybe_serialize($value);
 
 		$data = array(
 			'option_name'  => $key,
@@ -388,7 +388,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -400,9 +400,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-customer-stats.php

Doc Comments +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
 	 * @access public
 	 * @since 1.5
 	 *
-	 * @return array $data The data for the CSV file
+	 * @return boolean $data The data for the CSV file
 	 */
 	public function get_data() {
 

Spacing +37 added lines, -37 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -55,32 +55,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',
 		);
 
-		$customers = Give()->customers->get_customers( $args );
+		$customers = Give()->customers->get_customers($args);
 
-		if ( $customers ) {
+		if ($customers) {
 
-			$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 ( $customers as $customer ) {
+			foreach ($customers as $customer) {
 
-				$attached_payment_ids = explode( ',', $customer->payment_ids );
+				$attached_payment_ids = explode(',', $customer->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(
@@ -91,29 +91,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_customer_recount_should_increase_value', true, $payment ) ) {
-								$purchase_value += give_get_payment_amount( $payment->ID );
+							if (apply_filters('give_customer_recount_should_increase_value', true, $payment)) {
+								$purchase_value += give_get_payment_amount($payment->ID);
 							}
 
-							if ( apply_filters( 'give_customer_recount_should_increase_count', true, $payment ) ) {
-								$purchase_count ++;
+							if (apply_filters('give_customer_recount_should_increase_count', true, $payment)) {
+								$purchase_count++;
 							}
 
 						}
@@ -123,7 +123,7 @@ 
 
 				}
 
-				$payment_ids = implode( ',', $payment_ids );
+				$payment_ids = implode(',', $payment_ids);
 
 				$customer_update_data = array(
 					'purchase_count' => $purchase_count,
@@ -131,8 +131,8 @@ 
 					'payment_ids'    => $payment_ids,
 				);
 
-				$customer_instance = new Give_Customer( $customer->id );
-				$customer_instance->update( $customer_update_data );
+				$customer_instance = new Give_Customer($customer->id);
+				$customer_instance->update($customer_update_data);
 
 			}
 
@@ -152,21 +152,21 @@ 
 	public function get_percentage_complete() {
 
 		$args = array(
-			'number'  => - 1,
+			'number'  => -1,
 			'orderby' => 'id',
 			'order'   => 'DESC',
 		);
 
-		$customers = Give()->customers->get_customers( $args );
-		$total     = count( $customers );
+		$customers = Give()->customers->get_customers($args);
+		$total     = count($customers);
 
 		$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,19 +191,19 @@ 
 	 */
 	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->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;
 		}
@@ -213,10 +213,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-form-stats.php

Doc Comments +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
 	 * @since 1.5
 	 * @global object $wpdb Used to query the database using the WordPress
 	 *   Database API
-	 * @return array $data The data for the CSV file
+	 * @return boolean $data The data for the CSV file
 	 */
 	public function get_data() {
 		global $give_logs, $wpdb;

Spacing +66 added lines, -66 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -62,23 +62,23 @@ 
 	public function get_data() {
 		global $give_logs, $wpdb;
 
-		$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(
 			'post_parent'    => $this->form_id,
 			'post_type'      => 'give_log',
 			'posts_per_page' => $this->per_step,
@@ -86,49 +86,49 @@ 
 			'paged'          => $this->step,
 			'log_type'       => 'sale',
 			'fields'         => 'ids',
-		) );
+		));
 
-		$log_ids              = $give_logs->get_connected_logs( $args, 'sale' );
+		$log_ids              = $give_logs->get_connected_logs($args, 'sale');
 		$this->_log_ids_debug = array();
 
-		if ( $log_ids ) {
-			$log_ids     = implode( ',', $log_ids );
-			$payment_ids = $wpdb->get_col( "SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)" );
-			unset( $log_ids );
+		if ($log_ids) {
+			$log_ids     = implode(',', $log_ids);
+			$payment_ids = $wpdb->get_col("SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)");
+			unset($log_ids);
 
-			$payment_ids = implode( ',', $payment_ids );
-			$payments    = $wpdb->get_results( "SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (" . $payment_ids . ")" );
-			unset( $payment_ids );
+			$payment_ids = implode(',', $payment_ids);
+			$payments    = $wpdb->get_results("SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (".$payment_ids.")");
+			unset($payment_ids);
 
-			foreach ( $payments as $payment ) {
+			foreach ($payments as $payment) {
 
-				$payment = new Give_Payment( $payment->ID );
+				$payment = new Give_Payment($payment->ID);
 
 				//Ensure acceptible 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;
 				}
 
 				$this->_log_ids_debug[] = $payment->ID;
 
-				$totals['sales'] ++;
+				$totals['sales']++;
 				$totals['earnings'] += $payment->total;
 
 			}
 
-			$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', $totals['earnings'] );
+		give_update_meta($this->form_id, '_give_form_sales', $totals['sales']);
+		give_update_meta($this->form_id, '_give_form_earnings', $totals['earnings']);
 
 		return false;
 
@@ -143,54 +143,54 @@ 
 	public function get_percentage_complete() {
 		global $give_logs, $wpdb;
 
-		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(
 				'post_parent' => $this->form_id,
 				'post_type'   => 'give_log',
 				'post_status' => 'publish',
 				'log_type'    => 'sale',
 				'fields'      => 'ids',
 				'nopaging'    => true,
-			) );
+			));
 
-			$log_ids = $give_logs->get_connected_logs( $args, 'sale' );
+			$log_ids = $give_logs->get_connected_logs($args, 'sale');
 
-			if ( $log_ids ) {
-				$log_ids     = implode( ',', $log_ids );
-				$payment_ids = $wpdb->get_col( "SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)" );
-				unset( $log_ids );
+			if ($log_ids) {
+				$log_ids     = implode(',', $log_ids);
+				$payment_ids = $wpdb->get_col("SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)");
+				unset($log_ids);
 
-				$payment_ids = implode( ',', $payment_ids );
-				$payments    = $wpdb->get_results( "SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (" . $payment_ids . ")" );
-				unset( $payment_ids );
+				$payment_ids = implode(',', $payment_ids);
+				$payments    = $wpdb->get_results("SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (".$payment_ids.")");
+				unset($payment_ids);
 
-				foreach ( $payments as $payment ) {
-					if ( in_array( $payment->post_status, $accepted_statuses ) ) {
+				foreach ($payments as $payment) {
+					if (in_array($payment->post_status, $accepted_statuses)) {
 						continue;
 					}
 
-					$total ++;
+					$total++;
 				}
 			}
 
-			$this->store_data( 'give_recount_total_' . $this->form_id, $total );
+			$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;
 		}
 
@@ -204,8 +204,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;
 	}
 
 	/**
@@ -216,31 +216,31 @@ 
 	 */
 	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;
 		}
 	}
 
 	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);
 		}
 	}
 
@@ -268,11 +268,11 @@ 
 	 *
 	 * @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));
 
-		return empty( $value ) ? false : maybe_unserialize( $value );
+		return empty($value) ? false : maybe_unserialize($value);
 	}
 
 	/**
@@ -285,10 +285,10 @@ 
 	 *
 	 * @return void
 	 */
-	private function store_data( $key, $value ) {
+	private function store_data($key, $value) {
 		global $wpdb;
 
-		$value = maybe_serialize( $value );
+		$value = maybe_serialize($value);
 
 		$data = array(
 			'option_name'  => $key,
@@ -302,7 +302,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -314,9 +314,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-customer-stats.php

Doc Comments +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
 	 * @since 1.5
 	 * @global object $wpdb Used to query the database using the WordPress
 	 *   Database API
-	 * @return array $data The data for the CSV file
+	 * @return boolean $data The data for the CSV file
 	 */
 	public function get_data() {
 

Indentation +2 added lines, -2 removed lines

@@ -188,8 +188,8 @@
 
 			$payment_ids = implode( ',', $payment_ids );
 			$customer->update( array( 'payment_ids'    => $payment_ids,
-			                          'purchase_count' => $purchase_count,
-			                          'purchase_value' => $pending_total
+									  'purchase_count' => $purchase_count,
+									  'purchase_value' => $pending_total
 			) );
 
 			$this->done    = true;

Spacing +70 added lines, -70 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -54,39 +54,39 @@ 
 	 */
 	public function get_data() {
 
-		$customer = new Give_Customer( $this->customer_id );
-		$payments = $this->get_stored_data( 'give_recount_customer_payments_' . $customer->id, array() );
+		$customer = new Give_Customer($this->customer_id);
+		$payments = $this->get_stored_data('give_recount_customer_payments_'.$customer->id, array());
 
-		$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_customer_pending_total' . $customer->id, 0 );
+		if (count($step_items) > 0) {
+			$pending_total = (float) $this->get_stored_data('give_stats_customer_pending_total'.$customer->id, 0);
 			$step_total    = 0;
 
-			$found_payment_ids = $this->get_stored_data( 'give_stats_found_payments_' . $customer->id, array() );
+			$found_payment_ids = $this->get_stored_data('give_stats_found_payments_'.$customer->id, array());
 
-			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' . $customer->id, array() );
+					$missing_payments   = $this->get_stored_data('give_stats_missing_payments'.$customer->id, array());
 					$missing_payments[] = $payment->ID;
-					$this->store_data( 'give_stats_missing_payments' . $customer->id, $missing_payments );
+					$this->store_data('give_stats_missing_payments'.$customer->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_customer_recount_sholud_increase_value', true, $payment ) ) {
-						$payment_amount = give_get_payment_amount( $payment->ID );
+					if (apply_filters('give_customer_recount_sholud_increase_value', true, $payment)) {
+						$payment_amount = give_get_payment_amount($payment->ID);
 						$step_total += $payment_amount;
 					}
 
@@ -95,8 +95,8 @@ 
 			}
 
 			$updated_total = $pending_total + $step_total;
-			$this->store_data( 'give_stats_customer_pending_total' . $customer->id, $updated_total );
-			$this->store_data( 'give_stats_found_payments_' . $customer->id, $found_payment_ids );
+			$this->store_data('give_stats_customer_pending_total'.$customer->id, $updated_total);
+			$this->store_data('give_stats_found_payments_'.$customer->id, $found_payment_ids);
 
 			return true;
 		}
@@ -113,16 +113,16 @@ 
 	 */
 	public function get_percentage_complete() {
 
-		$payments = $this->get_stored_data( 'give_recount_customer_payments_' . $this->customer_id );
-		$total    = count( $payments );
+		$payments = $this->get_stored_data('give_recount_customer_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;
 		}
 
@@ -136,8 +136,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;
 	}
 
 	/**
@@ -148,62 +148,62 @@ 
 	 */
 	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 {
-			$customer    = new Give_Customer( $this->customer_id );
-			$payment_ids = get_option( 'give_stats_found_payments_' . $customer->id, array() );
-			$this->delete_data( 'give_stats_found_payments_' . $customer->id );
+			$customer    = new Give_Customer($this->customer_id);
+			$payment_ids = get_option('give_stats_found_payments_'.$customer->id, array());
+			$this->delete_data('give_stats_found_payments_'.$customer->id);
 
-			$removed_payments = array_unique( get_option( 'give_stats_missing_payments' . $customer->id, array() ) );
+			$removed_payments = array_unique(get_option('give_stats_missing_payments'.$customer->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_customer_recount_should_increase_count', true, $payment ) ) {
-					$purchase_count ++;
+				$payment = get_post($payment_id);
+				if (apply_filters('give_customer_recount_should_increase_count', true, $payment)) {
+					$purchase_count++;
 				}
 			}
 
-			$this->delete_data( 'give_stats_missing_payments' . $customer->id );
+			$this->delete_data('give_stats_missing_payments'.$customer->id);
 
-			$pending_total = $this->get_stored_data( 'give_stats_customer_pending_total' . $customer->id, 0 );
-			$this->delete_data( 'give_stats_customer_pending_total' . $customer->id );
-			$this->delete_data( 'give_recount_customer_stats_' . $customer->id );
-			$this->delete_data( 'give_recount_customer_payments_' . $this->customer_id );
+			$pending_total = $this->get_stored_data('give_stats_customer_pending_total'.$customer->id, 0);
+			$this->delete_data('give_stats_customer_pending_total'.$customer->id);
+			$this->delete_data('give_recount_customer_stats_'.$customer->id);
+			$this->delete_data('give_recount_customer_payments_'.$this->customer_id);
 
-			$payment_ids = implode( ',', $payment_ids );
-			$customer->update( array( 'payment_ids'    => $payment_ids,
+			$payment_ids = implode(',', $payment_ids);
+			$customer->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
-			$customer = new Give_Customer( $this->customer_id );
-			$customer->update( array( 'purchase_value' => give_format_amount( 0 ), 'purchase_count' => 0 ) );
+			$customer = new Give_Customer($this->customer_id);
+			$customer->update(array('purchase_value' => give_format_amount(0), 'purchase_count' => 0));
 
-			$attached_payment_ids = explode( ',', $customer->payment_ids );
+			$attached_payment_ids = explode(',', $customer->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_customer_payments_' . $customer->id, $payments );
+			$this->store_data('give_recount_customer_payments_'.$customer->id, $payments);
 		}
 	}
 
@@ -276,11 +276,11 @@ 
 	 *
 	 * @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));
 
-		return empty( $value ) ? false : maybe_unserialize( $value );
+		return empty($value) ? false : maybe_unserialize($value);
 	}
 
 	/**
@@ -293,10 +293,10 @@ 
 	 *
 	 * @return void
 	 */
-	private function store_data( $key, $value ) {
+	private function store_data($key, $value) {
 		global $wpdb;
 
-		$value = maybe_serialize( $value );
+		$value = maybe_serialize($value);
 
 		$data = array(
 			'option_name'  => $key,
@@ -310,7 +310,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -322,9 +322,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/logs/class-api-requests-logs-list-table.php

Doc Comments +1 added lines, -1 removed lines

@@ -157,7 +157,7 @@
 	 *
 	 * @access public
 	 * @since  1.0
-	 * @return string|bool String if search is present, false otherwise
+	 * @return string|false String if search is present, false otherwise
 	 */
 	public function get_search() {
 		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;

Spacing +65 added lines, -65 removed lines

@@ -10,13 +10,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -45,11 +45,11 @@ 
 		global $status, $page;
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => give_get_forms_label_singular(),    // Singular name of the listed records
-			'plural'   => give_get_forms_label_plural(),        // Plural name of the listed records
-			'ajax'     => false,// Does this table support ajax?
-		) );
+		parent::__construct(array(
+			'singular' => give_get_forms_label_singular(), // Singular name of the listed records
+			'plural'   => give_get_forms_label_plural(), // Plural name of the listed records
+			'ajax'     => false, // Does this table support ajax?
+		));
 	}
 
 	/**
@@ -63,20 +63,20 @@ 
 	 *
 	 * @return void
 	 */
-	public function search_box( $text, $input_id ) {
-		$input_id = $input_id . '-search-input';
+	public function search_box($text, $input_id) {
+		$input_id = $input_id.'-search-input';
 
-		if ( ! empty( $_REQUEST['orderby'] ) ) {
-			echo '<input type="hidden" name="orderby" value="' . esc_attr( $_REQUEST['orderby'] ) . '" />';
+		if ( ! empty($_REQUEST['orderby'])) {
+			echo '<input type="hidden" name="orderby" value="'.esc_attr($_REQUEST['orderby']).'" />';
 		}
-		if ( ! empty( $_REQUEST['order'] ) ) {
-			echo '<input type="hidden" name="order" value="' . esc_attr( $_REQUEST['order'] ) . '" />';
+		if ( ! empty($_REQUEST['order'])) {
+			echo '<input type="hidden" name="order" value="'.esc_attr($_REQUEST['order']).'" />';
 		}
 		?>
 		<p class="search-box" role="search">
 			<label class="screen-reader-text" for="<?php echo $input_id ?>"><?php echo $text; ?>:</label>
 			<input type="search" id="<?php echo $input_id ?>" name="s" value="<?php _admin_search_query(); ?>"/>
-			<?php submit_button( $text, 'button', false, false, array( 'ID' => 'search-submit' ) ); ?>
+			<?php submit_button($text, 'button', false, false, array('ID' => 'search-submit')); ?>
 		</p>
 		<?php
 	}
@@ -90,10 +90,10 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'ID'      => esc_html__( 'Log ID', 'give' ),
-			'details' => esc_html__( 'Request Details', 'give' ),
-			'ip'      => esc_html__( 'Request IP', 'give' ),
-			'date'    => esc_html__( 'Date', 'give' ),
+			'ID'      => esc_html__('Log ID', 'give'),
+			'details' => esc_html__('Request Details', 'give'),
+			'ip'      => esc_html__('Request IP', 'give'),
+			'date'    => esc_html__('Date', 'give'),
 		);
 
 		return $columns;
@@ -110,10 +110,10 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
-		switch ( $column_name ) {
+	public function column_default($item, $column_name) {
+		switch ($column_name) {
 			default:
-				return $item[ $column_name ];
+				return $item[$column_name];
 		}
 	}
 
@@ -127,26 +127,26 @@ 
 	 *
 	 * @return void
 	 */
-	public function column_details( $item ) {
+	public function column_details($item) {
 		?>
-		<a href="#TB_inline?width=640&amp;inlineId=log-details-<?php echo $item['ID']; ?>" class="thickbox"><?php esc_html_e( 'View Request', 'give' ); ?></a>
+		<a href="#TB_inline?width=640&amp;inlineId=log-details-<?php echo $item['ID']; ?>" class="thickbox"><?php esc_html_e('View Request', 'give'); ?></a>
 		<div id="log-details-<?php echo $item['ID']; ?>" style="display:none;">
 			<?php
 
-			$request = get_post_field( 'post_excerpt', $item['ID'] );
-			$error   = get_post_field( 'post_content', $item['ID'] );
-			echo '<p><strong>' . esc_html__( 'API Request:', 'give' ) . '</strong></p>';
-			echo '<div>' . $request . '</div>';
-			if ( ! empty( $error ) ) {
-				echo '<p><strong>' . esc_html__( 'Error', 'give' ) . '</strong></p>';
-				echo '<div>' . esc_html( $error ) . '</div>';
+			$request = get_post_field('post_excerpt', $item['ID']);
+			$error   = get_post_field('post_content', $item['ID']);
+			echo '<p><strong>'.esc_html__('API Request:', 'give').'</strong></p>';
+			echo '<div>'.$request.'</div>';
+			if ( ! empty($error)) {
+				echo '<p><strong>'.esc_html__('Error', 'give').'</strong></p>';
+				echo '<div>'.esc_html($error).'</div>';
 			}
-			echo '<p><strong>' . esc_html__( 'API User:', 'give' ) . '</strong></p>';
-			echo '<div>' . give_get_meta( $item['ID'], '_give_log_user', true ) . '</div>';
-			echo '<p><strong>' . esc_html__( 'API Key:', 'give' ) . '</strong></p>';
-			echo '<div>' . give_get_meta( $item['ID'], '_give_log_key', true ) . '</div>';
-			echo '<p><strong>' . esc_html__( 'Request Date:', 'give' ) . '</strong></p>';
-			echo '<div>' . get_post_field( 'post_date', $item['ID'] ) . '</div>';
+			echo '<p><strong>'.esc_html__('API User:', 'give').'</strong></p>';
+			echo '<div>'.give_get_meta($item['ID'], '_give_log_user', true).'</div>';
+			echo '<p><strong>'.esc_html__('API Key:', 'give').'</strong></p>';
+			echo '<div>'.give_get_meta($item['ID'], '_give_log_key', true).'</div>';
+			echo '<p><strong>'.esc_html__('Request Date:', 'give').'</strong></p>';
+			echo '<div>'.get_post_field('post_date', $item['ID']).'</div>';
 			?>
 		</div>
 		<?php
@@ -160,7 +160,7 @@ 
 	 * @return string|bool String if search is present, false otherwise
 	 */
 	public function get_search() {
-		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;
+		return ! empty($_GET['s']) ? urldecode(trim($_GET['s'])) : false;
 	}
 
 
@@ -176,19 +176,19 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
-		if ( 'top' === $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+	protected function display_tablenav($which) {
+		if ('top' === $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav <?php echo esc_attr($which); ?>">
 
 			<div class="alignleft actions bulkactions">
-				<?php $this->bulk_actions( $which ); ?>
+				<?php $this->bulk_actions($which); ?>
 			</div>
 			<?php
-			$this->extra_tablenav( $which );
-			$this->pagination( $which );
+			$this->extra_tablenav($which);
+			$this->pagination($which);
 			?>
 
 			<br class="clear"/>
@@ -210,31 +210,31 @@ 
 
 		$search = $this->get_search();
 
-		if ( $search ) {
-			if ( filter_var( $search, FILTER_VALIDATE_IP ) ) {
+		if ($search) {
+			if (filter_var($search, FILTER_VALIDATE_IP)) {
 				// This is an IP address search
 				$key = '_give_log_request_ip';
-			} elseif ( is_email( $search ) ) {
+			} elseif (is_email($search)) {
 				// This is an email search
-				$userdata = get_user_by( 'email', $search );
+				$userdata = get_user_by('email', $search);
 
-				if ( $userdata ) {
+				if ($userdata) {
 					$search = $userdata->ID;
 				}
 
 				$key = '_give_log_user';
-			} elseif ( strlen( $search ) == 32 ) {
+			} elseif (strlen($search) == 32) {
 				// Look for an API key
 				$key = '_give_log_key';
-			} elseif ( stristr( $search, 'token:' ) ) {
+			} elseif (stristr($search, 'token:')) {
 				// Look for an API token
-				$search = str_ireplace( 'token:', '', $search );
+				$search = str_ireplace('token:', '', $search);
 				$key    = '_give_log_token';
 			} else {
 				// This is (probably) a user ID search
-				$userdata = get_userdata( $search );
+				$userdata = get_userdata($search);
 
-				if ( $userdata ) {
+				if ($userdata) {
 					$search = $userdata->ID;
 				}
 
@@ -260,7 +260,7 @@ 
 	 * @return int Current page number
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 	/**
@@ -270,7 +270,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	function bulk_actions( $which = '' ) {
+	function bulk_actions($which = '') {
 		give_log_views();
 	}
 
@@ -293,14 +293,14 @@ 
 			'meta_query' => $this->get_meta_query(),
 		);
 
-		$logs = $give_logs->get_connected_logs( $log_query );
+		$logs = $give_logs->get_connected_logs($log_query);
 
-		if ( $logs ) {
-			foreach ( $logs as $log ) {
+		if ($logs) {
+			foreach ($logs as $log) {
 
 				$logs_data[] = array(
 					'ID'   => $log->ID,
-					'ip'   => give_get_meta( $log->ID, '_give_log_request_ip', true ),
+					'ip'   => give_get_meta($log->ID, '_give_log_request_ip', true),
 					'date' => $log->post_date,
 				);
 			}
@@ -328,14 +328,14 @@ 
 		$columns               = $this->get_columns();
 		$hidden                = array(); // No hidden columns
 		$sortable              = $this->get_sortable_columns();
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 		$this->items           = $this->get_logs();
-		$total_items           = $give_logs->get_log_count( 0, 'api_request' );
+		$total_items           = $give_logs->get_log_count(0, 'api_request');
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 				'total_items' => $total_items,
 				'per_page'    => $this->per_page,
-				'total_pages' => ceil( $total_items / $this->per_page ),
+				'total_pages' => ceil($total_items / $this->per_page),
 			)
 		);
 	}

includes/admin/tools/logs/class-sales-logs-list-table.php

Doc Comments +1 added lines, -1 removed lines

@@ -157,7 +157,7 @@
 	 *
 	 * @access public
 	 * @since  1.0
-	 * @return string|bool string If search is present, false otherwise
+	 * @return string|false string If search is present, false otherwise
 	 */
 	public function get_search() {
 		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;

Spacing +83 added lines, -83 removed lines

@@ -9,13 +9,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -44,13 +44,13 @@ 
 		global $status, $page;
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => give_get_forms_label_singular(),    // Singular name of the listed records
-			'plural'   => give_get_forms_label_plural(),        // Plural name of the listed records
-			'ajax'     => false,// Does this table support ajax?
-		) );
+		parent::__construct(array(
+			'singular' => give_get_forms_label_singular(), // Singular name of the listed records
+			'plural'   => give_get_forms_label_plural(), // Plural name of the listed records
+			'ajax'     => false, // Does this table support ajax?
+		));
 
-		add_action( 'give_log_view_actions', array( $this, 'give_forms_filter' ) );
+		add_action('give_log_view_actions', array($this, 'give_forms_filter'));
 	}
 
 	/**
@@ -64,39 +64,39 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
+	public function column_default($item, $column_name) {
 
-		$payment = give_get_payment_by( 'id', $item['payment_id'] );
+		$payment = give_get_payment_by('id', $item['payment_id']);
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 			case 'form' :
-				$form_title = get_the_title( $item[ $column_name ] );
-				$form_title = empty( $form_title ) ? sprintf( __( 'Untitled (#%s)', 'give' ), $item[ $column_name ] ) : $form_title;
-				return '<a href="' . esc_url( add_query_arg( 'form', $item[ $column_name ] ) ) . '" >' . $form_title . '</a>';
+				$form_title = get_the_title($item[$column_name]);
+				$form_title = empty($form_title) ? sprintf(__('Untitled (#%s)', 'give'), $item[$column_name]) : $form_title;
+				return '<a href="'.esc_url(add_query_arg('form', $item[$column_name])).'" >'.$form_title.'</a>';
 
 			case 'user_id' :
-				return '<a href="' .
-					   admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&user=' . ( ! empty( $item['user_id'] ) ? urlencode( $item['user_id'] ) : give_get_payment_user_email( $item['payment_id'] ) ) ) .
-					   '">' . $item['user_name'] . '</a>';
+				return '<a href="'.
+					   admin_url('edit.php?post_type=give_forms&page=give-payment-history&user='.( ! empty($item['user_id']) ? urlencode($item['user_id']) : give_get_payment_user_email($item['payment_id']))).
+					   '">'.$item['user_name'].'</a>';
 
 			case 'amount' :
-				return give_currency_filter( give_format_amount( $item['amount'] ) );
+				return give_currency_filter(give_format_amount($item['amount']));
 
 			case 'status' :
 
-				$value = '<div class="give-donation-status status-' . sanitize_title( give_get_payment_status( $payment, true ) ) . '"><span class="give-donation-status-icon"></span> ' . give_get_payment_status( $payment, true ) . '</div>';
+				$value = '<div class="give-donation-status status-'.sanitize_title(give_get_payment_status($payment, true)).'"><span class="give-donation-status-icon"></span> '.give_get_payment_status($payment, true).'</div>';
 
-				if ( $payment->mode == 'test' ) {
-					$value .= ' <span class="give-item-label give-item-label-orange give-test-mode-transactions-label" data-tooltip="' . esc_attr__( 'This donation was made in test mode.', 'give' ) . '">' . __( 'Test', 'give' ) . '</span>';
+				if ($payment->mode == 'test') {
+					$value .= ' <span class="give-item-label give-item-label-orange give-test-mode-transactions-label" data-tooltip="'.esc_attr__('This donation was made in test mode.', 'give').'">'.__('Test', 'give').'</span>';
 				}
 
 				return $value;
 
 			case 'payment_id' :
-				return '<a href="' . admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-order-details&id=' . $item['payment_id'] ) . '">' . give_get_payment_number( $item['payment_id'] ) . '</a>';
+				return '<a href="'.admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-order-details&id='.$item['payment_id']).'">'.give_get_payment_number($item['payment_id']).'</a>';
 
 			default:
-				return $item[ $column_name ];
+				return $item[$column_name];
 		}
 	}
 
@@ -109,13 +109,13 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'ID'         => __( 'Log ID', 'give' ),
-			'user_id'    => __( 'Donor', 'give' ),
-			'form'       => __( 'Form', 'give' ),
-			'amount'     => __( 'Donation Amount', 'give' ),
-			'status'     => __( 'Status', 'give' ),
-			'payment_id' => __( 'Transaction ID', 'give' ),
-			'date'       => __( 'Date', 'give' ),
+			'ID'         => __('Log ID', 'give'),
+			'user_id'    => __('Donor', 'give'),
+			'form'       => __('Form', 'give'),
+			'amount'     => __('Donation Amount', 'give'),
+			'status'     => __('Status', 'give'),
+			'payment_id' => __('Transaction ID', 'give'),
+			'date'       => __('Date', 'give'),
 		);
 
 		return $columns;
@@ -129,7 +129,7 @@ 
 	 * @return int Current page number
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 	/**
@@ -140,7 +140,7 @@ 
 	 * @return mixed int If User ID, string If Email/Login
 	 */
 	public function get_filtered_user() {
-		return isset( $_GET['user'] ) ? absint( $_GET['user'] ) : false;
+		return isset($_GET['user']) ? absint($_GET['user']) : false;
 	}
 
 	/**
@@ -151,7 +151,7 @@ 
 	 * @return int Download ID
 	 */
 	public function get_filtered_give_form() {
-		return ! empty( $_GET['form'] ) ? absint( $_GET['form'] ) : false;
+		return ! empty($_GET['form']) ? absint($_GET['form']) : false;
 	}
 
 	/**
@@ -162,7 +162,7 @@ 
 	 * @return string|bool string If search is present, false otherwise
 	 */
 	public function get_search() {
-		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;
+		return ! empty($_GET['s']) ? urldecode(trim($_GET['s'])) : false;
 	}
 
 
@@ -178,20 +178,20 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
+	protected function display_tablenav($which) {
 
-		if ( 'top' === $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+		if ('top' === $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav <?php echo esc_attr($which); ?>">
 
 			<div class="alignleft actions bulkactions">
-				<?php $this->bulk_actions( $which ); ?>
+				<?php $this->bulk_actions($which); ?>
 			</div>
 			<?php
-			$this->extra_tablenav( $which );
-			$this->pagination( $which );
+			$this->extra_tablenav($which);
+			$this->pagination($which);
 			?>
 
 			<br class="clear"/>
@@ -214,7 +214,7 @@ 
 
 		$meta_query = array();
 
-		if ( $user ) {
+		if ($user) {
 			// Show only logs from a specific user
 			$meta_query[] = array(
 				'key'   => '_give_log_user_id',
@@ -223,8 +223,8 @@ 
 		}
 
 		$search = $this->get_search();
-		if ( $search ) {
-			if ( is_email( $search ) ) {
+		if ($search) {
+			if (is_email($search)) {
 				// This is an email search. We use this to ensure it works for guest users and logged-in users
 				$key     = '_give_log_user_info';
 				$compare = 'LIKE';
@@ -233,32 +233,32 @@ 
 				$key     = '_give_log_user_id';
 				$compare = 'LIKE';
 
-				if ( ! is_numeric( $search ) ) {
+				if ( ! is_numeric($search)) {
 					// Searching for user by username
-					$user = get_user_by( 'login', $search );
+					$user = get_user_by('login', $search);
 
-					if ( $user ) {
+					if ($user) {
 						// Found one, set meta value to user's ID
 						$search = $user->ID;
 					} else {
 						// No user found so let's do a real search query
-						$users = new WP_User_Query( array(
+						$users = new WP_User_Query(array(
 							'search'         => $search,
-							'search_columns' => array( 'user_url', 'user_nicename' ),
+							'search_columns' => array('user_url', 'user_nicename'),
 							'number'         => 1,
 							'fields'         => 'ids',
-						) );
+						));
 
 						$found_user = $users->get_results();
 
-						if ( $found_user ) {
+						if ($found_user) {
 							$search = $found_user[0];
 						}
 					}
 				}
 			}
 
-			if ( ! $this->file_search ) {
+			if ( ! $this->file_search) {
 				// Meta query only works for non file name searche
 				$meta_query[] = array(
 					'key'     => $key,
@@ -279,7 +279,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	function bulk_actions( $which = '' ) {
+	function bulk_actions($which = '') {
 		give_log_views();
 	}
 
@@ -291,24 +291,24 @@ 
 	 * @return void
 	 */
 	public function give_forms_filter() {
-		$give_forms = get_posts( array(
+		$give_forms = get_posts(array(
 			'post_type'              => 'give_forms',
 			'post_status'            => 'any',
-			'posts_per_page'         => - 1,
+			'posts_per_page'         => -1,
 			'orderby'                => 'title',
 			'order'                  => 'ASC',
 			'fields'                 => 'ids',
 			'update_post_meta_cache' => false,
 			'update_post_term_cache' => false,
-		) );
+		));
 
-		if ( $give_forms ) {
+		if ($give_forms) {
 			echo '<select name="form" id="give-log-form-filter">';
-			echo '<option value="0">' . __( 'All', 'give' ) . '</option>';
-			foreach ( $give_forms as $form ) {
-				$form_title = get_the_title( $form );
-				$form_title = empty( $form_title ) ? sprintf( __( 'Untitled (#%s)', 'give' ), $form ) : $form_title;
-				echo '<option value="' . $form . '"' . selected( $form, $this->get_filtered_give_form() ) . '>' . esc_html( $form_title ) . '</option>';
+			echo '<option value="0">'.__('All', 'give').'</option>';
+			foreach ($give_forms as $form) {
+				$form_title = get_the_title($form);
+				$form_title = empty($form_title) ? sprintf(__('Untitled (#%s)', 'give'), $form) : $form_title;
+				echo '<option value="'.$form.'"'.selected($form, $this->get_filtered_give_form()).'>'.esc_html($form_title).'</option>';
 			}
 			echo '</select>';
 		}
@@ -328,7 +328,7 @@ 
 
 		$logs_data = array();
 		$paged     = $this->get_paged();
-		$give_form = empty( $_GET['s'] ) ? $this->get_filtered_give_form() : null;
+		$give_form = empty($_GET['s']) ? $this->get_filtered_give_form() : null;
 		$user      = $this->get_filtered_user();
 
 		$log_query = array(
@@ -338,38 +338,38 @@ 
 			'meta_query'  => $this->get_meta_query(),
 		);
 
-		$cache_key = Give_Cache::get_key( 'get_logs', $log_query );
+		$cache_key = Give_Cache::get_key('get_logs', $log_query);
 
 		// Return result from cache if exist.
-		if ( ! ( $logs_data = Give_Cache::get( $cache_key ) ) ) {
-			$logs = $give_logs->get_connected_logs( $log_query );
+		if ( ! ($logs_data = Give_Cache::get($cache_key))) {
+			$logs = $give_logs->get_connected_logs($log_query);
 
-			if ( $logs ) {
-				foreach ( $logs as $log ) {
-					$payment_id = give_get_meta( $log->ID, '_give_log_payment_id', true );
+			if ($logs) {
+				foreach ($logs as $log) {
+					$payment_id = give_get_meta($log->ID, '_give_log_payment_id', true);
 
 					// Make sure this payment hasn't been deleted
-					if ( get_post( $payment_id ) ) :
-						$user_info      = give_get_payment_meta_user_info( $payment_id );
-						$payment_meta   = give_get_payment_meta( $payment_id );
-						$payment_amount = give_get_payment_amount( $payment_id );
+					if (get_post($payment_id)) :
+						$user_info      = give_get_payment_meta_user_info($payment_id);
+						$payment_meta   = give_get_payment_meta($payment_id);
+						$payment_amount = give_get_payment_amount($payment_id);
 
 						$logs_data[] = array(
-							'ID'         => '<span class="give-item-label give-item-label-gray">' . $log->ID . '</span>',
+							'ID'         => '<span class="give-item-label give-item-label-gray">'.$log->ID.'</span>',
 							'payment_id' => $payment_id,
 							'form'       => $log->post_parent,
 							'amount'     => $payment_amount,
 							'user_id'    => $user_info['id'],
-							'user_name'  => $user_info['first_name'] . ' ' . $user_info['last_name'],
-							'date'       => get_post_field( 'post_date', $payment_id ),
+							'user_name'  => $user_info['first_name'].' '.$user_info['last_name'],
+							'date'       => get_post_field('post_date', $payment_id),
 						);
 
 					endif;
 				}
 
 				// Cache results.
-				if ( ! empty( $logs_data ) ) {
-					Give_Cache::set( $cache_key, $logs_data );
+				if ( ! empty($logs_data)) {
+					Give_Cache::set($cache_key, $logs_data);
 				}
 			}
 		}
@@ -397,15 +397,15 @@ 
 		$columns               = $this->get_columns();
 		$hidden                = array();
 		$sortable              = $this->get_sortable_columns();
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 		$current_page          = $this->get_pagenum();
 		$this->items           = $this->get_logs();
-		$total_items           = $give_logs->get_log_count( $this->get_filtered_give_form(), 'sale', $this->get_meta_query() );
+		$total_items           = $give_logs->get_log_count($this->get_filtered_give_form(), 'sale', $this->get_meta_query());
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 				'total_items' => $total_items,
 				'per_page'    => $this->per_page,
-				'total_pages' => ceil( $total_items / $this->per_page ),
+				'total_pages' => ceil($total_items / $this->per_page),
 			)
 		);
 	}

includes/class-give-gravatars.php

Spacing +95 added lines, -95 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -62,50 +62,50 @@ 
 	 *
 	 * @return bool If the gravatar exists or not
 	 */
-	public function validate_gravatar( $id_or_email ) {
+	public function validate_gravatar($id_or_email) {
 		//id or email code borrowed from wp-includes/pluggable.php
 		$email = '';
-		if ( is_numeric( $id_or_email ) ) {
+		if (is_numeric($id_or_email)) {
 			$id   = (int) $id_or_email;
-			$user = get_userdata( $id );
-			if ( $user ) {
+			$user = get_userdata($id);
+			if ($user) {
 				$email = $user->user_email;
 			}
-		} elseif ( is_object( $id_or_email ) ) {
+		} elseif (is_object($id_or_email)) {
 			// No avatar for pingbacks or trackbacks
-			$allowed_comment_types = apply_filters( 'get_avatar_comment_types', array( 'comment' ) );
-			if ( ! empty( $id_or_email->comment_type ) && ! in_array( $id_or_email->comment_type, (array) $allowed_comment_types ) ) {
+			$allowed_comment_types = apply_filters('get_avatar_comment_types', array('comment'));
+			if ( ! empty($id_or_email->comment_type) && ! in_array($id_or_email->comment_type, (array) $allowed_comment_types)) {
 				return false;
 			}
 
-			if ( ! empty( $id_or_email->user_id ) ) {
+			if ( ! empty($id_or_email->user_id)) {
 				$id   = (int) $id_or_email->user_id;
-				$user = get_userdata( $id );
-				if ( $user ) {
+				$user = get_userdata($id);
+				if ($user) {
 					$email = $user->user_email;
 				}
-			} elseif ( ! empty( $id_or_email->comment_author_email ) ) {
+			} elseif ( ! empty($id_or_email->comment_author_email)) {
 				$email = $id_or_email->comment_author_email;
 			}
 		} else {
 			$email = $id_or_email;
 		}
 
-		$hashkey = md5( strtolower( trim( $email ) ) );
-		$uri     = 'http://www.gravatar.com/avatar/' . $hashkey . '?d=404';
+		$hashkey = md5(strtolower(trim($email)));
+		$uri     = 'http://www.gravatar.com/avatar/'.$hashkey.'?d=404';
 
-		$data = wp_cache_get( $hashkey );
-		if ( false === $data ) {
-			$response = wp_remote_head( $uri );
-			if ( is_wp_error( $response ) ) {
+		$data = wp_cache_get($hashkey);
+		if (false === $data) {
+			$response = wp_remote_head($uri);
+			if (is_wp_error($response)) {
 				$data = 'not200';
 			} else {
 				$data = $response['response']['code'];
 			}
-			wp_cache_set( $hashkey, $data, $group = '', $expire = 60 * 5 );
+			wp_cache_set($hashkey, $data, $group = '', $expire = 60 * 5);
 
 		}
-		if ( $data == '200' ) {
+		if ($data == '200') {
 			return true;
 		} else {
 			return false;
@@ -122,19 +122,19 @@ 
 	 *
 	 * @return array        IDs if logs, false otherwise
 	 */
-	public function get_log_ids( $form_id = '' ) {
+	public function get_log_ids($form_id = '') {
 
 		// get Give_Logging class
 		global $give_logs;
 
 		// get log for this form
-		$logs = $give_logs->get_logs( $form_id );
+		$logs = $give_logs->get_logs($form_id);
 
-		if ( $logs ) {
+		if ($logs) {
 			$log_ids = array();
 
 			// make an array with all the donor IDs
-			foreach ( $logs as $log ) {
+			foreach ($logs as $log) {
 				$log_ids[] = $log->ID;
 			}
 
@@ -155,51 +155,51 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function get_payment_ids( $form_id = '' ) {
+	public function get_payment_ids($form_id = '') {
 
 		$give_options = give_get_settings();
 
-		$log_ids = $this->get_log_ids( $form_id );
+		$log_ids = $this->get_log_ids($form_id);
 
-		if ( $log_ids ) {
+		if ($log_ids) {
 
 			$payment_ids = array();
 
-			foreach ( $log_ids as $id ) {
+			foreach ($log_ids as $id) {
 				// get the payment ID for each corresponding log ID
-				$payment_ids[] = give_get_meta( $id, '_give_log_payment_id', true );
+				$payment_ids[] = give_get_meta($id, '_give_log_payment_id', true);
 			}
 
 			// remove donors who have donated more than once so we can have unique avatars
 			$unique_emails = array();
 
-			foreach ( $payment_ids as $key => $id ) {
+			foreach ($payment_ids as $key => $id) {
 
-				$email = give_get_meta( $id, '_give_payment_user_email', true );
+				$email = give_get_meta($id, '_give_payment_user_email', true);
 
-				if ( isset ( $give_options['give_donors_gravatars_has_gravatar_account'] ) ) {
-					if ( ! $this->validate_gravatar( $email ) ) {
+				if (isset ($give_options['give_donors_gravatars_has_gravatar_account'])) {
+					if ( ! $this->validate_gravatar($email)) {
 						continue;
 					}
 				}
 
-				$unique_emails[ $id ] = give_get_meta( $id, '_give_payment_user_email', true );
+				$unique_emails[$id] = give_get_meta($id, '_give_payment_user_email', true);
 
 			}
 
 			$unique_ids = array();
 
 			// strip duplicate emails
-			$unique_emails = array_unique( $unique_emails );
+			$unique_emails = array_unique($unique_emails);
 
 			// convert the unique IDs back into simple array
-			foreach ( $unique_emails as $id => $email ) {
+			foreach ($unique_emails as $id => $email) {
 				$unique_ids[] = $id;
 			}
 
 			// randomize the payment IDs if enabled
-			if ( isset( $give_options['give_donors_gravatars_random_gravatars'] ) ) {
-				shuffle( $unique_ids );
+			if (isset($give_options['give_donors_gravatars_random_gravatars'])) {
+				shuffle($unique_ids);
 			}
 
 			// return our unique IDs
@@ -220,22 +220,22 @@ 
 	 *
 	 * @return string
 	 */
-	public function gravatars( $form_id = false, $title = '' ) {
+	public function gravatars($form_id = false, $title = '') {
 
 		// unique $payment_ids 
-		$payment_ids = $this->get_payment_ids( $form_id );
+		$payment_ids = $this->get_payment_ids($form_id);
 
 		$give_options = give_get_settings();
 
 		// return if no ID
-		if ( ! $form_id ) {
+		if ( ! $form_id) {
 			return;
 		}
 
 		// minimum amount of donations before showing gravatars
 		// if the number of items in array is not greater or equal to the number specified, then exit
-		if ( isset( $give_options['give_donors_gravatars_min_purchases_required'] ) && '' != $give_options['give_donors_gravatars_min_purchases_required'] ) {
-			if ( ! ( count( $payment_ids ) >= $give_options['give_donors_gravatars_min_purchases_required'] ) ) {
+		if (isset($give_options['give_donors_gravatars_min_purchases_required']) && '' != $give_options['give_donors_gravatars_min_purchases_required']) {
+			if ( ! (count($payment_ids) >= $give_options['give_donors_gravatars_min_purchases_required'])) {
 				return;
 			}
 		}
@@ -246,51 +246,51 @@ 
 		echo '<div id="give-purchase-gravatars">';
 
 
-		if ( isset ( $title ) ) {
+		if (isset ($title)) {
 
-			if ( $title ) {
-				echo apply_filters( 'give_donors_gravatars_title', '<h3 class="give-gravatars-title">' . esc_attr( $title ) . '</h3>' );
-			} elseif ( isset( $give_options['give_donors_gravatars_heading'] ) ) {
-				echo apply_filters( 'give_donors_gravatars_title', '<h3 class="give-gravatars-title">' . esc_attr( $give_options['give_donors_gravatars_heading'] ) . '</h2>' );
+			if ($title) {
+				echo apply_filters('give_donors_gravatars_title', '<h3 class="give-gravatars-title">'.esc_attr($title).'</h3>');
+			} elseif (isset($give_options['give_donors_gravatars_heading'])) {
+				echo apply_filters('give_donors_gravatars_title', '<h3 class="give-gravatars-title">'.esc_attr($give_options['give_donors_gravatars_heading']).'</h2>');
 			}
 
 		}
 		echo '<ul class="give-purchase-gravatars-list">';
 		$i = 0;
 
-		if ( $payment_ids ) {
-			foreach ( $payment_ids as $id ) {
+		if ($payment_ids) {
+			foreach ($payment_ids as $id) {
 
 				// Give saves a blank option even when the control is turned off, hence the extra check
-				if ( isset( $give_options['give_donors_gravatars_maximum_number'] ) && '' != $give_options['give_donors_gravatars_maximum_number'] && $i == $give_options['give_donors_gravatars_maximum_number'] ) {
+				if (isset($give_options['give_donors_gravatars_maximum_number']) && '' != $give_options['give_donors_gravatars_maximum_number'] && $i == $give_options['give_donors_gravatars_maximum_number']) {
 					continue;
 				}
 
 				// get the payment meta
-				$payment_meta = give_get_meta( $id, '_give_payment_meta', true );
+				$payment_meta = give_get_meta($id, '_give_payment_meta', true);
 
 				// unserialize the payment meta
-				$user_info = maybe_unserialize( $payment_meta['user_info'] );
+				$user_info = maybe_unserialize($payment_meta['user_info']);
 
 				// get donor's first name
 				$name = $user_info['first_name'];
 
 				// get donor's email
-				$email = give_get_meta( $id, '_give_payment_user_email', true );
+				$email = give_get_meta($id, '_give_payment_user_email', true);
 
 				// set gravatar size and provide filter
-				$size = isset( $give_options['give_donors_gravatars_gravatar_size'] ) ? apply_filters( 'give_donors_gravatars_gravatar_size', $give_options['give_donors_gravatars_gravatar_size'] ) : '';
+				$size = isset($give_options['give_donors_gravatars_gravatar_size']) ? apply_filters('give_donors_gravatars_gravatar_size', $give_options['give_donors_gravatars_gravatar_size']) : '';
 
 				// default image
-				$default_image = apply_filters( 'give_donors_gravatars_gravatar_default_image', false );
+				$default_image = apply_filters('give_donors_gravatars_gravatar_default_image', false);
 
 				// assemble output
 				$output .= '<li>';
 
-				$output .= get_avatar( $email, $size, $default_image, $name );
+				$output .= get_avatar($email, $size, $default_image, $name);
 				$output .= '</li>';
 
-				$i ++;
+				$i++;
 
 			} // end foreach
 		}
@@ -299,7 +299,7 @@ 
 		echo '</ul>';
 		echo '</div>';
 
-		return apply_filters( 'give_donors_gravatars', ob_get_clean() );
+		return apply_filters('give_donors_gravatars', ob_get_clean());
 	}
 
 	/**
@@ -311,7 +311,7 @@ 
 	 * @return void
 	 */
 	public function register_widget() {
-		register_widget( 'Give_Donors_Gravatars_Widget' );
+		register_widget('Give_Donors_Gravatars_Widget');
 	}
 
 	/**
@@ -325,19 +325,19 @@ 
 	 *
 	 * @return string
 	 */
-	public function shortcode( $atts, $content = null ) {
+	public function shortcode($atts, $content = null) {
 
-		$atts = shortcode_atts( array(
+		$atts = shortcode_atts(array(
 			'id'    => '',
 			'title' => ''
-		), $atts, 'give_donors_gravatars' );
+		), $atts, 'give_donors_gravatars');
 
 		// if no ID is passed on single give_forms pages, get the correct ID
-		if ( is_singular( 'give_forms' ) ) {
+		if (is_singular('give_forms')) {
 			$id = get_the_ID();
 		}
 
-		$content = $this->gravatars( $atts['id'], $atts['title'] );
+		$content = $this->gravatars($atts['id'], $atts['title']);
 
 		return $content;
 
@@ -353,56 +353,56 @@ 
 	 *
 	 * @return array           Gravatar settings.
 	 */
-	public function settings( $settings ) {
+	public function settings($settings) {
 
 		$give_gravatar_settings = array(
 			array(
-				'name' => esc_html__( 'Donator Gravatars', 'give' ),
+				'name' => esc_html__('Donator Gravatars', 'give'),
 				'desc' => '<hr>',
 				'id'   => 'give_title',
 				'type' => 'give_title'
 			),
 			array(
-				'name' => esc_html__( 'Heading', 'give' ),
-				'desc' => esc_html__( 'The heading to display above the Gravatars.', 'give' ),
+				'name' => esc_html__('Heading', 'give'),
+				'desc' => esc_html__('The heading to display above the Gravatars.', 'give'),
 				'type' => 'text',
 				'id'   => 'give_donors_gravatars_heading'
 			),
 			array(
-				'name'    => esc_html__( 'Gravatar Size', 'give' ),
-				'desc'    => esc_html__( 'The size of each Gravatar in pixels (512px maximum).', 'give' ),
+				'name'    => esc_html__('Gravatar Size', 'give'),
+				'desc'    => esc_html__('The size of each Gravatar in pixels (512px maximum).', 'give'),
 				'type'    => 'text_small',
 				'id'      => 'give_donors_gravatars_gravatar_size',
 				'default' => '64'
 			),
 			array(
-				'name' => esc_html__( 'Minimum Unique Donations Required', 'give' ),
-				'desc' => esc_html__( 'The minimum number of unique donations a form must have before the Gravatars are shown. Leave blank for no minimum.', 'give' ),
+				'name' => esc_html__('Minimum Unique Donations Required', 'give'),
+				'desc' => esc_html__('The minimum number of unique donations a form must have before the Gravatars are shown. Leave blank for no minimum.', 'give'),
 				'type' => 'text_small',
 				'id'   => 'give_donors_gravatars_min_purchases_required',
 			),
 			array(
-				'name'    => esc_html__( 'Maximum Gravatars To Show', 'give' ),
-				'desc'    => esc_html__( 'The maximum number of gravatars to show. Leave blank for no limit.', 'give' ),
+				'name'    => esc_html__('Maximum Gravatars To Show', 'give'),
+				'desc'    => esc_html__('The maximum number of gravatars to show. Leave blank for no limit.', 'give'),
 				'type'    => 'text',
 				'id'      => 'give_donors_gravatars_maximum_number',
 				'default' => '20',
 			),
 			array(
-				'name' => esc_html__( 'Gravatar Visibility', 'give' ),
-				'desc' => esc_html__( 'Show only donors with a Gravatar account.', 'give' ),
+				'name' => esc_html__('Gravatar Visibility', 'give'),
+				'desc' => esc_html__('Show only donors with a Gravatar account.', 'give'),
 				'id'   => 'give_donors_gravatars_has_gravatar_account',
 				'type' => 'checkbox',
 			),
 			array(
-				'name' => esc_html__( 'Randomize Gravatars', 'give' ),
-				'desc' => esc_html__( 'Randomize the Gravatars.', 'give' ),
+				'name' => esc_html__('Randomize Gravatars', 'give'),
+				'desc' => esc_html__('Randomize the Gravatars.', 'give'),
 				'id'   => 'give_donors_gravatars_random_gravatars',
 				'type' => 'checkbox',
 			),
 		);
 
-		return array_merge( $settings, $give_gravatar_settings );
+		return array_merge($settings, $give_gravatar_settings);
 	}
 
 }
@@ -428,7 +428,7 @@ 
 		// widget settings
 		$widget_ops = array(
 			'classname'   => 'give-donors-gravatars',
-			'description' => esc_html__( 'Displays gravatars of people who have donated using your your form. Will only show on the single form page.', 'give' ),
+			'description' => esc_html__('Displays gravatars of people who have donated using your your form. Will only show on the single form page.', 'give'),
 		);
 
 		// widget control settings
@@ -441,7 +441,7 @@ 
 		// create the widget
 		parent::__construct(
 			'give_donors_gravatars_widget',
-			esc_html__( 'Give Donors Gravatars', 'give' ),
+			esc_html__('Give Donors Gravatars', 'give'),
 			$widget_ops,
 			$control_ops
 		);
@@ -461,29 +461,29 @@ 
 	 *
 	 * @return void
 	 */
-	public function widget( $args, $instance ) {
+	public function widget($args, $instance) {
 
 		//@TODO: Don't extract it!!!
-		extract( $args );
+		extract($args);
 
-		if ( ! is_singular( 'give_forms' ) ) {
+		if ( ! is_singular('give_forms')) {
 			return;
 		}
 
 		// Variables from widget settings
-		$title = apply_filters( 'widget_title', $instance['title'] );
+		$title = apply_filters('widget_title', $instance['title']);
 
 		// Used by themes. Opens the widget
 		echo $before_widget;
 
 		// Display the widget title
-		if ( $title ) {
-			echo $before_title . $title . $after_title;
+		if ($title) {
+			echo $before_title.$title.$after_title;
 		}
 
 		$gravatars = new Give_Donors_Gravatars();
 
-		echo $gravatars->gravatars( get_the_ID(), null ); // remove title
+		echo $gravatars->gravatars(get_the_ID(), null); // remove title
 
 		// Used by themes. Closes the widget
 		echo $after_widget;
@@ -503,11 +503,11 @@ 
 	 *
 	 * @return array Updated settings to save.
 	 */
-	public function update( $new_instance, $old_instance ) {
+	public function update($new_instance, $old_instance) {
 
 		$instance = $old_instance;
 
-		$instance['title'] = strip_tags( $new_instance['title'] );
+		$instance['title'] = strip_tags($new_instance['title']);
 
 		return $instance;
 
@@ -525,19 +525,19 @@ 
 	 *
 	 * @return void
 	 */
-	public function form( $instance ) {
+	public function form($instance) {
 
 		// Set up some default widget settings.
 		$defaults = array(
 			'title' => '',
 		);
 
-		$instance = wp_parse_args( (array) $instance, $defaults ); ?>
+		$instance = wp_parse_args((array) $instance, $defaults); ?>
 
 		<!-- Title -->
 		<p>
-			<label for="<?php echo $this->get_field_id( 'title' ); ?>"><?php esc_html_e( 'Title:', 'give' ) ?></label>
-			<input class="widefat" id="<?php echo $this->get_field_id( 'title' ); ?>" name="<?php echo $this->get_field_name( 'title' ); ?>" type="text" value="<?php echo $instance['title']; ?>" />
+			<label for="<?php echo $this->get_field_id('title'); ?>"><?php esc_html_e('Title:', 'give') ?></label>
+			<input class="widefat" id="<?php echo $this->get_field_id('title'); ?>" name="<?php echo $this->get_field_name('title'); ?>" type="text" value="<?php echo $instance['title']; ?>" />
 		</p>
 
 		<?php

includes/class-give-donate-form.php

Spacing +119 added lines, -119 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -337,11 +337,11 @@ 
 	 * @param  bool  $_id   Post id. Default is false.
 	 * @param  array $_args Arguments passed.
 	 */
-	public function __construct( $_id = false, $_args = array() ) {
+	public function __construct($_id = false, $_args = array()) {
 
-		$donation_form = WP_Post::get_instance( $_id );
+		$donation_form = WP_Post::get_instance($_id);
 
-		return $this->setup_donation_form( $donation_form );
+		return $this->setup_donation_form($donation_form);
 	}
 
 	/**
@@ -354,23 +354,23 @@ 
 	 *
 	 * @return bool                   If the setup was successful or not.
 	 */
-	private function setup_donation_form( $donation_form ) {
+	private function setup_donation_form($donation_form) {
 
-		if ( ! is_object( $donation_form ) ) {
+		if ( ! is_object($donation_form)) {
 			return false;
 		}
 
-		if ( ! is_a( $donation_form, 'WP_Post' ) ) {
+		if ( ! is_a($donation_form, 'WP_Post')) {
 			return false;
 		}
 
-		if ( 'give_forms' !== $donation_form->post_type ) {
+		if ('give_forms' !== $donation_form->post_type) {
 			return false;
 		}
 
-		foreach ( $donation_form as $key => $value ) {
+		foreach ($donation_form as $key => $value) {
 
-			switch ( $key ) {
+			switch ($key) {
 
 				default:
 					$this->$key = $value;
@@ -394,16 +394,16 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function __get( $key ) {
+	public function __get($key) {
 
-		if ( method_exists( $this, 'get_' . $key ) ) {
+		if (method_exists($this, 'get_'.$key)) {
 
-			return call_user_func( array( $this, 'get_' . $key ) );
+			return call_user_func(array($this, 'get_'.$key));
 
 		} else {
 
 			/* translators: %s: property key */
-			return new WP_Error( 'give-form-invalid-property', sprintf( esc_html__( 'Can\'t get property %s.', 'give' ), $key ) );
+			return new WP_Error('give-form-invalid-property', sprintf(esc_html__('Can\'t get property %s.', 'give'), $key));
 
 		}
 
@@ -419,30 +419,30 @@ 
 	 *
 	 * @return bool|int    False if data isn't passed and class not instantiated for creation, or New Form ID.
 	 */
-	public function create( $data = array() ) {
+	public function create($data = array()) {
 
-		if ( $this->id != 0 ) {
+		if ($this->id != 0) {
 			return false;
 		}
 
 		$defaults = array(
 			'post_type'   => 'give_forms',
 			'post_status' => 'draft',
-			'post_title'  => __( 'New Donation Form', 'give' ),
+			'post_title'  => __('New Donation Form', 'give'),
 		);
 
-		$args = wp_parse_args( $data, $defaults );
+		$args = wp_parse_args($data, $defaults);
 
 		/**
 		 * Fired before a donation form is created
 		 *
 		 * @param array $args The post object arguments used for creation.
 		 */
-		do_action( 'give_form_pre_create', $args );
+		do_action('give_form_pre_create', $args);
 
-		$id = wp_insert_post( $args, true );
+		$id = wp_insert_post($args, true);
 
-		$donation_form = WP_Post::get_instance( $id );
+		$donation_form = WP_Post::get_instance($id);
 
 		/**
 		 * Fired after a donation form is created
@@ -450,9 +450,9 @@ 
 		 * @param int   $id   The post ID of the created item.
 		 * @param array $args The post object arguments used for creation.
 		 */
-		do_action( 'give_form_post_create', $id, $args );
+		do_action('give_form_post_create', $id, $args);
 
-		return $this->setup_donation_form( $donation_form );
+		return $this->setup_donation_form($donation_form);
 
 	}
 
@@ -477,7 +477,7 @@ 
 	 * @return string Donation form name.
 	 */
 	public function get_name() {
-		return get_the_title( $this->ID );
+		return get_the_title($this->ID);
 	}
 
 	/**
@@ -490,13 +490,13 @@ 
 	 */
 	public function get_price() {
 
-		if ( ! isset( $this->price ) ) {
+		if ( ! isset($this->price)) {
 
-			$this->price = give_get_meta( $this->ID, '_give_set_price', true );
+			$this->price = give_get_meta($this->ID, '_give_set_price', true);
 
-			if ( $this->price ) {
+			if ($this->price) {
 
-				$this->price = give_sanitize_amount( $this->price );
+				$this->price = give_sanitize_amount($this->price);
 
 			} else {
 
@@ -514,7 +514,7 @@ 
 		 * @param string     $price The donation form price.
 		 * @param string|int $id    The form ID.
 		 */
-		return apply_filters( 'give_get_set_price', $this->price, $this->ID );
+		return apply_filters('give_get_set_price', $this->price, $this->ID);
 	}
 
 	/**
@@ -527,14 +527,14 @@ 
 	 */
 	public function get_minimum_price() {
 
-		if ( ! isset( $this->minimum_price ) ) {
+		if ( ! isset($this->minimum_price)) {
 
-			$allow_custom_amount = give_get_meta( $this->ID, '_give_custom_amount', true );
-			$this->minimum_price = give_get_meta( $this->ID, '_give_custom_amount_minimum', true );
+			$allow_custom_amount = give_get_meta($this->ID, '_give_custom_amount', true);
+			$this->minimum_price = give_get_meta($this->ID, '_give_custom_amount_minimum', true);
 
-			if ( give_is_setting_enabled( $allow_custom_amount ) && $this->minimum_price ) {
+			if (give_is_setting_enabled($allow_custom_amount) && $this->minimum_price) {
 
-				$this->minimum_price = give_sanitize_amount( $this->minimum_price );
+				$this->minimum_price = give_sanitize_amount($this->minimum_price);
 
 			} else {
 
@@ -544,7 +544,7 @@ 
 
 		}
 
-		return apply_filters( 'give_get_set_minimum_price', $this->minimum_price, $this->ID );
+		return apply_filters('give_get_set_minimum_price', $this->minimum_price, $this->ID);
 	}
 
 	/**
@@ -557,9 +557,9 @@ 
 	 */
 	public function get_prices() {
 
-		if ( ! isset( $this->prices ) ) {
+		if ( ! isset($this->prices)) {
 
-			$this->prices = give_get_meta( $this->ID, '_give_donation_levels', true );
+			$this->prices = give_get_meta($this->ID, '_give_donation_levels', true);
 
 		}
 
@@ -571,7 +571,7 @@ 
 		 * @param array      $prices The array of mulit-level prices.
 		 * @param int|string $ID     The ID of the form.
 		 */
-		return apply_filters( 'give_get_donation_levels', $this->prices, $this->ID );
+		return apply_filters('give_get_donation_levels', $this->prices, $this->ID);
 
 	}
 
@@ -585,13 +585,13 @@ 
 	 */
 	public function get_goal() {
 
-		if ( ! isset( $this->goal ) ) {
+		if ( ! isset($this->goal)) {
 
-			$this->goal = give_get_meta( $this->ID, '_give_set_goal', true );
+			$this->goal = give_get_meta($this->ID, '_give_set_goal', true);
 
-			if ( $this->goal ) {
+			if ($this->goal) {
 
-				$this->goal = give_sanitize_amount( $this->goal );
+				$this->goal = give_sanitize_amount($this->goal);
 
 			} else {
 
@@ -601,7 +601,7 @@ 
 
 		}
 
-		return apply_filters( 'give_get_set_goal', $this->goal, $this->ID );
+		return apply_filters('give_get_set_goal', $this->goal, $this->ID);
 
 	}
 
@@ -615,10 +615,10 @@ 
 	 */
 	public function is_single_price_mode() {
 
-		$option = give_get_meta( $this->ID, '_give_price_option', true );
+		$option = give_get_meta($this->ID, '_give_price_option', true);
 		$ret    = 0;
 
-		if ( empty( $option ) || $option === 'set' ) {
+		if (empty($option) || $option === 'set') {
 			$ret = 1;
 		}
 
@@ -630,7 +630,7 @@ 
 		 * @param bool       $ret Is donation form in single price mode?
 		 * @param int|string $ID The ID of the donation form.
 		 */
-		return (bool) apply_filters( 'give_single_price_option_mode', $ret, $this->ID );
+		return (bool) apply_filters('give_single_price_option_mode', $ret, $this->ID);
 
 	}
 
@@ -644,10 +644,10 @@ 
 	 */
 	public function is_custom_price_mode() {
 
-		$option = give_get_meta( $this->ID, '_give_custom_amount', true );
+		$option = give_get_meta($this->ID, '_give_custom_amount', true);
 		$ret    = 0;
 
-		if ( give_is_setting_enabled( $option ) ) {
+		if (give_is_setting_enabled($option)) {
 			$ret = 1;
 		}
 
@@ -659,7 +659,7 @@ 
 		 * @param bool       $ret Is donation form in custom price mode?
 		 * @param int|string $ID  The ID of the donation form.
 		 */
-		return (bool) apply_filters( 'give_custom_price_option_mode', $ret, $this->ID );
+		return (bool) apply_filters('give_custom_price_option_mode', $ret, $this->ID);
 
 	}
 
@@ -675,10 +675,10 @@ 
 	 */
 	public function has_variable_prices() {
 
-		$option = give_get_meta( $this->ID, '_give_price_option', true );
+		$option = give_get_meta($this->ID, '_give_price_option', true);
 		$ret    = 0;
 
-		if ( $option === 'multi' ) {
+		if ($option === 'multi') {
 			$ret = 1;
 		}
 
@@ -688,7 +688,7 @@ 
 		 * @param bool       $ret Does donation form have variable prices?
 		 * @param int|string $ID  The ID of the donation form.
 		 */
-		return (bool) apply_filters( 'give_has_variable_prices', $ret, $this->ID );
+		return (bool) apply_filters('give_has_variable_prices', $ret, $this->ID);
 
 	}
 
@@ -702,17 +702,17 @@ 
 	 */
 	public function get_type() {
 
-		if ( ! isset( $this->type ) ) {
+		if ( ! isset($this->type)) {
 
-			$this->type = give_get_meta( $this->ID, '_give_price_option', true );
+			$this->type = give_get_meta($this->ID, '_give_price_option', true);
 
-			if ( empty( $this->type ) ) {
+			if (empty($this->type)) {
 				$this->type = 'set';
 			}
 
 		}
 
-		return apply_filters( 'give_get_form_type', $this->type, $this->ID );
+		return apply_filters('give_get_form_type', $this->type, $this->ID);
 
 	}
 
@@ -728,23 +728,23 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_form_classes( $args ) {
+	public function get_form_classes($args) {
 
-		$float_labels_option = give_is_float_labels_enabled( $args )
+		$float_labels_option = give_is_float_labels_enabled($args)
 			? 'float-labels-enabled'
 			: '';
 
-		$form_classes_array = apply_filters( 'give_form_classes', array(
+		$form_classes_array = apply_filters('give_form_classes', array(
 			'give-form',
-			'give-form-' . $this->ID,
-			'give-form-type-' . $this->get_type(),
+			'give-form-'.$this->ID,
+			'give-form-type-'.$this->get_type(),
 			$float_labels_option,
-		), $this->ID, $args );
+		), $this->ID, $args);
 
 		// Remove empty class names.
-		$form_classes_array = array_filter( $form_classes_array );
+		$form_classes_array = array_filter($form_classes_array);
 
-		return implode( ' ', $form_classes_array );
+		return implode(' ', $form_classes_array);
 
 	}
 
@@ -759,24 +759,24 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_form_wrap_classes( $args ) {
+	public function get_form_wrap_classes($args) {
 
-		$display_option = ( isset( $args['display_style'] ) && ! empty( $args['display_style'] ) )
+		$display_option = (isset($args['display_style']) && ! empty($args['display_style']))
 			? $args['display_style']
-			: give_get_meta( $this->ID, '_give_payment_display', true );
+			: give_get_meta($this->ID, '_give_payment_display', true);
 
 		// If admin want to show only button for form then user inbuilt modal functionality.
-		if( 'button' === $display_option ) {
+		if ('button' === $display_option) {
 			$display_option = 'modal give-display-button-only';
 		}
 
-		$form_wrap_classes_array = apply_filters( 'give_form_wrap_classes', array(
+		$form_wrap_classes_array = apply_filters('give_form_wrap_classes', array(
 			'give-form-wrap',
-			'give-display-' . $display_option,
-		), $this->ID, $args );
+			'give-display-'.$display_option,
+		), $this->ID, $args);
 
 
-		return implode( ' ', $form_wrap_classes_array );
+		return implode(' ', $form_wrap_classes_array);
 
 	}
 
@@ -791,7 +791,7 @@ 
 	public function is_set_type_donation_form() {
 		$form_type = $this->get_type();
 
-		return ( 'set' === $form_type ? true : false );
+		return ('set' === $form_type ? true : false);
 	}
 
 	/**
@@ -805,7 +805,7 @@ 
 	public function is_multi_type_donation_form() {
 		$form_type = $this->get_type();
 
-		return ( 'multi' === $form_type ? true : false );
+		return ('multi' === $form_type ? true : false);
 
 	}
 
@@ -819,15 +819,15 @@ 
 	 */
 	public function get_sales() {
 
-		if ( ! isset( $this->sales ) ) {
+		if ( ! isset($this->sales)) {
 
-			if ( '' == give_get_meta( $this->ID, '_give_form_sales', true ) ) {
-				add_post_meta( $this->ID, '_give_form_sales', 0 );
+			if ('' == give_get_meta($this->ID, '_give_form_sales', true)) {
+				add_post_meta($this->ID, '_give_form_sales', 0);
 			} // End if
 
-			$this->sales = give_get_meta( $this->ID, '_give_form_sales', true );
+			$this->sales = give_get_meta($this->ID, '_give_form_sales', true);
 
-			if ( $this->sales < 0 ) {
+			if ($this->sales < 0) {
 				// Never let sales be less than zero
 				$this->sales = 0;
 			}
@@ -848,13 +848,13 @@ 
 	 *
 	 * @return int|false     New number of total sales.
 	 */
-	public function increase_sales( $quantity = 1 ) {
+	public function increase_sales($quantity = 1) {
 
-		$sales       = give_get_form_sales_stats( $this->ID );
-		$quantity    = absint( $quantity );
+		$sales       = give_get_form_sales_stats($this->ID);
+		$quantity    = absint($quantity);
 		$total_sales = $sales + $quantity;
 
-		if ( $this->update_meta( '_give_form_sales', $total_sales ) ) {
+		if ($this->update_meta('_give_form_sales', $total_sales)) {
 
 			$this->sales = $total_sales;
 
@@ -875,17 +875,17 @@ 
 	 *
 	 * @return int|false     New number of total sales.
 	 */
-	public function decrease_sales( $quantity = 1 ) {
+	public function decrease_sales($quantity = 1) {
 
-		$sales = give_get_form_sales_stats( $this->ID );
+		$sales = give_get_form_sales_stats($this->ID);
 
 		// Only decrease if not already zero
-		if ( $sales > 0 ) {
+		if ($sales > 0) {
 
-			$quantity    = absint( $quantity );
+			$quantity    = absint($quantity);
 			$total_sales = $sales - $quantity;
 
-			if ( $this->update_meta( '_give_form_sales', $total_sales ) ) {
+			if ($this->update_meta('_give_form_sales', $total_sales)) {
 
 				$this->sales = $sales;
 
@@ -909,15 +909,15 @@ 
 	 */
 	public function get_earnings() {
 
-		if ( ! isset( $this->earnings ) ) {
+		if ( ! isset($this->earnings)) {
 
-			if ( '' == give_get_meta( $this->ID, '_give_form_earnings', true ) ) {
-				add_post_meta( $this->ID, '_give_form_earnings', 0 );
+			if ('' == give_get_meta($this->ID, '_give_form_earnings', true)) {
+				add_post_meta($this->ID, '_give_form_earnings', 0);
 			}
 
-			$this->earnings = give_get_meta( $this->ID, '_give_form_earnings', true );
+			$this->earnings = give_get_meta($this->ID, '_give_form_earnings', true);
 
-			if ( $this->earnings < 0 ) {
+			if ($this->earnings < 0) {
 				// Never let earnings be less than zero
 				$this->earnings = 0;
 			}
@@ -938,12 +938,12 @@ 
 	 *
 	 * @return float|false
 	 */
-	public function increase_earnings( $amount = 0 ) {
+	public function increase_earnings($amount = 0) {
 
-		$earnings   = give_get_form_earnings_stats( $this->ID );
+		$earnings   = give_get_form_earnings_stats($this->ID);
 		$new_amount = $earnings + (float) $amount;
 
-		if ( $this->update_meta( '_give_form_earnings', $new_amount ) ) {
+		if ($this->update_meta('_give_form_earnings', $new_amount)) {
 
 			$this->earnings = $new_amount;
 
@@ -965,16 +965,16 @@ 
 	 *
 	 * @return float|false
 	 */
-	public function decrease_earnings( $amount ) {
+	public function decrease_earnings($amount) {
 
-		$earnings = give_get_form_earnings_stats( $this->ID );
+		$earnings = give_get_form_earnings_stats($this->ID);
 
-		if ( $earnings > 0 ) {
+		if ($earnings > 0) {
 			// Only decrease if greater than zero
 			$new_amount = $earnings - (float) $amount;
 
 
-			if ( $this->update_meta( '_give_form_earnings', $new_amount ) ) {
+			if ($this->update_meta('_give_form_earnings', $new_amount)) {
 
 				$this->earnings = $new_amount;
 
@@ -998,22 +998,22 @@ 
 	 *
 	 * @return bool
 	 */
-	public function is_free( $price_id = false ) {
+	public function is_free($price_id = false) {
 
 		$is_free          = false;
-		$variable_pricing = give_has_variable_prices( $this->ID );
+		$variable_pricing = give_has_variable_prices($this->ID);
 
-		if ( $variable_pricing && ! is_null( $price_id ) && $price_id !== false ) {
-			$price = give_get_price_option_amount( $this->ID, $price_id );
-		} elseif ( ! $variable_pricing ) {
-			$price = give_get_meta( $this->ID, '_give_set_price', true );
+		if ($variable_pricing && ! is_null($price_id) && $price_id !== false) {
+			$price = give_get_price_option_amount($this->ID, $price_id);
+		} elseif ( ! $variable_pricing) {
+			$price = give_get_meta($this->ID, '_give_set_price', true);
 		}
 
-		if ( isset( $price ) && (float) $price == 0 ) {
+		if (isset($price) && (float) $price == 0) {
 			$is_free = true;
 		}
 
-		return (bool) apply_filters( 'give_is_free_donation', $is_free, $this->ID, $price_id );
+		return (bool) apply_filters('give_is_free_donation', $is_free, $this->ID, $price_id);
 
 	}
 
@@ -1040,9 +1040,9 @@ 
 		$is_close_form = apply_filters(
 			'give_is_close_donation_form',
 			(
-			give_is_setting_enabled( give_get_meta( $this->ID, '_give_goal_option', true ) ) )
-			&& give_is_setting_enabled( give_get_meta( $this->ID, '_give_close_form_when_goal_achieved', true ) )
-			&& ( $this->get_goal() <= $this->get_earnings()
+			give_is_setting_enabled(give_get_meta($this->ID, '_give_goal_option', true)) )
+			&& give_is_setting_enabled(give_get_meta($this->ID, '_give_close_form_when_goal_achieved', true))
+			&& ($this->get_goal() <= $this->get_earnings()
 			),
 			$this->ID
 		);
@@ -1061,29 +1061,29 @@ 
 	 *
 	 * @return bool                            The result of the update query.
 	 */
-	private function update_meta( $meta_key = '', $meta_value = '' ) {
+	private function update_meta($meta_key = '', $meta_value = '') {
 
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
-		if ( empty( $meta_key ) ) {
+		if (empty($meta_key)) {
 			return false;
 		}
 
 		// Make sure if it needs to be serialized, we do
-		$meta_value = maybe_serialize( $meta_value );
+		$meta_value = maybe_serialize($meta_value);
 
-		if ( is_numeric( $meta_value ) ) {
-			$value_type = is_float( $meta_value ) ? '%f' : '%d';
+		if (is_numeric($meta_value)) {
+			$value_type = is_float($meta_value) ? '%f' : '%d';
 		} else {
 			$value_type = "'%s'";
 		}
 
-		$sql = $wpdb->prepare( "UPDATE $wpdb->postmeta SET meta_value = $value_type WHERE post_id = $this->ID AND meta_key = '%s'", $meta_value, $meta_key );
+		$sql = $wpdb->prepare("UPDATE $wpdb->postmeta SET meta_value = $value_type WHERE post_id = $this->ID AND meta_key = '%s'", $meta_value, $meta_key);
 
-		if ( $wpdb->query( $sql ) ) {
+		if ($wpdb->query($sql)) {
 
-			clean_post_cache( $this->ID );
+			clean_post_cache($this->ID);
 
 			return true;
 

includes/emails/template.php

Spacing +104 added lines, -104 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -41,8 +41,8 @@ 
  *
  * @return string $message Fully formatted message
  */
-function give_email_template_tags( $message, $payment_data, $payment_id, $admin_notice = false ) {
-	return give_do_email_tags( $message, $payment_id );
+function give_email_template_tags($message, $payment_data, $payment_id, $admin_notice = false) {
+	return give_do_email_tags($message, $payment_id);
 }
 
 /**
@@ -56,42 +56,42 @@ 
  *
  * @return string $message Fully formatted message
  */
-function give_email_preview_template_tags( $message ) {
+function give_email_preview_template_tags($message) {
 
-	$price = give_currency_filter( give_format_amount( 10.50 ) );
+	$price = give_currency_filter(give_format_amount(10.50));
 
 	$gateway = 'PayPal';
 
-	$receipt_id = strtolower( md5( uniqid() ) );
+	$receipt_id = strtolower(md5(uniqid()));
 
-	$payment_id = rand( 1, 100 );
+	$payment_id = rand(1, 100);
 
-	$receipt_link_url = esc_url( add_query_arg( array( 'payment_key' => $receipt_id, 'give_action' => 'view_receipt' ), home_url() ) );
+	$receipt_link_url = esc_url(add_query_arg(array('payment_key' => $receipt_id, 'give_action' => 'view_receipt'), home_url()));
 	$receipt_link = sprintf(
 		'<a href="%1$s">%2$s</a>',
 		$receipt_link_url,
-		esc_html__( 'View the receipt in your browser &raquo;', 'give' )
+		esc_html__('View the receipt in your browser &raquo;', 'give')
 	);
 
 	$user = wp_get_current_user();
 
-	$message = str_replace( '{name}', $user->display_name, $message );
-	$message = str_replace( '{fullname}', $user->display_name, $message );
-	$message = str_replace( '{username}', $user->user_login, $message );
-	$message = str_replace( '{date}', date( give_date_format(), current_time( 'timestamp' ) ), $message );
-	$message = str_replace( '{amount}', $price, $message );
-	$message = str_replace( '{price}', $price, $message );
-	$message = str_replace( '{donation}', esc_html__( 'Sample Donation Form Title', 'give' ), $message );
-	$message = str_replace( '{form_title}', esc_html__( 'Sample Donation Form Title - Sample Donation Level', 'give' ), $message );
-	$message = str_replace( '{receipt_id}', $receipt_id, $message );
-	$message = str_replace( '{payment_method}', $gateway, $message );
-	$message = str_replace( '{sitename}', get_bloginfo( 'name' ), $message );
-	$message = str_replace( '{payment_id}', $payment_id, $message );
-	$message = str_replace( '{receipt_link}', $receipt_link, $message );
-	$message = str_replace( '{receipt_link_url}', $receipt_link_url, $message );
-	$message = str_replace( '{pdf_receipt}', '<a href="#">Download Receipt</a>', $message );
-
-	return wpautop( apply_filters( 'give_email_preview_template_tags', $message ) );
+	$message = str_replace('{name}', $user->display_name, $message);
+	$message = str_replace('{fullname}', $user->display_name, $message);
+	$message = str_replace('{username}', $user->user_login, $message);
+	$message = str_replace('{date}', date(give_date_format(), current_time('timestamp')), $message);
+	$message = str_replace('{amount}', $price, $message);
+	$message = str_replace('{price}', $price, $message);
+	$message = str_replace('{donation}', esc_html__('Sample Donation Form Title', 'give'), $message);
+	$message = str_replace('{form_title}', esc_html__('Sample Donation Form Title - Sample Donation Level', 'give'), $message);
+	$message = str_replace('{receipt_id}', $receipt_id, $message);
+	$message = str_replace('{payment_method}', $gateway, $message);
+	$message = str_replace('{sitename}', get_bloginfo('name'), $message);
+	$message = str_replace('{payment_id}', $payment_id, $message);
+	$message = str_replace('{receipt_link}', $receipt_link, $message);
+	$message = str_replace('{receipt_link_url}', $receipt_link_url, $message);
+	$message = str_replace('{pdf_receipt}', '<a href="#">Download Receipt</a>', $message);
+
+	return wpautop(apply_filters('give_email_preview_template_tags', $message));
 }
 
 /**
@@ -103,23 +103,23 @@ 
  * @since  1.0
  * @return array|bool
  */
-function give_email_template_preview( $array ) {
+function give_email_template_preview($array) {
 
-	if ( ! current_user_can( 'manage_give_settings' ) ) {
+	if ( ! current_user_can('manage_give_settings')) {
 		return false;
 	}
 	$custom_field = array(
-		'name' => esc_html__( 'Preview Email', 'give' ),
-		'desc' => esc_html__( 'Click the buttons to preview or send test emails.', 'give' ),
+		'name' => esc_html__('Preview Email', 'give'),
+		'desc' => esc_html__('Click the buttons to preview or send test emails.', 'give'),
 		'id'   => 'give_email_preview_buttons',
 		'type' => 'email_preview_buttons'
 	);
 
-	return give_settings_array_insert( $array, 'donation_subject', array( $custom_field ) );
+	return give_settings_array_insert($array, 'donation_subject', array($custom_field));
 
 }
 
-add_filter( 'give_settings_emails', 'give_email_template_preview' );
+add_filter('give_settings_emails', 'give_email_template_preview');
 
 /**
  * Output Email Template Preview Buttons.
@@ -131,12 +131,12 @@ 
 function give_email_preview_buttons_callback() {
 	ob_start();
 	?>
-	<a href="<?php echo esc_url( add_query_arg( array( 'give_action' => 'preview_email' ), home_url() ) ); ?>" class="button-secondary" target="_blank"><?php esc_html_e( 'Preview Donation Receipt', 'give' ); ?></a>
-	<a href="<?php echo wp_nonce_url( add_query_arg( array(
+	<a href="<?php echo esc_url(add_query_arg(array('give_action' => 'preview_email'), home_url())); ?>" class="button-secondary" target="_blank"><?php esc_html_e('Preview Donation Receipt', 'give'); ?></a>
+	<a href="<?php echo wp_nonce_url(add_query_arg(array(
 		'give_action'  => 'send_test_email',
 		'give-message' => 'sent-test-email',
 		'tag'          => 'emails'
-	) ), 'give-test-email' ); ?>" aria-label="<?php esc_attr_e( 'Send demo donation receipt to the emails listed below.', 'give' ); ?>" class="button-secondary"><?php esc_html_e( 'Send Test Email', 'give' ); ?></a>
+	)), 'give-test-email'); ?>" aria-label="<?php esc_attr_e('Send demo donation receipt to the emails listed below.', 'give'); ?>" class="button-secondary"><?php esc_html_e('Send Test Email', 'give'); ?></a>
 	<?php
 	echo ob_get_clean();
 }
@@ -149,46 +149,46 @@ 
  */
 function give_display_email_template_preview() {
 
-	if ( empty( $_GET['give_action'] ) ) {
+	if (empty($_GET['give_action'])) {
 		return;
 	}
 
-	if ( 'preview_email' !== $_GET['give_action'] ) {
+	if ('preview_email' !== $_GET['give_action']) {
 		return;
 	}
 
-	if ( ! current_user_can( 'manage_give_settings' ) ) {
+	if ( ! current_user_can('manage_give_settings')) {
 		return;
 	}
 
 
-	Give()->emails->heading = esc_html__( 'Donation Receipt', 'give' );
+	Give()->emails->heading = esc_html__('Donation Receipt', 'give');
 
-	$payment_id = (int) isset( $_GET['preview_id'] ) ? $_GET['preview_id'] : '';
+	$payment_id = (int) isset($_GET['preview_id']) ? $_GET['preview_id'] : '';
 
 	echo give_get_preview_email_header();
 
 	//Are we previewing an actual payment?
-	if ( ! empty( $payment_id ) ) {
+	if ( ! empty($payment_id)) {
 
-		$content = give_get_email_body_content( $payment_id );
+		$content = give_get_email_body_content($payment_id);
 
-		$preview_content = give_do_email_tags( $content, $payment_id );
+		$preview_content = give_do_email_tags($content, $payment_id);
 
 	} else {
 
 		//No payment ID, use sample preview content
-		$preview_content = give_email_preview_template_tags( give_get_email_body_content( 0, array() ) );
+		$preview_content = give_email_preview_template_tags(give_get_email_body_content(0, array()));
 	}
 
 
-	echo Give()->emails->build_email( $preview_content );
+	echo Give()->emails->build_email($preview_content);
 
 	exit;
 
 }
 
-add_action( 'init', 'give_display_email_template_preview' );
+add_action('init', 'give_display_email_template_preview');
 
 /**
  * Email Template Body.
@@ -200,18 +200,18 @@ 
  *
  * @return string $email_body Body of the email
  */
-function give_get_email_body_content( $payment_id = 0, $payment_data = array() ) {
+function give_get_email_body_content($payment_id = 0, $payment_data = array()) {
 
 	$default_email_body = give_get_default_donation_receipt_email();
 
-	$email_content = give_get_option( 'donation_receipt' );
-	$email_content = isset( $email_content ) ? stripslashes( $email_content ) : $default_email_body;
+	$email_content = give_get_option('donation_receipt');
+	$email_content = isset($email_content) ? stripslashes($email_content) : $default_email_body;
 
-	$email_body = wpautop( $email_content );
+	$email_body = wpautop($email_content);
 
-	$email_body = apply_filters( 'give_donation_receipt_' . Give()->emails->get_template(), $email_body, $payment_id, $payment_data );
+	$email_body = apply_filters('give_donation_receipt_'.Give()->emails->get_template(), $email_body, $payment_id, $payment_data);
 
-	return apply_filters( 'give_donation_receipt', $email_body, $payment_id, $payment_data );
+	return apply_filters('give_donation_receipt', $email_body, $payment_id, $payment_data);
 }
 
 /**
@@ -224,36 +224,36 @@ 
  *
  * @return string $email_body Body of the email
  */
-function give_get_donation_notification_body_content( $payment_id = 0, $payment_data = array() ) {
+function give_get_donation_notification_body_content($payment_id = 0, $payment_data = array()) {
 
-	$user_info = maybe_unserialize( $payment_data['user_info'] );
-	$email     = give_get_payment_user_email( $payment_id );
+	$user_info = maybe_unserialize($payment_data['user_info']);
+	$email     = give_get_payment_user_email($payment_id);
 
-	if ( isset( $user_info['id'] ) && $user_info['id'] > 0 ) {
-		$user_data = get_userdata( $user_info['id'] );
+	if (isset($user_info['id']) && $user_info['id'] > 0) {
+		$user_data = get_userdata($user_info['id']);
 		$name      = $user_data->display_name;
-	} elseif ( isset( $user_info['first_name'] ) && isset( $user_info['last_name'] ) ) {
-		$name = $user_info['first_name'] . ' ' . $user_info['last_name'];
+	} elseif (isset($user_info['first_name']) && isset($user_info['last_name'])) {
+		$name = $user_info['first_name'].' '.$user_info['last_name'];
 	} else {
 		$name = $email;
 	}
 
-	$gateway = give_get_gateway_admin_label( give_get_meta( $payment_id, '_give_payment_gateway', true ) );
+	$gateway = give_get_gateway_admin_label(give_get_meta($payment_id, '_give_payment_gateway', true));
 
-	$default_email_body = esc_html__( 'Hello', 'give' ) . "\n\n";
-	$default_email_body .= esc_html__( 'A donation has been made.', 'give' ) . "\n\n";
-	$default_email_body .= esc_html__( 'Donation:', 'give' ) . "\n\n";
-	$default_email_body .= esc_html__( 'Donor:', 'give' ) . ' ' . html_entity_decode( $name, ENT_COMPAT, 'UTF-8' ) . "\n";
-	$default_email_body .= esc_html__( 'Amount:', 'give' ) . ' ' . html_entity_decode( give_currency_filter( give_format_amount( give_get_payment_amount( $payment_id ) ) ), ENT_COMPAT, 'UTF-8' ) . "\n";
-	$default_email_body .= esc_html__( 'Payment Method:', 'give' ) . ' ' . $gateway . "\n\n";
-	$default_email_body .= esc_html__( 'Thank you', 'give' );
+	$default_email_body = esc_html__('Hello', 'give')."\n\n";
+	$default_email_body .= esc_html__('A donation has been made.', 'give')."\n\n";
+	$default_email_body .= esc_html__('Donation:', 'give')."\n\n";
+	$default_email_body .= esc_html__('Donor:', 'give').' '.html_entity_decode($name, ENT_COMPAT, 'UTF-8')."\n";
+	$default_email_body .= esc_html__('Amount:', 'give').' '.html_entity_decode(give_currency_filter(give_format_amount(give_get_payment_amount($payment_id))), ENT_COMPAT, 'UTF-8')."\n";
+	$default_email_body .= esc_html__('Payment Method:', 'give').' '.$gateway."\n\n";
+	$default_email_body .= esc_html__('Thank you', 'give');
 
-	$email = give_get_option( 'donation_notification' );
-	$email = isset( $email ) ? stripslashes( $email ) : $default_email_body;
+	$email = give_get_option('donation_notification');
+	$email = isset($email) ? stripslashes($email) : $default_email_body;
 
-	$email_body = give_do_email_tags( $email, $payment_id );
+	$email_body = give_do_email_tags($email, $payment_id);
 
-	return apply_filters( 'give_donation_notification', wpautop( $email_body ), $payment_id, $payment_data );
+	return apply_filters('give_donation_notification', wpautop($email_body), $payment_id, $payment_data);
 }
 
 /**
@@ -266,19 +266,19 @@ 
  * @since  1.0
  */
 function give_render_receipt_in_browser() {
-	if ( ! isset( $_GET['payment_key'] ) ) {
-		wp_die( esc_html__( 'Missing donation payment key.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 400 ) );
+	if ( ! isset($_GET['payment_key'])) {
+		wp_die(esc_html__('Missing donation payment key.', 'give'), esc_html__('Error', 'give'), array('response' => 400));
 	}
 
-	$key = urlencode( $_GET['payment_key'] );
+	$key = urlencode($_GET['payment_key']);
 
 	ob_start();
 	//Disallows caching of the page
-	header( "Last-Modified: " . gmdate( "D, d M Y H:i:s" ) . " GMT" );
-	header( "Cache-Control: no-store, no-cache, must-revalidate" ); // HTTP/1.1
-	header( "Cache-Control: post-check=0, pre-check=0", false );
-	header( "Pragma: no-cache" ); // HTTP/1.0
-	header( "Expires: Sat, 23 Oct 1977 05:00:00 PST" ); // Date in the past
+	header("Last-Modified: ".gmdate("D, d M Y H:i:s")." GMT");
+	header("Cache-Control: no-store, no-cache, must-revalidate"); // HTTP/1.1
+	header("Cache-Control: post-check=0, pre-check=0", false);
+	header("Pragma: no-cache"); // HTTP/1.0
+	header("Expires: Sat, 23 Oct 1977 05:00:00 PST"); // Date in the past
 	?>
 	<!DOCTYPE html>
 	<html lang="en">
@@ -289,10 +289,10 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_receipt_head' );
+		do_action('give_receipt_head');
 		?>
 	</head>
-	<body class="<?php echo apply_filters( 'give_receipt_page_body_class', 'give_receipt_page' ); ?>">
+	<body class="<?php echo apply_filters('give_receipt_page_body_class', 'give_receipt_page'); ?>">
 
 	<div id="give_receipt_wrapper">
 		<?php
@@ -301,16 +301,16 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_render_receipt_in_browser_before' );
+		do_action('give_render_receipt_in_browser_before');
 
-		echo do_shortcode( '[give_receipt payment_key=' . $key . ']' );
+		echo do_shortcode('[give_receipt payment_key='.$key.']');
 
 		/**
 		 * Fires in the receipt template after the content.
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_render_receipt_in_browser_after' );
+		do_action('give_render_receipt_in_browser_after');
 		?>
 	</div>
 
@@ -320,7 +320,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_receipt_footer' );
+	do_action('give_receipt_footer');
 	?>
 	</body>
 	</html>
@@ -329,7 +329,7 @@ 
 	die();
 }
 
-add_action( 'give_view_receipt', 'give_render_receipt_in_browser' );
+add_action('give_view_receipt', 'give_render_receipt_in_browser');
 
 
 /**
@@ -344,29 +344,29 @@ 
 
 	//Payment receipt switcher
 	$payment_count = give_count_payments()->publish;
-	$payment_id    = (int) isset( $_GET['preview_id'] ) ? $_GET['preview_id'] : '';
+	$payment_id    = (int) isset($_GET['preview_id']) ? $_GET['preview_id'] : '';
 
-	if ( $payment_count <= 0 ) {
+	if ($payment_count <= 0) {
 		return false;
 	}
 
 	//Get payments.
-	$payments = new Give_Payments_Query( array(
+	$payments = new Give_Payments_Query(array(
 		'number' => 100
-	) );
+	));
 	$payments = $payments->get_payments();
 	$options  = array();
 
 	//Provide nice human readable options.
-	if ( $payments ) {
-		$options[0] = esc_html__( '- Select a donation -', 'give' );
-		foreach ( $payments as $payment ) {
+	if ($payments) {
+		$options[0] = esc_html__('- Select a donation -', 'give');
+		foreach ($payments as $payment) {
 
-			$options[ $payment->ID ] = esc_html( '#' . $payment->ID . ' - ' . $payment->email . ' - ' . $payment->form_title );
+			$options[$payment->ID] = esc_html('#'.$payment->ID.' - '.$payment->email.' - '.$payment->form_title);
 
 		}
 	} else {
-		$options[0] = esc_html__( 'No donations found.', 'give' );
+		$options[0] = esc_html__('No donations found.', 'give');
 	}
 
 	//Start constructing HTML output.
@@ -379,16 +379,16 @@ 
 			        var selected_trans = transactions.options[transactions.selectedIndex];
 				        console.log(selected_trans);
 				        if (selected_trans){
-				            var url_string = "' . get_bloginfo( 'url' ) . '?give_action=preview_email&preview_id=" + selected_trans.value;
+				            var url_string = "' . get_bloginfo('url').'?give_action=preview_email&preview_id=" + selected_trans.value;
 				                window.location = url_string;
 				        }
 				    }
 			    </script>';
 
-	$transaction_header .= '<label for="give_preview_email_payment_id" style="font-size:12px;color:#333;margin:0 4px 0 0;">' . esc_html__( 'Preview email with a donation:', 'give' ) . '</label>';
+	$transaction_header .= '<label for="give_preview_email_payment_id" style="font-size:12px;color:#333;margin:0 4px 0 0;">'.esc_html__('Preview email with a donation:', 'give').'</label>';
 
 	//The select field with 100 latest transactions
-	$transaction_header .= Give()->html->select( array(
+	$transaction_header .= Give()->html->select(array(
 		'name'             => 'preview_email_payment_id',
 		'selected'         => $payment_id,
 		'id'               => 'give_preview_email_payment_id',
@@ -398,12 +398,12 @@ 
 		'select_atts'      => 'onchange="change_preview()">',
 		'show_option_all'  => false,
 		'show_option_none' => false
-	) );
+	));
 
 	//Closing tag
 	$transaction_header .= '</div>';
 
-	return apply_filters( 'give_preview_email_receipt_header', $transaction_header );
+	return apply_filters('give_preview_email_receipt_header', $transaction_header);
 
 }
 
@@ -417,7 +417,7 @@ 
 function give_receipt_head_content() {
 
 	//Title.
-	$output = '<title>' . esc_html__( 'Donation Receipt', 'give' ) . '</title>';
+	$output = '<title>'.esc_html__('Donation Receipt', 'give').'</title>';
 
 	//Meta.
 	$output .= '<meta charset="utf-8"/>
@@ -431,10 +431,10 @@ 
 		<meta name="robots" content="noindex, nofollow"/>';
 
 	//CSS
-	$output .= '<link rel="stylesheet" href="' . give_get_stylesheet_uri() . '?ver=' . GIVE_VERSION . '">';
+	$output .= '<link rel="stylesheet" href="'.give_get_stylesheet_uri().'?ver='.GIVE_VERSION.'">';
 
-	echo apply_filters( 'give_receipt_head_content', $output );
+	echo apply_filters('give_receipt_head_content', $output);
 
 }
 
-add_action( 'give_receipt_head', 'give_receipt_head_content' );
\ No newline at end of file
+add_action('give_receipt_head', 'give_receipt_head_content');
\ No newline at end of file