WordImpress / Give

includes/admin/upgrades/views/upgrades-complete.php

Spacing +3 added lines, -3 removed lines

@@ -10,17 +10,17 @@
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 ?>
 <div class="wrap" id="poststuff">
 	<div id="give-updates">
-		<h1 id="give-updates-h1"><?php esc_html_e( 'Give - Updates Complete', 'give' ); ?></h1>
+		<h1 id="give-updates-h1"><?php esc_html_e('Give - Updates Complete', 'give'); ?></h1>
 		<hr class="wp-header-end">
 
 		<div class="give-update-panel-content">
-			<p><?php esc_html_e( 'Congratulations! You are running the latest versions of Give and its add-ons.', 'give' ); ?></p>
+			<p><?php esc_html_e('Congratulations! You are running the latest versions of Give and its add-ons.', 'give'); ?></p>
 		</div>
 
 	</div>

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

Spacing +71 added lines, -71 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -39,30 +39,30 @@ 
 	 */
 	public function csv_cols() {
 		$cols = array(
-			'id'        => __( 'ID', 'give' ), // unaltered payment ID (use for querying).
-			'seq_id'    => __( 'Payment Number', 'give' ), // sequential payment ID.
-			'email'     => __( 'Email', 'give' ),
-			'first'     => __( 'First Name', 'give' ),
-			'last'      => __( 'Last Name', 'give' ),
-			'address1'  => __( 'Address 1', 'give' ),
-			'address2'  => __( 'Address 2', 'give' ),
-			'city'      => __( 'City', 'give' ),
-			'state'     => __( 'State', 'give' ),
-			'country'   => __( 'Country', 'give' ),
-			'zip'       => __( 'Zip / Postal Code', 'give' ),
-			'form_id'   => __( 'Form ID', 'give' ),
-			'form_name' => __( 'Form Name', 'give' ),
-			'amount'    => __( 'Amount', 'give' ) . ' (' . give_currency_symbol( '', true ) . ')',
-			'gateway'   => __( 'Payment Method', 'give' ),
-			'trans_id'  => __( 'Transaction ID', 'give' ),
-			'key'       => __( 'Key', 'give' ),
-			'date'      => __( 'Date', 'give' ),
-			'user'      => __( 'User', 'give' ),
-			'status'    => __( 'Status', 'give' )
+			'id'        => __('ID', 'give'), // unaltered payment ID (use for querying).
+			'seq_id'    => __('Payment Number', 'give'), // sequential payment ID.
+			'email'     => __('Email', 'give'),
+			'first'     => __('First Name', 'give'),
+			'last'      => __('Last Name', 'give'),
+			'address1'  => __('Address 1', 'give'),
+			'address2'  => __('Address 2', 'give'),
+			'city'      => __('City', 'give'),
+			'state'     => __('State', 'give'),
+			'country'   => __('Country', 'give'),
+			'zip'       => __('Zip / Postal Code', 'give'),
+			'form_id'   => __('Form ID', 'give'),
+			'form_name' => __('Form Name', 'give'),
+			'amount'    => __('Amount', 'give').' ('.give_currency_symbol('', true).')',
+			'gateway'   => __('Payment Method', 'give'),
+			'trans_id'  => __('Transaction ID', 'give'),
+			'key'       => __('Key', 'give'),
+			'date'      => __('Date', 'give'),
+			'user'      => __('User', 'give'),
+			'status'    => __('Status', 'give')
 		);
 
-		if ( ! give_get_option( 'enable_sequential' ) ) {
-			unset( $cols['seq_id'] );
+		if ( ! give_get_option('enable_sequential')) {
+			unset($cols['seq_id']);
 		}
 
 		return $cols;
@@ -86,12 +86,12 @@ 
 			'status' => $this->status
 		);
 
-		if ( ! empty( $this->start ) || ! empty( $this->end ) ) {
+		if ( ! empty($this->start) || ! empty($this->end)) {
 
 			$args['date_query'] = array(
 				array(
-					'after'     => date( 'Y-n-d 00:00:00', strtotime( $this->start ) ),
-					'before'    => date( 'Y-n-d 23:59:59', strtotime( $this->end ) ),
+					'after'     => date('Y-n-d 00:00:00', strtotime($this->start)),
+					'before'    => date('Y-n-d 23:59:59', strtotime($this->end)),
 					'inclusive' => true
 				)
 			);
@@ -99,11 +99,11 @@ 
 		}
 
 		// Add category or tag to payment query if any.
-		if ( ! empty( $this->categories ) || ! empty( $this->tags ) ) {
+		if ( ! empty($this->categories) || ! empty($this->tags)) {
 			$form_args = array(
 				'post_type'      => 'give_forms',
 				'post_status'    => 'publish',
-				'posts_per_page' => - 1,
+				'posts_per_page' => -1,
 				'fields'         => 'ids',
 				'tax_query'      => array(
 					'relation' => 'AND',
@@ -111,72 +111,72 @@ 
 			);
 
 
-			if ( ! empty( $this->categories ) ) {
+			if ( ! empty($this->categories)) {
 				$form_args['tax_query'][] = array(
 					'taxonomy' => 'give_forms_category',
 					'terms'    => $this->categories,
 				);
 			}
 
-			if ( ! empty( $this->tags ) ) {
+			if ( ! empty($this->tags)) {
 				$form_args['tax_query'][] = array(
 					'taxonomy' => 'give_forms_tag',
 					'terms'    => $this->tags,
 				);
 			}
 
-			$forms = new WP_Query( $form_args );
+			$forms = new WP_Query($form_args);
 
-			if ( empty( $forms->posts ) ) {
+			if (empty($forms->posts)) {
 				return array();
 			}
 
 			$args['give_forms'] = $forms->posts;
 		}
 
-		$payments = give_get_payments( $args );
+		$payments = give_get_payments($args);
 
-		if ( $payments ) {
+		if ($payments) {
 
-			foreach ( $payments as $payment ) {
-				$payment_meta = give_get_payment_meta( $payment->ID );
-				$user_info    = give_get_payment_meta_user_info( $payment->ID );
-				$total        = give_donation_amount( $payment->ID, array( 'type' => 'stats' ) );
-				$user_id      = isset( $user_info['id'] ) && $user_info['id'] != - 1 ? $user_info['id'] : $user_info['email'];
+			foreach ($payments as $payment) {
+				$payment_meta = give_get_payment_meta($payment->ID);
+				$user_info    = give_get_payment_meta_user_info($payment->ID);
+				$total        = give_donation_amount($payment->ID, array('type' => 'stats'));
+				$user_id      = isset($user_info['id']) && $user_info['id'] != - 1 ? $user_info['id'] : $user_info['email'];
 
-				if ( is_numeric( $user_id ) ) {
-					$user = get_userdata( $user_id );
+				if (is_numeric($user_id)) {
+					$user = get_userdata($user_id);
 				} else {
 					$user = false;
 				}
 
 				$data[] = array(
 					'id'        => $payment->ID,
-					'seq_id'    => give_get_payment_number( $payment->ID ),
+					'seq_id'    => give_get_payment_number($payment->ID),
 					'email'     => $payment_meta['email'],
 					'first'     => $user_info['first_name'],
 					'last'      => $user_info['last_name'],
-					'address1'  => isset( $user_info['address']['line1'] ) ? $user_info['address']['line1'] : '',
-					'address2'  => isset( $user_info['address']['line2'] ) ? $user_info['address']['line2'] : '',
-					'city'      => isset( $user_info['address']['city'] ) ? $user_info['address']['city'] : '',
-					'state'     => isset( $user_info['address']['state'] ) ? $user_info['address']['state'] : '',
-					'country'   => isset( $user_info['address']['country'] ) ? $user_info['address']['country'] : '',
-					'zip'       => isset( $user_info['address']['zip'] ) ? $user_info['address']['zip'] : '',
-					'form_id'   => isset( $payment_meta['form_id'] ) ? $payment_meta['form_id'] : '',
-					'form_name' => isset( $payment_meta['form_title'] ) ? $payment_meta['form_title'] : '',
-					'amount'    => html_entity_decode( give_format_amount( $total, array( 'sanitize' => false ) ) ),
-					'gateway'   => give_get_gateway_admin_label( give_get_meta( $payment->ID, '_give_payment_gateway', true ) ),
-					'trans_id'  => give_get_payment_transaction_id( $payment->ID ),
+					'address1'  => isset($user_info['address']['line1']) ? $user_info['address']['line1'] : '',
+					'address2'  => isset($user_info['address']['line2']) ? $user_info['address']['line2'] : '',
+					'city'      => isset($user_info['address']['city']) ? $user_info['address']['city'] : '',
+					'state'     => isset($user_info['address']['state']) ? $user_info['address']['state'] : '',
+					'country'   => isset($user_info['address']['country']) ? $user_info['address']['country'] : '',
+					'zip'       => isset($user_info['address']['zip']) ? $user_info['address']['zip'] : '',
+					'form_id'   => isset($payment_meta['form_id']) ? $payment_meta['form_id'] : '',
+					'form_name' => isset($payment_meta['form_title']) ? $payment_meta['form_title'] : '',
+					'amount'    => html_entity_decode(give_format_amount($total, array('sanitize' => false))),
+					'gateway'   => give_get_gateway_admin_label(give_get_meta($payment->ID, '_give_payment_gateway', true)),
+					'trans_id'  => give_get_payment_transaction_id($payment->ID),
 					'key'       => $payment_meta['key'],
 					'date'      => $payment->post_date,
-					'user'      => $user ? $user->display_name : __( 'guest', 'give' ),
-					'status'    => give_get_payment_status( $payment, true )
+					'user'      => $user ? $user->display_name : __('guest', 'give'),
+					'status'    => give_get_payment_status($payment, true)
 				);
 
 			}
 
-			$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;
 
@@ -196,27 +196,27 @@ 
 
 		$status = $this->status;
 		$args   = array(
-			'start-date' => date( 'n/d/Y', strtotime( $this->start ) ),
-			'end-date'   => date( 'n/d/Y', strtotime( $this->end ) ),
+			'start-date' => date('n/d/Y', strtotime($this->start)),
+			'end-date'   => date('n/d/Y', strtotime($this->end)),
 		);
 
-		if ( 'any' == $status ) {
+		if ('any' == $status) {
 
-			$total = array_sum( (array) give_count_payments( $args ) );
+			$total = array_sum((array) give_count_payments($args));
 
 		} else {
 
-			$total = give_count_payments( $args )->$status;
+			$total = give_count_payments($args)->$status;
 
 		}
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( 30 * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = ((30 * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -230,11 +230,11 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing.
 	 */
-	public function set_properties( $request ) {
-		$this->start      = isset( $request['start'] ) ? sanitize_text_field( $request['start'] ) : '';
-		$this->end        = isset( $request['end'] ) ? sanitize_text_field( $request['end'] ) : '';
-		$this->status     = isset( $request['status'] ) ? sanitize_text_field( $request['status'] ) : 'complete';
-		$this->categories = isset( $request['give_forms_categories'] ) ? give_clean( $request['give_forms_categories'] ) : array();
-		$this->tags       = isset( $request['give_forms_tags'] ) ? give_clean( $request['give_forms_tags'] ) : array();
+	public function set_properties($request) {
+		$this->start      = isset($request['start']) ? sanitize_text_field($request['start']) : '';
+		$this->end        = isset($request['end']) ? sanitize_text_field($request['end']) : '';
+		$this->status     = isset($request['status']) ? sanitize_text_field($request['status']) : 'complete';
+		$this->categories = isset($request['give_forms_categories']) ? give_clean($request['give_forms_categories']) : array();
+		$this->tags       = isset($request['give_forms_tags']) ? give_clean($request['give_forms_tags']) : array();
 	}
 }

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

Spacing +7 added lines, -7 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -42,9 +42,9 @@ 
 		give_ignore_user_abort();
 
 		nocache_headers();
-		header( 'Content-Type: application/json; charset=utf-8' );
-		header( 'Content-Disposition: attachment; filename=' . apply_filters( 'give_core_settings_export_filename', 'give-export-' . $this->export_type . '-' . date( 'n' ) . '-' . date( 'Y' ) ) . '.json' );
-		header( 'Expires: 0' );
+		header('Content-Type: application/json; charset=utf-8');
+		header('Content-Disposition: attachment; filename='.apply_filters('give_core_settings_export_filename', 'give-export-'.$this->export_type.'-'.date('n').'-'.date('Y')).'.json');
+		header('Expires: 0');
 	}
 
 	/**
@@ -54,7 +54,7 @@ 
 	 * @since 1.8.17
 	 */
 	public function json_core_settings_export() {
-		echo wp_json_encode( get_option( 'give_settings' ) );
+		echo wp_json_encode(get_option('give_settings'));
 	}
 
 	/**
@@ -64,8 +64,8 @@ 
 	 * @since  1.8.17
 	 */
 	public function export() {
-		if ( ! $this->can_export() ) {
-			wp_die( __( 'You do not have permission to export data.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(__('You do not have permission to export data.', 'give'), __('Error', 'give'), array('response' => 403));
 		}
 
 		// Set headers.

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

Spacing +24 added lines, -24 removed lines

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

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

Spacing +25 added lines, -25 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -41,9 +41,9 @@ 
 		give_ignore_user_abort();
 
 		nocache_headers();
-		header( 'Content-Type: text/csv; charset=utf-8' );
-		header( 'Content-Disposition: attachment; filename=' . apply_filters( 'give_earnings_export_filename', 'give-export-' . $this->export_type . '-' . date( 'n' ) . '-' . date( 'Y' ) ) . '.csv' );
-		header( "Expires: 0" );
+		header('Content-Type: text/csv; charset=utf-8');
+		header('Content-Disposition: attachment; filename='.apply_filters('give_earnings_export_filename', 'give-export-'.$this->export_type.'-'.date('n').'-'.date('Y')).'.csv');
+		header("Expires: 0");
 
 	}
 
@@ -57,10 +57,10 @@ 
 	public function csv_cols() {
 
 		$cols = array(
-			'date'      => __( 'Date', 'give' ),
-			'donations' => __( 'Donations', 'give' ),
+			'date'      => __('Date', 'give'),
+			'donations' => __('Donations', 'give'),
 			/* translators: %s: currency */
-			'earnings'  => sprintf( __( 'Income (%s)', 'give' ), give_currency_symbol('', true) )
+			'earnings'  => sprintf(__('Income (%s)', 'give'), give_currency_symbol('', true))
 		);
 
 		return $cols;
@@ -75,28 +75,28 @@ 
 	 */
 	public function get_data() {
 
-		$start_year  = isset( $_POST['start_year'] ) ? absint( $_POST['start_year'] ) : date( 'Y' );
-		$end_year    = isset( $_POST['end_year'] ) ? absint( $_POST['end_year'] ) : date( 'Y' );
-		$start_month = isset( $_POST['start_month'] ) ? absint( $_POST['start_month'] ) : date( 'n' );
-		$end_month   = isset( $_POST['end_month'] ) ? absint( $_POST['end_month'] ) : date( 'n' );
+		$start_year  = isset($_POST['start_year']) ? absint($_POST['start_year']) : date('Y');
+		$end_year    = isset($_POST['end_year']) ? absint($_POST['end_year']) : date('Y');
+		$start_month = isset($_POST['start_month']) ? absint($_POST['start_month']) : date('n');
+		$end_month   = isset($_POST['end_month']) ? absint($_POST['end_month']) : date('n');
 
 		$data  = array();
 		$year  = $start_year;
 		$stats = new Give_Payment_Stats;
 
-		while ( $year <= $end_year ) {
+		while ($year <= $end_year) {
 
-			if ( $year == $start_year && $year == $end_year ) {
+			if ($year == $start_year && $year == $end_year) {
 
 				$m1 = $start_month;
 				$m2 = $end_month;
 
-			} elseif ( $year == $start_year ) {
+			} elseif ($year == $start_year) {
 
 				$m1 = $start_month;
 				$m2 = 12;
 
-			} elseif ( $year == $end_year ) {
+			} elseif ($year == $end_year) {
 
 				$m1 = 1;
 				$m2 = $end_month;
@@ -108,28 +108,28 @@ 
 
 			}
 
-			while ( $m1 <= $m2 ) {
+			while ($m1 <= $m2) {
 
-				$date1 = mktime( 0, 0, 0, $m1, 1, $year );
-				$date2 = mktime( 0, 0, 0, $m1, cal_days_in_month( CAL_GREGORIAN, $m1, $year ), $year );
+				$date1 = mktime(0, 0, 0, $m1, 1, $year);
+				$date2 = mktime(0, 0, 0, $m1, cal_days_in_month(CAL_GREGORIAN, $m1, $year), $year);
 
 				$data[] = array(
-					'date'      => date_i18n( 'F Y', $date1 ),
-					'donations' => $stats->get_sales( 0, $date1, $date2 ),
-					'earnings'  => give_format_amount( $stats->get_earnings( 0, $date1, $date2 ), array( 'sanitize' => false ) ),
+					'date'      => date_i18n('F Y', $date1),
+					'donations' => $stats->get_sales(0, $date1, $date2),
+					'earnings'  => give_format_amount($stats->get_earnings(0, $date1, $date2), array('sanitize' => false)),
 				);
 
-				$m1 ++;
+				$m1++;
 
 			}
 
 
-			$year ++;
+			$year++;
 
 		}
 
-		$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;
 	}

includes/class-give-roles.php

Spacing +88 added lines, -88 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -35,9 +35,9 @@ 
 	 * @access public
 	 */
 	public function __construct() {
-		add_filter( 'give_map_meta_cap', array( $this, 'meta_caps' ), 10, 4 );
-		add_filter( 'woocommerce_disable_admin_bar', array( $this, 'manage_admin_dashboard' ), 10, 1 );
-		add_filter( 'woocommerce_prevent_admin_access', array( $this, 'manage_admin_dashboard' ), 10 );
+		add_filter('give_map_meta_cap', array($this, 'meta_caps'), 10, 4);
+		add_filter('woocommerce_disable_admin_bar', array($this, 'manage_admin_dashboard'), 10, 1);
+		add_filter('woocommerce_prevent_admin_access', array($this, 'manage_admin_dashboard'), 10);
 	}
 
 	/**
@@ -51,7 +51,7 @@ 
 	 * @return void
 	 */
 	public function add_roles() {
-		add_role( 'give_manager', __( 'Give Manager', 'give' ), array(
+		add_role('give_manager', __('Give Manager', 'give'), array(
 			'read'                   => true,
 			'edit_posts'             => true,
 			'delete_posts'           => true,
@@ -80,25 +80,25 @@ 
 			'publish_posts'          => true,
 			'read_private_pages'     => true,
 			'read_private_posts'     => true,
-		) );
+		));
 
-		add_role( 'give_accountant', __( 'Give Accountant', 'give' ), array(
+		add_role('give_accountant', __('Give Accountant', 'give'), array(
 			'read'         => true,
 			'edit_posts'   => false,
 			'delete_posts' => false,
-		) );
+		));
 
-		add_role( 'give_worker', __( 'Give Worker', 'give' ), array(
+		add_role('give_worker', __('Give Worker', 'give'), array(
 			'read'         => true,
 			'edit_posts'   => true,
 			'edit_pages'   => true,
 			'upload_files' => true,
 			'delete_posts' => false,
-		) );
+		));
 
-		add_role( 'give_donor', __( 'Give Donor', 'give' ), array(
+		add_role('give_donor', __('Give Donor', 'give'), array(
 			'read' => true,
-		) );
+		));
 
 	}
 
@@ -117,53 +117,53 @@ 
 	public function add_caps() {
 		global $wp_roles;
 
-		if ( class_exists( 'WP_Roles' ) ) {
-			if ( ! isset( $wp_roles ) ) {
+		if (class_exists('WP_Roles')) {
+			if ( ! isset($wp_roles)) {
 				$wp_roles = new WP_Roles();
 			}
 		}
 
-		if ( is_object( $wp_roles ) ) {
-			$wp_roles->add_cap( 'give_manager', 'view_give_reports' );
-			$wp_roles->add_cap( 'give_manager', 'view_give_sensitive_data' );
-			$wp_roles->add_cap( 'give_manager', 'export_give_reports' );
-			$wp_roles->add_cap( 'give_manager', 'manage_give_settings' );
+		if (is_object($wp_roles)) {
+			$wp_roles->add_cap('give_manager', 'view_give_reports');
+			$wp_roles->add_cap('give_manager', 'view_give_sensitive_data');
+			$wp_roles->add_cap('give_manager', 'export_give_reports');
+			$wp_roles->add_cap('give_manager', 'manage_give_settings');
 
-			$wp_roles->add_cap( 'administrator', 'view_give_reports' );
-			$wp_roles->add_cap( 'administrator', 'view_give_sensitive_data' );
-			$wp_roles->add_cap( 'administrator', 'export_give_reports' );
-			$wp_roles->add_cap( 'administrator', 'manage_give_settings' );
-			$wp_roles->add_cap( 'administrator', 'view_give_payments' );
+			$wp_roles->add_cap('administrator', 'view_give_reports');
+			$wp_roles->add_cap('administrator', 'view_give_sensitive_data');
+			$wp_roles->add_cap('administrator', 'export_give_reports');
+			$wp_roles->add_cap('administrator', 'manage_give_settings');
+			$wp_roles->add_cap('administrator', 'view_give_payments');
 
 			// Add the main post type capabilities.
 			$capabilities = $this->get_core_caps();
-			foreach ( $capabilities as $cap_group ) {
-				foreach ( $cap_group as $cap ) {
-					$wp_roles->add_cap( 'administrator', $cap );
-					$wp_roles->add_cap( 'give_manager', $cap );
+			foreach ($capabilities as $cap_group) {
+				foreach ($cap_group as $cap) {
+					$wp_roles->add_cap('administrator', $cap);
+					$wp_roles->add_cap('give_manager', $cap);
 				}
 			}
 
 			// Add Capabilities to Give Workers User Role.
-			$wp_roles->add_cap( 'give_worker', 'edit_give_payments' );
-			$wp_roles->add_cap( 'give_worker', 'delete_give_forms' );
-			$wp_roles->add_cap( 'give_worker', 'delete_others_give_forms' );
-			$wp_roles->add_cap( 'give_worker', 'delete_private_give_forms' );
-			$wp_roles->add_cap( 'give_worker', 'delete_published_give_forms' );
-			$wp_roles->add_cap( 'give_worker', 'edit_give_forms' );
-			$wp_roles->add_cap( 'give_worker', 'edit_others_give_forms' );
-			$wp_roles->add_cap( 'give_worker', 'edit_private_give_forms' );
-			$wp_roles->add_cap( 'give_worker', 'edit_published_give_forms' );
-			$wp_roles->add_cap( 'give_worker', 'publish_give_forms' );
-			$wp_roles->add_cap( 'give_worker', 'read_private_give_forms' );
+			$wp_roles->add_cap('give_worker', 'edit_give_payments');
+			$wp_roles->add_cap('give_worker', 'delete_give_forms');
+			$wp_roles->add_cap('give_worker', 'delete_others_give_forms');
+			$wp_roles->add_cap('give_worker', 'delete_private_give_forms');
+			$wp_roles->add_cap('give_worker', 'delete_published_give_forms');
+			$wp_roles->add_cap('give_worker', 'edit_give_forms');
+			$wp_roles->add_cap('give_worker', 'edit_others_give_forms');
+			$wp_roles->add_cap('give_worker', 'edit_private_give_forms');
+			$wp_roles->add_cap('give_worker', 'edit_published_give_forms');
+			$wp_roles->add_cap('give_worker', 'publish_give_forms');
+			$wp_roles->add_cap('give_worker', 'read_private_give_forms');
 
 			// Add Capabilities to Give Accountant User Role.
-			$wp_roles->add_cap( 'give_accountant', 'edit_give_forms' );
-			$wp_roles->add_cap( 'give_accountant', 'read_private_give_forms' );
-			$wp_roles->add_cap( 'give_accountant', 'view_give_reports' );
-			$wp_roles->add_cap( 'give_accountant', 'export_give_reports' );
-			$wp_roles->add_cap( 'give_accountant', 'edit_give_payments' );
-			$wp_roles->add_cap( 'give_accountant', 'view_give_payments' );
+			$wp_roles->add_cap('give_accountant', 'edit_give_forms');
+			$wp_roles->add_cap('give_accountant', 'read_private_give_forms');
+			$wp_roles->add_cap('give_accountant', 'view_give_reports');
+			$wp_roles->add_cap('give_accountant', 'export_give_reports');
+			$wp_roles->add_cap('give_accountant', 'edit_give_payments');
+			$wp_roles->add_cap('give_accountant', 'view_give_payments');
 
 		}
 	}
@@ -181,10 +181,10 @@ 
 	public function get_core_caps() {
 		$capabilities = array();
 
-		$capability_types = array( 'give_form', 'give_payment' );
+		$capability_types = array('give_form', 'give_payment');
 
-		foreach ( $capability_types as $capability_type ) {
-			$capabilities[ $capability_type ] = array(
+		foreach ($capability_types as $capability_type) {
+			$capabilities[$capability_type] = array(
 				// Post type.
 				"edit_{$capability_type}",
 				"read_{$capability_type}",
@@ -230,22 +230,22 @@ 
 	 *
 	 * @return array  $caps    Meta capabilities.
 	 */
-	public function meta_caps( $caps, $cap, $user_id, $args ) {
+	public function meta_caps($caps, $cap, $user_id, $args) {
 
-		switch ( $cap ) {
+		switch ($cap) {
 
 			case 'view_give_form_stats' :
 
-				if ( empty( $args[0] ) ) {
+				if (empty($args[0])) {
 					break;
 				}
 
-				$form = get_post( $args[0] );
-				if ( empty( $form ) ) {
+				$form = get_post($args[0]);
+				if (empty($form)) {
 					break;
 				}
 
-				if ( user_can( $user_id, 'view_give_reports' ) || $user_id == $form->post_author ) {
+				if (user_can($user_id, 'view_give_reports') || $user_id == $form->post_author) {
 					$caps = array();
 				}
 
@@ -272,57 +272,57 @@ 
 
 		global $wp_roles;
 
-		if ( class_exists( 'WP_Roles' ) ) {
-			if ( ! isset( $wp_roles ) ) {
+		if (class_exists('WP_Roles')) {
+			if ( ! isset($wp_roles)) {
 				$wp_roles = new WP_Roles();
 			}
 		}
 
-		if ( is_object( $wp_roles ) ) {
+		if (is_object($wp_roles)) {
 			// Give Manager Capabilities.
-			$wp_roles->remove_cap( 'give_manager', 'view_give_reports' );
-			$wp_roles->remove_cap( 'give_manager', 'view_give_sensitive_data' );
-			$wp_roles->remove_cap( 'give_manager', 'export_give_reports' );
-			$wp_roles->remove_cap( 'give_manager', 'manage_give_settings' );
+			$wp_roles->remove_cap('give_manager', 'view_give_reports');
+			$wp_roles->remove_cap('give_manager', 'view_give_sensitive_data');
+			$wp_roles->remove_cap('give_manager', 'export_give_reports');
+			$wp_roles->remove_cap('give_manager', 'manage_give_settings');
 
 			// Site Administrator Capabilities.
-			$wp_roles->remove_cap( 'administrator', 'view_give_reports' );
-			$wp_roles->remove_cap( 'administrator', 'view_give_sensitive_data' );
-			$wp_roles->remove_cap( 'administrator', 'export_give_reports' );
-			$wp_roles->remove_cap( 'administrator', 'manage_give_settings' );
-			$wp_roles->remove_cap( 'administrator', 'view_give_payments' );
+			$wp_roles->remove_cap('administrator', 'view_give_reports');
+			$wp_roles->remove_cap('administrator', 'view_give_sensitive_data');
+			$wp_roles->remove_cap('administrator', 'export_give_reports');
+			$wp_roles->remove_cap('administrator', 'manage_give_settings');
+			$wp_roles->remove_cap('administrator', 'view_give_payments');
 
 			// Remove the Main Post Type Capabilities.
 			$capabilities = $this->get_core_caps();
 
-			foreach ( $capabilities as $cap_group ) {
-				foreach ( $cap_group as $cap ) {
-					$wp_roles->remove_cap( 'give_manager', $cap );
-					$wp_roles->remove_cap( 'administrator', $cap );
+			foreach ($capabilities as $cap_group) {
+				foreach ($cap_group as $cap) {
+					$wp_roles->remove_cap('give_manager', $cap);
+					$wp_roles->remove_cap('administrator', $cap);
 
 				}
 			}
 
 			// Remove capabilities from the Give Worker role.
-			$wp_roles->remove_cap( 'give_worker', 'edit_give_payments' );
-			$wp_roles->remove_cap( 'give_worker', 'delete_give_forms' );
-			$wp_roles->remove_cap( 'give_worker', 'delete_others_give_forms' );
-			$wp_roles->remove_cap( 'give_worker', 'delete_private_give_forms' );
-			$wp_roles->remove_cap( 'give_worker', 'delete_published_give_forms' );
-			$wp_roles->remove_cap( 'give_worker', 'edit_give_forms' );
-			$wp_roles->remove_cap( 'give_worker', 'edit_others_give_forms' );
-			$wp_roles->remove_cap( 'give_worker', 'edit_private_give_forms' );
-			$wp_roles->remove_cap( 'give_worker', 'edit_published_give_forms' );
-			$wp_roles->remove_cap( 'give_worker', 'publish_give_forms' );
-			$wp_roles->remove_cap( 'give_worker', 'read_private_give_forms' );
+			$wp_roles->remove_cap('give_worker', 'edit_give_payments');
+			$wp_roles->remove_cap('give_worker', 'delete_give_forms');
+			$wp_roles->remove_cap('give_worker', 'delete_others_give_forms');
+			$wp_roles->remove_cap('give_worker', 'delete_private_give_forms');
+			$wp_roles->remove_cap('give_worker', 'delete_published_give_forms');
+			$wp_roles->remove_cap('give_worker', 'edit_give_forms');
+			$wp_roles->remove_cap('give_worker', 'edit_others_give_forms');
+			$wp_roles->remove_cap('give_worker', 'edit_private_give_forms');
+			$wp_roles->remove_cap('give_worker', 'edit_published_give_forms');
+			$wp_roles->remove_cap('give_worker', 'publish_give_forms');
+			$wp_roles->remove_cap('give_worker', 'read_private_give_forms');
 
 			// Remove Capabilities from Give Accountant User Role.
-			$wp_roles->remove_cap( 'give_accountant', 'edit_give_forms' );
-			$wp_roles->remove_cap( 'give_accountant', 'read_private_give_forms' );
-			$wp_roles->remove_cap( 'give_accountant', 'view_give_reports' );
-			$wp_roles->remove_cap( 'give_accountant', 'export_give_reports' );
-			$wp_roles->remove_cap( 'give_accountant', 'edit_give_payments' );
-			$wp_roles->remove_cap( 'give_accountant', 'view_give_payments' );
+			$wp_roles->remove_cap('give_accountant', 'edit_give_forms');
+			$wp_roles->remove_cap('give_accountant', 'read_private_give_forms');
+			$wp_roles->remove_cap('give_accountant', 'view_give_reports');
+			$wp_roles->remove_cap('give_accountant', 'export_give_reports');
+			$wp_roles->remove_cap('give_accountant', 'edit_give_payments');
+			$wp_roles->remove_cap('give_accountant', 'view_give_payments');
 
 		}
 	}
@@ -345,7 +345,7 @@ 
 		$current_user = wp_get_current_user();
 
 		// If user with "Give Accountant" user role is logged-in .
-		if ( 0 !== $current_user->ID && in_array( 'give_accountant', (array) $current_user->roles, true ) ) {
+		if (0 !== $current_user->ID && in_array('give_accountant', (array) $current_user->roles, true)) {
 
 			// Return false, means no prevention.
 			return false;

includes/currency-functions.php

Spacing +138 added lines, -138 removed lines

@@ -20,19 +20,19 @@ 
  *
  * @return string The currency code
  */
-function give_get_currency( $donation_or_form_id = null, $args = array() ) {
+function give_get_currency($donation_or_form_id = null, $args = array()) {
 
 	// Get currency from donation
-	if ( is_numeric( $donation_or_form_id ) && 'give_payment' === get_post_type( $donation_or_form_id ) ) {
-		$donation_meta = give_get_meta( $donation_or_form_id, '_give_payment_meta', true );
+	if (is_numeric($donation_or_form_id) && 'give_payment' === get_post_type($donation_or_form_id)) {
+		$donation_meta = give_get_meta($donation_or_form_id, '_give_payment_meta', true);
 
-		if ( ! empty( $donation_meta['currency'] ) ) {
+		if ( ! empty($donation_meta['currency'])) {
 			$currency = $donation_meta['currency'];
 		} else {
-			$currency = give_get_option( 'currency', 'USD' );
+			$currency = give_get_option('currency', 'USD');
 		}
 	} else {
-		$currency = give_get_option( 'currency', 'USD' );
+		$currency = give_get_option('currency', 'USD');
 	}
 
 	/**
@@ -40,7 +40,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	return apply_filters( 'give_currency', $currency, $donation_or_form_id, $args );
+	return apply_filters('give_currency', $currency, $donation_or_form_id, $args);
 }
 
 /**
@@ -52,9 +52,9 @@ 
  */
 function give_get_currency_position() {
 
-	$currency_pos = give_get_option( 'currency_position', 'before' );
+	$currency_pos = give_get_option('currency_position', 'before');
 
-	return apply_filters( 'give_currency_position', $currency_pos );
+	return apply_filters('give_currency_position', $currency_pos);
 }
 
 /**
@@ -67,7 +67,7 @@ 
 function give_get_currencies_list() {
 	$currencies = array(
 		'USD' => array(
-			'admin_label' => __( 'US Dollars ($)', 'give' ),
+			'admin_label' => __('US Dollars ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -77,7 +77,7 @@ 
 			),
 		),
 		'EUR' => array(
-			'admin_label' => __( 'Euros (€)', 'give' ),
+			'admin_label' => __('Euros (€)', 'give'),
 			'symbol'      => '€',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -87,7 +87,7 @@ 
 			),
 		),
 		'GBP' => array(
-			'admin_label' => __( 'Pounds Sterling (£)', 'give' ),
+			'admin_label' => __('Pounds Sterling (£)', 'give'),
 			'symbol'      => '£',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -97,7 +97,7 @@ 
 			),
 		),
 		'AUD' => array(
-			'admin_label' => __( 'Australian Dollars ($)', 'give' ),
+			'admin_label' => __('Australian Dollars ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -107,7 +107,7 @@ 
 			),
 		),
 		'BRL' => array(
-			'admin_label' => __( 'Brazilian Real (R$)', 'give' ),
+			'admin_label' => __('Brazilian Real (R$)', 'give'),
 			'symbol'      => 'R$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -117,7 +117,7 @@ 
 			),
 		),
 		'CAD' => array(
-			'admin_label' => __( 'Canadian Dollars ($)', 'give' ),
+			'admin_label' => __('Canadian Dollars ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -127,7 +127,7 @@ 
 			),
 		),
 		'CZK' => array(
-			'admin_label' => __( 'Czech Koruna (Kč)', 'give' ),
+			'admin_label' => __('Czech Koruna (Kč)', 'give'),
 			'symbol'      => 'Kč',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -137,7 +137,7 @@ 
 			),
 		),
 		'DKK' => array(
-			'admin_label' => __( 'Danish Krone (kr.)', 'give' ),
+			'admin_label' => __('Danish Krone (kr.)', 'give'),
 			'symbol'      => ' kr. ',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -147,7 +147,7 @@ 
 			),
 		),
 		'HKD' => array(
-			'admin_label' => __( 'Hong Kong Dollar ($)', 'give' ),
+			'admin_label' => __('Hong Kong Dollar ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -157,7 +157,7 @@ 
 			),
 		),
 		'HUF' => array(
-			'admin_label' => __( 'Hungarian Forint (Ft)', 'give' ),
+			'admin_label' => __('Hungarian Forint (Ft)', 'give'),
 			'symbol'      => 'Ft',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -167,7 +167,7 @@ 
 			),
 		),
 		'ILS' => array(
-			'admin_label' => __( 'Israeli Shekel (₪)', 'give' ),
+			'admin_label' => __('Israeli Shekel (₪)', 'give'),
 			'symbol'      => '₪',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -177,7 +177,7 @@ 
 			),
 		),
 		'JPY' => array(
-			'admin_label' => __( 'Japanese Yen (¥)', 'give' ),
+			'admin_label' => __('Japanese Yen (¥)', 'give'),
 			'symbol'      => '¥',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -187,7 +187,7 @@ 
 			),
 		),
 		'MYR' => array(
-			'admin_label' => __( 'Malaysian Ringgits (RM)', 'give' ),
+			'admin_label' => __('Malaysian Ringgits (RM)', 'give'),
 			'symbol'      => 'RM',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -197,7 +197,7 @@ 
 			),
 		),
 		'MXN' => array(
-			'admin_label' => __( 'Mexican Peso ($)', 'give' ),
+			'admin_label' => __('Mexican Peso ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -207,7 +207,7 @@ 
 			),
 		),
 		'MAD' => array(
-			'admin_label' => __( 'Moroccan Dirham (.د.م)', 'give' ),
+			'admin_label' => __('Moroccan Dirham (.د.م)', 'give'),
 			'symbol'      => '.د.م',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -217,7 +217,7 @@ 
 			),
 		),
 		'NZD' => array(
-			'admin_label' => __( 'New Zealand Dollar ($)', 'give' ),
+			'admin_label' => __('New Zealand Dollar ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -227,7 +227,7 @@ 
 			),
 		),
 		'NOK' => array(
-			'admin_label' => __( 'Norwegian Krone (Kr.)', 'give' ),
+			'admin_label' => __('Norwegian Krone (Kr.)', 'give'),
 			'symbol'      => 'kr.',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -237,7 +237,7 @@ 
 			),
 		),
 		'PHP' => array(
-			'admin_label' => __( 'Philippine Pesos (₱)', 'give' ),
+			'admin_label' => __('Philippine Pesos (₱)', 'give'),
 			'symbol'      => '₱',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -247,7 +247,7 @@ 
 			),
 		),
 		'PLN' => array(
-			'admin_label' => __( 'Polish Zloty (zł)', 'give' ),
+			'admin_label' => __('Polish Zloty (zł)', 'give'),
 			'symbol'      => 'zł',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -257,7 +257,7 @@ 
 			),
 		),
 		'SGD' => array(
-			'admin_label' => __( 'Singapore Dollar ($)', 'give' ),
+			'admin_label' => __('Singapore Dollar ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -267,7 +267,7 @@ 
 			),
 		),
 		'KRW' => array(
-			'admin_label' => __( 'South Korean Won (₩)', 'give' ),
+			'admin_label' => __('South Korean Won (₩)', 'give'),
 			'symbol'      => '₩',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -277,7 +277,7 @@ 
 			),
 		),
 		'ZAR' => array(
-			'admin_label' => __( 'South African Rand (R)', 'give' ),
+			'admin_label' => __('South African Rand (R)', 'give'),
 			'symbol'      => 'R',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -287,7 +287,7 @@ 
 			),
 		),
 		'SEK' => array(
-			'admin_label' => __( 'Swedish Krona (kr)', 'give' ),
+			'admin_label' => __('Swedish Krona (kr)', 'give'),
 			'symbol'      => ' kr. ',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -297,7 +297,7 @@ 
 			),
 		),
 		'CHF' => array(
-			'admin_label' => __( 'Swiss Franc (Fr)', 'give' ),
+			'admin_label' => __('Swiss Franc (Fr)', 'give'),
 			'symbol'      => 'Fr',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -307,7 +307,7 @@ 
 			),
 		),
 		'TWD' => array(
-			'admin_label' => __( 'Taiwan New Dollars (NT$)', 'give' ),
+			'admin_label' => __('Taiwan New Dollars (NT$)', 'give'),
 			'symbol'      => 'NT$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -317,7 +317,7 @@ 
 			),
 		),
 		'THB' => array(
-			'admin_label' => __( 'Thai Baht (฿)', 'give' ),
+			'admin_label' => __('Thai Baht (฿)', 'give'),
 			'symbol'      => '฿',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -327,7 +327,7 @@ 
 			),
 		),
 		'INR' => array(
-			'admin_label' => __( 'Indian Rupee (₹)', 'give' ),
+			'admin_label' => __('Indian Rupee (₹)', 'give'),
 			'symbol'      => '₹',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -337,7 +337,7 @@ 
 			),
 		),
 		'TRY' => array(
-			'admin_label' => __( 'Turkish Lira (₺)', 'give' ),
+			'admin_label' => __('Turkish Lira (₺)', 'give'),
 			'symbol'      => '₺',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -347,7 +347,7 @@ 
 			),
 		),
 		'IRR' => array(
-			'admin_label' => __( 'Iranian Rial (﷼)', 'give' ),
+			'admin_label' => __('Iranian Rial (﷼)', 'give'),
 			'symbol'      => '﷼',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -357,7 +357,7 @@ 
 			),
 		),
 		'RUB' => array(
-			'admin_label' => __( 'Russian Rubles (руб)', 'give' ),
+			'admin_label' => __('Russian Rubles (руб)', 'give'),
 			'symbol'      => '₽',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -367,7 +367,7 @@ 
 			),
 		),
 		'AED' => array(
-			'admin_label' => __( 'United Arab Emirates dirham (د.إ)', 'give' ),
+			'admin_label' => __('United Arab Emirates dirham (د.إ)', 'give'),
 			'symbol'      => 'د.إ',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -377,7 +377,7 @@ 
 			),
 		),
 		'AMD' => array(
-			'admin_label' => __( 'Armenian dram (AMD)', 'give' ),
+			'admin_label' => __('Armenian dram (AMD)', 'give'),
 			'symbol'      => 'AMD', // Add backward compatibility. Using AMD in place of ֏
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -387,7 +387,7 @@ 
 			),
 		),
 		'ANG' => array(
-			'admin_label' => __( 'Netherlands Antillean guilder (ƒ)', 'give' ),
+			'admin_label' => __('Netherlands Antillean guilder (ƒ)', 'give'),
 			'symbol'      => 'ƒ',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -397,7 +397,7 @@ 
 			),
 		),
 		'ARS' => array(
-			'admin_label' => __( 'Argentine peso ($)', 'give' ),
+			'admin_label' => __('Argentine peso ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -407,7 +407,7 @@ 
 			),
 		),
 		'AWG' => array(
-			'admin_label' => __( 'Aruban florin (ƒ)', 'give' ),
+			'admin_label' => __('Aruban florin (ƒ)', 'give'),
 			'symbol'      => 'ƒ',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -417,7 +417,7 @@ 
 			),
 		),
 		'BAM' => array(
-			'admin_label' => __( 'Bosnia and Herzegovina convertible mark (KM)', 'give' ),
+			'admin_label' => __('Bosnia and Herzegovina convertible mark (KM)', 'give'),
 			'symbol'      => 'KM',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -427,7 +427,7 @@ 
 			),
 		),
 		'BDT' => array(
-			'admin_label' => __( 'Bangladeshi taka (৳)', 'give' ),
+			'admin_label' => __('Bangladeshi taka (৳)', 'give'),
 			'symbol'      => '৳',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -437,7 +437,7 @@ 
 			),
 		),
 		'BHD' => array(
-			'admin_label' => __( 'Bahraini dinar (.د.ب)', 'give' ),
+			'admin_label' => __('Bahraini dinar (.د.ب)', 'give'),
 			'symbol'      => '.د.ب',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -447,7 +447,7 @@ 
 			),
 		),
 		'BMD' => array(
-			'admin_label' => __( 'Bermudian dollar (BD$)', 'give' ),
+			'admin_label' => __('Bermudian dollar (BD$)', 'give'),
 			'symbol'      => 'BD$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -457,7 +457,7 @@ 
 			),
 		),
 		'BND' => array(
-			'admin_label' => __( 'Brunei dollar (B$)', 'give' ),
+			'admin_label' => __('Brunei dollar (B$)', 'give'),
 			'symbol'      => 'B$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -467,7 +467,7 @@ 
 			),
 		),
 		'BOB' => array(
-			'admin_label' => __( 'Bolivian boliviano (Bs.)', 'give' ),
+			'admin_label' => __('Bolivian boliviano (Bs.)', 'give'),
 			'symbol'      => 'Bs.',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -477,7 +477,7 @@ 
 			),
 		),
 		'BSD' => array(
-			'admin_label' => __( 'Bahamian dollar (B$)', 'give' ),
+			'admin_label' => __('Bahamian dollar (B$)', 'give'),
 			'symbol'      => 'B$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -487,7 +487,7 @@ 
 			),
 		),
 		'BWP' => array(
-			'admin_label' => __( 'Botswana pula (P)', 'give' ),
+			'admin_label' => __('Botswana pula (P)', 'give'),
 			'symbol'      => 'P',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -497,7 +497,7 @@ 
 			),
 		),
 		'BZD' => array(
-			'admin_label' => __( 'Belizean dollar (BZ$)', 'give' ),
+			'admin_label' => __('Belizean dollar (BZ$)', 'give'),
 			'symbol'      => 'BZ$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -507,7 +507,7 @@ 
 			),
 		),
 		'CLP' => array(
-			'admin_label' => __( 'Chilean peso ($)', 'give' ),
+			'admin_label' => __('Chilean peso ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -517,7 +517,7 @@ 
 			),
 		),
 		'CNY' => array(
-			'admin_label' => __( 'Chinese yuan (¥)', 'give' ),
+			'admin_label' => __('Chinese yuan (¥)', 'give'),
 			'symbol'      => '¥',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -527,7 +527,7 @@ 
 			),
 		),
 		'COP' => array(
-			'admin_label' => __( 'Colombian peso ($)', 'give' ),
+			'admin_label' => __('Colombian peso ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -537,7 +537,7 @@ 
 			),
 		),
 		'CRC' => array(
-			'admin_label' => __( 'Costa Rican colón (₡)', 'give' ),
+			'admin_label' => __('Costa Rican colón (₡)', 'give'),
 			'symbol'      => '₡',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -547,7 +547,7 @@ 
 			),
 		),
 		'CUC' => array(
-			'admin_label' => __( 'Cuban convertible peso (₱)', 'give' ),
+			'admin_label' => __('Cuban convertible peso (₱)', 'give'),
 			'symbol'      => '₱',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -557,7 +557,7 @@ 
 			),
 		),
 		'CUP' => array(
-			'admin_label' => __( 'Cuban convertible peso (₱)', 'give' ),
+			'admin_label' => __('Cuban convertible peso (₱)', 'give'),
 			'symbol'      => '₱',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -567,7 +567,7 @@ 
 			),
 		),
 		'DOP' => array(
-			'admin_label' => __( 'Dominican peso (RD$)', 'give' ),
+			'admin_label' => __('Dominican peso (RD$)', 'give'),
 			'symbol'      => 'RD$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -577,7 +577,7 @@ 
 			),
 		),
 		'EGP' => array(
-			'admin_label' => __( 'Egyptian pound (E£)', 'give' ),
+			'admin_label' => __('Egyptian pound (E£)', 'give'),
 			'symbol'      => 'E£',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -587,7 +587,7 @@ 
 			),
 		),
 		'GIP' => array(
-			'admin_label' => __( 'Gibraltar pound (£)', 'give' ),
+			'admin_label' => __('Gibraltar pound (£)', 'give'),
 			'symbol'      => '£',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -597,7 +597,7 @@ 
 			),
 		),
 		'GTQ' => array(
-			'admin_label' => __( 'Guatemalan quetzal (Q)', 'give' ),
+			'admin_label' => __('Guatemalan quetzal (Q)', 'give'),
 			'symbol'      => 'Q',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -607,7 +607,7 @@ 
 			),
 		),
 		'HNL' => array(
-			'admin_label' => __( 'Honduran lempira (L)', 'give' ),
+			'admin_label' => __('Honduran lempira (L)', 'give'),
 			'symbol'      => 'L',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -617,7 +617,7 @@ 
 			),
 		),
 		'HRK' => array(
-			'admin_label' => __( 'Croatian kuna (kn)', 'give' ),
+			'admin_label' => __('Croatian kuna (kn)', 'give'),
 			'symbol'      => 'kn',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -627,7 +627,7 @@ 
 			),
 		),
 		'IDR' => array(
-			'admin_label' => __( 'Indonesian rupiah (Rp)', 'give' ),
+			'admin_label' => __('Indonesian rupiah (Rp)', 'give'),
 			'symbol'      => 'Rp',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -637,7 +637,7 @@ 
 			),
 		),
 		'ISK' => array(
-			'admin_label' => __( 'Icelandic króna (kr)', 'give' ),
+			'admin_label' => __('Icelandic króna (kr)', 'give'),
 			'symbol'      => 'kr',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -647,7 +647,7 @@ 
 			),
 		),
 		'JMD' => array(
-			'admin_label' => __( 'Jamaican dollar (J$)', 'give' ),
+			'admin_label' => __('Jamaican dollar (J$)', 'give'),
 			'symbol'      => 'j$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -657,7 +657,7 @@ 
 			),
 		),
 		'JOD' => array(
-			'admin_label' => __( 'Jordanian dinar (د.ا)', 'give' ),
+			'admin_label' => __('Jordanian dinar (د.ا)', 'give'),
 			'symbol'      => 'د.ا',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -667,7 +667,7 @@ 
 			),
 		),
 		'KES' => array(
-			'admin_label' => __( 'Kenyan shilling (KSh)', 'give' ),
+			'admin_label' => __('Kenyan shilling (KSh)', 'give'),
 			'symbol'      => 'KSh',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -677,7 +677,7 @@ 
 			),
 		),
 		'KWD' => array(
-			'admin_label' => __( 'Kuwaiti dinar (د.ك)', 'give' ),
+			'admin_label' => __('Kuwaiti dinar (د.ك)', 'give'),
 			'symbol'      => 'د.ك',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -687,7 +687,7 @@ 
 			),
 		),
 		'KYD' => array(
-			'admin_label' => __( 'Cayman Islands dollar (KY$)', 'give' ),
+			'admin_label' => __('Cayman Islands dollar (KY$)', 'give'),
 			'symbol'      => 'KY$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -697,7 +697,7 @@ 
 			),
 		),
 		'MKD' => array(
-			'admin_label' => __( 'Macedonian denar (ден)', 'give' ),
+			'admin_label' => __('Macedonian denar (ден)', 'give'),
 			'symbol'      => 'ден',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -707,7 +707,7 @@ 
 			),
 		),
 		'NPR' => array(
-			'admin_label' => __( 'Nepalese rupee (NɌs)', 'give' ),
+			'admin_label' => __('Nepalese rupee (NɌs)', 'give'),
 			'symbol'      => '₨',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -717,7 +717,7 @@ 
 			),
 		),
 		'OMR' => array(
-			'admin_label' => __( 'Omani rial (ر.ع.)', 'give' ),
+			'admin_label' => __('Omani rial (ر.ع.)', 'give'),
 			'symbol'      => 'ر.ع.',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -727,7 +727,7 @@ 
 			),
 		),
 		'PEN' => array(
-			'admin_label' => __( 'Peruvian nuevo sol (S/.)', 'give' ),
+			'admin_label' => __('Peruvian nuevo sol (S/.)', 'give'),
 			'symbol'      => 'S/.',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -737,7 +737,7 @@ 
 			),
 		),
 		'PKR' => array(
-			'admin_label' => __( 'Pakistani rupee (Ɍs)', 'give' ),
+			'admin_label' => __('Pakistani rupee (Ɍs)', 'give'),
 			'symbol'      => '₨',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -747,7 +747,7 @@ 
 			),
 		),
 		'RON' => array(
-			'admin_label' => __( 'Romanian leu (L)', 'give' ),
+			'admin_label' => __('Romanian leu (L)', 'give'),
 			'symbol'      => 'L',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -757,7 +757,7 @@ 
 			),
 		),
 		'SAR' => array(
-			'admin_label' => __( 'Saudi riyal (ر.س)', 'give' ),
+			'admin_label' => __('Saudi riyal (ر.س)', 'give'),
 			'symbol'      => 'ر.س',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -767,7 +767,7 @@ 
 			),
 		),
 		'SZL' => array(
-			'admin_label' => __( 'Swazi lilangeni (L)', 'give' ),
+			'admin_label' => __('Swazi lilangeni (L)', 'give'),
 			'symbol'      => 'L',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -777,7 +777,7 @@ 
 			),
 		),
 		'TOP' => array(
-			'admin_label' => __( 'Tongan paʻanga (T$)', 'give' ),
+			'admin_label' => __('Tongan paʻanga (T$)', 'give'),
 			'symbol'      => 'T$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -787,7 +787,7 @@ 
 			),
 		),
 		'TZS' => array(
-			'admin_label' => __( 'Tanzanian shilling (TSh)', 'give' ),
+			'admin_label' => __('Tanzanian shilling (TSh)', 'give'),
 			'symbol'      => 'TSh',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -797,7 +797,7 @@ 
 			),
 		),
 		'UAH' => array(
-			'admin_label' => __( 'Ukrainian hryvnia (₴)', 'give' ),
+			'admin_label' => __('Ukrainian hryvnia (₴)', 'give'),
 			'symbol'      => '₴',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -807,7 +807,7 @@ 
 			),
 		),
 		'UYU' => array(
-			'admin_label' => __( 'Uruguayan peso ($U)', 'give' ),
+			'admin_label' => __('Uruguayan peso ($U)', 'give'),
 			'symbol'      => '$U',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -817,7 +817,7 @@ 
 			),
 		),
 		'VEF' => array(
-			'admin_label' => __( 'Venezuelan bolívar (Bs)', 'give' ),
+			'admin_label' => __('Venezuelan bolívar (Bs)', 'give'),
 			'symbol'      => 'Bs',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -827,7 +827,7 @@ 
 			),
 		),
 		'XCD' => array(
-			'admin_label' => __( 'East Caribbean dollar (EC$)', 'give' ),
+			'admin_label' => __('East Caribbean dollar (EC$)', 'give'),
 			'symbol'      => 'EC$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -852,7 +852,7 @@ 
 	 *
 	 * @param array $currencies
 	 */
-	return (array) apply_filters( 'give_currencies', $currencies );
+	return (array) apply_filters('give_currencies', $currencies);
 }
 
 /**
@@ -864,22 +864,22 @@ 
  *
  * @return array $currencies A list of the available currencies
  */
-function give_get_currencies( $info = 'admin_label' ) {
+function give_get_currencies($info = 'admin_label') {
 
 	$currencies = give_get_currencies_list();
 
 	// Backward compatibility: handle old way of currency registration.
 	// Backward compatibility: Return desired result.
-	if ( ! empty( $currencies ) ) {
-		foreach ( $currencies as $currency_code => $currency_setting ) {
-			if ( is_string( $currency_setting ) ) {
-				$currencies[ $currency_code ] = array(
+	if ( ! empty($currencies)) {
+		foreach ($currencies as $currency_code => $currency_setting) {
+			if (is_string($currency_setting)) {
+				$currencies[$currency_code] = array(
 					'admin_label' => $currency_setting,
 				);
 			}
 
-			$currencies[ $currency_code ] = wp_parse_args(
-				$currencies[ $currency_code ],
+			$currencies[$currency_code] = wp_parse_args(
+				$currencies[$currency_code],
 				array(
 					'admin_label' => '',
 					'symbol'      => $currency_code,
@@ -888,8 +888,8 @@ 
 			);
 		}
 
-		if ( ! empty( $info ) && is_string( $info ) && 'all' !== $info ) {
-			$currencies = wp_list_pluck( $currencies, $info );
+		if ( ! empty($info) && is_string($info) && 'all' !== $info) {
+			$currencies = wp_list_pluck($currencies, $info);
 		}
 	}
 
@@ -906,12 +906,12 @@ 
  *
  * @return array
  */
-function give_currency_symbols( $decode_currencies = false ) {
-	$currencies = give_get_currencies( 'symbol' );
+function give_currency_symbols($decode_currencies = false) {
+	$currencies = give_get_currencies('symbol');
 
-	if ( $decode_currencies ) {
-		array_walk( $currencies, function ( &$currency_symbol ) {
-			$currency_symbol = html_entity_decode( $currency_symbol, ENT_COMPAT, 'UTF-8' );
+	if ($decode_currencies) {
+		array_walk($currencies, function(&$currency_symbol) {
+			$currency_symbol = html_entity_decode($currency_symbol, ENT_COMPAT, 'UTF-8');
 		} );
 	}
 
@@ -922,7 +922,7 @@ 
 	 *
 	 * @param array $currencies
 	 */
-	return apply_filters( 'give_currency_symbols', $currencies );
+	return apply_filters('give_currency_symbols', $currencies);
 }
 
 
@@ -939,14 +939,14 @@ 
  *
  * @return string           The symbol to use for the currency
  */
-function give_currency_symbol( $currency = '', $decode_currency = false ) {
+function give_currency_symbol($currency = '', $decode_currency = false) {
 
-	if ( empty( $currency ) ) {
+	if (empty($currency)) {
 		$currency = give_get_currency();
 	}
 
-	$currencies = give_currency_symbols( $decode_currency );
-	$symbol     = array_key_exists( $currency, $currencies ) ? $currencies[ $currency ] : $currency;
+	$currencies = give_currency_symbols($decode_currency);
+	$symbol     = array_key_exists($currency, $currencies) ? $currencies[$currency] : $currency;
 
 	/**
 	 * Filter the currency symbol
@@ -956,7 +956,7 @@ 
 	 * @param string $symbol
 	 * @param string $currency
 	 */
-	return apply_filters( 'give_currency_symbol', $symbol, $currency );
+	return apply_filters('give_currency_symbol', $symbol, $currency);
 }
 
 
@@ -969,13 +969,13 @@ 
  *
  * @return string
  */
-function give_get_currency_name( $currency_code ) {
+function give_get_currency_name($currency_code) {
 	$currency_name  = '';
 	$currency_names = give_get_currencies();
 
-	if ( $currency_code && array_key_exists( $currency_code, $currency_names ) ) {
-		$currency_name = explode( '(', $currency_names[ $currency_code ] );
-		$currency_name = trim( current( $currency_name ) );
+	if ($currency_code && array_key_exists($currency_code, $currency_names)) {
+		$currency_name = explode('(', $currency_names[$currency_code]);
+		$currency_name = trim(current($currency_name));
 	}
 
 	/**
@@ -986,7 +986,7 @@ 
 	 * @param string $currency_name
 	 * @param string $currency_code
 	 */
-	return apply_filters( 'give_currency_name', $currency_name, $currency_code );
+	return apply_filters('give_currency_name', $currency_name, $currency_code);
 }
 
 /**
@@ -999,20 +999,20 @@ 
  *
  * @return mixed|string
  */
-function give_currency_filter( $price = '', $args = array() ) {
+function give_currency_filter($price = '', $args = array()) {
 
 	// Get functions arguments.
 	$func_args = func_get_args();
 
 	// Backward compatibility: modify second param to array
-	if ( isset( $func_args[1] ) && is_string( $func_args[1] ) ) {
+	if (isset($func_args[1]) && is_string($func_args[1])) {
 		$args = array(
-			'currency_code'   => isset( $func_args[1] ) ? $func_args[1] : '',
-			'decode_currency' => isset( $func_args[2] ) ? $func_args[2] : false,
-			'form_id'         => isset( $func_args[3] ) ? $func_args[3] : '',
+			'currency_code'   => isset($func_args[1]) ? $func_args[1] : '',
+			'decode_currency' => isset($func_args[2]) ? $func_args[2] : false,
+			'form_id'         => isset($func_args[3]) ? $func_args[3] : '',
 		);
 
-		give_doing_it_wrong( __FUNCTION__, 'Pass second argument as Array.', GIVE_VERSION );
+		give_doing_it_wrong(__FUNCTION__, 'Pass second argument as Array.', GIVE_VERSION);
 	}
 
 	// Set default values.
@@ -1025,22 +1025,22 @@ 
 		)
 	);
 
-	if ( empty( $args['currency_code'] ) || ! array_key_exists( (string) $args['currency_code'], give_get_currencies() ) ) {
-		$args['currency_code'] = give_get_currency( $args['form_id'] );
+	if (empty($args['currency_code']) || ! array_key_exists((string) $args['currency_code'], give_get_currencies())) {
+		$args['currency_code'] = give_get_currency($args['form_id']);
 	}
 
-	$args['position'] = give_get_option( 'currency_position', 'before' );
+	$args['position'] = give_get_option('currency_position', 'before');
 
 	$negative = $price < 0;
 
-	if ( $negative ) {
+	if ($negative) {
 		// Remove proceeding "-".
-		$price = substr( $price, 1 );
+		$price = substr($price, 1);
 	}
 
-	$args['symbol'] = give_currency_symbol( $args['currency_code'], $args['decode_currency'] );
+	$args['symbol'] = give_currency_symbol($args['currency_code'], $args['decode_currency']);
 
-	switch ( $args['currency_code'] ) :
+	switch ($args['currency_code']) :
 		case 'GBP' :
 		case 'BRL' :
 		case 'EUR' :
@@ -1070,13 +1070,13 @@ 
 		case 'MAD' :
 		case 'KRW' :
 		case 'ZAR' :
-			$formatted = ( 'before' === $args['position'] ? $args['symbol'] . $price : $price . $args['symbol'] );
+			$formatted = ('before' === $args['position'] ? $args['symbol'].$price : $price.$args['symbol']);
 			break;
 		case 'NOK':
-			$formatted = ( 'before' === $args['position'] ? $args['symbol'] . ' ' . $price : $price . ' ' . $args['symbol'] );
+			$formatted = ('before' === $args['position'] ? $args['symbol'].' '.$price : $price.' '.$args['symbol']);
 			break;
 		default:
-			$formatted = ( 'before' === $args['position'] ? $args['symbol'] . ' ' . $price : $price . ' ' . $args['symbol'] );
+			$formatted = ('before' === $args['position'] ? $args['symbol'].' '.$price : $price.' '.$args['symbol']);
 			break;
 	endswitch;
 
@@ -1085,7 +1085,7 @@ 
 	 *
 	 * @since 1.8.17
 	 */
-	$formatted = apply_filters( 'give_currency_filter', $formatted, $args, $price );
+	$formatted = apply_filters('give_currency_filter', $formatted, $args, $price);
 
 	/**
 	 * Filter formatted amount with currency
@@ -1099,16 +1099,16 @@ 
 	 *           filter name will be give_usd_currency_filter_after
 	 */
 	$formatted = apply_filters(
-		'give_' . strtolower( $args['currency_code'] ) . "_currency_filter_{$args['position']}",
+		'give_'.strtolower($args['currency_code'])."_currency_filter_{$args['position']}",
 		$formatted,
 		$args['currency_code'],
 		$price,
 		$args
 	);
 
-	if ( $negative ) {
+	if ($negative) {
 		// Prepend the minus sign before the currency sign.
-		$formatted = '-' . $formatted;
+		$formatted = '-'.$formatted;
 	}
 
 	return $formatted;
@@ -1125,7 +1125,7 @@ 
  *
  * @return bool
  */
-function give_is_zero_based_currency( $currency = '' ) {
+function give_is_zero_based_currency($currency = '') {
 	$zero_based_currency = array(
 		'PYG', // Paraguayan Guarani.
 		'GNF', // Guinean Franc.
@@ -1145,12 +1145,12 @@ 
 	);
 
 	// Set default currency.
-	if ( empty( $currency ) ) {
+	if (empty($currency)) {
 		$currency = give_get_currency();
 	}
 
 	// Check for Zero Based Currency.
-	if ( in_array( $currency, $zero_based_currency ) ) {
+	if (in_array($currency, $zero_based_currency)) {
 		return true;
 	}
 
@@ -1165,7 +1165,7 @@ 
  *
  * @return bool
  */
-function give_is_right_to_left_supported_currency( $currency = '' ) {
+function give_is_right_to_left_supported_currency($currency = '') {
 	$zero_based_currency = apply_filters(
 		'give_right_to_left_supported_currency',
 		array(
@@ -1181,12 +1181,12 @@ 
 	);
 
 	// Set default currency.
-	if ( empty( $currency ) ) {
+	if (empty($currency)) {
 		$currency = give_get_currency();
 	}
 
 	// Check for Zero Based Currency.
-	if ( in_array( $currency, $zero_based_currency ) ) {
+	if (in_array($currency, $zero_based_currency)) {
 		return true;
 	}
 

includes/admin/admin-actions.php

Doc Comments +3 added lines, -3 removed lines

@@ -40,7 +40,7 @@ 
  *
  * @since 1.8
  *
- * @return bool
+ * @return false|null
  */
 function give_redirect_to_clean_url_admin_pages() {
 	// Give admin pages.
@@ -992,7 +992,7 @@ 
  * @access public
  * @since  2.0
  *
- * @return bool
+ * @return false|null
  */
 function give_update_donor_name_on_user_update( $user_id = 0 ) {
 
@@ -1032,7 +1032,7 @@ 
  * @param  int          $user_id       User ID.
  * @param  WP_User|bool $old_user_data User data.
  *
- * @return bool
+ * @return false|null
  */
 function give_update_donor_email_on_user_update( $user_id = 0, $old_user_data = false ) {
 

Spacing +279 added lines, -285 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -20,19 +20,19 @@ 
  * @since 1.8
  */
 function give_load_wp_editor() {
-	if ( ! isset( $_POST['wp_editor'] ) ) {
+	if ( ! isset($_POST['wp_editor'])) {
 		die();
 	}
 
-	$wp_editor                     = json_decode( base64_decode( $_POST['wp_editor'] ), true );
+	$wp_editor                     = json_decode(base64_decode($_POST['wp_editor']), true);
 	$wp_editor[2]['textarea_name'] = $_POST['textarea_name'];
 
-	wp_editor( $wp_editor[0], $_POST['wp_editor_id'], $wp_editor[2] );
+	wp_editor($wp_editor[0], $_POST['wp_editor_id'], $wp_editor[2]);
 
 	die();
 }
 
-add_action( 'wp_ajax_give_load_wp_editor', 'give_load_wp_editor' );
+add_action('wp_ajax_give_load_wp_editor', 'give_load_wp_editor');
 
 
 /**
@@ -52,13 +52,13 @@ 
 	);
 
 	// Get current page.
-	$current_page = isset( $_GET['page'] ) ? esc_attr( $_GET['page'] ) : '';
+	$current_page = isset($_GET['page']) ? esc_attr($_GET['page']) : '';
 
 	// Bailout.
 	if (
-		empty( $current_page )
-		|| empty( $_GET['_wp_http_referer'] )
-		|| ! in_array( $current_page, $give_pages )
+		empty($current_page)
+		|| empty($_GET['_wp_http_referer'])
+		|| ! in_array($current_page, $give_pages)
 	) {
 		return false;
 	}
@@ -68,21 +68,21 @@ 
 	 *
 	 * @since 1.8
 	 */
-	$redirect = apply_filters( "give_validate_{$current_page}", true );
+	$redirect = apply_filters("give_validate_{$current_page}", true);
 
-	if ( $redirect ) {
+	if ($redirect) {
 		// Redirect.
 		wp_redirect(
 			remove_query_arg(
-				array( '_wp_http_referer', '_wpnonce' ),
-				wp_unslash( $_SERVER['REQUEST_URI'] )
+				array('_wp_http_referer', '_wpnonce'),
+				wp_unslash($_SERVER['REQUEST_URI'])
 			)
 		);
 		exit;
 	}
 }
 
-add_action( 'admin_init', 'give_redirect_to_clean_url_admin_pages' );
+add_action('admin_init', 'give_redirect_to_clean_url_admin_pages');
 
 
 /**
@@ -96,25 +96,25 @@ 
  */
 function give_hide_outdated_php_notice() {
 
-	if ( ! isset( $_POST['_give_hide_outdated_php_notices_shortly'] ) ) {
+	if ( ! isset($_POST['_give_hide_outdated_php_notices_shortly'])) {
 		give_die();
 	}
 
 	// Transient key name.
 	$transient_key = "_give_hide_outdated_php_notices_shortly";
 
-	if ( Give_Cache::get( $transient_key, true ) ) {
+	if (Give_Cache::get($transient_key, true)) {
 		return;
 	}
 
 	// Hide notice for 24 hours.
-	Give_Cache::set( $transient_key, true, DAY_IN_SECONDS, true );
+	Give_Cache::set($transient_key, true, DAY_IN_SECONDS, true);
 
 	give_die();
 
 }
 
-add_action( 'wp_ajax_give_hide_outdated_php_notice', 'give_hide_outdated_php_notice' );
+add_action('wp_ajax_give_hide_outdated_php_notice', 'give_hide_outdated_php_notice');
 
 /**
  * Register admin notices.
@@ -123,27 +123,27 @@ 
  */
 function _give_register_admin_notices() {
 	// Bailout.
-	if ( ! is_admin() ) {
+	if ( ! is_admin()) {
 		return;
 	}
 
 	// Bulk action notices.
 	if (
-		isset( $_GET['action'] ) &&
-		! empty( $_GET['action'] )
+		isset($_GET['action']) &&
+		! empty($_GET['action'])
 	) {
 
 		// Add payment bulk notice.
 		if (
-			current_user_can( 'edit_give_payments' ) &&
-			isset( $_GET['payment'] ) &&
-			! empty( $_GET['payment'] )
+			current_user_can('edit_give_payments') &&
+			isset($_GET['payment']) &&
+			! empty($_GET['payment'])
 		) {
-			$payment_count = isset( $_GET['payment'] ) ? count( $_GET['payment'] ) : 0;
+			$payment_count = isset($_GET['payment']) ? count($_GET['payment']) : 0;
 
-			switch ( $_GET['action'] ) {
+			switch ($_GET['action']) {
 				case 'delete':
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'bulk_action_delete',
 						'type'        => 'updated',
 						'description' => sprintf(
@@ -155,12 +155,12 @@ 
 							),
 							$payment_count ),
 						'show'        => true,
-					) );
+					));
 
 					break;
 
 				case 'resend-receipt':
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'bulk_action_resend_receipt',
 						'type'        => 'updated',
 						'description' => sprintf(
@@ -173,7 +173,7 @@ 
 							$payment_count
 						),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'set-status-publish' :
@@ -185,7 +185,7 @@ 
 				case 'set-status-cancelled' :
 				case 'set-status-abandoned' :
 				case 'set-status-preapproval' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'bulk_action_status_change',
 						'type'        => 'updated',
 						'description' => _n(
@@ -195,251 +195,251 @@ 
 							'give'
 						),
 						'show'        => true,
-					) );
+					));
 					break;
 			}
 		}
 	}
 
 	// Add give message notices.
-	if ( ! empty( $_GET['give-message'] ) ) {
+	if ( ! empty($_GET['give-message'])) {
 		// Donation reports errors.
-		if ( current_user_can( 'view_give_reports' ) ) {
-			switch ( $_GET['give-message'] ) {
+		if (current_user_can('view_give_reports')) {
+			switch ($_GET['give-message']) {
 				case 'donation_deleted' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donation-deleted',
 						'type'        => 'updated',
-						'description' => __( 'The donation has been deleted.', 'give' ),
+						'description' => __('The donation has been deleted.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'email_sent' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-payment-sent',
 						'type'        => 'updated',
-						'description' => __( 'The donation receipt has been resent.', 'give' ),
+						'description' => __('The donation receipt has been resent.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'refreshed-reports' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-refreshed-reports',
 						'type'        => 'updated',
-						'description' => __( 'The reports cache has been cleared.', 'give' ),
+						'description' => __('The reports cache has been cleared.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'donation-note-deleted' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donation-note-deleted',
 						'type'        => 'updated',
-						'description' => __( 'The donation note has been deleted.', 'give' ),
+						'description' => __('The donation note has been deleted.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 			}
 		}
 
 		// Give settings notices and errors.
-		if ( current_user_can( 'manage_give_settings' ) ) {
-			switch ( $_GET['give-message'] ) {
+		if (current_user_can('manage_give_settings')) {
+			switch ($_GET['give-message']) {
 				case 'settings-imported' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-settings-imported',
 						'type'        => 'updated',
-						'description' => __( 'The settings have been imported.', 'give' ),
+						'description' => __('The settings have been imported.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'api-key-generated' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-api-key-generated',
 						'type'        => 'updated',
-						'description' => __( 'API keys have been generated.', 'give' ),
+						'description' => __('API keys have been generated.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'api-key-exists' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-api-key-exists',
 						'type'        => 'updated',
-						'description' => __( 'The specified user already has API keys.', 'give' ),
+						'description' => __('The specified user already has API keys.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'api-key-regenerated' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-api-key-regenerated',
 						'type'        => 'updated',
-						'description' => __( 'API keys have been regenerated.', 'give' ),
+						'description' => __('API keys have been regenerated.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'api-key-revoked' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-api-key-revoked',
 						'type'        => 'updated',
-						'description' => __( 'API keys have been revoked.', 'give' ),
+						'description' => __('API keys have been revoked.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'sent-test-email' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-sent-test-email',
 						'type'        => 'updated',
-						'description' => __( 'The test email has been sent.', 'give' ),
+						'description' => __('The test email has been sent.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'matched-success-failure-page':
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-matched-success-failure-page',
 						'type'        => 'updated',
-						'description' => __( 'You cannot set the success and failed pages to the same page', 'give' ),
+						'description' => __('You cannot set the success and failed pages to the same page', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 			}
 		}
 		// Payments errors.
-		if ( current_user_can( 'edit_give_payments' ) ) {
-			switch ( $_GET['give-message'] ) {
+		if (current_user_can('edit_give_payments')) {
+			switch ($_GET['give-message']) {
 				case 'note-added' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-note-added',
 						'type'        => 'updated',
-						'description' => __( 'The donation note has been added.', 'give' ),
+						'description' => __('The donation note has been added.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'payment-updated' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-payment-updated',
 						'type'        => 'updated',
-						'description' => __( 'The donation has been updated.', 'give' ),
+						'description' => __('The donation has been updated.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 			}
 		}
 
 		// Donor Notices.
-		if ( current_user_can( 'edit_give_payments' ) ) {
-			switch ( $_GET['give-message'] ) {
+		if (current_user_can('edit_give_payments')) {
+			switch ($_GET['give-message']) {
 				case 'donor-deleted' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-deleted',
 						'type'        => 'updated',
-						'description' => __( 'The selected donor(s) has been deleted.', 'give' ),
+						'description' => __('The selected donor(s) has been deleted.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'donor-donations-deleted' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-donations-deleted',
 						'type'        => 'updated',
-						'description' => __( 'The selected donor(s) and its associated donations has been deleted.', 'give' ),
+						'description' => __('The selected donor(s) and its associated donations has been deleted.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'confirm-delete-donor' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-confirm-delete-donor',
 						'type'        => 'updated',
-						'description' => __( 'You must confirm to delete the selected donor(s).', 'give' ),
+						'description' => __('You must confirm to delete the selected donor(s).', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'invalid-donor-id' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-invalid-donor-id',
 						'type'        => 'updated',
-						'description' => __( 'Invalid Donor ID.', 'give' ),
+						'description' => __('Invalid Donor ID.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'donor-delete-failed' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-delete-failed',
 						'type'        => 'error',
-						'description' => __( 'Unable to delete selected donor(s).', 'give' ),
+						'description' => __('Unable to delete selected donor(s).', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'email-added' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-email-added',
 						'type'        => 'updated',
-						'description' => __( 'Donor email added.', 'give' ),
+						'description' => __('Donor email added.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'email-removed' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-email-removed',
 						'type'        => 'updated',
-						'description' => __( 'Donor email removed.', 'give' ),
+						'description' => __('Donor email removed.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'email-remove-failed' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-email-remove-failed',
 						'type'        => 'updated',
-						'description' => __( 'Failed to remove donor email.', 'give' ),
+						'description' => __('Failed to remove donor email.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'primary-email-updated' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-primary-email-updated',
 						'type'        => 'updated',
-						'description' => __( 'Primary email updated for donor.', 'give' ),
+						'description' => __('Primary email updated for donor.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'primary-email-failed' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-primary-email-failed',
 						'type'        => 'updated',
-						'description' => __( 'Failed to set primary email.', 'give' ),
+						'description' => __('Failed to set primary email.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'reconnect-user' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-reconnect-user',
 						'type'        => 'updated',
-						'description' => __( 'User has been successfully connected with Donor.', 'give' ),
+						'description' => __('User has been successfully connected with Donor.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'profile-updated' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-profile-updated',
 						'type'        => 'updated',
-						'description' => __( 'Donor information updated successfully.', 'give' ),
+						'description' => __('Donor information updated successfully.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 			}
 		}
 	}
 }
 
-add_action( 'admin_notices', '_give_register_admin_notices', - 1 );
+add_action('admin_notices', '_give_register_admin_notices', - 1);
 
 
 /**
@@ -449,31 +449,30 @@ 
  *
  * @return bool
  */
-function _give_show_test_mode_notice_in_admin_bar( $wp_admin_bar ) {
-	$is_test_mode = ! empty( $_POST['test_mode'] ) ?
-		give_is_setting_enabled( $_POST['test_mode'] ) :
-		give_is_test_mode();
+function _give_show_test_mode_notice_in_admin_bar($wp_admin_bar) {
+	$is_test_mode = ! empty($_POST['test_mode']) ?
+		give_is_setting_enabled($_POST['test_mode']) : give_is_test_mode();
 
 	if (
-		! current_user_can( 'view_give_reports' ) ||
+		! current_user_can('view_give_reports') ||
 		! $is_test_mode
 	) {
 		return false;
 	}
 
 	// Add the main site admin menu item.
-	$wp_admin_bar->add_menu( array(
+	$wp_admin_bar->add_menu(array(
 		'id'     => 'give-test-notice',
-		'href'   => admin_url( 'edit.php?post_type=give_forms&page=give-settings&tab=gateways' ),
+		'href'   => admin_url('edit.php?post_type=give_forms&page=give-settings&tab=gateways'),
 		'parent' => 'top-secondary',
-		'title'  => __( 'Give Test Mode Active', 'give' ),
-		'meta'   => array( 'class' => 'give-test-mode-active' ),
-	) );
+		'title'  => __('Give Test Mode Active', 'give'),
+		'meta'   => array('class' => 'give-test-mode-active'),
+	));
 
 	return true;
 }
 
-add_action( 'admin_bar_menu', '_give_show_test_mode_notice_in_admin_bar', 1000, 1 );
+add_action('admin_bar_menu', '_give_show_test_mode_notice_in_admin_bar', 1000, 1);
 
 /**
  * Add Link to Import page in from donation archive and donation single page
@@ -482,12 +481,12 @@ 
  */
 function give_import_page_link_callback() {
 	?>
-	<a href="<?php echo esc_url( give_import_page_url() ); ?>"
-	   class="page-import-action page-title-action"><?php _e( 'Import Donations', 'give' ); ?></a>
+	<a href="<?php echo esc_url(give_import_page_url()); ?>"
+	   class="page-import-action page-title-action"><?php _e('Import Donations', 'give'); ?></a>
 
 	<?php
 	// Check if view donation single page only.
-	if ( ! empty( $_REQUEST['view'] ) && 'view-payment-details' === (string) give_clean( $_REQUEST['view'] ) && 'give-payment-history' === give_clean( $_REQUEST['page'] ) ) {
+	if ( ! empty($_REQUEST['view']) && 'view-payment-details' === (string) give_clean($_REQUEST['view']) && 'give-payment-history' === give_clean($_REQUEST['page'])) {
 		?>
 		<style type="text/css">
 			.wrap #transaction-details-heading {
@@ -498,7 +497,7 @@ 
 	}
 }
 
-add_action( 'give_payments_page_top', 'give_import_page_link_callback', 11 );
+add_action('give_payments_page_top', 'give_import_page_link_callback', 11);
 
 /**
  * Load donation import ajax callback
@@ -510,9 +509,9 @@ 
  */
 function give_donation_import_callback() {
 	$import_setting = array();
-	$fields         = isset( $_POST['fields'] ) ? $_POST['fields'] : null;
+	$fields         = isset($_POST['fields']) ? $_POST['fields'] : null;
 
-	parse_str( $fields );
+	parse_str($fields);
 
 	$import_setting['create_user'] = $create_user;
 	$import_setting['mode']        = $mode;
@@ -521,57 +520,57 @@ 
 	$import_setting['delete_csv']  = $delete_csv;
 
 	// Parent key id.
-	$main_key = maybe_unserialize( $main_key );
-
-	$current    = absint( $_REQUEST['current'] );
-	$total_ajax = absint( $_REQUEST['total_ajax'] );
-	$start      = absint( $_REQUEST['start'] );
-	$end        = absint( $_REQUEST['end'] );
-	$next       = absint( $_REQUEST['next'] );
-	$total      = absint( $_REQUEST['total'] );
-	$per_page   = absint( $_REQUEST['per_page'] );
-	if ( empty( $delimiter ) ) {
+	$main_key = maybe_unserialize($main_key);
+
+	$current    = absint($_REQUEST['current']);
+	$total_ajax = absint($_REQUEST['total_ajax']);
+	$start      = absint($_REQUEST['start']);
+	$end        = absint($_REQUEST['end']);
+	$next       = absint($_REQUEST['next']);
+	$total      = absint($_REQUEST['total']);
+	$per_page   = absint($_REQUEST['per_page']);
+	if (empty($delimiter)) {
 		$delimiter = ',';
 	}
 
 	// Processing done here.
-	$raw_data = give_get_donation_data_from_csv( $csv, $start, $end, $delimiter );
-	$raw_key  = maybe_unserialize( $mapto );
+	$raw_data = give_get_donation_data_from_csv($csv, $start, $end, $delimiter);
+	$raw_key  = maybe_unserialize($mapto);
 
 	// Prevent normal emails.
-	remove_action( 'give_complete_donation', 'give_trigger_donation_receipt', 999 );
-	remove_action( 'give_insert_user', 'give_new_user_notification', 10 );
-	remove_action( 'give_insert_payment', 'give_payment_save_page_data' );
-
-	$current_key  = $start;
-	foreach ( $raw_data as $row_data ) {
-		$import_setting['donation_key']  = $current_key;
-		give_save_import_donation_to_db( $raw_key, $row_data, $main_key, $import_setting );
+	remove_action('give_complete_donation', 'give_trigger_donation_receipt', 999);
+	remove_action('give_insert_user', 'give_new_user_notification', 10);
+	remove_action('give_insert_payment', 'give_payment_save_page_data');
+
+	$current_key = $start;
+	foreach ($raw_data as $row_data) {
+		$import_setting['donation_key'] = $current_key;
+		give_save_import_donation_to_db($raw_key, $row_data, $main_key, $import_setting);
 		$current_key++;
 	}
 
 	// Check if function exists or not.
-	if ( function_exists( 'give_payment_save_page_data' ) ) {
-		add_action( 'give_insert_payment', 'give_payment_save_page_data' );
+	if (function_exists('give_payment_save_page_data')) {
+		add_action('give_insert_payment', 'give_payment_save_page_data');
 	}
-	add_action( 'give_insert_user', 'give_new_user_notification', 10, 2 );
-	add_action( 'give_complete_donation', 'give_trigger_donation_receipt', 999 );
+	add_action('give_insert_user', 'give_new_user_notification', 10, 2);
+	add_action('give_complete_donation', 'give_trigger_donation_receipt', 999);
 
-	if ( $next == false ) {
+	if ($next == false) {
 		$json_data = array(
 			'success' => true,
-			'message' => __( 'All donation uploaded successfully!', 'give' ),
+			'message' => __('All donation uploaded successfully!', 'give'),
 		);
 	} else {
 		$index_start = $start;
 		$index_end   = $end;
 		$last        = false;
 		$next        = true;
-		if ( $next ) {
+		if ($next) {
 			$index_start = $index_start + $per_page;
-			$index_end   = $per_page + ( $index_start - 1 );
+			$index_end   = $per_page + ($index_start - 1);
 		}
-		if ( $index_end >= $total ) {
+		if ($index_end >= $total) {
 			$index_end = $total;
 			$last      = true;
 		}
@@ -585,27 +584,27 @@ 
 		);
 	}
 
-	$url              = give_import_page_url( array(
+	$url = give_import_page_url(array(
 		'step'          => '4',
 		'importer-type' => 'import_donations',
 		'csv'           => $csv,
 		'total'         => $total,
 		'delete_csv'    => $import_setting['delete_csv'],
-		'success'       => ( isset( $json_data['success'] ) ? $json_data['success'] : '' ),
-	) );
+		'success'       => (isset($json_data['success']) ? $json_data['success'] : ''),
+	));
 	$json_data['url'] = $url;
 
-	$current ++;
+	$current++;
 	$json_data['current'] = $current;
 
-	$percentage              = ( 100 / ( $total_ajax + 1 ) ) * $current;
+	$percentage              = (100 / ($total_ajax + 1)) * $current;
 	$json_data['percentage'] = $percentage;
 
-	$json_data = apply_filters( 'give_import_ajax_responces', $json_data, $fields );
-	wp_die( json_encode( $json_data ) );
+	$json_data = apply_filters('give_import_ajax_responces', $json_data, $fields);
+	wp_die(json_encode($json_data));
 }
 
-add_action( 'wp_ajax_give_donation_import', 'give_donation_import_callback' );
+add_action('wp_ajax_give_donation_import', 'give_donation_import_callback');
 
 /**
  * Load core settings import ajax callback
@@ -617,8 +616,8 @@ 
  */
 
 function give_core_settings_import_callback() {
-	$fields = isset( $_POST['fields'] ) ? $_POST['fields'] : null;
-	parse_str( $fields, $fields );
+	$fields = isset($_POST['fields']) ? $_POST['fields'] : null;
+	parse_str($fields, $fields);
 
 	$json_data['success'] = false;
 
@@ -633,22 +632,22 @@ 
 	 *
 	 * @return array $fields
 	 */
-	$fields = (array) apply_filters( 'give_import_core_settings_fields', $fields );
+	$fields = (array) apply_filters('give_import_core_settings_fields', $fields);
 
-	$file_name = ( ! empty( $fields['file_name'] ) ? give_clean( $fields['file_name'] ) : false );
+	$file_name = ( ! empty($fields['file_name']) ? give_clean($fields['file_name']) : false);
 
-	if ( ! empty( $file_name ) ) {
-		$type = ( ! empty( $fields['type'] ) ? give_clean( $fields['type'] ) : 'merge' );
+	if ( ! empty($file_name)) {
+		$type = ( ! empty($fields['type']) ? give_clean($fields['type']) : 'merge');
 
 		// Get the json data from the file and then alter it in array format
-		$json_string   = give_get_core_settings_json( $file_name );
-		$json_to_array = json_decode( $json_string, true );
+		$json_string   = give_get_core_settings_json($file_name);
+		$json_to_array = json_decode($json_string, true);
 
 		// get the current settign from the options table.
-		$host_give_options = get_option( 'give_settings', array() );
+		$host_give_options = get_option('give_settings', array());
 
 		// Save old settins for backup.
-		update_option( 'give_settings_old', $host_give_options );
+		update_option('give_settings_old', $host_give_options);
 
 		/**
 		 * Filter to Modify Core Settings that are being going to get import in options table as give settings.
@@ -664,9 +663,9 @@ 
 		 *
 		 * @return array $json_to_array Setting that are being going to get imported
 		 */
-		$json_to_array = (array) apply_filters( 'give_import_core_settings_data', $json_to_array, $type, $host_give_options, $fields );
+		$json_to_array = (array) apply_filters('give_import_core_settings_data', $json_to_array, $type, $host_give_options, $fields);
 
-		update_option( 'give_settings', $json_to_array );
+		update_option('give_settings', $json_to_array);
 
 		$json_data['success'] = true;
 	}
@@ -682,16 +681,16 @@ 
 	 *
 	 * @return array $url
 	 */
-	$json_data['url'] = give_import_page_url( (array) apply_filters( 'give_import_core_settings_success_url', array(
-		'step'          => ( empty( $json_data['success'] ) ? '1' : '3' ),
+	$json_data['url'] = give_import_page_url((array) apply_filters('give_import_core_settings_success_url', array(
+		'step'          => (empty($json_data['success']) ? '1' : '3'),
 		'importer-type' => 'import_core_setting',
-		'success'       => ( empty( $json_data['success'] ) ? '0' : '1' ),
-	) ) );
+		'success'       => (empty($json_data['success']) ? '0' : '1'),
+	)));
 
-	wp_send_json( $json_data );
+	wp_send_json($json_data);
 }
 
-add_action( 'wp_ajax_give_core_settings_import', 'give_core_settings_import_callback' );
+add_action('wp_ajax_give_core_settings_import', 'give_core_settings_import_callback');
 
 /**
  * Initializes blank slate content if a list table is empty.
@@ -703,7 +702,7 @@ 
 	$blank_slate->init();
 }
 
-add_action( 'current_screen', 'give_blank_slate' );
+add_action('current_screen', 'give_blank_slate');
 
 /**
  * Validate Fields of User Profile
@@ -716,24 +715,24 @@ 
  *
  * @return mixed
  */
-function give_validate_user_profile( $errors, $update, $user ) {
+function give_validate_user_profile($errors, $update, $user) {
 
-	if ( ! empty( $_POST['action'] ) && ( 'adduser' === $_POST['action'] || 'createuser' === $_POST['action'] ) ) {
+	if ( ! empty($_POST['action']) && ('adduser' === $_POST['action'] || 'createuser' === $_POST['action'])) {
 		return;
 	}
 
-	if ( ! empty( $user->ID ) ) {
-		$donor = Give()->donors->get_donor_by( 'user_id', $user->ID );
+	if ( ! empty($user->ID)) {
+		$donor = Give()->donors->get_donor_by('user_id', $user->ID);
 
-		if ( $donor ) {
+		if ($donor) {
 			// If Donor is attached with User, then validate first name.
-			if ( empty( $_POST['first_name'] ) ) {
+			if (empty($_POST['first_name'])) {
 				$errors->add(
 					'empty_first_name',
 					sprintf(
 						'<strong>%1$s:</strong> %2$s',
-						__( 'ERROR', 'give' ),
-						__( 'Please enter your first name.', 'give' )
+						__('ERROR', 'give'),
+						__('Please enter your first name.', 'give')
 					)
 				);
 			}
@@ -742,7 +741,7 @@ 
 
 }
 
-add_action( 'user_profile_update_errors', 'give_validate_user_profile', 10, 3 );
+add_action('user_profile_update_errors', 'give_validate_user_profile', 10, 3);
 
 /**
  * Show Donor Information on User Profile Page.
@@ -751,19 +750,19 @@ 
  *
  * @since 2.0
  */
-function give_donor_information_profile_fields( $user ) {
-	$donor = Give()->donors->get_donor_by( 'user_id', $user->ID );
+function give_donor_information_profile_fields($user) {
+	$donor = Give()->donors->get_donor_by('user_id', $user->ID);
 
 	// Display Donor Information, only if donor is attached with User.
-	if ( ! empty( $donor->user_id ) ) {
+	if ( ! empty($donor->user_id)) {
 		?>
 		<table class="form-table">
 			<tbody>
 			<tr>
-				<th scope="row"><?php _e( 'Donor', 'give' ); ?></th>
+				<th scope="row"><?php _e('Donor', 'give'); ?></th>
 				<td>
-					<a href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ); ?>">
-						<?php _e( 'View Donor Information', 'give' ); ?>
+					<a href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id); ?>">
+						<?php _e('View Donor Information', 'give'); ?>
 					</a>
 				</td>
 			</tr>
@@ -773,7 +772,7 @@ 
 	}
 }
 
-add_action( 'personal_options', 'give_donor_information_profile_fields' );
+add_action('personal_options', 'give_donor_information_profile_fields');
 /**
  * Get Array of WP User Roles.
  *
@@ -785,8 +784,8 @@ 
 	$user_roles = array();
 
 	// Loop through User Roles.
-	foreach ( get_editable_roles() as $role_name => $role_info ):
-		$user_roles[ $role_name ] = $role_info['name'];
+	foreach (get_editable_roles() as $role_name => $role_info):
+		$user_roles[$role_name] = $role_info['name'];
 	endforeach;
 
 	return $user_roles;
@@ -803,44 +802,42 @@ 
 function __give_ajax_donor_manage_addresses() {
 	// Bailout.
 	if (
-		empty( $_POST['form'] ) ||
-		empty( $_POST['donorID'] )
+		empty($_POST['form']) ||
+		empty($_POST['donorID'])
 	) {
-		wp_send_json_error( array( 'error' => 1 ) );
+		wp_send_json_error(array('error' => 1));
 	}
 
-	$post                  = give_clean( wp_parse_args( $_POST ) );
-	$donorID               = absint( $post['donorID'] );
-	$form_data             = give_clean( wp_parse_args( $post['form'] ) );
-	$is_multi_address_type = ( 'billing' === $form_data['address-id'] || false !== strpos( $form_data['address-id'], '_' ) );
-	$address_type          = false !== strpos( $form_data['address-id'], '_' ) ?
-		array_shift( explode( '_', $form_data['address-id'] ) ) :
-		$form_data['address-id'];
-	$address_id            = false !== strpos( $form_data['address-id'], '_' ) ?
-		array_pop( explode( '_', $form_data['address-id'] ) ) :
-		null;
+	$post                  = give_clean(wp_parse_args($_POST));
+	$donorID               = absint($post['donorID']);
+	$form_data             = give_clean(wp_parse_args($post['form']));
+	$is_multi_address_type = ('billing' === $form_data['address-id'] || false !== strpos($form_data['address-id'], '_'));
+	$address_type          = false !== strpos($form_data['address-id'], '_') ?
+		array_shift(explode('_', $form_data['address-id'])) : $form_data['address-id'];
+	$address_id            = false !== strpos($form_data['address-id'], '_') ?
+		array_pop(explode('_', $form_data['address-id'])) : null;
 	$response_data         = array(
 		'action' => $form_data['address-action'],
 		'id'     => $form_data['address-id'],
 	);
 
 	// Security check.
-	if ( ! wp_verify_nonce( $form_data['_wpnonce'], 'give-manage-donor-addresses' ) ) {
-		wp_send_json_error( array(
+	if ( ! wp_verify_nonce($form_data['_wpnonce'], 'give-manage-donor-addresses')) {
+		wp_send_json_error(array(
 				'error'     => 1,
 				'error_msg' => wp_sprintf(
 					'<div class="notice notice-error"><p>%s</p></div>',
-					__( 'Error: Security issue.', 'give' )
+					__('Error: Security issue.', 'give')
 				)
 			)
 		);
 	}
 
-	$donor = new Give_Donor( $donorID );
+	$donor = new Give_Donor($donorID);
 
 	// Verify donor.
-	if ( ! $donor->id ) {
-		wp_send_json_error( array( 'error' => 3 ) );
+	if ( ! $donor->id) {
+		wp_send_json_error(array('error' => 3));
 	}
 
 	// Unset all data except address.
@@ -851,57 +848,55 @@ 
 	);
 
 	// Process action.
-	switch ( $response_data['action'] ) {
+	switch ($response_data['action']) {
 
 		case 'add':
-			if ( ! $donor->add_address( "{$address_type}[]", $form_data ) ) {
-				wp_send_json_error( array(
+			if ( ! $donor->add_address("{$address_type}[]", $form_data)) {
+				wp_send_json_error(array(
 						'error'     => 1,
 						'error_msg' => wp_sprintf(
 							'<div class="notice notice-error"><p>%s</p></div>',
-							__( 'Error: Unable to save the address. Please check if address already exist.', 'give' )
+							__('Error: Unable to save the address. Please check if address already exist.', 'give')
 						)
 					)
 				);
 			}
 
-			$total_addresses = count( $donor->address[ $address_type ] );
+			$total_addresses = count($donor->address[$address_type]);
 
 			$address_index = $is_multi_address_type ?
-				$total_addresses - 1 :
-				$address_type;
+				$total_addresses - 1 : $address_type;
 
 			$address_id = $is_multi_address_type ?
-				end( array_keys( $donor->address[ $address_type ] ) ) :
-				$address_type;
+				end(array_keys($donor->address[$address_type])) : $address_type;
 
 			$response_data['address_html'] = __give_get_format_address(
-				end( $donor->address['billing'] ),
+				end($donor->address['billing']),
 				array(
 					// We can add only billing address from donor screen.
 					'type'  => 'billing',
 					'id'    => $address_id,
-					'index' => ++ $address_index,
+					'index' => ++$address_index,
 				)
 			);
-			$response_data['success_msg']  = wp_sprintf(
+			$response_data['success_msg'] = wp_sprintf(
 				'<div class="notice updated"><p>%s</p></div>',
-				__( 'Successfully added a new address to the donor.', 'give' )
+				__('Successfully added a new address to the donor.', 'give')
 			);
 
-			if ( $is_multi_address_type ) {
+			if ($is_multi_address_type) {
 				$response_data['id'] = "{$response_data['id']}_{$address_index}";
 			}
 
 			break;
 
 		case 'remove':
-			if ( ! $donor->remove_address( $response_data['id'] ) ) {
-				wp_send_json_error( array(
+			if ( ! $donor->remove_address($response_data['id'])) {
+				wp_send_json_error(array(
 						'error'     => 2,
 						'error_msg' => wp_sprintf(
 							'<div class="notice notice-error"><p>%s</p></div>',
-							__( 'Error: Unable to delete address.', 'give' )
+							__('Error: Unable to delete address.', 'give')
 						)
 					)
 				);
@@ -909,18 +904,18 @@ 
 
 			$response_data['success_msg'] = wp_sprintf(
 				'<div class="notice updated"><p>%s</p></div>',
-				__( 'Successfully removed a address of donor.', 'give' )
+				__('Successfully removed a address of donor.', 'give')
 			);
 
 			break;
 
 		case 'update':
-			if ( ! $donor->update_address( $response_data['id'], $form_data ) ) {
-				wp_send_json_error( array(
+			if ( ! $donor->update_address($response_data['id'], $form_data)) {
+				wp_send_json_error(array(
 						'error'     => 3,
 						'error_msg' => wp_sprintf(
 							'<div class="notice notice-error"><p>%s</p></div>',
-							__( 'Error: Unable to update address. Please check if address already exist.', 'give' )
+							__('Error: Unable to update address. Please check if address already exist.', 'give')
 						)
 					)
 				);
@@ -928,26 +923,25 @@ 
 
 			$response_data['address_html'] = __give_get_format_address(
 				$is_multi_address_type ?
-					$donor->address[ $address_type ][ $address_id ] :
-					$donor->address[ $address_type ],
+					$donor->address[$address_type][$address_id] : $donor->address[$address_type],
 				array(
 					'type'  => $address_type,
 					'id'    => $address_id,
 					'index' => $address_id,
 				)
 			);
-			$response_data['success_msg']  = wp_sprintf(
+			$response_data['success_msg'] = wp_sprintf(
 				'<div class="notice updated"><p>%s</p></div>',
-				__( 'Successfully updated a address of donor', 'give' )
+				__('Successfully updated a address of donor', 'give')
 			);
 
 			break;
 	}
 
-	wp_send_json_success( $response_data );
+	wp_send_json_success($response_data);
 }
 
-add_action( 'wp_ajax_donor_manage_addresses', '__give_ajax_donor_manage_addresses' );
+add_action('wp_ajax_donor_manage_addresses', '__give_ajax_donor_manage_addresses');
 
 /**
  * Admin donor billing address label
@@ -958,13 +952,13 @@ 
  *
  * @return string
  */
-function __give_donor_billing_address_label( $address_label ) {
-	$address_label = __( 'Billing Address', 'give' );
+function __give_donor_billing_address_label($address_label) {
+	$address_label = __('Billing Address', 'give');
 
 	return $address_label;
 }
 
-add_action( 'give_donor_billing_address_label', '__give_donor_billing_address_label' );
+add_action('give_donor_billing_address_label', '__give_donor_billing_address_label');
 
 /**
  * Admin donor personal address label
@@ -975,13 +969,13 @@ 
  *
  * @return string
  */
-function __give_donor_personal_address_label( $address_label ) {
-	$address_label = __( 'Personal Address', 'give' );
+function __give_donor_personal_address_label($address_label) {
+	$address_label = __('Personal Address', 'give');
 
 	return $address_label;
 }
 
-add_action( 'give_donor_personal_address_label', '__give_donor_personal_address_label' );
+add_action('give_donor_personal_address_label', '__give_donor_personal_address_label');
 
 /**
  * Update Donor Information when User Profile is updated from admin.
@@ -994,32 +988,32 @@ 
  *
  * @return bool
  */
-function give_update_donor_name_on_user_update( $user_id = 0 ) {
+function give_update_donor_name_on_user_update($user_id = 0) {
 
-	if ( current_user_can( 'edit_user', $user_id ) ) {
+	if (current_user_can('edit_user', $user_id)) {
 
-		$donor = new Give_Donor( $user_id, true );
+		$donor = new Give_Donor($user_id, true);
 
 		// Bailout, if donor doesn't exists.
-		if ( ! $donor ) {
+		if ( ! $donor) {
 			return false;
 		}
 
 		// Get User First name and Last name.
-		$first_name = ( $_POST['first_name'] ) ? give_clean( $_POST['first_name'] ) : get_user_meta( $user_id, 'first_name', true );
-		$last_name  = ( $_POST['last_name'] ) ? give_clean( $_POST['last_name'] ) : get_user_meta( $user_id, 'last_name', true );
-		$full_name  = strip_tags( wp_unslash( trim( "{$first_name} {$last_name}" ) ) );
+		$first_name = ($_POST['first_name']) ? give_clean($_POST['first_name']) : get_user_meta($user_id, 'first_name', true);
+		$last_name  = ($_POST['last_name']) ? give_clean($_POST['last_name']) : get_user_meta($user_id, 'last_name', true);
+		$full_name  = strip_tags(wp_unslash(trim("{$first_name} {$last_name}")));
 
 		// Assign User First name and Last name to Donor.
-		Give()->donors->update( $donor->id, array( 'name' => $full_name ) );
-		Give()->donor_meta->update_meta( $donor->id, '_give_donor_first_name', $first_name );
-		Give()->donor_meta->update_meta( $donor->id, '_give_donor_last_name', $last_name );
+		Give()->donors->update($donor->id, array('name' => $full_name));
+		Give()->donor_meta->update_meta($donor->id, '_give_donor_first_name', $first_name);
+		Give()->donor_meta->update_meta($donor->id, '_give_donor_last_name', $last_name);
 
 	}
 }
 
-add_action( 'edit_user_profile_update', 'give_update_donor_name_on_user_update', 10 );
-add_action( 'personal_options_update', 'give_update_donor_name_on_user_update', 10 );
+add_action('edit_user_profile_update', 'give_update_donor_name_on_user_update', 10);
+add_action('personal_options_update', 'give_update_donor_name_on_user_update', 10);
 
 
 /**
@@ -1034,29 +1028,29 @@ 
  *
  * @return bool
  */
-function give_update_donor_email_on_user_update( $user_id = 0, $old_user_data = false ) {
+function give_update_donor_email_on_user_update($user_id = 0, $old_user_data = false) {
 
-	$donor = new Give_Donor( $user_id, true );
+	$donor = new Give_Donor($user_id, true);
 
-	if ( ! $donor ) {
+	if ( ! $donor) {
 		return false;
 	}
 
-	$user = get_userdata( $user_id );
+	$user = get_userdata($user_id);
 
-	if ( ! empty( $user ) && $user->user_email !== $donor->email ) {
+	if ( ! empty($user) && $user->user_email !== $donor->email) {
 
-		$success = Give()->donors->update( $donor->id, array( 'email' => $user->user_email ) );
+		$success = Give()->donors->update($donor->id, array('email' => $user->user_email));
 
-		if ( $success ) {
+		if ($success) {
 			// Update some payment meta if we need to
-			$payments_array = explode( ',', $donor->payment_ids );
+			$payments_array = explode(',', $donor->payment_ids);
 
-			if ( ! empty( $payments_array ) ) {
+			if ( ! empty($payments_array)) {
 
-				foreach ( $payments_array as $payment_id ) {
+				foreach ($payments_array as $payment_id) {
 
-					give_update_payment_meta( $payment_id, 'email', $user->user_email );
+					give_update_payment_meta($payment_id, 'email', $user->user_email);
 
 				}
 
@@ -1070,7 +1064,7 @@ 
 			 * @param  WP_User    $user  WordPress User object.
 			 * @param  Give_Donor $donor Give donor object.
 			 */
-			do_action( 'give_update_donor_email_on_user_update', $user, $donor );
+			do_action('give_update_donor_email_on_user_update', $user, $donor);
 
 		}
 
@@ -1078,4 +1072,4 @@ 
 
 }
 
-add_action( 'profile_update', 'give_update_donor_email_on_user_update', 10, 2 );
\ No newline at end of file
+add_action('profile_update', 'give_update_donor_email_on_user_update', 10, 2);
\ No newline at end of file

includes/admin/donors/donor-functions.php

Doc Comments +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
  * Connect and Reconnect Donor with User profile.
  * @todo $address is unnecessary param because we are store address to user.
  *
- * @param object $donor      Donor Object.
+ * @param Give_Donor $donor      Donor Object.
  * @param array  $donor_data Donor Post Variables.
  * @param array  $address    Address Information.
  *

Spacing +24 added lines, -24 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,7 +23,7 @@ 
  *
  * @return array        The altered list of views.
  */
-function give_register_default_donor_views( $views ) {
+function give_register_default_donor_views($views) {
 
 	$default_views = array(
 		'overview' => 'give_donor_view',
@@ -31,11 +31,11 @@ 
 		'notes'    => 'give_donor_notes_view',
 	);
 
-	return array_merge( $views, $default_views );
+	return array_merge($views, $default_views);
 
 }
 
-add_filter( 'give_donor_views', 'give_register_default_donor_views', 1, 1 );
+add_filter('give_donor_views', 'give_register_default_donor_views', 1, 1);
 
 /**
  * Register a tab for the single donor view.
@@ -46,23 +46,23 @@ 
  *
  * @return array       The altered list of tabs
  */
-function give_register_default_donor_tabs( $tabs ) {
+function give_register_default_donor_tabs($tabs) {
 
 	$default_tabs = array(
 		'overview' => array(
 			'dashicon' => 'dashicons-admin-users',
-			'title' => __( 'Donor Profile', 'give' ),
+			'title' => __('Donor Profile', 'give'),
 		),
 		'notes'    => array(
 			'dashicon' => 'dashicons-admin-comments',
-			'title' => __( 'Donor Notes', 'give' ),
+			'title' => __('Donor Notes', 'give'),
 		),
 	);
 
-	return array_merge( $tabs, $default_tabs );
+	return array_merge($tabs, $default_tabs);
 }
 
-add_filter( 'give_donor_tabs', 'give_register_default_donor_tabs', 1, 1 );
+add_filter('give_donor_tabs', 'give_register_default_donor_tabs', 1, 1);
 
 /**
  * Register the Delete icon as late as possible so it's at the bottom.
@@ -73,17 +73,17 @@ 
  *
  * @return array       The altered list of tabs, with 'delete' at the bottom.
  */
-function give_register_delete_donor_tab( $tabs ) {
+function give_register_delete_donor_tab($tabs) {
 
 	$tabs['delete'] = array(
 		'dashicon' => 'dashicons-trash',
-		'title'    => __( 'Delete Donor', 'give' ),
+		'title'    => __('Delete Donor', 'give'),
 	);
 
 	return $tabs;
 }
 
-add_filter( 'give_donor_tabs', 'give_register_delete_donor_tab', PHP_INT_MAX, 1 );
+add_filter('give_donor_tabs', 'give_register_delete_donor_tab', PHP_INT_MAX, 1);
 
 /**
  * Connect and Reconnect Donor with User profile.
@@ -97,7 +97,7 @@ 
  *
  * @return array
  */
-function give_connect_user_donor_profile( $donor, $donor_data, $address ) {
+function give_connect_user_donor_profile($donor, $donor_data, $address) {
 
 	$donor_id         = $donor->id;
 	$previous_user_id = $donor->user_id;
@@ -111,31 +111,31 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_pre_edit_donor', $donor_id, $donor_data, $address );
+	do_action('give_pre_edit_donor', $donor_id, $donor_data, $address);
 
 	$output = array();
 
-	if ( $donor->update( $donor_data ) ) {
+	if ($donor->update($donor_data)) {
 
 		// Create and Update Donor First Name and Last Name in Meta Fields.
-		$donor->update_meta( '_give_donor_first_name', $donor_data['first_name'] );
-		$donor->update_meta( '_give_donor_last_name', $donor_data['last_name'] );
+		$donor->update_meta('_give_donor_first_name', $donor_data['first_name']);
+		$donor->update_meta('_give_donor_last_name', $donor_data['last_name']);
 
 		// Fetch disconnected user id, if exists.
-		$disconnected_user_id = $donor->get_meta( '_give_disconnected_user_id', true );
+		$disconnected_user_id = $donor->get_meta('_give_disconnected_user_id', true);
 
 		// Flag User and Donor Disconnection.
-		delete_user_meta( $disconnected_user_id, '_give_is_donor_disconnected' );
+		delete_user_meta($disconnected_user_id, '_give_is_donor_disconnected');
 
 		// Check whether the disconnected user id and the reconnected user id are same or not.
 		// If both are same then delete user id store in donor meta.
-		if( $donor_data['user_id'] === $disconnected_user_id ) {
-			delete_user_meta( $disconnected_user_id, '_give_disconnected_donor_id' );
-			$donor->delete_meta( '_give_disconnected_user_id' );
+		if ($donor_data['user_id'] === $disconnected_user_id) {
+			delete_user_meta($disconnected_user_id, '_give_disconnected_donor_id');
+			$donor->delete_meta('_give_disconnected_user_id');
 		}
 
 		$output['success']       = true;
-		$donor_data              = array_merge( $donor_data, $address );
+		$donor_data              = array_merge($donor_data, $address);
 		$output['customer_info'] = $donor_data;
 
 	} else {
@@ -152,7 +152,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_post_edit_donor', $donor_id, $donor_data );
+	do_action('give_post_edit_donor', $donor_id, $donor_data);
 
 
 	return $output;