WordImpress / Give

includes/template-functions.php

Spacing +81 added lines, -81 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,7 +21,7 @@ 
  * @return string
  */
 function give_get_templates_dir() {
-	return GIVE_PLUGIN_DIR . 'templates';
+	return GIVE_PLUGIN_DIR.'templates';
 }
 
 /**
@@ -31,7 +31,7 @@ 
  * @return string
  */
 function give_get_templates_url() {
-	return GIVE_PLUGIN_URL . 'templates';
+	return GIVE_PLUGIN_URL.'templates';
 }
 
 /**
@@ -44,24 +44,24 @@ 
  * @param string $template_path Template file path. Default is empty.
  * @param string $default_path  Default path. Default is empty.
  */
-function give_get_template( $template_name, $args = array(), $template_path = '', $default_path = '' ) {
-	if ( ! empty( $args ) && is_array( $args ) ) {
-		extract( $args );
+function give_get_template($template_name, $args = array(), $template_path = '', $default_path = '') {
+	if ( ! empty($args) && is_array($args)) {
+		extract($args);
 	}
 
-	$template_names = array( $template_name . '.php' );
+	$template_names = array($template_name.'.php');
 
-	$located = give_locate_template( $template_names, $template_path, $default_path );
+	$located = give_locate_template($template_names, $template_path, $default_path);
 
-	if ( ! file_exists( $located ) ) {
+	if ( ! file_exists($located)) {
 		/* translators: %s: the template */
-		Give()->notices->print_frontend_notice( sprintf( __( 'The %s template was not found.', 'give' ), $located ), true );
+		Give()->notices->print_frontend_notice(sprintf(__('The %s template was not found.', 'give'), $located), true);
 
 		return;
 	}
 
 	// Allow 3rd party plugin filter template file from their plugin.
-	$located = apply_filters( 'give_get_template', $located, $template_name, $args, $template_path, $default_path );
+	$located = apply_filters('give_get_template', $located, $template_name, $args, $template_path, $default_path);
 
 	/**
 	 * Fires in give template, before the file is included.
@@ -75,9 +75,9 @@ 
 	 * @param string $located       Template file filter by 3rd party plugin.
 	 * @param array  $args          Passed arguments.
 	 */
-	do_action( 'give_before_template_part', $template_name, $template_path, $located, $args );
+	do_action('give_before_template_part', $template_name, $template_path, $located, $args);
 
-	include( $located );
+	include($located);
 
 	/**
 	 * Fires in give template, after the file is included.
@@ -91,7 +91,7 @@ 
 	 * @param string $located       Template file filter by 3rd party plugin.
 	 * @param array  $args          Passed arguments.
 	 */
-	do_action( 'give_after_template_part', $template_name, $template_path, $located, $args );
+	do_action('give_after_template_part', $template_name, $template_path, $located, $args);
 }
 
 /**
@@ -107,7 +107,7 @@ 
  *
  * @return string
  */
-function give_get_template_part( $slug, $name = null, $load = true ) {
+function give_get_template_part($slug, $name = null, $load = true) {
 
 	/**
 	 * Fires in give template part, before the template part is retrieved.
@@ -119,20 +119,20 @@ 
 	 * @param string $slug Template part file slug {slug}.php.
 	 * @param string $name Template part file name {slug}-{name}.php.
 	 */
-	do_action( "get_template_part_{$slug}", $slug, $name );
+	do_action("get_template_part_{$slug}", $slug, $name);
 
 	// Setup possible parts
 	$templates = array();
-	if ( isset( $name ) ) {
-		$templates[] = $slug . '-' . $name . '.php';
+	if (isset($name)) {
+		$templates[] = $slug.'-'.$name.'.php';
 	}
-	$templates[] = $slug . '.php';
+	$templates[] = $slug.'.php';
 
 	// Allow template parts to be filtered
-	$templates = apply_filters( 'give_get_template_part', $templates, $slug, $name );
+	$templates = apply_filters('give_get_template_part', $templates, $slug, $name);
 
 	// Return the part that is found
-	return give_locate_template( $templates, $load, false );
+	return give_locate_template($templates, $load, false);
 }
 
 /**
@@ -153,37 +153,37 @@ 
  *
  * @return string The template filename if one is located.
  */
-function give_locate_template( $template_names, $load = false, $require_once = true ) {
+function give_locate_template($template_names, $load = false, $require_once = true) {
 	// No file found yet
 	$located = false;
 
 	// Try to find a template file
-	foreach ( (array) $template_names as $template_name ) {
+	foreach ((array) $template_names as $template_name) {
 
 		// Continue if template is empty
-		if ( empty( $template_name ) ) {
+		if (empty($template_name)) {
 			continue;
 		}
 
 		// Trim off any slashes from the template name
-		$template_name = ltrim( $template_name, '/' );
+		$template_name = ltrim($template_name, '/');
 
 		// try locating this template file by looping through the template paths
-		foreach ( give_get_theme_template_paths() as $template_path ) {
+		foreach (give_get_theme_template_paths() as $template_path) {
 
-			if ( file_exists( $template_path . $template_name ) ) {
-				$located = $template_path . $template_name;
+			if (file_exists($template_path.$template_name)) {
+				$located = $template_path.$template_name;
 				break;
 			}
 		}
 
-		if ( $located ) {
+		if ($located) {
 			break;
 		}
 	}
 
-	if ( ( true == $load ) && ! empty( $located ) ) {
-		load_template( $located, $require_once );
+	if ((true == $load) && ! empty($located)) {
+		load_template($located, $require_once);
 	}
 
 	return $located;
@@ -200,17 +200,17 @@ 
 	$template_dir = give_get_theme_template_dir_name();
 
 	$file_paths = array(
-		1   => trailingslashit( get_stylesheet_directory() ) . $template_dir,
-		10  => trailingslashit( get_template_directory() ) . $template_dir,
+		1   => trailingslashit(get_stylesheet_directory()).$template_dir,
+		10  => trailingslashit(get_template_directory()).$template_dir,
 		100 => give_get_templates_dir(),
 	);
 
-	$file_paths = apply_filters( 'give_template_paths', $file_paths );
+	$file_paths = apply_filters('give_template_paths', $file_paths);
 
 	// sort the file paths based on priority
-	ksort( $file_paths, SORT_NUMERIC );
+	ksort($file_paths, SORT_NUMERIC);
 
-	return array_map( 'trailingslashit', $file_paths );
+	return array_map('trailingslashit', $file_paths);
 }
 
 /**
@@ -222,7 +222,7 @@ 
  * @return string
  */
 function give_get_theme_template_dir_name() {
-	return trailingslashit( apply_filters( 'give_templates_dir', 'give' ) );
+	return trailingslashit(apply_filters('give_templates_dir', 'give'));
 }
 
 /**
@@ -232,10 +232,10 @@ 
  * @return void
  */
 function give_version_in_header() {
-	echo '<meta name="generator" content="Give v' . GIVE_VERSION . '" />' . "\n";
+	echo '<meta name="generator" content="Give v'.GIVE_VERSION.'" />'."\n";
 }
 
-add_action( 'wp_head', 'give_version_in_header' );
+add_action('wp_head', 'give_version_in_header');
 
 /**
  * Determines if we're currently on the Donations History page.
@@ -245,9 +245,9 @@ 
  */
 function give_is_donation_history_page() {
 
-	$ret = is_page( give_get_option( 'history_page' ) );
+	$ret = is_page(give_get_option('history_page'));
 
-	return apply_filters( 'give_is_donation_history_page', $ret );
+	return apply_filters('give_is_donation_history_page', $ret);
 }
 
 /**
@@ -259,25 +259,25 @@ 
  *
  * @return array Modified array of classes
  */
-function give_add_body_classes( $class ) {
+function give_add_body_classes($class) {
 	$classes = (array) $class;
 
-	if ( give_is_success_page() ) {
+	if (give_is_success_page()) {
 		$classes[] = 'give-success';
 		$classes[] = 'give-page';
 	}
 
-	if ( give_is_failed_transaction_page() ) {
+	if (give_is_failed_transaction_page()) {
 		$classes[] = 'give-failed-transaction';
 		$classes[] = 'give-page';
 	}
 
-	if ( give_is_donation_history_page() ) {
+	if (give_is_donation_history_page()) {
 		$classes[] = 'give-donation-history';
 		$classes[] = 'give-page';
 	}
 
-	if ( give_is_test_mode() ) {
+	if (give_is_test_mode()) {
 		$classes[] = 'give-test-mode';
 		$classes[] = 'give-page';
 	}
@@ -286,7 +286,7 @@ 
 	/* @var WP_Theme $current_theme */
 	$current_theme = wp_get_theme();
 
-	switch ( $current_theme->get_template() ) {
+	switch ($current_theme->get_template()) {
 
 		case 'Divi':
 			$classes[] = 'give-divi';
@@ -303,10 +303,10 @@ 
 
 	}
 
-	return array_unique( $classes );
+	return array_unique($classes);
 }
 
-add_filter( 'body_class', 'give_add_body_classes' );
+add_filter('body_class', 'give_add_body_classes');
 
 
 /**
@@ -322,22 +322,22 @@ 
  *
  * @return array
  */
-function give_add_post_class( $classes, $class = '', $post_id = '' ) {
-	if ( ! $post_id || 'give_forms' !== get_post_type( $post_id ) ) {
+function give_add_post_class($classes, $class = '', $post_id = '') {
+	if ( ! $post_id || 'give_forms' !== get_post_type($post_id)) {
 		return $classes;
 	}
 
 	//@TODO: Add classes for custom taxonomy and form configurations (multi vs single donations, etc).
 
-	if ( false !== ( $key = array_search( 'hentry', $classes ) ) ) {
-		unset( $classes[ $key ] );
+	if (false !== ($key = array_search('hentry', $classes))) {
+		unset($classes[$key]);
 	}
 
 	return $classes;
 }
 
 
-add_filter( 'post_class', 'give_add_post_class', 20, 3 );
+add_filter('post_class', 'give_add_post_class', 20, 3);
 
 /**
  * Get the placeholder image URL for forms etc
@@ -347,74 +347,74 @@ 
  */
 function give_get_placeholder_img_src() {
 
-	$placeholder_url = '//placehold.it/600x600&text=' . urlencode( esc_attr__( 'Give Placeholder Image', 'give' ) );
+	$placeholder_url = '//placehold.it/600x600&text='.urlencode(esc_attr__('Give Placeholder Image', 'give'));
 
-	return apply_filters( 'give_placeholder_img_src', $placeholder_url );
+	return apply_filters('give_placeholder_img_src', $placeholder_url);
 }
 
 
 /**
  * Global
  */
-if ( ! function_exists( 'give_output_content_wrapper' ) ) {
+if ( ! function_exists('give_output_content_wrapper')) {
 
 	/**
 	 * Output the start of the page wrapper.
 	 */
 	function give_output_content_wrapper() {
-		give_get_template_part( 'global/wrapper-start' );
+		give_get_template_part('global/wrapper-start');
 	}
 }
-if ( ! function_exists( 'give_output_content_wrapper_end' ) ) {
+if ( ! function_exists('give_output_content_wrapper_end')) {
 
 	/**
 	 * Output the end of the page wrapper.
 	 */
 	function give_output_content_wrapper_end() {
-		give_get_template_part( 'global/wrapper-end' );
+		give_get_template_part('global/wrapper-end');
 	}
 }
 
 /**
  * Single Give Form
  */
-if ( ! function_exists( 'give_left_sidebar_pre_wrap' ) ) {
+if ( ! function_exists('give_left_sidebar_pre_wrap')) {
 	function give_left_sidebar_pre_wrap() {
-		echo apply_filters( 'give_left_sidebar_pre_wrap', '<div id="give-sidebar-left" class="give-sidebar give-single-form-sidebar-left">' );
+		echo apply_filters('give_left_sidebar_pre_wrap', '<div id="give-sidebar-left" class="give-sidebar give-single-form-sidebar-left">');
 	}
 }
 
-if ( ! function_exists( 'give_left_sidebar_post_wrap' ) ) {
+if ( ! function_exists('give_left_sidebar_post_wrap')) {
 	function give_left_sidebar_post_wrap() {
-		echo apply_filters( 'give_left_sidebar_post_wrap', '</div>' );
+		echo apply_filters('give_left_sidebar_post_wrap', '</div>');
 	}
 }
 
-if ( ! function_exists( 'give_get_forms_sidebar' ) ) {
+if ( ! function_exists('give_get_forms_sidebar')) {
 	function give_get_forms_sidebar() {
-		give_get_template_part( 'single-give-form/sidebar' );
+		give_get_template_part('single-give-form/sidebar');
 	}
 }
 
-if ( ! function_exists( 'give_show_form_images' ) ) {
+if ( ! function_exists('give_show_form_images')) {
 
 	/**
 	 * Output the donation form featured image.
 	 */
 	function give_show_form_images() {
-		if ( give_is_setting_enabled( give_get_option( 'form_featured_img' ) ) ) {
-			give_get_template_part( 'single-give-form/featured-image' );
+		if (give_is_setting_enabled(give_get_option('form_featured_img'))) {
+			give_get_template_part('single-give-form/featured-image');
 		}
 	}
 }
 
-if ( ! function_exists( 'give_template_single_title' ) ) {
+if ( ! function_exists('give_template_single_title')) {
 
 	/**
 	 * Output the form title.
 	 */
 	function give_template_single_title() {
-		give_get_template_part( 'single-give-form/title' );
+		give_get_template_part('single-give-form/title');
 	}
 }
 
@@ -422,7 +422,7 @@ 
  * Conditional Functions
  */
 
-if ( ! function_exists( 'is_give_form' ) ) {
+if ( ! function_exists('is_give_form')) {
 
 	/**
 	 * is_give_form
@@ -434,11 +434,11 @@ 
 	 * @return bool
 	 */
 	function is_give_form() {
-		return is_singular( array( 'give_form' ) );
+		return is_singular(array('give_form'));
 	}
 }
 
-if ( ! function_exists( 'is_give_category' ) ) {
+if ( ! function_exists('is_give_category')) {
 
 	/**
 	 * is_give_category
@@ -453,12 +453,12 @@ 
 	 *
 	 * @return bool
 	 */
-	function is_give_category( $term = '' ) {
-		return is_tax( 'give_forms_category', $term );
+	function is_give_category($term = '') {
+		return is_tax('give_forms_category', $term);
 	}
 }
 
-if ( ! function_exists( 'is_give_tag' ) ) {
+if ( ! function_exists('is_give_tag')) {
 
 	/**
 	 * is_give_tag
@@ -473,12 +473,12 @@ 
 	 *
 	 * @return bool
 	 */
-	function is_give_tag( $term = '' ) {
-		return is_tax( 'give_forms_tag', $term );
+	function is_give_tag($term = '') {
+		return is_tax('give_forms_tag', $term);
 	}
 }
 
-if ( ! function_exists( 'is_give_taxonomy' ) ) {
+if ( ! function_exists('is_give_taxonomy')) {
 
 	/**
 	 * is_give_taxonomy
@@ -490,6 +490,6 @@ 
 	 * @return bool
 	 */
 	function is_give_taxonomy() {
-		return is_tax( get_object_taxonomies( 'give_form' ) );
+		return is_tax(get_object_taxonomies('give_form'));
 	}
 }

uninstall.php

Spacing +40 added lines, -40 removed lines

@@ -10,61 +10,61 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'WP_UNINSTALL_PLUGIN' ) ) {
+if ( ! defined('WP_UNINSTALL_PLUGIN')) {
 	exit;
 }
 
 // Load Give file.
-include_once( 'give.php' );
+include_once('give.php');
 
 global $wpdb, $wp_roles;
 
 
-if ( give_is_setting_enabled( give_get_option( 'uninstall_on_delete' ) ) ) {
+if (give_is_setting_enabled(give_get_option('uninstall_on_delete'))) {
 
 	// Delete All the Custom Post Types.
-	$give_taxonomies = array( 'form_category', 'form_tag', 'give_log_type' );
-	$give_post_types = array( 'give_forms', 'give_payment', 'give_log' );
-	foreach ( $give_post_types as $post_type ) {
+	$give_taxonomies = array('form_category', 'form_tag', 'give_log_type');
+	$give_post_types = array('give_forms', 'give_payment', 'give_log');
+	foreach ($give_post_types as $post_type) {
 
-		$give_taxonomies = array_merge( $give_taxonomies, get_object_taxonomies( $post_type ) );
-		$items           = get_posts( array(
+		$give_taxonomies = array_merge($give_taxonomies, get_object_taxonomies($post_type));
+		$items           = get_posts(array(
 			'post_type'   => $post_type,
 			'post_status' => 'any',
-			'numberposts' => - 1,
+			'numberposts' => -1,
 			'fields'      => 'ids',
-		) );
+		));
 
-		if ( $items ) {
-			foreach ( $items as $item ) {
-				wp_delete_post( $item, true );
+		if ($items) {
+			foreach ($items as $item) {
+				wp_delete_post($item, true);
 			}
 		}
 	}
 
 	// Delete All the Terms & Taxonomies.
-	foreach ( array_unique( array_filter( $give_taxonomies ) ) as $taxonomy ) {
+	foreach (array_unique(array_filter($give_taxonomies)) as $taxonomy) {
 
-		$terms = $wpdb->get_results( $wpdb->prepare( "SELECT t.*, tt.* FROM $wpdb->terms AS t INNER JOIN $wpdb->term_taxonomy AS tt ON t.term_id = tt.term_id WHERE tt.taxonomy IN ('%s') ORDER BY t.name ASC", $taxonomy ) );
+		$terms = $wpdb->get_results($wpdb->prepare("SELECT t.*, tt.* FROM $wpdb->terms AS t INNER JOIN $wpdb->term_taxonomy AS tt ON t.term_id = tt.term_id WHERE tt.taxonomy IN ('%s') ORDER BY t.name ASC", $taxonomy));
 
 		// Delete Terms.
-		if ( $terms ) {
-			foreach ( $terms as $term ) {
-				$wpdb->delete( $wpdb->term_taxonomy, array( 'term_taxonomy_id' => $term->term_taxonomy_id ) );
-				$wpdb->delete( $wpdb->terms, array( 'term_id' => $term->term_id ) );
+		if ($terms) {
+			foreach ($terms as $term) {
+				$wpdb->delete($wpdb->term_taxonomy, array('term_taxonomy_id' => $term->term_taxonomy_id));
+				$wpdb->delete($wpdb->terms, array('term_id' => $term->term_id));
 			}
 		}
 
 		// Delete Taxonomies.
-		$wpdb->delete( $wpdb->term_taxonomy, array( 'taxonomy' => $taxonomy ), array( '%s' ) );
+		$wpdb->delete($wpdb->term_taxonomy, array('taxonomy' => $taxonomy), array('%s'));
 	}
 
 	// Delete the Plugin Pages.
-	$give_created_pages = array( 'success_page', 'failure_page', 'history_page' );
-	foreach ( $give_created_pages as $p ) {
-		$page = give_get_option( $p, false );
-		if ( $page ) {
-			wp_delete_post( $page, true );
+	$give_created_pages = array('success_page', 'failure_page', 'history_page');
+	foreach ($give_created_pages as $p) {
+		$page = give_get_option($p, false);
+		if ($page) {
+			wp_delete_post($page, true);
 		}
 	}
 
@@ -73,21 +73,21 @@ 
 	Give()->roles->remove_caps();
 
 	// Delete the Roles.
-	$give_roles = array( 'give_manager', 'give_accountant', 'give_worker', 'give_donor' );
-	foreach ( $give_roles as $role ) {
-		remove_role( $role );
+	$give_roles = array('give_manager', 'give_accountant', 'give_worker', 'give_donor');
+	foreach ($give_roles as $role) {
+		remove_role($role);
 	}
 
 	// Remove all database tables.
-	$wpdb->query( 'DROP TABLE IF EXISTS ' . $wpdb->prefix . 'give_donors' );
-	$wpdb->query( 'DROP TABLE IF EXISTS ' . $wpdb->prefix . 'give_customers' );
-	$wpdb->query( 'DROP TABLE IF EXISTS ' . $wpdb->prefix . 'give_customermeta' );
+	$wpdb->query('DROP TABLE IF EXISTS '.$wpdb->prefix.'give_donors');
+	$wpdb->query('DROP TABLE IF EXISTS '.$wpdb->prefix.'give_customers');
+	$wpdb->query('DROP TABLE IF EXISTS '.$wpdb->prefix.'give_customermeta');
 
 	// Cleanup Cron Events.
-	wp_clear_scheduled_hook( 'give_daily_scheduled_events' );
-	wp_clear_scheduled_hook( 'give_weekly_scheduled_events' );
-	wp_clear_scheduled_hook( 'give_daily_cron' );
-	wp_clear_scheduled_hook( 'give_weekly_cron' );
+	wp_clear_scheduled_hook('give_daily_scheduled_events');
+	wp_clear_scheduled_hook('give_weekly_scheduled_events');
+	wp_clear_scheduled_hook('give_daily_cron');
+	wp_clear_scheduled_hook('give_weekly_cron');
 
 	// Get all options.
 	$give_option_names = $wpdb->get_col(
@@ -97,16 +97,16 @@ 
 		)
 	);
 
-	if ( ! empty( $give_option_names ) ) {
+	if ( ! empty($give_option_names)) {
 		// Convert option name to transient or option name.
 		$new_give_option_names = array();
 
 		// Delete all the Plugin Options.
-		foreach ( $give_option_names as $option ) {
-			if ( false !== strpos( $option, 'give_cache' ) ) {
-				Give_Cache::delete( $option );
+		foreach ($give_option_names as $option) {
+			if (false !== strpos($option, 'give_cache')) {
+				Give_Cache::delete($option);
 			} else {
-				delete_option( $option );
+				delete_option($option);
 			}
 		}
 	}

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

Spacing +35 added lines, -35 removed lines

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

includes/admin/admin-filters.php

Doc Comments +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
  *
  * @param   string $value
  *
- * @return  mixed
+ * @return  string
  */
 function __give_validate_decimal_separator_setting_field( $value ) {
 	$thousand_separator = give_clean( $_POST['thousands_separator'] );

Spacing +58 added lines, -58 removed lines

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

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

Doc Comments +2 added lines, -2 removed lines

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

Indentation +2 added lines, -2 removed lines

@@ -871,8 +871,8 @@
 		 */
 		private function is_donations_import_page() {
 			return 'import' === give_get_current_setting_tab() &&
-			       isset( $_GET['importer-type'] ) &&
-			       $this->importer_type === give_clean( $_GET['importer-type'] );
+				   isset( $_GET['importer-type'] ) &&
+				   $this->importer_type === give_clean( $_GET['importer-type'] );
 		}
 	}
 

Spacing +189 added lines, -195 removed lines

@@ -11,11 +11,11 @@ 
  * @since       1.8.14
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Import_Donations' ) ) {
+if ( ! class_exists('Give_Import_Donations')) {
 
 	/**
 	 * Give_Import_Donations.
@@ -57,7 +57,7 @@ 
 		 * @access private
 		 */
 		private function __construct() {
-			self::$per_page  = ! empty( $_GET['per_page'] ) ? absint( $_GET['per_page'] ) : self::$per_page;
+			self::$per_page = ! empty($_GET['per_page']) ? absint($_GET['per_page']) : self::$per_page;
 		}
 
 		/**
@@ -69,7 +69,7 @@ 
 		 * @return static
 		 */
 		public static function get_instance() {
-			if ( null === static::$instance ) {
+			if (null === static::$instance) {
 				self::$instance = new static();
 			}
 
@@ -96,27 +96,27 @@ 
 		 * @return void
 		 */
 		private function setup_hooks() {
-			if ( ! $this->is_donations_import_page() ) {
+			if ( ! $this->is_donations_import_page()) {
 				return;
 			}
 
 			// Do not render main import tools page.
-			remove_action( 'give_admin_field_tools_import', array( 'Give_Settings_Import', 'render_import_field', ) );
+			remove_action('give_admin_field_tools_import', array('Give_Settings_Import', 'render_import_field',));
 
 
 			// Render donation import page
-			add_action( 'give_admin_field_tools_import', array( $this, 'render_page' ) );
+			add_action('give_admin_field_tools_import', array($this, 'render_page'));
 
 			// Print the HTML.
-			add_action( 'give_tools_import_donations_form_start', array( $this, 'html' ), 10 );
+			add_action('give_tools_import_donations_form_start', array($this, 'html'), 10);
 
 			// Run when form submit.
-			add_action( 'give-tools_save_import', array( $this, 'save' ) );
+			add_action('give-tools_save_import', array($this, 'save'));
 
-			add_action( 'give-tools_update_notices', array( $this, 'update_notices' ), 11, 1 );
+			add_action('give-tools_update_notices', array($this, 'update_notices'), 11, 1);
 
 			// Used to add submit button.
-			add_action( 'give_tools_import_donations_form_end', array( $this, 'submit' ), 10 );
+			add_action('give_tools_import_donations_form_end', array($this, 'submit'), 10);
 		}
 
 		/**
@@ -128,9 +128,9 @@ 
 		 *
 		 * @return mixed
 		 */
-		public function update_notices( $messages ) {
-			if ( ! empty( $_GET['tab'] ) && 'import' === give_clean( $_GET['tab'] ) ) {
-				unset( $messages['give-setting-updated'] );
+		public function update_notices($messages) {
+			if ( ! empty($_GET['tab']) && 'import' === give_clean($_GET['tab'])) {
+				unset($messages['give-setting-updated']);
 			}
 
 			return $messages;
@@ -142,7 +142,7 @@ 
 		 * @since 1.8.14
 		 */
 		public function submit() {
-			wp_nonce_field( 'give-save-settings', '_give-save-settings' );
+			wp_nonce_field('give-save-settings', '_give-save-settings');
 			?>
 			<input type="hidden" class="import-step" id="import-step" name="step" value="<?php echo $this->get_step(); ?>"/>
 			<input type="hidden" class="importer-type" value="<?php echo $this->importer_type; ?>"/>
@@ -164,7 +164,7 @@ 
 				<table class="widefat export-options-table give-table <?php echo "step-{$step}"; ?>" id="<?php echo "step-{$step}"; ?>">
 					<tbody>
 						<?php
-						switch ( $this->get_step() ) {
+						switch ($this->get_step()) {
 							case 1:
 								$this->render_media_csv();
 								break;
@@ -181,7 +181,7 @@ 
 								$this->import_success();
 						}
 
-						if ( false === $this->check_for_dropdown_or_import() ) {
+						if (false === $this->check_for_dropdown_or_import()) {
 							?>
 							<tr valign="top">
 								<th></th>
@@ -189,7 +189,7 @@ 
 									<input type="submit"
 										   class="button button-primary button-large button-secondary <?php echo "step-{$step}"; ?>"
 										   id="recount-stats-submit"
-										   value="<?php esc_attr_e( 'Submit', 'give' ); ?>"/>
+										   value="<?php esc_attr_e('Submit', 'give'); ?>"/>
 								</th>
 							</tr>
 							<?php
@@ -208,56 +208,56 @@ 
 		 */
 		public function import_success() {
 
-			$delete_csv = ( ! empty( $_GET['delete_csv'] ) ? absint( $_GET['delete_csv'] ) : false );
-			$csv        = ( ! empty( $_GET['csv'] ) ? absint( $_GET['csv'] ) : false );
-			if ( ! empty( $delete_csv ) && ! empty( $csv ) ) {
-				wp_delete_attachment( $csv, true );
+			$delete_csv = ( ! empty($_GET['delete_csv']) ? absint($_GET['delete_csv']) : false);
+			$csv        = ( ! empty($_GET['csv']) ? absint($_GET['csv']) : false);
+			if ( ! empty($delete_csv) && ! empty($csv)) {
+				wp_delete_attachment($csv, true);
 			}
 
 			$report      = give_import_donation_report();
 			$report_html = array(
 				'duplicate_donor'    => array(
-					__( '%s duplicate %s detected', 'give' ),
-					__( 'donor', 'give' ),
-					__( 'donors', 'give' ),
+					__('%s duplicate %s detected', 'give'),
+					__('donor', 'give'),
+					__('donors', 'give'),
 				),
 				'create_donor'       => array(
-					__( '%s %s created', 'give' ),
-					__( 'donor', 'give' ),
-					__( 'donors', 'give' ),
+					__('%s %s created', 'give'),
+					__('donor', 'give'),
+					__('donors', 'give'),
 				),
 				'create_form'        => array(
-					__( '%s donation %s created', 'give' ),
-					__( 'form', 'give' ),
-					__( 'forms', 'give' ),
+					__('%s donation %s created', 'give'),
+					__('form', 'give'),
+					__('forms', 'give'),
 				),
 				'duplicate_donation' => array(
-					__( '%s duplicate %s detected', 'give' ),
-					__( 'donation', 'give' ),
-					__( 'donations', 'give' ),
+					__('%s duplicate %s detected', 'give'),
+					__('donation', 'give'),
+					__('donations', 'give'),
 				),
 				'create_donation'    => array(
-					__( '%s %s imported', 'give' ),
-					__( 'donation', 'give' ),
-					__( 'donations', 'give' ),
+					__('%s %s imported', 'give'),
+					__('donation', 'give'),
+					__('donations', 'give'),
 				),
 			);
-			$total       = (int) $_GET['total'];
-			-- $total;
+			$total = (int) $_GET['total'];
+			--$total;
 			$success = (bool) $_GET['success'];
 			?>
 			<tr valign="top" class="give-import-dropdown">
 				<th colspan="2">
 					<h2>
 						<?php
-						if ( $success ) {
+						if ($success) {
 							echo sprintf(
-								__( 'Import complete! %s donations processed', 'give' ),
+								__('Import complete! %s donations processed', 'give'),
 								"<strong>{$total}</strong>"
 							);
 						} else {
 							echo sprintf(
-								__( 'Failed to import %s donations', 'give' ),
+								__('Failed to import %s donations', 'give'),
 								"<strong>{$total}</strong>"
 							);
 						}
@@ -265,25 +265,25 @@ 
 					</h2>
 
 					<?php
-					$text      = __( 'Import Donation', 'give' );
+					$text      = __('Import Donation', 'give');
 					$query_arg = array(
 						'post_type' => 'give_forms',
 						'page'      => 'give-tools',
 						'tab'       => 'import',
 					);
-					if ( $success ) {
+					if ($success) {
 						$query_arg = array(
 							'post_type' => 'give_forms',
 							'page'      => 'give-payment-history',
 						);
-						$text      = __( 'View Donations', 'give' );
+						$text = __('View Donations', 'give');
 					}
 
-					foreach ( $report as $key => $value ) {
-						if ( array_key_exists( $key, $report_html ) && ! empty( $value ) ) {
+					foreach ($report as $key => $value) {
+						if (array_key_exists($key, $report_html) && ! empty($value)) {
 							?>
 							<p>
-								<?php echo esc_html( wp_sprintf( $report_html[ $key ][0], $value, _n( $report_html[ $key ][1], $report_html[ $key ][2], $value, 'give' ) ) ); ?>
+								<?php echo esc_html(wp_sprintf($report_html[$key][0], $value, _n($report_html[$key][1], $report_html[$key][2], $value, 'give'))); ?>
 							</p>
 							<?php
 						}
@@ -291,7 +291,7 @@ 
 					?>
 
 					<p>
-						<a class="button button-large button-secondary" href="<?php echo add_query_arg( $query_arg, admin_url( 'edit.php' ) ); ?>"><?php echo $text; ?></a>
+						<a class="button button-large button-secondary" href="<?php echo add_query_arg($query_arg, admin_url('edit.php')); ?>"><?php echo $text; ?></a>
 					</p>
 				</th>
 			</tr>
@@ -308,26 +308,26 @@ 
 			give_import_donation_report_reset();
 
 			$csv         = (int) $_REQUEST['csv'];
-			$delimiter   = ( ! empty( $_REQUEST['delimiter'] ) ? give_clean( $_REQUEST['delimiter'] ) : 'csv' );
+			$delimiter   = ( ! empty($_REQUEST['delimiter']) ? give_clean($_REQUEST['delimiter']) : 'csv');
 			$index_start = 1;
 			$index_end   = 1;
 			$next        = true;
-			$total       = self::get_csv_total( $csv );
-			if ( self::$per_page < $total ) {
-				$total_ajax = ceil( $total / self::$per_page );
+			$total       = self::get_csv_total($csv);
+			if (self::$per_page < $total) {
+				$total_ajax = ceil($total / self::$per_page);
 				$index_end  = self::$per_page;
 			} else {
 				$total_ajax = 1;
 				$index_end  = $total;
 				$next       = false;
 			}
-			$current_percentage = 100 / ( $total_ajax + 1 );
+			$current_percentage = 100 / ($total_ajax + 1);
 
 			?>
 			<tr valign="top" class="give-import-dropdown">
 				<th colspan="2">
-					<h2 id="give-import-title"><?php esc_html_e( 'Importing', 'give' ) ?></h2>
-					<p class="give-field-description"><?php esc_html_e( 'Your donations are now being imported...', 'give' ) ?></p>
+					<h2 id="give-import-title"><?php esc_html_e('Importing', 'give') ?></h2>
+					<p class="give-field-description"><?php esc_html_e('Your donations are now being imported...', 'give') ?></p>
 				</th>
 			</tr>
 
@@ -346,7 +346,7 @@ 
 						<div style="width: <?php echo $current_percentage; ?>%"></div>
 					</div>
 					<input type="hidden" value="3" name="step">
-					<input type="hidden" value='<?php echo maybe_serialize( $_REQUEST['mapto'] ); ?>' name="mapto"
+					<input type="hidden" value='<?php echo maybe_serialize($_REQUEST['mapto']); ?>' name="mapto"
 						   class="mapto">
 					<input type="hidden" value="<?php echo $_REQUEST['csv']; ?>" name="csv" class="csv">
 					<input type="hidden" value="<?php echo $_REQUEST['mode']; ?>" name="mode" class="mode">
@@ -355,7 +355,7 @@ 
 					<input type="hidden" value="<?php echo $_REQUEST['delete_csv']; ?>" name="delete_csv"
 						   class="delete_csv">
 					<input type="hidden" value="<?php echo $delimiter; ?>" name="delimiter">
-					<input type="hidden" value='<?php echo maybe_serialize( self::get_importer( $csv, 0, $delimiter ) ); ?>'
+					<input type="hidden" value='<?php echo maybe_serialize(self::get_importer($csv, 0, $delimiter)); ?>'
 						   name="main_key"
 						   class="main_key">
 				</th>
@@ -376,20 +376,20 @@ 
 		 */
 		public function check_for_dropdown_or_import() {
 			$return = true;
-			if ( isset( $_REQUEST['mapto'] ) ) {
+			if (isset($_REQUEST['mapto'])) {
 				$mapto = (array) $_REQUEST['mapto'];
-				if ( false === in_array( 'form_title', $mapto ) && false === in_array( 'form_id', $mapto ) ) {
-					Give_Admin_Settings::add_error( 'give-import-csv-form', __( 'In order to import donations, a column must be mapped to either the "Donation Form Title" or "Donation Form ID" field. Please map a column to one of those fields.', 'give' ) );
+				if (false === in_array('form_title', $mapto) && false === in_array('form_id', $mapto)) {
+					Give_Admin_Settings::add_error('give-import-csv-form', __('In order to import donations, a column must be mapped to either the "Donation Form Title" or "Donation Form ID" field. Please map a column to one of those fields.', 'give'));
 					$return = false;
 				}
 
-				if ( false === in_array( 'amount', $mapto ) ) {
-					Give_Admin_Settings::add_error( 'give-import-csv-amount', __( 'In order to import donations, a column must be mapped to the "Amount" field. Please map a column to that field.', 'give' ) );
+				if (false === in_array('amount', $mapto)) {
+					Give_Admin_Settings::add_error('give-import-csv-amount', __('In order to import donations, a column must be mapped to the "Amount" field. Please map a column to that field.', 'give'));
 					$return = false;
 				}
 
-				if ( false === in_array( 'email', $mapto ) && false === in_array( 'donor_id', $mapto ) ) {
-					Give_Admin_Settings::add_error( 'give-import-csv-donor', __( 'In order to import donations, a column must be mapped to either the "Donor Email" or "Donor ID" field. Please map a column to that field.', 'give' ) );
+				if (false === in_array('email', $mapto) && false === in_array('donor_id', $mapto)) {
+					Give_Admin_Settings::add_error('give-import-csv-donor', __('In order to import donations, a column must be mapped to either the "Donor Email" or "Donor ID" field. Please map a column to that field.', 'give'));
 					$return = false;
 				}
 			} else {
@@ -406,10 +406,10 @@ 
 		 */
 		public function render_dropdown() {
 			$csv       = (int) $_GET['csv'];
-			$delimiter = ( ! empty( $_GET['delimiter'] ) ? give_clean( $_GET['delimiter'] ) : 'csv' );
+			$delimiter = ( ! empty($_GET['delimiter']) ? give_clean($_GET['delimiter']) : 'csv');
 
 			// TO check if the CSV files that is being add is valid or not if not then redirect to first step again
-			if ( ! $this->is_valid_csv( $csv ) ) {
+			if ( ! $this->is_valid_csv($csv)) {
 				$url = give_import_page_url();
 				?>
 				<script type="text/javascript">
@@ -420,27 +420,27 @@ 
 				?>
 				<tr valign="top" class="give-import-dropdown">
 					<th colspan="2">
-						<h2 id="give-import-title"><?php esc_html_e( 'Map CSV fields to donations', 'give' ) ?></h2>
-						<p class="give-field-description"><?php esc_html_e( 'Select fields from your CSV file to map against donations fields or to ignore during import.', 'give' ) ?></p>
+						<h2 id="give-import-title"><?php esc_html_e('Map CSV fields to donations', 'give') ?></h2>
+						<p class="give-field-description"><?php esc_html_e('Select fields from your CSV file to map against donations fields or to ignore during import.', 'give') ?></p>
 					</th>
 				</tr>
 
 				<tr valign="top" class="give-import-dropdown">
-					<th><b><?php esc_html_e( 'Column name', 'give' ); ?></b></th>
-					<th><b><?php esc_html_e( 'Map to field', 'give' ); ?></b></th>
+					<th><b><?php esc_html_e('Column name', 'give'); ?></b></th>
+					<th><b><?php esc_html_e('Map to field', 'give'); ?></b></th>
 				</tr>
 
 				<?php
-				$raw_key   = $this->get_importer( $csv, 0, $delimiter );
-				$mapto     = (array) ( isset( $_REQUEST['mapto'] ) ? $_REQUEST['mapto'] : array() );
+				$raw_key   = $this->get_importer($csv, 0, $delimiter);
+				$mapto     = (array) (isset($_REQUEST['mapto']) ? $_REQUEST['mapto'] : array());
 
-				foreach ( $raw_key as $index => $value ) {
+				foreach ($raw_key as $index => $value) {
 					?>
 					<tr valign="top" class="give-import-option">
 						<th><?php echo $value; ?></th>
 						<th>
 							<?php
-							$this->get_columns( $index, $value, $mapto );
+							$this->get_columns($index, $value, $mapto);
 							?>
 						</th>
 					</tr>
@@ -455,14 +455,14 @@ 
 		 *
 		 * @return string
 		 */
-		public function selected( $option_value, $value ) {
-			$option_value = strtolower( $option_value );
-			$value = strtolower( $value );
+		public function selected($option_value, $value) {
+			$option_value = strtolower($option_value);
+			$value = strtolower($value);
 
 			$selected = '';
-			if ( stristr( $value, $option_value ) ) {
+			if (stristr($value, $option_value)) {
 				$selected = 'selected';
-			} elseif ( strrpos( $value, '_' ) && stristr( $option_value, __( 'Import as Meta', 'give' ) ) ) {
+			} elseif (strrpos($value, '_') && stristr($option_value, __('Import as Meta', 'give'))) {
 				$selected = 'selected';
 			}
 
@@ -481,28 +481,28 @@ 
 		 *
 		 * @return void
 		 */
-		private function get_columns( $index, $value = false, $mapto = array() ) {
+		private function get_columns($index, $value = false, $mapto = array()) {
 			$default       = give_import_default_options();
-			$current_mapto = (string) ( ! empty( $mapto[ $index ] ) ? $mapto[ $index ] : '' );
+			$current_mapto = (string) ( ! empty($mapto[$index]) ? $mapto[$index] : '');
 			?>
 			<select name="mapto[<?php echo $index; ?>]">
-				<?php $this->get_dropdown_option_html( $default, $current_mapto, $value ); ?>
+				<?php $this->get_dropdown_option_html($default, $current_mapto, $value); ?>
 
-				<optgroup label="<?php _e( 'Donations', 'give' ); ?>">
+				<optgroup label="<?php _e('Donations', 'give'); ?>">
 					<?php
-					$this->get_dropdown_option_html( give_import_donations_options(), $current_mapto, $value );
+					$this->get_dropdown_option_html(give_import_donations_options(), $current_mapto, $value);
 					?>
 				</optgroup>
 
-				<optgroup label="<?php _e( 'Donors', 'give' ); ?>">
+				<optgroup label="<?php _e('Donors', 'give'); ?>">
 					<?php
-					$this->get_dropdown_option_html( give_import_donor_options(), $current_mapto, $value );
+					$this->get_dropdown_option_html(give_import_donor_options(), $current_mapto, $value);
 					?>
 				</optgroup>
 
-				<optgroup label="<?php _e( 'Forms', 'give' ); ?>">
+				<optgroup label="<?php _e('Forms', 'give'); ?>">
 					<?php
-					$this->get_dropdown_option_html( give_import_donation_form_options(), $current_mapto, $value );
+					$this->get_dropdown_option_html(give_import_donation_form_options(), $current_mapto, $value);
 					?>
 				</optgroup>
 
@@ -513,7 +513,7 @@ 
 				 *
 				 * @since 1.8.15
 				 */
-				do_action( 'give_import_dropdown_option', $index, $value, $mapto, $current_mapto );
+				do_action('give_import_dropdown_option', $index, $value, $mapto, $current_mapto);
 				?>
 			</select>
 			<?php
@@ -531,16 +531,16 @@ 
 		 *
 		 * @return void
 		 */
-		public function get_dropdown_option_html( $options, $current_mapto, $value = false ) {
-			foreach ( $options as $option => $option_value ) {
+		public function get_dropdown_option_html($options, $current_mapto, $value = false) {
+			foreach ($options as $option => $option_value) {
 				$option_value_texts = (array) $option_value;
 				$option_text = $option_value_texts[0];
 
-				$checked = ( ( $current_mapto === $option ) ? 'selected' : false );
-				if ( empty( $checked ) ) {
-					foreach ( $option_value_texts as $option_value_text ) {
-						$checked = $this->selected( $option_value_text, $value );
-						if ( $checked ) {
+				$checked = (($current_mapto === $option) ? 'selected' : false);
+				if (empty($checked)) {
+					foreach ($option_value_texts as $option_value_text) {
+						$checked = $this->selected($option_value_text, $value);
+						if ($checked) {
 							break;
 						}
 					}
@@ -564,13 +564,13 @@ 
 		 *
 		 * @return bool|int
 		 */
-		public function get_csv_total( $file_id ) {
+		public function get_csv_total($file_id) {
 			$total = false;
-			if ( $file_id ) {
-				$file_dir = get_attached_file( $file_id );
-				if ( $file_dir ) {
-					$file = new SplFileObject( $file_dir, 'r' );
-					$file->seek( PHP_INT_MAX );
+			if ($file_id) {
+				$file_dir = get_attached_file($file_id);
+				if ($file_dir) {
+					$file = new SplFileObject($file_dir, 'r');
+					$file->seek(PHP_INT_MAX);
 					$total = $file->key() + 1;
 				}
 			}
@@ -589,7 +589,7 @@ 
 		 *
 		 * @return array|bool $raw_data title of the CSV file fields
 		 */
-		public function get_importer( $file_id, $index = 0, $delimiter = 'csv' ) {
+		public function get_importer($file_id, $index = 0, $delimiter = 'csv') {
 			/**
 			 * Filter to modify delimiter of Import.
 			 *
@@ -597,16 +597,16 @@ 
 			 *
 			 * Return string $delimiter.
 			 */
-			$delimiter = (string) apply_filters( 'give_import_delimiter_set', $delimiter );
+			$delimiter = (string) apply_filters('give_import_delimiter_set', $delimiter);
 
 			$raw_data = false;
-			$file_dir = get_attached_file( $file_id );
-			if ( $file_dir ) {
-				if ( false !== ( $handle = fopen( $file_dir, 'r' ) ) ) {
-					$raw_data = fgetcsv( $handle, $index, $delimiter );
+			$file_dir = get_attached_file($file_id);
+			if ($file_dir) {
+				if (false !== ($handle = fopen($file_dir, 'r'))) {
+					$raw_data = fgetcsv($handle, $index, $delimiter);
 					// Remove BOM signature from the first item.
-					if ( isset( $raw_data[0] ) ) {
-						$raw_data[0] = $this->remove_utf8_bom( $raw_data[0] );
+					if (isset($raw_data[0])) {
+						$raw_data[0] = $this->remove_utf8_bom($raw_data[0]);
 					}
 				}
 			}
@@ -623,9 +623,9 @@ 
 		 *
 		 * @return string
 		 */
-		public function remove_utf8_bom( $string ) {
-			if ( 'efbbbf' === substr( bin2hex( $string ), 0, 6 ) ) {
-				$string = substr( $string, 3 );
+		public function remove_utf8_bom($string) {
+			if ('efbbbf' === substr(bin2hex($string), 0, 6)) {
+				$string = substr($string, 3);
 			}
 
 			return $string;
@@ -641,17 +641,17 @@ 
 			$step = $this->get_step();
 			?>
 			<ol class="give-progress-steps">
-				<li class="<?php echo( 1 === $step ? 'active' : '' ); ?>">
-					<?php esc_html_e( 'Upload CSV file', 'give' ); ?>
+				<li class="<?php echo(1 === $step ? 'active' : ''); ?>">
+					<?php esc_html_e('Upload CSV file', 'give'); ?>
 				</li>
-				<li class="<?php echo( 2 === $step ? 'active' : '' ); ?>">
-					<?php esc_html_e( 'Column mapping', 'give' ); ?>
+				<li class="<?php echo(2 === $step ? 'active' : ''); ?>">
+					<?php esc_html_e('Column mapping', 'give'); ?>
 				</li>
-				<li class="<?php echo( 3 === $step ? 'active' : '' ); ?>">
-					<?php esc_html_e( 'Import', 'give' ); ?>
+				<li class="<?php echo(3 === $step ? 'active' : ''); ?>">
+					<?php esc_html_e('Import', 'give'); ?>
 				</li>
-				<li class="<?php echo( 4 === $step ? 'active' : '' ); ?>">
-					<?php esc_html_e( 'Done!', 'give' ); ?>
+				<li class="<?php echo(4 === $step ? 'active' : ''); ?>">
+					<?php esc_html_e('Done!', 'give'); ?>
 				</li>
 			</ol>
 			<?php
@@ -665,16 +665,16 @@ 
 		 * @return int $step on which step doest the import is on.
 		 */
 		public function get_step() {
-			$step    = (int) ( isset( $_REQUEST['step'] ) ? give_clean( $_REQUEST['step'] ) : 0 );
+			$step    = (int) (isset($_REQUEST['step']) ? give_clean($_REQUEST['step']) : 0);
 			$on_step = 1;
 
-			if ( empty( $step ) || 1 === $step ) {
+			if (empty($step) || 1 === $step) {
 				$on_step = 1;
-			} elseif ( $this->check_for_dropdown_or_import() ) {
+			} elseif ($this->check_for_dropdown_or_import()) {
 				$on_step = 3;
-			} elseif ( 2 === $step ) {
+			} elseif (2 === $step) {
 				$on_step = 2;
-			} elseif ( 4 === $step ) {
+			} elseif (4 === $step) {
 				$on_step = 4;
 			}
 
@@ -687,7 +687,7 @@ 
 		 * @since 1.8.14
 		 */
 		public function render_page() {
-			include_once GIVE_PLUGIN_DIR . 'includes/admin/tools/views/html-admin-page-import-donations.php';
+			include_once GIVE_PLUGIN_DIR.'includes/admin/tools/views/html-admin-page-import-donations.php';
 		}
 
 		/**
@@ -702,37 +702,34 @@ 
 			?>
 			<tr valign="top">
 				<th colspan="2">
-					<h2 id="give-import-title"><?php esc_html_e( 'Import donations from a CSV file', 'give' ) ?></h2>
-					<p class="give-field-description"><?php esc_html_e( 'This tool allows you to import or add donation data to your give form(s) via a CSV file.', 'give' ) ?></p>
+					<h2 id="give-import-title"><?php esc_html_e('Import donations from a CSV file', 'give') ?></h2>
+					<p class="give-field-description"><?php esc_html_e('This tool allows you to import or add donation data to your give form(s) via a CSV file.', 'give') ?></p>
 				</th>
 			</tr>
 			<?php
-			$csv         = ( isset( $_POST['csv'] ) ? give_clean( $_POST['csv'] ) : '' );
-			$csv_id      = ( isset( $_POST['csv_id'] ) ? give_clean( $_POST['csv_id'] ) : '' );
-			$delimiter   = ( isset( $_POST['delimiter'] ) ? give_clean( $_POST['delimiter'] ) : 'csv' );
-			$mode        = empty( $_POST['mode'] ) ?
-				'disabled' :
-				( give_is_setting_enabled( give_clean( $_POST['mode'] ) ) ? 'enabled' : 'disabled' );
-			$create_user = empty( $_POST['create_user'] ) ?
-				'enabled' :
-				( give_is_setting_enabled( give_clean( $_POST['create_user'] ) ) ? 'enabled' : 'disabled' );
-			$delete_csv  = empty( $_POST['delete_csv'] ) ?
-				'enabled' :
-				( give_is_setting_enabled( give_clean( $_POST['delete_csv'] ) ) ? 'enabled' : 'disabled' );
+			$csv         = (isset($_POST['csv']) ? give_clean($_POST['csv']) : '');
+			$csv_id      = (isset($_POST['csv_id']) ? give_clean($_POST['csv_id']) : '');
+			$delimiter   = (isset($_POST['delimiter']) ? give_clean($_POST['delimiter']) : 'csv');
+			$mode        = empty($_POST['mode']) ?
+				'disabled' : (give_is_setting_enabled(give_clean($_POST['mode'])) ? 'enabled' : 'disabled');
+			$create_user = empty($_POST['create_user']) ?
+				'enabled' : (give_is_setting_enabled(give_clean($_POST['create_user'])) ? 'enabled' : 'disabled');
+			$delete_csv  = empty($_POST['delete_csv']) ?
+				'enabled' : (give_is_setting_enabled(give_clean($_POST['delete_csv'])) ? 'enabled' : 'disabled');
 
 			// Reset csv and csv_id if csv
-			if ( empty( $csv_id ) || ! $this->is_valid_csv( $csv_id, $csv ) ) {
+			if (empty($csv_id) || ! $this->is_valid_csv($csv_id, $csv)) {
 				$csv_id = $csv = '';
 			}
-			$per_page = isset( $_POST['per_page'] ) ? absint( $_POST['per_page'] ) : self::$per_page;
+			$per_page = isset($_POST['per_page']) ? absint($_POST['per_page']) : self::$per_page;
 
 			$settings = array(
 				array(
 					'id'          => 'csv',
-					'name'        => __( 'Choose a CSV file:', 'give' ),
+					'name'        => __('Choose a CSV file:', 'give'),
 					'type'        => 'file',
-					'attributes'  => array( 'editing' => 'false', 'library' => 'text' ),
-					'description' => __( 'The file must be a Comma Seperated Version (CSV) file type only.', 'give' ),
+					'attributes'  => array('editing' => 'false', 'library' => 'text'),
+					'description' => __('The file must be a Comma Seperated Version (CSV) file type only.', 'give'),
 					'fvalue'      => 'url',
 					'default'     => $csv,
 				),
@@ -743,61 +740,61 @@ 
 				),
 				array(
 					'id'          => 'delimiter',
-					'name'        => __( 'CSV Delimiter:', 'give' ),
-					'description' => __( 'In case your CSV file supports a different type of separator (or delimiter) -- like a tab or space -- you can set that here.', 'give' ),
+					'name'        => __('CSV Delimiter:', 'give'),
+					'description' => __('In case your CSV file supports a different type of separator (or delimiter) -- like a tab or space -- you can set that here.', 'give'),
 					'default'     => $delimiter,
 					'type'        => 'select',
 					'options'     => array(
-						'csv'                  => esc_html__( 'Comma', 'give' ),
-						'tab-separated-values' => esc_html__( 'Tab', 'give' ),
+						'csv'                  => esc_html__('Comma', 'give'),
+						'tab-separated-values' => esc_html__('Tab', 'give'),
 					),
 				),
 				array(
 					'id'          => 'mode',
-					'name'        => __( 'Test Mode:', 'give' ),
-					'description' => __( 'Test mode allows you to preview what this import would look like without making any actual changes to your site or your database.', 'give' ),
+					'name'        => __('Test Mode:', 'give'),
+					'description' => __('Test mode allows you to preview what this import would look like without making any actual changes to your site or your database.', 'give'),
 					'default'     => $mode,
 					'type'        => 'radio_inline',
 					'options'     => array(
-						'enabled'  => __( 'Enabled', 'give' ),
-						'disabled' => __( 'Disabled', 'give' ),
+						'enabled'  => __('Enabled', 'give'),
+						'disabled' => __('Disabled', 'give'),
 					),
 				),
 				array(
 					'id'          => 'create_user',
-					'name'        => __( 'Create WP users for new donors:', 'give' ),
-					'description' => __( 'The importer can create WordPress user accounts based on the names and email addresses of the donations in your CSV file. Enable this option if you\'d like the importer to do that.', 'give' ),
+					'name'        => __('Create WP users for new donors:', 'give'),
+					'description' => __('The importer can create WordPress user accounts based on the names and email addresses of the donations in your CSV file. Enable this option if you\'d like the importer to do that.', 'give'),
 					'default'     => $create_user,
 					'type'        => 'radio_inline',
 					'options'     => array(
-						'enabled'  => __( 'Enabled', 'give' ),
-						'disabled' => __( 'Disabled', 'give' ),
+						'enabled'  => __('Enabled', 'give'),
+						'disabled' => __('Disabled', 'give'),
 					),
 				),
 				array(
 					'id'          => 'delete_csv',
-					'name'        => __( 'Delete CSV after import:', 'give' ),
-					'description' => __( 'Your CSV file will be uploaded via the WordPress Media Library. It\'s a good idea to delete it after the import is finished so that your sensitive data is not accessible on the web. Disable this only if you plan to delete the file manually later.', 'give' ),
+					'name'        => __('Delete CSV after import:', 'give'),
+					'description' => __('Your CSV file will be uploaded via the WordPress Media Library. It\'s a good idea to delete it after the import is finished so that your sensitive data is not accessible on the web. Disable this only if you plan to delete the file manually later.', 'give'),
 					'default'     => $delete_csv,
 					'type'        => 'radio_inline',
 					'options'     => array(
-						'enabled'  => __( 'Enabled', 'give' ),
-						'disabled' => __( 'Disabled', 'give' ),
+						'enabled'  => __('Enabled', 'give'),
+						'disabled' => __('Disabled', 'give'),
 					),
 				),
 				array(
 					'id'          => 'per_page',
-					'name'        => __( 'Process Rows Per Batch:', 'give' ),
+					'name'        => __('Process Rows Per Batch:', 'give'),
 					'type'        => 'number',
-					'description' => __( 'Determine how many rows you would like to import per cycle.', 'give' ),
+					'description' => __('Determine how many rows you would like to import per cycle.', 'give'),
 					'default'     => $per_page,
 					'class'       => 'give-text-small',
 				),
 			);
 
-			$settings = apply_filters( 'give_import_file_upload_html', $settings );
+			$settings = apply_filters('give_import_file_upload_html', $settings);
 
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
 		}
 
 		/**
@@ -810,27 +807,24 @@ 
 			$step = $this->get_step();
 
 			// Validation for first step.
-			if ( 1 === $step ) {
-				$csv_id = absint( $_POST['csv_id'] );
+			if (1 === $step) {
+				$csv_id = absint($_POST['csv_id']);
 
-				if ( $this->is_valid_csv( $csv_id, esc_url( $_POST['csv'] ) ) ) {
+				if ($this->is_valid_csv($csv_id, esc_url($_POST['csv']))) {
 
-					$url = give_import_page_url( (array) apply_filters( 'give_import_step_two_url', array(
+					$url = give_import_page_url((array) apply_filters('give_import_step_two_url', array(
 						'step'          => '2',
 						'importer-type' => $this->importer_type,
 						'csv'           => $csv_id,
-						'delimiter'     => isset( $_REQUEST['delimiter'] ) ? give_clean( $_REQUEST['delimiter'] ) : 'csv',
-						'mode'          => empty( $_POST['mode'] ) ?
-							'0' :
-							( give_is_setting_enabled( give_clean( $_POST['mode'] ) ) ? '1' : '0' ),
-						'create_user'   => empty( $_POST['create_user'] ) ?
-							'0' :
-							( give_is_setting_enabled( give_clean( $_POST['create_user'] ) ) ? '1' : '0' ),
-						'delete_csv'    => empty( $_POST['delete_csv'] ) ?
-							'1' :
-							( give_is_setting_enabled( give_clean( $_POST['delete_csv'] ) ) ? '1' : '0' ),
-						'per_page'      => isset( $_POST['per_page'] ) ? absint( $_POST['per_page'] ) : self::$per_page,
-					) ) );
+						'delimiter'     => isset($_REQUEST['delimiter']) ? give_clean($_REQUEST['delimiter']) : 'csv',
+						'mode'          => empty($_POST['mode']) ?
+							'0' : (give_is_setting_enabled(give_clean($_POST['mode'])) ? '1' : '0'),
+						'create_user'   => empty($_POST['create_user']) ?
+							'0' : (give_is_setting_enabled(give_clean($_POST['create_user'])) ? '1' : '0'),
+						'delete_csv'    => empty($_POST['delete_csv']) ?
+							'1' : (give_is_setting_enabled(give_clean($_POST['delete_csv'])) ? '1' : '0'),
+						'per_page'      => isset($_POST['per_page']) ? absint($_POST['per_page']) : self::$per_page,
+					)));
 					?>
 					<script type="text/javascript">
 						window.location = "<?php echo $url; ?>"
@@ -851,25 +845,25 @@ 
 		 *
 		 * @return bool $has_error CSV is valid or not.
 		 */
-		private function is_valid_csv( $csv = false, $match_url = '' ) {
+		private function is_valid_csv($csv = false, $match_url = '') {
 			$is_valid_csv = true;
 
-			if ( $csv ) {
-				$csv_url = wp_get_attachment_url( $csv );
+			if ($csv) {
+				$csv_url = wp_get_attachment_url($csv);
 
-				$delimiter = ( ! empty( $_REQUEST['delimiter'] ) ? give_clean( $_REQUEST['delimiter'] ) : 'csv' );
+				$delimiter = ( ! empty($_REQUEST['delimiter']) ? give_clean($_REQUEST['delimiter']) : 'csv');
 
 				if (
 					! $csv_url ||
-					( ! empty( $match_url ) && ( $csv_url !== $match_url ) ) ||
-					( ( $mime_type = get_post_mime_type( $csv ) ) && ! strpos( $mime_type, $delimiter ) )
+					( ! empty($match_url) && ($csv_url !== $match_url)) ||
+					(($mime_type = get_post_mime_type($csv)) && ! strpos($mime_type, $delimiter))
 				) {
 					$is_valid_csv = false;
-					Give_Admin_Settings::add_error( 'give-import-csv', __( 'Please upload or provide a valid CSV file.', 'give' ) );
+					Give_Admin_Settings::add_error('give-import-csv', __('Please upload or provide a valid CSV file.', 'give'));
 				}
 			} else {
 				$is_valid_csv = false;
-				Give_Admin_Settings::add_error( 'give-import-csv', __( 'Please upload or provide a valid CSV file.', 'give' ) );
+				Give_Admin_Settings::add_error('give-import-csv', __('Please upload or provide a valid CSV file.', 'give'));
 			}
 
 			return $is_valid_csv;
@@ -885,8 +879,8 @@ 
 		 * @param $field
 		 * @param $option_value
 		 */
-		public function render_import_field( $field, $option_value ) {
-			include_once GIVE_PLUGIN_DIR . 'includes/admin/tools/views/html-admin-page-imports.php';
+		public function render_import_field($field, $option_value) {
+			include_once GIVE_PLUGIN_DIR.'includes/admin/tools/views/html-admin-page-imports.php';
 		}
 
 		/**
@@ -897,8 +891,8 @@ 
 		 */
 		private function is_donations_import_page() {
 			return 'import' === give_get_current_setting_tab() &&
-			       isset( $_GET['importer-type'] ) &&
-			       $this->importer_type === give_clean( $_GET['importer-type'] );
+			       isset($_GET['importer-type']) &&
+			       $this->importer_type === give_clean($_GET['importer-type']);
 		}
 	}
 

includes/class-give-donor.php

Doc Comments +2 added lines, -2 removed lines

@@ -902,7 +902,7 @@ 
 	 * @access public
 	 *
 	 * @param  string $meta_key Metadata key. Default is empty.
-	 * @param  mixed $meta_value Metadata value.
+	 * @param  integer $meta_value Metadata value.
 	 * @param  mixed $prev_value Optional. Previous value to check before removing. Default is empty.
 	 *
 	 * @return bool               False on failure, true if success.
@@ -918,7 +918,7 @@ 
 	 * @access public
 	 *
 	 * @param  string $meta_key Metadata name. Default is empty.
-	 * @param  mixed $meta_value Optional. Metadata value. Default is empty.
+	 * @param  string $meta_value Optional. Metadata value. Default is empty.
 	 *
 	 * @return bool               False for failure. True for success.
 	 */

Spacing +172 added lines, -172 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -139,32 +139,32 @@ 
 	 * @param bool $_id_or_email
 	 * @param bool $by_user_id
 	 */
-	public function __construct( $_id_or_email = false, $by_user_id = false ) {
+	public function __construct($_id_or_email = false, $by_user_id = false) {
 
 		$this->db = Give()->donors;
 
 		if (
 			false === $_id_or_email
-			|| ( is_numeric( $_id_or_email ) && (int) $_id_or_email !== absint( $_id_or_email ) )
+			|| (is_numeric($_id_or_email) && (int) $_id_or_email !== absint($_id_or_email))
 		) {
 			return false;
 		}
 
-		$by_user_id = is_bool( $by_user_id ) ? $by_user_id : false;
+		$by_user_id = is_bool($by_user_id) ? $by_user_id : false;
 
-		if ( is_numeric( $_id_or_email ) ) {
+		if (is_numeric($_id_or_email)) {
 			$field = $by_user_id ? 'user_id' : 'id';
 		} else {
 			$field = 'email';
 		}
 
-		$donor = $this->db->get_donor_by( $field, $_id_or_email );
+		$donor = $this->db->get_donor_by($field, $_id_or_email);
 
-		if ( empty( $donor ) || ! is_object( $donor ) ) {
+		if (empty($donor) || ! is_object($donor)) {
 			return false;
 		}
 
-		$this->setup_donor( $donor );
+		$this->setup_donor($donor);
 
 	}
 
@@ -180,15 +180,15 @@ 
 	 *
 	 * @return bool             If the setup was successful or not.
 	 */
-	private function setup_donor( $donor ) {
+	private function setup_donor($donor) {
 
-		if ( ! is_object( $donor ) ) {
+		if ( ! is_object($donor)) {
 			return false;
 		}
 
-		foreach ( $donor as $key => $value ) {
+		foreach ($donor as $key => $value) {
 
-			switch ( $key ) {
+			switch ($key) {
 
 				case 'notes':
 					$this->$key = $this->get_notes();
@@ -202,11 +202,11 @@ 
 		}
 
 		// Get donor's all email including primary email.
-		$this->emails = (array) $this->get_meta( 'additional_email', false );
-		$this->emails = array( 'primary' => $this->email ) + $this->emails;
+		$this->emails = (array) $this->get_meta('additional_email', false);
+		$this->emails = array('primary' => $this->email) + $this->emails;
 
 		// Donor ID and email are the only things that are necessary, make sure they exist.
-		if ( ! empty( $this->id ) && ! empty( $this->email ) ) {
+		if ( ! empty($this->id) && ! empty($this->email)) {
 			return true;
 		}
 
@@ -223,16 +223,16 @@ 
 	 *
 	 * @return mixed|\WP_Error
 	 */
-	public function __get( $key ) {
+	public function __get($key) {
 
-		if ( method_exists( $this, 'get_' . $key ) ) {
+		if (method_exists($this, 'get_'.$key)) {
 
-			return call_user_func( array( $this, 'get_' . $key ) );
+			return call_user_func(array($this, 'get_'.$key));
 
 		} else {
 
 			/* translators: %s: property key */
-			return new WP_Error( 'give-donor-invalid-property', sprintf( esc_html__( 'Can\'t get property %s.', 'give' ), $key ) );
+			return new WP_Error('give-donor-invalid-property', sprintf(esc_html__('Can\'t get property %s.', 'give'), $key));
 
 		}
 
@@ -248,9 +248,9 @@ 
 	 *
 	 * @return bool|int    False if not a valid creation, donor ID if user is found or valid creation.
 	 */
-	public function create( $data = array() ) {
+	public function create($data = array()) {
 
-		if ( $this->id != 0 || empty( $data ) ) {
+		if ($this->id != 0 || empty($data)) {
 			return false;
 		}
 
@@ -258,15 +258,15 @@ 
 			'payment_ids' => '',
 		);
 
-		$args = wp_parse_args( $data, $defaults );
-		$args = $this->sanitize_columns( $args );
+		$args = wp_parse_args($data, $defaults);
+		$args = $this->sanitize_columns($args);
 
-		if ( empty( $args['email'] ) || ! is_email( $args['email'] ) ) {
+		if (empty($args['email']) || ! is_email($args['email'])) {
 			return false;
 		}
 
-		if ( ! empty( $args['payment_ids'] ) && is_array( $args['payment_ids'] ) ) {
-			$args['payment_ids'] = implode( ',', array_unique( array_values( $args['payment_ids'] ) ) );
+		if ( ! empty($args['payment_ids']) && is_array($args['payment_ids'])) {
+			$args['payment_ids'] = implode(',', array_unique(array_values($args['payment_ids'])));
 		}
 
 		/**
@@ -276,18 +276,18 @@ 
 		 *
 		 * @param array $args Donor attributes.
 		 */
-		do_action( 'give_donor_pre_create', $args );
+		do_action('give_donor_pre_create', $args);
 
 		$created = false;
 
 		// The DB class 'add' implies an update if the donor being asked to be created already exists
-		if ( $this->db->add( $data ) ) {
+		if ($this->db->add($data)) {
 
 			// We've successfully added/updated the donor, reset the class vars with the new data
-			$donor = $this->db->get_donor_by( 'email', $args['email'] );
+			$donor = $this->db->get_donor_by('email', $args['email']);
 
 			// Setup the donor data with the values from DB
-			$this->setup_donor( $donor );
+			$this->setup_donor($donor);
 
 			$created = $this->id;
 		}
@@ -300,7 +300,7 @@ 
 		 * @param bool|int $created False if not a valid creation, donor ID if user is found or valid creation.
 		 * @param array $args Customer attributes.
 		 */
-		do_action( 'give_donor_post_create', $created, $args );
+		do_action('give_donor_post_create', $created, $args);
 
 		return $created;
 
@@ -316,13 +316,13 @@ 
 	 *
 	 * @return bool        If the update was successful or not.
 	 */
-	public function update( $data = array() ) {
+	public function update($data = array()) {
 
-		if ( empty( $data ) ) {
+		if (empty($data)) {
 			return false;
 		}
 
-		$data = $this->sanitize_columns( $data );
+		$data = $this->sanitize_columns($data);
 
 		/**
 		 * Fires before updating donors.
@@ -332,14 +332,14 @@ 
 		 * @param int $donor_id Donor id.
 		 * @param array $data Donor attributes.
 		 */
-		do_action( 'give_donor_pre_update', $this->id, $data );
+		do_action('give_donor_pre_update', $this->id, $data);
 
 		$updated = false;
 
-		if ( $this->db->update( $this->id, $data ) ) {
+		if ($this->db->update($this->id, $data)) {
 
-			$donor = $this->db->get_donor_by( 'id', $this->id );
-			$this->setup_donor( $donor );
+			$donor = $this->db->get_donor_by('id', $this->id);
+			$this->setup_donor($donor);
 
 			$updated = true;
 		}
@@ -353,7 +353,7 @@ 
 		 * @param int $donor_id Donor id.
 		 * @param array $data Donor attributes.
 		 */
-		do_action( 'give_donor_post_update', $updated, $this->id, $data );
+		do_action('give_donor_post_update', $updated, $this->id, $data);
 
 		return $updated;
 	}
@@ -371,27 +371,27 @@ 
 	 *
 	 * @return bool            If the attachment was successfully.
 	 */
-	public function attach_payment( $payment_id = 0, $update_stats = true ) {
+	public function attach_payment($payment_id = 0, $update_stats = true) {
 
-		if ( empty( $payment_id ) ) {
+		if (empty($payment_id)) {
 			return false;
 		}
 
-		if ( empty( $this->payment_ids ) ) {
+		if (empty($this->payment_ids)) {
 
 			$new_payment_ids = $payment_id;
 
 		} else {
 
-			$payment_ids = array_map( 'absint', explode( ',', $this->payment_ids ) );
+			$payment_ids = array_map('absint', explode(',', $this->payment_ids));
 
-			if ( in_array( $payment_id, $payment_ids ) ) {
+			if (in_array($payment_id, $payment_ids)) {
 				$update_stats = false;
 			}
 
 			$payment_ids[] = $payment_id;
 
-			$new_payment_ids = implode( ',', array_unique( array_values( $payment_ids ) ) );
+			$new_payment_ids = implode(',', array_unique(array_values($payment_ids)));
 
 		}
 
@@ -403,20 +403,20 @@ 
 		 * @param int $payment_id Payment id.
 		 * @param int $donor_id Customer id.
 		 */
-		do_action( 'give_donor_pre_attach_payment', $payment_id, $this->id );
+		do_action('give_donor_pre_attach_payment', $payment_id, $this->id);
 
-		$payment_added = $this->update( array( 'payment_ids' => $new_payment_ids ) );
+		$payment_added = $this->update(array('payment_ids' => $new_payment_ids));
 
-		if ( $payment_added ) {
+		if ($payment_added) {
 
 			$this->payment_ids = $new_payment_ids;
 
 			// We added this payment successfully, increment the stats
-			if ( $update_stats ) {
-				$payment_amount = give_donation_amount( $payment_id, array( 'type' => 'stats' ) );
+			if ($update_stats) {
+				$payment_amount = give_donation_amount($payment_id, array('type' => 'stats'));
 
-				if ( ! empty( $payment_amount ) ) {
-					$this->increase_value( $payment_amount );
+				if ( ! empty($payment_amount)) {
+					$this->increase_value($payment_amount);
 				}
 
 				$this->increase_purchase_count();
@@ -432,7 +432,7 @@ 
 		 * @param int $payment_id Payment id.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_post_attach_payment', $payment_added, $payment_id, $this->id );
+		do_action('give_donor_post_attach_payment', $payment_added, $payment_id, $this->id);
 
 		return $payment_added;
 	}
@@ -450,33 +450,33 @@ 
 	 *
 	 * @return boolean               If the removal was successful.
 	 */
-	public function remove_payment( $payment_id = 0, $update_stats = true ) {
+	public function remove_payment($payment_id = 0, $update_stats = true) {
 
-		if ( empty( $payment_id ) ) {
+		if (empty($payment_id)) {
 			return false;
 		}
 
-		$payment = new Give_Payment( $payment_id );
+		$payment = new Give_Payment($payment_id);
 
-		if ( 'publish' !== $payment->status && 'revoked' !== $payment->status ) {
+		if ('publish' !== $payment->status && 'revoked' !== $payment->status) {
 			$update_stats = false;
 		}
 
 		$new_payment_ids = '';
 
-		if ( ! empty( $this->payment_ids ) ) {
+		if ( ! empty($this->payment_ids)) {
 
-			$payment_ids = array_map( 'absint', explode( ',', $this->payment_ids ) );
+			$payment_ids = array_map('absint', explode(',', $this->payment_ids));
 
-			$pos = array_search( $payment_id, $payment_ids );
-			if ( false === $pos ) {
+			$pos = array_search($payment_id, $payment_ids);
+			if (false === $pos) {
 				return false;
 			}
 
-			unset( $payment_ids[ $pos ] );
-			$payment_ids = array_filter( $payment_ids );
+			unset($payment_ids[$pos]);
+			$payment_ids = array_filter($payment_ids);
 
-			$new_payment_ids = implode( ',', array_unique( array_values( $payment_ids ) ) );
+			$new_payment_ids = implode(',', array_unique(array_values($payment_ids)));
 
 		}
 
@@ -488,20 +488,20 @@ 
 		 * @param int $payment_id Payment id.
 		 * @param int $donor_id Customer id.
 		 */
-		do_action( 'give_donor_pre_remove_payment', $payment_id, $this->id );
+		do_action('give_donor_pre_remove_payment', $payment_id, $this->id);
 
-		$payment_removed = $this->update( array( 'payment_ids' => $new_payment_ids ) );
+		$payment_removed = $this->update(array('payment_ids' => $new_payment_ids));
 
-		if ( $payment_removed ) {
+		if ($payment_removed) {
 
 			$this->payment_ids = $new_payment_ids;
 
-			if ( $update_stats ) {
+			if ($update_stats) {
 				// We removed this payment successfully, decrement the stats
-				$payment_amount = give_donation_amount( $payment_id );
+				$payment_amount = give_donation_amount($payment_id);
 
-				if ( ! empty( $payment_amount ) ) {
-					$this->decrease_value( $payment_amount );
+				if ( ! empty($payment_amount)) {
+					$this->decrease_value($payment_amount);
 				}
 
 				$this->decrease_donation_count();
@@ -517,7 +517,7 @@ 
 		 * @param int $payment_id Payment id.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_post_remove_payment', $payment_removed, $payment_id, $this->id );
+		do_action('give_donor_post_remove_payment', $payment_removed, $payment_id, $this->id);
 
 		return $payment_removed;
 
@@ -533,10 +533,10 @@ 
 	 *
 	 * @return int        The donation count.
 	 */
-	public function increase_purchase_count( $count = 1 ) {
+	public function increase_purchase_count($count = 1) {
 
 		// Make sure it's numeric and not negative.
-		if ( ! is_numeric( $count ) || $count != absint( $count ) ) {
+		if ( ! is_numeric($count) || $count != absint($count)) {
 			return false;
 		}
 
@@ -550,9 +550,9 @@ 
 		 * @param int $count The number to increase by.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_pre_increase_donation_count', $count, $this->id );
+		do_action('give_donor_pre_increase_donation_count', $count, $this->id);
 
-		if ( $this->update( array( 'purchase_count' => $new_total ) ) ) {
+		if ($this->update(array('purchase_count' => $new_total))) {
 			$this->purchase_count = $new_total;
 		}
 
@@ -565,7 +565,7 @@ 
 		 * @param int $count The number increased by.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_post_increase_donation_count', $this->purchase_count, $count, $this->id );
+		do_action('give_donor_post_increase_donation_count', $this->purchase_count, $count, $this->id);
 
 		return $this->purchase_count;
 	}
@@ -580,16 +580,16 @@ 
 	 *
 	 * @return mixed      If successful, the new count, otherwise false.
 	 */
-	public function decrease_donation_count( $count = 1 ) {
+	public function decrease_donation_count($count = 1) {
 
 		// Make sure it's numeric and not negative
-		if ( ! is_numeric( $count ) || $count != absint( $count ) ) {
+		if ( ! is_numeric($count) || $count != absint($count)) {
 			return false;
 		}
 
 		$new_total = (int) $this->purchase_count - (int) $count;
 
-		if ( $new_total < 0 ) {
+		if ($new_total < 0) {
 			$new_total = 0;
 		}
 
@@ -601,9 +601,9 @@ 
 		 * @param int $count The number to decrease by.
 		 * @param int $donor_id Customer id.
 		 */
-		do_action( 'give_donor_pre_decrease_donation_count', $count, $this->id );
+		do_action('give_donor_pre_decrease_donation_count', $count, $this->id);
 
-		if ( $this->update( array( 'purchase_count' => $new_total ) ) ) {
+		if ($this->update(array('purchase_count' => $new_total))) {
 			$this->purchase_count = $new_total;
 		}
 
@@ -616,7 +616,7 @@ 
 		 * @param int $count The number decreased by.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_post_decrease_donation_count', $this->purchase_count, $count, $this->id );
+		do_action('give_donor_post_decrease_donation_count', $this->purchase_count, $count, $this->id);
 
 		return $this->purchase_count;
 	}
@@ -631,9 +631,9 @@ 
 	 *
 	 * @return mixed        If successful, the new value, otherwise false.
 	 */
-	public function increase_value( $value = 0.00 ) {
+	public function increase_value($value = 0.00) {
 
-		$new_value = floatval( $this->purchase_value ) + $value;
+		$new_value = floatval($this->purchase_value) + $value;
 
 		/**
 		 * Fires before increasing donor lifetime value.
@@ -643,9 +643,9 @@ 
 		 * @param float $value The value to increase by.
 		 * @param int $donor_id Customer id.
 		 */
-		do_action( 'give_donor_pre_increase_value', $value, $this->id );
+		do_action('give_donor_pre_increase_value', $value, $this->id);
 
-		if ( $this->update( array( 'purchase_value' => $new_value ) ) ) {
+		if ($this->update(array('purchase_value' => $new_value))) {
 			$this->purchase_value = $new_value;
 		}
 
@@ -658,7 +658,7 @@ 
 		 * @param float $value The value increased by.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_post_increase_value', $this->purchase_value, $value, $this->id );
+		do_action('give_donor_post_increase_value', $this->purchase_value, $value, $this->id);
 
 		return $this->purchase_value;
 	}
@@ -673,11 +673,11 @@ 
 	 *
 	 * @return mixed        If successful, the new value, otherwise false.
 	 */
-	public function decrease_value( $value = 0.00 ) {
+	public function decrease_value($value = 0.00) {
 
-		$new_value = floatval( $this->purchase_value ) - $value;
+		$new_value = floatval($this->purchase_value) - $value;
 
-		if ( $new_value < 0 ) {
+		if ($new_value < 0) {
 			$new_value = 0.00;
 		}
 
@@ -689,9 +689,9 @@ 
 		 * @param float $value The value to decrease by.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_pre_decrease_value', $value, $this->id );
+		do_action('give_donor_pre_decrease_value', $value, $this->id);
 
-		if ( $this->update( array( 'purchase_value' => $new_value ) ) ) {
+		if ($this->update(array('purchase_value' => $new_value))) {
 			$this->purchase_value = $new_value;
 		}
 
@@ -704,7 +704,7 @@ 
 		 * @param float $value The value decreased by.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_post_decrease_value', $this->purchase_value, $value, $this->id );
+		do_action('give_donor_post_decrease_value', $this->purchase_value, $value, $this->id);
 
 		return $this->purchase_value;
 	}
@@ -723,7 +723,7 @@ 
 	 *
 	 * @return mixed              If successful, the new donation stat value, otherwise false.
 	 */
-	public function update_donation_value( $curr_amount, $new_amount ) {
+	public function update_donation_value($curr_amount, $new_amount) {
 		/**
 		 * Payment total difference value can be:
 		 *  zero   (in case amount not change)
@@ -733,15 +733,15 @@ 
 		$payment_total_diff = $new_amount - $curr_amount;
 
 		// We do not need to update donation stat if donation did not change.
-		if ( ! $payment_total_diff ) {
+		if ( ! $payment_total_diff) {
 			return false;
 		}
 
-		if ( $payment_total_diff > 0 ) {
-			$this->increase_value( $payment_total_diff );
+		if ($payment_total_diff > 0) {
+			$this->increase_value($payment_total_diff);
 		} else {
 			// Pass payment total difference as +ve value to decrease amount from user lifetime stat.
-			$this->decrease_value( - $payment_total_diff );
+			$this->decrease_value( -$payment_total_diff );
 		}
 
 		return $this->purchase_value;
@@ -758,15 +758,15 @@ 
 	 *
 	 * @return array       The notes requested.
 	 */
-	public function get_notes( $length = 20, $paged = 1 ) {
+	public function get_notes($length = 20, $paged = 1) {
 
-		$length = is_numeric( $length ) ? $length : 20;
-		$offset = is_numeric( $paged ) && $paged != 1 ? ( ( absint( $paged ) - 1 ) * $length ) : 0;
+		$length = is_numeric($length) ? $length : 20;
+		$offset = is_numeric($paged) && $paged != 1 ? ((absint($paged) - 1) * $length) : 0;
 
 		$all_notes   = $this->get_raw_notes();
-		$notes_array = array_reverse( array_filter( explode( "\n\n", $all_notes ) ) );
+		$notes_array = array_reverse(array_filter(explode("\n\n", $all_notes)));
 
-		$desired_notes = array_slice( $notes_array, $offset, $length );
+		$desired_notes = array_slice($notes_array, $offset, $length);
 
 		return $desired_notes;
 
@@ -783,9 +783,9 @@ 
 	public function get_notes_count() {
 
 		$all_notes   = $this->get_raw_notes();
-		$notes_array = array_reverse( array_filter( explode( "\n\n", $all_notes ) ) );
+		$notes_array = array_reverse(array_filter(explode("\n\n", $all_notes)));
 
-		return count( $notes_array );
+		return count($notes_array);
 
 	}
 
@@ -798,7 +798,7 @@ 
 	 *
 	 * @return string|float
 	 */
-	public function get_total_donation_amount( $args = array() ) {
+	public function get_total_donation_amount($args = array()) {
 
 		/**
 		 * Filter total donation amount.
@@ -809,7 +809,7 @@ 
 		 * @param integer      $donor_id       Donor ID.
 		 * @param array        $args           Pass additional data.
 		 */
-		return apply_filters( 'give_get_total_donation_amount', $this->purchase_value, $this->id, $args );
+		return apply_filters('give_get_total_donation_amount', $this->purchase_value, $this->id, $args);
 	}
 
 	/**
@@ -822,22 +822,22 @@ 
 	 *
 	 * @return string|boolean The new note if added successfully, false otherwise.
 	 */
-	public function add_note( $note = '' ) {
+	public function add_note($note = '') {
 
-		$note = trim( $note );
-		if ( empty( $note ) ) {
+		$note = trim($note);
+		if (empty($note)) {
 			return false;
 		}
 
 		$notes = $this->get_raw_notes();
 
-		if ( empty( $notes ) ) {
+		if (empty($notes)) {
 			$notes = '';
 		}
 
-		$note_string = date_i18n( 'F j, Y H:i:s', current_time( 'timestamp' ) ) . ' - ' . $note;
-		$new_note    = apply_filters( 'give_customer_add_note_string', $note_string );
-		$notes       .= "\n\n" . $new_note;
+		$note_string = date_i18n('F j, Y H:i:s', current_time('timestamp')).' - '.$note;
+		$new_note    = apply_filters('give_customer_add_note_string', $note_string);
+		$notes .= "\n\n".$new_note;
 
 		/**
 		 * Fires before donor note is added.
@@ -847,11 +847,11 @@ 
 		 * @param string $new_note New note to add.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_pre_add_note', $new_note, $this->id );
+		do_action('give_donor_pre_add_note', $new_note, $this->id);
 
-		$updated = $this->update( array( 'notes' => $notes ) );
+		$updated = $this->update(array('notes' => $notes));
 
-		if ( $updated ) {
+		if ($updated) {
 			$this->notes = $this->get_notes();
 		}
 
@@ -864,7 +864,7 @@ 
 		 * @param string $new_note New note added.
 		 * @param int $donor_id Donor id.
 		 */
-		do_action( 'give_donor_post_add_note', $this->notes, $new_note, $this->id );
+		do_action('give_donor_post_add_note', $this->notes, $new_note, $this->id);
 
 		// Return the formatted note, so we can test, as well as update any displays
 		return $new_note;
@@ -881,7 +881,7 @@ 
 	 */
 	private function get_raw_notes() {
 
-		$all_notes = $this->db->get_column( 'notes', $this->id );
+		$all_notes = $this->db->get_column('notes', $this->id);
 
 		return $all_notes;
 
@@ -898,8 +898,8 @@ 
 	 *
 	 * @return mixed            Will be an array if $single is false. Will be value of meta data field if $single is true.
 	 */
-	public function get_meta( $meta_key = '', $single = true ) {
-		return Give()->donor_meta->get_meta( $this->id, $meta_key, $single );
+	public function get_meta($meta_key = '', $single = true) {
+		return Give()->donor_meta->get_meta($this->id, $meta_key, $single);
 	}
 
 	/**
@@ -914,8 +914,8 @@ 
 	 *
 	 * @return bool               False for failure. True for success.
 	 */
-	public function add_meta( $meta_key = '', $meta_value, $unique = false ) {
-		return Give()->donor_meta->add_meta( $this->id, $meta_key, $meta_value, $unique );
+	public function add_meta($meta_key = '', $meta_value, $unique = false) {
+		return Give()->donor_meta->add_meta($this->id, $meta_key, $meta_value, $unique);
 	}
 
 	/**
@@ -930,8 +930,8 @@ 
 	 *
 	 * @return bool               False on failure, true if success.
 	 */
-	public function update_meta( $meta_key = '', $meta_value, $prev_value = '' ) {
-		return Give()->donor_meta->update_meta( $this->id, $meta_key, $meta_value, $prev_value );
+	public function update_meta($meta_key = '', $meta_value, $prev_value = '') {
+		return Give()->donor_meta->update_meta($this->id, $meta_key, $meta_value, $prev_value);
 	}
 
 	/**
@@ -945,8 +945,8 @@ 
 	 *
 	 * @return bool               False for failure. True for success.
 	 */
-	public function delete_meta( $meta_key = '', $meta_value = '' ) {
-		return Give()->donor_meta->delete_meta( $this->id, $meta_key, $meta_value );
+	public function delete_meta($meta_key = '', $meta_value = '') {
+		return Give()->donor_meta->delete_meta($this->id, $meta_key, $meta_value);
 	}
 
 	/**
@@ -959,51 +959,51 @@ 
 	 *
 	 * @return array       The sanitized data, based off column defaults.
 	 */
-	private function sanitize_columns( $data ) {
+	private function sanitize_columns($data) {
 
 		$columns        = $this->db->get_columns();
 		$default_values = $this->db->get_column_defaults();
 
-		foreach ( $columns as $key => $type ) {
+		foreach ($columns as $key => $type) {
 
 			// Only sanitize data that we were provided
-			if ( ! array_key_exists( $key, $data ) ) {
+			if ( ! array_key_exists($key, $data)) {
 				continue;
 			}
 
-			switch ( $type ) {
+			switch ($type) {
 
 				case '%s':
-					if ( 'email' == $key ) {
-						$data[ $key ] = sanitize_email( $data[ $key ] );
-					} elseif ( 'notes' == $key ) {
-						$data[ $key ] = strip_tags( $data[ $key ] );
+					if ('email' == $key) {
+						$data[$key] = sanitize_email($data[$key]);
+					} elseif ('notes' == $key) {
+						$data[$key] = strip_tags($data[$key]);
 					} else {
-						$data[ $key ] = sanitize_text_field( $data[ $key ] );
+						$data[$key] = sanitize_text_field($data[$key]);
 					}
 					break;
 
 				case '%d':
-					if ( ! is_numeric( $data[ $key ] ) || (int) $data[ $key ] !== absint( $data[ $key ] ) ) {
-						$data[ $key ] = $default_values[ $key ];
+					if ( ! is_numeric($data[$key]) || (int) $data[$key] !== absint($data[$key])) {
+						$data[$key] = $default_values[$key];
 					} else {
-						$data[ $key ] = absint( $data[ $key ] );
+						$data[$key] = absint($data[$key]);
 					}
 					break;
 
 				case '%f':
 					// Convert what was given to a float
-					$value = floatval( $data[ $key ] );
+					$value = floatval($data[$key]);
 
-					if ( ! is_float( $value ) ) {
-						$data[ $key ] = $default_values[ $key ];
+					if ( ! is_float($value)) {
+						$data[$key] = $default_values[$key];
 					} else {
-						$data[ $key ] = $value;
+						$data[$key] = $value;
 					}
 					break;
 
 				default:
-					$data[ $key ] = sanitize_text_field( $data[ $key ] );
+					$data[$key] = sanitize_text_field($data[$key]);
 					break;
 
 			}
@@ -1023,33 +1023,33 @@ 
 	 *
 	 * @return bool            If the email was added successfully
 	 */
-	public function add_email( $email = '', $primary = false ) {
-		if ( ! is_email( $email ) ) {
+	public function add_email($email = '', $primary = false) {
+		if ( ! is_email($email)) {
 			return false;
 		}
-		$existing = new Give_Donor( $email );
+		$existing = new Give_Donor($email);
 
-		if ( $existing->id > 0 ) {
+		if ($existing->id > 0) {
 			// Email address already belongs to another donor
 			return false;
 		}
 
-		if ( email_exists( $email ) ) {
-			$user = get_user_by( 'email', $email );
-			if ( $user->ID != $this->user_id ) {
+		if (email_exists($email)) {
+			$user = get_user_by('email', $email);
+			if ($user->ID != $this->user_id) {
 				return false;
 			}
 		}
 
-		do_action( 'give_donor_pre_add_email', $email, $this->id, $this );
+		do_action('give_donor_pre_add_email', $email, $this->id, $this);
 
 		// Add is used to ensure duplicate emails are not added
-		$ret = (bool) $this->add_meta( 'additional_email', $email );
+		$ret = (bool) $this->add_meta('additional_email', $email);
 
-		do_action( 'give_donor_post_add_email', $email, $this->id, $this );
+		do_action('give_donor_post_add_email', $email, $this->id, $this);
 
-		if ( $ret && true === $primary ) {
-			$this->set_primary_email( $email );
+		if ($ret && true === $primary) {
+			$this->set_primary_email($email);
 		}
 
 		return $ret;
@@ -1065,16 +1065,16 @@ 
 	 *
 	 * @return bool          If the email was removed successfully.
 	 */
-	public function remove_email( $email = '' ) {
-		if ( ! is_email( $email ) ) {
+	public function remove_email($email = '') {
+		if ( ! is_email($email)) {
 			return false;
 		}
 
-		do_action( 'give_donor_pre_remove_email', $email, $this->id, $this );
+		do_action('give_donor_pre_remove_email', $email, $this->id, $this);
 
-		$ret = (bool) $this->delete_meta( 'additional_email', $email );
+		$ret = (bool) $this->delete_meta('additional_email', $email);
 
-		do_action( 'give_donor_post_remove_email', $email, $this->id, $this );
+		do_action('give_donor_post_remove_email', $email, $this->id, $this);
 
 		return $ret;
 	}
@@ -1091,16 +1091,16 @@ 
 	 *
 	 * @return bool                      If the email was set as primary successfully.
 	 */
-	public function set_primary_email( $new_primary_email = '' ) {
-		if ( ! is_email( $new_primary_email ) ) {
+	public function set_primary_email($new_primary_email = '') {
+		if ( ! is_email($new_primary_email)) {
 			return false;
 		}
 
-		do_action( 'give_donor_pre_set_primary_email', $new_primary_email, $this->id, $this );
+		do_action('give_donor_pre_set_primary_email', $new_primary_email, $this->id, $this);
 
-		$existing = new Give_Donor( $new_primary_email );
+		$existing = new Give_Donor($new_primary_email);
 
-		if ( $existing->id > 0 && (int) $existing->id !== (int) $this->id ) {
+		if ($existing->id > 0 && (int) $existing->id !== (int) $this->id) {
 			// This email belongs to another donor.
 			return false;
 		}
@@ -1108,21 +1108,21 @@ 
 		$old_email = $this->email;
 
 		// Update donor record with new email.
-		$update = $this->update( array( 'email' => $new_primary_email ) );
+		$update = $this->update(array('email' => $new_primary_email));
 
 		// Remove new primary from list of additional emails.
-		$remove = $this->remove_email( $new_primary_email );
+		$remove = $this->remove_email($new_primary_email);
 
 		// Add old email to additional emails list.
-		$add = $this->add_email( $old_email );
+		$add = $this->add_email($old_email);
 
 		$ret = $update && $remove && $add;
 
-		if ( $ret ) {
+		if ($ret) {
 			$this->email = $new_primary_email;
 		}
 
-		do_action( 'give_donor_post_set_primary_email', $new_primary_email, $this->id, $this );
+		do_action('give_donor_post_set_primary_email', $new_primary_email, $this->id, $this);
 
 		return $ret;
 	}

includes/admin/donors/donor-functions.php

Spacing +28 added lines, -28 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,7 +23,7 @@ 
  *
  * @return array        The altered list of views.
  */
-function give_register_default_donor_views( $views ) {
+function give_register_default_donor_views($views) {
 
 	$default_views = array(
 		'overview' => 'give_donor_view',
@@ -31,11 +31,11 @@ 
 		'notes'    => 'give_donor_notes_view',
 	);
 
-	return array_merge( $views, $default_views );
+	return array_merge($views, $default_views);
 
 }
 
-add_filter( 'give_donor_views', 'give_register_default_donor_views', 1, 1 );
+add_filter('give_donor_views', 'give_register_default_donor_views', 1, 1);
 
 /**
  * Register a tab for the single donor view.
@@ -46,23 +46,23 @@ 
  *
  * @return array       The altered list of tabs
  */
-function give_register_default_donor_tabs( $tabs ) {
+function give_register_default_donor_tabs($tabs) {
 
 	$default_tabs = array(
 		'overview' => array(
 			'dashicon' => 'dashicons-admin-users',
-			'title' => __( 'Donor Profile', 'give' ),
+			'title' => __('Donor Profile', 'give'),
 		),
 		'notes'    => array(
 			'dashicon' => 'dashicons-admin-comments',
-			'title' => __( 'Donor Notes', 'give' ),
+			'title' => __('Donor Notes', 'give'),
 		),
 	);
 
-	return array_merge( $tabs, $default_tabs );
+	return array_merge($tabs, $default_tabs);
 }
 
-add_filter( 'give_donor_tabs', 'give_register_default_donor_tabs', 1, 1 );
+add_filter('give_donor_tabs', 'give_register_default_donor_tabs', 1, 1);
 
 /**
  * Register the Delete icon as late as possible so it's at the bottom.
@@ -73,17 +73,17 @@ 
  *
  * @return array       The altered list of tabs, with 'delete' at the bottom.
  */
-function give_register_delete_donor_tab( $tabs ) {
+function give_register_delete_donor_tab($tabs) {
 
 	$tabs['delete'] = array(
 		'dashicon' => 'dashicons-trash',
-		'title'    => __( 'Delete Donor', 'give' ),
+		'title'    => __('Delete Donor', 'give'),
 	);
 
 	return $tabs;
 }
 
-add_filter( 'give_donor_tabs', 'give_register_delete_donor_tab', PHP_INT_MAX, 1 );
+add_filter('give_donor_tabs', 'give_register_delete_donor_tab', PHP_INT_MAX, 1);
 
 /**
  * Connect and Reconnect Donor with User profile.
@@ -96,7 +96,7 @@ 
  *
  * @return array
  */
-function give_connect_user_donor_profile( $donor, $donor_data, $address ) {
+function give_connect_user_donor_profile($donor, $donor_data, $address) {
 
 	$donor_id         = $donor->id;
 	$previous_user_id = $donor->user_id;
@@ -110,40 +110,40 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_pre_edit_donor', $donor_id, $donor_data, $address );
+	do_action('give_pre_edit_donor', $donor_id, $donor_data, $address);
 
 	$output = array();
 
-	if ( $donor->update( $donor_data ) ) {
+	if ($donor->update($donor_data)) {
 
-		if ( ! empty( $donor->user_id ) && $donor->user_id > 0 ) {
-			update_user_meta( $donor->user_id, '_give_user_address', $address );
+		if ( ! empty($donor->user_id) && $donor->user_id > 0) {
+			update_user_meta($donor->user_id, '_give_user_address', $address);
 		}
 
 		// Update some donation meta if we need to.
-		$payments_array = explode( ',', $donor->payment_ids );
+		$payments_array = explode(',', $donor->payment_ids);
 
-		if ( $donor->user_id !== $previous_user_id ) {
-			foreach ( $payments_array as $payment_id ) {
-				give_update_payment_meta( $payment_id, '_give_payment_user_id', $donor->user_id );
+		if ($donor->user_id !== $previous_user_id) {
+			foreach ($payments_array as $payment_id) {
+				give_update_payment_meta($payment_id, '_give_payment_user_id', $donor->user_id);
 			}
 		}
 
 		// Fetch disconnected user id, if exists.
-		$disconnected_user_id = $donor->get_meta( '_give_disconnected_user_id', true );
+		$disconnected_user_id = $donor->get_meta('_give_disconnected_user_id', true);
 
 		// Flag User and Donor Disconnection.
-		delete_user_meta( $disconnected_user_id, '_give_is_donor_disconnected' );
+		delete_user_meta($disconnected_user_id, '_give_is_donor_disconnected');
 
 		// Check whether the disconnected user id and the reconnected user id are same or not.
 		// If both are same then delete user id store in donor meta.
-		if( $donor_data['user_id'] === $disconnected_user_id ) {
-			delete_user_meta( $disconnected_user_id, '_give_disconnected_donor_id' );
-			$donor->delete_meta( '_give_disconnected_user_id' );
+		if ($donor_data['user_id'] === $disconnected_user_id) {
+			delete_user_meta($disconnected_user_id, '_give_disconnected_donor_id');
+			$donor->delete_meta('_give_disconnected_user_id');
 		}
 
 		$output['success']       = true;
-		$donor_data              = array_merge( $donor_data, $address );
+		$donor_data              = array_merge($donor_data, $address);
 		$output['customer_info'] = $donor_data;
 
 	} else {
@@ -160,7 +160,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_post_edit_donor', $donor_id, $donor_data );
+	do_action('give_post_edit_donor', $donor_id, $donor_data);
 
 
 	return $output;

includes/admin/upgrades/views/upgrades.php

Spacing +14 added lines, -14 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -18,33 +18,33 @@ 
 ?>
 <div class="wrap" id="poststuff">
 	<div id="give-updates">
-		<h1 id="give-updates-h1"><?php esc_html_e( 'Give - Updates', 'give' ); ?></h1>
+		<h1 id="give-updates-h1"><?php esc_html_e('Give - Updates', 'give'); ?></h1>
 		<hr class="wp-header-end">
 
 		<div class="give-update-panel-content">
-			<p><?php printf( __( 'Give regularly receives new features, bug fixes, and enhancements. It is important to always stay up-to-date with latest version of Give core and its add-ons.  Please create a backup of your site before updating. To update add-ons be sure your <a href="%1$s">license keys</a> are activated.', 'give' ), 'https://givewp.com/my-account/' ); ?></p>
+			<p><?php printf(__('Give regularly receives new features, bug fixes, and enhancements. It is important to always stay up-to-date with latest version of Give core and its add-ons.  Please create a backup of your site before updating. To update add-ons be sure your <a href="%1$s">license keys</a> are activated.', 'give'), 'https://givewp.com/my-account/'); ?></p>
 		</div>
 
 		<?php $update_counter = 1; ?>
 
 		<?php $db_updates = $give_updates->get_db_update_count(); ?>
-		<?php if ( ! empty( $db_updates ) ) : ?>
-			<?php $db_update_url = add_query_arg( array(
+		<?php if ( ! empty($db_updates)) : ?>
+			<?php $db_update_url = add_query_arg(array(
 				'type' => 'database',
-			) ); ?>
+			)); ?>
 			<div id="give-db-updates">
 				<div class="postbox-container">
 					<div class="postbox">
-						<h2 class="hndle"><?php _e( 'Database Updates', 'give' ); ?></h2>
+						<h2 class="hndle"><?php _e('Database Updates', 'give'); ?></h2>
 						<div class="inside">
 							<div class="panel-content">
-								<p class="give-update-button"><?php echo sprintf( __( 'Give needs to update the database. <a href="%s">Update now</a>', 'give' ), $db_update_url ); ?></p>
+								<p class="give-update-button"><?php echo sprintf(__('Give needs to update the database. <a href="%s">Update now</a>', 'give'), $db_update_url); ?></p>
 							</div>
 							<div class="progress-container give-hidden">
 								<p class="update-message"
 								   data-update-count="<?php echo $db_updates; ?>"
 								   data-resume-update="<?php echo $give_updates->resume_updates(); ?>">
-									<strong><?php echo sprintf( __( 'Update 1 of %s', 'give' ), $db_updates ); ?></strong>
+									<strong><?php echo sprintf(__('Update 1 of %s', 'give'), $db_updates); ?></strong>
 								</p>
 								<div class="progress-content"></div>
 							</div>
@@ -61,17 +61,17 @@ 
 		<?php endif; ?>
 
 		<?php $plugin_updates = $give_updates->get_plugin_update_count(); ?>
-		<?php if ( ! empty( $plugin_updates ) ) : ?>
-			<?php $plugin_update_url = add_query_arg( array(
+		<?php if ( ! empty($plugin_updates)) : ?>
+			<?php $plugin_update_url = add_query_arg(array(
 				's' => 'Give',
-			), admin_url( '/plugins.php' ) ); ?>
+			), admin_url('/plugins.php')); ?>
 			<div id="give-plugin-updates">
 				<div class="postbox-container">
 					<div class="postbox">
-						<h2 class="hndle"><?php _e( 'Add-on Updates', 'give' ); ?></h2>
+						<h2 class="hndle"><?php _e('Add-on Updates', 'give'); ?></h2>
 						<div class="inside">
 							<div class="panel-content">
-								<p><?php echo sprintf( __( 'There %1$s %2$s Give %3$s that %4$s to be updated. <a href="%5$s">Update now</a>', 'give' ), _n( 'is', 'are', $plugin_updates, 'give' ), $plugin_updates, _n( 'add-on', 'add-ons', $plugin_updates, 'give' ), _n( 'needs', 'need', $plugin_updates, 'give' ), $plugin_update_url  ); ?></p>
+								<p><?php echo sprintf(__('There %1$s %2$s Give %3$s that %4$s to be updated. <a href="%5$s">Update now</a>', 'give'), _n('is', 'are', $plugin_updates, 'give'), $plugin_updates, _n('add-on', 'add-ons', $plugin_updates, 'give'), _n('needs', 'need', $plugin_updates, 'give'), $plugin_update_url); ?></p>
 								<?php include_once 'plugins-update-section.php'; ?>
 							</div>
 						</div>

includes/admin/payments/payments-history.php

Spacing +22 added lines, -22 removed lines

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