WordImpress / Give

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 +109 added lines, -111 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -70,12 +70,12 @@ 
 	 *
 	 * @param int $_step
 	 */
-	public function __construct( $_step = 1 ) {
+	public function __construct($_step = 1) {
 
-		parent::__construct( $_step );
+		parent::__construct($_step);
 
 		// Filter to change the filename.
-		add_filter( 'give_export_filename', array( $this, 'give_export_filename' ), 10, 2 );
+		add_filter('give_export_filename', array($this, 'give_export_filename'), 10, 2);
 	}
 
 	/**
@@ -88,19 +88,19 @@ 
 	 *
 	 * @return string $filename file name.
 	 */
-	public function give_export_filename( $filename, $export_type ) {
+	public function give_export_filename($filename, $export_type) {
 
-		if ( $this->export_type !== $export_type ) {
+		if ($this->export_type !== $export_type) {
 			return $filename;
 		}
 
-		$forms = empty( $_GET['forms'] ) ? 0 : absint( $_GET['forms'] );
+		$forms = empty($_GET['forms']) ? 0 : absint($_GET['forms']);
 
-		if ( $forms ) {
-			$slug     = get_post_field( 'post_name', get_post( $forms ) );
-			$filename = 'give-export-donors-' . $slug . '-' . date( 'm-d-Y' );
+		if ($forms) {
+			$slug     = get_post_field('post_name', get_post($forms));
+			$filename = 'give-export-donors-'.$slug.'-'.date('m-d-Y');
 		} else {
-			$filename = 'give-export-donors-all-forms-' . date( 'm-d-Y' );
+			$filename = 'give-export-donors-all-forms-'.date('m-d-Y');
 		}
 
 		return $filename;
@@ -113,23 +113,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;
 	}
 
 	/**
@@ -142,12 +142,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);
 	}
 
 	/**
@@ -159,14 +159,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;
 	}
@@ -178,38 +178,38 @@ 
 	 *
 	 * @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 'donor_created_date' :
-					$cols['donor_created_date'] = esc_html__( 'Donor Created Date', 'give' );
+					$cols['donor_created_date'] = esc_html__('Donor Created 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;
 			}
 		}
@@ -230,29 +230,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',
@@ -261,44 +261,42 @@ 
 				);
 			}
 
-			$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->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->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.
@@ -308,7 +306,7 @@ 
 		} else {
 
 			// Export all donors.
-			$offset = 30 * ( $this->step - 1 );
+			$offset = 30 * ($this->step - 1);
 
 			$args = array(
 				'number' => 30,
@@ -316,29 +314,29 @@ 
 			);
 
 			// 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;
 				}
 
-				$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;
 	}
@@ -354,18 +352,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;
 		}
 
@@ -381,49 +379,49 @@ 
 	 *
 	 * @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'] ) ) {
-			$donor_name              = give_get_donor_name_by( $donor->id, 'donor' );
-			$data[ $i ]['full_name'] = $donor_name;
+		if ( ! empty($columns['full_name'])) {
+			$donor_name              = give_get_donor_name_by($donor->id, 'donor');
+			$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['donor_created_date'] ) ) {
-			$data[ $i ]['donor_created_date'] = date_i18n( give_date_format(), strtotime( $donor->date_created ) );
+		if ( ! empty($columns['donor_created_date'])) {
+			$data[$i]['donor_created_date'] = 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];
 
 	}
 
@@ -433,9 +431,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/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/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 +89 added lines, -89 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -98,8 +98,8 @@ 
 	/**
 	 * Constructor.
 	 */
-	public function __construct( $_step = 1 ) {
-		parent::__construct( $_step );
+	public function __construct($_step = 1) {
+		parent::__construct($_step);
 
 		$this->is_writable = true;
 	}
@@ -118,32 +118,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);
 		}
 	}
 
@@ -154,17 +154,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,
@@ -172,16 +172,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.
@@ -200,23 +200,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();
 	}
@@ -228,33 +228,33 @@ 
 	 * @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() ) {
+		if ( ! $this->can_export()) {
 			wp_die(
-				esc_html__( 'You do not have permission to delete test transactions.', 'give' ),
-				esc_html__( 'Error', 'give' ),
-				array( 'response' => 403 )
+				esc_html__('You do not have permission to delete test transactions.', 'give'),
+				esc_html__('Error', 'give'),
+				array('response' => 403)
 			);
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
-			update_option( 'give_earnings_total', give_get_total_earnings( true ), false );
-			Give_Cache::delete( Give_Cache::get_key( 'give_estimated_monthly_stats' ) );
+			update_option('give_earnings_total', give_get_total_earnings(true), false);
+			Give_Cache::delete(Give_Cache::get_key('give_estimated_monthly_stats'));
 
-			$this->delete_option( $this->donation_key );
+			$this->delete_option($this->donation_key);
 
 			$this->done    = true;
-			$this->message = __( 'Test donor and transactions successfully deleted.', 'give' );
+			$this->message = __('Test donor and transactions successfully deleted.', 'give');
 
 			return false;
 		}
@@ -272,12 +272,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;
 
@@ -288,68 +288,68 @@ 
 		$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;
@@ -361,24 +361,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];
 	}
 
 	/**
@@ -390,8 +390,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);
 	}
 
 	/**
@@ -404,8 +404,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);
 	}
 
 	/**
@@ -417,8 +417,8 @@ 
 	 *
 	 * @return void
 	 */
-	public function delete_option( $key ) {
-		delete_option( $key );
+	public function delete_option($key) {
+		delete_option($key);
 	}
 
 	/**
@@ -429,12 +429,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;
@@ -445,6 +445,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 +185 added lines, -189 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 ) ) {
-			$currency = give_get_meta( $id_or_currency_code, '_give_payment_currency', true );
+			$setting = $currencies[$id_or_currency_code]['setting'];
+		} elseif (is_numeric($id_or_currency_code) && 'give_payment' === get_post_type($id_or_currency_code)) {
+			$currency = give_get_meta($id_or_currency_code, '_give_payment_currency', true);
 
 			if (
-				! empty( $currency) &&
+				! empty($currency) &&
 				$give_options['currency'] !== $currency
 			) {
-				$setting = $currencies[ $currency ]['setting'];
+				$setting = $currencies[$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);
 }
 
 
@@ -164,10 +163,10 @@ 
  *
  * @return string $amount Newly sanitized amount
  */
-function give_sanitize_amount_for_db( $number, $args = array() ) {
+function give_sanitize_amount_for_db($number, $args = array()) {
 	$args['number_decimals'] = 6;
 
-	return give_maybe_sanitize_amount( $number, $args );
+	return give_maybe_sanitize_amount($number, $args);
 }
 
 /**
@@ -180,19 +179,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,
 		);
 	}
 
@@ -205,24 +204,23 @@ 
 		)
 	);
 
-	$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)
 	) {
-		return number_format( $number, $number_decimals, '.', '' );
+		return number_format($number, $number_decimals, '.', '');
 	} elseif (
 		// Decimal formatted number.
 		// If number of decimal place set to non zero and
@@ -230,34 +228,34 @@ 
 		// then number will be consider as decimal formatted which means number is already sanitized.
 		$number_decimals
 		&& '.' === $thousand_separator
-		&& 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, '.', '' );
+		&& 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, '.', '');
 	}
 
 	// 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);
 }
 
 /**
@@ -274,10 +272,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;
 	}
 
@@ -285,10 +283,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,
 		);
 	}
 
@@ -304,58 +302,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'), '.');
 	}
 
 	/**
@@ -363,7 +361,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	return apply_filters( 'give_sanitize_amount', $number );
+	return apply_filters('give_sanitize_amount', $number);
 }
 
 /**
@@ -376,9 +374,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,
 		);
@@ -391,60 +389,59 @@ 
 		'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'] ) ) {
-		$args['currency'] = give_get_meta( $args['donation_id'], '_give_payment_currency', true );
+	if ($args['donation_id'] && empty($args['currency'])) {
+		$args['currency'] = give_get_meta($args['donation_id'], '_give_payment_currency', true);
 	}
 
 	$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( wp_unslash( $var ) ) : $var;
+		return is_scalar($var) ? sanitize_text_field(wp_unslash($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':
@@ -663,22 +659,22 @@ 
  *
  * @return bool
  */
-function give_validate_nonce( $nonce, $action = - 1, $wp_die_args = array() ) {
+function give_validate_nonce($nonce, $action = - 1, $wp_die_args = array()) {
 
 	// Verify nonce.
-	$verify_nonce = wp_verify_nonce( $nonce, $action );
+	$verify_nonce = wp_verify_nonce($nonce, $action);
 
 	// On ajax request send nonce verification status.
-	if ( wp_doing_ajax() ) {
+	if (wp_doing_ajax()) {
 		return $verify_nonce;
 	}
 
-	if ( ! $verify_nonce ) {
+	if ( ! $verify_nonce) {
 		$wp_die_args = wp_parse_args(
 			$wp_die_args,
 			array(
-				'message' => __( 'Nonce verification has failed.', 'give' ),
-				'title'   => __( 'Error', 'give' ),
+				'message' => __('Nonce verification has failed.', 'give'),
+				'title'   => __('Error', 'give'),
 				'args'    => array(
 					'response' => 403,
 				),
@@ -705,16 +701,16 @@ 
  *
  * @return bool
  */
-function give_verify_donation_form_nonce( $nonce = '', $form_id ) {
+function give_verify_donation_form_nonce($nonce = '', $form_id) {
 
 	// Form nonce action.
 	$nonce_action = "give_donation_form_nonce_{$form_id}";
 
 	// Nonce validation.
-	$verify_nonce = give_validate_nonce( $nonce, $nonce_action );
+	$verify_nonce = give_validate_nonce($nonce, $nonce_action);
 
-	if ( ! $verify_nonce ) {
-		give_set_error( 'donation_form_nonce', __( 'Nonce verification has failed.', 'give' ) );
+	if ( ! $verify_nonce) {
+		give_set_error('donation_form_nonce', __('Nonce verification has failed.', 'give'));
 	}
 
 	return $verify_nonce;
@@ -734,43 +730,43 @@ 
  *
  * @return mixed
  */
-function give_check_variable( $variable, $conditional = '', $default = false, $array_key_name = '' ) {
+function give_check_variable($variable, $conditional = '', $default = false, $array_key_name = '') {
 	// Get value from array if array key non empty.
-	if( empty( $array_key_name ) ) {
-		switch ( $conditional ) {
+	if (empty($array_key_name)) {
+		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;
 		}
 	} else {
-		$isset = array_key_exists( $array_key_name, $variable );
+		$isset = array_key_exists($array_key_name, $variable);
 
-		switch ( $conditional ) {
+		switch ($conditional) {
 			case 'isset_empty':
-				$variable = ( $isset && ! empty( $variable[ $array_key_name ] ) ) ? $variable[ $array_key_name ] : $default;
+				$variable = ($isset && ! empty($variable[$array_key_name])) ? $variable[$array_key_name] : $default;
 				break;
 
 			case 'empty':
-				$variable = ! empty( $variable[ $array_key_name ] ) ? $variable[ $array_key_name ] : $default;
+				$variable = ! empty($variable[$array_key_name]) ? $variable[$array_key_name] : $default;
 				break;
 
 			case 'null':
-				$variable = $isset && ! is_null( $variable[ $array_key_name ] ) ? $variable[ $array_key_name ] : $default;
+				$variable = $isset && ! is_null($variable[$array_key_name]) ? $variable[$array_key_name] : $default;
 				break;
 
 			default:
-				$variable = $isset && isset( $variable[ $array_key_name ] ) ? $variable[ $array_key_name ] : $default;
+				$variable = $isset && isset($variable[$array_key_name]) ? $variable[$array_key_name] : $default;
 		}
 	}
 

includes/admin/shortcodes/shortcode-give-goal.php

Spacing +19 added lines, -19 removed lines

@@ -9,7 +9,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,10 +23,10 @@ 
 	 */
 	public function __construct() {
 
-		$this->shortcode['title'] = esc_html__( 'Donation Form Goal', 'give' );
-		$this->shortcode['label'] = esc_html__( 'Donation Form Goal', 'give' );
+		$this->shortcode['title'] = esc_html__('Donation Form Goal', 'give');
+		$this->shortcode['label'] = esc_html__('Donation Form Goal', 'give');
 
-		parent::__construct( 'give_goal' );
+		parent::__construct('give_goal');
 	}
 
 	/**
@@ -38,8 +38,8 @@ 
 
 		$create_form_link = sprintf(
 		/* translators: %s: create new form URL */
-			__( '<a href="%s">Create</a> a new Donation Form.', 'give' ),
-			admin_url( 'post-new.php?post_type=give_forms' )
+			__('<a href="%s">Create</a> a new Donation Form.', 'give'),
+			admin_url('post-new.php?post_type=give_forms')
 		);
 
 		return array(
@@ -51,35 +51,35 @@ 
 					'meta_value' => 'enabled',
 				),
 				'name'        => 'id',
-				'tooltip'     => esc_attr__( 'Select a Donation Form', 'give' ),
-				'placeholder' => '- ' . esc_attr__( 'Select a Donation Form', 'give' ) . ' -',
+				'tooltip'     => esc_attr__('Select a Donation Form', 'give'),
+				'placeholder' => '- '.esc_attr__('Select a Donation Form', 'give').' -',
 				'required'    => array(
-					'alert' => esc_html__( 'You must first select a Form!', 'give' ),
-					'error' => sprintf( '<p class="strong">%s</p><p class="no-margin">%s</p>', esc_html__( 'No forms found.', 'give' ), $create_form_link ),
+					'alert' => esc_html__('You must first select a Form!', 'give'),
+					'error' => sprintf('<p class="strong">%s</p><p class="no-margin">%s</p>', esc_html__('No forms found.', 'give'), $create_form_link),
 				),
 			),
 			array(
 				'type' => 'container',
-				'html' => sprintf( '<p class="strong margin-top">%s</p>', esc_html__( 'Optional settings', 'give' ) ),
+				'html' => sprintf('<p class="strong margin-top">%s</p>', esc_html__('Optional settings', 'give')),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'show_text',
-				'label'   => esc_attr__( 'Show Text:', 'give' ),
-				'tooltip' => esc_attr__( 'This text displays the amount of income raised compared to the goal.', 'give' ),
+				'label'   => esc_attr__('Show Text:', 'give'),
+				'tooltip' => esc_attr__('This text displays the amount of income raised compared to the goal.', 'give'),
 				'options' => array(
-					'true'  => esc_html__( 'Show', 'give' ),
-					'false' => esc_html__( 'Hide', 'give' ),
+					'true'  => esc_html__('Show', 'give'),
+					'false' => esc_html__('Hide', 'give'),
 				),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'show_bar',
-				'label'   => esc_attr__( 'Show Progress Bar:', 'give' ),
-				'tooltip' => esc_attr__( 'Do you want to display the goal\'s progress bar?', 'give' ),
+				'label'   => esc_attr__('Show Progress Bar:', 'give'),
+				'tooltip' => esc_attr__('Do you want to display the goal\'s progress bar?', 'give'),
 				'options' => array(
-					'true'  => esc_html__( 'Show', 'give' ),
-					'false' => esc_html__( 'Hide', 'give' ),
+					'true'  => esc_html__('Show', 'give'),
+					'false' => esc_html__('Hide', 'give'),
 				),
 			),
 		);

templates/shortcode-register.php

Spacing +13 added lines, -13 removed lines

@@ -2,7 +2,7 @@ 
 /**
  * This template is used to display the registration form with [give_register]
  */
-Give()->notices->render_frontend_notices( 0 ); ?>
+Give()->notices->render_frontend_notices(0); ?>
 
 <form id="give-register-form" class="give-form" action="" method="post">
 	<?php
@@ -13,11 +13,11 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_register_form_fields_top' );
+	do_action('give_register_form_fields_top');
 	?>
 
 	<fieldset>
-		<legend><?php esc_html_e( 'Register a New Account', 'give' ); ?></legend>
+		<legend><?php esc_html_e('Register a New Account', 'give'); ?></legend>
 
 		<?php
 		/**
@@ -27,26 +27,26 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_register_form_fields_before' );
+		do_action('give_register_form_fields_before');
 		?>
 
 		<div class="form-row form-row-first form-row-responsive">
-			<label for="give-user-login"><?php esc_html_e( 'Username', 'give' ); ?></label>
+			<label for="give-user-login"><?php esc_html_e('Username', 'give'); ?></label>
 			<input id="give-user-login" class="required give-input" type="text" name="give_user_login" required aria-required="true" />
 		</div>
 
 		<div class="form-row form-row-last form-row-responsive">
-			<label for="give-user-email"><?php esc_html_e( 'Email', 'give' ); ?></label>
+			<label for="give-user-email"><?php esc_html_e('Email', 'give'); ?></label>
 			<input id="give-user-email" class="required give-input" type="email" name="give_user_email" required aria-required="true" />
 		</div>
 
 		<div class="form-row form-row-first form-row-responsive">
-			<label for="give-user-pass"><?php esc_html_e( 'Password', 'give' ); ?></label>
+			<label for="give-user-pass"><?php esc_html_e('Password', 'give'); ?></label>
 			<input id="give-user-pass" class="password required give-input" type="password" name="give_user_pass" required aria-required="true" />
 		</div>
 
 		<div class="form-row form-row-last form-row-responsive">
-			<label for="give-user-pass2"><?php esc_html_e( 'Confirm PW', 'give' ); ?></label>
+			<label for="give-user-pass2"><?php esc_html_e('Confirm PW', 'give'); ?></label>
 			<input id="give-user-pass2" class="password required give-input" type="password" name="give_user_pass2" required aria-required="true" />
 		</div>
 
@@ -58,17 +58,17 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_register_form_fields_before_submit' );
+		do_action('give_register_form_fields_before_submit');
 		?>
 
 		<div class="give-hidden">
 			<input type="hidden" name="give_honeypot" value="" />
 			<input type="hidden" name="give_action" value="user_register" />
-			<input type="hidden" name="give_redirect" value="<?php echo esc_url( $give_register_redirect ); ?>" />
+			<input type="hidden" name="give_redirect" value="<?php echo esc_url($give_register_redirect); ?>" />
 		</div>
 
 		<div class="form-row">
-			<input class="button" name="give_register_submit" type="submit" value="<?php esc_attr_e( 'Register', 'give' ); ?>" />
+			<input class="button" name="give_register_submit" type="submit" value="<?php esc_attr_e('Register', 'give'); ?>" />
 		</div>
 
 		<?php
@@ -79,7 +79,7 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_register_form_fields_after' );
+		do_action('give_register_form_fields_after');
 		?>
 
 	</fieldset>
@@ -92,6 +92,6 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_register_form_fields_bottom' );
+	do_action('give_register_form_fields_bottom');
 	?>
 </form>

templates/global/wrapper-start.php

Spacing +4 added lines, -4 removed lines

@@ -9,13 +9,13 @@ 
  * @since       1.0

  */

 

-if ( ! defined( 'ABSPATH' ) ) {

+if ( ! defined('ABSPATH')) {

 	exit; // Exit if accessed directly.

 }

 

-$template = strtolower( get_option( 'template' ) );

+$template = strtolower(get_option('template'));

 

-switch ( $template ) {

+switch ($template) {

 	case 'twentyeleven' :

 		echo '<div id="primary" class="give-wrap"><div id="content" role="main" class="twentyeleven">';

 		break;

@@ -65,6 +65,6 @@ 
 		echo '<div class="wrapper hentry" style="box-sizing: border-box;">';

 		break;

 	default :

-		echo apply_filters( 'give_default_wrapper_start', '<div id="container" class="give-wrap container"><div id="content" role="main">' );

+		echo apply_filters('give_default_wrapper_start', '<div id="container" class="give-wrap container"><div id="content" role="main">');

 		break;

 }
\ No newline at end of file

templates/global/wrapper-end.php

Spacing +5 added lines, -5 removed lines

@@ -9,13 +9,13 @@ 
  * @since       1.0

  */

 

-if ( ! defined( 'ABSPATH' ) ) {

+if ( ! defined('ABSPATH')) {

 	exit; // Exit if accessed directly.

 }

 

-$template = get_option( 'template' );

+$template = get_option('template');

 

-switch ( $template ) {

+switch ($template) {

 	case 'twentyeleven' :

 		echo '</div></div>';

 		break;

@@ -27,7 +27,7 @@ 
 		break;

 	case 'twentyfourteen' :

 		echo '</div></div></div>';

-		get_sidebar( 'content' );

+		get_sidebar('content');

 		break;

 	case 'twentyfifteen' :

 		echo '</div></div>';

@@ -57,6 +57,6 @@ 
 		echo '</div>';

 		break;

 	default :

-		echo apply_filters( 'give_default_wrapper_end', '</div></div>' );

+		echo apply_filters('give_default_wrapper_end', '</div></div>');

 		break;

 }
\ No newline at end of file