WordImpress / Give

includes/admin/shortcodes/shortcode-give-goal.php

Spacing +19 added lines, -19 removed lines

@@ -9,7 +9,7 @@ 
  * @since       1.3.0
  */
 
-defined( 'ABSPATH' ) or exit;
+defined('ABSPATH') or exit;
 
 class Give_Shortcode_Donation_Form_Goal extends Give_Shortcode_Generator {
 
@@ -18,10 +18,10 @@ 
 	 */
 	public function __construct() {
 
-		$this->shortcode['title'] = __( 'Donation Form Goal', 'give' );
-		$this->shortcode['label'] = __( 'Donation Form Goal', 'give' );
+		$this->shortcode['title'] = __('Donation Form Goal', 'give');
+		$this->shortcode['label'] = __('Donation Form Goal', 'give');
 
-		parent::__construct( 'give_goal' );
+		parent::__construct('give_goal');
 	}
 
 	/**
@@ -31,8 +31,8 @@ 
 	 */
 	public function define_fields() {
 
-		$create_form_link = sprintf( __( '%sClick here%s to create a new Donation Form.', 'give' ),
-			'<a href="' . admin_url( 'post-new.php?post_type=give_forms' ) . '">',
+		$create_form_link = sprintf(__('%sClick here%s to create a new Donation Form.', 'give'),
+			'<a href="'.admin_url('post-new.php?post_type=give_forms').'">',
 			'</a>'
 		);
 
@@ -43,35 +43,35 @@ 
 					'post_type' => 'give_forms',
 				),
 				'name'        => 'id',
-				'tooltip'     => __( 'Select a Donation Form', 'give' ),
-				'placeholder' => sprintf( '– %s –', __( 'Select a Form', 'give' ) ),
+				'tooltip'     => __('Select a Donation Form', 'give'),
+				'placeholder' => sprintf('– %s –', __('Select a Form', 'give')),
 				'required'    => array(
-					'alert' => __( 'You must first select a Form!', 'give' ),
-					'error' => sprintf( '<p class="strong">%s</p><p class="no-margin">%s</p>', __( 'No donation forms were found!', 'give' ), $create_form_link ),
+					'alert' => __('You must first select a Form!', 'give'),
+					'error' => sprintf('<p class="strong">%s</p><p class="no-margin">%s</p>', __('No donation forms were found!', 'give'), $create_form_link),
 				),
 			),
 			array(
 				'type' => 'container',
-				'html' => sprintf( '<p class="strong margin-top">%s</p>', __( 'Optional settings', 'give' ) ),
+				'html' => sprintf('<p class="strong margin-top">%s</p>', __('Optional settings', 'give')),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'show_text',
-				'label'   => __( 'Show Text:', 'give' ),
-				'tooltip' => __( 'This text displays the amount of income raised compared to the goal.', 'give' ),
+				'label'   => __('Show Text:', 'give'),
+				'tooltip' => __('This text displays the amount of income raised compared to the goal.', 'give'),
 				'options' => array(
-					'true'  => __( 'Show', 'give' ),
-					'false' => __( 'Hide', 'give' ),
+					'true'  => __('Show', 'give'),
+					'false' => __('Hide', 'give'),
 				),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'show_bar',
-				'label'   => __( 'Show Progress Bar:', 'give' ),
-				'tooltip' => __( 'Do you want to display the goal\'s progress bar?', 'give' ),
+				'label'   => __('Show Progress Bar:', 'give'),
+				'tooltip' => __('Do you want to display the goal\'s progress bar?', 'give'),
 				'options' => array(
-					'true'  => __( 'Show', 'give' ),
-					'false' => __( 'Hide', 'give' ),
+					'true'  => __('Show', 'give'),
+					'false' => __('Hide', 'give'),
 				),
 			),
 		);

includes/admin/shortcodes/shortcode-give-login.php

Spacing +6 added lines, -6 removed lines

@@ -9,7 +9,7 @@ 
  * @since       1.3.0
  */
 
-defined( 'ABSPATH' ) or exit;
+defined('ABSPATH') or exit;
 
 class Give_Shortcode_Login extends Give_Shortcode_Generator {
 
@@ -18,10 +18,10 @@ 
 	 */
 	public function __construct() {
 
-		$this->shortcode['title'] = __( 'Login', 'give' );
-		$this->shortcode['label'] = __( 'Login', 'give' );
+		$this->shortcode['title'] = __('Login', 'give');
+		$this->shortcode['label'] = __('Login', 'give');
 
-		parent::__construct( 'give_login' );
+		parent::__construct('give_login');
 	}
 
 	/**
@@ -34,13 +34,13 @@ 
 		return array(
 			array(
 				'type' => 'container',
-				'html' => sprintf( '<p class="no-margin">%s</p>', __( 'Redirect URL (optional):', 'give' ) ),
+				'html' => sprintf('<p class="no-margin">%s</p>', __('Redirect URL (optional):', 'give')),
 			),
 			array(
 				'type'     => 'textbox',
 				'name'     => 'redirect',
 				'minWidth' => 320,
-				'tooltip'  => __( 'Enter an URL here to redirect to after login.', 'give' ),
+				'tooltip'  => __('Enter an URL here to redirect to after login.', 'give'),
 			),
 		);
 	}

includes/admin/shortcodes/shortcode-give-profile-editor.php

Spacing +3 added lines, -3 removed lines

@@ -9,7 +9,7 @@ 
  * @since       1.3.0
  */
 
-defined( 'ABSPATH' ) or exit;
+defined('ABSPATH') or exit;
 
 class Give_Shortcode_Profile_Editor extends Give_Shortcode_Generator {
 
@@ -18,9 +18,9 @@ 
 	 */
 	public function __construct() {
 
-		$this->shortcode['label'] = __( 'Profile Editor', 'give' );
+		$this->shortcode['label'] = __('Profile Editor', 'give');
 
-		parent::__construct( 'give_profile_editor' );
+		parent::__construct('give_profile_editor');
 	}
 }
 

includes/admin/shortcodes/shortcode-give-receipt.php

Spacing +20 added lines, -20 removed lines

@@ -9,7 +9,7 @@ 
  * @since       1.3.0
  */
 
-defined( 'ABSPATH' ) or exit;
+defined('ABSPATH') or exit;
 
 class Give_Shortcode_Donation_Receipt extends Give_Shortcode_Generator {
 
@@ -18,10 +18,10 @@ 
 	 */
 	public function __construct() {
 
-		$this->shortcode['title'] = __( 'Donation Receipt', 'give' );
-		$this->shortcode['label'] = __( 'Donation Receipt', 'give' );
+		$this->shortcode['title'] = __('Donation Receipt', 'give');
+		$this->shortcode['label'] = __('Donation Receipt', 'give');
 
-		parent::__construct( 'give_receipt' );
+		parent::__construct('give_receipt');
 	}
 
 	/**
@@ -34,51 +34,51 @@ 
 		return array(
 			array(
 				'type' => 'container',
-				'html' => sprintf( '<p class="strong">%s</p>', __( 'Optional settings', 'give' ) ),
+				'html' => sprintf('<p class="strong">%s</p>', __('Optional settings', 'give')),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'price',
-				'label'   => __( 'Show Price:', 'give' ),
+				'label'   => __('Show Price:', 'give'),
 				'options' => array(
-					'true'  => __( 'Show', 'give' ),
-					'false' => __( 'Hide', 'give' ),
+					'true'  => __('Show', 'give'),
+					'false' => __('Hide', 'give'),
 				),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'date',
-				'label'   => __( 'Show Date:', 'give' ),
+				'label'   => __('Show Date:', 'give'),
 				'options' => array(
-					'true'  => __( 'Show', 'give' ),
-					'false' => __( 'Hide', 'give' ),
+					'true'  => __('Show', 'give'),
+					'false' => __('Hide', 'give'),
 				),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'payment_key',
-				'label'   => __( 'Show Payment Key:', 'give' ),
+				'label'   => __('Show Payment Key:', 'give'),
 				'options' => array(
-					'true'  => __( 'Show', 'give' ),
-					'false' => __( 'Hide', 'give' ),
+					'true'  => __('Show', 'give'),
+					'false' => __('Hide', 'give'),
 				),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'payment_method',
-				'label'   => __( 'Show Payment Method:', 'give' ),
+				'label'   => __('Show Payment Method:', 'give'),
 				'options' => array(
-					'true'  => __( 'Show', 'give' ),
-					'false' => __( 'Hide', 'give' ),
+					'true'  => __('Show', 'give'),
+					'false' => __('Hide', 'give'),
 				),
 			),
 			array(
 				'type'    => 'listbox',
 				'name'    => 'payment_id',
-				'label'   => __( 'Show Payment ID:', 'give' ),
+				'label'   => __('Show Payment ID:', 'give'),
 				'options' => array(
-					'true'  => __( 'Show', 'give' ),
-					'false' => __( 'Hide', 'give' ),
+					'true'  => __('Show', 'give'),
+					'false' => __('Hide', 'give'),
 				),
 			),
 		);

includes/admin/shortcodes/shortcode-give-register.php

Spacing +6 added lines, -6 removed lines

@@ -9,7 +9,7 @@ 
  * @since       1.3.0
  */
 
-defined( 'ABSPATH' ) or exit;
+defined('ABSPATH') or exit;
 
 class Give_Shortcode_Register extends Give_Shortcode_Generator {
 
@@ -18,10 +18,10 @@ 
 	 */
 	public function __construct() {
 
-		$this->shortcode['title'] = __( 'Register', 'give' );
-		$this->shortcode['label'] = __( 'Register', 'give' );
+		$this->shortcode['title'] = __('Register', 'give');
+		$this->shortcode['label'] = __('Register', 'give');
 
-		parent::__construct( 'give_register' );
+		parent::__construct('give_register');
 	}
 
 	/**
@@ -34,13 +34,13 @@ 
 		return array(
 			array(
 				'type' => 'container',
-				'html' => sprintf( '<p class="no-margin">%s</p>', __( 'Redirect URL (optional):', 'give' ) ),
+				'html' => sprintf('<p class="no-margin">%s</p>', __('Redirect URL (optional):', 'give')),
 			),
 			array(
 				'type'     => 'textbox',
 				'name'     => 'redirect',
 				'minWidth' => 320,
-				'tooltip'  => __( 'Enter an URL here to redirect to after registering.', 'give' ),
+				'tooltip'  => __('Enter an URL here to redirect to after registering.', 'give'),
 			),
 		);
 	}

includes/admin/upgrades/upgrades.php

Spacing +17 added lines, -17 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,12 +21,12 @@ 
  * @return void
  */
 function give_upgrades_screen() {
-	$action = isset( $_GET['give-upgrade'] ) ? sanitize_text_field( $_GET['give-upgrade'] ) : '';
-	$step   = isset( $_GET['step'] ) ? absint( $_GET['step'] ) : 1;
-	$total  = isset( $_GET['total'] ) ? absint( $_GET['total'] ) : false;
-	$custom = isset( $_GET['custom'] ) ? absint( $_GET['custom'] ) : 0;
-	$number = isset( $_GET['number'] ) ? absint( $_GET['number'] ) : 100;
-	$steps  = round( ( $total / $number ), 0 );
+	$action = isset($_GET['give-upgrade']) ? sanitize_text_field($_GET['give-upgrade']) : '';
+	$step   = isset($_GET['step']) ? absint($_GET['step']) : 1;
+	$total  = isset($_GET['total']) ? absint($_GET['total']) : false;
+	$custom = isset($_GET['custom']) ? absint($_GET['custom']) : 0;
+	$number = isset($_GET['number']) ? absint($_GET['number']) : 100;
+	$steps  = round(($total / $number), 0);
 
 	$doing_upgrade_args = array(
 		'page'         => 'give-upgrades',
@@ -36,25 +36,25 @@ 
 		'custom'       => $custom,
 		'steps'        => $steps
 	);
-	update_option( 'give_doing_upgrade', $doing_upgrade_args );
-	if ( $step > $steps ) {
+	update_option('give_doing_upgrade', $doing_upgrade_args);
+	if ($step > $steps) {
 		// Prevent a weird case where the estimate was off. Usually only a couple.
 		$steps = $step;
 	}
 	?>
 	<div class="wrap">
-		<h2><?php _e( 'Give - Upgrades', 'give' ); ?></h2>
+		<h2><?php _e('Give - Upgrades', 'give'); ?></h2>
 
-		<?php if ( ! empty( $action ) ) : ?>
+		<?php if ( ! empty($action)) : ?>
 
 			<div id="give-upgrade-status">
-				<p style="font-size: 20px;max-width: 900px;"><?php _e( 'The upgrade process has started, please be patient and do not close this window or navigate away from this page. This could take several minutes depending on the upgrade and the size of your website. You will be automatically redirected when the upgrade is finished.', 'give' ); ?>
-					<img src="<?php echo GIVE_PLUGIN_URL . '/assets/images/spinner.gif'; ?>" id="give-upgrade-loader" style="  position: relative; top: 3px; left: 6px;" />
+				<p style="font-size: 20px;max-width: 900px;"><?php _e('The upgrade process has started, please be patient and do not close this window or navigate away from this page. This could take several minutes depending on the upgrade and the size of your website. You will be automatically redirected when the upgrade is finished.', 'give'); ?>
+					<img src="<?php echo GIVE_PLUGIN_URL.'/assets/images/spinner.gif'; ?>" id="give-upgrade-loader" style="  position: relative; top: 3px; left: 6px;" />
 				</p>
 
-				<?php if ( ! empty( $total ) ) : ?>
+				<?php if ( ! empty($total)) : ?>
 					<p>
-						<strong><?php printf( __( 'Step %d of approximately %d running', 'give' ), $step, $steps ); ?></strong>
+						<strong><?php printf(__('Step %d of approximately %d running', 'give'), $step, $steps); ?></strong>
 					</p>
 				<?php endif; ?>
 			</div>
@@ -68,8 +68,8 @@ 
 
 			<div id="give-upgrade-status">
 				<p style="font-size: 20px;max-width: 900px;">
-					<?php _e( 'The upgrade process has started, please be patient and do not close this window or navigate away from this page. This could take several minutes depending on the upgrade and the size of your website. You will be automatically redirected when the upgrade is finished.', 'give' ); ?>
-					<img src="<?php echo GIVE_PLUGIN_URL . '/assets/images/spinner.gif'; ?>" id="give-upgrade-loader" style="  position: relative; top: 3px; left: 6px;" />
+					<?php _e('The upgrade process has started, please be patient and do not close this window or navigate away from this page. This could take several minutes depending on the upgrade and the size of your website. You will be automatically redirected when the upgrade is finished.', 'give'); ?>
+					<img src="<?php echo GIVE_PLUGIN_URL.'/assets/images/spinner.gif'; ?>" id="give-upgrade-loader" style="  position: relative; top: 3px; left: 6px;" />
 				</p>
 			</div>
 			<script type="text/javascript">

Braces +5 added lines, -2 removed lines

@@ -64,11 +64,14 @@
 				}, 250 );
 			</script>
 
-		<?php else : ?>
+		<?php else {
+	: ?>
 
 			<div id="give-upgrade-status">
 				<p style="font-size: 20px;max-width: 900px;">
-					<?php _e( 'The upgrade process has started, please be patient and do not close this window or navigate away from this page. This could take several minutes depending on the upgrade and the size of your website. You will be automatically redirected when the upgrade is finished.', 'give' ); ?>
+					<?php _e( 'The upgrade process has started, please be patient and do not close this window or navigate away from this page. This could take several minutes depending on the upgrade and the size of your website. You will be automatically redirected when the upgrade is finished.', 'give' );
+}
+?>
 					<img src="<?php echo GIVE_PLUGIN_URL . '/assets/images/spinner.gif'; ?>" id="give-upgrade-loader" style="  position: relative; top: 3px; left: 6px;" />
 				</p>
 			</div>

includes/ajax-functions.php

Spacing +66 added lines, -66 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,27 +25,27 @@ 
 function give_test_ajax_works() {
 
 	// Check if the Airplane Mode plugin is installed
-	if ( class_exists( 'Airplane_Mode_Core' ) ) {
+	if (class_exists('Airplane_Mode_Core')) {
 
 		$airplane = Airplane_Mode_Core::getInstance();
 
-		if ( method_exists( $airplane, 'enabled' ) ) {
+		if (method_exists($airplane, 'enabled')) {
 
-			if ( $airplane->enabled() ) {
+			if ($airplane->enabled()) {
 				return true;
 			}
 
 		} else {
 
-			if ( $airplane->check_status() == 'on' ) {
+			if ($airplane->check_status() == 'on') {
 				return true;
 			}
 		}
 	}
 
-	add_filter( 'block_local_requests', '__return_false' );
+	add_filter('block_local_requests', '__return_false');
 
-	if ( get_transient( '_give_ajax_works' ) ) {
+	if (get_transient('_give_ajax_works')) {
 		return true;
 	}
 
@@ -57,35 +57,35 @@ 
 		)
 	);
 
-	$ajax  = wp_remote_post( give_get_ajax_url(), $params );
+	$ajax  = wp_remote_post(give_get_ajax_url(), $params);
 	$works = true;
 
-	if ( is_wp_error( $ajax ) ) {
+	if (is_wp_error($ajax)) {
 
 		$works = false;
 
 	} else {
 
-		if( empty( $ajax['response'] ) ) {
+		if (empty($ajax['response'])) {
 			$works = false;
 		}
 
-		if( empty( $ajax['response']['code'] ) || 200 !== (int) $ajax['response']['code'] ) {
+		if (empty($ajax['response']['code']) || 200 !== (int) $ajax['response']['code']) {
 			$works = false;
 		}
 
-		if( empty( $ajax['response']['message'] ) || 'OK' !== $ajax['response']['message'] ) {
+		if (empty($ajax['response']['message']) || 'OK' !== $ajax['response']['message']) {
 			$works = false;
 		}
 
-		if( ! isset( $ajax['body'] ) || 0 !== (int) $ajax['body'] ) {
+		if ( ! isset($ajax['body']) || 0 !== (int) $ajax['body']) {
 			$works = false;
 		}
 
 	}
 
-	if ( $works ) {
-		set_transient( '_give_ajax_works', '1', DAY_IN_SECONDS );
+	if ($works) {
+		set_transient('_give_ajax_works', '1', DAY_IN_SECONDS);
 	}
 
 	return $works;
@@ -99,16 +99,16 @@ 
  * @return string
  */
 function give_get_ajax_url() {
-	$scheme = defined( 'FORCE_SSL_ADMIN' ) && FORCE_SSL_ADMIN ? 'https' : 'admin';
+	$scheme = defined('FORCE_SSL_ADMIN') && FORCE_SSL_ADMIN ? 'https' : 'admin';
 
 	$current_url = give_get_current_page_url();
-	$ajax_url    = admin_url( 'admin-ajax.php', $scheme );
+	$ajax_url    = admin_url('admin-ajax.php', $scheme);
 
-	if ( preg_match( '/^https/', $current_url ) && ! preg_match( '/^https/', $ajax_url ) ) {
-		$ajax_url = preg_replace( '/^http/', 'https', $ajax_url );
+	if (preg_match('/^https/', $current_url) && ! preg_match('/^https/', $ajax_url)) {
+		$ajax_url = preg_replace('/^http/', 'https', $ajax_url);
 	}
 
-	return apply_filters( 'give_ajax_url', $ajax_url );
+	return apply_filters('give_ajax_url', $ajax_url);
 }
 
 /**
@@ -118,11 +118,11 @@ 
  * @return void
  */
 function give_load_checkout_login_fields() {
-	do_action( 'give_purchase_form_login_fields' );
+	do_action('give_purchase_form_login_fields');
 	give_die();
 }
 
-add_action( 'wp_ajax_nopriv_give_checkout_login', 'give_load_checkout_login_fields' );
+add_action('wp_ajax_nopriv_give_checkout_login', 'give_load_checkout_login_fields');
 
 /**
  * Load Checkout Register Fields via AJAX
@@ -131,11 +131,11 @@ 
  * @return void
  */
 function give_load_checkout_register_fields() {
-	do_action( 'give_purchase_form_register_fields' );
+	do_action('give_purchase_form_register_fields');
 	give_die();
 }
 
-add_action( 'wp_ajax_nopriv_checkout_register', 'give_load_checkout_register_fields' );
+add_action('wp_ajax_nopriv_checkout_register', 'give_load_checkout_register_fields');
 
 /**
  * Get Form Title via AJAX (used only in WordPress Admin)
@@ -144,9 +144,9 @@ 
  * @return void
  */
 function give_ajax_get_form_title() {
-	if ( isset( $_POST['form_id'] ) ) {
-		$title = get_the_title( $_POST['form_id'] );
-		if ( $title ) {
+	if (isset($_POST['form_id'])) {
+		$title = get_the_title($_POST['form_id']);
+		if ($title) {
 			echo $title;
 		} else {
 			echo 'fail';
@@ -155,8 +155,8 @@ 
 	give_die();
 }
 
-add_action( 'wp_ajax_give_get_form_title', 'give_ajax_get_form_title' );
-add_action( 'wp_ajax_nopriv_give_get_form_title', 'give_ajax_get_form_title' );
+add_action('wp_ajax_give_get_form_title', 'give_ajax_get_form_title');
+add_action('wp_ajax_nopriv_give_get_form_title', 'give_ajax_get_form_title');
 
 /**
  * Retrieve a states drop down
@@ -166,23 +166,23 @@ 
  */
 function give_ajax_get_states_field() {
 
-	if ( empty( $_POST['country'] ) ) {
+	if (empty($_POST['country'])) {
 		$_POST['country'] = give_get_country();
 	}
-	$states = give_get_states( $_POST['country'] );
+	$states = give_get_states($_POST['country']);
 
-	if ( ! empty( $states ) ) {
+	if ( ! empty($states)) {
 
 		$args = array(
 			'name'             => $_POST['field_name'],
 			'id'               => $_POST['field_name'],
-			'class'            => $_POST['field_name'] . '  give-select',
-			'options'          => give_get_states( $_POST['country'] ),
+			'class'            => $_POST['field_name'].'  give-select',
+			'options'          => give_get_states($_POST['country']),
 			'show_option_all'  => false,
 			'show_option_none' => false
 		);
 
-		$response = Give()->html->select( $args );
+		$response = Give()->html->select($args);
 
 	} else {
 
@@ -194,8 +194,8 @@ 
 	give_die();
 }
 
-add_action( 'wp_ajax_give_get_states', 'give_ajax_get_states_field' );
-add_action( 'wp_ajax_nopriv_give_get_states', 'give_ajax_get_states_field' );
+add_action('wp_ajax_give_get_states', 'give_ajax_get_states_field');
+add_action('wp_ajax_nopriv_give_get_states', 'give_ajax_get_states_field');
 
 /**
  * Retrieve a states drop down
@@ -206,17 +206,17 @@ 
 function give_ajax_form_search() {
 	global $wpdb;
 
-	$search  = esc_sql( sanitize_text_field( $_GET['s'] ) );
+	$search  = esc_sql(sanitize_text_field($_GET['s']));
 	$results = array();
-	if ( current_user_can( 'edit_give_forms' ) ) {
-		$items = $wpdb->get_results( "SELECT ID,post_title FROM $wpdb->posts WHERE `post_type` = 'give_forms' AND `post_title` LIKE '%$search%' LIMIT 50" );
+	if (current_user_can('edit_give_forms')) {
+		$items = $wpdb->get_results("SELECT ID,post_title FROM $wpdb->posts WHERE `post_type` = 'give_forms' AND `post_title` LIKE '%$search%' LIMIT 50");
 	} else {
-		$items = $wpdb->get_results( "SELECT ID,post_title FROM $wpdb->posts WHERE `post_type` = 'give_forms' AND `post_status` = 'publish' AND `post_title` LIKE '%$search%' LIMIT 50" );
+		$items = $wpdb->get_results("SELECT ID,post_title FROM $wpdb->posts WHERE `post_type` = 'give_forms' AND `post_status` = 'publish' AND `post_title` LIKE '%$search%' LIMIT 50");
 	}
 
-	if ( $items ) {
+	if ($items) {
 
-		foreach ( $items as $item ) {
+		foreach ($items as $item) {
 
 			$results[] = array(
 				'id'   => $item->ID,
@@ -228,18 +228,18 @@ 
 
 		$items[] = array(
 			'id'   => 0,
-			'name' => __( 'No results found', 'give' )
+			'name' => __('No results found', 'give')
 		);
 
 	}
 
-	echo json_encode( $results );
+	echo json_encode($results);
 
 	give_die();
 }
 
-add_action( 'wp_ajax_give_form_search', 'give_ajax_form_search' );
-add_action( 'wp_ajax_nopriv_give_form_search', 'give_ajax_form_search' );
+add_action('wp_ajax_give_form_search', 'give_ajax_form_search');
+add_action('wp_ajax_nopriv_give_form_search', 'give_ajax_form_search');
 
 /**
  * Search the donors database via Ajax
@@ -250,21 +250,21 @@ 
 function give_ajax_donor_search() {
 	global $wpdb;
 
-	$search  = esc_sql( sanitize_text_field( $_GET['s'] ) );
+	$search  = esc_sql(sanitize_text_field($_GET['s']));
 	$results = array();
-	if ( ! current_user_can( 'view_give_reports' ) ) {
+	if ( ! current_user_can('view_give_reports')) {
 		$donors = array();
 	} else {
-		$donors = $wpdb->get_results( "SELECT id,name,email FROM {$wpdb->prefix}give_donors WHERE `name` LIKE '%$search%' OR `email` LIKE '%$search%' LIMIT 50" );
+		$donors = $wpdb->get_results("SELECT id,name,email FROM {$wpdb->prefix}give_donors WHERE `name` LIKE '%$search%' OR `email` LIKE '%$search%' LIMIT 50");
 	}
 
-	if ( $donors ) {
+	if ($donors) {
 
-		foreach ( $donors as $donor ) {
+		foreach ($donors as $donor) {
 
 			$results[] = array(
 				'id'   => $donor->id,
-				'name' => $donor->name . '(' . $donor->email . ')'
+				'name' => $donor->name.'('.$donor->email.')'
 			);
 		}
 
@@ -272,17 +272,17 @@ 
 
 		$donors[] = array(
 			'id'   => 0,
-			'name' => __( 'No results found', 'give' )
+			'name' => __('No results found', 'give')
 		);
 
 	}
 
-	echo json_encode( $results );
+	echo json_encode($results);
 
 	give_die();
 }
 
-add_action( 'wp_ajax_give_donor_search', 'give_ajax_donor_search' );
+add_action('wp_ajax_give_donor_search', 'give_ajax_donor_search');
 
 
 /**
@@ -293,30 +293,30 @@ 
  */
 function give_ajax_search_users() {
 
-	if ( current_user_can( 'manage_give_settings' ) ) {
+	if (current_user_can('manage_give_settings')) {
 
-		$search_query = trim( $_POST['user_name'] );
+		$search_query = trim($_POST['user_name']);
 
-		$found_users = get_users( array(
+		$found_users = get_users(array(
 				'number' => 9999,
-				'search' => $search_query . '*'
+				'search' => $search_query.'*'
 			)
 		);
 
 		$user_list = '<ul>';
-		if ( $found_users ) {
-			foreach ( $found_users as $user ) {
-				$user_list .= '<li><a href="#" data-login="' . esc_attr( $user->user_login ) . '">' . esc_html( $user->user_login ) . '</a></li>';
+		if ($found_users) {
+			foreach ($found_users as $user) {
+				$user_list .= '<li><a href="#" data-login="'.esc_attr($user->user_login).'">'.esc_html($user->user_login).'</a></li>';
 			}
 		} else {
-			$user_list .= '<li>' . __( 'No users found', 'give' ) . '</li>';
+			$user_list .= '<li>'.__('No users found', 'give').'</li>';
 		}
 		$user_list .= '</ul>';
 
-		echo json_encode( array( 'results' => $user_list ) );
+		echo json_encode(array('results' => $user_list));
 
 	}
 	die();
 }
 
-add_action( 'wp_ajax_give_search_users', 'give_ajax_search_users' );
\ No newline at end of file
+add_action('wp_ajax_give_search_users', 'give_ajax_search_users');
\ No newline at end of file

includes/api/class-give-api-v1.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) exit;
+if ( ! defined('ABSPATH')) exit;
 
 /**
  * Give_API_V1 Class

Braces +3 added lines, -1 removed lines

@@ -10,7 +10,9 @@
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) exit;
+if ( ! defined( 'ABSPATH' ) ) {
+	exit;
+}
 
 /**
  * Give_API_V1 Class

includes/class-give-customer.php

Spacing +126 added lines, -126 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -96,29 +96,29 @@ 
 	 *
 	 * @since 1.0
 	 */
-	public function __construct( $_id_or_email = false, $by_user_id = false ) {
+	public function __construct($_id_or_email = false, $by_user_id = false) {
 
 		$this->db = new Give_DB_Customers;
 
-		if ( false === $_id_or_email || ( is_numeric( $_id_or_email ) && (int) $_id_or_email !== absint( $_id_or_email ) ) ) {
+		if (false === $_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';
 		}
 
-		$customer = $this->db->get_customer_by( $field, $_id_or_email );
+		$customer = $this->db->get_customer_by($field, $_id_or_email);
 
-		if ( empty( $customer ) || ! is_object( $customer ) ) {
+		if (empty($customer) || ! is_object($customer)) {
 			return false;
 		}
 
-		$this->setup_customer( $customer );
+		$this->setup_customer($customer);
 
 	}
 
@@ -131,15 +131,15 @@ 
 	 *
 	 * @return bool             If the setup was successful or not
 	 */
-	private function setup_customer( $customer ) {
+	private function setup_customer($customer) {
 
-		if ( ! is_object( $customer ) ) {
+		if ( ! is_object($customer)) {
 			return false;
 		}
 
-		foreach ( $customer as $key => $value ) {
+		foreach ($customer as $key => $value) {
 
-			switch ( $key ) {
+			switch ($key) {
 
 				case 'notes':
 					$this->$key = $this->get_notes();
@@ -154,7 +154,7 @@ 
 		}
 
 		// Customer 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;
 		}
 
@@ -167,15 +167,15 @@ 
 	 *
 	 * @since 1.0
 	 */
-	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 {
 
-			return new WP_Error( 'give-customer-invalid-property', sprintf( __( 'Can\'t get property %s', 'give' ), $key ) );
+			return new WP_Error('give-customer-invalid-property', sprintf(__('Can\'t get property %s', 'give'), $key));
 
 		}
 
@@ -190,9 +190,9 @@ 
 	 *
 	 * @return mixed        False if not a valid creation, Customer 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;
 		}
 
@@ -200,34 +200,34 @@ 
 			'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'])));
 		}
 
-		do_action( 'give_customer_pre_create', $args );
+		do_action('give_customer_pre_create', $args);
 
 		$created = false;
 
 		// The DB class 'add' implies an update if the customer being asked to be created already exists
-		if ( $this->db->add( $data ) ) {
+		if ($this->db->add($data)) {
 
 			// We've successfully added/updated the customer, reset the class vars with the new data
-			$customer = $this->db->get_customer_by( 'email', $args['email'] );
+			$customer = $this->db->get_customer_by('email', $args['email']);
 
 			// Setup the customer data with the values from DB
-			$this->setup_customer( $customer );
+			$this->setup_customer($customer);
 
 			$created = $this->id;
 		}
 
-		do_action( 'give_customer_post_create', $created, $args );
+		do_action('give_customer_post_create', $created, $args);
 
 		return $created;
 
@@ -242,27 +242,27 @@ 
 	 *
 	 * @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);
 
-		do_action( 'give_customer_pre_update', $this->id, $data );
+		do_action('give_customer_pre_update', $this->id, $data);
 
 		$updated = false;
 
-		if ( $this->db->update( $this->id, $data ) ) {
+		if ($this->db->update($this->id, $data)) {
 
-			$customer = $this->db->get_customer_by( 'id', $this->id );
-			$this->setup_customer( $customer );
+			$customer = $this->db->get_customer_by('id', $this->id);
+			$this->setup_customer($customer);
 
 			$updated = true;
 		}
 
-		do_action( 'give_customer_post_update', $updated, $this->id, $data );
+		do_action('give_customer_post_update', $updated, $this->id, $data);
 
 		return $updated;
 	}
@@ -278,44 +278,44 @@ 
 	 *
 	 * @return bool            If the attachment was successfuly
 	 */
-	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)));
 
 		}
 
-		do_action( 'give_customer_pre_attach_payment', $payment_id, $this->id );
+		do_action('give_customer_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_get_payment_amount( $payment_id );
+			if ($update_stats) {
+				$payment_amount = give_get_payment_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();
@@ -323,7 +323,7 @@ 
 
 		}
 
-		do_action( 'give_customer_post_attach_payment', $payment_added, $payment_id, $this->id );
+		do_action('give_customer_post_attach_payment', $payment_added, $payment_id, $this->id);
 
 		return $payment_added;
 	}
@@ -339,44 +339,44 @@ 
 	 *
 	 * @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;
 		}
 
 		$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)));
 
 		}
 
-		do_action( 'give_customer_pre_remove_payment', $payment_id, $this->id );
+		do_action('give_customer_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_get_payment_amount( $payment_id );
+				$payment_amount = give_get_payment_amount($payment_id);
 
-				if ( ! empty( $payment_amount ) ) {
-					$this->decrease_value( $payment_amount );
+				if ( ! empty($payment_amount)) {
+					$this->decrease_value($payment_amount);
 				}
 
 				$this->decrease_purchase_count();
@@ -384,7 +384,7 @@ 
 
 		}
 
-		do_action( 'give_customer_post_remove_payment', $payment_removed, $payment_id, $this->id );
+		do_action('give_customer_post_remove_payment', $payment_removed, $payment_id, $this->id);
 
 		return $payment_removed;
 
@@ -399,22 +399,22 @@ 
 	 *
 	 * @return int            The purchase 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;
 		}
 
 		$new_total = (int) $this->purchase_count + (int) $count;
 
-		do_action( 'give_customer_pre_increase_purchase_count', $count, $this->id );
+		do_action('give_customer_pre_increase_purchase_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;
 		}
 
-		do_action( 'give_customer_post_increase_purchase_count', $this->purchase_count, $count, $this->id );
+		do_action('give_customer_post_increase_purchase_count', $this->purchase_count, $count, $this->id);
 
 		return $this->purchase_count;
 	}
@@ -428,26 +428,26 @@ 
 	 *
 	 * @return mixed          If successful, the new count, otherwise false
 	 */
-	public function decrease_purchase_count( $count = 1 ) {
+	public function decrease_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;
 		}
 
 		$new_total = (int) $this->purchase_count - (int) $count;
 
-		if ( $new_total < 0 ) {
+		if ($new_total < 0) {
 			$new_total = 0;
 		}
 
-		do_action( 'give_customer_pre_decrease_purchase_count', $count, $this->id );
+		do_action('give_customer_pre_decrease_purchase_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;
 		}
 
-		do_action( 'give_customer_post_decrease_purchase_count', $this->purchase_count, $count, $this->id );
+		do_action('give_customer_post_decrease_purchase_count', $this->purchase_count, $count, $this->id);
 
 		return $this->purchase_count;
 	}
@@ -461,17 +461,17 @@ 
 	 *
 	 * @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;
 
-		do_action( 'give_customer_pre_increase_value', $value, $this->id );
+		do_action('give_customer_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;
 		}
 
-		do_action( 'give_customer_post_increase_value', $this->purchase_value, $value, $this->id );
+		do_action('give_customer_post_increase_value', $this->purchase_value, $value, $this->id);
 
 		return $this->purchase_value;
 	}
@@ -485,21 +485,21 @@ 
 	 *
 	 * @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;
 		}
 
-		do_action( 'give_customer_pre_decrease_value', $value, $this->id );
+		do_action('give_customer_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;
 		}
 
-		do_action( 'give_customer_post_decrease_value', $this->purchase_value, $value, $this->id );
+		do_action('give_customer_post_decrease_value', $this->purchase_value, $value, $this->id);
 
 		return $this->purchase_value;
 	}
@@ -514,15 +514,15 @@ 
 	 *
 	 * @return array           The notes requsted
 	 */
-	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;
 
@@ -537,9 +537,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);
 
 	}
 
@@ -552,32 +552,32 @@ 
 	 *
 	 * @return string|boolean The new note if added succesfully, 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;
 
-		do_action( 'give_customer_pre_add_note', $new_note, $this->id );
+		do_action('give_customer_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();
 		}
 
-		do_action( 'give_customer_post_add_note', $this->notes, $new_note, $this->id );
+		do_action('give_customer_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;
@@ -592,7 +592,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;
 
@@ -607,51 +607,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;
 
 			}