WordImpress / Give

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

Spacing +27 added lines, -28 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,14 +25,14 @@ 
  */
 function give_do_ajax_export() {
 
-	require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/class-batch-export.php';
+	require_once GIVE_PLUGIN_DIR.'includes/admin/tools/export/class-batch-export.php';
 
-	parse_str( $_POST['form'], $form );
+	parse_str($_POST['form'], $form);
 
 	$_REQUEST = $form = (array) $form;
 
-	if ( ! wp_verify_nonce( $_REQUEST['give_ajax_export'], 'give_ajax_export' ) ) {
-		die( '-2' );
+	if ( ! wp_verify_nonce($_REQUEST['give_ajax_export'], 'give_ajax_export')) {
+		die('-2');
 	}
 
 	/**
@@ -42,28 +42,28 @@ 
 	 *
 	 * @param string $class Export class.
 	 */
-	do_action( 'give_batch_export_class_include', $form['give-export-class'] );
+	do_action('give_batch_export_class_include', $form['give-export-class']);
 
-	$step   = absint( $_POST['step'] );
-	$class  = sanitize_text_field( $form['give-export-class'] );
+	$step   = absint($_POST['step']);
+	$class  = sanitize_text_field($form['give-export-class']);
 
 	/* @var Give_Batch_Export $export */
-	$export = new $class( $step );
+	$export = new $class($step);
 
-	if ( ! $export->can_export() ) {
-		die( '-1' );
+	if ( ! $export->can_export()) {
+		die('-1');
 	}
 
-	if ( ! $export->is_writable ) {
+	if ( ! $export->is_writable) {
 		$json_args = array(
 			'error'   => true,
-			'message' => esc_html__( 'Export location or file not writable.', 'give' )
+			'message' => esc_html__('Export location or file not writable.', 'give')
 		);
 		echo json_encode($json_args);
 		exit;
 	}
 
-	$export->set_properties( give_clean( $_REQUEST ) );
+	$export->set_properties(give_clean($_REQUEST));
 
 	$export->pre_fetch();
 
@@ -71,7 +71,7 @@ 
 
 	$percentage = $export->get_percentage_complete();
 
-	if ( $ret ) {
+	if ($ret) {
 
 		$step += 1;
 		$json_data = array(
@@ -79,18 +79,17 @@ 
 			'percentage' => $percentage
 		);
 
-	} elseif ( true === $export->is_empty ) {
+	} elseif (true === $export->is_empty) {
 
 		$json_data = array(
 			'error'   => true,
-			'message' => esc_html__( 'No data found for export parameters.', 'give' )
+			'message' => esc_html__('No data found for export parameters.', 'give')
 		);
 
-	} elseif ( true === $export->done && true === $export->is_void ) {
+	} elseif (true === $export->done && true === $export->is_void) {
 
-		$message = ! empty( $export->message ) ?
-			$export->message :
-			esc_html__( 'Batch Processing Complete', 'give' );
+		$message = ! empty($export->message) ?
+			$export->message : esc_html__('Batch Processing Complete', 'give');
 
 		$json_data = array(
 			'success' => true,
@@ -99,23 +98,23 @@ 
 
 	} else {
 		
-		$args = array_merge( $_REQUEST, array(
+		$args = array_merge($_REQUEST, array(
 			'step'        => $step,
 			'class'       => $class,
-			'nonce'       => wp_create_nonce( 'give-batch-export' ),
+			'nonce'       => wp_create_nonce('give-batch-export'),
 			'give_action' => 'form_batch_export',
-		) );
+		));
 
 		$json_data = array(
 			'step' => 'done',
-			'url' => add_query_arg( $args, admin_url() )
+			'url' => add_query_arg($args, admin_url())
 		);
 
 	}
 
-	$export->unset_properties( give_clean( $_REQUEST ), $export );
-	echo json_encode( $json_data );
+	$export->unset_properties(give_clean($_REQUEST), $export);
+	echo json_encode($json_data);
 	exit;
 }
 
-add_action( 'wp_ajax_give_do_ajax_export', 'give_do_ajax_export' );
+add_action('wp_ajax_give_do_ajax_export', 'give_do_ajax_export');

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

Indentation +1 added lines, -1 removed lines

@@ -245,7 +245,7 @@
 
 					// Continue if donor already included.
 					if ( empty( $payment->customer_id ) ||
-					     in_array( $payment->customer_id, $cached_donor_ids )
+						 in_array( $payment->customer_id, $cached_donor_ids )
 					) {
 						continue;
 					}

Spacing +103 added lines, -105 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -70,23 +70,23 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 
 		// Set data from form submission
-		if ( isset( $_POST['form'] ) ) {
-			parse_str( $_POST['form'], $this->data );
+		if (isset($_POST['form'])) {
+			parse_str($_POST['form'], $this->data);
 		}
 
 		$this->form = $this->data['forms'];
 
 		// Setup donor ids cache.
-		if ( ! empty( $this->form ) ) {
+		if ( ! empty($this->form)) {
 			// Cache donor ids to output unique list of donor.
-			$this->query_id = give_clean( $_REQUEST['give_export_option']['query_id'] );
+			$this->query_id = give_clean($_REQUEST['give_export_option']['query_id']);
 			$this->cache_donor_ids();
 		}
 
-		$this->price_id = ! empty( $request['give_price_option'] ) && 'all' !== $request['give_price_option'] ? absint( $request['give_price_option'] ) : null;
+		$this->price_id = ! empty($request['give_price_option']) && 'all' !== $request['give_price_option'] ? absint($request['give_price_option']) : null;
 
 	}
 
@@ -101,12 +101,12 @@ 
 		// Fetch already cached donor ids.
 		$donor_ids = $this->donor_ids;
 
-		if ( $cached_donor_ids = Give_Cache::get( $this->query_id, true ) ) {
-			$donor_ids = array_unique( array_merge( $cached_donor_ids, $this->donor_ids ) );
+		if ($cached_donor_ids = Give_Cache::get($this->query_id, true)) {
+			$donor_ids = array_unique(array_merge($cached_donor_ids, $this->donor_ids));
 		}
 
-		$donor_ids = array_values( $donor_ids );
-		Give_Cache::set( $this->query_id, $donor_ids, HOUR_IN_SECONDS, true );
+		$donor_ids = array_values($donor_ids);
+		Give_Cache::set($this->query_id, $donor_ids, HOUR_IN_SECONDS, true);
 	}
 
 	/**
@@ -118,14 +118,14 @@ 
 	 */
 	public function csv_cols() {
 
-		$columns = isset( $this->data['give_export_option'] ) ? $this->data['give_export_option'] : array();
+		$columns = isset($this->data['give_export_option']) ? $this->data['give_export_option'] : array();
 
 		// We need columns.
-		if ( empty( $columns ) ) {
+		if (empty($columns)) {
 			return false;
 		}
 
-		$cols = $this->get_cols( $columns );
+		$cols = $this->get_cols($columns);
 
 		return $cols;
 	}
@@ -137,41 +137,41 @@ 
 	 *
 	 * @return array
 	 */
-	private function get_cols( $columns ) {
+	private function get_cols($columns) {
 
 		$cols = array();
 
-		foreach ( $columns as $key => $value ) {
+		foreach ($columns as $key => $value) {
 
-			switch ( $key ) {
+			switch ($key) {
 				case 'full_name' :
-					$cols['full_name'] = esc_html__( 'Full Name', 'give' );
+					$cols['full_name'] = esc_html__('Full Name', 'give');
 					break;
 				case 'email' :
-					$cols['email'] = esc_html__( 'Email Address', 'give' );
+					$cols['email'] = esc_html__('Email Address', 'give');
 					break;
 				case 'address' :
-					$cols['address_line1']   = esc_html__( 'Address', 'give' );
-					$cols['address_line2']   = esc_html__( 'Address 2', 'give' );
-					$cols['address_city']    = esc_html__( 'City', 'give' );
-					$cols['address_state']   = esc_html__( 'State', 'give' );
-					$cols['address_zip']     = esc_html__( 'Zip', 'give' );
-					$cols['address_country'] = esc_html__( 'Country', 'give' );
+					$cols['address_line1']   = esc_html__('Address', 'give');
+					$cols['address_line2']   = esc_html__('Address 2', 'give');
+					$cols['address_city']    = esc_html__('City', 'give');
+					$cols['address_state']   = esc_html__('State', 'give');
+					$cols['address_zip']     = esc_html__('Zip', 'give');
+					$cols['address_country'] = esc_html__('Country', 'give');
 					break;
 				case 'userid' :
-					$cols['userid'] = esc_html__( 'User ID', 'give' );
+					$cols['userid'] = esc_html__('User ID', 'give');
 					break;
 				case 'donation_form' :
-					$cols['donation_form'] = esc_html__( 'Donation Form', 'give' );
+					$cols['donation_form'] = esc_html__('Donation Form', 'give');
 					break;
 				case 'date_first_donated' :
-					$cols['date_first_donated'] = esc_html__( 'First Donation Date', 'give' );
+					$cols['date_first_donated'] = esc_html__('First Donation Date', 'give');
 					break;
 				case 'donations' :
-					$cols['donations'] = esc_html__( 'Number of Donations', 'give' );
+					$cols['donations'] = esc_html__('Number of Donations', 'give');
 					break;
 				case 'donation_sum' :
-					$cols['donation_sum'] = esc_html__( 'Sum of Donations', 'give' );
+					$cols['donation_sum'] = esc_html__('Sum of Donations', 'give');
 					break;
 			}
 		}
@@ -192,29 +192,29 @@ 
 		$i = 0;
 
 		$data             = array();
-		$cached_donor_ids = Give_Cache::get( $this->query_id, true );
+		$cached_donor_ids = Give_Cache::get($this->query_id, true);
 
-		if ( ! empty( $this->form ) ) {
+		if ( ! empty($this->form)) {
 
 			// Export donors for a specific donation form and also within specified timeframe
 			$args = array(
 				'output'     => 'payments', // Use 'posts' to get standard post objects
-				'post_type'  => array( 'give_payment' ),
+				'post_type'  => array('give_payment'),
 				'number'     => 30,
 				'paged'      => $this->step,
 				'status'     => 'publish',
 				'meta_key'   => '_give_payment_form_id',
-				'meta_value' => absint( $this->form ),
+				'meta_value' => absint($this->form),
 			);
 
 			// Check for date option filter
-			if ( ! empty( $this->data['donor_export_start_date'] ) || ! empty( $this->data['donor_export_end_date'] ) ) {
-				$args['start_date'] = ! empty( $this->data['donor_export_start_date'] ) ? date( 'Y-n-d 00:00:00', strtotime( $this->data['donor_export_start_date'] ) ) : date( 'Y-n-d 23:59:59', '1970-1-01 00:00:00' );
-				$args['end_date']   = ! empty( $this->data['donor_export_end_date'] ) ? date( 'Y-n-d 23:59:59', strtotime( $this->data['donor_export_end_date'] ) ) : date( 'Y-n-d 23:59:59', current_time( 'timestamp' ) );
+			if ( ! empty($this->data['donor_export_start_date']) || ! empty($this->data['donor_export_end_date'])) {
+				$args['start_date'] = ! empty($this->data['donor_export_start_date']) ? date('Y-n-d 00:00:00', strtotime($this->data['donor_export_start_date'])) : date('Y-n-d 23:59:59', '1970-1-01 00:00:00');
+				$args['end_date']   = ! empty($this->data['donor_export_end_date']) ? date('Y-n-d 23:59:59', strtotime($this->data['donor_export_end_date'])) : date('Y-n-d 23:59:59', current_time('timestamp'));
 			}
 
 			// Check for price option.
-			if ( null !== $this->price_id ) {
+			if (null !== $this->price_id) {
 				$args['meta_query'] = array(
 					array(
 						'key'   => '_give_payment_price_id',
@@ -223,45 +223,43 @@ 
 				);
 			}
 
-			$payments_query = new Give_Payments_Query( $args );
+			$payments_query = new Give_Payments_Query($args);
 			$payments       = $payments_query->get_payments();
 
-			if ( $payments ) {
+			if ($payments) {
 				/* @var Give_Payment $payment */
-				foreach ( $payments as $payment ) {
+				foreach ($payments as $payment) {
 					// Set donation sum.
-					$this->payment_stats[ $payment->customer_id ]['donation_sum'] = isset( $this->payment_stats[ $payment->customer_id ]['donation_sum'] ) ?
-						$this->payment_stats[ $payment->customer_id ]['donation_sum'] :
-						0;
-					$this->payment_stats[ $payment->customer_id ]['donation_sum'] += $payment->total;
+					$this->payment_stats[$payment->customer_id]['donation_sum'] = isset($this->payment_stats[$payment->customer_id]['donation_sum']) ?
+						$this->payment_stats[$payment->customer_id]['donation_sum'] : 0;
+					$this->payment_stats[$payment->customer_id]['donation_sum'] += $payment->total;
 
 					// Set donation count.
-					$this->payment_stats[ $payment->customer_id ]['donations'] = isset( $this->payment_stats[ $payment->customer_id ]['donations'] ) ?
-						++ $this->payment_stats[ $payment->customer_id ]['donations'] :
-						1;
+					$this->payment_stats[$payment->customer_id]['donations'] = isset($this->payment_stats[$payment->customer_id]['donations']) ?
+						++$this->payment_stats[$payment->customer_id]['donations'] : 1;
 
 					// Set donation form name.
-					$this->payment_stats[ $payment->customer_id ]['form_title'] = $payment->form_title;
+					$this->payment_stats[$payment->customer_id]['form_title'] = $payment->form_title;
 
 					// Continue if donor already included.
-					if ( empty( $payment->customer_id ) ||
-					     in_array( $payment->customer_id, $cached_donor_ids )
+					if (empty($payment->customer_id) ||
+					     in_array($payment->customer_id, $cached_donor_ids)
 					) {
 						continue;
 					}
 
 					$this->donor_ids[] = $cached_donor_ids[] = $payment->customer_id;
 
-					$i ++;
+					$i++;
 				}
 
-				if ( ! empty( $this->donor_ids ) ) {
-					foreach ( $this->donor_ids as $donor_id ) {
-						$donor                      = Give()->donors->get_donor_by( 'id', $donor_id );
-						$donor->donation_form_title = $this->payment_stats[ $donor_id ]['form_title'];
-						$donor->purchase_count      = $this->payment_stats[ $donor_id ]['donations'];
-						$donor->purchase_value      = $this->payment_stats[ $donor_id ]['donation_sum'];
-						$data[]                     = $this->set_donor_data( $i, $data, $donor );
+				if ( ! empty($this->donor_ids)) {
+					foreach ($this->donor_ids as $donor_id) {
+						$donor                      = Give()->donors->get_donor_by('id', $donor_id);
+						$donor->donation_form_title = $this->payment_stats[$donor_id]['form_title'];
+						$donor->purchase_count      = $this->payment_stats[$donor_id]['donations'];
+						$donor->purchase_value      = $this->payment_stats[$donor_id]['donation_sum'];
+						$data[]                     = $this->set_donor_data($i, $data, $donor);
 					}
 
 					// Cache donor ids only if admin export donor for specific form.
@@ -271,7 +269,7 @@ 
 		} else {
 
 			// Export all donors.
-			$offset = 30 * ( $this->step - 1 );
+			$offset = 30 * ($this->step - 1);
 
 			$args = array(
 				'number' => 30,
@@ -279,31 +277,31 @@ 
 			);
 
 			// Check for date option filter
-			if ( ! empty( $this->data['donor_export_start_date'] ) || ! empty( $this->data['donor_export_end_date'] ) ) {
+			if ( ! empty($this->data['donor_export_start_date']) || ! empty($this->data['donor_export_end_date'])) {
 				$args['date'] = array(
-					'start' => ! empty( $this->data['donor_export_start_date'] ) ? date( 'Y-n-d 00:00:00', strtotime( $this->data['donor_export_start_date'] ) ) : date( 'Y-n-d 23:59:59', '1970-1-01 00:00:00' ),
-					'end'   => ! empty( $this->data['donor_export_end_date'] ) ? date( 'Y-n-d 23:59:59', strtotime( $this->data['donor_export_end_date'] ) ) : date( 'Y-n-d 23:59:59', current_time( 'timestamp' ) ),
+					'start' => ! empty($this->data['donor_export_start_date']) ? date('Y-n-d 00:00:00', strtotime($this->data['donor_export_start_date'])) : date('Y-n-d 23:59:59', '1970-1-01 00:00:00'),
+					'end'   => ! empty($this->data['donor_export_end_date']) ? date('Y-n-d 23:59:59', strtotime($this->data['donor_export_end_date'])) : date('Y-n-d 23:59:59', current_time('timestamp')),
 				);
 			}
 
-			$donors = Give()->donors->get_donors( $args );
+			$donors = Give()->donors->get_donors($args);
 
-			foreach ( $donors as $donor ) {
+			foreach ($donors as $donor) {
 
 				// Continue if donor already included.
-				if ( empty( $donor->id ) || empty( $donor->payment_ids ) ) {
+				if (empty($donor->id) || empty($donor->payment_ids)) {
 					continue;
 				}
 
-				$payment                    = new Give_Payment( $donor->payment_ids );
+				$payment                    = new Give_Payment($donor->payment_ids);
 				$donor->donation_form_title = $payment->form_title;
-				$data[]                     = $this->set_donor_data( $i, $data, $donor );
-				$i ++;
+				$data[]                     = $this->set_donor_data($i, $data, $donor);
+				$i++;
 			}
 		}// End if().
 
-		$data = apply_filters( 'give_export_get_data', $data );
-		$data = apply_filters( "give_export_get_data_{$this->export_type}", $data );
+		$data = apply_filters('give_export_get_data', $data);
+		$data = apply_filters("give_export_get_data_{$this->export_type}", $data);
 
 		return $data;
 	}
@@ -319,18 +317,18 @@ 
 		$percentage = 0;
 
 		// We can't count the number when getting them for a specific form.
-		if ( empty( $this->form ) ) {
+		if (empty($this->form)) {
 
 			$total = Give()->donors->count();
 
-			if ( $total > 0 ) {
+			if ($total > 0) {
 
-				$percentage = ( ( 30 * $this->step ) / $total ) * 100;
+				$percentage = ((30 * $this->step) / $total) * 100;
 
 			}
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -346,51 +344,51 @@ 
 	 *
 	 * @return mixed
 	 */
-	private function set_donor_data( $i, $data, $donor ) {
+	private function set_donor_data($i, $data, $donor) {
 
 		$columns = $this->csv_cols();
 
 		// Set address variable
 		$address = '';
-		if ( isset( $donor->user_id ) && $donor->user_id > 0 ) {
-			$address = give_get_donor_address( $donor->user_id );
+		if (isset($donor->user_id) && $donor->user_id > 0) {
+			$address = give_get_donor_address($donor->user_id);
 		}
 
 		// Set columns
-		if ( ! empty( $columns['full_name'] ) ) {
-			$data[ $i ]['full_name'] = $donor->name;
+		if ( ! empty($columns['full_name'])) {
+			$data[$i]['full_name'] = $donor->name;
 		}
-		if ( ! empty( $columns['email'] ) ) {
-			$data[ $i ]['email'] = $donor->email;
+		if ( ! empty($columns['email'])) {
+			$data[$i]['email'] = $donor->email;
 		}
-		if ( ! empty( $columns['address_line1'] ) ) {
-
-			$data[ $i ]['address_line1']   = isset( $address['line1'] ) ? $address['line1'] : '';
-			$data[ $i ]['address_line2']   = isset( $address['line2'] ) ? $address['line2'] : '';
-			$data[ $i ]['address_city']    = isset( $address['city'] ) ? $address['city'] : '';
-			$data[ $i ]['address_state']   = isset( $address['state'] ) ? $address['state'] : '';
-			$data[ $i ]['address_zip']     = isset( $address['zip'] ) ? $address['zip'] : '';
-			$data[ $i ]['address_country'] = isset( $address['country'] ) ? $address['country'] : '';
+		if ( ! empty($columns['address_line1'])) {
+
+			$data[$i]['address_line1']   = isset($address['line1']) ? $address['line1'] : '';
+			$data[$i]['address_line2']   = isset($address['line2']) ? $address['line2'] : '';
+			$data[$i]['address_city']    = isset($address['city']) ? $address['city'] : '';
+			$data[$i]['address_state']   = isset($address['state']) ? $address['state'] : '';
+			$data[$i]['address_zip']     = isset($address['zip']) ? $address['zip'] : '';
+			$data[$i]['address_country'] = isset($address['country']) ? $address['country'] : '';
 		}
-		if ( ! empty( $columns['userid'] ) ) {
-			$data[ $i ]['userid'] = ! empty( $donor->user_id ) ? $donor->user_id : '';
+		if ( ! empty($columns['userid'])) {
+			$data[$i]['userid'] = ! empty($donor->user_id) ? $donor->user_id : '';
 		}
-		if ( ! empty( $columns['donation_form'] ) ) {
-			$data[ $i ]['donation_form'] = ! empty( $donor->donation_form_title ) ? $donor->donation_form_title : '';
+		if ( ! empty($columns['donation_form'])) {
+			$data[$i]['donation_form'] = ! empty($donor->donation_form_title) ? $donor->donation_form_title : '';
 		}
-		if ( ! empty( $columns['date_first_donated'] ) ) {
-			$data[ $i ]['date_first_donated'] = date_i18n( give_date_format(), strtotime( $donor->date_created ) );
+		if ( ! empty($columns['date_first_donated'])) {
+			$data[$i]['date_first_donated'] = date_i18n(give_date_format(), strtotime($donor->date_created));
 		}
-		if ( ! empty( $columns['donations'] ) ) {
-			$data[ $i ]['donations'] = $donor->purchase_count;
+		if ( ! empty($columns['donations'])) {
+			$data[$i]['donations'] = $donor->purchase_count;
 		}
-		if ( ! empty( $columns['donation_sum'] ) ) {
-			$data[ $i ]['donation_sum'] = give_format_amount( $donor->purchase_value, array( 'sanitize' => false ) );
+		if ( ! empty($columns['donation_sum'])) {
+			$data[$i]['donation_sum'] = give_format_amount($donor->purchase_value, array('sanitize' => false));
 		}
 
-		$data[ $i ] = apply_filters( 'give_export_set_donor_data', $data[ $i ], $donor );
+		$data[$i] = apply_filters('give_export_set_donor_data', $data[$i], $donor);
 
-		return $data[ $i ];
+		return $data[$i];
 
 	}
 
@@ -400,9 +398,9 @@ 
 	 * @param array             $request
 	 * @param Give_Batch_Export $export
 	 */
-	public function unset_properties( $request, $export ) {
-		if ( $export->done ) {
-			Give_Cache::delete( "give_cache_{$this->query_id}" );
+	public function unset_properties($request, $export) {
+		if ($export->done) {
+			Give_Cache::delete("give_cache_{$this->query_id}");
 		}
 	}
 }
\ No newline at end of file

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

Spacing +38 added lines, -38 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -118,14 +118,14 @@ 
 	 *
 	 * @param int $_step
 	 */
-	public function __construct( $_step = 1 ) {
+	public function __construct($_step = 1) {
 
 		$upload_dir     = wp_upload_dir();
 		$this->filetype = '.csv';
-		$this->filename = 'give-' . $this->export_type . $this->filetype;
-		$this->file     = trailingslashit( $upload_dir['basedir'] ) . $this->filename;
+		$this->filename = 'give-'.$this->export_type.$this->filetype;
+		$this->file     = trailingslashit($upload_dir['basedir']).$this->filename;
 
-		if ( ! is_writeable( $upload_dir['basedir'] ) ) {
+		if ( ! is_writeable($upload_dir['basedir'])) {
 			$this->is_writable = false;
 		}
 
@@ -141,22 +141,22 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( esc_html__( 'You do not have permission to export data.', 'give' ), esc_html__( 'Error', 'give' ), array(
+		if ( ! $this->can_export()) {
+			wp_die(esc_html__('You do not have permission to export data.', 'give'), esc_html__('Error', 'give'), array(
 				'response' => 403,
-			) );
+			));
 		}
 
-		if ( $this->step < 2 ) {
+		if ($this->step < 2) {
 
 			// Make sure we start with a fresh file on step 1.
-			@unlink( $this->file );
+			@unlink($this->file);
 			$this->print_csv_cols();
 		}
 
 		$rows = $this->print_csv_rows();
 
-		if ( $rows ) {
+		if ($rows) {
 			return true;
 		} else {
 			return false;
@@ -176,14 +176,14 @@ 
 		$col_data = '';
 		$cols     = $this->get_csv_cols();
 		$i        = 1;
-		foreach ( $cols as $col_id => $column ) {
-			$col_data .= '"' . addslashes( $column ) . '"';
-			$col_data .= $i == count( $cols ) ? '' : ',';
-			$i ++;
+		foreach ($cols as $col_id => $column) {
+			$col_data .= '"'.addslashes($column).'"';
+			$col_data .= $i == count($cols) ? '' : ',';
+			$i++;
 		}
 		$col_data .= "\r\n";
 
-		$this->stash_step_data( $col_data );
+		$this->stash_step_data($col_data);
 
 		return $col_data;
 
@@ -202,23 +202,23 @@ 
 		$data     = $this->get_data();
 		$cols     = $this->get_csv_cols();
 
-		if ( $data ) {
+		if ($data) {
 
 			// Output each row
-			foreach ( $data as $row ) {
+			foreach ($data as $row) {
 				$i = 1;
-				foreach ( $row as $col_id => $column ) {
+				foreach ($row as $col_id => $column) {
 					// Make sure the column is valid
-					if ( array_key_exists( $col_id, $cols ) ) {
-						$row_data .= '"' . addslashes( preg_replace( '/"/', "'", $column ) ) . '"';
-						$row_data .= $i == count( $cols ) ? '' : ',';
-						$i ++;
+					if (array_key_exists($col_id, $cols)) {
+						$row_data .= '"'.addslashes(preg_replace('/"/', "'", $column)).'"';
+						$row_data .= $i == count($cols) ? '' : ',';
+						$i++;
 					}
 				}
 				$row_data .= "\r\n";
 			}
 
-			$this->stash_step_data( $row_data );
+			$this->stash_step_data($row_data);
 
 			return $row_data;
 		}
@@ -246,18 +246,18 @@ 
 
 		$file = '';
 
-		if ( @file_exists( $this->file ) ) {
+		if (@file_exists($this->file)) {
 
-			if ( ! is_writeable( $this->file ) ) {
+			if ( ! is_writeable($this->file)) {
 				$this->is_writable = false;
 			}
 
-			$file = @file_get_contents( $this->file );
+			$file = @file_get_contents($this->file);
 
 		} else {
 
-			@file_put_contents( $this->file, '' );
-			@chmod( $this->file, 0664 );
+			@file_put_contents($this->file, '');
+			@chmod($this->file, 0664);
 
 		}
 
@@ -273,18 +273,18 @@ 
 	 *
 	 * @return void
 	 */
-	protected function stash_step_data( $data = '' ) {
+	protected function stash_step_data($data = '') {
 
 		$file = $this->get_file();
 		$file .= $data;
-		@file_put_contents( $this->file, $file );
+		@file_put_contents($this->file, $file);
 
 		// If we have no rows after this step, mark it as an empty export.
-		$file_rows    = file( $this->file, FILE_SKIP_EMPTY_LINES );
+		$file_rows    = file($this->file, FILE_SKIP_EMPTY_LINES);
 		$default_cols = $this->get_csv_cols();
-		$default_cols = empty( $default_cols ) ? 0 : 1;
+		$default_cols = empty($default_cols) ? 0 : 1;
 
-		$this->is_empty = count( $file_rows ) == $default_cols ? true : false;
+		$this->is_empty = count($file_rows) == $default_cols ? true : false;
 
 	}
 
@@ -302,7 +302,7 @@ 
 
 		$file = $this->get_file();
 
-		@unlink( $this->file );
+		@unlink($this->file);
 
 		echo $file;
 
@@ -311,7 +311,7 @@ 
 		 *
 		 * @since 1.8
 		 */
-		do_action( 'give_file_export_complete', $_REQUEST );
+		do_action('give_file_export_complete', $_REQUEST);
 
 		give_die();
 	}
@@ -323,7 +323,7 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing.
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 	}
 
 	/**
@@ -334,7 +334,7 @@ 
 	 * @param array             $request The Form Data passed into the batch processing.
 	 * @param Give_Batch_Export $export
 	 */
-	public function unset_properties( $request, $export ) {
+	public function unset_properties($request, $export) {
 	}
 
 	/**

includes/admin/tools/logs/class-gateway-error-logs-list-table.php

Spacing +50 added lines, -50 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,11 +44,11 @@ 
 	 */
 	public function __construct() {
 		// 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.
+		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?.
-		) );
+		));
 	}
 
 	/**
@@ -62,19 +62,19 @@ 
 	 *
 	 * @return string Column Name.
 	 */
-	public function column_default( $item, $column_name ) {
+	public function column_default($item, $column_name) {
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 			case 'ID' :
 				return $item['ID_label'];
 			case 'payment_id' :
-				return empty( $item['payment_id'] ) ? esc_html__( 'n/a', 'give' ) : sprintf( "<a href=\"%s\" target=\"_blank\">{$item['payment_id']}</a>", get_edit_post_link( $item['payment_id'] ) );
+				return empty($item['payment_id']) ? esc_html__('n/a', 'give') : sprintf("<a href=\"%s\" target=\"_blank\">{$item['payment_id']}</a>", get_edit_post_link($item['payment_id']));
 			case 'gateway' :
-				return empty( $item['gateway'] ) ? esc_html__( 'n/a', 'give' ) : $item['gateway'];
+				return empty($item['gateway']) ? esc_html__('n/a', 'give') : $item['gateway'];
 			case 'error' :
-				return get_the_title( $item['ID'] ) ? get_the_title( $item['ID'] ) : esc_html__( 'Payment Error', 'give' );
+				return get_the_title($item['ID']) ? get_the_title($item['ID']) : esc_html__('Payment Error', 'give');
 			default:
-				return $item[ $column_name ];
+				return $item[$column_name];
 		}
 	}
 
@@ -88,27 +88,27 @@ 
 	 *
 	 * @return void
 	 */
-	public function column_message( $item ) { ?>
-		<a href="#TB_inline?width=640&amp;inlineId=log-message-<?php echo $item['ID']; ?>" class="thickbox give-error-log-details-link button button-small" data-tooltip="<?php esc_attr_e( 'View Log Message', 'give' ); ?>"><span class="dashicons dashicons-visibility"></span></a>
+	public function column_message($item) { ?>
+		<a href="#TB_inline?width=640&amp;inlineId=log-message-<?php echo $item['ID']; ?>" class="thickbox give-error-log-details-link button button-small" data-tooltip="<?php esc_attr_e('View Log Message', 'give'); ?>"><span class="dashicons dashicons-visibility"></span></a>
 		<div id="log-message-<?php echo $item['ID']; ?>" style="display:none;">
 			<?php
 
-			$log_message = get_post_field( 'post_content', $item['ID'] );
+			$log_message = get_post_field('post_content', $item['ID']);
 
-			$serialized = strpos( $log_message, '{"' );
+			$serialized = strpos($log_message, '{"');
 
 			// Check to see if the log message contains serialized information
-			if ( $serialized !== false ) {
-				$length = strlen( $log_message ) - $serialized;
-				$intro  = substr( $log_message, 0, - $length );
-				$data   = substr( $log_message, $serialized, strlen( $log_message ) - 1 );
+			if ($serialized !== false) {
+				$length = strlen($log_message) - $serialized;
+				$intro  = substr($log_message, 0, - $length);
+				$data   = substr($log_message, $serialized, strlen($log_message) - 1);
 
-				echo wpautop( $intro );
-				echo wpautop( '<strong>' . esc_html__( 'Log data:', 'give' ) . '</strong>' );
-				echo '<div style="word-wrap: break-word;">' . wpautop( $data ) . '</div>';
+				echo wpautop($intro);
+				echo wpautop('<strong>'.esc_html__('Log data:', 'give').'</strong>');
+				echo '<div style="word-wrap: break-word;">'.wpautop($data).'</div>';
 			} else {
 				// No serialized data found
-				echo wpautop( $log_message );
+				echo wpautop($log_message);
 			}
 			?>
 		</div>
@@ -124,12 +124,12 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'ID'         => esc_html__( 'Log ID', 'give' ),
-			'error'      => esc_html__( 'Error', 'give' ),
-			'gateway'    => esc_html__( 'Gateway', 'give' ),
-			'payment_id' => esc_html__( 'Donation ID', 'give' ),
-			'date'       => esc_html__( 'Date', 'give' ),
-			'message'    => esc_html__( 'Details', 'give' )
+			'ID'         => esc_html__('Log ID', 'give'),
+			'error'      => esc_html__('Error', 'give'),
+			'gateway'    => esc_html__('Gateway', 'give'),
+			'payment_id' => esc_html__('Donation ID', 'give'),
+			'date'       => esc_html__('Date', 'give'),
+			'message'    => esc_html__('Details', 'give')
 		);
 
 		return $columns;
@@ -143,7 +143,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;
 	}
 
 	/**
@@ -153,7 +153,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	public function bulk_actions( $which = '' ) {
+	public function bulk_actions($which = '') {
 		give_log_views();
 	}
 
@@ -171,7 +171,7 @@ 
 
 		// Prevent the queries from getting cached.
 		// Without this there are occasional memory issues for some installs.
-		wp_suspend_cache_addition( true );
+		wp_suspend_cache_addition(true);
 
 		$logs_data = array();
 		$paged     = $this->get_paged();
@@ -181,17 +181,17 @@ 
 			'posts_per_page' => $this->per_page,
 		);
 
-		$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,
-					'ID_label'   => '<span class=\'give-item-label give-item-label-gray\'>' . $log->ID . '</span>',
+					'ID_label'   => '<span class=\'give-item-label give-item-label-gray\'>'.$log->ID.'</span>',
 					'payment_id' => $log->post_parent,
 					'error'      => 'error',
-					'gateway'    => give_get_payment_gateway( $log->post_parent ),
+					'gateway'    => give_get_payment_gateway($log->post_parent),
 					'date'       => $log->post_date
 				);
 			}
@@ -213,19 +213,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"/>
@@ -252,14 +252,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, 'gateway_error' );
+		$total_items           = $give_logs->get_log_count(0, 'gateway_error');
 
-		$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/reports/class-donor-reports-table.php

Doc Comments +1 added lines, -1 removed lines

@@ -143,7 +143,7 @@
 	 *
 	 * @access public
 	 * @since  1.0
-	 * @return mixed 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 +67 added lines, -67 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';
 }
 
 /**
@@ -62,11 +62,11 @@ 
 		global $status, $page;
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => esc_html__( 'Donor', 'give' ),     // Singular name of the listed records
-			'plural'   => esc_html__( 'Donors', 'give' ),    // Plural name of the listed records
-			'ajax'     => false,// Does this table support ajax?
-		) );
+		parent::__construct(array(
+			'singular' => esc_html__('Donor', 'give'), // Singular name of the listed records
+			'plural'   => esc_html__('Donors', 'give'), // Plural name of the listed records
+			'ajax'     => false, // Does this table support ajax?
+		));
 
 	}
 
@@ -83,7 +83,7 @@ 
 	 *
 	 * @return false
 	 */
-	public function search_box( $text, $input_id ) {
+	public function search_box($text, $input_id) {
 		return false;
 	}
 
@@ -98,20 +98,20 @@ 
 	 *
 	 * @return void
 	 */
-	public function give_search_box( $text, $input_id ) {
-		$input_id = $input_id . '-search-input';
+	public function give_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 donor-search" 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
 	}
@@ -124,33 +124,33 @@ 
 	 *
 	 * @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 give-clearfix <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav give-clearfix <?php echo esc_attr($which); ?>">
 
-			<?php if ( 'top' === $which ) { ?>
+			<?php if ('top' === $which) { ?>
 				<h2 class="alignleft reports-earnings-title">
-					<?php esc_html_e( 'Donors Report', 'give' ); ?>
+					<?php esc_html_e('Donors Report', 'give'); ?>
 				</h2>
 			<?php } ?>
 
 			<div class="alignright tablenav-right">
 				<div class="actions bulkactions">
 					<?php
-					if ( 'top' === $which ) {
-						$this->give_search_box( esc_html__( 'Search Donors', 'give' ), 'give-donors-report-search' );
+					if ('top' === $which) {
+						$this->give_search_box(esc_html__('Search Donors', 'give'), 'give-donors-report-search');
 					}
 
-					$this->bulk_actions( $which ); ?>
+					$this->bulk_actions($which); ?>
 
 				</div>
 				<?php
-				$this->extra_tablenav( $which );
-				$this->pagination( $which );
+				$this->extra_tablenav($which);
+				$this->pagination($which);
 				?>
 			</div>
 
@@ -172,35 +172,35 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
+	public function column_default($item, $column_name) {
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 
 			case 'name' :
-				$name = '#' . $item['id'] . ' ';
-				$name .= ! empty( $item['name'] ) ? $item['name'] : '<em>' . esc_html__( 'Unnamed Donor', 'give' ) . '</em>';
-				$view_url = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $item['id'] );
-				$value    = '<a href="' . esc_url( $view_url ) . '">' . $name . '</a>';
+				$name = '#'.$item['id'].' ';
+				$name .= ! empty($item['name']) ? $item['name'] : '<em>'.esc_html__('Unnamed Donor', 'give').'</em>';
+				$view_url = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$item['id']);
+				$value    = '<a href="'.esc_url($view_url).'">'.$name.'</a>';
 				break;
 
 			case 'num_donations' :
 				$value = sprintf(
 					'<a href="%s">%s</a>',
-					admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&donor=' . absint( $item['id'] ) ),
-						esc_html( $item['num_donations'] )
+					admin_url('edit.php?post_type=give_forms&page=give-payment-history&donor='.absint($item['id'])),
+						esc_html($item['num_donations'])
 					);
 				break;
 
 			case 'amount_spent' :
-				$value = give_currency_filter( give_format_amount( $item[ $column_name ], array( 'sanitize' => false ) ) );
+				$value = give_currency_filter(give_format_amount($item[$column_name], array('sanitize' => false)));
 				break;
 
 			default:
-				$value = isset( $item[ $column_name ] ) ? $item[ $column_name ] : null;
+				$value = isset($item[$column_name]) ? $item[$column_name] : null;
 				break;
 		}
 
-		return apply_filters( "give_report_column_{$column_name}", $value, $item['id'] );
+		return apply_filters("give_report_column_{$column_name}", $value, $item['id']);
 	}
 
 	/**
@@ -212,13 +212,13 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'name'          => __( 'Name', 'give' ),
-			'email'         => __( 'Email', 'give' ),
-			'num_donations' => __( 'Donations', 'give' ),
-			'amount_spent'  => __( 'Total Donated', 'give' ),
+			'name'          => __('Name', 'give'),
+			'email'         => __('Email', 'give'),
+			'num_donations' => __('Donations', 'give'),
+			'amount_spent'  => __('Total Donated', 'give'),
 		);
 
-		return apply_filters( 'give_report_donor_columns', $columns );
+		return apply_filters('give_report_donor_columns', $columns);
 
 	}
 
@@ -231,10 +231,10 @@ 
 	 */
 	public function get_sortable_columns() {
 		return array(
-			'id'            => array( 'id', true ),
-			'name'          => array( 'name', true ),
-			'num_donations' => array( 'purchase_count', false ),
-			'amount_spent'  => array( 'purchase_value', false ),
+			'id'            => array('id', true),
+			'name'          => array('name', true),
+			'num_donations' => array('purchase_count', false),
+			'amount_spent'  => array('purchase_value', false),
 		);
 	}
 
@@ -245,7 +245,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	public function bulk_actions( $which = '' ) {
+	public function bulk_actions($which = '') {
 
 	}
 
@@ -257,7 +257,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;
 	}
 
 	/**
@@ -268,7 +268,7 @@ 
 	 * @return mixed 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;
 	}
 
 	/**
@@ -287,15 +287,15 @@ 
 
 		// Get donor query.
 		$args   = $this->get_donor_query();
-		$donors = Give()->donors->get_donors( $args );
+		$donors = Give()->donors->get_donors($args);
 
-		if ( $donors ) {
+		if ($donors) {
 
-			$this->count = count( $donors );
+			$this->count = count($donors);
 
-			foreach ( $donors as $donor ) {
+			foreach ($donors as $donor) {
 
-				$user_id = ! empty( $donor->user_id ) ? absint( $donor->user_id ) : 0;
+				$user_id = ! empty($donor->user_id) ? absint($donor->user_id) : 0;
 
 				$data[] = array(
 					'id'            => $donor->id,
@@ -323,9 +323,9 @@ 
 
 		$_donor_query['number'] = -1;
 		$_donor_query['offset'] = 0;
-		$donors = Give()->donors->get_donors( $_donor_query );
+		$donors = Give()->donors->get_donors($_donor_query);
 
-		return count( $donors );
+		return count($donors);
 	}
 
 	/**
@@ -337,10 +337,10 @@ 
 	 */
 	public function get_donor_query() {
 		$paged   = $this->get_paged();
-		$offset  = $this->per_page * ( $paged - 1 );
+		$offset  = $this->per_page * ($paged - 1);
 		$search  = $this->get_search();
-		$order   = isset( $_GET['order'] ) ? sanitize_text_field( $_GET['order'] ) : 'DESC';
-		$orderby = isset( $_GET['orderby'] ) ? sanitize_text_field( $_GET['orderby'] ) : 'id';
+		$order   = isset($_GET['order']) ? sanitize_text_field($_GET['order']) : 'DESC';
+		$orderby = isset($_GET['orderby']) ? sanitize_text_field($_GET['orderby']) : 'id';
 
 		$args = array(
 			'number'  => $this->per_page,
@@ -349,10 +349,10 @@ 
 			'orderby' => $orderby,
 		);
 
-		if( $search ) {
-			if ( is_email( $search ) ) {
+		if ($search) {
+			if (is_email($search)) {
 				$args['email'] = $search;
-			} elseif ( is_numeric( $search ) ) {
+			} elseif (is_numeric($search)) {
 				$args['id'] = $search;
 			} else {
 				$args['name'] = $search;
@@ -379,16 +379,16 @@ 
 		$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->reports_data();
 
 		$this->total = $this->get_donor_count();
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 			'total_items' => $this->total,
 			'per_page'    => $this->per_page,
-			'total_pages' => ceil( $this->total / $this->per_page ),
-		) );
+			'total_pages' => ceil($this->total / $this->per_page),
+		));
 	}
 }

includes/admin/tools/logs/logs.php

Spacing +22 added lines, -22 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -22,7 +22,7 @@ 
  */
 function give_get_logs_tab() {
 
-	require( GIVE_PLUGIN_DIR . 'includes/admin/tools/logs/logs.php' );
+	require(GIVE_PLUGIN_DIR.'includes/admin/tools/logs/logs.php');
 
 	// Get current section.
 	$current_section = $_GET['section'] = give_get_current_setting_section();
@@ -32,7 +32,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( "give_logs_view_{$current_section}" );
+	do_action("give_logs_view_{$current_section}");
 }
 
 
@@ -46,7 +46,7 @@ 
  */
 function give_logs_view_sales() {
 
-	include GIVE_PLUGIN_DIR . 'includes/admin/tools/logs/class-sales-logs-list-table.php';
+	include GIVE_PLUGIN_DIR.'includes/admin/tools/logs/class-sales-logs-list-table.php';
 
 	$logs_table = new Give_Sales_Log_Table();
 	$logs_table->prepare_items();
@@ -59,7 +59,7 @@ 
 		 *
 		 * @since 1.8.12
 		 */
-		do_action( 'give_logs_donations_top' );
+		do_action('give_logs_donations_top');
 
 		$logs_table->display(); ?>
 		<input type="hidden" name="post_type" value="give_forms"/>
@@ -73,14 +73,14 @@ 
 		 *
 		 * @since 1.8.12
 		 */
-		do_action( 'give_logs_donations_bottom' );
+		do_action('give_logs_donations_bottom');
 		?>
 
 	</div>
 	<?php
 }
 
-add_action( 'give_logs_view_sales', 'give_logs_view_sales' );
+add_action('give_logs_view_sales', 'give_logs_view_sales');
 
 
 /**
@@ -92,7 +92,7 @@ 
  * @return void
  */
 function give_logs_view_gateway_errors() {
-	include( GIVE_PLUGIN_DIR . 'includes/admin/tools/logs/class-gateway-error-logs-list-table.php' );
+	include(GIVE_PLUGIN_DIR.'includes/admin/tools/logs/class-gateway-error-logs-list-table.php');
 
 	$logs_table = new Give_Gateway_Error_Log_Table();
 	$logs_table->prepare_items();
@@ -105,7 +105,7 @@ 
 		 *
 		 * @since 1.8.12
 		 */
-		do_action( 'give_logs_payment_error_top' );
+		do_action('give_logs_payment_error_top');
 
 		$logs_table->display(); ?>
 		<input type="hidden" name="post_type" value="give_forms"/>
@@ -119,14 +119,14 @@ 
 		 *
 		 * @since 1.8.12
 		 */
-		do_action( 'give_logs_payment_error_bottom' );
+		do_action('give_logs_payment_error_bottom');
 		?>
 
 	</div>
 	<?php
 }
 
-add_action( 'give_logs_view_gateway_errors', 'give_logs_view_gateway_errors' );
+add_action('give_logs_view_gateway_errors', 'give_logs_view_gateway_errors');
 
 /**
  * API Request Logs
@@ -138,7 +138,7 @@ 
  * @return void
  */
 function give_logs_view_api_requests() {
-	include( GIVE_PLUGIN_DIR . 'includes/admin/tools/logs/class-api-requests-logs-list-table.php' );
+	include(GIVE_PLUGIN_DIR.'includes/admin/tools/logs/class-api-requests-logs-list-table.php');
 
 	$logs_table = new Give_API_Request_Log_Table();
 	$logs_table->prepare_items();
@@ -151,9 +151,9 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_logs_api_requests_top' );
+		do_action('give_logs_api_requests_top');
 
-		$logs_table->search_box( esc_html__( 'Search', 'give' ), 'give-api-requests' );
+		$logs_table->search_box(esc_html__('Search', 'give'), 'give-api-requests');
 		$logs_table->display();
 		?>
 		<input type="hidden" name="post_type" value="give_forms"/>
@@ -167,14 +167,14 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_logs_api_requests_bottom' );
+		do_action('give_logs_api_requests_bottom');
 		?>
 
 	</div>
 	<?php
 }
 
-add_action( 'give_logs_view_api_requests', 'give_logs_view_api_requests' );
+add_action('give_logs_view_api_requests', 'give_logs_view_api_requests');
 
 /**
  * Renders the log views drop down.
@@ -186,11 +186,11 @@ 
 	$current_section = give_get_current_setting_section();
 
 	// If there are not any event attach to action then do not show form.
-	if ( ! has_action( 'give_log_view_actions' ) ) {
+	if ( ! has_action('give_log_view_actions')) {
 		return;
 	}
 	?>
-	<form id="give-logs-filter" method="get" action="<?php echo 'edit.php?post_type=give_forms&page=give-tools&tab=logs&section=' . $current_section; ?>">
+	<form id="give-logs-filter" method="get" action="<?php echo 'edit.php?post_type=give_forms&page=give-tools&tab=logs&section='.$current_section; ?>">
 		<?php
 		/**
 		 * Fires after displaying the reports page views drop down.
@@ -199,14 +199,14 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_log_view_actions' );
+		do_action('give_log_view_actions');
 		?>
 
 		<input type="hidden" name="post_type" value="give_forms"/>
 		<input type="hidden" name="page" value="give-tools"/>
 		<input type="hidden" name="tab" value="logs"/>
 
-		<?php submit_button( esc_html__( 'Apply', 'give' ), 'secondary', 'submit', false ); ?>
+		<?php submit_button(esc_html__('Apply', 'give'), 'secondary', 'submit', false); ?>
 	</form>
 	<?php
 }
@@ -218,7 +218,7 @@ 
  *
  * @return string
  */
-function give_tools_set_form_method( $method ) {
+function give_tools_set_form_method($method) {
 	return 'get';
 }
-add_filter( 'give-tools_form_method_tab_logs', 'give_tools_set_form_method', 10 );
+add_filter('give-tools_form_method_tab_logs', 'give_tools_set_form_method', 10);

give.php

Doc Comments +1 added lines, -1 removed lines

@@ -521,7 +521,7 @@
  * Example: <?php $give = Give(); ?>
  *
  * @since 1.0
- * @return object|Give
+ * @return Give
  */
 function Give() {
 	return Give::instance();

Spacing +151 added lines, -151 removed lines

@@ -40,11 +40,11 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! class_exists( 'Give' ) ) :
+if ( ! class_exists('Give')) :
 
 	/**
 	 * Main Give Class
@@ -202,7 +202,7 @@ 
 		 * @return    Give
 		 */
 		public static function instance() {
-			if ( is_null( self::$_instance ) ) {
+			if (is_null(self::$_instance)) {
 				self::$_instance = new self();
 			}
 
@@ -214,13 +214,13 @@ 
 		 */
 		public function __construct() {
 			// PHP version
-			if ( ! defined( 'GIVE_REQUIRED_PHP_VERSION' ) ) {
-				define( 'GIVE_REQUIRED_PHP_VERSION', '5.3' );
+			if ( ! defined('GIVE_REQUIRED_PHP_VERSION')) {
+				define('GIVE_REQUIRED_PHP_VERSION', '5.3');
 			}
 
 			// Bailout: Need minimum php version to load plugin.
-			if ( function_exists( 'phpversion' ) && version_compare( GIVE_REQUIRED_PHP_VERSION, phpversion(), '>' ) ) {
-				add_action( 'admin_notices', array( $this, 'minmum_phpversion_notice' ) );
+			if (function_exists('phpversion') && version_compare(GIVE_REQUIRED_PHP_VERSION, phpversion(), '>')) {
+				add_action('admin_notices', array($this, 'minmum_phpversion_notice'));
 
 				return;
 			}
@@ -230,7 +230,7 @@ 
 			$this->includes();
 			$this->init_hooks();
 
-			do_action( 'give_loaded' );
+			do_action('give_loaded');
 		}
 
 		/**
@@ -239,8 +239,8 @@ 
 		 * @since  1.8.9
 		 */
 		private function init_hooks() {
-			register_activation_hook( __FILE__, 'give_install' );
-			add_action( 'plugins_loaded', array( $this, 'init' ), 0 );
+			register_activation_hook(__FILE__, 'give_install');
+			add_action('plugins_loaded', array($this, 'init'), 0);
 		}
 		/**
 		 * Init Give when WordPress Initializes.
@@ -254,7 +254,7 @@ 
 			 *
 			 * @since 1.8.9
 			 */
-			do_action( 'before_give_init' );
+			do_action('before_give_init');
 
 			// Set up localization.
 			$this->load_textdomain();
@@ -279,7 +279,7 @@ 
 			 *
 			 * @since 1.8.7
 			 */
-			do_action( 'give_init', $this );
+			do_action('give_init', $this);
 
 		}
 
@@ -296,7 +296,7 @@ 
 		 */
 		public function __clone() {
 			// Cloning instances of the class is forbidden.
-			_doing_it_wrong( __FUNCTION__, __( 'Cheatin’ huh?', 'give' ), '1.0' );
+			_doing_it_wrong(__FUNCTION__, __('Cheatin’ huh?', 'give'), '1.0');
 		}
 
 		/**
@@ -309,7 +309,7 @@ 
 		 */
 		public function __wakeup() {
 			// Unserializing instances of the class is forbidden.
-			_doing_it_wrong( __FUNCTION__, __( 'Cheatin’ huh?', 'give' ), '1.0' );
+			_doing_it_wrong(__FUNCTION__, __('Cheatin’ huh?', 'give'), '1.0');
 		}
 
 		/**
@@ -323,33 +323,33 @@ 
 		private function setup_constants() {
 
 			// Plugin version
-			if ( ! defined( 'GIVE_VERSION' ) ) {
-				define( 'GIVE_VERSION', '1.8.17' );
+			if ( ! defined('GIVE_VERSION')) {
+				define('GIVE_VERSION', '1.8.17');
 			}
 
 			// Plugin Folder Path
-			if ( ! defined( 'GIVE_PLUGIN_DIR' ) ) {
-				define( 'GIVE_PLUGIN_DIR', plugin_dir_path( __FILE__ ) );
+			if ( ! defined('GIVE_PLUGIN_DIR')) {
+				define('GIVE_PLUGIN_DIR', plugin_dir_path(__FILE__));
 			}
 
 			// Plugin Folder URL
-			if ( ! defined( 'GIVE_PLUGIN_URL' ) ) {
-				define( 'GIVE_PLUGIN_URL', plugin_dir_url( __FILE__ ) );
+			if ( ! defined('GIVE_PLUGIN_URL')) {
+				define('GIVE_PLUGIN_URL', plugin_dir_url(__FILE__));
 			}
 
 			// Plugin Basename aka: "give/give.php"
-			if ( ! defined( 'GIVE_PLUGIN_BASENAME' ) ) {
-				define( 'GIVE_PLUGIN_BASENAME', plugin_basename( __FILE__ ) );
+			if ( ! defined('GIVE_PLUGIN_BASENAME')) {
+				define('GIVE_PLUGIN_BASENAME', plugin_basename(__FILE__));
 			}
 
 			// Plugin Root File
-			if ( ! defined( 'GIVE_PLUGIN_FILE' ) ) {
-				define( 'GIVE_PLUGIN_FILE', __FILE__ );
+			if ( ! defined('GIVE_PLUGIN_FILE')) {
+				define('GIVE_PLUGIN_FILE', __FILE__);
 			}
 
 			// Make sure CAL_GREGORIAN is defined
-			if ( ! defined( 'CAL_GREGORIAN' ) ) {
-				define( 'CAL_GREGORIAN', 1 );
+			if ( ! defined('CAL_GREGORIAN')) {
+				define('CAL_GREGORIAN', 1);
 			}
 		}
 
@@ -364,133 +364,133 @@ 
 		private function includes() {
 			global $give_options;
 
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/class-admin-settings.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/class-give-settings.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/class-admin-settings.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/class-give-settings.php';
 			$give_options = give_get_settings();
 
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-cron.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/give-metabox-functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-cache.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/post-types.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/scripts.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/ajax-functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/actions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/filters.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/api/class-give-api.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-notices.php';
-
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-roles.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-template-loader.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-donate-form.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-db.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-db-donors.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-db-donor-meta.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-donor.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-stats.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-session.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-html-elements.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-logging.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-license-handler.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-email-access.php';
-
-			require_once GIVE_PLUGIN_DIR . 'includes/country-functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/template-functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/misc-functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/import-functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/forms/functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/forms/template.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/forms/widget.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/shortcodes.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/formatting.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/currency-functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/price-functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/error-tracking.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/process-donation.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/login-register.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/user-functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/plugin-compatibility.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/deprecated/deprecated-classes.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/deprecated/deprecated-functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/deprecated/deprecated-actions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/deprecated/deprecated-filters.php';
-
-			require_once GIVE_PLUGIN_DIR . 'includes/payments/functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/payments/actions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/payments/class-payment-stats.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/payments/class-payments-query.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/payments/class-give-payment.php';
-
-			require_once GIVE_PLUGIN_DIR . 'includes/gateways/functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/gateways/actions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/gateways/paypal-standard.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/gateways/offline-donations.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/gateways/manual.php';
-
-			require_once GIVE_PLUGIN_DIR . 'includes/emails/class-give-emails.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/emails/class-give-email-tags.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/emails/functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/emails/template.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/emails/actions.php';
-
-			require_once GIVE_PLUGIN_DIR . 'includes/donors/class-give-donors-query.php';
-
-			if ( defined( 'WP_CLI' ) && WP_CLI ) {
-				require_once GIVE_PLUGIN_DIR . 'includes/class-give-cli-commands.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-cron.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/give-metabox-functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-cache.php';
+			require_once GIVE_PLUGIN_DIR.'includes/post-types.php';
+			require_once GIVE_PLUGIN_DIR.'includes/scripts.php';
+			require_once GIVE_PLUGIN_DIR.'includes/ajax-functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/actions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/filters.php';
+			require_once GIVE_PLUGIN_DIR.'includes/api/class-give-api.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-notices.php';
+
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-roles.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-template-loader.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-donate-form.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-db.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-db-donors.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-db-donor-meta.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-donor.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-stats.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-session.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-html-elements.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-logging.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-license-handler.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-email-access.php';
+
+			require_once GIVE_PLUGIN_DIR.'includes/country-functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/template-functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/misc-functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/import-functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/forms/functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/forms/template.php';
+			require_once GIVE_PLUGIN_DIR.'includes/forms/widget.php';
+			require_once GIVE_PLUGIN_DIR.'includes/shortcodes.php';
+			require_once GIVE_PLUGIN_DIR.'includes/formatting.php';
+			require_once GIVE_PLUGIN_DIR.'includes/currency-functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/price-functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/error-tracking.php';
+			require_once GIVE_PLUGIN_DIR.'includes/process-donation.php';
+			require_once GIVE_PLUGIN_DIR.'includes/login-register.php';
+			require_once GIVE_PLUGIN_DIR.'includes/user-functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/plugin-compatibility.php';
+			require_once GIVE_PLUGIN_DIR.'includes/deprecated/deprecated-classes.php';
+			require_once GIVE_PLUGIN_DIR.'includes/deprecated/deprecated-functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/deprecated/deprecated-actions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/deprecated/deprecated-filters.php';
+
+			require_once GIVE_PLUGIN_DIR.'includes/payments/functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/payments/actions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/payments/class-payment-stats.php';
+			require_once GIVE_PLUGIN_DIR.'includes/payments/class-payments-query.php';
+			require_once GIVE_PLUGIN_DIR.'includes/payments/class-give-payment.php';
+
+			require_once GIVE_PLUGIN_DIR.'includes/gateways/functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/gateways/actions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/gateways/paypal-standard.php';
+			require_once GIVE_PLUGIN_DIR.'includes/gateways/offline-donations.php';
+			require_once GIVE_PLUGIN_DIR.'includes/gateways/manual.php';
+
+			require_once GIVE_PLUGIN_DIR.'includes/emails/class-give-emails.php';
+			require_once GIVE_PLUGIN_DIR.'includes/emails/class-give-email-tags.php';
+			require_once GIVE_PLUGIN_DIR.'includes/emails/functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/emails/template.php';
+			require_once GIVE_PLUGIN_DIR.'includes/emails/actions.php';
+
+			require_once GIVE_PLUGIN_DIR.'includes/donors/class-give-donors-query.php';
+
+			if (defined('WP_CLI') && WP_CLI) {
+				require_once GIVE_PLUGIN_DIR.'includes/class-give-cli-commands.php';
 			}
 
-			if ( is_admin() || ( defined( 'WP_CLI' ) && WP_CLI ) ) {
+			if (is_admin() || (defined('WP_CLI') && WP_CLI)) {
 
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/admin-footer.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/welcome.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/admin-pages.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/class-api-keys-table.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/class-i18n-module.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/admin-actions.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/admin-filters.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/add-ons.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/plugins.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/dashboard-widgets.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/class-blank-slate.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/admin-footer.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/welcome.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/admin-pages.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/class-api-keys-table.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/class-i18n-module.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/admin-actions.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/admin-filters.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/add-ons.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/plugins.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/dashboard-widgets.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/class-blank-slate.php';
 
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/payments/actions.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/payments/payments-history.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/payments/actions.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/payments/payments-history.php';
 
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/donors/donors.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/donors/donor-functions.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/donors/donor-actions.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/donors/donors.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/donors/donor-functions.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/donors/donor-actions.php';
 
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/forms/metabox.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/forms/class-metabox-form-data.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/forms/dashboard-columns.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/forms/metabox.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/forms/class-metabox-form-data.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/forms/dashboard-columns.php';
 
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/export-functions.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/class-export.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/export-actions.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/pdf-reports.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/tools/export/export-functions.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/tools/export/class-export.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/tools/export/export-actions.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/tools/export/pdf-reports.php';
 
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/reports/reports.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/reports/class-give-graph.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/reports/graphing.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/reports/reports.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/reports/class-give-graph.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/reports/graphing.php';
 
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/logs/logs.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/tools/logs/logs.php';
 
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/tools-actions.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/tools-actions.php';
 
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/abstract-shortcode-generator.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/class-shortcode-button.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-form.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-goal.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-login.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-register.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-profile-editor.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-donation-history.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-receipt.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/shortcodes/abstract-shortcode-generator.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/shortcodes/class-shortcode-button.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/shortcodes/shortcode-give-form.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/shortcodes/shortcode-give-goal.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/shortcodes/shortcode-give-login.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/shortcodes/shortcode-give-register.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/shortcodes/shortcode-give-profile-editor.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/shortcodes/shortcode-give-donation-history.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/shortcodes/shortcode-give-receipt.php';
 
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/class-give-updates.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/class-give-updates.php';
 
 			}// End if().
 
-			require_once GIVE_PLUGIN_DIR . 'includes/install.php';
+			require_once GIVE_PLUGIN_DIR.'includes/install.php';
 
 		}
 
@@ -505,16 +505,16 @@ 
 		public function load_textdomain() {
 
 			// Set filter for Give's languages directory
-			$give_lang_dir = dirname( plugin_basename( GIVE_PLUGIN_FILE ) ) . '/languages/';
-			$give_lang_dir = apply_filters( 'give_languages_directory', $give_lang_dir );
+			$give_lang_dir = dirname(plugin_basename(GIVE_PLUGIN_FILE)).'/languages/';
+			$give_lang_dir = apply_filters('give_languages_directory', $give_lang_dir);
 
 			// Traditional WordPress plugin locale filter.
-			$locale = is_admin() && function_exists( 'get_user_locale' ) ? get_user_locale() : get_locale();
-			$locale = apply_filters( 'plugin_locale', $locale, 'give' );
+			$locale = is_admin() && function_exists('get_user_locale') ? get_user_locale() : get_locale();
+			$locale = apply_filters('plugin_locale', $locale, 'give');
 
-			unload_textdomain( 'give' );
-			load_textdomain( 'give', WP_LANG_DIR . '/give/give-' . $locale . '.mo' );
-			load_plugin_textdomain( 'give', false, $give_lang_dir );
+			unload_textdomain('give');
+			load_textdomain('give', WP_LANG_DIR.'/give/give-'.$locale.'.mo');
+			load_plugin_textdomain('give', false, $give_lang_dir);
 
 		}
 
@@ -527,17 +527,17 @@ 
 		 */
 		public function minmum_phpversion_notice() {
 			// Bailout.
-			if ( ! is_admin() ) {
+			if ( ! is_admin()) {
 				return;
 			}
 
-			$notice_desc = '<p><strong>' . __( 'Your site could be faster and more secure with a newer PHP version.', 'give' ) . '</strong></p>';
-			$notice_desc .= '<p>' . __( 'Hey, we\'ve noticed that you\'re running an outdated version of PHP. PHP is the programming language that WordPress and Give are built on. The version that is currently used for your site is no longer supported. Newer versions of PHP are both faster and more secure. In fact, your version of PHP no longer receives security updates, which is why we\'re sending you this notice.', 'give' ) . '</p>';
-			$notice_desc .= '<p>' . __( 'Hosts have the ability to update your PHP version, but sometimes they don\'t dare to do that because they\'re afraid they\'ll break your site.', 'give' ) . '</p>';
-			$notice_desc .= '<p><strong>' . __( 'To which version should I update?', 'give' ) . '</strong></p>';
-			$notice_desc .= '<p>' . __( 'You should update your PHP version to either 5.6 or to 7.0 or 7.1. On a normal WordPress site, switching to PHP 5.6 should never cause issues. We would however actually recommend you switch to PHP7. There are some plugins that are not ready for PHP7 though, so do some testing first. PHP7 is much faster than PHP 5.6. It\'s also the only PHP version still in active development and therefore the better option for your site in the long run.', 'give' ) . '</p>';
-			$notice_desc .= '<p><strong>' . __( 'Can\'t update? Ask your host!', 'give' ) . '</strong></p>';
-			$notice_desc .= '<p>' . sprintf( __( 'If you cannot upgrade your PHP version yourself, you can send an email to your host. If they don\'t want to upgrade your PHP version, we would suggest you switch hosts. Have a look at one of the recommended %1$sWordPress hosting partners%2$s.', 'give' ), sprintf( '<a href="%1$s" target="_blank">', esc_url( 'https://wordpress.org/hosting/' ) ), '</a>' ) . '</p>';
+			$notice_desc = '<p><strong>'.__('Your site could be faster and more secure with a newer PHP version.', 'give').'</strong></p>';
+			$notice_desc .= '<p>'.__('Hey, we\'ve noticed that you\'re running an outdated version of PHP. PHP is the programming language that WordPress and Give are built on. The version that is currently used for your site is no longer supported. Newer versions of PHP are both faster and more secure. In fact, your version of PHP no longer receives security updates, which is why we\'re sending you this notice.', 'give').'</p>';
+			$notice_desc .= '<p>'.__('Hosts have the ability to update your PHP version, but sometimes they don\'t dare to do that because they\'re afraid they\'ll break your site.', 'give').'</p>';
+			$notice_desc .= '<p><strong>'.__('To which version should I update?', 'give').'</strong></p>';
+			$notice_desc .= '<p>'.__('You should update your PHP version to either 5.6 or to 7.0 or 7.1. On a normal WordPress site, switching to PHP 5.6 should never cause issues. We would however actually recommend you switch to PHP7. There are some plugins that are not ready for PHP7 though, so do some testing first. PHP7 is much faster than PHP 5.6. It\'s also the only PHP version still in active development and therefore the better option for your site in the long run.', 'give').'</p>';
+			$notice_desc .= '<p><strong>'.__('Can\'t update? Ask your host!', 'give').'</strong></p>';
+			$notice_desc .= '<p>'.sprintf(__('If you cannot upgrade your PHP version yourself, you can send an email to your host. If they don\'t want to upgrade your PHP version, we would suggest you switch hosts. Have a look at one of the recommended %1$sWordPress hosting partners%2$s.', 'give'), sprintf('<a href="%1$s" target="_blank">', esc_url('https://wordpress.org/hosting/')), '</a>').'</p>';
 
 			echo sprintf(
 				'<div class="notice notice-error">%1$s</div>',

includes/admin/tools/data/class-give-tools-delete-test-donors.php

Doc Comments +4 added lines, -1 removed lines

@@ -216,7 +216,7 @@ 
 	 * Return the calculated completion percentage.
 	 *
 	 * @since 1.8.12
-	 * @return int
+	 * @return double
 	 */
 	public function get_percentage_complete() {
 		return ceil( ( 100 * $this->step_completed ) / $this->total_step );
@@ -353,6 +353,9 @@ 
 		return true;
 	}
 
+	/**
+	 * @param integer $page
+	 */
 	public function get_delete_ids( $donation_ids, $page ) {
 		$index            = $page --;
 		$count            = count( $donation_ids );

Spacing +87 added lines, -87 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -109,32 +109,32 @@ 
 		$donor_ids    = array();
 
 		// Check if the ajax request if running for the first time.
-		if ( 1 === (int) $this->step ) {
+		if (1 === (int) $this->step) {
 			// Delete all the donation ids.
-			$this->delete_option( $this->donation_key );
+			$this->delete_option($this->donation_key);
 			// Delete all the donor ids.
-			$this->delete_option( $this->donor_key );
+			$this->delete_option($this->donor_key);
 
 			// Delete all the step and set to 'count' which if the first step in the process of deleting the donors.
-			$this->update_option( $this->step_key, 'count' );
+			$this->update_option($this->step_key, 'count');
 
 			// Delete tha page count of the step.
-			$this->update_option( $this->step_on_key, '0' );
+			$this->update_option($this->step_on_key, '0');
 		} else {
 			// Get the old donors list.
-			$donor_ids = $this->get_option( $this->donor_key );
+			$donor_ids = $this->get_option($this->donor_key);
 
 			// Get the old donation list.
-			$donation_ids = $this->get_option( $this->donation_key );
+			$donation_ids = $this->get_option($this->donation_key);
 		}
 
 		// Get the step and check for it if it's on the first step( 'count' ) or not.
 		$step = (int) $this->get_step();
-		if ( 1 === $step ) {
+		if (1 === $step) {
 			/**
 			 * Will add or update the donation and donor data by running wp query.
 			 */
-			$this->count( $step, $donation_ids, $donor_ids );
+			$this->count($step, $donation_ids, $donor_ids);
 		}
 	}
 
@@ -145,17 +145,17 @@ 
 	 * @param array $donation_ids Contain the list of all the donation id's that has being add before this
 	 * @param array $donor_ids Contain the list of all the donors id's that has being add before this
 	 */
-	private function count( $step, $donation_ids = array(), $donor_ids = array() ) {
+	private function count($step, $donation_ids = array(), $donor_ids = array()) {
 
 		// Get the Page count by default it's zero.
 		$paged = (int) $this->get_step_page();
 		// Incresed the page count by one.
-		++ $paged;
+		++$paged;
 
 		/**
 		 * Filter add to alter the argument before the wp quest run
 		 */
-		$args = apply_filters( 'give_tools_reset_stats_total_args', array(
+		$args = apply_filters('give_tools_reset_stats_total_args', array(
 			'post_type'      => 'give_payment',
 			'post_status'    => 'any',
 			'posts_per_page' => $this->per_step,
@@ -163,16 +163,16 @@ 
 			// ONLY TEST MODE TRANSACTIONS!!!
 			'meta_key'       => '_give_payment_mode',
 			'meta_value'     => 'test',
-		) );
+		));
 
 		// Reset the post data.
 		wp_reset_postdata();
 		// Getting the new donation.
-		$donation_posts = new WP_Query( $args );
+		$donation_posts = new WP_Query($args);
 
 		// The Loop.
-		if ( $donation_posts->have_posts() ) {
-			while ( $donation_posts->have_posts() ) {
+		if ($donation_posts->have_posts()) {
+			while ($donation_posts->have_posts()) {
 				$donation_posts->the_post();
 				global $post;
 				// Add the donation id in side the array.
@@ -191,23 +191,23 @@ 
 		$max_num_pages = (int) $donation_posts->max_num_pages;
 
 		// Check current page is less then max number of page or not
-		if ( $paged < $max_num_pages ) {
+		if ($paged < $max_num_pages) {
 			// Update the curretn page virable for the next step
-			$this->update_option( $this->step_on_key, $paged );
+			$this->update_option($this->step_on_key, $paged);
 
 			// Calculating percentage.
 			$page_remain          = $max_num_pages - $paged;
-			$this->total_step     = (int) $max_num_pages + ( $total_donation / $this->per_step ) + ( ( $page_remain * 2 ) * count( $donor_ids ) );
+			$this->total_step     = (int) $max_num_pages + ($total_donation / $this->per_step) + (($page_remain * 2) * count($donor_ids));
 			$this->step_completed = $paged;
 		} else {
-			$donation_ids_count = count( $donor_ids );
-			$this->update_option( $this->step_key, 'donation' );
-			$this->update_option( $this->step_on_key, '0' );
+			$donation_ids_count = count($donor_ids);
+			$this->update_option($this->step_key, 'donation');
+			$this->update_option($this->step_on_key, '0');
 		}
 
-		$donor_ids = array_unique( $donor_ids );
-		$this->update_option( $this->donor_key, $donor_ids );
-		$this->update_option( $this->donation_key, $donation_ids );
+		$donor_ids = array_unique($donor_ids);
+		$this->update_option($this->donor_key, $donor_ids);
+		$this->update_option($this->donation_key, $donation_ids);
 
 		wp_reset_postdata();
 	}
@@ -219,34 +219,34 @@ 
 	 * @return int
 	 */
 	public function get_percentage_complete() {
-		return ceil( ( 100 * $this->step_completed ) / $this->total_step );
+		return ceil((100 * $this->step_completed) / $this->total_step);
 	}
 
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( __( 'You do not have permission to delete test transactions.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(__('You do not have permission to delete test transactions.', 'give'), __('Error', 'give'), array('response' => 403));
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
-			update_option( 'give_earnings_total', give_get_total_earnings( true ) );
-			Give_Cache::delete( Give_Cache::get_key( 'give_estimated_monthly_stats' ) );
+			update_option('give_earnings_total', give_get_total_earnings(true));
+			Give_Cache::delete(Give_Cache::get_key('give_estimated_monthly_stats'));
 
-			$this->delete_option( $this->donation_key );
+			$this->delete_option($this->donation_key);
 
 			// Reset the sequential order numbers
-			if ( give_get_option( 'enable_sequential' ) ) {
-				delete_option( 'give_last_payment_number' );
+			if (give_get_option('enable_sequential')) {
+				delete_option('give_last_payment_number');
 			}
 
 			$this->done    = true;
-			$this->message = __( 'Test donor and transactions successfully deleted.', 'give' );
+			$this->message = __('Test donor and transactions successfully deleted.', 'give');
 
 			return false;
 		}
@@ -264,12 +264,12 @@ 
 	public function get_data() {
 
 		// Get the donation id's.
-		$donation_ids = $this->get_option( $this->donation_key );
+		$donation_ids = $this->get_option($this->donation_key);
 
 		/**
 		 * Return false id not test donation is found.
 		 */
-		if ( empty( $donation_ids ) ) {
+		if (empty($donation_ids)) {
 			$this->is_empty   = true;
 			$this->total_step = 1;
 
@@ -280,70 +280,70 @@ 
 		$step = (int) $this->get_step();
 
 		// get teh donor ids.
-		$donor_ids = $this->get_option( $this->donor_key );
+		$donor_ids = $this->get_option($this->donor_key);
 
 		// In step to we delete all the donation in loop.
-		if ( 2 === $step ) {
+		if (2 === $step) {
 			$pass_to_donor = false;
 			$page          = (int) $this->get_step_page();
-			$page ++;
-			$count = count( $donation_ids );
+			$page++;
+			$count = count($donation_ids);
 
-			$this->total_step     = ( ( count( $donation_ids ) / $this->per_step ) * 2 ) + count( $donor_ids );
+			$this->total_step     = ((count($donation_ids) / $this->per_step) * 2) + count($donor_ids);
 			$this->step_completed = $page;
 
 
-			if ( $count > $this->per_step ) {
+			if ($count > $this->per_step) {
 
-				$this->update_option( $this->step_on_key, $page );
-				$donation_ids = $this->get_delete_ids( $donation_ids, $page );
-				$current_page = (int) ceil( $count / $this->per_step );
+				$this->update_option($this->step_on_key, $page);
+				$donation_ids = $this->get_delete_ids($donation_ids, $page);
+				$current_page = (int) ceil($count / $this->per_step);
 
-				if ( $page === $current_page ) {
+				if ($page === $current_page) {
 					$pass_to_donor = true;
 				}
 			} else {
 				$pass_to_donor = true;
 			}
 
-			if ( true === $pass_to_donor ) {
-				$this->update_option( $this->step_key, 'donor' );
-				$this->update_option( $this->step_on_key, '0' );
+			if (true === $pass_to_donor) {
+				$this->update_option($this->step_key, 'donor');
+				$this->update_option($this->step_on_key, '0');
 			}
 
-			foreach ( $donation_ids as $item ) {
+			foreach ($donation_ids as $item) {
 				// Delete the main payment.
-				give_delete_donation( absint( $item ) );
+				give_delete_donation(absint($item));
 			}
-			do_action( 'give_delete_log_cache' );
+			do_action('give_delete_log_cache');
 		}
 
 
 		// Here we delete all the donor
-		if ( 3 === $step ) {
+		if (3 === $step) {
 			$page  = (int) $this->get_step_page();
-			$count = count( $donor_ids );
+			$count = count($donor_ids);
 
-			$this->total_step     = ( ( count( $donation_ids ) / $this->per_step ) * 2 ) + count( $donor_ids );
-			$this->step_completed = $page + ( count( $donation_ids ) / $this->per_step );
+			$this->total_step     = ((count($donation_ids) / $this->per_step) * 2) + count($donor_ids);
+			$this->step_completed = $page + (count($donation_ids) / $this->per_step);
 
-			$args = apply_filters( 'give_tools_reset_stats_total_args', array(
+			$args = apply_filters('give_tools_reset_stats_total_args', array(
 				'post_type'      => 'give_payment',
 				'post_status'    => 'any',
 				'posts_per_page' => 1,
 				'meta_key'       => '_give_payment_mode',
 				'meta_value'     => 'live',
-				'author'         => $donor_ids[ $page ],
-			) );
+				'author'         => $donor_ids[$page],
+			));
 
-			$donation_posts = get_posts( $args );
-			if ( empty( $donation_posts ) ) {
-				Give()->donors->delete_by_user_id( $donor_ids[ $page ] );
+			$donation_posts = get_posts($args);
+			if (empty($donation_posts)) {
+				Give()->donors->delete_by_user_id($donor_ids[$page]);
 			}
 
-			$page ++;
-			$this->update_option( $this->step_on_key, $page );
-			if ( $count === $page ) {
+			$page++;
+			$this->update_option($this->step_on_key, $page);
+			if ($count === $page) {
 				$this->is_empty = false;
 
 				return false;
@@ -355,24 +355,24 @@ 
 		return true;
 	}
 
-	public function get_delete_ids( $donation_ids, $page ) {
-		$index            = $page --;
-		$count            = count( $donation_ids );
+	public function get_delete_ids($donation_ids, $page) {
+		$index            = $page--;
+		$count            = count($donation_ids);
 		$temp             = 0;
 		$current_page     = 0;
 		$post_delete      = $this->per_step;
 		$page_donation_id = array();
 
-		foreach ( $donation_ids as $item ) {
-			$temp ++;
-			$page_donation_id[ $current_page ][] = $item;
-			if ( $temp === $post_delete ) {
-				$current_page ++;
+		foreach ($donation_ids as $item) {
+			$temp++;
+			$page_donation_id[$current_page][] = $item;
+			if ($temp === $post_delete) {
+				$current_page++;
 				$temp = 0;
 			}
 		}
 
-		return $page_donation_id[ $page ];
+		return $page_donation_id[$page];
 	}
 
 	/**
@@ -384,8 +384,8 @@ 
 	 *
 	 * @return mixed       Returns the data from the database
 	 */
-	public function get_option( $key, $defalut_value = false ) {
-		return get_option( $key, $defalut_value );
+	public function get_option($key, $defalut_value = false) {
+		return get_option($key, $defalut_value);
 	}
 
 	/**
@@ -398,8 +398,8 @@ 
 	 *
 	 * @return void
 	 */
-	public function update_option( $key, $value ) {
-		update_option( $key, $value, false );
+	public function update_option($key, $value) {
+		update_option($key, $value, false);
 	}
 
 	/**
@@ -411,8 +411,8 @@ 
 	 *
 	 * @return void
 	 */
-	public function delete_option( $key ) {
-		delete_option( $key );
+	public function delete_option($key) {
+		delete_option($key);
 	}
 
 	/**
@@ -423,12 +423,12 @@ 
 	 * @return int|string
 	 */
 	private function get_step() {
-		$step_key = (string) $this->get_option( $this->step_key, false );
-		if ( 'count' === $step_key ) {
+		$step_key = (string) $this->get_option($this->step_key, false);
+		if ('count' === $step_key) {
 			return 1;
-		} elseif ( 'donation' === $step_key ) {
+		} elseif ('donation' === $step_key) {
 			return 2;
-		} elseif ( 'donor' === $step_key ) {
+		} elseif ('donor' === $step_key) {
 			return 3;
 		} else {
 			return $step_key;
@@ -439,6 +439,6 @@ 
 	 * Get the current $page value in the ajax.
 	 */
 	private function get_step_page() {
-		return $this->get_option( $this->step_on_key, false );
+		return $this->get_option($this->step_on_key, false);
 	}
 }

includes/formatting.php

Doc Comments +1 added lines, -1 removed lines

@@ -340,7 +340,7 @@
  * @since 1.0
  *
  * @param int|float|string $amount   Formatted or sanitized price
- * @param int|bool         $dp       number of decimals
+ * @param boolean         $dp       number of decimals
  * @param bool             $sanitize Whether or not sanitize number
  *
  * @return string $amount Newly formatted amount or Price Not Available

Spacing +171 added lines, -175 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,37 +23,37 @@ 
  *
  * @return mixed
  */
-function give_get_currency_formatting_settings( $id_or_currency_code = null ) {
+function give_get_currency_formatting_settings($id_or_currency_code = null) {
 	$give_options = give_get_settings();
 	$setting      = array();
 
-	if ( ! empty( $id_or_currency_code ) ) {
-		$currencies = give_get_currencies( 'all' );
+	if ( ! empty($id_or_currency_code)) {
+		$currencies = give_get_currencies('all');
 
 		// Set default formatting setting only if currency not set as global currency.
 		if (
-			is_string( $id_or_currency_code ) &&
-			! empty( $give_options['currency'] ) &&
+			is_string($id_or_currency_code) &&
+			! empty($give_options['currency']) &&
 			$id_or_currency_code !== $give_options['currency'] &&
-			array_key_exists( $id_or_currency_code, $currencies )
+			array_key_exists($id_or_currency_code, $currencies)
 		) {
-			$setting = $currencies[ $id_or_currency_code ]['setting'];
-		} elseif ( is_numeric( $id_or_currency_code ) && 'give_payment' === get_post_type( $id_or_currency_code ) ) {
-			$donation_meta = give_get_meta( $id_or_currency_code, '_give_payment_meta', true );
+			$setting = $currencies[$id_or_currency_code]['setting'];
+		} elseif (is_numeric($id_or_currency_code) && 'give_payment' === get_post_type($id_or_currency_code)) {
+			$donation_meta = give_get_meta($id_or_currency_code, '_give_payment_meta', true);
 
 			if (
-				! empty( $donation_meta['currency'] ) &&
+				! empty($donation_meta['currency']) &&
 				$give_options['currency'] !== $donation_meta['currency']
 			) {
-				$setting = $currencies[ $donation_meta['currency'] ]['setting'];
+				$setting = $currencies[$donation_meta['currency']]['setting'];
 			}
 		}
 	}
 
-	if ( empty( $setting ) ) {
+	if (empty($setting)) {
 		// Set thousand separator.
-		$thousand_separator = isset( $give_options['thousands_separator'] ) ? $give_options['thousands_separator'] : ',';
-		$thousand_separator = empty( $thousand_separator ) ? ' ' : $thousand_separator;
+		$thousand_separator = isset($give_options['thousands_separator']) ? $give_options['thousands_separator'] : ',';
+		$thousand_separator = empty($thousand_separator) ? ' ' : $thousand_separator;
 
 		// Set decimal separator.
 		$default_decimal_separators = array(
@@ -61,17 +61,16 @@ 
 			',' => '.',
 		);
 
-		$default_decimal_separator = in_array( $thousand_separator, $default_decimal_separators ) ?
-			$default_decimal_separators[ $thousand_separator ] :
-			'.';
+		$default_decimal_separator = in_array($thousand_separator, $default_decimal_separators) ?
+			$default_decimal_separators[$thousand_separator] : '.';
 
-		$decimal_separator = ! empty( $give_options['decimal_separator'] ) ? $give_options['decimal_separator'] : $default_decimal_separator;
+		$decimal_separator = ! empty($give_options['decimal_separator']) ? $give_options['decimal_separator'] : $default_decimal_separator;
 
 		$setting = array(
-			'currency_position'   => give_get_option( 'currency_position', 'before' ),
+			'currency_position'   => give_get_option('currency_position', 'before'),
 			'thousands_separator' => $thousand_separator,
 			'decimal_separator'   => $decimal_separator,
-			'number_decimals'     => give_get_option( 'number_decimals', 0 ),
+			'number_decimals'     => give_get_option('number_decimals', 0),
 		);
 	}
 
@@ -80,7 +79,7 @@ 
 	 *
 	 * @since 1.8.15
 	 */
-	return apply_filters( 'give_get_currency_formatting_settings', $setting, $id_or_currency_code );
+	return apply_filters('give_get_currency_formatting_settings', $setting, $id_or_currency_code);
 }
 
 /**
@@ -92,16 +91,16 @@ 
  *
  * @return mixed
  */
-function give_get_price_decimals( $id_or_currency_code = null ) {
+function give_get_price_decimals($id_or_currency_code = null) {
 	// Set currency on basis of donation id.
-	if ( empty( $id_or_currency_code ) ) {
+	if (empty($id_or_currency_code)) {
 		$id_or_currency_code = give_get_currency();
 	}
 
 	$number_of_decimals = 0;
 
-	if ( ! give_is_zero_based_currency( $id_or_currency_code ) ) {
-		$setting            = give_get_currency_formatting_settings( $id_or_currency_code );
+	if ( ! give_is_zero_based_currency($id_or_currency_code)) {
+		$setting            = give_get_currency_formatting_settings($id_or_currency_code);
 		$number_of_decimals = $setting['number_decimals'];
 	}
 
@@ -110,7 +109,7 @@ 
 	 *
 	 * @since 1.6
 	 */
-	return apply_filters( 'give_sanitize_amount_decimals', $number_of_decimals, $id_or_currency_code );
+	return apply_filters('give_sanitize_amount_decimals', $number_of_decimals, $id_or_currency_code);
 }
 
 /**
@@ -122,15 +121,15 @@ 
  *
  * @return mixed
  */
-function give_get_price_thousand_separator( $id_or_currency_code = null ) {
-	$setting = give_get_currency_formatting_settings( $id_or_currency_code );
+function give_get_price_thousand_separator($id_or_currency_code = null) {
+	$setting = give_get_currency_formatting_settings($id_or_currency_code);
 
 	/**
 	 * Filter the thousand separator
 	 *
 	 * @since 1.6
 	 */
-	return apply_filters( 'give_get_price_thousand_separator', $setting['thousands_separator'], $id_or_currency_code );
+	return apply_filters('give_get_price_thousand_separator', $setting['thousands_separator'], $id_or_currency_code);
 }
 
 /**
@@ -142,15 +141,15 @@ 
  *
  * @return mixed
  */
-function give_get_price_decimal_separator( $id_or_currency_code = null ) {
-	$setting = give_get_currency_formatting_settings( $id_or_currency_code );
+function give_get_price_decimal_separator($id_or_currency_code = null) {
+	$setting = give_get_currency_formatting_settings($id_or_currency_code);
 
 	/**
 	 * Filter the thousand separator
 	 *
 	 * @since 1.6
 	 */
-	return apply_filters( 'give_get_price_decimal_separator', $setting['decimal_separator'], $id_or_currency_code );
+	return apply_filters('give_get_price_decimal_separator', $setting['decimal_separator'], $id_or_currency_code);
 }
 
 
@@ -163,8 +162,8 @@ 
  *
  * @return string $amount Newly sanitized amount
  */
-function give_sanitize_amount_for_db( $number ) {
-	return give_maybe_sanitize_amount( $number, array( 'number_decimals' => 6 ) );
+function give_sanitize_amount_for_db($number) {
+	return give_maybe_sanitize_amount($number, array('number_decimals' => 6));
 }
 
 /**
@@ -177,19 +176,19 @@ 
  *
  * @return string $amount Newly sanitized amount
  */
-function give_maybe_sanitize_amount( $number, $args = array() ) {
+function give_maybe_sanitize_amount($number, $args = array()) {
 	// Bailout.
-	if ( empty( $number ) || ( ! is_numeric( $number ) && ! is_string( $number ) ) ) {
+	if (empty($number) || ( ! is_numeric($number) && ! is_string($number))) {
 		return $number;
 	}
 
 	$func_args = func_get_args();
 
 	// Backward compatibility.
-	if ( isset( $func_args[1] ) && ( is_bool( $func_args[1] ) || is_numeric( $func_args[1] ) ) ) {
+	if (isset($func_args[1]) && (is_bool($func_args[1]) || is_numeric($func_args[1]))) {
 		$args = array(
 			'number_decimals' => $func_args[1],
-			'trim_zeros'      => isset( $func_args[2] ) ? $func_args[2] : false,
+			'trim_zeros'      => isset($func_args[2]) ? $func_args[2] : false,
 		);
 	}
 
@@ -202,23 +201,22 @@ 
 		)
 	);
 
-	$thousand_separator = give_get_price_thousand_separator( $args['currency'] );
-	$decimal_separator  = give_get_price_decimal_separator( $args['currency'] );
-	$number_decimals    = is_bool( $args['number_decimals'] ) ?
-		give_get_price_decimals() :
-		$args['number_decimals'];
+	$thousand_separator = give_get_price_thousand_separator($args['currency']);
+	$decimal_separator  = give_get_price_decimal_separator($args['currency']);
+	$number_decimals    = is_bool($args['number_decimals']) ?
+		give_get_price_decimals() : $args['number_decimals'];
 
 	// Explode number by . decimal separator.
-	$number_parts = explode( '.', $number );
+	$number_parts = explode('.', $number);
 
 	// Remove currency symbols from number if any.
-	$number = trim( str_replace( give_currency_symbols( true ), '', $number ) );
+	$number = trim(str_replace(give_currency_symbols(true), '', $number));
 
 	if (
 		// Non formatted number.
 		(
-			( false === strpos( $number, $thousand_separator ) ) &&
-			( false === strpos( $number, $decimal_separator ) )
+			(false === strpos($number, $thousand_separator)) &&
+			(false === strpos($number, $decimal_separator))
 		) ||
 
 		// Decimal formatted number.
@@ -228,35 +226,35 @@ 
 		(
 			$number_decimals &&
 			'.' === $thousand_separator &&
-			false !== strpos( $number, $thousand_separator ) &&
-			false === strpos( $number, $decimal_separator ) &&
-			2 === count( $number_parts ) &&
-			( $number_decimals >= strlen( $number_parts[1] ) )
+			false !== strpos($number, $thousand_separator) &&
+			false === strpos($number, $decimal_separator) &&
+			2 === count($number_parts) &&
+			($number_decimals >= strlen($number_parts[1]))
 		)
 	) {
-		return number_format( $number, $number_decimals, '.', '' );
+		return number_format($number, $number_decimals, '.', '');
 	}
 
 	// Handle thousand separator as '.'
 	// Handle sanitize database values.
-	$is_db_sanitize_val = ( 2 === count( $number_parts ) &&
-							is_numeric( $number_parts[0] ) &&
-							is_numeric( $number_parts[1] ) &&
-							( 6 === strlen( $number_parts[1] ) ) );
+	$is_db_sanitize_val = (2 === count($number_parts) &&
+							is_numeric($number_parts[0]) &&
+							is_numeric($number_parts[1]) &&
+							(6 === strlen($number_parts[1])));
 
-	if ( $is_db_sanitize_val ) {
+	if ($is_db_sanitize_val) {
 		// Sanitize database value.
-		return number_format( $number, $number_decimals, '.', '' );
+		return number_format($number, $number_decimals, '.', '');
 
 	} elseif (
 		'.' === $thousand_separator &&
-		false !== strpos( $number, $thousand_separator )
+		false !== strpos($number, $thousand_separator)
 	) {
 		// Fix point thousand separator value.
-		$number = str_replace( '.', '', $number );
+		$number = str_replace('.', '', $number);
 	}
 
-	return give_sanitize_amount( $number, $args );
+	return give_sanitize_amount($number, $args);
 }
 
 /**
@@ -273,10 +271,10 @@ 
  *
  * @return string $amount Newly sanitized amount
  */
-function give_sanitize_amount( $number, $args = array() ) {
+function give_sanitize_amount($number, $args = array()) {
 
 	// Bailout.
-	if ( empty( $number ) || ( ! is_numeric( $number ) && ! is_string( $number ) ) ) {
+	if (empty($number) || ( ! is_numeric($number) && ! is_string($number))) {
 		return $number;
 	}
 
@@ -284,10 +282,10 @@ 
 	$func_args = func_get_args();
 
 	// Backward compatibility.
-	if ( isset( $func_args[1] ) && ( is_bool( $func_args[1] ) || is_numeric( $func_args[1] ) ) ) {
+	if (isset($func_args[1]) && (is_bool($func_args[1]) || is_numeric($func_args[1]))) {
 		$args = array(
 			'number_decimals' => $func_args[1],
-			'trim_zeros'      => isset( $func_args[2] ) ? $func_args[2] : false,
+			'trim_zeros'      => isset($func_args[2]) ? $func_args[2] : false,
 		);
 	}
 
@@ -303,58 +301,58 @@ 
 	// Remove slash from amount.
 	// If thousand or decimal separator is set to ' then in $_POST or $_GET param we will get an escaped number.
 	// To prevent notices and warning remove slash from amount/number.
-	$number = wp_unslash( $number );
+	$number = wp_unslash($number);
 
-	$thousand_separator = give_get_price_thousand_separator( $args['currency'] );
+	$thousand_separator = give_get_price_thousand_separator($args['currency']);
 
 	$locale   = localeconv();
 	$decimals = array(
-		give_get_price_decimal_separator( $args['currency'] ),
+		give_get_price_decimal_separator($args['currency']),
 		$locale['decimal_point'],
 		$locale['mon_decimal_point'],
 	);
 
 	// Remove locale from string
-	if ( ! is_float( $number ) ) {
-		$number = str_replace( $decimals, '.', $number );
+	if ( ! is_float($number)) {
+		$number = str_replace($decimals, '.', $number);
 	}
 
 	// Remove thousand amount formatting if amount has.
 	// This condition use to add backward compatibility to version before 1.6, because before version 1.6 we were saving formatted amount to db.
 	// Do not replace thousand separator from price if it is same as decimal separator, because it will be already replace by above code.
-	if ( ! in_array( $thousand_separator, $decimals ) && ( false !== strpos( $number, $thousand_separator ) ) ) {
-		$number = str_replace( $thousand_separator, '', $number );
-	} elseif ( in_array( $thousand_separator, $decimals ) ) {
-		$number = preg_replace( '/\.(?=.*\.)/', '', $number );
+	if ( ! in_array($thousand_separator, $decimals) && (false !== strpos($number, $thousand_separator))) {
+		$number = str_replace($thousand_separator, '', $number);
+	} elseif (in_array($thousand_separator, $decimals)) {
+		$number = preg_replace('/\.(?=.*\.)/', '', $number);
 	}
 
 	// Remove non numeric entity before decimal separator.
-	$number     = preg_replace( '/[^0-9\.]/', '', $number );
-	$default_dp = give_get_price_decimals( $args['currency'] );
+	$number     = preg_replace('/[^0-9\.]/', '', $number);
+	$default_dp = give_get_price_decimals($args['currency']);
 
 	// Reset negative amount to zero.
-	if ( 0 > $number ) {
-		$number = number_format( 0, $default_dp, '.' );
+	if (0 > $number) {
+		$number = number_format(0, $default_dp, '.');
 	}
 
 	// If number does not have decimal then add number of decimals to it.
 	if (
-		false === strpos( $number, '.' )
-		|| ( $default_dp > strlen( substr( $number, strpos( $number, '.' ) + 1 ) ) )
+		false === strpos($number, '.')
+		|| ($default_dp > strlen(substr($number, strpos($number, '.') + 1)))
 	) {
-		$number = number_format( $number, $default_dp, '.', '' );
+		$number = number_format($number, $default_dp, '.', '');
 	}
 
 	// Format number by custom number of decimals.
-	if ( false !== $args['number_decimals'] ) {
-		$dp     = intval( is_bool( $args['number_decimals'] ) ? $default_dp : $args['number_decimals'] );
-		$dp     = apply_filters( 'give_sanitize_amount_decimals', $dp, $number );
-		$number = number_format( floatval( $number ), $dp, '.', '' );
+	if (false !== $args['number_decimals']) {
+		$dp     = intval(is_bool($args['number_decimals']) ? $default_dp : $args['number_decimals']);
+		$dp     = apply_filters('give_sanitize_amount_decimals', $dp, $number);
+		$number = number_format(floatval($number), $dp, '.', '');
 	}
 
 	// Trim zeros.
-	if ( $args['trim_zeros'] && strstr( $number, '.' ) ) {
-		$number = rtrim( rtrim( $number, '0' ), '.' );
+	if ($args['trim_zeros'] && strstr($number, '.')) {
+		$number = rtrim(rtrim($number, '0'), '.');
 	}
 
 	/**
@@ -362,7 +360,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	return apply_filters( 'give_sanitize_amount', $number );
+	return apply_filters('give_sanitize_amount', $number);
 }
 
 /**
@@ -375,9 +373,9 @@ 
  *
  * @return string $amount   Newly formatted amount or Price Not Available
  */
-function give_format_amount( $amount, $args = array() ) {
+function give_format_amount($amount, $args = array()) {
 	// Backward compatibility.
-	if ( is_bool( $args ) ) {
+	if (is_bool($args)) {
 		$args = array(
 			'decimal' => $args,
 		);
@@ -390,61 +388,60 @@ 
 		'currency'    => '',
 	);
 
-	$args = wp_parse_args( $args, $default_args );
+	$args = wp_parse_args($args, $default_args);
 
 	// Set Currency based on donation id, if required.
-	if ( $args['donation_id'] && empty( $args['currency'] ) ) {
-		$donation_meta    = give_get_meta( $args['donation_id'], '_give_payment_meta', true );
+	if ($args['donation_id'] && empty($args['currency'])) {
+		$donation_meta    = give_get_meta($args['donation_id'], '_give_payment_meta', true);
 		$args['currency'] = $donation_meta['currency'];
 	}
 
 	$formatted     = 0;
-	$currency      = ! empty( $args['currency'] ) ? $args['currency'] : give_get_currency( $args['donation_id'] );
-	$thousands_sep = give_get_price_thousand_separator( $currency );
-	$decimal_sep   = give_get_price_decimal_separator( $currency );
-	$decimals      = ! empty( $args['decimal'] ) ? give_get_price_decimals( $currency ) : 0;
+	$currency      = ! empty($args['currency']) ? $args['currency'] : give_get_currency($args['donation_id']);
+	$thousands_sep = give_get_price_thousand_separator($currency);
+	$decimal_sep   = give_get_price_decimal_separator($currency);
+	$decimals      = ! empty($args['decimal']) ? give_get_price_decimals($currency) : 0;
 
-	if ( ! empty( $amount ) ) {
+	if ( ! empty($amount)) {
 		// Sanitize amount before formatting.
-		$amount = ! empty( $args['sanitize'] ) ?
-			give_maybe_sanitize_amount( $amount, array( 'number_decimals' => $decimals, 'currency' => $currency ) ) :
-			number_format( $amount, $decimals, '.', '' );
+		$amount = ! empty($args['sanitize']) ?
+			give_maybe_sanitize_amount($amount, array('number_decimals' => $decimals, 'currency' => $currency)) : number_format($amount, $decimals, '.', '');
 
-		switch ( $currency ) {
+		switch ($currency) {
 			case 'INR':
 				$decimal_amount = '';
 
 				// Extract decimals from amount
-				if ( ( $pos = strpos( $amount, '.' ) ) !== false ) {
-					if ( ! empty( $decimals ) ) {
-						$decimal_amount = substr( round( substr( $amount, $pos ), $decimals ), 1 );
-						$amount         = substr( $amount, 0, $pos );
-
-						if ( ! $decimal_amount ) {
-							$decimal_amount = substr( "{$decimal_sep}0000000000", 0, ( $decimals + 1 ) );
-						} elseif ( ( $decimals + 1 ) > strlen( $decimal_amount ) ) {
-							$decimal_amount = substr( "{$decimal_amount}000000000", 0, ( $decimals + 1 ) );
+				if (($pos = strpos($amount, '.')) !== false) {
+					if ( ! empty($decimals)) {
+						$decimal_amount = substr(round(substr($amount, $pos), $decimals), 1);
+						$amount         = substr($amount, 0, $pos);
+
+						if ( ! $decimal_amount) {
+							$decimal_amount = substr("{$decimal_sep}0000000000", 0, ($decimals + 1));
+						} elseif (($decimals + 1) > strlen($decimal_amount)) {
+							$decimal_amount = substr("{$decimal_amount}000000000", 0, ($decimals + 1));
 						}
 					} else {
-						$amount = number_format( $amount, $decimals, $decimal_sep, '' );
+						$amount = number_format($amount, $decimals, $decimal_sep, '');
 					}
 				}
 
 				// Extract last 3 from amount
-				$result = substr( $amount, - 3 );
-				$amount = substr( $amount, 0, - 3 );
+				$result = substr($amount, - 3);
+				$amount = substr($amount, 0, - 3);
 
 				// Apply digits 2 by 2
-				while ( strlen( $amount ) > 0 ) {
-					$result = substr( $amount, - 2 ) . $thousands_sep . $result;
-					$amount = substr( $amount, 0, - 2 );
+				while (strlen($amount) > 0) {
+					$result = substr($amount, - 2).$thousands_sep.$result;
+					$amount = substr($amount, 0, - 2);
 				}
 
-				$formatted = $result . $decimal_amount;
+				$formatted = $result.$decimal_amount;
 				break;
 
 			default:
-				$formatted = number_format( $amount, $decimals, $decimal_sep, $thousands_sep );
+				$formatted = number_format($amount, $decimals, $decimal_sep, $thousands_sep);
 		}
 	}
 
@@ -453,7 +450,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	return apply_filters( 'give_format_amount', $formatted, $amount, $decimals, $decimal_sep, $thousands_sep, $currency, $args );
+	return apply_filters('give_format_amount', $formatted, $amount, $decimals, $decimal_sep, $thousands_sep, $currency, $args);
 }
 
 
@@ -471,17 +468,17 @@ 
  *
  * @return string  formatted amount number with large number names.
  */
-function give_human_format_large_amount( $amount, $args = array() ) {
+function give_human_format_large_amount($amount, $args = array()) {
 	// Sanitize amount.
-	$sanitize_amount = give_maybe_sanitize_amount( $amount );
+	$sanitize_amount = give_maybe_sanitize_amount($amount);
 
 	// Bailout.
-	if ( ! floatval( $sanitize_amount ) ) {
+	if ( ! floatval($sanitize_amount)) {
 		return '0';
 	};
 
 	// Set default currency;
-	if ( empty( $args['currency'] ) ) {
+	if (empty($args['currency'])) {
 		$args['currency'] = give_get_currency();
 	}
 
@@ -489,37 +486,37 @@ 
 	$thousands_sep = give_get_price_thousand_separator();
 
 	// Explode amount to calculate name of large numbers.
-	$amount_array = explode( $thousands_sep, $amount );
+	$amount_array = explode($thousands_sep, $amount);
 
 	// Calculate amount parts count.
-	$amount_count_parts = count( $amount_array );
+	$amount_count_parts = count($amount_array);
 
 	// Human format amount (default).
 	$human_format_amount = $amount;
 
-	switch ( $args['currency'] ) {
+	switch ($args['currency']) {
 		case 'INR':
 			// Calculate large number formatted amount.
-			if ( 4 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s arab', 'give' ), round( ( $sanitize_amount / 1000000000 ), 2 ) );
-			} elseif ( 3 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s crore', 'give' ), round( ( $sanitize_amount / 10000000 ), 2 ) );
-			} elseif ( 2 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s lakh', 'give' ), round( ( $sanitize_amount / 100000 ), 2 ) );
+			if (4 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s arab', 'give'), round(($sanitize_amount / 1000000000), 2));
+			} elseif (3 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s crore', 'give'), round(($sanitize_amount / 10000000), 2));
+			} elseif (2 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s lakh', 'give'), round(($sanitize_amount / 100000), 2));
 			}
 			break;
 		default:
 			// Calculate large number formatted amount.
-			if ( 4 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s trillion', 'give' ), round( ( $sanitize_amount / 1000000000000 ), 2 ) );
-			} elseif ( 3 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s billion', 'give' ), round( ( $sanitize_amount / 1000000000 ), 2 ) );
-			} elseif ( 2 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s million', 'give' ), round( ( $sanitize_amount / 1000000 ), 2 ) );
+			if (4 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s trillion', 'give'), round(($sanitize_amount / 1000000000000), 2));
+			} elseif (3 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s billion', 'give'), round(($sanitize_amount / 1000000000), 2));
+			} elseif (2 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s million', 'give'), round(($sanitize_amount / 1000000), 2));
 			}
 	}
 
-	return apply_filters( 'give_human_format_large_amount', $human_format_amount, $amount, $sanitize_amount );
+	return apply_filters('give_human_format_large_amount', $human_format_amount, $amount, $sanitize_amount);
 }
 
 /**
@@ -533,17 +530,16 @@ 
  *
  * @return string $amount Newly formatted amount or Price Not Available
  */
-function give_format_decimal( $amount, $dp = false, $sanitize = true ) {
+function give_format_decimal($amount, $dp = false, $sanitize = true) {
 	$decimal_separator = give_get_price_decimal_separator();
 	$formatted_amount  = $sanitize ?
-		give_maybe_sanitize_amount( $amount, array( 'number_decimals' => $dp ) ) :
-		number_format( $amount, ( is_bool( $dp ) ? give_get_price_decimals() : $dp ), '.', '' );
+		give_maybe_sanitize_amount($amount, array('number_decimals' => $dp)) : number_format($amount, (is_bool($dp) ? give_get_price_decimals() : $dp), '.', '');
 
-	if ( false !== strpos( $formatted_amount, '.' ) ) {
-		$formatted_amount = str_replace( '.', $decimal_separator, $formatted_amount );
+	if (false !== strpos($formatted_amount, '.')) {
+		$formatted_amount = str_replace('.', $decimal_separator, $formatted_amount);
 	}
 
-	return apply_filters( 'give_format_decimal', $formatted_amount, $amount, $decimal_separator );
+	return apply_filters('give_format_decimal', $formatted_amount, $amount, $decimal_separator);
 }
 
 /**
@@ -555,7 +551,7 @@ 
  *
  * @return string                  Date format string
  */
-function give_date_format( $date_context = '' ) {
+function give_date_format($date_context = '') {
 	/**
 	 * Filter the date context
 	 *
@@ -576,19 +572,19 @@ 
 	 *
 	 *    }
 	 */
-	$date_format_contexts = apply_filters( 'give_date_format_contexts', array() );
+	$date_format_contexts = apply_filters('give_date_format_contexts', array());
 
 	// Set date format to default date format.
-	$date_format = get_option( 'date_format' );
+	$date_format = get_option('date_format');
 
 	// Update date format if we have non empty date format context array and non empty date format string for that context.
-	if ( $date_context && ! empty( $date_format_contexts ) && array_key_exists( $date_context, $date_format_contexts ) ) {
-		$date_format = ! empty( $date_format_contexts[ $date_context ] )
-			? $date_format_contexts[ $date_context ]
+	if ($date_context && ! empty($date_format_contexts) && array_key_exists($date_context, $date_format_contexts)) {
+		$date_format = ! empty($date_format_contexts[$date_context])
+			? $date_format_contexts[$date_context]
 			: $date_format;
 	}
 
-	return apply_filters( 'give_date_format', $date_format );
+	return apply_filters('give_date_format', $date_format);
 }
 
 /**
@@ -602,8 +598,8 @@ 
  *
  * @return string
  */
-function give_get_cache_key( $action, $query_args ) {
-	return Give_Cache::get_key( $action, $query_args );
+function give_get_cache_key($action, $query_args) {
+	return Give_Cache::get_key($action, $query_args);
 }
 
 /**
@@ -616,11 +612,11 @@ 
  *
  * @return string|array
  */
-function give_clean( $var ) {
-	if ( is_array( $var ) ) {
-		return array_map( 'give_clean', $var );
+function give_clean($var) {
+	if (is_array($var)) {
+		return array_map('give_clean', $var);
 	} else {
-		return is_scalar( $var ) ? sanitize_text_field( $var ) : $var;
+		return is_scalar($var) ? sanitize_text_field($var) : $var;
 	}
 }
 
@@ -633,10 +629,10 @@ 
  *
  * @return int
  */
-function give_let_to_num( $size ) {
-	$l   = substr( $size, - 1 );
-	$ret = substr( $size, 0, - 1 );
-	switch ( strtoupper( $l ) ) {
+function give_let_to_num($size) {
+	$l   = substr($size, - 1);
+	$ret = substr($size, 0, - 1);
+	switch (strtoupper($l)) {
 		case 'P':
 			$ret *= 1024;
 		case 'T':
@@ -661,19 +657,19 @@ 
  * @param int   $action
  * @param array $wp_die_args
  */
-function give_validate_nonce( $nonce, $action = - 1, $wp_die_args = array() ) {
+function give_validate_nonce($nonce, $action = - 1, $wp_die_args = array()) {
 
 	$default_wp_die_args = array(
-		'message' => esc_html__( 'Nonce verification has failed.', 'give' ),
-		'title'   => esc_html__( 'Error', 'give' ),
+		'message' => esc_html__('Nonce verification has failed.', 'give'),
+		'title'   => esc_html__('Error', 'give'),
 		'args'    => array(
 			'response' => 403,
 		),
 	);
 
-	$wp_die_args = wp_parse_args( $wp_die_args, $default_wp_die_args );
+	$wp_die_args = wp_parse_args($wp_die_args, $default_wp_die_args);
 
-	if ( ! wp_verify_nonce( $nonce, $action ) ) {
+	if ( ! wp_verify_nonce($nonce, $action)) {
 		wp_die(
 			$wp_die_args['message'],
 			$wp_die_args['title'],
@@ -695,23 +691,23 @@ 
  *
  * @return mixed
  */
-function give_check_variable( $variable, $conditional = '', $default = false ) {
+function give_check_variable($variable, $conditional = '', $default = false) {
 
-	switch ( $conditional ) {
+	switch ($conditional) {
 		case 'isset_empty':
-			$variable = ( isset( $variable ) && ! empty( $variable ) ) ? $variable : $default;
+			$variable = (isset($variable) && ! empty($variable)) ? $variable : $default;
 			break;
 
 		case 'empty':
-			$variable = ! empty( $variable ) ? $variable : $default;
+			$variable = ! empty($variable) ? $variable : $default;
 			break;
 
 		case 'null':
-			$variable = ! is_null( $variable ) ? $variable : $default;
+			$variable = ! is_null($variable) ? $variable : $default;
 			break;
 
 		default:
-			$variable = isset( $variable ) ? $variable : $default;
+			$variable = isset($variable) ? $variable : $default;
 
 	}