WordImpress / Give

includes/plugin-compatibility.php

Indentation +2 added lines, -2 removed lines

@@ -39,8 +39,8 @@
 	}
 
 	if ( ( is_plugin_active( 'wordpress-seo/wp-seo.php' )
-	       || is_plugin_active( 'wordpress-seo-premium/wp-seo-premium.php' ) )
-	     && class_exists( 'WPSEO_Sitemaps_Cache' )
+		   || is_plugin_active( 'wordpress-seo-premium/wp-seo-premium.php' ) )
+		 && class_exists( 'WPSEO_Sitemaps_Cache' )
 	) {
 
 		$forms_singular_option = give_get_option( 'forms_singular' );

Spacing +16 added lines, -16 removed lines

@@ -19,10 +19,10 @@ 
  * @return void
  */
 function give_disable_mandrill_nl2br() {
-	add_filter( 'mandrill_nl2br', '__return_false' );
+	add_filter('mandrill_nl2br', '__return_false');
 }
 
-add_action( 'give_email_send_before', 'give_disable_mandrill_nl2br' );
+add_action('give_email_send_before', 'give_disable_mandrill_nl2br');
 
 
 /**
@@ -34,33 +34,33 @@ 
  */
 function give_clear_seo_sitemap_cache_on_settings_change() {
 	// Load required file if the fn 'is_plugin_active' doesn't exists.
-	if ( ! function_exists( 'is_plugin_active' ) ) {
-		require_once ABSPATH . 'wp-admin/includes/plugin.php';
+	if ( ! function_exists('is_plugin_active')) {
+		require_once ABSPATH.'wp-admin/includes/plugin.php';
 	}
 
-	if ( ( is_plugin_active( 'wordpress-seo/wp-seo.php' )
-	       || is_plugin_active( 'wordpress-seo-premium/wp-seo-premium.php' ) )
-	     && class_exists( 'WPSEO_Sitemaps_Cache' )
+	if ((is_plugin_active('wordpress-seo/wp-seo.php')
+	       || is_plugin_active('wordpress-seo-premium/wp-seo-premium.php'))
+	     && class_exists('WPSEO_Sitemaps_Cache')
 	) {
 
-		$forms_singular_option = give_get_option( 'forms_singular' );
-		$forms_archive_option  = give_get_option( 'forms_singular' );
+		$forms_singular_option = give_get_option('forms_singular');
+		$forms_archive_option  = give_get_option('forms_singular');
 
 		// If there is change detected for Single Form View and Form Archives options then proceed.
 		if (
-			( isset( $_POST['forms_singular'] ) && $_POST['forms_singular'] !== $forms_singular_option ) ||
-			( isset( $_POST['forms_archives'] ) && $_POST['forms_archives'] !== $forms_archive_option )
+			(isset($_POST['forms_singular']) && $_POST['forms_singular'] !== $forms_singular_option) ||
+			(isset($_POST['forms_archives']) && $_POST['forms_archives'] !== $forms_archive_option)
 		) {
 			// If Yoast SEO or Yoast SEO Premium plugin exists, then update seo sitemap cache.
 			$yoast_sitemaps_cache = new WPSEO_Sitemaps_Cache();
-			if ( method_exists( $yoast_sitemaps_cache, 'clear' ) ) {
+			if (method_exists($yoast_sitemaps_cache, 'clear')) {
 				WPSEO_Sitemaps_Cache::clear();
 			}
 		}
 	}
 }
 
-add_action( 'give-settings_save_display', 'give_clear_seo_sitemap_cache_on_settings_change' );
+add_action('give-settings_save_display', 'give_clear_seo_sitemap_cache_on_settings_change');
 
 /**
  * This is support for the plugin Elementor. This function
@@ -78,12 +78,12 @@ 
 	/**
 	 * Is the plugin: Elementor activated?
 	 */
-	if ( is_plugin_active( 'elementor/elementor.php' ) ) {
+	if (is_plugin_active('elementor/elementor.php')) {
 
 		/**
 		 * Check user is on the Elementor's editor page, then hide Give Shortcodes Button.
 		 */
-		if ( isset( $_GET['action'] ) && 'elementor' === give_clean( $_GET['action'] ) ) {
+		if (isset($_GET['action']) && 'elementor' === give_clean($_GET['action'])) {
 			return false;
 		}
 	}
@@ -91,4 +91,4 @@ 
 	return true;
 }
 
-add_filter( 'give_shortcode_button_condition', 'give_elementor_hide_shortcodes_button', 11 );
+add_filter('give_shortcode_button_condition', 'give_elementor_hide_shortcodes_button', 11);

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

Braces +4 added lines, -1 removed lines

@@ -95,5 +95,8 @@
 		<?php endif; ?>
 		<?php echo $form_close_tag; ?>
 	</div>
-<?php else : echo '<div class="error"><p>' . __( 'Oops, this settings page does not exist.', 'give' ) . '</p></div>'; ?>
+<?php else {
+	: echo '<div class="error"><p>' . __( 'Oops, this settings page does not exist.', 'give' ) . '</p></div>';
+}
+?>
 <?php endif; ?>
\ No newline at end of file

Spacing +22 added lines, -22 removed lines

@@ -2,12 +2,12 @@ 
 /**
  * Admin View: Settings
  */
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Bailout: Do not output anything if setting tab is not defined.
-if ( ! empty( $tabs ) && array_key_exists( give_get_current_setting_tab(), $tabs ) ) :
+if ( ! empty($tabs) && array_key_exists(give_get_current_setting_tab(), $tabs)) :
 	/**
 	 * Filter the form action.
 	 *
@@ -17,7 +17,7 @@ 
 	 *
 	 * @since 1.8
 	 */
-	$form_method = apply_filters( self::$setting_filter_prefix . '_form_method_tab_' . $current_tab, 'post' );
+	$form_method = apply_filters(self::$setting_filter_prefix.'_form_method_tab_'.$current_tab, 'post');
 
 	/**
 	 * Filter the main form tab.
@@ -29,14 +29,14 @@ 
 	 *
 	 * @since 1.8
 	 */
-	$form_open_tag  = apply_filters( self::$setting_filter_prefix . '_open_form', '<form method="' . $form_method . '" id="give-mainform" action="" enctype="multipart/form-data">' );
-	$form_close_tag = apply_filters( self::$setting_filter_prefix . '_close_form', '</form>' );
+	$form_open_tag  = apply_filters(self::$setting_filter_prefix.'_open_form', '<form method="'.$form_method.'" id="give-mainform" action="" enctype="multipart/form-data">');
+	$form_close_tag = apply_filters(self::$setting_filter_prefix.'_close_form', '</form>');
 
-	$wrapper_class = implode( ' ',
+	$wrapper_class = implode(' ',
 		array(
-			self::$setting_filter_prefix . '-setting-page',
-			self::$setting_filter_prefix . '-' . give_get_current_setting_section() . '-section',
-			self::$setting_filter_prefix . '-' . give_get_current_setting_tab() . '-tab',
+			self::$setting_filter_prefix.'-setting-page',
+			self::$setting_filter_prefix.'-'.give_get_current_setting_section().'-section',
+			self::$setting_filter_prefix.'-'.give_get_current_setting_tab().'-tab',
 		)
 	);
 	?>
@@ -46,8 +46,8 @@ 
 
 		/* @var Give_Settings_Page $current_setting_obj */
 		if (
-			! empty( $current_setting_obj ) &&
-			method_exists( $current_setting_obj, 'get_heading_html' )
+			! empty($current_setting_obj) &&
+			method_exists($current_setting_obj, 'get_heading_html')
 		) {
 			echo $current_setting_obj->get_heading_html();
 		} else {
@@ -55,7 +55,7 @@ 
 			// Backward compatibility.
 			echo sprintf(
 				'<h1 class="wp-heading-inline">%s</h1><hr class="wp-header-end">',
-				esc_html( $tabs[ $current_tab ] )
+				esc_html($tabs[$current_tab])
 			);
 		}
 
@@ -63,8 +63,8 @@ 
 		?>
 		<div class="nav-tab-wrapper give-nav-tab-wrapper">
 			<?php
-			foreach ( $tabs as $name => $label ) {
-				echo '<a href="' . admin_url( "edit.php?post_type=give_forms&page=" . self::$setting_filter_prefix . "&tab={$name}" ) . '" class="nav-tab ' . ( $current_tab === $name ? 'nav-tab-active' : 'give-mobile-hidden' ) . '">' . $label . '</a>';
+			foreach ($tabs as $name => $label) {
+				echo '<a href="'.admin_url("edit.php?post_type=give_forms&page=".self::$setting_filter_prefix."&tab={$name}").'" class="nav-tab '.($current_tab === $name ? 'nav-tab-active' : 'give-mobile-hidden').'">'.$label.'</a>';
 			}
 
 			/**
@@ -76,10 +76,10 @@ 
 			 *
 			 * @since 1.8
 			 */
-			do_action( self::$setting_filter_prefix . '_tabs' );
+			do_action(self::$setting_filter_prefix.'_tabs');
 			?>
 			<div class="give-sub-nav-tab-wrapper">
-				<a href="#" id="give-show-sub-nav" class="nav-tab give-not-tab" title="<?php _e( 'View remaining setting tabs', 'give' ); ?>"><span class="dashicons dashicons-arrow-down-alt2"></span></span>
+				<a href="#" id="give-show-sub-nav" class="nav-tab give-not-tab" title="<?php _e('View remaining setting tabs', 'give'); ?>"><span class="dashicons dashicons-arrow-down-alt2"></span></span>
 				</a>
 				<nav class="give-sub-nav-tab give-hidden"></nav>
 			</div>
@@ -95,7 +95,7 @@ 
 		 *
 		 * @since 1.8
 		 */
-		do_action( self::$setting_filter_prefix . "_sections_{$current_tab}_page" );
+		do_action(self::$setting_filter_prefix."_sections_{$current_tab}_page");
 
 		/**
 		 * Trigger Action.
@@ -106,16 +106,16 @@ 
 		 *
 		 * @since 1.8
 		 */
-		do_action( self::$setting_filter_prefix . "_settings_{$current_tab}_page" );
+		do_action(self::$setting_filter_prefix."_settings_{$current_tab}_page");
 
-		wp_nonce_field( 'give-save-settings', '_give-save-settings' );
+		wp_nonce_field('give-save-settings', '_give-save-settings');
 
-		if ( empty( $GLOBALS['give_hide_save_button'] ) ) : ?>
+		if (empty($GLOBALS['give_hide_save_button'])) : ?>
 			<div class="give-submit-wrap">
-				<input name="save" class="button-primary give-save-button" type="submit" value="<?php _e( 'Save changes', 'give' ); ?>"/>
+				<input name="save" class="button-primary give-save-button" type="submit" value="<?php _e('Save changes', 'give'); ?>"/>
 			</div>
 		<?php endif; ?>
 		<?php echo $form_close_tag; ?>
 	</div>
-<?php else : echo '<div class="error"><p>' . __( 'Oops, this settings page does not exist.', 'give' ) . '</p></div>'; ?>
+<?php else : echo '<div class="error"><p>'.__('Oops, this settings page does not exist.', 'give').'</p></div>'; ?>
 <?php endif; ?>
\ No newline at end of file

includes/admin/views/blank-slate.php

Spacing +10 added lines, -10 removed lines

@@ -17,23 +17,23 @@ 
 ?>
 
 <div class="give-blank-slate">
-	<?php if ( ! empty( $content['image_url'] ) ) : ?>
-		<img class="give-blank-slate__image" src="<?php echo esc_url( $content['image_url'] ); ?>" alt="<?php echo esc_attr( $content['image_alt'] ); ?>">
+	<?php if ( ! empty($content['image_url'])) : ?>
+		<img class="give-blank-slate__image" src="<?php echo esc_url($content['image_url']); ?>" alt="<?php echo esc_attr($content['image_alt']); ?>">
 	<?php endif; ?>
 
-	<?php if ( ! empty( $content['heading'] ) ) : ?>
-		<h2 class="give-blank-slate__heading"><?php esc_html_e( $content['heading'] ); ?></h2>
+	<?php if ( ! empty($content['heading'])) : ?>
+		<h2 class="give-blank-slate__heading"><?php esc_html_e($content['heading']); ?></h2>
 	<?php endif; ?>
 
-	<?php if ( ! empty( $content['message'] ) ) : ?>
-		<p class="give-blank-slate__message"><?php esc_html_e( $content['message'] ); ?></p>
+	<?php if ( ! empty($content['message'])) : ?>
+		<p class="give-blank-slate__message"><?php esc_html_e($content['message']); ?></p>
 	<?php endif; ?>
 
-	<?php if ( ! empty( $content['cta_text']) && ! empty( $content['cta_link'] ) ) : ?>
-		<a class="give-blank-slate__cta button button-primary" href="<?php echo esc_url( $content['cta_link'] ); ?>"><?php esc_html_e( $content['cta_text'] ); ?></a>
+	<?php if ( ! empty($content['cta_text']) && ! empty($content['cta_link'])) : ?>
+		<a class="give-blank-slate__cta button button-primary" href="<?php echo esc_url($content['cta_link']); ?>"><?php esc_html_e($content['cta_text']); ?></a>
 	<?php endif; ?>
 
-	<?php if ( ! empty( $content['help'] ) ) : ?>
+	<?php if ( ! empty($content['help'])) : ?>
 		<p class="give-blank-slate__help">
 			<?php
 			$allowed_html = array(
@@ -47,7 +47,7 @@ 
 				'code'   => array(),
 			);
 
-			echo wp_kses( $content['help'], $allowed_html );
+			echo wp_kses($content['help'], $allowed_html);
 			?>
 		</p>
 	<?php endif; ?>

includes/admin/tools/data/tools-actions.php

Spacing +13 added lines, -13 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -20,10 +20,10 @@ 
  * @since  1.5
  */
 function give_register_batch_recount_export_classes() {
-	add_action( 'give_batch_export_class_include', 'give_include_batch_export_class', 10, 1 );
+	add_action('give_batch_export_class_include', 'give_include_batch_export_class', 10, 1);
 }
 
-add_action( 'give_register_batch_exporter', 'give_register_batch_recount_export_classes', 10 );
+add_action('give_register_batch_exporter', 'give_register_batch_recount_export_classes', 10);
 
 
 /**
@@ -35,39 +35,39 @@ 
  *
  * @return void
  */
-function give_include_batch_export_class( $class ) {
-	switch ( $class ) {
+function give_include_batch_export_class($class) {
+	switch ($class) {
 
 		case 'Give_Tools_Delete_Donors':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-delete-test-donors.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-delete-test-donors.php';
 			break;
 
 		case 'Give_Tools_Import_Donors':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-delete-import-donors.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-delete-import-donors.php';
 			break;
 
 		case 'Give_Tools_Delete_Test_Transactions':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-delete-test-transactions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-delete-test-transactions.php';
 			break;
 
 		case 'Give_Tools_Recount_Donor_Stats':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-recount-donor-stats.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-recount-donor-stats.php';
 			break;
 
 		case 'Give_Tools_Reset_Stats':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-reset-stats.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-reset-stats.php';
 			break;
 
 		case 'Give_Tools_Recount_All_Stats':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-recount-all-stats.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-recount-all-stats.php';
 			break;
 
 		case 'Give_Tools_Recount_Form_Stats':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-recount-form-stats.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-recount-form-stats.php';
 			break;
 
 		case 'Give_Tools_Recount_Income':
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/class-give-tools-recount-income.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/class-give-tools-recount-income.php';
 			break;
 	}
 }

includes/admin/reports/class-give-graph.php

Spacing +35 added lines, -35 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -81,15 +81,15 @@ 
 	 * @param array $_data
 	 * @param array $options
 	 */
-	public function __construct( $_data, $options = array() ) {
+	public function __construct($_data, $options = array()) {
 
-		$this->data      = $_data;
+		$this->data = $_data;
 
 		// Generate unique ID
-		$this->id = md5( rand() );
+		$this->id = md5(rand());
 
 		// Setup default options;
-		$this->options = apply_filters( 'give_graph_args', array(
+		$this->options = apply_filters('give_graph_args', array(
 			'y_mode'          => null,
 			'x_mode'          => null,
 			'y_decimals'      => 0,
@@ -107,9 +107,9 @@ 
 			'lines'           => false,
 			'points'          => true,
 			'dataType'        => array()
-		) );
+		));
 
-		$this->options = wp_parse_args( $options, $this->options );
+		$this->options = wp_parse_args($options, $this->options);
 	}
 
 	/**
@@ -120,8 +120,8 @@ 
 	 *
 	 * @since 1.0
 	 */
-	public function set( $key, $value ) {
-		$this->options[ $key ] = $value;
+	public function set($key, $value) {
+		$this->options[$key] = $value;
 	}
 
 	/**
@@ -131,8 +131,8 @@ 
 	 *
 	 * @since 1.0
 	 */
-	public function get( $key ) {
-		return isset( $this->options[ $key ] ) ? $this->options[ $key ] : false;
+	public function get($key) {
+		return isset($this->options[$key]) ? $this->options[$key] : false;
 	}
 
 	/**
@@ -141,7 +141,7 @@ 
 	 * @since 1.0
 	 */
 	public function get_data() {
-		return apply_filters( 'give_get_graph_data', $this->data, $this );
+		return apply_filters('give_get_graph_data', $this->data, $this);
 	}
 
 	/**
@@ -151,19 +151,19 @@ 
 	 */
 	public function load_scripts() {
 		// Use minified libraries if SCRIPT_DEBUG is turned off
-		$suffix = ( defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG ) ? '' : '.min';
+		$suffix = (defined('SCRIPT_DEBUG') && SCRIPT_DEBUG) ? '' : '.min';
 
-		wp_register_script( 'jquery-flot-orderbars', GIVE_PLUGIN_URL . 'assets/js/plugins/jquery.flot.orderBars' . $suffix . '.js', array('jquery-flot'), GIVE_VERSION );
-		wp_enqueue_script( 'jquery-flot-orderbars' );
+		wp_register_script('jquery-flot-orderbars', GIVE_PLUGIN_URL.'assets/js/plugins/jquery.flot.orderBars'.$suffix.'.js', array('jquery-flot'), GIVE_VERSION);
+		wp_enqueue_script('jquery-flot-orderbars');
 
-		wp_register_script( 'jquery-flot-time', GIVE_PLUGIN_URL . 'assets/js/plugins/jquery.flot.time' . $suffix . '.js', array('jquery-flot'), GIVE_VERSION );
-		wp_enqueue_script( 'jquery-flot-time' );
+		wp_register_script('jquery-flot-time', GIVE_PLUGIN_URL.'assets/js/plugins/jquery.flot.time'.$suffix.'.js', array('jquery-flot'), GIVE_VERSION);
+		wp_enqueue_script('jquery-flot-time');
 
-		wp_register_script( 'jquery-flot-resize', GIVE_PLUGIN_URL . 'assets/js/plugins/jquery.flot.resize' . $suffix . '.js', array('jquery-flot'), GIVE_VERSION );
-		wp_enqueue_script( 'jquery-flot-resize' );
+		wp_register_script('jquery-flot-resize', GIVE_PLUGIN_URL.'assets/js/plugins/jquery.flot.resize'.$suffix.'.js', array('jquery-flot'), GIVE_VERSION);
+		wp_enqueue_script('jquery-flot-resize');
 
-		wp_register_script( 'jquery-flot', GIVE_PLUGIN_URL . 'assets/js/plugins/jquery.flot' . $suffix . '.js', false, GIVE_VERSION );
-		wp_enqueue_script( 'jquery-flot' );
+		wp_register_script('jquery-flot', GIVE_PLUGIN_URL.'assets/js/plugins/jquery.flot'.$suffix.'.js', false, GIVE_VERSION);
+		wp_enqueue_script('jquery-flot');
 
 	}
 
@@ -190,14 +190,14 @@ 
 					[
 						<?php
 							$order = 0;
-							foreach( $this->get_data() as $label => $data ) :
+							foreach ($this->get_data() as $label => $data) :
 						?>
 						{
-							label : "<?php echo esc_attr( $label ); ?>",
-							id    : "<?php echo sanitize_key( $label ); ?>",
-							dataType  : '<?php echo ( ! empty( $this->options['dataType'][$order] ) ? $this->options['dataType'][$order] : 'count' ); ?>',
+							label : "<?php echo esc_attr($label); ?>",
+							id    : "<?php echo sanitize_key($label); ?>",
+							dataType  : '<?php echo ( ! empty($this->options['dataType'][$order]) ? $this->options['dataType'][$order] : 'count'); ?>',
 							// data format is: [ point on x, value on y ]
-							data  : [<?php foreach( $data as $point ) { echo '[' . implode( ',', $point ) . '],'; } ?>],
+							data  : [<?php foreach ($data as $point) { echo '['.implode(',', $point).'],'; } ?>],
 							points: {
 								show: <?php echo $this->options['points'] ? 'true' : 'false'; ?>,
 							},
@@ -212,7 +212,7 @@ 
 								fill     : true,
 								fillColor: {colors: [{opacity: 0.4}, {opacity: 0.1}]}
 							},
-							<?php if( $this->options[ 'multiple_y_axes' ] ) : ?>
+							<?php if ($this->options['multiple_y_axes']) : ?>
 							yaxis : <?php echo $yaxis_count; ?>
 							<?php endif; ?>
 
@@ -226,10 +226,10 @@ 
 						grid: {
 							show           : true,
 							aboveData      : false,
-							color          : "<?php echo $this->options[ 'color' ]; ?>",
-							backgroundColor: "<?php echo $this->options[ 'bgcolor' ]; ?>",
-							borderColor    : "<?php echo $this->options[ 'bordercolor' ]; ?>",
-							borderWidth    : <?php echo absint( $this->options[ 'borderwidth' ] ); ?>,
+							color          : "<?php echo $this->options['color']; ?>",
+							backgroundColor: "<?php echo $this->options['bgcolor']; ?>",
+							borderColor    : "<?php echo $this->options['bordercolor']; ?>",
+							borderWidth    : <?php echo absint($this->options['borderwidth']); ?>,
 							clickable      : false,
 							hoverable      : true
 						},
@@ -240,7 +240,7 @@ 
 							mode        : "<?php echo $this->options['x_mode']; ?>",
 							timeFormat  : "<?php echo $this->options['x_mode'] == 'time' ? $this->options['time_format'] : ''; ?>",
 							tickSize    : "<?php echo $this->options['x_mode'] == 'time' ? '' : 1; ?>",
-							<?php if( $this->options['x_mode'] != 'time' ) : ?>
+							<?php if ($this->options['x_mode'] != 'time') : ?>
 							tickDecimals: <?php echo $this->options['x_decimals']; ?>
 							<?php endif; ?>
 						},
@@ -249,7 +249,7 @@ 
 							min         : 0,
 							mode        : "<?php echo $this->options['y_mode']; ?>",
 							timeFormat  : "<?php echo $this->options['y_mode'] == 'time' ? $this->options['time_format'] : ''; ?>",
-							<?php if( $this->options['y_mode'] != 'time' ) : ?>
+							<?php if ($this->options['y_mode'] != 'time') : ?>
 							tickDecimals: <?php echo $this->options['y_decimals']; ?>,
 							<?php endif; ?>
 							tickFormatter: function(val) {
@@ -323,7 +323,7 @@ 
 		 *
 		 * @param Give_Graph $this Graph object.
 		 */
-		do_action( 'give_before_graph', $this );
+		do_action('give_before_graph', $this);
 
 		// Build the graph.
 		echo $this->build_graph();
@@ -335,7 +335,7 @@ 
 		 *
 		 * @param Give_Graph $this Graph object.
 		 */
-		do_action( 'give_after_graph', $this );
+		do_action('give_after_graph', $this);
 	}
 
 }

includes/admin/tools/import/class-give-import-donations.php

Doc Comments +2 added lines, -2 removed lines

@@ -454,7 +454,7 @@ 
 
 		/**
 		 * @param $option_value
-		 * @param $value
+		 * @param boolean $value
 		 *
 		 * @return string
 		 */
@@ -544,7 +544,7 @@ 
 		 *
 		 * @since 1.8.14
 		 *
-		 * @param $file_id
+		 * @param integer $file_id
 		 *
 		 * @return bool|int
 		 */

Indentation +9 added lines, -9 removed lines

@@ -201,13 +201,13 @@ 
 									   id="recount-stats-submit"
 									   value="
 									       <?php
-								       /**
-								        * Filter to modify donation importer submit button text.
-								        *
-								        * @since 2.1
-								        */
-								       echo apply_filters( 'give_import_donation_submit_button_text', __( 'Submit', 'give' ) );
-								       ?>
+									   /**
+									    * Filter to modify donation importer submit button text.
+									    *
+									    * @since 2.1
+									    */
+									   echo apply_filters( 'give_import_donation_submit_button_text', __( 'Submit', 'give' ) );
+									   ?>
 											"/>
 							</th>
 							<th>
@@ -1079,8 +1079,8 @@ 
 		 */
 		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'] );
+				   isset( $_GET['importer-type'] ) &&
+				   $this->importer_type === give_clean( $_GET['importer-type'] );
 		}
 	}
 

Spacing +231 added lines, -237 removed lines

@@ -11,11 +11,11 @@ 
  * @since       1.8.14
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Import_Donations' ) ) {
+if ( ! class_exists('Give_Import_Donations')) {
 
 	/**
 	 * Give_Import_Donations.
@@ -66,7 +66,7 @@ 
 		 * @access private
 		 */
 		private function __construct() {
-			self::$per_page = ! empty( $_GET['per_page'] ) ? absint( $_GET['per_page'] ) : self::$per_page;
+			self::$per_page = ! empty($_GET['per_page']) ? absint($_GET['per_page']) : self::$per_page;
 		}
 
 		/**
@@ -78,7 +78,7 @@ 
 		 * @return static
 		 */
 		public static function get_instance() {
-			if ( null === static::$instance ) {
+			if (null === static::$instance) {
 				self::$instance = new static();
 			}
 
@@ -105,27 +105,27 @@ 
 		 * @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_donations_form_start', array( $this, 'html' ), 10 );
+			add_action('give_tools_import_donations_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_donations_form_end', array( $this, 'submit' ), 10 );
+			add_action('give_tools_import_donations_form_end', array($this, 'submit'), 10);
 		}
 
 		/**
@@ -137,9 +137,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;
@@ -151,7 +151,7 @@ 
 		 * @since 1.8.14
 		 */
 		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(); ?>"/>
@@ -172,11 +172,11 @@ 
 			?>
 			<section>
 				<table
-						class="widefat export-options-table give-table <?php echo "step-{$step}"; ?> <?php echo( 1 === $step && ! empty( $this->is_csv_valid ) ? 'give-hidden' : '' ); ?>  "
+						class="widefat export-options-table give-table <?php echo "step-{$step}"; ?> <?php echo(1 === $step && ! empty($this->is_csv_valid) ? 'give-hidden' : ''); ?>  "
 						id="<?php echo "step-{$step}"; ?>">
 					<tbody>
 					<?php
-					switch ( $step ) {
+					switch ($step) {
 						case 1:
 							$this->render_media_csv();
 							break;
@@ -192,7 +192,7 @@ 
 						case 4:
 							$this->import_success();
 					}
-					if ( false === $this->check_for_dropdown_or_import() ) {
+					if (false === $this->check_for_dropdown_or_import()) {
 						?>
 						<tr valign="top">
 							<th>
@@ -206,7 +206,7 @@ 
 								        *
 								        * @since 2.1
 								        */
-								       echo apply_filters( 'give_import_donation_submit_button_text', __( 'Submit', 'give' ) );
+								       echo apply_filters('give_import_donation_submit_button_text', __('Submit', 'give'));
 								       ?>
 											"/>
 							</th>
@@ -217,7 +217,7 @@ 
 								 *
 								 * @since 2.1
 								 */
-								do_action( 'give_import_donation_submit_button' );
+								do_action('give_import_donation_submit_button');
 								?>
 							</th>
 						</tr>
@@ -237,70 +237,70 @@ 
 		 */
 		public function import_success() {
 
-			$delete_csv = ( ! empty( $_GET['delete_csv'] ) ? absint( $_GET['delete_csv'] ) : false );
-			$csv        = ( ! empty( $_GET['csv'] ) ? absint( $_GET['csv'] ) : false );
-			if ( ! empty( $delete_csv ) && ! empty( $csv ) ) {
-				wp_delete_attachment( $csv, true );
+			$delete_csv = ( ! empty($_GET['delete_csv']) ? absint($_GET['delete_csv']) : false);
+			$csv        = ( ! empty($_GET['csv']) ? absint($_GET['csv']) : false);
+			if ( ! empty($delete_csv) && ! empty($csv)) {
+				wp_delete_attachment($csv, true);
 			}
 
 			$report = give_import_donation_report();
 
 			$report_html = array(
 				'duplicate_donor'    => array(
-					__( '%s duplicate %s detected', 'give' ),
-					__( '%s duplicate %s will be detected', 'give' ),
-					__( 'donor', 'give' ),
-					__( 'donors', 'give' ),
+					__('%s duplicate %s detected', 'give'),
+					__('%s duplicate %s will be detected', 'give'),
+					__('donor', 'give'),
+					__('donors', 'give'),
 				),
 				'create_donor'       => array(
-					__( '%s %s created', 'give' ),
-					__( '%s %s will be going to get created', 'give' ),
-					__( 'donor', 'give' ),
-					__( 'donors', 'give' ),
+					__('%s %s created', 'give'),
+					__('%s %s will be going to get created', 'give'),
+					__('donor', 'give'),
+					__('donors', 'give'),
 				),
 				'create_form'        => array(
-					__( '%s donation %s created', 'give' ),
-					__( '%s donation %s will be going to get created', 'give' ),
-					__( 'form', 'give' ),
-					__( 'forms', 'give' ),
+					__('%s donation %s created', 'give'),
+					__('%s donation %s will be going to get created', 'give'),
+					__('form', 'give'),
+					__('forms', 'give'),
 				),
 				'duplicate_donation' => array(
-					__( '%s duplicate %s detected', 'give' ),
-					__( '%s duplicate %s will be detected', 'give' ),
-					__( 'donation', 'give' ),
-					__( 'donations', 'give' ),
+					__('%s duplicate %s detected', 'give'),
+					__('%s duplicate %s will be detected', 'give'),
+					__('donation', 'give'),
+					__('donations', 'give'),
 				),
 				'create_donation'    => array(
-					__( '%s %s imported', 'give' ),
-					__( '%s %s will going to get imported', 'give' ),
-					__( 'donation', 'give' ),
-					__( 'donations', 'give' ),
+					__('%s %s imported', 'give'),
+					__('%s %s will going to get imported', 'give'),
+					__('donation', 'give'),
+					__('donations', 'give'),
 				),
 			);
-			$total       = (int) $_GET['total'];
-			-- $total;
+			$total = (int) $_GET['total'];
+			--$total;
 			$success = (bool) $_GET['success'];
-			$dry_run = empty( $_GET['dry_run'] ) ? 0 : absint( $_GET['dry_run'] );
+			$dry_run = empty($_GET['dry_run']) ? 0 : absint($_GET['dry_run']);
 			?>
 			<tr valign="top" class="give-import-dropdown">
 				<th colspan="2">
 					<h2>
 						<?php
-						if ( $success ) {
-							if ( $dry_run ) {
+						if ($success) {
+							if ($dry_run) {
 								printf(
-									_n( 'Dry run import complete! %s donation processed', 'Dry run import complete! %s donations processed', $total, 'give' ),
+									_n('Dry run import complete! %s donation processed', 'Dry run import complete! %s donations processed', $total, 'give'),
 									"<strong>{$total}</strong>"
 								);
 							} else {
 								printf(
-									_n( 'Import complete! %s donation processed', 'Import complete! %s donations processed', $total, 'give' ),
+									_n('Import complete! %s donation processed', 'Import complete! %s donations processed', $total, 'give'),
 									"<strong>{$total}</strong>"
 								);
 							}
 						} else {
 							printf(
-								_n( 'Failed to import %s donation', 'Failed to import %s donations', $total, 'give' ),
+								_n('Failed to import %s donation', 'Failed to import %s donations', $total, 'give'),
 								"<strong>{$total}</strong>"
 							);
 						}
@@ -308,16 +308,16 @@ 
 					</h2>
 
 					<?php
-					$text      = __( 'Import Donation', 'give' );
+					$text      = __('Import Donation', 'give');
 					$query_arg = array(
 						'post_type' => 'give_forms',
 						'page'      => 'give-tools',
 						'tab'       => 'import',
 					);
-					if ( $success ) {
+					if ($success) {
 
 
-						if ( $dry_run ) {
+						if ($dry_run) {
 							$query_arg = array(
 								'post_type'     => 'give_forms',
 								'page'          => 'give-tools',
@@ -325,25 +325,25 @@ 
 								'importer-type' => 'import_donations',
 							);
 
-							$text = __( 'Start Import', 'give' );
+							$text = __('Start Import', 'give');
 						} else {
 							$query_arg = array(
 								'post_type' => 'give_forms',
 								'page'      => 'give-payment-history',
 							);
-							$text      = __( 'View Donations', 'give' );
+							$text = __('View Donations', 'give');
 						}
 					}
 
-					foreach ( $report as $key => $value ) {
-						if ( array_key_exists( $key, $report_html ) && ! empty( $value ) ) {
-							$key_name = $report_html[ $key ][2];
-							if ( $value > 1 ) {
-								$key_name = $report_html[ $key ][3];
+					foreach ($report as $key => $value) {
+						if (array_key_exists($key, $report_html) && ! empty($value)) {
+							$key_name = $report_html[$key][2];
+							if ($value > 1) {
+								$key_name = $report_html[$key][3];
 							}
 							?>
 							<p>
-								<?php printf( $report_html[ $key ][ $dry_run ], $value, $key_name ); ?>
+								<?php printf($report_html[$key][$dry_run], $value, $key_name); ?>
 							</p>
 							<?php
 						}
@@ -352,7 +352,7 @@ 
 
 					<p>
 						<a class="button button-large button-secondary"
-						   href="<?php echo add_query_arg( $query_arg, admin_url( 'edit.php' ) ); ?>"><?php echo $text; ?></a>
+						   href="<?php echo add_query_arg($query_arg, admin_url('edit.php')); ?>"><?php echo $text; ?></a>
 					</p>
 				</th>
 			</tr>
@@ -369,26 +369,26 @@ 
 			give_import_donation_report_reset();
 
 			$csv         = (int) $_REQUEST['csv'];
-			$delimiter   = ( ! empty( $_REQUEST['delimiter'] ) ? give_clean( $_REQUEST['delimiter'] ) : 'csv' );
+			$delimiter   = ( ! empty($_REQUEST['delimiter']) ? give_clean($_REQUEST['delimiter']) : 'csv');
 			$index_start = 1;
 			$index_end   = 1;
 			$next        = true;
-			$total       = self::get_csv_total( $csv );
-			if ( self::$per_page < $total ) {
-				$total_ajax = ceil( $total / self::$per_page );
+			$total       = self::get_csv_total($csv);
+			if (self::$per_page < $total) {
+				$total_ajax = ceil($total / self::$per_page);
 				$index_end  = self::$per_page;
 			} else {
 				$total_ajax = 1;
 				$index_end  = $total;
 				$next       = false;
 			}
-			$current_percentage = 100 / ( $total_ajax + 1 );
+			$current_percentage = 100 / ($total_ajax + 1);
 
 			?>
 			<tr valign="top" class="give-import-dropdown">
 				<th colspan="2">
-					<h2 id="give-import-title"><?php _e( 'Importing', 'give' ) ?></h2>
-					<p class="give-field-description"><?php _e( 'Your donations are now being imported...', 'give' ) ?></p>
+					<h2 id="give-import-title"><?php _e('Importing', 'give') ?></h2>
+					<p class="give-field-description"><?php _e('Your donations are now being imported...', 'give') ?></p>
 				</th>
 			</tr>
 
@@ -407,7 +407,7 @@ 
 						<div style="width: <?php echo $current_percentage; ?>%"></div>
 					</div>
 					<input type="hidden" value="3" name="step">
-					<input type="hidden" value='<?php echo maybe_serialize( $_REQUEST['mapto'] ); ?>' name="mapto"
+					<input type="hidden" value='<?php echo maybe_serialize($_REQUEST['mapto']); ?>' name="mapto"
 						   class="mapto">
 					<input type="hidden" value="<?php echo $_REQUEST['csv']; ?>" name="csv" class="csv">
 					<input type="hidden" value="<?php echo $_REQUEST['mode']; ?>" name="mode" class="mode">
@@ -416,9 +416,9 @@ 
 					<input type="hidden" value="<?php echo $_REQUEST['delete_csv']; ?>" name="delete_csv"
 						   class="delete_csv">
 					<input type="hidden" value="<?php echo $delimiter; ?>" name="delimiter">
-					<input type="hidden" value="<?php echo absint( $_REQUEST['dry_run'] ); ?>" name="dry_run">
+					<input type="hidden" value="<?php echo absint($_REQUEST['dry_run']); ?>" name="dry_run">
 					<input type="hidden"
-						   value='<?php echo maybe_serialize( self::get_importer( $csv, 0, $delimiter ) ); ?>'
+						   value='<?php echo maybe_serialize(self::get_importer($csv, 0, $delimiter)); ?>'
 						   name="main_key"
 						   class="main_key">
 				</th>
@@ -433,20 +433,20 @@ 
 		 */
 		public function check_for_dropdown_or_import() {
 			$return = true;
-			if ( isset( $_REQUEST['mapto'] ) ) {
+			if (isset($_REQUEST['mapto'])) {
 				$mapto = (array) $_REQUEST['mapto'];
-				if ( false === in_array( 'form_title', $mapto ) && false === in_array( 'form_id', $mapto ) ) {
-					Give_Admin_Settings::add_error( 'give-import-csv-form', __( 'In order to import donations, a column must be mapped to either the "Donation Form Title" or "Donation Form ID" field. Please map a column to one of those fields.', 'give' ) );
+				if (false === in_array('form_title', $mapto) && false === in_array('form_id', $mapto)) {
+					Give_Admin_Settings::add_error('give-import-csv-form', __('In order to import donations, a column must be mapped to either the "Donation Form Title" or "Donation Form ID" field. Please map a column to one of those fields.', 'give'));
 					$return = false;
 				}
 
-				if ( false === in_array( 'amount', $mapto ) ) {
-					Give_Admin_Settings::add_error( 'give-import-csv-amount', __( 'In order to import donations, a column must be mapped to the "Amount" field. Please map a column to that field.', 'give' ) );
+				if (false === in_array('amount', $mapto)) {
+					Give_Admin_Settings::add_error('give-import-csv-amount', __('In order to import donations, a column must be mapped to the "Amount" field. Please map a column to that field.', 'give'));
 					$return = false;
 				}
 
-				if ( false === in_array( 'email', $mapto ) && false === in_array( 'donor_id', $mapto ) ) {
-					Give_Admin_Settings::add_error( 'give-import-csv-donor', __( 'In order to import donations, a column must be mapped to either the "Donor Email" or "Donor ID" field. Please map a column to that field.', 'give' ) );
+				if (false === in_array('email', $mapto) && false === in_array('donor_id', $mapto)) {
+					Give_Admin_Settings::add_error('give-import-csv-donor', __('In order to import donations, a column must be mapped to either the "Donor Email" or "Donor ID" field. Please map a column to that field.', 'give'));
 					$return = false;
 				}
 			} else {
@@ -463,10 +463,10 @@ 
 		 */
 		public function render_dropdown() {
 			$csv       = (int) $_GET['csv'];
-			$delimiter = ( ! empty( $_GET['delimiter'] ) ? give_clean( $_GET['delimiter'] ) : 'csv' );
+			$delimiter = ( ! empty($_GET['delimiter']) ? give_clean($_GET['delimiter']) : 'csv');
 
 			// TO check if the CSV files that is being add is valid or not if not then redirect to first step again
-			if ( ! $this->is_valid_csv( $csv ) ) {
+			if ( ! $this->is_valid_csv($csv)) {
 				$url = give_import_page_url();
 				?>
 				<input type="hidden" name="csv_not_valid" class="csv_not_valid" value="<?php echo $url; ?>"/>
@@ -475,11 +475,11 @@ 
 				?>
 				<tr valign="top" class="give-import-dropdown">
 					<th colspan="2">
-						<h2 id="give-import-title"><?php _e( 'Map CSV fields to donations', 'give' ) ?></h2>
+						<h2 id="give-import-title"><?php _e('Map CSV fields to donations', 'give') ?></h2>
 
-						<p class="give-import-donation-required-fields-title"><?php _e( 'Required Fields' ); ?></p>
+						<p class="give-import-donation-required-fields-title"><?php _e('Required Fields'); ?></p>
 
-						<p class="give-field-description"><?php _e( 'These fields are required for the import to submitted' ); ?></p>
+						<p class="give-field-description"><?php _e('These fields are required for the import to submitted'); ?></p>
 
 						<ul class="give-import-donation-required-fields">
 							<li class="give-import-donation-required-email"
@@ -487,7 +487,7 @@ 
 								<span class="give-import-donation-required-symbol dashicons dashicons-no-alt"></span>
 								<span class="give-import-donation-required-text">
 									<?php
-									_e( 'Email Address', 'give' );
+									_e('Email Address', 'give');
 									?>
 								</span>
 							</li>
@@ -497,7 +497,7 @@ 
 								<span class="give-import-donation-required-symbol dashicons dashicons-no-alt"></span>
 								<span class="give-import-donation-required-text">
 									<?php
-									_e( 'First Name', 'give' );
+									_e('First Name', 'give');
 									?>
 								</span>
 							</li>
@@ -507,7 +507,7 @@ 
 								<span class="give-import-donation-required-symbol dashicons dashicons-no-alt"></span>
 								<span class="give-import-donation-required-text">
 									<?php
-									_e( 'Donation Amount', 'give' );
+									_e('Donation Amount', 'give');
 									?>
 								</span>
 							</li>
@@ -517,32 +517,32 @@ 
 								<span class="give-import-donation-required-symbol dashicons dashicons-no-alt"></span>
 								<span class="give-import-donation-required-text">
 									<?php
-									_e( 'Form Title or ID', 'give' );
+									_e('Form Title or ID', 'give');
 									?>
 								</span>
 							</li>
 						</ul>
 
-						<p class="give-field-description"><?php _e( 'Select fields from your CSV file to map against donations fields or to ignore during import.', 'give' ) ?></p>
+						<p class="give-field-description"><?php _e('Select fields from your CSV file to map against donations fields or to ignore during import.', 'give') ?></p>
 					</th>
 				</tr>
 
 				<tr valign="top" class="give-import-dropdown">
-					<th><b><?php _e( 'Column name', 'give' ); ?></b></th>
-					<th><b><?php _e( 'Map to field', 'give' ); ?></b></th>
+					<th><b><?php _e('Column name', 'give'); ?></b></th>
+					<th><b><?php _e('Map to field', 'give'); ?></b></th>
 				</tr>
 
 				<?php
-				$raw_key = $this->get_importer( $csv, 0, $delimiter );
-				$mapto   = (array) ( isset( $_REQUEST['mapto'] ) ? $_REQUEST['mapto'] : array() );
+				$raw_key = $this->get_importer($csv, 0, $delimiter);
+				$mapto   = (array) (isset($_REQUEST['mapto']) ? $_REQUEST['mapto'] : array());
 
-				foreach ( $raw_key as $index => $value ) {
+				foreach ($raw_key as $index => $value) {
 					?>
 					<tr valign="top" class="give-import-option">
 						<th><?php echo $value; ?></th>
 						<th>
 							<?php
-							$this->get_columns( $index, $value, $mapto );
+							$this->get_columns($index, $value, $mapto);
 							?>
 						</th>
 					</tr>
@@ -557,14 +557,14 @@ 
 		 *
 		 * @return string
 		 */
-		public function selected( $option_value, $value ) {
-			$option_value = strtolower( $option_value );
-			$value        = strtolower( $value );
+		public function selected($option_value, $value) {
+			$option_value = strtolower($option_value);
+			$value        = strtolower($value);
 
 			$selected = '';
-			if ( stristr( $value, $option_value ) ) {
+			if (stristr($value, $option_value)) {
 				$selected = 'selected';
-			} elseif ( strrpos( $value, 'give_' ) && stristr( $option_value, __( 'Import as Meta', 'give' ) ) ) {
+			} elseif (strrpos($value, 'give_') && stristr($option_value, __('Import as Meta', 'give'))) {
 				$selected = 'selected';
 			}
 
@@ -583,28 +583,28 @@ 
 		 *
 		 * @return void
 		 */
-		private function get_columns( $index, $value = false, $mapto = array() ) {
+		private function get_columns($index, $value = false, $mapto = array()) {
 			$default       = give_import_default_options();
-			$current_mapto = (string) ( ! empty( $mapto[ $index ] ) ? $mapto[ $index ] : '' );
+			$current_mapto = (string) ( ! empty($mapto[$index]) ? $mapto[$index] : '');
 			?>
 			<select name="mapto[<?php echo $index; ?>]">
-				<?php $this->get_dropdown_option_html( $default, $current_mapto, $value ); ?>
+				<?php $this->get_dropdown_option_html($default, $current_mapto, $value); ?>
 
-				<optgroup label="<?php _e( 'Donations', 'give' ); ?>">
+				<optgroup label="<?php _e('Donations', 'give'); ?>">
 					<?php
-					$this->get_dropdown_option_html( give_import_donations_options(), $current_mapto, $value );
+					$this->get_dropdown_option_html(give_import_donations_options(), $current_mapto, $value);
 					?>
 				</optgroup>
 
-				<optgroup label="<?php _e( 'Donors', 'give' ); ?>">
+				<optgroup label="<?php _e('Donors', 'give'); ?>">
 					<?php
-					$this->get_dropdown_option_html( give_import_donor_options(), $current_mapto, $value );
+					$this->get_dropdown_option_html(give_import_donor_options(), $current_mapto, $value);
 					?>
 				</optgroup>
 
-				<optgroup label="<?php _e( 'Forms', 'give' ); ?>">
+				<optgroup label="<?php _e('Forms', 'give'); ?>">
 					<?php
-					$this->get_dropdown_option_html( give_import_donation_form_options(), $current_mapto, $value );
+					$this->get_dropdown_option_html(give_import_donation_form_options(), $current_mapto, $value);
 					?>
 				</optgroup>
 
@@ -615,7 +615,7 @@ 
 				 *
 				 * @since 1.8.15
 				 */
-				do_action( 'give_import_dropdown_option', $index, $value, $mapto, $current_mapto );
+				do_action('give_import_dropdown_option', $index, $value, $mapto, $current_mapto);
 				?>
 			</select>
 			<?php
@@ -633,23 +633,23 @@ 
 		 *
 		 * @return void
 		 */
-		public function get_dropdown_option_html( $options, $current_mapto, $value = false ) {
+		public function get_dropdown_option_html($options, $current_mapto, $value = false) {
 
-			foreach ( $options as $option => $option_value ) {
+			foreach ($options as $option => $option_value) {
 				$ignore = array();
-				if ( isset( $option_value['ignore'] ) ) {
+				if (isset($option_value['ignore'])) {
 					$ignore = $option_value['ignore'];
-					unset( $option_value['ignore'] );
+					unset($option_value['ignore']);
 				}
 
 				$option_value_texts = (array) $option_value;
 				$option_text        = $option_value_texts[0];
 
-				$checked = ( ( $current_mapto === $option ) ? 'selected' : false );
-				if ( empty( $checked ) && ! in_array( $value, $ignore ) ) {
-					foreach ( $option_value_texts as $option_value_text ) {
-						$checked = $this->selected( $option_value_text, $value );
-						if ( $checked ) {
+				$checked = (($current_mapto === $option) ? 'selected' : false);
+				if (empty($checked) && ! in_array($value, $ignore)) {
+					foreach ($option_value_texts as $option_value_text) {
+						$checked = $this->selected($option_value_text, $value);
+						if ($checked) {
 							break;
 						}
 					}
@@ -673,12 +673,12 @@ 
 		 *
 		 * @return bool|int
 		 */
-		public function get_csv_total( $file_id ) {
+		public function get_csv_total($file_id) {
 			$total = false;
-			if ( $file_id ) {
-				$file_dir = get_attached_file( $file_id );
-				if ( $file_dir ) {
-					$total = $this->get_csv_data_from_file_dir( $file_dir );
+			if ($file_id) {
+				$file_dir = get_attached_file($file_id);
+				if ($file_dir) {
+					$total = $this->get_csv_data_from_file_dir($file_dir);
 				}
 			}
 
@@ -694,11 +694,11 @@ 
 		 *
 		 * @return bool|int
 		 */
-		public function get_csv_data_from_file_dir( $file_dir ) {
+		public function get_csv_data_from_file_dir($file_dir) {
 			$total = false;
-			if ( $file_dir ) {
-				$file = new SplFileObject( $file_dir, 'r' );
-				$file->seek( PHP_INT_MAX );
+			if ($file_dir) {
+				$file = new SplFileObject($file_dir, 'r');
+				$file->seek(PHP_INT_MAX);
 				$total = $file->key() + 1;
 			}
 
@@ -716,7 +716,7 @@ 
 		 *
 		 * @return array|bool $raw_data title of the CSV file fields
 		 */
-		public function get_importer( $file_id, $index = 0, $delimiter = 'csv' ) {
+		public function get_importer($file_id, $index = 0, $delimiter = 'csv') {
 			/**
 			 * Filter to modify delimiter of Import.
 			 *
@@ -724,16 +724,16 @@ 
 			 *
 			 * Return string $delimiter.
 			 */
-			$delimiter = (string) apply_filters( 'give_import_delimiter_set', $delimiter );
+			$delimiter = (string) apply_filters('give_import_delimiter_set', $delimiter);
 
 			$raw_data = false;
-			$file_dir = get_attached_file( $file_id );
-			if ( $file_dir ) {
-				if ( false !== ( $handle = fopen( $file_dir, 'r' ) ) ) {
-					$raw_data = fgetcsv( $handle, $index, $delimiter );
+			$file_dir = get_attached_file($file_id);
+			if ($file_dir) {
+				if (false !== ($handle = fopen($file_dir, 'r'))) {
+					$raw_data = fgetcsv($handle, $index, $delimiter);
 					// Remove BOM signature from the first item.
-					if ( isset( $raw_data[0] ) ) {
-						$raw_data[0] = $this->remove_utf8_bom( $raw_data[0] );
+					if (isset($raw_data[0])) {
+						$raw_data[0] = $this->remove_utf8_bom($raw_data[0]);
 					}
 				}
 			}
@@ -750,9 +750,9 @@ 
 		 *
 		 * @return string
 		 */
-		public function remove_utf8_bom( $string ) {
-			if ( 'efbbbf' === substr( bin2hex( $string ), 0, 6 ) ) {
-				$string = substr( $string, 3 );
+		public function remove_utf8_bom($string) {
+			if ('efbbbf' === substr(bin2hex($string), 0, 6)) {
+				$string = substr($string, 3);
 			}
 
 			return $string;
@@ -767,17 +767,17 @@ 
 			$step = $this->get_step();
 			?>
 			<ol class="give-progress-steps">
-				<li class="<?php echo( 1 === $step ? 'active' : '' ); ?>">
-					<?php _e( 'Upload CSV file', 'give' ); ?>
+				<li class="<?php echo(1 === $step ? 'active' : ''); ?>">
+					<?php _e('Upload CSV file', 'give'); ?>
 				</li>
-				<li class="<?php echo( 2 === $step ? 'active' : '' ); ?>">
-					<?php _e( 'Column mapping', 'give' ); ?>
+				<li class="<?php echo(2 === $step ? 'active' : ''); ?>">
+					<?php _e('Column mapping', 'give'); ?>
 				</li>
-				<li class="<?php echo( 3 === $step ? 'active' : '' ); ?>">
-					<?php _e( 'Import', 'give' ); ?>
+				<li class="<?php echo(3 === $step ? 'active' : ''); ?>">
+					<?php _e('Import', 'give'); ?>
 				</li>
-				<li class="<?php echo( 4 === $step ? 'active' : '' ); ?>">
-					<?php _e( 'Done!', 'give' ); ?>
+				<li class="<?php echo(4 === $step ? 'active' : ''); ?>">
+					<?php _e('Done!', 'give'); ?>
 				</li>
 			</ol>
 			<?php
@@ -791,16 +791,16 @@ 
 		 * @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 ( $this->check_for_dropdown_or_import() ) {
+			} elseif ($this->check_for_dropdown_or_import()) {
 				$on_step = 3;
-			} elseif ( 2 === $step ) {
+			} elseif (2 === $step) {
 				$on_step = 2;
-			} elseif ( 4 === $step ) {
+			} elseif (4 === $step) {
 				$on_step = 4;
 			}
 
@@ -813,7 +813,7 @@ 
 		 * @since 1.8.14
 		 */
 		public function render_page() {
-			include_once GIVE_PLUGIN_DIR . 'includes/admin/tools/views/html-admin-page-import-donations.php';
+			include_once GIVE_PLUGIN_DIR.'includes/admin/tools/views/html-admin-page-import-donations.php';
 		}
 
 		/**
@@ -822,18 +822,18 @@ 
 		 * @since 2.1
 		 */
 		public function give_import_donation_submit_button_render_media_csv() {
-			$dry_run = isset( $_POST['dry_run'] ) ? absint( $_POST['dry_run'] ) : 1;
+			$dry_run = isset($_POST['dry_run']) ? absint($_POST['dry_run']) : 1;
 			?>
 			<div>
 				<label for="dry_run">
 					<input type="hidden" name="dry_run" value="0"/>
 					<input type="checkbox" name="dry_run" id="dry_run" class="dry_run"
-						   value="1" <?php checked( 1, $dry_run ); ?> >
-					<strong><?php _e( 'Dry Run', 'give' ); ?></strong>
+						   value="1" <?php checked(1, $dry_run); ?> >
+					<strong><?php _e('Dry Run', 'give'); ?></strong>
 				</label>
 				<p class="give-field-description">
 					<?php
-					_e( 'Preview what the import would look like without making any default changes to your site or your database.', 'give' );
+					_e('Preview what the import would look like without making any default changes to your site or your database.', 'give');
 					?>
 				</p>
 			</div>
@@ -849,8 +849,8 @@ 
 		 *
 		 * @return string
 		 */
-		function give_import_donation_submit_text_render_media_csv( $text ) {
-			return __( 'Begin Import', 'give' );
+		function give_import_donation_submit_text_render_media_csv($text) {
+			return __('Begin Import', 'give');
 		}
 
 		/**
@@ -862,60 +862,57 @@ 
 		 * @return void
 		 */
 		public function render_media_csv() {
-			add_filter( 'give_import_donation_submit_button_text', array(
+			add_filter('give_import_donation_submit_button_text', array(
 				$this,
 				'give_import_donation_submit_text_render_media_csv'
-			) );
-			add_action( 'give_import_donation_submit_button', array(
+			));
+			add_action('give_import_donation_submit_button', array(
 				$this,
 				'give_import_donation_submit_button_render_media_csv'
-			) );
+			));
 			?>
 			<tr valign="top">
 				<th colspan="2">
-					<h2 id="give-import-title"><?php _e( 'Import donations from a CSV file', 'give' ) ?></h2>
-					<p class="give-field-description"><?php _e( 'This tool allows you to import or add donation data to your give form(s) via a CSV file.', 'give' ) ?></p>
+					<h2 id="give-import-title"><?php _e('Import donations from a CSV file', 'give') ?></h2>
+					<p class="give-field-description"><?php _e('This tool allows you to import or add donation data to your give form(s) via a CSV file.', 'give') ?></p>
 				</th>
 			</tr>
 			<?php
-			$csv         = ( isset( $_POST['csv'] ) ? give_clean( $_POST['csv'] ) : '' );
-			$csv_id      = ( isset( $_POST['csv_id'] ) ? give_clean( $_POST['csv_id'] ) : '' );
-			$delimiter   = ( isset( $_POST['delimiter'] ) ? give_clean( $_POST['delimiter'] ) : 'csv' );
-			$mode        = empty( $_POST['mode'] ) ?
-				'disabled' :
-				( give_is_setting_enabled( give_clean( $_POST['mode'] ) ) ? 'enabled' : 'disabled' );
-			$create_user = empty( $_POST['create_user'] ) ?
-				'disabled' :
-				( give_is_setting_enabled( give_clean( $_POST['create_user'] ) ) ? 'enabled' : 'disabled' );
-			$delete_csv  = empty( $_POST['delete_csv'] ) ?
-				'enabled' :
-				( give_is_setting_enabled( give_clean( $_POST['delete_csv'] ) ) ? 'enabled' : 'disabled' );
+			$csv         = (isset($_POST['csv']) ? give_clean($_POST['csv']) : '');
+			$csv_id      = (isset($_POST['csv_id']) ? give_clean($_POST['csv_id']) : '');
+			$delimiter   = (isset($_POST['delimiter']) ? give_clean($_POST['delimiter']) : 'csv');
+			$mode        = empty($_POST['mode']) ?
+				'disabled' : (give_is_setting_enabled(give_clean($_POST['mode'])) ? 'enabled' : 'disabled');
+			$create_user = empty($_POST['create_user']) ?
+				'disabled' : (give_is_setting_enabled(give_clean($_POST['create_user'])) ? 'enabled' : 'disabled');
+			$delete_csv  = empty($_POST['delete_csv']) ?
+				'enabled' : (give_is_setting_enabled(give_clean($_POST['delete_csv'])) ? 'enabled' : 'disabled');
 
 			// Reset csv and csv_id if csv
-			if ( empty( $csv_id ) || ! $this->is_valid_csv( $csv_id, $csv ) ) {
+			if (empty($csv_id) || ! $this->is_valid_csv($csv_id, $csv)) {
 				$csv_id = $csv = '';
 			}
-			$per_page = isset( $_POST['per_page'] ) ? absint( $_POST['per_page'] ) : self::$per_page;
+			$per_page = isset($_POST['per_page']) ? absint($_POST['per_page']) : self::$per_page;
 
 			$sample_file_text = sprintf(
 				'%s <a href="%s">%s</a>.',
-				__( 'Download the sample file', 'give' ),
-				esc_url( GIVE_PLUGIN_URL . 'sample-data/sample-data.csv' ),
-				__( 'here', 'give' )
+				__('Download the sample file', 'give'),
+				esc_url(GIVE_PLUGIN_URL.'sample-data/sample-data.csv'),
+				__('here', 'give')
 			);
 
 			$csv_description = sprintf(
 				'%1$s %2$s',
-				__( 'The file must be a Comma Seperated Version (CSV) file type only.', 'give' ),
+				__('The file must be a Comma Seperated Version (CSV) file type only.', 'give'),
 				$sample_file_text
 			);
 
 			$settings = array(
 				array(
 					'id'          => 'csv',
-					'name'        => __( 'Choose a CSV file:', 'give' ),
+					'name'        => __('Choose a CSV file:', 'give'),
 					'type'        => 'file',
-					'attributes'  => array( 'editing' => 'false', 'library' => 'text' ),
+					'attributes'  => array('editing' => 'false', 'library' => 'text'),
 					'description' => $csv_description,
 					'fvalue'      => 'url',
 					'default'     => $csv,
@@ -927,62 +924,62 @@ 
 				),
 				array(
 					'id'          => 'delimiter',
-					'name'        => __( 'CSV Delimiter:', 'give' ),
-					'description' => __( 'In case your CSV file supports a different type of separator (or delimiter) -- like a tab or space -- you can set that here.', 'give' ),
+					'name'        => __('CSV Delimiter:', 'give'),
+					'description' => __('In case your CSV file supports a different type of separator (or delimiter) -- like a tab or space -- you can set that here.', 'give'),
 					'default'     => $delimiter,
 					'type'        => 'select',
 					'options'     => array(
-						'csv'                  => __( 'Comma', 'give' ),
-						'tab-separated-values' => __( 'Tab', 'give' ),
+						'csv'                  => __('Comma', 'give'),
+						'tab-separated-values' => __('Tab', 'give'),
 					),
 				),
 				array(
 					'id'          => 'mode',
-					'name'        => __( 'Test Mode:', 'give' ),
-					'description' => __( 'Select whether you would like these donations to be marked as "test" donations within the database. By default, they will be marked as live donations.', 'give' ),
+					'name'        => __('Test Mode:', 'give'),
+					'description' => __('Select whether you would like these donations to be marked as "test" donations within the database. By default, they will be marked as live donations.', 'give'),
 					'default'     => $mode,
 					'type'        => 'radio_inline',
 					'options'     => array(
-						'enabled'  => __( 'Enabled', 'give' ),
-						'disabled' => __( 'Disabled', 'give' ),
+						'enabled'  => __('Enabled', 'give'),
+						'disabled' => __('Disabled', 'give'),
 					),
 				),
 				array(
 					'id'          => 'create_user',
-					'name'        => __( 'Create WP users for new donors:', 'give' ),
-					'description' => __( 'The importer can create WordPress user accounts based on the names and email addresses of the donations in your CSV file. Enable this option if you\'d like the importer to do that.', 'give' ),
+					'name'        => __('Create WP users for new donors:', 'give'),
+					'description' => __('The importer can create WordPress user accounts based on the names and email addresses of the donations in your CSV file. Enable this option if you\'d like the importer to do that.', 'give'),
 					'default'     => $create_user,
 					'type'        => 'radio_inline',
 					'options'     => array(
-						'enabled'  => __( 'Enabled', 'give' ),
-						'disabled' => __( 'Disabled', 'give' ),
+						'enabled'  => __('Enabled', 'give'),
+						'disabled' => __('Disabled', 'give'),
 					),
 				),
 				array(
 					'id'          => 'delete_csv',
-					'name'        => __( 'Delete CSV after import:', 'give' ),
-					'description' => __( 'Your CSV file will be uploaded via the WordPress Media Library. It\'s a good idea to delete it after the import is finished so that your sensitive data is not accessible on the web. Disable this only if you plan to delete the file manually later.', 'give' ),
+					'name'        => __('Delete CSV after import:', 'give'),
+					'description' => __('Your CSV file will be uploaded via the WordPress Media Library. It\'s a good idea to delete it after the import is finished so that your sensitive data is not accessible on the web. Disable this only if you plan to delete the file manually later.', 'give'),
 					'default'     => $delete_csv,
 					'type'        => 'radio_inline',
 					'options'     => array(
-						'enabled'  => __( 'Enabled', 'give' ),
-						'disabled' => __( 'Disabled', 'give' ),
+						'enabled'  => __('Enabled', 'give'),
+						'disabled' => __('Disabled', 'give'),
 					),
 				),
 				array(
 					'id'          => 'per_page',
-					'name'        => __( 'Process Rows Per Batch:', 'give' ),
+					'name'        => __('Process Rows Per Batch:', 'give'),
 					'type'        => 'number',
-					'description' => __( 'Determine how many rows you would like to import per cycle.', 'give' ),
+					'description' => __('Determine how many rows you would like to import per cycle.', 'give'),
 					'default'     => $per_page,
 					'class'       => 'give-text-small',
 				),
 			);
 
-			$settings = apply_filters( 'give_import_file_upload_html', $settings );
+			$settings = apply_filters('give_import_file_upload_html', $settings);
 
-			if ( empty( $this->is_csv_valid ) ) {
-				Give_Admin_Settings::output_fields( $settings, 'give_settings' );
+			if (empty($this->is_csv_valid)) {
+				Give_Admin_Settings::output_fields($settings, 'give_settings');
 			} else {
 				?>
 				<input type="hidden" name="is_csv_valid" class="is_csv_valid"
@@ -1001,28 +998,25 @@ 
 			$step = $this->get_step();
 
 			// Validation for first step.
-			if ( 1 === $step ) {
-				$csv_id = absint( $_POST['csv_id'] );
+			if (1 === $step) {
+				$csv_id = absint($_POST['csv_id']);
 
-				if ( $this->is_valid_csv( $csv_id, esc_url( $_POST['csv'] ) ) ) {
+				if ($this->is_valid_csv($csv_id, esc_url($_POST['csv']))) {
 
-					$url = give_import_page_url( (array) apply_filters( 'give_import_step_two_url', array(
+					$url = give_import_page_url((array) apply_filters('give_import_step_two_url', array(
 						'step'          => '2',
 						'importer-type' => $this->importer_type,
 						'csv'           => $csv_id,
-						'delimiter'     => isset( $_REQUEST['delimiter'] ) ? give_clean( $_REQUEST['delimiter'] ) : 'csv',
-						'mode'          => empty( $_POST['mode'] ) ?
-							'0' :
-							( give_is_setting_enabled( give_clean( $_POST['mode'] ) ) ? '1' : '0' ),
-						'create_user'   => empty( $_POST['create_user'] ) ?
-							'0' :
-							( give_is_setting_enabled( give_clean( $_POST['create_user'] ) ) ? '1' : '0' ),
-						'delete_csv'    => empty( $_POST['delete_csv'] ) ?
-							'1' :
-							( give_is_setting_enabled( give_clean( $_POST['delete_csv'] ) ) ? '1' : '0' ),
-						'per_page'      => isset( $_POST['per_page'] ) ? absint( $_POST['per_page'] ) : self::$per_page,
-						'dry_run'       => isset( $_POST['dry_run'] ) ? absint( $_POST['dry_run'] ) : 0,
-					) ) );
+						'delimiter'     => isset($_REQUEST['delimiter']) ? give_clean($_REQUEST['delimiter']) : 'csv',
+						'mode'          => empty($_POST['mode']) ?
+							'0' : (give_is_setting_enabled(give_clean($_POST['mode'])) ? '1' : '0'),
+						'create_user'   => empty($_POST['create_user']) ?
+							'0' : (give_is_setting_enabled(give_clean($_POST['create_user'])) ? '1' : '0'),
+						'delete_csv'    => empty($_POST['delete_csv']) ?
+							'1' : (give_is_setting_enabled(give_clean($_POST['delete_csv'])) ? '1' : '0'),
+						'per_page'      => isset($_POST['per_page']) ? absint($_POST['per_page']) : self::$per_page,
+						'dry_run'       => isset($_POST['dry_run']) ? absint($_POST['dry_run']) : 0,
+					)));
 
 					$this->is_csv_valid = $url;
 				}
@@ -1040,25 +1034,25 @@ 
 		 *
 		 * @return bool $has_error CSV is valid or not.
 		 */
-		private function is_valid_csv( $csv = false, $match_url = '' ) {
+		private function is_valid_csv($csv = false, $match_url = '') {
 			$is_valid_csv = true;
 
-			if ( $csv ) {
-				$csv_url = wp_get_attachment_url( $csv );
+			if ($csv) {
+				$csv_url = wp_get_attachment_url($csv);
 
-				$delimiter = ( ! empty( $_REQUEST['delimiter'] ) ? give_clean( $_REQUEST['delimiter'] ) : 'csv' );
+				$delimiter = ( ! empty($_REQUEST['delimiter']) ? give_clean($_REQUEST['delimiter']) : 'csv');
 
 				if (
 					! $csv_url ||
-					( ! empty( $match_url ) && ( $csv_url !== $match_url ) ) ||
-					( ( $mime_type = get_post_mime_type( $csv ) ) && ! strpos( $mime_type, $delimiter ) )
+					( ! empty($match_url) && ($csv_url !== $match_url)) ||
+					(($mime_type = get_post_mime_type($csv)) && ! strpos($mime_type, $delimiter))
 				) {
 					$is_valid_csv = false;
-					Give_Admin_Settings::add_error( 'give-import-csv', __( 'Please upload or provide a valid CSV file.', 'give' ) );
+					Give_Admin_Settings::add_error('give-import-csv', __('Please upload or provide a valid CSV file.', 'give'));
 				}
 			} else {
 				$is_valid_csv = false;
-				Give_Admin_Settings::add_error( 'give-import-csv', __( 'Please upload or provide a valid CSV file.', 'give' ) );
+				Give_Admin_Settings::add_error('give-import-csv', __('Please upload or provide a valid CSV file.', 'give'));
 			}
 
 			return $is_valid_csv;
@@ -1074,8 +1068,8 @@ 
 		 * @param $field
 		 * @param $option_value
 		 */
-		public function render_import_field( $field, $option_value ) {
-			include_once GIVE_PLUGIN_DIR . 'includes/admin/tools/views/html-admin-page-imports.php';
+		public function render_import_field($field, $option_value) {
+			include_once GIVE_PLUGIN_DIR.'includes/admin/tools/views/html-admin-page-imports.php';
 		}
 
 		/**
@@ -1086,8 +1080,8 @@ 
 		 */
 		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'] );
+			       isset($_GET['importer-type']) &&
+			       $this->importer_type === give_clean($_GET['importer-type']);
 		}
 	}
 

includes/forms/template.php

Braces +5 added lines, -2 removed lines

@@ -1020,9 +1020,12 @@
 				    }
 				    ?>
 			    </select>
-		    <?php else : ?>
+		    <?php else {
+	: ?>
 			    <input type="text" size="6" name="card_state" id="card_state" class="card_state give-input"
-			           placeholder="<?php echo $label; ?>" value="<?php echo $selected_state; ?>"/>
+			           placeholder="<?php echo $label;
+}
+?>" value="<?php echo $selected_state; ?>"/>
 		    <?php endif; ?>
 	    </p>
 

Doc Comments +7 added lines, -8 removed lines

@@ -21,7 +21,7 @@ 
  *
  * @since 1.0
  *
- * @return string Donation form.
+ * @return false|null Donation form.
  */
 function give_get_donation_form( $args = array() ) {
 
@@ -1482,7 +1482,7 @@ 
  *
  * @param  int $form_id The form ID.
  *
- * @return bool
+ * @return false|null
  */
 function give_terms_agreement( $form_id ) {
 	$form_option = give_get_meta( $form_id, '_give_terms_option', true );
@@ -1676,7 +1676,7 @@ 
  * @param  int   $form_id The form ID.
  * @param  array $args    An array of form arguments.
  *
- * @return mixed
+ * @return boolean
  */
 function give_show_goal_progress( $form_id, $args = array() ) {
 
@@ -1703,7 +1703,7 @@ 
  * @param  int $total      Total amount based on shortcode parameter.
  * @param  int $total_goal Total Goal amount passed by Admin.
  *
- * @return mixed
+ * @return boolean
  */
 function give_show_goal_totals_progress( $total, $total_goal ) {
 
@@ -1727,7 +1727,7 @@ 
  *
  * @since  1.8
  *
- * @param  $form_id
+ * @param  integer $form_id
  * @param  $args
  *
  * @return mixed|string
@@ -1768,7 +1768,7 @@ 
  * @param  int   $form_id The form ID.
  * @param  array $args    An array of form arguments.
  *
- * @return void|bool
+ * @return false|null
  */
 function give_form_content( $form_id, $args ) {
 
@@ -2058,7 +2058,7 @@ 
  *
  * @since 2.1
  *
- * @return array
+ * @return string[]
  */
 function add_class_for_form_grid( $class, $id, $args ) {
 	$class[] = 'give-form-grid-wrap';
@@ -2069,7 +2069,6 @@ 
 /**
  * Add hidden field to Form Grid page
  *
- * @param int              $form_id The form ID.
  * @param array            $args    An array of form arguments.
  * @param Give_Donate_Form $form    Form object.
  *

Indentation +12 added lines, -12 removed lines

@@ -996,10 +996,10 @@ 
 			    <?php echo( give_field_is_required( 'billing_country', $form_id ) ? ' required aria-required="true" ' : '' ); ?>
 		    >
 			    <?php
-			    foreach ( $countries as $country_code => $country ) {
-				    echo '<option value="' . esc_attr( $country_code ) . '"' . selected( $country_code, $selected_country, false ) . '>' . $country . '</option>';
-			    }
-			    ?>
+				foreach ( $countries as $country_code => $country ) {
+					echo '<option value="' . esc_attr( $country_code ) . '"' . selected( $country_code, $selected_country, false ) . '>' . $country . '</option>';
+				}
+				?>
 		    </select>
 	    </p>
 
@@ -1071,7 +1071,7 @@ 
 		    <label for="card_state" class="give-label">
 			    <span class="state-label-text"><?php echo $label; ?></span>
 			    <?php if ( give_field_is_required( 'card_state', $form_id ) ) :
-				    ?>
+					?>
 				    <span class="give-required-indicator <?php echo( array_key_exists( $selected_country, $states_not_required_country_list ) ? 'give-hidden' : '' ) ?> ">*</span>
 			    <?php endif; ?>
 			    <span class="give-tooltip give-icon give-icon-question"
@@ -1079,7 +1079,7 @@ 
 		    </label>
 		    <?php
 
-		    if ( ! empty( $states ) ) : ?>
+			if ( ! empty( $states ) ) : ?>
 			    <select
 					    name="card_state"
 					    autocomplete="address-level4"
@@ -1087,10 +1087,10 @@ 
 					    class="card_state give-select<?php echo( give_field_is_required( 'card_state', $form_id ) ? ' required' : '' ); ?>"
 				    <?php echo( give_field_is_required( 'card_state', $form_id ) ? ' required aria-required="true" ' : '' ); ?>>
 				    <?php
-				    foreach ( $states as $state_code => $state ) {
-					    echo '<option value="' . $state_code . '"' . selected( $state_code, $selected_state, false ) . '>' . $state . '</option>';
-				    }
-				    ?>
+					foreach ( $states as $state_code => $state ) {
+						echo '<option value="' . $state_code . '"' . selected( $state_code, $selected_state, false ) . '>' . $state . '</option>';
+					}
+					?>
 			    </select>
 		    <?php else : ?>
 			    <input type="text" size="6" name="card_state" id="card_state" class="card_state give-input"
@@ -1292,8 +1292,8 @@ 
 				<a href="<?php echo remove_query_arg( 'login' ); ?>" class="give-checkout-register-cancel"
 				   data-action="give_checkout_register">
 					<?php if ( give_logged_in_only( $form_id ) ) {
-					    _e( 'Register as a part of your donation &raquo;', 'give' );
-                    } else {
+						_e( 'Register as a part of your donation &raquo;', 'give' );
+					} else {
 						 _e( 'Register or donate as a guest &raquo;', 'give' );
 					} ?>
 				</a>

Spacing +490 added lines, -492 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,35 +23,35 @@ 
  *
  * @return string Donation form.
  */
-function give_get_donation_form( $args = array() ) {
+function give_get_donation_form($args = array()) {
 
 	global $post;
 	static $count = 1;
 
-	$form_id = is_object( $post ) ? $post->ID : 0;
+	$form_id = is_object($post) ? $post->ID : 0;
 
-	if ( isset( $args['id'] ) ) {
+	if (isset($args['id'])) {
 		$form_id = $args['id'];
 	}
 
-	$defaults = apply_filters( 'give_form_args_defaults', array(
+	$defaults = apply_filters('give_form_args_defaults', array(
 		'form_id' => $form_id,
-	) );
+	));
 
-	$args = wp_parse_args( $args, $defaults );
+	$args = wp_parse_args($args, $defaults);
 
-	$form = new Give_Donate_Form( $args['form_id'] );
+	$form = new Give_Donate_Form($args['form_id']);
 
 	// Bail out, if no form ID.
-	if ( empty( $form->ID ) ) {
+	if (empty($form->ID)) {
 		return false;
 	}
 
 	$args['id_prefix'] = "{$form_id}-{$count}";
-	$payment_mode      = give_get_chosen_gateway( $form->ID );
+	$payment_mode      = give_get_chosen_gateway($form->ID);
 
 	$form_action = add_query_arg(
-		apply_filters( 'give_form_action_args', array(
+		apply_filters('give_form_action_args', array(
 				'payment-mode' => $payment_mode,
 			)
 		),
@@ -60,17 +60,17 @@ 
 
 	// Sanity Check: Donation form not published or user doesn't have permission to view drafts.
 	if (
-		( 'publish' !== $form->post_status && ! current_user_can( 'edit_give_forms', $form->ID ) )
-		|| ( 'trash' === $form->post_status )
+		('publish' !== $form->post_status && ! current_user_can('edit_give_forms', $form->ID))
+		|| ('trash' === $form->post_status)
 	) {
 		return false;
 	}
 
 	// Get the form wrap CSS classes.
-	$form_wrap_classes = $form->get_form_wrap_classes( $args );
+	$form_wrap_classes = $form->get_form_wrap_classes($args);
 
 	// Get the <form> tag wrap CSS classes.
-	$form_classes = $form->get_form_classes( $args );
+	$form_classes = $form->get_form_classes($args);
 
 	ob_start();
 
@@ -82,21 +82,21 @@ 
 	 * @param int   $form_id The form ID.
 	 * @param array $args    An array of form arguments.
 	 */
-	do_action( 'give_pre_form_output', $form->ID, $args, $form );
+	do_action('give_pre_form_output', $form->ID, $args, $form);
 
 	?>
 	<div id="give-form-<?php echo $form->ID; ?>-wrap" class="<?php echo $form_wrap_classes; ?>">
 		<?php
-		if ( $form->is_close_donation_form() ) {
+		if ($form->is_close_donation_form()) {
 
-			$form_title = ! is_singular( 'give_forms' ) ? apply_filters( 'give_form_title', '<h2 class="give-form-title">' . get_the_title( $form_id ) . '</h2>' ) : '';
+			$form_title = ! is_singular('give_forms') ? apply_filters('give_form_title', '<h2 class="give-form-title">'.get_the_title($form_id).'</h2>') : '';
 
 			// Get Goal thank you message.
-			$goal_achieved_message = get_post_meta( $form->ID, '_give_form_goal_achieved_message', true );
-			$goal_achieved_message = ! empty( $goal_achieved_message ) ? $form_title . apply_filters( 'the_content', $goal_achieved_message ) : '';
+			$goal_achieved_message = get_post_meta($form->ID, '_give_form_goal_achieved_message', true);
+			$goal_achieved_message = ! empty($goal_achieved_message) ? $form_title.apply_filters('the_content', $goal_achieved_message) : '';
 
 			// Print thank you message.
-			echo apply_filters( 'give_goal_closed_output', $goal_achieved_message, $form->ID, $form );
+			echo apply_filters('give_goal_closed_output', $goal_achieved_message, $form->ID, $form);
 
 		} else {
 			/**
@@ -104,13 +104,13 @@ 
 			 * 1. if show_title params set to true
 			 * 2. if admin set form display_style to button
 			 */
-			$form_title = apply_filters( 'give_form_title', '<h2 class="give-form-title">' . get_the_title( $form_id ) . '</h2>' );
+			$form_title = apply_filters('give_form_title', '<h2 class="give-form-title">'.get_the_title($form_id).'</h2>');
 			if (
 				(
-					( isset( $args['show_title'] ) && $args['show_title'] == true )
-					|| ( 'button' === get_post_meta( $form_id, '_give_payment_display', true ) )
+					(isset($args['show_title']) && $args['show_title'] == true)
+					|| ('button' === get_post_meta($form_id, '_give_payment_display', true))
 				)
-				&& ! doing_action( 'give_single_form_summary' )
+				&& ! doing_action('give_single_form_summary')
 			) {
 				echo $form_title;
 			}
@@ -124,13 +124,13 @@ 
 			 * @param array            $args    An array of form arguments.
 			 * @param Give_Donate_Form $form    Form object.
 			 */
-			do_action( 'give_pre_form', $form->ID, $args, $form );
+			do_action('give_pre_form', $form->ID, $args, $form);
 
 			// Set form html tags.
 			$form_html_tags = array(
 				'id'      => "give-form-{$args['id_prefix']}",
 				'class'   => $form_classes,
-				'action'  => esc_url_raw( $form_action ),
+				'action'  => esc_url_raw($form_action),
 				'data-id' => $args['id_prefix'],
 			);
 
@@ -142,9 +142,9 @@ 
 			 * @param array            $form_html_tags Array of form html tags.
 			 * @param Give_Donate_Form $form           Form object.
 			 */
-			$form_html_tags = apply_filters( 'give_form_html_tags', (array) $form_html_tags, $form );
+			$form_html_tags = apply_filters('give_form_html_tags', (array) $form_html_tags, $form);
 			?>
-			<form <?php echo give_get_attribute_str( $form_html_tags ); ?> method="post">
+			<form <?php echo give_get_attribute_str($form_html_tags); ?> method="post">
 				<!-- The following field is for robots only, invisible to humans: -->
 				<span class="give-hidden" style="display: none !important;">
 					<label for="give-form-honeypot-<?php echo $form_id; ?>"></label>
@@ -162,7 +162,7 @@ 
 				 * @param array            $args    An array of form arguments.
 				 * @param Give_Donate_Form $form    Form object.
 				 */
-				do_action( 'give_donation_form_top', $form->ID, $args, $form );
+				do_action('give_donation_form_top', $form->ID, $args, $form);
 
 				/**
 				 * Fires while outputting donation form, for payment gateway fields.
@@ -173,7 +173,7 @@ 
 				 * @param array            $args    An array of form arguments.
 				 * @param Give_Donate_Form $form    Form object.
 				 */
-				do_action( 'give_payment_mode_select', $form->ID, $args, $form );
+				do_action('give_payment_mode_select', $form->ID, $args, $form);
 
 				/**
 				 * Fires while outputting donation form, after all other fields.
@@ -184,7 +184,7 @@ 
 				 * @param array            $args    An array of form arguments.
 				 * @param Give_Donate_Form $form    Form object.
 				 */
-				do_action( 'give_donation_form_bottom', $form->ID, $args, $form );
+				do_action('give_donation_form_bottom', $form->ID, $args, $form);
 
 				?>
 			</form>
@@ -199,12 +199,12 @@ 
 			 * @param array            $args    An array of form arguments.
 			 * @param Give_Donate_Form $form    Form object.
 			 */
-			do_action( 'give_post_form', $form->ID, $args, $form );
+			do_action('give_post_form', $form->ID, $args, $form);
 
 		}
 		?>
 
-	</div><!--end #give-form-<?php echo absint( $form->ID ); ?>-->
+	</div><!--end #give-form-<?php echo absint($form->ID); ?>-->
 	<?php
 
 	/**
@@ -215,12 +215,12 @@ 
 	 * @param int   $form_id The form ID.
 	 * @param array $args    An array of form arguments.
 	 */
-	do_action( 'give_post_form_output', $form->ID, $args );
+	do_action('give_post_form_output', $form->ID, $args);
 
 	$final_output = ob_get_clean();
-	$count ++;
+	$count++;
 
-	echo apply_filters( 'give_donate_form', $final_output, $args );
+	echo apply_filters('give_donate_form', $final_output, $args);
 }
 
 /**
@@ -237,11 +237,11 @@ 
  *
  * @return string
  */
-function give_show_purchase_form( $form_id, $args ) {
+function give_show_purchase_form($form_id, $args) {
 
-	$payment_mode = give_get_chosen_gateway( $form_id );
+	$payment_mode = give_get_chosen_gateway($form_id);
 
-	if ( ! isset( $form_id ) && isset( $_POST['give_form_id'] ) ) {
+	if ( ! isset($form_id) && isset($_POST['give_form_id'])) {
 		$form_id = $_POST['give_form_id'];
 	}
 
@@ -250,33 +250,33 @@ 
 	 *
 	 * @since 1.7
 	 */
-	do_action( 'give_payment_fields_top', $form_id );
+	do_action('give_payment_fields_top', $form_id);
 
-	if ( give_can_checkout() && isset( $form_id ) ) {
+	if (give_can_checkout() && isset($form_id)) {
 
 		/**
 		 * Fires while displaying donation form, before registration login.
 		 *
 		 * @since 1.7
 		 */
-		do_action( 'give_donation_form_before_register_login', $form_id, $args );
+		do_action('give_donation_form_before_register_login', $form_id, $args);
 
 		/**
 		 * Fire when register/login form fields render.
 		 *
 		 * @since 1.7
 		 */
-		do_action( 'give_donation_form_register_login_fields', $form_id, $args );
+		do_action('give_donation_form_register_login_fields', $form_id, $args);
 
 		/**
 		 * Fire when credit card form fields render.
 		 *
 		 * @since 1.7
 		 */
-		do_action( 'give_donation_form_before_cc_form', $form_id, $args );
+		do_action('give_donation_form_before_cc_form', $form_id, $args);
 
 		// Load the credit card form and allow gateways to load their own if they wish.
-		if ( has_action( 'give_' . $payment_mode . '_cc_form' ) ) {
+		if (has_action('give_'.$payment_mode.'_cc_form')) {
 			/**
 			 * Fires while displaying donation form, credit card form fields for a given gateway.
 			 *
@@ -284,7 +284,7 @@ 
 			 *
 			 * @param int $form_id The form ID.
 			 */
-			do_action( "give_{$payment_mode}_cc_form", $form_id, $args );
+			do_action("give_{$payment_mode}_cc_form", $form_id, $args);
 		} else {
 			/**
 			 * Fires while displaying donation form, credit card form fields.
@@ -293,7 +293,7 @@ 
 			 *
 			 * @param int $form_id The form ID.
 			 */
-			do_action( 'give_cc_form', $form_id, $args );
+			do_action('give_cc_form', $form_id, $args);
 		}
 
 		/**
@@ -301,7 +301,7 @@ 
 		 *
 		 * @since 1.7
 		 */
-		do_action( 'give_donation_form_after_cc_form', $form_id, $args );
+		do_action('give_donation_form_after_cc_form', $form_id, $args);
 
 	} else {
 		/**
@@ -309,7 +309,7 @@ 
 		 *
 		 * @since 1.7
 		 */
-		do_action( 'give_donation_form_no_access', $form_id );
+		do_action('give_donation_form_no_access', $form_id);
 
 	}
 
@@ -318,10 +318,10 @@ 
 	 *
 	 * @since 1.7
 	 */
-	do_action( 'give_payment_fields_bottom', $form_id, $args );
+	do_action('give_payment_fields_bottom', $form_id, $args);
 }
 
-add_action( 'give_donation_form', 'give_show_purchase_form', 10, 2 );
+add_action('give_donation_form', 'give_show_purchase_form', 10, 2);
 
 /**
  * Give Show Login/Register Form Fields.
@@ -332,11 +332,11 @@ 
  *
  * @return void
  */
-function give_show_register_login_fields( $form_id ) {
+function give_show_register_login_fields($form_id) {
 
-	$show_register_form = give_show_login_register_option( $form_id );
+	$show_register_form = give_show_login_register_option($form_id);
 
-	if ( ( $show_register_form === 'registration' || ( $show_register_form === 'both' && ! isset( $_GET['login'] ) ) ) && ! is_user_logged_in() ) :
+	if (($show_register_form === 'registration' || ($show_register_form === 'both' && ! isset($_GET['login']))) && ! is_user_logged_in()) :
 		?>
 		<div id="give-checkout-login-register-<?php echo $form_id; ?>">
 			<?php
@@ -345,11 +345,11 @@ 
 			 *
 			 * @since 1.7
 			 */
-			do_action( 'give_donation_form_register_fields', $form_id );
+			do_action('give_donation_form_register_fields', $form_id);
 			?>
 		</div>
 		<?php
-	elseif ( ( $show_register_form === 'login' || ( $show_register_form === 'both' && isset( $_GET['login'] ) ) ) && ! is_user_logged_in() ) :
+	elseif (($show_register_form === 'login' || ($show_register_form === 'both' && isset($_GET['login']))) && ! is_user_logged_in()) :
 		?>
 		<div id="give-checkout-login-register-<?php echo $form_id; ?>">
 			<?php
@@ -358,23 +358,23 @@ 
 			 *
 			 * @since 1.7
 			 */
-			do_action( 'give_donation_form_login_fields', $form_id );
+			do_action('give_donation_form_login_fields', $form_id);
 			?>
 		</div>
 		<?php
 	endif;
 
-	if ( ( ! isset( $_GET['login'] ) && is_user_logged_in() ) || ! isset( $show_register_form ) || 'none' === $show_register_form || 'login' === $show_register_form ) {
+	if (( ! isset($_GET['login']) && is_user_logged_in()) || ! isset($show_register_form) || 'none' === $show_register_form || 'login' === $show_register_form) {
 		/**
 		 * Fire when user info render.
 		 *
 		 * @since 1.7
 		 */
-		do_action( 'give_donation_form_after_user_info', $form_id );
+		do_action('give_donation_form_after_user_info', $form_id);
 	}
 }
 
-add_action( 'give_donation_form_register_login_fields', 'give_show_register_login_fields' );
+add_action('give_donation_form_register_login_fields', 'give_show_register_login_fields');
 
 /**
  * Donation Amount Field.
@@ -389,16 +389,16 @@ 
  *
  * @return void
  */
-function give_output_donation_amount_top( $form_id = 0, $args = array() ) {
+function give_output_donation_amount_top($form_id = 0, $args = array()) {
 
 	$give_options        = give_get_settings();
-	$variable_pricing    = give_has_variable_prices( $form_id );
-	$allow_custom_amount = give_get_meta( $form_id, '_give_custom_amount', true );
-	$currency_position   = isset( $give_options['currency_position'] ) ? $give_options['currency_position'] : 'before';
-	$symbol              = give_currency_symbol( give_get_currency( $form_id, $args ) );
-	$currency_output     = '<span class="give-currency-symbol give-currency-position-' . $currency_position . '">' . $symbol . '</span>';
-	$default_amount      = give_format_amount( give_get_default_form_amount( $form_id ), array( 'sanitize' => false, 'currency' => give_get_currency( $form_id ) ) );
-	$custom_amount_text  = give_get_meta( $form_id, '_give_custom_amount_text', true );
+	$variable_pricing    = give_has_variable_prices($form_id);
+	$allow_custom_amount = give_get_meta($form_id, '_give_custom_amount', true);
+	$currency_position   = isset($give_options['currency_position']) ? $give_options['currency_position'] : 'before';
+	$symbol              = give_currency_symbol(give_get_currency($form_id, $args));
+	$currency_output     = '<span class="give-currency-symbol give-currency-position-'.$currency_position.'">'.$symbol.'</span>';
+	$default_amount      = give_format_amount(give_get_default_form_amount($form_id), array('sanitize' => false, 'currency' => give_get_currency($form_id)));
+	$custom_amount_text  = give_get_meta($form_id, '_give_custom_amount_text', true);
 
 	/**
 	 * Fires while displaying donation form, before donation level fields.
@@ -408,20 +408,20 @@ 
 	 * @param int   $form_id The form ID.
 	 * @param array $args    An array of form arguments.
 	 */
-	do_action( 'give_before_donation_levels', $form_id, $args );
+	do_action('give_before_donation_levels', $form_id, $args);
 
 	// Set Price, No Custom Amount Allowed means hidden price field.
-	if ( ! give_is_setting_enabled( $allow_custom_amount ) ) {
+	if ( ! give_is_setting_enabled($allow_custom_amount)) {
 		?>
-		<label class="give-hidden" for="give-amount-hidden"><?php esc_html_e( 'Donation Amount:', 'give' ); ?></label>
+		<label class="give-hidden" for="give-amount-hidden"><?php esc_html_e('Donation Amount:', 'give'); ?></label>
 		<input id="give-amount" class="give-amount-hidden" type="hidden" name="give-amount"
 			   value="<?php echo $default_amount; ?>" required aria-required="true"/>
 		<div class="set-price give-donation-amount form-row-wide">
-			<?php if ( $currency_position == 'before' ) {
+			<?php if ($currency_position == 'before') {
 				echo $currency_output;
 			} ?>
 			<span id="give-amount-text" class="give-text-input give-amount-top"><?php echo $default_amount; ?></span>
-			<?php if ( $currency_position == 'after' ) {
+			<?php if ($currency_position == 'after') {
 				echo $currency_output;
 			} ?>
 		</div>
@@ -431,13 +431,13 @@ 
 		?>
 		<div class="give-total-wrap">
 			<div class="give-donation-amount form-row-wide">
-				<?php if ( $currency_position == 'before' ) {
+				<?php if ($currency_position == 'before') {
 					echo $currency_output;
 				} ?>
-				<label class="give-hidden" for="give-amount"><?php esc_html_e( 'Donation Amount:', 'give' ); ?></label>
+				<label class="give-hidden" for="give-amount"><?php esc_html_e('Donation Amount:', 'give'); ?></label>
 				<input class="give-text-input give-amount-top" id="give-amount" name="give-amount" type="tel"
 					   placeholder="" value="<?php echo $default_amount; ?>" autocomplete="off">
-				<?php if ( $currency_position == 'after' ) {
+				<?php if ($currency_position == 'after') {
 					echo $currency_output;
 				} ?>
 			</div>
@@ -452,16 +452,16 @@ 
 	 * @param int   $form_id The form ID.
 	 * @param array $args    An array of form arguments.
 	 */
-	do_action( 'give_after_donation_amount', $form_id, $args );
+	do_action('give_after_donation_amount', $form_id, $args);
 
 	// Custom Amount Text
-	if ( ! $variable_pricing && give_is_setting_enabled( $allow_custom_amount ) && ! empty( $custom_amount_text ) ) { ?>
+	if ( ! $variable_pricing && give_is_setting_enabled($allow_custom_amount) && ! empty($custom_amount_text)) { ?>
 		<p class="give-custom-amount-text"><?php echo $custom_amount_text; ?></p>
 	<?php }
 
 	// Output Variable Pricing Levels.
-	if ( $variable_pricing ) {
-		give_output_levels( $form_id );
+	if ($variable_pricing) {
+		give_output_levels($form_id);
 	}
 
 	/**
@@ -472,10 +472,10 @@ 
 	 * @param int   $form_id The form ID.
 	 * @param array $args    An array of form arguments.
 	 */
-	do_action( 'give_after_donation_levels', $form_id, $args );
+	do_action('give_after_donation_levels', $form_id, $args);
 }
 
-add_action( 'give_donation_form_top', 'give_output_donation_amount_top', 10, 2 );
+add_action('give_donation_form_top', 'give_output_donation_amount_top', 10, 2);
 
 /**
  * Outputs the Donation Levels in various formats such as dropdown, radios, and buttons.
@@ -486,7 +486,7 @@ 
  *
  * @return string Donation levels.
  */
-function give_output_levels( $form_id ) {
+function give_output_levels($form_id) {
 
 	/**
 	 * Filter the variable pricing
@@ -499,47 +499,47 @@ 
 	 * @param array $prices Array of variable prices.
 	 * @param int   $form   Form ID.
 	 */
-	$prices = apply_filters( 'give_form_variable_prices', give_get_variable_prices( $form_id ), $form_id );
+	$prices = apply_filters('give_form_variable_prices', give_get_variable_prices($form_id), $form_id);
 
 
-	$display_style      = give_get_meta( $form_id, '_give_display_style', true );
-	$custom_amount      = give_get_meta( $form_id, '_give_custom_amount', true );
-	$custom_amount_text = give_get_meta( $form_id, '_give_custom_amount_text', true );
+	$display_style      = give_get_meta($form_id, '_give_display_style', true);
+	$custom_amount      = give_get_meta($form_id, '_give_custom_amount', true);
+	$custom_amount_text = give_get_meta($form_id, '_give_custom_amount_text', true);
 
-	if ( empty( $custom_amount_text ) ) {
-		$custom_amount_text = esc_html__( 'Give a Custom Amount', 'give' );
+	if (empty($custom_amount_text)) {
+		$custom_amount_text = esc_html__('Give a Custom Amount', 'give');
 	}
 
 	$output = '';
 
-	switch ( $display_style ) {
+	switch ($display_style) {
 		case 'buttons':
 
 			$output .= '<ul id="give-donation-level-button-wrap" class="give-donation-levels-wrap give-list-inline">';
 
-			foreach ( $prices as $price ) {
-				$level_text    = apply_filters( 'give_form_level_text', ! empty( $price['_give_text'] ) ? $price['_give_text'] : give_currency_filter( give_format_amount( $price['_give_amount'], array( 'sanitize' => false ) ), array( 'currency_code' => give_get_currency( $form_id ) ) ), $form_id, $price );
-				$level_classes = apply_filters( 'give_form_level_classes', 'give-donation-level-btn give-btn give-btn-level-' . $price['_give_id']['level_id'] . ' ' . ( give_is_default_level_id( $price ) ? 'give-default-level' : '' ), $form_id, $price );
+			foreach ($prices as $price) {
+				$level_text    = apply_filters('give_form_level_text', ! empty($price['_give_text']) ? $price['_give_text'] : give_currency_filter(give_format_amount($price['_give_amount'], array('sanitize' => false)), array('currency_code' => give_get_currency($form_id))), $form_id, $price);
+				$level_classes = apply_filters('give_form_level_classes', 'give-donation-level-btn give-btn give-btn-level-'.$price['_give_id']['level_id'].' '.(give_is_default_level_id($price) ? 'give-default-level' : ''), $form_id, $price);
 
-				$formatted_amount = give_format_amount( $price['_give_amount'], array(
+				$formatted_amount = give_format_amount($price['_give_amount'], array(
 					'sanitize' => false,
-					'currency' => give_get_currency( $form_id ),
-				) );
+					'currency' => give_get_currency($form_id),
+				));
 
 				$output .= sprintf(
 					'<li><button type="button" data-price-id="%1$s" class="%2$s" value="%3$s" data-default="%4$s">%5$s</button></li>',
 					$price['_give_id']['level_id'],
 					$level_classes,
 					$formatted_amount,
-					array_key_exists( '_give_default', $price ) ? 1 : 0,
+					array_key_exists('_give_default', $price) ? 1 : 0,
 					$level_text
 				);
 			}
 
 			// Custom Amount.
 			if (
-				give_is_setting_enabled( $custom_amount )
-				&& ! empty( $custom_amount_text )
+				give_is_setting_enabled($custom_amount)
+				&& ! empty($custom_amount_text)
 			) {
 
 				$output .= sprintf(
@@ -556,30 +556,30 @@ 
 
 			$output .= '<ul id="give-donation-level-radio-list" class="give-donation-levels-wrap">';
 
-			foreach ( $prices as $price ) {
-				$level_text    = apply_filters( 'give_form_level_text', ! empty( $price['_give_text'] ) ? $price['_give_text'] : give_currency_filter( give_format_amount( $price['_give_amount'], array( 'sanitize' => false ) ), array( 'currency_code' => give_get_currency( $form_id ) ) ), $form_id, $price );
-				$level_classes = apply_filters( 'give_form_level_classes', 'give-radio-input give-radio-input-level give-radio-level-' . $price['_give_id']['level_id'] . ( give_is_default_level_id( $price ) ? ' give-default-level' : '' ), $form_id, $price );
+			foreach ($prices as $price) {
+				$level_text    = apply_filters('give_form_level_text', ! empty($price['_give_text']) ? $price['_give_text'] : give_currency_filter(give_format_amount($price['_give_amount'], array('sanitize' => false)), array('currency_code' => give_get_currency($form_id))), $form_id, $price);
+				$level_classes = apply_filters('give_form_level_classes', 'give-radio-input give-radio-input-level give-radio-level-'.$price['_give_id']['level_id'].(give_is_default_level_id($price) ? ' give-default-level' : ''), $form_id, $price);
 
-				$formatted_amount = give_format_amount( $price['_give_amount'], array(
+				$formatted_amount = give_format_amount($price['_give_amount'], array(
 					'sanitize' => false,
-					'currency' => give_get_currency( $form_id ),
-				) );
+					'currency' => give_get_currency($form_id),
+				));
 
 				$output .= sprintf(
 					'<li><input type="radio" data-price-id="%1$s" class="%2$s" value="%3$s" name="give-radio-donation-level" id="give-radio-level-%1$s" %4$s data-default="%5$s"><label for="give-radio-level-%1$s">%6$s</label></li>',
 					$price['_give_id']['level_id'],
 					$level_classes,
 					$formatted_amount,
-					( give_is_default_level_id( $price ) ? 'checked="checked"' : '' ),
-					array_key_exists( '_give_default', $price ) ? 1 : 0,
+					(give_is_default_level_id($price) ? 'checked="checked"' : ''),
+					array_key_exists('_give_default', $price) ? 1 : 0,
 					$level_text
 				);
 			}
 
 			// Custom Amount.
 			if (
-				give_is_setting_enabled( $custom_amount )
-				&& ! empty( $custom_amount_text )
+				give_is_setting_enabled($custom_amount)
+				&& ! empty($custom_amount_text)
 			) {
 				$output .= sprintf(
 					'<li><input type="radio" data-price-id="custom" class="give-radio-input give-radio-input-level give-radio-level-custom" name="give-radio-donation-level" id="give-radio-level-custom" value="custom"><label for="give-radio-level-custom">%1$s</label></li>',
@@ -593,33 +593,33 @@ 
 
 		case 'dropdown':
 
-			$output .= '<label for="give-donation-level-select-' . $form_id . '" class="give-hidden">' . esc_html__( 'Choose Your Donation Amount', 'give' ) . ':</label>';
-			$output .= '<select id="give-donation-level-select-' . $form_id . '" class="give-select give-select-level give-donation-levels-wrap">';
+			$output .= '<label for="give-donation-level-select-'.$form_id.'" class="give-hidden">'.esc_html__('Choose Your Donation Amount', 'give').':</label>';
+			$output .= '<select id="give-donation-level-select-'.$form_id.'" class="give-select give-select-level give-donation-levels-wrap">';
 
 			// first loop through prices.
-			foreach ( $prices as $price ) {
-				$level_text    = apply_filters( 'give_form_level_text', ! empty( $price['_give_text'] ) ? $price['_give_text'] : give_currency_filter( give_format_amount( $price['_give_amount'], array( 'sanitize' => false ) ), array( 'currency_code' => give_get_currency( $form_id ) ) ), $form_id, $price );
-				$level_classes = apply_filters( 'give_form_level_classes', 'give-donation-level-' . $price['_give_id']['level_id'] . ( give_is_default_level_id( $price ) ? ' give-default-level' : '' ), $form_id,
-				$price );
+			foreach ($prices as $price) {
+				$level_text    = apply_filters('give_form_level_text', ! empty($price['_give_text']) ? $price['_give_text'] : give_currency_filter(give_format_amount($price['_give_amount'], array('sanitize' => false)), array('currency_code' => give_get_currency($form_id))), $form_id, $price);
+				$level_classes = apply_filters('give_form_level_classes', 'give-donation-level-'.$price['_give_id']['level_id'].(give_is_default_level_id($price) ? ' give-default-level' : ''), $form_id,
+				$price);
 
-				$formatted_amount = give_format_amount( $price['_give_amount'], array(
+				$formatted_amount = give_format_amount($price['_give_amount'], array(
 					'sanitize' => false,
-					'currency' => give_get_currency( $form_id ),
-				) );
+					'currency' => give_get_currency($form_id),
+				));
 
 				$output .= sprintf(
 					'<option data-price-id="%1$s" class="%2$s" value="%3$s" %4$s data-default="%5$s">%6$s</option>',
 					$price['_give_id']['level_id'],
 					$level_classes,
 					$formatted_amount,
-					( give_is_default_level_id( $price ) ? 'selected="selected"' : '' ),
-					array_key_exists( '_give_default', $price ) ? 1 : 0,
+					(give_is_default_level_id($price) ? 'selected="selected"' : ''),
+					array_key_exists('_give_default', $price) ? 1 : 0,
 					$level_text
 				);
 			}
 
 			// Custom Amount.
-			if ( give_is_setting_enabled( $custom_amount ) && ! empty( $custom_amount_text ) ) {
+			if (give_is_setting_enabled($custom_amount) && ! empty($custom_amount_text)) {
 				$output .= sprintf(
 					'<option data-price-id="custom" class="give-donation-level-custom" value="custom">%1$s</option>',
 					$custom_amount_text
@@ -631,7 +631,7 @@ 
 			break;
 	}
 
-	echo apply_filters( 'give_form_level_output', $output, $form_id );
+	echo apply_filters('give_form_level_output', $output, $form_id);
 }
 
 /**
@@ -646,27 +646,27 @@ 
  *
  * @return string Checkout button.
  */
-function give_display_checkout_button( $form_id, $args ) {
+function give_display_checkout_button($form_id, $args) {
 
-	$display_option = ( isset( $args['display_style'] ) && ! empty( $args['display_style'] ) )
+	$display_option = (isset($args['display_style']) && ! empty($args['display_style']))
 		? $args['display_style']
-		: give_get_meta( $form_id, '_give_payment_display', true );
+		: give_get_meta($form_id, '_give_payment_display', true);
 
-	if ( 'button' === $display_option ) {
+	if ('button' === $display_option) {
 		$display_option = 'modal';
-	} elseif ( $display_option === 'onpage' ) {
+	} elseif ($display_option === 'onpage') {
 		return '';
 	}
 
-	$display_label_field = give_get_meta( $form_id, '_give_reveal_label', true );
-	$display_label       = ! empty( $args['continue_button_title'] ) ? $args['continue_button_title'] : ( ! empty( $display_label_field ) ? $display_label_field : esc_html__( 'Donate Now', 'give' ) );
+	$display_label_field = give_get_meta($form_id, '_give_reveal_label', true);
+	$display_label       = ! empty($args['continue_button_title']) ? $args['continue_button_title'] : ( ! empty($display_label_field) ? $display_label_field : esc_html__('Donate Now', 'give'));
 
-	$output = '<button type="button" class="give-btn give-btn-' . $display_option . '">' . $display_label . '</button>';
+	$output = '<button type="button" class="give-btn give-btn-'.$display_option.'">'.$display_label.'</button>';
 
-	echo apply_filters( 'give_display_checkout_button', $output );
+	echo apply_filters('give_display_checkout_button', $output);
 }
 
-add_action( 'give_after_donation_levels', 'give_display_checkout_button', 10, 2 );
+add_action('give_after_donation_levels', 'give_display_checkout_button', 10, 2);
 
 /**
  * Shows the User Info fields in the Personal Info box, more fields can be added via the hooks provided.
@@ -679,52 +679,52 @@ 
  *
  * @return void
  */
-function give_user_info_fields( $form_id ) {
+function give_user_info_fields($form_id) {
 
 	// Get user info.
-	$give_user_info = _give_get_prefill_form_field_values( $form_id );
-	$title          = ! empty( $give_user_info['give_title'] ) ? $give_user_info['give_title'] : '';
-	$first_name     = ! empty( $give_user_info['give_first'] ) ? $give_user_info['give_first'] : '';
-	$last_name      = ! empty( $give_user_info['give_last'] ) ? $give_user_info['give_last'] : '';
-	$company_name   = ! empty( $give_user_info['company_name'] ) ? $give_user_info['company_name'] : '';
-	$email          = ! empty( $give_user_info['give_email'] ) ? $give_user_info['give_email'] : '';
-	$title_prefixes = give_get_name_title_prefixes( $form_id );
+	$give_user_info = _give_get_prefill_form_field_values($form_id);
+	$title          = ! empty($give_user_info['give_title']) ? $give_user_info['give_title'] : '';
+	$first_name     = ! empty($give_user_info['give_first']) ? $give_user_info['give_first'] : '';
+	$last_name      = ! empty($give_user_info['give_last']) ? $give_user_info['give_last'] : '';
+	$company_name   = ! empty($give_user_info['company_name']) ? $give_user_info['company_name'] : '';
+	$email          = ! empty($give_user_info['give_email']) ? $give_user_info['give_email'] : '';
+	$title_prefixes = give_get_name_title_prefixes($form_id);
 
 	/**
 	 * Fire before user personal information fields
 	 *
 	 * @since 1.7
 	 */
-	do_action( 'give_donation_form_before_personal_info', $form_id );
+	do_action('give_donation_form_before_personal_info', $form_id);
 
 	$title_prefix_classes = '';
-	if ( give_is_name_title_prefix_enabled( $form_id ) ) {
+	if (give_is_name_title_prefix_enabled($form_id)) {
 		$title_prefix_classes = 'give-title-prefix-wrap';
 	}
 	?>
-	<fieldset id="give_checkout_user_info" class="<?php echo esc_html( $title_prefix_classes ); ?>">
+	<fieldset id="give_checkout_user_info" class="<?php echo esc_html($title_prefix_classes); ?>">
 		<legend>
-			<?php echo esc_html( apply_filters( 'give_checkout_personal_info_text', __( 'Personal Info', 'give' ) ) ); ?>
+			<?php echo esc_html(apply_filters('give_checkout_personal_info_text', __('Personal Info', 'give'))); ?>
 		</legend>
 
-		<?php if ( give_is_name_title_prefix_enabled( $form_id ) && is_array( $title_prefixes ) && count( $title_prefixes ) > 0 ) { ?>
+		<?php if (give_is_name_title_prefix_enabled($form_id) && is_array($title_prefixes) && count($title_prefixes) > 0) { ?>
 			<p id="give-title-wrap" class="form-row form-row-title form-row-responsive">
 				<label class="give-label" for="give-title">
-					<?php esc_attr_e( 'Title', 'give' ); ?>
-					<?php if ( give_field_is_required( 'give_title', $form_id ) ) : ?>
+					<?php esc_attr_e('Title', 'give'); ?>
+					<?php if (give_field_is_required('give_title', $form_id)) : ?>
 						<span class="give-required-indicator">*</span>
 					<?php endif ?>
-					<?php echo Give()->tooltips->render_help( __( 'We will use this to personalize your account experience.', 'give' ) ); ?>
+					<?php echo Give()->tooltips->render_help(__('We will use this to personalize your account experience.', 'give')); ?>
 				</label>
 				<select
 						class="give-input required"
 						type="text"
 						name="give_title"
 						id="give-title"
-					<?php echo( give_field_is_required( 'give_title', $form_id ) ? ' required aria-required="true" ' : '' ); ?>
+					<?php echo(give_field_is_required('give_title', $form_id) ? ' required aria-required="true" ' : ''); ?>
 				>
-					<?php foreach ( $title_prefixes as $key => $value ) { ?>
-						<option value="<?php echo esc_html( $value ); ?>" <?php selected( $value, $title, true ); ?>><?php echo esc_html( $value ); ?></option>
+					<?php foreach ($title_prefixes as $key => $value) { ?>
+						<option value="<?php echo esc_html($value); ?>" <?php selected($value, $title, true); ?>><?php echo esc_html($value); ?></option>
 					<?php } ?>
 				</select>
 			</p>
@@ -732,63 +732,63 @@ 
 
 		<p id="give-first-name-wrap" class="form-row form-row-first form-row-responsive">
 			<label class="give-label" for="give-first">
-				<?php esc_attr_e( 'First Name', 'give' ); ?>
-				<?php if ( give_field_is_required( 'give_first', $form_id ) ) : ?>
+				<?php esc_attr_e('First Name', 'give'); ?>
+				<?php if (give_field_is_required('give_first', $form_id)) : ?>
 					<span class="give-required-indicator">*</span>
 				<?php endif ?>
-				<?php echo Give()->tooltips->render_help( __( 'We will use this to personalize your account experience.', 'give' ) ); ?>
+				<?php echo Give()->tooltips->render_help(__('We will use this to personalize your account experience.', 'give')); ?>
 			</label>
 			<input
 					class="give-input required"
 					type="text"
 					name="give_first"
 					autocomplete="given-name"
-					placeholder="<?php esc_attr_e( 'First Name', 'give' ); ?>"
+					placeholder="<?php esc_attr_e('First Name', 'give'); ?>"
 					id="give-first"
-					value="<?php echo esc_html( $first_name ); ?>"
-				<?php echo( give_field_is_required( 'give_first', $form_id ) ? ' required aria-required="true" ' : '' ); ?>
+					value="<?php echo esc_html($first_name); ?>"
+				<?php echo(give_field_is_required('give_first', $form_id) ? ' required aria-required="true" ' : ''); ?>
 			/>
 		</p>
 
 		<p id="give-last-name-wrap" class="form-row form-row-last form-row-responsive">
 			<label class="give-label" for="give-last">
-				<?php esc_attr_e( 'Last Name', 'give' ); ?>
-				<?php if ( give_field_is_required( 'give_last', $form_id ) ) : ?>
+				<?php esc_attr_e('Last Name', 'give'); ?>
+				<?php if (give_field_is_required('give_last', $form_id)) : ?>
 					<span class="give-required-indicator">*</span>
 				<?php endif ?>
-				<?php echo Give()->tooltips->render_help( __( 'We will use this as well to personalize your account experience.', 'give' ) ); ?>
+				<?php echo Give()->tooltips->render_help(__('We will use this as well to personalize your account experience.', 'give')); ?>
 			</label>
 
 			<input
-					class="give-input<?php echo( give_field_is_required( 'give_last', $form_id ) ? ' required' : '' ); ?>"
+					class="give-input<?php echo(give_field_is_required('give_last', $form_id) ? ' required' : ''); ?>"
 					type="text"
 					name="give_last"
 					autocomplete="family-name"
 					id="give-last"
-					placeholder="<?php esc_attr_e( 'Last Name', 'give' ); ?>"
-					value="<?php echo esc_html( $last_name ); ?>"
-				<?php echo( give_field_is_required( 'give_last', $form_id ) ? ' required aria-required="true" ' : '' ); ?>
+					placeholder="<?php esc_attr_e('Last Name', 'give'); ?>"
+					value="<?php echo esc_html($last_name); ?>"
+				<?php echo(give_field_is_required('give_last', $form_id) ? ' required aria-required="true" ' : ''); ?>
 			/>
 		</p>
 
-		<?php if ( give_is_company_field_enabled( $form_id ) ) : ?>
-			<?php $give_company = give_field_is_required( 'give_company_name', $form_id ); ?>
+		<?php if (give_is_company_field_enabled($form_id)) : ?>
+			<?php $give_company = give_field_is_required('give_company_name', $form_id); ?>
 			<p id="give-company-wrap" class="form-row form-row-wide">
 				<label class="give-label" for="give-company">
-					<?php esc_attr_e( 'Company Name', 'give' ); ?>
-					<?php if ( $give_company ) : ?>
+					<?php esc_attr_e('Company Name', 'give'); ?>
+					<?php if ($give_company) : ?>
 						<span class="give-required-indicator">*</span>
 					<?php endif; ?>
-					<?php echo Give()->tooltips->render_help( __( 'Donate on behalf of Company', 'give' ) ); ?>
+					<?php echo Give()->tooltips->render_help(__('Donate on behalf of Company', 'give')); ?>
 				</label>
 				<input
-					class="give-input<?php echo( $give_company ? ' required' : '' ); ?>"
+					class="give-input<?php echo($give_company ? ' required' : ''); ?>"
 					type="text"
 					name="give_company_name"
-					placeholder="<?php esc_attr_e( 'Company Name', 'give' ); ?>"
+					placeholder="<?php esc_attr_e('Company Name', 'give'); ?>"
 					id="give-company"
-					value="<?php echo esc_html( $company_name ); ?>"
-					<?php echo( $give_company ? ' required aria-required="true" ' : '' ); ?>
+					value="<?php echo esc_html($company_name); ?>"
+					<?php echo($give_company ? ' required aria-required="true" ' : ''); ?>
 				/>
 			</p>
 		<?php endif ?>
@@ -799,31 +799,31 @@ 
 		 *
 		 * @since 1.7
 		 */
-		do_action( 'give_donation_form_before_email', $form_id );
+		do_action('give_donation_form_before_email', $form_id);
 		?>
 		<p id="give-email-wrap" class="form-row form-row-wide">
 			<label class="give-label" for="give-email">
-				<?php esc_attr_e( 'Email Address', 'give' ); ?>
-				<?php if ( give_field_is_required( 'give_email', $form_id ) ) { ?>
+				<?php esc_attr_e('Email Address', 'give'); ?>
+				<?php if (give_field_is_required('give_email', $form_id)) { ?>
 					<span class="give-required-indicator">*</span>
 				<?php } ?>
-				<?php echo Give()->tooltips->render_help( __( 'We will send the donation receipt to this address.', 'give' ) ); ?>
+				<?php echo Give()->tooltips->render_help(__('We will send the donation receipt to this address.', 'give')); ?>
 			</label>
 			<input
 					class="give-input required"
 					type="email"
 					name="give_email"
 					autocomplete="email"
-					placeholder="<?php esc_attr_e( 'Email Address', 'give' ); ?>"
+					placeholder="<?php esc_attr_e('Email Address', 'give'); ?>"
 					id="give-email"
-					value="<?php echo esc_html( $email ); ?>"
-				<?php echo( give_field_is_required( 'give_email', $form_id ) ? ' required aria-required="true" ' : '' ); ?>
+					value="<?php echo esc_html($email); ?>"
+				<?php echo(give_field_is_required('give_email', $form_id) ? ' required aria-required="true" ' : ''); ?>
 			/>
 
 		</p>
 
-		<?php if ( give_is_anonymous_donation_field_enabled( $form_id ) ) : ?>
-			<?php $is_anonymous_donation = isset( $_POST['give_anonymous_donation'] ) ? absint( $_POST['give_anonymous_donation'] ) : 0; ?>
+		<?php if (give_is_anonymous_donation_field_enabled($form_id)) : ?>
+			<?php $is_anonymous_donation = isset($_POST['give_anonymous_donation']) ? absint($_POST['give_anonymous_donation']) : 0; ?>
 			<p id="give-anonymous-donation-wrap" class="form-row form-row-wide">
 				<label class="give-label" for="give-anonymous-donation">
 				<input
@@ -832,35 +832,35 @@ 
 						name="give_anonymous_donation"
 						id="give-anonymous-donation"
 						value="1"
-					<?php echo( give_field_is_required( 'give_anonymous_donation', $form_id ) ? ' required aria-required="true" ' : '' ); ?>
-					<?php checked( 1, $is_anonymous_donation ); ?>
+					<?php echo(give_field_is_required('give_anonymous_donation', $form_id) ? ' required aria-required="true" ' : ''); ?>
+					<?php checked(1, $is_anonymous_donation); ?>
 				>
-					<?php _e( 'Make this an anonymous donation', 'give' ); ?>
-					<?php if ( give_field_is_required( 'give_comment', $form_id ) ) { ?>
+					<?php _e('Make this an anonymous donation', 'give'); ?>
+					<?php if (give_field_is_required('give_comment', $form_id)) { ?>
 						<span class="give-required-indicator">*</span>
 					<?php } ?>
-					<?php echo Give()->tooltips->render_help( esc_html__( 'Would you like to prevent this donation from being displayed publicy?', 'give' ) ); ?>
+					<?php echo Give()->tooltips->render_help(esc_html__('Would you like to prevent this donation from being displayed publicy?', 'give')); ?>
 				</label>
 			</p>
 		<?php endif; ?>
 
-		<?php if ( give_is_donor_comment_field_enabled( $form_id ) ) : ?>
+		<?php if (give_is_donor_comment_field_enabled($form_id)) : ?>
 			<p id="give-comment-wrap" class="form-row form-row-wide">
 				<label class="give-label" for="give-comment">
-					<?php _e( 'Comment', 'give' ); ?>
-					<?php if ( give_field_is_required( 'give_comment', $form_id ) ) { ?>
+					<?php _e('Comment', 'give'); ?>
+					<?php if (give_field_is_required('give_comment', $form_id)) { ?>
 						<span class="give-required-indicator">*</span>
 					<?php } ?>
-					<?php echo Give()->tooltips->render_help( __( 'Would you like to add a comment to this donation?', 'give' ) ); ?>
+					<?php echo Give()->tooltips->render_help(__('Would you like to add a comment to this donation?', 'give')); ?>
 				</label>
 
 				<textarea
 					class="give-input required"
 					name="give_comment"
-					placeholder="<?php _e( 'Leave a comment', 'give' ); ?>"
+					placeholder="<?php _e('Leave a comment', 'give'); ?>"
 					id="give-comment"
-					<?php echo( give_field_is_required( 'give_comment', $form_id ) ? ' required aria-required="true" ' : '' ); ?>
-				><?php echo isset( $_POST['give_comment'] ) ? give_clean( $_POST['give_comment'] ) : ''; ?></textarea>
+					<?php echo(give_field_is_required('give_comment', $form_id) ? ' required aria-required="true" ' : ''); ?>
+				><?php echo isset($_POST['give_comment']) ? give_clean($_POST['give_comment']) : ''; ?></textarea>
 
 			</p>
 		<?php endif; ?>
@@ -870,14 +870,14 @@ 
 		 *
 		 * @since 1.7
 		 */
-		do_action( 'give_donation_form_after_email', $form_id );
+		do_action('give_donation_form_after_email', $form_id);
 
 		/**
 		 * Fire after personal email field
 		 *
 		 * @since 1.7
 		 */
-		do_action( 'give_donation_form_user_info', $form_id );
+		do_action('give_donation_form_user_info', $form_id);
 		?>
 	</fieldset>
 	<?php
@@ -886,11 +886,11 @@ 
 	 *
 	 * @since 1.7
 	 */
-	do_action( 'give_donation_form_after_personal_info', $form_id );
+	do_action('give_donation_form_after_personal_info', $form_id);
 }
 
-add_action( 'give_donation_form_after_user_info', 'give_user_info_fields' );
-add_action( 'give_register_fields_before', 'give_user_info_fields' );
+add_action('give_donation_form_after_user_info', 'give_user_info_fields');
+add_action('give_register_fields_before', 'give_user_info_fields');
 
 /**
  * Renders the credit card info form.
@@ -901,7 +901,7 @@ 
  *
  * @return void
  */
-function give_get_cc_form( $form_id ) {
+function give_get_cc_form($form_id) {
 
 	ob_start();
 
@@ -912,50 +912,50 @@ 
 	 *
 	 * @param int $form_id The form ID.
 	 */
-	do_action( 'give_before_cc_fields', $form_id );
+	do_action('give_before_cc_fields', $form_id);
 	?>
 	<fieldset id="give_cc_fields-<?php echo $form_id ?>" class="give-do-validate">
-		<legend><?php echo apply_filters( 'give_credit_card_fieldset_heading', esc_html__( 'Credit Card Info', 'give' ) ); ?></legend>
-		<?php if ( is_ssl() ) : ?>
+		<legend><?php echo apply_filters('give_credit_card_fieldset_heading', esc_html__('Credit Card Info', 'give')); ?></legend>
+		<?php if (is_ssl()) : ?>
 			<div id="give_secure_site_wrapper-<?php echo $form_id ?>">
 				<span class="give-icon padlock"></span>
-				<span><?php _e( 'This is a secure SSL encrypted payment.', 'give' ); ?></span>
+				<span><?php _e('This is a secure SSL encrypted payment.', 'give'); ?></span>
 			</div>
 		<?php endif; ?>
 		<p id="give-card-number-wrap-<?php echo $form_id ?>" class="form-row form-row-two-thirds form-row-responsive">
 			<label for="card_number-<?php echo $form_id ?>" class="give-label">
-				<?php _e( 'Card Number', 'give' ); ?>
+				<?php _e('Card Number', 'give'); ?>
 				<span class="give-required-indicator">*</span>
-				<?php echo Give()->tooltips->render_help( __( 'The (typically) 16 digits on the front of your credit card.', 'give' ) ); ?>
+				<?php echo Give()->tooltips->render_help(__('The (typically) 16 digits on the front of your credit card.', 'give')); ?>
 				<span class="card-type"></span>
 			</label>
 
 			<input type="tel" autocomplete="off" name="card_number" id="card_number-<?php echo $form_id ?>"
-				   class="card-number give-input required" placeholder="<?php _e( 'Card number', 'give' ); ?>"
+				   class="card-number give-input required" placeholder="<?php _e('Card number', 'give'); ?>"
 				   required aria-required="true"/>
 		</p>
 
 		<p id="give-card-cvc-wrap-<?php echo $form_id ?>" class="form-row form-row-one-third form-row-responsive">
 			<label for="card_cvc-<?php echo $form_id ?>" class="give-label">
-				<?php _e( 'CVC', 'give' ); ?>
+				<?php _e('CVC', 'give'); ?>
 				<span class="give-required-indicator">*</span>
-				<?php echo Give()->tooltips->render_help( __( 'The 3 digit (back) or 4 digit (front) value on your card.', 'give' ) ); ?>
+				<?php echo Give()->tooltips->render_help(__('The 3 digit (back) or 4 digit (front) value on your card.', 'give')); ?>
 			</label>
 
 			<input type="tel" size="4" autocomplete="off" name="card_cvc" id="card_cvc-<?php echo $form_id ?>"
-				   class="card-cvc give-input required" placeholder="<?php _e( 'Security code', 'give' ); ?>"
+				   class="card-cvc give-input required" placeholder="<?php _e('Security code', 'give'); ?>"
 				   required aria-required="true"/>
 		</p>
 
 		<p id="give-card-name-wrap-<?php echo $form_id ?>" class="form-row form-row-two-thirds form-row-responsive">
 			<label for="card_name-<?php echo $form_id ?>" class="give-label">
-				<?php _e( 'Name on the Card', 'give' ); ?>
+				<?php _e('Name on the Card', 'give'); ?>
 				<span class="give-required-indicator">*</span>
-				<?php echo Give()->tooltips->render_help( __( 'The name printed on the front of your credit card.', 'give' ) ); ?>
+				<?php echo Give()->tooltips->render_help(__('The name printed on the front of your credit card.', 'give')); ?>
 			</label>
 
 			<input type="text" autocomplete="off" name="card_name" id="card_name-<?php echo $form_id ?>"
-				   class="card-name give-input required" placeholder="<?php esc_attr_e( 'Card name', 'give' ); ?>"
+				   class="card-name give-input required" placeholder="<?php esc_attr_e('Card name', 'give'); ?>"
 				   required aria-required="true"/>
 		</p>
 		<?php
@@ -966,19 +966,19 @@ 
 		 *
 		 * @param int $form_id The form ID.
 		 */
-		do_action( 'give_before_cc_expiration' );
+		do_action('give_before_cc_expiration');
 		?>
 		<p class="card-expiration form-row form-row-one-third form-row-responsive">
 			<label for="card_expiry-<?php echo $form_id ?>" class="give-label">
-				<?php _e( 'Expiration', 'give' ); ?>
+				<?php _e('Expiration', 'give'); ?>
 				<span class="give-required-indicator">*</span>
-				<?php echo Give()->tooltips->render_help( __( 'The date your credit card expires, typically on the front of the card.', 'give' ) ); ?>
+				<?php echo Give()->tooltips->render_help(__('The date your credit card expires, typically on the front of the card.', 'give')); ?>
 			</label>
 
 			<input type="hidden" id="card_exp_month-<?php echo $form_id ?>" name="card_exp_month" class="card-expiry-month"/>
 			<input type="hidden" id="card_exp_year-<?php echo $form_id ?>" name="card_exp_year" class="card-expiry-year"/>
 
-			<input type="tel" autocomplete="off" name="card_expiry" id="card_expiry-<?php echo $form_id ?>" class="card-expiry give-input required" placeholder="<?php esc_attr_e( 'MM / YY', 'give' ); ?>" required aria-required="true"/>
+			<input type="tel" autocomplete="off" name="card_expiry" id="card_expiry-<?php echo $form_id ?>" class="card-expiry give-input required" placeholder="<?php esc_attr_e('MM / YY', 'give'); ?>" required aria-required="true"/>
 		</p>
 		<?php
 		/**
@@ -988,7 +988,7 @@ 
 		 *
 		 * @param int $form_id The form ID.
 		 */
-		do_action( 'give_after_cc_expiration', $form_id );
+		do_action('give_after_cc_expiration', $form_id);
 		?>
 	</fieldset>
 	<?php
@@ -999,12 +999,12 @@ 
 	 *
 	 * @param int $form_id The form ID.
 	 */
-	do_action( 'give_after_cc_fields', $form_id );
+	do_action('give_after_cc_fields', $form_id);
 
 	echo ob_get_clean();
 }
 
-add_action( 'give_cc_form', 'give_get_cc_form' );
+add_action('give_cc_form', 'give_get_cc_form');
 
 /**
  * Outputs the default credit card address fields.
@@ -1015,20 +1015,20 @@ 
  *
  * @return void
  */
-function give_default_cc_address_fields( $form_id ) {
+function give_default_cc_address_fields($form_id) {
 	// Get user info.
-	$give_user_info = _give_get_prefill_form_field_values( $form_id );
+	$give_user_info = _give_get_prefill_form_field_values($form_id);
 
 	$logged_in = is_user_logged_in();
 
-	if ( $logged_in ) {
-		$user_address = give_get_donor_address( get_current_user_id() );
+	if ($logged_in) {
+		$user_address = give_get_donor_address(get_current_user_id());
 	}
 
 	ob_start();
 	?>
 	<fieldset id="give_cc_address" class="cc-address">
-		<legend><?php echo apply_filters( 'give_billing_details_fieldset_heading', esc_html__( 'Billing Details', 'give' ) ); ?></legend>
+		<legend><?php echo apply_filters('give_billing_details_fieldset_heading', esc_html__('Billing Details', 'give')); ?></legend>
 		<?php
 		/**
 		 * Fires while rendering credit card billing form, before address fields.
@@ -1037,36 +1037,36 @@ 
 		 *
 		 * @param int $form_id The form ID.
 		 */
-		do_action( 'give_cc_billing_top' );
+		do_action('give_cc_billing_top');
 
 		// For Country.
 		$selected_country = give_get_country();
-		if ( ! empty( $give_user_info['billing_country'] ) && '*' !== $give_user_info['billing_country'] ) {
+		if ( ! empty($give_user_info['billing_country']) && '*' !== $give_user_info['billing_country']) {
 			$selected_country = $give_user_info['billing_country'];
 		}
 		$countries = give_get_country_list();
 
 		// For state.
 		$selected_state = '';
-		if ( $selected_country === give_get_country() ) {
+		if ($selected_country === give_get_country()) {
 			// Get default selected state by admin.
 			$selected_state = give_get_state();
 		}
 		// Get the last payment made by user states.
-		if ( ! empty( $give_user_info['card_state'] ) && '*' !== $give_user_info['card_state'] ) {
+		if ( ! empty($give_user_info['card_state']) && '*' !== $give_user_info['card_state']) {
 			$selected_state = $give_user_info['card_state'];
 		}
 		// Get the country code.
-		if ( ! empty( $give_user_info['billing_country'] ) && '*' !== $give_user_info['billing_country'] ) {
+		if ( ! empty($give_user_info['billing_country']) && '*' !== $give_user_info['billing_country']) {
 			$selected_country = $give_user_info['billing_country'];
 		}
-		$label        = __( 'State', 'give' );
+		$label        = __('State', 'give');
 		$states_label = give_get_states_label();
 		// Check if $country code exists in the array key for states label.
-		if ( array_key_exists( $selected_country, $states_label ) ) {
-			$label = $states_label[ $selected_country ];
+		if (array_key_exists($selected_country, $states_label)) {
+			$label = $states_label[$selected_country];
 		}
-		$states = give_get_states( $selected_country );
+		$states = give_get_states($selected_country);
 		// Get the country list that do not have any states init.
 		$no_states_country = give_no_states_country_list();
 		// Get the country list that does not require states.
@@ -1074,24 +1074,24 @@ 
 		?>
 	    <p id="give-card-country-wrap" class="form-row form-row-wide">
 		    <label for="billing_country" class="give-label">
-			    <?php esc_html_e( 'Country', 'give' ); ?>
-			    <?php if ( give_field_is_required( 'billing_country', $form_id ) ) : ?>
+			    <?php esc_html_e('Country', 'give'); ?>
+			    <?php if (give_field_is_required('billing_country', $form_id)) : ?>
 				    <span class="give-required-indicator">*</span>
 			    <?php endif; ?>
 			    <span class="give-tooltip give-icon give-icon-question"
-			          data-tooltip="<?php esc_attr_e( 'The country for your billing address.', 'give' ); ?>"></span>
+			          data-tooltip="<?php esc_attr_e('The country for your billing address.', 'give'); ?>"></span>
 		    </label>
 
 		    <select
 				    name="billing_country"
 				    autocomplete="country-name"
 				    id="billing_country"
-				    class="billing-country billing_country give-select<?php echo( give_field_is_required( 'billing_country', $form_id ) ? ' required' : '' ); ?>"
-			    <?php echo( give_field_is_required( 'billing_country', $form_id ) ? ' required aria-required="true" ' : '' ); ?>
+				    class="billing-country billing_country give-select<?php echo(give_field_is_required('billing_country', $form_id) ? ' required' : ''); ?>"
+			    <?php echo(give_field_is_required('billing_country', $form_id) ? ' required aria-required="true" ' : ''); ?>
 		    >
 			    <?php
-			    foreach ( $countries as $country_code => $country ) {
-				    echo '<option value="' . esc_attr( $country_code ) . '"' . selected( $country_code, $selected_country, false ) . '>' . $country . '</option>';
+			    foreach ($countries as $country_code => $country) {
+				    echo '<option value="'.esc_attr($country_code).'"'.selected($country_code, $selected_country, false).'>'.$country.'</option>';
 			    }
 			    ?>
 		    </select>
@@ -1099,12 +1099,12 @@ 
 
 		<p id="give-card-address-wrap" class="form-row form-row-wide">
 			<label for="card_address" class="give-label">
-				<?php _e( 'Address 1', 'give' ); ?>
+				<?php _e('Address 1', 'give'); ?>
 				<?php
-				if ( give_field_is_required( 'card_address', $form_id ) ) : ?>
+				if (give_field_is_required('card_address', $form_id)) : ?>
 					<span class="give-required-indicator">*</span>
 				<?php endif; ?>
-				<?php echo Give()->tooltips->render_help( __( 'The primary billing address for your credit card.', 'give' ) ); ?>
+				<?php echo Give()->tooltips->render_help(__('The primary billing address for your credit card.', 'give')); ?>
 			</label>
 
 			<input
@@ -1112,20 +1112,20 @@ 
 					id="card_address"
 					name="card_address"
 					autocomplete="address-line1"
-					class="card-address give-input<?php echo( give_field_is_required( 'card_address', $form_id ) ? ' required' : '' ); ?>"
-					placeholder="<?php _e( 'Address line 1', 'give' ); ?>"
-					value="<?php echo isset( $give_user_info['card_address'] ) ? $give_user_info['card_address'] : ''; ?>"
-				<?php echo( give_field_is_required( 'card_address', $form_id ) ? '  required aria-required="true" ' : '' ); ?>
+					class="card-address give-input<?php echo(give_field_is_required('card_address', $form_id) ? ' required' : ''); ?>"
+					placeholder="<?php _e('Address line 1', 'give'); ?>"
+					value="<?php echo isset($give_user_info['card_address']) ? $give_user_info['card_address'] : ''; ?>"
+				<?php echo(give_field_is_required('card_address', $form_id) ? '  required aria-required="true" ' : ''); ?>
 			/>
 		</p>
 
 		<p id="give-card-address-2-wrap" class="form-row form-row-wide">
 			<label for="card_address_2" class="give-label">
-				<?php _e( 'Address 2', 'give' ); ?>
-				<?php if ( give_field_is_required( 'card_address_2', $form_id ) ) : ?>
+				<?php _e('Address 2', 'give'); ?>
+				<?php if (give_field_is_required('card_address_2', $form_id)) : ?>
 					<span class="give-required-indicator">*</span>
 				<?php endif; ?>
-				<?php echo Give()->tooltips->render_help( __( '(optional) The suite, apartment number, post office box (etc) associated with your billing address.', 'give' ) ); ?>
+				<?php echo Give()->tooltips->render_help(__('(optional) The suite, apartment number, post office box (etc) associated with your billing address.', 'give')); ?>
 			</label>
 
 			<input
@@ -1133,56 +1133,56 @@ 
 					id="card_address_2"
 					name="card_address_2"
 					autocomplete="address-line2"
-					class="card-address-2 give-input<?php echo( give_field_is_required( 'card_address_2', $form_id ) ? ' required' : '' ); ?>"
-					placeholder="<?php _e( 'Address line 2', 'give' ); ?>"
-					value="<?php echo isset( $give_user_info['card_address_2'] ) ? $give_user_info['card_address_2'] : ''; ?>"
-				<?php echo( give_field_is_required( 'card_address_2', $form_id ) ? ' required aria-required="true" ' : '' ); ?>
+					class="card-address-2 give-input<?php echo(give_field_is_required('card_address_2', $form_id) ? ' required' : ''); ?>"
+					placeholder="<?php _e('Address line 2', 'give'); ?>"
+					value="<?php echo isset($give_user_info['card_address_2']) ? $give_user_info['card_address_2'] : ''; ?>"
+				<?php echo(give_field_is_required('card_address_2', $form_id) ? ' required aria-required="true" ' : ''); ?>
 			/>
 		</p>
 
 		<p id="give-card-city-wrap" class="form-row form-row-wide">
 			<label for="card_city" class="give-label">
-				<?php _e( 'City', 'give' ); ?>
-				<?php if ( give_field_is_required( 'card_city', $form_id ) ) : ?>
+				<?php _e('City', 'give'); ?>
+				<?php if (give_field_is_required('card_city', $form_id)) : ?>
 					<span class="give-required-indicator">*</span>
 				<?php endif; ?>
-				<?php echo Give()->tooltips->render_help( __( 'The city for your billing address.', 'give' ) ); ?>
+				<?php echo Give()->tooltips->render_help(__('The city for your billing address.', 'give')); ?>
 			</label>
 			<input
 					type="text"
 					id="card_city"
 					name="card_city"
 					autocomplete="address-level3"
-					class="card-city give-input<?php echo( give_field_is_required( 'card_city', $form_id ) ? ' required' : '' ); ?>"
-					placeholder="<?php _e( 'City', 'give' ); ?>"
-					value="<?php echo isset( $give_user_info['card_city'] ) ? $give_user_info['card_city'] : ''; ?>"
-				<?php echo( give_field_is_required( 'card_city', $form_id ) ? ' required aria-required="true" ' : '' ); ?>
+					class="card-city give-input<?php echo(give_field_is_required('card_city', $form_id) ? ' required' : ''); ?>"
+					placeholder="<?php _e('City', 'give'); ?>"
+					value="<?php echo isset($give_user_info['card_city']) ? $give_user_info['card_city'] : ''; ?>"
+				<?php echo(give_field_is_required('card_city', $form_id) ? ' required aria-required="true" ' : ''); ?>
 			/>
 		</p>
 
 	    <p id="give-card-state-wrap"
-	       class="form-row form-row-first form-row-responsive <?php echo ( ! empty( $selected_country ) && array_key_exists( $selected_country, $no_states_country ) ) ? 'give-hidden' : ''; ?> ">
+	       class="form-row form-row-first form-row-responsive <?php echo ( ! empty($selected_country) && array_key_exists($selected_country, $no_states_country)) ? 'give-hidden' : ''; ?> ">
 		    <label for="card_state" class="give-label">
 			    <span class="state-label-text"><?php echo $label; ?></span>
-			    <?php if ( give_field_is_required( 'card_state', $form_id ) ) :
+			    <?php if (give_field_is_required('card_state', $form_id)) :
 				    ?>
-				    <span class="give-required-indicator <?php echo( array_key_exists( $selected_country, $states_not_required_country_list ) ? 'give-hidden' : '' ) ?> ">*</span>
+				    <span class="give-required-indicator <?php echo(array_key_exists($selected_country, $states_not_required_country_list) ? 'give-hidden' : '') ?> ">*</span>
 			    <?php endif; ?>
 			    <span class="give-tooltip give-icon give-icon-question"
-			          data-tooltip="<?php esc_attr_e( 'The state, province, or county for your billing address.', 'give' ); ?>"></span>
+			          data-tooltip="<?php esc_attr_e('The state, province, or county for your billing address.', 'give'); ?>"></span>
 		    </label>
 		    <?php
 
-		    if ( ! empty( $states ) ) : ?>
+		    if ( ! empty($states)) : ?>
 			    <select
 					    name="card_state"
 					    autocomplete="address-level4"
 					    id="card_state"
-					    class="card_state give-select<?php echo( give_field_is_required( 'card_state', $form_id ) ? ' required' : '' ); ?>"
-				    <?php echo( give_field_is_required( 'card_state', $form_id ) ? ' required aria-required="true" ' : '' ); ?>>
+					    class="card_state give-select<?php echo(give_field_is_required('card_state', $form_id) ? ' required' : ''); ?>"
+				    <?php echo(give_field_is_required('card_state', $form_id) ? ' required aria-required="true" ' : ''); ?>>
 				    <?php
-				    foreach ( $states as $state_code => $state ) {
-					    echo '<option value="' . $state_code . '"' . selected( $state_code, $selected_state, false ) . '>' . $state . '</option>';
+				    foreach ($states as $state_code => $state) {
+					    echo '<option value="'.$state_code.'"'.selected($state_code, $selected_state, false).'>'.$state.'</option>';
 				    }
 				    ?>
 			    </select>
@@ -1194,11 +1194,11 @@ 
 
 		<p id="give-card-zip-wrap" class="form-row form-row-last form-row-responsive">
 			<label for="card_zip" class="give-label">
-				<?php _e( 'Zip / Postal Code', 'give' ); ?>
-				<?php if ( give_field_is_required( 'card_zip', $form_id ) ) : ?>
+				<?php _e('Zip / Postal Code', 'give'); ?>
+				<?php if (give_field_is_required('card_zip', $form_id)) : ?>
 					<span class="give-required-indicator">*</span>
 				<?php endif; ?>
-				<?php echo Give()->tooltips->render_help( __( 'The ZIP Code or postal code for your billing address.', 'give' ) ); ?>
+				<?php echo Give()->tooltips->render_help(__('The ZIP Code or postal code for your billing address.', 'give')); ?>
 			</label>
 
 			<input
@@ -1207,10 +1207,10 @@ 
 					id="card_zip"
 					name="card_zip"
 					autocomplete="postal-code"
-					class="card-zip give-input<?php echo( give_field_is_required( 'card_zip', $form_id ) ? ' required' : '' ); ?>"
-					placeholder="<?php _e( 'Zip / Postal Code', 'give' ); ?>"
-					value="<?php echo isset( $give_user_info['card_zip'] ) ? $give_user_info['card_zip'] : ''; ?>"
-				<?php echo( give_field_is_required( 'card_zip', $form_id ) ? ' required aria-required="true" ' : '' ); ?>
+					class="card-zip give-input<?php echo(give_field_is_required('card_zip', $form_id) ? ' required' : ''); ?>"
+					placeholder="<?php _e('Zip / Postal Code', 'give'); ?>"
+					value="<?php echo isset($give_user_info['card_zip']) ? $give_user_info['card_zip'] : ''; ?>"
+				<?php echo(give_field_is_required('card_zip', $form_id) ? ' required aria-required="true" ' : ''); ?>
 			/>
 		</p>
 		<?php
@@ -1221,14 +1221,14 @@ 
 		 *
 		 * @param int $form_id The form ID.
 		 */
-		do_action( 'give_cc_billing_bottom' );
+		do_action('give_cc_billing_bottom');
 		?>
 	</fieldset>
 	<?php
 	echo ob_get_clean();
 }
 
-add_action( 'give_after_cc_fields', 'give_default_cc_address_fields' );
+add_action('give_after_cc_fields', 'give_default_cc_address_fields');
 
 
 /**
@@ -1241,15 +1241,15 @@ 
  *
  * @return string
  */
-function give_get_register_fields( $form_id ) {
+function give_get_register_fields($form_id) {
 
 	global $user_ID;
 
-	if ( is_user_logged_in() ) {
-		$user_data = get_userdata( $user_ID );
+	if (is_user_logged_in()) {
+		$user_data = get_userdata($user_ID);
 	}
 
-	$show_register_form = give_show_login_register_option( $form_id );
+	$show_register_form = give_show_login_register_option($form_id);
 
 	ob_start(); ?>
 	<fieldset id="give-register-fields-<?php echo $form_id; ?>">
@@ -1262,7 +1262,7 @@ 
 		 *
 		 * @param int $form_id The form ID.
 		 */
-		do_action( 'give_register_fields_before', $form_id );
+		do_action('give_register_fields_before', $form_id);
 		?>
 
 		<fieldset id="give-register-account-fields-<?php echo $form_id; ?>">
@@ -1274,24 +1274,24 @@ 
 			 *
 			 * @param int $form_id The form ID.
 			 */
-			do_action( 'give_register_account_fields_before', $form_id );
+			do_action('give_register_account_fields_before', $form_id);
 
-			$class = ( 'registration' === $show_register_form) ? 'form-row-wide' : 'form-row-first';
+			$class = ('registration' === $show_register_form) ? 'form-row-wide' : 'form-row-first';
 			?>
-			<div id="give-create-account-wrap-<?php echo $form_id; ?>" class="form-row <?php echo esc_attr( $class ); ?> form-row-responsive">
+			<div id="give-create-account-wrap-<?php echo $form_id; ?>" class="form-row <?php echo esc_attr($class); ?> form-row-responsive">
 				<label for="give-create-account-<?php echo $form_id; ?>">
 					<?php
 					// Add attributes to checkbox, if Guest Checkout is disabled.
-					$is_guest_checkout = give_get_meta( $form_id, '_give_logged_in_only', true );
-					$id                = 'give-create-account-' . $form_id;
-					if ( ! give_is_setting_enabled( $is_guest_checkout ) ) {
+					$is_guest_checkout = give_get_meta($form_id, '_give_logged_in_only', true);
+					$id                = 'give-create-account-'.$form_id;
+					if ( ! give_is_setting_enabled($is_guest_checkout)) {
 						echo Give()->tooltips->render(
 							array(
 								'tag_content' => sprintf(
 									'<input type="checkbox" name="give_create_account" value="on" id="%s" class="give-input give-disabled" checked />',
 									$id
 								),
-								'label'       => __( 'Registration is required to donate.', 'give' ),
+								'label'       => __('Registration is required to donate.', 'give'),
 							) );
 					} else {
 						?>
@@ -1299,22 +1299,22 @@ 
 						<?php
 					}
 
-					_e( 'Create an account', 'give' );
-					echo Give()->tooltips->render_help( __( 'Create an account on the site to see and manage donation history.', 'give' ) );
+					_e('Create an account', 'give');
+					echo Give()->tooltips->render_help(__('Create an account on the site to see and manage donation history.', 'give'));
 					echo str_replace(
 						'/>',
-						'data-time="' . time() . '" data-nonce-life="' . give_get_nonce_life() . '"/>',
-						give_get_nonce_field( "give_form_create_user_nonce_{$form_id}", 'give-form-user-register-hash', false )
+						'data-time="'.time().'" data-nonce-life="'.give_get_nonce_life().'"/>',
+						give_get_nonce_field("give_form_create_user_nonce_{$form_id}", 'give-form-user-register-hash', false)
 					);
 					?>
 				</label>
 			</div>
 
-			<?php if ( 'both' === $show_register_form ) { ?>
+			<?php if ('both' === $show_register_form) { ?>
 				<div class="give-login-account-wrap form-row form-row-last form-row-responsive">
-					<p class="give-login-message"><?php esc_html_e( 'Already have an account?', 'give' ); ?>&nbsp;
-						<a href="<?php echo esc_url( add_query_arg( 'login', 1 ) ); ?>" class="give-checkout-login"
-						   data-action="give_checkout_login"><?php esc_html_e( 'Login', 'give' ); ?></a>
+					<p class="give-login-message"><?php esc_html_e('Already have an account?', 'give'); ?>&nbsp;
+						<a href="<?php echo esc_url(add_query_arg('login', 1)); ?>" class="give-checkout-login"
+						   data-action="give_checkout_login"><?php esc_html_e('Login', 'give'); ?></a>
 					</p>
 					<p class="give-loading-text">
 						<span class="give-loading-animation"></span>
@@ -1330,7 +1330,7 @@ 
 			 *
 			 * @param int $form_id The form ID.
 			 */
-			do_action( 'give_register_account_fields_after', $form_id );
+			do_action('give_register_account_fields_after', $form_id);
 			?>
 		</fieldset>
 
@@ -1342,7 +1342,7 @@ 
 		 *
 		 * @param int $form_id The form ID.
 		 */
-		do_action( 'give_register_fields_after', $form_id );
+		do_action('give_register_fields_after', $form_id);
 		?>
 
 		<input type="hidden" name="give-purchase-var" value="needs-to-register"/>
@@ -1353,7 +1353,7 @@ 
 		 *
 		 * @since 1.7
 		 */
-		do_action( 'give_donation_form_user_info', $form_id );
+		do_action('give_donation_form_user_info', $form_id);
 		?>
 
 	</fieldset>
@@ -1361,7 +1361,7 @@ 
 	echo ob_get_clean();
 }
 
-add_action( 'give_donation_form_register_fields', 'give_get_register_fields' );
+add_action('give_donation_form_register_fields', 'give_get_register_fields');
 
 /**
  * Gets the login fields for the login form on the checkout. This function hooks
@@ -1374,28 +1374,28 @@ 
  *
  * @return string
  */
-function give_get_login_fields( $form_id ) {
+function give_get_login_fields($form_id) {
 
-	$form_id            = isset( $_POST['form_id'] ) ? $_POST['form_id'] : $form_id;
-	$show_register_form = give_show_login_register_option( $form_id );
+	$form_id            = isset($_POST['form_id']) ? $_POST['form_id'] : $form_id;
+	$show_register_form = give_show_login_register_option($form_id);
 
 	ob_start();
 	?>
 	<fieldset id="give-login-fields-<?php echo $form_id; ?>">
-		<legend><?php echo apply_filters( 'give_account_login_fieldset_heading', __( 'Login to Your Account', 'give' ) );
-			if ( ! give_logged_in_only( $form_id ) ) {
-				echo ' <span class="sub-text">' . __( '(optional)', 'give' ) . '</span>';
+		<legend><?php echo apply_filters('give_account_login_fieldset_heading', __('Login to Your Account', 'give'));
+			if ( ! give_logged_in_only($form_id)) {
+				echo ' <span class="sub-text">'.__('(optional)', 'give').'</span>';
 			} ?>
 		</legend>
-		<?php if ( $show_register_form == 'both' ) { ?>
+		<?php if ($show_register_form == 'both') { ?>
 			<p class="give-new-account-link">
-				<?php _e( 'Don\'t have an account?', 'give' ); ?>&nbsp;
-				<a href="<?php echo remove_query_arg( 'login' ); ?>" class="give-checkout-register-cancel"
+				<?php _e('Don\'t have an account?', 'give'); ?>&nbsp;
+				<a href="<?php echo remove_query_arg('login'); ?>" class="give-checkout-register-cancel"
 				   data-action="give_checkout_register">
-					<?php if ( give_logged_in_only( $form_id ) ) {
-					    _e( 'Register as a part of your donation &raquo;', 'give' );
+					<?php if (give_logged_in_only($form_id)) {
+					    _e('Register as a part of your donation &raquo;', 'give');
                     } else {
-						 _e( 'Register or donate as a guest &raquo;', 'give' );
+						 _e('Register or donate as a guest &raquo;', 'give');
 					} ?>
 				</a>
 			</p>
@@ -1416,35 +1416,35 @@ 
 		<div class="give-user-login-fields-container">
 			<div id="give-user-login-wrap-<?php echo $form_id; ?>" class="form-row form-row-first form-row-responsive">
 				<label class="give-label" for="give-user-login-<?php echo $form_id; ?>">
-					<?php _e( 'Username', 'give' ); ?>
-					<?php if ( give_logged_in_only( $form_id ) ) { ?>
+					<?php _e('Username', 'give'); ?>
+					<?php if (give_logged_in_only($form_id)) { ?>
 						<span class="give-required-indicator">*</span>
 					<?php } ?>
 				</label>
 
-				<input class="give-input<?php echo ( give_logged_in_only( $form_id ) ) ? ' required' : ''; ?>" type="text"
+				<input class="give-input<?php echo (give_logged_in_only($form_id)) ? ' required' : ''; ?>" type="text"
 					   name="give_user_login" id="give-user-login-<?php echo $form_id; ?>" value=""
-					   placeholder="<?php _e( 'Your username', 'give' ); ?>"<?php echo ( give_logged_in_only( $form_id ) ) ? ' required aria-required="true" ' : ''; ?>/>
+					   placeholder="<?php _e('Your username', 'give'); ?>"<?php echo (give_logged_in_only($form_id)) ? ' required aria-required="true" ' : ''; ?>/>
 			</div>
 
 			<div id="give-user-pass-wrap-<?php echo $form_id; ?>"
 				 class="give_login_password form-row form-row-last form-row-responsive">
 				<label class="give-label" for="give-user-pass-<?php echo $form_id; ?>">
-					<?php _e( 'Password', 'give' ); ?>
-					<?php if ( give_logged_in_only( $form_id ) ) { ?>
+					<?php _e('Password', 'give'); ?>
+					<?php if (give_logged_in_only($form_id)) { ?>
 						<span class="give-required-indicator">*</span>
 					<?php } ?>
 				</label>
-				<input class="give-input<?php echo ( give_logged_in_only( $form_id ) ) ? ' required' : ''; ?>"
+				<input class="give-input<?php echo (give_logged_in_only($form_id)) ? ' required' : ''; ?>"
 					   type="password" name="give_user_pass" id="give-user-pass-<?php echo $form_id; ?>"
-					   placeholder="<?php _e( 'Your password', 'give' ); ?>"<?php echo ( give_logged_in_only( $form_id ) ) ? ' required aria-required="true" ' : ''; ?>/>
+					   placeholder="<?php _e('Your password', 'give'); ?>"<?php echo (give_logged_in_only($form_id)) ? ' required aria-required="true" ' : ''; ?>/>
 				<input type="hidden" name="give-purchase-var" value="needs-to-login"/>
 			</div>
 
 			<div id="give-forgot-password-wrap-<?php echo $form_id; ?>" class="give_login_forgot_password">
 				 <span class="give-forgot-password ">
 					 <a href="<?php echo wp_lostpassword_url() ?>"
-						target="_blank"><?php _e( 'Reset Password', 'give' ) ?></a>
+						target="_blank"><?php _e('Reset Password', 'give') ?></a>
 				 </span>
 			</div>
 		</div>
@@ -1452,11 +1452,11 @@ 
 
 		<div id="give-user-login-submit-<?php echo $form_id; ?>" class="give-clearfix">
 			<input type="submit" class="give-submit give-btn button" name="give_login_submit"
-				   value="<?php _e( 'Login', 'give' ); ?>"/>
-			<?php if ( $show_register_form !== 'login' ) { ?>
+				   value="<?php _e('Login', 'give'); ?>"/>
+			<?php if ($show_register_form !== 'login') { ?>
 				<input type="button" data-action="give_cancel_login"
 					   class="give-cancel-login give-checkout-register-cancel give-btn button" name="give_login_cancel"
-					   value="<?php _e( 'Cancel', 'give' ); ?>"/>
+					   value="<?php _e('Cancel', 'give'); ?>"/>
 			<?php } ?>
 			<span class="give-loading-animation"></span>
 		</div>
@@ -1468,14 +1468,14 @@ 
 		 *
 		 * @param int $form_id The form ID.
 		 */
-		do_action( 'give_checkout_login_fields_after', $form_id );
+		do_action('give_checkout_login_fields_after', $form_id);
 		?>
 	</fieldset><!--end #give-login-fields-->
 	<?php
 	echo ob_get_clean();
 }
 
-add_action( 'give_donation_form_login_fields', 'give_get_login_fields', 10, 1 );
+add_action('give_donation_form_login_fields', 'give_get_login_fields', 10, 1);
 
 /**
  * Payment Mode Select.
@@ -1491,10 +1491,10 @@ 
  *
  * @return void
  */
-function give_payment_mode_select( $form_id, $args ) {
+function give_payment_mode_select($form_id, $args) {
 
-	$gateways  = give_get_enabled_payment_gateways( $form_id );
-	$id_prefix = ! empty( $args['id_prefix'] ) ? $args['id_prefix'] : '';
+	$gateways  = give_get_enabled_payment_gateways($form_id);
+	$id_prefix = ! empty($args['id_prefix']) ? $args['id_prefix'] : '';
 
 	/**
 	 * Fires while selecting payment gateways, before the fields.
@@ -1503,10 +1503,10 @@ 
 	 *
 	 * @param int $form_id The form ID.
 	 */
-	do_action( 'give_payment_mode_top', $form_id );
+	do_action('give_payment_mode_top', $form_id);
 	?>
 
-	<fieldset id="give-payment-mode-select" <?php if ( count( $gateways ) <= 1 ) {
+	<fieldset id="give-payment-mode-select" <?php if (count($gateways) <= 1) {
 		echo 'style="display: none;"';
 	} ?>>
 		<?php
@@ -1517,10 +1517,10 @@ 
 		 *
 		 * @param int $form_id The form ID.
 		 */
-		do_action( 'give_payment_mode_before_gateways_wrap' );
+		do_action('give_payment_mode_before_gateways_wrap');
 		?>
 		<legend
-				class="give-payment-mode-label"><?php echo apply_filters( 'give_checkout_payment_method_text', esc_html__( 'Select Payment Method', 'give' ) ); ?>
+				class="give-payment-mode-label"><?php echo apply_filters('give_checkout_payment_method_text', esc_html__('Select Payment Method', 'give')); ?>
 			<span class="give-loading-text"><span
 						class="give-loading-animation"></span>
             </span>
@@ -1533,37 +1533,36 @@ 
 			 *
 			 * @since 1.7
 			 */
-			do_action( 'give_payment_mode_before_gateways' )
+			do_action('give_payment_mode_before_gateways')
 			?>
 			<ul id="give-gateway-radio-list">
 				<?php
 				/**
 				 * Loop through the active payment gateways.
 				 */
-				$selected_gateway = give_get_chosen_gateway( $form_id );
+				$selected_gateway = give_get_chosen_gateway($form_id);
 				$give_settings    = give_get_settings();
-				$gateways_label   = array_key_exists( 'gateways_label', $give_settings ) ?
-					$give_settings['gateways_label'] :
-					array();
+				$gateways_label   = array_key_exists('gateways_label', $give_settings) ?
+					$give_settings['gateways_label'] : array();
 
-				foreach ( $gateways as $gateway_id => $gateway ) :
+				foreach ($gateways as $gateway_id => $gateway) :
 					//Determine the default gateway.
-					$checked = checked( $gateway_id, $selected_gateway, false );
+					$checked = checked($gateway_id, $selected_gateway, false);
 					$checked_class = $checked ? ' class="give-gateway-option-selected"' : ''; ?>
 					<li<?php echo $checked_class ?>>
 						<input type="radio" name="payment-mode" class="give-gateway"
-							   id="give-gateway-<?php echo esc_attr( $gateway_id . '-' . $id_prefix ); ?>"
-							   value="<?php echo esc_attr( $gateway_id ); ?>"<?php echo $checked; ?>>
+							   id="give-gateway-<?php echo esc_attr($gateway_id.'-'.$id_prefix); ?>"
+							   value="<?php echo esc_attr($gateway_id); ?>"<?php echo $checked; ?>>
 
 						<?php
 						$label = $gateway['checkout_label'];
-						if ( ! empty( $gateways_label[ $gateway_id  ] ) ) {
-							$label = $gateways_label[ $gateway_id ];
+						if ( ! empty($gateways_label[$gateway_id])) {
+							$label = $gateways_label[$gateway_id];
 						}
 						?>
-						<label for="give-gateway-<?php echo esc_attr( $gateway_id . '-' . $id_prefix ); ?>"
+						<label for="give-gateway-<?php echo esc_attr($gateway_id.'-'.$id_prefix); ?>"
 							   class="give-gateway-option"
-							   id="give-gateway-option-<?php echo esc_attr( $gateway_id ); ?>"> <?php echo esc_html( $label ); ?></label>
+							   id="give-gateway-option-<?php echo esc_attr($gateway_id); ?>"> <?php echo esc_html($label); ?></label>
 					</li>
 					<?php
 				endforeach;
@@ -1575,7 +1574,7 @@ 
 			 *
 			 * @since 1.7
 			 */
-			do_action( 'give_payment_mode_after_gateways' );
+			do_action('give_payment_mode_after_gateways');
 			?>
 		</div>
 		<?php
@@ -1586,7 +1585,7 @@ 
 		 *
 		 * @param int $form_id The form ID.
 		 */
-		do_action( 'give_payment_mode_after_gateways_wrap' );
+		do_action('give_payment_mode_after_gateways_wrap');
 		?>
 	</fieldset>
 
@@ -1598,7 +1597,7 @@ 
 	 *
 	 * @param int $form_id The form ID.
 	 */
-	do_action( 'give_payment_mode_bottom', $form_id );
+	do_action('give_payment_mode_bottom', $form_id);
 	?>
 
 	<div id="give_purchase_form_wrap">
@@ -1609,7 +1608,7 @@ 
 		 *
 		 * @since 1.7
 		 */
-		do_action( 'give_donation_form', $form_id, $args );
+		do_action('give_donation_form', $form_id, $args);
 		?>
 
 	</div>
@@ -1620,10 +1619,10 @@ 
 	 *
 	 * @since 1.7
 	 */
-	do_action( 'give_donation_form_wrap_bottom', $form_id );
+	do_action('give_donation_form_wrap_bottom', $form_id);
 }
 
-add_action( 'give_payment_mode_select', 'give_payment_mode_select', 10, 2 );
+add_action('give_payment_mode_select', 'give_payment_mode_select', 10, 2);
 
 /**
  * Renders the Checkout Agree to Terms, this displays a checkbox for users to
@@ -1636,22 +1635,22 @@ 
  *
  * @return bool
  */
-function give_terms_agreement( $form_id ) {
-	$form_option = give_get_meta( $form_id, '_give_terms_option', true );
+function give_terms_agreement($form_id) {
+	$form_option = give_get_meta($form_id, '_give_terms_option', true);
 
 	// Bailout if per form and global term and conditions is not setup.
 	if (
-		give_is_setting_enabled( $form_option, 'global' )
-		&& give_is_setting_enabled( give_get_option( 'terms' ) )
+		give_is_setting_enabled($form_option, 'global')
+		&& give_is_setting_enabled(give_get_option('terms'))
 	) {
-		$label         = give_get_option( 'agree_to_terms_label', esc_html__( 'Agree to Terms?', 'give' ) );
-		$terms         = $terms = give_get_option( 'agreement_text', '' );
-		$edit_term_url = admin_url( 'edit.php?post_type=give_forms&page=give-settings&tab=display&section=term-and-conditions' );
+		$label         = give_get_option('agree_to_terms_label', esc_html__('Agree to Terms?', 'give'));
+		$terms         = $terms = give_get_option('agreement_text', '');
+		$edit_term_url = admin_url('edit.php?post_type=give_forms&page=give-settings&tab=display&section=term-and-conditions');
 
-	} elseif ( give_is_setting_enabled( $form_option ) ) {
-		$label         = ( $label = give_get_meta( $form_id, '_give_agree_label', true ) ) ? stripslashes( $label ) : esc_html__( 'Agree to Terms?', 'give' );
-		$terms         = give_get_meta( $form_id, '_give_agree_text', true );
-		$edit_term_url = admin_url( 'post.php?post=' . $form_id . '&action=edit#form_terms_options' );
+	} elseif (give_is_setting_enabled($form_option)) {
+		$label         = ($label = give_get_meta($form_id, '_give_agree_label', true)) ? stripslashes($label) : esc_html__('Agree to Terms?', 'give');
+		$terms         = give_get_meta($form_id, '_give_agree_text', true);
+		$edit_term_url = admin_url('post.php?post='.$form_id.'&action=edit#form_terms_options');
 
 	} else {
 		return false;
@@ -1659,9 +1658,9 @@ 
 
 
 	// Bailout: Check if term and conditions text is empty or not.
-	if ( empty( $terms ) ) {
-		if ( is_user_logged_in() && current_user_can( 'edit_give_forms' ) ) {
-			echo sprintf( __( 'Please enter valid terms and conditions in <a href="%s">this form\'s settings</a>.', 'give' ), $edit_term_url );
+	if (empty($terms)) {
+		if (is_user_logged_in() && current_user_can('edit_give_forms')) {
+			echo sprintf(__('Please enter valid terms and conditions in <a href="%s">this form\'s settings</a>.', 'give'), $edit_term_url);
 		}
 
 		return false;
@@ -1672,11 +1671,11 @@ 
 	 *
 	 * @since  2.1.5
 	 */
-	$terms = apply_filters( 'give_the_term_content',  wpautop( do_shortcode( $terms ) ), $terms, $form_id );
+	$terms = apply_filters('give_the_term_content', wpautop(do_shortcode($terms)), $terms, $form_id);
 
 	?>
 	<fieldset id="give_terms_agreement">
-		<legend><?php echo apply_filters( 'give_terms_agreement_text', esc_html__( 'Terms', 'give' ) ); ?></legend>
+		<legend><?php echo apply_filters('give_terms_agreement_text', esc_html__('Terms', 'give')); ?></legend>
 		<div id="give_terms" class="give_terms-<?php echo $form_id; ?>" style="display:none;">
 			<?php
 			/**
@@ -1684,7 +1683,7 @@ 
 			 *
 			 * @since 1.0
 			 */
-			do_action( 'give_before_terms' );
+			do_action('give_before_terms');
 
 			echo $terms;
 			/**
@@ -1692,14 +1691,14 @@ 
 			 *
 			 * @since 1.0
 			 */
-			do_action( 'give_after_terms' );
+			do_action('give_after_terms');
 			?>
 		</div>
 		<div id="give_show_terms">
 			<a href="#" class="give_terms_links give_terms_links-<?php echo $form_id; ?>" role="button"
-			   aria-controls="give_terms"><?php esc_html_e( 'Show Terms', 'give' ); ?></a>
+			   aria-controls="give_terms"><?php esc_html_e('Show Terms', 'give'); ?></a>
 			<a href="#" class="give_terms_links give_terms_links-<?php echo $form_id; ?>" role="button"
-			   aria-controls="give_terms" style="display:none;"><?php esc_html_e( 'Hide Terms', 'give' ); ?></a>
+			   aria-controls="give_terms" style="display:none;"><?php esc_html_e('Hide Terms', 'give'); ?></a>
 		</div>
 
 		<input name="give_agree_to_terms" class="required" type="checkbox"
@@ -1710,7 +1709,7 @@ 
 	<?php
 }
 
-add_action( 'give_donation_form_after_cc_form', 'give_terms_agreement', 8888, 1 );
+add_action('give_donation_form_after_cc_form', 'give_terms_agreement', 8888, 1);
 
 /**
  * Checkout Final Total.
@@ -1723,15 +1722,14 @@ 
  *
  * @return void
  */
-function give_checkout_final_total( $form_id ) {
+function give_checkout_final_total($form_id) {
 
-	$total = isset( $_POST['give_total'] ) ?
-		apply_filters( 'give_donation_total', give_maybe_sanitize_amount( $_POST['give_total'] ) ) :
-		give_get_default_form_amount( $form_id );
+	$total = isset($_POST['give_total']) ?
+		apply_filters('give_donation_total', give_maybe_sanitize_amount($_POST['give_total'])) : give_get_default_form_amount($form_id);
 
 
 	// Only proceed if give_total available.
-	if ( empty( $total ) ) {
+	if (empty($total)) {
 		return;
 	}
 	?>
@@ -1742,18 +1740,18 @@ 
 		 *
 		 * @since 2.0.5
 		 */
-		do_action( 'give_donation_final_total_label_before', $form_id );
+		do_action('give_donation_final_total_label_before', $form_id);
 		?>
 		<span class="give-donation-total-label">
-			<?php echo apply_filters( 'give_donation_total_label', esc_html__( 'Donation Total:', 'give' ) ); ?>
+			<?php echo apply_filters('give_donation_total_label', esc_html__('Donation Total:', 'give')); ?>
 		</span>
 		<span class="give-final-total-amount"
-			  data-total="<?php echo give_format_amount( $total, array( 'sanitize' => false ) ); ?>">
+			  data-total="<?php echo give_format_amount($total, array('sanitize' => false)); ?>">
 			<?php
-			echo give_currency_filter( give_format_amount( $total, array(
+			echo give_currency_filter(give_format_amount($total, array(
 				'sanitize' => false,
-				'currency' => give_get_currency( $form_id ),
-			) ), array( 'currency_code' => give_get_currency( $form_id ) ) ); ?>
+				'currency' => give_get_currency($form_id),
+			)), array('currency_code' => give_get_currency($form_id))); ?>
 		</span>
 		<?php
 		/**
@@ -1761,13 +1759,13 @@ 
 		 *
 		 * @since 2.0.5
 		 */
-		do_action( 'give_donation_final_total_label_after', $form_id );
+		do_action('give_donation_final_total_label_after', $form_id);
 		?>
 	</p>
 	<?php
 }
 
-add_action( 'give_donation_form_before_submit', 'give_checkout_final_total', 999 );
+add_action('give_donation_form_before_submit', 'give_checkout_final_total', 999);
 
 /**
  * Renders the Checkout Submit section.
@@ -1779,7 +1777,7 @@ 
  *
  * @return void
  */
-function give_checkout_submit( $form_id, $args ) {
+function give_checkout_submit($form_id, $args) {
 	?>
 	<fieldset id="give_purchase_submit" class="give-donation-submit">
 		<?php
@@ -1788,24 +1786,24 @@ 
 		 *
 		 * @since 1.7
 		 */
-		do_action( 'give_donation_form_before_submit', $form_id, $args );
+		do_action('give_donation_form_before_submit', $form_id, $args);
 
-		give_checkout_hidden_fields( $form_id );
+		give_checkout_hidden_fields($form_id);
 
-		echo give_get_donation_form_submit_button( $form_id );
+		echo give_get_donation_form_submit_button($form_id);
 
 		/**
 		 * Fire after donation form submit.
 		 *
 		 * @since 1.7
 		 */
-		do_action( 'give_donation_form_after_submit', $form_id, $args );
+		do_action('give_donation_form_after_submit', $form_id, $args);
 		?>
 	</fieldset>
 	<?php
 }
 
-add_action( 'give_donation_form_after_cc_form', 'give_checkout_submit', 9999, 2 );
+add_action('give_donation_form_after_cc_form', 'give_checkout_submit', 9999, 2);
 
 /**
  * Give Donation form submit button.
@@ -1816,10 +1814,10 @@ 
  *
  * @return string
  */
-function give_get_donation_form_submit_button( $form_id ) {
+function give_get_donation_form_submit_button($form_id) {
 
-	$display_label_field = give_get_meta( $form_id, '_give_checkout_label', true );
-	$display_label       = ( ! empty( $display_label_field ) ? $display_label_field : esc_html__( 'Donate Now', 'give' ) );
+	$display_label_field = give_get_meta($form_id, '_give_checkout_label', true);
+	$display_label       = ( ! empty($display_label_field) ? $display_label_field : esc_html__('Donate Now', 'give'));
 	ob_start();
 	?>
 	<div class="give-submit-button-wrap give-clearfix">
@@ -1828,7 +1826,7 @@ 
 		<span class="give-loading-animation"></span>
 	</div>
 	<?php
-	return apply_filters( 'give_donation_form_submit_button', ob_get_clean(), $form_id );
+	return apply_filters('give_donation_form_submit_button', ob_get_clean(), $form_id);
 }
 
 /**
@@ -1843,22 +1841,22 @@ 
  *
  * @return mixed
  */
-function give_show_goal_progress( $form_id, $args = array() ) {
+function give_show_goal_progress($form_id, $args = array()) {
 
 	ob_start();
-	give_get_template( 'shortcode-goal', array( 'form_id' => $form_id, 'args' => $args ) );
+	give_get_template('shortcode-goal', array('form_id' => $form_id, 'args' => $args));
 
 	/**
 	 * Filter progress bar output
 	 *
 	 * @since 2.0
 	 */
-	echo apply_filters( 'give_goal_output', ob_get_clean(), $form_id, $args );
+	echo apply_filters('give_goal_output', ob_get_clean(), $form_id, $args);
 
 	return true;
 }
 
-add_action( 'give_pre_form', 'give_show_goal_progress', 10, 2 );
+add_action('give_pre_form', 'give_show_goal_progress', 10, 2);
 
 /**
  * Show Give Totals Progress.
@@ -1870,22 +1868,22 @@ 
  *
  * @return mixed
  */
-function give_show_goal_totals_progress( $total, $total_goal ) {
+function give_show_goal_totals_progress($total, $total_goal) {
 
 	// Bail out if total goal is set as an array.
-	if ( isset( $total_goal ) && is_array( $total_goal ) ) {
+	if (isset($total_goal) && is_array($total_goal)) {
 		return false;
 	}
 
 	ob_start();
-	give_get_template( 'shortcode-totals-progress', array( 'total' => $total, 'total_goal' => $total_goal ) );
+	give_get_template('shortcode-totals-progress', array('total' => $total, 'total_goal' => $total_goal));
 
-	echo apply_filters( 'give_total_progress_output', ob_get_clean() );
+	echo apply_filters('give_total_progress_output', ob_get_clean());
 
 	return true;
 }
 
-add_action( 'give_pre_form', 'give_show_goal_totals_progress', 10, 2 );
+add_action('give_pre_form', 'give_show_goal_totals_progress', 10, 2);
 
 /**
  * Get form content position.
@@ -1897,10 +1895,10 @@ 
  *
  * @return mixed|string
  */
-function give_get_form_content_placement( $form_id, $args ) {
+function give_get_form_content_placement($form_id, $args) {
 	$show_content = '';
 
-	if ( isset( $args['show_content'] ) && ! empty( $args['show_content'] ) ) {
+	if (isset($args['show_content']) && ! empty($args['show_content'])) {
 		// Content positions.
 		$content_placement = array(
 			'above' => 'give_pre_form',
@@ -1908,18 +1906,18 @@ 
 		);
 
 		// Check if content position already decoded.
-		if ( in_array( $args['show_content'], $content_placement ) ) {
+		if (in_array($args['show_content'], $content_placement)) {
 			return $args['show_content'];
 		}
 
-		$show_content = ( 'none' !== $args['show_content'] ? $content_placement[ $args['show_content'] ] : '' );
+		$show_content = ('none' !== $args['show_content'] ? $content_placement[$args['show_content']] : '');
 
-	} elseif ( give_is_setting_enabled( give_get_meta( $form_id, '_give_display_content', true ) ) ) {
-		$show_content = give_get_meta( $form_id, '_give_content_placement', true );
+	} elseif (give_is_setting_enabled(give_get_meta($form_id, '_give_display_content', true))) {
+		$show_content = give_get_meta($form_id, '_give_content_placement', true);
 
-	} elseif ( 'none' !== give_get_meta( $form_id, '_give_content_option', true ) ) {
+	} elseif ('none' !== give_get_meta($form_id, '_give_content_option', true)) {
 		// Backward compatibility for _give_content_option for v18.
-		$show_content = give_get_meta( $form_id, '_give_content_option', true );
+		$show_content = give_get_meta($form_id, '_give_content_option', true);
 	}
 
 	return $show_content;
@@ -1935,20 +1933,20 @@ 
  *
  * @return void|bool
  */
-function give_form_content( $form_id, $args ) {
+function give_form_content($form_id, $args) {
 
-	$show_content = give_get_form_content_placement( $form_id, $args );
+	$show_content = give_get_form_content_placement($form_id, $args);
 
 	// Bailout.
-	if ( empty( $show_content ) ) {
+	if (empty($show_content)) {
 		return false;
 	}
 
 	// Add action according to value.
-	add_action( $show_content, 'give_form_display_content', 10, 2 );
+	add_action($show_content, 'give_form_display_content', 10, 2);
 }
 
-add_action( 'give_pre_form_output', 'give_form_content', 10, 2 );
+add_action('give_pre_form_output', 'give_form_content', 10, 2);
 
 /**
  * Renders Post Form Content.
@@ -1962,14 +1960,14 @@ 
  *
  * @return void
  */
-function give_form_display_content( $form_id, $args ) {
-	$content      = give_get_meta( $form_id, '_give_form_content', true );
-	$show_content = give_get_form_content_placement( $form_id, $args );
-
-	if ( give_is_setting_enabled( give_get_option( 'the_content_filter' ) ) ) {
-		$content = apply_filters( 'the_content', $content );
-	} else{
-		$content = wpautop( do_shortcode( $content ) );
+function give_form_display_content($form_id, $args) {
+	$content      = give_get_meta($form_id, '_give_form_content', true);
+	$show_content = give_get_form_content_placement($form_id, $args);
+
+	if (give_is_setting_enabled(give_get_option('the_content_filter'))) {
+		$content = apply_filters('the_content', $content);
+	} else {
+		$content = wpautop(do_shortcode($content));
 	}
 
 	$output = sprintf(
@@ -1988,11 +1986,11 @@ 
 	 * @param int    $form_id
 	 * @param array  $args
 	 */
-	echo apply_filters( 'give_form_content_output', $output, $form_id, $args );
+	echo apply_filters('give_form_content_output', $output, $form_id, $args);
 
 	// remove action to prevent content output on addition forms on page.
 	// @see: https://github.com/WordImpress/Give/issues/634.
-	remove_action( $show_content, 'give_form_display_content' );
+	remove_action($show_content, 'give_form_display_content');
 }
 
 /**
@@ -2004,7 +2002,7 @@ 
  *
  * @return void
  */
-function give_checkout_hidden_fields( $form_id ) {
+function give_checkout_hidden_fields($form_id) {
 
 	/**
 	 * Fires while rendering hidden checkout fields, before the fields.
@@ -2013,13 +2011,13 @@ 
 	 *
 	 * @param int $form_id The form ID.
 	 */
-	do_action( 'give_hidden_fields_before', $form_id );
+	do_action('give_hidden_fields_before', $form_id);
 
-	if ( is_user_logged_in() ) { ?>
+	if (is_user_logged_in()) { ?>
 		<input type="hidden" name="give-user-id" value="<?php echo get_current_user_id(); ?>"/>
 	<?php } ?>
 	<input type="hidden" name="give_action" value="purchase"/>
-	<input type="hidden" name="give-gateway" value="<?php echo give_get_chosen_gateway( $form_id ); ?>"/>
+	<input type="hidden" name="give-gateway" value="<?php echo give_get_chosen_gateway($form_id); ?>"/>
 	<?php
 	/**
 	 * Fires while rendering hidden checkout fields, after the fields.
@@ -2028,7 +2026,7 @@ 
 	 *
 	 * @param int $form_id The form ID.
 	 */
-	do_action( 'give_hidden_fields_after', $form_id );
+	do_action('give_hidden_fields_after', $form_id);
 
 }
 
@@ -2043,20 +2041,20 @@ 
  *
  * @return string $content Filtered content.
  */
-function give_filter_success_page_content( $content ) {
+function give_filter_success_page_content($content) {
 
 	$give_options = give_get_settings();
 
-	if ( isset( $give_options['success_page'] ) && isset( $_GET['payment-confirmation'] ) && is_page( $give_options['success_page'] ) ) {
-		if ( has_filter( 'give_payment_confirm_' . $_GET['payment-confirmation'] ) ) {
-			$content = apply_filters( 'give_payment_confirm_' . $_GET['payment-confirmation'], $content );
+	if (isset($give_options['success_page']) && isset($_GET['payment-confirmation']) && is_page($give_options['success_page'])) {
+		if (has_filter('give_payment_confirm_'.$_GET['payment-confirmation'])) {
+			$content = apply_filters('give_payment_confirm_'.$_GET['payment-confirmation'], $content);
 		}
 	}
 
 	return $content;
 }
 
-add_filter( 'the_content', 'give_filter_success_page_content' );
+add_filter('the_content', 'give_filter_success_page_content');
 
 /**
  * Test Mode Frontend Warning.
@@ -2067,12 +2065,12 @@ 
  */
 function give_test_mode_frontend_warning() {
 
-	if ( give_is_test_mode() ) {
-		echo '<div class="give_error give_warning" id="give_error_test_mode"><p><strong>' . esc_html__( 'Notice:', 'give' ) . '</strong> ' . esc_html__( 'Test mode is enabled. While in test mode no live donations are processed.', 'give' ) . '</p></div>';
+	if (give_is_test_mode()) {
+		echo '<div class="give_error give_warning" id="give_error_test_mode"><p><strong>'.esc_html__('Notice:', 'give').'</strong> '.esc_html__('Test mode is enabled. While in test mode no live donations are processed.', 'give').'</p></div>';
 	}
 }
 
-add_action( 'give_pre_form', 'give_test_mode_frontend_warning', 10 );
+add_action('give_pre_form', 'give_test_mode_frontend_warning', 10);
 
 /**
  * Members-only Form.
@@ -2086,21 +2084,21 @@ 
  *
  * @return string
  */
-function give_members_only_form( $final_output, $args ) {
+function give_members_only_form($final_output, $args) {
 
-	$form_id = isset( $args['form_id'] ) ? $args['form_id'] : 0;
+	$form_id = isset($args['form_id']) ? $args['form_id'] : 0;
 
 	//Sanity Check: Must have form_id & not be logged in.
-	if ( empty( $form_id ) || is_user_logged_in() ) {
+	if (empty($form_id) || is_user_logged_in()) {
 		return $final_output;
 	}
 
 	//Logged in only and Register / Login set to none.
-	if ( give_logged_in_only( $form_id ) && give_show_login_register_option( $form_id ) == 'none' ) {
+	if (give_logged_in_only($form_id) && give_show_login_register_option($form_id) == 'none') {
 
-		$final_output = Give()->notices->print_frontend_notice( esc_html__( 'Please log in in order to complete your donation.', 'give' ), false );
+		$final_output = Give()->notices->print_frontend_notice(esc_html__('Please log in in order to complete your donation.', 'give'), false);
 
-		return apply_filters( 'give_members_only_output', $final_output, $form_id );
+		return apply_filters('give_members_only_output', $final_output, $form_id);
 
 	}
 
@@ -2108,7 +2106,7 @@ 
 
 }
 
-add_filter( 'give_donate_form', 'give_members_only_form', 10, 2 );
+add_filter('give_donate_form', 'give_members_only_form', 10, 2);
 
 
 /**
@@ -2120,40 +2118,40 @@ 
  * @param array            $args
  * @param Give_Donate_Form $form
  */
-function __give_form_add_donation_hidden_field( $form_id, $args, $form ) {
-	$id_prefix = ! empty( $args['id_prefix'] ) ? $args['id_prefix'] : '';
+function __give_form_add_donation_hidden_field($form_id, $args, $form) {
+	$id_prefix = ! empty($args['id_prefix']) ? $args['id_prefix'] : '';
 	?>
 	<input type="hidden" name="give-form-id-prefix" value="<?php echo $id_prefix; ?>"/>
-	<input type="hidden" name="give-form-id" value="<?php echo intval( $form_id ); ?>"/>
-	<input type="hidden" name="give-form-title" value="<?php echo esc_html( $form->post_title ); ?>"/>
-	<input type="hidden" name="give-current-url" value="<?php echo esc_url( give_get_current_page_url() ); ?>"/>
-	<input type="hidden" name="give-form-url" value="<?php echo esc_url( give_get_current_page_url() ); ?>"/>
+	<input type="hidden" name="give-form-id" value="<?php echo intval($form_id); ?>"/>
+	<input type="hidden" name="give-form-title" value="<?php echo esc_html($form->post_title); ?>"/>
+	<input type="hidden" name="give-current-url" value="<?php echo esc_url(give_get_current_page_url()); ?>"/>
+	<input type="hidden" name="give-form-url" value="<?php echo esc_url(give_get_current_page_url()); ?>"/>
 	<?php
 	// Get the custom option amount.
-	$custom_amount = give_get_meta( $form_id, '_give_custom_amount', true );
+	$custom_amount = give_get_meta($form_id, '_give_custom_amount', true);
 
 	// If custom amount enabled.
-	if ( give_is_setting_enabled( $custom_amount ) ) {
+	if (give_is_setting_enabled($custom_amount)) {
 		?>
 		<input type="hidden" name="give-form-minimum"
-		       value="<?php echo give_maybe_sanitize_amount( give_get_form_minimum_price( $form_id ) ); ?>"/>
+		       value="<?php echo give_maybe_sanitize_amount(give_get_form_minimum_price($form_id)); ?>"/>
 		<input type="hidden" name="give-form-maximum"
-		       value="<?php echo give_maybe_sanitize_amount( give_get_form_maximum_price( $form_id ) ); ?>"/>
+		       value="<?php echo give_maybe_sanitize_amount(give_get_form_maximum_price($form_id)); ?>"/>
 		<?php
 	}
 
 	// WP nonce field.
 	echo str_replace(
 		'/>',
-		'data-time="' . time() . '" data-nonce-life="' . give_get_nonce_life() . '"/>',
-		give_get_nonce_field( "give_donation_form_nonce_{$form_id}", 'give-form-hash', false )
+		'data-time="'.time().'" data-nonce-life="'.give_get_nonce_life().'"/>',
+		give_get_nonce_field("give_donation_form_nonce_{$form_id}", 'give-form-hash', false)
 	);
 
 	// Price ID hidden field for variable (multi-level) donation forms.
-	if ( give_has_variable_prices( $form_id ) ) {
+	if (give_has_variable_prices($form_id)) {
 		// Get the default price ID.
-		$default_price = give_form_get_default_level( $form_id );
-		$price_id      = isset( $default_price['_give_id']['level_id'] ) ? $default_price['_give_id']['level_id'] : 0;
+		$default_price = give_form_get_default_level($form_id);
+		$price_id      = isset($default_price['_give_id']['level_id']) ? $default_price['_give_id']['level_id'] : 0;
 
 		echo sprintf(
 			'<input type="hidden" name="give-price-id" value="%s"/>',
@@ -2162,7 +2160,7 @@ 
 	}
 }
 
-add_action( 'give_donation_form_top', '__give_form_add_donation_hidden_field', 0, 3 );
+add_action('give_donation_form_top', '__give_form_add_donation_hidden_field', 0, 3);
 
 /**
  * Add currency settings on donation form.
@@ -2174,20 +2172,20 @@ 
  *
  * @return array
  */
-function __give_form_add_currency_settings( $form_html_tags, $form ) {
-	$form_currency     = give_get_currency( $form->ID );
-	$currency_settings = give_get_currency_formatting_settings( $form_currency );
+function __give_form_add_currency_settings($form_html_tags, $form) {
+	$form_currency     = give_get_currency($form->ID);
+	$currency_settings = give_get_currency_formatting_settings($form_currency);
 
 	// Check if currency exist.
-	if ( empty( $currency_settings ) ) {
+	if (empty($currency_settings)) {
 		return $form_html_tags;
 	}
 
-	$form_html_tags['data-currency_symbol'] = give_currency_symbol( $form_currency );
+	$form_html_tags['data-currency_symbol'] = give_currency_symbol($form_currency);
 	$form_html_tags['data-currency_code']   = $form_currency;
 
-	if ( ! empty( $currency_settings ) ) {
-		foreach ( $currency_settings as $key => $value ) {
+	if ( ! empty($currency_settings)) {
+		foreach ($currency_settings as $key => $value) {
 			$form_html_tags["data-{$key}"] = $value;
 		}
 	}
@@ -2195,7 +2193,7 @@ 
 	return $form_html_tags;
 }
 
-add_filter( 'give_form_html_tags', '__give_form_add_currency_settings', 0, 2 );
+add_filter('give_form_html_tags', '__give_form_add_currency_settings', 0, 2);
 
 /**
  * Adds classes to progress bar container.
@@ -2206,7 +2204,7 @@ 
  *
  * @return string
  */
-function add_give_goal_progress_class( $class_goal ) {
+function add_give_goal_progress_class($class_goal) {
 	$class_goal = 'progress progress-striped active';
 
 	return $class_goal;
@@ -2221,7 +2219,7 @@ 
  *
  * @return string
  */
-function add_give_goal_progress_bar_class( $class_bar ) {
+function add_give_goal_progress_bar_class($class_bar) {
 	$class_bar = 'bar';
 
 	return $class_bar;
@@ -2238,12 +2236,12 @@ 
  *
  * @return array
  */
-function add_class_for_form_grid( $class, $id, $args ) {
+function add_class_for_form_grid($class, $id, $args) {
 	$class[] = 'give-form-grid-wrap';
 
-	foreach ( $class as $index => $item ) {
-		if( false !== strpos( $item, 'give-display-' ) ) {
-			unset( $class[$index] );
+	foreach ($class as $index => $item) {
+		if (false !== strpos($item, 'give-display-')) {
+			unset($class[$index]);
 		}
 	}
 
@@ -2259,7 +2257,7 @@ 
  *
  * @since 2.1
  */
-function give_is_form_grid_page_hidden_field( $id, $args, $form ) {
+function give_is_form_grid_page_hidden_field($id, $args, $form) {
 	echo '<input type="hidden" name="is-form-grid" value="true" />';
 }
 
@@ -2274,28 +2272,28 @@ 
  * @since 2.1
  * @return string
  */
-function give_redirect_and_popup_form( $redirect, $args ) {
+function give_redirect_and_popup_form($redirect, $args) {
 
 	// Check the page has Form Grid.
-	$is_form_grid = isset( $_POST['is-form-grid'] ) ? give_clean( $_POST['is-form-grid'] ) : '';
+	$is_form_grid = isset($_POST['is-form-grid']) ? give_clean($_POST['is-form-grid']) : '';
 
-	if ( 'true' === $is_form_grid ) {
+	if ('true' === $is_form_grid) {
 
-		$payment_mode = give_clean( $_POST['payment-mode'] );
+		$payment_mode = give_clean($_POST['payment-mode']);
 		$form_id = $args['form-id'];
 
 		// Get the URL without Query parameters.
-		$redirect = strtok( $redirect, '?' );
+		$redirect = strtok($redirect, '?');
 
 		// Add query parameters 'form-id' and 'payment-mode'.
-		$redirect = add_query_arg( array(
+		$redirect = add_query_arg(array(
 			'form-id'      => $form_id,
 			'payment-mode' => $payment_mode,
-		), $redirect );
+		), $redirect);
 	}
 
 	// Return the modified URL.
 	return $redirect;
 }
 
-add_filter( 'give_send_back_to_checkout', 'give_redirect_and_popup_form', 10, 2 );
+add_filter('give_send_back_to_checkout', 'give_redirect_and_popup_form', 10, 2);

includes/gateways/manual.php

Spacing +15 added lines, -15 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -20,7 +20,7 @@ 
  * @since 1.0
  * @return void
  */
-add_action( 'give_manual_cc_form', '__return_false' );
+add_action('give_manual_cc_form', '__return_false');
 
 /**
  * Processes the donation data and uses the Manual Payment gateway to record
@@ -32,44 +32,44 @@ 
  *
  * @return void
  */
-function give_manual_payment( $purchase_data ) {
+function give_manual_payment($purchase_data) {
 
-	if ( ! wp_verify_nonce( $purchase_data['gateway_nonce'], 'give-gateway' ) ) {
-		wp_die( esc_html__( 'Nonce verification failed.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! wp_verify_nonce($purchase_data['gateway_nonce'], 'give-gateway')) {
+		wp_die(esc_html__('Nonce verification failed.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 	}
 
 	//Create payment_data array
 	$payment_data = array(
 		'price'           => $purchase_data['price'],
 		'give_form_title' => $purchase_data['post_data']['give-form-title'],
-		'give_form_id'    => intval( $purchase_data['post_data']['give-form-id'] ),
+		'give_form_id'    => intval($purchase_data['post_data']['give-form-id']),
 		'give_price_id'   => isset($purchase_data['post_data']['give-price-id']) ? $purchase_data['post_data']['give-price-id'] : '',
 		'date'            => $purchase_data['date'],
 		'user_email'      => $purchase_data['user_email'],
 		'purchase_key'    => $purchase_data['purchase_key'],
-		'currency'        => give_get_currency( $purchase_data['post_data']['give-form-id'], $purchase_data ),
+		'currency'        => give_get_currency($purchase_data['post_data']['give-form-id'], $purchase_data),
 		'user_info'       => $purchase_data['user_info'],
 		'status'          => 'pending'
 	);
 	// Record the pending payment
-	$payment = give_insert_payment( $payment_data );
+	$payment = give_insert_payment($payment_data);
 
-	if ( $payment ) {
-		give_update_payment_status( $payment, 'publish' );
+	if ($payment) {
+		give_update_payment_status($payment, 'publish');
 		give_send_to_success_page();
 	} else {
 		give_record_gateway_error(
-			esc_html__( 'Payment Error', 'give' ),
+			esc_html__('Payment Error', 'give'),
 			sprintf(
 				/* translators: %s: payment data */
-				esc_html__( 'The payment creation failed while processing a manual (free or test) donation. Payment data: %s', 'give' ),
-				json_encode( $payment_data )
+				esc_html__('The payment creation failed while processing a manual (free or test) donation. Payment data: %s', 'give'),
+				json_encode($payment_data)
 			),
 			$payment
 		);
 		// If errors are present, send the user back to the donation page so they can be corrected
-		give_send_back_to_checkout( '?payment-mode=' . $purchase_data['post_data']['give-gateway'] );
+		give_send_back_to_checkout('?payment-mode='.$purchase_data['post_data']['give-gateway']);
 	}
 }
 
-add_action( 'give_gateway_manual', 'give_manual_payment' );
+add_action('give_gateway_manual', 'give_manual_payment');

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

Doc Comments +1 added lines, -1 removed lines

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

Spacing +96 added lines, -96 removed lines

@@ -12,13 +12,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded.
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -61,11 +61,11 @@ 
 	public function __construct() {
 
 		// Set parent defaults.
-		parent::__construct( array(
-			'singular' => __( 'Donor', 'give' ), // Singular name of the listed records.
-			'plural'   => __( 'Donors', 'give' ), // Plural name of the listed records.
+		parent::__construct(array(
+			'singular' => __('Donor', 'give'), // Singular name of the listed records.
+			'plural'   => __('Donors', 'give'), // Plural name of the listed records.
 			'ajax'     => false, // Does this table support ajax?.
-		) );
+		));
 
 	}
 
@@ -80,23 +80,23 @@ 
 	 *
 	 * @return void
 	 */
-	public function search_box( $text, $input_id ) {
-		$input_id = $input_id . '-search-input';
+	public function search_box($text, $input_id) {
+		$input_id = $input_id.'-search-input';
 
-		if ( ! empty( $_REQUEST['orderby'] ) ) {
-			echo sprintf( '<input type="hidden" name="orderby" value="%1$s" />', esc_attr( $_REQUEST['orderby'] ) );
+		if ( ! empty($_REQUEST['orderby'])) {
+			echo sprintf('<input type="hidden" name="orderby" value="%1$s" />', esc_attr($_REQUEST['orderby']));
 		}
 
-		if ( ! empty( $_REQUEST['order'] ) ) {
-			echo sprintf( '<input type="hidden" name="order" value="%1$s" />', esc_attr( $_REQUEST['order'] ) );
+		if ( ! empty($_REQUEST['order'])) {
+			echo sprintf('<input type="hidden" name="order" value="%1$s" />', esc_attr($_REQUEST['order']));
 		}
 		?>
 		<p class="search-box" role="search">
 			<label class="screen-reader-text" for="<?php echo $input_id ?>"><?php echo $text; ?>:</label>
 			<input type="search" id="<?php echo $input_id ?>" name="s" value="<?php _admin_search_query(); ?>"/>
-			<?php submit_button( $text, 'button', false, false, array(
+			<?php submit_button($text, 'button', false, false, array(
 				'ID' => 'search-submit',
-			) ); ?>
+			)); ?>
 		</p>
 		<?php
 	}
@@ -112,32 +112,32 @@ 
 	 *
 	 * @return string Column Name.
 	 */
-	public function column_default( $donor, $column_name ) {
+	public function column_default($donor, $column_name) {
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 
 			case 'num_donations' :
 				$value = sprintf(
 					'<a href="%s">%s</a>',
-					admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&donor=' . absint( $donor['id'] ) ),
-					esc_html( $donor['num_donations'] )
+					admin_url('edit.php?post_type=give_forms&page=give-payment-history&donor='.absint($donor['id'])),
+					esc_html($donor['num_donations'])
 				);
 				break;
 
 			case 'amount_spent' :
-				$value = give_currency_filter( give_format_amount( $donor[ $column_name ], array( 'sanitize' => false ) ) );
+				$value = give_currency_filter(give_format_amount($donor[$column_name], array('sanitize' => false)));
 				break;
 
 			case 'date_created' :
-				$value = date_i18n( give_date_format(), strtotime( $donor['date_created'] ) );
+				$value = date_i18n(give_date_format(), strtotime($donor['date_created']));
 				break;
 
 			default:
-				$value = isset( $donor[ $column_name ] ) ? $donor[ $column_name ] : null;
+				$value = isset($donor[$column_name]) ? $donor[$column_name] : null;
 				break;
 		}
 
-		return apply_filters( "give_donors_column_{$column_name}", $value, $donor['id'] );
+		return apply_filters("give_donors_column_{$column_name}", $value, $donor['id']);
 
 	}
 
@@ -151,7 +151,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function column_cb( $donor ) {
+	public function column_cb($donor) {
 		return sprintf(
 			'<input class="donor-selector" type="checkbox" name="%1$s[]" value="%2$d" data-name="%3$s" />',
 			$this->_args['singular'],
@@ -170,12 +170,12 @@ 
 	 *
 	 * @return string
 	 */
-	public function column_name( $donor ) {
-		$name     = ! empty( $donor['name'] ) ? $donor['name'] : '<em>' . __( 'Unnamed Donor', 'give' ) . '</em>';
-		$view_url = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor['id'] );
-		$actions  = $this->get_row_actions( $donor );
+	public function column_name($donor) {
+		$name     = ! empty($donor['name']) ? $donor['name'] : '<em>'.__('Unnamed Donor', 'give').'</em>';
+		$view_url = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor['id']);
+		$actions  = $this->get_row_actions($donor);
 
-		return '<a href="' . esc_url( $view_url ) . '">' . $name . '</a>' . $this->row_actions( $actions );
+		return '<a href="'.esc_url($view_url).'">'.$name.'</a>'.$this->row_actions($actions);
 	}
 
 	/**
@@ -189,14 +189,14 @@ 
 	public function get_columns() {
 		$columns = array(
 			'cb'            => '<input type="checkbox" />', // Render a checkbox instead of text.
-			'name'          => __( 'Name', 'give' ),
-			'email'         => __( 'Email', 'give' ),
-			'num_donations' => __( 'Donations', 'give' ),
-			'amount_spent'  => __( 'Total Donated', 'give' ),
-			'date_created'  => __( 'Date Created', 'give' ),
+			'name'          => __('Name', 'give'),
+			'email'         => __('Email', 'give'),
+			'num_donations' => __('Donations', 'give'),
+			'amount_spent'  => __('Total Donated', 'give'),
+			'date_created'  => __('Date Created', 'give'),
 		);
 
-		return apply_filters( 'give_list_donors_columns', $columns );
+		return apply_filters('give_list_donors_columns', $columns);
 
 	}
 
@@ -210,13 +210,13 @@ 
 	public function get_sortable_columns() {
 
 		$columns = array(
-			'date_created'  => array( 'date_created', true ),
-			'name'          => array( 'name', true ),
-			'num_donations' => array( 'purchase_count', false ),
-			'amount_spent'  => array( 'purchase_value', false ),
+			'date_created'  => array('date_created', true),
+			'name'          => array('name', true),
+			'num_donations' => array('purchase_count', false),
+			'amount_spent'  => array('purchase_value', false),
 		);
 
-		return apply_filters( 'give_list_donors_sortable_columns', $columns );
+		return apply_filters('give_list_donors_sortable_columns', $columns);
 	}
 
 	/**
@@ -229,15 +229,15 @@ 
 	 *
 	 * @return array An array of action links.
 	 */
-	public function get_row_actions( $donor ) {
+	public function get_row_actions($donor) {
 
 		$actions = array(
-			'view'   => sprintf( '<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor['id'] ), sprintf( esc_attr__( 'View "%s"', 'give' ), $donor['name'] ), __( 'View Donor', 'give' ) ),
-			'notes'  => sprintf( '<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=notes&id=' . $donor['id'] ), sprintf( esc_attr__( 'Notes for "%s"', 'give' ), $donor['name'] ), __( 'Notes', 'give' ) ),
-			'delete' => sprintf( '<a class="%1$s" data-id="%2$s" href="#" aria-label="%3$s">%4$s</a>', 'give-single-donor-delete', $donor['id'],sprintf( esc_attr__( 'Delete "%s"', 'give' ), $donor['name'] ), __( 'Delete', 'give' ) ),
+			'view'   => sprintf('<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor['id']), sprintf(esc_attr__('View "%s"', 'give'), $donor['name']), __('View Donor', 'give')),
+			'notes'  => sprintf('<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url('edit.php?post_type=give_forms&page=give-donors&view=notes&id='.$donor['id']), sprintf(esc_attr__('Notes for "%s"', 'give'), $donor['name']), __('Notes', 'give')),
+			'delete' => sprintf('<a class="%1$s" data-id="%2$s" href="#" aria-label="%3$s">%4$s</a>', 'give-single-donor-delete', $donor['id'], sprintf(esc_attr__('Delete "%s"', 'give'), $donor['name']), __('Delete', 'give')),
 		);
 
-		return apply_filters( 'give_donor_row_actions', $actions, $donor );
+		return apply_filters('give_donor_row_actions', $actions, $donor);
 
 	}
 
@@ -250,7 +250,7 @@ 
 	 * @return int Current page number.
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 	/**
@@ -262,7 +262,7 @@ 
 	 * @return mixed string If search is present, false otherwise.
 	 */
 	public function get_search() {
-		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;
+		return ! empty($_GET['s']) ? urldecode(trim($_GET['s'])) : false;
 	}
 
 	/**
@@ -275,7 +275,7 @@ 
 	 */
 	public function get_bulk_actions() {
 		$actions = array(
-			'delete' => __( 'Delete', 'give' ),
+			'delete' => __('Delete', 'give'),
 		);
 		return $actions;
 	}
@@ -288,19 +288,19 @@ 
 	 * @access protected
 	 * @since  1.8.16
 	 */
-	protected function display_tablenav( $which ) {
-		if ( 'top' === $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'], '_wpnonce', false );
+	protected function display_tablenav($which) {
+		if ('top' === $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural'], '_wpnonce', false);
 		}
 		?>
-		<div class="tablenav <?php echo esc_attr( $which ); ?>">
-			<?php if ( $this->has_items() ) : ?>
+		<div class="tablenav <?php echo esc_attr($which); ?>">
+			<?php if ($this->has_items()) : ?>
 				<div class="alignleft actions bulkactions">
-					<?php $this->bulk_actions( $which ); ?>
+					<?php $this->bulk_actions($which); ?>
 				</div>
 			<?php endif;
-			$this->extra_tablenav( $which );
-			$this->pagination( $which );
+			$this->extra_tablenav($which);
+			$this->pagination($which);
 			?>
 			<br class="clear" />
 		</div>
@@ -321,17 +321,17 @@ 
 
 		// Get donor query.
 		$args   = $this->get_donor_query();
-		$donors = Give()->donors->get_donors( $args );
+		$donors = Give()->donors->get_donors($args);
 
-		if ( $donors ) {
+		if ($donors) {
 
-			foreach ( $donors as $donor ) {
+			foreach ($donors as $donor) {
 
-				$user_id      = ! empty( $donor->user_id ) ? intval( $donor->user_id ) : 0;
-				$title_prefix = Give()->donor_meta->get_meta( $donor->id, '_give_donor_title_prefix', true );
+				$user_id      = ! empty($donor->user_id) ? intval($donor->user_id) : 0;
+				$title_prefix = Give()->donor_meta->get_meta($donor->id, '_give_donor_title_prefix', true);
 
 				// If title prefix is set, then update the donor name.
-				$donor->name = give_get_donor_name_with_title_prefixes( $title_prefix, $donor->name );
+				$donor->name = give_get_donor_name_with_title_prefixes($title_prefix, $donor->name);
 
 				$data[] = array(
 					'id'            => $donor->id,
@@ -345,7 +345,7 @@ 
 			}
 		}
 
-		return apply_filters( 'give_donors_column_query_data', $data );
+		return apply_filters('give_donors_column_query_data', $data);
 	}
 
 	/**
@@ -360,9 +360,9 @@ 
 
 		$_donor_query['number'] = - 1;
 		$_donor_query['offset'] = 0;
-		$donors                 = Give()->donors->get_donors( $_donor_query );
+		$donors                 = Give()->donors->get_donors($_donor_query);
 
-		return count( $donors );
+		return count($donors);
 	}
 
 	/**
@@ -375,10 +375,10 @@ 
 	 */
 	public function get_donor_query() {
 		$paged   = $this->get_paged();
-		$offset  = $this->per_page * ( $paged - 1 );
+		$offset  = $this->per_page * ($paged - 1);
 		$search  = $this->get_search();
-		$order   = isset( $_GET['order'] ) ? sanitize_text_field( $_GET['order'] ) : 'DESC';
-		$orderby = isset( $_GET['orderby'] ) ? sanitize_text_field( $_GET['orderby'] ) : 'id';
+		$order   = isset($_GET['order']) ? sanitize_text_field($_GET['order']) : 'DESC';
+		$orderby = isset($_GET['orderby']) ? sanitize_text_field($_GET['orderby']) : 'id';
 
 		$args = array(
 			'number'  => $this->per_page,
@@ -387,10 +387,10 @@ 
 			'orderby' => $orderby,
 		);
 
-		if ( $search ) {
-			if ( is_email( $search ) ) {
+		if ($search) {
+			if (is_email($search)) {
 				$args['email'] = $search;
-			} elseif ( is_numeric( $search ) ) {
+			} elseif (is_numeric($search)) {
 				$args['id'] = $search;
 			} else {
 				$args['name'] = $search;
@@ -408,9 +408,9 @@ 
 	 * @since  1.8.17
 	 * @access public
 	 */
-	public function single_row( $item ) {
-		echo sprintf( '<tr id="donor-%1$d" data-id="%2$d" data-name="%3$s">', $item['id'], $item['id'], $item['name'] );
-		$this->single_row_columns( $item );
+	public function single_row($item) {
+		echo sprintf('<tr id="donor-%1$d" data-id="%2$d" data-name="%3$s">', $item['id'], $item['id'], $item['name']);
+		$this->single_row_columns($item);
 		echo '</tr>';
 	}
 
@@ -423,17 +423,17 @@ 
 	public function display() {
 		$singular = $this->_args['singular'];
 
-		$this->display_tablenav( 'top' );
+		$this->display_tablenav('top');
 
-		$this->screen->render_screen_reader_content( 'heading_list' );
+		$this->screen->render_screen_reader_content('heading_list');
 
-		$get_data = give_clean( $_GET ); // WPCS: input var ok, sanitization ok, CSRF ok.
+		$get_data = give_clean($_GET); // WPCS: input var ok, sanitization ok, CSRF ok.
 
-		$search_keyword = ! empty( $get_data['s'] ) ? $get_data['s'] : '';
-		$order          = ! empty( $get_data['order'] ) ? $get_data['order'] : 'DESC';
-		$order_by       = ! empty( $get_data['orderby'] ) ? $get_data['orderby'] : 'ID';
+		$search_keyword = ! empty($get_data['s']) ? $get_data['s'] : '';
+		$order          = ! empty($get_data['order']) ? $get_data['order'] : 'DESC';
+		$order_by       = ! empty($get_data['orderby']) ? $get_data['orderby'] : 'ID';
 		?>
-		<table class="wp-list-table <?php echo implode( ' ', $this->get_table_classes() ); ?>">
+		<table class="wp-list-table <?php echo implode(' ', $this->get_table_classes()); ?>">
 			<thead>
 			<tr>
 				<?php $this->print_column_headers(); ?>
@@ -441,7 +441,7 @@ 
 			</thead>
 
 			<tbody id="the-list"<?php
-			if ( $singular ) {
+			if ($singular) {
 				echo " data-wp-lists='list:$singular'";
 			} ?>>
 			<tr class="hidden"></tr>
@@ -449,7 +449,7 @@ 
 				<td colspan="6" class="colspanchange">
 
 					<fieldset class="inline-edit-col-left">
-						<legend class="inline-edit-legend"><?php esc_attr_e( 'BULK DELETE', 'give' ); ?></legend>
+						<legend class="inline-edit-legend"><?php esc_attr_e('BULK DELETE', 'give'); ?></legend>
 						<div class="inline-edit-col">
 							<div id="bulk-titles">
 								<div id="give-bulk-donors" class="give-bulk-donors">
@@ -462,22 +462,22 @@ 
 						<div class="inline-edit-col">
 							<label>
 								<input class="give-donor-delete-confirm" type="checkbox" name="give-donor-delete-confirm"/>
-								<?php esc_attr_e( 'Are you sure you want to delete the selected donor(s)?', 'give' ); ?>
+								<?php esc_attr_e('Are you sure you want to delete the selected donor(s)?', 'give'); ?>
 							</label>
 							<label>
 								<input class="give-donor-delete-records" type="checkbox" name="give-donor-delete-records"/>
-								<?php esc_attr_e( 'Delete all associated donations and records?', 'give' ); ?>
+								<?php esc_attr_e('Delete all associated donations and records?', 'give'); ?>
 							</label>
 						</div>
 					</fieldset>
 
 					<p class="submit inline-edit-save">
 						<input type="hidden" name="give_action" value="delete_bulk_donor"/>
-						<input type="hidden" name="s" value="<?php echo esc_html( $search_keyword ); ?>"/>
-						<input type="hidden" name="orderby" value="<?php echo esc_html( $order_by ); ?>"/>
-						<input type="hidden" name="order" value="<?php echo esc_html( $order ); ?>"/>
-						<button type="button" id="give-bulk-delete-cancel" class="button cancel alignleft"><?php esc_attr_e( 'Cancel', 'give' ); ?></button>
-						<input type="submit" id="give-bulk-delete-button" disabled class="button button-primary alignright" value="<?php esc_attr_e( 'Delete', 'give' ); ?>">
+						<input type="hidden" name="s" value="<?php echo esc_html($search_keyword); ?>"/>
+						<input type="hidden" name="orderby" value="<?php echo esc_html($order_by); ?>"/>
+						<input type="hidden" name="order" value="<?php echo esc_html($order); ?>"/>
+						<button type="button" id="give-bulk-delete-cancel" class="button cancel alignleft"><?php esc_attr_e('Cancel', 'give'); ?></button>
+						<input type="submit" id="give-bulk-delete-button" disabled class="button button-primary alignright" value="<?php esc_attr_e('Delete', 'give'); ?>">
 						<br class="clear">
 					</p>
 				</td>
@@ -487,13 +487,13 @@ 
 
 			<tfoot>
 			<tr>
-				<?php $this->print_column_headers( false ); ?>
+				<?php $this->print_column_headers(false); ?>
 			</tr>
 			</tfoot>
 
 		</table>
 		<?php
-		$this->display_tablenav( 'bottom' );
+		$this->display_tablenav('bottom');
 	}
 
 	/**
@@ -510,16 +510,16 @@ 
 		$hidden   = array(); // No hidden columns.
 		$sortable = $this->get_sortable_columns();
 
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 
 		$this->items = $this->donor_data();
 
 		$this->total = $this->get_donor_count();
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 			'total_items' => $this->total,
 			'per_page'    => $this->per_page,
-			'total_pages' => ceil( $this->total / $this->per_page ),
-		) );
+			'total_pages' => ceil($this->total / $this->per_page),
+		));
 	}
 }