WordImpress / Give

includes/admin/tools/import/class-give-import-core-settings.php

Spacing +75 added lines, -75 removed lines

@@ -11,11 +11,11 @@ 
  * @since       1.8.17
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Import_Core_Settings' ) ) {
+if ( ! class_exists('Give_Import_Core_Settings')) {
 
 	/**
 	 * Give_Import_Core_Settings.
@@ -77,7 +77,7 @@ 
 		 * @return static
 		 */
 		public static function get_instance() {
-			if ( null === static::$instance ) {
+			if (null === static::$instance) {
 				self::$instance = new static();
 			}
 
@@ -104,26 +104,26 @@ 
 		 * @return void
 		 */
 		private function setup_hooks() {
-			if ( ! $this->is_donations_import_page() ) {
+			if ( ! $this->is_donations_import_page()) {
 				return;
 			}
 
 			// Do not render main import tools page.
-			remove_action( 'give_admin_field_tools_import', array( 'Give_Settings_Import', 'render_import_field', ) );
+			remove_action('give_admin_field_tools_import', array('Give_Settings_Import', 'render_import_field',));
 
 			// Render donation import page
-			add_action( 'give_admin_field_tools_import', array( $this, 'render_page' ) );
+			add_action('give_admin_field_tools_import', array($this, 'render_page'));
 
 			// Print the HTML.
-			add_action( 'give_tools_import_core_settings_form_start', array( $this, 'html' ), 10 );
+			add_action('give_tools_import_core_settings_form_start', array($this, 'html'), 10);
 
 			// Run when form submit.
-			add_action( 'give-tools_save_import', array( $this, 'save' ) );
+			add_action('give-tools_save_import', array($this, 'save'));
 
-			add_action( 'give-tools_update_notices', array( $this, 'update_notices' ), 11, 1 );
+			add_action('give-tools_update_notices', array($this, 'update_notices'), 11, 1);
 
 			// Used to add submit button.
-			add_action( 'give_tools_import_core_settings_form_end', array( $this, 'submit' ), 10 );
+			add_action('give_tools_import_core_settings_form_end', array($this, 'submit'), 10);
 		}
 
 		/**
@@ -135,9 +135,9 @@ 
 		 *
 		 * @return mixed
 		 */
-		public function update_notices( $messages ) {
-			if ( ! empty( $_GET['tab'] ) && 'import' === give_clean( $_GET['tab'] ) ) {
-				unset( $messages['give-setting-updated'] );
+		public function update_notices($messages) {
+			if ( ! empty($_GET['tab']) && 'import' === give_clean($_GET['tab'])) {
+				unset($messages['give-setting-updated']);
 			}
 
 			return $messages;
@@ -149,7 +149,7 @@ 
 		 * @since 1.8.17
 		 */
 		public function submit() {
-			wp_nonce_field( 'give-save-settings', '_give-save-settings' );
+			wp_nonce_field('give-save-settings', '_give-save-settings');
 			?>
 			<input type="hidden" class="import-step" id="import-step" name="step" value="<?php echo $this->get_step(); ?>"/>
 			<input type="hidden" class="importer-type" value="<?php echo $this->importer_type; ?>"/>
@@ -169,11 +169,11 @@ 
 			?>
 			<section>
 				<table
-					class="widefat export-options-table give-table <?php echo "step-{$step}"; ?> <?php echo( 1 === $step && ! empty( $this->is_json_valid ) ? 'give-hidden' : '' ); ?> "
+					class="widefat export-options-table give-table <?php echo "step-{$step}"; ?> <?php echo(1 === $step && ! empty($this->is_json_valid) ? 'give-hidden' : ''); ?> "
 					id="<?php echo "step-{$step}"; ?>">
 					<tbody>
 					<?php
-					switch ( $step ) {
+					switch ($step) {
 						case 1:
 							$this->render_upload_html();
 							break;
@@ -200,14 +200,14 @@ 
 		public function import_success() {
 			// Imported successfully
 
-			$success = (bool) ( isset( $_GET['success'] ) ? give_clean( $_GET['success'] ) : false );
-			$undo = (bool) ( isset( $_GET['undo'] ) ? give_clean( $_GET['undo'] ) : false );
+			$success = (bool) (isset($_GET['success']) ? give_clean($_GET['success']) : false);
+			$undo = (bool) (isset($_GET['undo']) ? give_clean($_GET['undo']) : false);
 			$query_arg_setting = array(
 				'post_type' => 'give_forms',
 				'page'      => 'give-settings',
 			);
 
-			if ( $undo ) {
+			if ($undo) {
 				$success = false;
 			}
 
@@ -220,30 +220,30 @@ 
 				'undo'      => 'true',
 			);
 
-			$title = __( 'Settings Importing Completed!', 'give' );
-			if ( $success ) {
+			$title = __('Settings Importing Completed!', 'give');
+			if ($success) {
 				$query_arg_success['undo'] = '1';
 				$query_arg_success['step'] = '3';
 				$query_arg_success['success'] = '1';
-				$text = __( 'Undo Importing', 'give' );
+				$text = __('Undo Importing', 'give');
 			} else {
-				if ( $undo ) {
-					$host_give_options = get_option( 'give_settings_old', array() );
-					update_option( 'give_settings', $host_give_options );
-					$title = __( 'Undo of Setting Imported Completed!', 'give' );
+				if ($undo) {
+					$host_give_options = get_option('give_settings_old', array());
+					update_option('give_settings', $host_give_options);
+					$title = __('Undo of Setting Imported Completed!', 'give');
 				} else {
-					$title = __( 'Failed to import', 'give' );
+					$title = __('Failed to import', 'give');
 				}
 
-				$text = __( 'Importing Again', 'give' );
+				$text = __('Importing Again', 'give');
 			}
 			?>
 			<tr valign="top" class="give-import-dropdown">
 				<th colspan="2">
 					<h2><?php echo $title; ?></h2>
 					<p>
-						<a class="button button-large button-secondary" href="<?php echo add_query_arg( $query_arg_success, admin_url( 'edit.php' ) ); ?>"><?php echo $text; ?></a>
-						<a class="button button-large button-secondary" href="<?php echo add_query_arg( $query_arg_setting, admin_url( 'edit.php' ) ); ?>"><?php echo __( 'View Settings', 'give' ); ?></a>
+						<a class="button button-large button-secondary" href="<?php echo add_query_arg($query_arg_success, admin_url('edit.php')); ?>"><?php echo $text; ?></a>
+						<a class="button button-large button-secondary" href="<?php echo add_query_arg($query_arg_setting, admin_url('edit.php')); ?>"><?php echo __('View Settings', 'give'); ?></a>
 					</p>
 				</th>
 			</tr>
@@ -256,14 +256,14 @@ 
 		 * @since 1.8.17
 		 */
 		public function start_import() {
-			$type      = ( ! empty( $_GET['type'] ) ? give_clean( $_GET['type'] ) : 'replace' );
-			$file_name = ( ! empty( $_GET['file_name'] ) ? give_clean( $_GET['file_name'] ) : '' );
+			$type      = ( ! empty($_GET['type']) ? give_clean($_GET['type']) : 'replace');
+			$file_name = ( ! empty($_GET['file_name']) ? give_clean($_GET['file_name']) : '');
 
 			?>
 			<tr valign="top" class="give-import-dropdown">
 				<th colspan="2">
-					<h2 id="give-import-title"><?php esc_html_e( 'Importing', 'give' ) ?></h2>
-					<p class="give-field-description"><?php esc_html_e( 'Your settings are now being imported...', 'give' ) ?></p>
+					<h2 id="give-import-title"><?php esc_html_e('Importing', 'give') ?></h2>
+					<p class="give-field-description"><?php esc_html_e('Your settings are now being imported...', 'give') ?></p>
 				</th>
 			</tr>
 
@@ -290,14 +290,14 @@ 
 			$step = $this->get_step();
 			?>
 			<ol class="give-progress-steps">
-				<li class="<?php echo( 1 === $step ? 'active' : '' ); ?>">
-					<?php esc_html_e( 'Upload JSON file', 'give' ); ?>
+				<li class="<?php echo(1 === $step ? 'active' : ''); ?>">
+					<?php esc_html_e('Upload JSON file', 'give'); ?>
 				</li>
-				<li class="<?php echo( 2 === $step ? 'active' : '' ); ?>">
-					<?php esc_html_e( 'Import', 'give' ); ?>
+				<li class="<?php echo(2 === $step ? 'active' : ''); ?>">
+					<?php esc_html_e('Import', 'give'); ?>
 				</li>
-				<li class="<?php echo( 3 === $step ? 'active' : '' ); ?>">
-					<?php esc_html_e( 'Done!', 'give' ); ?>
+				<li class="<?php echo(3 === $step ? 'active' : ''); ?>">
+					<?php esc_html_e('Done!', 'give'); ?>
 				</li>
 			</ol>
 			<?php
@@ -311,14 +311,14 @@ 
 		 * @return int $step on which step doest the import is on.
 		 */
 		public function get_step() {
-			$step    = (int) ( isset( $_REQUEST['step'] ) ? give_clean( $_REQUEST['step'] ) : 0 );
+			$step    = (int) (isset($_REQUEST['step']) ? give_clean($_REQUEST['step']) : 0);
 			$on_step = 1;
 
-			if ( empty( $step ) || 1 === $step ) {
+			if (empty($step) || 1 === $step) {
 				$on_step = 1;
-			} elseif ( 2 === $step ) {
+			} elseif (2 === $step) {
 				$on_step = 2;
-			} elseif ( 3 === $step ) {
+			} elseif (3 === $step) {
 				$on_step = 3;
 			}
 
@@ -331,7 +331,7 @@ 
 		 * @since 1.8.17
 		 */
 		public function render_page() {
-			include_once GIVE_PLUGIN_DIR . 'includes/admin/tools/views/html-admin-page-import-core-settings.php';
+			include_once GIVE_PLUGIN_DIR.'includes/admin/tools/views/html-admin-page-import-core-settings.php';
 		}
 
 		/**
@@ -343,28 +343,28 @@ 
 		 * @return void
 		 */
 		public function render_upload_html() {
-			$json = ( isset( $_POST['json'] ) ? give_clean( $_POST['json'] ) : '' );
-			$type = ( isset( $_POST['type'] ) ? give_clean( $_POST['type'] ) : 'merge' );
+			$json = (isset($_POST['json']) ? give_clean($_POST['json']) : '');
+			$type = (isset($_POST['type']) ? give_clean($_POST['type']) : 'merge');
 			$step = $this->get_step();
 
 			?>
 			<tr valign="top">
 				<th colspan="2">
-					<h2 id="give-import-title"><?php esc_html_e( 'Import Core Settings from a JSON file', 'give' ) ?></h2>
-					<p class="give-field-description"><?php esc_html_e( 'This tool allows you to import Give settings from another Give installation. Settings imported contain data from Give core as well as any of our Premium Add-ons.', 'give' ) ?></p>
+					<h2 id="give-import-title"><?php esc_html_e('Import Core Settings from a JSON file', 'give') ?></h2>
+					<p class="give-field-description"><?php esc_html_e('This tool allows you to import Give settings from another Give installation. Settings imported contain data from Give core as well as any of our Premium Add-ons.', 'give') ?></p>
 				</th>
 			</tr>
 
 			<tr valign="top">
 				<th scope="row" class="titledesc">
-					<label for="json"><?php esc_html_e( 'Choose a JSON file:', 'give' ) ?></label>
+					<label for="json"><?php esc_html_e('Choose a JSON file:', 'give') ?></label>
 				</th>
 				<td class="give-forminp">
 					<div class="give-field-wrap">
 						<label for="json">
 							<input type="file" name="json" class="give-upload-json-file" value="<?php echo $json; ?>"
 							       accept=".json">
-							<p class="give-field-description"><?php esc_html_e( 'The file type must be JSON.', 'give' )?></p>
+							<p class="give-field-description"><?php esc_html_e('The file type must be JSON.', 'give')?></p>
 						</label>
 					</div>
 				</td>
@@ -373,21 +373,21 @@ 
 			$settings = array(
 				array(
 					'id'          => 'type',
-					'name'        => __( 'Merge Type:', 'give' ),
-					'description' => __( 'Select "Merge" to retain existing settings, or "Replace" to overwrite with the settings from the JSON file', 'give' ),
+					'name'        => __('Merge Type:', 'give'),
+					'description' => __('Select "Merge" to retain existing settings, or "Replace" to overwrite with the settings from the JSON file', 'give'),
 					'default'     => $type,
 					'type'        => 'radio_inline',
 					'options'     => array(
-						'merge'   => __( 'Merge', 'give' ),
-						'replace' => __( 'Replace', 'give' ),
+						'merge'   => __('Merge', 'give'),
+						'replace' => __('Replace', 'give'),
 					),
 				),
 			);
 
-			$settings = apply_filters( 'give_import_core_setting_html', $settings );
+			$settings = apply_filters('give_import_core_setting_html', $settings);
 
-			if ( empty( $this->is_json_valid ) ) {
-				Give_Admin_Settings::output_fields( $settings, 'give_settings' );
+			if (empty($this->is_json_valid)) {
+				Give_Admin_Settings::output_fields($settings, 'give_settings');
 				?>
 				<tr valign="top">
 					<th></th>
@@ -395,7 +395,7 @@ 
 						<input type="submit"
 						       class="button button-primary button-large button-secondary <?php echo "step-{$step}"; ?>"
 						       id="recount-stats-submit"
-						       value="<?php esc_attr_e( 'Submit', 'give' ); ?>"/>
+						       value="<?php esc_attr_e('Submit', 'give'); ?>"/>
 					</th>
 				</tr>
 				<?php
@@ -417,20 +417,20 @@ 
 			$step = $this->get_step();
 
 			// Validation for first step.
-			if ( 1 === $step ) {
-				$type          = ( ! empty( $_REQUEST['type'] ) ? give_clean( $_REQUEST['type'] ) : 'replace' );
+			if (1 === $step) {
+				$type          = ( ! empty($_REQUEST['type']) ? give_clean($_REQUEST['type']) : 'replace');
 				$core_settings = self::upload_widget_settings_file();
-				if ( ! empty( $core_settings['error'] ) ) {
-					Give_Admin_Settings::add_error( 'give-import-csv', __( 'Please upload a valid JSON settings file.', 'give' ) );
+				if ( ! empty($core_settings['error'])) {
+					Give_Admin_Settings::add_error('give-import-csv', __('Please upload a valid JSON settings file.', 'give'));
 				} else {
-					$file_path = explode( '/', $core_settings['file'] );
-					$count     = ( count( $file_path ) - 1 );
-					$url       = give_import_page_url( (array) apply_filters( 'give_import_core_settings_importing_url', array(
+					$file_path = explode('/', $core_settings['file']);
+					$count     = (count($file_path) - 1);
+					$url       = give_import_page_url((array) apply_filters('give_import_core_settings_importing_url', array(
 						'step'          => '2',
 						'importer-type' => $this->importer_type,
 						'type'          => $type,
-						'file_name'     => $file_path[ $count ],
-					) ) );
+						'file_name'     => $file_path[$count],
+					)));
 
 
 					$this->is_json_valid = $url;
@@ -445,7 +445,7 @@ 
 		 * @return bool
 		 */
 		private function is_donations_import_page() {
-			return 'import' === give_get_current_setting_tab() && isset( $_GET['importer-type'] ) && $this->importer_type === give_clean( $_GET['importer-type'] );
+			return 'import' === give_get_current_setting_tab() && isset($_GET['importer-type']) && $this->importer_type === give_clean($_GET['importer-type']);
 		}
 
 		/**
@@ -454,14 +454,14 @@ 
 		 */
 		public static function upload_widget_settings_file() {
 			$upload = false;
-			if ( isset( $_FILES['json'] ) ) {
-				add_filter( 'upload_mimes', array( __CLASS__, 'json_upload_mimes' ) );
+			if (isset($_FILES['json'])) {
+				add_filter('upload_mimes', array(__CLASS__, 'json_upload_mimes'));
 
-				$upload = wp_handle_upload( $_FILES['json'], array( 'test_form' => false ) );
+				$upload = wp_handle_upload($_FILES['json'], array('test_form' => false));
 
-				remove_filter( 'upload_mimes', array( __CLASS__, 'json_upload_mimes' ) );
+				remove_filter('upload_mimes', array(__CLASS__, 'json_upload_mimes'));
 			} else {
-				Give_Admin_Settings::add_error( 'give-import-csv', __( 'Please upload or provide a valid JSON file.', 'give' ) );
+				Give_Admin_Settings::add_error('give-import-csv', __('Please upload or provide a valid JSON file.', 'give'));
 			}
 
 			return $upload;
@@ -472,7 +472,7 @@ 
 		 *
 		 * @param array $existing_mimes
 		 */
-		public static function json_upload_mimes( $existing_mimes = array() ) {
+		public static function json_upload_mimes($existing_mimes = array()) {
 			$existing_mimes['json'] = 'application/json';
 
 			return $existing_mimes;

includes/admin/dashboard-widgets.php

Spacing +32 added lines, -32 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,12 +21,12 @@ 
  * @return void
  */
 function give_register_dashboard_widgets() {
-	if ( current_user_can( apply_filters( 'give_dashboard_stats_cap', 'view_give_reports' ) ) ) {
-		wp_add_dashboard_widget( 'give_dashboard_sales', __( 'Give: Donation Statistics', 'give' ), 'give_dashboard_sales_widget' );
+	if (current_user_can(apply_filters('give_dashboard_stats_cap', 'view_give_reports'))) {
+		wp_add_dashboard_widget('give_dashboard_sales', __('Give: Donation Statistics', 'give'), 'give_dashboard_sales_widget');
 	}
 }
 
-add_action( 'wp_dashboard_setup', 'give_register_dashboard_widgets', 10 );
+add_action('wp_dashboard_setup', 'give_register_dashboard_widgets', 10);
 
 /**
  * Sales Summary Dashboard Widget
@@ -38,7 +38,7 @@ 
  */
 function give_dashboard_sales_widget() {
 
-	if ( ! current_user_can( apply_filters( 'give_dashboard_stats_cap', 'view_give_reports' ) ) ) {
+	if ( ! current_user_can(apply_filters('give_dashboard_stats_cap', 'view_give_reports'))) {
 		return;
 	}
 	$stats = new Give_Payment_Stats(); ?>
@@ -46,27 +46,27 @@ 
 	<div class="give-dashboard-widget">
 
 		<div class="give-dashboard-today give-clearfix">
-			<h3 class="give-dashboard-date-today"><?php echo date_i18n( _x( 'F j, Y', 'dashboard widget', 'give' ) ); ?></h3>
+			<h3 class="give-dashboard-date-today"><?php echo date_i18n(_x('F j, Y', 'dashboard widget', 'give')); ?></h3>
 
 			<p class="give-dashboard-happy-day"><?php
 				printf(
 				/* translators: %s: day of the week */
-					__( 'Happy %s!', 'give' ),
-					date_i18n( 'l', current_time( 'timestamp' ) )
+					__('Happy %s!', 'give'),
+					date_i18n('l', current_time('timestamp'))
 				);
 			?></p>
 
 			<p class="give-dashboard-today-earnings"><?php
-				$earnings_today = $stats->get_earnings( 0, 'today', false );
-				echo give_currency_filter( give_format_amount( $earnings_today, array( 'sanitize' => false ) ) );
+				$earnings_today = $stats->get_earnings(0, 'today', false);
+				echo give_currency_filter(give_format_amount($earnings_today, array('sanitize' => false)));
 			?></p>
 
 			<p class="give-donations-today"><?php
-				$donations_today = $stats->get_sales( 0, 'today', false );
+				$donations_today = $stats->get_sales(0, 'today', false);
 				printf(
 					/* translators: %s: daily donation count */
-					__( '%s donations today', 'give' ),
-					give_format_amount( $donations_today, array( 'decimal' => false, 'sanitize' => false ) )
+					__('%s donations today', 'give'),
+					give_format_amount($donations_today, array('decimal' => false, 'sanitize' => false))
 				);
 			?></p>
 
@@ -76,34 +76,34 @@ 
 		<table class="give-table-stats">
 			<thead style="display: none;">
 			<tr>
-				<th><?php _e( 'This Week', 'give' ); ?></th>
-				<th><?php _e( 'This Month', 'give' ); ?></th>
-				<th><?php _e( 'Past 30 Days', 'give' ); ?></th>
+				<th><?php _e('This Week', 'give'); ?></th>
+				<th><?php _e('This Month', 'give'); ?></th>
+				<th><?php _e('Past 30 Days', 'give'); ?></th>
 			</tr>
 			</thead>
 			<tbody>
 			<tr id="give-table-stats-tr-1">
 				<td>
-					<p class="give-dashboard-stat-total"><?php echo give_currency_filter( give_format_amount( $stats->get_earnings( 0, 'this_week' ), array( 'sanitize' => false ) ) ); ?></p>
+					<p class="give-dashboard-stat-total"><?php echo give_currency_filter(give_format_amount($stats->get_earnings(0, 'this_week'), array('sanitize' => false))); ?></p>
 
-					<p class="give-dashboard-stat-total-label"><?php _e( 'This Week', 'give' ); ?></p>
+					<p class="give-dashboard-stat-total-label"><?php _e('This Week', 'give'); ?></p>
 				</td>
 				<td>
-					<p class="give-dashboard-stat-total"><?php echo give_currency_filter( give_format_amount( $stats->get_earnings( 0, 'this_month' ), array( 'sanitize' => false ) ) ); ?></p>
+					<p class="give-dashboard-stat-total"><?php echo give_currency_filter(give_format_amount($stats->get_earnings(0, 'this_month'), array('sanitize' => false))); ?></p>
 
-					<p class="give-dashboard-stat-total-label"><?php _e( 'This Month', 'give' ); ?></p>
+					<p class="give-dashboard-stat-total-label"><?php _e('This Month', 'give'); ?></p>
 				</td>
 			</tr>
 			<tr id="give-table-stats-tr-2">
 				<td>
-					<p class="give-dashboard-stat-total"><?php echo give_currency_filter( give_format_amount( $stats->get_earnings( 0, 'last_month' ), array( 'sanitize' => false ) ) ) ?></p>
+					<p class="give-dashboard-stat-total"><?php echo give_currency_filter(give_format_amount($stats->get_earnings(0, 'last_month'), array('sanitize' => false))) ?></p>
 
-					<p class="give-dashboard-stat-total-label"><?php _e( 'Last Month', 'give' ); ?></p>
+					<p class="give-dashboard-stat-total-label"><?php _e('Last Month', 'give'); ?></p>
 				</td>
 				<td>
-					<p class="give-dashboard-stat-total"><?php echo give_currency_filter( give_format_amount( $stats->get_earnings( 0, 'this_quarter' ), array( 'sanitize' => false ) ) ) ?></p>
+					<p class="give-dashboard-stat-total"><?php echo give_currency_filter(give_format_amount($stats->get_earnings(0, 'this_quarter'), array('sanitize' => false))) ?></p>
 
-					<p class="give-dashboard-stat-total-label"><?php _e( 'This Quarter', 'give' ); ?></p>
+					<p class="give-dashboard-stat-total-label"><?php _e('This Quarter', 'give'); ?></p>
 				</td>
 			</tr>
 			</tbody>
@@ -123,24 +123,24 @@ 
  *
  * @return array
  */
-function give_dashboard_at_a_glance_widget( $items ) {
+function give_dashboard_at_a_glance_widget($items) {
 
-	$num_posts = wp_count_posts( 'give_forms' );
+	$num_posts = wp_count_posts('give_forms');
 
-	if ( $num_posts && $num_posts->publish ) {
+	if ($num_posts && $num_posts->publish) {
 
 		$text = sprintf(
 			/* translators: %s: number of posts published */
-			_n( '%s Give Form', '%s Give Forms', $num_posts->publish, 'give' ),
+			_n('%s Give Form', '%s Give Forms', $num_posts->publish, 'give'),
 			$num_posts->publish
 		);
 
-		$text = sprintf( $text, number_format_i18n( $num_posts->publish ) );
+		$text = sprintf($text, number_format_i18n($num_posts->publish));
 
-		if ( current_user_can( 'edit_give_forms', get_current_user_id() ) ) {
+		if (current_user_can('edit_give_forms', get_current_user_id())) {
 			$text = sprintf(
 				'<a class="give-forms-count" href="%1$s">%2$s</a>',
-				admin_url( 'edit.php?post_type=give_forms' ),
+				admin_url('edit.php?post_type=give_forms'),
 				$text
 			);
 		} else {
@@ -156,4 +156,4 @@ 
 	return $items;
 }
 
-add_filter( 'dashboard_glance_items', 'give_dashboard_at_a_glance_widget', 1, 1 );
+add_filter('dashboard_glance_items', 'give_dashboard_at_a_glance_widget', 1, 1);

includes/admin/payments/payments-history.php

Spacing +23 added lines, -23 removed lines

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

includes/admin/payments/actions.php

Spacing +124 added lines, -124 removed lines

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

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

Indentation +4 added lines, -4 removed lines

@@ -323,10 +323,10 @@
 										</div>
 
 										<?php
-                                        // Display the transaction ID present.
-                                        // The transaction ID is the charge ID from the gateway.
-                                        // For instance, stripe "ch_BzvwYCchqOy5Nt".
-                                        if ( $transaction_id != $payment_id ) : ?>
+										// Display the transaction ID present.
+										// The transaction ID is the charge ID from the gateway.
+										// For instance, stripe "ch_BzvwYCchqOy5Nt".
+										if ( $transaction_id != $payment_id ) : ?>
 											<div class="give-order-tx-id give-admin-box-inside">
 												<p>
 													<strong><?php _e( 'Transaction ID:', 'give' ); ?> <span class="give-tooltip give-icon give-icon-question"  data-tooltip="<?php echo sprintf( esc_attr__( 'The transaction ID within %s.', 'give' ), $gateway); ?>"></span></strong>&nbsp;

Spacing +149 added lines, -149 removed lines

@@ -10,13 +10,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! current_user_can( 'view_give_payments' ) ) {
+if ( ! current_user_can('view_give_payments')) {
 	wp_die(
-		__( 'Sorry, you are not allowed to access this page.', 'give' ), __( 'Error', 'give' ), array(
+		__('Sorry, you are not allowed to access this page.', 'give'), __('Error', 'give'), array(
 			'response' => 403,
 		)
 	);
@@ -28,35 +28,35 @@ 
  * @since 1.0
  * @return void
  */
-if ( ! isset( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
-	wp_die( __( 'Donation ID not supplied. Please try again.', 'give' ), __( 'Error', 'give' ), array( 'response' => 400 ) );
+if ( ! isset($_GET['id']) || ! is_numeric($_GET['id'])) {
+	wp_die(__('Donation ID not supplied. Please try again.', 'give'), __('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( __( 'The specified ID does not belong to a donation. Please try again.', 'give' ), __( 'Error', 'give' ), array( 'response' => 400 ) );
+if (empty($payment_exists)) {
+	wp_die(__('The specified ID does not belong to a donation. Please try again.', 'give'), __('Error', 'give'), array('response' => 400));
 }
 
 $number       = $payment->number;
 $payment_meta = $payment->get_meta();
 
-$company_name   = ! empty( $payment_meta['_give_donation_company'] ) ? esc_attr( $payment_meta['_give_donation_company'] ) : '';
-$transaction_id = esc_attr( $payment->transaction_id );
+$company_name   = ! empty($payment_meta['_give_donation_company']) ? esc_attr($payment_meta['_give_donation_company']) : '';
+$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;
 $currency_code  = $payment->currency;
 $payment_mode   = $payment->mode;
-$base_url       = admin_url( 'edit.php?post_type=give_forms&page=give-payment-history' );
+$base_url       = admin_url('edit.php?post_type=give_forms&page=give-payment-history');
 
 ?>
 <div class="wrap give-wrap">
@@ -65,13 +65,13 @@ 
 	<?php
 		printf(
 			/* translators: %s: donation number */
-			esc_html__( 'Donation %s', 'give' ),
+			esc_html__('Donation %s', 'give'),
 			$number
 		);
-		if ( $payment_mode == 'test' ) {
+		if ($payment_mode == 'test') {
 			echo Give()->tooltips->render_span(array(
-				'label' => __( 'This donation was made in test mode.', 'give' ),
-				'tag_content' => __( 'Test Donation', 'give' ),
+				'label' => __('This donation was made in test mode.', 'give'),
+				'tag_content' => __('Test Donation', 'give'),
 				'position'=> 'right',
 				'attributes' => array(
 					'id' => 'test-payment-label',
@@ -90,7 +90,7 @@ 
 	 *
 	 * @param int $payment_id Payment id.
 	 */
-	do_action( 'give_view_donation_details_before', $payment_id );
+	do_action('give_view_donation_details_before', $payment_id);
 	?>
 
 	<hr class="wp-header-end">
@@ -104,7 +104,7 @@ 
 		 *
 		 * @param int $payment_id Payment id.
 		 */
-		do_action( 'give_view_donation_details_form_top', $payment_id );
+		do_action('give_view_donation_details_form_top', $payment_id);
 		?>
 		<div id="poststuff">
 			<div id="give-dashboard-widgets-wrap">
@@ -120,16 +120,16 @@ 
 							 *
 							 * @param int $payment_id Payment id.
 							 */
-							do_action( 'give_view_donation_details_sidebar_before', $payment_id );
+							do_action('give_view_donation_details_sidebar_before', $payment_id);
 							?>
 
 							<div id="give-order-update" class="postbox give-order-data">
 
 								<div class="give-order-top">
-									<h3 class="hndle"><?php _e( 'Update Donation', 'give' ); ?></h3>
+									<h3 class="hndle"><?php _e('Update Donation', 'give'); ?></h3>
 
 									<?php
-									if ( current_user_can( 'view_give_payments' ) ) {
+									if (current_user_can('view_give_payments')) {
 										echo sprintf(
 											'<span class="delete-donation" id="delete-donation-%d"><a class="delete-single-donation delete-donation-button dashicons dashicons-trash" href="%s" aria-label="%s"></a></span>',
 											$payment_id,
@@ -141,7 +141,7 @@ 
 													), $base_url
 												), 'give_donation_nonce'
 											),
-											sprintf( __( 'Delete Donation %s', 'give' ), $payment_id )
+											sprintf(__('Delete Donation %s', 'give'), $payment_id)
 										);
 									}
 									?>
@@ -158,33 +158,33 @@ 
 										 *
 										 * @param int $payment_id Payment id.
 										 */
-										do_action( 'give_view_donation_details_totals_before', $payment_id );
+										do_action('give_view_donation_details_totals_before', $payment_id);
 										?>
 
 										<div class="give-admin-box-inside">
 											<p>
-												<label for="give-payment-status" class="strong"><?php _e( 'Status:', 'give' ); ?></label>&nbsp;
+												<label for="give-payment-status" class="strong"><?php _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 _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 _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 _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 _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>
 
@@ -198,14 +198,14 @@ 
 										 *
 										 * @param int $payment_id Payment id.
 										 */
-										do_action( 'give_view_donation_details_update_inner', $payment_id );
+										do_action('give_view_donation_details_update_inner', $payment_id);
 										?>
 
 										<div class="give-order-payment give-admin-box-inside">
 											<p>
-												<label for="give-payment-total" class="strong"><?php _e( 'Total Donation:', 'give' ); ?></label>&nbsp;
-												<?php echo give_currency_symbol( $payment->currency ); ?>
-												&nbsp;<input id="give-payment-total" name="give-payment-total" type="text" class="small-text give-price-field" value="<?php echo esc_attr( give_format_decimal( give_donation_amount( $payment_id ), false, false ) ); ?>"/>
+												<label for="give-payment-total" class="strong"><?php _e('Total Donation:', 'give'); ?></label>&nbsp;
+												<?php echo give_currency_symbol($payment->currency); ?>
+												&nbsp;<input id="give-payment-total" name="give-payment-total" type="text" class="small-text give-price-field" value="<?php echo esc_attr(give_format_decimal(give_donation_amount($payment_id), false, false)); ?>"/>
 											</p>
 										</div>
 
@@ -217,7 +217,7 @@ 
 										 *
 										 * @param int $payment_id Payment id.
 										 */
-										do_action( 'give_view_donation_details_totals_after', $payment_id );
+										do_action('give_view_donation_details_totals_after', $payment_id);
 										?>
 
 									</div>
@@ -235,17 +235,17 @@ 
 									 *
 									 * @param int $payment_id Payment id.
 									 */
-									do_action( 'give_view_donation_details_update_before', $payment_id );
+									do_action('give_view_donation_details_update_before', $payment_id);
 									?>
 
 									<div id="major-publishing-actions">
 										<div id="publishing-action">
 
 											<input type="submit" class="button button-primary right"
-											       value="<?php _e( 'Save Donation', 'give' ); ?>"/>
+											       value="<?php _e('Save Donation', 'give'); ?>"/>
 
 											<?php
-											if ( give_is_payment_complete( $payment_id ) ) {
+											if (give_is_payment_complete($payment_id)) {
 												echo sprintf(
 													'<a href="%1$s" id="give-resend-receipt" class="button-secondary right">%2$s</a>',
 													esc_url(
@@ -256,7 +256,7 @@ 
 															)
 														)
 													),
-													__( 'Resend Receipt', 'give' )
+													__('Resend Receipt', 'give')
 												);
 											}
 											?>
@@ -271,7 +271,7 @@ 
 									 *
 									 * @param int $payment_id Payment id.
 									 */
-									do_action( 'give_view_donation_details_update_after', $payment_id );
+									do_action('give_view_donation_details_update_after', $payment_id);
 									?>
 
 								</div>
@@ -282,7 +282,7 @@ 
 
 							<div id="give-order-details" class="postbox give-order-data">
 
-								<h3 class="hndle"><?php _e( 'Donation Meta', 'give' ); ?></h3>
+								<h3 class="hndle"><?php _e('Donation Meta', 'give'); ?></h3>
 
 								<div class="inside">
 									<div class="give-admin-box">
@@ -295,30 +295,30 @@ 
 										 *
 										 * @param int $payment_id Payment id.
 										 */
-										do_action( 'give_view_donation_details_payment_meta_before', $payment_id );
+										do_action('give_view_donation_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 _e( 'Gateway:', 'give' ); ?></strong>&nbsp;
-													<?php echo give_get_gateway_admin_label( $gateway ); ?>
+													<strong><?php _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 _e( 'Key:', 'give' ); ?></strong>&nbsp;
-												<?php echo give_get_payment_key( $payment_id ); ?>
+												<strong><?php _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 _e( 'IP:', 'give' ); ?></strong>&nbsp;
-												<?php echo esc_html( give_get_payment_user_ip( $payment_id ) ); ?>
+												<strong><?php _e('IP:', 'give'); ?></strong>&nbsp;
+												<?php echo esc_html(give_get_payment_user_ip($payment_id)); ?>
 											</p>
 										</div>
 
@@ -326,18 +326,18 @@ 
                                         // Display the transaction ID present.
                                         // The transaction ID is the charge ID from the gateway.
                                         // For instance, stripe "ch_BzvwYCchqOy5Nt".
-                                        if ( $transaction_id != $payment_id ) : ?>
+                                        if ($transaction_id != $payment_id) : ?>
 											<div class="give-order-tx-id give-admin-box-inside">
 												<p>
-													<strong><?php _e( 'Transaction ID:', 'give' ); ?> <span class="give-tooltip give-icon give-icon-question"  data-tooltip="<?php echo sprintf( esc_attr__( 'The transaction ID within %s.', 'give' ), $gateway); ?>"></span></strong>&nbsp;
-													<?php echo apply_filters( "give_payment_details_transaction_id-{$gateway}", $transaction_id, $payment_id ); ?>
+													<strong><?php _e('Transaction ID:', 'give'); ?> <span class="give-tooltip give-icon give-icon-question"  data-tooltip="<?php echo sprintf(esc_attr__('The transaction ID within %s.', 'give'), $gateway); ?>"></span></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 _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 _e('View all donations for this donor &raquo;', 'give'); ?></a>
 											</p>
 										</div>
 
@@ -349,7 +349,7 @@ 
 										 *
 										 * @param int $payment_id Payment id.
 										 */
-										do_action( 'give_view_donation_details_payment_meta_after', $payment_id );
+										do_action('give_view_donation_details_payment_meta_after', $payment_id);
 										?>
 
 									</div>
@@ -369,7 +369,7 @@ 
 							 *
 							 * @param int $payment_id Payment id.
 							 */
-							do_action( 'give_view_donation_details_sidebar_after', $payment_id );
+							do_action('give_view_donation_details_sidebar_after', $payment_id);
 							?>
 
 						</div>
@@ -389,31 +389,31 @@ 
 							 *
 							 * @param int $payment_id Payment id.
 							 */
-							do_action( 'give_view_donation_details_main_before', $payment_id );
+							do_action('give_view_donation_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 _e( 'Donation Information', 'give' ); ?></h3>
+								<h3 class="hndle"><?php _e('Donation Information', 'give'); ?></h3>
 
 								<div class="inside">
 
 									<div class="column-container">
 										<div class="column">
 											<p>
-												<strong><?php _e( 'Donation Form ID:', 'give' ); ?></strong><br>
+												<strong><?php _e('Donation Form ID:', 'give'); ?></strong><br>
 												<?php
-												if ( $payment_meta['form_id'] ) :
+												if ($payment_meta['form_id']) :
 													printf(
 														'<a href="%1$s">%2$s</a>',
-														admin_url( 'post.php?action=edit&post=' . $payment_meta['form_id'] ),
+														admin_url('post.php?action=edit&post='.$payment_meta['form_id']),
 														$payment_meta['form_id']
 													);
 												endif;
 												?>
 											</p>
 											<p>
-												<strong><?php esc_html_e( 'Donation Form Title:', 'give' ); ?></strong><br>
+												<strong><?php esc_html_e('Donation Form Title:', 'give'); ?></strong><br>
 												<?php
 												echo Give()->html->forms_dropdown(
 													array(
@@ -429,21 +429,21 @@ 
 										</div>
 										<div class="column">
 											<p>
-												<strong><?php _e( 'Donation Date:', 'give' ); ?></strong><br>
-												<?php echo date_i18n( give_date_format(), $payment_date ); ?>
+												<strong><?php _e('Donation Date:', 'give'); ?></strong><br>
+												<?php echo date_i18n(give_date_format(), $payment_date); ?>
 											</p>
 											<p>
-												<strong><?php _e( 'Donation Level:', 'give' ); ?></strong><br>
+												<strong><?php _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 ) ) {
-														_e( 'n/a', 'give' );
+													$var_prices = give_has_variable_prices($payment_meta['form_id']);
+													if (empty($var_prices)) {
+														_e('n/a', 'give');
 													} else {
 														$prices_atts = array();
-														if ( $variable_prices = give_get_variable_prices( $payment_meta['form_id'] ) ) {
-															foreach ( $variable_prices as $variable_price ) {
-																$prices_atts[ $variable_price['_give_id']['level_id'] ] = give_format_amount( $variable_price['_give_amount'], array( 'sanitize' => false ) );
+														if ($variable_prices = give_get_variable_prices($payment_meta['form_id'])) {
+															foreach ($variable_prices as $variable_price) {
+																$prices_atts[$variable_price['_give_id']['level_id']] = give_format_amount($variable_price['_give_amount'], array('sanitize' => false));
 															}
 														}
 														// Variable price dropdown options.
@@ -452,12 +452,12 @@ 
 															'name'             => 'give-variable-price',
 															'chosen'           => true,
 															'show_option_all'  => '',
-															'show_option_none' => ( '' === get_post_meta( $payment_id, '_give_payment_price_id', true ) ? __( 'None', 'give' ) : '' ),
-															'select_atts'      => 'data-prices=' . esc_attr( wp_json_encode( $prices_atts ) ),
+															'show_option_none' => ('' === get_post_meta($payment_id, '_give_payment_price_id', true) ? __('None', 'give') : ''),
+															'select_atts'      => 'data-prices='.esc_attr(wp_json_encode($prices_atts)),
 															'selected'         => $payment_meta['price_id'],
 														);
 														// Render variable prices select tag html.
-														give_get_form_variable_price_dropdown( $variable_price_dropdown_option, true );
+														give_get_form_variable_price_dropdown($variable_price_dropdown_option, true);
 													}
 													?>
 												</span>
@@ -465,8 +465,8 @@ 
 										</div>
 										<div class="column">
 											<p>
-												<strong><?php esc_html_e( 'Total Donation:', 'give' ); ?></strong><br>
-												<?php echo give_donation_amount( $payment, true ); ?>
+												<strong><?php esc_html_e('Total Donation:', 'give'); ?></strong><br>
+												<?php echo give_donation_amount($payment, true); ?>
 											</p>
 
 											<p>
@@ -480,7 +480,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);
 
 
 												/**
@@ -492,7 +492,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 donation details page, in the donation-information metabox, before the body elements.
@@ -503,7 +503,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 donation details page, in the donation-information metabox, after the body elements.
@@ -514,7 +514,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>
@@ -534,59 +534,59 @@ 
 							 *
 							 * @param int $payment_id Payment id.
 							 */
-							do_action( 'give_view_donation_details_donor_detail_before', $payment_id );
+							do_action('give_view_donation_details_donor_detail_before', $payment_id);
 							?>
 
 							<div id="give-donor-details" class="postbox">
-								<h3 class="hndle"><?php _e( 'Donor Details', 'give' ); ?></h3>
+								<h3 class="hndle"><?php _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 _e( 'Donor ID:', 'give' ); ?></strong><br>
+												<strong><?php _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
 													);
 												}
 												?>
-												<span>(<a href="#new" class="give-payment-new-donor"><?php _e( 'Create New Donor', 'give' ); ?></a>)</span>
+												<span>(<a href="#new" class="give-payment-new-donor"><?php _e('Create New Donor', 'give'); ?></a>)</span>
 											</p>
 											<p>
-												<strong><?php _e( 'Donor Since:', 'give' ); ?></strong><br>
-												<?php echo date_i18n( give_date_format(), strtotime( $donor->date_created ) ) ?>
+												<strong><?php _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 _e( 'Donor Name:', 'give' ); ?></strong><br>
+												<strong><?php _e('Donor Name:', 'give'); ?></strong><br>
 												<?php
-												$donor_billing_name = give_get_donor_name_by( $payment_id, 'donation' );
-												$donor_name         = give_get_donor_name_by( $donor_id, 'donor' );
+												$donor_billing_name = give_get_donor_name_by($payment_id, 'donation');
+												$donor_name         = give_get_donor_name_by($donor_id, 'donor');
 
 												// Check whether the donor name and WP_User name is same or not.
-												if ( sanitize_title( $donor_billing_name ) != sanitize_title( $donor_name ) ) {
-													echo $donor_billing_name . ' (<a href="' . esc_url( admin_url( "edit.php?post_type=give_forms&page=give-donors&view=overview&id=$donor_id" ) ) . '">' . $donor_name . '</a>)';
+												if (sanitize_title($donor_billing_name) != sanitize_title($donor_name)) {
+													echo $donor_billing_name.' (<a href="'.esc_url(admin_url("edit.php?post_type=give_forms&page=give-donors&view=overview&id=$donor_id")).'">'.$donor_name.'</a>)';
 												} else {
 													echo $donor_name;
 												}
 												?>
 											</p>
 											<p>
-												<strong><?php _e( 'Donor Email:', 'give' ); ?></strong><br>
+												<strong><?php _e('Donor Email:', 'give'); ?></strong><br>
 												<?php echo $donor->email; ?>
 											</p>
 										</div>
 										<div class="column">
 											<p>
-												<strong><?php _e( 'Change Donor:', 'give' ); ?></strong><br>
+												<strong><?php _e('Change Donor:', 'give'); ?></strong><br>
 												<?php
 												echo Give()->html->donor_dropdown(
 													array(
@@ -597,9 +597,9 @@ 
 												?>
 											</p>
 											<p>
-												<?php if ( ! empty( $company_name ) ) {
+												<?php if ( ! empty($company_name)) {
 													?>
-													<strong><?php esc_html_e( 'Company Name:', 'give' ); ?></strong><br>
+													<strong><?php esc_html_e('Company Name:', 'give'); ?></strong><br>
 													<?php
 													echo $company_name;
 												} ?>
@@ -610,19 +610,19 @@ 
 									<div class="column-container new-donor" style="display: none">
 										<div class="column">
 											<p>
-												<label for="give-new-donor-first-name"><?php _e( 'New Donor First Name:', 'give' ); ?></label>
+												<label for="give-new-donor-first-name"><?php _e('New Donor First Name:', 'give'); ?></label>
 												<input id="give-new-donor-first-name" type="text" name="give-new-donor-first-name" value="" class="medium-text"/>
 											</p>
 										</div>
 										<div class="column">
 											<p>
-												<label for="give-new-donor-last-name"><?php _e( 'New Donor Last Name:', 'give' ); ?></label>
+												<label for="give-new-donor-last-name"><?php _e('New Donor Last Name:', 'give'); ?></label>
 												<input id="give-new-donor-last-name" type="text" name="give-new-donor-last-name" value="" class="medium-text"/>
 											</p>
 										</div>
 										<div class="column">
 											<p>
-												<label for="give-new-donor-email"><?php _e( 'New Donor Email:', 'give' ); ?></label>
+												<label for="give-new-donor-email"><?php _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>
@@ -630,9 +630,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 _e( 'Cancel', 'give' ); ?></a>
+												<a href="#cancel" class="give-payment-new-donor-cancel give-delete"><?php _e('Cancel', 'give'); ?></a>
 												<br>
-												<em><?php _e( 'Click "Save Donation" to create new donor.', 'give' ); ?></em>
+												<em><?php _e('Click "Save Donation" to create new donor.', 'give'); ?></em>
 											</p>
 										</div>
 									</div>
@@ -647,7 +647,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.
@@ -656,7 +656,7 @@ 
 									 *
 									 * @param int $payment_id Payment id.
 									 */
-									do_action( 'give_payment_view_details', $payment_id );
+									do_action('give_payment_view_details', $payment_id);
 									?>
 
 								</div>
@@ -672,11 +672,11 @@ 
 							 *
 							 * @param int $payment_id Payment id.
 							 */
-							do_action( 'give_view_donation_details_billing_before', $payment_id );
+							do_action('give_view_donation_details_billing_before', $payment_id);
 							?>
 
 							<div id="give-billing-details" class="postbox">
-								<h3 class="hndle"><?php _e( 'Billing Address', 'give' ); ?></h3>
+								<h3 class="hndle"><?php _e('Billing Address', 'give'); ?></h3>
 
 								<div class="inside">
 
@@ -686,9 +686,9 @@ 
 											<div class="data column-container">
 
 												<?php
-												$address['country'] = ( ! empty( $address['country'] ) ? $address['country'] : give_get_country() );
+												$address['country'] = ( ! empty($address['country']) ? $address['country'] : give_get_country());
 
-												$address['state'] = ( ! empty( $address['state'] ) ? $address['state'] : '' );
+												$address['state'] = ( ! empty($address['state']) ? $address['state'] : '');
 
 												// Get the country list that does not have any states init.
 												$no_states_country = give_no_states_country_list();
@@ -696,7 +696,7 @@ 
 
 												<div class="row">
 													<div id="give-order-address-country-wrap">
-														<label class="order-data-address-line"><?php _e( 'Country:', 'give' ); ?></label>
+														<label class="order-data-address-line"><?php _e('Country:', 'give'); ?></label>
 														<?php
 														echo Give()->html->select(
 															array(
@@ -706,8 +706,8 @@ 
 																'show_option_all'  => false,
 																'show_option_none' => false,
 																'chosen'           => true,
-																'placeholder'      => esc_attr__( 'Select a country', 'give' ),
-																'data'             => array( 'search-type' => 'no_ajax' ),
+																'placeholder'      => esc_attr__('Select a country', 'give'),
+																'data'             => array('search-type' => 'no_ajax'),
 															)
 														);
 														?>
@@ -716,35 +716,35 @@ 
 
 												<div class="row">
 													<div class="give-wrap-address-line1">
-														<label for="give-payment-address-line1" class="order-data-address"><?php _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 _e('Address 1:', 'give'); ?></label>
+														<input id="give-payment-address-line1" type="text" name="give-payment-address[0][line1]" value="<?php echo esc_attr($address['line1']); ?>" class="medium-text"/>
 													</div>
 												</div>
 
 												<div class="row">
 													<div class="give-wrap-address-line2">
-														<label for="give-payment-address-line2" class="order-data-address-line"><?php _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 _e('Address 2:', 'give'); ?></label>
+														<input id="give-payment-address-line2" type="text" name="give-payment-address[0][line2]" value="<?php echo esc_attr($address['line2']); ?>" class="medium-text"/>
 													</div>
 												</div>
 
 												<div class="row">
 													<div class="give-wrap-address-city">
-														<label for="give-payment-address-city" class="order-data-address-line"><?php esc_html_e( 'City:', 'give' ); ?></label>
-														<input id="give-payment-address-city" type="text" name="give-payment-address[0][city]" value="<?php echo esc_attr( $address['city'] ); ?>" class="medium-text"/>
+														<label for="give-payment-address-city" class="order-data-address-line"><?php esc_html_e('City:', 'give'); ?></label>
+														<input id="give-payment-address-city" type="text" name="give-payment-address[0][city]" value="<?php echo esc_attr($address['city']); ?>" class="medium-text"/>
 													</div>
 												</div>
 
 												<?php
-												$state_exists = ( ! empty( $address['country'] ) && array_key_exists( $address['country'], $no_states_country ) ? true : false );
+												$state_exists = ( ! empty($address['country']) && array_key_exists($address['country'], $no_states_country) ? true : false);
 												?>
 												<div class="row">
-													<div class="<?php echo( ! empty( $state_exists ) ? 'column-full' : 'column' ); ?> give-column give-column-state">
-														<div id="give-order-address-state-wrap" class="<?php echo( ! empty( $state_exists ) ? 'give-hidden' : '' ); ?>">
-															<label for="give-payment-address-state" class="order-data-address-line"><?php esc_html_e( 'State / Province / County:', 'give' ); ?></label>
+													<div class="<?php echo( ! empty($state_exists) ? 'column-full' : 'column'); ?> give-column give-column-state">
+														<div id="give-order-address-state-wrap" class="<?php echo( ! empty($state_exists) ? 'give-hidden' : ''); ?>">
+															<label for="give-payment-address-state" class="order-data-address-line"><?php esc_html_e('State / Province / County:', 'give'); ?></label>
 															<?php
-															$states = give_get_states( $address['country'] );
-															if ( ! empty( $states ) ) {
+															$states = give_get_states($address['country']);
+															if ( ! empty($states)) {
 																echo Give()->html->select(
 																	array(
 																		'options'          => $states,
@@ -753,23 +753,23 @@ 
 																		'show_option_all'  => false,
 																		'show_option_none' => false,
 																		'chosen'           => true,
-																		'placeholder'      => esc_attr__( 'Select a state', 'give' ),
-																		'data'             => array( 'search-type' => 'no_ajax' ),
+																		'placeholder'      => esc_attr__('Select a state', 'give'),
+																		'data'             => array('search-type' => 'no_ajax'),
 																	)
 																);
 															} else {
 																?>
-																<input id="give-payment-address-state" type="text" name="give-payment-address[0][state]" value="<?php echo esc_attr( $address['state'] ); ?>" class="medium-text"/>
+																<input id="give-payment-address-state" type="text" name="give-payment-address[0][state]" value="<?php echo esc_attr($address['state']); ?>" class="medium-text"/>
 																<?php
 															}
 															?>
 														</div>
 													</div>
 
-													<div class="<?php echo( ! empty( $state_exists ) ? 'column-full' : 'column' ); ?> give-column give-column-zip">
+													<div class="<?php echo( ! empty($state_exists) ? 'column-full' : 'column'); ?> give-column give-column-zip">
 														<div class="give-wrap-address-zip">
-															<label for="give-payment-address-zip" class="order-data-address-line"><?php _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 _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>
@@ -788,7 +788,7 @@ 
 									 *
 									 * @param int $payment_id Payment id.
 									 */
-									do_action( 'give_payment_billing_details', $payment_id );
+									do_action('give_payment_billing_details', $payment_id);
 									?>
 
 								</div>
@@ -804,34 +804,34 @@ 
 							 *
 							 * @param int $payment_id Payment id.
 							 */
-							do_action( 'give_view_donation_details_billing_after', $payment_id );
+							do_action('give_view_donation_details_billing_after', $payment_id);
 							?>
 
 							<div id="give-payment-notes" class="postbox">
-								<h3 class="hndle"><?php _e( 'Donation Notes', 'give' ); ?></h3>
+								<h3 class="hndle"><?php _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 _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 _e('Add Note', 'give'); ?></button>
 									</div>
 
 								</div>
@@ -847,7 +847,7 @@ 
 							 *
 							 * @param int $payment_id Payment id.
 							 */
-							do_action( 'give_view_donation_details_main_after', $payment_id );
+							do_action('give_view_donation_details_main_after', $payment_id);
 							?>
 
 						</div>
@@ -869,11 +869,11 @@ 
 		 *
 		 * @param int $payment_id Payment id.
 		 */
-		do_action( 'give_view_donation_details_form_bottom', $payment_id );
+		do_action('give_view_donation_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
@@ -884,6 +884,6 @@ 
 	 *
 	 * @param int $payment_id Payment id.
 	 */
-	do_action( 'give_view_donation_details_after', $payment_id );
+	do_action('give_view_donation_details_after', $payment_id);
 	?>
 </div><!-- /.wrap -->

includes/admin/forms/dashboard-columns.php

Spacing +72 added lines, -72 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -27,34 +27,34 @@ 
  * @return array $form_columns Updated array of forms columns
  *  Post Type List Table
  */
-function give_form_columns( $give_form_columns ) {
+function give_form_columns($give_form_columns) {
 
 	// Standard columns
 	$give_form_columns = array(
 		'cb'            => '<input type="checkbox"/>',
-		'title'         => __( 'Name', 'give' ),
-		'form_category' => __( 'Categories', 'give' ),
-		'form_tag'      => __( 'Tags', 'give' ),
-		'price'         => __( 'Amount', 'give' ),
-		'goal'          => __( 'Goal', 'give' ),
-		'donations'     => __( 'Donations', 'give' ),
-		'earnings'      => __( 'Income', 'give' ),
-		'shortcode'     => __( 'Shortcode', 'give' ),
-		'date'          => __( 'Date', 'give' ),
+		'title'         => __('Name', 'give'),
+		'form_category' => __('Categories', 'give'),
+		'form_tag'      => __('Tags', 'give'),
+		'price'         => __('Amount', 'give'),
+		'goal'          => __('Goal', 'give'),
+		'donations'     => __('Donations', 'give'),
+		'earnings'      => __('Income', 'give'),
+		'shortcode'     => __('Shortcode', 'give'),
+		'date'          => __('Date', 'give'),
 	);
 
 	// Does the user want categories / tags?
-	if ( ! give_is_setting_enabled( give_get_option( 'categories', 'disabled' ) ) ) {
-		unset( $give_form_columns['form_category'] );
+	if ( ! give_is_setting_enabled(give_get_option('categories', 'disabled'))) {
+		unset($give_form_columns['form_category']);
 	}
-	if ( ! give_is_setting_enabled( give_get_option( 'tags', 'disabled' ) ) ) {
-		unset( $give_form_columns['form_tag'] );
+	if ( ! give_is_setting_enabled(give_get_option('tags', 'disabled'))) {
+		unset($give_form_columns['form_tag']);
 	}
 
-	return apply_filters( 'give_forms_columns', $give_form_columns );
+	return apply_filters('give_forms_columns', $give_form_columns);
 }
 
-add_filter( 'manage_edit-give_forms_columns', 'give_form_columns' );
+add_filter('manage_edit-give_forms_columns', 'give_form_columns');
 
 /**
  * Render Give Form Columns
@@ -66,70 +66,70 @@ 
  *
  * @return void
  */
-function give_render_form_columns( $column_name, $post_id ) {
-	if ( get_post_type( $post_id ) == 'give_forms' ) {
+function give_render_form_columns($column_name, $post_id) {
+	if (get_post_type($post_id) == 'give_forms') {
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 			case 'form_category':
-				echo get_the_term_list( $post_id, 'give_forms_category', '', ', ', '' );
+				echo get_the_term_list($post_id, 'give_forms_category', '', ', ', '');
 				break;
 			case 'form_tag':
-				echo get_the_term_list( $post_id, 'give_forms_tag', '', ', ', '' );
+				echo get_the_term_list($post_id, 'give_forms_tag', '', ', ', '');
 				break;
 			case 'price':
-				if ( give_has_variable_prices( $post_id ) ) {
-					echo give_price_range( $post_id );
+				if (give_has_variable_prices($post_id)) {
+					echo give_price_range($post_id);
 				} else {
-					echo give_price( $post_id, false );
-					printf( '<input type="hidden" class="formprice-%1$s" value="%2$s" />', esc_attr( $post_id ), esc_attr( give_get_form_price( $post_id ) ) );
+					echo give_price($post_id, false);
+					printf('<input type="hidden" class="formprice-%1$s" value="%2$s" />', esc_attr($post_id), esc_attr(give_get_form_price($post_id)));
 				}
 				break;
 			case 'goal':
-				if ( give_is_setting_enabled( give_get_meta( $post_id, '_give_goal_option', true ) ) ) {
+				if (give_is_setting_enabled(give_get_meta($post_id, '_give_goal_option', true))) {
 
-					echo give_admin_form_goal_stats( $post_id );
+					echo give_admin_form_goal_stats($post_id);
 
 				} else {
-					_e( 'No Goal Set', 'give' );
+					_e('No Goal Set', 'give');
 				}
 
 				printf(
 					'<input type="hidden" class="formgoal-%1$s" value="%2$s" />',
-					esc_attr( $post_id ),
-					give_get_form_goal( $post_id )
+					esc_attr($post_id),
+					give_get_form_goal($post_id)
 				);
 
 				break;
 			case 'donations':
-				if ( current_user_can( 'view_give_form_stats', $post_id ) ) {
+				if (current_user_can('view_give_form_stats', $post_id)) {
 					printf(
 						'<a href="%1$s">%2$s</a>',
-						esc_url( admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&form_id=' . $post_id ) ),
-						give_get_form_sales_stats( $post_id )
+						esc_url(admin_url('edit.php?post_type=give_forms&page=give-payment-history&form_id='.$post_id)),
+						give_get_form_sales_stats($post_id)
 					);
 				} else {
 					echo '-';
 				}
 				break;
 			case 'earnings':
-				if ( current_user_can( 'view_give_form_stats', $post_id ) ) {
+				if (current_user_can('view_give_form_stats', $post_id)) {
 					printf(
 						'<a href="%1$s">%2$s</a>',
-						esc_url( admin_url( 'edit.php?post_type=give_forms&page=give-reports&tab=forms&form-id=' . $post_id ) ),
-						give_currency_filter( give_format_amount( give_get_form_earnings_stats( $post_id ), array( 'sanitize' => false ) ) )
+						esc_url(admin_url('edit.php?post_type=give_forms&page=give-reports&tab=forms&form-id='.$post_id)),
+						give_currency_filter(give_format_amount(give_get_form_earnings_stats($post_id), array('sanitize' => false)))
 					);
 				} else {
 					echo '-';
 				}
 				break;
 			case 'shortcode':
-				printf( '<input onclick="this.setSelectionRange(0, this.value.length)" type="text" class="shortcode-input" readonly="" value="[give_form id=&#34;%s&#34;]"', absint( $post_id ) );
+				printf('<input onclick="this.setSelectionRange(0, this.value.length)" type="text" class="shortcode-input" readonly="" value="[give_form id=&#34;%s&#34;]"', absint($post_id));
 				break;
 		}// End switch().
 	}// End if().
 }
 
-add_action( 'manage_posts_custom_column', 'give_render_form_columns', 10, 2 );
+add_action('manage_posts_custom_column', 'give_render_form_columns', 10, 2);
 
 /**
  * Registers the sortable columns in the list table
@@ -140,7 +140,7 @@ 
  *
  * @return array $columns Array of sortable columns
  */
-function give_sortable_form_columns( $columns ) {
+function give_sortable_form_columns($columns) {
 	$columns['price']     = 'amount';
 	$columns['sales']     = 'sales';
 	$columns['earnings']  = 'earnings';
@@ -150,7 +150,7 @@ 
 	return $columns;
 }
 
-add_filter( 'manage_edit-give_forms_sortable_columns', 'give_sortable_form_columns' );
+add_filter('manage_edit-give_forms_sortable_columns', 'give_sortable_form_columns');
 
 /**
  * Sorts Columns in the Forms List Table
@@ -161,13 +161,13 @@ 
  *
  * @return array $vars Array of all the sort variables.
  */
-function give_sort_forms( $vars ) {
+function give_sort_forms($vars) {
 	// Check if we're viewing the "give_forms" post type.
-	if ( ! isset( $vars['post_type'] ) || ! isset( $vars['orderby'] ) || 'give_forms' !== $vars['post_type'] ) {
+	if ( ! isset($vars['post_type']) || ! isset($vars['orderby']) || 'give_forms' !== $vars['post_type']) {
 		return $vars;
 	}
 
-	switch ( $vars['orderby'] ) {
+	switch ($vars['orderby']) {
 		// Check if 'orderby' is set to "sales".
 		case 'sales':
 			$vars = array_merge(
@@ -192,7 +192,7 @@ 
 
 		// Check if "orderby" is set to "price/amount".
 		case 'amount':
-			$multi_level_meta_key = ( 'asc' === $vars['order'] ) ? '_give_levels_minimum_amount' : '_give_levels_maximum_amount';
+			$multi_level_meta_key = ('asc' === $vars['order']) ? '_give_levels_minimum_amount' : '_give_levels_maximum_amount';
 
 			$vars['orderby']    = 'meta_value_num';
 			$vars['meta_query'] = array(
@@ -244,17 +244,17 @@ 
  *
  * @return array       Array of all sort variables.
  */
-function give_filter_forms( $vars ) {
-	if ( isset( $vars['post_type'] ) && 'give_forms' == $vars['post_type'] ) {
+function give_filter_forms($vars) {
+	if (isset($vars['post_type']) && 'give_forms' == $vars['post_type']) {
 
 		// If an author ID was passed, use it
-		if ( isset( $_REQUEST['author'] ) && ! current_user_can( 'view_give_reports' ) ) {
+		if (isset($_REQUEST['author']) && ! current_user_can('view_give_reports')) {
 
 			$author_id = $_REQUEST['author'];
-			if ( (int) $author_id !== get_current_user_id() ) {
-				wp_die( esc_html__( 'You do not have permission to view this data.', 'give' ), esc_html__( 'Error', 'give' ), array(
+			if ((int) $author_id !== get_current_user_id()) {
+				wp_die(esc_html__('You do not have permission to view this data.', 'give'), esc_html__('Error', 'give'), array(
 					'response' => 403,
-				) );
+				));
 			}
 			$vars = array_merge(
 				$vars,
@@ -278,11 +278,11 @@ 
  * @return void
  */
 function give_forms_load() {
-	add_filter( 'request', 'give_sort_forms' );
-	add_filter( 'request', 'give_filter_forms' );
+	add_filter('request', 'give_sort_forms');
+	add_filter('request', 'give_filter_forms');
 }
 
-add_action( 'load-edit.php', 'give_forms_load', 9999 );
+add_action('load-edit.php', 'give_forms_load', 9999);
 
 /**
  * Remove Forms Month Filter
@@ -296,17 +296,17 @@ 
  * @global      $typenow The post type we are viewing.
  * @return array Empty array disables the dropdown.
  */
-function give_remove_month_filter( $dates ) {
+function give_remove_month_filter($dates) {
 	global $typenow;
 
-	if ( $typenow == 'give_forms' ) {
+	if ($typenow == 'give_forms') {
 		$dates = array();
 	}
 
 	return $dates;
 }
 
-add_filter( 'months_dropdown_results', 'give_remove_month_filter', 99 );
+add_filter('months_dropdown_results', 'give_remove_month_filter', 99);
 
 /**
  * Updates price when saving post
@@ -317,23 +317,23 @@ 
  *
  * @return int|null
  */
-function give_price_save_quick_edit( $post_id ) {
-	if ( ! isset( $_POST['post_type'] ) || 'give_forms' !== $_POST['post_type'] ) {
+function give_price_save_quick_edit($post_id) {
+	if ( ! isset($_POST['post_type']) || 'give_forms' !== $_POST['post_type']) {
 		return;
 	}
-	if ( ! current_user_can( 'edit_post', $post_id ) ) {
+	if ( ! current_user_can('edit_post', $post_id)) {
 		return $post_id;
 	}
-	if ( defined( 'DOING_AUTOSAVE' ) && DOING_AUTOSAVE ) {
+	if (defined('DOING_AUTOSAVE') && DOING_AUTOSAVE) {
 		return $post_id;
 	}
 
-	if ( isset( $_REQUEST['_give_regprice'] ) ) {
-		give_update_meta( $post_id, '_give_set_price', give_sanitize_amount_for_db( strip_tags( stripslashes( $_REQUEST['_give_regprice'] ) ) ) );
+	if (isset($_REQUEST['_give_regprice'])) {
+		give_update_meta($post_id, '_give_set_price', give_sanitize_amount_for_db(strip_tags(stripslashes($_REQUEST['_give_regprice']))));
 	}
 }
 
-add_action( 'save_post', 'give_price_save_quick_edit' );
+add_action('save_post', 'give_price_save_quick_edit');
 
 /**
  * Process bulk edit actions via AJAX
@@ -343,18 +343,18 @@ 
  */
 function give_save_bulk_edit() {
 
-	$post_ids = ( isset( $_POST['post_ids'] ) && ! empty( $_POST['post_ids'] ) ) ? $_POST['post_ids'] : array();
+	$post_ids = (isset($_POST['post_ids']) && ! empty($_POST['post_ids'])) ? $_POST['post_ids'] : array();
 
-	if ( ! empty( $post_ids ) && is_array( $post_ids ) ) {
-		$price = isset( $_POST['price'] ) ? strip_tags( stripslashes( $_POST['price'] ) ) : 0;
-		foreach ( $post_ids as $post_id ) {
+	if ( ! empty($post_ids) && is_array($post_ids)) {
+		$price = isset($_POST['price']) ? strip_tags(stripslashes($_POST['price'])) : 0;
+		foreach ($post_ids as $post_id) {
 
-			if ( ! current_user_can( 'edit_post', $post_id ) ) {
+			if ( ! current_user_can('edit_post', $post_id)) {
 				continue;
 			}
 
-			if ( ! empty( $price ) ) {
-				give_update_meta( $post_id, '_give_set_price', give_sanitize_amount_for_db( $price ) );
+			if ( ! empty($price)) {
+				give_update_meta($post_id, '_give_set_price', give_sanitize_amount_for_db($price));
 			}
 		}
 	}
@@ -362,4 +362,4 @@ 
 	die();
 }
 
-add_action( 'wp_ajax_give_save_bulk_edit', 'give_save_bulk_edit' );
+add_action('wp_ajax_give_save_bulk_edit', 'give_save_bulk_edit');

includes/admin/admin-filters.php

Spacing +60 added lines, -60 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -27,30 +27,30 @@ 
  *
  * @return  mixed
  */
-function __give_sanitize_number_decimals_setting_field( $value ) {
+function __give_sanitize_number_decimals_setting_field($value) {
 	$value_changed = false;
 	$old_value     = $value;
 
-	if ( isset( $_POST['decimal_separator'] ) ) {
-		$value         = ! empty( $_POST['decimal_separator'] ) ? $value : 0;
+	if (isset($_POST['decimal_separator'])) {
+		$value         = ! empty($_POST['decimal_separator']) ? $value : 0;
 		$value_changed = true;
 	}
 
-	if ( $value_changed && ( $old_value !== $value ) ) {
-		Give_Admin_Settings::add_error( 'give-number-decimal', __( 'The \'Number of Decimals\' option has been automatically set to zero because the \'Decimal Separator\' is not set.', 'give' ) );
+	if ($value_changed && ($old_value !== $value)) {
+		Give_Admin_Settings::add_error('give-number-decimal', __('The \'Number of Decimals\' option has been automatically set to zero because the \'Decimal Separator\' is not set.', 'give'));
 	}
 
-	$value = absint( $value );
+	$value = absint($value);
 
-	if ( 6 <= $value ) {
+	if (6 <= $value) {
 		$value = 5;
-		Give_Admin_Settings::add_error( 'give-number-decimal', __( 'The \'Number of Decimals\' option has been automatically set to 5 because you entered a number higher than the maximum allowed.', 'give' ) );
+		Give_Admin_Settings::add_error('give-number-decimal', __('The \'Number of Decimals\' option has been automatically set to 5 because you entered a number higher than the maximum allowed.', 'give'));
 	}
 
-	return absint( $value );
+	return absint($value);
 }
 
-add_filter( 'give_admin_settings_sanitize_option_number_decimals', '__give_sanitize_number_decimals_setting_field', 10 );
+add_filter('give_admin_settings_sanitize_option_number_decimals', '__give_sanitize_number_decimals_setting_field', 10);
 
 
 /**
@@ -66,20 +66,20 @@ 
  *
  * @return  mixed
  */
-function __give_validate_decimal_separator_setting_field( $value ) {
-	$thousand_separator = isset( $_POST['thousands_separator'] ) ? give_clean( $_POST['thousands_separator'] ) : '';
-	$decimal_separator  = isset( $_POST['decimal_separator'] ) ? give_clean( $_POST['decimal_separator'] ) : '';
+function __give_validate_decimal_separator_setting_field($value) {
+	$thousand_separator = isset($_POST['thousands_separator']) ? give_clean($_POST['thousands_separator']) : '';
+	$decimal_separator  = isset($_POST['decimal_separator']) ? give_clean($_POST['decimal_separator']) : '';
 
-	if ( $decimal_separator === $thousand_separator ) {
+	if ($decimal_separator === $thousand_separator) {
 		$value                    = '';
 		$_POST['number_decimals'] = 0;
-		Give_Admin_Settings::add_error( 'give-decimal-separator', __( 'The \'Decimal Separator\' option has automatically been set to empty because it can not be equal to the \'Thousand Separator\'', 'give' ) );
+		Give_Admin_Settings::add_error('give-decimal-separator', __('The \'Decimal Separator\' option has automatically been set to empty because it can not be equal to the \'Thousand Separator\'', 'give'));
 	}
 
 	return $value;
 }
 
-add_filter( 'give_admin_settings_sanitize_option_decimal_separator', '__give_validate_decimal_separator_setting_field', 10 );
+add_filter('give_admin_settings_sanitize_option_decimal_separator', '__give_validate_decimal_separator_setting_field', 10);
 
 /**
  * Change $delimiter text to symbol.
@@ -90,16 +90,16 @@ 
  *
  * @return string $delimiter.
  */
-function __give_import_delimiter_set_callback( $delimiter ) {
+function __give_import_delimiter_set_callback($delimiter) {
 	$delimite_type = array(
 		'csv'                  => ',',
 		'tab-separated-values' => "\t",
 	);
 
-	return ( array_key_exists( $delimiter, $delimite_type ) ? $delimite_type[ $delimiter ] : ',' );
+	return (array_key_exists($delimiter, $delimite_type) ? $delimite_type[$delimiter] : ',');
 }
 
-add_filter( 'give_import_delimiter_set', '__give_import_delimiter_set_callback', 10 );
+add_filter('give_import_delimiter_set', '__give_import_delimiter_set_callback', 10);
 
 /**
  * Give unset the page id from the core setting data from the json files.
@@ -111,17 +111,17 @@ 
  *
  * @return array $json_to_array
  */
-function give_import_core_settings_merge_pages( $json_to_array, $type ) {
-	if ( 'merge' === $type ) {
-		unset( $json_to_array['success_page'] );
-		unset( $json_to_array['failure_page'] );
-		unset( $json_to_array['history_page'] );
+function give_import_core_settings_merge_pages($json_to_array, $type) {
+	if ('merge' === $type) {
+		unset($json_to_array['success_page']);
+		unset($json_to_array['failure_page']);
+		unset($json_to_array['history_page']);
 	}
 
 	return $json_to_array;
 }
 
-add_filter( 'give_import_core_settings_data', 'give_import_core_settings_merge_pages', 11, 2 );
+add_filter('give_import_core_settings_data', 'give_import_core_settings_merge_pages', 11, 2);
 
 /**
  * Give check the image size from the core setting data from the json files.
@@ -133,18 +133,18 @@ 
  *
  * @return array $json_to_array
  */
-function give_import_core_settings_merge_image_size( $json_to_array, $type ) {
-	if ( 'merge' === $type ) {
+function give_import_core_settings_merge_image_size($json_to_array, $type) {
+	if ('merge' === $type) {
 		// Featured image sizes import under Display Options > Post Types > Featured Image Size.
 		if (
-			! empty( $json_to_array['form_featured_img'] )
-			&& ! empty( $json_to_array['featured_image_size'] )
-			&& give_is_setting_enabled( $json_to_array['form_featured_img'] )
+			! empty($json_to_array['form_featured_img'])
+			&& ! empty($json_to_array['featured_image_size'])
+			&& give_is_setting_enabled($json_to_array['form_featured_img'])
 		) {
 			$images_sizes = get_intermediate_image_sizes();
 
-			if ( ! in_array( $json_to_array['featured_image_size'], $images_sizes, true ) ) {
-				unset( $json_to_array['featured_image_size'] );
+			if ( ! in_array($json_to_array['featured_image_size'], $images_sizes, true)) {
+				unset($json_to_array['featured_image_size']);
 			}
 		}
 	}
@@ -152,7 +152,7 @@ 
 	return $json_to_array;
 }
 
-add_filter( 'give_import_core_settings_data', 'give_import_core_settings_merge_image_size', 12, 2 );
+add_filter('give_import_core_settings_data', 'give_import_core_settings_merge_image_size', 12, 2);
 
 /**
  * Give upload the image logo from the core setting data from the json files.
@@ -164,24 +164,24 @@ 
  *
  * @return array $json_to_array
  */
-function give_import_core_settings_merge_upload_image( $json_to_array, $type ) {
-	if ( 'merge' === $type ) {
+function give_import_core_settings_merge_upload_image($json_to_array, $type) {
+	if ('merge' === $type) {
 		// Emails > Email Settings > Logo.
-		if ( ! empty( $json_to_array['email_logo'] ) ) {
+		if ( ! empty($json_to_array['email_logo'])) {
 
 			// Need to require these files.
-			if ( ! function_exists( 'media_handle_upload' ) ) {
-				require_once( ABSPATH . 'wp-admin/includes/image.php' );
-				require_once( ABSPATH . 'wp-admin/includes/file.php' );
-				require_once( ABSPATH . 'wp-admin/includes/media.php' );
+			if ( ! function_exists('media_handle_upload')) {
+				require_once(ABSPATH.'wp-admin/includes/image.php');
+				require_once(ABSPATH.'wp-admin/includes/file.php');
+				require_once(ABSPATH.'wp-admin/includes/media.php');
 			}
 
 			$url     = $json_to_array['email_logo'];
-			$new_url = media_sideload_image( $url, 0, null, 'src' );
-			if ( ! is_wp_error( $new_url ) ) {
+			$new_url = media_sideload_image($url, 0, null, 'src');
+			if ( ! is_wp_error($new_url)) {
 				$json_to_array['email_logo'] = $new_url;
 			} else {
-				unset( $json_to_array['email_logo'] );
+				unset($json_to_array['email_logo']);
 			}
 		}
 	}
@@ -189,7 +189,7 @@ 
 	return $json_to_array;
 }
 
-add_filter( 'give_import_core_settings_data', 'give_import_core_settings_merge_upload_image', 13, 2 );
+add_filter('give_import_core_settings_data', 'give_import_core_settings_merge_upload_image', 13, 2);
 
 /**
  * Give unset the license key from the core setting data from the json files.
@@ -201,12 +201,12 @@ 
  *
  * @return array $json_to_array
  */
-function give_import_core_settings_merge_license_key( $json_to_array, $type ) {
-	if ( 'merge' === $type ) {
-		foreach ( $json_to_array as $key => $value ) {
-			$is_license_key = strpos( '_license_key', $key );
-			if ( ! empty( $is_license_key ) ) {
-				unset( $json_to_array[ $key ] );
+function give_import_core_settings_merge_license_key($json_to_array, $type) {
+	if ('merge' === $type) {
+		foreach ($json_to_array as $key => $value) {
+			$is_license_key = strpos('_license_key', $key);
+			if ( ! empty($is_license_key)) {
+				unset($json_to_array[$key]);
 			}
 		}
 	}
@@ -214,7 +214,7 @@ 
 	return $json_to_array;
 }
 
-add_filter( 'give_import_core_settings_data', 'give_import_core_settings_merge_license_key', 14, 2 );
+add_filter('give_import_core_settings_data', 'give_import_core_settings_merge_license_key', 14, 2);
 
 /**
  * Give merge the json data and setting data.
@@ -227,16 +227,16 @@ 
  *
  * @return array $json_to_array
  */
-function give_import_core_settings_merge_data( $json_to_array, $type, $host_give_options ) {
-	if ( 'merge' === $type ) {
-		$json_to_array_merge = array_merge( $host_give_options, $json_to_array );
+function give_import_core_settings_merge_data($json_to_array, $type, $host_give_options) {
+	if ('merge' === $type) {
+		$json_to_array_merge = array_merge($host_give_options, $json_to_array);
 		$json_to_array       = $json_to_array_merge;
 	}
 
 	return $json_to_array;
 }
 
-add_filter( 'give_import_core_settings_data', 'give_import_core_settings_merge_data', 1000, 3 );
+add_filter('give_import_core_settings_data', 'give_import_core_settings_merge_data', 1000, 3);
 
 /**
  * Backward Compatibility - Cleanup User Roles.
@@ -247,11 +247,11 @@ 
  *
  * @return mixed
  */
-function give_bc_1817_cleanup_user_roles( $caps ) {
+function give_bc_1817_cleanup_user_roles($caps) {
 
 	if (
-		! give_has_upgrade_completed( 'v1817_cleanup_user_roles' ) &&
-		! isset( $caps['view_give_payments'] )
+		! give_has_upgrade_completed('v1817_cleanup_user_roles') &&
+		! isset($caps['view_give_payments'])
 	) {
 		give_v1817_process_cleanup_user_roles();
 	}
@@ -259,4 +259,4 @@ 
 	return $caps;
 }
 
-add_filter( 'user_has_cap', 'give_bc_1817_cleanup_user_roles' );
+add_filter('user_has_cap', 'give_bc_1817_cleanup_user_roles');

includes/class-give-cli-commands.php

Spacing +221 added lines, -221 removed lines

@@ -7,12 +7,12 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Add give command.
-WP_CLI::add_command( 'give', 'GIVE_CLI_COMMAND' );
+WP_CLI::add_command('give', 'GIVE_CLI_COMMAND');
 
 
 /**
@@ -76,8 +76,8 @@ 
 	 *
 	 * @subcommand    logo
 	 */
-	public function ascii( $args, $assoc_args ) {
-		WP_CLI::log( file_get_contents( GIVE_PLUGIN_DIR . 'assets/dist/images/give-ascii-logo.txt' ) );
+	public function ascii($args, $assoc_args) {
+		WP_CLI::log(file_get_contents(GIVE_PLUGIN_DIR.'assets/dist/images/give-ascii-logo.txt'));
 	}
 
 
@@ -102,54 +102,54 @@ 
 	 *
 	 * @subcommand    details
 	 */
-	public function details( $args, $assoc_args ) {
+	public function details($args, $assoc_args) {
 
 		/**
 		 * Plugin Information
 		 */
-		WP_CLI::log( $this->color_message( __( 'Give Version: ', 'give' ) ) . GIVE_VERSION );
+		WP_CLI::log($this->color_message(__('Give Version: ', 'give')).GIVE_VERSION);
 
 		/**
 		 * General Information.
 		 */
-		WP_CLI::log( "\n####   " . $this->color_message( __( 'General information', 'give' ) ) . '   ####' );
+		WP_CLI::log("\n####   ".$this->color_message(__('General information', 'give')).'   ####');
 
-		$success_page = give_get_option( 'success_page' );
-		$failure_page = give_get_option( 'failure_page' );
-		$history_page = give_get_option( 'history_page' );
+		$success_page = give_get_option('success_page');
+		$failure_page = give_get_option('failure_page');
+		$history_page = give_get_option('history_page');
 
-		WP_CLI::log( $this->color_message( sprintf( __( 'Success Page: ', 'give' ) ) ) . ( $success_page ? "[{$success_page}] " . get_permalink( $success_page ) : __( 'Not Set', 'give' ) ) );
-		WP_CLI::log( $this->color_message( __( 'Failed Donation Page: ', 'give' ) ) . ( $failure_page ? "[{$failure_page}] " . get_permalink( $failure_page ) : __( 'Not Set', 'give' ) ) );
-		WP_CLI::log( $this->color_message( __( 'Donation History Page: ', 'give' ) ) . ( $history_page ? "[{$history_page}] " . get_permalink( $history_page ) : __( 'Not Set', 'give' ) ) );
-		WP_CLI::log( $this->color_message( __( 'Country: ', 'give' ) ) . give_get_country() );
+		WP_CLI::log($this->color_message(sprintf(__('Success Page: ', 'give'))).($success_page ? "[{$success_page}] ".get_permalink($success_page) : __('Not Set', 'give')));
+		WP_CLI::log($this->color_message(__('Failed Donation Page: ', 'give')).($failure_page ? "[{$failure_page}] ".get_permalink($failure_page) : __('Not Set', 'give')));
+		WP_CLI::log($this->color_message(__('Donation History Page: ', 'give')).($history_page ? "[{$history_page}] ".get_permalink($history_page) : __('Not Set', 'give')));
+		WP_CLI::log($this->color_message(__('Country: ', 'give')).give_get_country());
 
 		/**
 		 * Currency Information.
 		 */
-		$default_gateway = give_get_option( 'default_gateway' );
+		$default_gateway = give_get_option('default_gateway');
 
-		WP_CLI::log( "\n####   " . $this->color_message( __( 'Currency Information', 'give' ) ) . '   ####' );
+		WP_CLI::log("\n####   ".$this->color_message(__('Currency Information', 'give')).'   ####');
 
-		WP_CLI::log( $this->color_message( __( 'Currency: ', 'give' ), give_get_currency() ) );
-		WP_CLI::log( $this->color_message( __( 'Currency Position: ', 'give' ), give_get_currency_position() ) );
-		WP_CLI::log( $this->color_message( __( 'Thousand Separator: ', 'give' ), give_get_price_thousand_separator() ) );
-		WP_CLI::log( $this->color_message( __( 'Decimal Separator: ', 'give' ), give_get_price_decimal_separator() ) );
-		WP_CLI::log( $this->color_message( __( 'Number of Decimals: ', 'give' ), give_get_price_decimals() ) );
-		WP_CLI::log( $this->color_message( __( 'Test Mode: ', 'give' ), ( give_get_option( 'test_mode' ) ? __( 'Yes', 'give' ) : __( 'No', 'give' ) ) ) );
-		WP_CLI::log( $this->color_message( __( 'Default Gateway: ', 'give' ), ( $default_gateway ? $default_gateway : __( 'Not Set', 'give' ) ) ) );
+		WP_CLI::log($this->color_message(__('Currency: ', 'give'), give_get_currency()));
+		WP_CLI::log($this->color_message(__('Currency Position: ', 'give'), give_get_currency_position()));
+		WP_CLI::log($this->color_message(__('Thousand Separator: ', 'give'), give_get_price_thousand_separator()));
+		WP_CLI::log($this->color_message(__('Decimal Separator: ', 'give'), give_get_price_decimal_separator()));
+		WP_CLI::log($this->color_message(__('Number of Decimals: ', 'give'), give_get_price_decimals()));
+		WP_CLI::log($this->color_message(__('Test Mode: ', 'give'), (give_get_option('test_mode') ? __('Yes', 'give') : __('No', 'give'))));
+		WP_CLI::log($this->color_message(__('Default Gateway: ', 'give'), ($default_gateway ? $default_gateway : __('Not Set', 'give'))));
 
 		// Payment gateways Information.
-		$gateways = give_get_ordered_payment_gateways( give_get_payment_gateways() );
-		WP_CLI::log( $this->color_message( __( 'Enabled Gateways: ', 'give' ) ) );
+		$gateways = give_get_ordered_payment_gateways(give_get_payment_gateways());
+		WP_CLI::log($this->color_message(__('Enabled Gateways: ', 'give')));
 
-		if ( ! empty( $gateways ) ) {
+		if ( ! empty($gateways)) {
 			self::$counter = 1;
-			foreach ( $gateways as $gateway ) {
-				WP_CLI::log( '  ' . $this->color_message( self::$counter, $gateway['admin_label'] ) );
-				self::$counter ++;
+			foreach ($gateways as $gateway) {
+				WP_CLI::log('  '.$this->color_message(self::$counter, $gateway['admin_label']));
+				self::$counter++;
 			}
 		} else {
-			WP_CLI::log( __( 'Not any payment gateways found', 'give' ) );
+			WP_CLI::log(__('Not any payment gateways found', 'give'));
 		}
 	}
 
@@ -181,15 +181,15 @@ 
 	 *
 	 * @subcommand    forms
 	 */
-	public function forms( $args, $assoc_args ) {
+	public function forms($args, $assoc_args) {
 		global $wp_query;
-		$form_id = isset( $assoc_args ) && array_key_exists( 'id', $assoc_args ) ? absint( $assoc_args['id'] ) : false;
-		$number  = isset( $assoc_args ) && array_key_exists( 'number', $assoc_args ) ? absint( $assoc_args['number'] ) : 10;
+		$form_id = isset($assoc_args) && array_key_exists('id', $assoc_args) ? absint($assoc_args['id']) : false;
+		$number  = isset($assoc_args) && array_key_exists('number', $assoc_args) ? absint($assoc_args['number']) : 10;
 		$start   = time();
 
 		// Cache previous number query var.
 		$is_set_number = $cache_per_page = false;
-		if ( isset( $wp_query->query_vars['number'] ) ) {
+		if (isset($wp_query->query_vars['number'])) {
 			$cache_per_page = $wp_query->query_vars['number'];
 			$is_set_number  = true;
 		}
@@ -198,22 +198,22 @@ 
 		$wp_query->query_vars['number'] = $number;
 
 		// Get forms.
-		$forms = $form_id ? $this->api->get_forms( $form_id ) : $this->api->get_forms();
+		$forms = $form_id ? $this->api->get_forms($form_id) : $this->api->get_forms();
 
 		// Reset number query var.
-		if ( $is_set_number ) {
+		if ($is_set_number) {
 			$wp_query->query_vars['number'] = $cache_per_page;
 		}
 
 		// Bailout.
-		if ( array_key_exists( 'error', $forms ) ) {
+		if (array_key_exists('error', $forms)) {
 
-			WP_CLI::warning( $forms['error'] );
+			WP_CLI::warning($forms['error']);
 
 			return;
-		} elseif ( empty( $forms['forms'] ) ) {
+		} elseif (empty($forms['forms'])) {
 
-			WP_CLI::error( __( 'No forms found.', 'give' ) );
+			WP_CLI::error(__('No forms found.', 'give'));
 
 			return;
 		}
@@ -221,25 +221,25 @@ 
 		// Param to check if form typeis already showed or not.
 		$is_show_form_type = false;
 
-		if ( 1 === count( $forms ) && $form_id ) {
+		if (1 === count($forms) && $form_id) {
 			// Show single form.
-			foreach ( $forms['forms'][0] as $key => $info ) {
-				switch ( $key ) {
+			foreach ($forms['forms'][0] as $key => $info) {
+				switch ($key) {
 					case 'stats':
-						$this->color_main_heading( ucfirst( $key ) );
+						$this->color_main_heading(ucfirst($key));
 
-						foreach ( $info as $heading => $data ) {
-							$this->color_sub_heading( ucfirst( $heading ) );
-							switch ( $heading ) {
+						foreach ($info as $heading => $data) {
+							$this->color_sub_heading(ucfirst($heading));
+							switch ($heading) {
 								default:
-									foreach ( $data as $subheading => $subdata ) {
+									foreach ($data as $subheading => $subdata) {
 
-										switch ( $subheading ) {
+										switch ($subheading) {
 											case 'earnings':
-												WP_CLI::log( $this->color_message( $subheading . ': ', give_currency_filter( $subdata ) ) );
+												WP_CLI::log($this->color_message($subheading.': ', give_currency_filter($subdata)));
 												break;
 											default:
-												WP_CLI::log( $this->color_message( $subheading . ': ', $subdata ) );
+												WP_CLI::log($this->color_message($subheading.': ', $subdata));
 										}
 									}
 							}
@@ -249,26 +249,26 @@ 
 					case 'pricing':
 					case 'info':
 					default:
-						$this->color_main_heading( ucfirst( $key ) );
+						$this->color_main_heading(ucfirst($key));
 
 						// Show form type.
-						if ( ! $is_show_form_type ) {
-							$form              = new Give_Donate_Form( $form_id );
+						if ( ! $is_show_form_type) {
+							$form              = new Give_Donate_Form($form_id);
 							$is_show_form_type = true;
 
-							WP_CLI::log( $this->color_message( __( 'form type', 'give' ), $form->get_type() ) );
+							WP_CLI::log($this->color_message(__('form type', 'give'), $form->get_type()));
 						}
 
-						foreach ( $info as $heading => $data ) {
+						foreach ($info as $heading => $data) {
 
-							switch ( $heading ) {
+							switch ($heading) {
 								case 'id':
-									WP_CLI::log( $this->color_message( $heading, $data ) );
+									WP_CLI::log($this->color_message($heading, $data));
 									break;
 
 								default:
-									$data = empty( $data ) ? __( 'Not set', 'give' ) : $data;
-									WP_CLI::log( $this->color_message( $heading, $data ) );
+									$data = empty($data) ? __('Not set', 'give') : $data;
+									WP_CLI::log($this->color_message($heading, $data));
 							}
 						}
 				}// End switch().
@@ -279,37 +279,37 @@ 
 			$is_table_first_row_set = false;
 			$table_column_count     = 0;
 
-			WP_CLI::line( $this->color_message( sprintf( __( '%d donation forms found', 'give' ), count( $forms['forms'] ) ), '', false ) );
+			WP_CLI::line($this->color_message(sprintf(__('%d donation forms found', 'give'), count($forms['forms'])), '', false));
 
-			foreach ( $forms['forms'] as $index => $form_data ) {
+			foreach ($forms['forms'] as $index => $form_data) {
 
 				// Default table data.
 				$table_first_row = array();
 				$table_row       = array();
 
-				foreach ( $form_data['info'] as $key => $form ) {
+				foreach ($form_data['info'] as $key => $form) {
 
 					// Do not show thumbnail, content and link in table.
-					if ( in_array( $key, array( 'content', 'thumbnail', 'link' ), true ) ) {
+					if (in_array($key, array('content', 'thumbnail', 'link'), true)) {
 						continue;
 					}
 
-					if ( ! $is_table_first_row_set ) {
+					if ( ! $is_table_first_row_set) {
 						$table_first_row[] = $key;
 					}
 
 					$table_row[] = $form;
 
-					if ( 'status' === $key ) {
+					if ('status' === $key) {
 						// First array item will be an form id in our case.
-						$form = new Give_Donate_Form( absint( $table_row[0] ) );
+						$form = new Give_Donate_Form(absint($table_row[0]));
 
 						$table_row[] = $form->get_type();
 					}
 				}
 
 				// Set table first row.
-				if ( ! $is_table_first_row_set ) {
+				if ( ! $is_table_first_row_set) {
 
 					// Add extra column to table.
 					$table_first_row[] = 'type';
@@ -322,7 +322,7 @@ 
 				$table_data[] = $table_row;
 			}// End foreach().
 
-			$this->display_table( $table_data );
+			$this->display_table($table_data);
 		}// End if().
 	}
 
@@ -375,59 +375,59 @@ 
 	 *
 	 * @subcommand    donors
 	 */
-	public function donors( $args, $assoc_args ) {
+	public function donors($args, $assoc_args) {
 		global $wp_query;
-		$donor_id = isset( $assoc_args ) && array_key_exists( 'id', $assoc_args ) ? absint( $assoc_args['id'] ) : false;
-		$email    = isset( $assoc_args ) && array_key_exists( 'email', $assoc_args ) ? $assoc_args['email'] : false;
-		$name     = isset( $assoc_args ) && array_key_exists( 'name', $assoc_args ) ? $assoc_args['name'] : '';
-		$create   = isset( $assoc_args ) && array_key_exists( 'create', $assoc_args ) ? $assoc_args['create'] : false;
-		$number   = isset( $assoc_args ) && array_key_exists( 'number', $assoc_args ) ? $assoc_args['number'] : 10;
-		$form_id  = isset( $assoc_args ) && array_key_exists( 'form-id', $assoc_args ) ? $assoc_args['form-id'] : 0;
-		$format   = isset( $assoc_args ) && array_key_exists( 'format', $assoc_args ) ? $assoc_args['format'] : 'table';
+		$donor_id = isset($assoc_args) && array_key_exists('id', $assoc_args) ? absint($assoc_args['id']) : false;
+		$email    = isset($assoc_args) && array_key_exists('email', $assoc_args) ? $assoc_args['email'] : false;
+		$name     = isset($assoc_args) && array_key_exists('name', $assoc_args) ? $assoc_args['name'] : '';
+		$create   = isset($assoc_args) && array_key_exists('create', $assoc_args) ? $assoc_args['create'] : false;
+		$number   = isset($assoc_args) && array_key_exists('number', $assoc_args) ? $assoc_args['number'] : 10;
+		$form_id  = isset($assoc_args) && array_key_exists('form-id', $assoc_args) ? $assoc_args['form-id'] : 0;
+		$format   = isset($assoc_args) && array_key_exists('format', $assoc_args) ? $assoc_args['format'] : 'table';
 		$start    = time();
 
-		if ( $create ) {
-			if ( 80 < $create ) {
-				WP_CLI::warning( 'Currently we can only generate maximum 80 donors.', 'give' );
+		if ($create) {
+			if (80 < $create) {
+				WP_CLI::warning('Currently we can only generate maximum 80 donors.', 'give');
 				$create = 80;
 			}
 
 			$number = 1;
 
-			if ( isset( $assoc_args['email'] ) && ! is_email( $email ) ) {
-				WP_CLI::warning( 'Wrong email address provided.', 'give' );
+			if (isset($assoc_args['email']) && ! is_email($email)) {
+				WP_CLI::warning('Wrong email address provided.', 'give');
 
 				return;
 			}
 
 			// Create one or more donors.
-			if ( ! $email ) {
+			if ( ! $email) {
 				// If no email is specified, look to see if we are generating arbitrary donor accounts.
-				$number = is_numeric( $create ) ? absint( $create ) : 1;
+				$number = is_numeric($create) ? absint($create) : 1;
 			}
 
-			for ( $i = 0; $i < $number; $i ++ ) {
+			for ($i = 0; $i < $number; $i++) {
 				$name  = $name ? $name : $this->get_random_name();
-				$email = $email ? $email : $this->get_random_email( $name );
+				$email = $email ? $email : $this->get_random_email($name);
 
 				$args = array(
 					'email' => $email,
 					'name'  => $name,
 				);
 
-				$donor_id = Give()->donors->add( $args );
+				$donor_id = Give()->donors->add($args);
 
-				if ( $donor_id ) {
-					WP_CLI::line( $this->color_message( sprintf( __( 'Donor #%d created successfully', 'give' ), $donor_id ) ) );
+				if ($donor_id) {
+					WP_CLI::line($this->color_message(sprintf(__('Donor #%d created successfully', 'give'), $donor_id)));
 				} else {
-					WP_CLI::error( __( 'Failed to create donor', 'give' ) );
+					WP_CLI::error(__('Failed to create donor', 'give'));
 				}
 
 				// Reset email and name to false so it is generated on the next loop (if creating donors).
 				$email = $name = false;
 			}
 
-			WP_CLI::line( $this->color_message( sprintf( __( '%1$d donors created in %2$d seconds', 'give' ), $number, time() - $start ) ) );
+			WP_CLI::line($this->color_message(sprintf(__('%1$d donors created in %2$d seconds', 'give'), $number, time() - $start)));
 
 		} else {
 			// Counter.
@@ -441,7 +441,7 @@ 
 			 */
 			// Cache previous number query var.
 			$is_set_number = $cache_per_page = false;
-			if ( isset( $wp_query->query_vars['number'] ) ) {
+			if (isset($wp_query->query_vars['number'])) {
 				$cache_per_page = $wp_query->query_vars['number'];
 				$is_set_number  = true;
 			}
@@ -450,24 +450,24 @@ 
 			$wp_query->query_vars['number'] = $number;
 
 			// Get donors.
-			if ( $form_id ) {
+			if ($form_id) {
 				// @TODO: Allow user to get a list of donors by donation status.
-				$donors = $this->get_donors_by_form_id( $form_id );
+				$donors = $this->get_donors_by_form_id($form_id);
 			} else {
-				$donors = $this->api->get_donors( $search );
+				$donors = $this->api->get_donors($search);
 			}
 
 			// Reset number query var.
-			if ( $is_set_number ) {
+			if ($is_set_number) {
 				$wp_query->query_vars['number'] = $cache_per_page;
 			}
 
-			if ( isset( $donors['error'] ) ) {
-				WP_CLI::error( $donors['error'] );
+			if (isset($donors['error'])) {
+				WP_CLI::error($donors['error']);
 			}
 
-			if ( empty( $donors ) ) {
-				WP_CLI::error( __( 'No donors found.', 'give' ) );
+			if (empty($donors)) {
+				WP_CLI::error(__('No donors found.', 'give'));
 
 				return;
 			}
@@ -475,24 +475,24 @@ 
 			$table_data             = array();
 			$is_table_first_row_set = false;
 
-			foreach ( $donors['donors'] as $donor_data ) {
+			foreach ($donors['donors'] as $donor_data) {
 				// Set default table row data.
-				$table_first_row = array( __( 's_no', 'give' ) );
-				$table_row       = array( self::$counter );
+				$table_first_row = array(__('s_no', 'give'));
+				$table_row       = array(self::$counter);
 
-				foreach ( $donor_data as $key => $donor ) {
-					switch ( $key ) {
+				foreach ($donor_data as $key => $donor) {
+					switch ($key) {
 						case 'stats':
-							foreach ( $donor as $heading => $data ) {
+							foreach ($donor as $heading => $data) {
 
 								// Get first row.
-								if ( ! $is_table_first_row_set ) {
+								if ( ! $is_table_first_row_set) {
 									$table_first_row[] = $heading;
 								}
 
-								switch ( $heading ) {
+								switch ($heading) {
 									case 'total_spent':
-										$table_row[] = give_currency_filter( $data );
+										$table_row[] = give_currency_filter($data);
 										break;
 
 									default:
@@ -503,10 +503,10 @@ 
 
 						case 'info':
 						default:
-							foreach ( $donor as $heading => $data ) {
+							foreach ($donor as $heading => $data) {
 
 								// Get first row.
-								if ( ! $is_table_first_row_set ) {
+								if ( ! $is_table_first_row_set) {
 									$table_first_row[] = $heading;
 								}
 
@@ -516,7 +516,7 @@ 
 				}
 
 				// Add first row data to table data.
-				if ( ! $is_table_first_row_set ) {
+				if ( ! $is_table_first_row_set) {
 					$table_data[]           = $table_first_row;
 					$is_table_first_row_set = true;
 				}
@@ -525,44 +525,44 @@ 
 				$table_data[] = $table_row;
 
 				// Increase counter.
-				self::$counter ++;
+				self::$counter++;
 			}// End foreach().
 
-			switch ( $format ) {
+			switch ($format) {
 				case 'json':
 					$table_column_name = $table_data[0];
-					unset( $table_data[0] );
+					unset($table_data[0]);
 
 					$new_table_data = array();
-					foreach ( $table_data as $index => $data ) {
-						foreach ( $data as $key => $value ) {
-							$new_table_data[ $index ][ $table_column_name[ $key ] ] = $value;
+					foreach ($table_data as $index => $data) {
+						foreach ($data as $key => $value) {
+							$new_table_data[$index][$table_column_name[$key]] = $value;
 						}
 					}
 
-					WP_CLI::log( json_encode( $new_table_data ) );
+					WP_CLI::log(json_encode($new_table_data));
 					break;
 
 				case 'csv':
-					$file_path = trailingslashit( WP_CONTENT_DIR ) . 'uploads/give_donors_' . date( 'Y_m_d_s', current_time( 'timestamp' ) ) . '.csv';
-					$fp        = fopen( $file_path, 'w' );
+					$file_path = trailingslashit(WP_CONTENT_DIR).'uploads/give_donors_'.date('Y_m_d_s', current_time('timestamp')).'.csv';
+					$fp        = fopen($file_path, 'w');
 
-					if ( is_writable( $file_path ) ) {
-						foreach ( $table_data as $fields ) {
-							fputcsv( $fp, $fields );
+					if (is_writable($file_path)) {
+						foreach ($table_data as $fields) {
+							fputcsv($fp, $fields);
 						}
 
-						fclose( $fp );
+						fclose($fp);
 
-						WP_CLI::success( "Donors list csv created successfully: {$file_path}" );
+						WP_CLI::success("Donors list csv created successfully: {$file_path}");
 					} else {
-						WP_CLI::warning( "Unable to create donors list csv file: {$file_path} (May folder do not have write permission)" );
+						WP_CLI::warning("Unable to create donors list csv file: {$file_path} (May folder do not have write permission)");
 					}
 
 					break;
 
 				default:
-					$this->display_table( $table_data );
+					$this->display_table($table_data);
 			}// End switch().
 		}// End if().
 	}
@@ -592,13 +592,13 @@ 
 	 *
 	 * @subcommand    donations
 	 */
-	public function donations( $args, $assoc_args ) {
+	public function donations($args, $assoc_args) {
 		global $wp_query;
-		$number = isset( $assoc_args ) && array_key_exists( 'number', $assoc_args ) ? $assoc_args['number'] : 10;
+		$number = isset($assoc_args) && array_key_exists('number', $assoc_args) ? $assoc_args['number'] : 10;
 
 		// Cache previous number query var.
 		$is_set_number = $cache_per_page = false;
-		if ( isset( $wp_query->query_vars['number'] ) ) {
+		if (isset($wp_query->query_vars['number'])) {
 			$cache_per_page = $wp_query->query_vars['number'];
 			$is_set_number  = true;
 		}
@@ -610,46 +610,46 @@ 
 		$donations = $this->api->get_recent_donations();
 
 		// Reset number query var.
-		if ( $is_set_number ) {
+		if ($is_set_number) {
 			$wp_query->query_vars['number'] = $cache_per_page;
 		}
 
-		if ( empty( $donations ) ) {
-			WP_CLI::error( __( 'No donations found.', 'give' ) );
+		if (empty($donations)) {
+			WP_CLI::error(__('No donations found.', 'give'));
 
 			return;
 		}
 
 		self::$counter = 1;
 
-		foreach ( $donations['donations'] as $key => $donation ) {
-			$this->color_main_heading( sprintf( __( '%1$s. Donation #%2$s', 'give' ), self::$counter, $donation['ID'] ), 'Y' );
-			self::$counter ++;
+		foreach ($donations['donations'] as $key => $donation) {
+			$this->color_main_heading(sprintf(__('%1$s. Donation #%2$s', 'give'), self::$counter, $donation['ID']), 'Y');
+			self::$counter++;
 
-			foreach ( $donation as $column => $data ) {
+			foreach ($donation as $column => $data) {
 
-				if ( is_array( $data ) ) {
-					$this->color_sub_heading( $column );
-					foreach ( $data as $subcolumn => $subdata ) {
+				if (is_array($data)) {
+					$this->color_sub_heading($column);
+					foreach ($data as $subcolumn => $subdata) {
 
 						// Decode html codes.
-						switch ( $subcolumn ) {
+						switch ($subcolumn) {
 							case 'name':
-								$subdata = html_entity_decode( $subdata );
+								$subdata = html_entity_decode($subdata);
 								break;
 						}
 
 						// @TODO Check if multi dimension array information is importent to show or not. For example inside donation array we have array for fees data inside payment meta.
-						if ( is_array( $subdata ) ) {
+						if (is_array($subdata)) {
 							continue;
 						}
 
-						WP_CLI::log( $this->color_message( $subcolumn, $subdata ) );
+						WP_CLI::log($this->color_message($subcolumn, $subdata));
 					}
 					continue;
 				}
 
-				WP_CLI::log( $this->color_message( $column, $data ) );
+				WP_CLI::log($this->color_message($column, $data));
 			}
 		}
 	}
@@ -689,27 +689,27 @@ 
 	 *
 	 * @return        void
 	 */
-	public function report( $args, $assoc_args ) {
+	public function report($args, $assoc_args) {
 		$stats      = new Give_Payment_Stats();
-		$date       = isset( $assoc_args ) && array_key_exists( 'date', $assoc_args ) ? $assoc_args['date'] : false;
-		$start_date = isset( $assoc_args ) && array_key_exists( 'start-date', $assoc_args ) ? $assoc_args['start-date'] : false;
-		$end_date   = isset( $assoc_args ) && array_key_exists( 'end-date', $assoc_args ) ? $assoc_args['end-date'] : false;
-		$form_id    = isset( $assoc_args ) && array_key_exists( 'id', $assoc_args ) ? $assoc_args['id'] : 0;
+		$date       = isset($assoc_args) && array_key_exists('date', $assoc_args) ? $assoc_args['date'] : false;
+		$start_date = isset($assoc_args) && array_key_exists('start-date', $assoc_args) ? $assoc_args['start-date'] : false;
+		$end_date   = isset($assoc_args) && array_key_exists('end-date', $assoc_args) ? $assoc_args['end-date'] : false;
+		$form_id    = isset($assoc_args) && array_key_exists('id', $assoc_args) ? $assoc_args['id'] : 0;
 
-		if ( ! empty( $date ) ) {
+		if ( ! empty($date)) {
 			$start_date = $date;
 			$end_date   = false;
-		} elseif ( empty( $date ) && empty( $start_date ) ) {
+		} elseif (empty($date) && empty($start_date)) {
 			$start_date = 'this_month';
 			$end_date   = false;
 		}
 
 		// Get stats.
-		$earnings = $stats->get_earnings( $form_id, $start_date, $end_date );
-		$sales    = $stats->get_sales( $form_id, $start_date, $end_date );
+		$earnings = $stats->get_earnings($form_id, $start_date, $end_date);
+		$sales    = $stats->get_sales($form_id, $start_date, $end_date);
 
-		WP_CLI::line( $this->color_message( __( 'Earnings', 'give' ), give_currency_filter( $earnings, array( 'decode_currency' => true ) ) ) );
-		WP_CLI::line( $this->color_message( __( 'Sales', 'give' ), $sales ) );
+		WP_CLI::line($this->color_message(__('Earnings', 'give'), give_currency_filter($earnings, array('decode_currency' => true))));
+		WP_CLI::line($this->color_message(__('Sales', 'give'), $sales));
 	}
 
 
@@ -736,27 +736,27 @@ 
 	 *
 	 * @subcommand    cache
 	 */
-	public function cache( $args, $assoc_args ) {
-		$action = isset( $assoc_args ) && array_key_exists( 'action', $assoc_args ) ? $assoc_args['action'] : false;
+	public function cache($args, $assoc_args) {
+		$action = isset($assoc_args) && array_key_exists('action', $assoc_args) ? $assoc_args['action'] : false;
 
 		// Bailout.
-		if ( ! $action || ! in_array( $action, array( 'delete' ), true ) ) {
-			WP_CLI::warning( __( 'Type wp give cache --action=delete to delete all stat transients', 'give' ) );
+		if ( ! $action || ! in_array($action, array('delete'), true)) {
+			WP_CLI::warning(__('Type wp give cache --action=delete to delete all stat transients', 'give'));
 
 			return;
 		}
 
-		switch ( $action ) {
+		switch ($action) {
 			case 'delete':
 				// Reset counter.
 				self::$counter = 1;
 
-				if ( $this->delete_stats_transients() ) {
+				if ($this->delete_stats_transients()) {
 					// Report .eading.
-					WP_CLI::success( 'Give cache deleted.' );
+					WP_CLI::success('Give cache deleted.');
 				} else {
 					// Report .eading.
-					WP_CLI::warning( 'We did not find any Give plugin cache to delete :)' );
+					WP_CLI::warning('We did not find any Give plugin cache to delete :)');
 				}
 				break;
 		}
@@ -785,27 +785,27 @@ 
 			ARRAY_A
 		);
 
-		if ( ! empty( $stat_option_names ) ) {
+		if ( ! empty($stat_option_names)) {
 
-			foreach ( $stat_option_names as $option_name ) {
+			foreach ($stat_option_names as $option_name) {
 				$error       = false;
 				$option_name = $option_name['option_name'];
 
-				switch ( true ) {
-					case ( false !== strpos( $option_name, 'transient' ) ):
-						$option_name = str_replace( '_transient_', '', $option_name );
-						$error       = delete_transient( $option_name );
+				switch (true) {
+					case (false !== strpos($option_name, 'transient')):
+						$option_name = str_replace('_transient_', '', $option_name);
+						$error       = delete_transient($option_name);
 						break;
 
 					default:
-						$error = delete_option( $option_name );
+						$error = delete_option($option_name);
 				}
 
-				if ( $error ) {
-					WP_CLI::log( $this->color_message( self::$counter, $option_name ) );
-					self::$counter ++;
+				if ($error) {
+					WP_CLI::log($this->color_message(self::$counter, $option_name));
+					self::$counter++;
 				} else {
-					WP_CLI::log( $this->color_message( __( 'Error while deleting this transient', 'give' ), $option_name ) );
+					WP_CLI::log($this->color_message(__('Error while deleting this transient', 'give'), $option_name));
 				}
 			}
 
@@ -826,13 +826,13 @@ 
 	 *
 	 * @return   string
 	 */
-	private function color_message( $heading, $message = '', $colon = true, $color = 'g' ) {
+	private function color_message($heading, $message = '', $colon = true, $color = 'g') {
 		// Add colon.
-		if ( $colon ) {
-			$heading = $heading . ': ';
+		if ($colon) {
+			$heading = $heading.': ';
 		}
 
-		return WP_CLI::colorize( "%{$color}" . $heading . '%n' ) . $message;
+		return WP_CLI::colorize("%{$color}".$heading.'%n').$message;
 	}
 
 
@@ -847,8 +847,8 @@ 
 	 *
 	 * @return    void
 	 */
-	private function color_main_heading( $heading, $color = 'g' ) {
-		WP_CLI::log( "\n######   " . $this->color_message( $heading, '', false, $color ) . '   ######' );
+	private function color_main_heading($heading, $color = 'g') {
+		WP_CLI::log("\n######   ".$this->color_message($heading, '', false, $color).'   ######');
 	}
 
 	/**
@@ -861,8 +861,8 @@ 
 	 *
 	 * @return    void
 	 */
-	private function color_sub_heading( $subheading ) {
-		WP_CLI::log( "\n--->" . $subheading . '', '', false );
+	private function color_sub_heading($subheading) {
+		WP_CLI::log("\n--->".$subheading.'', '', false);
 	}
 
 
@@ -876,17 +876,17 @@ 
 	 *
 	 * @return    void
 	 */
-	private function display_table( $data ) {
+	private function display_table($data) {
 		$table = new \cli\Table();
 
 		// Set table header.
-		$table->setHeaders( $data[0] );
+		$table->setHeaders($data[0]);
 
 		// Remove table header.
-		unset( $data[0] );
+		unset($data[0]);
 
 		// Set table data.
-		$table->setRows( $data );
+		$table->setRows($data);
 
 		// Display table.
 		$table->display();
@@ -903,14 +903,14 @@ 
 	 * @return array
 	 */
 
-	private function get_donors_by_form_id( $form_id ) {
+	private function get_donors_by_form_id($form_id) {
 		$donors = array();
 
 		$donations = new Give_Payments_Query(
 			array(
-				'give_forms' => array( $form_id ),
-				'number'     => - 1,
-				'status'     => array( 'publish' ),
+				'give_forms' => array($form_id),
+				'number'     => -1,
+				'status'     => array('publish'),
 			)
 		);
 
@@ -918,16 +918,16 @@ 
 		$skip_donors = array();
 
 		/* @var Give_Payment|object $donation Payment object. */
-		foreach ( $donations as $donation ) {
+		foreach ($donations as $donation) {
 
-			if ( in_array( $donation->customer_id, $skip_donors, true ) ) {
+			if (in_array($donation->customer_id, $skip_donors, true)) {
 				continue;
 			}
 
-			if ( ! empty( $donors ) ) {
-				$donors['donors'][] = current( current( $this->api->get_donors( (int) $donation->customer_id ) ) );
+			if ( ! empty($donors)) {
+				$donors['donors'][] = current(current($this->api->get_donors((int) $donation->customer_id)));
 			} else {
-				$donors = array_merge( $donors, $this->api->get_donors( (int) $donation->customer_id ) );
+				$donors = array_merge($donors, $this->api->get_donors((int) $donation->customer_id));
 			}
 
 			$skip_donors[] = $donation->customer_id;
@@ -972,15 +972,15 @@ 
 		);
 
 		// Generate a random forename.
-		$random_name = $names[ mt_rand( 0, sizeof( $names ) - 1 ) ];
+		$random_name = $names[mt_rand(0, sizeof($names) - 1)];
 
 		// Generate a random surname.
-		$random_surname = $surnames[ mt_rand( 0, sizeof( $surnames ) - 1 ) ];
+		$random_surname = $surnames[mt_rand(0, sizeof($surnames) - 1)];
 
 		// Generate name.
 		$name = "{$random_name} {$random_surname}";
 
-		if ( in_array( $name, $this->new_donor_names ) ) {
+		if (in_array($name, $this->new_donor_names)) {
 			$name = $this->get_random_name();
 		}
 
@@ -999,8 +999,8 @@ 
 	 *
 	 * @return string
 	 */
-	private function get_random_email( $name ) {
-		return implode( '.', explode( ' ', strtolower( $name ) ) ) . '@test.com';
+	private function get_random_email($name) {
+		return implode('.', explode(' ', strtolower($name))).'@test.com';
 	}
 
 	/**
@@ -1015,59 +1015,59 @@ 
 	 * @when after_wp_load
 	 * @subcommand test-mode
 	 */
-	public function test_mode( $args, $assoc ) {
+	public function test_mode($args, $assoc) {
 
 		// Return if associative arguments are not specified.
-		if ( empty( $assoc ) ) {
-			WP_CLI::error( "--enable or --disable flag is missing." );
+		if (empty($assoc)) {
+			WP_CLI::error("--enable or --disable flag is missing.");
 			return;
 		}
 
-		$enabled_gateways = give_get_option( 'gateways' );
-		$default_gateway  = give_get_option( 'default_gateway' );
+		$enabled_gateways = give_get_option('gateways');
+		$default_gateway  = give_get_option('default_gateway');
 
 
 		// Enable Test Mode.
-		if ( true === WP_CLI\Utils\get_flag_value( $assoc, 'enable' ) ) {
+		if (true === WP_CLI\Utils\get_flag_value($assoc, 'enable')) {
 
 			// Set `Test Mode` to `enabled`.
-			give_update_option( 'test_mode', 'enabled' );
+			give_update_option('test_mode', 'enabled');
 
 
 			// Enable `Test Donation` gateway.
 			$enabled_gateways['manual'] = "1";
-			give_update_option( 'gateways', $enabled_gateways );
+			give_update_option('gateways', $enabled_gateways);
 
 
 			// Set `Test Donation` as default gateway.
-			add_option( 'give_test_mode_default_gateway', $default_gateway );
-			give_update_option( 'default_gateway', 'manual' );
+			add_option('give_test_mode_default_gateway', $default_gateway);
+			give_update_option('default_gateway', 'manual');
 
 
 			// Show success message on completion.
-			WP_CLI::success( 'Give Test mode enabled' );
+			WP_CLI::success('Give Test mode enabled');
 		}
 
 		// Disable Test Mode.
-		if ( true === WP_CLI\Utils\get_flag_value( $assoc, 'disable' ) ) {
+		if (true === WP_CLI\Utils\get_flag_value($assoc, 'disable')) {
 
 			// Set `Test Mode` to `disabled`.
-			give_update_option( 'test_mode', 'disabled' );
+			give_update_option('test_mode', 'disabled');
 
 
 			// Disable `Test Donation` gateway.
-			unset( $enabled_gateways['manual'] );
-			give_update_option( 'gateways', $enabled_gateways );
+			unset($enabled_gateways['manual']);
+			give_update_option('gateways', $enabled_gateways);
 
 
 			// Backup `Default Gateway` setting for restore on test mode disable.
-			$default_gateway_backup = get_option( 'give_test_mode_default_gateway' );
-			give_update_option( 'default_gateway', $default_gateway_backup );
-			delete_option( 'give_test_mode_default_gateway' );
+			$default_gateway_backup = get_option('give_test_mode_default_gateway');
+			give_update_option('default_gateway', $default_gateway_backup);
+			delete_option('give_test_mode_default_gateway');
 
 
 			// Show success message on completion.
-			WP_CLI::success( 'Give Test mode disabled' );
+			WP_CLI::success('Give Test mode disabled');
 		}
 	}
 }

includes/actions.php

Spacing +71 added lines, -71 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,14 +25,14 @@ 
  */
 function give_get_actions() {
 
-	$_get_action = ! empty( $_GET['give_action'] ) ? $_GET['give_action'] : null;
+	$_get_action = ! empty($_GET['give_action']) ? $_GET['give_action'] : null;
 
 	// Add backward compatibility to give-action param ( $_GET )
-	if ( empty( $_get_action ) ) {
-		$_get_action = ! empty( $_GET['give-action'] ) ? $_GET['give-action'] : null;
+	if (empty($_get_action)) {
+		$_get_action = ! empty($_GET['give-action']) ? $_GET['give-action'] : null;
 	}
 
-	if ( isset( $_get_action ) ) {
+	if (isset($_get_action)) {
 		/**
 		 * Fires in WordPress init or admin init, when give_action is present in $_GET.
 		 *
@@ -40,12 +40,12 @@ 
 		 *
 		 * @param array $_GET Array of HTTP GET variables.
 		 */
-		do_action( "give_{$_get_action}", $_GET );
+		do_action("give_{$_get_action}", $_GET);
 	}
 
 }
 
-add_action( 'init', 'give_get_actions' );
+add_action('init', 'give_get_actions');
 
 /**
  * Hooks Give actions, when present in the $_POST super global. Every give_action
@@ -58,14 +58,14 @@ 
  */
 function give_post_actions() {
 
-	$_post_action = ! empty( $_POST['give_action'] ) ? $_POST['give_action'] : null;
+	$_post_action = ! empty($_POST['give_action']) ? $_POST['give_action'] : null;
 
 	// Add backward compatibility to give-action param ( $_POST ).
-	if ( empty( $_post_action ) ) {
-		$_post_action = ! empty( $_POST['give-action'] ) ? $_POST['give-action'] : null;
+	if (empty($_post_action)) {
+		$_post_action = ! empty($_POST['give-action']) ? $_POST['give-action'] : null;
 	}
 
-	if ( isset( $_post_action ) ) {
+	if (isset($_post_action)) {
 		/**
 		 * Fires in WordPress init or admin init, when give_action is present in $_POST.
 		 *
@@ -73,12 +73,12 @@ 
 		 *
 		 * @param array $_POST Array of HTTP POST variables.
 		 */
-		do_action( "give_{$_post_action}", $_POST );
+		do_action("give_{$_post_action}", $_POST);
 	}
 
 }
 
-add_action( 'init', 'give_post_actions' );
+add_action('init', 'give_post_actions');
 
 /**
  * Connect WordPress user with Donor.
@@ -90,17 +90,17 @@ 
  *
  * @return void
  */
-function give_connect_donor_to_wpuser( $user_id, $user_data ) {
+function give_connect_donor_to_wpuser($user_id, $user_data) {
 	/* @var Give_Donor $donor */
-	$donor = new Give_Donor( $user_data['user_email'] );
+	$donor = new Give_Donor($user_data['user_email']);
 
 	// Validate donor id and check if do nor is already connect to wp user or not.
-	if ( $donor->id && ! $donor->user_id ) {
+	if ($donor->id && ! $donor->user_id) {
 
 		// Update donor user_id.
-		if ( $donor->update( array( 'user_id' => $user_id ) ) ) {
-			$donor_note = sprintf( esc_html__( 'WordPress user #%d is connected to #%d', 'give' ), $user_id, $donor->id );
-			$donor->add_note( $donor_note );
+		if ($donor->update(array('user_id' => $user_id))) {
+			$donor_note = sprintf(esc_html__('WordPress user #%d is connected to #%d', 'give'), $user_id, $donor->id);
+			$donor->add_note($donor_note);
 
 			// Update user_id meta in payments.
 			// if( ! empty( $donor->payment_ids ) && ( $donations = explode( ',', $donor->payment_ids ) ) ) {
@@ -113,7 +113,7 @@ 
 	}
 }
 
-add_action( 'give_insert_user', 'give_connect_donor_to_wpuser', 10, 2 );
+add_action('give_insert_user', 'give_connect_donor_to_wpuser', 10, 2);
 
 
 /**
@@ -127,45 +127,45 @@ 
  */
 function give_validate_license_when_site_migrated() {
 	// Store current site address if not already stored.
-	$home_url_parts              = parse_url( home_url() );
-	$home_url                    = isset( $home_url_parts['host'] ) ? $home_url_parts['host'] : false;
-	$home_url                    .= isset( $home_url_parts['path'] ) ? $home_url_parts['path'] : '';
-	$site_address_before_migrate = get_option( 'give_site_address_before_migrate' );
+	$home_url_parts              = parse_url(home_url());
+	$home_url                    = isset($home_url_parts['host']) ? $home_url_parts['host'] : false;
+	$home_url .= isset($home_url_parts['path']) ? $home_url_parts['path'] : '';
+	$site_address_before_migrate = get_option('give_site_address_before_migrate');
 
 	// Need $home_url to proceed.
-	if ( ! $home_url ) {
+	if ( ! $home_url) {
 		return;
 	}
 
 	// Save site address.
-	if ( ! $site_address_before_migrate ) {
+	if ( ! $site_address_before_migrate) {
 		// Update site address.
-		update_option( 'give_site_address_before_migrate', $home_url );
+		update_option('give_site_address_before_migrate', $home_url);
 
 		return;
 	}
 
 	// Backwards compat. for before when we were storing URL scheme.
-	if ( strpos( $site_address_before_migrate, 'http' ) ) {
-		$site_address_before_migrate = parse_url( $site_address_before_migrate );
-		$site_address_before_migrate = isset( $site_address_before_migrate['host'] ) ? $site_address_before_migrate['host'] : false;
+	if (strpos($site_address_before_migrate, 'http')) {
+		$site_address_before_migrate = parse_url($site_address_before_migrate);
+		$site_address_before_migrate = isset($site_address_before_migrate['host']) ? $site_address_before_migrate['host'] : false;
 
 		// Add path for multisite installs.
-		$site_address_before_migrate .= isset( $site_address_before_migrate['path'] ) ? $site_address_before_migrate['path'] : '';
+		$site_address_before_migrate .= isset($site_address_before_migrate['path']) ? $site_address_before_migrate['path'] : '';
 	}
 
 	// If the two URLs don't match run CRON.
-	if ( $home_url !== $site_address_before_migrate ) {
+	if ($home_url !== $site_address_before_migrate) {
 		// Immediately run cron.
-		wp_schedule_single_event( time(), 'give_validate_license_when_site_migrated' );
+		wp_schedule_single_event(time(), 'give_validate_license_when_site_migrated');
 
 		// Update site address.
-		update_option( 'give_site_address_before_migrate', $home_url );
+		update_option('give_site_address_before_migrate', $home_url);
 	}
 
 }
 
-add_action( 'admin_init', 'give_validate_license_when_site_migrated' );
+add_action('admin_init', 'give_validate_license_when_site_migrated');
 
 
 /**
@@ -175,19 +175,19 @@ 
  *
  * @param $data
  */
-function give_donor_batch_export_complete( $data ) {
+function give_donor_batch_export_complete($data) {
 	// Remove donor ids cache.
 	if (
-		isset( $data['class'] )
+		isset($data['class'])
 		&& 'Give_Batch_Donors_Export' === $data['class']
-		&& ! empty( $data['forms'] )
-		&& isset( $data['give_export_option']['query_id'] )
+		&& ! empty($data['forms'])
+		&& isset($data['give_export_option']['query_id'])
 	) {
-		Give_Cache::delete( Give_Cache::get_key( $data['give_export_option']['query_id'] ) );
+		Give_Cache::delete(Give_Cache::get_key($data['give_export_option']['query_id']));
 	}
 }
 
-add_action( 'give_file_export_complete', 'give_donor_batch_export_complete' );
+add_action('give_file_export_complete', 'give_donor_batch_export_complete');
 
 /**
  * Print css for wordpress setting pages.
@@ -198,12 +198,12 @@ 
 	/* @var WP_Screen $screen */
 	$screen = get_current_screen();
 
-	if ( ! ( $screen instanceof WP_Screen ) ) {
+	if ( ! ($screen instanceof WP_Screen)) {
 		return false;
 	}
 
-	switch ( true ) {
-		case ( 'plugins' === $screen->base || 'plugins-network' === $screen->base ):
+	switch (true) {
+		case ('plugins' === $screen->base || 'plugins-network' === $screen->base):
 			?>
 			<style>
 				tr.active.update + tr.give-addon-notice-tr td {
@@ -240,7 +240,7 @@ 
 	}
 }
 
-add_action( 'admin_head', 'give_admin_quick_css' );
+add_action('admin_head', 'give_admin_quick_css');
 
 
 /**
@@ -252,31 +252,31 @@ 
  *
  * @return void
  */
-function give_set_donation_levels_max_min_amount( $form_id ) {
+function give_set_donation_levels_max_min_amount($form_id) {
 	if (
-		( 'set' === $_POST['_give_price_option'] ) ||
-		( in_array( '_give_donation_levels', $_POST ) && count( $_POST['_give_donation_levels'] ) <= 0 ) ||
-		! ( $donation_levels_amounts = wp_list_pluck( $_POST['_give_donation_levels'], '_give_amount' ) )
+		('set' === $_POST['_give_price_option']) ||
+		(in_array('_give_donation_levels', $_POST) && count($_POST['_give_donation_levels']) <= 0) ||
+		! ($donation_levels_amounts = wp_list_pluck($_POST['_give_donation_levels'], '_give_amount'))
 	) {
 		// Delete old meta.
-		give_delete_meta( $form_id, '_give_levels_minimum_amount' );
-		give_delete_meta( $form_id, '_give_levels_maximum_amount' );
+		give_delete_meta($form_id, '_give_levels_minimum_amount');
+		give_delete_meta($form_id, '_give_levels_maximum_amount');
 
 		return;
 	}
 
 	// Sanitize donation level amounts.
-	$donation_levels_amounts = array_map( 'give_maybe_sanitize_amount', $donation_levels_amounts );
+	$donation_levels_amounts = array_map('give_maybe_sanitize_amount', $donation_levels_amounts);
 
-	$min_amount = min( $donation_levels_amounts );
-	$max_amount = max( $donation_levels_amounts );
+	$min_amount = min($donation_levels_amounts);
+	$max_amount = max($donation_levels_amounts);
 
 	// Set Minimum and Maximum amount for Multi Level Donation Forms.
-	give_update_meta( $form_id, '_give_levels_minimum_amount', $min_amount ? give_sanitize_amount_for_db( $min_amount ) : 0 );
-	give_update_meta( $form_id, '_give_levels_maximum_amount', $max_amount ? give_sanitize_amount_for_db( $max_amount ) : 0 );
+	give_update_meta($form_id, '_give_levels_minimum_amount', $min_amount ? give_sanitize_amount_for_db($min_amount) : 0);
+	give_update_meta($form_id, '_give_levels_maximum_amount', $max_amount ? give_sanitize_amount_for_db($max_amount) : 0);
 }
 
-add_action( 'give_pre_process_give_forms_meta', 'give_set_donation_levels_max_min_amount', 30 );
+add_action('give_pre_process_give_forms_meta', 'give_set_donation_levels_max_min_amount', 30);
 
 
 /**
@@ -286,23 +286,23 @@ 
  *
  * @param int $payment_id
  */
-function _give_save_donor_billing_address( $payment_id ) {
-	$donor_id  = absint( give_get_payment_donor_id( $payment_id ));
+function _give_save_donor_billing_address($payment_id) {
+	$donor_id = absint(give_get_payment_donor_id($payment_id));
 
 	// Bailout
-	if ( ! $donor_id ) {
+	if ( ! $donor_id) {
 		return;
 	}
 
 
 	/* @var Give_Donor $donor */
-	$donor = new Give_Donor( $donor_id );
+	$donor = new Give_Donor($donor_id);
 
 	// Save address.
-	$donor->add_address( 'billing[]', give_get_donation_address( $payment_id ) );
+	$donor->add_address('billing[]', give_get_donation_address($payment_id));
 }
 
-add_action( 'give_complete_donation', '_give_save_donor_billing_address', 9999 );
+add_action('give_complete_donation', '_give_save_donor_billing_address', 9999);
 
 
 /**
@@ -312,23 +312,23 @@ 
  *
  * @param array $args
  */
-function give_update_log_form_id( $args ) {
-	$new_form_id = absint( $args[0] );
-	$payment_id  = absint( $args[1] );
-	$logs        = Give()->logs->get_logs( $payment_id );
+function give_update_log_form_id($args) {
+	$new_form_id = absint($args[0]);
+	$payment_id  = absint($args[1]);
+	$logs        = Give()->logs->get_logs($payment_id);
 
 	// Bailout.
-	if ( empty( $logs ) ) {
+	if (empty($logs)) {
 		return;
 	}
 
 	/* @var object $log */
-	foreach ( $logs as $log ) {
-		Give()->logs->logmeta_db->update_meta( $log->ID, '_give_log_form_id', $new_form_id );
+	foreach ($logs as $log) {
+		Give()->logs->logmeta_db->update_meta($log->ID, '_give_log_form_id', $new_form_id);
 	}
 
 	// Delete cache.
 	Give()->logs->delete_cache();
 }
 
-add_action( 'give_update_log_form_id', 'give_update_log_form_id' );
\ No newline at end of file
+add_action('give_update_log_form_id', 'give_update_log_form_id');
\ No newline at end of file