WordImpress / Give

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

Spacing +34 added lines, -34 removed lines

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

includes/admin/upgrades/class-give-updates.php

Braces +5 added lines, -2 removed lines

@@ -618,8 +618,11 @@
 				<a href="<?php echo esc_url( add_query_arg( array( 'give-restart-db-upgrades' => 1 ), admin_url( 'edit.php?post_type=give_forms&page=give-updates' ) ) ); ?>" class="button button-primary give-restart-updater-btn">
 					<?php _e( 'Restart the updater', 'give' ); ?>
 				</a>
-			<?php else: ?>
-				<strong><?php _e( 'Database Update', 'give' ); ?></strong>
+			<?php else {
+	: ?>
+				<strong><?php _e( 'Database Update', 'give' );
+}
+?></strong>
 				&nbsp;&#8211;&nbsp;<?php _e( 'An unexpected issue occurred during the database update which caused it to stop automatically. Please contact support for assistance.', 'give' ); ?>
 			<?php
 			endif;

Spacing +247 added lines, -254 removed lines

@@ -80,34 +80,34 @@ 
 	 *
 	 * @param array $args
 	 */
-	public function register( $args ) {
+	public function register($args) {
 		$args_default = array(
 			'id'       => '',
 			'version'  => '',
 			'callback' => '',
 		);
 
-		$args = wp_parse_args( $args, $args_default );
+		$args = wp_parse_args($args, $args_default);
 
 		// You can only register database upgrade.
 		$args['type'] = 'database';
 
 		// Bailout.
 		if (
-			empty( $args['id'] ) ||
-			empty( $args['version'] ) ||
-			empty( $args['callback'] ) ||
-			! is_callable( $args['callback'] )
+			empty($args['id']) ||
+			empty($args['version']) ||
+			empty($args['callback']) ||
+			! is_callable($args['callback'])
 		) {
 			return;
 		}
 
 		// Change depend param to array.
-		if ( isset( $args['depend'] ) && is_string( $args['depend'] ) ) {
-			$args['depend'] = array( $args['depend'] );
+		if (isset($args['depend']) && is_string($args['depend'])) {
+			$args['depend'] = array($args['depend']);
 		}
 
-		$this->updates[ $args['type'] ][] = $args;
+		$this->updates[$args['type']][] = $args;
 	}
 
 	/**
@@ -118,7 +118,7 @@ 
 	 * @return static
 	 */
 	static function get_instance() {
-		if ( is_null( self::$instance ) ) {
+		if (is_null(self::$instance)) {
 			self::$instance = new self();
 		}
 
@@ -136,27 +136,27 @@ 
 		/**
 		 * Load file
 		 */
-		require_once GIVE_PLUGIN_DIR . 'includes/class-give-background-updater.php';
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/upgrade-functions.php';
+		require_once GIVE_PLUGIN_DIR.'includes/class-give-background-updater.php';
+		require_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/upgrade-functions.php';
 
 		self::$background_updater = new Give_Background_Updater();
 
 		/**
 		 * Setup hooks.
 		 */
-		add_action( 'init', array( $this, '__register_upgrade' ), 9999 );
-		add_action( 'give_set_upgrade_completed', array( $this, '__flush_resume_updates' ), 9999 );
-		add_action( 'wp_ajax_give_db_updates_info', array( $this, '__give_db_updates_info' ) );
-		add_action( 'wp_ajax_give_run_db_updates', array( $this, '__give_start_updating' ) );
-		add_action( 'admin_init', array( $this, '__redirect_admin' ) );
-		add_action( 'admin_init', array( $this, '__pause_db_update' ), - 1 );
-		add_action( 'admin_init', array( $this, '__restart_db_update' ), - 1 );
-		add_action( 'admin_notices', array( $this, '__show_notice' ) );
-		add_action( 'give_restart_db_upgrade', array( $this, '__health_background_update' ) );
-
-		if ( is_admin() ) {
-			add_action( 'admin_init', array( $this, '__change_donations_label' ), 9999 );
-			add_action( 'admin_menu', array( $this, '__register_menu' ), 9999 );
+		add_action('init', array($this, '__register_upgrade'), 9999);
+		add_action('give_set_upgrade_completed', array($this, '__flush_resume_updates'), 9999);
+		add_action('wp_ajax_give_db_updates_info', array($this, '__give_db_updates_info'));
+		add_action('wp_ajax_give_run_db_updates', array($this, '__give_start_updating'));
+		add_action('admin_init', array($this, '__redirect_admin'));
+		add_action('admin_init', array($this, '__pause_db_update'), - 1);
+		add_action('admin_init', array($this, '__restart_db_update'), - 1);
+		add_action('admin_notices', array($this, '__show_notice'));
+		add_action('give_restart_db_upgrade', array($this, '__health_background_update'));
+
+		if (is_admin()) {
+			add_action('admin_init', array($this, '__change_donations_label'), 9999);
+			add_action('admin_menu', array($this, '__register_menu'), 9999);
 		}
 	}
 
@@ -170,12 +170,12 @@ 
 		$addons         = give_get_plugins();
 		$plugin_updates = get_plugin_updates();
 
-		foreach ( $addons as $key => $info ) {
-			if ( 'active' != $info['Status'] || 'add-on' != $info['Type'] || empty( $plugin_updates[ $key ] ) ) {
+		foreach ($addons as $key => $info) {
+			if ('active' != $info['Status'] || 'add-on' != $info['Type'] || empty($plugin_updates[$key])) {
 				continue;
 			}
 
-			$this->updates['plugin'][] = array_merge( $info, (array) $plugin_updates[ $key ] );
+			$this->updates['plugin'][] = array_merge($info, (array) $plugin_updates[$key]);
 		}
 	}
 
@@ -187,7 +187,7 @@ 
 	 * @access public
 	 */
 	public function __register_upgrade() {
-		if ( ! is_admin() ) {
+		if ( ! is_admin()) {
 			return;
 		}
 
@@ -196,7 +196,7 @@ 
 		 *
 		 * @since 1.8.12
 		 */
-		do_action( 'give_register_updates', $this );
+		do_action('give_register_updates', $this);
 	}
 
 	/**
@@ -209,23 +209,22 @@ 
 		global $menu;
 
 		// Bailout.
-		if ( empty( $menu ) || ! $this->get_total_update_count() ) {
+		if (empty($menu) || ! $this->get_total_update_count()) {
 			return;
 		}
 
-		$is_update = ( $this->is_doing_updates() && ! self::$background_updater->is_paused_process() );
+		$is_update = ($this->is_doing_updates() && ! self::$background_updater->is_paused_process());
 
-		foreach ( $menu as $index => $menu_item ) {
-			if ( 'edit.php?post_type=give_forms' !== $menu_item[2] ) {
+		foreach ($menu as $index => $menu_item) {
+			if ('edit.php?post_type=give_forms' !== $menu_item[2]) {
 				continue;
 			}
 
-			$menu[ $index ][0] = sprintf(
+			$menu[$index][0] = sprintf(
 				'%1$s <span class="update-plugins"><span class="plugin-count give-update-progress-count">%2$s%3$s</span></span>',
-				__( 'Donations', 'give' ),
+				__('Donations', 'give'),
 				$is_update ?
-					$this->get_db_update_processing_percentage() :
-					$this->get_total_update_count(),
+					$this->get_db_update_processing_percentage() : $this->get_total_update_count(),
 				$is_update ? '%' : ''
 			);
 
@@ -244,41 +243,40 @@ 
 		$this->__register_plugin_addon_updates();
 
 		// Bailout.
-		if ( ! $this->get_total_update_count() ) {
+		if ( ! $this->get_total_update_count()) {
 			// Show complete update message if still on update setting page.
-			if ( isset( $_GET['page'] ) && 'give-updates' === $_GET['page'] ) {
+			if (isset($_GET['page']) && 'give-updates' === $_GET['page']) {
 				// Upgrades
 				add_submenu_page(
 					'edit.php?post_type=give_forms',
-					esc_html__( 'Give Updates Complete', 'give' ),
-					__( 'Updates', 'give' ),
+					esc_html__('Give Updates Complete', 'give'),
+					__('Updates', 'give'),
 					'manage_give_settings',
 					'give-updates',
-					array( $this, 'render_complete_page' )
+					array($this, 'render_complete_page')
 				);
 			}
 
 			return;
 		}
 
-		$is_update = ( $this->is_doing_updates() && ! self::$background_updater->is_paused_process() );
+		$is_update = ($this->is_doing_updates() && ! self::$background_updater->is_paused_process());
 
 		// Upgrades
 		add_submenu_page(
 			'edit.php?post_type=give_forms',
-			esc_html__( 'Give Updates', 'give' ),
+			esc_html__('Give Updates', 'give'),
 			sprintf(
 				'%1$s <span class="update-plugins"%2$s><span class="plugin-count give-update-progress-count">%3$s%4$s</span></span>',
-				__( 'Updates', 'give' ),
-				isset( $_GET['give-pause-db-upgrades'] ) ? ' style="display:none;"' : '',
+				__('Updates', 'give'),
+				isset($_GET['give-pause-db-upgrades']) ? ' style="display:none;"' : '',
 				$is_update ?
-					$this->get_db_update_processing_percentage() :
-					$this->get_total_update_count(),
+					$this->get_db_update_processing_percentage() : $this->get_total_update_count(),
 				$is_update ? '%' : ''
 			),
 			'manage_give_settings',
 			'give-updates',
-			array( $this, 'render_page' )
+			array($this, 'render_page')
 		);
 	}
 
@@ -293,13 +291,13 @@ 
 		// Show db upgrade completed notice.
 		if (
 			! wp_doing_ajax() &&
-			current_user_can( 'manage_give_settings' ) &&
-			get_option( 'give_show_db_upgrade_complete_notice' ) &&
-			! isset( $_GET['give-db-update-completed'] )
+			current_user_can('manage_give_settings') &&
+			get_option('give_show_db_upgrade_complete_notice') &&
+			! isset($_GET['give-db-update-completed'])
 		) {
-			delete_option( 'give_show_db_upgrade_complete_notice' );
+			delete_option('give_show_db_upgrade_complete_notice');
 
-			wp_redirect( admin_url( 'edit.php?post_type=give_forms&page=give-updates&give-db-update-completed=give_db_upgrade_completed' ) );
+			wp_redirect(admin_url('edit.php?post_type=give_forms&page=give-updates&give-db-update-completed=give_db_upgrade_completed'));
 			exit();
 		}
 	}
@@ -315,15 +313,15 @@ 
 	 *
 	 * @return bool
 	 */
-	public function __pause_db_update( $force = false ) {
+	public function __pause_db_update($force = false) {
 		// Bailout.
 		if (
 			! $force &&
 			(
 				wp_doing_ajax() ||
-				! isset( $_GET['page'] ) ||
+				! isset($_GET['page']) ||
 				'give-updates' !== $_GET['page'] ||
-				! isset( $_GET['give-pause-db-upgrades'] ) ||
+				! isset($_GET['give-pause-db-upgrades']) ||
 				self::$background_updater->is_paused_process()
 			)
 
@@ -331,13 +329,13 @@ 
 			return false;
 		}
 
-		delete_option( 'give_upgrade_error' );
+		delete_option('give_upgrade_error');
 
-		$this->__health_background_update( $this );
+		$this->__health_background_update($this);
 		$batch = self::$background_updater->get_all_batch();
 
 		// Bailout: if batch is empty
-		if ( empty( $batch->data ) ) {
+		if (empty($batch->data)) {
 			return false;
 		}
 
@@ -346,25 +344,25 @@ 
 
 		// Do not stop background process immediately if task running.
 		// @see Give_Background_Updater::lock_process
-		if ( ! $force && self::$background_updater->is_process_running() ) {
-			update_option( 'give_pause_upgrade', 1 );
+		if ( ! $force && self::$background_updater->is_process_running()) {
+			update_option('give_pause_upgrade', 1);
 
 			return true;
 		}
 
-		update_option( 'give_paused_batches', $batch, 'no' );
-		delete_option( $batch->key );
-		delete_site_transient( self::$background_updater->get_identifier() . '_process_lock' );
-		wp_clear_scheduled_hook( self::$background_updater->get_cron_identifier() );
+		update_option('give_paused_batches', $batch, 'no');
+		delete_option($batch->key);
+		delete_site_transient(self::$background_updater->get_identifier().'_process_lock');
+		wp_clear_scheduled_hook(self::$background_updater->get_cron_identifier());
 
-		Give()->logs->add( 'Update Pause', print_r( $batch, true ), 0, 'update' );
+		Give()->logs->add('Update Pause', print_r($batch, true), 0, 'update');
 
 		/**
 		 * Fire action when pause db updates
 		 *
 		 * @since 2.0.1
 		 */
-		do_action( 'give_pause_db_upgrade', $this );
+		do_action('give_pause_db_upgrade', $this);
 
 		return true;
 	}
@@ -381,31 +379,31 @@ 
 		// Bailout.
 		if (
 			wp_doing_ajax() ||
-			! isset( $_GET['page'] ) ||
+			! isset($_GET['page']) ||
 			'give-updates' !== $_GET['page'] ||
-			! isset( $_GET['give-restart-db-upgrades'] ) ||
+			! isset($_GET['give-restart-db-upgrades']) ||
 			! self::$background_updater->is_paused_process()
 		) {
 			return false;
 		}
 
 		Give_Background_Updater::flush_cache();
-		$batch = get_option( 'give_paused_batches' );
+		$batch = get_option('give_paused_batches');
 
-		if ( ! empty( $batch ) ) {
-			wp_cache_delete( $batch->key, 'options' );
-			update_option( $batch->key, $batch->data );
+		if ( ! empty($batch)) {
+			wp_cache_delete($batch->key, 'options');
+			update_option($batch->key, $batch->data);
 
-			delete_option( 'give_paused_batches' );
+			delete_option('give_paused_batches');
 
-			Give()->logs->add( 'Update Restart', print_r( $batch, true ), 0, 'update' );
+			Give()->logs->add('Update Restart', print_r($batch, true), 0, 'update');
 
 
 			/** Fire action when restart db updates
 			 *
 			 * @since 2.0.1
 			 */
-			do_action( 'give_restart_db_upgrade', $this );
+			do_action('give_restart_db_upgrade', $this);
 
 			self::$background_updater->dispatch();
 		}
@@ -421,53 +419,53 @@ 
 	 *
 	 * @param Give_Updates $give_updates
 	 */
-	public function __health_background_update( $give_updates ) {
-		if ( ! $this->is_doing_updates() ) {
+	public function __health_background_update($give_updates) {
+		if ( ! $this->is_doing_updates()) {
 			return;
 		}
 
 		Give_Background_Updater::flush_cache();
 
 		$batch                = Give_Updates::$background_updater->get_all_batch();
-		$batch_data_count     = count( $batch->data );
-		$all_updates          = $give_updates->get_updates( 'database', 'all' );
-		$all_update_ids       = wp_list_pluck( $all_updates, 'id' );
-		$all_batch_update_ids = ! empty( $batch->data ) ? wp_list_pluck( $batch->data, 'id' ) : array();
+		$batch_data_count     = count($batch->data);
+		$all_updates          = $give_updates->get_updates('database', 'all');
+		$all_update_ids       = wp_list_pluck($all_updates, 'id');
+		$all_batch_update_ids = ! empty($batch->data) ? wp_list_pluck($batch->data, 'id') : array();
 		$log_data             = '';
-		$doing_upgrade_args   = get_option( 'give_doing_upgrade' );
+		$doing_upgrade_args   = get_option('give_doing_upgrade');
 
-		if ( ! empty( $doing_upgrade_args ) ) {
-			$log_data .= 'Doing update:' . "\n";
-			$log_data .= print_r( $doing_upgrade_args, true ) . "\n";
+		if ( ! empty($doing_upgrade_args)) {
+			$log_data .= 'Doing update:'."\n";
+			$log_data .= print_r($doing_upgrade_args, true)."\n";
 		}
 
 		/**
 		 * Add remove upgrade from batch
 		 */
-		if ( ! empty( $batch->data ) ) {
+		if ( ! empty($batch->data)) {
 
-			foreach ( $batch->data as $index => $update ) {
-				$log_data = print_r( $update, true ) . "\n";
+			foreach ($batch->data as $index => $update) {
+				$log_data = print_r($update, true)."\n";
 
-				if ( ! is_callable( $update['callback'] ) ) {
-					$log_data .= 'Removing missing callback update: ' . "{$update['id']}\n";
-					unset( $batch->data[ $index ] );
-				} elseif ( give_has_upgrade_completed( $update['id'] ) ) {
-					$log_data .= 'Removing already completed update: ' . "{$update['id']}\n";
-					unset( $batch->data[ $index ] );
+				if ( ! is_callable($update['callback'])) {
+					$log_data .= 'Removing missing callback update: '."{$update['id']}\n";
+					unset($batch->data[$index]);
+				} elseif (give_has_upgrade_completed($update['id'])) {
+					$log_data .= 'Removing already completed update: '."{$update['id']}\n";
+					unset($batch->data[$index]);
 				}
 
-				if ( ! empty( $update['depend'] ) ) {
+				if ( ! empty($update['depend'])) {
 
-					foreach ( $update['depend'] as $depend ) {
-						if ( give_has_upgrade_completed( $depend ) ) {
-							$log_data .= 'Completed update: ' . "{$depend}\n";
+					foreach ($update['depend'] as $depend) {
+						if (give_has_upgrade_completed($depend)) {
+							$log_data .= 'Completed update: '."{$depend}\n";
 							continue;
 						}
 
-						if ( in_array( $depend, $all_update_ids ) && ! in_array( $depend, $all_batch_update_ids ) ) {
-							$log_data .= 'Adding missing update: ' . "{$depend}\n";
-							array_unshift( $batch->data, $all_updates[ array_search( $depend, $all_update_ids ) ] );
+						if (in_array($depend, $all_update_ids) && ! in_array($depend, $all_batch_update_ids)) {
+							$log_data .= 'Adding missing update: '."{$depend}\n";
+							array_unshift($batch->data, $all_updates[array_search($depend, $all_update_ids)]);
 						}
 					}
 				}
@@ -477,52 +475,52 @@ 
 		/**
 		 * Add new upgrade to batch
 		 */
-		if ( $new_updates = $this->get_updates( 'database', 'new' ) ) {
-			$all_batch_update_ids = ! empty( $batch->data ) ? wp_list_pluck( $batch->data, 'id' ) : array();
+		if ($new_updates = $this->get_updates('database', 'new')) {
+			$all_batch_update_ids = ! empty($batch->data) ? wp_list_pluck($batch->data, 'id') : array();
 
-			foreach ( $new_updates as $index => $new_update ) {
-				if ( give_has_upgrade_completed( $new_update['id'] ) || in_array( $new_update['id'], $all_batch_update_ids ) ) {
-					unset( $new_updates[ $index ] );
+			foreach ($new_updates as $index => $new_update) {
+				if (give_has_upgrade_completed($new_update['id']) || in_array($new_update['id'], $all_batch_update_ids)) {
+					unset($new_updates[$index]);
 				}
 			}
 
-			if ( ! empty( $new_updates ) ) {
-				$log_data .= 'Adding new update: ' . "\n";
-				$log_data .= print_r( $new_updates, true ) . "\n";
+			if ( ! empty($new_updates)) {
+				$log_data .= 'Adding new update: '."\n";
+				$log_data .= print_r($new_updates, true)."\n";
 
-				$batch->data = array_merge( (array) $batch->data, $new_updates );
-				update_option( 'give_db_update_count', ( absint( get_option( 'give_db_update_count' ) ) + count( $new_updates ) ) );
+				$batch->data = array_merge((array) $batch->data, $new_updates);
+				update_option('give_db_update_count', (absint(get_option('give_db_update_count')) + count($new_updates)));
 			}
 		}
 
 		/**
 		 * Fix batch
 		 */
-		if ( empty( $batch->data ) ) {
+		if (empty($batch->data)) {
 			// Complete batch if do not have any data to process.
-			self::$background_updater->delete( $batch->key );
+			self::$background_updater->delete($batch->key);
 
-			if ( self::$background_updater->has_queue() ) {
-				$this->__health_background_update( $this );
+			if (self::$background_updater->has_queue()) {
+				$this->__health_background_update($this);
 			} else {
-				delete_site_transient( self::$background_updater->get_identifier() . '_process_lock' );
-				wp_clear_scheduled_hook( self::$background_updater->get_cron_identifier() );
+				delete_site_transient(self::$background_updater->get_identifier().'_process_lock');
+				wp_clear_scheduled_hook(self::$background_updater->get_cron_identifier());
 
 				self::$background_updater->complete();
 			}
 
-		} elseif ( $batch_data_count !== count( $batch->data ) ) {
+		} elseif ($batch_data_count !== count($batch->data)) {
 
-			$log_data .= 'Updating batch' . "\n";
-			$log_data .= print_r( $batch, true );
+			$log_data .= 'Updating batch'."\n";
+			$log_data .= print_r($batch, true);
 
-			if ( ! empty( $batch->key ) ) {
-				wp_cache_delete( $batch->key, 'options' );
-				update_option( $batch->key, $batch->data );
+			if ( ! empty($batch->key)) {
+				wp_cache_delete($batch->key, 'options');
+				update_option($batch->key, $batch->data);
 			} else {
 
-				foreach ( $batch->data as $data ) {
-					Give_Updates::$background_updater->push_to_queue( $data );
+				foreach ($batch->data as $data) {
+					Give_Updates::$background_updater->push_to_queue($data);
 				}
 
 				Give_Updates::$background_updater->save();
@@ -533,31 +531,31 @@ 
 		/**
 		 * Fix give_doing_upgrade option
 		 */
-		if( $fresh_new_db_count = $this->get_total_new_db_update_count( true ) ) {
-			update_option( 'give_db_update_count', $fresh_new_db_count );
+		if ($fresh_new_db_count = $this->get_total_new_db_update_count(true)) {
+			update_option('give_db_update_count', $fresh_new_db_count);
 		}
 
 		$doing_upgrade_args['update']           = 1;
-		$doing_upgrade_args['heading']          = sprintf( 'Update %s of %s', 1, $fresh_new_db_count );
-		$doing_upgrade_args['total_percentage'] = $this->get_db_update_processing_percentage( true );
+		$doing_upgrade_args['heading']          = sprintf('Update %s of %s', 1, $fresh_new_db_count);
+		$doing_upgrade_args['total_percentage'] = $this->get_db_update_processing_percentage(true);
 
 		// Remove already completed update from info.
 		if (
-			empty( $doing_upgrade_args['update_info'] )
-			|| give_has_upgrade_completed( $doing_upgrade_args['update_info']['id'] )
+			empty($doing_upgrade_args['update_info'])
+			|| give_has_upgrade_completed($doing_upgrade_args['update_info']['id'])
 		) {
-			$doing_upgrade_args['update_info'] = current( array_values( $batch->data ) );
+			$doing_upgrade_args['update_info'] = current(array_values($batch->data));
 			$doing_upgrade_args['step']        = 1;
 		}
 
 		// Check if dependency completed or not.
-		if ( isset( $doing_upgrade_args['update_info']['depend'] ) ) {
-			foreach ( $doing_upgrade_args['update_info']['depend'] as $depend ) {
-				if ( give_has_upgrade_completed( $depend ) ) {
+		if (isset($doing_upgrade_args['update_info']['depend'])) {
+			foreach ($doing_upgrade_args['update_info']['depend'] as $depend) {
+				if (give_has_upgrade_completed($depend)) {
 					continue;
 				}
 
-				$doing_upgrade_args['update_info']      = $all_updates[ array_search( $depend, $all_update_ids ) ];
+				$doing_upgrade_args['update_info']      = $all_updates[array_search($depend, $all_update_ids)];
 				$doing_upgrade_args['step']             = 1;
 				$doing_upgrade_args['percentage']       = 0;
 				$doing_upgrade_args['total_percentage'] = 0;
@@ -566,14 +564,14 @@ 
 			}
 		}
 
-		if( ! empty( $doing_upgrade_args['update_info'] ) ) {
-			update_option( 'give_doing_upgrade', $doing_upgrade_args );
+		if ( ! empty($doing_upgrade_args['update_info'])) {
+			update_option('give_doing_upgrade', $doing_upgrade_args);
 
-			$log_data .= 'Updated doing update:' . "\n";
-			$log_data .= print_r( $doing_upgrade_args, true ) . "\n";
+			$log_data .= 'Updated doing update:'."\n";
+			$log_data .= print_r($doing_upgrade_args, true)."\n";
 		}
 
-		Give()->logs->add( 'Update Health Check', $log_data, 0, 'update' );
+		Give()->logs->add('Update Health Check', $log_data, 0, 'update');
 	}
 
 
@@ -587,61 +585,61 @@ 
 		$current_screen = get_current_screen();
 
 		// Bailout.
-		if ( ! current_user_can( 'manage_give_settings' ) ) {
+		if ( ! current_user_can('manage_give_settings')) {
 			return;
 		}
 
 		// Run DB updates.
-		if ( ! empty( $_GET['give-run-db-update'] ) ) {
+		if ( ! empty($_GET['give-run-db-update'])) {
 			$this->run_db_update();
 		}
 
 
 		// Bailout.
-		if ( in_array( $current_screen->base, array( 'give_forms_page_give-updates', 'update-core' ) ) ) {
+		if (in_array($current_screen->base, array('give_forms_page_give-updates', 'update-core'))) {
 			return;
 		}
 
 		// Show notice if upgrade paused.
-		if ( self::$background_updater->is_paused_process() ) {
+		if (self::$background_updater->is_paused_process()) {
 			ob_start();
 
-			$upgrade_error = get_option( 'give_upgrade_error' );
-			if ( ! $upgrade_error ) : ?>
-				<strong><?php _e( 'Database Update', 'give' ); ?></strong>
-				&nbsp;&#8211;&nbsp;<?php _e( 'GiveWP needs to update your database to the latest version. The following process will make updates to your site\'s database. Please create a backup before proceeding.', 'give' ); ?>
+			$upgrade_error = get_option('give_upgrade_error');
+			if ( ! $upgrade_error) : ?>
+				<strong><?php _e('Database Update', 'give'); ?></strong>
+				&nbsp;&#8211;&nbsp;<?php _e('GiveWP needs to update your database to the latest version. The following process will make updates to your site\'s database. Please create a backup before proceeding.', 'give'); ?>
 				<br>
 				<br>
-				<a href="<?php echo esc_url( add_query_arg( array( 'give-restart-db-upgrades' => 1 ), admin_url( 'edit.php?post_type=give_forms&page=give-updates' ) ) ); ?>" class="button button-primary give-restart-updater-btn">
-					<?php _e( 'Restart the updater', 'give' ); ?>
+				<a href="<?php echo esc_url(add_query_arg(array('give-restart-db-upgrades' => 1), admin_url('edit.php?post_type=give_forms&page=give-updates'))); ?>" class="button button-primary give-restart-updater-btn">
+					<?php _e('Restart the updater', 'give'); ?>
 				</a>
 			<?php else: ?>
-				<strong><?php _e( 'Database Update', 'give' ); ?></strong>
-				&nbsp;&#8211;&nbsp;<?php _e( 'An unexpected issue occurred during the database update which caused it to stop automatically. Please contact support for assistance.', 'give' ); ?>
+				<strong><?php _e('Database Update', 'give'); ?></strong>
+				&nbsp;&#8211;&nbsp;<?php _e('An unexpected issue occurred during the database update which caused it to stop automatically. Please contact support for assistance.', 'give'); ?>
 			<?php
 			endif;
 			$desc_html = ob_get_clean();
 
-			Give()->notices->register_notice( array(
+			Give()->notices->register_notice(array(
 				'id'          => 'give_upgrade_db',
 				'type'        => 'error',
 				'dismissible' => false,
 				'description' => $desc_html,
-			) );
+			));
 		}
 
 		// Bailout if doing upgrades.
-		if ( $this->is_doing_updates() ) {
+		if ($this->is_doing_updates()) {
 			return;
 		}
 
 		// Show notice if ajax is not working.
-		if ( ! give_test_ajax_works() ) {
+		if ( ! give_test_ajax_works()) {
 			Give()->notices->register_notice(
 				array(
 					'id'          => 'give_db_upgrade_ajax_inaccessible',
 					'type'        => 'error',
-					'description' => sprintf( '%1$s <a href="%2$s" target="_blank">%3$s</a>', __( 'Give needs to upgrade the database but cannot because AJAX does not appear accessible. This could be because your website is password protected, in maintenance mode, or has a specific hosting configuration or plugin active that is preventing access.', 'give' ), 'http://docs.givewp.com/admin-ajax-error', __( 'Read More', 'give' ) . ' &raquo;' ),
+					'description' => sprintf('%1$s <a href="%2$s" target="_blank">%3$s</a>', __('Give needs to upgrade the database but cannot because AJAX does not appear accessible. This could be because your website is password protected, in maintenance mode, or has a specific hosting configuration or plugin active that is preventing access.', 'give'), 'http://docs.givewp.com/admin-ajax-error', __('Read More', 'give').' &raquo;'),
 					'show'        => true,
 				)
 			);
@@ -650,40 +648,40 @@ 
 		}
 
 		// Show db upgrade completed notice.
-		if ( ! empty( $_GET['give-db-update-completed'] ) ) {
-			Give()->notices->register_notice( array(
+		if ( ! empty($_GET['give-db-update-completed'])) {
+			Give()->notices->register_notice(array(
 				'id'          => 'give_db_upgrade_completed',
 				'type'        => 'updated',
-				'description' => __( 'Give database updates completed successfully. Thank you for updating to the latest version!', 'give' ),
+				'description' => __('Give database updates completed successfully. Thank you for updating to the latest version!', 'give'),
 				'show'        => true,
-			) );
+			));
 
 			// Start update.
-		} elseif ( ! empty( $_GET['give-run-db-update'] ) ) {
+		} elseif ( ! empty($_GET['give-run-db-update'])) {
 			$this->run_db_update();
 
 			// Show run the update notice.
-		} elseif ( $this->get_total_new_db_update_count() ) {
+		} elseif ($this->get_total_new_db_update_count()) {
 			ob_start();
 			?>
 			<p>
-				<strong><?php _e( 'Database Update', 'give' ); ?></strong>
-				&nbsp;&#8211;&nbsp;<?php _e( 'GiveWP needs to update your database to the latest version. The following process will make updates to your site\'s database. Please create a complete backup before proceeding.', 'give' ); ?>
+				<strong><?php _e('Database Update', 'give'); ?></strong>
+				&nbsp;&#8211;&nbsp;<?php _e('GiveWP needs to update your database to the latest version. The following process will make updates to your site\'s database. Please create a complete backup before proceeding.', 'give'); ?>
 			</p>
 			<p class="submit">
-				<a href="<?php echo esc_url( add_query_arg( array( 'give-run-db-update' => 1 ), admin_url( 'edit.php?post_type=give_forms&page=give-updates' ) ) ); ?>" class="button button-primary give-run-update-now">
-					<?php _e( 'Run the updater', 'give' ); ?>
+				<a href="<?php echo esc_url(add_query_arg(array('give-run-db-update' => 1), admin_url('edit.php?post_type=give_forms&page=give-updates'))); ?>" class="button button-primary give-run-update-now">
+					<?php _e('Run the updater', 'give'); ?>
 				</a>
 			</p>
 			<?php
 			$desc_html = ob_get_clean();
 
-			Give()->notices->register_notice( array(
+			Give()->notices->register_notice(array(
 				'id'          => 'give_upgrade_db',
 				'type'        => 'updated',
 				'dismissible' => false,
 				'description' => $desc_html,
-			) );
+			));
 		}
 	}
 
@@ -694,7 +692,7 @@ 
 	 * @access public
 	 */
 	public function render_complete_page() {
-		include_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/views/upgrades-complete.php';
+		include_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/views/upgrades-complete.php';
 	}
 
 	/**
@@ -704,7 +702,7 @@ 
 	 * @access public
 	 */
 	public function render_page() {
-		include_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/views/upgrades.php';
+		include_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/views/upgrades.php';
 	}
 
 	/**
@@ -715,26 +713,26 @@ 
 	 */
 	private function run_db_update() {
 		// Bailout.
-		if ( $this->is_doing_updates() || ! $this->get_total_new_db_update_count() ) {
+		if ($this->is_doing_updates() || ! $this->get_total_new_db_update_count()) {
 			return;
 		}
 
-		$updates = $this->get_updates( 'database', 'new' );
+		$updates = $this->get_updates('database', 'new');
 
-		foreach ( $updates as $update ) {
-			self::$background_updater->push_to_queue( $update );
+		foreach ($updates as $update) {
+			self::$background_updater->push_to_queue($update);
 		}
 
-		add_option( 'give_db_update_count', count( $updates ), '', 'no' );
+		add_option('give_db_update_count', count($updates), '', 'no');
 
-		add_option( 'give_doing_upgrade', array(
+		add_option('give_doing_upgrade', array(
 			'update_info'      => $updates[0],
 			'step'             => 1,
 			'update'           => 1,
-			'heading'          => sprintf( 'Update %s of %s', 1, count( $updates ) ),
+			'heading'          => sprintf('Update %s of %s', 1, count($updates)),
 			'percentage'       => 0,
 			'total_percentage' => 0,
-		), '', 'no' );
+		), '', 'no');
 
 		self::$background_updater->save()->dispatch();
 	}
@@ -748,14 +746,13 @@ 
 	 */
 	public function __flush_resume_updates() {
 		//delete_option( 'give_doing_upgrade' );
-		update_option( 'give_version', preg_replace( '/[^0-9.].*/', '', GIVE_VERSION ) );
+		update_option('give_version', preg_replace('/[^0-9.].*/', '', GIVE_VERSION));
 
 		// Reset counter.
 		$this->step = $this->percentage = 0;
 
-		$this->update = ( $this->get_total_db_update_count() > $this->update ) ?
-			( $this->update + 1 ) :
-			$this->update;
+		$this->update = ($this->get_total_db_update_count() > $this->update) ?
+			($this->update + 1) : $this->update;
 	}
 
 
@@ -770,7 +767,7 @@ 
 	public function __give_start_updating() {
 		// Check permission.
 		if (
-			! current_user_can( 'manage_give_settings' ) ||
+			! current_user_can('manage_give_settings') ||
 			$this->is_doing_updates()
 		) {
 			wp_send_json_error();
@@ -778,7 +775,7 @@ 
 
 		// @todo: validate nonce
 		// @todo: set http method to post
-		if ( empty( $_POST['run_db_update'] ) ) {
+		if (empty($_POST['run_db_update'])) {
 			wp_send_json_error();
 		}
 
@@ -797,34 +794,34 @@ 
 	 * @return string
 	 */
 	public function __give_db_updates_info() {
-		$update_info   = get_option( 'give_doing_upgrade' );
+		$update_info   = get_option('give_doing_upgrade');
 		$response_type = '';
 
-		if ( self::$background_updater->is_paused_process() ) {
+		if (self::$background_updater->is_paused_process()) {
 			$update_info = array(
-				'message'    => __( 'The updates have been paused.', 'give' ),
+				'message'    => __('The updates have been paused.', 'give'),
 				'heading'    => '',
 				'percentage' => 0,
 			);
 
-			if ( get_option( 'give_upgrade_error' ) ) {
-				$update_info['message'] = __( 'An unexpected issue occurred during the database update which caused it to stop automatically. Please contact support for assistance.', 'give' );
+			if (get_option('give_upgrade_error')) {
+				$update_info['message'] = __('An unexpected issue occurred during the database update which caused it to stop automatically. Please contact support for assistance.', 'give');
 			}
 
 			$response_type = 'error';
 
-		} elseif ( empty( $update_info ) || ! $this->get_total_new_db_update_count( true ) ) {
+		} elseif (empty($update_info) || ! $this->get_total_new_db_update_count(true)) {
 			$update_info   = array(
-				'message'    => __( 'Give database updates completed successfully. Thank you for updating to the latest version!', 'give' ),
-				'heading'    => __( 'Updates Completed.', 'give' ),
+				'message'    => __('Give database updates completed successfully. Thank you for updating to the latest version!', 'give'),
+				'heading'    => __('Updates Completed.', 'give'),
 				'percentage' => 0,
 			);
 			$response_type = 'success';
 
-			delete_option( 'give_show_db_upgrade_complete_notice' );
+			delete_option('give_show_db_upgrade_complete_notice');
 		}
 
-		$this->send_ajax_response( $update_info, $response_type );
+		$this->send_ajax_response($update_info, $response_type);
 	}
 
 	/**
@@ -836,7 +833,7 @@ 
 	 * @param        $data
 	 * @param string $type
 	 */
-	public function send_ajax_response( $data, $type = '' ) {
+	public function send_ajax_response($data, $type = '') {
 		$default = array(
 			'message'    => '',
 			'heading'    => '',
@@ -846,24 +843,24 @@ 
 		);
 
 		// Set data.
-		$data = wp_parse_args( $data, $default );
+		$data = wp_parse_args($data, $default);
 
 		// Enable cache.
 		Give_Cache::enable();
 
-		switch ( $type ) {
+		switch ($type) {
 			case 'success':
-				wp_send_json_success( $data );
+				wp_send_json_success($data);
 				break;
 
 			case 'error':
-				wp_send_json_error( $data );
+				wp_send_json_error($data);
 				break;
 
 			default:
-				wp_send_json( array(
+				wp_send_json(array(
 					'data' => $data,
-				) );
+				));
 				break;
 		}
 	}
@@ -877,12 +874,12 @@ 
 	 * @param $total
 	 * @param $current_total
 	 */
-	public function set_percentage( $total, $current_total ) {
+	public function set_percentage($total, $current_total) {
 		// Set percentage.
-		$this->percentage = $total ? ( ( $current_total ) / $total ) * 100 : 0;
+		$this->percentage = $total ? (($current_total) / $total) * 100 : 0;
 
 		// Verify percentage.
-		$this->percentage = ( 100 < $this->percentage ) ? 100 : $this->percentage;
+		$this->percentage = (100 < $this->percentage) ? 100 : $this->percentage;
 	}
 
 	/**
@@ -895,22 +892,22 @@ 
 	 *
 	 * @return bool|null
 	 */
-	public function is_parent_updates_completed( $update ) {
+	public function is_parent_updates_completed($update) {
 		// Bailout.
-		if ( empty( $update['depend'] ) ) {
+		if (empty($update['depend'])) {
 			return true;
 		}
 
 		// Check if dependency is valid or not.
-		if ( ! $this->has_valid_dependency( $update ) ) {
+		if ( ! $this->has_valid_dependency($update)) {
 			return null;
 		}
 
 		$is_dependency_completed = true;
 
-		foreach ( $update['depend'] as $depend ) {
+		foreach ($update['depend'] as $depend) {
 
-			if ( ! give_has_upgrade_completed( $depend ) ) {
+			if ( ! give_has_upgrade_completed($depend)) {
 				$is_dependency_completed = false;
 				break;
 			}
@@ -927,7 +924,7 @@ 
 	 * @return bool
 	 */
 	public function is_doing_updates() {
-		return (bool) get_option( 'give_doing_upgrade' );
+		return (bool) get_option('give_doing_upgrade');
 	}
 
 
@@ -941,7 +938,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public function has_valid_dependency( $update ) {
+	public function has_valid_dependency($update) {
 		$is_valid_dependency = true;
 		// $update_ids          = wp_list_pluck( $this->get_updates( 'database', 'all' ), 'id' );
 		//
@@ -967,33 +964,33 @@ 
 	 *
 	 * @return array
 	 */
-	public function get_updates( $update_type = '', $status = 'all' ) {
+	public function get_updates($update_type = '', $status = 'all') {
 		// return all updates.
-		if ( empty( $update_type ) ) {
+		if (empty($update_type)) {
 			return $this->updates;
 		}
 
 		// Get specific update.
-		$updates = ! empty( $this->updates[ $update_type ] ) ? $this->updates[ $update_type ] : array();
+		$updates = ! empty($this->updates[$update_type]) ? $this->updates[$update_type] : array();
 
 		// Bailout.
-		if ( empty( $updates ) ) {
+		if (empty($updates)) {
 			return $updates;
 		}
 
-		switch ( $status ) {
+		switch ($status) {
 			case 'new':
 				// Remove already completed updates.
-				wp_cache_delete( 'give_completed_upgrades', 'options' );
+				wp_cache_delete('give_completed_upgrades', 'options');
 				$completed_updates = give_get_completed_upgrades();
 
-				if ( ! empty( $completed_updates ) ) {
-					foreach ( $updates as $index => $update ) {
-						if ( in_array( $update['id'], $completed_updates ) ) {
-							unset( $updates[ $index ] );
+				if ( ! empty($completed_updates)) {
+					foreach ($updates as $index => $update) {
+						if (in_array($update['id'], $completed_updates)) {
+							unset($updates[$index]);
 						}
 					}
-					$updates = array_values( $updates );
+					$updates = array_values($updates);
 				}
 
 				break;
@@ -1010,7 +1007,7 @@ 
 	 * @return int
 	 */
 	public function get_total_plugin_update_count() {
-		return count( $this->get_updates( 'plugin' ) );
+		return count($this->get_updates('plugin'));
 	}
 
 	/**
@@ -1025,7 +1022,7 @@ 
 		$db_update_count     = $this->get_pending_db_update_count();
 		$plugin_update_count = $this->get_total_plugin_update_count();
 
-		return ( $db_update_count + $plugin_update_count );
+		return ($db_update_count + $plugin_update_count);
 	}
 
 	/**
@@ -1037,7 +1034,7 @@ 
 	 * @return int
 	 */
 	public function get_pending_db_update_count() {
-		return count( $this->get_updates( 'database', 'new' ) );
+		return count($this->get_updates('database', 'new'));
 	}
 
 	/**
@@ -1049,7 +1046,7 @@ 
 	 * @return int
 	 */
 	public function get_total_db_update_count() {
-		return count( $this->get_updates( 'database', 'all' ) );
+		return count($this->get_updates('database', 'all'));
 	}
 
 	/**
@@ -1062,10 +1059,9 @@ 
 	 *
 	 * @return int
 	 */
-	public function get_total_new_db_update_count( $refresh = false ) {
+	public function get_total_new_db_update_count($refresh = false) {
 		$update_count = $this->is_doing_updates() && ! $refresh ?
-			get_option( 'give_db_update_count' ) :
-			$this->get_pending_db_update_count();
+			get_option('give_db_update_count') : $this->get_pending_db_update_count();
 
 		return $update_count;
 	}
@@ -1080,11 +1076,11 @@ 
 	 *
 	 * @return int
 	 */
-	public function get_running_db_update( $refresh = false ) {
+	public function get_running_db_update($refresh = false) {
 		$current_update = 1;
 
-		if ( $this->is_doing_updates() && ! $refresh ) {
-			$current_update = get_option( 'give_doing_upgrade' );
+		if ($this->is_doing_updates() && ! $refresh) {
+			$current_update = get_option('give_doing_upgrade');
 			$current_update = $current_update['update'];
 		}
 
@@ -1101,25 +1097,23 @@ 
 	 *
 	 * @return float|int
 	 */
-	public function get_db_update_processing_percentage( $refresh = false ) {
+	public function get_db_update_processing_percentage($refresh = false) {
 		// Bailout.
-		if ( ! $this->get_total_new_db_update_count( $refresh ) ) {
+		if ( ! $this->get_total_new_db_update_count($refresh)) {
 			return 0;
 		}
 
-		$resume_update            = get_option( 'give_doing_upgrade' );
-		$update_count_percentages = ( ( $this->get_running_db_update( $refresh ) - 1 ) / $this->get_total_new_db_update_count( $refresh ) ) * 100;
-		$update_percentage_share  = ( 1 / $this->get_total_new_db_update_count() ) * 100;
-		$upgrade_percentage       = ( ( $resume_update['percentage'] * $update_percentage_share ) / 100 );
+		$resume_update            = get_option('give_doing_upgrade');
+		$update_count_percentages = (($this->get_running_db_update($refresh) - 1) / $this->get_total_new_db_update_count($refresh)) * 100;
+		$update_percentage_share  = (1 / $this->get_total_new_db_update_count()) * 100;
+		$upgrade_percentage       = (($resume_update['percentage'] * $update_percentage_share) / 100);
 
 		$final_percentage = $update_count_percentages + $upgrade_percentage;
 
 		return $this->is_doing_updates() ?
-			( absint( $final_percentage ) ?
-				absint( $final_percentage ) :
-				round( $final_percentage, 2 )
-			) :
-			0;
+			(absint($final_percentage) ?
+				absint($final_percentage) : round($final_percentage, 2)
+			) : 0;
 	}
 
 
@@ -1131,8 +1125,8 @@ 
 	 * @return array
 	 */
 	public function get_update_ids() {
-		$all_updates    = $this->get_updates( 'database', 'all' );
-		$all_update_ids = wp_list_pluck( $all_updates, 'id' );
+		$all_updates    = $this->get_updates('database', 'all');
+		$all_update_ids = wp_list_pluck($all_updates, 'id');
 
 		return $all_update_ids;
 	}
@@ -1147,10 +1141,9 @@ 
 	 *
 	 * @return float|int
 	 */
-	public function get_offset( $process_item_count ) {
-		return ( 1 === $this->step ) ?
-			0 :
-			( $this->step - 1 ) * $process_item_count;
+	public function get_offset($process_item_count) {
+		return (1 === $this->step) ?
+			0 : ($this->step - 1) * $process_item_count;
 	}
 }
 

includes/admin/admin-footer.php

Spacing +6 added lines, -6 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,14 +24,14 @@ 
  *
  * @return      string
  */
-function give_admin_rate_us( $footer_text ) {
+function give_admin_rate_us($footer_text) {
 	global $typenow;
 
-	if ( 'give_forms' === $typenow ) {
+	if ('give_forms' === $typenow) {
 		$rate_text = sprintf(
 			/* translators: %s: Link to 5 star rating */
-			__( 'If you like <strong>Give</strong> please leave us a %s rating. It takes a minute and helps a lot. Thanks in advance!', 'give' ),
-			'<a href="https://wordpress.org/support/view/plugin-reviews/give?filter=5#postform" target="_blank" class="give-rating-link" style="text-decoration:none;" data-rated="' . esc_attr__( 'Thanks :)', 'give' ) . '">&#9733;&#9733;&#9733;&#9733;&#9733;</a>'
+			__('If you like <strong>Give</strong> please leave us a %s rating. It takes a minute and helps a lot. Thanks in advance!', 'give'),
+			'<a href="https://wordpress.org/support/view/plugin-reviews/give?filter=5#postform" target="_blank" class="give-rating-link" style="text-decoration:none;" data-rated="'.esc_attr__('Thanks :)', 'give').'">&#9733;&#9733;&#9733;&#9733;&#9733;</a>'
 		);
 
 		return $rate_text;
@@ -40,4 +40,4 @@ 
 	}
 }
 
-add_filter( 'admin_footer_text', 'give_admin_rate_us' );
+add_filter('admin_footer_text', 'give_admin_rate_us');

includes/admin/admin-pages.php

Spacing +81 added lines, -81 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -32,7 +32,7 @@ 
 	global $give_settings_page, $give_payments_page, $give_reports_page, $give_add_ons_page, $give_donors_page, $give_tools_page;
 
 	//Payments
-	$give_payment       = get_post_type_object( 'give_payment' );
+	$give_payment       = get_post_type_object('give_payment');
 	$give_payments_page = add_submenu_page(
 		'edit.php?post_type=give_forms',
 		$give_payment->labels->name,
@@ -45,8 +45,8 @@ 
 	//Donors
 	$give_donors_page = add_submenu_page(
 		'edit.php?post_type=give_forms',
-		esc_html__( 'Donors', 'give' ),
-		esc_html__( 'Donors', 'give' ),
+		esc_html__('Donors', 'give'),
+		esc_html__('Donors', 'give'),
 		'view_give_reports',
 		'give-donors',
 		'give_donors_page'
@@ -55,8 +55,8 @@ 
 	//Reports`
 	$give_reports_page = add_submenu_page(
 		'edit.php?post_type=give_forms',
-		esc_html__( 'Donation Reports', 'give' ),
-		esc_html__( 'Reports', 'give' ),
+		esc_html__('Donation Reports', 'give'),
+		esc_html__('Reports', 'give'),
 		'view_give_reports',
 		'give-reports',
 		array(
@@ -68,8 +68,8 @@ 
 	//Settings
 	$give_settings_page = add_submenu_page(
 		'edit.php?post_type=give_forms',
-		esc_html__( 'Give Settings', 'give' ),
-		esc_html__( 'Settings', 'give' ),
+		esc_html__('Give Settings', 'give'),
+		esc_html__('Settings', 'give'),
 		'manage_give_settings',
 		'give-settings',
 		array(
@@ -81,8 +81,8 @@ 
 	//Tools.
 	$give_tools_page = add_submenu_page(
 		'edit.php?post_type=give_forms',
-		esc_html__( 'Give Tools', 'give' ),
-		esc_html__( 'Tools', 'give' ),
+		esc_html__('Give Tools', 'give'),
+		esc_html__('Tools', 'give'),
 		'manage_give_settings',
 		'give-tools',
 		array(
@@ -94,15 +94,15 @@ 
 	//Add-ons
 	$give_add_ons_page = add_submenu_page(
 		'edit.php?post_type=give_forms',
-		esc_html__( 'Give Add-ons', 'give' ),
-		esc_html__( 'Add-ons', 'give' ),
+		esc_html__('Give Add-ons', 'give'),
+		esc_html__('Add-ons', 'give'),
 		'install_plugins',
 		'give-addons',
 		'give_add_ons_page'
 	);
 }
 
-add_action( 'admin_menu', 'give_add_options_links', 10 );
+add_action('admin_menu', 'give_add_options_links', 10);
 
 /**
  *  Determines whether the current admin page is a Give admin page.
@@ -118,28 +118,28 @@ 
  *
  * @return bool True if Give admin page.
  */
-function give_is_admin_page( $passed_page = '', $passed_view = '' ) {
+function give_is_admin_page($passed_page = '', $passed_view = '') {
 	global $pagenow, $typenow;
 
 	$found          = true;
-	$get_query_args = ! empty( $_GET ) ? @array_map( 'strtolower', $_GET ) : array();
+	$get_query_args = ! empty($_GET) ? @array_map('strtolower', $_GET) : array();
 
 	// Set default argument, if not passed.
-	$query_args = wp_parse_args( $get_query_args, array_fill_keys( array( 'post_type', 'action', 'taxonomy', 'page', 'view', 'tab' ), false ) );
+	$query_args = wp_parse_args($get_query_args, array_fill_keys(array('post_type', 'action', 'taxonomy', 'page', 'view', 'tab'), false));
 
-	switch ( $passed_page ) {
+	switch ($passed_page) {
 		case 'categories':
 		case 'tags':
-			$has_view = in_array( $passed_view, array( 'list-table', 'edit', 'new' ), true );
+			$has_view = in_array($passed_view, array('list-table', 'edit', 'new'), true);
 
 			if (
-				! in_array( $query_args['taxonomy'], array( 'give_forms_category', 'give_forms_tag' ), true ) &&
+				! in_array($query_args['taxonomy'], array('give_forms_category', 'give_forms_tag'), true) &&
 				'edit-tags.php' !== $pagenow &&
 				(
 					$has_view ||
 					(
-						( in_array( $passed_view, array( 'list-table', 'new' ), true ) && 'edit' === $query_args['action'] ) ||
-						( 'edit' !== $passed_view && 'edit' !== $query_args['action'] ) &&
+						(in_array($passed_view, array('list-table', 'new'), true) && 'edit' === $query_args['action']) ||
+						('edit' !== $passed_view && 'edit' !== $query_args['action']) &&
 						! $has_view
 					)
 				)
@@ -149,18 +149,18 @@ 
 			break;
 		// Give Donation form page.
 		case 'give_forms':
-			$has_view = in_array( $passed_view, array( 'new', 'list-table', 'edit' ), true );
+			$has_view = in_array($passed_view, array('new', 'list-table', 'edit'), true);
 
 			if (
 				'give_forms' !== $typenow &&
 				(
-					( 'list-table' !== $passed_view && 'edit.php' !== $pagenow ) &&
-					( 'edit' !== $passed_view && 'post.php' !== $pagenow ) &&
-					( 'new' !== $passed_view && 'post-new.php' !== $pagenow )
+					('list-table' !== $passed_view && 'edit.php' !== $pagenow) &&
+					('edit' !== $passed_view && 'post.php' !== $pagenow) &&
+					('new' !== $passed_view && 'post-new.php' !== $pagenow)
 				) ||
 				(
 					! $has_view &&
-					( 'post-new.php' !== $pagenow && 'give_forms' !== $query_args['post_type'] )
+					('post-new.php' !== $pagenow && 'give_forms' !== $query_args['post_type'])
 				)
 			) {
 				$found = false;
@@ -168,13 +168,13 @@ 
 			break;
 		// Give Donors page.
 		case 'donors':
-			$has_view = array_intersect( array( $passed_view, $query_args['view'] ), array( 'list-table', 'overview', 'notes' ) );
+			$has_view = array_intersect(array($passed_view, $query_args['view']), array('list-table', 'overview', 'notes'));
 
 			if (
-				( 'give-donors' !== $query_args['page'] || 'edit.php' !== $pagenow ) &&
+				('give-donors' !== $query_args['page'] || 'edit.php' !== $pagenow) &&
 				(
-					( $passed_view !== $query_args['view'] || ! empty( $has_view ) ) ||
-					( false !== $query_args['view'] && 'list-table' !== $passed_view )
+					($passed_view !== $query_args['view'] || ! empty($has_view)) ||
+					(false !== $query_args['view'] && 'list-table' !== $passed_view)
 				)
 			) {
 				$found = false;
@@ -183,12 +183,12 @@ 
 		// Give Donations page.
 		case 'payments':
 			if (
-				( 'give-payment-history' !== $query_args['page'] || 'edit.php' !== $pagenow ) &&
+				('give-payment-history' !== $query_args['page'] || 'edit.php' !== $pagenow) &&
 				(
-					! in_array( $passed_view, array( 'list-table', 'edit' ), true ) ||
+					! in_array($passed_view, array('list-table', 'edit'), true) ||
 					(
-						( 'list-table' !== $passed_view && false !== $query_args['view'] ) ||
-						( 'edit' !== $passed_view && 'view-payment-details' !== $query_args['view'] )
+						('list-table' !== $passed_view && false !== $query_args['view']) ||
+						('edit' !== $passed_view && 'view-payment-details' !== $query_args['view'])
 					)
 				)
 			) {
@@ -199,20 +199,20 @@ 
 		case 'settings':
 		case 'addons':
 			// Get current tab.
-			$current_tab       = empty( $passed_view ) ? $query_args['tab'] : $passed_view;
-			$give_setting_page = in_array( $query_args['page'], array( 'give-reports', 'give-settings', 'give-addons' ), true );
+			$current_tab       = empty($passed_view) ? $query_args['tab'] : $passed_view;
+			$give_setting_page = in_array($query_args['page'], array('give-reports', 'give-settings', 'give-addons'), true);
 
 			// Check if it's Give Setting page or not.
 			if (
-				( 'edit.php' !== $pagenow || ! $give_setting_page ) &&
-				! Give_Admin_Settings::is_setting_page( $current_tab )
+				('edit.php' !== $pagenow || ! $give_setting_page) &&
+				! Give_Admin_Settings::is_setting_page($current_tab)
 			) {
 				$found = false;
 			}
 			break;
 		default:
 			global $give_payments_page, $give_settings_page, $give_reports_page, $give_system_info_page, $give_add_ons_page, $give_settings_export, $give_donors_page, $give_tools_page;
-			$admin_pages = apply_filters( 'give_admin_pages', array(
+			$admin_pages = apply_filters('give_admin_pages', array(
 				$give_payments_page,
 				$give_settings_page,
 				$give_reports_page,
@@ -222,11 +222,11 @@ 
 				$give_donors_page,
 				$give_tools_page,
 				'widgets.php',
-			) );
+			));
 
-			$found = ( 'give_forms' === $typenow || in_array( $pagenow, array_merge( $admin_pages, array( 'index.php', 'post-new.php', 'post.php' ) ), true ) ) ? true : false;
+			$found = ('give_forms' === $typenow || in_array($pagenow, array_merge($admin_pages, array('index.php', 'post-new.php', 'post.php')), true)) ? true : false;
 	}
-	return (bool) apply_filters( 'give_is_admin_page', $found, $query_args['page'], $query_args['view'], $passed_page, $passed_view );
+	return (bool) apply_filters('give_is_admin_page', $found, $query_args['page'], $query_args['view'], $passed_page, $passed_view);
 }
 
 /**
@@ -236,37 +236,37 @@ 
  * @param  array $settings
  * @return array
  */
-function give_settings_page_pages( $settings ) {
-	include( 'abstract-admin-settings-page.php' );
-	include( 'settings/class-settings-cmb2-backward-compatibility.php' );
+function give_settings_page_pages($settings) {
+	include('abstract-admin-settings-page.php');
+	include('settings/class-settings-cmb2-backward-compatibility.php');
 
 	$settings = array(
 		// General settings.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/settings/class-settings-general.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/settings/class-settings-general.php'),
 
 		// Payment Gateways Settings.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/settings/class-settings-gateways.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/settings/class-settings-gateways.php'),
 
 		// Display settings.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/settings/class-settings-display.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/settings/class-settings-display.php'),
 
 		// Emails settings.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/settings/class-settings-email.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/settings/class-settings-email.php'),
 
 		// Addons settings.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/settings/class-settings-addon.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/settings/class-settings-addon.php'),
 
 		// License settings.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/settings/class-settings-license.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/settings/class-settings-license.php'),
 
 		// Advanced settings.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/settings/class-settings-advanced.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/settings/class-settings-advanced.php'),
 	);
 
 	// Output.
 	return $settings;
 }
-add_filter( 'give-settings_get_settings_pages', 'give_settings_page_pages', 0, 1 );
+add_filter('give-settings_get_settings_pages', 'give_settings_page_pages', 0, 1);
 
 
 /**
@@ -276,25 +276,25 @@ 
  * @param  array $settings
  * @return array
  */
-function give_reports_page_pages( $settings ) {
-	include( 'abstract-admin-settings-page.php' );
+function give_reports_page_pages($settings) {
+	include('abstract-admin-settings-page.php');
 
 	$settings = array(
 		// Earnings.
-		include( 'reports/class-earnings-report.php' ),
+		include('reports/class-earnings-report.php'),
 
 		// Forms.
-		include( 'reports/class-forms-report.php' ),
+		include('reports/class-forms-report.php'),
 
 		// Gateways.
-		include( 'reports/class-gateways-report.php' ),
+		include('reports/class-gateways-report.php'),
 
 	);
 
 	// Output.
 	return $settings;
 }
-add_filter( 'give-reports_get_settings_pages', 'give_reports_page_pages', 0, 1 );
+add_filter('give-reports_get_settings_pages', 'give_reports_page_pages', 0, 1);
 
 /**
  * Add setting tab to give-settings page
@@ -303,34 +303,34 @@ 
  * @param  array $settings
  * @return array
  */
-function give_tools_page_pages( $settings ) {
-	include( 'abstract-admin-settings-page.php' );
+function give_tools_page_pages($settings) {
+	include('abstract-admin-settings-page.php');
 
 	$settings = array(
 
 		// Export.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/tools/class-settings-export.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/tools/class-settings-export.php'),
 
 		// Import
-		include_once( GIVE_PLUGIN_DIR . 'includes/admin/tools/class-settings-import.php' ),
+		include_once(GIVE_PLUGIN_DIR.'includes/admin/tools/class-settings-import.php'),
 
 		// Logs.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/tools/class-settings-logs.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/tools/class-settings-logs.php'),
 
 		// API.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/tools/class-settings-api.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/tools/class-settings-api.php'),
 
 		// Data.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/tools/class-settings-data.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/tools/class-settings-data.php'),
 
 		// System Info.
-		include( GIVE_PLUGIN_DIR . 'includes/admin/tools/class-settings-system-info.php' ),
+		include(GIVE_PLUGIN_DIR.'includes/admin/tools/class-settings-system-info.php'),
 	);
 
 	// Output.
 	return $settings;
 }
-add_filter( 'give-tools_get_settings_pages', 'give_tools_page_pages', 0, 1 );
+add_filter('give-tools_get_settings_pages', 'give_tools_page_pages', 0, 1);
 
 /**
  * Set default tools page tab.
@@ -339,10 +339,10 @@ 
  * @param  string $default_tab Default tab name.
  * @return string
  */
-function give_set_default_tab_form_tools_page( $default_tab ) {
+function give_set_default_tab_form_tools_page($default_tab) {
 	return 'export';
 }
-add_filter( 'give_default_setting_tab_give-tools', 'give_set_default_tab_form_tools_page', 10, 1 );
+add_filter('give_default_setting_tab_give-tools', 'give_set_default_tab_form_tools_page', 10, 1);
 
 
 /**
@@ -352,10 +352,10 @@ 
  * @param  string $default_tab Default tab name.
  * @return string
  */
-function give_set_default_tab_form_reports_page( $default_tab ) {
+function give_set_default_tab_form_reports_page($default_tab) {
 	return 'earnings';
 }
-add_filter( 'give_default_setting_tab_give-reports', 'give_set_default_tab_form_reports_page', 10, 1 );
+add_filter('give_default_setting_tab_give-reports', 'give_set_default_tab_form_reports_page', 10, 1);
 
 
 /**
@@ -368,19 +368,19 @@ 
  *
  * @return array
  */
-function give_add_display_page_states( $post_states, $post ) {
+function give_add_display_page_states($post_states, $post) {
 
-	switch ( $post->ID ) {
-		case give_get_option( 'success_page' ):
-			$post_states['give_successfully_page'] = __( 'Donation Success Page', 'give' );
+	switch ($post->ID) {
+		case give_get_option('success_page'):
+			$post_states['give_successfully_page'] = __('Donation Success Page', 'give');
 			break;
 
-		case give_get_option( 'failure_page' ):
-			$post_states['give_failure_page'] = __( 'Donation Failed Page', 'give' );
+		case give_get_option('failure_page'):
+			$post_states['give_failure_page'] = __('Donation Failed Page', 'give');
 			break;
 
-		case give_get_option( 'history_page' ):
-			$post_states['give_history_page'] = __( 'Donation History Page', 'give' );
+		case give_get_option('history_page'):
+			$post_states['give_history_page'] = __('Donation History Page', 'give');
 			break;
 	}
 
@@ -388,4 +388,4 @@ 
 }
 
 // Add a post display state for special Give pages.
-add_filter( 'display_post_states', 'give_add_display_page_states', 10, 2 );
+add_filter('display_post_states', 'give_add_display_page_states', 10, 2);

includes/admin/class-api-keys-table.php

Spacing +66 added lines, -66 removed lines

@@ -10,13 +10,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -53,11 +53,11 @@ 
 		global $status, $page;
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => esc_html__( 'API Key', 'give' ),     // Singular name of the listed records
-			'plural'   => esc_html__( 'API Keys', 'give' ),    // Plural name of the listed records
+		parent::__construct(array(
+			'singular' => esc_html__('API Key', 'give'), // Singular name of the listed records
+			'plural'   => esc_html__('API Keys', 'give'), // Plural name of the listed records
 			'ajax'     => false, // Does this table support ajax?
-		) );
+		));
 
 		$this->query();
 	}
@@ -73,8 +73,8 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $item, $column_name ) {
-		return $item[ $column_name ];
+	public function column_default($item, $column_name) {
+		return $item[$column_name];
 	}
 
 	/**
@@ -87,8 +87,8 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_key( $item ) {
-		return '<input onClick="this.setSelectionRange(0, this.value.length)" readonly="readonly" type="text" class="large-text" value="' . esc_attr( $item['key'] ) . '"/>';
+	public function column_key($item) {
+		return '<input onClick="this.setSelectionRange(0, this.value.length)" readonly="readonly" type="text" class="large-text" value="'.esc_attr($item['key']).'"/>';
 	}
 
 	/**
@@ -101,8 +101,8 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_token( $item ) {
-		return '<input onClick="this.setSelectionRange(0, this.value.length)" readonly="readonly" type="text" class="large-text" value="' . esc_attr( $item['token'] ) . '"/>';
+	public function column_token($item) {
+		return '<input onClick="this.setSelectionRange(0, this.value.length)" readonly="readonly" type="text" class="large-text" value="'.esc_attr($item['token']).'"/>';
 	}
 
 	/**
@@ -115,8 +115,8 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_secret( $item ) {
-		return '<input onClick="this.setSelectionRange(0, this.value.length)" readonly="readonly" type="text" class="large-text" value="' . esc_attr( $item['secret'] ) . '"/>';
+	public function column_secret($item) {
+		return '<input onClick="this.setSelectionRange(0, this.value.length)" readonly="readonly" type="text" class="large-text" value="'.esc_attr($item['secret']).'"/>';
 	}
 
 	/**
@@ -126,46 +126,46 @@ 
 	 * @since  1.1
 	 * @return string
 	 */
-	public function column_user( $item ) {
+	public function column_user($item) {
 
 		$actions = array();
 
-		if ( apply_filters( 'give_api_log_requests', true ) ) {
+		if (apply_filters('give_api_log_requests', true)) {
 			$actions['view'] = sprintf(
 				'<a href="%s">%s</a>',
-				esc_url( add_query_arg( array(
+				esc_url(add_query_arg(array(
 					'section'   => 'api_requests',
 					'post_type' => 'give_forms',
 					'page'      => 'give-tools',
 					'tab'       => 'logs',
 					's'         => $item['email'],
-				), 'edit.php' ) ),
-				esc_html__( 'View API Log', 'give' )
+				), 'edit.php')),
+				esc_html__('View API Log', 'give')
 			);
 		}
 
 		$actions['reissue'] = sprintf(
 			'<a href="%s" class="give-regenerate-api-key">%s</a>',
-			esc_url( wp_nonce_url( add_query_arg( array(
+			esc_url(wp_nonce_url(add_query_arg(array(
 				'user_id'          => $item['id'],
 				'give_action'      => 'process_api_key',
 				'give_api_process' => 'regenerate',
-			) ), 'give-api-nonce' ) ),
-			esc_html__( 'Reissue', 'give' )
+			)), 'give-api-nonce')),
+			esc_html__('Reissue', 'give')
 		);
-		$actions['revoke']  = sprintf(
+		$actions['revoke'] = sprintf(
 			'<a href="%s" class="give-revoke-api-key give-delete">%s</a>',
-			esc_url( wp_nonce_url( add_query_arg( array(
+			esc_url(wp_nonce_url(add_query_arg(array(
 				'user_id'          => $item['id'],
 				'give_action'      => 'process_api_key',
 				'give_api_process' => 'revoke',
-			) ), 'give-api-nonce' ) ),
-			esc_html__( 'Revoke', 'give' )
+			)), 'give-api-nonce')),
+			esc_html__('Revoke', 'give')
 		);
 
-		$actions = apply_filters( 'give_api_row_actions', array_filter( $actions ) );
+		$actions = apply_filters('give_api_row_actions', array_filter($actions));
 
-		return sprintf( '%1$s %2$s', $item['user'], $this->row_actions( $actions ) );
+		return sprintf('%1$s %2$s', $item['user'], $this->row_actions($actions));
 	}
 
 	/**
@@ -189,10 +189,10 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'user'   => esc_html__( 'Username', 'give' ),
-			'key'    => esc_html__( 'Public Key', 'give' ),
-			'token'  => esc_html__( 'Token', 'give' ),
-			'secret' => esc_html__( 'Secret Key', 'give' ),
+			'user'   => esc_html__('Username', 'give'),
+			'key'    => esc_html__('Public Key', 'give'),
+			'token'  => esc_html__('Token', 'give'),
+			'secret' => esc_html__('Secret Key', 'give'),
 		);
 
 		return $columns;
@@ -206,20 +206,20 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
-		if ( 'top' === $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+	protected function display_tablenav($which) {
+		if ('top' === $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav <?php echo esc_attr($which); ?>">
 
 			<div class="alignleft actions bulkactions">
-				<?php $this->bulk_actions( $which ); ?>
+				<?php $this->bulk_actions($which); ?>
 			</div>
 
 			<?php
-			$this->extra_tablenav( $which );
-			$this->pagination( $which );
+			$this->extra_tablenav($which);
+			$this->pagination($which);
 			?>
 
 			<br class="clear"/>
@@ -237,18 +237,18 @@ 
 	 *
 	 * @return void
 	 */
-	function bulk_actions( $which = '' ) {
+	function bulk_actions($which = '') {
 		// These aren't really bulk actions but this outputs the markup in the right place.
 		static $give_api_is_bottom;
 
-		if ( $give_api_is_bottom ) {
+		if ($give_api_is_bottom) {
 			return;
 		}
 		?>
 		<input type="hidden" name="give_action" value="process_api_key"/>
 		<input type="hidden" name="give_api_process" value="generate"/>
 		<?php
-		wp_nonce_field( 'give-api-nonce' );
+		wp_nonce_field('give-api-nonce');
 		/**
 		 * API Key user search.
 		 */
@@ -257,9 +257,9 @@ 
 			'name' => 'user_id',
 		);
 
-		echo Give()->html->ajax_user_search( $args );
+		echo Give()->html->ajax_user_search($args);
 
-		submit_button( esc_html__( 'Generate New API Keys', 'give' ), 'secondary', 'submit', false );
+		submit_button(esc_html__('Generate New API Keys', 'give'), 'secondary', 'submit', false);
 
 		$give_api_is_bottom = true;
 	}
@@ -272,7 +272,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;
 	}
 
 	/**
@@ -283,21 +283,21 @@ 
 	 * @return array
 	 */
 	public function query() {
-		$users = get_users( array(
+		$users = get_users(array(
 			'meta_value' => 'give_user_secret_key',
 			'number'     => $this->per_page,
-			'offset'     => $this->per_page * ( $this->get_paged() - 1 ),
-		) );
-		$keys  = array();
-
-		foreach ( $users as $user ) {
-			$keys[ $user->ID ]['id']    = $user->ID;
-			$keys[ $user->ID ]['email'] = $user->user_email;
-			$keys[ $user->ID ]['user']  = '<a href="' . add_query_arg( 'user_id', $user->ID, 'user-edit.php' ) . '"><strong>' . $user->user_login . '</strong></a>';
-
-			$keys[ $user->ID ]['key']    = Give()->api->get_user_public_key( $user->ID );
-			$keys[ $user->ID ]['secret'] = Give()->api->get_user_secret_key( $user->ID );
-			$keys[ $user->ID ]['token']  = Give()->api->get_token( $user->ID );
+			'offset'     => $this->per_page * ($this->get_paged() - 1),
+		));
+		$keys = array();
+
+		foreach ($users as $user) {
+			$keys[$user->ID]['id']    = $user->ID;
+			$keys[$user->ID]['email'] = $user->user_email;
+			$keys[$user->ID]['user']  = '<a href="'.add_query_arg('user_id', $user->ID, 'user-edit.php').'"><strong>'.$user->user_login.'</strong></a>';
+
+			$keys[$user->ID]['key']    = Give()->api->get_user_public_key($user->ID);
+			$keys[$user->ID]['secret'] = Give()->api->get_user_secret_key($user->ID);
+			$keys[$user->ID]['token']  = Give()->api->get_token($user->ID);
 		}
 
 		return $keys;
@@ -314,9 +314,9 @@ 
 	public function total_items() {
 		global $wpdb;
 
-		$total_items = Give_Cache::get( 'give_total_api_keys', true );
+		$total_items = Give_Cache::get('give_total_api_keys', true);
 
-		if ( ! $total_items ) {
+		if ( ! $total_items) {
 			$total_items = $wpdb->get_var(
 				$wpdb->prepare(
 					"SELECT count(user_id)
@@ -325,7 +325,7 @@ 
 				)
 			);
 
-			Give_Cache::set( 'give_total_api_keys', $total_items, HOUR_IN_SECONDS, true );
+			Give_Cache::set('give_total_api_keys', $total_items, HOUR_IN_SECONDS, true);
 		}
 
 		return $total_items;
@@ -344,7 +344,7 @@ 
 		$hidden   = array(); // No hidden columns
 		$sortable = array(); // Not sortable... for now
 
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 
 		$data = $this->query();
 
@@ -352,10 +352,10 @@ 
 
 		$this->items = $data;
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 			'total_items' => $total_items,
 			'per_page'    => $this->per_page,
-			'total_pages' => ceil( $total_items / $this->per_page ),
-		) );
+			'total_pages' => ceil($total_items / $this->per_page),
+		));
 	}
 }

includes/admin/donors/donors.php

Spacing +218 added lines, -218 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,7 +25,7 @@ 
  *
  * @return string
  */
-function __give_get_format_address( $address, $address_args = array() ) {
+function __give_get_format_address($address, $address_args = array()) {
 	$address_html = '';
 	$address_args = wp_parse_args(
 		$address_args,
@@ -40,7 +40,7 @@ 
 	$address_id = $address_args['type'];
 
 	// Bailout.
-	if ( empty( $address ) || ! is_array( $address ) ) {
+	if (empty($address) || ! is_array($address)) {
 		return $address_html;
 	}
 
@@ -49,19 +49,19 @@ 
 	$address_html .= sprintf(
 		'<span data-address-type="line1">%1$s</span>%2$s',
 		$address['line1'],
-		( ! empty( $address['line2'] ) ? '<br>' : '' )
+		( ! empty($address['line2']) ? '<br>' : '')
 	);
 	$address_html .= sprintf(
 		'<span data-address-type="line2">%1$s</span>%2$s',
 		$address['line2'],
-		( ! empty( $address['city'] ) ? '<br>' : '' )
+		( ! empty($address['city']) ? '<br>' : '')
 	);
 	$address_html .= sprintf(
 		'<span data-address-type="city">%1$s</span><span data-address-type="state">%2$s</span><span data-address-type="zip">%3$s</span>%4$s',
 		$address['city'],
-		( ! empty( $address['state'] ) ? ", {$address['state']}" : '' ),
-		( ! empty( $address['zip'] ) ? " {$address['zip']}" : '' ),
-		( ! empty( $address['country'] ) ? '<br>' : '' )
+		( ! empty($address['state']) ? ", {$address['state']}" : ''),
+		( ! empty($address['zip']) ? " {$address['zip']}" : ''),
+		( ! empty($address['country']) ? '<br>' : '')
 	);
 	$address_html .= sprintf(
 		'<span data-address-type="country">%s</span><br>',
@@ -71,8 +71,8 @@ 
 	// Address action.
 	$address_html .= sprintf(
 		'<br><a href="#" class="js-edit">%1$s</a> | <a href="#" class="js-remove">%2$s</a>',
-		__( 'Edit', 'give' ),
-		__( 'Remove', 'give' )
+		__('Edit', 'give'),
+		__('Remove', 'give')
 	);
 
 	/**
@@ -80,14 +80,14 @@ 
 	 *
 	 * @since 2.0
 	 */
-	$address_label = apply_filters( "give_donor_{$address_args['type']}_address_label", ucfirst( $address_args['type'] ), $address_args );
+	$address_label = apply_filters("give_donor_{$address_args['type']}_address_label", ucfirst($address_args['type']), $address_args);
 
 	// Set unique id and index for multi type address.
-	if ( isset( $address_args['index'] ) ) {
+	if (isset($address_args['index'])) {
 		$address_label = "{$address_label} #{$address_args['index']}";
 	}
 
-	if ( isset( $address_args['id'] ) ) {
+	if (isset($address_args['id'])) {
 		$address_id = "{$address_id}_{$address_args['id']}";
 	}
 
@@ -112,9 +112,9 @@ 
  */
 function give_donors_page() {
 	$default_views  = give_donor_views();
-	$requested_view = isset( $_GET['view'] ) ? sanitize_text_field( $_GET['view'] ) : 'donors';
-	if ( array_key_exists( $requested_view, $default_views ) && function_exists( $default_views[ $requested_view ] ) ) {
-		give_render_donor_view( $requested_view, $default_views );
+	$requested_view = isset($_GET['view']) ? sanitize_text_field($_GET['view']) : 'donors';
+	if (array_key_exists($requested_view, $default_views) && function_exists($default_views[$requested_view])) {
+		give_render_donor_view($requested_view, $default_views);
 	} else {
 		give_donors_list();
 	}
@@ -130,7 +130,7 @@ 
 
 	$views = array();
 
-	return apply_filters( 'give_donor_views', $views );
+	return apply_filters('give_donor_views', $views);
 
 }
 
@@ -144,7 +144,7 @@ 
 
 	$tabs = array();
 
-	return apply_filters( 'give_donor_tabs', $tabs );
+	return apply_filters('give_donor_tabs', $tabs);
 
 }
 
@@ -156,7 +156,7 @@ 
  */
 function give_donors_list() {
 
-	include GIVE_PLUGIN_DIR . 'includes/admin/donors/class-donor-table.php';
+	include GIVE_PLUGIN_DIR.'includes/admin/donors/class-donor-table.php';
 
 	$donors_table = new Give_Donor_List_Table();
 	$donors_table->prepare_items();
@@ -169,12 +169,12 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_donors_table_top' );
+		do_action('give_donors_table_top');
 		?>
 
 		<hr class="wp-header-end">
-		<form id="give-donors-search-filter" method="get" action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors' ); ?>">
-			<?php $donors_table->search_box( __( 'Search Donors', 'give' ), 'give-donors' ); ?>
+		<form id="give-donors-search-filter" method="get" action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors'); ?>">
+			<?php $donors_table->search_box(__('Search Donors', 'give'), 'give-donors'); ?>
 			<input type="hidden" name="post_type" value="give_forms" />
 			<input type="hidden" name="page" value="give-donors" />
 			<input type="hidden" name="view" value="donors" />
@@ -191,7 +191,7 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_donors_table_bottom' );
+		do_action('give_donors_table_bottom');
 		?>
 	</div>
 	<?php
@@ -207,33 +207,33 @@ 
  *
  * @return void
  */
-function give_render_donor_view( $view, $callbacks ) {
+function give_render_donor_view($view, $callbacks) {
 
 	$render = true;
 
-	$donor_view_role = apply_filters( 'give_view_donors_role', 'view_give_reports' );
+	$donor_view_role = apply_filters('give_view_donors_role', 'view_give_reports');
 
-	if ( ! current_user_can( $donor_view_role ) ) {
-		give_set_error( 'give-no-access', __( 'You are not permitted to view this data.', 'give' ) );
+	if ( ! current_user_can($donor_view_role)) {
+		give_set_error('give-no-access', __('You are not permitted to view this data.', 'give'));
 		$render = false;
 	}
 
-	if ( ! isset( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
-		give_set_error( 'give-invalid_donor', __( 'Invalid Donor ID.', 'give' ) );
+	if ( ! isset($_GET['id']) || ! is_numeric($_GET['id'])) {
+		give_set_error('give-invalid_donor', __('Invalid Donor ID.', 'give'));
 		$render = false;
 	}
 
 	$donor_id          = (int) $_GET['id'];
-	$reconnect_user_id = ! empty( $_GET['user_id'] ) ? (int) $_GET['user_id'] : '';
-	$donor             = new Give_Donor( $donor_id );
+	$reconnect_user_id = ! empty($_GET['user_id']) ? (int) $_GET['user_id'] : '';
+	$donor             = new Give_Donor($donor_id);
 
 	// Reconnect User with Donor profile.
-	if ( $reconnect_user_id ) {
-		give_connect_user_donor_profile( $donor, array( 'user_id' => $reconnect_user_id ), array() );
+	if ($reconnect_user_id) {
+		give_connect_user_donor_profile($donor, array('user_id' => $reconnect_user_id), array());
 	}
 
-	if ( empty( $donor->id ) ) {
-		give_set_error( 'give-invalid_donor', __( 'Invalid Donor ID.', 'give' ) );
+	if (empty($donor->id)) {
+		give_set_error('give-invalid_donor', __('Invalid Donor ID.', 'give'));
 		$render = false;
 	}
 
@@ -242,9 +242,9 @@ 
 
 	<div class='wrap'>
 
-		<?php if ( give_get_errors() ) : ?>
+		<?php if (give_get_errors()) : ?>
 			<div class="error settings-error">
-				<?php Give()->notices->render_frontend_notices( 0 ); ?>
+				<?php Give()->notices->render_frontend_notices(0); ?>
 			</div>
 		<?php endif; ?>
 
@@ -252,7 +252,7 @@ 
 			<?php
 			printf(
 			/* translators: %s: donor first name */
-				__( 'Edit Donor: %s %s', 'give' ),
+				__('Edit Donor: %s %s', 'give'),
 				$donor->get_first_name(),
 				$donor->get_last_name()
 			);
@@ -261,26 +261,26 @@ 
 
 		<hr class="wp-header-end">
 
-		<?php if ( $donor && $render ) : ?>
+		<?php if ($donor && $render) : ?>
 
 			<h2 class="nav-tab-wrapper">
 				<?php
-				foreach ( $donor_tabs as $key => $tab ) :
+				foreach ($donor_tabs as $key => $tab) :
 					$active = $key === $view ? true : false;
 					$class  = $active ? 'nav-tab nav-tab-active' : 'nav-tab';
 					printf(
-						'<a href="%1$s" class="%2$s"><span class="dashicons %3$s"></span>%4$s</a>' . "\n",
-						esc_url( admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=' . $key . '&id=' . $donor->id ) ),
-						esc_attr( $class ),
-						sanitize_html_class( $tab['dashicon'] ),
-						esc_html( $tab['title'] )
+						'<a href="%1$s" class="%2$s"><span class="dashicons %3$s"></span>%4$s</a>'."\n",
+						esc_url(admin_url('edit.php?post_type=give_forms&page=give-donors&view='.$key.'&id='.$donor->id)),
+						esc_attr($class),
+						sanitize_html_class($tab['dashicon']),
+						esc_html($tab['title'])
 					);
 				endforeach;
 				?>
 			</h2>
 
 			<div id="give-donor-card-wrapper">
-				<?php $callbacks[ $view ]( $donor ) ?>
+				<?php $callbacks[$view]($donor) ?>
 			</div>
 
 		<?php endif; ?>
@@ -300,9 +300,9 @@ 
  *
  * @return void
  */
-function give_donor_view( $donor ) {
+function give_donor_view($donor) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
 	/**
 	 * Fires in donor profile screen, above the donor card.
@@ -311,11 +311,11 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_card_top', $donor );
+	do_action('give_donor_card_top', $donor);
 
 	// Set Read only to the fields which needs to be locked.
 	$read_only = '';
-	if ( $donor->user_id ) {
+	if ($donor->user_id) {
 		$read_only = 'readonly="readonly"';
 	}
 	?>
@@ -323,32 +323,32 @@ 
 	<div id="donor-summary" class="info-wrapper donor-section postbox">
 
 		<form id="edit-donor-info" method="post"
-		      action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ); ?>">
+		      action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id); ?>">
 
 			<div class="donor-info">
 
 				<div class="donor-bio-header clearfix">
 
 					<div class="avatar-wrap left" id="donor-avatar">
-						<?php echo get_avatar( $donor->email ); ?>
+						<?php echo get_avatar($donor->email); ?>
 					</div>
 
 					<div id="donor-name-wrap" class="left">
 						<span class="donor-name info-item edit-item">
 							<input <?php echo $read_only; ?> size="15" data-key="first_name"
 							                                 name="customerinfo[first_name]" type="text"
-							                                 value="<?php esc_attr_e( $donor->get_first_name() ); ?>"
-							                                 placeholder="<?php _e( 'First Name', 'give' ); ?>"/>
-							<?php if ( $donor->user_id ) : ?>
+							                                 value="<?php esc_attr_e($donor->get_first_name()); ?>"
+							                                 placeholder="<?php _e('First Name', 'give'); ?>"/>
+							<?php if ($donor->user_id) : ?>
 								<a href="#" class="give-lock-block">
 									<i class="give-icon give-icon-locked"></i>
 								</a>
 							<?php endif; ?>
 							<input <?php echo $read_only; ?> size="15" data-key="last_name"
 							                                 name="customerinfo[last_name]" type="text"
-							                                 value="<?php esc_attr_e( $donor->get_last_name() ); ?>"
-							                                 placeholder="<?php _e( 'Last Name', 'give' ); ?>"/>
-							<?php if ( $donor->user_id ) : ?>
+							                                 value="<?php esc_attr_e($donor->get_last_name()); ?>"
+							                                 placeholder="<?php _e('Last Name', 'give'); ?>"/>
+							<?php if ($donor->user_id) : ?>
 								<a href="#" class="give-lock-block">
 									<i class="give-icon give-icon-locked"></i>
 								</a>
@@ -358,12 +358,12 @@ 
 									data-key="name"><?php echo $donor->name; ?></span></span>
 					</div>
 					<p class="donor-since info-item">
-						<?php _e( 'Donor since', 'give' ); ?>
-						<?php echo date_i18n( give_date_format(), strtotime( $donor->date_created ) ) ?>
+						<?php _e('Donor since', 'give'); ?>
+						<?php echo date_i18n(give_date_format(), strtotime($donor->date_created)) ?>
 					</p>
-					<?php if ( current_user_can( $donor_edit_role ) ) : ?>
+					<?php if (current_user_can($donor_edit_role)) : ?>
 						<a href="#" id="edit-donor"
-						   class="button info-item editable donor-edit-link"><?php _e( 'Edit Donor', 'give' ); ?></a>
+						   class="button info-item editable donor-edit-link"><?php _e('Edit Donor', 'give'); ?></a>
 					<?php endif; ?>
 				</div>
 				<!-- /donor-bio-header -->
@@ -373,11 +373,11 @@ 
 					<table class="widefat striped">
 						<tbody>
 						<tr>
-							<th scope="col"><label for="tablecell"><?php _e( 'Donor ID:', 'give' ); ?></label></th>
+							<th scope="col"><label for="tablecell"><?php _e('Donor ID:', 'give'); ?></label></th>
 							<td><?php echo $donor->id; ?></td>
 						</tr>
 						<tr>
-							<th scope="col"><label for="tablecell"><?php _e( 'User ID:', 'give' ); ?></label></th>
+							<th scope="col"><label for="tablecell"><?php _e('User ID:', 'give'); ?></label></th>
 							<td>
 									<span class="donor-user-id info-item edit-item">
 										<?php
@@ -394,22 +394,22 @@ 
 											'data'  => $data_atts,
 										);
 
-										if ( ! empty( $user_id ) ) {
-											$userdata              = get_userdata( $user_id );
+										if ( ! empty($user_id)) {
+											$userdata              = get_userdata($user_id);
 											$user_args['selected'] = $user_id;
 										}
 
-										echo Give()->html->ajax_user_search( $user_args );
+										echo Give()->html->ajax_user_search($user_args);
 										?>
 									</span>
 
 								<span class="donor-user-id info-item editable">
-										<?php if ( ! empty( $userdata ) ) : ?>
-											<span data-key="user_id">#<?php echo $donor->user_id . ' - ' . $userdata->display_name; ?></span>
+										<?php if ( ! empty($userdata)) : ?>
+											<span data-key="user_id">#<?php echo $donor->user_id.' - '.$userdata->display_name; ?></span>
 										<?php else: ?>
-											<span data-key="user_id"><?php _e( 'Unregistered', 'give' ); ?></span>
+											<span data-key="user_id"><?php _e('Unregistered', 'give'); ?></span>
 										<?php endif; ?>
-									<?php if ( current_user_can( $donor_edit_role ) && intval( $donor->user_id ) > 0 ):
+									<?php if (current_user_can($donor_edit_role) && intval($donor->user_id) > 0):
 
 										echo sprintf(
 											'- <span class="disconnect-user">
@@ -418,11 +418,11 @@ 
 				                                       | <span class="view-user-profile">
  											                <a id="view-user-profile" href="%3$s" aria-label="%4$s">%5$s</a>
  										               </span>',
-											__( 'Disconnects the current user ID from this donor record.', 'give' ),
-											__( 'Disconnect User', 'give' ),
-											'user-edit.php?user_id=' . $donor->user_id,
-											__( 'View User Profile of current user ID.', 'give' ),
-											__( 'View User Profile', 'give' )
+											__('Disconnects the current user ID from this donor record.', 'give'),
+											__('Disconnect User', 'give'),
+											'user-edit.php?user_id='.$donor->user_id,
+											__('View User Profile of current user ID.', 'give'),
+											__('View User Profile', 'give')
 										);
 
 										endif; ?>
@@ -431,13 +431,13 @@ 
 						</tr>
 
 						<?php
-						$donor_company = $donor->get_meta( '_give_donor_company', true );
+						$donor_company = $donor->get_meta('_give_donor_company', true);
 
-						if ( ! empty( $donor_company ) ) {
+						if ( ! empty($donor_company)) {
 							?>
 							<tr class="alternate">
 								<th scope="col">
-									<label for="tablecell"><?php _e( 'Company Name:', 'give' ); ?></label>
+									<label for="tablecell"><?php _e('Company Name:', 'give'); ?></label>
 								</th>
 								<td><?php echo $donor_company; ?></td>
 							</tr>
@@ -453,11 +453,11 @@ 
 
 			<span id="donor-edit-actions" class="edit-item">
 				<input type="hidden" data-key="id" name="customerinfo[id]" value="<?php echo $donor->id; ?>"/>
-				<?php wp_nonce_field( 'edit-donor', '_wpnonce', false, true ); ?>
+				<?php wp_nonce_field('edit-donor', '_wpnonce', false, true); ?>
 				<input type="hidden" name="give_action" value="edit-donor"/>
 				<input type="submit" id="give-edit-donor-save" class="button-secondary"
-				       value="<?php _e( 'Update Donor', 'give' ); ?>"/>
-				<a id="give-edit-donor-cancel" href="" class="delete"><?php _e( 'Cancel', 'give' ); ?></a>
+				       value="<?php _e('Update Donor', 'give'); ?>"/>
+				<a id="give-edit-donor-cancel" href="" class="delete"><?php _e('Cancel', 'give'); ?></a>
 			</span>
 
 		</form>
@@ -472,24 +472,24 @@ 
 	 *
 	 * @param Give_Donor $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_before_stats', $donor );
+	do_action('give_donor_before_stats', $donor);
 	?>
 
 	<div id="donor-stats-wrapper" class="donor-section postbox clear">
 		<ul>
 			<li>
-				<a href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&donor=' . absint( $donor->id ) ); ?>">
+				<a href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-payment-history&donor='.absint($donor->id)); ?>">
 					<span class="dashicons dashicons-heart"></span>
 					<?php
 					// Completed Donations.
-					$completed_donations_text = sprintf( _n( '%d Completed Donation', '%d Completed Donations', $donor->purchase_count, 'give' ), $donor->purchase_count );
-					echo apply_filters( 'give_donor_completed_donations', $completed_donations_text, $donor );
+					$completed_donations_text = sprintf(_n('%d Completed Donation', '%d Completed Donations', $donor->purchase_count, 'give'), $donor->purchase_count);
+					echo apply_filters('give_donor_completed_donations', $completed_donations_text, $donor);
 					?>
 				</a>
 			</li>
 			<li>
 				<span class="dashicons dashicons-chart-area"></span>
-				<?php echo give_currency_filter( give_format_amount( $donor->get_total_donation_amount(), array( 'sanitize' => false ) ) ); ?> <?php _e( 'Lifetime Donations', 'give' ); ?>
+				<?php echo give_currency_filter(give_format_amount($donor->get_total_donation_amount(), array('sanitize' => false))); ?> <?php _e('Lifetime Donations', 'give'); ?>
 			</li>
 			<?php
 			/**
@@ -501,7 +501,7 @@ 
 			 *
 			 * @param object $donor The donor object being displayed.
 			 */
-			do_action( 'give_donor_stats_list', $donor );
+			do_action('give_donor_stats_list', $donor);
 			?>
 		</ul>
 	</div>
@@ -514,11 +514,11 @@ 
 	 *
 	 * @param Give_Donor $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_before_address', $donor );
+	do_action('give_donor_before_address', $donor);
 	?>
 
 	<div id="donor-address-wrapper" class="donor-section clear">
-		<h3><?php _e( 'Addresses', 'give' ); ?></h3>
+		<h3><?php _e('Addresses', 'give'); ?></h3>
 
 		<div class="postbox">
 			<div class="give-spinner-wrapper">
@@ -528,16 +528,16 @@ 
 				<div class="all-address">
 					<div class="give-grid-row">
 						<?php
-						if ( ! empty( $donor->address ) ) :
+						if ( ! empty($donor->address)) :
 							// Default address always will be at zero array index.
 							$is_set_as_default = null;
 
-							foreach ( $donor->address as $address_type => $addresses ) {
+							foreach ($donor->address as $address_type => $addresses) {
 
-								switch ( true ) {
-									case is_array( end( $addresses ) ):
+								switch (true) {
+									case is_array(end($addresses)):
 										$index = 1;
-										foreach ( $addresses as $id => $address ) {
+										foreach ($addresses as $id => $address) {
 											echo __give_get_format_address(
 												$address,
 												array(
@@ -547,11 +547,11 @@ 
 												)
 											);
 
-											$index ++;
+											$index++;
 										}
 										break;
 
-									case is_string( end( $addresses ) ):
+									case is_string(end($addresses)):
 										echo __give_get_format_address(
 											$addresses,
 											array(
@@ -564,13 +564,13 @@ 
 						endif;
 						?>
 					</div>
-					<span class="give-no-address-message<?php if ( ! empty( $donor->address ) ) {
+					<span class="give-no-address-message<?php if ( ! empty($donor->address)) {
 						echo ' give-hidden';
 					} ?>">
-						<?php _e( 'This donor does not have any addresses saved.', 'give' ); ?>
+						<?php _e('This donor does not have any addresses saved.', 'give'); ?>
 					</span>
 					<button class="button add-new-address">
-						<?php _e( 'Add Address', 'give' ); ?>
+						<?php _e('Add Address', 'give'); ?>
 					</button>
 				</div>
 
@@ -580,26 +580,26 @@ 
 							<tbody>
 							<tr>
 								<th class="col">
-									<label class="country"><?php esc_html_e( 'Country:', 'give' ); ?></label>
+									<label class="country"><?php esc_html_e('Country:', 'give'); ?></label>
 								</th>
 								<td>
 									<?php
-									echo Give()->html->select( array(
+									echo Give()->html->select(array(
 										'options'          => give_get_country_list(),
 										'name'             => 'country',
-										'selected'         => give_get_option( 'base_country' ),
+										'selected'         => give_get_option('base_country'),
 										'show_option_all'  => false,
 										'show_option_none' => false,
 										'chosen'           => true,
-										'placeholder'      => esc_attr__( 'Select a country', 'give' ),
-										'data'             => array( 'search-type' => 'no_ajax' ),
-									) );
+										'placeholder'      => esc_attr__('Select a country', 'give'),
+										'data'             => array('search-type' => 'no_ajax'),
+									));
 									?>
 								</td>
 							</tr>
 							<tr>
 								<th class="col">
-									<label for="line1"><?php esc_html_e( 'Address 1:', 'give' ); ?></label>
+									<label for="line1"><?php esc_html_e('Address 1:', 'give'); ?></label>
 								</th>
 								<td>
 									<input id="line1" name="line1" type="text" class="medium-text"/>
@@ -607,7 +607,7 @@ 
 							</tr>
 							<tr>
 								<th class="col">
-									<label for="line2"><?php esc_html_e( 'Address 2:', 'give' ); ?></label>
+									<label for="line2"><?php esc_html_e('Address 2:', 'give'); ?></label>
 								</th>
 								<td>
 									<input id="line2" type="text" name="line2" value="" class="medium-text"/>
@@ -616,7 +616,7 @@ 
 							</tr>
 							<tr>
 								<th class="col">
-									<label for="city"><?php esc_html_e( 'City:', 'give' ); ?></label>
+									<label for="city"><?php esc_html_e('City:', 'give'); ?></label>
 								</th>
 								<td>
 									<input id="city" type="text" name="city" value="" class="medium-text"/>
@@ -624,41 +624,41 @@ 
 							</tr>
 							<?php
 							$no_states_country = give_no_states_country_list();
-							$base_country      = give_get_option( 'base_country' );
-							if ( ! array_key_exists( $base_country, $no_states_country ) ) {
+							$base_country      = give_get_option('base_country');
+							if ( ! array_key_exists($base_country, $no_states_country)) {
 								?>
 								<tr class="give-field-wrap">
 									<th class="col">
-										<label for="state"><?php esc_html_e( 'State / Province / County:', 'give' ); ?></label>
+										<label for="state"><?php esc_html_e('State / Province / County:', 'give'); ?></label>
 									</th>
 									<td>
 										<?php
-										$states     = give_get_states( $base_country );
+										$states     = give_get_states($base_country);
 										$state_args = array(
 											'name'  => 'state',
 											'class' => 'regular-text',
 										);
 
-										if ( empty( $states ) ) {
+										if (empty($states)) {
 
 											// Show Text field, if empty states.
-											$state_args = wp_parse_args( $state_args, array(
-												'value' => give_get_option( 'base_state' ),
-											) );
-											echo Give()->html->text( $state_args );
+											$state_args = wp_parse_args($state_args, array(
+												'value' => give_get_option('base_state'),
+											));
+											echo Give()->html->text($state_args);
 										} else {
 
 											// Show Chosen DropDown, if states are not empty.
-											$state_args = wp_parse_args( $state_args, array(
+											$state_args = wp_parse_args($state_args, array(
 												'options'          => $states,
-												'selected'         => give_get_option( 'base_state' ),
+												'selected'         => give_get_option('base_state'),
 												'show_option_all'  => false,
 												'show_option_none' => false,
 												'chosen'           => true,
-												'placeholder'      => __( 'Select a state', 'give' ),
-												'data'             => array( 'search-type' => 'no_ajax' ),
-											) );
-											echo Give()->html->select( $state_args );
+												'placeholder'      => __('Select a state', 'give'),
+												'data'             => array('search-type' => 'no_ajax'),
+											));
+											echo Give()->html->select($state_args);
 										}
 										?>
 									</td>
@@ -668,7 +668,7 @@ 
 							?>
 							<tr>
 								<th class="col">
-									<label for="zip"><?php esc_html_e( 'Zip / Postal Code:', 'give' ); ?></label>
+									<label for="zip"><?php esc_html_e('Zip / Postal Code:', 'give'); ?></label>
 								</th>
 								<td>
 									<input id="zip" type="text" name="zip" value="" class="medium-text"/>
@@ -676,12 +676,12 @@ 
 							</tr>
 							<tr>
 								<td colspan="2">
-									<?php wp_nonce_field( 'give-manage-donor-addresses', '_wpnonce', false ); ?>
+									<?php wp_nonce_field('give-manage-donor-addresses', '_wpnonce', false); ?>
 									<input type="hidden" name="address-action" value="add">
 									<input type="hidden" name="address-id" value="">
 									<input type="submit" class="button button-primary js-save"
-									       value="<?php _e( 'Save', 'give' ); ?>">&nbsp;&nbsp;<button
-											class="button js-cancel"><?php _e( 'Cancel', 'give' ); ?></button>
+									       value="<?php _e('Save', 'give'); ?>">&nbsp;&nbsp;<button
+											class="button js-cancel"><?php _e('Cancel', 'give'); ?></button>
 								</td>
 							</tr>
 							</tbody>
@@ -700,7 +700,7 @@ 
 	 *
 	 * @param Give_Donor $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_before_tables_wrapper', $donor );
+	do_action('give_donor_before_tables_wrapper', $donor);
 	?>
 
 	<div id="donor-tables-wrapper" class="donor-section">
@@ -713,46 +713,46 @@ 
 		 *
 		 * @param object $donor The donor object being displayed.
 		 */
-		do_action( 'give_donor_before_tables', $donor );
+		do_action('give_donor_before_tables', $donor);
 		?>
 
-		<h3><?php _e( 'Donor Emails', 'give' ); ?></h3>
+		<h3><?php _e('Donor Emails', 'give'); ?></h3>
 
 		<table class="wp-list-table widefat striped emails">
 			<thead>
 			<tr>
-				<th><?php _e( 'Email', 'give' ); ?></th>
-				<th><?php _e( 'Actions', 'give' ); ?></th>
+				<th><?php _e('Email', 'give'); ?></th>
+				<th><?php _e('Actions', 'give'); ?></th>
 			</tr>
 			</thead>
 
 			<tbody>
-			<?php if ( ! empty( $donor->emails ) ) { ?>
+			<?php if ( ! empty($donor->emails)) { ?>
 
-				<?php foreach ( $donor->emails as $key => $email ) : ?>
+				<?php foreach ($donor->emails as $key => $email) : ?>
 					<tr data-key="<?php echo $key; ?>">
 						<td>
 							<?php echo $email; ?>
-							<?php if ( 'primary' === $key ) : ?>
+							<?php if ('primary' === $key) : ?>
 								<span class="dashicons dashicons-star-filled primary-email-icon"></span>
 							<?php endif; ?>
 						</td>
 						<td>
-							<?php if ( 'primary' !== $key ) : ?>
+							<?php if ('primary' !== $key) : ?>
 								<?php
-								$base_url    = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id );
-								$promote_url = wp_nonce_url( add_query_arg( array(
-									'email'       => rawurlencode( $email ),
+								$base_url    = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id);
+								$promote_url = wp_nonce_url(add_query_arg(array(
+									'email'       => rawurlencode($email),
 									'give_action' => 'set_donor_primary_email',
-								), $base_url ), 'give-set-donor-primary-email' );
-								$remove_url  = wp_nonce_url( add_query_arg( array(
-									'email'       => rawurlencode( $email ),
+								), $base_url), 'give-set-donor-primary-email');
+								$remove_url = wp_nonce_url(add_query_arg(array(
+									'email'       => rawurlencode($email),
 									'give_action' => 'remove_donor_email',
-								), $base_url ), 'give-remove-donor-email' );
+								), $base_url), 'give-remove-donor-email');
 								?>
-								<a href="<?php echo $promote_url; ?>"><?php _e( 'Make Primary', 'give' ); ?></a>
+								<a href="<?php echo $promote_url; ?>"><?php _e('Make Primary', 'give'); ?></a>
 								&nbsp;|&nbsp;
-								<a href="<?php echo $remove_url; ?>" class="delete"><?php _e( 'Remove', 'give' ); ?></a>
+								<a href="<?php echo $remove_url; ?>" class="delete"><?php _e('Remove', 'give'); ?></a>
 							<?php endif; ?>
 						</td>
 					</tr>
@@ -762,14 +762,14 @@ 
 					<td colspan="2" class="add-donor-email-td">
 						<div class="add-donor-email-wrapper">
 							<input type="hidden" name="donor-id" value="<?php echo $donor->id; ?>"/>
-							<?php wp_nonce_field( 'give_add_donor_email', 'add_email_nonce', false, true ); ?>
+							<?php wp_nonce_field('give_add_donor_email', 'add_email_nonce', false, true); ?>
 							<input type="email" name="additional-email" value=""
-							       placeholder="<?php _e( 'Email Address', 'give' ); ?>"/>&nbsp;
+							       placeholder="<?php _e('Email Address', 'give'); ?>"/>&nbsp;
 							<input type="checkbox" name="make-additional-primary" value="1"
 							       id="make-additional-primary"/>&nbsp;<label
-									for="make-additional-primary"><?php _e( 'Make Primary', 'give' ); ?></label>
+									for="make-additional-primary"><?php _e('Make Primary', 'give'); ?></label>
 							<button class="button-secondary give-add-donor-email"
-							        id="add-donor-email"><?php _e( 'Add Email', 'give' ); ?></button>
+							        id="add-donor-email"><?php _e('Add Email', 'give'); ?></button>
 							<span class="spinner"></span>
 						</div>
 						<div class="notice-wrap"></div>
@@ -777,50 +777,50 @@ 
 				</tr>
 			<?php } else { ?>
 				<tr>
-					<td colspan="2"><?php _e( 'No Emails Found', 'give' ); ?></td>
+					<td colspan="2"><?php _e('No Emails Found', 'give'); ?></td>
 				</tr>
 			<?php }// End if().
 			?>
 			</tbody>
 		</table>
 
-		<h3><?php _e( 'Recent Donations', 'give' ); ?></h3>
+		<h3><?php _e('Recent Donations', 'give'); ?></h3>
 		<?php
-		$payment_ids = explode( ',', $donor->payment_ids );
-		$payments    = give_get_payments( array(
+		$payment_ids = explode(',', $donor->payment_ids);
+		$payments    = give_get_payments(array(
 			'post__in' => $payment_ids,
-		) );
-		$payments    = array_slice( $payments, 0, 10 );
+		));
+		$payments    = array_slice($payments, 0, 10);
 		?>
 		<table class="wp-list-table widefat striped payments">
 			<thead>
 			<tr>
-				<th scope="col"><?php _e( 'ID', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Amount', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Date', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Status', 'give' ); ?></th>
-				<th scope="col"><?php _e( 'Actions', 'give' ); ?></th>
+				<th scope="col"><?php _e('ID', 'give'); ?></th>
+				<th scope="col"><?php _e('Amount', 'give'); ?></th>
+				<th scope="col"><?php _e('Date', 'give'); ?></th>
+				<th scope="col"><?php _e('Status', 'give'); ?></th>
+				<th scope="col"><?php _e('Actions', 'give'); ?></th>
 			</tr>
 			</thead>
 			<tbody>
-			<?php if ( ! empty( $payments ) ) { ?>
-				<?php foreach ( $payments as $payment ) : ?>
+			<?php if ( ! empty($payments)) { ?>
+				<?php foreach ($payments as $payment) : ?>
 					<tr>
-						<td><?php echo Give()->seq_donation_number->get_serial_code( $payment->ID ); ?></td>
-						<td><?php echo give_donation_amount( $payment->ID, array( 'currency' => true, 'amount' => true, 'type' => 'donor' ) ); ?></td>
-						<td><?php echo date_i18n( give_date_format(), strtotime( $payment->post_date ) ); ?></td>
-						<td><?php echo give_get_payment_status( $payment, true ); ?></td>
+						<td><?php echo Give()->seq_donation_number->get_serial_code($payment->ID); ?></td>
+						<td><?php echo give_donation_amount($payment->ID, array('currency' => true, 'amount' => true, 'type' => 'donor')); ?></td>
+						<td><?php echo date_i18n(give_date_format(), strtotime($payment->post_date)); ?></td>
+						<td><?php echo give_get_payment_status($payment, true); ?></td>
 						<td>
 							<?php
 							printf(
 								'<a href="%1$s" aria-label="%2$s">%3$s</a>',
-								admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id=' . $payment->ID ),
+								admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-payment-details&id='.$payment->ID),
 								sprintf(
 								/* translators: %s: Donation ID */
-									esc_attr__( 'View Donation %s.', 'give' ),
+									esc_attr__('View Donation %s.', 'give'),
 									$payment->ID
 								),
-								__( 'View Donation', 'give' )
+								__('View Donation', 'give')
 							);
 							?>
 
@@ -835,47 +835,47 @@ 
 							 * @param object $donor   The donor object being displayed.
 							 * @param object $payment The payment object being displayed.
 							 */
-							do_action( 'give_donor_recent_purchases_actions', $donor, $payment );
+							do_action('give_donor_recent_purchases_actions', $donor, $payment);
 							?>
 						</td>
 					</tr>
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<tr>
-					<td colspan="5"><?php _e( 'No donations found.', 'give' ); ?></td>
+					<td colspan="5"><?php _e('No donations found.', 'give'); ?></td>
 				</tr>
 			<?php }// End if().
 			?>
 			</tbody>
 		</table>
 
-		<h3><?php _e( 'Completed Forms', 'give' ); ?></h3>
+		<h3><?php _e('Completed Forms', 'give'); ?></h3>
 		<?php
-		$donations = give_get_users_completed_donations( $donor->email );
+		$donations = give_get_users_completed_donations($donor->email);
 		?>
 		<table class="wp-list-table widefat striped donations">
 			<thead>
 			<tr>
-				<th scope="col"><?php _e( 'Form', 'give' ); ?></th>
-				<th scope="col" width="120px"><?php _e( 'Actions', 'give' ); ?></th>
+				<th scope="col"><?php _e('Form', 'give'); ?></th>
+				<th scope="col" width="120px"><?php _e('Actions', 'give'); ?></th>
 			</tr>
 			</thead>
 			<tbody>
-			<?php if ( ! empty( $donations ) ) { ?>
-				<?php foreach ( $donations as $donation ) : ?>
+			<?php if ( ! empty($donations)) { ?>
+				<?php foreach ($donations as $donation) : ?>
 					<tr>
 						<td><?php echo $donation->post_title; ?></td>
 						<td>
 							<?php
 							printf(
 								'<a href="%1$s" aria-label="%2$s">%3$s</a>',
-								esc_url( admin_url( 'post.php?action=edit&post=' . $donation->ID ) ),
+								esc_url(admin_url('post.php?action=edit&post='.$donation->ID)),
 								sprintf(
 								/* translators: %s: form name */
-									esc_attr__( 'View Form %s.', 'give' ),
+									esc_attr__('View Form %s.', 'give'),
 									$donation->post_title
 								),
-								__( 'View Form', 'give' )
+								__('View Form', 'give')
 							);
 							?>
 						</td>
@@ -883,7 +883,7 @@ 
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<tr>
-					<td colspan="2"><?php _e( 'No completed donations found.', 'give' ); ?></td>
+					<td colspan="2"><?php _e('No completed donations found.', 'give'); ?></td>
 				</tr>
 			<?php } ?>
 			</tbody>
@@ -897,7 +897,7 @@ 
 		 *
 		 * @param object $donor The donor object being displayed.
 		 */
-		do_action( 'give_donor_after_tables', $donor );
+		do_action('give_donor_after_tables', $donor);
 		?>
 
 	</div>
@@ -910,7 +910,7 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_card_bottom', $donor );
+	do_action('give_donor_card_bottom', $donor);
 
 }
 
@@ -923,31 +923,31 @@ 
  *
  * @return void
  */
-function give_donor_notes_view( $donor ) {
+function give_donor_notes_view($donor) {
 
-	$paged       = isset( $_GET['paged'] ) && is_numeric( $_GET['paged'] ) ? $_GET['paged'] : 1;
-	$paged       = absint( $paged );
+	$paged       = isset($_GET['paged']) && is_numeric($_GET['paged']) ? $_GET['paged'] : 1;
+	$paged       = absint($paged);
 	$note_count  = $donor->get_notes_count();
-	$per_page    = apply_filters( 'give_donor_notes_per_page', 20 );
-	$total_pages = ceil( $note_count / $per_page );
-	$donor_notes = $donor->get_notes( $per_page, $paged );
+	$per_page    = apply_filters('give_donor_notes_per_page', 20);
+	$total_pages = ceil($note_count / $per_page);
+	$donor_notes = $donor->get_notes($per_page, $paged);
 	?>
 
 	<div id="donor-notes-wrapper">
 		<div class="donor-notes-header">
-			<?php echo get_avatar( $donor->email, 30 ); ?> <span><?php echo $donor->name; ?></span>
+			<?php echo get_avatar($donor->email, 30); ?> <span><?php echo $donor->name; ?></span>
 		</div>
-		<h3><?php _e( 'Notes', 'give' ); ?></h3>
+		<h3><?php _e('Notes', 'give'); ?></h3>
 
-		<?php if ( 1 == $paged ) : ?>
+		<?php if (1 == $paged) : ?>
 			<div style="display: block; margin-bottom: 55px;">
 				<form id="give-add-donor-note" method="post"
-				      action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=notes&id=' . $donor->id ); ?>">
+				      action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=notes&id='.$donor->id); ?>">
 					<textarea id="donor-note" name="donor_note" class="donor-note-input" rows="10"></textarea>
 					<br/>
 					<input type="hidden" id="donor-id" name="customer_id" value="<?php echo $donor->id; ?>"/>
 					<input type="hidden" name="give_action" value="add-donor-note"/>
-					<?php wp_nonce_field( 'add-donor-note', 'add_donor_note_nonce', true, true ); ?>
+					<?php wp_nonce_field('add-donor-note', 'add_donor_note_nonce', true, true); ?>
 					<input id="add-donor-note" class="right button-primary" type="submit" value="Add Note"/>
 				</form>
 			</div>
@@ -962,26 +962,26 @@ 
 			'show_all' => true,
 		);
 
-		echo paginate_links( $pagination_args );
+		echo paginate_links($pagination_args);
 		?>
 
 		<div id="give-donor-notes" class="postbox">
-			<?php if ( count( $donor_notes ) > 0 ) { ?>
-				<?php foreach ( $donor_notes as $key => $note ) : ?>
+			<?php if (count($donor_notes) > 0) { ?>
+				<?php foreach ($donor_notes as $key => $note) : ?>
 					<div class="donor-note-wrapper dashboard-comment-wrap comment-item">
 					<span class="note-content-wrap">
-						<?php echo stripslashes( $note ); ?>
+						<?php echo stripslashes($note); ?>
 					</span>
 					</div>
 				<?php endforeach; ?>
 			<?php } else { ?>
 				<div class="give-no-donor-notes">
-					<?php _e( 'No donor notes found.', 'give' ); ?>
+					<?php _e('No donor notes found.', 'give'); ?>
 				</div>
 			<?php } ?>
 		</div>
 
-		<?php echo paginate_links( $pagination_args ); ?>
+		<?php echo paginate_links($pagination_args); ?>
 
 	</div>
 
@@ -997,9 +997,9 @@ 
  *
  * @return void
  */
-function give_donor_delete_view( $donor ) {
+function give_donor_delete_view($donor) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
 	/**
 	 * Fires in donor delete screen, above the content.
@@ -1008,16 +1008,16 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_delete_top', $donor );
+	do_action('give_donor_delete_top', $donor);
 	?>
 
 	<div class="info-wrapper donor-section">
 
 		<form id="delete-donor" method="post"
-		      action="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=delete&id=' . $donor->id ); ?>">
+		      action="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=delete&id='.$donor->id); ?>">
 
 			<div class="donor-notes-header">
-				<?php echo get_avatar( $donor->email, 30 ); ?> <span><?php echo $donor->name; ?></span>
+				<?php echo get_avatar($donor->email, 30); ?> <span><?php echo $donor->name; ?></span>
 			</div>
 
 
@@ -1025,20 +1025,20 @@ 
 
 				<span class="delete-donor-options">
 					<p>
-						<?php echo Give()->html->checkbox( array(
+						<?php echo Give()->html->checkbox(array(
 							'name' => 'give-donor-delete-confirm',
-						) ); ?>
-						<label for="give-donor-delete-confirm"><?php _e( 'Are you sure you want to delete this donor?', 'give' ); ?></label>
+						)); ?>
+						<label for="give-donor-delete-confirm"><?php _e('Are you sure you want to delete this donor?', 'give'); ?></label>
 					</p>
 
 					<p>
-						<?php echo Give()->html->checkbox( array(
+						<?php echo Give()->html->checkbox(array(
 							'name'    => 'give-donor-delete-records',
 							'options' => array(
 								'disabled' => true,
 							),
-						) ); ?>
-						<label for="give-donor-delete-records"><?php _e( 'Delete all associated donations and records?', 'give' ); ?></label>
+						)); ?>
+						<label for="give-donor-delete-records"><?php _e('Delete all associated donations and records?', 'give'); ?></label>
 					</p>
 
 					<?php
@@ -1051,19 +1051,19 @@ 
 					 *
 					 * @param object $donor The donor object being displayed.
 					 */
-					do_action( 'give_donor_delete_inputs', $donor );
+					do_action('give_donor_delete_inputs', $donor);
 					?>
 				</span>
 
 				<span id="donor-edit-actions">
 					<input type="hidden" name="customer_id" value="<?php echo $donor->id; ?>"/>
-					<?php wp_nonce_field( 'delete-donor', '_wpnonce', false, true ); ?>
+					<?php wp_nonce_field('delete-donor', '_wpnonce', false, true); ?>
 					<input type="hidden" name="give_action" value="delete-donor"/>
 					<input type="submit" disabled="disabled" id="give-delete-donor" class="button-primary"
-					       value="<?php _e( 'Delete Donor', 'give' ); ?>"/>
+					       value="<?php _e('Delete Donor', 'give'); ?>"/>
 					<a id="give-delete-donor-cancel"
-					   href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ); ?>"
-					   class="delete"><?php _e( 'Cancel', 'give' ); ?></a>
+					   href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id); ?>"
+					   class="delete"><?php _e('Cancel', 'give'); ?></a>
 				</span>
 
 			</div>
@@ -1079,5 +1079,5 @@ 
 	 *
 	 * @param object $donor The donor object being displayed.
 	 */
-	do_action( 'give_donor_delete_bottom', $donor );
+	do_action('give_donor_delete_bottom', $donor);
 }

Braces +5 added lines, -2 removed lines

@@ -406,8 +406,11 @@
 								<span class="donor-user-id info-item editable">
 										<?php if ( ! empty( $userdata ) ) : ?>
 											<span data-key="user_id">#<?php echo $donor->user_id . ' - ' . $userdata->display_name; ?></span>
-										<?php else: ?>
-											<span data-key="user_id"><?php _e( 'Unregistered', 'give' ); ?></span>
+										<?php else {
+	: ?>
+											<span data-key="user_id"><?php _e( 'Unregistered', 'give' );
+}
+?></span>
 										<?php endif; ?>
 									<?php if ( current_user_can( $donor_edit_role ) && intval( $donor->user_id ) > 0 ):
 

includes/admin/add-ons.php

Spacing +15 added lines, -15 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,14 +25,14 @@ 
 function give_add_ons_page() {
 	?>
 	<div class="wrap" id="give-add-ons">
-		<h1><?php echo esc_html( get_admin_page_title() ); ?>
-			&nbsp;&mdash;&nbsp;<a href="https://givewp.com/addons/" class="button-primary give-view-addons-all" target="_blank"><?php esc_html_e( 'View All Add-ons', 'give' ); ?>
+		<h1><?php echo esc_html(get_admin_page_title()); ?>
+			&nbsp;&mdash;&nbsp;<a href="https://givewp.com/addons/" class="button-primary give-view-addons-all" target="_blank"><?php esc_html_e('View All Add-ons', 'give'); ?>
 				<span class="dashicons dashicons-external"></span></a>
 		</h1>
 
 		<hr class="wp-header-end">
 
-		<p><?php esc_html_e( 'The following Add-ons extend the functionality of Give.', 'give' ); ?></p>
+		<p><?php esc_html_e('The following Add-ons extend the functionality of Give.', 'give'); ?></p>
 		<?php give_add_ons_feed(); ?>
 	</div>
 	<?php
@@ -49,27 +49,27 @@ 
 function give_add_ons_feed() {
 
 	$addons_debug = false; //set to true to debug
-	$cache        = Give_Cache::get( 'give_add_ons_feed', true );
+	$cache        = Give_Cache::get('give_add_ons_feed', true);
 
-	if ( false === $cache || ( true === $addons_debug && true === WP_DEBUG ) ) {
-		if ( function_exists( 'vip_safe_wp_remote_get' ) ) {
-			$feed = vip_safe_wp_remote_get( 'https://givewp.com/downloads/feed/', false, 3, 1, 20, array( 'sslverify' => false ) );
+	if (false === $cache || (true === $addons_debug && true === WP_DEBUG)) {
+		if (function_exists('vip_safe_wp_remote_get')) {
+			$feed = vip_safe_wp_remote_get('https://givewp.com/downloads/feed/', false, 3, 1, 20, array('sslverify' => false));
 		} else {
-			$feed = wp_remote_get( 'https://givewp.com/downloads/feed/', array( 'sslverify' => false ) );
+			$feed = wp_remote_get('https://givewp.com/downloads/feed/', array('sslverify' => false));
 		}
 
-		if ( ! is_wp_error( $feed ) && ! empty( $feed ) ) {
-			if ( isset( $feed['body'] ) && strlen( $feed['body'] ) > 0 ) {
-				$cache = wp_remote_retrieve_body( $feed );
-				Give_Cache::set( 'give_add_ons_feed', $cache, HOUR_IN_SECONDS, true );
+		if ( ! is_wp_error($feed) && ! empty($feed)) {
+			if (isset($feed['body']) && strlen($feed['body']) > 0) {
+				$cache = wp_remote_retrieve_body($feed);
+				Give_Cache::set('give_add_ons_feed', $cache, HOUR_IN_SECONDS, true);
 			}
 		} else {
 			$cache = sprintf(
 				'<div class="error"><p>%s</p></div>',
-				esc_html__( 'There was an error retrieving the Give Add-ons list from the server. Please try again later.', 'give' )
+				esc_html__('There was an error retrieving the Give Add-ons list from the server. Please try again later.', 'give')
 			);
 		}
 	}
 
-	echo wp_kses_post( $cache );
+	echo wp_kses_post($cache);
 }

includes/admin/class-blank-slate.php

Spacing +55 added lines, -55 removed lines

@@ -9,7 +9,7 @@ 
  * @since       1.8.13
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -75,63 +75,63 @@ 
 	 */
 	public function init() {
 		// Bail early if screen cannot be detected.
-		if ( empty( $this->screen ) ) {
+		if (empty($this->screen)) {
 			return null;
 		}
 
 		$content = array();
 
 		// Define content and hook into the appropriate action.
-		switch ( $this->screen ) {
+		switch ($this->screen) {
 			// Forms screen.
 			case 'edit-give_forms':
-				$this->form = $this->post_exists( 'give_forms' );
+				$this->form = $this->post_exists('give_forms');
 
-				if ( $this->form ) {
+				if ($this->form) {
 					// Form exists. Bail out.
 					return false;
 				} else {
 					// No forms exist.
-					$content = $this->get_content( 'no_forms' );
+					$content = $this->get_content('no_forms');
 				}
 
-				add_action( 'manage_posts_extra_tablenav', array( $this, 'render' ) );
+				add_action('manage_posts_extra_tablenav', array($this, 'render'));
 				break;
 			// Donations screen.
 			case 'give_forms_page_give-payment-history':
-				$this->form     = $this->post_exists( 'give_forms' );
-				$this->donation = $this->post_exists( 'give_payment' );
+				$this->form     = $this->post_exists('give_forms');
+				$this->donation = $this->post_exists('give_payment');
 
-				if ( $this->donation ) {
+				if ($this->donation) {
 					// Donation exists. Bail out.
 					return false;
-				} elseif ( ! $this->form ) {
+				} elseif ( ! $this->form) {
 					// No forms and no donations exist.
-					$content = $this->get_content( 'no_donations_or_forms' );
+					$content = $this->get_content('no_donations_or_forms');
 				} else {
 					// No donations exist but a form does exist.
-					$content = $this->get_content( 'no_donations' );
+					$content = $this->get_content('no_donations');
 				}
 
-				add_action( 'give_payments_page_bottom', array( $this, 'render' ) );
+				add_action('give_payments_page_bottom', array($this, 'render'));
 				break;
 			// Donors screen.
 			case 'give_forms_page_give-donors':
-				$this->form  = $this->post_exists( 'give_forms' );
+				$this->form  = $this->post_exists('give_forms');
 				$this->donor = $this->donor_exists();
 
-				if ( $this->donor ) {
+				if ($this->donor) {
 					// Donor exists. Bail out.
 					return false;
-				} elseif ( ! $this->form ) {
+				} elseif ( ! $this->form) {
 					// No forms and no donors exist.
-					$content = $this->get_content( 'no_donors_or_forms' );
+					$content = $this->get_content('no_donors_or_forms');
 				} else {
 					// No donors exist but a form does exist.
-					$content = $this->get_content( 'no_donors' );
+					$content = $this->get_content('no_donors');
 				}
 
-				add_action( 'give_donors_table_bottom', array( $this, 'render' ) );
+				add_action('give_donors_table_bottom', array($this, 'render'));
 				break;
 			default:
 				return null;
@@ -140,7 +140,7 @@ 
 		$this->content = $content;
 
 		// Hide non-essential UI elements.
-		add_action( 'admin_head', array( $this, 'hide_ui' ) );
+		add_action('admin_head', array($this, 'hide_ui'));
 	}
 
 	/**
@@ -150,9 +150,9 @@ 
 	 *
 	 * @param string $which The location of the list table hook: 'top' or 'bottom'.
 	 */
-	public function render( $which = 'bottom' ) {
+	public function render($which = 'bottom') {
 		// Bail out to prevent content from rendering twice.
-		if ( 'top' === $which ) {
+		if ('top' === $which) {
 			return null;
 		}
 
@@ -177,9 +177,9 @@ 
 		 *
 		 * @param string $screen The current screen ID.
 		 */
-		$content = apply_filters( 'give_blank_slate_content', $this->content, $screen );
+		$content = apply_filters('give_blank_slate_content', $this->content, $screen);
 
-		$template_path = GIVE_PLUGIN_DIR . 'includes/admin/views/blank-slate.php';
+		$template_path = GIVE_PLUGIN_DIR.'includes/admin/views/blank-slate.php';
 
 		include $template_path;
 	}
@@ -214,17 +214,17 @@ 
 	 * @param string $post_type Post type used in the query.
 	 * @return bool True if post exists, otherwise false.
 	 */
-	private function post_exists( $post_type ) {
+	private function post_exists($post_type) {
 		// Attempt to get a single post of the post type.
-		$query = new WP_Query( array(
+		$query = new WP_Query(array(
 			'post_type'              => $post_type,
 			'posts_per_page'         => 1,
 			'no_found_rows'          => false,
 			'update_post_meta_cache' => false,
 			'update_post_term_cache' => false,
 			'fields'                 => 'ids',
-			'post_status'            => array( 'any', 'trash' ),
-		) );
+			'post_status'            => array('any', 'trash'),
+		));
 
 		return $query->have_posts();
 	}
@@ -237,9 +237,9 @@ 
 	 * @return bool True if donor exists, otherwise false.
 	 */
 	private function donor_exists() {
-		$donors = Give()->donors->get_donors( array( 'number' => 1 ) );
+		$donors = Give()->donors->get_donors(array('number' => 1));
 
-		return ! empty( $donors );
+		return ! empty($donors);
 	}
 
 	/**
@@ -250,18 +250,18 @@ 
 	 * @param string $context The key used to determine which content is returned.
 	 * @return array Blank slate content.
 	 */
-	private function get_content( $context ) {
+	private function get_content($context) {
 		// Define default content.
 		$defaults = array(
-			'image_url' => GIVE_PLUGIN_URL . 'assets/dist/images/give-icon-full-circle.svg',
-			'image_alt' => __( 'Give Icon', 'give' ),
-			'heading'   => __( 'No donation forms  found.', 'give' ),
-			'message'   => __( 'The first step towards accepting online donations is to create a form.', 'give' ),
-			'cta_text'  => __( 'Create Donation Form', 'give' ),
-			'cta_link'  => admin_url( 'post-new.php?post_type=give_forms' ),
+			'image_url' => GIVE_PLUGIN_URL.'assets/dist/images/give-icon-full-circle.svg',
+			'image_alt' => __('Give Icon', 'give'),
+			'heading'   => __('No donation forms  found.', 'give'),
+			'message'   => __('The first step towards accepting online donations is to create a form.', 'give'),
+			'cta_text'  => __('Create Donation Form', 'give'),
+			'cta_link'  => admin_url('post-new.php?post_type=give_forms'),
 			'help'      => sprintf(
 				/* translators: 1: Opening anchor tag. 2: Closing anchor tag. */
-				__( 'Need help? Get started with %1$sGive 101%2$s.', 'give' ),
+				__('Need help? Get started with %1$sGive 101%2$s.', 'give'),
 				'<a href="http://docs.givewp.com/give101/" target="_blank">',
 				'</a>'
 			),
@@ -270,42 +270,42 @@ 
 		// Define contextual content.
 		$content = array(
 			'no_donations_or_forms' => array(
-				'heading' => __( 'No donations found.', 'give' ),
-				'message' => __( 'Your donation history will appear here, but first, you need a donation form!', 'give' ),
+				'heading' => __('No donations found.', 'give'),
+				'message' => __('Your donation history will appear here, but first, you need a donation form!', 'give'),
 			),
 			'no_donations'          => array(
-				'heading'  => __( 'No donations found.', 'give' ),
-				'message'  => __( 'When your first donation arrives, a record of the donation will appear here.', 'give' ),
-				'cta_text' => __( 'View All Forms', 'give' ),
-				'cta_link' => admin_url( 'edit.php?post_type=give_forms' ),
+				'heading'  => __('No donations found.', 'give'),
+				'message'  => __('When your first donation arrives, a record of the donation will appear here.', 'give'),
+				'cta_text' => __('View All Forms', 'give'),
+				'cta_link' => admin_url('edit.php?post_type=give_forms'),
 				'help'     => sprintf(
 					/* translators: 1: Opening anchor tag. 2: Closing anchor tag. */
-					__( 'Need help? Learn more about %1$sDonations%2$s.', 'give' ),
+					__('Need help? Learn more about %1$sDonations%2$s.', 'give'),
 					'<a href="http://docs.givewp.com/core-donations/">',
 					'</a>'
 				),
 			),
 			'no_donors_or_forms'    => array(
-				'heading' => __( 'No donors  found.', 'give' ),
-				'message' => __( 'Your donor history will appear here, but first, you need a donation form!', 'give' ),
+				'heading' => __('No donors  found.', 'give'),
+				'message' => __('Your donor history will appear here, but first, you need a donation form!', 'give'),
 			),
 			'no_donors'             => array(
-				'heading'  => __( 'No donors found.', 'give' ),
-				'message'  => __( 'When your first donation arrives, the donor will appear here.', 'give' ),
-				'cta_text' => __( 'View All Forms', 'give' ),
-				'cta_link' => admin_url( 'edit.php?post_type=give_forms' ),
+				'heading'  => __('No donors found.', 'give'),
+				'message'  => __('When your first donation arrives, the donor will appear here.', 'give'),
+				'cta_text' => __('View All Forms', 'give'),
+				'cta_link' => admin_url('edit.php?post_type=give_forms'),
 				'help'     => sprintf(
 					/* translators: 1: Opening anchor tag. 2: Closing anchor tag. */
-					__( 'Need help? Learn more about %1$sDonors%2$s.', 'give' ),
+					__('Need help? Learn more about %1$sDonors%2$s.', 'give'),
 					'<a href="http://docs.givewp.com/core-donors/">',
 					'</a>'
 				),
 			),
 		);
 
-		if ( isset( $content[ $context ] ) ) {
+		if (isset($content[$context])) {
 			// Merge contextual content with defaults.
-			return wp_parse_args( $content[ $context ], $defaults );
+			return wp_parse_args($content[$context], $defaults);
 		} else {
 			// Return defaults if context is undefined.
 			return $defaults;

includes/admin/tools/data/class-give-tools-recount-form-stats.php

Spacing +56 added lines, -56 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -53,8 +53,8 @@ 
 	/**
 	 * Constructor.
 	 */
-	public function __construct( $_step = 1 ) {
-		parent::__construct( $_step );
+	public function __construct($_step = 1) {
+		parent::__construct($_step);
 
 		$this->is_writable = true;
 	}
@@ -69,43 +69,43 @@ 
 	 */
 	public function get_data() {
 
-		$accepted_statuses = apply_filters( 'give_recount_accepted_statuses', array( 'publish' ) );
+		$accepted_statuses = apply_filters('give_recount_accepted_statuses', array('publish'));
 
-		if ( $this->step == 1 ) {
-			$this->delete_data( 'give_temp_recount_form_stats' );
+		if ($this->step == 1) {
+			$this->delete_data('give_temp_recount_form_stats');
 		}
 
-		$totals = $this->get_stored_data( 'give_temp_recount_form_stats' );
+		$totals = $this->get_stored_data('give_temp_recount_form_stats');
 
-		if ( false === $totals ) {
+		if (false === $totals) {
 			$totals = array(
 				'earnings' => (float) 0,
 				'sales'    => 0,
 			);
-			$this->store_data( 'give_temp_recount_form_stats', $totals );
+			$this->store_data('give_temp_recount_form_stats', $totals);
 		}
 
-		$args = apply_filters( 'give_recount_form_stats_args', array(
+		$args = apply_filters('give_recount_form_stats_args', array(
 			'give_forms' => $this->form_id,
 			'number'     => $this->per_step,
 			'status'     => $accepted_statuses,
 			'paged'      => $this->step,
 			'fields'     => 'ids',
-		) );
+		));
 
-		$payments = new Give_Payments_Query( $args );
+		$payments = new Give_Payments_Query($args);
 		$payments = $payments->get_payments();
 
-		if ( $payments ) {
-			foreach ( $payments as $payment ) {
+		if ($payments) {
+			foreach ($payments as $payment) {
 
 				// Ensure acceptable status only.
-				if ( ! in_array( $payment->post_status, $accepted_statuses ) ) {
+				if ( ! in_array($payment->post_status, $accepted_statuses)) {
 					continue;
 				}
 
 				// Ensure only payments for this form are counted.
-				if ( $payment->form_id != $this->form_id ) {
+				if ($payment->form_id != $this->form_id) {
 					continue;
 				}
 
@@ -116,25 +116,25 @@ 
 				 */
 				$earning_amount = apply_filters(
 					'give_donation_amount',
-					give_format_amount( $payment->total, array( 'donation_id' => $payment->ID ) ),
+					give_format_amount($payment->total, array('donation_id' => $payment->ID)),
 					$payment->total,
 					$payment->ID,
-					array( 'type' => 'stats', 'currency' => false, 'amount' => false )
+					array('type' => 'stats', 'currency' => false, 'amount' => false)
 				);
 
-				$totals['sales'] ++;
-				$totals['earnings'] += (float) give_maybe_sanitize_amount( $earning_amount );
+				$totals['sales']++;
+				$totals['earnings'] += (float) give_maybe_sanitize_amount($earning_amount);
 
 			}
 
-			$this->store_data( 'give_temp_recount_form_stats', $totals );
+			$this->store_data('give_temp_recount_form_stats', $totals);
 
 			return true;
 		}
 
 
-		give_update_meta( $this->form_id, '_give_form_sales', $totals['sales'] );
-		give_update_meta( $this->form_id, '_give_form_earnings', give_sanitize_amount_for_db( $totals['earnings'] ) );
+		give_update_meta($this->form_id, '_give_form_sales', $totals['sales']);
+		give_update_meta($this->form_id, '_give_form_earnings', give_sanitize_amount_for_db($totals['earnings']));
 
 		return false;
 	}
@@ -146,35 +146,35 @@ 
 	 * @return int
 	 */
 	public function get_percentage_complete() {
-		if ( $this->step == 1 ) {
-			$this->delete_data( 'give_recount_total_' . $this->form_id );
+		if ($this->step == 1) {
+			$this->delete_data('give_recount_total_'.$this->form_id);
 		}
 
-		$accepted_statuses = apply_filters( 'give_recount_accepted_statuses', array( 'publish' ) );
-		$total             = $this->get_stored_data( 'give_recount_total_' . $this->form_id );
+		$accepted_statuses = apply_filters('give_recount_accepted_statuses', array('publish'));
+		$total             = $this->get_stored_data('give_recount_total_'.$this->form_id);
 
-		if ( false === $total ) {
+		if (false === $total) {
 			$total = 0;
-			$args  = apply_filters( 'give_recount_form_stats_total_args', array(
+			$args  = apply_filters('give_recount_form_stats_total_args', array(
 				'give_forms' => $this->form_id,
-				'number'     => - 1,
+				'number'     => -1,
 				'status'     => $accepted_statuses,
 				'fields'     => 'ids',
-			) );
+			));
 
-			$payments = new Give_Payments_Query( $args );
-			$total    = count( $payments->get_payments() );
-			$this->store_data( 'give_recount_total_' . $this->form_id, $total );
+			$payments = new Give_Payments_Query($args);
+			$total    = count($payments->get_payments());
+			$this->store_data('give_recount_total_'.$this->form_id, $total);
 
 		}
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( $this->per_step * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = (($this->per_step * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -188,8 +188,8 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
-		$this->form_id = isset( $request['form_id'] ) ? sanitize_text_field( $request['form_id'] ) : false;
+	public function set_properties($request) {
+		$this->form_id = isset($request['form_id']) ? sanitize_text_field($request['form_id']) : false;
 	}
 
 	/**
@@ -200,21 +200,21 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( esc_html__( 'You do not have permission to recount stats.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(esc_html__('You do not have permission to recount stats.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
-			$this->delete_data( 'give_recount_total_' . $this->form_id );
-			$this->delete_data( 'give_temp_recount_form_stats' );
+			$this->delete_data('give_recount_total_'.$this->form_id);
+			$this->delete_data('give_temp_recount_form_stats');
 			$this->done    = true;
-			$this->message = sprintf( esc_html__( 'Donation counts and income amount statistics successfully recounted for "%s".', 'give' ), get_the_title( $this->form_id ) );
+			$this->message = sprintf(esc_html__('Donation counts and income amount statistics successfully recounted for "%s".', 'give'), get_the_title($this->form_id));
 
 			return false;
 		}
@@ -248,17 +248,17 @@ 
 	 *
 	 * @return mixed       Returns the data from the database
 	 */
-	private function get_stored_data( $key ) {
+	private function get_stored_data($key) {
 		global $wpdb;
-		$value = $wpdb->get_var( $wpdb->prepare( "SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key ) );
+		$value = $wpdb->get_var($wpdb->prepare("SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key));
 
-		if ( empty( $value ) ) {
+		if (empty($value)) {
 			return false;
 		}
 
-		$maybe_json = json_decode( $value );
-		if ( ! is_null( $maybe_json ) ) {
-			$value = json_decode( $value, true );
+		$maybe_json = json_decode($value);
+		if ( ! is_null($maybe_json)) {
+			$value = json_decode($value, true);
 		}
 
 		return $value;
@@ -274,10 +274,10 @@ 
 	 *
 	 * @return void
 	 */
-	private function store_data( $key, $value ) {
+	private function store_data($key, $value) {
 		global $wpdb;
 
-		$value = is_array( $value ) ? wp_json_encode( $value ) : esc_attr( $value );
+		$value = is_array($value) ? wp_json_encode($value) : esc_attr($value);
 
 		$data = array(
 			'option_name'  => $key,
@@ -291,7 +291,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -303,9 +303,9 @@ 
 	 *
 	 * @return void
 	 */
-	private function delete_data( $key ) {
+	private function delete_data($key) {
 		global $wpdb;
-		$wpdb->delete( $wpdb->options, array( 'option_name' => $key ) );
+		$wpdb->delete($wpdb->options, array('option_name' => $key));
 	}
 
 }
\ No newline at end of file