WordImpress / Give

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

Spacing +38 added lines, -38 removed lines

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

includes/admin/tools/views/html-admin-page-exports.php

Spacing +56 added lines, -56 removed lines

@@ -3,7 +3,7 @@ 
  * Admin View: Exports
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 } ?>
 
@@ -18,14 +18,14 @@ 
 				 *
 				 * @since 1.0
 				 */
-				do_action( 'give_tools_tab_export_content_top' );
+				do_action('give_tools_tab_export_content_top');
 				?>
 
 				<table class="widefat export-options-table give-table">
 					<thead>
 					<tr>
-						<th scope="col"><?php esc_html_e( 'Export Type', 'give' ); ?></th>
-						<th scope="col"><?php esc_html_e( 'Export Options', 'give' ); ?></th>
+						<th scope="col"><?php esc_html_e('Export Type', 'give'); ?></th>
+						<th scope="col"><?php esc_html_e('Export Options', 'give'); ?></th>
 					</tr>
 					</thead>
 					<tbody>
@@ -38,42 +38,42 @@ 
 					 *
 					 * @since 1.0
 					 */
-					do_action( 'give_tools_tab_export_table_top' );
+					do_action('give_tools_tab_export_table_top');
 					?>
 					<tr class="give-export-pdf-sales-earnings">
 						<td scope="row" class="row-title">
 							<h3>
-								<span><?php esc_html_e( 'Export PDF of Donations and Income', 'give' ); ?></span>
+								<span><?php esc_html_e('Export PDF of Donations and Income', 'give'); ?></span>
 							</h3>
-							<p><?php esc_html_e( 'Download a PDF of Donations and Income reports for all forms for the current year.', 'give' ); ?></p>
+							<p><?php esc_html_e('Download a PDF of Donations and Income reports for all forms for the current year.', 'give'); ?></p>
 						</td>
 						<td>
-							<a class="button" href="<?php echo wp_nonce_url( add_query_arg( array( 'give-action' => 'generate_pdf' ) ), 'give_generate_pdf' ); ?>">
-								<?php esc_html_e( 'Generate PDF', 'give' ); ?>
+							<a class="button" href="<?php echo wp_nonce_url(add_query_arg(array('give-action' => 'generate_pdf')), 'give_generate_pdf'); ?>">
+								<?php esc_html_e('Generate PDF', 'give'); ?>
 							</a>
 						</td>
 					</tr>
 					<tr class="alternate give-export-sales-earnings">
 						<td scope="row" class="row-title">
 							<h3>
-								<span><?php esc_html_e( 'Export Income and Donation Stats', 'give' ); ?></span>
+								<span><?php esc_html_e('Export Income and Donation Stats', 'give'); ?></span>
 							</h3>
-							<p><?php esc_html_e( 'Download a CSV of income and donations over time.', 'give' ); ?></p>
+							<p><?php esc_html_e('Download a CSV of income and donations over time.', 'give'); ?></p>
 						</td>
 						<td>
 							<form method="post">
 								<?php
 								printf(
 								/* translators: 1: start date dropdown 2: end date dropdown */
-									esc_html__( '%1$s to %2$s', 'give' ),
-									Give()->html->year_dropdown( 'start_year' ) . ' ' . Give()->html->month_dropdown( 'start_month' ),
-									Give()->html->year_dropdown( 'end_year' ) . ' ' . Give()->html->month_dropdown( 'end_month' )
+									esc_html__('%1$s to %2$s', 'give'),
+									Give()->html->year_dropdown('start_year').' '.Give()->html->month_dropdown('start_month'),
+									Give()->html->year_dropdown('end_year').' '.Give()->html->month_dropdown('end_month')
 								);
 								?>
 								<input type="hidden" name="give-action"
 								       value="earnings_export"/>
 								<input type="submit"
-								       value="<?php esc_attr_e( 'Generate CSV', 'give' ); ?>"
+								       value="<?php esc_attr_e('Generate CSV', 'give'); ?>"
 								       class="button-secondary"/>
 							</form>
 						</td>
@@ -81,37 +81,37 @@ 
 					<tr class="give-export-payment-history">
 						<td scope="row" class="row-title">
 							<h3>
-								<span><?php esc_html_e( 'Export Donation History', 'give' ); ?></span>
+								<span><?php esc_html_e('Export Donation History', 'give'); ?></span>
 							</h3>
-							<p><?php esc_html_e( 'Download a CSV of all donations recorded.', 'give' ); ?></p>
+							<p><?php esc_html_e('Download a CSV of all donations recorded.', 'give'); ?></p>
 						</td>
 						<td>
 							<form id="give-export-payments"
 							      class="give-export-form" method="post">
 								<?php
-								echo Give()->html->date_field( array(
+								echo Give()->html->date_field(array(
 									'id'          => 'give-payment-export-start',
 									'name'        => 'start',
-									'placeholder' => esc_attr__( 'Start date', 'give' ),
-								) );
+									'placeholder' => esc_attr__('Start date', 'give'),
+								));
 
-								echo Give()->html->date_field( array(
+								echo Give()->html->date_field(array(
 									'id'          => 'give-payment-export-end',
 									'name'        => 'end',
-									'placeholder' => esc_attr__( 'End date', 'give' ),
-								) );
+									'placeholder' => esc_attr__('End date', 'give'),
+								));
 								?>
 								<select name="status">
-									<option value="any"><?php esc_html_e( 'All Statuses', 'give' ); ?></option>
+									<option value="any"><?php esc_html_e('All Statuses', 'give'); ?></option>
 									<?php
 									$statuses = give_get_payment_statuses();
-									foreach ( $statuses as $status => $label ) {
-										echo '<option value="' . $status . '">' . $label . '</option>';
+									foreach ($statuses as $status => $label) {
+										echo '<option value="'.$status.'">'.$label.'</option>';
 									}
 									?>
 								</select>
 								<?php
-								if ( give_is_setting_enabled( give_get_option( 'categories' ) ) ) {
+								if (give_is_setting_enabled(give_get_option('categories'))) {
 									echo Give()->html->category_dropdown(
 										'give_forms_categories[]',
 										0,
@@ -121,12 +121,12 @@ 
 											'multiple'        => true,
 											'selected'        => array(),
 											'show_option_all' => false,
-											'placeholder'     => __( 'Choose one or more from categories', 'give' ),
+											'placeholder'     => __('Choose one or more from categories', 'give'),
 										)
 									);
 								}
 
-								if ( give_is_setting_enabled( give_get_option( 'tags' ) ) ) {
+								if (give_is_setting_enabled(give_get_option('tags'))) {
 									echo Give()->html->tags_dropdown(
 										'give_forms_tags[]',
 										0,
@@ -136,18 +136,18 @@ 
 											'multiple'        => true,
 											'selected'        => array(),
 											'show_option_all' => false,
-											'placeholder'     => __( 'Choose one or more from tags', 'give' ),
+											'placeholder'     => __('Choose one or more from tags', 'give'),
 										)
 									);
 								}
 
-								wp_nonce_field( 'give_ajax_export', 'give_ajax_export' );
+								wp_nonce_field('give_ajax_export', 'give_ajax_export');
 								?>
 								<input type="hidden" name="give-export-class"
 								       value="Give_Batch_Payments_Export"/>
 								<span>
 									<input type="submit"
-									       value="<?php esc_attr_e( 'Generate CSV', 'give' ); ?>"
+									       value="<?php esc_attr_e('Generate CSV', 'give'); ?>"
 									       class="button-secondary"/>
 									<span class="spinner"></span>
 								</span>
@@ -157,9 +157,9 @@ 
 					<tr class="alternate give-export-donors">
 						<td scope="row" class="row-title">
 							<h3>
-								<span><?php esc_html_e( 'Export Donors in CSV', 'give' ); ?></span>
+								<span><?php esc_html_e('Export Donors in CSV', 'give'); ?></span>
 							</h3>
-							<p><?php esc_html_e( 'Download an export of donors for all donation forms or only those who have given to a particular form.', 'give' ); ?></p>
+							<p><?php esc_html_e('Download an export of donors for all donation forms or only those who have given to a particular form.', 'give'); ?></p>
 						</td>
 						<td>
 							<form method="post" id="give_donor_export"
@@ -167,99 +167,99 @@ 
 
 								<?php
 								// Start Date form field for donors
-								echo Give()->html->date_field( array(
+								echo Give()->html->date_field(array(
 									'id'          => 'give_donor_export_start_date',
 									'name'        => 'donor_export_start_date',
-									'placeholder' => esc_attr__( 'Start date', 'give' ),
-								) );
+									'placeholder' => esc_attr__('Start date', 'give'),
+								));
 
 								// End Date form field for donors
-								echo Give()->html->date_field( array(
+								echo Give()->html->date_field(array(
 									'id'          => 'give_donor_export_end_date',
 									'name'        => 'donor_export_end_date',
-									'placeholder' => esc_attr__( 'End date', 'give' ),
-								) );
+									'placeholder' => esc_attr__('End date', 'give'),
+								));
 
 								// Donation forms dropdown for donors export
-								echo Give()->html->forms_dropdown( array(
+								echo Give()->html->forms_dropdown(array(
 									'name'   => 'forms',
 									'id'     => 'give_donor_export_form',
 									'chosen' => true,
-								) );
+								));
 								?>
 
 								<input type="submit"
-								       value="<?php esc_attr_e( 'Generate CSV', 'give' ); ?>"
+								       value="<?php esc_attr_e('Generate CSV', 'give'); ?>"
 								       class="button-secondary"/>
 
 								<div id="export-donor-options-wrap"
 								     class="give-clearfix">
-									<p><?php esc_html_e( 'Export Columns:', 'give' ); ?></p>
+									<p><?php esc_html_e('Export Columns:', 'give'); ?></p>
 									<ul id="give-export-option-ul">
 										<li>
 											<label for="give-export-fullname">
 												<input type="checkbox" checked
 												       name="give_export_option[full_name]"
-												       id="give-export-fullname"><?php esc_html_e( 'Name', 'give' ); ?>
+												       id="give-export-fullname"><?php esc_html_e('Name', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-email">
 												<input type="checkbox" checked
 												       name="give_export_option[email]"
-												       id="give-export-email"><?php esc_html_e( 'Email', 'give' ); ?>
+												       id="give-export-email"><?php esc_html_e('Email', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-address">
 												<input type="checkbox" checked
 												       name="give_export_option[address]"
-												       id="give-export-address"><?php esc_html_e( 'Address', 'give' ); ?>
+												       id="give-export-address"><?php esc_html_e('Address', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-userid">
 												<input type="checkbox" checked
 												       name="give_export_option[userid]"
-												       id="give-export-userid"><?php esc_html_e( 'User ID', 'give' ); ?>
+												       id="give-export-userid"><?php esc_html_e('User ID', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-donation-form">
 												<input type="checkbox" checked
 												       name="give_export_option[donation_form]"
-												       id="give-export-donation-form"><?php esc_html_e( 'Donation Form', 'give' ); ?>
+												       id="give-export-donation-form"><?php esc_html_e('Donation Form', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-first-donation-date">
 												<input type="checkbox" checked
 												       name="give_export_option[date_first_donated]"
-												       id="give-export-first-donation-date"><?php esc_html_e( 'First Donation Date', 'give' ); ?>
+												       id="give-export-first-donation-date"><?php esc_html_e('First Donation Date', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-donation-number">
 												<input type="checkbox" checked
 												       name="give_export_option[donations]"
-												       id="give-export-donation-number"><?php esc_html_e( 'Number of Donations', 'give' ); ?>
+												       id="give-export-donation-number"><?php esc_html_e('Number of Donations', 'give'); ?>
 											</label>
 										</li>
 										<li>
 											<label for="give-export-donation-sum">
 												<input type="checkbox" checked
 												       name="give_export_option[donation_sum]"
-												       id="give-export-donation-sum"><?php esc_html_e( 'Total Donated', 'give' ); ?>
+												       id="give-export-donation-sum"><?php esc_html_e('Total Donated', 'give'); ?>
 											</label>
 										</li>
 									</ul>
 								</div>
-								<?php wp_nonce_field( 'give_ajax_export', 'give_ajax_export' ); ?>
+								<?php wp_nonce_field('give_ajax_export', 'give_ajax_export'); ?>
 								<input type="hidden" name="give-export-class"
 								       value="Give_Batch_Donors_Export"/>
 								<input type="hidden"
 								       name="give_export_option[query_id]"
-								       value="<?php echo uniqid( 'give_' ); ?>"/>
+								       value="<?php echo uniqid('give_'); ?>"/>
 							</form>
 						</td>
 					</tr>
@@ -272,7 +272,7 @@ 
 					 *
 					 * @since 1.0
 					 */
-					do_action( 'give_tools_tab_export_table_bottom' );
+					do_action('give_tools_tab_export_table_bottom');
 					?>
 					</tbody>
 				</table>
@@ -283,7 +283,7 @@ 
 				 *
 				 * @since 1.0
 				 */
-				do_action( 'give_tools_tab_export_content_bottom' );
+				do_action('give_tools_tab_export_content_bottom');
 				?>
 
 			</div>

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

Indentation +10 added lines, -10 removed lines

@@ -514,16 +514,16 @@
 											<p>
 												<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' );
-
-                                                // 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>)';
-                                                }else{
-                                                    echo $donor_name;
-                                                }
-                                                ?>
+												$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>)';
+												}else{
+													echo $donor_name;
+												}
+												?>
 											</p>
 											<p>
 												<strong><?php esc_html_e( 'Donor Email:', 'give' ); ?></strong><br>

Braces +1 added lines, -1 removed lines

@@ -520,7 +520,7 @@
                                                 // 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>)';
-                                                }else{
+                                                } else{
                                                     echo $donor_name;
                                                 }
                                                 ?>

Spacing +143 added lines, -143 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -20,27 +20,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;
@@ -52,11 +52,11 @@ 
 	<h1 id="transaction-details-heading"><?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>
 
@@ -68,7 +68,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);
 	?>
 	<form id="give-edit-order-form" method="post">
 		<?php
@@ -79,7 +79,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">
@@ -95,12 +95,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">
@@ -113,33 +113,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>
 
@@ -153,13 +153,13 @@ 
 										 *
 										 * @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_get_payment_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_get_payment_amount($payment_id), false, false)); ?>"/>
 											</p>
 										</div>
 
@@ -171,7 +171,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>
@@ -189,17 +189,17 @@ 
 									 *
 									 * @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' ); ?>"/>
-											<?php if ( give_is_payment_complete( $payment_id ) ) : ?>
-												<a href="<?php echo esc_url( add_query_arg( array(
+											<input type="submit" class="button button-primary right" value="<?php esc_attr_e('Save Donation', 'give'); ?>"/>
+											<?php if (give_is_payment_complete($payment_id)) : ?>
+												<a href="<?php echo esc_url(add_query_arg(array(
 													'give-action' => 'email_links',
 													'purchase_id' => $payment_id,
-												) ) ); ?>" id="give-resend-receipt" class="button-secondary right"><?php esc_html_e( 'Resend Receipt', 'give' ); ?></a>
+												))); ?>" id="give-resend-receipt" class="button-secondary right"><?php esc_html_e('Resend Receipt', 'give'); ?></a>
 											<?php endif; ?>
 										</div>
 										<div class="clear"></div>
@@ -213,7 +213,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>
@@ -224,7 +224,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">
@@ -237,44 +237,44 @@ 
 										 *
 										 * @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&user=' . urlencode( esc_attr( give_get_payment_user_email( $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&user='.urlencode(esc_attr(give_get_payment_user_email($payment_id)))); ?>
+												<a href="<?php echo $purchase_url; ?>"><?php esc_html_e('View all donations for this donor &raquo;', 'give'); ?></a>
 											</p>
 										</div>
 
@@ -286,7 +286,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>
@@ -306,7 +306,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>
@@ -326,56 +326,56 @@ 
 							 *
 							 * @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" target="_blank">#%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>
-												<?php echo Give()->html->forms_dropdown( array(
+												<strong><?php esc_html_e('Donation Form Title:', 'give'); ?></strong><br>
+												<?php echo Give()->html->forms_dropdown(array(
 													'selected' => $payment_meta['form_id'],
 													'name'   => 'give-payment-form-select',
 													'id'     => 'give-payment-form-select',
 													'chosen' => true,
-												) ); ?>
+												)); ?>
 											</p>
 										</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 = '';
-														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.
@@ -384,12 +384,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( 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(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>
@@ -397,8 +397,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
@@ -411,7 +411,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);
 
 
 												/**
@@ -423,7 +423,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.
@@ -434,7 +434,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.
@@ -445,7 +445,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>
@@ -465,67 +465,67 @@ 
 							 *
 							 * @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" target="_blank">#%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>)';
-                                                }else{
+                                                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(
+												echo Give()->html->donor_dropdown(array(
 													'selected' => $donor->id,
 													'name'     => 'donor-id',
-												) );
+												));
 												?>
 											</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>
@@ -533,13 +533,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>
@@ -547,9 +547,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>
@@ -565,7 +565,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.
@@ -574,7 +574,7 @@ 
 									 *
 									 * @param int $payment_id Payment id.
 									 */
-									do_action( 'give_payment_view_details', $payment_id );
+									do_action('give_payment_view_details', $payment_id);
 									?>
 
 								</div>
@@ -590,11 +590,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">
 
@@ -604,59 +604,59 @@ 
 											<div class="data column-container">
 												<div class="column">
 													<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 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="column">
 													<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 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 class="column">
 													<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(
+														echo Give()->html->select(array(
 															'options'          => give_get_country_list(),
 															'name'             => 'give-payment-address[0][country]',
 															'selected'         => $address['country'],
 															'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'),
+														));
 														?>
 													</div>
 													<div id="give-order-address-state-wrap">
-														<label for="give-payment-address-state" class="order-data-address-line"><?php esc_html_e( 'State / Province / County:', 'give' ); ?></label>
+														<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 ) ) {
-															echo Give()->html->select( array(
+														$states = give_get_states($address['country']);
+														if ( ! empty($states)) {
+															echo Give()->html->select(array(
 																'options'          => $states,
 																'name'             => 'give-payment-address[0][state]',
 																'selected'         => $address['state'],
 																'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>
@@ -676,7 +676,7 @@ 
 									 *
 									 * @param int $payment_id Payment id.
 									 */
-									do_action( 'give_payment_billing_details', $payment_id );
+									do_action('give_payment_billing_details', $payment_id);
 									?>
 
 								</div>
@@ -692,32 +692,32 @@ 
 							 *
 							 * @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>
@@ -733,7 +733,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>
@@ -755,11 +755,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
@@ -770,6 +770,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/payments-history.php

Spacing +22 added lines, -22 removed lines

@@ -10,7 +10,7 @@ 
 */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,12 +25,12 @@ 
 */
 function give_payment_history_page() {
 
-	$give_payment = get_post_type_object( 'give_payment' );
+	$give_payment = get_post_type_object('give_payment');
 
-	if ( isset( $_GET['view'] ) && 'view-payment-details' == $_GET['view'] ) {
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/payments/view-payment-details.php';
+	if (isset($_GET['view']) && 'view-payment-details' == $_GET['view']) {
+		require_once GIVE_PLUGIN_DIR.'includes/admin/payments/view-payment-details.php';
 	} else {
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/payments/class-payments-table.php';
+		require_once GIVE_PLUGIN_DIR.'includes/admin/payments/class-payments-table.php';
 		$payments_table = new Give_Payment_History_Table();
 		$payments_table->prepare_items();
 	?>
@@ -44,17 +44,17 @@ 
 		 *
 		 * @since 1.7
 		 */
-		do_action( 'give_payments_page_top' );
+		do_action('give_payments_page_top');
 		?>
 
-		<form id="give-payments-advanced-filter" method="get" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-payment-history' ); ?>">
+		<form id="give-payments-advanced-filter" method="get" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-payment-history'); ?>">
 			<input type="hidden" name="post_type" value="give_forms" />
 			<input type="hidden" name="page" value="give-payment-history" />
 			<?php $payments_table->views() ?>
 			<?php $payments_table->advanced_filters(); ?>
 		</form>
 
-		<form id="give-payments-filter" method="get" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-payment-history' ); ?>">
+		<form id="give-payments-filter" method="get" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-payment-history'); ?>">
 			<input type="hidden" name="post_type" value="give_forms" />
 			<input type="hidden" name="page" value="give-payment-history" />
 			<?php $payments_table->display() ?>
@@ -66,7 +66,7 @@ 
 		 *
 		 * @since 1.7
 		 */
-		do_action( 'give_payments_page_bottom' );
+		do_action('give_payments_page_bottom');
 		?>
 
 	</div>
@@ -83,29 +83,29 @@ 
  * @param $title
  * @return string
  */
-function give_view_order_details_title( $admin_title, $title ) {
+function give_view_order_details_title($admin_title, $title) {
 
-	if ( 'give_forms_page_give-payment-history' != get_current_screen()->base ) {
+	if ('give_forms_page_give-payment-history' != get_current_screen()->base) {
 		return $admin_title;
 	}
 
-	if( ! isset( $_GET['give-action'] ) ) {
+	if ( ! isset($_GET['give-action'])) {
 		return $admin_title;
 	}
 
-	switch( $_GET['give-action'] ) :
+	switch ($_GET['give-action']) :
 
 		case 'view-payment-details' :
 			$title = sprintf(
 				/* translators: %s: admin title */
-				esc_html__( 'View Donation Details - %s', 'give' ),
+				esc_html__('View Donation Details - %s', 'give'),
 				$admin_title
 			);
 			break;
 		case 'edit-payment' :
 			$title = sprintf(
 				/* translators: %s: admin title */
-				esc_html__( 'Edit Donation - %s', 'give' ),
+				esc_html__('Edit Donation - %s', 'give'),
 				$admin_title
 			);
 			break;
@@ -116,7 +116,7 @@ 
 
 	return $title;
 }
-add_filter( 'admin_title', 'give_view_order_details_title', 10, 2 );
+add_filter('admin_title', 'give_view_order_details_title', 10, 2);
 
 /**
  * Intercept default Edit post links for Give payments and rewrite them to the View Order Details screen.
@@ -128,20 +128,20 @@ 
  * @param $context
  * @return string
  */
-function give_override_edit_post_for_payment_link( $url, $post_id = 0, $context ) {
+function give_override_edit_post_for_payment_link($url, $post_id = 0, $context) {
 
-	$post = get_post( $post_id );
+	$post = get_post($post_id);
 
-	if( ! $post ) {
+	if ( ! $post) {
 		return $url;
 	}
 
-	if( 'give_payment' != $post->post_type ) {
+	if ('give_payment' != $post->post_type) {
 		return $url;
 	}
 
-	$url = admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id=' . $post_id );
+	$url = admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id='.$post_id);
 
 	return $url;
 }
-add_filter( 'get_edit_post_link', 'give_override_edit_post_for_payment_link', 10, 3 );
+add_filter('get_edit_post_link', 'give_override_edit_post_for_payment_link', 10, 3);

includes/admin/donors/donor-functions.php

Spacing +12 added lines, -12 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,7 +23,7 @@ 
  *
  * @return array        The altered list of views.
  */
-function give_register_default_donor_views( $views ) {
+function give_register_default_donor_views($views) {
 
 	$default_views = array(
 		'overview' => 'give_donor_view',
@@ -31,11 +31,11 @@ 
 		'notes'    => 'give_donor_notes_view',
 	);
 
-	return array_merge( $views, $default_views );
+	return array_merge($views, $default_views);
 
 }
 
-add_filter( 'give_donor_views', 'give_register_default_donor_views', 1, 1 );
+add_filter('give_donor_views', 'give_register_default_donor_views', 1, 1);
 
 /**
  * Register a tab for the single donor view.
@@ -46,23 +46,23 @@ 
  *
  * @return array       The altered list of tabs
  */
-function give_register_default_donor_tabs( $tabs ) {
+function give_register_default_donor_tabs($tabs) {
 
 	$default_tabs = array(
 		'overview' => array(
 			'dashicon' => 'dashicons-admin-users',
-			'title' => __( 'Donor Profile', 'give' ),
+			'title' => __('Donor Profile', 'give'),
 		),
 		'notes'    => array(
 			'dashicon' => 'dashicons-admin-comments',
-			'title' => __( 'Donor Notes', 'give' ),
+			'title' => __('Donor Notes', 'give'),
 		),
 	);
 
-	return array_merge( $tabs, $default_tabs );
+	return array_merge($tabs, $default_tabs);
 }
 
-add_filter( 'give_donor_tabs', 'give_register_default_donor_tabs', 1, 1 );
+add_filter('give_donor_tabs', 'give_register_default_donor_tabs', 1, 1);
 
 /**
  * Register the Delete icon as late as possible so it's at the bottom.
@@ -73,14 +73,14 @@ 
  *
  * @return array       The altered list of tabs, with 'delete' at the bottom.
  */
-function give_register_delete_donor_tab( $tabs ) {
+function give_register_delete_donor_tab($tabs) {
 
 	$tabs['delete'] = array(
 		'dashicon' => 'dashicons-trash',
-		'title' => esc_html__( 'Delete Donor', 'give' ),
+		'title' => esc_html__('Delete Donor', 'give'),
 	);
 
 	return $tabs;
 }
 
-add_filter( 'give_donor_tabs', 'give_register_delete_donor_tab', PHP_INT_MAX, 1 );
+add_filter('give_donor_tabs', 'give_register_delete_donor_tab', PHP_INT_MAX, 1);

includes/class-give-html-elements.php

Spacing +151 added lines, -151 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -35,7 +35,7 @@ 
 	 *
 	 * @return string       Donations dropdown.
 	 */
-	public function donations_dropdown( $args = array() ) {
+	public function donations_dropdown($args = array()) {
 
 		$defaults = array(
 			'name'        => 'donations',
@@ -45,32 +45,32 @@ 
 			'selected'    => 0,
 			'chosen'      => false,
 			'number'      => 30,
-			'placeholder' => __( 'Select a donation', 'give' ),
+			'placeholder' => __('Select a donation', 'give'),
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		$payments = new Give_Payments_Query( array(
+		$payments = new Give_Payments_Query(array(
 			'number' => $args['number'],
-		) );
+		));
 
 		$payments = $payments->get_payments();
 
 		$options = array();
 
 		// Provide nice human readable options.
-		if ( $payments ) {
+		if ($payments) {
 			$options[0] = $args['placeholder'];
-			foreach ( $payments as $payment ) {
+			foreach ($payments as $payment) {
 
-				$options[ absint( $payment->ID ) ] = esc_html( '#' . $payment->ID . ' - ' . $payment->email . ' - ' . $payment->form_title );
+				$options[absint($payment->ID)] = esc_html('#'.$payment->ID.' - '.$payment->email.' - '.$payment->form_title);
 
 			}
 		} else {
-			$options[0] = __( 'No donations found.', 'give' );
+			$options[0] = __('No donations found.', 'give');
 		}
 
-		$output = $this->select( array(
+		$output = $this->select(array(
 			'name'             => $args['name'],
 			'selected'         => $args['selected'],
 			'id'               => $args['id'],
@@ -82,7 +82,7 @@ 
 			'select_atts'      => $args['select_atts'],
 			'show_option_all'  => false,
 			'show_option_none' => false,
-		) );
+		));
 
 		return $output;
 	}
@@ -99,7 +99,7 @@ 
 	 *
 	 * @return string      Give forms dropdown.
 	 */
-	public function forms_dropdown( $args = array() ) {
+	public function forms_dropdown($args = array()) {
 
 		$defaults = array(
 			'name'        => 'forms',
@@ -109,39 +109,39 @@ 
 			'selected'    => 0,
 			'chosen'      => false,
 			'number'      => 30,
-			'placeholder' => esc_attr__( 'Select a Donation Form', 'give' ),
+			'placeholder' => esc_attr__('Select a Donation Form', 'give'),
 			'data'        => array(
 				'search-type' => 'form',
 			),
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		$forms = get_posts( array(
+		$forms = get_posts(array(
 			'post_type'      => 'give_forms',
 			'orderby'        => 'title',
 			'order'          => 'ASC',
 			'posts_per_page' => $args['number'],
-		) );
+		));
 
 		$options = array();
 
 		// Ensure the selected.
-		if ( false !== $args['selected'] && $args['selected'] !== 0 ) {
-			$options[ $args['selected'] ] = get_the_title( $args['selected'] );
+		if (false !== $args['selected'] && $args['selected'] !== 0) {
+			$options[$args['selected']] = get_the_title($args['selected']);
 		}
 
-		if ( $forms ) {
+		if ($forms) {
 			$options[0] = $args['placeholder'];
-			foreach ( $forms as $form ) {
-				$form_title                     = empty( $form->post_title ) ? sprintf( __( 'Untitled (#%s)', 'give' ), $form->ID ) : $form->post_title;
-				$options[ absint( $form->ID ) ] = esc_html( $form_title );
+			foreach ($forms as $form) {
+				$form_title                     = empty($form->post_title) ? sprintf(__('Untitled (#%s)', 'give'), $form->ID) : $form->post_title;
+				$options[absint($form->ID)] = esc_html($form_title);
 			}
 		} else {
-			$options[0] = esc_html__( 'No forms found.', 'give' );
+			$options[0] = esc_html__('No forms found.', 'give');
 		}
 
-		$output = $this->select( array(
+		$output = $this->select(array(
 			'name'             => $args['name'],
 			'selected'         => $args['selected'],
 			'id'               => $args['id'],
@@ -153,7 +153,7 @@ 
 			'show_option_all'  => false,
 			'show_option_none' => false,
 			'data'             => $args['data'],
-		) );
+		));
 
 		return $output;
 	}
@@ -170,7 +170,7 @@ 
 	 *
 	 * @return string      Donors dropdown.
 	 */
-	public function donor_dropdown( $args = array() ) {
+	public function donor_dropdown($args = array()) {
 
 		$defaults = array(
 			'name'        => 'donors',
@@ -179,57 +179,57 @@ 
 			'multiple'    => false,
 			'selected'    => 0,
 			'chosen'      => true,
-			'placeholder' => esc_attr__( 'Select a Donor', 'give' ),
+			'placeholder' => esc_attr__('Select a Donor', 'give'),
 			'number'      => 30,
 			'data'        => array(
 				'search-type' => 'donor',
 			),
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		$donors = Give()->donors->get_donors( array(
+		$donors = Give()->donors->get_donors(array(
 			'number' => $args['number']
-		) );
+		));
 
 		$options = array();
 
-		if ( $donors ) {
-			$options[0] = esc_html__( 'No donor attached', 'give' );
-			foreach ( $donors as $donor ) {
-				$options[ absint( $donor->id ) ] = esc_html( $donor->name . ' (' . $donor->email . ')' );
+		if ($donors) {
+			$options[0] = esc_html__('No donor attached', 'give');
+			foreach ($donors as $donor) {
+				$options[absint($donor->id)] = esc_html($donor->name.' ('.$donor->email.')');
 			}
 		} else {
-			$options[0] = esc_html__( 'No donors found.', 'give' );
+			$options[0] = esc_html__('No donors found.', 'give');
 		}
 
-		if ( ! empty( $args['selected'] ) ) {
+		if ( ! empty($args['selected'])) {
 
 			// If a selected customer has been specified, we need to ensure it's in the initial list of customers displayed.
-			if ( ! array_key_exists( $args['selected'], $options ) ) {
+			if ( ! array_key_exists($args['selected'], $options)) {
 
-				$donor = new Give_Donor( $args['selected'] );
+				$donor = new Give_Donor($args['selected']);
 
-				if ( $donor ) {
+				if ($donor) {
 
-					$options[ absint( $args['selected'] ) ] = esc_html( $donor->name . ' (' . $donor->email . ')' );
+					$options[absint($args['selected'])] = esc_html($donor->name.' ('.$donor->email.')');
 
 				}
 			}
 		}
 
-		$output = $this->select( array(
+		$output = $this->select(array(
 			'name'             => $args['name'],
 			'selected'         => $args['selected'],
 			'id'               => $args['id'],
-			'class'            => $args['class'] . ' give-customer-select',
+			'class'            => $args['class'].' give-customer-select',
 			'options'          => $options,
 			'multiple'         => $args['multiple'],
 			'chosen'           => $args['chosen'],
 			'show_option_all'  => false,
 			'show_option_none' => false,
 			'data'             => $args['data'],
-		) );
+		));
 
 		return $output;
 	}
@@ -248,21 +248,21 @@ 
 	 *
 	 * @return string           Categories dropdown.
 	 */
-	public function category_dropdown( $name = 'give_forms_categories', $selected = 0, $args = array() ) {
-		$categories = get_terms( 'give_forms_category', apply_filters( 'give_forms_category_dropdown', array() ) );
+	public function category_dropdown($name = 'give_forms_categories', $selected = 0, $args = array()) {
+		$categories = get_terms('give_forms_category', apply_filters('give_forms_category_dropdown', array()));
 		$options    = array();
 
-		foreach ( $categories as $category ) {
-			$options[ absint( $category->term_id ) ] = esc_html( $category->name );
+		foreach ($categories as $category) {
+			$options[absint($category->term_id)] = esc_html($category->name);
 		}
 
-		$output = $this->select( wp_parse_args( $args, array(
+		$output = $this->select(wp_parse_args($args, array(
 			'name'             => $name,
 			'selected'         => $selected,
 			'options'          => $options,
-			'show_option_all'  => esc_html__( 'All Categories', 'give' ),
+			'show_option_all'  => esc_html__('All Categories', 'give'),
 			'show_option_none' => false,
-		) ) );
+		)));
 
 		return $output;
 	}
@@ -281,21 +281,21 @@ 
 	 *
 	 * @return string           Tags dropdown.
 	 */
-	public function tags_dropdown( $name = 'give_forms_tags', $selected = 0, $args = array() ) {
-		$tags    = get_terms( 'give_forms_tag', apply_filters( 'give_forms_tag_dropdown', array() ) );
+	public function tags_dropdown($name = 'give_forms_tags', $selected = 0, $args = array()) {
+		$tags    = get_terms('give_forms_tag', apply_filters('give_forms_tag_dropdown', array()));
 		$options = array();
 
-		foreach ( $tags as $tag ) {
-			$options[ absint( $tag->term_id ) ] = esc_html( $tag->name );
+		foreach ($tags as $tag) {
+			$options[absint($tag->term_id)] = esc_html($tag->name);
 		}
 
-		$output = $this->select( wp_parse_args( $args, array(
+		$output = $this->select(wp_parse_args($args, array(
 			'name'             => $name,
 			'selected'         => $selected,
 			'options'          => $options,
-			'show_option_all'  => esc_html__( 'All Tags', 'give' ),
+			'show_option_all'  => esc_html__('All Tags', 'give'),
 			'show_option_none' => false,
-		) ) );
+		)));
 
 		return $output;
 	}
@@ -315,25 +315,25 @@ 
 	 *
 	 * @return string               Years dropdown.
 	 */
-	public function year_dropdown( $name = 'year', $selected = 0, $years_before = 5, $years_after = 0 ) {
-		$current    = date( 'Y' );
-		$start_year = $current - absint( $years_before );
-		$end_year   = $current + absint( $years_after );
-		$selected   = empty( $selected ) ? date( 'Y' ) : $selected;
+	public function year_dropdown($name = 'year', $selected = 0, $years_before = 5, $years_after = 0) {
+		$current    = date('Y');
+		$start_year = $current - absint($years_before);
+		$end_year   = $current + absint($years_after);
+		$selected   = empty($selected) ? date('Y') : $selected;
 		$options    = array();
 
-		while ( $start_year <= $end_year ) {
-			$options[ absint( $start_year ) ] = $start_year;
-			$start_year ++;
+		while ($start_year <= $end_year) {
+			$options[absint($start_year)] = $start_year;
+			$start_year++;
 		}
 
-		$output = $this->select( array(
+		$output = $this->select(array(
 			'name'             => $name,
 			'selected'         => $selected,
 			'options'          => $options,
 			'show_option_all'  => false,
 			'show_option_none' => false,
-		) );
+		));
 
 		return $output;
 	}
@@ -351,23 +351,23 @@ 
 	 *
 	 * @return string           Months dropdown.
 	 */
-	public function month_dropdown( $name = 'month', $selected = 0 ) {
+	public function month_dropdown($name = 'month', $selected = 0) {
 		$month    = 1;
 		$options  = array();
-		$selected = empty( $selected ) ? date( 'n' ) : $selected;
+		$selected = empty($selected) ? date('n') : $selected;
 
-		while ( $month <= 12 ) {
-			$options[ absint( $month ) ] = give_month_num_to_name( $month );
-			$month ++;
+		while ($month <= 12) {
+			$options[absint($month)] = give_month_num_to_name($month);
+			$month++;
 		}
 
-		$output = $this->select( array(
+		$output = $this->select(array(
 			'name'             => $name,
 			'selected'         => $selected,
 			'options'          => $options,
 			'show_option_all'  => false,
 			'show_option_none' => false,
-		) );
+		));
 
 		return $output;
 	}
@@ -384,7 +384,7 @@ 
 	 *
 	 * @return string      The dropdown.
 	 */
-	public function select( $args = array() ) {
+	public function select($args = array()) {
 		$defaults = array(
 			'options'          => array(),
 			'name'             => null,
@@ -395,67 +395,67 @@ 
 			'placeholder'      => null,
 			'multiple'         => false,
 			'select_atts'      => false,
-			'show_option_all'  => __( 'All', 'give' ),
-			'show_option_none' => __( 'None', 'give' ),
+			'show_option_all'  => __('All', 'give'),
+			'show_option_none' => __('None', 'give'),
 			'data'             => array(),
 			'readonly'         => false,
 			'disabled'         => false,
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
 		$data_elements = '';
-		foreach ( $args['data'] as $key => $value ) {
-			$data_elements .= ' data-' . esc_attr( $key ) . '="' . esc_attr( $value ) . '"';
+		foreach ($args['data'] as $key => $value) {
+			$data_elements .= ' data-'.esc_attr($key).'="'.esc_attr($value).'"';
 		}
 
-		if ( $args['multiple'] ) {
+		if ($args['multiple']) {
 			$multiple = ' MULTIPLE';
 		} else {
 			$multiple = '';
 		}
 
-		if ( $args['chosen'] ) {
+		if ($args['chosen']) {
 			$args['class'] .= ' give-select-chosen';
 		}
 
-		if ( $args['placeholder'] ) {
+		if ($args['placeholder']) {
 			$placeholder = $args['placeholder'];
 		} else {
 			$placeholder = '';
 		}
 
-		$output = '<select name="' . esc_attr( $args['name'] ) . '" id="' . esc_attr( sanitize_key( str_replace( '-', '_', $args['id'] ) ) ) . '" class="give-select ' . esc_attr( $args['class'] ) . '"' . $multiple . ' ' . $args['select_atts'] . ' data-placeholder="' . $placeholder . '"' . $data_elements . '>';
+		$output = '<select name="'.esc_attr($args['name']).'" id="'.esc_attr(sanitize_key(str_replace('-', '_', $args['id']))).'" class="give-select '.esc_attr($args['class']).'"'.$multiple.' '.$args['select_atts'].' data-placeholder="'.$placeholder.'"'.$data_elements.'>';
 
-		if ( $args['show_option_all'] ) {
-			if ( $args['multiple'] ) {
-				$selected = selected( true, in_array( 0, $args['selected'] ), false );
+		if ($args['show_option_all']) {
+			if ($args['multiple']) {
+				$selected = selected(true, in_array(0, $args['selected']), false);
 			} else {
-				$selected = selected( $args['selected'], 0, false );
+				$selected = selected($args['selected'], 0, false);
 			}
-			$output .= '<option value="all"' . $selected . '>' . esc_html( $args['show_option_all'] ) . '</option>';
+			$output .= '<option value="all"'.$selected.'>'.esc_html($args['show_option_all']).'</option>';
 		}
 
-		if ( ! empty( $args['options'] ) ) {
+		if ( ! empty($args['options'])) {
 
-			if ( $args['show_option_none'] ) {
-				if ( $args['multiple'] ) {
-					$selected = selected( true, in_array( - 1, $args['selected'] ), false );
+			if ($args['show_option_none']) {
+				if ($args['multiple']) {
+					$selected = selected(true, in_array( -1, $args['selected'] ), false);
 				} else {
-					$selected = selected( $args['selected'], - 1, false );
+					$selected = selected($args['selected'], - 1, false);
 				}
-				$output .= '<option value="-1"' . $selected . '>' . esc_html( $args['show_option_none'] ) . '</option>';
+				$output .= '<option value="-1"'.$selected.'>'.esc_html($args['show_option_none']).'</option>';
 			}
 
-			foreach ( $args['options'] as $key => $option ) {
+			foreach ($args['options'] as $key => $option) {
 
-				if ( $args['multiple'] && is_array( $args['selected'] ) ) {
-					$selected = selected( true, in_array( $key, $args['selected'] ), false );
+				if ($args['multiple'] && is_array($args['selected'])) {
+					$selected = selected(true, in_array($key, $args['selected']), false);
 				} else {
-					$selected = selected( $args['selected'], $key, false );
+					$selected = selected($args['selected'], $key, false);
 				}
 
-				$output .= '<option value="' . esc_attr( $key ) . '"' . $selected . '>' . esc_html( $option ) . '</option>';
+				$output .= '<option value="'.esc_attr($key).'"'.$selected.'>'.esc_html($option).'</option>';
 			}
 		}
 
@@ -476,7 +476,7 @@ 
 	 *
 	 * @return string      The checkbox.
 	 */
-	public function checkbox( $args = array() ) {
+	public function checkbox($args = array()) {
 		$defaults = array(
 			'name'    => null,
 			'current' => null,
@@ -487,16 +487,16 @@ 
 			),
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
 		$options = '';
-		if ( ! empty( $args['options']['disabled'] ) ) {
+		if ( ! empty($args['options']['disabled'])) {
 			$options .= ' disabled="disabled"';
-		} elseif ( ! empty( $args['options']['readonly'] ) ) {
+		} elseif ( ! empty($args['options']['readonly'])) {
 			$options .= ' readonly';
 		}
 
-		$output = '<input type="checkbox"' . $options . ' name="' . esc_attr( $args['name'] ) . '" id="' . esc_attr( $args['name'] ) . '" class="' . $args['class'] . ' ' . esc_attr( $args['name'] ) . '" ' . checked( 1, $args['current'], false ) . ' />';
+		$output = '<input type="checkbox"'.$options.' name="'.esc_attr($args['name']).'" id="'.esc_attr($args['name']).'" class="'.$args['class'].' '.esc_attr($args['name']).'" '.checked(1, $args['current'], false).' />';
 
 		return $output;
 	}
@@ -513,22 +513,22 @@ 
 	 *
 	 * @return string      The text field.
 	 */
-	public function text( $args = array() ) {
+	public function text($args = array()) {
 		// Backwards compatibility.
-		if ( func_num_args() > 1 ) {
+		if (func_num_args() > 1) {
 			$args = func_get_args();
 
 			$name  = $args[0];
-			$value = isset( $args[1] ) ? $args[1] : '';
-			$label = isset( $args[2] ) ? $args[2] : '';
-			$desc  = isset( $args[3] ) ? $args[3] : '';
+			$value = isset($args[1]) ? $args[1] : '';
+			$label = isset($args[2]) ? $args[2] : '';
+			$desc  = isset($args[3]) ? $args[3] : '';
 		}
 
 		$defaults = array(
-			'name'         => isset( $name ) ? $name : 'text',
-			'value'        => isset( $value ) ? $value : null,
-			'label'        => isset( $label ) ? $label : null,
-			'desc'         => isset( $desc ) ? $desc : null,
+			'name'         => isset($name) ? $name : 'text',
+			'value'        => isset($value) ? $value : null,
+			'label'        => isset($label) ? $label : null,
+			'desc'         => isset($desc) ? $desc : null,
 			'placeholder'  => '',
 			'class'        => 'regular-text',
 			'disabled'     => false,
@@ -536,29 +536,29 @@ 
 			'data'         => false,
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
 		$disabled = '';
-		if ( $args['disabled'] ) {
+		if ($args['disabled']) {
 			$disabled = ' disabled="disabled"';
 		}
 
 		$data = '';
-		if ( ! empty( $args['data'] ) ) {
-			foreach ( $args['data'] as $key => $value ) {
-				$data .= 'data-' . $key . '="' . $value . '" ';
+		if ( ! empty($args['data'])) {
+			foreach ($args['data'] as $key => $value) {
+				$data .= 'data-'.$key.'="'.$value.'" ';
 			}
 		}
 
-		$output = '<span id="give-' . sanitize_key( $args['name'] ) . '-wrap">';
+		$output = '<span id="give-'.sanitize_key($args['name']).'-wrap">';
 
-		$output .= '<label class="give-label" for="give-' . sanitize_key( $args['name'] ) . '">' . esc_html( $args['label'] ) . '</label>';
+		$output .= '<label class="give-label" for="give-'.sanitize_key($args['name']).'">'.esc_html($args['label']).'</label>';
 
-		if ( ! empty( $args['desc'] ) ) {
-			$output .= '<span class="give-description">' . esc_html( $args['desc'] ) . '</span>';
+		if ( ! empty($args['desc'])) {
+			$output .= '<span class="give-description">'.esc_html($args['desc']).'</span>';
 		}
 
-		$output .= '<input type="text" name="' . esc_attr( $args['name'] ) . '" id="' . esc_attr( $args['name'] ) . '" autocomplete="' . esc_attr( $args['autocomplete'] ) . '" value="' . esc_attr( $args['value'] ) . '" placeholder="' . esc_attr( $args['placeholder'] ) . '" class="' . $args['class'] . '" ' . $data . '' . $disabled . '/>';
+		$output .= '<input type="text" name="'.esc_attr($args['name']).'" id="'.esc_attr($args['name']).'" autocomplete="'.esc_attr($args['autocomplete']).'" value="'.esc_attr($args['value']).'" placeholder="'.esc_attr($args['placeholder']).'" class="'.$args['class'].'" '.$data.''.$disabled.'/>';
 
 		$output .= '</span>';
 
@@ -577,15 +577,15 @@ 
 	 *
 	 * @return string      The date picker.
 	 */
-	public function date_field( $args = array() ) {
+	public function date_field($args = array()) {
 
-		if ( empty( $args['class'] ) ) {
+		if (empty($args['class'])) {
 			$args['class'] = 'give_datepicker';
-		} elseif ( ! strpos( $args['class'], 'give_datepicker' ) ) {
+		} elseif ( ! strpos($args['class'], 'give_datepicker')) {
 			$args['class'] .= ' give_datepicker';
 		}
 
-		return $this->text( $args );
+		return $this->text($args);
 	}
 
 	/**
@@ -600,7 +600,7 @@ 
 	 *
 	 * @return string      The textarea.
 	 */
-	public function textarea( $args = array() ) {
+	public function textarea($args = array()) {
 		$defaults = array(
 			'name'     => 'textarea',
 			'value'    => null,
@@ -610,21 +610,21 @@ 
 			'disabled' => false,
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
 		$disabled = '';
-		if ( $args['disabled'] ) {
+		if ($args['disabled']) {
 			$disabled = ' disabled="disabled"';
 		}
 
-		$output = '<span id="give-' . sanitize_key( $args['name'] ) . '-wrap">';
+		$output = '<span id="give-'.sanitize_key($args['name']).'-wrap">';
 
-		$output .= '<label class="give-label" for="give-' . sanitize_key( $args['name'] ) . '">' . esc_html( $args['label'] ) . '</label>';
+		$output .= '<label class="give-label" for="give-'.sanitize_key($args['name']).'">'.esc_html($args['label']).'</label>';
 
-		$output .= '<textarea name="' . esc_attr( $args['name'] ) . '" id="' . esc_attr( $args['name'] ) . '" class="' . $args['class'] . '"' . $disabled . '>' . esc_attr( $args['value'] ) . '</textarea>';
+		$output .= '<textarea name="'.esc_attr($args['name']).'" id="'.esc_attr($args['name']).'" class="'.$args['class'].'"'.$disabled.'>'.esc_attr($args['value']).'</textarea>';
 
-		if ( ! empty( $args['desc'] ) ) {
-			$output .= '<span class="give-description">' . esc_html( $args['desc'] ) . '</span>';
+		if ( ! empty($args['desc'])) {
+			$output .= '<span class="give-description">'.esc_html($args['desc']).'</span>';
 		}
 
 		$output .= '</span>';
@@ -644,7 +644,7 @@ 
 	 *
 	 * @return string      The text field with ajax search.
 	 */
-	public function ajax_user_search( $args = array() ) {
+	public function ajax_user_search($args = array()) {
 
 		$defaults = array(
 			'name'        => 'users',
@@ -655,13 +655,13 @@ 
 			'chosen'      => true,
 			'number'      => 30,
 			'select_atts' => '',
-			'placeholder' => __( 'Select a user', 'give' ),
+			'placeholder' => __('Select a user', 'give'),
 			'data'        => array(
 				'search-type' => 'user',
 			),
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
 		// Set initial args.
 		$get_users_args = array(
@@ -670,31 +670,31 @@ 
 
 		// Ensure selected user is not included in initial query.
 		// This is because sites with many users, it's not a guarantee the selected user will be returned.
-		if ( ! empty( $args['selected'] ) ) {
+		if ( ! empty($args['selected'])) {
 			$get_users_args['exclude'] = $args['selected'];
 		}
 
 		// Initial users array.
-		$users = apply_filters( 'give_ajax_user_search_initial_results', get_users( $get_users_args ), $args );
+		$users = apply_filters('give_ajax_user_search_initial_results', get_users($get_users_args), $args);
 
 		// Now add the selected user to the $users array if the arg is present.
-		if ( ! empty( $args['selected'] ) ) {
-			$selected_user =  apply_filters( 'give_ajax_user_search_selected_results', get_users( "include={$args['selected']}" ), $args );;
-			$users         = array_merge( $users, $selected_user );
+		if ( ! empty($args['selected'])) {
+			$selected_user = apply_filters('give_ajax_user_search_selected_results', get_users("include={$args['selected']}"), $args); ;
+			$users         = array_merge($users, $selected_user);
 		}
 
 		$options = array();
 
-		if ( $users ) {
+		if ($users) {
 			$options[0] = $args['placeholder'];
-			foreach ( $users as $user ) {
-				$options[ absint( $user->ID ) ] = esc_html( $user->user_login . ' (' . $user->user_email . ')' );
+			foreach ($users as $user) {
+				$options[absint($user->ID)] = esc_html($user->user_login.' ('.$user->user_email.')');
 			}
 		} else {
-			$options[0] = __( 'No users found.', 'give' );
+			$options[0] = __('No users found.', 'give');
 		}
 
-		$output = $this->select( array(
+		$output = $this->select(array(
 			'name'             => $args['name'],
 			'selected'         => $args['selected'],
 			'id'               => $args['id'],
@@ -707,7 +707,7 @@ 
 			'show_option_all'  => false,
 			'show_option_none' => false,
 			'data'             => $args['data'],
-		) );
+		));
 
 		return $output;
 

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

Spacing +50 added lines, -50 removed lines

@@ -9,13 +9,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded.
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -44,11 +44,11 @@ 
 	 */
 	public function __construct() {
 		// Set parent defaults.
-		parent::__construct( array(
-			'singular' => give_get_forms_label_singular(),    // Singular name of the listed records.
-			'plural'   => give_get_forms_label_plural(),        // Plural name of the listed records.
+		parent::__construct(array(
+			'singular' => give_get_forms_label_singular(), // Singular name of the listed records.
+			'plural'   => give_get_forms_label_plural(), // Plural name of the listed records.
 			'ajax'     => false                        // Does this table support ajax?.
-		) );
+		));
 	}
 
 	/**
@@ -62,19 +62,19 @@ 
 	 *
 	 * @return string Column Name.
 	 */
-	public function column_default( $item, $column_name ) {
+	public function column_default($item, $column_name) {
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 			case 'ID' :
 				return $item['ID_label'];
 			case 'payment_id' :
-				return empty( $item['payment_id'] ) ? esc_html__( 'n/a', 'give' ) : sprintf( "<a href=\"%s\" target=\"_blank\">{$item['payment_id']}</a>", get_edit_post_link( $item['payment_id'] ) );
+				return empty($item['payment_id']) ? esc_html__('n/a', 'give') : sprintf("<a href=\"%s\" target=\"_blank\">{$item['payment_id']}</a>", get_edit_post_link($item['payment_id']));
 			case 'gateway' :
-				return empty( $item['gateway'] ) ? esc_html__( 'n/a', 'give' ) : $item['gateway'];
+				return empty($item['gateway']) ? esc_html__('n/a', 'give') : $item['gateway'];
 			case 'error' :
-				return get_the_title( $item['ID'] ) ? get_the_title( $item['ID'] ) : esc_html__( 'Payment Error', 'give' );
+				return get_the_title($item['ID']) ? get_the_title($item['ID']) : esc_html__('Payment Error', 'give');
 			default:
-				return $item[ $column_name ];
+				return $item[$column_name];
 		}
 	}
 
@@ -88,27 +88,27 @@ 
 	 *
 	 * @return void
 	 */
-	public function column_message( $item ) { ?>
-		<a href="#TB_inline?width=640&amp;inlineId=log-message-<?php echo $item['ID']; ?>" class="thickbox give-error-log-details-link button button-small" data-tooltip="<?php esc_attr_e( 'View Log Message', 'give' ); ?>"><span class="dashicons dashicons-visibility"></span></a>
+	public function column_message($item) { ?>
+		<a href="#TB_inline?width=640&amp;inlineId=log-message-<?php echo $item['ID']; ?>" class="thickbox give-error-log-details-link button button-small" data-tooltip="<?php esc_attr_e('View Log Message', 'give'); ?>"><span class="dashicons dashicons-visibility"></span></a>
 		<div id="log-message-<?php echo $item['ID']; ?>" style="display:none;">
 			<?php
 
-			$log_message = get_post_field( 'post_content', $item['ID'] );
+			$log_message = get_post_field('post_content', $item['ID']);
 
-			$serialized = strpos( $log_message, '{"' );
+			$serialized = strpos($log_message, '{"');
 
 			// Check to see if the log message contains serialized information
-			if ( $serialized !== false ) {
-				$length = strlen( $log_message ) - $serialized;
-				$intro  = substr( $log_message, 0, - $length );
-				$data   = substr( $log_message, $serialized, strlen( $log_message ) - 1 );
+			if ($serialized !== false) {
+				$length = strlen($log_message) - $serialized;
+				$intro  = substr($log_message, 0, - $length);
+				$data   = substr($log_message, $serialized, strlen($log_message) - 1);
 
-				echo wpautop( $intro );
-				echo wpautop( '<strong>' . esc_html__( 'Log data:', 'give' ) . '</strong>' );
-				echo '<div style="word-wrap: break-word;">' . wpautop( $data ) . '</div>';
+				echo wpautop($intro);
+				echo wpautop('<strong>'.esc_html__('Log data:', 'give').'</strong>');
+				echo '<div style="word-wrap: break-word;">'.wpautop($data).'</div>';
 			} else {
 				// No serialized data found
-				echo wpautop( $log_message );
+				echo wpautop($log_message);
 			}
 			?>
 		</div>
@@ -124,12 +124,12 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'ID'         => esc_html__( 'Log ID', 'give' ),
-			'error'      => esc_html__( 'Error', 'give' ),
-			'gateway'    => esc_html__( 'Gateway', 'give' ),
-			'payment_id' => esc_html__( 'Donation ID', 'give' ),
-			'date'       => esc_html__( 'Date', 'give' ),
-			'message'    => esc_html__( 'Details', 'give' )
+			'ID'         => esc_html__('Log ID', 'give'),
+			'error'      => esc_html__('Error', 'give'),
+			'gateway'    => esc_html__('Gateway', 'give'),
+			'payment_id' => esc_html__('Donation ID', 'give'),
+			'date'       => esc_html__('Date', 'give'),
+			'message'    => esc_html__('Details', 'give')
 		);
 
 		return $columns;
@@ -143,7 +143,7 @@ 
 	 * @return int Current page number
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 	/**
@@ -153,7 +153,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	public function bulk_actions( $which = '' ) {
+	public function bulk_actions($which = '') {
 		give_log_views();
 	}
 
@@ -171,7 +171,7 @@ 
 
 		// Prevent the queries from getting cached.
 		// Without this there are occasional memory issues for some installs.
-		wp_suspend_cache_addition( true );
+		wp_suspend_cache_addition(true);
 
 		$logs_data = array();
 		$paged     = $this->get_paged();
@@ -181,17 +181,17 @@ 
 			'posts_per_page' => $this->per_page,
 		);
 
-		$logs = $give_logs->get_connected_logs( $log_query );
+		$logs = $give_logs->get_connected_logs($log_query);
 
-		if ( $logs ) {
-			foreach ( $logs as $log ) {
+		if ($logs) {
+			foreach ($logs as $log) {
 
 				$logs_data[] = array(
 					'ID'         => $log->ID,
-					'ID_label'   => '<span class=\'give-item-label give-item-label-gray\'>' . $log->ID . '</span>',
+					'ID_label'   => '<span class=\'give-item-label give-item-label-gray\'>'.$log->ID.'</span>',
 					'payment_id' => $log->post_parent,
 					'error'      => 'error',
-					'gateway'    => give_get_payment_gateway( $log->post_parent ),
+					'gateway'    => give_get_payment_gateway($log->post_parent),
 					'date'       => $log->post_date
 				);
 			}
@@ -213,19 +213,19 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
-		if ( 'top' === $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+	protected function display_tablenav($which) {
+		if ('top' === $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav <?php echo esc_attr($which); ?>">
 
 			<div class="alignleft actions bulkactions">
-				<?php $this->bulk_actions( $which ); ?>
+				<?php $this->bulk_actions($which); ?>
 			</div>
 			<?php
-			$this->extra_tablenav( $which );
-			$this->pagination( $which );
+			$this->extra_tablenav($which);
+			$this->pagination($which);
 			?>
 
 			<br class="clear"/>
@@ -252,14 +252,14 @@ 
 		$columns               = $this->get_columns();
 		$hidden                = array(); // No hidden columns
 		$sortable              = $this->get_sortable_columns();
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 		$this->items           = $this->get_logs();
-		$total_items           = $give_logs->get_log_count( 0, 'gateway_error' );
+		$total_items           = $give_logs->get_log_count(0, 'gateway_error');
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 				'total_items' => $total_items,
 				'per_page'    => $this->per_page,
-				'total_pages' => ceil( $total_items / $this->per_page ),
+				'total_pages' => ceil($total_items / $this->per_page),
 			)
 		);
 	}

includes/admin/reports/class-donor-reports-table.php

Doc Comments +1 added lines, -1 removed lines

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

Indentation +2 added lines, -2 removed lines

@@ -185,8 +185,8 @@
 				
 			case 'num_donations' :
 				$value = '<a href="' .
-				         admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&user=' . urlencode( $item['email'] )
-				         ) . '">' . esc_html( $item['num_donations'] ) . '</a>';
+						 admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&user=' . urlencode( $item['email'] )
+						 ) . '">' . esc_html( $item['num_donations'] ) . '</a>';
 				break;
 
 			case 'amount_spent' :

Spacing +68 added lines, -68 removed lines

@@ -10,13 +10,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -62,11 +62,11 @@ 
 		global $status, $page;
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => esc_html__( 'Donor', 'give' ),     // Singular name of the listed records
-			'plural'   => esc_html__( 'Donors', 'give' ),    // Plural name of the listed records
-			'ajax'     => false,// Does this table support ajax?
-		) );
+		parent::__construct(array(
+			'singular' => esc_html__('Donor', 'give'), // Singular name of the listed records
+			'plural'   => esc_html__('Donors', 'give'), // Plural name of the listed records
+			'ajax'     => false, // Does this table support ajax?
+		));
 
 	}
 
@@ -83,7 +83,7 @@ 
 	 *
 	 * @return false
 	 */
-	public function search_box( $text, $input_id ) {
+	public function search_box($text, $input_id) {
 		return false;
 	}
 
@@ -98,20 +98,20 @@ 
 	 *
 	 * @return void
 	 */
-	public function give_search_box( $text, $input_id ) {
-		$input_id = $input_id . '-search-input';
+	public function give_search_box($text, $input_id) {
+		$input_id = $input_id.'-search-input';
 
-		if ( ! empty( $_REQUEST['orderby'] ) ) {
-			echo '<input type="hidden" name="orderby" value="' . esc_attr( $_REQUEST['orderby'] ) . '" />';
+		if ( ! empty($_REQUEST['orderby'])) {
+			echo '<input type="hidden" name="orderby" value="'.esc_attr($_REQUEST['orderby']).'" />';
 		}
-		if ( ! empty( $_REQUEST['order'] ) ) {
-			echo '<input type="hidden" name="order" value="' . esc_attr( $_REQUEST['order'] ) . '" />';
+		if ( ! empty($_REQUEST['order'])) {
+			echo '<input type="hidden" name="order" value="'.esc_attr($_REQUEST['order']).'" />';
 		}
 		?>
 		<p class="search-box donor-search" role="search">
 			<label class="screen-reader-text" for="<?php echo $input_id ?>"><?php echo $text; ?>:</label>
 			<input type="search" id="<?php echo $input_id ?>" name="s" value="<?php _admin_search_query(); ?>" />
-			<?php submit_button( $text, 'button', false, false, array( 'ID' => 'search-submit' ) ); ?>
+			<?php submit_button($text, 'button', false, false, array('ID' => 'search-submit')); ?>
 		</p>
 	<?php
 	}
@@ -124,33 +124,33 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
+	protected function display_tablenav($which) {
 
-		if ( 'top' === $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+		if ('top' === $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav give-clearfix <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav give-clearfix <?php echo esc_attr($which); ?>">
 
-			<?php if ( 'top' === $which ) { ?>
+			<?php if ('top' === $which) { ?>
 				<h3 class="alignleft reports-earnings-title">
-					<span><?php esc_html_e( 'Donors Report', 'give' ); ?></span>
+					<span><?php esc_html_e('Donors Report', 'give'); ?></span>
 				</h3>
 			<?php } ?>
 
 			<div class="alignright tablenav-right">
 				<div class="actions bulkactions">
 					<?php
-					if ( 'top' === $which ) {
-						$this->give_search_box( esc_html__( 'Search Donors', 'give' ), 'give-donors-report-search' );
+					if ('top' === $which) {
+						$this->give_search_box(esc_html__('Search Donors', 'give'), 'give-donors-report-search');
 					}
 
-					$this->bulk_actions( $which ); ?>
+					$this->bulk_actions($which); ?>
 
 				</div>
 				<?php
-				$this->extra_tablenav( $which );
-				$this->pagination( $which );
+				$this->extra_tablenav($which);
+				$this->pagination($which);
 				?>
 			</div>
 
@@ -172,33 +172,33 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
+	public function column_default($item, $column_name) {
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 
 			case 'name' :
-				$name = '#' . $item['id'] . ' ';
-				$name .= ! empty( $item['name'] ) ? $item['name'] : '<em>' . esc_html__( 'Unnamed Donor', 'give' ) . '</em>';
-				$view_url = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $item['id'] );
-				$value    = '<a href="' . esc_url( $view_url ) . '">' . $name . '</a>';
+				$name = '#'.$item['id'].' ';
+				$name .= ! empty($item['name']) ? $item['name'] : '<em>'.esc_html__('Unnamed Donor', 'give').'</em>';
+				$view_url = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$item['id']);
+				$value    = '<a href="'.esc_url($view_url).'">'.$name.'</a>';
 				break;
 
 			case 'num_donations' :
-				$value = '<a href="' .
-				         admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&user=' . urlencode( $item['email'] )
-				         ) . '">' . esc_html( $item['num_donations'] ) . '</a>';
+				$value = '<a href="'.
+				         admin_url('edit.php?post_type=give_forms&page=give-payment-history&user='.urlencode($item['email'])
+				         ).'">'.esc_html($item['num_donations']).'</a>';
 				break;
 
 			case 'amount_spent' :
-				$value = give_currency_filter( give_format_amount( $item[ $column_name ], array( 'sanitize' => false ) ) );
+				$value = give_currency_filter(give_format_amount($item[$column_name], array('sanitize' => false)));
 				break;
 
 			default:
-				$value = isset( $item[ $column_name ] ) ? $item[ $column_name ] : null;
+				$value = isset($item[$column_name]) ? $item[$column_name] : null;
 				break;
 		}
 
-		return apply_filters( "give_report_column_{$column_name}", $value, $item['id'] );
+		return apply_filters("give_report_column_{$column_name}", $value, $item['id']);
 	}
 
 	/**
@@ -210,13 +210,13 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'name'          => __( 'Name', 'give' ),
-			'email'         => __( 'Email', 'give' ),
-			'num_donations' => __( 'Donations', 'give' ),
-			'amount_spent'  => __( 'Total Donated', 'give' ),
+			'name'          => __('Name', 'give'),
+			'email'         => __('Email', 'give'),
+			'num_donations' => __('Donations', 'give'),
+			'amount_spent'  => __('Total Donated', 'give'),
 		);
 
-		return apply_filters( 'give_report_donor_columns', $columns );
+		return apply_filters('give_report_donor_columns', $columns);
 
 	}
 
@@ -229,10 +229,10 @@ 
 	 */
 	public function get_sortable_columns() {
 		return array(
-			'id'            => array( 'id', true ),
-			'name'          => array( 'name', true ),
-			'num_donations' => array( 'purchase_count', false ),
-			'amount_spent'  => array( 'purchase_value', false ),
+			'id'            => array('id', true),
+			'name'          => array('name', true),
+			'num_donations' => array('purchase_count', false),
+			'amount_spent'  => array('purchase_value', false),
 		);
 	}
 
@@ -243,7 +243,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	public function bulk_actions( $which = '' ) {
+	public function bulk_actions($which = '') {
 
 	}
 
@@ -255,7 +255,7 @@ 
 	 * @return int Current page number
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 	/**
@@ -266,7 +266,7 @@ 
 	 * @return mixed string If search is present, false otherwise
 	 */
 	public function get_search() {
-		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;
+		return ! empty($_GET['s']) ? urldecode(trim($_GET['s'])) : false;
 	}
 
 	/**
@@ -285,15 +285,15 @@ 
 
 		// Get donor query.
 		$args   = $this->get_donor_query();
-		$donors = Give()->donors->get_donors( $args );
+		$donors = Give()->donors->get_donors($args);
 
-		if ( $donors ) {
+		if ($donors) {
 
-			$this->count = count( $donors );
+			$this->count = count($donors);
 
-			foreach ( $donors as $donor ) {
+			foreach ($donors as $donor) {
 
-				$user_id = ! empty( $donor->user_id ) ? absint( $donor->user_id ) : 0;
+				$user_id = ! empty($donor->user_id) ? absint($donor->user_id) : 0;
 
 				$data[] = array(
 					'id'            => $donor->id,
@@ -321,9 +321,9 @@ 
 
 		$_donor_query['number'] = -1;
 		$_donor_query['offset'] = 0;
-		$donors = Give()->donors->get_donors( $_donor_query );
+		$donors = Give()->donors->get_donors($_donor_query);
 
-		return count( $donors );
+		return count($donors);
 	}
 
 	/**
@@ -335,10 +335,10 @@ 
 	 */
 	public function get_donor_query() {
 		$paged   = $this->get_paged();
-		$offset  = $this->per_page * ( $paged - 1 );
+		$offset  = $this->per_page * ($paged - 1);
 		$search  = $this->get_search();
-		$order   = isset( $_GET['order'] ) ? sanitize_text_field( $_GET['order'] ) : 'DESC';
-		$orderby = isset( $_GET['orderby'] ) ? sanitize_text_field( $_GET['orderby'] ) : 'id';
+		$order   = isset($_GET['order']) ? sanitize_text_field($_GET['order']) : 'DESC';
+		$orderby = isset($_GET['orderby']) ? sanitize_text_field($_GET['orderby']) : 'id';
 
 		$args = array(
 			'number'  => $this->per_page,
@@ -347,10 +347,10 @@ 
 			'orderby' => $orderby,
 		);
 
-		if( $search ) {
-			if ( is_email( $search ) ) {
+		if ($search) {
+			if (is_email($search)) {
 				$args['email'] = $search;
-			} elseif ( is_numeric( $search ) ) {
+			} elseif (is_numeric($search)) {
 				$args['id'] = $search;
 			} else {
 				$args['name'] = $search;
@@ -377,16 +377,16 @@ 
 		$hidden   = array(); // No hidden columns
 		$sortable = $this->get_sortable_columns();
 
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 
 		$this->items = $this->reports_data();
 
 		$this->total = $this->get_donor_count();
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 			'total_items' => $this->total,
 			'per_page'    => $this->per_page,
-			'total_pages' => ceil( $this->total / $this->per_page ),
-		) );
+			'total_pages' => ceil($this->total / $this->per_page),
+		));
 	}
 }

includes/admin/reports/class-gateways-report.php

Spacing +16 added lines, -16 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Gateways_Report' ) ) :
+if ( ! class_exists('Give_Gateways_Report')) :
 
 	/**
 	 * Give_Gateways_Report.
@@ -43,16 +43,16 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'gateways';
-			$this->label = esc_html__( 'Donation Methods', 'give' );
+			$this->label = esc_html__('Donation Methods', 'give');
 
-			add_filter( 'give-reports_tabs_array', array( $this, 'add_settings_page' ), 20 );
-			add_action( "give-reports_settings_{$this->id}_page", array( $this, 'output' ) );
-			add_action( 'give_admin_field_report_gateways', array( $this, 'render_report_gateways_field' ), 10, 2 );
+			add_filter('give-reports_tabs_array', array($this, 'add_settings_page'), 20);
+			add_action("give-reports_settings_{$this->id}_page", array($this, 'output'));
+			add_action('give_admin_field_report_gateways', array($this, 'render_report_gateways_field'), 10, 2);
 
 			// Do not use main form for this tab.
-			if( give_get_current_setting_tab() === $this->id ) {
-				add_action( 'give-reports_open_form', '__return_empty_string' );
-				add_action( 'give-reports_close_form', '__return_empty_string' );
+			if (give_get_current_setting_tab() === $this->id) {
+				add_action('give-reports_open_form', '__return_empty_string');
+				add_action('give-reports_close_form', '__return_empty_string');
 			}
 		}
 
@@ -63,8 +63,8 @@ 
 		 * @param  array $pages Lst of pages.
 		 * @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;
 		}
@@ -86,7 +86,7 @@ 
 			 * @param  array $settings
 			 */
 			$settings = apply_filters(
-				'give_get_settings_' . $this->id,
+				'give_get_settings_'.$this->id,
 				array(
 					array(
 						'id'   => 'give_reports_gateways',
@@ -95,7 +95,7 @@ 
 					),
 					array(
 						'id'   => 'gateways',
-						'name' => esc_html__( 'Gateways', 'give' ),
+						'name' => esc_html__('Gateways', 'give'),
 						'type' => 'report_gateways',
 					),
 					array(
@@ -119,7 +119,7 @@ 
 		public function output() {
 			$settings = $this->get_settings();
 
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
 		}
 
 		/**
@@ -131,8 +131,8 @@ 
 		 * @param $field
 		 * @param $option_value
 		 */
-		public function render_report_gateways_field( $field, $option_value ) {
-			do_action( 'give_reports_view_gateways');
+		public function render_report_gateways_field($field, $option_value) {
+			do_action('give_reports_view_gateways');
 		}
 	}