WordImpress / Give

includes/scripts.php

Spacing +173 added lines, -173 removed lines

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

includes/class-give-donor.php

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 );
+			if ($update_stats) {
+				$payment_amount = give_donation_amount($payment_id);
 
-				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/actions.php

Spacing +60 added lines, -60 removed lines

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

includes/user-functions.php

Spacing +144 added lines, -144 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -30,54 +30,54 @@ 
  *
  * @return bool|array List of all user donations.
  */
-function give_get_users_donations( $user = 0, $number = 20, $pagination = false, $status = 'complete' ) {
+function give_get_users_donations($user = 0, $number = 20, $pagination = false, $status = 'complete') {
 
-	if ( empty( $user ) ) {
+	if (empty($user)) {
 		$user = get_current_user_id();
 	}
 
-	if ( 0 === $user && ! Give()->email_access->token_exists ) {
+	if (0 === $user && ! Give()->email_access->token_exists) {
 		return false;
 	}
 
 	$status = $status === 'complete' ? 'publish' : $status;
 	$paged = 1;
 
-	if ( $pagination ) {
-		if ( get_query_var( 'paged' ) ) {
-			$paged = get_query_var( 'paged' );
-		} elseif ( get_query_var( 'page' ) ) {
-			$paged = get_query_var( 'page' );
+	if ($pagination) {
+		if (get_query_var('paged')) {
+			$paged = get_query_var('paged');
+		} elseif (get_query_var('page')) {
+			$paged = get_query_var('page');
 		}
 	}
 
-	$args = apply_filters( 'give_get_users_donations_args', array(
+	$args = apply_filters('give_get_users_donations_args', array(
 		'user'    => $user,
 		'number'  => $number,
 		'status'  => $status,
 		'orderby' => 'date',
-	) );
+	));
 
-	if ( $pagination ) {
+	if ($pagination) {
 		$args['page'] = $paged;
 	} else {
 		$args['nopaging'] = true;
 	}
 
-	$by_user_id = is_numeric( $user ) ? true : false;
-	$donor   = new Give_Donor( $user, $by_user_id );
+	$by_user_id = is_numeric($user) ? true : false;
+	$donor = new Give_Donor($user, $by_user_id);
 
-	if ( ! empty( $donor->payment_ids ) ) {
+	if ( ! empty($donor->payment_ids)) {
 
-		unset( $args['user'] );
-		$args['post__in'] = array_map( 'absint', explode( ',', $donor->payment_ids ) );
+		unset($args['user']);
+		$args['post__in'] = array_map('absint', explode(',', $donor->payment_ids));
 
 	}
 
-	$donations = give_get_payments( apply_filters( 'give_get_users_donations_args', $args ) );
+	$donations = give_get_payments(apply_filters('give_get_users_donations_args', $args));
 
 	// No donations
-	if ( ! $donations ) {
+	if ( ! $donations) {
 		return false;
 	}
 
@@ -96,65 +96,65 @@ 
  *
  * @return bool|object List of unique forms donated by user
  */
-function give_get_users_completed_donations( $user = 0, $status = 'complete' ) {
-	if ( empty( $user ) ) {
+function give_get_users_completed_donations($user = 0, $status = 'complete') {
+	if (empty($user)) {
 		$user = get_current_user_id();
 	}
 
-	if ( empty( $user ) ) {
+	if (empty($user)) {
 		return false;
 	}
 
-	$by_user_id = is_numeric( $user ) ? true : false;
+	$by_user_id = is_numeric($user) ? true : false;
 
-	$donor = new Give_Donor( $user, $by_user_id );
+	$donor = new Give_Donor($user, $by_user_id);
 
-	if ( empty( $donor->payment_ids ) ) {
+	if (empty($donor->payment_ids)) {
 		return false;
 	}
 
 	// Get all the items donated.
-	$payment_ids    = array_reverse( explode( ',', $donor->payment_ids ) );
-	$limit_payments = apply_filters( 'give_users_completed_donations_payments', 50 );
-	if ( ! empty( $limit_payments ) ) {
-		$payment_ids = array_slice( $payment_ids, 0, $limit_payments );
+	$payment_ids    = array_reverse(explode(',', $donor->payment_ids));
+	$limit_payments = apply_filters('give_users_completed_donations_payments', 50);
+	if ( ! empty($limit_payments)) {
+		$payment_ids = array_slice($payment_ids, 0, $limit_payments);
 	}
 	$donation_data = array();
-	foreach ( $payment_ids as $payment_id ) {
-		$donation_data[] = give_get_payment_meta( $payment_id );
+	foreach ($payment_ids as $payment_id) {
+		$donation_data[] = give_get_payment_meta($payment_id);
 	}
 
-	if ( empty( $donation_data ) ) {
+	if (empty($donation_data)) {
 		return false;
 	}
 
 	// Grab only the post ids "form_id" of the forms donated on this order
 	$completed_donations_ids = array();
-	foreach ( $donation_data as $donation_meta ) {
-		$completed_donations_ids[] = isset( $donation_meta['form_id'] ) ? $donation_meta['form_id'] : '';
+	foreach ($donation_data as $donation_meta) {
+		$completed_donations_ids[] = isset($donation_meta['form_id']) ? $donation_meta['form_id'] : '';
 	}
 
-	if ( empty( $completed_donations_ids ) ) {
+	if (empty($completed_donations_ids)) {
 		return false;
 	}
 
 	// Only include each donation once
-	$form_ids = array_unique( $completed_donations_ids );
+	$form_ids = array_unique($completed_donations_ids);
 
 	// Make sure we still have some products and a first item
-	if ( empty( $form_ids ) || ! isset( $form_ids[0] ) ) {
+	if (empty($form_ids) || ! isset($form_ids[0])) {
 		return false;
 	}
 
-	$post_type = get_post_type( $form_ids[0] );
+	$post_type = get_post_type($form_ids[0]);
 
-	$args = apply_filters( 'give_get_users_completed_donations_args', array(
+	$args = apply_filters('give_get_users_completed_donations_args', array(
 		'include'        => $form_ids,
 		'post_type'      => $post_type,
-		'posts_per_page' => - 1,
-	) );
+		'posts_per_page' => -1,
+	));
 
-	return apply_filters( 'give_users_completed_donations_list', get_posts( $args ) );
+	return apply_filters('give_users_completed_donations_list', get_posts($args));
 }
 
 
@@ -170,12 +170,12 @@ 
  *
  * @return      bool True if has donated, false other wise.
  */
-function give_has_donations( $user_id = null ) {
-	if ( empty( $user_id ) ) {
+function give_has_donations($user_id = null) {
+	if (empty($user_id)) {
 		$user_id = get_current_user_id();
 	}
 
-	if ( give_get_users_donations( $user_id, 1 ) ) {
+	if (give_get_users_donations($user_id, 1)) {
 		return true; // User has at least one donation.
 	}
 
@@ -196,23 +196,23 @@ 
  *
  * @return      array
  */
-function give_get_donation_stats_by_user( $user = '' ) {
+function give_get_donation_stats_by_user($user = '') {
 
 	$field = '';
 
-	if ( is_email( $user ) ) {
+	if (is_email($user)) {
 		$field = 'email';
-	} elseif ( is_numeric( $user ) ) {
+	} elseif (is_numeric($user)) {
 		$field = 'user_id';
 	}
 
-	$stats    = array();
-	$donor = Give()->donors->get_donor_by( $field, $user );
+	$stats = array();
+	$donor = Give()->donors->get_donor_by($field, $user);
 
-	if ( $donor ) {
-		$donor = new Give_Donor( $donor->id );
-		$stats['purchases']   = absint( $donor->purchase_count );
-		$stats['total_spent'] = give_maybe_sanitize_amount( $donor->get_total_donation_amount() );
+	if ($donor) {
+		$donor = new Give_Donor($donor->id);
+		$stats['purchases']   = absint($donor->purchase_count);
+		$stats['total_spent'] = give_maybe_sanitize_amount($donor->get_total_donation_amount());
 	}
 
 	/**
@@ -220,7 +220,7 @@ 
 	 *
 	 * @since 1.7
 	 */
-	$stats = (array) apply_filters( 'give_donation_stats_by_user', $stats, $user );
+	$stats = (array) apply_filters('give_donation_stats_by_user', $stats, $user);
 
 	return $stats;
 }
@@ -238,21 +238,21 @@ 
  *
  * @return      int The total number of donations.
  */
-function give_count_donations_of_donor( $user = null ) {
+function give_count_donations_of_donor($user = null) {
 
 	// Logged in?
-	if ( empty( $user ) ) {
+	if (empty($user)) {
 		$user = get_current_user_id();
 	}
 
 	// Email access?
-	if ( empty( $user ) && Give()->email_access->token_email ) {
+	if (empty($user) && Give()->email_access->token_email) {
 		$user = Give()->email_access->token_email;
 	}
 
-	$stats = ! empty( $user ) ? give_get_donation_stats_by_user( $user ) : false;
+	$stats = ! empty($user) ? give_get_donation_stats_by_user($user) : false;
 
-	return isset( $stats['purchases'] ) ? $stats['purchases'] : 0;
+	return isset($stats['purchases']) ? $stats['purchases'] : 0;
 }
 
 /**
@@ -265,9 +265,9 @@ 
  *
  * @return      float The total amount the user has spent
  */
-function give_donation_total_of_user( $user = null ) {
+function give_donation_total_of_user($user = null) {
 
-	$stats = give_get_donation_stats_by_user( $user );
+	$stats = give_get_donation_stats_by_user($user);
 
 	return $stats['total_spent'];
 }
@@ -283,40 +283,40 @@ 
  *
  * @return bool
  */
-function give_validate_username( $username, $form_id = 0 ) {
+function give_validate_username($username, $form_id = 0) {
 	$valid = true;
 
 	// Validate username.
-	if ( ! empty( $username ) ) {
+	if ( ! empty($username)) {
 
 		// Sanitize username.
-		$sanitized_user_name = sanitize_user( $username, false );
+		$sanitized_user_name = sanitize_user($username, false);
 
 		// We have an user name, check if it already exists.
-		if ( username_exists( $username ) ) {
+		if (username_exists($username)) {
 			// Username already registered.
-			give_set_error( 'username_unavailable', __( 'Username already taken.', 'give' ) );
+			give_set_error('username_unavailable', __('Username already taken.', 'give'));
 			$valid = false;
 
 			// Check if it's valid.
-		} elseif ( $sanitized_user_name !== $username ) {
+		} elseif ($sanitized_user_name !== $username) {
 			// Invalid username.
-			if ( is_multisite() ) {
-				give_set_error( 'username_invalid', __( 'Invalid username. Only lowercase letters (a-z) and numbers are allowed.', 'give' ) );
+			if (is_multisite()) {
+				give_set_error('username_invalid', __('Invalid username. Only lowercase letters (a-z) and numbers are allowed.', 'give'));
 				$valid = false;
 			} else {
-				give_set_error( 'username_invalid', __( 'Invalid username.', 'give' ) );
+				give_set_error('username_invalid', __('Invalid username.', 'give'));
 				$valid = false;
 			}
 		}
 	} else {
 		// Username is empty.
-		give_set_error( 'username_empty', __( 'Enter a username.', 'give' ) );
+		give_set_error('username_empty', __('Enter a username.', 'give'));
 		$valid = false;
 
 		// Check if guest checkout is disable for form.
-		if ( $form_id && give_logged_in_only( $form_id ) ) {
-			give_set_error( 'registration_required', __( 'You must register or login to complete your donation.', 'give' ) );
+		if ($form_id && give_logged_in_only($form_id)) {
+			give_set_error('registration_required', __('You must register or login to complete your donation.', 'give'));
 			$valid = false;
 		}
 	}
@@ -330,7 +330,7 @@ 
 	 * @param string $username
 	 * @param bool   $form_id
 	 */
-	$valid = (bool) apply_filters( 'give_validate_username', $valid, $username, $form_id );
+	$valid = (bool) apply_filters('give_validate_username', $valid, $username, $form_id);
 
 	return $valid;
 }
@@ -346,25 +346,25 @@ 
  *
  * @return bool
  */
-function give_validate_user_email( $email, $registering_new_user = false ) {
+function give_validate_user_email($email, $registering_new_user = false) {
 	$valid = true;
 
-	if ( empty( $email ) ) {
+	if (empty($email)) {
 		// No email.
-		give_set_error( 'email_empty', __( 'Enter an email.', 'give' ) );
+		give_set_error('email_empty', __('Enter an email.', 'give'));
 		$valid = false;
 
-	} elseif ( ! is_email( $email ) ) {
+	} elseif ( ! is_email($email)) {
 		// Validate email.
-		give_set_error( 'email_invalid', __( 'Invalid email.', 'give' ) );
+		give_set_error('email_invalid', __('Invalid email.', 'give'));
 		$valid = false;
 
-	} elseif ( $registering_new_user ) {
+	} elseif ($registering_new_user) {
 
 		// If donor email is not primary
-		if ( ! email_exists( $email ) && give_donor_email_exists( $email ) && give_is_additional_email( $email ) ) {
+		if ( ! email_exists($email) && give_donor_email_exists($email) && give_is_additional_email($email)) {
 			// Check if email exists.
-			give_set_error( 'email_used', __( 'The email address provided is already active for another user.', 'give' ) );
+			give_set_error('email_used', __('The email address provided is already active for another user.', 'give'));
 			$valid = false;
 		}
 	}
@@ -378,7 +378,7 @@ 
 	 * @param string $email
 	 * @param bool   $registering_new_user
 	 */
-	$valid = (bool) apply_filters( 'give_validate_user_email', $valid, $email, $registering_new_user );
+	$valid = (bool) apply_filters('give_validate_user_email', $valid, $email, $registering_new_user);
 
 	return $valid;
 }
@@ -394,34 +394,34 @@ 
  *
  * @return bool
  */
-function give_validate_user_password( $password = '', $confirm_password = '', $registering_new_user = false ) {
+function give_validate_user_password($password = '', $confirm_password = '', $registering_new_user = false) {
 	$valid = true;
 
 	// Passwords Validation For New Donors Only
-	if ( $registering_new_user ) {
+	if ($registering_new_user) {
 		// Password or confirmation missing.
-		if ( ! $password ) {
+		if ( ! $password) {
 			// The password is invalid.
-			give_set_error( 'password_empty', __( 'Enter a password.', 'give' ) );
+			give_set_error('password_empty', __('Enter a password.', 'give'));
 			$valid = false;
-		} elseif ( ! $confirm_password ) {
+		} elseif ( ! $confirm_password) {
 			// Confirmation password is invalid.
-			give_set_error( 'confirmation_empty', __( 'Enter the password confirmation.', 'give' ) );
+			give_set_error('confirmation_empty', __('Enter the password confirmation.', 'give'));
 			$valid = false;
 		}
 	}
 	// Passwords Validation For New Donors as well as Existing Donors
-	if ( $password || $confirm_password ) {
-		if ( strlen( $password ) < 6 || strlen( $confirm_password ) < 6 ) {
+	if ($password || $confirm_password) {
+		if (strlen($password) < 6 || strlen($confirm_password) < 6) {
 			// Seems Weak Password
-			give_set_error( 'password_weak', __( 'Passwords should have at least 6 characters.', 'give' ) );
+			give_set_error('password_weak', __('Passwords should have at least 6 characters.', 'give'));
 			$valid = false;
 		}
-		if ( $password && $confirm_password ) {
+		if ($password && $confirm_password) {
 			// Verify confirmation matches.
-			if ( $password != $confirm_password ) {
+			if ($password != $confirm_password) {
 				// Passwords do not match
-				give_set_error( 'password_mismatch', __( 'Passwords you entered do not match. Please try again.', 'give' ) );
+				give_set_error('password_mismatch', __('Passwords you entered do not match. Please try again.', 'give'));
 				$valid = false;
 			}
 		}
@@ -437,7 +437,7 @@ 
 	 * @param string $confirm_password
 	 * @param bool   $registering_new_user
 	 */
-	$valid = (bool) apply_filters( 'give_validate_user_email', $valid, $password, $confirm_password, $registering_new_user );
+	$valid = (bool) apply_filters('give_validate_user_email', $valid, $password, $confirm_password, $registering_new_user);
 
 	return $valid;
 }
@@ -455,20 +455,20 @@ 
  *
  * @return      void
  */
-function give_add_past_donations_to_new_user( $user_id ) {
+function give_add_past_donations_to_new_user($user_id) {
 
-	$email = get_the_author_meta( 'user_email', $user_id );
+	$email = get_the_author_meta('user_email', $user_id);
 
-	$payments = give_get_payments( array(
+	$payments = give_get_payments(array(
 		'output' => 'payments',
 		's' => $email,
-	) );
+	));
 
-	if ( $payments ) {
-		foreach ( $payments as $payment ) {
+	if ($payments) {
+		foreach ($payments as $payment) {
 			/* @var $payment \Give_Payment */
 
-			if ( intval( give_get_payment_user_id( $payment->ID ) ) > 0 ) {
+			if (intval(give_get_payment_user_id($payment->ID)) > 0) {
 				continue;
 			}
 
@@ -480,7 +480,7 @@ 
 
 }
 
-add_action( 'user_register', 'give_add_past_donations_to_new_user' );
+add_action('user_register', 'give_add_past_donations_to_new_user');
 
 
 /**
@@ -506,34 +506,34 @@ 
  *
  * @return        array The donor's address, if any
  */
-function give_get_donor_address( $user_id = 0 ) {
-	if ( empty( $user_id ) ) {
+function give_get_donor_address($user_id = 0) {
+	if (empty($user_id)) {
 		$user_id = get_current_user_id();
 	}
 
-	$address = (array) get_user_meta( $user_id, '_give_user_address', true );
+	$address = (array) get_user_meta($user_id, '_give_user_address', true);
 
-	if ( ! isset( $address['line1'] ) ) {
+	if ( ! isset($address['line1'])) {
 		$address['line1'] = '';
 	}
 
-	if ( ! isset( $address['line2'] ) ) {
+	if ( ! isset($address['line2'])) {
 		$address['line2'] = '';
 	}
 
-	if ( ! isset( $address['city'] ) ) {
+	if ( ! isset($address['city'])) {
 		$address['city'] = '';
 	}
 
-	if ( ! isset( $address['zip'] ) ) {
+	if ( ! isset($address['zip'])) {
 		$address['zip'] = '';
 	}
 
-	if ( ! isset( $address['country'] ) ) {
+	if ( ! isset($address['country'])) {
 		$address['country'] = '';
 	}
 
-	if ( ! isset( $address['state'] ) ) {
+	if ( ! isset($address['state'])) {
 		$address['state'] = '';
 	}
 
@@ -553,53 +553,53 @@ 
  *
  * @return        void
  */
-function give_new_user_notification( $user_id = 0, $user_data = array() ) {
+function give_new_user_notification($user_id = 0, $user_data = array()) {
 
-	if ( empty( $user_id ) || empty( $user_data ) ) {
+	if (empty($user_id) || empty($user_data)) {
 		return;
 	}
-	$blogname = wp_specialchars_decode( get_option( 'blogname' ), ENT_QUOTES );
+	$blogname = wp_specialchars_decode(get_option('blogname'), ENT_QUOTES);
 
 	// New User Registration: Email sends to the site admin.
 	$emails = Give()->emails;
-	$emails->__set( 'heading', esc_html__( 'New User Registration', 'give' ) );
+	$emails->__set('heading', esc_html__('New User Registration', 'give'));
 
 	/* translators: %s: site name */
-	$message = sprintf( esc_attr__( 'A new user has registered on %s:', 'give' ), $blogname ) . "\r\n\r\n";
+	$message = sprintf(esc_attr__('A new user has registered on %s:', 'give'), $blogname)."\r\n\r\n";
 	/* translators: %s: user login */
-	$message .= '<strong>' . esc_attr__( 'Username:', 'give' ) . '</strong> ' . $user_data['user_login'] . "\r\n";
+	$message .= '<strong>'.esc_attr__('Username:', 'give').'</strong> '.$user_data['user_login']."\r\n";
 	/* translators: %s: user email */
-	$message .= '<strong>' . esc_attr__( 'E-mail:', 'give' ) . '</strong> ' . $user_data['user_email'] . "\r\n\r\n";
+	$message .= '<strong>'.esc_attr__('E-mail:', 'give').'</strong> '.$user_data['user_email']."\r\n\r\n";
 
-	$message .= '<a href="' . admin_url( 'user-edit.php?user_id=' . $user_id ) . '" target="_blank"> ' . esc_attr__( 'Click here to view &raquo;', 'give' ) . '</a>' . "\r\n";
+	$message .= '<a href="'.admin_url('user-edit.php?user_id='.$user_id).'" target="_blank"> '.esc_attr__('Click here to view &raquo;', 'give').'</a>'."\r\n";
 
 	$emails->send(
-		get_option( 'admin_email' ),
+		get_option('admin_email'),
 		sprintf(
 			/* translators: %s: site name */
-			esc_attr__( '[%s] New User Registration', 'give' ),
+			esc_attr__('[%s] New User Registration', 'give'),
 			$blogname
 		),
 		$message
 	);
 
 	// Account Information: Email sends to donor who registered.
-	$emails->__set( 'heading', esc_html__( 'Account Information', 'give' ) );
+	$emails->__set('heading', esc_html__('Account Information', 'give'));
 
-	$message = sprintf( esc_attr__( 'The following email contains your account information for %s:', 'give' ), $blogname ) . "\r\n\r\n";
+	$message = sprintf(esc_attr__('The following email contains your account information for %s:', 'give'), $blogname)."\r\n\r\n";
 
 	/* translators: %s: user login */
-	$message .= '<strong>' . esc_attr__( 'Username:', 'give' ) . '</strong> ' . $user_data['user_login'] . "\r\n";
+	$message .= '<strong>'.esc_attr__('Username:', 'give').'</strong> '.$user_data['user_login']."\r\n";
 	/* translators: %s: password */
-	$message .= '<strong>' . esc_attr__( 'Password:', 'give' ) . '</strong> ' . esc_attr__( '[Password entered during donation]', 'give' ) . "\r\n\r\n";
+	$message .= '<strong>'.esc_attr__('Password:', 'give').'</strong> '.esc_attr__('[Password entered during donation]', 'give')."\r\n\r\n";
 
-	$message .= '<a href="' . wp_login_url() . '" target="_blank"> ' . esc_attr__( 'Click here to login &raquo;', 'give' ) . '</a>' . "\r\n";
+	$message .= '<a href="'.wp_login_url().'" target="_blank"> '.esc_attr__('Click here to login &raquo;', 'give').'</a>'."\r\n";
 
 	$emails->send(
 		$user_data['user_email'],
 		sprintf(
 			/* translators: %s: site name */
-			esc_attr__( '[%s] Your username and password', 'give' ),
+			esc_attr__('[%s] Your username and password', 'give'),
 			$blogname
 		),
 		$message
@@ -607,7 +607,7 @@ 
 
 }
 
-add_action( 'give_insert_user', 'give_new_user_notification', 10, 2 );
+add_action('give_insert_user', 'give_new_user_notification', 10, 2);
 
 
 /**
@@ -623,34 +623,34 @@ 
  *
  * @return      string
  */
-function give_get_donor_name_by( $id = 0, $from = 'donation' ) {
+function give_get_donor_name_by($id = 0, $from = 'donation') {
 
 	// ID shouldn't be empty
-	if ( empty( $id ) ) {
+	if (empty($id)) {
 		return;
 	}
 
 	$name = '';
 
-	switch ( $from ) {
+	switch ($from) {
 
 		case 'donation':
 
-			$user_info = give_get_payment_meta_user_info( $id );
-			$name = trim( "{$user_info['first_name']} {$user_info['last_name']}" );
+			$user_info = give_get_payment_meta_user_info($id);
+			$name = trim("{$user_info['first_name']} {$user_info['last_name']}");
 
 		break;
 
 		case 'donor':
 
-			$donor = new Give_Donor( $id );
+			$donor = new Give_Donor($id);
 			$name = $donor->name;
 
 		break;
 
 	}
 
-	return trim( $name );
+	return trim($name);
 
 }
 
@@ -662,8 +662,8 @@ 
  * @param  string $email Donor Email.
  * @return boolean  The user's ID on success, and false on failure.
  */
-function give_donor_email_exists( $email ) {
-	if ( Give()->donors->get_donor_by( 'email', $email ) ) {
+function give_donor_email_exists($email) {
+	if (Give()->donors->get_donor_by('email', $email)) {
 		return true;
 	}
 	return false;
@@ -678,12 +678,12 @@ 
  *
  * @return bool
  */
-function give_is_additional_email( $email ) {
+function give_is_additional_email($email) {
 	global $wpdb;
-	$meta_table  = Give()->donor_meta->table_name;
-	$donor_id = $wpdb->get_var( $wpdb->prepare( "SELECT customer_id FROM {$meta_table} WHERE meta_key = 'additional_email' AND meta_value = %s LIMIT 1", $email ) );
+	$meta_table = Give()->donor_meta->table_name;
+	$donor_id = $wpdb->get_var($wpdb->prepare("SELECT customer_id FROM {$meta_table} WHERE meta_key = 'additional_email' AND meta_value = %s LIMIT 1", $email));
 
-	if ( empty( $donor_id ) ) {
+	if (empty($donor_id)) {
 		return false;
 	}
 	return true;

includes/payments/class-payments-query.php

Spacing +102 added lines, -102 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -65,10 +65,10 @@ 
 	 *
 	 * @param  $args array The array of arguments that can be passed in and used for setting up this payment query.
 	 */
-	public function __construct( $args = array() ) {
+	public function __construct($args = array()) {
 		$defaults = array(
 			'output'          => 'payments',
-			'post_type'       => array( 'give_payment' ),
+			'post_type'       => array('give_payment'),
 			'start_date'      => false,
 			'end_date'        => false,
 			'number'          => 20,
@@ -90,7 +90,7 @@ 
 			'give_forms'      => null,
 		);
 
-		$this->args = $this->_args = wp_parse_args( $args, $defaults );
+		$this->args = $this->_args = wp_parse_args($args, $defaults);
 
 		$this->init();
 	}
@@ -104,11 +104,11 @@ 
 	 * @param $query_var
 	 * @param $value
 	 */
-	public function __set( $query_var, $value ) {
-		if ( in_array( $query_var, array( 'meta_query', 'tax_query' ) ) ) {
-			$this->args[ $query_var ][] = $value;
+	public function __set($query_var, $value) {
+		if (in_array($query_var, array('meta_query', 'tax_query'))) {
+			$this->args[$query_var][] = $value;
 		} else {
-			$this->args[ $query_var ] = $value;
+			$this->args[$query_var] = $value;
 		}
 	}
 
@@ -120,8 +120,8 @@ 
 	 *
 	 * @param $query_var
 	 */
-	public function __unset( $query_var ) {
-		unset( $this->args[ $query_var ] );
+	public function __unset($query_var) {
+		unset($this->args[$query_var]);
 	}
 
 	/**
@@ -161,7 +161,7 @@ 
 		$this->give_forms();
 		$this->gateway_filter();
 
-		add_filter( 'posts_orderby', array( $this, 'custom_orderby' ), 10, 2 );
+		add_filter('posts_orderby', array($this, 'custom_orderby'), 10, 2);
 	}
 
 	/**
@@ -172,7 +172,7 @@ 
 	 */
 	private function unset_filters() {
 		$this->date_filter_post();
-		remove_filter( 'posts_orderby', array( $this, 'custom_orderby' ) );
+		remove_filter('posts_orderby', array($this, 'custom_orderby'));
 	}
 
 
@@ -199,9 +199,9 @@ 
 		 *
 		 * @param Give_Payments_Query $this Payments query object.
 		 */
-		do_action( 'give_pre_get_payments', $this );
+		do_action('give_pre_get_payments', $this);
 
-		$query          = new WP_Query( $this->args );
+		$query          = new WP_Query($this->args);
 		$this->payments = array();
 
 		$custom_output = array(
@@ -209,18 +209,18 @@ 
 			'give_payments',
 		);
 
-		if ( ! in_array( $this->args['output'], $custom_output ) ) {
+		if ( ! in_array($this->args['output'], $custom_output)) {
 			return $query->posts;
 		}
 
-		if ( $query->have_posts() ) {
-			while ( $query->have_posts() ) {
+		if ($query->have_posts()) {
+			while ($query->have_posts()) {
 				$query->the_post();
 
 				$payment_id = get_post()->ID;
-				$payment    = new Give_Payment( $payment_id );
+				$payment    = new Give_Payment($payment_id);
 
-				$this->payments[] = apply_filters( 'give_payment', $payment, $payment_id, $this );
+				$this->payments[] = apply_filters('give_payment', $payment, $payment_id, $this);
 			}
 
 			wp_reset_postdata();
@@ -236,7 +236,7 @@ 
 		 *
 		 * @param Give_Payments_Query $this Payments query object.
 		 */
-		do_action( 'give_post_get_payments', $this );
+		do_action('give_post_get_payments', $this);
 
 		return $this->payments;
 	}
@@ -250,13 +250,13 @@ 
 	 * @return void
 	 */
 	public function date_filter_pre() {
-		if ( ! ( $this->args['start_date'] || $this->args['end_date'] ) ) {
+		if ( ! ($this->args['start_date'] || $this->args['end_date'])) {
 			return;
 		}
 
-		$this->setup_dates( $this->args['start_date'], $this->args['end_date'] );
+		$this->setup_dates($this->args['start_date'], $this->args['end_date']);
 
-		add_filter( 'posts_where', array( $this, 'payments_where' ) );
+		add_filter('posts_where', array($this, 'payments_where'));
 	}
 
 	/**
@@ -269,11 +269,11 @@ 
 	 * @return void
 	 */
 	public function date_filter_post() {
-		if ( ! ( $this->args['start_date'] || $this->args['end_date'] ) ) {
+		if ( ! ($this->args['start_date'] || $this->args['end_date'])) {
 			return;
 		}
 
-		remove_filter( 'posts_where', array( $this, 'payments_where' ) );
+		remove_filter('posts_where', array($this, 'payments_where'));
 	}
 
 	/**
@@ -285,12 +285,12 @@ 
 	 * @return void
 	 */
 	public function status() {
-		if ( ! isset( $this->args['status'] ) ) {
+		if ( ! isset($this->args['status'])) {
 			return;
 		}
 
-		$this->__set( 'post_status', $this->args['status'] );
-		$this->__unset( 'status' );
+		$this->__set('post_status', $this->args['status']);
+		$this->__unset('status');
 	}
 
 	/**
@@ -302,12 +302,12 @@ 
 	 * @return void
 	 */
 	public function page() {
-		if ( ! isset( $this->args['page'] ) ) {
+		if ( ! isset($this->args['page'])) {
 			return;
 		}
 
-		$this->__set( 'paged', $this->args['page'] );
-		$this->__unset( 'page' );
+		$this->__set('paged', $this->args['page']);
+		$this->__unset('page');
 	}
 
 	/**
@@ -320,17 +320,17 @@ 
 	 */
 	public function per_page() {
 
-		if ( ! isset( $this->args['number'] ) ) {
+		if ( ! isset($this->args['number'])) {
 			return;
 		}
 
-		if ( $this->args['number'] == - 1 ) {
-			$this->__set( 'nopaging', true );
+		if ($this->args['number'] == - 1) {
+			$this->__set('nopaging', true);
 		} else {
-			$this->__set( 'posts_per_page', $this->args['number'] );
+			$this->__set('posts_per_page', $this->args['number']);
 		}
 
-		$this->__unset( 'number' );
+		$this->__unset('number');
 	}
 
 	/**
@@ -342,12 +342,12 @@ 
 	 * @return void
 	 */
 	public function month() {
-		if ( ! isset( $this->args['month'] ) ) {
+		if ( ! isset($this->args['month'])) {
 			return;
 		}
 
-		$this->__set( 'monthnum', $this->args['month'] );
-		$this->__unset( 'month' );
+		$this->__set('monthnum', $this->args['month']);
+		$this->__unset('month');
 	}
 
 	/**
@@ -359,23 +359,23 @@ 
 	 * @return void
 	 */
 	public function orderby() {
-		switch ( $this->args['orderby'] ) {
+		switch ($this->args['orderby']) {
 			case 'amount':
-				$this->__set( 'orderby', 'meta_value_num' );
-				$this->__set( 'meta_key', '_give_payment_total' );
+				$this->__set('orderby', 'meta_value_num');
+				$this->__set('meta_key', '_give_payment_total');
 				break;
 
 			case 'status':
-				$this->__set( 'orderby', 'post_status' );
+				$this->__set('orderby', 'post_status');
 				break;
 
 			case 'donation_form':
-				$this->__set( 'orderby', 'meta_value' );
-				$this->__set( 'meta_key', '_give_payment_form_title' );
+				$this->__set('orderby', 'meta_value');
+				$this->__set('meta_key', '_give_payment_form_title');
 				break;
 
 			default:
-				$this->__set( 'orderby', $this->args['orderby'] );
+				$this->__set('orderby', $this->args['orderby']);
 				break;
 		}
 	}
@@ -392,19 +392,19 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function custom_orderby( $order, $query ) {
+	public function custom_orderby($order, $query) {
 
-		if ( ! empty( $query->query['post_type'] ) ) {
-			$post_types = is_array( $query->query['post_type'] ) ? $query->query['post_type'] : array( $query->query['post_type'] );
+		if ( ! empty($query->query['post_type'])) {
+			$post_types = is_array($query->query['post_type']) ? $query->query['post_type'] : array($query->query['post_type']);
 
-			if ( ! in_array( 'give_payment', $post_types ) || is_array( $query->query['orderby'] ) ) {
+			if ( ! in_array('give_payment', $post_types) || is_array($query->query['orderby'])) {
 				return $order;
 			}
 
 			global $wpdb;
-			switch ( $query->query['orderby'] ) {
+			switch ($query->query['orderby']) {
 				case 'post_status':
-					$order = $wpdb->posts . '.post_status ' . strtoupper( $query->query['order'] );
+					$order = $wpdb->posts.'.post_status '.strtoupper($query->query['order']);
 					break;
 			}
 		}
@@ -421,11 +421,11 @@ 
 	 * @return void
 	 */
 	public function user() {
-		if ( is_null( $this->args['user'] ) ) {
+		if (is_null($this->args['user'])) {
 			return;
 		}
 
-		if ( is_numeric( $this->args['user'] ) ) {
+		if (is_numeric($this->args['user'])) {
 			$user_key = '_give_payment_user_id';
 		} else {
 			$user_key = '_give_payment_user_email';
@@ -447,7 +447,7 @@ 
 	 * @return  void
 	 */
 	public function donor() {
-		if ( is_null( $this->args['donor'] ) || ! is_numeric( $this->args['donor'] ) ) {
+		if (is_null($this->args['donor']) || ! is_numeric($this->args['donor'])) {
 			return;
 		}
 
@@ -469,33 +469,33 @@ 
 	 */
 	public function search() {
 
-		if ( ! isset( $this->args['s'] ) ) {
+		if ( ! isset($this->args['s'])) {
 			return;
 		}
 
-		$search = trim( $this->args['s'] );
+		$search = trim($this->args['s']);
 
-		if ( empty( $search ) ) {
+		if (empty($search)) {
 			return;
 		}
 
-		$is_email = is_email( $search ) || strpos( $search, '@' ) !== false;
-		$is_user  = strpos( $search, strtolower( 'user:' ) ) !== false;
+		$is_email = is_email($search) || strpos($search, '@') !== false;
+		$is_user  = strpos($search, strtolower('user:')) !== false;
 
-		if ( ! empty( $this->args['search_in_notes'] ) ) {
+		if ( ! empty($this->args['search_in_notes'])) {
 
-			$notes = give_get_payment_notes( 0, $search );
+			$notes = give_get_payment_notes(0, $search);
 
-			if ( ! empty( $notes ) ) {
+			if ( ! empty($notes)) {
 
-				$payment_ids = wp_list_pluck( (array) $notes, 'comment_post_ID' );
+				$payment_ids = wp_list_pluck((array) $notes, 'comment_post_ID');
 
-				$this->__set( 'post__in', $payment_ids );
+				$this->__set('post__in', $payment_ids);
 			}
 
-			$this->__unset( 's' );
+			$this->__unset('s');
 
-		} elseif ( $is_email || strlen( $search ) == 32 ) {
+		} elseif ($is_email || strlen($search) == 32) {
 
 			$key         = $is_email ? '_give_payment_user_email' : '_give_payment_purchase_key';
 			$search_meta = array(
@@ -504,19 +504,19 @@ 
 				'compare' => 'LIKE',
 			);
 
-			$this->__set( 'meta_query', $search_meta );
-			$this->__unset( 's' );
+			$this->__set('meta_query', $search_meta);
+			$this->__unset('s');
 
-		} elseif ( $is_user ) {
+		} elseif ($is_user) {
 
 			$search_meta = array(
 				'key'   => '_give_payment_user_id',
-				'value' => trim( str_replace( 'user:', '', strtolower( $search ) ) ),
+				'value' => trim(str_replace('user:', '', strtolower($search))),
 			);
 
-			$this->__set( 'meta_query', $search_meta );
+			$this->__set('meta_query', $search_meta);
 
-			if ( give_get_option( 'enable_sequential' ) ) {
+			if (give_get_option('enable_sequential')) {
 
 				$search_meta = array(
 					'key'     => '_give_payment_number',
@@ -524,19 +524,19 @@ 
 					'compare' => 'LIKE',
 				);
 
-				$this->__set( 'meta_query', $search_meta );
+				$this->__set('meta_query', $search_meta);
 
 				$this->args['meta_query']['relation'] = 'OR';
 
 			}
 
-			$this->__unset( 's' );
+			$this->__unset('s');
 
 		} elseif (
-			give_get_option( 'enable_sequential' ) &&
+			give_get_option('enable_sequential') &&
 			(
-				false !== strpos( $search, give_get_option( 'sequential_prefix' ) ) ||
-				false !== strpos( $search, give_get_option( 'sequential_postfix' ) )
+				false !== strpos($search, give_get_option('sequential_prefix')) ||
+				false !== strpos($search, give_get_option('sequential_postfix'))
 			)
 		) {
 
@@ -546,29 +546,29 @@ 
 				'compare' => 'LIKE',
 			);
 
-			$this->__set( 'meta_query', $search_meta );
-			$this->__unset( 's' );
+			$this->__set('meta_query', $search_meta);
+			$this->__unset('s');
 
-		} elseif ( is_numeric( $search ) ) {
+		} elseif (is_numeric($search)) {
 
-			$post = get_post( $search );
+			$post = get_post($search);
 
-			if ( is_object( $post ) && $post->post_type == 'give_payment' ) {
+			if (is_object($post) && $post->post_type == 'give_payment') {
 
 				$arr   = array();
 				$arr[] = $search;
-				$this->__set( 'post__in', $arr );
-				$this->__unset( 's' );
+				$this->__set('post__in', $arr);
+				$this->__unset('s');
 			}
-		} elseif ( '#' == substr( $search, 0, 1 ) ) {
+		} elseif ('#' == substr($search, 0, 1)) {
 
-			$search = str_replace( '#:', '', $search );
-			$search = str_replace( '#', '', $search );
-			$this->__set( 'give_forms', $search );
-			$this->__unset( 's' );
+			$search = str_replace('#:', '', $search);
+			$search = str_replace('#', '', $search);
+			$this->__set('give_forms', $search);
+			$this->__unset('s');
 
 		} else {
-			$this->__set( 's', $search );
+			$this->__set('s', $search);
 
 		}
 
@@ -583,8 +583,8 @@ 
 	 * @return void
 	 */
 	public function mode() {
-		if ( empty( $this->args['mode'] ) || $this->args['mode'] == 'all' ) {
-			$this->__unset( 'mode' );
+		if (empty($this->args['mode']) || $this->args['mode'] == 'all') {
+			$this->__unset('mode');
 
 			return;
 		}
@@ -606,10 +606,10 @@ 
 	 * @return void
 	 */
 	public function children() {
-		if ( empty( $this->args['children'] ) ) {
-			$this->__set( 'post_parent', 0 );
+		if (empty($this->args['children'])) {
+			$this->__set('post_parent', 0);
 		}
-		$this->__unset( 'children' );
+		$this->__unset('children');
 	}
 
 	/**
@@ -622,13 +622,13 @@ 
 	 */
 	public function give_forms() {
 
-		if ( empty( $this->args['give_forms'] ) ) {
+		if (empty($this->args['give_forms'])) {
 			return;
 		}
 
 		$compare = '=';
 
-		if ( is_array( $this->args['give_forms'] ) ) {
+		if (is_array($this->args['give_forms'])) {
 			$compare = 'IN';
 		}
 
@@ -642,7 +642,7 @@ 
 			)
 		);
 
-		$this->__unset( 'give_forms' );
+		$this->__unset('give_forms');
 
 	}
 
@@ -656,13 +656,13 @@ 
 	 */
 	public function gateway_filter() {
 
-		if ( empty( $this->args['gateway'] ) ) {
+		if (empty($this->args['gateway'])) {
 			return;
 		}
 
 		$compare = '=';
 
-		if ( is_array( $this->args['gateway'] ) ) {
+		if (is_array($this->args['gateway'])) {
 			$compare = 'IN';
 		}
 
@@ -676,7 +676,7 @@ 
 			)
 		);
 
-		$this->__unset( 'gateway' );
+		$this->__unset('gateway');
 
 	}
 

includes/payments/class-payment-stats.php

Spacing +41 added lines, -41 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -38,17 +38,17 @@ 
 	 *
 	 * @return float|int                Total amount of donations based on the passed arguments.
 	 */
-	public function get_sales( $form_id = 0, $start_date = false, $end_date = false, $status = 'publish' ) {
+	public function get_sales($form_id = 0, $start_date = false, $end_date = false, $status = 'publish') {
 
-		$this->setup_dates( $start_date, $end_date );
+		$this->setup_dates($start_date, $end_date);
 
 		// Make sure start date is valid
-		if ( is_wp_error( $this->start_date ) ) {
+		if (is_wp_error($this->start_date)) {
 			return $this->start_date;
 		}
 
 		// Make sure end date is valid
-		if ( is_wp_error( $this->end_date ) ) {
+		if (is_wp_error($this->end_date)) {
 			return $this->end_date;
 		}
 
@@ -57,18 +57,18 @@ 
 			'start_date' => $this->start_date,
 			'end_date'   => $this->end_date,
 			'fields'     => 'ids',
-			'number'     => - 1,
+			'number'     => -1,
 		);
 
-		if ( ! empty( $form_id ) ) {
+		if ( ! empty($form_id)) {
 			$args['give_forms'] = $form_id;
 		}
 
 		/* @var Give_Payments_Query $payments */
-		$payments = new Give_Payments_Query( $args );
+		$payments = new Give_Payments_Query($args);
 		$payments = $payments->get_payments();
 
-		return count( $payments );
+		return count($payments);
 	}
 
 
@@ -85,16 +85,16 @@ 
 	 *
 	 * @return float|int                Total amount of donations based on the passed arguments.
 	 */
-	public function get_earnings( $form_id = 0, $start_date = false, $end_date = false, $gateway_id = false ) {
-		$this->setup_dates( $start_date, $end_date );
+	public function get_earnings($form_id = 0, $start_date = false, $end_date = false, $gateway_id = false) {
+		$this->setup_dates($start_date, $end_date);
 
 		// Make sure start date is valid
-		if ( is_wp_error( $this->start_date ) ) {
+		if (is_wp_error($this->start_date)) {
 			return $this->start_date;
 		}
 
 		// Make sure end date is valid
-		if ( is_wp_error( $this->end_date ) ) {
+		if (is_wp_error($this->end_date)) {
 			return $this->end_date;
 		}
 
@@ -104,12 +104,12 @@ 
 			'start_date' => $this->start_date,
 			'end_date'   => $this->end_date,
 			'fields'     => 'ids',
-			'number'     => - 1,
+			'number'     => -1,
 		);
 
 
 		// Filter by Gateway ID meta_key
-		if ( $gateway_id ) {
+		if ($gateway_id) {
 			$args['meta_query'][] = array(
 				'key'   => '_give_payment_gateway',
 				'value' => $gateway_id,
@@ -117,39 +117,39 @@ 
 		}
 
 		// Filter by Gateway ID meta_key
-		if ( $form_id ) {
+		if ($form_id) {
 			$args['meta_query'][] = array(
 				'key'   => '_give_payment_form_id',
 				'value' => $form_id,
 			);
 		}
 
-		if ( ! empty( $args['meta_query'] ) && 1 < count( $args['meta_query'] ) ) {
+		if ( ! empty($args['meta_query']) && 1 < count($args['meta_query'])) {
 			$args['meta_query']['relation'] = 'AND';
 		}
 
-		$args = apply_filters( 'give_stats_earnings_args', $args );
-		$key  = Give_Cache::get_key( 'give_stats', $args );
+		$args = apply_filters('give_stats_earnings_args', $args);
+		$key  = Give_Cache::get_key('give_stats', $args);
 
 		// Set transient for faster stats.
-		$earnings = Give_Cache::get( $key );
+		$earnings = Give_Cache::get($key);
 
-		if ( false === $earnings ) {
+		if (false === $earnings) {
 
 			$this->timestamp = false;
-			$payments        = new Give_Payments_Query( $args );
+			$payments        = new Give_Payments_Query($args);
 			$payments        = $payments->get_payments();
 			$earnings        = 0;
 
-			if ( ! empty( $payments ) ) {
-				foreach ( $payments as $payment ) {
-					$earnings += give_donation_amount( $payment->ID );
+			if ( ! empty($payments)) {
+				foreach ($payments as $payment) {
+					$earnings += give_donation_amount($payment->ID);
 				}
 
 			}
 
 			// Cache the results for one hour.
-			Give_Cache::set( $key, give_sanitize_amount_for_db( $earnings ), 60 * 60 );
+			Give_Cache::set($key, give_sanitize_amount_for_db($earnings), 60 * 60);
 		}
 
 		/**
@@ -163,10 +163,10 @@ 
 		 * @param  string|bool $end_date   Earning end date.
 		 * @param  string|bool $gateway_id Payment gateway id.
 		 */
-		$earnings = apply_filters( 'give_get_earnings', $earnings, $form_id, $start_date, $end_date, $gateway_id );
+		$earnings = apply_filters('give_get_earnings', $earnings, $form_id, $start_date, $end_date, $gateway_id);
 
 		//return earnings
-		return round( $earnings, give_get_price_decimals( $form_id ) );
+		return round($earnings, give_get_price_decimals($form_id));
 
 	}
 
@@ -183,17 +183,17 @@ 
 	 *
 	 * @return float|int                Total amount of donations based on the passed arguments.
 	 */
-	public function get_earnings_cache_key( $form_id = 0, $start_date = false, $end_date = false, $gateway_id = false ) {
+	public function get_earnings_cache_key($form_id = 0, $start_date = false, $end_date = false, $gateway_id = false) {
 
-		$this->setup_dates( $start_date, $end_date );
+		$this->setup_dates($start_date, $end_date);
 
 		// Make sure start date is valid
-		if ( is_wp_error( $this->start_date ) ) {
+		if (is_wp_error($this->start_date)) {
 			return $this->start_date;
 		}
 
 		// Make sure end date is valid
-		if ( is_wp_error( $this->end_date ) ) {
+		if (is_wp_error($this->end_date)) {
 			return $this->end_date;
 		}
 
@@ -203,12 +203,12 @@ 
 			'start_date' => $this->start_date,
 			'end_date'   => $this->end_date,
 			'fields'     => 'ids',
-			'number'     => - 1,
+			'number'     => -1,
 		);
 
 
 		// Filter by Gateway ID meta_key
-		if ( $gateway_id ) {
+		if ($gateway_id) {
 			$args['meta_query'][] = array(
 				'key'   => '_give_payment_gateway',
 				'value' => $gateway_id,
@@ -216,19 +216,19 @@ 
 		}
 
 		// Filter by Gateway ID meta_key
-		if ( $form_id ) {
+		if ($form_id) {
 			$args['meta_query'][] = array(
 				'key'   => '_give_payment_form_id',
 				'value' => $form_id,
 			);
 		}
 
-		if ( ! empty( $args['meta_query'] ) && 1 < count( $args['meta_query'] ) ) {
+		if ( ! empty($args['meta_query']) && 1 < count($args['meta_query'])) {
 			$args['meta_query']['relation'] = 'AND';
 		}
 
-		$args = apply_filters( 'give_stats_earnings_args', $args );
-		$key  = Give_Cache::get_key( 'give_stats', $args );
+		$args = apply_filters('give_stats_earnings_args', $args);
+		$key  = Give_Cache::get_key('give_stats', $args);
 
 		//return earnings
 		return $key;
@@ -245,16 +245,16 @@ 
 	 *
 	 * @return array       Best selling forms
 	 */
-	public function get_best_selling( $number = 10 ) {
+	public function get_best_selling($number = 10) {
 
 		global $wpdb;
 
-		$give_forms = $wpdb->get_results( $wpdb->prepare(
+		$give_forms = $wpdb->get_results($wpdb->prepare(
 			"SELECT post_id as form_id, max(meta_value) as sales
 				FROM $wpdb->postmeta WHERE meta_key='_give_form_sales' AND meta_value > 0
 				GROUP BY meta_value+0
 				DESC LIMIT %d;", $number
-		) );
+		));
 
 		return $give_forms;
 	}

includes/class-notices.php

Spacing +103 added lines, -103 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -53,21 +53,21 @@ 
 	 * @since 1.8.9
 	 */
 	public function __construct() {
-		add_action( 'admin_notices', array( $this, 'render_admin_notices' ), 999 );
-		add_action( 'give_dismiss_notices', array( $this, 'dismiss_notices' ) );
+		add_action('admin_notices', array($this, 'render_admin_notices'), 999);
+		add_action('give_dismiss_notices', array($this, 'dismiss_notices'));
 
-		add_action( 'give_frontend_notices', array( $this, 'render_frontend_notices' ), 999 );
-		add_action( 'give_donation_form_before_personal_info', array( $this, 'render_frontend_notices' ) );
-		add_action( 'give_ajax_donation_errors', array( $this, 'render_frontend_notices' ) );
+		add_action('give_frontend_notices', array($this, 'render_frontend_notices'), 999);
+		add_action('give_donation_form_before_personal_info', array($this, 'render_frontend_notices'));
+		add_action('give_ajax_donation_errors', array($this, 'render_frontend_notices'));
 
 		/**
 		 * Backward compatibility for deprecated params.
 		 *
 		 * @since 1.8.14
 		 */
-		add_filter( 'give_register_notice_args', array( $this, 'bc_deprecated_params' ) );
-		add_filter( 'give_frontend_errors_args', array( $this, 'bc_deprecated_params' ) );
-		add_filter( 'give_frontend_notice_args', array( $this, 'bc_deprecated_params' ) );
+		add_filter('give_register_notice_args', array($this, 'bc_deprecated_params'));
+		add_filter('give_frontend_errors_args', array($this, 'bc_deprecated_params'));
+		add_filter('give_frontend_notice_args', array($this, 'bc_deprecated_params'));
 	}
 
 	/**
@@ -80,19 +80,19 @@ 
 	 *
 	 * @return array
 	 */
-	public function bc_deprecated_params( $args ) {
+	public function bc_deprecated_params($args) {
 		/**
 		 *  Param: auto_dismissible
 		 *  deprecated in 1.8.14
 		 *
 		 *  Check if auto_dismissible is set and it true then unset and change dismissible parameter value to auto
 		 */
-		if ( isset( $args['auto_dismissible'] ) ) {
-			if ( ! empty( $args['auto_dismissible'] ) ) {
+		if (isset($args['auto_dismissible'])) {
+			if ( ! empty($args['auto_dismissible'])) {
 				$args['dismissible'] = 'auto';
 			}
 			// unset auto_dismissible as it has been deprecated.
-			unset( $args['auto_dismissible'] );
+			unset($args['auto_dismissible']);
 		}
 
 		return $args;
@@ -108,9 +108,9 @@ 
 	 *
 	 * @return bool
 	 */
-	public function register_notice( $notice_args ) {
+	public function register_notice($notice_args) {
 		// Bailout.
-		if ( empty( $notice_args['id'] ) || array_key_exists( $notice_args['id'], self::$notices ) ) {
+		if (empty($notice_args['id']) || array_key_exists($notice_args['id'], self::$notices)) {
 			return false;
 		}
 
@@ -159,39 +159,39 @@ 
 		 *
 		 * @since 1.8.14
 		 */
-		$notice_args = apply_filters( 'give_register_notice_args', $notice_args );
+		$notice_args = apply_filters('give_register_notice_args', $notice_args);
 
 		// Set extra dismiss links if any.
-		if ( false !== strpos( $notice_args['description'], 'data-dismiss-interval' ) ) {
+		if (false !== strpos($notice_args['description'], 'data-dismiss-interval')) {
 
-			preg_match_all( "/data-([^\"]*)=\"([^\"]*)\"/", $notice_args['description'], $extra_notice_dismiss_link );
+			preg_match_all("/data-([^\"]*)=\"([^\"]*)\"/", $notice_args['description'], $extra_notice_dismiss_link);
 
-			if ( ! empty( $extra_notice_dismiss_link ) ) {
-				$extra_notice_dismiss_links = array_chunk( current( $extra_notice_dismiss_link ), 3 );
-				foreach ( $extra_notice_dismiss_links as $extra_notice_dismiss_link ) {
+			if ( ! empty($extra_notice_dismiss_link)) {
+				$extra_notice_dismiss_links = array_chunk(current($extra_notice_dismiss_link), 3);
+				foreach ($extra_notice_dismiss_links as $extra_notice_dismiss_link) {
 					// Create array og key ==> value by parsing query string created after renaming data attributes.
-					$data_attribute_query_str = str_replace( array( 'data-', '-', '"' ), array(
+					$data_attribute_query_str = str_replace(array('data-', '-', '"'), array(
 						'',
 						'_',
 						'',
-					), implode( '&', $extra_notice_dismiss_link ) );
+					), implode('&', $extra_notice_dismiss_link));
 
-					$notice_args['extra_links'][] = wp_parse_args( $data_attribute_query_str );
+					$notice_args['extra_links'][] = wp_parse_args($data_attribute_query_str);
 				}
 			}
 		}
 
 
-		self::$notices[ $notice_args['id'] ] = $notice_args;
+		self::$notices[$notice_args['id']] = $notice_args;
 
 		// Auto set show param if not already set.
-		if ( ! isset( self::$notices[ $notice_args['id'] ]['show'] ) ) {
-			self::$notices[ $notice_args['id'] ]['show'] = $this->is_notice_dismissed( $notice_args ) ? false : true;
+		if ( ! isset(self::$notices[$notice_args['id']]['show'])) {
+			self::$notices[$notice_args['id']]['show'] = $this->is_notice_dismissed($notice_args) ? false : true;
 		}
 
 		// Auto set time interval for shortly.
-		if ( 'shortly' === self::$notices[ $notice_args['id'] ]['dismiss_interval'] ) {
-			self::$notices[ $notice_args['id'] ]['dismiss_interval_time'] = DAY_IN_SECONDS;
+		if ('shortly' === self::$notices[$notice_args['id']]['dismiss_interval']) {
+			self::$notices[$notice_args['id']]['dismiss_interval_time'] = DAY_IN_SECONDS;
 		}
 
 		return true;
@@ -205,51 +205,51 @@ 
 	 */
 	public function render_admin_notices() {
 		// Bailout.
-		if ( empty( self::$notices ) ) {
+		if (empty(self::$notices)) {
 			return;
 		}
 
 		$output = '';
 
-		foreach ( self::$notices as $notice_id => $notice ) {
+		foreach (self::$notices as $notice_id => $notice) {
 			// Check flag set to true to show notice.
-			if ( ! $notice['show'] ) {
+			if ( ! $notice['show']) {
 				continue;
 			}
 
 
 			// Render custom html.
-			if( ! empty( $notice['description_html'] ) ) {
+			if ( ! empty($notice['description_html'])) {
 				$output .= "{$notice['description_html']} \n";
 				continue;
 			}
 
 			// Check if notice dismissible or not.
-			if ( ! self::$has_auto_dismissible_notice ) {
-				self::$has_auto_dismissible_notice = ( 'auto' === $notice['dismissible'] );
+			if ( ! self::$has_auto_dismissible_notice) {
+				self::$has_auto_dismissible_notice = ('auto' === $notice['dismissible']);
 			}
 
 			// Check if notice dismissible or not.
-			if ( ! self::$has_dismiss_interval_notice ) {
+			if ( ! self::$has_dismiss_interval_notice) {
 				self::$has_dismiss_interval_notice = $notice['dismiss_interval'];
 			}
 
-			$css_id = ( false === strpos( $notice['id'], 'give' ) ? "give-{$notice['id']}" : $notice['id'] );
+			$css_id = (false === strpos($notice['id'], 'give') ? "give-{$notice['id']}" : $notice['id']);
 
-			$css_class = 'give-notice notice ' . ( empty( $notice['dismissible'] ) ? 'non' : 'is' ) . "-dismissible {$notice['type']} notice-{$notice['type']}";
-			$output    .= sprintf(
-				'<div id="%1$s" class="%2$s" data-dismissible="%3$s" data-dismissible-type="%4$s" data-dismiss-interval="%5$s" data-notice-id="%6$s" data-security="%7$s" data-dismiss-interval-time="%8$s" style="display: none">' . " \n",
+			$css_class = 'give-notice notice '.(empty($notice['dismissible']) ? 'non' : 'is')."-dismissible {$notice['type']} notice-{$notice['type']}";
+			$output .= sprintf(
+				'<div id="%1$s" class="%2$s" data-dismissible="%3$s" data-dismissible-type="%4$s" data-dismiss-interval="%5$s" data-notice-id="%6$s" data-security="%7$s" data-dismiss-interval-time="%8$s" style="display: none">'." \n",
 				$css_id,
 				$css_class,
-				give_clean( $notice['dismissible'] ),
+				give_clean($notice['dismissible']),
 				$notice['dismissible_type'],
 				$notice['dismiss_interval'],
 				$notice['id'],
-				empty( $notice['dismissible_type'] ) ? '' : wp_create_nonce( "give_edit_{$notice_id}_notice" ),
+				empty($notice['dismissible_type']) ? '' : wp_create_nonce("give_edit_{$notice_id}_notice"),
 				$notice['dismiss_interval_time']
 			);
 
-			$output .= ( 0 === strpos( $notice['description'], '<div' ) || 0 === strpos( $notice['description'], '<p' ) ? $notice['description'] : "<p>{$notice['description']}</p>" );
+			$output .= (0 === strpos($notice['description'], '<div') || 0 === strpos($notice['description'], '<p') ? $notice['description'] : "<p>{$notice['description']}</p>");
 			$output .= "</div> \n";
 		}
 
@@ -267,18 +267,18 @@ 
 	 *
 	 * @param int $form_id
 	 */
-	public function render_frontend_notices( $form_id = 0 ) {
+	public function render_frontend_notices($form_id = 0) {
 		$errors = give_get_errors();
 
-		$request_form_id = isset( $_REQUEST['form-id'] ) ? absint( $_REQUEST['form-id'] ) : 0;
+		$request_form_id = isset($_REQUEST['form-id']) ? absint($_REQUEST['form-id']) : 0;
 
 		// Sanity checks first: Ensure that gateway returned errors display on the appropriate form.
-		if ( ! isset( $_POST['give_ajax'] ) && $request_form_id !== $form_id ) {
+		if ( ! isset($_POST['give_ajax']) && $request_form_id !== $form_id) {
 			return;
 		}
 
-		if ( $errors ) {
-			self::print_frontend_errors( $errors );
+		if ($errors) {
+			self::print_frontend_errors($errors);
 
 			give_clear_errors();
 		}
@@ -291,7 +291,7 @@ 
 	 * @access private
 	 */
 	private function print_js() {
-		if ( self::$has_auto_dismissible_notice ) :
+		if (self::$has_auto_dismissible_notice) :
 			?>
 			<script>
 				jQuery(document).ready(function () {
@@ -307,7 +307,7 @@ 
 			<?php
 		endif;
 
-		if ( self::$has_dismiss_interval_notice ) :
+		if (self::$has_dismiss_interval_notice) :
 			?>
 			<script>
 				jQuery(document).ready(function () {
@@ -386,29 +386,29 @@ 
 	 * @access public
 	 */
 	public function dismiss_notices() {
-		$_post     = give_clean( $_POST );
-		$notice_id = esc_attr( $_post['notice_id'] );
+		$_post     = give_clean($_POST);
+		$notice_id = esc_attr($_post['notice_id']);
 
 		// Bailout.
 		if (
-			empty( $notice_id ) ||
-			empty( $_post['dismissible_type'] ) ||
-			empty( $_post['dismiss_interval'] ) ||
-			! check_ajax_referer( "give_edit_{$notice_id}_notice", '_wpnonce' )
+			empty($notice_id) ||
+			empty($_post['dismissible_type']) ||
+			empty($_post['dismiss_interval']) ||
+			! check_ajax_referer("give_edit_{$notice_id}_notice", '_wpnonce')
 		) {
 			wp_send_json_error();
 		}
 
-		$notice_key = Give()->notices->get_notice_key( $notice_id, $_post['dismiss_interval'] );
-		if ( 'user' === $_post['dismissible_type'] ) {
+		$notice_key = Give()->notices->get_notice_key($notice_id, $_post['dismiss_interval']);
+		if ('user' === $_post['dismissible_type']) {
 			$current_user = wp_get_current_user();
-			$notice_key   = Give()->notices->get_notice_key( $notice_id, $_post['dismiss_interval'], $current_user->ID );
+			$notice_key   = Give()->notices->get_notice_key($notice_id, $_post['dismiss_interval'], $current_user->ID);
 		}
 
-		$notice_dismiss_time = ! empty( $_post['dismiss_interval_time'] ) ? $_post['dismiss_interval_time'] : null;
+		$notice_dismiss_time = ! empty($_post['dismiss_interval_time']) ? $_post['dismiss_interval_time'] : null;
 
 		// Save option to hide notice.
-		Give_Cache::set( $notice_key, true, $notice_dismiss_time, true );
+		Give_Cache::set($notice_key, true, $notice_dismiss_time, true);
 
 		wp_send_json_success();
 	}
@@ -426,18 +426,18 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_notice_key( $notice_id, $dismiss_interval = null, $user_id = 0 ) {
+	public function get_notice_key($notice_id, $dismiss_interval = null, $user_id = 0) {
 		$notice_key = "_give_notice_{$notice_id}";
 
-		if ( ! empty( $dismiss_interval ) ) {
+		if ( ! empty($dismiss_interval)) {
 			$notice_key .= "_{$dismiss_interval}";
 		}
 
-		if ( $user_id ) {
+		if ($user_id) {
 			$notice_key .= "_{$user_id}";
 		}
 
-		$notice_key = sanitize_key( $notice_key );
+		$notice_key = sanitize_key($notice_key);
 
 		return $notice_key;
 	}
@@ -450,11 +450,11 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_dismiss_link( $notice_args ) {
+	public function get_dismiss_link($notice_args) {
 		$notice_args = wp_parse_args(
 			$notice_args,
 			array(
-				'title'                 => __( 'Click here', 'give' ),
+				'title'                 => __('Click here', 'give'),
 				'dismissible_type'      => '',
 				'dismiss_interval'      => '',
 				'dismiss_interval_time' => null,
@@ -481,31 +481,31 @@ 
 	 *
 	 * @return bool|null
 	 */
-	public function is_notice_dismissed( $notice ) {
-		$notice_key          = $this->get_notice_key( $notice['id'], $notice['dismiss_interval'] );
+	public function is_notice_dismissed($notice) {
+		$notice_key          = $this->get_notice_key($notice['id'], $notice['dismiss_interval']);
 		$is_notice_dismissed = false;
 
-		if ( 'user' === $notice['dismissible_type'] ) {
+		if ('user' === $notice['dismissible_type']) {
 			$current_user = wp_get_current_user();
-			$notice_key   = Give()->notices->get_notice_key( $notice['id'], $notice['dismiss_interval'], $current_user->ID );
+			$notice_key   = Give()->notices->get_notice_key($notice['id'], $notice['dismiss_interval'], $current_user->ID);
 		}
 
-		$notice_data = Give_Cache::get( $notice_key, true );
+		$notice_data = Give_Cache::get($notice_key, true);
 
 		// Find notice dismiss link status if notice has extra dismissible links.
-		if ( ( empty( $notice_data ) || is_wp_error( $notice_data ) ) && ! empty( $notice['extra_links'] ) ) {
+		if ((empty($notice_data) || is_wp_error($notice_data)) && ! empty($notice['extra_links'])) {
 
-			foreach ( $notice['extra_links'] as $extra_link ) {
-				$new_notice_data = wp_parse_args( $extra_link, $notice );
-				unset( $new_notice_data['extra_links'] );
+			foreach ($notice['extra_links'] as $extra_link) {
+				$new_notice_data = wp_parse_args($extra_link, $notice);
+				unset($new_notice_data['extra_links']);
 
-				if ( $is_notice_dismissed = $this->is_notice_dismissed( $new_notice_data ) ) {
+				if ($is_notice_dismissed = $this->is_notice_dismissed($new_notice_data)) {
 					return $is_notice_dismissed;
 				}
 			}
 		}
 
-		$is_notice_dismissed = ! empty( $notice_data ) && ! is_wp_error( $notice_data );
+		$is_notice_dismissed = ! empty($notice_data) && ! is_wp_error($notice_data);
 
 		return $is_notice_dismissed;
 	}
@@ -519,9 +519,9 @@ 
 	 *
 	 * @param array $errors
 	 */
-	public static function print_frontend_errors( $errors ) {
+	public static function print_frontend_errors($errors) {
 		// Bailout.
-		if ( ! $errors ) {
+		if ( ! $errors) {
 			return;
 		}
 
@@ -536,37 +536,37 @@ 
 		);
 
 		// Note: we will remove give_errors class in future.
-		$classes = apply_filters( 'give_error_class', array( 'give_notices', 'give_errors' ) );
+		$classes = apply_filters('give_error_class', array('give_notices', 'give_errors'));
 
-		echo sprintf( '<div class="%s">', implode( ' ', $classes ) );
+		echo sprintf('<div class="%s">', implode(' ', $classes));
 
 		// Loop error codes and display errors.
-		foreach ( $errors as $error_id => $error ) {
+		foreach ($errors as $error_id => $error) {
 			// Backward compatibility v<1.8.11
-			if ( is_string( $error ) ) {
+			if (is_string($error)) {
 				$error = array(
 					'message'     => $error,
 					'notice_args' => array(),
 				);
 			}
 
-			$notice_args = wp_parse_args( $error['notice_args'], $default_notice_args );
+			$notice_args = wp_parse_args($error['notice_args'], $default_notice_args);
 
 			/**
 			 * Filter to modify Frontend Errors args before errors is display.
 			 *
 			 * @since 1.8.14
 			 */
-			$notice_args = apply_filters( 'give_frontend_errors_args', $notice_args );
+			$notice_args = apply_filters('give_frontend_errors_args', $notice_args);
 
 			echo sprintf(
 				'<div class="give_error give_notice" id="give_error_%1$s" data-dismissible="%2$s" data-dismiss-interval="%3$d">
 						<p><strong>%4$s</strong>: %5$s</p>
 					</div>',
 				$error_id,
-				give_clean( $notice_args['dismissible'] ),
-				absint( $notice_args['dismiss_interval'] ),
-				esc_html__( 'Error', 'give' ),
+				give_clean($notice_args['dismissible']),
+				absint($notice_args['dismiss_interval']),
+				esc_html__('Error', 'give'),
 				$error['message']
 			);
 		}
@@ -588,8 +588,8 @@ 
 	 *
 	 * @return  string
 	 */
-	public static function print_frontend_notice( $message, $echo = true, $notice_type = 'warning', $notice_args = array() ) {
-		if ( empty( $message ) ) {
+	public static function print_frontend_notice($message, $echo = true, $notice_type = 'warning', $notice_args = array()) {
+		if (empty($message)) {
 			return '';
 		}
 
@@ -603,14 +603,14 @@ 
 			'dismiss_interval' => 5000,
 		);
 
-		$notice_args = wp_parse_args( $notice_args, $default_notice_args );
+		$notice_args = wp_parse_args($notice_args, $default_notice_args);
 
 		/**
 		 * Filter to modify Frontend notice args before notices is display.
 		 *
 		 * @since 1.8.14
 		 */
-		$notice_args = apply_filters( 'give_frontend_notice_args', $notice_args );
+		$notice_args = apply_filters('give_frontend_notice_args', $notice_args);
 
 		// Note: we will remove give_errors class in future.
 		$error = sprintf(
@@ -620,12 +620,12 @@ 
 				</p>
 			</div>',
 			$notice_type,
-			give_clean( $notice_args['dismissible'] ),
-			absint( $notice_args['dismiss_interval'] ),
+			give_clean($notice_args['dismissible']),
+			absint($notice_args['dismiss_interval']),
 			$message
 		);
 
-		if ( ! $echo ) {
+		if ( ! $echo) {
 			return $error;
 		}
 
@@ -645,24 +645,24 @@ 
 	 *
 	 * @return string
 	 */
-	public function print_admin_notices( $notice_args = array() ) {
+	public function print_admin_notices($notice_args = array()) {
 		// Bailout.
-		if ( empty( $notice_args['description'] ) ) {
+		if (empty($notice_args['description'])) {
 			return '';
 		}
 
-		$defaults    = array(
+		$defaults = array(
 			'id'          => '',
 			'echo'        => true,
 			'notice_type' => 'warning',
 			'dismissible' => true,
 		);
-		$notice_args = wp_parse_args( $notice_args, $defaults );
+		$notice_args = wp_parse_args($notice_args, $defaults);
 
 		$output    = '';
-		$css_id    = ! empty( $notice_args['id'] ) ? $notice_args['id'] : uniqid( 'give-inline-notice-' );
+		$css_id    = ! empty($notice_args['id']) ? $notice_args['id'] : uniqid('give-inline-notice-');
 		$css_class = "notice-{$notice_args['notice_type']} give-notice notice inline";
-		$css_class .= ( $notice_args['dismissible'] ) ? ' is-dismissible' : '';
+		$css_class .= ($notice_args['dismissible']) ? ' is-dismissible' : '';
 		$output    .= sprintf(
 			'<div id="%1$s" class="%2$s"><p>%3$s</p></div>',
 			$css_id,
@@ -670,7 +670,7 @@ 
 			$notice_args['description']
 		);
 
-		if ( ! $notice_args['echo'] ) {
+		if ( ! $notice_args['echo']) {
 			return $output;
 		}
 

includes/shortcodes.php

Spacing +116 added lines, -116 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,40 +23,40 @@ 
  *
  * @return string|bool
  */
-function give_donation_history( $atts ) {
+function give_donation_history($atts) {
 
-	$donation_history_args = shortcode_atts( array(
+	$donation_history_args = shortcode_atts(array(
 		'id'             => true,
 		'date'           => true,
 		'donor'          => false,
 		'amount'         => true,
 		'status'         => false,
 		'payment_method' => false,
-	), $atts, 'donation_history' );
+	), $atts, 'donation_history');
 
 	// Always show receipt link.
 	$donation_history_args['details'] = true;
 
 	// Set Donation History Shortcode Arguments in session variable.
-	Give()->session->set( 'give_donation_history_args', $donation_history_args );
+	Give()->session->set('give_donation_history_args', $donation_history_args);
 
 	// If payment_key query arg exists, return receipt instead of donation history.
-	if ( isset( $_GET['payment_key'] ) ) {
+	if (isset($_GET['payment_key'])) {
 		ob_start();
-		echo give_receipt_shortcode( array() );
+		echo give_receipt_shortcode(array());
 
 		// Display donation history link only if it is not accessed via Receipt Access Link.
-		if ( give_get_receipt_session() ) {
+		if (give_get_receipt_session()) {
 			echo sprintf(
 				'<a href="%s">%s</a>',
-				esc_url( give_get_history_page_uri() ),
-				__( '&laquo; Return to All Donations', 'give' )
+				esc_url(give_get_history_page_uri()),
+				__('&laquo; Return to All Donations', 'give')
 			);
 		}
 		return ob_get_clean();
 	}
 
-	$email_access = give_get_option( 'email_access' );
+	$email_access = give_get_option('email_access');
 
 	/**
 	 * Determine access
@@ -67,31 +67,31 @@ 
 	if (
 		is_user_logged_in() ||
 		false !== Give()->session->get_session_expiration() ||
-		( give_is_setting_enabled( $email_access ) && Give()->email_access->token_exists ) ||
+		(give_is_setting_enabled($email_access) && Give()->email_access->token_exists) ||
 		true === give_get_history_session()
 	) {
 		ob_start();
-		give_get_template_part( 'history', 'donations' );
+		give_get_template_part('history', 'donations');
 
 		return ob_get_clean();
 
-	} elseif ( give_is_setting_enabled( $email_access ) ) {
+	} elseif (give_is_setting_enabled($email_access)) {
 		// Is Email-based access enabled?
 		ob_start();
-		give_get_template_part( 'email', 'login-form' );
+		give_get_template_part('email', 'login-form');
 
 		return ob_get_clean();
 
 	} else {
 
-		$output = apply_filters( 'give_donation_history_nonuser_message', Give()->notices->print_frontend_notice( __( 'You must be logged in to view your donation history. Please login using your account or create an account using the same email you used to donate with.', 'give' ), false ) );
-		$output .= do_shortcode( '[give_login]' );
+		$output = apply_filters('give_donation_history_nonuser_message', Give()->notices->print_frontend_notice(__('You must be logged in to view your donation history. Please login using your account or create an account using the same email you used to donate with.', 'give'), false));
+		$output .= do_shortcode('[give_login]');
 
 		return $output;
 	}
 }
 
-add_shortcode( 'donation_history', 'give_donation_history' );
+add_shortcode('donation_history', 'give_donation_history');
 
 /**
  * Donation Form Shortcode
@@ -104,8 +104,8 @@ 
  *
  * @return string
  */
-function give_form_shortcode( $atts ) {
-	$atts = shortcode_atts( array(
+function give_form_shortcode($atts) {
+	$atts = shortcode_atts(array(
 		'id'                    => '',
 		'show_title'            => true,
 		'show_goal'             => true,
@@ -113,21 +113,21 @@ 
 		'float_labels'          => '',
 		'display_style'         => '',
 		'continue_button_title' => '',
-	), $atts, 'give_form' );
+	), $atts, 'give_form');
 
 	// Convert string to bool.
-	$atts['show_title'] = filter_var( $atts['show_title'], FILTER_VALIDATE_BOOLEAN );
-	$atts['show_goal']  = filter_var( $atts['show_goal'], FILTER_VALIDATE_BOOLEAN );
+	$atts['show_title'] = filter_var($atts['show_title'], FILTER_VALIDATE_BOOLEAN);
+	$atts['show_goal']  = filter_var($atts['show_goal'], FILTER_VALIDATE_BOOLEAN);
 
 	//get the Give Form
 	ob_start();
-	give_get_donation_form( $atts );
+	give_get_donation_form($atts);
 	$final_output = ob_get_clean();
 
-	return apply_filters( 'give_donate_form', $final_output, $atts );
+	return apply_filters('give_donate_form', $final_output, $atts);
 }
 
-add_shortcode( 'give_form', 'give_form_shortcode' );
+add_shortcode('give_form', 'give_form_shortcode');
 
 /**
  * Donation Form Goal Shortcode.
@@ -140,37 +140,37 @@ 
  *
  * @return string
  */
-function give_goal_shortcode( $atts ) {
-	$atts = shortcode_atts( array(
+function give_goal_shortcode($atts) {
+	$atts = shortcode_atts(array(
 		'id'        => '',
 		'show_text' => true,
 		'show_bar'  => true,
-	), $atts, 'give_goal' );
+	), $atts, 'give_goal');
 
 
 	//get the Give Form.
 	ob_start();
 
 	//Sanity check 1: ensure there is an ID Provided.
-	if ( empty( $atts['id'] ) ) {
-		Give()->notices->print_frontend_notice( __( 'The shortcode is missing Donation Form ID attribute.', 'give' ), true );
+	if (empty($atts['id'])) {
+		Give()->notices->print_frontend_notice(__('The shortcode is missing Donation Form ID attribute.', 'give'), true);
 	}
 
 	//Sanity check 2: Check the form even has Goals enabled.
-	if ( ! give_is_setting_enabled( give_get_meta( $atts['id'], '_give_goal_option', true ) ) ) {
+	if ( ! give_is_setting_enabled(give_get_meta($atts['id'], '_give_goal_option', true))) {
 
-		Give()->notices->print_frontend_notice( __( 'The form does not have Goals enabled.', 'give' ), true );
+		Give()->notices->print_frontend_notice(__('The form does not have Goals enabled.', 'give'), true);
 	} else {
 		//Passed all sanity checks: output Goal.
-		give_show_goal_progress( $atts['id'], $atts );
+		give_show_goal_progress($atts['id'], $atts);
 	}
 
 	$final_output = ob_get_clean();
 
-	return apply_filters( 'give_goal_shortcode_output', $final_output, $atts );
+	return apply_filters('give_goal_shortcode_output', $final_output, $atts);
 }
 
-add_shortcode( 'give_goal', 'give_goal_shortcode' );
+add_shortcode('give_goal', 'give_goal_shortcode');
 
 
 /**
@@ -187,22 +187,22 @@ 
  *
  * @return string
  */
-function give_login_form_shortcode( $atts ) {
-	$atts = shortcode_atts( array(
+function give_login_form_shortcode($atts) {
+	$atts = shortcode_atts(array(
 		// Add backward compatibility for redirect attribute.
 		'redirect' => '',
 
 		'login-redirect'  => '',
 		'logout-redirect' => '',
-	), $atts, 'give_login' );
+	), $atts, 'give_login');
 
 	// Check login-redirect attribute first, if it empty or not found then check for redirect attribute and add value of this to login-redirect attribute.
-	$atts['login-redirect'] = ! empty( $atts['login-redirect'] ) ? $atts['login-redirect'] : ( ! empty( $atts['redirect'] ) ? $atts['redirect'] : '' );
+	$atts['login-redirect'] = ! empty($atts['login-redirect']) ? $atts['login-redirect'] : ( ! empty($atts['redirect']) ? $atts['redirect'] : '');
 
-	return give_login_form( $atts['login-redirect'], $atts['logout-redirect'] );
+	return give_login_form($atts['login-redirect'], $atts['logout-redirect']);
 }
 
-add_shortcode( 'give_login', 'give_login_form_shortcode' );
+add_shortcode('give_login', 'give_login_form_shortcode');
 
 /**
  * Register Shortcode.
@@ -217,15 +217,15 @@ 
  *
  * @return string
  */
-function give_register_form_shortcode( $atts ) {
-	$atts = shortcode_atts( array(
+function give_register_form_shortcode($atts) {
+	$atts = shortcode_atts(array(
 		'redirect' => '',
-	), $atts, 'give_register' );
+	), $atts, 'give_register');
 
-	return give_register_form( $atts['redirect'] );
+	return give_register_form($atts['redirect']);
 }
 
-add_shortcode( 'give_register', 'give_register_form_shortcode' );
+add_shortcode('give_register', 'give_register_form_shortcode');
 
 /**
  * Receipt Shortcode.
@@ -238,12 +238,12 @@ 
  *
  * @return string
  */
-function give_receipt_shortcode( $atts ) {
+function give_receipt_shortcode($atts) {
 
 	global $give_receipt_args;
 
-	$give_receipt_args = shortcode_atts( array(
-		'error'          => __( 'You are missing the payment key to view this donation receipt.', 'give' ),
+	$give_receipt_args = shortcode_atts(array(
+		'error'          => __('You are missing the payment key to view this donation receipt.', 'give'),
 		'price'          => true,
 		'donor'          => true,
 		'date'           => true,
@@ -252,49 +252,49 @@ 
 		'payment_id'     => true,
 		'payment_status' => false,
 		'status_notice'  => true,
-	), $atts, 'give_receipt' );
+	), $atts, 'give_receipt');
 
 	//set $session var
 	$session = give_get_purchase_session();
 
 	//set payment key var
-	if ( isset( $_GET['payment_key'] ) ) {
-		$payment_key = urldecode( $_GET['payment_key'] );
-	} elseif ( $session ) {
+	if (isset($_GET['payment_key'])) {
+		$payment_key = urldecode($_GET['payment_key']);
+	} elseif ($session) {
 		$payment_key = $session['purchase_key'];
-	} elseif ( $give_receipt_args['payment_key'] ) {
+	} elseif ($give_receipt_args['payment_key']) {
 		$payment_key = $give_receipt_args['payment_key'];
 	}
 
-	$email_access = give_get_option( 'email_access' );
+	$email_access = give_get_option('email_access');
 
 	// No payment_key found & Email Access is Turned on:
-	if ( ! isset( $payment_key ) && give_is_setting_enabled( $email_access ) && ! Give()->email_access->token_exists ) {
+	if ( ! isset($payment_key) && give_is_setting_enabled($email_access) && ! Give()->email_access->token_exists) {
 
 		ob_start();
 
-		give_get_template_part( 'email-login-form' );
+		give_get_template_part('email-login-form');
 
 		return ob_get_clean();
 
-	} elseif ( ! isset( $payment_key ) ) {
+	} elseif ( ! isset($payment_key)) {
 
-		return Give()->notices->print_frontend_notice( $give_receipt_args['error'], false, 'error' );
+		return Give()->notices->print_frontend_notice($give_receipt_args['error'], false, 'error');
 
 	}
 
-	$user_can_view = give_can_view_receipt( $payment_key );
+	$user_can_view = give_can_view_receipt($payment_key);
 
 	// Key was provided, but user is logged out. Offer them the ability to login and view the receipt.
-	if ( ! $user_can_view && give_is_setting_enabled( $email_access ) && ! Give()->email_access->token_exists ) {
+	if ( ! $user_can_view && give_is_setting_enabled($email_access) && ! Give()->email_access->token_exists) {
 
 		ob_start();
 
-		give_get_template_part( 'email-login-form' );
+		give_get_template_part('email-login-form');
 
 		return ob_get_clean();
 
-	} elseif ( ! $user_can_view ) {
+	} elseif ( ! $user_can_view) {
 
 		global $give_login_redirect;
 
@@ -302,9 +302,9 @@ 
 
 		ob_start();
 
-		Give()->notices->print_frontend_notice( apply_filters( 'give_must_be_logged_in_error_message', __( 'You must be logged in to view this donation receipt.', 'give' ) ) );
+		Give()->notices->print_frontend_notice(apply_filters('give_must_be_logged_in_error_message', __('You must be logged in to view this donation receipt.', 'give')));
 
-		give_get_template_part( 'shortcode', 'login' );
+		give_get_template_part('shortcode', 'login');
 
 		$login_form = ob_get_clean();
 
@@ -318,20 +318,20 @@ 
 	 * or if user is logged out and donation was made as a guest, the donation session is checked for
 	 * or if user is logged in and the user can view sensitive shop data.
 	 */
-	if ( ! apply_filters( 'give_user_can_view_receipt', $user_can_view, $give_receipt_args ) ) {
-		return Give()->notices->print_frontend_notice( $give_receipt_args['error'], false, 'error' );
+	if ( ! apply_filters('give_user_can_view_receipt', $user_can_view, $give_receipt_args)) {
+		return Give()->notices->print_frontend_notice($give_receipt_args['error'], false, 'error');
 	}
 
 	ob_start();
 
-	give_get_template_part( 'shortcode', 'receipt' );
+	give_get_template_part('shortcode', 'receipt');
 
 	$display = ob_get_clean();
 
 	return $display;
 }
 
-add_shortcode( 'give_receipt', 'give_receipt_shortcode' );
+add_shortcode('give_receipt', 'give_receipt_shortcode');
 
 /**
  * Profile Editor Shortcode.
@@ -350,25 +350,25 @@ 
  *
  * @return string Output generated from the profile editor
  */
-function give_profile_editor_shortcode( $atts ) {
+function give_profile_editor_shortcode($atts) {
 
 	ob_start();
 
 	// Restrict access to donor profile, if donor and user are disconnected.
-	$is_donor_disconnected = get_user_meta( get_current_user_id(), '_give_is_donor_disconnected', true );
-	if( is_user_logged_in() && $is_donor_disconnected ) {
-		Give()->notices->print_frontend_notice( __( 'Your Donor and User profile are no longer connected. Please contact the site administrator.', 'give' ), true, 'error' );
+	$is_donor_disconnected = get_user_meta(get_current_user_id(), '_give_is_donor_disconnected', true);
+	if (is_user_logged_in() && $is_donor_disconnected) {
+		Give()->notices->print_frontend_notice(__('Your Donor and User profile are no longer connected. Please contact the site administrator.', 'give'), true, 'error');
 		return false;
 	}
 
-	give_get_template_part( 'shortcode', 'profile-editor' );
+	give_get_template_part('shortcode', 'profile-editor');
 
 	$display = ob_get_clean();
 
 	return $display;
 }
 
-add_shortcode( 'give_profile_editor', 'give_profile_editor_shortcode' );
+add_shortcode('give_profile_editor', 'give_profile_editor_shortcode');
 
 /**
  * Process Profile Updater Form.
@@ -381,32 +381,32 @@ 
  *
  * @return bool
  */
-function give_process_profile_editor_updates( $data ) {
+function give_process_profile_editor_updates($data) {
 	// Profile field change request
-	if ( empty( $_POST['give_profile_editor_submit'] ) && ! is_user_logged_in() ) {
+	if (empty($_POST['give_profile_editor_submit']) && ! is_user_logged_in()) {
 		return false;
 	}
 
 	// Nonce security
-	if ( ! wp_verify_nonce( $data['give_profile_editor_nonce'], 'give-profile-editor-nonce' ) ) {
+	if ( ! wp_verify_nonce($data['give_profile_editor_nonce'], 'give-profile-editor-nonce')) {
 		return false;
 	}
 
 	$user_id       = get_current_user_id();
-	$old_user_data = get_userdata( $user_id );
-
-	$display_name     = isset( $data['give_display_name'] ) ? sanitize_text_field( $data['give_display_name'] ) : $old_user_data->display_name;
-	$first_name       = isset( $data['give_first_name'] ) ? sanitize_text_field( $data['give_first_name'] ) : $old_user_data->first_name;
-	$last_name        = isset( $data['give_last_name'] ) ? sanitize_text_field( $data['give_last_name'] ) : $old_user_data->last_name;
-	$email            = isset( $data['give_email'] ) ? sanitize_email( $data['give_email'] ) : $old_user_data->user_email;
-	$line1            = ( isset( $data['give_address_line1'] ) ? sanitize_text_field( $data['give_address_line1'] ) : '' );
-	$line2            = ( isset( $data['give_address_line2'] ) ? sanitize_text_field( $data['give_address_line2'] ) : '' );
-	$city             = ( isset( $data['give_address_city'] ) ? sanitize_text_field( $data['give_address_city'] ) : '' );
-	$state            = ( isset( $data['give_address_state'] ) ? sanitize_text_field( $data['give_address_state'] ) : '' );
-	$zip              = ( isset( $data['give_address_zip'] ) ? sanitize_text_field( $data['give_address_zip'] ) : '' );
-	$country          = ( isset( $data['give_address_country'] ) ? sanitize_text_field( $data['give_address_country'] ) : '' );
-	$password         = ! empty( $data['give_new_user_pass1'] ) ? $data['give_new_user_pass1'] : '';
-	$confirm_password = ! empty( $data['give_new_user_pass2'] ) ? $data['give_new_user_pass2'] : '';
+	$old_user_data = get_userdata($user_id);
+
+	$display_name     = isset($data['give_display_name']) ? sanitize_text_field($data['give_display_name']) : $old_user_data->display_name;
+	$first_name       = isset($data['give_first_name']) ? sanitize_text_field($data['give_first_name']) : $old_user_data->first_name;
+	$last_name        = isset($data['give_last_name']) ? sanitize_text_field($data['give_last_name']) : $old_user_data->last_name;
+	$email            = isset($data['give_email']) ? sanitize_email($data['give_email']) : $old_user_data->user_email;
+	$line1            = (isset($data['give_address_line1']) ? sanitize_text_field($data['give_address_line1']) : '');
+	$line2            = (isset($data['give_address_line2']) ? sanitize_text_field($data['give_address_line2']) : '');
+	$city             = (isset($data['give_address_city']) ? sanitize_text_field($data['give_address_city']) : '');
+	$state            = (isset($data['give_address_state']) ? sanitize_text_field($data['give_address_state']) : '');
+	$zip              = (isset($data['give_address_zip']) ? sanitize_text_field($data['give_address_zip']) : '');
+	$country          = (isset($data['give_address_country']) ? sanitize_text_field($data['give_address_country']) : '');
+	$password         = ! empty($data['give_new_user_pass1']) ? $data['give_new_user_pass1'] : '';
+	$confirm_password = ! empty($data['give_new_user_pass2']) ? $data['give_new_user_pass2'] : '';
 
 	$userdata = array(
 		'ID'           => $user_id,
@@ -417,8 +417,8 @@ 
 		'user_pass'    => $password,
 	);
 
-	if( empty( $line1 ) || empty( $city ) || empty( $state ) || empty( $zip ) || empty( $country ) ) {
-		give_set_error( 'give-empty-address-fields', __( 'Please fill in the required address fields.', 'give' ) );
+	if (empty($line1) || empty($city) || empty($state) || empty($zip) || empty($country)) {
+		give_set_error('give-empty-address-fields', __('Please fill in the required address fields.', 'give'));
 	}
 
 	$address = array(
@@ -438,48 +438,48 @@ 
 	 * @param int $user_id The ID of the user.
 	 * @param array $userdata User info, including ID, first name, last name, display name and email.
 	 */
-	do_action( 'give_pre_update_user_profile', $user_id, $userdata );
+	do_action('give_pre_update_user_profile', $user_id, $userdata);
 
 	// Validate First Name.
-	if( empty( $first_name ) ) {
-		give_set_error( 'give-empty-first-name', __( 'Please enter first name.', 'give' ) );
+	if (empty($first_name)) {
+		give_set_error('give-empty-first-name', __('Please enter first name.', 'give'));
 	}
 
 	// Make sure to validate passwords for existing Donors.
-	give_validate_user_password( $password, $confirm_password );
+	give_validate_user_password($password, $confirm_password);
 
-	if ( empty( $email ) ) {
+	if (empty($email)) {
 		// Make sure email should not be empty.
-		give_set_error( 'email_empty', __( 'The email you entered is empty.', 'give' ) );
+		give_set_error('email_empty', __('The email you entered is empty.', 'give'));
 
-	} else if ( ! is_email( $email ) ) {
+	} else if ( ! is_email($email)) {
 		// Make sure email should be valid.
-		give_set_error( 'email_not_valid', __( 'The email you entered is not valid. Please use another', 'give' ) );
+		give_set_error('email_not_valid', __('The email you entered is not valid. Please use another', 'give'));
 
-	} else if ( $email != $old_user_data->user_email ) {
+	} else if ($email != $old_user_data->user_email) {
 		// Make sure the new email doesn't belong to another user
-		if ( email_exists( $email ) ) {
-			give_set_error( 'user_email_exists', __( 'The email you entered belongs to another user. Please use another.', 'give' ) );
-		} elseif ( Give()->donors->get_donor_by( 'email', $email ) ){
+		if (email_exists($email)) {
+			give_set_error('user_email_exists', __('The email you entered belongs to another user. Please use another.', 'give'));
+		} elseif (Give()->donors->get_donor_by('email', $email)) {
 			// Make sure the new email doesn't belong to another user
-			give_set_error( 'donor_email_exists', __( 'The email you entered belongs to another donor. Please use another.', 'give' ) );
+			give_set_error('donor_email_exists', __('The email you entered belongs to another donor. Please use another.', 'give'));
 		}
 	}
 
 	// Check for errors
 	$errors = give_get_errors();
 
-	if ( $errors ) {
+	if ($errors) {
 		// Send back to the profile editor if there are errors
-		wp_redirect( $data['give_redirect'] );
+		wp_redirect($data['give_redirect']);
 		give_die();
 	}
 
 	// Update the user
-	$meta    = update_user_meta( $user_id, '_give_user_address', $address );
-	$updated = wp_update_user( $userdata );
+	$meta    = update_user_meta($user_id, '_give_user_address', $address);
+	$updated = wp_update_user($userdata);
 
-	if ( $updated ) {
+	if ($updated) {
 
 		/**
 		 * Fires after updating user profile.
@@ -489,12 +489,12 @@ 
 		 * @param int $user_id The ID of the user.
 		 * @param array $userdata User info, including ID, first name, last name, display name and email.
 		 */
-		do_action( 'give_user_profile_updated', $user_id, $userdata );
-		wp_redirect( add_query_arg( 'updated', 'true', $data['give_redirect'] ) );
+		do_action('give_user_profile_updated', $user_id, $userdata);
+		wp_redirect(add_query_arg('updated', 'true', $data['give_redirect']));
 		give_die();
 	}
 
 	return false;
 }
 
-add_action( 'give_edit_user_profile', 'give_process_profile_editor_updates' );
+add_action('give_edit_user_profile', 'give_process_profile_editor_updates');

includes/currency-functions.php

Spacing +137 added lines, -137 removed lines

@@ -20,19 +20,19 @@ 
  *
  * @return string The currency code
  */
-function give_get_currency( $donation_or_form_id = null, $args = array() ) {
+function give_get_currency($donation_or_form_id = null, $args = array()) {
 
 	// Get currency from donation
-	if ( is_numeric( $donation_or_form_id ) && 'give_payment' === get_post_type( $donation_or_form_id ) ) {
-		$donation_meta = give_get_meta( $donation_or_form_id, '_give_payment_meta', true );
+	if (is_numeric($donation_or_form_id) && 'give_payment' === get_post_type($donation_or_form_id)) {
+		$donation_meta = give_get_meta($donation_or_form_id, '_give_payment_meta', true);
 
-		if ( ! empty( $donation_meta['currency'] ) ) {
+		if ( ! empty($donation_meta['currency'])) {
 			$currency = $donation_meta['currency'];
 		} else {
-			$currency = give_get_option( 'currency', 'USD' );
+			$currency = give_get_option('currency', 'USD');
 		}
 	} else {
-		$currency = give_get_option( 'currency', 'USD' );
+		$currency = give_get_option('currency', 'USD');
 	}
 
 	/**
@@ -40,7 +40,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	return apply_filters( 'give_currency', $currency, $donation_or_form_id, $args );
+	return apply_filters('give_currency', $currency, $donation_or_form_id, $args);
 }
 
 /**
@@ -52,9 +52,9 @@ 
  */
 function give_get_currency_position() {
 
-	$currency_pos = give_get_option( 'currency_position', 'before' );
+	$currency_pos = give_get_option('currency_position', 'before');
 
-	return apply_filters( 'give_currency_position', $currency_pos );
+	return apply_filters('give_currency_position', $currency_pos);
 }
 
 /**
@@ -67,7 +67,7 @@ 
 function give_get_currencies_list() {
 	$currencies = array(
 		'USD' => array(
-			'admin_label' => __( 'US Dollars ($)', 'give' ),
+			'admin_label' => __('US Dollars ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -77,7 +77,7 @@ 
 			),
 		),
 		'EUR' => array(
-			'admin_label' => __( 'Euros (€)', 'give' ),
+			'admin_label' => __('Euros (€)', 'give'),
 			'symbol'      => '€',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -87,7 +87,7 @@ 
 			),
 		),
 		'GBP' => array(
-			'admin_label' => __( 'Pounds Sterling (£)', 'give' ),
+			'admin_label' => __('Pounds Sterling (£)', 'give'),
 			'symbol'      => '£',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -97,7 +97,7 @@ 
 			),
 		),
 		'AUD' => array(
-			'admin_label' => __( 'Australian Dollars ($)', 'give' ),
+			'admin_label' => __('Australian Dollars ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -107,7 +107,7 @@ 
 			),
 		),
 		'BRL' => array(
-			'admin_label' => __( 'Brazilian Real (R$)', 'give' ),
+			'admin_label' => __('Brazilian Real (R$)', 'give'),
 			'symbol'      => 'R$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -117,7 +117,7 @@ 
 			),
 		),
 		'CAD' => array(
-			'admin_label' => __( 'Canadian Dollars ($)', 'give' ),
+			'admin_label' => __('Canadian Dollars ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -127,7 +127,7 @@ 
 			),
 		),
 		'CZK' => array(
-			'admin_label' => __( 'Czech Koruna (Kč)', 'give' ),
+			'admin_label' => __('Czech Koruna (Kč)', 'give'),
 			'symbol'      => 'Kč',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -137,7 +137,7 @@ 
 			),
 		),
 		'DKK' => array(
-			'admin_label' => __( 'Danish Krone (kr.)', 'give' ),
+			'admin_label' => __('Danish Krone (kr.)', 'give'),
 			'symbol'      => ' kr. ',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -147,7 +147,7 @@ 
 			),
 		),
 		'HKD' => array(
-			'admin_label' => __( 'Hong Kong Dollar ($)', 'give' ),
+			'admin_label' => __('Hong Kong Dollar ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -157,7 +157,7 @@ 
 			),
 		),
 		'HUF' => array(
-			'admin_label' => __( 'Hungarian Forint (Ft)', 'give' ),
+			'admin_label' => __('Hungarian Forint (Ft)', 'give'),
 			'symbol'      => 'Ft',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -167,7 +167,7 @@ 
 			),
 		),
 		'ILS' => array(
-			'admin_label' => __( 'Israeli Shekel (₪)', 'give' ),
+			'admin_label' => __('Israeli Shekel (₪)', 'give'),
 			'symbol'      => '₪',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -177,7 +177,7 @@ 
 			),
 		),
 		'JPY' => array(
-			'admin_label' => __( 'Japanese Yen (¥)', 'give' ),
+			'admin_label' => __('Japanese Yen (¥)', 'give'),
 			'symbol'      => '¥',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -187,7 +187,7 @@ 
 			),
 		),
 		'MYR' => array(
-			'admin_label' => __( 'Malaysian Ringgits (RM)', 'give' ),
+			'admin_label' => __('Malaysian Ringgits (RM)', 'give'),
 			'symbol'      => 'RM',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -197,7 +197,7 @@ 
 			),
 		),
 		'MXN' => array(
-			'admin_label' => __( 'Mexican Peso ($)', 'give' ),
+			'admin_label' => __('Mexican Peso ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -207,7 +207,7 @@ 
 			),
 		),
 		'MAD' => array(
-			'admin_label' => __( 'Moroccan Dirham (.د.م)', 'give' ),
+			'admin_label' => __('Moroccan Dirham (.د.م)', 'give'),
 			'symbol'      => '.د.م',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -217,7 +217,7 @@ 
 			),
 		),
 		'NZD' => array(
-			'admin_label' => __( 'New Zealand Dollar ($)', 'give' ),
+			'admin_label' => __('New Zealand Dollar ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -227,7 +227,7 @@ 
 			),
 		),
 		'NOK' => array(
-			'admin_label' => __( 'Norwegian Krone (Kr.)', 'give' ),
+			'admin_label' => __('Norwegian Krone (Kr.)', 'give'),
 			'symbol'      => 'kr.',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -237,7 +237,7 @@ 
 			),
 		),
 		'PHP' => array(
-			'admin_label' => __( 'Philippine Pesos (₱)', 'give' ),
+			'admin_label' => __('Philippine Pesos (₱)', 'give'),
 			'symbol'      => '₱',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -247,7 +247,7 @@ 
 			),
 		),
 		'PLN' => array(
-			'admin_label' => __( 'Polish Zloty (zł)', 'give' ),
+			'admin_label' => __('Polish Zloty (zł)', 'give'),
 			'symbol'      => 'zł',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -257,7 +257,7 @@ 
 			),
 		),
 		'SGD' => array(
-			'admin_label' => __( 'Singapore Dollar ($)', 'give' ),
+			'admin_label' => __('Singapore Dollar ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -267,7 +267,7 @@ 
 			),
 		),
 		'KRW' => array(
-			'admin_label' => __( 'South Korean Won (₩)', 'give' ),
+			'admin_label' => __('South Korean Won (₩)', 'give'),
 			'symbol'      => '₩',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -277,7 +277,7 @@ 
 			),
 		),
 		'ZAR' => array(
-			'admin_label' => __( 'South African Rand (R)', 'give' ),
+			'admin_label' => __('South African Rand (R)', 'give'),
 			'symbol'      => 'R',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -287,7 +287,7 @@ 
 			),
 		),
 		'SEK' => array(
-			'admin_label' => __( 'Swedish Krona (kr)', 'give' ),
+			'admin_label' => __('Swedish Krona (kr)', 'give'),
 			'symbol'      => ' kr. ',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -297,7 +297,7 @@ 
 			),
 		),
 		'CHF' => array(
-			'admin_label' => __( 'Swiss Franc (Fr)', 'give' ),
+			'admin_label' => __('Swiss Franc (Fr)', 'give'),
 			'symbol'      => 'Fr',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -307,7 +307,7 @@ 
 			),
 		),
 		'TWD' => array(
-			'admin_label' => __( 'Taiwan New Dollars (NT$)', 'give' ),
+			'admin_label' => __('Taiwan New Dollars (NT$)', 'give'),
 			'symbol'      => 'NT$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -317,7 +317,7 @@ 
 			),
 		),
 		'THB' => array(
-			'admin_label' => __( 'Thai Baht (฿)', 'give' ),
+			'admin_label' => __('Thai Baht (฿)', 'give'),
 			'symbol'      => '฿',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -327,7 +327,7 @@ 
 			),
 		),
 		'INR' => array(
-			'admin_label' => __( 'Indian Rupee (₹)', 'give' ),
+			'admin_label' => __('Indian Rupee (₹)', 'give'),
 			'symbol'      => '₹',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -337,7 +337,7 @@ 
 			),
 		),
 		'TRY' => array(
-			'admin_label' => __( 'Turkish Lira (₺)', 'give' ),
+			'admin_label' => __('Turkish Lira (₺)', 'give'),
 			'symbol'      => '₺',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -347,7 +347,7 @@ 
 			),
 		),
 		'IRR' => array(
-			'admin_label' => __( 'Iranian Rial (﷼)', 'give' ),
+			'admin_label' => __('Iranian Rial (﷼)', 'give'),
 			'symbol'      => '﷼',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -357,7 +357,7 @@ 
 			),
 		),
 		'RUB' => array(
-			'admin_label' => __( 'Russian Rubles (руб)', 'give' ),
+			'admin_label' => __('Russian Rubles (руб)', 'give'),
 			'symbol'      => '₽',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -367,7 +367,7 @@ 
 			),
 		),
 		'AED' => array(
-			'admin_label' => __( 'United Arab Emirates dirham (د.إ)', 'give' ),
+			'admin_label' => __('United Arab Emirates dirham (د.إ)', 'give'),
 			'symbol'      => 'د.إ',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -377,7 +377,7 @@ 
 			),
 		),
 		'AMD' => array(
-			'admin_label' => __( 'Armenian dram (AMD)', 'give' ),
+			'admin_label' => __('Armenian dram (AMD)', 'give'),
 			'symbol'      => 'AMD', // Add backward compatibility. Using AMD in place of ֏
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -387,7 +387,7 @@ 
 			),
 		),
 		'ANG' => array(
-			'admin_label' => __( 'Netherlands Antillean guilder (ƒ)', 'give' ),
+			'admin_label' => __('Netherlands Antillean guilder (ƒ)', 'give'),
 			'symbol'      => 'ƒ',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -397,7 +397,7 @@ 
 			),
 		),
 		'ARS' => array(
-			'admin_label' => __( 'Argentine peso ($)', 'give' ),
+			'admin_label' => __('Argentine peso ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -407,7 +407,7 @@ 
 			),
 		),
 		'AWG' => array(
-			'admin_label' => __( 'Aruban florin (ƒ)', 'give' ),
+			'admin_label' => __('Aruban florin (ƒ)', 'give'),
 			'symbol'      => 'ƒ',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -417,7 +417,7 @@ 
 			),
 		),
 		'BAM' => array(
-			'admin_label' => __( 'Bosnia and Herzegovina convertible mark (KM)', 'give' ),
+			'admin_label' => __('Bosnia and Herzegovina convertible mark (KM)', 'give'),
 			'symbol'      => 'KM',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -427,7 +427,7 @@ 
 			),
 		),
 		'BDT' => array(
-			'admin_label' => __( 'Bangladeshi taka (৳)', 'give' ),
+			'admin_label' => __('Bangladeshi taka (৳)', 'give'),
 			'symbol'      => '৳',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -437,7 +437,7 @@ 
 			),
 		),
 		'BHD' => array(
-			'admin_label' => __( 'Bahraini dinar (.د.ب)', 'give' ),
+			'admin_label' => __('Bahraini dinar (.د.ب)', 'give'),
 			'symbol'      => '.د.ب',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -447,7 +447,7 @@ 
 			),
 		),
 		'BMD' => array(
-			'admin_label' => __( 'Bermudian dollar (BD$)', 'give' ),
+			'admin_label' => __('Bermudian dollar (BD$)', 'give'),
 			'symbol'      => 'BD$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -457,7 +457,7 @@ 
 			),
 		),
 		'BND' => array(
-			'admin_label' => __( 'Brunei dollar (B$)', 'give' ),
+			'admin_label' => __('Brunei dollar (B$)', 'give'),
 			'symbol'      => 'B$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -467,7 +467,7 @@ 
 			),
 		),
 		'BOB' => array(
-			'admin_label' => __( 'Bolivian boliviano (Bs.)', 'give' ),
+			'admin_label' => __('Bolivian boliviano (Bs.)', 'give'),
 			'symbol'      => 'Bs.',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -477,7 +477,7 @@ 
 			),
 		),
 		'BSD' => array(
-			'admin_label' => __( 'Bahamian dollar (B$)', 'give' ),
+			'admin_label' => __('Bahamian dollar (B$)', 'give'),
 			'symbol'      => 'B$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -487,7 +487,7 @@ 
 			),
 		),
 		'BWP' => array(
-			'admin_label' => __( 'Botswana pula (P)', 'give' ),
+			'admin_label' => __('Botswana pula (P)', 'give'),
 			'symbol'      => 'P',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -497,7 +497,7 @@ 
 			),
 		),
 		'BZD' => array(
-			'admin_label' => __( 'Belizean dollar (BZ$)', 'give' ),
+			'admin_label' => __('Belizean dollar (BZ$)', 'give'),
 			'symbol'      => 'BZ$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -507,7 +507,7 @@ 
 			),
 		),
 		'CLP' => array(
-			'admin_label' => __( 'Chilean peso ($)', 'give' ),
+			'admin_label' => __('Chilean peso ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -517,7 +517,7 @@ 
 			),
 		),
 		'CNY' => array(
-			'admin_label' => __( 'Chinese yuan (¥)', 'give' ),
+			'admin_label' => __('Chinese yuan (¥)', 'give'),
 			'symbol'      => '¥',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -527,7 +527,7 @@ 
 			),
 		),
 		'COP' => array(
-			'admin_label' => __( 'Colombian peso ($)', 'give' ),
+			'admin_label' => __('Colombian peso ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -537,7 +537,7 @@ 
 			),
 		),
 		'CRC' => array(
-			'admin_label' => __( 'Costa Rican colón (₡)', 'give' ),
+			'admin_label' => __('Costa Rican colón (₡)', 'give'),
 			'symbol'      => '₡',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -547,7 +547,7 @@ 
 			),
 		),
 		'CUC' => array(
-			'admin_label' => __( 'Cuban convertible peso (₱)', 'give' ),
+			'admin_label' => __('Cuban convertible peso (₱)', 'give'),
 			'symbol'      => '₱',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -557,7 +557,7 @@ 
 			),
 		),
 		'CUP' => array(
-			'admin_label' => __( 'Cuban convertible peso (₱)', 'give' ),
+			'admin_label' => __('Cuban convertible peso (₱)', 'give'),
 			'symbol'      => '₱',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -567,7 +567,7 @@ 
 			),
 		),
 		'DOP' => array(
-			'admin_label' => __( 'Dominican peso (RD$)', 'give' ),
+			'admin_label' => __('Dominican peso (RD$)', 'give'),
 			'symbol'      => 'RD$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -577,7 +577,7 @@ 
 			),
 		),
 		'EGP' => array(
-			'admin_label' => __( 'Egyptian pound (E£)', 'give' ),
+			'admin_label' => __('Egyptian pound (E£)', 'give'),
 			'symbol'      => 'E£',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -587,7 +587,7 @@ 
 			),
 		),
 		'GIP' => array(
-			'admin_label' => __( 'Gibraltar pound (£)', 'give' ),
+			'admin_label' => __('Gibraltar pound (£)', 'give'),
 			'symbol'      => '£',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -597,7 +597,7 @@ 
 			),
 		),
 		'GTQ' => array(
-			'admin_label' => __( 'Guatemalan quetzal (Q)', 'give' ),
+			'admin_label' => __('Guatemalan quetzal (Q)', 'give'),
 			'symbol'      => 'Q',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -607,7 +607,7 @@ 
 			),
 		),
 		'HNL' => array(
-			'admin_label' => __( 'Honduran lempira (L)', 'give' ),
+			'admin_label' => __('Honduran lempira (L)', 'give'),
 			'symbol'      => 'L',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -617,7 +617,7 @@ 
 			),
 		),
 		'HRK' => array(
-			'admin_label' => __( 'Croatian kuna (kn)', 'give' ),
+			'admin_label' => __('Croatian kuna (kn)', 'give'),
 			'symbol'      => 'kn',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -627,7 +627,7 @@ 
 			),
 		),
 		'IDR' => array(
-			'admin_label' => __( 'Indonesian rupiah (Rp)', 'give' ),
+			'admin_label' => __('Indonesian rupiah (Rp)', 'give'),
 			'symbol'      => 'Rp',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -637,7 +637,7 @@ 
 			),
 		),
 		'ISK' => array(
-			'admin_label' => __( 'Icelandic króna (kr)', 'give' ),
+			'admin_label' => __('Icelandic króna (kr)', 'give'),
 			'symbol'      => 'kr',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -647,7 +647,7 @@ 
 			),
 		),
 		'JMD' => array(
-			'admin_label' => __( 'Jamaican dollar (J$)', 'give' ),
+			'admin_label' => __('Jamaican dollar (J$)', 'give'),
 			'symbol'      => 'j$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -657,7 +657,7 @@ 
 			),
 		),
 		'JOD' => array(
-			'admin_label' => __( 'Jordanian dinar (د.ا)', 'give' ),
+			'admin_label' => __('Jordanian dinar (د.ا)', 'give'),
 			'symbol'      => 'د.ا',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -667,7 +667,7 @@ 
 			),
 		),
 		'KES' => array(
-			'admin_label' => __( 'Kenyan shilling (KSh)', 'give' ),
+			'admin_label' => __('Kenyan shilling (KSh)', 'give'),
 			'symbol'      => 'KSh',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -677,7 +677,7 @@ 
 			),
 		),
 		'KWD' => array(
-			'admin_label' => __( 'Kuwaiti dinar (د.ك)', 'give' ),
+			'admin_label' => __('Kuwaiti dinar (د.ك)', 'give'),
 			'symbol'      => 'د.ك',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -687,7 +687,7 @@ 
 			),
 		),
 		'KYD' => array(
-			'admin_label' => __( 'Cayman Islands dollar (KY$)', 'give' ),
+			'admin_label' => __('Cayman Islands dollar (KY$)', 'give'),
 			'symbol'      => 'KY$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -697,7 +697,7 @@ 
 			),
 		),
 		'MKD' => array(
-			'admin_label' => __( 'Macedonian denar (ден)', 'give' ),
+			'admin_label' => __('Macedonian denar (ден)', 'give'),
 			'symbol'      => 'ден',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -707,7 +707,7 @@ 
 			),
 		),
 		'NPR' => array(
-			'admin_label' => __( 'Nepalese rupee (NɌs)', 'give' ),
+			'admin_label' => __('Nepalese rupee (NɌs)', 'give'),
 			'symbol'      => '₨',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -717,7 +717,7 @@ 
 			),
 		),
 		'OMR' => array(
-			'admin_label' => __( 'Omani rial (ر.ع.)', 'give' ),
+			'admin_label' => __('Omani rial (ر.ع.)', 'give'),
 			'symbol'      => 'ر.ع.',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -727,7 +727,7 @@ 
 			),
 		),
 		'PEN' => array(
-			'admin_label' => __( 'Peruvian nuevo sol (S/.)', 'give' ),
+			'admin_label' => __('Peruvian nuevo sol (S/.)', 'give'),
 			'symbol'      => 'S/.',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -737,7 +737,7 @@ 
 			),
 		),
 		'PKR' => array(
-			'admin_label' => __( 'Pakistani rupee (Ɍs)', 'give' ),
+			'admin_label' => __('Pakistani rupee (Ɍs)', 'give'),
 			'symbol'      => '₨',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -747,7 +747,7 @@ 
 			),
 		),
 		'RON' => array(
-			'admin_label' => __( 'Romanian leu (L)', 'give' ),
+			'admin_label' => __('Romanian leu (L)', 'give'),
 			'symbol'      => 'L',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -757,7 +757,7 @@ 
 			),
 		),
 		'SAR' => array(
-			'admin_label' => __( 'Saudi riyal (ر.س)', 'give' ),
+			'admin_label' => __('Saudi riyal (ر.س)', 'give'),
 			'symbol'      => 'ر.س',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -767,7 +767,7 @@ 
 			),
 		),
 		'SZL' => array(
-			'admin_label' => __( 'Swazi lilangeni (L)', 'give' ),
+			'admin_label' => __('Swazi lilangeni (L)', 'give'),
 			'symbol'      => 'L',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -777,7 +777,7 @@ 
 			),
 		),
 		'TOP' => array(
-			'admin_label' => __( 'Tongan paʻanga (T$)', 'give' ),
+			'admin_label' => __('Tongan paʻanga (T$)', 'give'),
 			'symbol'      => 'T$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -787,7 +787,7 @@ 
 			),
 		),
 		'TZS' => array(
-			'admin_label' => __( 'Tanzanian shilling (TSh)', 'give' ),
+			'admin_label' => __('Tanzanian shilling (TSh)', 'give'),
 			'symbol'      => 'TSh',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -797,7 +797,7 @@ 
 			),
 		),
 		'UAH' => array(
-			'admin_label' => __( 'Ukrainian hryvnia (₴)', 'give' ),
+			'admin_label' => __('Ukrainian hryvnia (₴)', 'give'),
 			'symbol'      => '₴',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -807,7 +807,7 @@ 
 			),
 		),
 		'UYU' => array(
-			'admin_label' => __( 'Uruguayan peso ($U)', 'give' ),
+			'admin_label' => __('Uruguayan peso ($U)', 'give'),
 			'symbol'      => '$U',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -817,7 +817,7 @@ 
 			),
 		),
 		'VEF' => array(
-			'admin_label' => __( 'Venezuelan bolívar (Bs)', 'give' ),
+			'admin_label' => __('Venezuelan bolívar (Bs)', 'give'),
 			'symbol'      => 'Bs',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -827,7 +827,7 @@ 
 			),
 		),
 		'XCD' => array(
-			'admin_label' => __( 'East Caribbean dollar (EC$)', 'give' ),
+			'admin_label' => __('East Caribbean dollar (EC$)', 'give'),
 			'symbol'      => 'EC$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -852,7 +852,7 @@ 
 	 *
 	 * @param array $currencies
 	 */
-	return (array) apply_filters( 'give_currencies', $currencies );
+	return (array) apply_filters('give_currencies', $currencies);
 }
 
 /**
@@ -864,22 +864,22 @@ 
  *
  * @return array $currencies A list of the available currencies
  */
-function give_get_currencies( $info = 'admin_label' ) {
+function give_get_currencies($info = 'admin_label') {
 
 	$currencies = give_get_currencies_list();
 
 	// Backward compatibility: handle old way of currency registration.
 	// Backward compatibility: Return desired result.
-	if ( ! empty( $currencies ) ) {
-		foreach ( $currencies as $currency_code => $currency_setting ) {
-			if ( is_string( $currency_setting ) ) {
-				$currencies[ $currency_code ] = array(
+	if ( ! empty($currencies)) {
+		foreach ($currencies as $currency_code => $currency_setting) {
+			if (is_string($currency_setting)) {
+				$currencies[$currency_code] = array(
 					'admin_label' => $currency_setting,
 				);
 			}
 
-			$currencies[ $currency_code ] = wp_parse_args(
-				$currencies[ $currency_code ],
+			$currencies[$currency_code] = wp_parse_args(
+				$currencies[$currency_code],
 				array(
 					'admin_label' => '',
 					'symbol'      => $currency_code,
@@ -888,8 +888,8 @@ 
 			);
 		}
 
-		if ( ! empty( $info ) && is_string( $info ) && 'all' !== $info ) {
-			$currencies = wp_list_pluck( $currencies, $info );
+		if ( ! empty($info) && is_string($info) && 'all' !== $info) {
+			$currencies = wp_list_pluck($currencies, $info);
 		}
 	}
 
@@ -906,12 +906,12 @@ 
  *
  * @return array
  */
-function give_currency_symbols( $decode_currencies = false ) {
-	$currencies = give_get_currencies( 'symbol' );
+function give_currency_symbols($decode_currencies = false) {
+	$currencies = give_get_currencies('symbol');
 
-	if ( $decode_currencies ) {
-		array_walk( $currencies, function ( &$currency_symbol ) {
-			$currency_symbol = html_entity_decode( $currency_symbol, ENT_COMPAT, 'UTF-8' );
+	if ($decode_currencies) {
+		array_walk($currencies, function(&$currency_symbol) {
+			$currency_symbol = html_entity_decode($currency_symbol, ENT_COMPAT, 'UTF-8');
 		} );
 	}
 
@@ -922,7 +922,7 @@ 
 	 *
 	 * @param array $currencies
 	 */
-	return apply_filters( 'give_currency_symbols', $currencies );
+	return apply_filters('give_currency_symbols', $currencies);
 }
 
 
@@ -939,14 +939,14 @@ 
  *
  * @return string           The symbol to use for the currency
  */
-function give_currency_symbol( $currency = '', $decode_currency = false ) {
+function give_currency_symbol($currency = '', $decode_currency = false) {
 
-	if ( empty( $currency ) ) {
+	if (empty($currency)) {
 		$currency = give_get_currency();
 	}
 
-	$currencies = give_currency_symbols( $decode_currency );
-	$symbol     = array_key_exists( $currency, $currencies ) ? $currencies[ $currency ] : $currency;
+	$currencies = give_currency_symbols($decode_currency);
+	$symbol     = array_key_exists($currency, $currencies) ? $currencies[$currency] : $currency;
 
 	/**
 	 * Filter the currency symbol
@@ -956,7 +956,7 @@ 
 	 * @param string $symbol
 	 * @param string $currency
 	 */
-	return apply_filters( 'give_currency_symbol', $symbol, $currency );
+	return apply_filters('give_currency_symbol', $symbol, $currency);
 }
 
 
@@ -969,13 +969,13 @@ 
  *
  * @return string
  */
-function give_get_currency_name( $currency_code ) {
+function give_get_currency_name($currency_code) {
 	$currency_name  = '';
 	$currency_names = give_get_currencies();
 
-	if ( $currency_code && array_key_exists( $currency_code, $currency_names ) ) {
-		$currency_name = explode( '(', $currency_names[ $currency_code ] );
-		$currency_name = trim( current( $currency_name ) );
+	if ($currency_code && array_key_exists($currency_code, $currency_names)) {
+		$currency_name = explode('(', $currency_names[$currency_code]);
+		$currency_name = trim(current($currency_name));
 	}
 
 	/**
@@ -986,7 +986,7 @@ 
 	 * @param string $currency_name
 	 * @param string $currency_code
 	 */
-	return apply_filters( 'give_currency_name', $currency_name, $currency_code );
+	return apply_filters('give_currency_name', $currency_name, $currency_code);
 }
 
 /**
@@ -999,17 +999,17 @@ 
  *
  * @return mixed|string
  */
-function give_currency_filter( $price = '', $args = array() ) {
+function give_currency_filter($price = '', $args = array()) {
 
 	// Get functions arguments.
 	$func_args = func_get_args();
 
 	// Backward compatibility: modify second param to array
-	if ( isset( $func_args[1] ) && is_string( $func_args[1] ) ) {
+	if (isset($func_args[1]) && is_string($func_args[1])) {
 		$args = array(
-			'currency_code'   => isset( $func_args[1] ) ? $func_args[1] : '',
-			'decode_currency' => isset( $func_args[2] ) ? $func_args[2] : false,
-			'form_id'         => isset( $func_args[3] ) ? $func_args[3] : '',
+			'currency_code'   => isset($func_args[1]) ? $func_args[1] : '',
+			'decode_currency' => isset($func_args[2]) ? $func_args[2] : false,
+			'form_id'         => isset($func_args[3]) ? $func_args[3] : '',
 		);
 	}
 
@@ -1023,22 +1023,22 @@ 
 		)
 	);
 
-	if ( empty( $args['currency_code'] ) || ! array_key_exists( (string) $args['currency_code'], give_get_currencies() ) ) {
-		$args['currency_code'] = give_get_currency( $args['form_id'] );
+	if (empty($args['currency_code']) || ! array_key_exists((string) $args['currency_code'], give_get_currencies())) {
+		$args['currency_code'] = give_get_currency($args['form_id']);
 	}
 
-	$args['position'] = give_get_option( 'currency_position', 'before' );
+	$args['position'] = give_get_option('currency_position', 'before');
 
 	$negative = $price < 0;
 
-	if ( $negative ) {
+	if ($negative) {
 		// Remove proceeding "-".
-		$price = substr( $price, 1 );
+		$price = substr($price, 1);
 	}
 
-	$args['symbol'] = give_currency_symbol( $args['currency_code'], $args['decode_currency'] );
+	$args['symbol'] = give_currency_symbol($args['currency_code'], $args['decode_currency']);
 
-	switch ( $args['currency_code'] ) :
+	switch ($args['currency_code']) :
 		case 'GBP' :
 		case 'BRL' :
 		case 'EUR' :
@@ -1068,13 +1068,13 @@ 
 		case 'MAD' :
 		case 'KRW' :
 		case 'ZAR' :
-			$formatted = ( 'before' === $args['position'] ? $args['symbol'] . $price : $price . $args['symbol'] );
+			$formatted = ('before' === $args['position'] ? $args['symbol'].$price : $price.$args['symbol']);
 			break;
 		case 'NOK':
-			$formatted = ( 'before' === $args['position'] ? $args['symbol'] . ' ' . $price : $price . ' ' . $args['symbol'] );
+			$formatted = ('before' === $args['position'] ? $args['symbol'].' '.$price : $price.' '.$args['symbol']);
 			break;
 		default:
-			$formatted = ( 'before' === $args['position'] ? $args['symbol'] . ' ' . $price : $price . ' ' . $args['symbol'] );
+			$formatted = ('before' === $args['position'] ? $args['symbol'].' '.$price : $price.' '.$args['symbol']);
 			break;
 	endswitch;
 
@@ -1083,7 +1083,7 @@ 
 	 *
 	 * @since 1.8.17
 	 */
-	$formatted = apply_filters( 'give_currency_filter', $formatted, $args, $price );
+	$formatted = apply_filters('give_currency_filter', $formatted, $args, $price);
 
 	/**
 	 * Filter formatted amount with currency
@@ -1097,16 +1097,16 @@ 
 	 *           filter name will be give_usd_currency_filter_after
 	 */
 	$formatted = apply_filters(
-		'give_' . strtolower( $args['currency_code'] ) . "_currency_filter_{$args['position']}",
+		'give_'.strtolower($args['currency_code'])."_currency_filter_{$args['position']}",
 		$formatted,
 		$args['currency_code'],
 		$price,
 		$args
 	);
 
-	if ( $negative ) {
+	if ($negative) {
 		// Prepend the minus sign before the currency sign.
-		$formatted = '-' . $formatted;
+		$formatted = '-'.$formatted;
 	}
 
 	return $formatted;
@@ -1123,7 +1123,7 @@ 
  *
  * @return bool
  */
-function give_is_zero_based_currency( $currency = '' ) {
+function give_is_zero_based_currency($currency = '') {
 	$zero_based_currency = array(
 		'PYG', // Paraguayan Guarani.
 		'GNF', // Guinean Franc.
@@ -1143,12 +1143,12 @@ 
 	);
 
 	// Set default currency.
-	if ( empty( $currency ) ) {
+	if (empty($currency)) {
 		$currency = give_get_currency();
 	}
 
 	// Check for Zero Based Currency.
-	if ( in_array( $currency, $zero_based_currency ) ) {
+	if (in_array($currency, $zero_based_currency)) {
 		return true;
 	}
 
@@ -1163,7 +1163,7 @@ 
  *
  * @return bool
  */
-function give_is_right_to_left_supported_currency( $currency = '' ) {
+function give_is_right_to_left_supported_currency($currency = '') {
 	$zero_based_currency = apply_filters(
 		'give_right_to_left_supported_currency',
 		array(
@@ -1179,12 +1179,12 @@ 
 	);
 
 	// Set default currency.
-	if ( empty( $currency ) ) {
+	if (empty($currency)) {
 		$currency = give_get_currency();
 	}
 
 	// Check for Zero Based Currency.
-	if ( in_array( $currency, $zero_based_currency ) ) {
+	if (in_array($currency, $zero_based_currency)) {
 		return true;
 	}