WordImpress / Give

includes/admin/donors/donors.php

Spacing +167 added lines, -167 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,9 +24,9 @@ 
  */
 function give_donors_page() {
 	$default_views  = give_donor_views();
-	$requested_view = isset( $_GET['view'] ) ? sanitize_text_field( $_GET['view'] ) : 'donors';
-	if ( array_key_exists( $requested_view, $default_views ) && function_exists( $default_views[ $requested_view ] ) ) {
-		give_render_donor_view( $requested_view, $default_views );
+	$requested_view = isset($_GET['view']) ? sanitize_text_field($_GET['view']) : 'donors';
+	if (array_key_exists($requested_view, $default_views) && function_exists($default_views[$requested_view])) {
+		give_render_donor_view($requested_view, $default_views);
 	} else {
 		give_donors_list();
 	}
@@ -42,7 +42,7 @@ 
 
 	$views = array();
 
-	return apply_filters( 'give_donor_views', $views );
+	return apply_filters('give_donor_views', $views);
 
 }
 
@@ -56,7 +56,7 @@ 
 
 	$tabs = array();
 
-	return apply_filters( 'give_donor_tabs', $tabs );
+	return apply_filters('give_donor_tabs', $tabs);
 
 }
 
@@ -67,7 +67,7 @@ 
  * @return void
  */
 function give_donors_list() {
-	include dirname( __FILE__ ) . '/class-donor-table.php';
+	include dirname(__FILE__).'/class-donor-table.php';
 
 	$donors_table = new Give_Donor_List_Table();
 	$donors_table->prepare_items();
@@ -80,11 +80,11 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_donors_table_top' );
+		do_action('give_donors_table_top');
 		?>
-		<form id="give-donors-filter" method="get" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors' ); ?>">
+		<form id="give-donors-filter" method="get" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors'); ?>">
 			<?php
-			$donors_table->search_box( __( 'Search Donors', 'give' ), 'give-donors' );
+			$donors_table->search_box(__('Search Donors', 'give'), 'give-donors');
 			$donors_table->display();
 			?>
 			<input type="hidden" name="post_type" value="give_forms" />
@@ -97,7 +97,7 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_donors_table_bottom' );
+		do_action('give_donors_table_bottom');
 		?>
 	</div>
 	<?php
@@ -113,27 +113,27 @@ 
  *
  * @return void
  */
-function give_render_donor_view( $view, $callbacks ) {
+function give_render_donor_view($view, $callbacks) {
 
 	$render = true;
 
-	$donor_view_role = apply_filters( 'give_view_donors_role', 'view_give_reports' );
+	$donor_view_role = apply_filters('give_view_donors_role', 'view_give_reports');
 
-	if ( ! current_user_can( $donor_view_role ) ) {
-		give_set_error( 'give-no-access', __( 'You are not permitted to view this data.', 'give' ) );
+	if ( ! current_user_can($donor_view_role)) {
+		give_set_error('give-no-access', __('You are not permitted to view this data.', 'give'));
 		$render = false;
 	}
 
-	if ( ! isset( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
-		give_set_error( 'give-invalid_donor', __( 'Invalid Donor ID.', 'give' ) );
+	if ( ! isset($_GET['id']) || ! is_numeric($_GET['id'])) {
+		give_set_error('give-invalid_donor', __('Invalid Donor ID.', 'give'));
 		$render = false;
 	}
 
 	$donor_id = (int) $_GET['id'];
-	$donor    = new Give_Donor( $donor_id );
+	$donor    = new Give_Donor($donor_id);
 
-	if ( empty( $donor->id ) ) {
-		give_set_error( 'give-invalid_donor', __( 'Invalid Donor ID.', 'give' ) );
+	if (empty($donor->id)) {
+		give_set_error('give-invalid_donor', __('Invalid Donor ID.', 'give'));
 		$render = false;
 	}
 
@@ -142,9 +142,9 @@ 
 
 	<div class='wrap'>
 
-		<?php if ( give_get_errors() ) : ?>
+		<?php if (give_get_errors()) : ?>
 			<div class="error settings-error">
-				<?php Give()->notices->render_frontend_notices( 0 ); ?>
+				<?php Give()->notices->render_frontend_notices(0); ?>
 			</div>
 		<?php endif; ?>
 		
@@ -152,32 +152,32 @@ 
 			<?php
 			printf(
 			/* translators: %s: donor number */
-				esc_html__( 'Donor %s', 'give' ),
+				esc_html__('Donor %s', 'give'),
 				$donor_id
 			);
 			?>
 		</h1>
 
-		<?php if ( $donor && $render ) : ?>
+		<?php if ($donor && $render) : ?>
 
 			<h2 class="nav-tab-wrapper">
 			<?php
-			foreach ( $donor_tabs as $key => $tab ) :
+			foreach ($donor_tabs as $key => $tab) :
 				$active = $key === $view ? true : false;
 				$class = $active ? 'nav-tab nav-tab-active' : 'nav-tab';
 				printf(
-					'<a href="%1$s" class="%2$s"><span class="dashicons %3$s"></span>%4$s</a>' . "\n",
-					esc_url( admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=' . $key . '&id=' . $donor->id ) ),
-					esc_attr( $class ),
-					sanitize_html_class( $tab['dashicon'] ),
-					esc_html( $tab['title'] )
+					'<a href="%1$s" class="%2$s"><span class="dashicons %3$s"></span>%4$s</a>'."\n",
+					esc_url(admin_url('edit.php?post_type=give_forms&page=give-donors&view='.$key.'&id='.$donor->id)),
+					esc_attr($class),
+					sanitize_html_class($tab['dashicon']),
+					esc_html($tab['title'])
 				);
 			endforeach;
 			?>
 			</h2>
 
 			<div id="give-donor-card-wrapper">
-				<?php $callbacks[ $view ]( $donor ) ?>
+				<?php $callbacks[$view]($donor) ?>
 			</div>
 
 		<?php endif; ?>
@@ -197,9 +197,9 @@ 
  *
  * @return void
  */
-function give_donor_view( $donor ) {
+function give_donor_view($donor) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
 	/**
 	 * Fires in donor profile screen, above the donor card.
@@ -208,32 +208,32 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_card_top', $donor );
+	do_action('give_donor_card_top', $donor);
 	?>
 
 	<div id="donor-summary" class="info-wrapper donor-section postbox">
 
-		<form id="edit-donor-info" method="post" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ); ?>">
+		<form id="edit-donor-info" method="post" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id); ?>">
 
 			<div class="donor-info">
 
 				<div class="donor-bio-header clearfix">
 
 					<div class="avatar-wrap left" id="donor-avatar">
-						<?php echo get_avatar( $donor->email ); ?>
+						<?php echo get_avatar($donor->email); ?>
 					</div>
 
 					<div id="donor-name-wrap" class="left">
 						<span class="donor-id">#<?php echo $donor->id; ?></span>
-						<span class="donor-name info-item edit-item"><input size="15" data-key="name" name="customerinfo[name]" type="text" value="<?php echo esc_attr( $donor->name ); ?>" placeholder="<?php _e( 'Donor Name', 'give' ); ?>" /></span>
+						<span class="donor-name info-item edit-item"><input size="15" data-key="name" name="customerinfo[name]" type="text" value="<?php echo esc_attr($donor->name); ?>" placeholder="<?php _e('Donor Name', 'give'); ?>" /></span>
 						<span class="donor-name info-item editable"><span data-key="name"><?php echo $donor->name; ?></span></span>
 					</div>
 					<p class="donor-since info-item">
-						<?php _e( 'Donor since', 'give' ); ?>
-						<?php echo date_i18n( give_date_format(), strtotime( $donor->date_created ) ) ?>
+						<?php _e('Donor since', 'give'); ?>
+						<?php echo date_i18n(give_date_format(), strtotime($donor->date_created)) ?>
 					</p>
-					<?php if ( current_user_can( $donor_edit_role ) ) : ?>
-						<a href="#" id="edit-donor" class="button info-item editable donor-edit-link"><?php _e( 'Edit Donor', 'give' ); ?></a>
+					<?php if (current_user_can($donor_edit_role)) : ?>
+						<a href="#" id="edit-donor" class="button info-item editable donor-edit-link"><?php _e('Edit Donor', 'give'); ?></a>
 					<?php endif; ?>
 				</div>
 				<!-- /donor-bio-header -->
@@ -243,7 +243,7 @@ 
 					<table class="widefat">
 						<tbody>
 						<tr class="alternate">
-							<th scope="col"><label for="tablecell"><?php _e( 'User:', 'give' ); ?></label></th>
+							<th scope="col"><label for="tablecell"><?php _e('User:', 'give'); ?></label></th>
 							<td>
 								<span class="donor-user-id info-item edit-item">
 									<?php
@@ -260,48 +260,48 @@ 
 										'data'  => $data_atts,
 									);
 
-									if ( ! empty( $user_id ) ) {
-										$userdata           = get_userdata( $user_id );
+									if ( ! empty($user_id)) {
+										$userdata = get_userdata($user_id);
 										$user_args['selected'] = $user_id;
 									}
 
-									echo Give()->html->ajax_user_search( $user_args );
+									echo Give()->html->ajax_user_search($user_args);
 									?>
 								</span>
 
 								<span class="donor-user-id info-item editable">
-									<?php if ( ! empty( $userdata ) ) { ?>
-										<span data-key="user_id">#<?php echo $donor->user_id . ' - ' . $userdata->display_name; ?></span>
+									<?php if ( ! empty($userdata)) { ?>
+										<span data-key="user_id">#<?php echo $donor->user_id.' - '.$userdata->display_name; ?></span>
 									<?php } else { ?>
-										<span data-key="user_id"><?php _e( 'None', 'give' ); ?></span>
+										<span data-key="user_id"><?php _e('None', 'give'); ?></span>
 									<?php } ?>
-									<?php if ( current_user_can( $donor_edit_role ) && intval( $donor->user_id ) > 0 ) { ?>
+									<?php if (current_user_can($donor_edit_role) && intval($donor->user_id) > 0) { ?>
 										<span class="disconnect-user">
  											-
- 											<a id="disconnect-donor" href="#disconnect" aria-label="<?php _e( 'Disconnects the current user ID from this donor record.', 'give' ); ?>">
- 												<?php _e( 'Disconnect User', 'give' ); ?>
+ 											<a id="disconnect-donor" href="#disconnect" aria-label="<?php _e('Disconnects the current user ID from this donor record.', 'give'); ?>">
+ 												<?php _e('Disconnect User', 'give'); ?>
 											</a>
  										</span>
  										<span class="view-user-profile">
  											|
- 											<a id="view-user-profile" href="<?php echo 'user-edit.php?user_id=' . $donor->user_id; ?>" aria-label="<?php _e( 'View User Profile of current user ID.', 'give' ); ?>">
- 												<?php _e( 'View User Profile', 'give' ); ?>
+ 											<a id="view-user-profile" href="<?php echo 'user-edit.php?user_id='.$donor->user_id; ?>" aria-label="<?php _e('View User Profile of current user ID.', 'give'); ?>">
+ 												<?php _e('View User Profile', 'give'); ?>
 											</a>
  										</span>
 									<?php } ?>
 								</span>
 							</td>
 						</tr>
-						<?php if ( isset( $donor->user_id ) && $donor->user_id > 0 ) : ?>
+						<?php if (isset($donor->user_id) && $donor->user_id > 0) : ?>
 
 							<tr>
-								<th scope="col"><?php _e( 'Address:', 'give' ); ?></th>
+								<th scope="col"><?php _e('Address:', 'give'); ?></th>
 								<td class="row-title">
 
 									<div class="donor-address-wrapper">
 
 										<?php
-										$address  = get_user_meta( $donor->user_id, '_give_user_address', true );
+										$address  = get_user_meta($donor->user_id, '_give_user_address', true);
 										$defaults = array(
 											'line1'   => '',
 											'line2'   => '',
@@ -311,10 +311,10 @@ 
 											'zip'     => '',
 										);
 
-										$address = wp_parse_args( $address, $defaults );
+										$address = wp_parse_args($address, $defaults);
 										?>
 
-										<?php if ( ! empty( $address ) ) { ?>
+										<?php if ( ! empty($address)) { ?>
 											<span class="donor-address info-item editable">
 												<span class="info-item" data-key="line1"><?php echo $address['line1']; ?></span>
 												<span class="info-item" data-key="line2"><?php echo $address['line2']; ?></span>
@@ -325,43 +325,43 @@ 
 											</span>
 										<?php } ?>
 										<span class="donor-address info-item edit-item">
-											<input class="info-item" type="text" data-key="line1" name="customerinfo[line1]" placeholder="<?php _e( 'Address 1', 'give' ); ?>" value="<?php echo $address['line1']; ?>" />
-											<input class="info-item" type="text" data-key="line2" name="customerinfo[line2]" placeholder="<?php _e( 'Address 2', 'give' ); ?>" value="<?php echo $address['line2']; ?>" />
-											<input class="info-item" type="text" data-key="city" name="customerinfo[city]" placeholder="<?php _e( 'City', 'give' ); ?>" value="<?php echo $address['city']; ?>" />
+											<input class="info-item" type="text" data-key="line1" name="customerinfo[line1]" placeholder="<?php _e('Address 1', 'give'); ?>" value="<?php echo $address['line1']; ?>" />
+											<input class="info-item" type="text" data-key="line2" name="customerinfo[line2]" placeholder="<?php _e('Address 2', 'give'); ?>" value="<?php echo $address['line2']; ?>" />
+											<input class="info-item" type="text" data-key="city" name="customerinfo[city]" placeholder="<?php _e('City', 'give'); ?>" value="<?php echo $address['city']; ?>" />
 											<select data-key="country" name="customerinfo[country]" id="billing_country" class="billing_country give-select edit-item">
 												<?php
 
 												$selected_country = $address['country'];
 
 												$countries = give_get_country_list();
-												foreach ( $countries as $country_code => $country ) {
-													echo '<option value="' . esc_attr( $country_code ) . '"' . selected( $country_code, $selected_country, false ) . '>' . $country . '</option>';
+												foreach ($countries as $country_code => $country) {
+													echo '<option value="'.esc_attr($country_code).'"'.selected($country_code, $selected_country, false).'>'.$country.'</option>';
 												}
 												?>
 											</select>
 											<?php
 											$selected_state = give_get_state();
-											$states         = give_get_states( $selected_country );
+											$states         = give_get_states($selected_country);
 
-											$selected_state = isset( $address['state'] ) ? $address['state'] : $selected_state;
+											$selected_state = isset($address['state']) ? $address['state'] : $selected_state;
 
-											if ( ! empty( $states ) ) {
+											if ( ! empty($states)) {
 												?>
 												<select data-key="state" name="customerinfo[state]" id="card_state" class="card_state give-select info-item">
 													<?php
-													foreach ( $states as $state_code => $state ) {
-														echo '<option value="' . $state_code . '"' . selected( $state_code, $selected_state, false ) . '>' . $state . '</option>';
+													foreach ($states as $state_code => $state) {
+														echo '<option value="'.$state_code.'"'.selected($state_code, $selected_state, false).'>'.$state.'</option>';
 													}
 													?>
 												</select>
 												<?php
 											} else {
 												?>
-												<input type="text" size="6" data-key="state" name="customerinfo[state]" id="card_state" class="card_state give-input info-item" placeholder="<?php _e( 'State / Province / County', 'give' ); ?>" />
+												<input type="text" size="6" data-key="state" name="customerinfo[state]" id="card_state" class="card_state give-input info-item" placeholder="<?php _e('State / Province / County', 'give'); ?>" />
 												<?php
 											}
 											?>
-											<input class="info-item" type="text" data-key="zip" name="customerinfo[zip]" placeholder="<?php _e( 'Zip / Postal Code', 'give' ); ?>" value="<?php echo $address['zip']; ?>" />
+											<input class="info-item" type="text" data-key="zip" name="customerinfo[zip]" placeholder="<?php _e('Zip / Postal Code', 'give'); ?>" value="<?php echo $address['zip']; ?>" />
 										</span>
 
 									</div>
@@ -377,10 +377,10 @@ 
 
 			<span id="donor-edit-actions" class="edit-item">
 				<input type="hidden" data-key="id" name="customerinfo[id]" value="<?php echo $donor->id; ?>" />
-				<?php wp_nonce_field( 'edit-donor', '_wpnonce', false, true ); ?>
+				<?php wp_nonce_field('edit-donor', '_wpnonce', false, true); ?>
 				<input type="hidden" name="give_action" value="edit-donor" />
-				<input type="submit" id="give-edit-donor-save" class="button-secondary" value="<?php _e( 'Update Donor', 'give' ); ?>" />
-				<a id="give-edit-donor-cancel" href="" class="delete"><?php _e( 'Cancel', 'give' ); ?></a>
+				<input type="submit" id="give-edit-donor-save" class="button-secondary" value="<?php _e('Update Donor', 'give'); ?>" />
+				<a id="give-edit-donor-cancel" href="" class="delete"><?php _e('Cancel', 'give'); ?></a>
 			</span>
 
 		</form>
@@ -395,24 +395,24 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_before_stats', $donor );
+	do_action('give_donor_before_stats', $donor);
 	?>
 
 	<div id="donor-stats-wrapper" class="donor-section postbox clear">
 		<ul>
 			<li>
-				<a href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&donor=' . absint( $donor->id ) ); ?>">
+				<a href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-payment-history&donor='.absint($donor->id)); ?>">
 					<span class="dashicons dashicons-heart"></span>
 					<?php
 					// Completed Donations
-					$completed_donations_text = sprintf( _n( '%d Completed Donation', '%d Completed Donations', $donor->purchase_count, 'give' ), $donor->purchase_count );
-					echo apply_filters( 'give_donor_completed_donations', $completed_donations_text, $donor );
+					$completed_donations_text = sprintf(_n('%d Completed Donation', '%d Completed Donations', $donor->purchase_count, 'give'), $donor->purchase_count);
+					echo apply_filters('give_donor_completed_donations', $completed_donations_text, $donor);
 					?>
 				</a>
 			</li>
 			<li>
 				<span class="dashicons dashicons-chart-area"></span>
-				<?php echo give_currency_filter( give_format_amount( $donor->purchase_value, array( 'sanitize' => false ) ) ); ?> <?php _e( 'Lifetime Donations', 'give' ); ?>
+				<?php echo give_currency_filter(give_format_amount($donor->purchase_value, array('sanitize' => false))); ?> <?php _e('Lifetime Donations', 'give'); ?>
 			</li>
 			<?php
 			/**
@@ -424,7 +424,7 @@ 
 			 *
 			 * @param object $donor The donor object being displayed.
 			 */
-			do_action( 'give_donor_stats_list', $donor );
+			do_action('give_donor_stats_list', $donor);
 			?>
 		</ul>
 	</div>
@@ -437,7 +437,7 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_before_tables_wrapper', $donor );
+	do_action('give_donor_before_tables_wrapper', $donor);
 	?>
 
 	<div id="donor-tables-wrapper" class="donor-section">
@@ -450,46 +450,46 @@ 
 		 *
 		 * @param object $donor The donor object being displayed.
 		 */
-		do_action( 'give_donor_before_tables', $donor );
+		do_action('give_donor_before_tables', $donor);
 		?>
 
-		<h3><?php _e( 'Donor Emails', 'give' ); ?></h3>
+		<h3><?php _e('Donor Emails', 'give'); ?></h3>
 
 		<table class="wp-list-table widefat striped emails">
 			<thead>
 				<tr>
-					<th><?php _e( 'Email', 'give' ); ?></th>
-					<th><?php _e( 'Actions', 'give' ); ?></th>
+					<th><?php _e('Email', 'give'); ?></th>
+					<th><?php _e('Actions', 'give'); ?></th>
 				</tr>
 			</thead>
 
 			<tbody>
-				<?php if ( ! empty( $donor->emails ) ) { ?>
+				<?php if ( ! empty($donor->emails)) { ?>
 
-					<?php foreach ( $donor->emails as $key => $email ) : ?>
+					<?php foreach ($donor->emails as $key => $email) : ?>
 						<tr data-key="<?php echo $key; ?>">
 							<td>
 								<?php echo $email; ?>
-								<?php if ( 'primary' === $key ) : ?>
+								<?php if ('primary' === $key) : ?>
 									<span class="dashicons dashicons-star-filled primary-email-icon"></span>
 								<?php endif; ?>
 							</td>
 							<td>
-								<?php if ( 'primary' !== $key ) : ?>
+								<?php if ('primary' !== $key) : ?>
 									<?php
-									$base_url    = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id );
-									$promote_url = wp_nonce_url( add_query_arg( array(
-										'email' => rawurlencode( $email ),
+									$base_url    = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id);
+									$promote_url = wp_nonce_url(add_query_arg(array(
+										'email' => rawurlencode($email),
 										'give_action' => 'set_donor_primary_email',
-									), $base_url ), 'give-set-donor-primary-email' );
-									$remove_url  = wp_nonce_url( add_query_arg( array(
-										'email' => rawurlencode( $email ),
+									), $base_url), 'give-set-donor-primary-email');
+									$remove_url = wp_nonce_url(add_query_arg(array(
+										'email' => rawurlencode($email),
 										'give_action' => 'remove_donor_email',
-									), $base_url ), 'give-remove-donor-email' );
+									), $base_url), 'give-remove-donor-email');
 									?>
-									<a href="<?php echo $promote_url; ?>"><?php _e( 'Make Primary', 'give' ); ?></a>
+									<a href="<?php echo $promote_url; ?>"><?php _e('Make Primary', 'give'); ?></a>
 									&nbsp;|&nbsp;
-									<a href="<?php echo $remove_url; ?>" class="delete"><?php _e( 'Remove', 'give' ); ?></a>
+									<a href="<?php echo $remove_url; ?>" class="delete"><?php _e('Remove', 'give'); ?></a>
 								<?php endif; ?>
 							</td>
 						</tr>
@@ -499,59 +499,59 @@ 
 						<td colspan="2" class="add-donor-email-td">
 							<div class="add-donor-email-wrapper">
 								<input type="hidden" name="donor-id" value="<?php echo $donor->id; ?>" />
-								<?php wp_nonce_field( 'give_add_donor_email', 'add_email_nonce', false, true ); ?>
-								<input type="email" name="additional-email" value="" placeholder="<?php _e( 'Email Address', 'give' ); ?>" />&nbsp;
-								<input type="checkbox" name="make-additional-primary" value="1" id="make-additional-primary" />&nbsp;<label for="make-additional-primary"><?php _e( 'Make Primary', 'give' ); ?></label>
-								<button class="button-secondary give-add-donor-email" id="add-donor-email"><?php _e( 'Add Email', 'give' ); ?></button>
+								<?php wp_nonce_field('give_add_donor_email', 'add_email_nonce', false, true); ?>
+								<input type="email" name="additional-email" value="" placeholder="<?php _e('Email Address', 'give'); ?>" />&nbsp;
+								<input type="checkbox" name="make-additional-primary" value="1" id="make-additional-primary" />&nbsp;<label for="make-additional-primary"><?php _e('Make Primary', 'give'); ?></label>
+								<button class="button-secondary give-add-donor-email" id="add-donor-email"><?php _e('Add Email', 'give'); ?></button>
 								<span class="spinner"></span>
 							</div>
 							<div class="notice-wrap"></div>
 						</td>
 					</tr>
 				<?php } else { ?>
-					<tr><td colspan="2"><?php _e( 'No Emails Found', 'give' ); ?></td></tr>
+					<tr><td colspan="2"><?php _e('No Emails Found', 'give'); ?></td></tr>
 				<?php }// End if().
 	?>
 			</tbody>
 		</table>
 
-		<h3><?php _e( 'Recent Donations', 'give' ); ?></h3>
+		<h3><?php _e('Recent Donations', 'give'); ?></h3>
 		<?php
-		$payment_ids = explode( ',', $donor->payment_ids );
-		$payments    = give_get_payments( array(
+		$payment_ids = explode(',', $donor->payment_ids);
+		$payments    = give_get_payments(array(
 			'post__in' => $payment_ids,
-		) );
-		$payments    = array_slice( $payments, 0, 10 );
+		));
+		$payments    = array_slice($payments, 0, 10);
 		?>
 		<table class="wp-list-table widefat striped payments">
 			<thead>
 			<tr>
-				<th scope="col"><?php _e( 'ID', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Amount', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Date', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Status', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Actions', 'give' ); ?></th>
+				<th scope="col"><?php _e('ID', 'give'); ?></th>
+				<th scope="col"><?php _e('Amount', 'give'); ?></th>
+				<th scope="col"><?php _e('Date', 'give'); ?></th>
+				<th scope="col"><?php _e('Status', 'give'); ?></th>
+				<th scope="col"><?php _e('Actions', 'give'); ?></th>
 			</tr>
 			</thead>
 			<tbody>
-			<?php if ( ! empty( $payments ) ) { ?>
-				<?php foreach ( $payments as $payment ) : ?>
+			<?php if ( ! empty($payments)) { ?>
+				<?php foreach ($payments as $payment) : ?>
 					<tr>
 						<td><?php echo $payment->ID; ?></td>
-						<td><?php echo give_payment_amount( $payment->ID ); ?></td>
-						<td><?php echo date_i18n( give_date_format(), strtotime( $payment->post_date ) ); ?></td>
-						<td><?php echo give_get_payment_status( $payment, true ); ?></td>
+						<td><?php echo give_payment_amount($payment->ID); ?></td>
+						<td><?php echo date_i18n(give_date_format(), strtotime($payment->post_date)); ?></td>
+						<td><?php echo give_get_payment_status($payment, true); ?></td>
 						<td>
 							<?php
 							printf(
 								'<a href="%1$s" aria-label="%2$s">%3$s</a>',
-								admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id=' . $payment->ID ),
+								admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id='.$payment->ID),
 								sprintf(
 									/* translators: %s: Donation ID */
-									esc_attr__( 'View Donation %s.', 'give' ),
+									esc_attr__('View Donation %s.', 'give'),
 									$payment->ID
 								),
-								__( 'View Donation', 'give' )
+								__('View Donation', 'give')
 							);
 							?>
 
@@ -566,47 +566,47 @@ 
 							 * @param object $donor The donor object being displayed.
 							 * @param object $payment  The payment object being displayed.
 							 */
-							do_action( 'give_donor_recent_purchases_actions', $donor, $payment );
+							do_action('give_donor_recent_purchases_actions', $donor, $payment);
 							?>
 						</td>
 					</tr>
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<tr>
-					<td colspan="5"><?php _e( 'No donations found.', 'give' ); ?></td>
+					<td colspan="5"><?php _e('No donations found.', 'give'); ?></td>
 				</tr>
 			<?php }// End if().
 	?>
 			</tbody>
 		</table>
 
-		<h3><?php _e( 'Completed Forms', 'give' ); ?></h3>
+		<h3><?php _e('Completed Forms', 'give'); ?></h3>
 		<?php
-		$donations = give_get_users_completed_donations( $donor->email );
+		$donations = give_get_users_completed_donations($donor->email);
 		?>
 		<table class="wp-list-table widefat striped donations">
 			<thead>
 			<tr>
-				<th scope="col"><?php _e( 'Form', 'give' ); ?></th>
-				<th scope="col" width="120px"><?php _e( 'Actions', 'give' ); ?></th>
+				<th scope="col"><?php _e('Form', 'give'); ?></th>
+				<th scope="col" width="120px"><?php _e('Actions', 'give'); ?></th>
 			</tr>
 			</thead>
 			<tbody>
-			<?php if ( ! empty( $donations ) ) { ?>
-				<?php foreach ( $donations as $donation ) : ?>
+			<?php if ( ! empty($donations)) { ?>
+				<?php foreach ($donations as $donation) : ?>
 					<tr>
 						<td><?php echo $donation->post_title; ?></td>
 						<td>
 							<?php
 							printf(
 								'<a href="%1$s" aria-label="%2$s">%3$s</a>',
-								esc_url( admin_url( 'post.php?action=edit&post=' . $donation->ID ) ),
+								esc_url(admin_url('post.php?action=edit&post='.$donation->ID)),
 								sprintf(
 									/* translators: %s: form name */
-									esc_attr__( 'View Form %s.', 'give' ),
+									esc_attr__('View Form %s.', 'give'),
 									$donation->post_title
 								),
-								__( 'View Form', 'give' )
+								__('View Form', 'give')
 							);
 							?>
 						</td>
@@ -614,7 +614,7 @@ 
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<tr>
-					<td colspan="2"><?php _e( 'No completed donations found.', 'give' ); ?></td>
+					<td colspan="2"><?php _e('No completed donations found.', 'give'); ?></td>
 				</tr>
 			<?php } ?>
 			</tbody>
@@ -628,7 +628,7 @@ 
 		 *
 		 * @param object $donor The donor object being displayed.
 		 */
-		do_action( 'give_donor_after_tables', $donor );
+		do_action('give_donor_after_tables', $donor);
 		?>
 
 	</div>
@@ -641,7 +641,7 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_card_bottom', $donor );
+	do_action('give_donor_card_bottom', $donor);
 
 }
 
@@ -654,30 +654,30 @@ 
  *
  * @return void
  */
-function give_donor_notes_view( $donor ) {
+function give_donor_notes_view($donor) {
 
-	$paged          = isset( $_GET['paged'] ) && is_numeric( $_GET['paged'] ) ? $_GET['paged'] : 1;
-	$paged          = absint( $paged );
+	$paged          = isset($_GET['paged']) && is_numeric($_GET['paged']) ? $_GET['paged'] : 1;
+	$paged          = absint($paged);
 	$note_count     = $donor->get_notes_count();
-	$per_page       = apply_filters( 'give_donor_notes_per_page', 20 );
-	$total_pages    = ceil( $note_count / $per_page );
-	$donor_notes = $donor->get_notes( $per_page, $paged );
+	$per_page       = apply_filters('give_donor_notes_per_page', 20);
+	$total_pages    = ceil($note_count / $per_page);
+	$donor_notes = $donor->get_notes($per_page, $paged);
 	?>
 
 	<div id="donor-notes-wrapper">
 		<div class="donor-notes-header">
-			<?php echo get_avatar( $donor->email, 30 ); ?> <span><?php echo $donor->name; ?></span>
+			<?php echo get_avatar($donor->email, 30); ?> <span><?php echo $donor->name; ?></span>
 		</div>
-		<h3><?php _e( 'Notes', 'give' ); ?></h3>
+		<h3><?php _e('Notes', 'give'); ?></h3>
 
-		<?php if ( 1 == $paged ) : ?>
+		<?php if (1 == $paged) : ?>
 			<div style="display: block; margin-bottom: 55px;">
-				<form id="give-add-donor-note" method="post" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=notes&id=' . $donor->id ); ?>">
+				<form id="give-add-donor-note" method="post" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=notes&id='.$donor->id); ?>">
 					<textarea id="donor-note" name="donor_note" class="donor-note-input" rows="10"></textarea>
 					<br />
 					<input type="hidden" id="donor-id" name="customer_id" value="<?php echo $donor->id; ?>" />
 					<input type="hidden" name="give_action" value="add-donor-note" />
-					<?php wp_nonce_field( 'add-donor-note', 'add_donor_note_nonce', true, true ); ?>
+					<?php wp_nonce_field('add-donor-note', 'add_donor_note_nonce', true, true); ?>
 					<input id="add-donor-note" class="right button-primary" type="submit" value="Add Note" />
 				</form>
 			</div>
@@ -692,26 +692,26 @@ 
 			'show_all' => true,
 		);
 
-		echo paginate_links( $pagination_args );
+		echo paginate_links($pagination_args);
 		?>
 
 		<div id="give-donor-notes" class="postbox">
-			<?php if ( count( $donor_notes ) > 0 ) { ?>
-				<?php foreach ( $donor_notes as $key => $note ) : ?>
+			<?php if (count($donor_notes) > 0) { ?>
+				<?php foreach ($donor_notes as $key => $note) : ?>
 					<div class="donor-note-wrapper dashboard-comment-wrap comment-item">
 					<span class="note-content-wrap">
-						<?php echo stripslashes( $note ); ?>
+						<?php echo stripslashes($note); ?>
 					</span>
 					</div>
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<div class="give-no-donor-notes">
-					<?php _e( 'No donor notes found.', 'give' ); ?>
+					<?php _e('No donor notes found.', 'give'); ?>
 				</div>
 			<?php } ?>
 		</div>
 
-		<?php echo paginate_links( $pagination_args ); ?>
+		<?php echo paginate_links($pagination_args); ?>
 
 	</div>
 
@@ -727,9 +727,9 @@ 
  *
  * @return void
  */
-function give_donor_delete_view( $donor ) {
+function give_donor_delete_view($donor) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
 	/**
 	 * Fires in donor delete screen, above the content.
@@ -738,15 +738,15 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_delete_top', $donor );
+	do_action('give_donor_delete_top', $donor);
 	?>
 
 	<div class="info-wrapper donor-section">
 
-		<form id="delete-donor" method="post" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=delete&id=' . $donor->id ); ?>">
+		<form id="delete-donor" method="post" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=delete&id='.$donor->id); ?>">
 
 			<div class="donor-notes-header">
-				<?php echo get_avatar( $donor->email, 30 ); ?> <span><?php echo $donor->name; ?></span>
+				<?php echo get_avatar($donor->email, 30); ?> <span><?php echo $donor->name; ?></span>
 			</div>
 
 
@@ -754,20 +754,20 @@ 
 
 				<span class="delete-donor-options">
 					<p>
-						<?php echo Give()->html->checkbox( array(
+						<?php echo Give()->html->checkbox(array(
 							'name' => 'give-donor-delete-confirm',
-						) ); ?>
-						<label for="give-donor-delete-confirm"><?php _e( 'Are you sure you want to delete this donor?', 'give' ); ?></label>
+						)); ?>
+						<label for="give-donor-delete-confirm"><?php _e('Are you sure you want to delete this donor?', 'give'); ?></label>
 					</p>
 
 					<p>
-						<?php echo Give()->html->checkbox( array(
+						<?php echo Give()->html->checkbox(array(
 							'name'    => 'give-donor-delete-records',
 							'options' => array(
 								'disabled' => true,
 							),
-						) ); ?>
-						<label for="give-donor-delete-records"><?php _e( 'Delete all associated donations and records?', 'give' ); ?></label>
+						)); ?>
+						<label for="give-donor-delete-records"><?php _e('Delete all associated donations and records?', 'give'); ?></label>
 					</p>
 
 					<?php
@@ -780,16 +780,16 @@ 
 					 *
 					 * @param object $donor The donor object being displayed.
 					 */
-					do_action( 'give_donor_delete_inputs', $donor );
+					do_action('give_donor_delete_inputs', $donor);
 					?>
 				</span>
 
 				<span id="donor-edit-actions">
 					<input type="hidden" name="customer_id" value="<?php echo $donor->id; ?>" />
-					<?php wp_nonce_field( 'delete-donor', '_wpnonce', false, true ); ?>
+					<?php wp_nonce_field('delete-donor', '_wpnonce', false, true); ?>
 					<input type="hidden" name="give_action" value="delete-donor" />
-					<input type="submit" disabled="disabled" id="give-delete-donor" class="button-primary" value="<?php _e( 'Delete Donor', 'give' ); ?>" />
-					<a id="give-delete-donor-cancel" href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ); ?>" class="delete"><?php _e( 'Cancel', 'give' ); ?></a>
+					<input type="submit" disabled="disabled" id="give-delete-donor" class="button-primary" value="<?php _e('Delete Donor', 'give'); ?>" />
+					<a id="give-delete-donor-cancel" href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id); ?>" class="delete"><?php _e('Cancel', 'give'); ?></a>
 				</span>
 
 			</div>
@@ -805,5 +805,5 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_delete_bottom', $donor );
+	do_action('give_donor_delete_bottom', $donor);
 }

includes/admin/views/html-admin-settings.php

Spacing +17 added lines, -17 removed lines

@@ -2,13 +2,13 @@ 
 /**
  * Admin View: Settings
  */
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 
 // Bailout: Do not output anything if setting tab is not defined.
-if( ! empty( $tabs ) && array_key_exists( give_get_current_setting_tab(), $tabs ) ) :
+if ( ! empty($tabs) && array_key_exists(give_get_current_setting_tab(), $tabs)) :
 	/**
 	 * Filter the form action.
 	 *
@@ -18,7 +18,7 @@ 
 	 *
 	 * @since 1.8
 	 */
-	$form_method    = apply_filters( self::$setting_filter_prefix . '_form_method_tab_' . $current_tab, 'post' );
+	$form_method = apply_filters(self::$setting_filter_prefix.'_form_method_tab_'.$current_tab, 'post');
 
 	/**
 	 * Filter the main form tab.
@@ -30,20 +30,20 @@ 
 	 *
 	 * @since 1.8
 	 */
-	$form_open_tag  = apply_filters( self::$setting_filter_prefix . '_open_form', '<form method="' . $form_method  . '" id="give-mainform" action="" enctype="multipart/form-data">' );
-	$form_close_tag = apply_filters( self::$setting_filter_prefix . '_close_form', '</form>' );
+	$form_open_tag  = apply_filters(self::$setting_filter_prefix.'_open_form', '<form method="'.$form_method.'" id="give-mainform" action="" enctype="multipart/form-data">');
+	$form_close_tag = apply_filters(self::$setting_filter_prefix.'_close_form', '</form>');
 	?>
-	<div class="wrap give-settings-page <?php echo self::$setting_filter_prefix . '-setting-page'; ?>">
+	<div class="wrap give-settings-page <?php echo self::$setting_filter_prefix.'-setting-page'; ?>">
 		<?php echo $form_open_tag; ?>
-		<h1 class="wp-heading-inline"><?php echo esc_html( $tabs[ $current_tab ] ); ?></h1>
+		<h1 class="wp-heading-inline"><?php echo esc_html($tabs[$current_tab]); ?></h1>
 		<?php
 		// Show messages.
 		self::show_messages();
 		?>
 		<div class="nav-tab-wrapper give-nav-tab-wrapper">
 			<?php
-			foreach ( $tabs as $name => $label ) {
-				echo '<a href="' . admin_url( "edit.php?post_type=give_forms&page=" . self::$setting_filter_prefix . "&tab={$name}" ) . '" class="nav-tab ' . ( $current_tab == $name ? 'nav-tab-active' : '' ) . '">' . $label . '</a>';
+			foreach ($tabs as $name => $label) {
+				echo '<a href="'.admin_url("edit.php?post_type=give_forms&page=".self::$setting_filter_prefix."&tab={$name}").'" class="nav-tab '.($current_tab == $name ? 'nav-tab-active' : '').'">'.$label.'</a>';
 			}
 
 			/**
@@ -55,11 +55,11 @@ 
 			 *
 			 * @since 1.8
 			 */
-			do_action( self::$setting_filter_prefix . '_tabs' );
+			do_action(self::$setting_filter_prefix.'_tabs');
 			?>
 		</div>
 		<div class="give-sub-nav-tab-wrapper">
-			<a href="#" id="give-show-sub-nav" class="nav-tab give-not-tab" title="<?php _e( 'View remaining setting tabs', 'give' ); ?>"><span class="dashicons dashicons-arrow-down-alt2"></span></span></a>
+			<a href="#" id="give-show-sub-nav" class="nav-tab give-not-tab" title="<?php _e('View remaining setting tabs', 'give'); ?>"><span class="dashicons dashicons-arrow-down-alt2"></span></span></a>
 			<nav class="give-sub-nav-tab give-hidden"></nav>
 		</div>
 		<?php
@@ -73,7 +73,7 @@ 
 		 *
 		 * @since 1.8
 		 */
-		do_action( self::$setting_filter_prefix . "_sections_{$current_tab}_page" );
+		do_action(self::$setting_filter_prefix."_sections_{$current_tab}_page");
 
 		/**
 		 * Trigger Action.
@@ -84,15 +84,15 @@ 
 		 *
 		 * @since 1.8
 		 */
-		do_action( self::$setting_filter_prefix . "_settings_{$current_tab}_page" );
+		do_action(self::$setting_filter_prefix."_settings_{$current_tab}_page");
 
-		if ( empty( $GLOBALS['give_hide_save_button'] ) ) : ?>
+		if (empty($GLOBALS['give_hide_save_button'])) : ?>
 			<div class="give-submit-wrap">
-				<?php wp_nonce_field( 'give-save-settings', '_give-save-settings' ); ?>
-				<input name="save" class="button-primary give-save-button" type="submit" value="<?php esc_attr_e( 'Save changes', 'give' ); ?>" />
+				<?php wp_nonce_field('give-save-settings', '_give-save-settings'); ?>
+				<input name="save" class="button-primary give-save-button" type="submit" value="<?php esc_attr_e('Save changes', 'give'); ?>" />
 			</div>
 		<?php endif; ?>
 		<?php echo $form_close_tag; ?>
 	</div>
-<?php else : echo '<div class="error"><p>' . __( 'Oops, this settings page does not exist.', 'give' ) . '</p></div>'; ?>
+<?php else : echo '<div class="error"><p>'.__('Oops, this settings page does not exist.', 'give').'</p></div>'; ?>
 <?php endif; ?>
\ No newline at end of file

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

Spacing +67 added lines, -67 removed lines

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

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

Spacing +58 added lines, -58 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';
 }
 
 /**
@@ -56,13 +56,13 @@ 
 		global $status, $page;
 
 		// 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?
-		) );
+		));
 
-		add_action( 'give_report_view_actions', array( $this, 'category_filter' ) );
+		add_action('give_report_view_actions', array($this, 'category_filter'));
 		$this->query();
 
 	}
@@ -78,32 +78,32 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
-		switch ( $column_name ) {
+	public function column_default($item, $column_name) {
+		switch ($column_name) {
 			case 'title':
-				$title = empty( $item['title'] ) ? sprintf( __( 'Untitled (#%s)', 'give' ), $item['ID'] ) : $item['title'];
+				$title = empty($item['title']) ? sprintf(__('Untitled (#%s)', 'give'), $item['ID']) : $item['title'];
 
 				return sprintf(
 					'<a href="%s">%s</a>',
-					get_edit_post_link( $item['ID'] ),
+					get_edit_post_link($item['ID']),
 					$title
 				);
 			case 'sales':
 				return sprintf(
 					'<a href="%s">%s</a>',
-					admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&form_id=' . urlencode( $item['ID'] ) ),
+					admin_url('edit.php?post_type=give_forms&page=give-payment-history&form_id='.urlencode($item['ID'])),
 					$item['sales']
 				);
 			case 'earnings' :
-				return give_currency_filter( give_format_amount( $item[ $column_name ], array( 'sanitize' => false ) ) );
+				return give_currency_filter(give_format_amount($item[$column_name], array('sanitize' => false)));
 			case 'average_sales' :
-				return round( $item[ $column_name ] );
+				return round($item[$column_name]);
 			case 'average_earnings' :
-				return give_currency_filter( give_format_amount( $item[ $column_name ], array( 'sanitize' => false ) ) );
+				return give_currency_filter(give_format_amount($item[$column_name], array('sanitize' => false)));
 			case 'details' :
-				return '<a href="' . admin_url( 'edit.php?post_type=give_forms&page=give-reports&tab=forms&form-id=' . $item['ID'] ) . '">' . esc_html__( 'View Detailed Report', 'give' ) . '</a>';
+				return '<a href="'.admin_url('edit.php?post_type=give_forms&page=give-reports&tab=forms&form-id='.$item['ID']).'">'.esc_html__('View Detailed Report', 'give').'</a>';
 			default:
-				return $item[ $column_name ];
+				return $item[$column_name];
 		}
 	}
 
@@ -117,12 +117,12 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'title'            => esc_html__( 'Form', 'give' ),
-			'sales'            => esc_html__( 'Donations', 'give' ),
-			'earnings'         => esc_html__( 'Income', 'give' ),
-			'average_sales'    => esc_html__( 'Monthly Average Donations', 'give' ),
-			'average_earnings' => esc_html__( 'Monthly Average Income', 'give' ),
-			'details'          => esc_html__( 'Detailed Report', 'give' )
+			'title'            => esc_html__('Form', 'give'),
+			'sales'            => esc_html__('Donations', 'give'),
+			'earnings'         => esc_html__('Income', 'give'),
+			'average_sales'    => esc_html__('Monthly Average Donations', 'give'),
+			'average_earnings' => esc_html__('Monthly Average Income', 'give'),
+			'details'          => esc_html__('Detailed Report', 'give')
 		);
 
 		return $columns;
@@ -138,9 +138,9 @@ 
 	 */
 	public function get_sortable_columns() {
 		return array(
-			'title'    => array( 'title', true ),
-			'sales'    => array( 'sales', false ),
-			'earnings' => array( 'earnings', false ),
+			'title'    => array('title', true),
+			'sales'    => array('sales', false),
+			'earnings' => array('earnings', false),
 		);
 	}
 
@@ -153,7 +153,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;
 	}
 
 	/**
@@ -165,7 +165,7 @@ 
 	 * @return int Category ID
 	 */
 	public function get_category() {
-		return isset( $_GET['category'] ) ? absint( $_GET['category'] ) : 0;
+		return isset($_GET['category']) ? absint($_GET['category']) : 0;
 	}
 
 	/**
@@ -176,7 +176,7 @@ 
 	 *
 	 * @return void
 	 */
-	public function bulk_actions( $which = '' ) {
+	public function bulk_actions($which = '') {
 
 	}
 
@@ -188,27 +188,27 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
+	protected function display_tablenav($which) {
 
-		if ( 'top' === $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+		if ('top' === $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav give-clearfix <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav give-clearfix <?php echo esc_attr($which); ?>">
 
-			<?php if ( 'top' === $which ) { ?>
+			<?php if ('top' === $which) { ?>
 				<h2 class="alignleft reports-earnings-title">
-					<?php esc_html_e( 'Donation Forms Report', 'give' ); ?>
+					<?php esc_html_e('Donation Forms Report', 'give'); ?>
 				</h2>
 			<?php } ?>
 
 			<div class="alignright tablenav-right">
 				<div class="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);
 				?>
 			</div>
 
@@ -228,9 +228,9 @@ 
 	 */
 	public function category_filter() {
 
-		$categories = get_terms( 'form_category' );
-		if ( $categories && ! is_wp_error( $categories ) ) {
-			echo Give()->html->category_dropdown( 'category', $this->get_category() );
+		$categories = get_terms('form_category');
+		if ($categories && ! is_wp_error($categories)) {
+			echo Give()->html->category_dropdown('category', $this->get_category());
 		}
 	}
 
@@ -244,8 +244,8 @@ 
 	 */
 	public function query() {
 
-		$orderby  = isset( $_GET['orderby'] ) ? $_GET['orderby'] : 'title';
-		$order    = isset( $_GET['order'] ) ? $_GET['order'] : 'DESC';
+		$orderby  = isset($_GET['orderby']) ? $_GET['orderby'] : 'title';
+		$order    = isset($_GET['order']) ? $_GET['order'] : 'DESC';
 		$category = $this->get_category();
 
 		$args = array(
@@ -258,7 +258,7 @@ 
 			'suppress_filters' => true
 		);
 
-		if ( ! empty( $category ) ) {
+		if ( ! empty($category)) {
 			$args['tax_query'] = array(
 				array(
 					'taxonomy' => 'form_category',
@@ -267,7 +267,7 @@ 
 			);
 		}
 
-		switch ( $orderby ) :
+		switch ($orderby) :
 			case 'title' :
 				$args['orderby'] = 'title';
 				break;
@@ -283,9 +283,9 @@ 
 				break;
 		endswitch;
 
-		$args = apply_filters( 'give_form_reports_prepare_items_args', $args, $this );
+		$args = apply_filters('give_form_reports_prepare_items_args', $args, $this);
 
-		$this->donation_forms = new WP_Query( $args );
+		$this->donation_forms = new WP_Query($args);
 
 		// Store total number of donation forms count.
 		$this->count = $this->donation_forms->found_posts;
@@ -305,15 +305,15 @@ 
 
 		$give_forms = $this->donation_forms->posts;
 
-		if ( $give_forms ) {
-			foreach ( $give_forms as $form ) {
+		if ($give_forms) {
+			foreach ($give_forms as $form) {
 				$reports_data[] = array(
 					'ID'               => $form,
-					'title'            => get_the_title( $form ),
-					'sales'            => give_get_form_sales_stats( $form ),
-					'earnings'         => give_get_form_earnings_stats( $form ),
-					'average_sales'    => give_get_average_monthly_form_sales( $form ),
-					'average_earnings' => give_get_average_monthly_form_earnings( $form )
+					'title'            => get_the_title($form),
+					'sales'            => give_get_form_sales_stats($form),
+					'earnings'         => give_get_form_earnings_stats($form),
+					'average_sales'    => give_get_average_monthly_form_sales($form),
+					'average_earnings' => give_get_average_monthly_form_earnings($form)
 				);
 			}
 		}
@@ -338,14 +338,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->reports_data();
 		$total_items = $this->count;
 
-		$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-gateways-reports-table.php

Spacing +38 added lines, -38 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';
 }
 
 /**
@@ -45,11 +45,11 @@ 
 		global $status, $page;
 
 		// 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?
-		) );
+		));
 
 	}
 
@@ -64,10 +64,10 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
-		switch ( $column_name ) {
+	public function column_default($item, $column_name) {
+		switch ($column_name) {
 			default:
-				return $item[ $column_name ];
+				return $item[$column_name];
 		}
 	}
 
@@ -80,11 +80,11 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'label'           => esc_attr__( 'Gateway', 'give' ),
-			'complete_sales'  => esc_attr__( 'Complete Payments', 'give' ),
-			'pending_sales'   => esc_attr__( 'Pending / Failed Payments', 'give' ),
-			'total_sales'     => esc_attr__( 'Total Payments', 'give' ),
-			'total_donations' => esc_attr__( 'Total Donated', 'give' )
+			'label'           => esc_attr__('Gateway', 'give'),
+			'complete_sales'  => esc_attr__('Complete Payments', 'give'),
+			'pending_sales'   => esc_attr__('Pending / Failed Payments', 'give'),
+			'total_sales'     => esc_attr__('Total Payments', 'give'),
+			'total_donations' => esc_attr__('Total Donated', 'give')
 		);
 
 		return $columns;
@@ -99,7 +99,7 @@ 
 	 */
 	public function get_sortable_columns() {
 		return array(
-			'total_donations' => array( 'total_donations', false )
+			'total_donations' => array('total_donations', false)
 		);
 	}
 
@@ -112,7 +112,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;
 	}
 
 
@@ -123,7 +123,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	public function bulk_actions( $which = '' ) {
+	public function bulk_actions($which = '') {
 
 	}
 
@@ -135,27 +135,27 @@ 
 	 *
 	 * @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 gateways-report-tablenav give-clearfix <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav gateways-report-tablenav give-clearfix <?php echo esc_attr($which); ?>">
 
-			<?php if ( 'top' === $which ) { ?>
+			<?php if ('top' === $which) { ?>
 				<h2 class="alignleft reports-earnings-title">
-					<?php esc_html_e( 'Donation Methods Report', 'give' ); ?>
+					<?php esc_html_e('Donation Methods Report', 'give'); ?>
 				</h2>
 			<?php } ?>
 
 			<div class="alignright tablenav-right">
 				<div class="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);
 				?>
 			</div>
 
@@ -177,17 +177,17 @@ 
 	 *
 	 * @return int
 	 */
-	public function give_sort_total_donations( $old_value, $new_value ) {
+	public function give_sort_total_donations($old_value, $new_value) {
 		// If no sort, default to label.
-		$orderby = ( ! empty( $_REQUEST['orderby'] ) ) ? $_REQUEST['orderby'] : 'label';
+		$orderby = ( ! empty($_REQUEST['orderby'])) ? $_REQUEST['orderby'] : 'label';
 
 		//If no order, default to asc.
-		$order = ( ! empty( $_REQUEST['order'] ) ) ? $_REQUEST['order'] : 'asc';
+		$order = ( ! empty($_REQUEST['order'])) ? $_REQUEST['order'] : 'asc';
 
 		//Determine sort order.
-		$result = strcmp( $old_value[ $orderby ], $new_value[ $orderby ] );
+		$result = strcmp($old_value[$orderby], $new_value[$orderby]);
 
-		return ( $order === 'asc' ) ? $result : -$result;
+		return ($order === 'asc') ? $result : -$result;
 	}
 
 
@@ -204,10 +204,10 @@ 
 		$gateways     = give_get_payment_gateways();
 		$stats        = new Give_Payment_Stats();
 
-		foreach ( $gateways as $gateway_id => $gateway ) {
+		foreach ($gateways as $gateway_id => $gateway) {
 
-			$complete_count = give_count_sales_by_gateway( $gateway_id, 'publish' );
-			$pending_count  = give_count_sales_by_gateway( $gateway_id, array( 'pending', 'failed' ) );
+			$complete_count = give_count_sales_by_gateway($gateway_id, 'publish');
+			$pending_count  = give_count_sales_by_gateway($gateway_id, array('pending', 'failed'));
 
 			$reports_data[] = array(
 				'ID'              => $gateway_id,
@@ -215,7 +215,7 @@ 
 				'complete_sales'  => $complete_count,
 				'pending_sales'   => $pending_count,
 				'total_sales'     => $complete_count + $pending_count,
-				'total_donations' => give_currency_filter( give_format_amount( $stats->get_earnings( 0, strtotime('04/13/2015' ), current_time('timestamp' ), $gateway_id ), array( 'sanitize' => false ) ) ),
+				'total_donations' => give_currency_filter(give_format_amount($stats->get_earnings(0, strtotime('04/13/2015'), current_time('timestamp'), $gateway_id), array('sanitize' => false))),
 			);
 		}
 
@@ -236,11 +236,11 @@ 
 		$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->reports_data();
 
 		// Sort Array when we are sorting data in array.
-		usort( $this->items, array( $this, 'give_sort_total_donations' ) );
+		usort($this->items, array($this, 'give_sort_total_donations'));
 
 	}
 }

includes/admin/reports/reports.php

Spacing +63 added lines, -63 removed lines

@@ -16,7 +16,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -29,8 +29,8 @@ 
  * @return void
  */
 function give_reports_page() {
-	$current_page = admin_url( 'edit.php?post_type=give_forms&page=give-reports' );
-	$active_tab   = isset( $_GET['tab'] ) ? sanitize_text_field( $_GET['tab'] ) : 'earnings';
+	$current_page = admin_url('edit.php?post_type=give_forms&page=give-reports');
+	$active_tab   = isset($_GET['tab']) ? sanitize_text_field($_GET['tab']) : 'earnings';
 	$views        = give_reports_default_views();
 	?>
 	<div class="wrap give-settings-page">
@@ -38,17 +38,17 @@ 
 		<h1 class="screen-reader-text"><?php echo get_admin_page_title(); ?></h1>
 
 		<h2 class="nav-tab-wrapper">
-			<?php foreach ( $views as $tab => $label ) { ?>
-				<a href="<?php echo esc_url( add_query_arg( array(
+			<?php foreach ($views as $tab => $label) { ?>
+				<a href="<?php echo esc_url(add_query_arg(array(
 					'tab'              => $tab,
 					'settings-updated' => false,
-				), $current_page ) ); ?>" class="nav-tab <?php echo $tab === $active_tab ? esc_attr( 'nav-tab-active' ) : ''; ?>"><?php echo esc_html( $label ); ?></a>
+				), $current_page)); ?>" class="nav-tab <?php echo $tab === $active_tab ? esc_attr('nav-tab-active') : ''; ?>"><?php echo esc_html($label); ?></a>
 			<?php } ?>
-			<?php if ( current_user_can( 'export_give_reports' ) ) { ?>
-				<a href="<?php echo esc_url( add_query_arg( array(
+			<?php if (current_user_can('export_give_reports')) { ?>
+				<a href="<?php echo esc_url(add_query_arg(array(
 					'tab'              => 'export',
 					'settings-updated' => false,
-				), $current_page ) ); ?>" class="nav-tab <?php echo 'export' === $active_tab ? esc_attr( 'nav-tab-active' ) : ''; ?>"><?php esc_html_e( 'Export', 'give' ); ?></a>
+				), $current_page)); ?>" class="nav-tab <?php echo 'export' === $active_tab ? esc_attr('nav-tab-active') : ''; ?>"><?php esc_html_e('Export', 'give'); ?></a>
 			<?php }
 			/**
 			 * Fires in the report tabs.
@@ -57,7 +57,7 @@ 
 			 *
 			 * @since 1.0
 			 */
-			do_action( 'give_reports_tabs' );
+			do_action('give_reports_tabs');
 			?>
 		</h2>
 
@@ -67,10 +67,10 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_reports_page_top' );
+		do_action('give_reports_page_top');
 
 		// Set $active_tab prior to hook firing.
-		if ( in_array( $active_tab, array_keys( $views ) ) ) {
+		if (in_array($active_tab, array_keys($views))) {
 			$active_tab = 'reports';
 		}
 
@@ -79,14 +79,14 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( "give_reports_tab_{$active_tab}" );
+		do_action("give_reports_tab_{$active_tab}");
 
 		/**
 		 * Fires after the report page.
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_reports_page_bottom' );
+		do_action('give_reports_page_bottom');
 		?>
 	</div><!-- .wrap -->
 	<?php
@@ -100,13 +100,13 @@ 
  */
 function give_reports_default_views() {
 	$views = array(
-		'earnings' => esc_html__( 'Income', 'give' ),
-		'forms'    => esc_html__( 'Forms', 'give' ),
-		'donors'   => esc_html__( 'Donors', 'give' ),
-		'gateways' => esc_html__( 'Donation Methods', 'give' ),
+		'earnings' => esc_html__('Income', 'give'),
+		'forms'    => esc_html__('Forms', 'give'),
+		'donors'   => esc_html__('Donors', 'give'),
+		'gateways' => esc_html__('Donation Methods', 'give'),
 	);
 
-	$views = apply_filters( 'give_report_views', $views );
+	$views = apply_filters('give_report_views', $views);
 
 	return $views;
 }
@@ -121,15 +121,15 @@ 
  * @since 1.0
  * @return string $view Report View
  */
-function give_get_reporting_view( $default = 'earnings' ) {
+function give_get_reporting_view($default = 'earnings') {
 
-	if ( ! isset( $_GET['view'] ) || ! in_array( $_GET['view'], array_keys( give_reports_default_views() ) ) ) {
+	if ( ! isset($_GET['view']) || ! in_array($_GET['view'], array_keys(give_reports_default_views()))) {
 		$view = $default;
 	} else {
 		$view = $_GET['view'];
 	}
 
-	return apply_filters( 'give_get_reporting_view', $view );
+	return apply_filters('give_get_reporting_view', $view);
 }
 
 /**
@@ -142,7 +142,7 @@ 
 	$current_view = 'earnings';
 	$views        = give_reports_default_views();
 
-	if ( isset( $_GET['tab'] ) && array_key_exists( $_GET['tab'], $views ) ) {
+	if (isset($_GET['tab']) && array_key_exists($_GET['tab'], $views)) {
 		$current_view = $_GET['tab'];
 	}
 
@@ -151,10 +151,10 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( "give_reports_view_{$current_view}" );
+	do_action("give_reports_view_{$current_view}");
 }
 
-add_action( 'give_reports_tab_reports', 'give_reports_tab_reports' );
+add_action('give_reports_tab_reports', 'give_reports_tab_reports');
 
 /**
  * Renders the Reports Page Views Drop Downs
@@ -164,19 +164,19 @@ 
  */
 function give_report_views() {
 	$views        = give_reports_default_views();
-	$current_view = isset( $_GET['view'] ) ? $_GET['view'] : 'earnings';
+	$current_view = isset($_GET['view']) ? $_GET['view'] : 'earnings';
 	/**
 	 * Fires before the report page actions form.
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_report_view_actions_before' );
+	do_action('give_report_view_actions_before');
 	?>
 	<form id="give-reports-filter" method="get">
 		<select id="give-reports-view" name="view">
-			<option value="-1"><?php esc_html_e( 'Report Type', 'give' ); ?></option>
-			<?php foreach ( $views as $view_id => $label ) : ?>
-				<option value="<?php echo esc_attr( $view_id ); ?>" <?php selected( $view_id, $current_view ); ?>><?php echo $label; ?></option>
+			<option value="-1"><?php esc_html_e('Report Type', 'give'); ?></option>
+			<?php foreach ($views as $view_id => $label) : ?>
+				<option value="<?php echo esc_attr($view_id); ?>" <?php selected($view_id, $current_view); ?>><?php echo $label; ?></option>
 			<?php endforeach; ?>
 		</select>
 
@@ -188,12 +188,12 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_report_view_actions' );
+		do_action('give_report_view_actions');
 		?>
 
 		<input type="hidden" name="post_type" value="give_forms"/>
 		<input type="hidden" name="page" value="give-reports"/>
-		<?php submit_button( esc_html__( 'Show', 'give' ), 'secondary', 'submit', false ); ?>
+		<?php submit_button(esc_html__('Show', 'give'), 'secondary', 'submit', false); ?>
 	</form>
 	<?php
 	/**
@@ -201,7 +201,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_report_view_actions_after' );
+	do_action('give_report_view_actions_after');
 }
 
 /**
@@ -214,11 +214,11 @@ 
  */
 function give_reports_forms_table() {
 
-	if ( isset( $_GET['form-id'] ) ) {
+	if (isset($_GET['form-id'])) {
 		return;
 	}
 
-	include( dirname( __FILE__ ) . '/class-form-reports-table.php' );
+	include(dirname(__FILE__).'/class-form-reports-table.php');
 
 	$give_table = new Give_Form_Reports_Table();
 	$give_table->prepare_items();
@@ -230,7 +230,7 @@ 
 	<?php
 }
 
-add_action( 'give_reports_view_forms', 'give_reports_forms_table' );
+add_action('give_reports_view_forms', 'give_reports_forms_table');
 
 /**
  * Renders the detailed report for a specific give form.
@@ -239,20 +239,20 @@ 
  * @return void
  */
 function give_reports_form_details() {
-	if ( ! isset( $_GET['form-id'] ) ) {
+	if ( ! isset($_GET['form-id'])) {
 		return;
 	}
 	?>
 	<div class="tablenav top reports-forms-details-wrap">
 		<div class="actions bulkactions">
-			<button onclick="history.go(-1);" class="button-secondary"><?php esc_html_e( 'Go Back', 'give' ); ?></button>
+			<button onclick="history.go(-1);" class="button-secondary"><?php esc_html_e('Go Back', 'give'); ?></button>
 		</div>
 	</div>
 	<?php
-	give_reports_graph_of_form( absint( $_GET['form-id'] ) );
+	give_reports_graph_of_form(absint($_GET['form-id']));
 }
 
-add_action( 'give_reports_view_forms', 'give_reports_form_details' );
+add_action('give_reports_view_forms', 'give_reports_form_details');
 
 /**
  * Renders the Reports Donors Table
@@ -263,7 +263,7 @@ 
  * @return void
  */
 function give_reports_donors_table() {
-	include( dirname( __FILE__ ) . '/class-donor-reports-table.php' );
+	include(dirname(__FILE__).'/class-donor-reports-table.php');
 
 	$give_table = new Give_Donor_Reports_Table();
 	$give_table->prepare_items();
@@ -275,9 +275,9 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_logs_donors_table_top' );
+		do_action('give_logs_donors_table_top');
 
-		$give_table->search_box( esc_html__( 'Search', 'give' ), 'give-donors' );
+		$give_table->search_box(esc_html__('Search', 'give'), 'give-donors');
 		$give_table->display();
 		?>
 		<input type="hidden" name="post_type" value="give_forms"/>
@@ -290,13 +290,13 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_logs_donors_table_bottom' );
+		do_action('give_logs_donors_table_bottom');
 		?>
 	</div>
 	<?php
 }
 
-add_action( 'give_reports_view_donors', 'give_reports_donors_table' );
+add_action('give_reports_view_donors', 'give_reports_donors_table');
 
 /**
  * Renders the Gateways Table
@@ -307,14 +307,14 @@ 
  * @return void
  */
 function give_reports_gateways_table() {
-	include( dirname( __FILE__ ) . '/class-gateways-reports-table.php' );
+	include(dirname(__FILE__).'/class-gateways-reports-table.php');
 
 	$give_table = new Give_Gateway_Reports_Table();
 	$give_table->prepare_items();
 	$give_table->display();
 }
 
-add_action( 'give_reports_view_gateways', 'give_reports_gateways_table' );
+add_action('give_reports_view_gateways', 'give_reports_gateways_table');
 
 /**
  * Renders the Reports Earnings Graphs
@@ -325,13 +325,13 @@ 
 function give_reports_earnings() {
 	?>
 	<div class="tablenav top reports-table-nav">
-		<h2 class="reports-earnings-title"><?php esc_html_e( 'Income Report', 'give' ); ?></h2>
+		<h2 class="reports-earnings-title"><?php esc_html_e('Income Report', 'give'); ?></h2>
 	</div>
 	<?php
 	give_reports_graph();
 }
 
-add_action( 'give_reports_view_earnings', 'give_reports_earnings' );
+add_action('give_reports_view_earnings', 'give_reports_earnings');
 
 
 /**
@@ -342,9 +342,9 @@ 
  */
 function give_estimated_monthly_stats() {
 
-	$estimated = Give_Cache::get( 'give_estimated_monthly_stats', true );
+	$estimated = Give_Cache::get('give_estimated_monthly_stats', true);
 
-	if ( false === $estimated ) {
+	if (false === $estimated) {
 
 		$estimated = array(
 			'earnings' => 0,
@@ -353,22 +353,22 @@ 
 
 		$stats = new Give_Payment_Stats;
 
-		$to_date_earnings = $stats->get_earnings( 0, 'this_month' );
-		$to_date_sales    = $stats->get_sales( 0, 'this_month' );
+		$to_date_earnings = $stats->get_earnings(0, 'this_month');
+		$to_date_sales    = $stats->get_sales(0, 'this_month');
 
-		$current_day   = date( 'd', current_time( 'timestamp' ) );
-		$current_month = date( 'n', current_time( 'timestamp' ) );
-		$current_year  = date( 'Y', current_time( 'timestamp' ) );
-		$days_in_month = cal_days_in_month( CAL_GREGORIAN, $current_month, $current_year );
+		$current_day   = date('d', current_time('timestamp'));
+		$current_month = date('n', current_time('timestamp'));
+		$current_year  = date('Y', current_time('timestamp'));
+		$days_in_month = cal_days_in_month(CAL_GREGORIAN, $current_month, $current_year);
 
-		$estimated['earnings'] = ( $to_date_earnings / $current_day ) * $days_in_month;
-		$estimated['sales']    = ( $to_date_sales / $current_day ) * $days_in_month;
+		$estimated['earnings'] = ($to_date_earnings / $current_day) * $days_in_month;
+		$estimated['sales']    = ($to_date_sales / $current_day) * $days_in_month;
 
 		// Cache for one day
-		Give_Cache::set( 'give_estimated_monthly_stats', $estimated, DAY_IN_SECONDS, true );
+		Give_Cache::set('give_estimated_monthly_stats', $estimated, DAY_IN_SECONDS, true);
 	}
 
-	return maybe_unserialize( $estimated );
+	return maybe_unserialize($estimated);
 }
 
 /**
@@ -381,7 +381,7 @@ 
 function give_reports_set_form_method() {
 	return 'get';
 }
-add_filter( 'give-reports_form_method_tab_forms', 'give_reports_set_form_method', 10 );
-add_filter( 'give-reports_form_method_tab_donors', 'give_reports_set_form_method', 10 );
+add_filter('give-reports_form_method_tab_forms', 'give_reports_set_form_method', 10);
+add_filter('give-reports_form_method_tab_donors', 'give_reports_set_form_method', 10);
 
 // @TODO: After release 1.8 Donations -> Reports generates with new setting api, so we can remove some old code from this file.

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

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" class="wp-heading-inline"><?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&donor=' . absint( give_get_payment_donor_id( $payment_id ) ) ); ?>
-												<a href="<?php echo $purchase_url; ?>"><?php esc_html_e( 'View all donations for this donor &raquo;', 'give' ); ?></a>
+											<p><?php $purchase_url = admin_url('edit.php?post_type=give_forms&page=give-payment-history&donor='.absint(give_get_payment_donor_id($payment_id))); ?>
+												<a href="<?php echo $purchase_url; ?>"><?php esc_html_e('View all donations for this donor &raquo;', 'give'); ?></a>
 											</p>
 										</div>
 
@@ -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">#%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">#%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/scripts.php

Spacing +168 added lines, -168 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,26 +25,26 @@ 
  */
 function give_load_scripts() {
 
-	$js_dir         = GIVE_PLUGIN_URL . 'assets/js/frontend/';
-	$js_plugins     = GIVE_PLUGIN_URL . 'assets/js/plugins/';
-	$scripts_footer = ( give_is_setting_enabled( give_get_option( 'scripts_footer' ) ) ) ? true : false;
+	$js_dir         = GIVE_PLUGIN_URL.'assets/js/frontend/';
+	$js_plugins     = GIVE_PLUGIN_URL.'assets/js/plugins/';
+	$scripts_footer = (give_is_setting_enabled(give_get_option('scripts_footer'))) ? true : false;
 
 	// Use minified libraries if SCRIPT_DEBUG is turned off.
-	$suffix = ( defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG ) ? '' : '.min';
+	$suffix = (defined('SCRIPT_DEBUG') && SCRIPT_DEBUG) ? '' : '.min';
 
 	// Localize / PHP to AJAX vars.
-	$localize_give_vars = apply_filters( 'give_global_script_vars', array(
+	$localize_give_vars = apply_filters('give_global_script_vars', array(
 		'ajaxurl'             => give_get_ajax_url(),
-		'checkout_nonce'      => wp_create_nonce( 'give_checkout_nonce' ),
+		'checkout_nonce'      => wp_create_nonce('give_checkout_nonce'),
 		'currency'            => give_get_currency(),
-		'currency_sign'       => give_currency_filter( '' ),
+		'currency_sign'       => give_currency_filter(''),
 		'currency_pos'        => give_get_currency_position(),
 		'thousands_separator' => give_get_price_thousand_separator(),
 		'decimal_separator'   => give_get_price_decimal_separator(),
-		'no_gateway'          => __( 'Please select a payment method.', 'give' ),
-		'bad_minimum'         => __( 'The minimum custom donation amount for this form is', 'give' ),
-		'general_loading'     => __( 'Loading...', 'give' ),
-		'purchase_loading'    => __( 'Please Wait...', 'give' ),
+		'no_gateway'          => __('Please select a payment method.', 'give'),
+		'bad_minimum'         => __('The minimum custom donation amount for this form is', 'give'),
+		'general_loading'     => __('Loading...', 'give'),
+		'purchase_loading'    => __('Please Wait...', 'give'),
 		'number_decimals'     => give_get_price_decimals(),
 		'give_version'        => GIVE_VERSION,
 		'magnific_options'    => apply_filters(
@@ -58,81 +58,81 @@ 
 			'give_form_translation_js',
 			array(
 				// Field name               Validation message.
-				'payment-mode'           => __( 'Please select payment mode.', 'give' ),
-				'give_first'             => __( 'Please enter your first name.', 'give' ),
-				'give_email'             => __( 'Please enter a valid email address.', 'give' ),
-				'give_user_login'        => __( 'Invalid username. Only lowercase letters (a-z) and numbers are allowed.', 'give' ),
-				'give_user_pass'         => __( 'Enter a password.', 'give' ),
-				'give_user_pass_confirm' => __( 'Enter the password confirmation.', 'give' ),
-				'give_agree_to_terms'    => __( 'You must agree to the terms and conditions.', 'give' ),
+				'payment-mode'           => __('Please select payment mode.', 'give'),
+				'give_first'             => __('Please enter your first name.', 'give'),
+				'give_email'             => __('Please enter a valid email address.', 'give'),
+				'give_user_login'        => __('Invalid username. Only lowercase letters (a-z) and numbers are allowed.', 'give'),
+				'give_user_pass'         => __('Enter a password.', 'give'),
+				'give_user_pass_confirm' => __('Enter the password confirmation.', 'give'),
+				'give_agree_to_terms'    => __('You must agree to the terms and conditions.', 'give'),
 			)
 		),
-	) );
+	));
 
-	$localize_give_ajax = apply_filters( 'give_global_ajax_vars', array(
+	$localize_give_ajax = apply_filters('give_global_ajax_vars', array(
 		'ajaxurl'         => give_get_ajax_url(),
-		'loading'         => __( 'Loading', 'give' ),
+		'loading'         => __('Loading', 'give'),
 		// General loading message.
-		'select_option'   => __( 'Please select an option', 'give' ),
+		'select_option'   => __('Please select an option', 'give'),
 		// Variable pricing error with multi-donation option enabled.
-		'default_gateway' => give_get_default_gateway( null ),
-		'permalinks'      => get_option( 'permalink_structure' ) ? '1' : '0',
+		'default_gateway' => give_get_default_gateway(null),
+		'permalinks'      => get_option('permalink_structure') ? '1' : '0',
 		'number_decimals' => give_get_price_decimals(),
-	) );
+	));
 
 	// DEBUG is On.
-	if ( defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG ) {
+	if (defined('SCRIPT_DEBUG') && SCRIPT_DEBUG) {
 
-		if ( give_is_cc_verify_enabled() ) {
-			wp_register_script( 'give-cc-validator', $js_plugins . 'jquery.payment' . $suffix . '.js', array( 'jquery' ), GIVE_VERSION, $scripts_footer );
-			wp_enqueue_script( 'give-cc-validator' );
+		if (give_is_cc_verify_enabled()) {
+			wp_register_script('give-cc-validator', $js_plugins.'jquery.payment'.$suffix.'.js', array('jquery'), GIVE_VERSION, $scripts_footer);
+			wp_enqueue_script('give-cc-validator');
 		}
 
-		wp_register_script( 'give-float-labels', $js_plugins . 'float-labels' . $suffix . '.js', array( 'jquery' ), GIVE_VERSION, $scripts_footer );
-		wp_enqueue_script( 'give-float-labels' );
+		wp_register_script('give-float-labels', $js_plugins.'float-labels'.$suffix.'.js', array('jquery'), GIVE_VERSION, $scripts_footer);
+		wp_enqueue_script('give-float-labels');
 
-		wp_register_script( 'give-blockui', $js_plugins . 'jquery.blockUI' . $suffix . '.js', array( 'jquery' ), GIVE_VERSION, $scripts_footer );
-		wp_enqueue_script( 'give-blockui' );
+		wp_register_script('give-blockui', $js_plugins.'jquery.blockUI'.$suffix.'.js', array('jquery'), GIVE_VERSION, $scripts_footer);
+		wp_enqueue_script('give-blockui');
 
-		wp_register_script( 'give-qtip', $js_plugins . 'jquery.qtip' . $suffix . '.js', array( 'jquery' ), GIVE_VERSION, $scripts_footer );
-		wp_enqueue_script( 'give-qtip' );
+		wp_register_script('give-qtip', $js_plugins.'jquery.qtip'.$suffix.'.js', array('jquery'), GIVE_VERSION, $scripts_footer);
+		wp_enqueue_script('give-qtip');
 
-		wp_register_script( 'give-accounting', $js_plugins . 'accounting' . $suffix . '.js', array( 'jquery' ), GIVE_VERSION, $scripts_footer );
-		wp_enqueue_script( 'give-accounting' );
+		wp_register_script('give-accounting', $js_plugins.'accounting'.$suffix.'.js', array('jquery'), GIVE_VERSION, $scripts_footer);
+		wp_enqueue_script('give-accounting');
 
-		wp_register_script( 'give-magnific', $js_plugins . 'jquery.magnific-popup' . $suffix . '.js', array( 'jquery' ), GIVE_VERSION, $scripts_footer );
-		wp_enqueue_script( 'give-magnific' );
+		wp_register_script('give-magnific', $js_plugins.'jquery.magnific-popup'.$suffix.'.js', array('jquery'), GIVE_VERSION, $scripts_footer);
+		wp_enqueue_script('give-magnific');
 
-		wp_register_script( 'give-checkout-global', $js_dir . 'give-checkout-global' . $suffix . '.js', array( 'jquery' ), GIVE_VERSION, $scripts_footer );
-		wp_enqueue_script( 'give-checkout-global' );
+		wp_register_script('give-checkout-global', $js_dir.'give-checkout-global'.$suffix.'.js', array('jquery'), GIVE_VERSION, $scripts_footer);
+		wp_enqueue_script('give-checkout-global');
 
 		// General scripts.
-		wp_register_script( 'give-scripts', $js_dir . 'give' . $suffix . '.js', array( 'jquery' ), GIVE_VERSION, $scripts_footer );
-		wp_enqueue_script( 'give-scripts' );
+		wp_register_script('give-scripts', $js_dir.'give'.$suffix.'.js', array('jquery'), GIVE_VERSION, $scripts_footer);
+		wp_enqueue_script('give-scripts');
 
 		// Load AJAX scripts, if enabled.
-		wp_register_script( 'give-ajax', $js_dir . 'give-ajax' . $suffix . '.js', array( 'jquery' ), GIVE_VERSION, $scripts_footer );
-		wp_enqueue_script( 'give-ajax' );
+		wp_register_script('give-ajax', $js_dir.'give-ajax'.$suffix.'.js', array('jquery'), GIVE_VERSION, $scripts_footer);
+		wp_enqueue_script('give-ajax');
 
 		// Localize / Pass AJAX vars from PHP,
-		wp_localize_script( 'give-checkout-global', 'give_global_vars', $localize_give_vars );
-		wp_localize_script( 'give-ajax', 'give_scripts', $localize_give_ajax );
+		wp_localize_script('give-checkout-global', 'give_global_vars', $localize_give_vars);
+		wp_localize_script('give-ajax', 'give_scripts', $localize_give_ajax);
 
 	} else {
 
 		// DEBUG is OFF (one JS file to rule them all!).
-		wp_register_script( 'give', $js_dir . 'give.all.min.js', array( 'jquery' ), GIVE_VERSION, $scripts_footer );
-		wp_enqueue_script( 'give' );
+		wp_register_script('give', $js_dir.'give.all.min.js', array('jquery'), GIVE_VERSION, $scripts_footer);
+		wp_enqueue_script('give');
 
 		// Localize / Pass AJAX vars from PHP.
-		wp_localize_script( 'give', 'give_global_vars', $localize_give_vars );
-		wp_localize_script( 'give', 'give_scripts', $localize_give_ajax );
+		wp_localize_script('give', 'give_global_vars', $localize_give_vars);
+		wp_localize_script('give', 'give_scripts', $localize_give_ajax);
 
 	}
 
 }
 
-add_action( 'wp_enqueue_scripts', 'give_load_scripts' );
+add_action('wp_enqueue_scripts', 'give_load_scripts');
 
 /**
  * Register styles.
@@ -145,16 +145,16 @@ 
  */
 function give_register_styles() {
 
-	if ( ! give_is_setting_enabled( give_get_option( 'css' ) ) ) {
+	if ( ! give_is_setting_enabled(give_get_option('css'))) {
 		return;
 	}
 
-	wp_register_style( 'give-styles', give_get_stylesheet_uri(), array(), GIVE_VERSION, 'all' );
-	wp_enqueue_style( 'give-styles' );
+	wp_register_style('give-styles', give_get_stylesheet_uri(), array(), GIVE_VERSION, 'all');
+	wp_enqueue_style('give-styles');
 
 }
 
-add_action( 'wp_enqueue_scripts', 'give_register_styles' );
+add_action('wp_enqueue_scripts', 'give_register_styles');
 
 
 /**
@@ -167,19 +167,19 @@ 
 function give_get_stylesheet_uri() {
 
 	// Use minified libraries if SCRIPT_DEBUG is turned off.
-	$suffix = ( defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG ) ? '' : '.min';
+	$suffix = (defined('SCRIPT_DEBUG') && SCRIPT_DEBUG) ? '' : '.min';
 
 	// LTR or RTL files.
-	$direction = ( is_rtl() ) ? '-rtl' : '';
+	$direction = (is_rtl()) ? '-rtl' : '';
 
-	$file          = 'give' . $direction . $suffix . '.css';
+	$file          = 'give'.$direction.$suffix.'.css';
 	$templates_dir = give_get_theme_template_dir_name();
 
-	$child_theme_style_sheet    = trailingslashit( get_stylesheet_directory() ) . $templates_dir . $file;
-	$child_theme_style_sheet_2  = trailingslashit( get_stylesheet_directory() ) . $templates_dir . 'give' . $direction . '.css';
-	$parent_theme_style_sheet   = trailingslashit( get_template_directory() ) . $templates_dir . $file;
-	$parent_theme_style_sheet_2 = trailingslashit( get_template_directory() ) . $templates_dir . 'give' . $direction . '.css';
-	$give_plugin_style_sheet    = trailingslashit( give_get_templates_dir() ) . $file;
+	$child_theme_style_sheet    = trailingslashit(get_stylesheet_directory()).$templates_dir.$file;
+	$child_theme_style_sheet_2  = trailingslashit(get_stylesheet_directory()).$templates_dir.'give'.$direction.'.css';
+	$parent_theme_style_sheet   = trailingslashit(get_template_directory()).$templates_dir.$file;
+	$parent_theme_style_sheet_2 = trailingslashit(get_template_directory()).$templates_dir.'give'.$direction.'.css';
+	$give_plugin_style_sheet    = trailingslashit(give_get_templates_dir()).$file;
 
 	$uri = false;
 
@@ -189,23 +189,23 @@ 
 	 * followed by non minified version, even if SCRIPT_DEBUG is not enabled.
 	 * This allows users to copy just give.css to their theme.
 	 */
-	if ( file_exists( $child_theme_style_sheet ) || ( ! empty( $suffix ) && ( $nonmin = file_exists( $child_theme_style_sheet_2 ) ) ) ) {
-		if ( ! empty( $nonmin ) ) {
-			$uri = trailingslashit( get_stylesheet_directory_uri() ) . $templates_dir . 'give' . $direction . '.css';
+	if (file_exists($child_theme_style_sheet) || ( ! empty($suffix) && ($nonmin = file_exists($child_theme_style_sheet_2)))) {
+		if ( ! empty($nonmin)) {
+			$uri = trailingslashit(get_stylesheet_directory_uri()).$templates_dir.'give'.$direction.'.css';
 		} else {
-			$uri = trailingslashit( get_stylesheet_directory_uri() ) . $templates_dir . $file;
+			$uri = trailingslashit(get_stylesheet_directory_uri()).$templates_dir.$file;
 		}
-	} elseif ( file_exists( $parent_theme_style_sheet ) || ( ! empty( $suffix ) && ( $nonmin = file_exists( $parent_theme_style_sheet_2 ) ) ) ) {
-		if ( ! empty( $nonmin ) ) {
-			$uri = trailingslashit( get_template_directory_uri() ) . $templates_dir . 'give' . $direction . '.css';
+	} elseif (file_exists($parent_theme_style_sheet) || ( ! empty($suffix) && ($nonmin = file_exists($parent_theme_style_sheet_2)))) {
+		if ( ! empty($nonmin)) {
+			$uri = trailingslashit(get_template_directory_uri()).$templates_dir.'give'.$direction.'.css';
 		} else {
-			$uri = trailingslashit( get_template_directory_uri() ) . $templates_dir . $file;
+			$uri = trailingslashit(get_template_directory_uri()).$templates_dir.$file;
 		}
-	} elseif ( file_exists( $give_plugin_style_sheet ) || file_exists( $give_plugin_style_sheet ) ) {
-		$uri = trailingslashit( give_get_templates_url() ) . $file;
+	} elseif (file_exists($give_plugin_style_sheet) || file_exists($give_plugin_style_sheet)) {
+		$uri = trailingslashit(give_get_templates_url()).$file;
 	}
 
-	return apply_filters( 'give_get_stylesheet_uri', $uri );
+	return apply_filters('give_get_stylesheet_uri', $uri);
 
 }
 
@@ -222,79 +222,79 @@ 
  *
  * @return void
  */
-function give_load_admin_scripts( $hook ) {
+function give_load_admin_scripts($hook) {
 
 	global $post, $post_type;
 
 	$give_options = give_get_settings();
 
 	// Directories of assets.
-	$js_dir     = GIVE_PLUGIN_URL . 'assets/js/admin/';
-	$js_plugins = GIVE_PLUGIN_URL . 'assets/js/plugins/';
-	$css_dir    = GIVE_PLUGIN_URL . 'assets/css/';
+	$js_dir     = GIVE_PLUGIN_URL.'assets/js/admin/';
+	$js_plugins = GIVE_PLUGIN_URL.'assets/js/plugins/';
+	$css_dir    = GIVE_PLUGIN_URL.'assets/css/';
 
 	// Use minified libraries if SCRIPT_DEBUG is turned off.
-	$suffix = ( defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG ) ? '' : '.min';
+	$suffix = (defined('SCRIPT_DEBUG') && SCRIPT_DEBUG) ? '' : '.min';
 
 	// LTR or RTL files.
-	$direction = ( is_rtl() ) ? '-rtl' : '';
+	$direction = (is_rtl()) ? '-rtl' : '';
 
 	// Global Admin.
-	wp_register_style( 'give-admin-bar-notification', $css_dir . 'adminbar-style.css' );
-	wp_enqueue_style( 'give-admin-bar-notification' );
+	wp_register_style('give-admin-bar-notification', $css_dir.'adminbar-style.css');
+	wp_enqueue_style('give-admin-bar-notification');
 
 	// Give Admin Only.
-	if ( ! apply_filters( 'give_load_admin_scripts', give_is_admin_page(), $hook ) ) {
+	if ( ! apply_filters('give_load_admin_scripts', give_is_admin_page(), $hook)) {
 		return;
 	}
 
 	// CSS.
-	wp_register_style( 'jquery-ui-css', $css_dir . 'jquery-ui-fresh' . $suffix . '.css' );
-	wp_enqueue_style( 'jquery-ui-css' );
-	wp_register_style( 'give-admin', $css_dir . 'give-admin' . $direction . $suffix . '.css', array(), GIVE_VERSION );
-	wp_enqueue_style( 'give-admin' );
-	wp_register_style( 'jquery-chosen', $css_dir . 'chosen' . $suffix . '.css', array(), GIVE_VERSION );
-	wp_enqueue_style( 'jquery-chosen' );
-	wp_enqueue_style( 'thickbox' );
-	wp_enqueue_style( 'wp-color-picker' );
+	wp_register_style('jquery-ui-css', $css_dir.'jquery-ui-fresh'.$suffix.'.css');
+	wp_enqueue_style('jquery-ui-css');
+	wp_register_style('give-admin', $css_dir.'give-admin'.$direction.$suffix.'.css', array(), GIVE_VERSION);
+	wp_enqueue_style('give-admin');
+	wp_register_style('jquery-chosen', $css_dir.'chosen'.$suffix.'.css', array(), GIVE_VERSION);
+	wp_enqueue_style('jquery-chosen');
+	wp_enqueue_style('thickbox');
+	wp_enqueue_style('wp-color-picker');
 
 
 	// JS.
-	wp_register_script( 'give-selector-cache', $js_plugins . 'selector-cache' . $suffix . '.js', array( 'jquery' ), GIVE_VERSION, false );
-	wp_enqueue_script( 'give-selector-cache' );
+	wp_register_script('give-selector-cache', $js_plugins.'selector-cache'.$suffix.'.js', array('jquery'), GIVE_VERSION, false);
+	wp_enqueue_script('give-selector-cache');
 
-	wp_register_script( 'jquery-chosen', $js_plugins . 'chosen.jquery' . $suffix . '.js', array( 'jquery' ), GIVE_VERSION );
-	wp_enqueue_script( 'jquery-chosen' );
+	wp_register_script('jquery-chosen', $js_plugins.'chosen.jquery'.$suffix.'.js', array('jquery'), GIVE_VERSION);
+	wp_enqueue_script('jquery-chosen');
 
-	wp_register_script( 'give-accounting', $js_plugins . 'accounting' . $suffix . '.js', array( 'jquery' ), GIVE_VERSION, false );
-	wp_enqueue_script( 'give-accounting' );
+	wp_register_script('give-accounting', $js_plugins.'accounting'.$suffix.'.js', array('jquery'), GIVE_VERSION, false);
+	wp_enqueue_script('give-accounting');
 
-	wp_enqueue_script( 'wp-color-picker' );
-	wp_enqueue_script( 'jquery-ui-datepicker' );
-	wp_enqueue_script( 'thickbox' );
+	wp_enqueue_script('wp-color-picker');
+	wp_enqueue_script('jquery-ui-datepicker');
+	wp_enqueue_script('thickbox');
 
-	wp_register_script( 'give-admin-scripts', $js_dir . 'admin-scripts' . $suffix . '.js', array( 'jquery', 'jquery-ui-datepicker', 'wp-color-picker', 'jquery-query' ), GIVE_VERSION, false );
-	wp_enqueue_script( 'give-admin-scripts' );
+	wp_register_script('give-admin-scripts', $js_dir.'admin-scripts'.$suffix.'.js', array('jquery', 'jquery-ui-datepicker', 'wp-color-picker', 'jquery-query'), GIVE_VERSION, false);
+	wp_enqueue_script('give-admin-scripts');
 
-	wp_register_script( 'jquery-flot', $js_plugins . 'jquery.flot' . $suffix . '.js' );
-	wp_enqueue_script( 'jquery-flot' );
+	wp_register_script('jquery-flot', $js_plugins.'jquery.flot'.$suffix.'.js');
+	wp_enqueue_script('jquery-flot');
 
-	wp_register_script( 'give-qtip', $js_plugins . 'jquery.qtip' . $suffix . '.js', array( 'jquery' ), GIVE_VERSION, false );
-	wp_enqueue_script( 'give-qtip' );
+	wp_register_script('give-qtip', $js_plugins.'jquery.qtip'.$suffix.'.js', array('jquery'), GIVE_VERSION, false);
+	wp_enqueue_script('give-qtip');
 
-	wp_register_script( 'give-repeatable-fields', $js_plugins . 'repeatable-fields' . $suffix . '.js', array( 'jquery' ), GIVE_VERSION, false );
-	wp_enqueue_script( 'give-repeatable-fields' );
+	wp_register_script('give-repeatable-fields', $js_plugins.'repeatable-fields'.$suffix.'.js', array('jquery'), GIVE_VERSION, false);
+	wp_enqueue_script('give-repeatable-fields');
 
 	// Forms CPT Script.
-	if ( $post_type === 'give_forms' ) {
-		wp_register_script( 'give-admin-forms-scripts', $js_dir . 'admin-forms' . $suffix . '.js', array( 'jquery' ), GIVE_VERSION, false );
-		wp_enqueue_script( 'give-admin-forms-scripts' );
+	if ($post_type === 'give_forms') {
+		wp_register_script('give-admin-forms-scripts', $js_dir.'admin-forms'.$suffix.'.js', array('jquery'), GIVE_VERSION, false);
+		wp_enqueue_script('give-admin-forms-scripts');
 	}
 
 	// Settings Scripts.
-	if ( isset( $_GET['page'] ) && $_GET['page'] == 'give-settings' ) {
-		wp_register_script( 'give-admin-settings-scripts', $js_dir . 'admin-settings' . $suffix . '.js', array( 'jquery' ), GIVE_VERSION, false );
-		wp_enqueue_script( 'give-admin-settings-scripts' );
+	if (isset($_GET['page']) && $_GET['page'] == 'give-settings') {
+		wp_register_script('give-admin-settings-scripts', $js_dir.'admin-settings'.$suffix.'.js', array('jquery'), GIVE_VERSION, false);
+		wp_enqueue_script('give-admin-settings-scripts');
 	}
 
 	// Price Separators.
@@ -302,80 +302,80 @@ 
 	$decimal_separator  = give_get_price_decimal_separator();
 
 	// Localize strings & variables for JS.
-	wp_localize_script( 'give-admin-scripts', 'give_vars', array(
-		'post_id'                        => isset( $post->ID ) ? $post->ID : null,
+	wp_localize_script('give-admin-scripts', 'give_vars', array(
+		'post_id'                        => isset($post->ID) ? $post->ID : null,
 		'give_version'                   => GIVE_VERSION,
 		'thousands_separator'            => $thousand_separator,
 		'decimal_separator'              => $decimal_separator,
-		'quick_edit_warning'             => __( 'Not available for variable priced forms.', 'give' ),
-		'delete_payment'                 => __( 'Are you sure you want to delete this payment?', 'give' ),
-		'delete_payment_note'            => __( 'Are you sure you want to delete this note?', 'give' ),
-		'revoke_api_key'                 => __( 'Are you sure you want to revoke this API key?', 'give' ),
-		'regenerate_api_key'             => __( 'Are you sure you want to regenerate this API key?', 'give' ),
-		'resend_receipt'                 => __( 'Are you sure you want to resend the donation receipt?', 'give' ),
-		'disconnect_user'                => __( 'Are you sure you want to disconnect the user from this donor?', 'give' ),
-		'one_option'                     => __( 'Choose a form', 'give' ),
-		'one_or_more_option'             => __( 'Choose one or more forms', 'give' ),
-		'currency_sign'                  => give_currency_filter( '' ),
-		'currency_pos'                   => isset( $give_options['currency_position'] ) ? $give_options['currency_position'] : 'before',
-		'currency_decimals'              => give_currency_decimal_filter( give_get_price_decimals() ),
-		'batch_export_no_class'          => __( 'You must choose a method.', 'give' ),
-		'batch_export_no_reqs'           => __( 'Required fields not completed.', 'give' ),
-		'reset_stats_warn'               => __( 'Are you sure you want to reset Give? This process is <strong><em>not reversible</em></strong> and will delete all data regardless of test or live mode. Please be sure you have a recent backup before proceeding.', 'give' ),
-		'delete_test_donor'               => __( 'Are you sure you want to delete all the test donors? This process will also delete test donations as well.', 'give' ),
-		'delete_import_donor'               => __( 'Are you sure you want to delete all the imported donors? This process will also delete imported donations as well.', 'give' ),
-		'price_format_guide'             => sprintf( __( 'Please enter amount in monetary decimal ( %1$s ) format without thousand separator ( %2$s ) .', 'give' ), $decimal_separator, $thousand_separator ),
+		'quick_edit_warning'             => __('Not available for variable priced forms.', 'give'),
+		'delete_payment'                 => __('Are you sure you want to delete this payment?', 'give'),
+		'delete_payment_note'            => __('Are you sure you want to delete this note?', 'give'),
+		'revoke_api_key'                 => __('Are you sure you want to revoke this API key?', 'give'),
+		'regenerate_api_key'             => __('Are you sure you want to regenerate this API key?', 'give'),
+		'resend_receipt'                 => __('Are you sure you want to resend the donation receipt?', 'give'),
+		'disconnect_user'                => __('Are you sure you want to disconnect the user from this donor?', 'give'),
+		'one_option'                     => __('Choose a form', 'give'),
+		'one_or_more_option'             => __('Choose one or more forms', 'give'),
+		'currency_sign'                  => give_currency_filter(''),
+		'currency_pos'                   => isset($give_options['currency_position']) ? $give_options['currency_position'] : 'before',
+		'currency_decimals'              => give_currency_decimal_filter(give_get_price_decimals()),
+		'batch_export_no_class'          => __('You must choose a method.', 'give'),
+		'batch_export_no_reqs'           => __('Required fields not completed.', 'give'),
+		'reset_stats_warn'               => __('Are you sure you want to reset Give? This process is <strong><em>not reversible</em></strong> and will delete all data regardless of test or live mode. Please be sure you have a recent backup before proceeding.', 'give'),
+		'delete_test_donor'               => __('Are you sure you want to delete all the test donors? This process will also delete test donations as well.', 'give'),
+		'delete_import_donor'               => __('Are you sure you want to delete all the imported donors? This process will also delete imported donations as well.', 'give'),
+		'price_format_guide'             => sprintf(__('Please enter amount in monetary decimal ( %1$s ) format without thousand separator ( %2$s ) .', 'give'), $decimal_separator, $thousand_separator),
 		/* translators : %s: Donation form options metabox */
-		'confirm_before_remove_row_text' => __( 'Do you want to delete this level?', 'give' ),
-		'matched_success_failure_page'   => __( 'You cannot set the success and failed pages to the same page', 'give' ),
-		'dismiss_notice_text'            => __( 'Dismiss this notice.', 'give' ),
-		'search_placeholder'             => __( 'Type to search all forms', 'give' ),
-		'search_placeholder_donor'       => __( 'Type to search all donors', 'give' ),
-		'search_placeholder_country'     => __( 'Type to search all countries', 'give' ),
-		'search_placeholder_state'       => __( 'Type to search all states/provinces', 'give' ),
+		'confirm_before_remove_row_text' => __('Do you want to delete this level?', 'give'),
+		'matched_success_failure_page'   => __('You cannot set the success and failed pages to the same page', 'give'),
+		'dismiss_notice_text'            => __('Dismiss this notice.', 'give'),
+		'search_placeholder'             => __('Type to search all forms', 'give'),
+		'search_placeholder_donor'       => __('Type to search all donors', 'give'),
+		'search_placeholder_country'     => __('Type to search all countries', 'give'),
+		'search_placeholder_state'       => __('Type to search all states/provinces', 'give'),
 		'bulk_action' => array(
 			'delete'    => array(
-				'zero'     => __( 'You must choose at least one or more payments to delete.', 'give' ),
-				'single'   => __( 'Are you sure you want to permanently delete this donation?', 'give' ),
-				'multiple' => __( 'Are you sure you want to permanently delete the selected {payment_count} donations?', 'give' ),
+				'zero'     => __('You must choose at least one or more payments to delete.', 'give'),
+				'single'   => __('Are you sure you want to permanently delete this donation?', 'give'),
+				'multiple' => __('Are you sure you want to permanently delete the selected {payment_count} donations?', 'give'),
 			),
 			'resend-receipt' => array(
-				'zero'     => __( 'You must choose at least one or more recipients to resend the email receipt.', 'give' ),
-				'single'   => __( 'Are you sure you want to resend the email receipt to this recipient?', 'give' ),
-				'multiple' => __( 'Are you sure you want to resend the emails receipt to {payment_count} recipients?', 'give' ),
+				'zero'     => __('You must choose at least one or more recipients to resend the email receipt.', 'give'),
+				'single'   => __('Are you sure you want to resend the email receipt to this recipient?', 'give'),
+				'multiple' => __('Are you sure you want to resend the emails receipt to {payment_count} recipients?', 'give'),
 			),
 			'set-to-status' => array(
-				'zero'      => __( 'You must choose at least one or more donations to set status to {status}.', 'give' ),
-				'single'    => __( 'Are you sure you want to set status of this donation to {status}?', 'give' ),
-				'multiple'  => __( 'Are you sure you want to set status of {payment_count} donations to {status}?', 'give' ),
+				'zero'      => __('You must choose at least one or more donations to set status to {status}.', 'give'),
+				'single'    => __('Are you sure you want to set status of this donation to {status}?', 'give'),
+				'multiple'  => __('Are you sure you want to set status of {payment_count} donations to {status}?', 'give'),
 			),
 		),
 		'metabox_fields' => array(
 			'media' => array(
-				'button_title' => __( 'Choose Image', 'give' ),
+				'button_title' => __('Choose Image', 'give'),
 			),
 			'file' => array(
-				'button_title' => __( 'Choose File', 'give' ),
+				'button_title' => __('Choose File', 'give'),
 			)
 		),
 		'chosen' => array(
-			'no_results_msg'  => __( 'No results match {search_term}', 'give' ),
-			'ajax_search_msg' => __( 'Searching results for match {search_term}', 'give' ),
+			'no_results_msg'  => __('No results match {search_term}', 'give'),
+			'ajax_search_msg' => __('Searching results for match {search_term}', 'give'),
 		),
-		'db_update_confirmation_msg' => __( 'The following process will make updates to your site\'s database. Please create a database backup before proceeding with updates.', 'give' ),
-        'error_message' => __( 'Something went wrong kindly try again!','give' ),
+		'db_update_confirmation_msg' => __('The following process will make updates to your site\'s database. Please create a database backup before proceeding with updates.', 'give'),
+        'error_message' => __('Something went wrong kindly try again!', 'give'),
         'give_donation_import' => 'give_donation_import',
-		'setting_not_save_message' => __( 'Changes you made may not be saved.','give' ),
-	) );
+		'setting_not_save_message' => __('Changes you made may not be saved.', 'give'),
+	));
 
-	if ( function_exists( 'wp_enqueue_media' ) && version_compare( get_bloginfo( 'version' ), '3.5', '>=' ) ) {
+	if (function_exists('wp_enqueue_media') && version_compare(get_bloginfo('version'), '3.5', '>=')) {
 		// call for new media manager.
 		wp_enqueue_media();
 	}
 
 }
 
-add_action( 'admin_enqueue_scripts', 'give_load_admin_scripts', 100 );
+add_action('admin_enqueue_scripts', 'give_load_admin_scripts', 100);
 
 /**
  * Admin Give Icon
@@ -390,13 +390,13 @@ 
 	?>
 	<style type="text/css" media="screen">
 
-		<?php if ( version_compare( get_bloginfo( 'version' ), '3.8-RC', '>=' ) || version_compare( get_bloginfo( 'version' ), '3.8', '>=' ) ) { ?>
+		<?php if (version_compare(get_bloginfo('version'), '3.8-RC', '>=') || version_compare(get_bloginfo('version'), '3.8', '>=')) { ?>
 		@font-face {
 			font-family: 'give-icomoon';
-			src: url('<?php echo GIVE_PLUGIN_URL . '/assets/fonts/icomoon.eot?ngjl88'; ?>');
-			src: url('<?php echo GIVE_PLUGIN_URL . '/assets/fonts/icomoon.eot?#iefixngjl88'?>') format('embedded-opentype'),
-			url('<?php echo GIVE_PLUGIN_URL . '/assets/fonts/icomoon.woff?ngjl88'; ?>') format('woff'),
-			url('<?php echo GIVE_PLUGIN_URL . '/assets/fonts/icomoon.svg?ngjl88#icomoon'; ?>') format('svg');
+			src: url('<?php echo GIVE_PLUGIN_URL.'/assets/fonts/icomoon.eot?ngjl88'; ?>');
+			src: url('<?php echo GIVE_PLUGIN_URL.'/assets/fonts/icomoon.eot?#iefixngjl88'?>') format('embedded-opentype'),
+			url('<?php echo GIVE_PLUGIN_URL.'/assets/fonts/icomoon.woff?ngjl88'; ?>') format('woff'),
+			url('<?php echo GIVE_PLUGIN_URL.'/assets/fonts/icomoon.svg?ngjl88#icomoon'; ?>') format('svg');
 			font-weight: normal;
 			font-style: normal;
 		}
@@ -415,4 +415,4 @@ 
 	<?php
 }
 
-add_action( 'admin_head', 'give_admin_icon' );
+add_action('admin_head', 'give_admin_icon');

includes/formatting.php

Spacing +148 added lines, -151 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,7 +23,7 @@ 
  * @return mixed
  */
 function give_get_price_decimals() {
-	return apply_filters( 'give_sanitize_amount_decimals', give_get_option( 'number_decimals', 2 ) );
+	return apply_filters('give_sanitize_amount_decimals', give_get_option('number_decimals', 2));
 }
 
 /**
@@ -35,8 +35,8 @@ 
  */
 function give_get_price_thousand_separator() {
 	$give_options       = give_get_settings();
-	$thousand_separator = isset( $give_options['thousands_separator'] ) ? $give_options['thousands_separator'] : ',';
-	$thousand_separator = empty( $thousand_separator ) ? ' ' : $thousand_separator;
+	$thousand_separator = isset($give_options['thousands_separator']) ? $give_options['thousands_separator'] : ',';
+	$thousand_separator = empty($thousand_separator) ? ' ' : $thousand_separator;
 
 	return $thousand_separator;
 }
@@ -55,11 +55,10 @@ 
 	);
 
 	$thousand_separator = give_get_price_thousand_separator();
-	$default_decimal_separator = in_array( $thousand_separator, $default_decimal_separators ) ?
-		$default_decimal_separators[$thousand_separator] :
-		'.';
+	$default_decimal_separator = in_array($thousand_separator, $default_decimal_separators) ?
+		$default_decimal_separators[$thousand_separator] : '.';
 
-	$decimal_separator = give_get_option( 'decimal_separator', $default_decimal_separator );
+	$decimal_separator = give_get_option('decimal_separator', $default_decimal_separator);
 
 	return $decimal_separator;
 }
@@ -74,8 +73,8 @@ 
  *
  * @return string $amount Newly sanitized amount
  */
-function give_sanitize_amount_for_db( $number ) {
-	return give_maybe_sanitize_amount( $number, 6 );
+function give_sanitize_amount_for_db($number) {
+	return give_maybe_sanitize_amount($number, 6);
 }
 
 /**
@@ -89,41 +88,41 @@ 
  *
  * @return string $amount Newly sanitized amount
  */
-function give_maybe_sanitize_amount( $number, $dp = false, $trim_zeros = false ) {
+function give_maybe_sanitize_amount($number, $dp = false, $trim_zeros = false) {
 	$thousand_separator = give_get_price_thousand_separator();
 	$decimal_separator  = give_get_price_decimal_separator();
 
 	// Bailout.
-	if( empty( $number ) || ( ! is_numeric( $number ) && ! is_string( $number ) ) ) {
+	if (empty($number) || ( ! is_numeric($number) && ! is_string($number))) {
 		return $number;
 	}elseif (
-		( false == strpos( $number, $thousand_separator ) ) &&
-		( false === strpos( $number, $decimal_separator ) )
+		(false == strpos($number, $thousand_separator)) &&
+		(false === strpos($number, $decimal_separator))
 	) {
-		return number_format( $number, ( is_bool( $dp ) ? give_get_price_decimals() : $dp ), '.', '' );
+		return number_format($number, (is_bool($dp) ? give_get_price_decimals() : $dp), '.', '');
 	}
 
 	// Handle thousand separator as '.'
 	// Handle sanitize database values.
-	$number_parts = explode( '.', $number );
-	$is_db_sanitize_val = ( 2 === count( $number_parts ) &&
-	                        is_numeric( $number_parts[0] ) &&
-	                        is_numeric( $number_parts[1] ) &&
-	                        ( 6 === strlen( $number_parts[1] ) ) );
+	$number_parts = explode('.', $number);
+	$is_db_sanitize_val = (2 === count($number_parts) &&
+	                        is_numeric($number_parts[0]) &&
+	                        is_numeric($number_parts[1]) &&
+	                        (6 === strlen($number_parts[1])));
 
-	if( $is_db_sanitize_val ) {
+	if ($is_db_sanitize_val) {
 		// Sanitize database value.
-		return number_format( $number, ( is_bool( $dp ) ? give_get_price_decimals() : $dp ), '.', '' );
+		return number_format($number, (is_bool($dp) ? give_get_price_decimals() : $dp), '.', '');
 
 	} elseif (
 		'.' === $thousand_separator &&
-		false !== strpos( $number, $thousand_separator )
-	){
+		false !== strpos($number, $thousand_separator)
+	) {
 		// Fix point thousand separator value.
-		$number = str_replace( '.', '', $number );
+		$number = str_replace('.', '', $number);
 	}
 
-	return give_sanitize_amount( $number, $dp, $trim_zeros );
+	return give_sanitize_amount($number, $dp, $trim_zeros);
 }
 
 /**
@@ -139,67 +138,67 @@ 
  *
  * @return string $amount Newly sanitized amount
  */
-function give_sanitize_amount( $number, $dp = false, $trim_zeros = false ) {
+function give_sanitize_amount($number, $dp = false, $trim_zeros = false) {
 
 	// Bailout.
-	if ( empty( $number ) || ( ! is_numeric( $number ) && ! is_string( $number ) ) ) {
+	if (empty($number) || ( ! is_numeric($number) && ! is_string($number))) {
 		return $number;
 	}
 
 	// Remove slash from amount.
 	// If thousand or decimal separator is set to ' then in $_POST or $_GET param we will get an escaped number.
 	// To prevent notices and warning remove slash from amount/number.
-	$number = wp_unslash( $number );
+	$number = wp_unslash($number);
 
 	$thousand_separator = give_get_price_thousand_separator();
 
 	$locale   = localeconv();
-	$decimals = array( give_get_price_decimal_separator(), $locale['decimal_point'], $locale['mon_decimal_point'] );
+	$decimals = array(give_get_price_decimal_separator(), $locale['decimal_point'], $locale['mon_decimal_point']);
 
 	// Remove locale from string
-	if ( ! is_float( $number ) ) {
-		$number = str_replace( $decimals, '.', $number );
+	if ( ! is_float($number)) {
+		$number = str_replace($decimals, '.', $number);
 	}
 
 	// Remove thousand amount formatting if amount has.
 	// This condition use to add backward compatibility to version before 1.6, because before version 1.6 we were saving formatted amount to db.
 	// Do not replace thousand separator from price if it is same as decimal separator, because it will be already replace by above code.
-	if ( ! in_array( $thousand_separator, $decimals ) && ( false !== strpos( $number, $thousand_separator ) ) ) {
-		$number = str_replace( $thousand_separator, '', $number );
-	} elseif ( in_array( $thousand_separator, $decimals ) ) {
-		$number = preg_replace( '/\.(?=.*\.)/', '', $number );
+	if ( ! in_array($thousand_separator, $decimals) && (false !== strpos($number, $thousand_separator))) {
+		$number = str_replace($thousand_separator, '', $number);
+	} elseif (in_array($thousand_separator, $decimals)) {
+		$number = preg_replace('/\.(?=.*\.)/', '', $number);
 	}
 
 	// Remove non numeric entity before decimal separator.
-	$number     = preg_replace( '/[^0-9\.]/', '', $number );
+	$number     = preg_replace('/[^0-9\.]/', '', $number);
 	$default_dp = give_get_price_decimals();
 
 	// Reset negative amount to zero.
-	if ( 0 > $number ) {
-		$number = number_format( 0, $default_dp, '.' );
+	if (0 > $number) {
+		$number = number_format(0, $default_dp, '.');
 	}
 
 	// If number does not have decimal then add number of decimals to it.
 	if (
-		false === strpos( $number, '.' )
-		|| ( $default_dp > strlen( substr( $number, strpos( $number, '.' ) + 1 ) ) )
+		false === strpos($number, '.')
+		|| ($default_dp > strlen(substr($number, strpos($number, '.') + 1)))
 	) {
-		$number = number_format( $number, $default_dp, '.', '' );
+		$number = number_format($number, $default_dp, '.', '');
 	}
 
 	// Format number by custom number of decimals.
-	if ( false !== $dp ) {
-		$dp     = intval( is_bool( $dp ) ? $default_dp : $dp );
-		$dp     = apply_filters( 'give_sanitize_amount_decimals', $dp, $number );
-		$number = number_format( floatval( $number ), $dp, '.', '' );
+	if (false !== $dp) {
+		$dp     = intval(is_bool($dp) ? $default_dp : $dp);
+		$dp     = apply_filters('give_sanitize_amount_decimals', $dp, $number);
+		$number = number_format(floatval($number), $dp, '.', '');
 	}
 
 	// Trim zeros.
-	if ( $trim_zeros && strstr( $number, '.' ) ) {
-		$number = rtrim( rtrim( $number, '0' ), '.' );
+	if ($trim_zeros && strstr($number, '.')) {
+		$number = rtrim(rtrim($number, '0'), '.');
 	}
 
-	return apply_filters( 'give_sanitize_amount', $number );
+	return apply_filters('give_sanitize_amount', $number);
 }
 
 /**
@@ -212,10 +211,10 @@ 
  *
  * @return string $amount   Newly formatted amount or Price Not Available
  */
-function give_format_amount( $amount, $args = array() ) {
+function give_format_amount($amount, $args = array()) {
 	// Backward compatibility.
-	if( is_bool( $args ) ) {
-		$args = array( 'decimal' => $args );
+	if (is_bool($args)) {
+		$args = array('decimal' => $args);
 	}
 
 	$default_args = array(
@@ -224,60 +223,59 @@ 
 		'currency' => give_get_currency(),
 	);
 
-	$args = wp_parse_args( $args, $default_args );
+	$args = wp_parse_args($args, $default_args);
 
 	$formatted     = 0;
 	$thousands_sep = give_get_price_thousand_separator();
 	$decimal_sep   = give_get_price_decimal_separator();
-	$decimals      = ! empty( $args['decimal'] ) ? give_get_price_decimals() : 0;
+	$decimals      = ! empty($args['decimal']) ? give_get_price_decimals() : 0;
 	$currency      = $args['currency'];
 
-	if ( ! empty( $amount ) ) {
+	if ( ! empty($amount)) {
 		// Sanitize amount before formatting.
-		$amount = ! empty( $args['sanitize'] ) ?
-			give_maybe_sanitize_amount( $amount, $decimals ) :
-			number_format( $amount, $decimals, '.', '' );
+		$amount = ! empty($args['sanitize']) ?
+			give_maybe_sanitize_amount($amount, $decimals) : number_format($amount, $decimals, '.', '');
 
-		switch ( $currency ) {
+		switch ($currency) {
 			case 'INR':
 				$decimal_amount = '';
 
 				// Extract decimals from amount
-				if ( ( $pos = strpos( $amount, '.' ) ) !== false ) {
-					if ( ! empty( $decimals ) ) {
-						$decimal_amount = substr( round( substr( $amount, $pos ), $decimals ), 1 );
-						$amount         = substr( $amount, 0, $pos );
-
-						if ( ! $decimal_amount ) {
-							$decimal_amount = substr( '.0000000000', 0, ( $decimals + 1 ) );
-						} elseif ( ( $decimals + 1 ) > strlen( $decimal_amount ) ) {
-							$decimal_amount = substr( "{$decimal_amount}000000000", 0, ( $decimals + 1 ) );
+				if (($pos = strpos($amount, '.')) !== false) {
+					if ( ! empty($decimals)) {
+						$decimal_amount = substr(round(substr($amount, $pos), $decimals), 1);
+						$amount         = substr($amount, 0, $pos);
+
+						if ( ! $decimal_amount) {
+							$decimal_amount = substr('.0000000000', 0, ($decimals + 1));
+						} elseif (($decimals + 1) > strlen($decimal_amount)) {
+							$decimal_amount = substr("{$decimal_amount}000000000", 0, ($decimals + 1));
 						}
 
 					} else {
-						$amount = number_format( $amount, $decimals, $decimal_sep, '' );
+						$amount = number_format($amount, $decimals, $decimal_sep, '');
 					}
 				}
 
 				// Extract last 3 from amount
-				$result = substr( $amount, - 3 );
-				$amount = substr( $amount, 0, - 3 );
+				$result = substr($amount, - 3);
+				$amount = substr($amount, 0, - 3);
 
 				// Apply digits 2 by 2
-				while ( strlen( $amount ) > 0 ) {
-					$result = substr( $amount, - 2 ) . $thousands_sep . $result;
-					$amount = substr( $amount, 0, - 2 );
+				while (strlen($amount) > 0) {
+					$result = substr($amount, - 2).$thousands_sep.$result;
+					$amount = substr($amount, 0, - 2);
 				}
 
-				$formatted = $result . $decimal_amount;
+				$formatted = $result.$decimal_amount;
 				break;
 
 			default:
-				$formatted = number_format( $amount, $decimals, $decimal_sep, $thousands_sep );
+				$formatted = number_format($amount, $decimals, $decimal_sep, $thousands_sep);
 		}
 	}
 
-	return apply_filters( 'give_format_amount', $formatted, $amount, $decimals, $decimal_sep, $thousands_sep, $currency, $args );
+	return apply_filters('give_format_amount', $formatted, $amount, $decimals, $decimal_sep, $thousands_sep, $currency, $args);
 }
 
 
@@ -295,51 +293,51 @@ 
  *
  * @return float|string  formatted amount number with large number names.
  */
-function give_human_format_large_amount( $amount, $args = array() ) {
+function give_human_format_large_amount($amount, $args = array()) {
 	$default_args = array(
 		'currency' => give_get_currency(),
 	);
 
-	$args = wp_parse_args( $args, $default_args );
+	$args = wp_parse_args($args, $default_args);
 
 	// Get thousand separator.
 	$thousands_sep = give_get_price_thousand_separator();
 
 	// Sanitize amount.
-	$sanitize_amount = give_maybe_sanitize_amount( $amount );
+	$sanitize_amount = give_maybe_sanitize_amount($amount);
 
 	// Explode amount to calculate name of large numbers.
-	$amount_array = explode( $thousands_sep, $amount );
+	$amount_array = explode($thousands_sep, $amount);
 
 	// Calculate amount parts count.
-	$amount_count_parts = count( $amount_array );
+	$amount_count_parts = count($amount_array);
 
 	// Human format amount (default).
 	$human_format_amount = $amount;
 
-	switch ( $args['currency'] ) {
+	switch ($args['currency']) {
 		case 'INR':
 			// Calculate large number formatted amount.
-			if ( 4 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s arab', 'give' ), round( ( $sanitize_amount / 1000000000 ), 2 ) );
-			} elseif ( 3 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s crore', 'give' ), round( ( $sanitize_amount / 10000000 ), 2 ) );
-			} elseif ( 2 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s lakh', 'give' ), round( ( $sanitize_amount / 100000 ), 2 ) );
+			if (4 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s arab', 'give'), round(($sanitize_amount / 1000000000), 2));
+			} elseif (3 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s crore', 'give'), round(($sanitize_amount / 10000000), 2));
+			} elseif (2 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s lakh', 'give'), round(($sanitize_amount / 100000), 2));
 			}
 			break;
 		default:
 			// Calculate large number formatted amount.
-			if ( 4 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s trillion', 'give' ), round( ( $sanitize_amount / 1000000000000 ), 2 ) );
-			} elseif ( 3 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s billion', 'give' ), round( ( $sanitize_amount / 1000000000 ), 2 ) );
-			} elseif ( 2 < $amount_count_parts ) {
-				$human_format_amount = sprintf( esc_html__( '%s million', 'give' ), round( ( $sanitize_amount / 1000000 ), 2 ) );
+			if (4 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s trillion', 'give'), round(($sanitize_amount / 1000000000000), 2));
+			} elseif (3 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s billion', 'give'), round(($sanitize_amount / 1000000000), 2));
+			} elseif (2 < $amount_count_parts) {
+				$human_format_amount = sprintf(esc_html__('%s million', 'give'), round(($sanitize_amount / 1000000), 2));
 			}
 	}
 
-	return apply_filters( 'give_human_format_large_amount', $human_format_amount, $amount, $sanitize_amount );
+	return apply_filters('give_human_format_large_amount', $human_format_amount, $amount, $sanitize_amount);
 }
 
 /**
@@ -353,17 +351,16 @@ 
  *
  * @return string $amount Newly formatted amount or Price Not Available
  */
-function give_format_decimal( $amount, $dp = false, $sanitize = true ) {
+function give_format_decimal($amount, $dp = false, $sanitize = true) {
 	$decimal_separator = give_get_price_decimal_separator();
 	$formatted_amount  = $sanitize ?
-		give_maybe_sanitize_amount( $amount, $dp ) :
-		number_format( $amount, ( is_bool( $dp ) ? give_get_price_decimals() : $dp ), '.', '' );
+		give_maybe_sanitize_amount($amount, $dp) : number_format($amount, (is_bool($dp) ? give_get_price_decimals() : $dp), '.', '');
 
-	if ( false !== strpos( $formatted_amount, '.' ) ) {
-		$formatted_amount = str_replace( '.', $decimal_separator, $formatted_amount );
+	if (false !== strpos($formatted_amount, '.')) {
+		$formatted_amount = str_replace('.', $decimal_separator, $formatted_amount);
 	}
 
-	return apply_filters( 'give_format_decimal', $formatted_amount, $amount, $decimal_separator );
+	return apply_filters('give_format_decimal', $formatted_amount, $amount, $decimal_separator);
 }
 
 /**
@@ -377,24 +374,24 @@ 
  *
  * @return mixed|string
  */
-function give_currency_filter( $price = '', $currency = '', $decode_currency = false ) {
+function give_currency_filter($price = '', $currency = '', $decode_currency = false) {
 
-	if ( empty( $currency ) || ! array_key_exists( (string) $currency, give_get_currencies() ) ) {
+	if (empty($currency) || ! array_key_exists((string) $currency, give_get_currencies())) {
 		$currency = give_get_currency();
 	}
 
-	$position = give_get_option( 'currency_position', 'before' );
+	$position = give_get_option('currency_position', 'before');
 
 	$negative = $price < 0;
 
-	if ( $negative ) {
+	if ($negative) {
 		// Remove proceeding "-".
-		$price = substr( $price, 1 );
+		$price = substr($price, 1);
 	}
 
-	$symbol = give_currency_symbol( $currency, $decode_currency );
+	$symbol = give_currency_symbol($currency, $decode_currency);
 
-	switch ( $currency ) :
+	switch ($currency) :
 		case 'GBP' :
 		case 'BRL' :
 		case 'EUR' :
@@ -423,13 +420,13 @@ 
 		case 'MAD' :
 		case 'KRW' :
 		case 'ZAR' :
-			$formatted = ( 'before' === $position ? $symbol . $price : $price . $symbol );
+			$formatted = ('before' === $position ? $symbol.$price : $price.$symbol);
 			break;
 		case 'NOK' :
-			$formatted = ( 'before' === $position ? $symbol . ' ' . $price : $price . ' ' . $symbol );
+			$formatted = ('before' === $position ? $symbol.' '.$price : $price.' '.$symbol);
 			break;
 		default :
-			$formatted = ( 'before' === $position ? $currency . ' ' . $price : $price . ' ' . $currency );
+			$formatted = ('before' === $position ? $currency.' '.$price : $price.' '.$currency);
 			break;
 	endswitch;
 
@@ -444,11 +441,11 @@ 
 	 *           and if currency is USD and currency position is after then
 	 *           filter name will be give_usd_currency_filter_after
 	 */
-	$formatted = apply_filters( 'give_' . strtolower( $currency ) . "_currency_filter_{$position}", $formatted, $currency, $price );
+	$formatted = apply_filters('give_'.strtolower($currency)."_currency_filter_{$position}", $formatted, $currency, $price);
 
-	if ( $negative ) {
+	if ($negative) {
 		// Prepend the minus sign before the currency sign.
-		$formatted = '-' . $formatted;
+		$formatted = '-'.$formatted;
 	}
 
 	return $formatted;
@@ -464,21 +461,21 @@ 
  */
 function give_currency_decimal_filter() {
 
-	remove_filter( 'give_sanitize_amount_decimals', 'give_currency_decimal_filter' );
+	remove_filter('give_sanitize_amount_decimals', 'give_currency_decimal_filter');
 
 	// Set default number of decimals.
 	$decimals = give_get_price_decimals();
 
-	add_filter( 'give_sanitize_amount_decimals', 'give_currency_decimal_filter' );
+	add_filter('give_sanitize_amount_decimals', 'give_currency_decimal_filter');
 
 	// Get number of decimals with backward compatibility ( version < 1.6 )
-	if ( 1 <= func_num_args() ) {
-		$decimals = ( false === func_get_arg( 0 ) ? $decimals : absint( func_get_arg( 0 ) ) );
+	if (1 <= func_num_args()) {
+		$decimals = (false === func_get_arg(0) ? $decimals : absint(func_get_arg(0)));
 	}
 
 	$currency = give_get_currency();
 
-	switch ( $currency ) {
+	switch ($currency) {
 		case 'RIAL' :
 		case 'JPY' :
 		case 'TWD' :
@@ -488,11 +485,11 @@ 
 			break;
 	}
 
-	return apply_filters( 'give_currency_decimal_count', $decimals, $currency );
+	return apply_filters('give_currency_decimal_count', $decimals, $currency);
 }
 
-add_filter( 'give_sanitize_amount_decimals', 'give_currency_decimal_filter' );
-add_filter( 'give_format_amount_decimals', 'give_currency_decimal_filter' );
+add_filter('give_sanitize_amount_decimals', 'give_currency_decimal_filter');
+add_filter('give_format_amount_decimals', 'give_currency_decimal_filter');
 
 
 /**
@@ -504,7 +501,7 @@ 
  *
  * @return string                  Date format string
  */
-function give_date_format( $date_context = '' ) {
+function give_date_format($date_context = '') {
 	/**
 	 * Filter the date context
 	 *
@@ -525,19 +522,19 @@ 
 	 *
 	 *    }
 	 */
-	$date_format_contexts = apply_filters( 'give_date_format_contexts', array() );
+	$date_format_contexts = apply_filters('give_date_format_contexts', array());
 
 	// Set date format to default date format.
-	$date_format = get_option( 'date_format' );
+	$date_format = get_option('date_format');
 
 	// Update date format if we have non empty date format context array and non empty date format string for that context.
-	if ( $date_context && ! empty( $date_format_contexts ) && array_key_exists( $date_context, $date_format_contexts ) ) {
-		$date_format = ! empty( $date_format_contexts[ $date_context ] )
-			? $date_format_contexts[ $date_context ]
+	if ($date_context && ! empty($date_format_contexts) && array_key_exists($date_context, $date_format_contexts)) {
+		$date_format = ! empty($date_format_contexts[$date_context])
+			? $date_format_contexts[$date_context]
 			: $date_format;
 	}
 
-	return apply_filters( 'give_date_format', $date_format );
+	return apply_filters('give_date_format', $date_format);
 }
 
 /**
@@ -551,8 +548,8 @@ 
  *
  * @return string
  */
-function give_get_cache_key( $action, $query_args ) {
-	return Give_Cache::get_key( $action, $query_args );
+function give_get_cache_key($action, $query_args) {
+	return Give_Cache::get_key($action, $query_args);
 }
 
 /**
@@ -565,11 +562,11 @@ 
  *
  * @return string|array
  */
-function give_clean( $var ) {
-	if ( is_array( $var ) ) {
-		return array_map( 'give_clean', $var );
+function give_clean($var) {
+	if (is_array($var)) {
+		return array_map('give_clean', $var);
 	} else {
-		return is_scalar( $var ) ? sanitize_text_field( $var ) : $var;
+		return is_scalar($var) ? sanitize_text_field($var) : $var;
 	}
 }
 
@@ -582,10 +579,10 @@ 
  *
  * @return int
  */
-function give_let_to_num( $size ) {
-	$l   = substr( $size, - 1 );
-	$ret = substr( $size, 0, - 1 );
-	switch ( strtoupper( $l ) ) {
+function give_let_to_num($size) {
+	$l   = substr($size, - 1);
+	$ret = substr($size, 0, - 1);
+	switch (strtoupper($l)) {
 		case 'P':
 			$ret *= 1024;
 		case 'T':
@@ -610,19 +607,19 @@ 
  * @param int   $action
  * @param array $wp_die_args
  */
-function give_validate_nonce( $nonce, $action = - 1, $wp_die_args = array() ) {
+function give_validate_nonce($nonce, $action = - 1, $wp_die_args = array()) {
 
 	$default_wp_die_args = array(
-		'message' => esc_html__( 'Nonce verification has failed.', 'give' ),
-		'title'   => esc_html__( 'Error', 'give' ),
+		'message' => esc_html__('Nonce verification has failed.', 'give'),
+		'title'   => esc_html__('Error', 'give'),
 		'args'    => array(
 			'response' => 403,
 		),
 	);
 
-	$wp_die_args = wp_parse_args( $wp_die_args, $default_wp_die_args );
+	$wp_die_args = wp_parse_args($wp_die_args, $default_wp_die_args);
 
-	if ( ! wp_verify_nonce( $nonce, $action ) ) {
+	if ( ! wp_verify_nonce($nonce, $action)) {
 		wp_die(
 			$wp_die_args['message'],
 			$wp_die_args['title'],
@@ -644,23 +641,23 @@ 
  *
  * @return mixed
  */
-function give_check_variable( $variable, $conditional = '', $default = false ) {
+function give_check_variable($variable, $conditional = '', $default = false) {
 
-	switch ( $conditional ) {
+	switch ($conditional) {
 		case 'isset_empty':
-			$variable = ( isset( $variable ) && ! empty( $variable ) ) ? $variable : $default;
+			$variable = (isset($variable) && ! empty($variable)) ? $variable : $default;
 			break;
 
 		case 'empty':
-			$variable = ! empty( $variable ) ? $variable : $default;
+			$variable = ! empty($variable) ? $variable : $default;
 			break;
 
 		case 'null':
-			$variable = ! is_null( $variable ) ? $variable : $default;
+			$variable = ! is_null($variable) ? $variable : $default;
 			break;
 
 		default:
-			$variable = isset( $variable ) ? $variable : $default;
+			$variable = isset($variable) ? $variable : $default;
 
 	}