WordImpress / Give

includes/admin/abstract-admin-settings-page.php

Spacing +30 added lines, -30 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_Page' ) ) :
+if ( ! class_exists('Give_Settings_Page')) :
 
 	/**
 	 * Give_Settings_Page.
@@ -70,25 +70,25 @@ 
 			// Get current setting page.
 			$this->current_setting_page = give_get_current_setting_page();
 
-			add_filter( "give_default_setting_tab_section_{$this->id}", array( $this, 'set_default_setting_tab' ), 10 );
-			add_filter( "{$this->current_setting_page}_tabs_array", array( $this, 'add_settings_page' ), 20 );
-			add_action( "{$this->current_setting_page}_settings_{$this->id}_page", array( $this, 'output' ) );
+			add_filter("give_default_setting_tab_section_{$this->id}", array($this, 'set_default_setting_tab'), 10);
+			add_filter("{$this->current_setting_page}_tabs_array", array($this, 'add_settings_page'), 20);
+			add_action("{$this->current_setting_page}_settings_{$this->id}_page", array($this, 'output'));
 
 			// Output section only if exist.
 			$sections = $this->get_sections();
-			if ( ! empty( $sections ) ) {
-				add_action( "{$this->current_setting_page}_sections_{$this->id}_page", array(
+			if ( ! empty($sections)) {
+				add_action("{$this->current_setting_page}_sections_{$this->id}_page", array(
 					$this,
 					'output_sections',
-				) );
+				));
 			}
 
 			// Save hide button by default.
 			$GLOBALS['give_hide_save_button'] = true;
 
 			// Enable saving feature.
-			if ( $this->enable_save ) {
-				add_action( "{$this->current_setting_page}_save_{$this->id}", array( $this, 'save' ) );
+			if ($this->enable_save) {
+				add_action("{$this->current_setting_page}_save_{$this->id}", array($this, 'save'));
 			}
 		}
 
@@ -113,7 +113,7 @@ 
 		 *
 		 * @return string
 		 */
-		function set_default_setting_tab( $setting_tab ) {
+		function set_default_setting_tab($setting_tab) {
 			return $this->default_tab;
 		}
 
@@ -126,8 +126,8 @@ 
 		 *
 		 * @return array
 		 */
-		public function add_settings_page( $pages ) {
-			$pages[ $this->id ] = $this->label;
+		public function add_settings_page($pages) {
+			$pages[$this->id] = $this->label;
 
 			return $pages;
 		}
@@ -146,7 +146,7 @@ 
 			 *
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, array() );
+			$settings = apply_filters('give_get_settings_'.$this->id, array());
 
 			// Output.
 			return $settings;
@@ -159,7 +159,7 @@ 
 		 * @return array
 		 */
 		public function get_sections() {
-			return apply_filters( 'give_get_sections_' . $this->id, array() );
+			return apply_filters('give_get_sections_'.$this->id, array());
 		}
 
 		/**
@@ -176,29 +176,29 @@ 
 			$sections = $this->get_sections();
 
 			// Show section settings only if setting section exist.
-			if ( $current_section && ! in_array( $current_section, array_keys( $sections ) ) ) {
-				echo '<div class="error"><p>' . __( 'Oops, this settings page does not exist.', 'give' ) . '</p></div>';
+			if ($current_section && ! in_array($current_section, array_keys($sections))) {
+				echo '<div class="error"><p>'.__('Oops, this settings page does not exist.', 'give').'</p></div>';
 				$GLOBALS['give_hide_save_button'] = true;
 
 				return;
 			}
 
 			// Bailout.
-			if ( empty( $sections ) ) {
+			if (empty($sections)) {
 				return;
 			}
 
-			if ( is_null( $this->current_setting_page ) ) {
+			if (is_null($this->current_setting_page)) {
 				$this->current_setting_page = give_get_current_setting_page();
 			}
 
 			echo '<ul class="subsubsub">';
 
 			// Get section keys.
-			$array_keys = array_keys( $sections );
+			$array_keys = array_keys($sections);
 
-			foreach ( $sections as $id => $label ) {
-				echo '<li><a href="' . admin_url( 'edit.php?post_type=give_forms&page=' . $this->current_setting_page . '&tab=' . $this->id . '&section=' . sanitize_title( $id ) ) . '" class="' . ( $current_section == $id ? 'current' : '' ) . '">' . $label . '</a> ' . ( end( $array_keys ) == $id ? '' : '|' ) . ' </li>';
+			foreach ($sections as $id => $label) {
+				echo '<li><a href="'.admin_url('edit.php?post_type=give_forms&page='.$this->current_setting_page.'&tab='.$this->id.'&section='.sanitize_title($id)).'" class="'.($current_section == $id ? 'current' : '').'">'.$label.'</a> '.(end($array_keys) == $id ? '' : '|').' </li>';
 			}
 
 			echo '</ul><br class="clear" /><hr>';
@@ -213,13 +213,13 @@ 
 		 * @return void
 		 */
 		public function output() {
-			if ( $this->enable_save ) {
+			if ($this->enable_save) {
 				$GLOBALS['give_hide_save_button'] = false;
 			}
 
 			$settings = $this->get_settings();
 
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
 		}
 
 		/**
@@ -232,14 +232,14 @@ 
 			$settings        = $this->get_settings();
 			$current_section = give_get_current_setting_section();
 
-			Give_Admin_Settings::save_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::save_fields($settings, 'give_settings');
 
 			/**
 			 * Trigger Action
 			 *
 			 * @since 1.8
 			 */
-			do_action( 'give_update_options_' . $this->id . '_' . $current_section );
+			do_action('give_update_options_'.$this->id.'_'.$current_section);
 		}
 
 		/**
@@ -256,11 +256,11 @@ 
 			$section         = $this->get_sections();
 			$current_section = give_get_current_setting_section();
 
-			if ( array_key_exists( $current_section, $section ) ) {
-				$heading[] = $section[ $current_section ];
+			if (array_key_exists($current_section, $section)) {
+				$heading[] = $section[$current_section];
 			}
 
-			return array_unique( $heading );
+			return array_unique($heading);
 		}
 
 		/**
@@ -274,7 +274,7 @@ 
 		public function get_heading_html() {
 			return sprintf(
 				'<h1 class="wp-heading-inline">%s</h1><hr class="wp-header-end">',
-				implode( ' <span class="give-settings-heading-sep dashicons dashicons-arrow-right-alt2"></span> ', $this->get_heading() )
+				implode(' <span class="give-settings-heading-sep dashicons dashicons-arrow-right-alt2"></span> ', $this->get_heading())
 			);
 		}
 	}

includes/admin/payments/view-payment-details.php

Spacing +141 added lines, -141 removed lines

@@ -10,13 +10,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! current_user_can( 'view_give_payments' ) ) {
+if ( ! current_user_can('view_give_payments')) {
 	wp_die(
-		__( 'Sorry, you are not allowed to access this page.', 'give' ), __( 'Error', 'give' ), array(
+		__('Sorry, you are not allowed to access this page.', 'give'), __('Error', 'give'), array(
 			'response' => 403,
 		)
 	);
@@ -28,27 +28,27 @@ 
  * @since 1.0
  * @return void
  */
-if ( ! isset( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
-	wp_die( esc_html__( 'Donation ID not supplied. Please try again.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 400 ) );
+if ( ! isset($_GET['id']) || ! is_numeric($_GET['id'])) {
+	wp_die(esc_html__('Donation ID not supplied. Please try again.', 'give'), esc_html__('Error', 'give'), array('response' => 400));
 }
 
 // Setup the variables
-$payment_id = absint( $_GET['id'] );
-$payment    = new Give_Payment( $payment_id );
+$payment_id = absint($_GET['id']);
+$payment    = new Give_Payment($payment_id);
 
 // Sanity check... fail if donation ID is invalid
 $payment_exists = $payment->ID;
-if ( empty( $payment_exists ) ) {
-	wp_die( esc_html__( 'The specified ID does not belong to a donation. Please try again.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 400 ) );
+if (empty($payment_exists)) {
+	wp_die(esc_html__('The specified ID does not belong to a donation. Please try again.', 'give'), esc_html__('Error', 'give'), array('response' => 400));
 }
 
 $number         = $payment->number;
 $payment_meta   = $payment->get_meta();
-$transaction_id = esc_attr( $payment->transaction_id );
+$transaction_id = esc_attr($payment->transaction_id);
 $user_id        = $payment->user_id;
 $donor_id       = $payment->customer_id;
-$payment_date   = strtotime( $payment->date );
-$user_info      = give_get_payment_meta_user_info( $payment_id );
+$payment_date   = strtotime($payment->date);
+$user_info      = give_get_payment_meta_user_info($payment_id);
 $address        = $payment->address;
 $currency_code  = $payment->currency;
 $gateway        = $payment->gateway;
@@ -61,11 +61,11 @@ 
 	<?php
 		printf(
 			/* translators: %s: donation number */
-			esc_html__( 'Donation %s', 'give' ),
+			esc_html__('Donation %s', 'give'),
 			$number
 		);
-		if ( $payment_mode == 'test' ) {
-			echo '<span id="test-payment-label" class="give-item-label give-item-label-orange" data-tooltip="' . esc_attr__( 'This donation was made in test mode.', 'give' ) . '" data-tooltip-my-position="center left" data-tooltip-target-position="center right">' . esc_html__( 'Test Donation', 'give' ) . '</span>';
+		if ($payment_mode == 'test') {
+			echo '<span id="test-payment-label" class="give-item-label give-item-label-orange" data-tooltip="'.esc_attr__('This donation was made in test mode.', 'give').'" data-tooltip-my-position="center left" data-tooltip-target-position="center right">'.esc_html__('Test Donation', 'give').'</span>';
 		}
 		?>
 		</h1>
@@ -78,7 +78,7 @@ 
 	 *
 	 * @param int $payment_id Payment id.
 	 */
-	do_action( 'give_view_order_details_before', $payment_id );
+	do_action('give_view_order_details_before', $payment_id);
 	?>
 
 	<hr class="wp-header-end">
@@ -92,7 +92,7 @@ 
 		 *
 		 * @param int $payment_id Payment id.
 		 */
-		do_action( 'give_view_order_details_form_top', $payment_id );
+		do_action('give_view_order_details_form_top', $payment_id);
 		?>
 		<div id="poststuff">
 			<div id="give-dashboard-widgets-wrap">
@@ -108,12 +108,12 @@ 
 							 *
 							 * @param int $payment_id Payment id.
 							 */
-							do_action( 'give_view_order_details_sidebar_before', $payment_id );
+							do_action('give_view_order_details_sidebar_before', $payment_id);
 							?>
 
 							<div id="give-order-update" class="postbox give-order-data">
 
-								<h3 class="hndle"><?php esc_html_e( 'Update Donation', 'give' ); ?></h3>
+								<h3 class="hndle"><?php esc_html_e('Update Donation', 'give'); ?></h3>
 
 								<div class="inside">
 									<div class="give-admin-box">
@@ -126,33 +126,33 @@ 
 										 *
 										 * @param int $payment_id Payment id.
 										 */
-										do_action( 'give_view_order_details_totals_before', $payment_id );
+										do_action('give_view_order_details_totals_before', $payment_id);
 										?>
 
 										<div class="give-admin-box-inside">
 											<p>
-												<label for="give-payment-status" class="strong"><?php esc_html_e( 'Status:', 'give' ); ?></label>&nbsp;
+												<label for="give-payment-status" class="strong"><?php esc_html_e('Status:', 'give'); ?></label>&nbsp;
 												<select id="give-payment-status" name="give-payment-status" class="medium-text">
-													<?php foreach ( give_get_payment_statuses() as $key => $status ) : ?>
-														<option value="<?php echo esc_attr( $key ); ?>"<?php selected( $payment->status, $key, true ); ?>><?php echo esc_html( $status ); ?></option>
+													<?php foreach (give_get_payment_statuses() as $key => $status) : ?>
+														<option value="<?php echo esc_attr($key); ?>"<?php selected($payment->status, $key, true); ?>><?php echo esc_html($status); ?></option>
 													<?php endforeach; ?>
 												</select>
-												<span class="give-donation-status status-<?php echo sanitize_title( $payment->status ); ?>"><span class="give-donation-status-icon"></span></span>
+												<span class="give-donation-status status-<?php echo sanitize_title($payment->status); ?>"><span class="give-donation-status-icon"></span></span>
 											</p>
 										</div>
 
 										<div class="give-admin-box-inside">
 											<p>
-												<label for="give-payment-date" class="strong"><?php esc_html_e( 'Date:', 'give' ); ?></label>&nbsp;
-												<input type="text" id="give-payment-date" name="give-payment-date" value="<?php echo esc_attr( date( 'm/d/Y', $payment_date ) ); ?>" class="medium-text give_datepicker"/>
+												<label for="give-payment-date" class="strong"><?php esc_html_e('Date:', 'give'); ?></label>&nbsp;
+												<input type="text" id="give-payment-date" name="give-payment-date" value="<?php echo esc_attr(date('m/d/Y', $payment_date)); ?>" class="medium-text give_datepicker"/>
 											</p>
 										</div>
 
 										<div class="give-admin-box-inside">
 											<p>
-												<label for="give-payment-time-hour" class="strong"><?php esc_html_e( 'Time:', 'give' ); ?></label>&nbsp;
-												<input type="number" step="1" max="24" id="give-payment-time-hour" name="give-payment-time-hour" value="<?php echo esc_attr( date_i18n( 'H', $payment_date ) ); ?>" class="small-text give-payment-time-hour"/>&nbsp;:&nbsp;
-												<input type="number" step="1" max="59" id="give-payment-time-min" name="give-payment-time-min" value="<?php echo esc_attr( date( 'i', $payment_date ) ); ?>" class="small-text give-payment-time-min"/>
+												<label for="give-payment-time-hour" class="strong"><?php esc_html_e('Time:', 'give'); ?></label>&nbsp;
+												<input type="number" step="1" max="24" id="give-payment-time-hour" name="give-payment-time-hour" value="<?php echo esc_attr(date_i18n('H', $payment_date)); ?>" class="small-text give-payment-time-hour"/>&nbsp;:&nbsp;
+												<input type="number" step="1" max="59" id="give-payment-time-min" name="give-payment-time-min" value="<?php echo esc_attr(date('i', $payment_date)); ?>" class="small-text give-payment-time-min"/>
 											</p>
 										</div>
 
@@ -166,14 +166,14 @@ 
 										 *
 										 * @param int $payment_id Payment id.
 										 */
-										do_action( 'give_view_order_details_update_inner', $payment_id );
+										do_action('give_view_order_details_update_inner', $payment_id);
 										?>
 
 										<div class="give-order-payment give-admin-box-inside">
 											<p>
-												<label for="give-payment-total" class="strong"><?php esc_html_e( 'Total Donation:', 'give' ); ?></label>&nbsp;
-												<?php echo give_currency_symbol( $payment->currency ); ?>
-												&nbsp;<input id="give-payment-total" name="give-payment-total" type="text" class="small-text give-price-field" value="<?php echo esc_attr( give_format_decimal( give_donation_amount( $payment_id ), false, false ) ); ?>"/>
+												<label for="give-payment-total" class="strong"><?php esc_html_e('Total Donation:', 'give'); ?></label>&nbsp;
+												<?php echo give_currency_symbol($payment->currency); ?>
+												&nbsp;<input id="give-payment-total" name="give-payment-total" type="text" class="small-text give-price-field" value="<?php echo esc_attr(give_format_decimal(give_donation_amount($payment_id), false, false)); ?>"/>
 											</p>
 										</div>
 
@@ -185,7 +185,7 @@ 
 										 *
 										 * @param int $payment_id Payment id.
 										 */
-										do_action( 'give_view_order_details_totals_after', $payment_id );
+										do_action('give_view_order_details_totals_after', $payment_id);
 										?>
 
 									</div>
@@ -203,14 +203,14 @@ 
 									 *
 									 * @param int $payment_id Payment id.
 									 */
-									do_action( 'give_view_order_details_update_before', $payment_id );
+									do_action('give_view_order_details_update_before', $payment_id);
 									?>
 
 									<div id="major-publishing-actions">
 										<div id="publishing-action">
-											<input type="submit" class="button button-primary right" value="<?php esc_attr_e( 'Save Donation', 'give' ); ?>"/>
+											<input type="submit" class="button button-primary right" value="<?php esc_attr_e('Save Donation', 'give'); ?>"/>
 											<?php
-											if ( give_is_payment_complete( $payment_id ) ) {
+											if (give_is_payment_complete($payment_id)) {
 												echo sprintf(
 													'<a href="%1$s" id="give-resend-receipt" class="button-secondary right">%2$s</a>',
 													esc_url(
@@ -221,7 +221,7 @@ 
 															)
 														)
 													),
-													__( 'Resend Receipt', 'give' )
+													__('Resend Receipt', 'give')
 												);
 											}
 											?>
@@ -237,7 +237,7 @@ 
 									 *
 									 * @param int $payment_id Payment id.
 									 */
-									do_action( 'give_view_order_details_update_after', $payment_id );
+									do_action('give_view_order_details_update_after', $payment_id);
 									?>
 
 								</div>
@@ -248,7 +248,7 @@ 
 
 							<div id="give-order-details" class="postbox give-order-data">
 
-								<h3 class="hndle"><?php esc_html_e( 'Donation Meta', 'give' ); ?></h3>
+								<h3 class="hndle"><?php esc_html_e('Donation Meta', 'give'); ?></h3>
 
 								<div class="inside">
 									<div class="give-admin-box">
@@ -261,45 +261,45 @@ 
 										 *
 										 * @param int $payment_id Payment id.
 										 */
-										do_action( 'give_view_order_details_payment_meta_before', $payment_id );
+										do_action('give_view_order_details_payment_meta_before', $payment_id);
 
-										$gateway = give_get_payment_gateway( $payment_id );
-										if ( $gateway ) :
+										$gateway = give_get_payment_gateway($payment_id);
+										if ($gateway) :
 										?>
 											<div class="give-order-gateway give-admin-box-inside">
 												<p>
-													<strong><?php esc_html_e( 'Gateway:', 'give' ); ?></strong>&nbsp;
-													<?php echo give_get_gateway_admin_label( $gateway ); ?>
+													<strong><?php esc_html_e('Gateway:', 'give'); ?></strong>&nbsp;
+													<?php echo give_get_gateway_admin_label($gateway); ?>
 												</p>
 											</div>
 										<?php endif; ?>
 
 										<div class="give-order-payment-key give-admin-box-inside">
 											<p>
-												<strong><?php esc_html_e( 'Key:', 'give' ); ?></strong>&nbsp;
-												<?php echo give_get_payment_key( $payment_id ); ?>
+												<strong><?php esc_html_e('Key:', 'give'); ?></strong>&nbsp;
+												<?php echo give_get_payment_key($payment_id); ?>
 											</p>
 										</div>
 
 										<div class="give-order-ip give-admin-box-inside">
 											<p>
-												<strong><?php esc_html_e( 'IP:', 'give' ); ?></strong>&nbsp;
-												<?php echo esc_html( give_get_payment_user_ip( $payment_id ) ); ?>
+												<strong><?php esc_html_e('IP:', 'give'); ?></strong>&nbsp;
+												<?php echo esc_html(give_get_payment_user_ip($payment_id)); ?>
 											</p>
 										</div>
 
-										<?php if ( $transaction_id ) : ?>
+										<?php if ($transaction_id) : ?>
 											<div class="give-order-tx-id give-admin-box-inside">
 												<p>
-													<strong><?php esc_html_e( 'Donation ID:', 'give' ); ?></strong>&nbsp;
-													<?php echo apply_filters( "give_payment_details_transaction_id-{$gateway}", $transaction_id, $payment_id ); ?>
+													<strong><?php esc_html_e('Donation ID:', 'give'); ?></strong>&nbsp;
+													<?php echo apply_filters("give_payment_details_transaction_id-{$gateway}", $transaction_id, $payment_id); ?>
 												</p>
 											</div>
 										<?php endif; ?>
 
 										<div class="give-admin-box-inside">
-											<p><?php $purchase_url = admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&donor=' . absint( give_get_payment_donor_id( $payment_id ) ) ); ?>
-												<a href="<?php echo $purchase_url; ?>"><?php esc_html_e( 'View all donations for this donor &raquo;', 'give' ); ?></a>
+											<p><?php $purchase_url = admin_url('edit.php?post_type=give_forms&page=give-payment-history&donor='.absint(give_get_payment_donor_id($payment_id))); ?>
+												<a href="<?php echo $purchase_url; ?>"><?php esc_html_e('View all donations for this donor &raquo;', 'give'); ?></a>
 											</p>
 										</div>
 
@@ -311,7 +311,7 @@ 
 										 *
 										 * @param int $payment_id Payment id.
 										 */
-										do_action( 'give_view_order_details_payment_meta_after', $payment_id );
+										do_action('give_view_order_details_payment_meta_after', $payment_id);
 										?>
 
 									</div>
@@ -331,7 +331,7 @@ 
 							 *
 							 * @param int $payment_id Payment id.
 							 */
-							do_action( 'give_view_order_details_sidebar_after', $payment_id );
+							do_action('give_view_order_details_sidebar_after', $payment_id);
 							?>
 
 						</div>
@@ -351,31 +351,31 @@ 
 							 *
 							 * @param int $payment_id Payment id.
 							 */
-							do_action( 'give_view_order_details_main_before', $payment_id );
+							do_action('give_view_order_details_main_before', $payment_id);
 							?>
 
 							<?php $column_count = 'columns-3'; ?>
 							<div id="give-donation-overview" class="postbox <?php echo $column_count; ?>">
-								<h3 class="hndle"><?php esc_html_e( 'Donation Information', 'give' ); ?></h3>
+								<h3 class="hndle"><?php esc_html_e('Donation Information', 'give'); ?></h3>
 
 								<div class="inside">
 
 									<div class="column-container">
 										<div class="column">
 											<p>
-												<strong><?php esc_html_e( 'Donation Form ID:', 'give' ); ?></strong><br>
+												<strong><?php esc_html_e('Donation Form ID:', 'give'); ?></strong><br>
 												<?php
-												if ( $payment_meta['form_id'] ) :
+												if ($payment_meta['form_id']) :
 													printf(
 														'<a href="%1$s">#%2$s</a>',
-														admin_url( 'post.php?action=edit&post=' . $payment_meta['form_id'] ),
+														admin_url('post.php?action=edit&post='.$payment_meta['form_id']),
 														$payment_meta['form_id']
 													);
 												endif;
 												?>
 											</p>
 											<p>
-												<strong><?php esc_html_e( 'Donation Form Title:', 'give' ); ?></strong><br>
+												<strong><?php esc_html_e('Donation Form Title:', 'give'); ?></strong><br>
 												<?php
 												echo Give()->html->forms_dropdown(
 													array(
@@ -391,21 +391,21 @@ 
 										</div>
 										<div class="column">
 											<p>
-												<strong><?php esc_html_e( 'Donation Date:', 'give' ); ?></strong><br>
-												<?php echo date_i18n( give_date_format(), $payment_date ); ?>
+												<strong><?php esc_html_e('Donation Date:', 'give'); ?></strong><br>
+												<?php echo date_i18n(give_date_format(), $payment_date); ?>
 											</p>
 											<p>
-												<strong><?php esc_html_e( 'Donation Level:', 'give' ); ?></strong><br>
+												<strong><?php esc_html_e('Donation Level:', 'give'); ?></strong><br>
 												<span class="give-donation-level">
 													<?php
-													$var_prices = give_has_variable_prices( $payment_meta['form_id'] );
-													if ( empty( $var_prices ) ) {
-														esc_html_e( 'n/a', 'give' );
+													$var_prices = give_has_variable_prices($payment_meta['form_id']);
+													if (empty($var_prices)) {
+														esc_html_e('n/a', 'give');
 													} else {
 														$prices_atts = array();
-														if ( $variable_prices = give_get_variable_prices( $payment_meta['form_id'] ) ) {
-															foreach ( $variable_prices as $variable_price ) {
-																$prices_atts[ $variable_price['_give_id']['level_id'] ] = give_format_amount( $variable_price['_give_amount'], array( 'sanitize' => false ) );
+														if ($variable_prices = give_get_variable_prices($payment_meta['form_id'])) {
+															foreach ($variable_prices as $variable_price) {
+																$prices_atts[$variable_price['_give_id']['level_id']] = give_format_amount($variable_price['_give_amount'], array('sanitize' => false));
 															}
 														}
 														// Variable price dropdown options.
@@ -414,12 +414,12 @@ 
 															'name'             => 'give-variable-price',
 															'chosen'           => true,
 															'show_option_all'  => '',
-															'show_option_none' => ( '' === get_post_meta( $payment_id, '_give_payment_price_id', true ) ? __( 'None', 'give' ) : '' ),
-															'select_atts'      => 'data-prices=' . esc_attr( wp_json_encode( $prices_atts ) ),
+															'show_option_none' => ('' === get_post_meta($payment_id, '_give_payment_price_id', true) ? __('None', 'give') : ''),
+															'select_atts'      => 'data-prices='.esc_attr(wp_json_encode($prices_atts)),
 															'selected'         => $payment_meta['price_id'],
 														);
 														// Render variable prices select tag html.
-														give_get_form_variable_price_dropdown( $variable_price_dropdown_option, true );
+														give_get_form_variable_price_dropdown($variable_price_dropdown_option, true);
 													}
 													?>
 												</span>
@@ -427,8 +427,8 @@ 
 										</div>
 										<div class="column">
 											<p>
-												<strong><?php esc_html_e( 'Total Donation:', 'give' ); ?></strong><br>
-												<?php echo give_currency_filter( give_format_amount( $payment->total, array( 'sanitize' => false ) ), give_get_payment_currency_code( $payment->ID ) ); ?>
+												<strong><?php esc_html_e('Total Donation:', 'give'); ?></strong><br>
+												<?php echo give_currency_filter(give_format_amount($payment->total, array('sanitize' => false)), give_get_payment_currency_code($payment->ID)); ?>
 											</p>
 											<p>
 												<?php
@@ -441,7 +441,7 @@ 
 												 *
 												 * @param int $payment_id Payment id.
 												 */
-												do_action( 'give_donation_details_thead_before', $payment_id );
+												do_action('give_donation_details_thead_before', $payment_id);
 
 
 												/**
@@ -453,7 +453,7 @@ 
 												 *
 												 * @param int $payment_id Payment id.
 												 */
-												do_action( 'give_donation_details_thead_after', $payment_id );
+												do_action('give_donation_details_thead_after', $payment_id);
 
 												/**
 												 * Fires in order details page, in the donation-information metabox, before the body elements.
@@ -464,7 +464,7 @@ 
 												 *
 												 * @param int $payment_id Payment id.
 												 */
-												do_action( 'give_donation_details_tbody_before', $payment_id );
+												do_action('give_donation_details_tbody_before', $payment_id);
 
 												/**
 												 * Fires in order details page, in the donation-information metabox, after the body elements.
@@ -475,7 +475,7 @@ 
 												 *
 												 * @param int $payment_id Payment id.
 												 */
-												do_action( 'give_donation_details_tbody_after', $payment_id );
+												do_action('give_donation_details_tbody_after', $payment_id);
 												?>
 											</p>
 										</div>
@@ -495,58 +495,58 @@ 
 							 *
 							 * @param int $payment_id Payment id.
 							 */
-							do_action( 'give_view_order_details_files_after', $payment_id );
+							do_action('give_view_order_details_files_after', $payment_id);
 							?>
 
 							<div id="give-donor-details" class="postbox">
-								<h3 class="hndle"><?php esc_html_e( 'Donor Details', 'give' ); ?></h3>
+								<h3 class="hndle"><?php esc_html_e('Donor Details', 'give'); ?></h3>
 
 								<div class="inside">
 
-									<?php $donor = new Give_Donor( $donor_id ); ?>
+									<?php $donor = new Give_Donor($donor_id); ?>
 
 									<div class="column-container donor-info">
 										<div class="column">
 											<p>
-												<strong><?php esc_html_e( 'Donor ID:', 'give' ); ?></strong><br>
+												<strong><?php esc_html_e('Donor ID:', 'give'); ?></strong><br>
 												<?php
-												if ( ! empty( $donor->id ) ) {
+												if ( ! empty($donor->id)) {
 													printf(
 														'<a href="%1$s">#%2$s</a>',
-														admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ),
+														admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id),
 														$donor->id
 													);
 												}
 												?>
 											</p>
 											<p>
-												<strong><?php esc_html_e( 'Donor Since:', 'give' ); ?></strong><br>
-												<?php echo date_i18n( give_date_format(), strtotime( $donor->date_created ) ); ?>
+												<strong><?php esc_html_e('Donor Since:', 'give'); ?></strong><br>
+												<?php echo date_i18n(give_date_format(), strtotime($donor->date_created)); ?>
 											</p>
 										</div>
 										<div class="column">
 											<p>
-												<strong><?php esc_html_e( 'Donor Name:', 'give' ); ?></strong><br>
+												<strong><?php esc_html_e('Donor Name:', 'give'); ?></strong><br>
 												<?php
-												$donor_billing_name = give_get_donor_name_by( $payment_id, 'donation' );
-												$donor_name         = give_get_donor_name_by( $donor_id, 'donor' );
+												$donor_billing_name = give_get_donor_name_by($payment_id, 'donation');
+												$donor_name         = give_get_donor_name_by($donor_id, 'donor');
 
 												// Check whether the donor name and WP_User name is same or not.
-												if ( sanitize_title( $donor_billing_name ) != sanitize_title( $donor_name ) ) {
-													echo $donor_billing_name . ' (<a href="' . esc_url( admin_url( "edit.php?post_type=give_forms&page=give-donors&view=overview&id=$donor_id" ) ) . '">' . $donor_name . '</a>)';
+												if (sanitize_title($donor_billing_name) != sanitize_title($donor_name)) {
+													echo $donor_billing_name.' (<a href="'.esc_url(admin_url("edit.php?post_type=give_forms&page=give-donors&view=overview&id=$donor_id")).'">'.$donor_name.'</a>)';
 												} else {
 													echo $donor_name;
 												}
 												?>
 											</p>
 											<p>
-												<strong><?php esc_html_e( 'Donor Email:', 'give' ); ?></strong><br>
+												<strong><?php esc_html_e('Donor Email:', 'give'); ?></strong><br>
 												<?php echo $donor->email; ?>
 											</p>
 										</div>
 										<div class="column">
 											<p>
-												<strong><?php esc_html_e( 'Change Donor:', 'give' ); ?></strong><br>
+												<strong><?php esc_html_e('Change Donor:', 'give'); ?></strong><br>
 												<?php
 												echo Give()->html->donor_dropdown(
 													array(
@@ -557,7 +557,7 @@ 
 												?>
 											</p>
 											<p>
-												<a href="#new" class="give-payment-new-donor"><?php esc_html_e( 'Create New Donor', 'give' ); ?></a>
+												<a href="#new" class="give-payment-new-donor"><?php esc_html_e('Create New Donor', 'give'); ?></a>
 											</p>
 										</div>
 									</div>
@@ -565,13 +565,13 @@ 
 									<div class="column-container new-donor" style="display: none">
 										<div class="column">
 											<p>
-												<label for="give-new-donor-name"><?php esc_html_e( 'New Donor Name:', 'give' ); ?></label>
+												<label for="give-new-donor-name"><?php esc_html_e('New Donor Name:', 'give'); ?></label>
 												<input id="give-new-donor-name" type="text" name="give-new-donor-name" value="" class="medium-text"/>
 											</p>
 										</div>
 										<div class="column">
 											<p>
-												<label for="give-new-donor-email"><?php esc_html_e( 'New Donor Email:', 'give' ); ?></label>
+												<label for="give-new-donor-email"><?php esc_html_e('New Donor Email:', 'give'); ?></label>
 												<input id="give-new-donor-email" type="email" name="give-new-donor-email" value="" class="medium-text"/>
 											</p>
 										</div>
@@ -579,9 +579,9 @@ 
 											<p>
 												<input type="hidden" name="give-current-donor" value="<?php echo $donor->id; ?>"/>
 												<input type="hidden" id="give-new-donor" name="give-new-donor" value="0"/>
-												<a href="#cancel" class="give-payment-new-donor-cancel give-delete"><?php esc_html_e( 'Cancel', 'give' ); ?></a>
+												<a href="#cancel" class="give-payment-new-donor-cancel give-delete"><?php esc_html_e('Cancel', 'give'); ?></a>
 												<br>
-												<em><?php esc_html_e( 'Click "Save Donation" to create new donor.', 'give' ); ?></em>
+												<em><?php esc_html_e('Click "Save Donation" to create new donor.', 'give'); ?></em>
 											</p>
 										</div>
 									</div>
@@ -597,7 +597,7 @@ 
 									 * @param array $payment_meta Payment meta.
 									 * @param array $user_info    User information.
 									 */
-									do_action( 'give_payment_personal_details_list', $payment_meta, $user_info );
+									do_action('give_payment_personal_details_list', $payment_meta, $user_info);
 
 									/**
 									 * Fires on the donation details page, in the donor-details metabox.
@@ -606,7 +606,7 @@ 
 									 *
 									 * @param int $payment_id Payment id.
 									 */
-									do_action( 'give_payment_view_details', $payment_id );
+									do_action('give_payment_view_details', $payment_id);
 									?>
 
 								</div>
@@ -622,11 +622,11 @@ 
 							 *
 							 * @param int $payment_id Payment id.
 							 */
-							do_action( 'give_view_order_details_billing_before', $payment_id );
+							do_action('give_view_order_details_billing_before', $payment_id);
 							?>
 
 							<div id="give-billing-details" class="postbox">
-								<h3 class="hndle"><?php esc_html_e( 'Billing Address', 'give' ); ?></h3>
+								<h3 class="hndle"><?php esc_html_e('Billing Address', 'give'); ?></h3>
 
 								<div class="inside">
 
@@ -636,9 +636,9 @@ 
 											<div class="data column-container">
 
 												<?php
-												$address['country'] = ( ! empty( $address['country'] ) ? $address['country'] : give_get_country() );
+												$address['country'] = ( ! empty($address['country']) ? $address['country'] : give_get_country());
 
-												$address['state'] = ( ! empty( $address['state'] ) ? $address['state'] : '' );
+												$address['state'] = ( ! empty($address['state']) ? $address['state'] : '');
 
 												// Get the country list that does not have any states init.
 												$no_states_country = give_no_states_country_list();
@@ -646,7 +646,7 @@ 
 
 												<div class="row">
 													<div id="give-order-address-country-wrap">
-														<label class="order-data-address-line"><?php esc_html_e( 'Country:', 'give' ); ?></label>
+														<label class="order-data-address-line"><?php esc_html_e('Country:', 'give'); ?></label>
 														<?php
 														echo Give()->html->select(
 															array(
@@ -656,8 +656,8 @@ 
 																'show_option_all'  => false,
 																'show_option_none' => false,
 																'chosen'           => true,
-																'placeholder'      => esc_attr__( 'Select a country', 'give' ),
-																'data'             => array( 'search-type' => 'no_ajax' ),
+																'placeholder'      => esc_attr__('Select a country', 'give'),
+																'data'             => array('search-type' => 'no_ajax'),
 															)
 														);
 														?>
@@ -666,35 +666,35 @@ 
 
 												<div class="row">
 													<div class="give-wrap-address-line1">
-														<label for="give-payment-address-line1" class="order-data-address"><?php esc_html_e( 'Address 1:', 'give' ); ?></label>
-														<input id="give-payment-address-line1" type="text" name="give-payment-address[0][line1]" value="<?php echo esc_attr( $address['line1'] ); ?>" class="medium-text"/>
+														<label for="give-payment-address-line1" class="order-data-address"><?php esc_html_e('Address 1:', 'give'); ?></label>
+														<input id="give-payment-address-line1" type="text" name="give-payment-address[0][line1]" value="<?php echo esc_attr($address['line1']); ?>" class="medium-text"/>
 													</div>
 												</div>
 
 												<div class="row">
 													<div class="give-wrap-address-line2">
-														<label for="give-payment-address-line2" class="order-data-address-line"><?php esc_html_e( 'Address 2:', 'give' ); ?></label>
-														<input id="give-payment-address-line2" type="text" name="give-payment-address[0][line2]" value="<?php echo esc_attr( $address['line2'] ); ?>" class="medium-text"/>
+														<label for="give-payment-address-line2" class="order-data-address-line"><?php esc_html_e('Address 2:', 'give'); ?></label>
+														<input id="give-payment-address-line2" type="text" name="give-payment-address[0][line2]" value="<?php echo esc_attr($address['line2']); ?>" class="medium-text"/>
 													</div>
 												</div>
 
 												<div class="row">
 													<div class="give-wrap-address-city">
-														<label for="give-payment-address-city" class="order-data-address-line"><?php esc_html_e( 'City:', 'give' ); ?></label>
-														<input id="give-payment-address-city" type="text" name="give-payment-address[0][city]" value="<?php echo esc_attr( $address['city'] ); ?>" class="medium-text"/>
+														<label for="give-payment-address-city" class="order-data-address-line"><?php esc_html_e('City:', 'give'); ?></label>
+														<input id="give-payment-address-city" type="text" name="give-payment-address[0][city]" value="<?php echo esc_attr($address['city']); ?>" class="medium-text"/>
 													</div>
 												</div>
 
 												<?php
-												$state_exists = ( ! empty( $address['country'] ) && array_key_exists( $address['country'], $no_states_country ) ? true : false );
+												$state_exists = ( ! empty($address['country']) && array_key_exists($address['country'], $no_states_country) ? true : false);
 												?>
 												<div class="row">
-													<div class="<?php echo( ! empty( $state_exists ) ? 'column-full' : 'column' ); ?> give-column give-column-state">
-														<div id="give-order-address-state-wrap" class="<?php echo( ! empty( $state_exists ) ? 'give-hidden' : '' ); ?>">
-															<label for="give-payment-address-state" class="order-data-address-line"><?php esc_html_e( 'State / Province / County:', 'give' ); ?></label>
+													<div class="<?php echo( ! empty($state_exists) ? 'column-full' : 'column'); ?> give-column give-column-state">
+														<div id="give-order-address-state-wrap" class="<?php echo( ! empty($state_exists) ? 'give-hidden' : ''); ?>">
+															<label for="give-payment-address-state" class="order-data-address-line"><?php esc_html_e('State / Province / County:', 'give'); ?></label>
 															<?php
-															$states = give_get_states( $address['country'] );
-															if ( ! empty( $states ) ) {
+															$states = give_get_states($address['country']);
+															if ( ! empty($states)) {
 																echo Give()->html->select(
 																	array(
 																		'options'          => $states,
@@ -703,23 +703,23 @@ 
 																		'show_option_all'  => false,
 																		'show_option_none' => false,
 																		'chosen'           => true,
-																		'placeholder'      => esc_attr__( 'Select a state', 'give' ),
-																		'data'             => array( 'search-type' => 'no_ajax' ),
+																		'placeholder'      => esc_attr__('Select a state', 'give'),
+																		'data'             => array('search-type' => 'no_ajax'),
 																	)
 																);
 															} else {
 																?>
-																<input id="give-payment-address-state" type="text" name="give-payment-address[0][state]" value="<?php echo esc_attr( $address['state'] ); ?>" class="medium-text"/>
+																<input id="give-payment-address-state" type="text" name="give-payment-address[0][state]" value="<?php echo esc_attr($address['state']); ?>" class="medium-text"/>
 																<?php
 															}
 															?>
 														</div>
 													</div>
 
-													<div class="<?php echo( ! empty( $state_exists ) ? 'column-full' : 'column' ); ?> give-column give-column-zip">
+													<div class="<?php echo( ! empty($state_exists) ? 'column-full' : 'column'); ?> give-column give-column-zip">
 														<div class="give-wrap-address-zip">
-															<label for="give-payment-address-zip" class="order-data-address-line"><?php esc_html_e( 'Zip / Postal Code:', 'give' ); ?></label>
-															<input id="give-payment-address-zip" type="text" name="give-payment-address[0][zip]" value="<?php echo esc_attr( $address['zip'] ); ?>" class="medium-text"/>
+															<label for="give-payment-address-zip" class="order-data-address-line"><?php esc_html_e('Zip / Postal Code:', 'give'); ?></label>
+															<input id="give-payment-address-zip" type="text" name="give-payment-address[0][zip]" value="<?php echo esc_attr($address['zip']); ?>" class="medium-text"/>
 														</div>
 													</div>
 												</div>
@@ -738,7 +738,7 @@ 
 									 *
 									 * @param int $payment_id Payment id.
 									 */
-									do_action( 'give_payment_billing_details', $payment_id );
+									do_action('give_payment_billing_details', $payment_id);
 									?>
 
 								</div>
@@ -754,33 +754,33 @@ 
 							 *
 							 * @param int $payment_id Payment id.
 							 */
-							do_action( 'give_view_order_details_billing_after', $payment_id );
+							do_action('give_view_order_details_billing_after', $payment_id);
 							?>
 
 							<div id="give-payment-notes" class="postbox">
-								<h3 class="hndle"><?php esc_html_e( 'Donation Notes', 'give' ); ?></h3>
+								<h3 class="hndle"><?php esc_html_e('Donation Notes', 'give'); ?></h3>
 
 								<div class="inside">
 									<div id="give-payment-notes-inner">
 										<?php
-										$notes = give_get_payment_notes( $payment_id );
-										if ( ! empty( $notes ) ) {
+										$notes = give_get_payment_notes($payment_id);
+										if ( ! empty($notes)) {
 											$no_notes_display = ' style="display:none;"';
-											foreach ( $notes as $note ) :
+											foreach ($notes as $note) :
 
-												echo give_get_payment_note_html( $note, $payment_id );
+												echo give_get_payment_note_html($note, $payment_id);
 
 											endforeach;
 										} else {
 											$no_notes_display = '';
 										}
-										echo '<p class="give-no-payment-notes"' . $no_notes_display . '>' . esc_html__( 'No donation notes.', 'give' ) . '</p>';
+										echo '<p class="give-no-payment-notes"'.$no_notes_display.'>'.esc_html__('No donation notes.', 'give').'</p>';
 										?>
 									</div>
 									<textarea name="give-payment-note" id="give-payment-note" class="large-text"></textarea>
 
 									<div class="give-clearfix">
-										<button id="give-add-payment-note" class="button button-secondary button-small" data-payment-id="<?php echo absint( $payment_id ); ?>"><?php esc_html_e( 'Add Note', 'give' ); ?></button>
+										<button id="give-add-payment-note" class="button button-secondary button-small" data-payment-id="<?php echo absint($payment_id); ?>"><?php esc_html_e('Add Note', 'give'); ?></button>
 									</div>
 
 								</div>
@@ -796,7 +796,7 @@ 
 							 *
 							 * @param int $payment_id Payment id.
 							 */
-							do_action( 'give_view_order_details_main_after', $payment_id );
+							do_action('give_view_order_details_main_after', $payment_id);
 							?>
 
 						</div>
@@ -818,11 +818,11 @@ 
 		 *
 		 * @param int $payment_id Payment id.
 		 */
-		do_action( 'give_view_order_details_form_bottom', $payment_id );
+		do_action('give_view_order_details_form_bottom', $payment_id);
 
-		wp_nonce_field( 'give_update_payment_details_nonce' );
+		wp_nonce_field('give_update_payment_details_nonce');
 		?>
-		<input type="hidden" name="give_payment_id" value="<?php echo esc_attr( $payment_id ); ?>"/>
+		<input type="hidden" name="give_payment_id" value="<?php echo esc_attr($payment_id); ?>"/>
 		<input type="hidden" name="give_action" value="update_payment_details"/>
 	</form>
 	<?php
@@ -833,6 +833,6 @@ 
 	 *
 	 * @param int $payment_id Payment id.
 	 */
-	do_action( 'give_view_order_details_after', $payment_id );
+	do_action('give_view_order_details_after', $payment_id);
 	?>
 </div><!-- /.wrap -->

includes/admin/payments/actions.php

Spacing +124 added lines, -124 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,52 +25,52 @@ 
  *
  * @return      void
  */
-function give_update_payment_details( $data ) {
+function give_update_payment_details($data) {
 
-	if ( ! current_user_can( 'edit_give_payments', $data['give_payment_id'] ) ) {
-		wp_die( esc_html__( 'You do not have permission to edit payments.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! current_user_can('edit_give_payments', $data['give_payment_id'])) {
+		wp_die(esc_html__('You do not have permission to edit payments.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 	}
 
-	check_admin_referer( 'give_update_payment_details_nonce' );
+	check_admin_referer('give_update_payment_details_nonce');
 
 	// Retrieve the payment ID.
-	$payment_id = absint( $data['give_payment_id'] );
+	$payment_id = absint($data['give_payment_id']);
 
 	/* @var Give_Payment $payment */
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
 	// Retrieve existing payment meta.
 	$meta      = $payment->get_meta();
 	$user_info = $payment->user_info;
 
 	$status = $data['give-payment-status'];
-	$date   = sanitize_text_field( $data['give-payment-date'] );
-	$hour   = sanitize_text_field( $data['give-payment-time-hour'] );
+	$date   = sanitize_text_field($data['give-payment-date']);
+	$hour   = sanitize_text_field($data['give-payment-time-hour']);
 
 	// Restrict to our high and low.
-	if ( $hour > 23 ) {
+	if ($hour > 23) {
 		$hour = 23;
-	} elseif ( $hour < 0 ) {
+	} elseif ($hour < 0) {
 		$hour = 00;
 	}
 
-	$minute = sanitize_text_field( $data['give-payment-time-min'] );
+	$minute = sanitize_text_field($data['give-payment-time-min']);
 
 	// Restrict to our high and low.
-	if ( $minute > 59 ) {
+	if ($minute > 59) {
 		$minute = 59;
-	} elseif ( $minute < 0 ) {
+	} elseif ($minute < 0) {
 		$minute = 00;
 	}
 
-	$address = give_clean( $data['give-payment-address'][0] );
+	$address = give_clean($data['give-payment-address'][0]);
 
 	$curr_total = $payment->total;
-	$new_total  = give_maybe_sanitize_amount( ( ! empty( $data['give-payment-total'] ) ? $data['give-payment-total'] : 0 ) );
-	$date       = date( 'Y-m-d', strtotime( $date ) ) . ' ' . $hour . ':' . $minute . ':00';
+	$new_total  = give_maybe_sanitize_amount(( ! empty($data['give-payment-total']) ? $data['give-payment-total'] : 0));
+	$date       = date('Y-m-d', strtotime($date)).' '.$hour.':'.$minute.':00';
 
-	$curr_donor_id = sanitize_text_field( $data['give-current-donor'] );
-	$new_donor_id  = sanitize_text_field( $data['donor-id'] );
+	$curr_donor_id = sanitize_text_field($data['give-current-donor']);
+	$new_donor_id  = sanitize_text_field($data['donor-id']);
 
 	/**
 	 * Fires before updating edited donation.
@@ -80,100 +80,100 @@ 
 	 *
 	 * @param int $payment_id The ID of the payment.
 	 */
-	do_action( 'give_update_edited_donation', $payment_id );
+	do_action('give_update_edited_donation', $payment_id);
 
 	$payment->date = $date;
 	$updated       = $payment->save();
 
-	if ( 0 === $updated ) {
-		wp_die( esc_html__( 'Error Updating Donation.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 400 ) );
+	if (0 === $updated) {
+		wp_die(esc_html__('Error Updating Donation.', 'give'), esc_html__('Error', 'give'), array('response' => 400));
 	}
 
 	$donor_changed = false;
 
-	if ( isset( $data['give-new-donor'] ) && $data['give-new-donor'] == '1' ) {
+	if (isset($data['give-new-donor']) && $data['give-new-donor'] == '1') {
 
-		$email = isset( $data['give-new-donor-email'] ) ? sanitize_text_field( $data['give-new-donor-email'] ) : '';
-		$names = isset( $data['give-new-donor-name'] ) ? sanitize_text_field( $data['give-new-donor-name'] ) : '';
+		$email = isset($data['give-new-donor-email']) ? sanitize_text_field($data['give-new-donor-email']) : '';
+		$names = isset($data['give-new-donor-name']) ? sanitize_text_field($data['give-new-donor-name']) : '';
 
-		if ( empty( $email ) || empty( $names ) ) {
-			wp_die( esc_html__( 'New donors require a name and email address.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 400 ) );
+		if (empty($email) || empty($names)) {
+			wp_die(esc_html__('New donors require a name and email address.', 'give'), esc_html__('Error', 'give'), array('response' => 400));
 		}
 
-		$donor = new Give_Donor( $email );
-		if ( empty( $donor->id ) ) {
-			$donor_data = array( 'name' => $names, 'email' => $email );
-			$user_id       = email_exists( $email );
-			if ( false !== $user_id ) {
+		$donor = new Give_Donor($email);
+		if (empty($donor->id)) {
+			$donor_data = array('name' => $names, 'email' => $email);
+			$user_id = email_exists($email);
+			if (false !== $user_id) {
 				$donor_data['user_id'] = $user_id;
 			}
 
-			if ( ! $donor->create( $donor_data ) ) {
+			if ( ! $donor->create($donor_data)) {
 				// Failed to crete the new donor, assume the previous donor.
 				$donor_changed = false;
-				$donor         = new Give_Donor( $curr_donor_id );
-				give_set_error( 'give-payment-new-donor-fail', __( 'Error creating new donor.', 'give' ) );
+				$donor         = new Give_Donor($curr_donor_id);
+				give_set_error('give-payment-new-donor-fail', __('Error creating new donor.', 'give'));
 			}
 		}
 
 		$new_donor_id = $donor->id;
 
-		$previous_donor = new Give_Donor( $curr_donor_id );
+		$previous_donor = new Give_Donor($curr_donor_id);
 
 		$donor_changed = true;
 
-	} elseif ( $curr_donor_id !== $new_donor_id ) {
+	} elseif ($curr_donor_id !== $new_donor_id) {
 
-		$donor = new Give_Donor( $new_donor_id );
+		$donor = new Give_Donor($new_donor_id);
 		$email    = $donor->email;
 		$names    = $donor->name;
 
-		$previous_donor = new Give_Donor( $curr_donor_id );
+		$previous_donor = new Give_Donor($curr_donor_id);
 
 		$donor_changed = true;
 
 	} else {
-		$donor = new Give_Donor( $curr_donor_id );
+		$donor = new Give_Donor($curr_donor_id);
 		$email    = $donor->email;
 		$names    = $donor->name;
 
 	}
 
-	if ( $donor_changed ) {
+	if ($donor_changed) {
 
 		// Setup first and last name from input values.
-		$names      = explode( ' ', $names );
-		$first_name = ! empty( $names[0] ) ? $names[0] : '';
+		$names      = explode(' ', $names);
+		$first_name = ! empty($names[0]) ? $names[0] : '';
 		$last_name  = '';
-		if ( ! empty( $names[1] ) ) {
-			unset( $names[0] );
-			$last_name = implode( ' ', $names );
+		if ( ! empty($names[1])) {
+			unset($names[0]);
+			$last_name = implode(' ', $names);
 		}
 
 		$payment->first_name = $first_name;
 		$payment->last_name  = $last_name;
 
 		// Remove the stats and payment from the previous donor and attach it to the new donor.
-		$previous_donor->remove_payment( $payment_id, false );
-		$donor->attach_payment( $payment_id, false );
+		$previous_donor->remove_payment($payment_id, false);
+		$donor->attach_payment($payment_id, false);
 
-		if ( 'publish' == $status ) {
+		if ('publish' == $status) {
 
 			// Reduce previous user donation count and amount.
 			$previous_donor->decrease_donation_count();
-			$previous_donor->decrease_value( $curr_total );
+			$previous_donor->decrease_value($curr_total);
 
 			// If donation was completed adjust stats of new donors.
 			$donor->increase_purchase_count();
-			$donor->increase_value( $new_total );
+			$donor->increase_value($new_total);
 		}
 
 		$payment->customer_id = $donor->id;
 	} else {
 
-		if ( 'publish' === $status ) {
+		if ('publish' === $status) {
 			// Update user donation stat.
-			$donor->update_donation_value( $curr_total, $new_total );
+			$donor->update_donation_value($curr_total, $new_total);
 		}
 	}
 
@@ -184,10 +184,10 @@ 
 	$payment->total      = $new_total;
 
 	// Check for payment notes.
-	if ( ! empty( $data['give-payment-note'] ) ) {
+	if ( ! empty($data['give-payment-note'])) {
 
-		$note = wp_kses( $data['give-payment-note'], array() );
-		give_insert_payment_note( $payment_id, $note );
+		$note = wp_kses($data['give-payment-note'], array());
+		give_insert_payment_note($payment_id, $note);
 
 	}
 
@@ -195,17 +195,17 @@ 
 	$payment->status = $status;
 
 	// Adjust total store earnings if the payment total has been changed.
-	if ( $new_total !== $curr_total && 'publish' == $status ) {
+	if ($new_total !== $curr_total && 'publish' == $status) {
 
-		if ( $new_total > $curr_total ) {
+		if ($new_total > $curr_total) {
 			// Increase if our new total is higher.
 			$difference = $new_total - $curr_total;
-			give_increase_total_earnings( $difference );
+			give_increase_total_earnings($difference);
 
-		} elseif ( $curr_total > $new_total ) {
+		} elseif ($curr_total > $new_total) {
 			// Decrease if our new total is lower.
 			$difference = $curr_total - $new_total;
-			give_decrease_total_earnings( $difference );
+			give_decrease_total_earnings($difference);
 
 		}
 	}
@@ -213,16 +213,16 @@ 
 	$payment->save();
 
 	// Get new give form ID.
-	$new_form_id     = absint( $data['give-payment-form-select'] );
-	$current_form_id = absint( $payment->get_meta( '_give_payment_form_id' ) );
+	$new_form_id     = absint($data['give-payment-form-select']);
+	$current_form_id = absint($payment->get_meta('_give_payment_form_id'));
 
 	// We are adding payment transfer code in last to remove any conflict with above functionality.
 	// For example: above code will automatically handle form stat (increase/decrease) when payment status changes.
 	// Check if user want to transfer current payment to new give form id.
-	if ( $new_form_id && $new_form_id != $current_form_id ) {
+	if ($new_form_id && $new_form_id != $current_form_id) {
 
 		// Get new give form title.
-		$new_form_title = get_the_title( $new_form_id );
+		$new_form_title = get_the_title($new_form_id);
 
 		// Update new give form data in payment data.
 		$payment_meta               = $payment->get_meta();
@@ -230,43 +230,43 @@ 
 		$payment_meta['form_id']    = $new_form_id;
 
 		// Update price id post meta data for set donation form.
-		if ( ! give_has_variable_prices( $new_form_id ) ) {
+		if ( ! give_has_variable_prices($new_form_id)) {
 			$payment_meta['price_id'] = '';
 		}
 
 		// Update payment give form meta data.
-		$payment->update_meta( '_give_payment_form_id', $new_form_id );
-		$payment->update_meta( '_give_payment_form_title', $new_form_title );
-		$payment->update_meta( '_give_payment_meta', $payment_meta );
+		$payment->update_meta('_give_payment_form_id', $new_form_id);
+		$payment->update_meta('_give_payment_form_title', $new_form_title);
+		$payment->update_meta('_give_payment_meta', $payment_meta);
 
 		// Update price id payment metadata.
-		if ( ! give_has_variable_prices( $new_form_id ) ) {
-			$payment->update_meta( '_give_payment_price_id', '' );
+		if ( ! give_has_variable_prices($new_form_id)) {
+			$payment->update_meta('_give_payment_price_id', '');
 		}
 
 		// If donation was completed, adjust stats of forms.
-		if ( 'publish' == $status ) {
+		if ('publish' == $status) {
 
 			// Decrease sale of old give form. For other payment status.
-			$current_form = new Give_Donate_Form( $current_form_id );
+			$current_form = new Give_Donate_Form($current_form_id);
 			$current_form->decrease_sales();
-			$current_form->decrease_earnings( $curr_total );
+			$current_form->decrease_earnings($curr_total);
 
 			// Increase sale of new give form.
-			$new_form = new Give_Donate_Form( $new_form_id );
+			$new_form = new Give_Donate_Form($new_form_id);
 			$new_form->increase_sales();
-			$new_form->increase_earnings( $new_total );
+			$new_form->increase_earnings($new_total);
 		}
 
 		// Re setup payment to update new meta value in object.
-		$payment->update_payment_setup( $payment->ID );
+		$payment->update_payment_setup($payment->ID);
 	}
 
 	// Update price id if current form is variable form.
 	/* @var Give_Donate_Form $form */
-	$form = new Give_Donate_Form( $payment->form_id );
+	$form = new Give_Donate_Form($payment->form_id);
 
-	if ( isset( $data['give-variable-price'] ) && $form->has_variable_prices() ) {
+	if (isset($data['give-variable-price']) && $form->has_variable_prices()) {
 
 		// Get payment meta data.
 		$payment_meta = $payment->get_meta();
@@ -275,9 +275,9 @@ 
 		$price_id = '';
 
 		// Get price info
-		if( 0 <= $data['give-variable-price'] ) {
-			foreach ( $form->prices as $variable_price ) {
-				if( $new_total === give_maybe_sanitize_amount( $variable_price['_give_amount'] ) ) {
+		if (0 <= $data['give-variable-price']) {
+			foreach ($form->prices as $variable_price) {
+				if ($new_total === give_maybe_sanitize_amount($variable_price['_give_amount'])) {
 					$price_info = $variable_price;
 					break;
 				}
@@ -285,15 +285,15 @@ 
 		}
 
 		// Set price id.
-		if( ! empty( $price_info ) ) {
+		if ( ! empty($price_info)) {
 			$price_id = $data['give-variable-price'];
 
-			if( $data['give-variable-price'] !== $price_info['_give_id']['level_id'] ) {
+			if ($data['give-variable-price'] !== $price_info['_give_id']['level_id']) {
 				// Set price id to amount match.
 				$price_id = $price_info['_give_id']['level_id'];
 			}
 
-		} elseif( $form->is_custom_price_mode() ){
+		} elseif ($form->is_custom_price_mode()) {
 			$price_id = 'custom';
 		}
 
@@ -301,11 +301,11 @@ 
 		$payment_meta['price_id'] = $price_id;
 
 		// Update payment give form meta data.
-		$payment->update_meta( '_give_payment_price_id', $price_id );
-		$payment->update_meta( '_give_payment_meta', $payment_meta );
+		$payment->update_meta('_give_payment_price_id', $price_id);
+		$payment->update_meta('_give_payment_meta', $payment_meta);
 
 		// Re setup payment to update new meta value in object.
-		$payment->update_payment_setup( $payment->ID );
+		$payment->update_payment_setup($payment->ID);
 	}
 
 	/**
@@ -316,13 +316,13 @@ 
 	 *
 	 * @param int $payment_id The ID of the payment.
 	 */
-	do_action( 'give_updated_edited_donation', $payment_id );
+	do_action('give_updated_edited_donation', $payment_id);
 
-	wp_safe_redirect( admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&give-message=payment-updated&id=' . $payment_id ) );
+	wp_safe_redirect(admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&give-message=payment-updated&id='.$payment_id));
 	exit;
 }
 
-add_action( 'give_update_payment_details', 'give_update_payment_details' );
+add_action('give_update_payment_details', 'give_update_payment_details');
 
 /**
  * Trigger a Donation Deletion.
@@ -333,48 +333,48 @@ 
  *
  * @return void
  */
-function give_trigger_donation_delete( $data ) {
-	if ( wp_verify_nonce( $data['_wpnonce'], 'give_donation_nonce' ) ) {
+function give_trigger_donation_delete($data) {
+	if (wp_verify_nonce($data['_wpnonce'], 'give_donation_nonce')) {
 
-		$payment_id = absint( $data['purchase_id'] );
+		$payment_id = absint($data['purchase_id']);
 
-		if ( ! current_user_can( 'edit_give_payments', $payment_id ) ) {
-			wp_die( __( 'You do not have permission to edit payments.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! current_user_can('edit_give_payments', $payment_id)) {
+			wp_die(__('You do not have permission to edit payments.', 'give'), __('Error', 'give'), array('response' => 403));
 		}
 
-		give_delete_donation( $payment_id );
-		wp_redirect( admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&give-message=donation_deleted' ) );
+		give_delete_donation($payment_id);
+		wp_redirect(admin_url('edit.php?post_type=give_forms&page=give-payment-history&give-message=donation_deleted'));
 		give_die();
 	}
 }
 
-add_action( 'give_delete_payment', 'give_trigger_donation_delete' );
+add_action('give_delete_payment', 'give_trigger_donation_delete');
 
 /**
  * AJAX Store Donation Note
  */
 function give_ajax_store_payment_note() {
 
-	$payment_id = absint( $_POST['payment_id'] );
-	$note       = wp_kses( $_POST['note'], array() );
+	$payment_id = absint($_POST['payment_id']);
+	$note       = wp_kses($_POST['note'], array());
 
-	if ( ! current_user_can( 'edit_give_payments', $payment_id ) ) {
-		wp_die( __( 'You do not have permission to edit payments.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! current_user_can('edit_give_payments', $payment_id)) {
+		wp_die(__('You do not have permission to edit payments.', 'give'), __('Error', 'give'), array('response' => 403));
 	}
 
-	if ( empty( $payment_id ) ) {
-		die( '-1' );
+	if (empty($payment_id)) {
+		die('-1');
 	}
 
-	if ( empty( $note ) ) {
-		die( '-1' );
+	if (empty($note)) {
+		die('-1');
 	}
 
-	$note_id = give_insert_payment_note( $payment_id, $note );
-	die( give_get_payment_note_html( $note_id ) );
+	$note_id = give_insert_payment_note($payment_id, $note);
+	die(give_get_payment_note_html($note_id));
 }
 
-add_action( 'wp_ajax_give_insert_payment_note', 'give_ajax_store_payment_note' );
+add_action('wp_ajax_give_insert_payment_note', 'give_ajax_store_payment_note');
 
 /**
  * Triggers a donation note deletion without ajax
@@ -385,24 +385,24 @@ 
  *
  * @return void
  */
-function give_trigger_payment_note_deletion( $data ) {
+function give_trigger_payment_note_deletion($data) {
 
-	if ( ! wp_verify_nonce( $data['_wpnonce'], 'give_delete_payment_note_' . $data['note_id'] ) ) {
+	if ( ! wp_verify_nonce($data['_wpnonce'], 'give_delete_payment_note_'.$data['note_id'])) {
 		return;
 	}
 
-	if ( ! current_user_can( 'edit_give_payments', $data['payment_id'] ) ) {
-		wp_die( esc_html__( 'You do not have permission to edit payments.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! current_user_can('edit_give_payments', $data['payment_id'])) {
+		wp_die(esc_html__('You do not have permission to edit payments.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 	}
 
-	$edit_order_url = admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&give-message=donation-note-deleted&id=' . absint( $data['payment_id'] ) );
+	$edit_order_url = admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&give-message=donation-note-deleted&id='.absint($data['payment_id']));
 
-	give_delete_payment_note( $data['note_id'], $data['payment_id'] );
+	give_delete_payment_note($data['note_id'], $data['payment_id']);
 
-	wp_redirect( $edit_order_url );
+	wp_redirect($edit_order_url);
 }
 
-add_action( 'give_delete_payment_note', 'give_trigger_payment_note_deletion' );
+add_action('give_delete_payment_note', 'give_trigger_payment_note_deletion');
 
 /**
  * Delete a payment note deletion with ajax
@@ -413,16 +413,16 @@ 
  */
 function give_ajax_delete_payment_note() {
 
-	if ( ! current_user_can( 'edit_give_payments', $_POST['payment_id'] ) ) {
-		wp_die( esc_html__( 'You do not have permission to edit payments.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! current_user_can('edit_give_payments', $_POST['payment_id'])) {
+		wp_die(esc_html__('You do not have permission to edit payments.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 	}
 
-	if ( give_delete_payment_note( $_POST['note_id'], $_POST['payment_id'] ) ) {
-		die( '1' );
+	if (give_delete_payment_note($_POST['note_id'], $_POST['payment_id'])) {
+		die('1');
 	} else {
-		die( '-1' );
+		die('-1');
 	}
 
 }
 
-add_action( 'wp_ajax_give_delete_payment_note', 'give_ajax_delete_payment_note' );
+add_action('wp_ajax_give_delete_payment_note', 'give_ajax_delete_payment_note');

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

Spacing +11 added lines, -11 removed lines

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

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 );
-				$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);
+				$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/pdf-reports.php

Spacing +159 added lines, -159 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly..
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,134 +25,134 @@ 
  *
  * @uses   give_pdf
  */
-function give_generate_pdf( $data ) {
+function give_generate_pdf($data) {
 
-	if ( ! current_user_can( 'view_give_reports' ) ) {
-		wp_die( __( 'You do not have permission to generate PDF sales reports.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! current_user_can('view_give_reports')) {
+		wp_die(__('You do not have permission to generate PDF sales reports.', 'give'), __('Error', 'give'), array('response' => 403));
 	}
 
-	if ( ! wp_verify_nonce( $_GET['_wpnonce'], 'give_generate_pdf' ) ) {
-		wp_die( __( 'Nonce verification failed.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! wp_verify_nonce($_GET['_wpnonce'], 'give_generate_pdf')) {
+		wp_die(__('Nonce verification failed.', 'give'), __('Error', 'give'), array('response' => 403));
 	}
 
-	if ( ! file_exists( GIVE_PLUGIN_DIR . '/includes/libraries/give-pdf.php' ) ) {
-		wp_die( __( 'Dependency missing.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! file_exists(GIVE_PLUGIN_DIR.'/includes/libraries/give-pdf.php')) {
+		wp_die(__('Dependency missing.', 'give'), __('Error', 'give'), array('response' => 403));
 	}
 
-	require_once GIVE_PLUGIN_DIR . '/includes/libraries/give-pdf.php';
+	require_once GIVE_PLUGIN_DIR.'/includes/libraries/give-pdf.php';
 
 	$daterange = utf8_decode(
 		sprintf(
 		/* translators: 1: start date 2: end date */
-			__( '%1$s to %2$s', 'give' ),
-			date_i18n( give_date_format(), mktime( 0, 0, 0, 1, 1, date( 'Y' ) ) ),
-			date_i18n( give_date_format() )
+			__('%1$s to %2$s', 'give'),
+			date_i18n(give_date_format(), mktime(0, 0, 0, 1, 1, date('Y'))),
+			date_i18n(give_date_format())
 		)
 	);
 
-	$categories_enabled = give_is_setting_enabled( give_get_option( 'categories', 'disabled' ) );
-	$tags_enabled       = give_is_setting_enabled( give_get_option( 'tags', 'disabled' ) );
+	$categories_enabled = give_is_setting_enabled(give_get_option('categories', 'disabled'));
+	$tags_enabled       = give_is_setting_enabled(give_get_option('tags', 'disabled'));
 
-	$pdf          = new Give_PDF( 'L', 'mm', 'A', true, 'UTF-8', false );
-	$default_font = apply_filters( 'give_pdf_default_font', 'Helvetica' );
+	$pdf          = new Give_PDF('L', 'mm', 'A', true, 'UTF-8', false);
+	$default_font = apply_filters('give_pdf_default_font', 'Helvetica');
 	$custom_font  = 'dejavusans';
 	$font_style   = '';
 
 	if (
-		file_exists( GIVE_PLUGIN_DIR . '/includes/libraries/tcpdf/fonts/CODE2000.TTF' ) &&
+		file_exists(GIVE_PLUGIN_DIR.'/includes/libraries/tcpdf/fonts/CODE2000.TTF') &&
 
 		// RIAL exist for backward compatibility.
-		in_array( give_get_currency(), array( 'RIAL', 'RUB', 'IRR' ) )
+		in_array(give_get_currency(), array('RIAL', 'RUB', 'IRR'))
 	) {
-		TCPDF_FONTS::addTTFfont( GIVE_PLUGIN_DIR . '/includes/libraries/tcpdf/fonts/CODE2000.TTF', '' );
+		TCPDF_FONTS::addTTFfont(GIVE_PLUGIN_DIR.'/includes/libraries/tcpdf/fonts/CODE2000.TTF', '');
 		$custom_font = 'CODE2000';
 		$font_style  = 'B';
 	}
 
-	$pdf->AddPage( 'L', 'A4' );
-	$pdf->setImageScale( 1.5 );
-	$pdf->SetTitle( utf8_decode( __( 'Donation report for the current year for all forms', 'give' ) ) );
-	$pdf->SetAuthor( utf8_decode( __( 'Give - Democratizing Generosity', 'give' ) ) );
-	$pdf->SetCreator( utf8_decode( __( 'Give - Democratizing Generosity', 'give' ) ) );
+	$pdf->AddPage('L', 'A4');
+	$pdf->setImageScale(1.5);
+	$pdf->SetTitle(utf8_decode(__('Donation report for the current year for all forms', 'give')));
+	$pdf->SetAuthor(utf8_decode(__('Give - Democratizing Generosity', 'give')));
+	$pdf->SetCreator(utf8_decode(__('Give - Democratizing Generosity', 'give')));
 
-	$pdf->Image( apply_filters( 'give_pdf_export_logo', GIVE_PLUGIN_URL . 'assets/images/give-logo-small.png' ), 247, 8 );
+	$pdf->Image(apply_filters('give_pdf_export_logo', GIVE_PLUGIN_URL.'assets/images/give-logo-small.png'), 247, 8);
 
-	$pdf->SetMargins( 8, 8, 8 );
-	$pdf->SetX( 8 );
+	$pdf->SetMargins(8, 8, 8);
+	$pdf->SetX(8);
 
-	$pdf->SetFont( $default_font, '', 16 );
-	$pdf->SetTextColor( 50, 50, 50 );
-	$pdf->Cell( 0, 3, utf8_decode( __( 'Donation report for the current year for all forms', 'give' ) ), 0, 2, 'L', false );
+	$pdf->SetFont($default_font, '', 16);
+	$pdf->SetTextColor(50, 50, 50);
+	$pdf->Cell(0, 3, utf8_decode(__('Donation report for the current year for all forms', 'give')), 0, 2, 'L', false);
 
-	$pdf->SetFont( $default_font, '', 13 );
-	$pdf->SetTextColor( 150, 150, 150 );
-	$pdf->Ln( 1 );
-	$pdf->Cell( 0, 6, utf8_decode( __( 'Date Range: ', 'give' ) ) . $daterange, 0, 2, 'L', false );
+	$pdf->SetFont($default_font, '', 13);
+	$pdf->SetTextColor(150, 150, 150);
+	$pdf->Ln(1);
+	$pdf->Cell(0, 6, utf8_decode(__('Date Range: ', 'give')).$daterange, 0, 2, 'L', false);
 	$pdf->Ln();
-	$pdf->SetTextColor( 50, 50, 50 );
-	$pdf->SetFont( $default_font, '', 14 );
-	$pdf->Cell( 0, 10, utf8_decode( __( 'Table View', 'give' ) ), 0, 2, 'L', false );
-	$pdf->SetFont( $default_font, '', 12 );
+	$pdf->SetTextColor(50, 50, 50);
+	$pdf->SetFont($default_font, '', 14);
+	$pdf->Cell(0, 10, utf8_decode(__('Table View', 'give')), 0, 2, 'L', false);
+	$pdf->SetFont($default_font, '', 12);
 
-	$pdf->SetFillColor( 238, 238, 238 );
-	$pdf->SetTextColor( 0, 0, 0, 100 ); // Set Black color.
-	$pdf->Cell( 50, 6, utf8_decode( __( 'Form Name', 'give' ) ), 1, 0, 'L', true );
-	$pdf->Cell( 50, 6, utf8_decode( __( 'Price', 'give' ) ), 1, 0, 'L', true );
+	$pdf->SetFillColor(238, 238, 238);
+	$pdf->SetTextColor(0, 0, 0, 100); // Set Black color.
+	$pdf->Cell(50, 6, utf8_decode(__('Form Name', 'give')), 1, 0, 'L', true);
+	$pdf->Cell(50, 6, utf8_decode(__('Price', 'give')), 1, 0, 'L', true);
 
 	// Display Categories Heading only, if user has opted for it.
-	if ( $categories_enabled ) {
-		$pdf->Cell( 45, 6, utf8_decode( __( 'Categories', 'give' ) ), 1, 0, 'L', true );
+	if ($categories_enabled) {
+		$pdf->Cell(45, 6, utf8_decode(__('Categories', 'give')), 1, 0, 'L', true);
 	}
 
 	// Display Tags Heading only, if user has opted for it.
-	if ( $tags_enabled ) {
-		$pdf->Cell( 45, 6, utf8_decode( __( 'Tags', 'give' ) ), 1, 0, 'L', true );
+	if ($tags_enabled) {
+		$pdf->Cell(45, 6, utf8_decode(__('Tags', 'give')), 1, 0, 'L', true);
 	}
 
-	$pdf->Cell( 45, 6, utf8_decode( __( 'Number of Donations', 'give' ) ), 1, 0, 'L', true );
-	$pdf->Cell( 45, 6, utf8_decode( __( 'Income to Date', 'give' ) ), 1, 1, 'L', true );
+	$pdf->Cell(45, 6, utf8_decode(__('Number of Donations', 'give')), 1, 0, 'L', true);
+	$pdf->Cell(45, 6, utf8_decode(__('Income to Date', 'give')), 1, 1, 'L', true);
 
 	// Set Custom Font to support various currencies.
-	$pdf->SetFont( apply_filters( 'give_pdf_custom_font', $custom_font ), $font_style, 12 );
+	$pdf->SetFont(apply_filters('give_pdf_custom_font', $custom_font), $font_style, 12);
 
-	$year       = date( 'Y' );
-	$give_forms = get_posts( array(
+	$year       = date('Y');
+	$give_forms = get_posts(array(
 		'post_type'      => 'give_forms',
 		'year'           => $year,
-		'posts_per_page' => - 1,
+		'posts_per_page' => -1,
 		'supply_filter'  => false,
-	) );
+	));
 
-	if ( $give_forms ) {
-		$pdf->SetWidths( array( 50, 50, 45, 45, 45, 45 ) );
+	if ($give_forms) {
+		$pdf->SetWidths(array(50, 50, 45, 45, 45, 45));
 
-		foreach ( $give_forms as $form ):
-			$pdf->SetFillColor( 255, 255, 255 );
+		foreach ($give_forms as $form):
+			$pdf->SetFillColor(255, 255, 255);
 
 			$title = $form->post_title;
 
-			if ( give_has_variable_prices( $form->ID ) ) {
-				$price = html_entity_decode( give_price_range( $form->ID, false ) );
+			if (give_has_variable_prices($form->ID)) {
+				$price = html_entity_decode(give_price_range($form->ID, false));
 			} else {
-				$price = give_currency_filter( give_get_form_price( $form->ID ), '', true );
+				$price = give_currency_filter(give_get_form_price($form->ID), '', true);
 			}
 
 			// Display Categories Data only, if user has opted for it.
 			$categories = array();
-			if ( $categories_enabled ) {
-				$categories = get_the_term_list( $form->ID, 'give_forms_category', '', ', ', '' );
-				$categories = ! is_wp_error( $categories ) ? strip_tags( $categories ) : '';
+			if ($categories_enabled) {
+				$categories = get_the_term_list($form->ID, 'give_forms_category', '', ', ', '');
+				$categories = ! is_wp_error($categories) ? strip_tags($categories) : '';
 			}
 
 			// Display Tags Data only, if user has opted for it.
 			$tags = array();
-			if ( $tags_enabled ) {
-				$tags = get_the_term_list( $form->ID, 'give_forms_tag', '', ', ', '' );
-				$tags = ! is_wp_error( $tags ) ? strip_tags( $tags ) : '';
+			if ($tags_enabled) {
+				$tags = get_the_term_list($form->ID, 'give_forms_tag', '', ', ', '');
+				$tags = ! is_wp_error($tags) ? strip_tags($tags) : '';
 			}
 
-			$sales    = give_get_form_sales_stats( $form->ID );
-			$earnings = give_currency_filter( give_format_amount( give_get_form_earnings_stats( $form->ID ), array( 'sanitize' => false, ) ), '', true );
+			$sales    = give_get_form_sales_stats($form->ID);
+			$earnings = give_currency_filter(give_format_amount(give_get_form_earnings_stats($form->ID), array('sanitize' => false,)), '', true);
 
 			// This will help filter data before appending it to PDF Receipt.
 			$prepare_pdf_data   = array();
@@ -160,53 +160,53 @@ 
 			$prepare_pdf_data[] = $price;
 
 			// Append Categories Data only, if user has opted for it.
-			if ( $categories_enabled ) {
+			if ($categories_enabled) {
 				$prepare_pdf_data[] = $categories;
 			}
 
 			// Append Tags Data only, if user has opted for it.
-			if ( $tags_enabled ) {
+			if ($tags_enabled) {
 				$prepare_pdf_data[] = $tags;
 			}
 
 			$prepare_pdf_data[] = $sales;
 			$prepare_pdf_data[] = $earnings;
 
-			$pdf->Row( $prepare_pdf_data );
+			$pdf->Row($prepare_pdf_data);
 
 		endforeach;
 	} else {
 
 		// Fix: Minor Styling Alignment Issue for PDF.
-		if ( $categories_enabled && $tags_enabled ) {
+		if ($categories_enabled && $tags_enabled) {
 			$no_found_width = 280;
-		} elseif ( $categories_enabled || $tags_enabled ) {
+		} elseif ($categories_enabled || $tags_enabled) {
 			$no_found_width = 235;
 		} else {
 			$no_found_width = 190;
 		}
-		$title = utf8_decode( __( 'No forms found.', 'give' ) );
-		$pdf->MultiCell( $no_found_width, 5, $title, 1, 'C', false, 1, '', '', true, 0, false, true, 0, 'T', false );
+		$title = utf8_decode(__('No forms found.', 'give'));
+		$pdf->MultiCell($no_found_width, 5, $title, 1, 'C', false, 1, '', '', true, 0, false, true, 0, 'T', false);
 	}// End if().
 	$pdf->Ln();
-	$pdf->SetTextColor( 50, 50, 50 );
-	$pdf->SetFont( $default_font, '', 14 );
+	$pdf->SetTextColor(50, 50, 50);
+	$pdf->SetFont($default_font, '', 14);
 
 	// Output Graph on a new page.
-	$pdf->AddPage( 'L', 'A4' );
-	$pdf->Cell( 0, 10, utf8_decode( __( 'Graph View', 'give' ) ), 0, 2, 'L', false );
-	$pdf->SetFont( $default_font, '', 12 );
+	$pdf->AddPage('L', 'A4');
+	$pdf->Cell(0, 10, utf8_decode(__('Graph View', 'give')), 0, 2, 'L', false);
+	$pdf->SetFont($default_font, '', 12);
 
-	$image = html_entity_decode( urldecode( give_draw_chart_image() ) );
-	$image = str_replace( ' ', '%20', $image );
+	$image = html_entity_decode(urldecode(give_draw_chart_image()));
+	$image = str_replace(' ', '%20', $image);
 
-	$pdf->SetX( 25 );
-	$pdf->Image( $image . '&file=.png' );
-	$pdf->Ln( 7 );
-	$pdf->Output( apply_filters( 'give_sales_earnings_pdf_export_filename', 'give-report-' . date_i18n( 'Y-m-d' ) ) . '.pdf', 'D' );
+	$pdf->SetX(25);
+	$pdf->Image($image.'&file=.png');
+	$pdf->Ln(7);
+	$pdf->Output(apply_filters('give_sales_earnings_pdf_export_filename', 'give-report-'.date_i18n('Y-m-d')).'.pdf', 'D');
 }
 
-add_action( 'give_generate_pdf', 'give_generate_pdf' );
+add_action('give_generate_pdf', 'give_generate_pdf');
 
 /**
  * Draws Chart for PDF Report.
@@ -223,38 +223,38 @@ 
  * @return string $chart->getUrl() URL for the Google Chart
  */
 function give_draw_chart_image() {
-	require_once GIVE_PLUGIN_DIR . '/includes/libraries/googlechartlib/GoogleChart.php';
-	require_once GIVE_PLUGIN_DIR . '/includes/libraries/googlechartlib/markers/GoogleChartShapeMarker.php';
-	require_once GIVE_PLUGIN_DIR . '/includes/libraries/googlechartlib/markers/GoogleChartTextMarker.php';
+	require_once GIVE_PLUGIN_DIR.'/includes/libraries/googlechartlib/GoogleChart.php';
+	require_once GIVE_PLUGIN_DIR.'/includes/libraries/googlechartlib/markers/GoogleChartShapeMarker.php';
+	require_once GIVE_PLUGIN_DIR.'/includes/libraries/googlechartlib/markers/GoogleChartTextMarker.php';
 
-	$chart = new GoogleChart( 'lc', 900, 330 );
+	$chart = new GoogleChart('lc', 900, 330);
 
 	$i        = 1;
 	$earnings = "";
 	$sales    = "";
 
-	while ( $i <= 12 ) :
-		$earnings .= give_get_earnings_by_date( null, $i, date( 'Y' ) ) . ",";
-		$sales    .= give_get_sales_by_date( null, $i, date( 'Y' ) ) . ",";
-		$i ++;
+	while ($i <= 12) :
+		$earnings .= give_get_earnings_by_date(null, $i, date('Y')).",";
+		$sales    .= give_get_sales_by_date(null, $i, date('Y')).",";
+		$i++;
 	endwhile;
 
-	$earnings_array = explode( ",", $earnings );
-	$sales_array    = explode( ",", $sales );
+	$earnings_array = explode(",", $earnings);
+	$sales_array    = explode(",", $sales);
 
 	$i = 0;
-	while ( $i <= 11 ) {
-		if ( empty( $sales_array[ $i ] ) ) {
-			$sales_array[ $i ] = 0;
+	while ($i <= 11) {
+		if (empty($sales_array[$i])) {
+			$sales_array[$i] = 0;
 		}
-		$i ++;
+		$i++;
 	}
 
 	$min_earnings   = 0;
-	$max_earnings   = max( $earnings_array );
-	$earnings_scale = round( $max_earnings, - 1 );
+	$max_earnings   = max($earnings_array);
+	$earnings_scale = round($max_earnings, - 1);
 
-	$data = new GoogleChartData( array(
+	$data = new GoogleChartData(array(
 		$earnings_array[0],
 		$earnings_array[1],
 		$earnings_array[2],
@@ -267,25 +267,25 @@ 
 		$earnings_array[9],
 		$earnings_array[10],
 		$earnings_array[11],
-	) );
+	));
 
-	$data->setLegend( __( 'Income', 'give' ) );
-	$data->setColor( '1b58a3' );
-	$chart->addData( $data );
+	$data->setLegend(__('Income', 'give'));
+	$data->setColor('1b58a3');
+	$chart->addData($data);
 
-	$shape_marker = new GoogleChartShapeMarker( GoogleChartShapeMarker::CIRCLE );
-	$shape_marker->setColor( '000000' );
-	$shape_marker->setSize( 7 );
-	$shape_marker->setBorder( 2 );
-	$shape_marker->setData( $data );
-	$chart->addMarker( $shape_marker );
+	$shape_marker = new GoogleChartShapeMarker(GoogleChartShapeMarker::CIRCLE);
+	$shape_marker->setColor('000000');
+	$shape_marker->setSize(7);
+	$shape_marker->setBorder(2);
+	$shape_marker->setData($data);
+	$chart->addMarker($shape_marker);
 
-	$value_marker = new GoogleChartTextMarker( GoogleChartTextMarker::VALUE );
-	$value_marker->setColor( '000000' );
-	$value_marker->setData( $data );
-	$chart->addMarker( $value_marker );
+	$value_marker = new GoogleChartTextMarker(GoogleChartTextMarker::VALUE);
+	$value_marker->setColor('000000');
+	$value_marker->setData($data);
+	$chart->addMarker($value_marker);
 
-	$data = new GoogleChartData( array(
+	$data = new GoogleChartData(array(
 		$sales_array[0],
 		$sales_array[1],
 		$sales_array[2],
@@ -298,46 +298,46 @@ 
 		$sales_array[9],
 		$sales_array[10],
 		$sales_array[11],
-	) );
-	$data->setLegend( __( 'Donations', 'give' ) );
-	$data->setColor( 'ff6c1c' );
-	$chart->addData( $data );
-
-	$chart->setTitle( __( 'Donations by Month for all Give Forms', 'give' ), '336699', 18 );
-
-	$chart->setScale( 0, $max_earnings );
-
-	$y_axis = new GoogleChartAxis( 'y' );
-	$y_axis->setDrawTickMarks( true )->setLabels( array( 0, $max_earnings ) );
-	$chart->addAxis( $y_axis );
-
-	$x_axis = new GoogleChartAxis( 'x' );
-	$x_axis->setTickMarks( 5 );
-	$x_axis->setLabels( array(
-		__( 'Jan', 'give' ),
-		__( 'Feb', 'give' ),
-		__( 'Mar', 'give' ),
-		__( 'Apr', 'give' ),
-		__( 'May', 'give' ),
-		__( 'June', 'give' ),
-		__( 'July', 'give' ),
-		__( 'Aug', 'give' ),
-		__( 'Sept', 'give' ),
-		__( 'Oct', 'give' ),
-		__( 'Nov', 'give' ),
-		__( 'Dec', 'give' ),
-	) );
-	$chart->addAxis( $x_axis );
-
-	$shape_marker = new GoogleChartShapeMarker( GoogleChartShapeMarker::CIRCLE );
-	$shape_marker->setSize( 6 );
-	$shape_marker->setBorder( 2 );
-	$shape_marker->setData( $data );
-	$chart->addMarker( $shape_marker );
-
-	$value_marker = new GoogleChartTextMarker( GoogleChartTextMarker::VALUE );
-	$value_marker->setData( $data );
-	$chart->addMarker( $value_marker );
+	));
+	$data->setLegend(__('Donations', 'give'));
+	$data->setColor('ff6c1c');
+	$chart->addData($data);
+
+	$chart->setTitle(__('Donations by Month for all Give Forms', 'give'), '336699', 18);
+
+	$chart->setScale(0, $max_earnings);
+
+	$y_axis = new GoogleChartAxis('y');
+	$y_axis->setDrawTickMarks(true)->setLabels(array(0, $max_earnings));
+	$chart->addAxis($y_axis);
+
+	$x_axis = new GoogleChartAxis('x');
+	$x_axis->setTickMarks(5);
+	$x_axis->setLabels(array(
+		__('Jan', 'give'),
+		__('Feb', 'give'),
+		__('Mar', 'give'),
+		__('Apr', 'give'),
+		__('May', 'give'),
+		__('June', 'give'),
+		__('July', 'give'),
+		__('Aug', 'give'),
+		__('Sept', 'give'),
+		__('Oct', 'give'),
+		__('Nov', 'give'),
+		__('Dec', 'give'),
+	));
+	$chart->addAxis($x_axis);
+
+	$shape_marker = new GoogleChartShapeMarker(GoogleChartShapeMarker::CIRCLE);
+	$shape_marker->setSize(6);
+	$shape_marker->setBorder(2);
+	$shape_marker->setData($data);
+	$chart->addMarker($shape_marker);
+
+	$value_marker = new GoogleChartTextMarker(GoogleChartTextMarker::VALUE);
+	$value_marker->setData($data);
+	$chart->addMarker($value_marker);
 
 	return $chart->getUrl();
 }

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

Spacing +10 added lines, -10 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -39,16 +39,16 @@ 
 	 * @return void
 	 */
 	public function headers() {
-		ignore_user_abort( true );
+		ignore_user_abort(true);
 
-		if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-			set_time_limit( 0 );
+		if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+			set_time_limit(0);
 		}
 
 		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');
 	}
 
 	/**
@@ -58,7 +58,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'));
 	}
 
 	/**
@@ -68,8 +68,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/export-actions.php

Spacing +29 added lines, -29 removed lines

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

includes/admin/tools/class-settings-api.php

Spacing +6 added lines, -6 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_API' ) ) :
+if ( ! class_exists('Give_Settings_API')) :
 
 	/**
 	 * Give_Settings_API.
@@ -34,7 +34,7 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'api';
-			$this->label = esc_html__( 'API', 'give' );
+			$this->label = esc_html__('API', 'give');
 
 			parent::__construct();
 		}
@@ -47,7 +47,7 @@ 
 		 */
 		public function get_settings() {
 			// Get settings.
-			$settings = apply_filters( 'give_settings_api', array(
+			$settings = apply_filters('give_settings_api', array(
 				array(
 					'id'   => 'give_tools_api',
 					'type' => 'title',
@@ -55,7 +55,7 @@ 
 				),
 				array(
 					'id'   => 'api',
-					'name' => esc_html__( 'API', 'give' ),
+					'name' => esc_html__('API', 'give'),
 					'type' => 'api',
 				),
 				array(
@@ -71,7 +71,7 @@ 
 			 * @since  1.8
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, $settings );
+			$settings = apply_filters('give_get_settings_'.$this->id, $settings);
 
 			// Output.
 			return $settings;