WordImpress / Give

includes/class-give-session.php

Spacing +76 added lines, -76 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -74,57 +74,57 @@ 
 	public function __construct() {
 
 		$this->use_php_sessions = $this->use_php_sessions();
-		$this->exp_option       = give_get_option( 'session_lifetime' );
+		$this->exp_option       = give_get_option('session_lifetime');
 
 		// PHP Sessions.
-		if ( $this->use_php_sessions ) {
+		if ($this->use_php_sessions) {
 
-			if ( is_multisite() ) {
+			if (is_multisite()) {
 
-				$this->prefix = '_' . get_current_blog_id();
+				$this->prefix = '_'.get_current_blog_id();
 
 			}
 
-			add_action( 'init', array( $this, 'maybe_start_session' ), - 2 );
+			add_action('init', array($this, 'maybe_start_session'), - 2);
 
 		} else {
 
-			if ( ! $this->should_start_session() ) {
+			if ( ! $this->should_start_session()) {
 				return;
 			}
 
 			// Use WP_Session.
-			if ( ! defined( 'WP_SESSION_COOKIE' ) ) {
-				define( 'WP_SESSION_COOKIE', 'give_wp_session' );
+			if ( ! defined('WP_SESSION_COOKIE')) {
+				define('WP_SESSION_COOKIE', 'give_wp_session');
 			}
 
-			if ( ! class_exists( 'Recursive_ArrayAccess' ) ) {
-				require_once GIVE_PLUGIN_DIR . 'includes/libraries/sessions/class-recursive-arrayaccess.php';
+			if ( ! class_exists('Recursive_ArrayAccess')) {
+				require_once GIVE_PLUGIN_DIR.'includes/libraries/sessions/class-recursive-arrayaccess.php';
 			}
 
 			// Include utilities class
-			if ( ! class_exists( 'WP_Session_Utils' ) ) {
-				require_once GIVE_PLUGIN_DIR . 'includes/libraries/sessions/class-wp-session-utils.php';
+			if ( ! class_exists('WP_Session_Utils')) {
+				require_once GIVE_PLUGIN_DIR.'includes/libraries/sessions/class-wp-session-utils.php';
 			}
-			if ( ! class_exists( 'WP_Session' ) ) {
-				require_once GIVE_PLUGIN_DIR . 'includes/libraries/sessions/class-wp-session.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/libraries/sessions/wp-session.php';
+			if ( ! class_exists('WP_Session')) {
+				require_once GIVE_PLUGIN_DIR.'includes/libraries/sessions/class-wp-session.php';
+				require_once GIVE_PLUGIN_DIR.'includes/libraries/sessions/wp-session.php';
 			}
 
-			add_filter( 'wp_session_expiration_variant', array( $this, 'set_expiration_variant_time' ), 99999 );
-			add_filter( 'wp_session_expiration', array( $this, 'set_expiration_time' ), 99999 );
+			add_filter('wp_session_expiration_variant', array($this, 'set_expiration_variant_time'), 99999);
+			add_filter('wp_session_expiration', array($this, 'set_expiration_time'), 99999);
 
 		}
 
 		// Init Session.
-		if ( empty( $this->session ) && ! $this->use_php_sessions ) {
-			add_action( 'plugins_loaded', array( $this, 'init' ), 9999 );
+		if (empty($this->session) && ! $this->use_php_sessions) {
+			add_action('plugins_loaded', array($this, 'init'), 9999);
 		} else {
-			add_action( 'init', array( $this, 'init' ), - 1 );
+			add_action('init', array($this, 'init'), - 1);
 		}
 
 		// Set cookie on Donation Completion page.
-		add_action( 'give_pre_process_donation', array( $this, 'set_session_cookies' ) );
+		add_action('give_pre_process_donation', array($this, 'set_session_cookies'));
 
 	}
 
@@ -140,8 +140,8 @@ 
 	 */
 	public function init() {
 
-		if ( $this->use_php_sessions ) {
-			$this->session = isset( $_SESSION[ 'give' . $this->prefix ] ) && is_array( $_SESSION[ 'give' . $this->prefix ] ) ? $_SESSION[ 'give' . $this->prefix ] : array();
+		if ($this->use_php_sessions) {
+			$this->session = isset($_SESSION['give'.$this->prefix]) && is_array($_SESSION['give'.$this->prefix]) ? $_SESSION['give'.$this->prefix] : array();
 		} else {
 			$this->session = WP_Session::get_instance();
 		}
@@ -176,37 +176,37 @@ 
 	 *
 	 * @return string|array      Session variable.
 	 */
-	public function get( $key ) {
-		$key    = sanitize_key( $key );
+	public function get($key) {
+		$key    = sanitize_key($key);
 		$return = false;
 
-		if ( isset( $this->session[ $key ] ) && ! empty( $this->session[ $key ] ) ) {
+		if (isset($this->session[$key]) && ! empty($this->session[$key])) {
 
-			preg_match( '/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $this->session[ $key ], $matches );
-			if ( ! empty( $matches ) ) {
-				$this->set( $key, null );
+			preg_match('/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $this->session[$key], $matches);
+			if ( ! empty($matches)) {
+				$this->set($key, null);
 
 				return false;
 			}
 
-			if ( is_numeric( $this->session[ $key ] ) ) {
-				$return = $this->session[ $key ];
+			if (is_numeric($this->session[$key])) {
+				$return = $this->session[$key];
 			} else {
 
-				$maybe_json = json_decode( $this->session[ $key ] );
+				$maybe_json = json_decode($this->session[$key]);
 
 				// Since json_last_error is PHP 5.3+, we have to rely on a `null` value for failing to parse JSON.
-				if ( is_null( $maybe_json ) ) {
-					$is_serialized = is_serialized( $this->session[ $key ] );
-					if ( $is_serialized ) {
-						$value = @unserialize( $this->session[ $key ] );
-						$this->set( $key, (array) $value );
+				if (is_null($maybe_json)) {
+					$is_serialized = is_serialized($this->session[$key]);
+					if ($is_serialized) {
+						$value = @unserialize($this->session[$key]);
+						$this->set($key, (array) $value);
 						$return = $value;
 					} else {
-						$return = $this->session[ $key ];
+						$return = $this->session[$key];
 					}
 				} else {
-					$return = json_decode( $this->session[ $key ], true );
+					$return = json_decode($this->session[$key], true);
 				}
 
 			}
@@ -229,21 +229,21 @@ 
 	 *
 	 * @return string        Session variable.
 	 */
-	public function set( $key, $value ) {
+	public function set($key, $value) {
 
-		$key = sanitize_key( $key );
+		$key = sanitize_key($key);
 
-		if ( is_array( $value ) ) {
-			$this->session[ $key ] = wp_json_encode( $value );
+		if (is_array($value)) {
+			$this->session[$key] = wp_json_encode($value);
 		} else {
-			$this->session[ $key ] = esc_attr( $value );
+			$this->session[$key] = esc_attr($value);
 		}
 
-		if ( $this->use_php_sessions ) {
-			$_SESSION[ 'give' . $this->prefix ] = $this->session;
+		if ($this->use_php_sessions) {
+			$_SESSION['give'.$this->prefix] = $this->session;
 		}
 
-		return $this->session[ $key ];
+		return $this->session[$key];
 	}
 
 	/**
@@ -258,10 +258,10 @@ 
 	 * @hook
 	 */
 	public function set_session_cookies() {
-		if ( ! headers_sent() ) {
-			$lifetime = current_time( 'timestamp' ) + $this->set_expiration_time();
-			@setcookie( session_name(), session_id(), $lifetime, COOKIEPATH, COOKIE_DOMAIN, false );
-			@setcookie( session_name() . '_expiration', $lifetime, $lifetime, COOKIEPATH, COOKIE_DOMAIN, false );
+		if ( ! headers_sent()) {
+			$lifetime = current_time('timestamp') + $this->set_expiration_time();
+			@setcookie(session_name(), session_id(), $lifetime, COOKIEPATH, COOKIE_DOMAIN, false);
+			@setcookie(session_name().'_expiration', $lifetime, $lifetime, COOKIEPATH, COOKIE_DOMAIN, false);
 		}
 	}
 
@@ -277,7 +277,7 @@ 
 	 */
 	public function set_expiration_variant_time() {
 
-		return ( ! empty( $this->exp_option ) ? ( intval( $this->exp_option ) - 3600 ) : 30 * 60 * 23 );
+		return ( ! empty($this->exp_option) ? (intval($this->exp_option) - 3600) : 30 * 60 * 23);
 	}
 
 	/**
@@ -292,7 +292,7 @@ 
 	 */
 	public function set_expiration_time() {
 
-		return ( ! empty( $this->exp_option ) ? intval( $this->exp_option ) : 30 * 60 * 24 );
+		return ( ! empty($this->exp_option) ? intval($this->exp_option) : 30 * 60 * 24);
 	}
 
 	/**
@@ -310,21 +310,21 @@ 
 		$ret = false;
 
 		// If the database variable is already set, no need to run auto detection.
-		$give_use_php_sessions = (bool) get_option( 'give_use_php_sessions' );
+		$give_use_php_sessions = (bool) get_option('give_use_php_sessions');
 
-		if ( ! $give_use_php_sessions ) {
+		if ( ! $give_use_php_sessions) {
 
 			// Attempt to detect if the server supports PHP sessions.
-			if ( function_exists( 'session_start' ) && ! ini_get( 'safe_mode' ) ) {
+			if (function_exists('session_start') && ! ini_get('safe_mode')) {
 
-				$this->set( 'give_use_php_sessions', 1 );
+				$this->set('give_use_php_sessions', 1);
 
-				if ( $this->get( 'give_use_php_sessions' ) ) {
+				if ($this->get('give_use_php_sessions')) {
 
 					$ret = true;
 
 					// Set the database option.
-					update_option( 'give_use_php_sessions', true );
+					update_option('give_use_php_sessions', true);
 
 				}
 			}
@@ -334,13 +334,13 @@ 
 		}
 
 		// Enable or disable PHP Sessions based on the GIVE_USE_PHP_SESSIONS constant.
-		if ( defined( 'GIVE_USE_PHP_SESSIONS' ) && GIVE_USE_PHP_SESSIONS ) {
+		if (defined('GIVE_USE_PHP_SESSIONS') && GIVE_USE_PHP_SESSIONS) {
 			$ret = true;
-		} elseif ( defined( 'GIVE_USE_PHP_SESSIONS' ) && ! GIVE_USE_PHP_SESSIONS ) {
+		} elseif (defined('GIVE_USE_PHP_SESSIONS') && ! GIVE_USE_PHP_SESSIONS) {
 			$ret = false;
 		}
 
-		return (bool) apply_filters( 'give_use_php_sessions', $ret );
+		return (bool) apply_filters('give_use_php_sessions', $ret);
 	}
 
 	/**
@@ -357,9 +357,9 @@ 
 
 		$start_session = true;
 
-		if ( ! empty( $_SERVER['REQUEST_URI'] ) ) {
+		if ( ! empty($_SERVER['REQUEST_URI'])) {
 
-			$blacklist = apply_filters( 'give_session_start_uri_blacklist', array(
+			$blacklist = apply_filters('give_session_start_uri_blacklist', array(
 				'feed',
 				'feed',
 				'feed/rss',
@@ -367,21 +367,21 @@ 
 				'feed/rdf',
 				'feed/atom',
 				'comments/feed/',
-			) );
-			$uri       = ltrim( $_SERVER['REQUEST_URI'], '/' );
-			$uri       = untrailingslashit( $uri );
-			if ( in_array( $uri, $blacklist ) ) {
+			));
+			$uri       = ltrim($_SERVER['REQUEST_URI'], '/');
+			$uri       = untrailingslashit($uri);
+			if (in_array($uri, $blacklist)) {
 				$start_session = false;
 			}
-			if ( false !== strpos( $uri, 'feed=' ) ) {
+			if (false !== strpos($uri, 'feed=')) {
 				$start_session = false;
 			}
-			if ( is_admin() ) {
+			if (is_admin()) {
 				$start_session = false;
 			}
 		}
 
-		return apply_filters( 'give_start_session', $start_session );
+		return apply_filters('give_start_session', $start_session);
 	}
 
 	/**
@@ -397,11 +397,11 @@ 
 	 */
 	public function maybe_start_session() {
 
-		if ( ! $this->should_start_session() ) {
+		if ( ! $this->should_start_session()) {
 			return;
 		}
 
-		if ( ! session_id() && ! headers_sent() ) {
+		if ( ! session_id() && ! headers_sent()) {
 			session_start();
 		}
 
@@ -420,9 +420,9 @@ 
 
 		$expiration = false;
 
-		if ( session_id() && isset( $_COOKIE[ session_name() . '_expiration' ] ) ) {
+		if (session_id() && isset($_COOKIE[session_name().'_expiration'])) {
 
-			$expiration = date( 'D, d M Y h:i:s', intval( $_COOKIE[ session_name() . '_expiration' ] ) );
+			$expiration = date('D, d M Y h:i:s', intval($_COOKIE[session_name().'_expiration']));
 
 		}
 

includes/admin/forms/class-metabox-form-data.php

Braces +15 added lines, -6 removed lines

@@ -662,9 +662,12 @@ 
 							<a href="#<?php echo $form_data_tab['id']; ?>" data-tab-id="<?php echo $form_data_tab['id']; ?>">
 								<?php if ( ! empty( $form_data_tab['icon-html'] ) ) : ?>
 									<?php echo $form_data_tab['icon-html']; ?>
-								<?php else : ?>
+								<?php else {
+	: ?>
 									<span class="give-icon give-icon-default"></span>
-								<?php endif; ?>
+								<?php endif;
+}
+?>
 								<span class="give-label"><?php echo $form_data_tab['label']; ?></span>
 							</a>
 							<?php if ( $this->has_sub_tab( $form_data_tab ) ) : ?>
@@ -674,9 +677,12 @@ 
 											<a href="#<?php echo $sub_tab['id']; ?>" data-tab-id="<?php echo $sub_tab['id']; ?>">
 												<?php if ( ! empty( $sub_tab['icon-html'] ) ) : ?>
 													<?php echo $sub_tab['icon-html']; ?>
-												<?php else : ?>
+												<?php else {
+	: ?>
 													<span class="give-icon give-icon-default"></span>
-												<?php endif; ?>
+												<?php endif;
+}
+?>
 												<span class="give-label"><?php echo $sub_tab['label']; ?></span>
 											</a>
 										</li>
@@ -703,11 +709,14 @@ 
 						</div>
 
 						<?php do_action( "give_after_{$setting['id']}_settings" ); ?>
-					<?php else : ?>
+					<?php else {
+	: ?>
 						<?php if ( $this->has_sub_tab( $setting ) ) : ?>
 							<?php if ( ! empty( $setting['sub-fields'] ) ) : ?>
 								<?php foreach ( $setting['sub-fields'] as $index => $sub_fields ) : ?>
-									<div id="<?php echo $sub_fields['id']; ?>" class="panel give_options_panel give-hidden">
+									<div id="<?php echo $sub_fields['id'];
+}
+?>" class="panel give_options_panel give-hidden">
 										<?php if ( ! empty( $sub_fields['fields'] ) ) : ?>
 											<?php foreach ( $sub_fields['fields'] as $sub_field ) : ?>
 												<?php give_render_field( $sub_field ); ?>

Spacing +297 added lines, -301 removed lines

@@ -46,24 +46,24 @@ 
 	 */
 	function __construct() {
 		$this->metabox_id    = 'give-metabox-form-data';
-		$this->metabox_label = __( 'Donation Form Options', 'give' );
+		$this->metabox_label = __('Donation Form Options', 'give');
 
 		// Setup.
-		add_action( 'admin_init', array( $this, 'setup' ) );
+		add_action('admin_init', array($this, 'setup'));
 
 		// Add metabox.
-		add_action( 'add_meta_boxes', array( $this, 'add_meta_box' ), 10 );
+		add_action('add_meta_boxes', array($this, 'add_meta_box'), 10);
 
 		// Save form meta.
-		add_action( 'save_post_give_forms', array( $this, 'save' ), 10, 2 );
+		add_action('save_post_give_forms', array($this, 'save'), 10, 2);
 
 		// cmb2 old setting loaders.
 		// add_filter( 'give_metabox_form_data_settings', array( $this, 'cmb2_metabox_settings' ) );
 		// Add offline donations options.
-		add_filter( 'give_metabox_form_data_settings', array( $this, 'add_offline_donations_setting_tab' ), 0, 1 );
+		add_filter('give_metabox_form_data_settings', array($this, 'add_offline_donations_setting_tab'), 0, 1);
 
 		// Maintain active tab query parameter after save.
-		add_filter( 'redirect_post_location', array( $this, 'maintain_active_tab' ), 10, 2 );
+		add_filter('redirect_post_location', array($this, 'maintain_active_tab'), 10, 2);
 	}
 
 	/**
@@ -87,7 +87,7 @@ 
 	 */
 	function get_settings() {
 		$post_id           = give_get_admin_post_id();
-		$price_placeholder = give_format_decimal( '1.00', false, false );
+		$price_placeholder = give_format_decimal('1.00', false, false);
 
 		// Start with an underscore to hide fields from custom fields list
 		$prefix = '_give_';
@@ -96,27 +96,27 @@ 
 			/**
 			 * Repeatable Field Groups
 			 */
-			'form_field_options'    => apply_filters( 'give_forms_field_options', array(
+			'form_field_options'    => apply_filters('give_forms_field_options', array(
 				'id'        => 'form_field_options',
-				'title'     => __( 'Donation Options', 'give' ),
+				'title'     => __('Donation Options', 'give'),
 				'icon-html' => '<span class="give-icon give-icon-heart"></span>',
-				'fields'    => apply_filters( 'give_forms_donation_form_metabox_fields', array(
+				'fields'    => apply_filters('give_forms_donation_form_metabox_fields', array(
 					// Donation Option.
 					array(
-						'name'        => __( 'Donation Option', 'give' ),
-						'description' => __( 'Do you want this form to have one set donation price or multiple levels (for example, $10, $20, $50)?', 'give' ),
-						'id'          => $prefix . 'price_option',
+						'name'        => __('Donation Option', 'give'),
+						'description' => __('Do you want this form to have one set donation price or multiple levels (for example, $10, $20, $50)?', 'give'),
+						'id'          => $prefix.'price_option',
 						'type'        => 'radio_inline',
 						'default'     => 'multi',
-						'options'     => apply_filters( 'give_forms_price_options', array(
-							'multi' => __( 'Multi-level Donation', 'give' ),
-							'set'   => __( 'Set Donation', 'give' ),
-						) ),
+						'options'     => apply_filters('give_forms_price_options', array(
+							'multi' => __('Multi-level Donation', 'give'),
+							'set'   => __('Set Donation', 'give'),
+						)),
 					),
 					array(
-						'name'        => __( 'Set Donation', 'give' ),
-						'description' => __( 'This is the set donation amount for this form. If you have a "Custom Amount Minimum" set, make sure it is less than this amount.', 'give' ),
-						'id'          => $prefix . 'set_price',
+						'name'        => __('Set Donation', 'give'),
+						'description' => __('This is the set donation amount for this form. If you have a "Custom Amount Minimum" set, make sure it is less than this amount.', 'give'),
+						'id'          => $prefix.'set_price',
 						'type'        => 'text_small',
 						'data_type'   => 'price',
 						'attributes'  => array(
@@ -126,34 +126,34 @@ 
 					),
 					// Display Style.
 					array(
-						'name'          => __( 'Display Style', 'give' ),
-						'description'   => __( 'Set how the donations levels will display on the form.', 'give' ),
-						'id'            => $prefix . 'display_style',
+						'name'          => __('Display Style', 'give'),
+						'description'   => __('Set how the donations levels will display on the form.', 'give'),
+						'id'            => $prefix.'display_style',
 						'type'          => 'radio_inline',
 						'default'       => 'buttons',
 						'options'       => array(
-							'buttons'  => __( 'Buttons', 'give' ),
-							'radios'   => __( 'Radios', 'give' ),
-							'dropdown' => __( 'Dropdown', 'give' ),
+							'buttons'  => __('Buttons', 'give'),
+							'radios'   => __('Radios', 'give'),
+							'dropdown' => __('Dropdown', 'give'),
 						),
 						'wrapper_class' => 'give-hidden',
 					),
 					// Custom Amount.
 					array(
-						'name'        => __( 'Custom Amount', 'give' ),
-						'description' => __( 'Do you want the user to be able to input their own donation amount?', 'give' ),
-						'id'          => $prefix . 'custom_amount',
+						'name'        => __('Custom Amount', 'give'),
+						'description' => __('Do you want the user to be able to input their own donation amount?', 'give'),
+						'id'          => $prefix.'custom_amount',
 						'type'        => 'radio_inline',
 						'default'     => 'disabled',
 						'options'     => array(
-							'enabled'  => __( 'Enabled', 'give' ),
-							'disabled' => __( 'Disabled', 'give' ),
+							'enabled'  => __('Enabled', 'give'),
+							'disabled' => __('Disabled', 'give'),
 						),
 					),
 					array(
-						'name'          => __( 'Minimum Amount', 'give' ),
-						'description'   => __( 'Enter the minimum custom donation amount.', 'give' ),
-						'id'            => $prefix . 'custom_amount_minimum',
+						'name'          => __('Minimum Amount', 'give'),
+						'description'   => __('Enter the minimum custom donation amount.', 'give'),
+						'id'            => $prefix.'custom_amount_minimum',
 						'type'          => 'text_small',
 						'data_type'     => 'price',
 						'attributes'    => array(
@@ -163,37 +163,37 @@ 
 						'wrapper_class' => 'give-hidden',
 					),
 					array(
-						'name'          => __( 'Custom Amount Text', 'give' ),
-						'description'   => __( 'This text appears as a label below the custom amount field for set donation forms. For multi-level forms the text will appear as it\'s own level (ie button, radio, or select option).', 'give' ),
-						'id'            => $prefix . 'custom_amount_text',
+						'name'          => __('Custom Amount Text', 'give'),
+						'description'   => __('This text appears as a label below the custom amount field for set donation forms. For multi-level forms the text will appear as it\'s own level (ie button, radio, or select option).', 'give'),
+						'id'            => $prefix.'custom_amount_text',
 						'type'          => 'text_medium',
 						'attributes'    => array(
 							'rows'        => 3,
-							'placeholder' => __( 'Give a Custom Amount', 'give' ),
+							'placeholder' => __('Give a Custom Amount', 'give'),
 						),
 						'wrapper_class' => 'give-hidden',
 					),
 					// Donation Levels.
 					array(
-						'id'            => $prefix . 'donation_levels',
+						'id'            => $prefix.'donation_levels',
 						'type'          => 'group',
 						'options'       => array(
-							'add_button'    => __( 'Add Level', 'give' ),
-							'header_title'  => __( 'Donation Level', 'give' ),
+							'add_button'    => __('Add Level', 'give'),
+							'header_title'  => __('Donation Level', 'give'),
 							'remove_button' => '<span class="dashicons dashicons-no"></span>',
 						),
 						'wrapper_class' => 'give-hidden',
 						// Fields array works the same, except id's only need to be unique for this group.
 						// Prefix is not needed.
-						'fields'        => apply_filters( 'give_donation_levels_table_row', array(
+						'fields'        => apply_filters('give_donation_levels_table_row', array(
 							array(
-								'name' => __( 'ID', 'give' ),
-								'id'   => $prefix . 'id',
+								'name' => __('ID', 'give'),
+								'id'   => $prefix.'id',
 								'type' => 'levels_id',
 							),
 							array(
-								'name'       => __( 'Amount', 'give' ),
-								'id'         => $prefix . 'amount',
+								'name'       => __('Amount', 'give'),
+								'id'         => $prefix.'amount',
 								'type'       => 'text_small',
 								'data_type'  => 'price',
 								'attributes' => array(
@@ -202,112 +202,112 @@ 
 								),
 							),
 							array(
-								'name'       => __( 'Text', 'give' ),
-								'id'         => $prefix . 'text',
+								'name'       => __('Text', 'give'),
+								'id'         => $prefix.'text',
 								'type'       => 'text',
 								'attributes' => array(
-									'placeholder' => __( 'Donation Level', 'give' ),
+									'placeholder' => __('Donation Level', 'give'),
 									'class'       => 'give-multilevel-text-field',
 								),
 							),
 							array(
-								'name' => __( 'Default', 'give' ),
-								'id'   => $prefix . 'default',
+								'name' => __('Default', 'give'),
+								'id'   => $prefix.'default',
 								'type' => 'give_default_radio_inline',
 							),
-						) ),
+						)),
 					),
 					array(
 						'name'  => 'donation_options_docs',
 						'type'  => 'docs_link',
 						'url'   => 'http://docs.givewp.com/form-donation-options',
-						'title' => __( 'Donation Options', 'give' ),
+						'title' => __('Donation Options', 'give'),
 					),
 				),
 					$post_id
 				),
-			) ),
+			)),
 
 			/**
 			 * Display Options
 			 */
-			'form_display_options'  => apply_filters( 'give_form_display_options', array(
+			'form_display_options'  => apply_filters('give_form_display_options', array(
 					'id'        => 'form_display_options',
-					'title'     => __( 'Form Display', 'give' ),
+					'title'     => __('Form Display', 'give'),
 					'icon-html' => '<span class="give-icon give-icon-display"></span>',
-					'fields'    => apply_filters( 'give_forms_display_options_metabox_fields', array(
+					'fields'    => apply_filters('give_forms_display_options_metabox_fields', array(
 						array(
-							'name'    => __( 'Display Options', 'give' ),
-							'desc'    => sprintf( __( 'How would you like to display donation information for this form?', 'give' ), '#' ),
-							'id'      => $prefix . 'payment_display',
+							'name'    => __('Display Options', 'give'),
+							'desc'    => sprintf(__('How would you like to display donation information for this form?', 'give'), '#'),
+							'id'      => $prefix.'payment_display',
 							'type'    => 'radio_inline',
 							'options' => array(
-								'onpage' => __( 'All Fields', 'give' ),
-								'modal'  => __( 'Modal', 'give' ),
-								'reveal' => __( 'Reveal', 'give' ),
-								'button' => __( 'Button', 'give' ),
+								'onpage' => __('All Fields', 'give'),
+								'modal'  => __('Modal', 'give'),
+								'reveal' => __('Reveal', 'give'),
+								'button' => __('Button', 'give'),
 							),
 							'default' => 'onpage',
 						),
 						array(
-							'id'            => $prefix . 'reveal_label',
-							'name'          => __( 'Continue Button', 'give' ),
-							'desc'          => __( 'The button label for displaying the additional payment fields.', 'give' ),
+							'id'            => $prefix.'reveal_label',
+							'name'          => __('Continue Button', 'give'),
+							'desc'          => __('The button label for displaying the additional payment fields.', 'give'),
 							'type'          => 'text_small',
 							'attributes'    => array(
-								'placeholder' => __( 'Donate Now', 'give' ),
+								'placeholder' => __('Donate Now', 'give'),
 							),
 							'wrapper_class' => 'give-hidden',
 						),
 						array(
-							'id'         => $prefix . 'checkout_label',
-							'name'       => __( 'Submit Button', 'give' ),
-							'desc'       => __( 'The button label for completing a donation.', 'give' ),
+							'id'         => $prefix.'checkout_label',
+							'name'       => __('Submit Button', 'give'),
+							'desc'       => __('The button label for completing a donation.', 'give'),
 							'type'       => 'text_small',
 							'attributes' => array(
-								'placeholder' => __( 'Donate Now', 'give' ),
+								'placeholder' => __('Donate Now', 'give'),
 							),
 						),
 						array(
-							'name' => __( 'Default Gateway', 'give' ),
-							'desc' => __( 'By default, the gateway for this form will inherit the global default gateway (set under Give > Settings > Payment Gateways). This option allows you to customize the default gateway for this form only.', 'give' ),
-							'id'   => $prefix . 'default_gateway',
+							'name' => __('Default Gateway', 'give'),
+							'desc' => __('By default, the gateway for this form will inherit the global default gateway (set under Give > Settings > Payment Gateways). This option allows you to customize the default gateway for this form only.', 'give'),
+							'id'   => $prefix.'default_gateway',
 							'type' => 'default_gateway',
 						),
 						array(
-							'name'    => __( 'Guest Donations', 'give' ),
-							'desc'    => __( 'Do you want to allow non-logged-in users to make donations?', 'give' ),
-							'id'      => $prefix . 'logged_in_only',
+							'name'    => __('Guest Donations', 'give'),
+							'desc'    => __('Do you want to allow non-logged-in users to make donations?', 'give'),
+							'id'      => $prefix.'logged_in_only',
 							'type'    => 'radio_inline',
 							'default' => 'enabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							),
 						),
 						array(
-							'name'    => __( 'Registration', 'give' ),
-							'desc'    => __( 'Display the registration and login forms in the payment section for non-logged-in users.', 'give' ),
-							'id'      => $prefix . 'show_register_form',
+							'name'    => __('Registration', 'give'),
+							'desc'    => __('Display the registration and login forms in the payment section for non-logged-in users.', 'give'),
+							'id'      => $prefix.'show_register_form',
 							'type'    => 'radio',
 							'options' => array(
-								'none'         => __( 'None', 'give' ),
-								'registration' => __( 'Registration', 'give' ),
-								'login'        => __( 'Login', 'give' ),
-								'both'         => __( 'Registration + Login', 'give' ),
+								'none'         => __('None', 'give'),
+								'registration' => __('Registration', 'give'),
+								'login'        => __('Login', 'give'),
+								'both'         => __('Registration + Login', 'give'),
 							),
 							'default' => 'none',
 						),
 						array(
-							'name'    => __( 'Floating Labels', 'give' ),
+							'name'    => __('Floating Labels', 'give'),
 							/* translators: %s: forms http://docs.givewp.com/form-floating-labels */
-							'desc'    => sprintf( __( 'Select the <a href="%s" target="_blank">floating labels</a> setting for this Give form. Be aware that if you have the "Disable CSS" option enabled, you will need to style the floating labels yourself.', 'give' ), esc_url( 'http://docs.givewp.com/form-floating-labels' ) ),
-							'id'      => $prefix . 'form_floating_labels',
+							'desc'    => sprintf(__('Select the <a href="%s" target="_blank">floating labels</a> setting for this Give form. Be aware that if you have the "Disable CSS" option enabled, you will need to style the floating labels yourself.', 'give'), esc_url('http://docs.givewp.com/form-floating-labels')),
+							'id'      => $prefix.'form_floating_labels',
 							'type'    => 'radio_inline',
 							'options' => array(
-								'global'   => __( 'Global Option', 'give' ),
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'global'   => __('Global Option', 'give'),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							),
 							'default' => 'global',
 						),
@@ -315,7 +315,7 @@ 
 							'name'  => 'form_display_docs',
 							'type'  => 'docs_link',
 							'url'   => 'http://docs.givewp.com/form-display-options',
-							'title' => __( 'Form Display', 'give' ),
+							'title' => __('Form Display', 'give'),
 						),
 					),
 						$post_id
@@ -326,27 +326,27 @@ 
 			/**
 			 * Donation Goals
 			 */
-			'donation_goal_options' => apply_filters( 'give_donation_goal_options', array(
+			'donation_goal_options' => apply_filters('give_donation_goal_options', array(
 				'id'        => 'donation_goal_options',
-				'title'     => __( 'Donation Goal', 'give' ),
+				'title'     => __('Donation Goal', 'give'),
 				'icon-html' => '<span class="give-icon give-icon-target"></span>',
-				'fields'    => apply_filters( 'give_forms_donation_goal_metabox_fields', array(
+				'fields'    => apply_filters('give_forms_donation_goal_metabox_fields', array(
 					// Goals
 					array(
-						'name'        => __( 'Donation Goal', 'give' ),
-						'description' => __( 'Do you want to set a donation goal for this form?', 'give' ),
-						'id'          => $prefix . 'goal_option',
+						'name'        => __('Donation Goal', 'give'),
+						'description' => __('Do you want to set a donation goal for this form?', 'give'),
+						'id'          => $prefix.'goal_option',
 						'type'        => 'radio_inline',
 						'default'     => 'disabled',
 						'options'     => array(
-							'enabled'  => __( 'Enabled', 'give' ),
-							'disabled' => __( 'Disabled', 'give' ),
+							'enabled'  => __('Enabled', 'give'),
+							'disabled' => __('Disabled', 'give'),
 						),
 					),
 					array(
-						'name'          => __( 'Goal Amount', 'give' ),
-						'description'   => __( 'This is the monetary goal amount you want to reach for this form.', 'give' ),
-						'id'            => $prefix . 'set_goal',
+						'name'          => __('Goal Amount', 'give'),
+						'description'   => __('This is the monetary goal amount you want to reach for this form.', 'give'),
+						'id'            => $prefix.'set_goal',
 						'type'          => 'text_small',
 						'data_type'     => 'price',
 						'attributes'    => array(
@@ -357,97 +357,97 @@ 
 					),
 
 					array(
-						'name'          => __( 'Goal Format', 'give' ),
-						'description'   => __( 'Do you want to display the total amount raised based on your monetary goal or a percentage? For instance, "$500 of $1,000 raised" or "50% funded".', 'give' ),
-						'id'            => $prefix . 'goal_format',
+						'name'          => __('Goal Format', 'give'),
+						'description'   => __('Do you want to display the total amount raised based on your monetary goal or a percentage? For instance, "$500 of $1,000 raised" or "50% funded".', 'give'),
+						'id'            => $prefix.'goal_format',
 						'type'          => 'radio_inline',
 						'default'       => 'amount',
 						'options'       => array(
-							'amount'     => __( 'Amount', 'give' ),
-							'percentage' => __( 'Percentage', 'give' ),
+							'amount'     => __('Amount', 'give'),
+							'percentage' => __('Percentage', 'give'),
 						),
 						'wrapper_class' => 'give-hidden',
 					),
 					array(
-						'name'          => __( 'Progress Bar Color', 'give' ),
-						'desc'          => __( 'Customize the color of the goal progress bar.', 'give' ),
-						'id'            => $prefix . 'goal_color',
+						'name'          => __('Progress Bar Color', 'give'),
+						'desc'          => __('Customize the color of the goal progress bar.', 'give'),
+						'id'            => $prefix.'goal_color',
 						'type'          => 'colorpicker',
 						'default'       => '#2bc253',
 						'wrapper_class' => 'give-hidden',
 					),
 
 					array(
-						'name'          => __( 'Close Form', 'give' ),
-						'desc'          => __( 'Do you want to close the donation forms and stop accepting donations once this goal has been met?', 'give' ),
-						'id'            => $prefix . 'close_form_when_goal_achieved',
+						'name'          => __('Close Form', 'give'),
+						'desc'          => __('Do you want to close the donation forms and stop accepting donations once this goal has been met?', 'give'),
+						'id'            => $prefix.'close_form_when_goal_achieved',
 						'type'          => 'radio_inline',
 						'default'       => 'disabled',
 						'options'       => array(
-							'enabled'  => __( 'Enabled', 'give' ),
-							'disabled' => __( 'Disabled', 'give' ),
+							'enabled'  => __('Enabled', 'give'),
+							'disabled' => __('Disabled', 'give'),
 						),
 						'wrapper_class' => 'give-hidden',
 					),
 					array(
-						'name'          => __( 'Goal Achieved Message', 'give' ),
-						'desc'          => __( 'Do you want to display a custom message when the goal is closed?', 'give' ),
-						'id'            => $prefix . 'form_goal_achieved_message',
+						'name'          => __('Goal Achieved Message', 'give'),
+						'desc'          => __('Do you want to display a custom message when the goal is closed?', 'give'),
+						'id'            => $prefix.'form_goal_achieved_message',
 						'type'          => 'wysiwyg',
-						'default'       => __( 'Thank you to all our donors, we have met our fundraising goal.', 'give' ),
+						'default'       => __('Thank you to all our donors, we have met our fundraising goal.', 'give'),
 						'wrapper_class' => 'give-hidden',
 					),
 					array(
 						'name'  => 'donation_goal_docs',
 						'type'  => 'docs_link',
 						'url'   => 'http://docs.givewp.com/form-donation-goal',
-						'title' => __( 'Donation Goal', 'give' ),
+						'title' => __('Donation Goal', 'give'),
 					),
 				),
 					$post_id
 				),
-			) ),
+			)),
 
 			/**
 			 * Content Field
 			 */
-			'form_content_options'  => apply_filters( 'give_forms_content_options', array(
+			'form_content_options'  => apply_filters('give_forms_content_options', array(
 				'id'        => 'form_content_options',
-				'title'     => __( 'Form Content', 'give' ),
+				'title'     => __('Form Content', 'give'),
 				'icon-html' => '<span class="give-icon give-icon-edit"></span>',
-				'fields'    => apply_filters( 'give_forms_content_options_metabox_fields', array(
+				'fields'    => apply_filters('give_forms_content_options_metabox_fields', array(
 
 					// Donation content.
 					array(
-						'name'        => __( 'Display Content', 'give' ),
-						'description' => __( 'Do you want to add custom content to this form?', 'give' ),
-						'id'          => $prefix . 'display_content',
+						'name'        => __('Display Content', 'give'),
+						'description' => __('Do you want to add custom content to this form?', 'give'),
+						'id'          => $prefix.'display_content',
 						'type'        => 'radio_inline',
 						'options'     => array(
-							'enabled'  => __( 'Enabled', 'give' ),
-							'disabled' => __( 'Disabled', 'give' ),
+							'enabled'  => __('Enabled', 'give'),
+							'disabled' => __('Disabled', 'give'),
 						),
 						'default'     => 'disabled',
 					),
 
 					// Content placement.
 					array(
-						'name'          => __( 'Content Placement', 'give' ),
-						'description'   => __( 'This option controls where the content appears within the donation form.', 'give' ),
-						'id'            => $prefix . 'content_placement',
+						'name'          => __('Content Placement', 'give'),
+						'description'   => __('This option controls where the content appears within the donation form.', 'give'),
+						'id'            => $prefix.'content_placement',
 						'type'          => 'radio_inline',
-						'options'       => apply_filters( 'give_forms_content_options_select', array(
-								'give_pre_form'  => __( 'Above fields', 'give' ),
-								'give_post_form' => __( 'Below fields', 'give' ),
+						'options'       => apply_filters('give_forms_content_options_select', array(
+								'give_pre_form'  => __('Above fields', 'give'),
+								'give_post_form' => __('Below fields', 'give'),
 							)
 						),
 						'default'       => 'give_pre_form',
 						'wrapper_class' => 'give-hidden',
 					),
 					array(
-						'name'          => __( 'Content', 'give' ),
-						'description'   => __( 'This content will display on the single give form page.', 'give' ),
-						'id'            => $prefix . 'form_content',
+						'name'          => __('Content', 'give'),
+						'description'   => __('This content will display on the single give form page.', 'give'),
+						'id'            => $prefix.'form_content',
 						'type'          => 'wysiwyg',
 						'wrapper_class' => 'give-hidden',
 					),
@@ -455,51 +455,51 @@ 
 						'name'  => 'form_content_docs',
 						'type'  => 'docs_link',
 						'url'   => 'http://docs.givewp.com/form-content',
-						'title' => __( 'Form Content', 'give' ),
+						'title' => __('Form Content', 'give'),
 					),
 				),
 					$post_id
 				),
-			) ),
+			)),
 
 			/**
 			 * Terms & Conditions
 			 */
-			'form_terms_options'    => apply_filters( 'give_forms_terms_options', array(
+			'form_terms_options'    => apply_filters('give_forms_terms_options', array(
 				'id'        => 'form_terms_options',
-				'title'     => __( 'Terms & Conditions', 'give' ),
+				'title'     => __('Terms & Conditions', 'give'),
 				'icon-html' => '<span class="give-icon give-icon-checklist"></span>',
-				'fields'    => apply_filters( 'give_forms_terms_options_metabox_fields', array(
+				'fields'    => apply_filters('give_forms_terms_options_metabox_fields', array(
 					// Donation Option
 					array(
-						'name'        => __( 'Terms and Conditions', 'give' ),
-						'description' => __( 'Do you want to require the donor to accept terms prior to being able to complete their donation?', 'give' ),
-						'id'          => $prefix . 'terms_option',
+						'name'        => __('Terms and Conditions', 'give'),
+						'description' => __('Do you want to require the donor to accept terms prior to being able to complete their donation?', 'give'),
+						'id'          => $prefix.'terms_option',
 						'type'        => 'radio_inline',
-						'options'     => apply_filters( 'give_forms_content_options_select', array(
-								'global'   => __( 'Global Option', 'give' ),
-								'enabled'  => __( 'Customize', 'give' ),
-								'disabled' => __( 'Disable', 'give' ),
+						'options'     => apply_filters('give_forms_content_options_select', array(
+								'global'   => __('Global Option', 'give'),
+								'enabled'  => __('Customize', 'give'),
+								'disabled' => __('Disable', 'give'),
 							)
 						),
 						'default'     => 'global',
 					),
 					array(
-						'id'            => $prefix . 'agree_label',
-						'name'          => __( 'Agreement Label', 'give' ),
-						'desc'          => __( 'The label shown next to the agree to terms check box. Add your own to customize or leave blank to use the default text placeholder.', 'give' ),
+						'id'            => $prefix.'agree_label',
+						'name'          => __('Agreement Label', 'give'),
+						'desc'          => __('The label shown next to the agree to terms check box. Add your own to customize or leave blank to use the default text placeholder.', 'give'),
 						'type'          => 'text',
 						'size'          => 'regular',
 						'attributes'    => array(
-							'placeholder' => __( 'Agree to Terms?', 'give' ),
+							'placeholder' => __('Agree to Terms?', 'give'),
 						),
 						'wrapper_class' => 'give-hidden',
 					),
 					array(
-						'id'            => $prefix . 'agree_text',
-						'name'          => __( 'Agreement Text', 'give' ),
-						'desc'          => __( 'This is the actual text which the user will have to agree to in order to make a donation.', 'give' ),
-						'default'       => give_get_option( 'agreement_text' ),
+						'id'            => $prefix.'agree_text',
+						'name'          => __('Agreement Text', 'give'),
+						'desc'          => __('This is the actual text which the user will have to agree to in order to make a donation.', 'give'),
+						'default'       => give_get_option('agreement_text'),
 						'type'          => 'wysiwyg',
 						'wrapper_class' => 'give-hidden',
 					),
@@ -507,18 +507,18 @@ 
 						'name'  => 'terms_docs',
 						'type'  => 'docs_link',
 						'url'   => 'http://docs.givewp.com/form-terms',
-						'title' => __( 'Terms and Conditions', 'give' ),
+						'title' => __('Terms and Conditions', 'give'),
 					),
 				),
 					$post_id
 				),
-			) ),
+			)),
 		);
 
 		/**
 		 * Filter the metabox tabbed panel settings.
 		 */
-		$settings = apply_filters( 'give_metabox_form_data_settings', $settings, $post_id );
+		$settings = apply_filters('give_metabox_form_data_settings', $settings, $post_id);
 
 		// Output.
 		return $settings;
@@ -535,8 +535,8 @@ 
 		add_meta_box(
 			$this->get_metabox_ID(),
 			$this->get_metabox_label(),
-			array( $this, 'output' ),
-			array( 'give_forms' ),
+			array($this, 'output'),
+			array('give_forms'),
 			'normal',
 			'high'
 		);
@@ -553,7 +553,7 @@ 
 	function enqueue_script() {
 		global $post;
 
-		if ( is_object( $post ) && 'give_forms' === $post->post_type ) {
+		if (is_object($post) && 'give_forms' === $post->post_type) {
 
 		}
 	}
@@ -591,32 +591,32 @@ 
 	public function get_tabs() {
 		$tabs = array();
 
-		if ( ! empty( $this->settings ) ) {
-			foreach ( $this->settings as $setting ) {
-				if ( ! isset( $setting['id'] ) || ! isset( $setting['title'] ) ) {
+		if ( ! empty($this->settings)) {
+			foreach ($this->settings as $setting) {
+				if ( ! isset($setting['id']) || ! isset($setting['title'])) {
 					continue;
 				}
 				$tab = array(
 					'id'        => $setting['id'],
 					'label'     => $setting['title'],
-					'icon-html' => ( ! empty( $setting['icon-html'] ) ? $setting['icon-html'] : '' ),
+					'icon-html' => ( ! empty($setting['icon-html']) ? $setting['icon-html'] : ''),
 				);
 
-				if ( $this->has_sub_tab( $setting ) ) {
-					if ( empty( $setting['sub-fields'] ) ) {
+				if ($this->has_sub_tab($setting)) {
+					if (empty($setting['sub-fields'])) {
 						$tab = array();
 					} else {
-						foreach ( $setting['sub-fields'] as $sub_fields ) {
+						foreach ($setting['sub-fields'] as $sub_fields) {
 							$tab['sub-fields'][] = array(
 								'id'        => $sub_fields['id'],
 								'label'     => $sub_fields['title'],
-								'icon-html' => ( ! empty( $sub_fields['icon-html'] ) ? $sub_fields['icon-html'] : '' ),
+								'icon-html' => ( ! empty($sub_fields['icon-html']) ? $sub_fields['icon-html'] : ''),
 							);
 						}
 					}
 				}
 
-				if ( ! empty( $tab ) ) {
+				if ( ! empty($tab)) {
 					$tabs[] = $tab;
 				}
 			}
@@ -634,33 +634,33 @@ 
 	 */
 	public function output() {
 		// Bailout.
-		if ( $form_data_tabs = $this->get_tabs() ) :
-			$active_tab = ! empty( $_GET['give_tab'] ) ? give_clean( $_GET['give_tab'] ) : 'form_field_options';
-			wp_nonce_field( 'give_save_form_meta', 'give_form_meta_nonce' );
+		if ($form_data_tabs = $this->get_tabs()) :
+			$active_tab = ! empty($_GET['give_tab']) ? give_clean($_GET['give_tab']) : 'form_field_options';
+			wp_nonce_field('give_save_form_meta', 'give_form_meta_nonce');
 			?>
 			<input id="give_form_active_tab" type="hidden" name="give_form_active_tab">
 			<div class="give-metabox-panel-wrap">
 				<ul class="give-form-data-tabs give-metabox-tabs">
-					<?php foreach ( $form_data_tabs as $index => $form_data_tab ) : ?>
+					<?php foreach ($form_data_tabs as $index => $form_data_tab) : ?>
 						<?php
 						// Determine if current tab is active.
 						$is_active = $active_tab === $form_data_tab['id'] ? true : false;
 						?>
-						<li class="<?php echo "{$form_data_tab['id']}_tab" . ( $is_active ? ' active' : '' ) . ( $this->has_sub_tab( $form_data_tab ) ? ' has-sub-fields' : '' ); ?>">
+						<li class="<?php echo "{$form_data_tab['id']}_tab".($is_active ? ' active' : '').($this->has_sub_tab($form_data_tab) ? ' has-sub-fields' : ''); ?>">
 							<a href="#<?php echo $form_data_tab['id']; ?>" data-tab-id="<?php echo $form_data_tab['id']; ?>">
-								<?php if ( ! empty( $form_data_tab['icon-html'] ) ) : ?>
+								<?php if ( ! empty($form_data_tab['icon-html'])) : ?>
 									<?php echo $form_data_tab['icon-html']; ?>
 								<?php else : ?>
 									<span class="give-icon give-icon-default"></span>
 								<?php endif; ?>
 								<span class="give-label"><?php echo $form_data_tab['label']; ?></span>
 							</a>
-							<?php if ( $this->has_sub_tab( $form_data_tab ) ) : ?>
+							<?php if ($this->has_sub_tab($form_data_tab)) : ?>
 								<ul class="give-metabox-sub-tabs give-hidden">
-									<?php foreach ( $form_data_tab['sub-fields'] as $sub_tab ) : ?>
+									<?php foreach ($form_data_tab['sub-fields'] as $sub_tab) : ?>
 										<li class="<?php echo "{$sub_tab['id']}_tab"; ?>">
 											<a href="#<?php echo $sub_tab['id']; ?>" data-tab-id="<?php echo $sub_tab['id']; ?>">
-												<?php if ( ! empty( $sub_tab['icon-html'] ) ) : ?>
+												<?php if ( ! empty($sub_tab['icon-html'])) : ?>
 													<?php echo $sub_tab['icon-html']; ?>
 												<?php else : ?>
 													<span class="give-icon give-icon-default"></span>
@@ -675,30 +675,30 @@ 
 					<?php endforeach; ?>
 				</ul>
 
-				<?php foreach ( $this->settings as $setting ) : ?>
-					<?php if ( ! $this->has_sub_tab( $setting ) ) : ?>
-						<?php do_action( "give_before_{$setting['id']}_settings" ); ?>
+				<?php foreach ($this->settings as $setting) : ?>
+					<?php if ( ! $this->has_sub_tab($setting)) : ?>
+						<?php do_action("give_before_{$setting['id']}_settings"); ?>
 						<?php
 						// Determine if current panel is active.
 						$is_active = $active_tab === $setting['id'] ? true : false;
 						?>
-						<div id="<?php echo $setting['id']; ?>" class="panel give_options_panel<?php echo( $is_active ? ' active' : '' ); ?>">
-							<?php if ( ! empty( $setting['fields'] ) ) : ?>
-								<?php foreach ( $setting['fields'] as $field ) : ?>
-									<?php give_render_field( $field ); ?>
+						<div id="<?php echo $setting['id']; ?>" class="panel give_options_panel<?php echo($is_active ? ' active' : ''); ?>">
+							<?php if ( ! empty($setting['fields'])) : ?>
+								<?php foreach ($setting['fields'] as $field) : ?>
+									<?php give_render_field($field); ?>
 								<?php endforeach; ?>
 							<?php endif; ?>
 						</div>
 
-						<?php do_action( "give_after_{$setting['id']}_settings" ); ?>
+						<?php do_action("give_after_{$setting['id']}_settings"); ?>
 					<?php else : ?>
-						<?php if ( $this->has_sub_tab( $setting ) ) : ?>
-							<?php if ( ! empty( $setting['sub-fields'] ) ) : ?>
-								<?php foreach ( $setting['sub-fields'] as $index => $sub_fields ) : ?>
+						<?php if ($this->has_sub_tab($setting)) : ?>
+							<?php if ( ! empty($setting['sub-fields'])) : ?>
+								<?php foreach ($setting['sub-fields'] as $index => $sub_fields) : ?>
 									<div id="<?php echo $sub_fields['id']; ?>" class="panel give_options_panel give-hidden">
-										<?php if ( ! empty( $sub_fields['fields'] ) ) : ?>
-											<?php foreach ( $sub_fields['fields'] as $sub_field ) : ?>
-												<?php give_render_field( $sub_field ); ?>
+										<?php if ( ! empty($sub_fields['fields'])) : ?>
+											<?php foreach ($sub_fields['fields'] as $sub_field) : ?>
+												<?php give_render_field($sub_field); ?>
 											<?php endforeach; ?>
 										<?php endif; ?>
 									</div>
@@ -722,9 +722,9 @@ 
 	 *
 	 * @return bool
 	 */
-	private function has_sub_tab( $field_setting ) {
+	private function has_sub_tab($field_setting) {
 		$has_sub_tab = false;
-		if ( array_key_exists( 'sub-fields', $field_setting ) ) {
+		if (array_key_exists('sub-fields', $field_setting)) {
 			$has_sub_tab = true;
 		}
 
@@ -739,13 +739,13 @@ 
 	 * @return array
 	 */
 	function cmb2_metabox_settings() {
-		$all_cmb2_settings   = apply_filters( 'cmb2_meta_boxes', array() );
+		$all_cmb2_settings   = apply_filters('cmb2_meta_boxes', array());
 		$give_forms_settings = $all_cmb2_settings;
 
 		// Filter settings: Use only give forms related settings.
-		foreach ( $all_cmb2_settings as $index => $setting ) {
-			if ( ! in_array( 'give_forms', $setting['object_types'] ) ) {
-				unset( $give_forms_settings[ $index ] );
+		foreach ($all_cmb2_settings as $index => $setting) {
+			if ( ! in_array('give_forms', $setting['object_types'])) {
+				unset($give_forms_settings[$index]);
 			}
 		}
 
@@ -763,96 +763,96 @@ 
 	 *
 	 * @return void
 	 */
-	public function save( $post_id, $post ) {
+	public function save($post_id, $post) {
 
 		// $post_id and $post are required.
-		if ( empty( $post_id ) || empty( $post ) ) {
+		if (empty($post_id) || empty($post)) {
 			return;
 		}
 
 		// Don't save meta boxes for revisions or autosaves.
-		if ( defined( 'DOING_AUTOSAVE' ) || is_int( wp_is_post_revision( $post ) ) || is_int( wp_is_post_autosave( $post ) ) ) {
+		if (defined('DOING_AUTOSAVE') || is_int(wp_is_post_revision($post)) || is_int(wp_is_post_autosave($post))) {
 			return;
 		}
 
 		// Check the nonce.
-		if ( empty( $_POST['give_form_meta_nonce'] ) || ! wp_verify_nonce( $_POST['give_form_meta_nonce'], 'give_save_form_meta' ) ) {
+		if (empty($_POST['give_form_meta_nonce']) || ! wp_verify_nonce($_POST['give_form_meta_nonce'], 'give_save_form_meta')) {
 			return;
 		}
 
 		// Check the post being saved == the $post_id to prevent triggering this call for other save_post events.
-		if ( empty( $_POST['post_ID'] ) || $_POST['post_ID'] != $post_id ) {
+		if (empty($_POST['post_ID']) || $_POST['post_ID'] != $post_id) {
 			return;
 		}
 
 		// Check user has permission to edit.
-		if ( ! current_user_can( 'edit_post', $post_id ) ) {
+		if ( ! current_user_can('edit_post', $post_id)) {
 			return;
 		}
 
 		// Fire action before saving form meta.
-		do_action( 'give_pre_process_give_forms_meta', $post_id, $post );
+		do_action('give_pre_process_give_forms_meta', $post_id, $post);
 
 		/**
 		 * Filter the meta key to save.
 		 * Third party addon developer can remove there meta keys from this array to handle saving data on there own.
 		 */
-		$form_meta_keys = apply_filters( 'give_process_form_meta_keys', $this->get_meta_keys_from_settings() );
+		$form_meta_keys = apply_filters('give_process_form_meta_keys', $this->get_meta_keys_from_settings());
 
 		// Save form meta data.
-		if ( ! empty( $form_meta_keys ) ) {
-			foreach ( $form_meta_keys as $form_meta_key ) {
+		if ( ! empty($form_meta_keys)) {
+			foreach ($form_meta_keys as $form_meta_key) {
 
 				// Set default value for checkbox fields.
 				if (
-					! isset( $_POST[ $form_meta_key ] )
-					&& ( 'checkbox' === $this->get_field_type( $form_meta_key ) )
+					! isset($_POST[$form_meta_key])
+					&& ('checkbox' === $this->get_field_type($form_meta_key))
 				) {
-					$_POST[ $form_meta_key ] = '';
+					$_POST[$form_meta_key] = '';
 				}
 
-				if ( isset( $_POST[ $form_meta_key ] ) ) {
-					$setting_field = $this->get_setting_field( $form_meta_key );
-					if ( ! empty( $setting_field['type'] ) ) {
-						switch ( $setting_field['type'] ) {
+				if (isset($_POST[$form_meta_key])) {
+					$setting_field = $this->get_setting_field($form_meta_key);
+					if ( ! empty($setting_field['type'])) {
+						switch ($setting_field['type']) {
 							case 'textarea':
 							case 'wysiwyg':
-								$form_meta_value = wp_kses_post( $_POST[ $form_meta_key ] );
+								$form_meta_value = wp_kses_post($_POST[$form_meta_key]);
 								break;
 
 							case 'group':
 								$form_meta_value = array();
 
-								foreach ( $_POST[ $form_meta_key ] as $index => $group ) {
+								foreach ($_POST[$form_meta_key] as $index => $group) {
 
 									// Do not save template input field values.
-									if ( '{{row-count-placeholder}}' === $index ) {
+									if ('{{row-count-placeholder}}' === $index) {
 										continue;
 									}
 
 									$group_meta_value = array();
-									foreach ( $group as $field_id => $field_value ) {
-										switch ( $this->get_field_type( $field_id, $form_meta_key ) ) {
+									foreach ($group as $field_id => $field_value) {
+										switch ($this->get_field_type($field_id, $form_meta_key)) {
 											case 'wysiwyg':
-												$group_meta_value[ $field_id ] = wp_kses_post( $field_value );
+												$group_meta_value[$field_id] = wp_kses_post($field_value);
 												break;
 
 											default:
-												$group_meta_value[ $field_id ] = give_clean( $field_value );
+												$group_meta_value[$field_id] = give_clean($field_value);
 										}
 									}
 
-									if ( ! empty( $group_meta_value ) ) {
-										$form_meta_value[ $index ] = $group_meta_value;
+									if ( ! empty($group_meta_value)) {
+										$form_meta_value[$index] = $group_meta_value;
 									}
 								}
 
 								// Arrange repeater field keys in order.
-								$form_meta_value = array_values( $form_meta_value );
+								$form_meta_value = array_values($form_meta_value);
 								break;
 
 							default:
-								$form_meta_value = give_clean( $_POST[ $form_meta_key ] );
+								$form_meta_value = give_clean($_POST[$form_meta_key]);
 						}// End switch().
 
 						/**
@@ -862,24 +862,24 @@ 
 						 */
 						$form_meta_value = apply_filters(
 							'give_pre_save_form_meta_value',
-							$this->sanitize_form_meta( $form_meta_value, $setting_field ),
+							$this->sanitize_form_meta($form_meta_value, $setting_field),
 							$form_meta_key,
 							$this,
 							$post_id
 						);
 
 						// Save data.
-						give_update_meta( $post_id, $form_meta_key, $form_meta_value );
+						give_update_meta($post_id, $form_meta_key, $form_meta_value);
 
 						// Fire after saving form meta key.
-						do_action( "give_save_{$form_meta_key}", $form_meta_key, $form_meta_value, $post_id, $post );
+						do_action("give_save_{$form_meta_key}", $form_meta_key, $form_meta_value, $post_id, $post);
 					}// End if().
 				}// End if().
 			}// End foreach().
 		}// End if().
 
 		// Fire action after saving form meta.
-		do_action( 'give_post_process_give_forms_meta', $post_id, $post );
+		do_action('give_post_process_give_forms_meta', $post_id, $post);
 	}
 
 
@@ -892,10 +892,10 @@ 
 	 *
 	 * @return string
 	 */
-	private function get_field_id( $field ) {
+	private function get_field_id($field) {
 		$field_id = '';
 
-		if ( array_key_exists( 'id', $field ) ) {
+		if (array_key_exists('id', $field)) {
 			$field_id = $field['id'];
 
 		}
@@ -912,12 +912,12 @@ 
 	 *
 	 * @return array
 	 */
-	private function get_fields_id( $setting ) {
+	private function get_fields_id($setting) {
 		$meta_keys = array();
 
-		if ( ! empty( $setting ) ) {
-			foreach ( $setting['fields'] as $field ) {
-				if ( $field_id = $this->get_field_id( $field ) ) {
+		if ( ! empty($setting)) {
+			foreach ($setting['fields'] as $field) {
+				if ($field_id = $this->get_field_id($field)) {
 					$meta_keys[] = $field_id;
 				}
 			}
@@ -935,14 +935,14 @@ 
 	 *
 	 * @return array
 	 */
-	private function get_sub_fields_id( $setting ) {
+	private function get_sub_fields_id($setting) {
 		$meta_keys = array();
 
-		if ( $this->has_sub_tab( $setting ) && ! empty( $setting['sub-fields'] ) ) {
-			foreach ( $setting['sub-fields'] as $fields ) {
-				if ( ! empty( $fields['fields'] ) ) {
-					foreach ( $fields['fields'] as $field ) {
-						if ( $field_id = $this->get_field_id( $field ) ) {
+		if ($this->has_sub_tab($setting) && ! empty($setting['sub-fields'])) {
+			foreach ($setting['sub-fields'] as $fields) {
+				if ( ! empty($fields['fields'])) {
+					foreach ($fields['fields'] as $field) {
+						if ($field_id = $this->get_field_id($field)) {
 							$meta_keys[] = $field_id;
 						}
 					}
@@ -964,14 +964,14 @@ 
 	private function get_meta_keys_from_settings() {
 		$meta_keys = array();
 
-		foreach ( $this->settings as $setting ) {
-			if ( $this->has_sub_tab( $setting ) ) {
-				$meta_key = $this->get_sub_fields_id( $setting );
+		foreach ($this->settings as $setting) {
+			if ($this->has_sub_tab($setting)) {
+				$meta_key = $this->get_sub_fields_id($setting);
 			} else {
-				$meta_key = $this->get_fields_id( $setting );
+				$meta_key = $this->get_fields_id($setting);
 			}
 
-			$meta_keys = array_merge( $meta_keys, $meta_key );
+			$meta_keys = array_merge($meta_keys, $meta_key);
 		}
 
 		return $meta_keys;
@@ -988,10 +988,10 @@ 
 	 *
 	 * @return string
 	 */
-	function get_field_type( $field_id, $group_id = '' ) {
-		$field = $this->get_setting_field( $field_id, $group_id );
+	function get_field_type($field_id, $group_id = '') {
+		$field = $this->get_setting_field($field_id, $group_id);
 
-		$type = array_key_exists( 'type', $field )
+		$type = array_key_exists('type', $field)
 			? $field['type']
 			: '';
 
@@ -1009,12 +1009,12 @@ 
 	 *
 	 * @return array
 	 */
-	private function get_field( $setting, $field_id ) {
+	private function get_field($setting, $field_id) {
 		$setting_field = array();
 
-		if ( ! empty( $setting['fields'] ) ) {
-			foreach ( $setting['fields'] as $field ) {
-				if ( array_key_exists( 'id', $field ) && $field['id'] === $field_id ) {
+		if ( ! empty($setting['fields'])) {
+			foreach ($setting['fields'] as $field) {
+				if (array_key_exists('id', $field) && $field['id'] === $field_id) {
 					$setting_field = $field;
 					break;
 				}
@@ -1034,12 +1034,12 @@ 
 	 *
 	 * @return array
 	 */
-	private function get_sub_field( $setting, $field_id ) {
+	private function get_sub_field($setting, $field_id) {
 		$setting_field = array();
 
-		if ( ! empty( $setting['sub-fields'] ) ) {
-			foreach ( $setting['sub-fields'] as $fields ) {
-				if ( $field = $this->get_field( $fields, $field_id ) ) {
+		if ( ! empty($setting['sub-fields'])) {
+			foreach ($setting['sub-fields'] as $fields) {
+				if ($field = $this->get_field($fields, $field_id)) {
 					$setting_field = $field;
 					break;
 				}
@@ -1059,17 +1059,17 @@ 
 	 *
 	 * @return array
 	 */
-	function get_setting_field( $field_id, $group_id = '' ) {
+	function get_setting_field($field_id, $group_id = '') {
 		$setting_field = array();
 
 		$_field_id = $field_id;
-		$field_id  = empty( $group_id ) ? $field_id : $group_id;
+		$field_id  = empty($group_id) ? $field_id : $group_id;
 
-		if ( ! empty( $this->settings ) ) {
-			foreach ( $this->settings as $setting ) {
+		if ( ! empty($this->settings)) {
+			foreach ($this->settings as $setting) {
 				if (
-					( $this->has_sub_tab( $setting ) && ( $setting_field = $this->get_sub_field( $setting, $field_id ) ) )
-					|| ( $setting_field = $this->get_field( $setting, $field_id ) )
+					($this->has_sub_tab($setting) && ($setting_field = $this->get_sub_field($setting, $field_id)))
+					|| ($setting_field = $this->get_field($setting, $field_id))
 				) {
 					break;
 				}
@@ -1077,9 +1077,9 @@ 
 		}
 
 		// Get field from group.
-		if ( ! empty( $group_id ) ) {
-			foreach ( $setting_field['fields'] as $field ) {
-				if ( array_key_exists( 'id', $field ) && $field['id'] === $_field_id ) {
+		if ( ! empty($group_id)) {
+			foreach ($setting_field['fields'] as $field) {
+				if (array_key_exists('id', $field) && $field['id'] === $_field_id) {
 					$setting_field = $field;
 				}
 			}
@@ -1098,14 +1098,14 @@ 
 	 *
 	 * @return mixed
 	 */
-	function add_offline_donations_setting_tab( $settings ) {
-		if ( give_is_gateway_active( 'offline' ) ) {
-			$settings['offline_donations_options'] = apply_filters( 'give_forms_offline_donations_options', array(
+	function add_offline_donations_setting_tab($settings) {
+		if (give_is_gateway_active('offline')) {
+			$settings['offline_donations_options'] = apply_filters('give_forms_offline_donations_options', array(
 				'id'        => 'offline_donations_options',
-				'title'     => __( 'Offline Donations', 'give' ),
+				'title'     => __('Offline Donations', 'give'),
 				'icon-html' => '<span class="give-icon give-icon-purse"></span>',
-				'fields'    => apply_filters( 'give_forms_offline_donations_metabox_fields', array() ),
-			) );
+				'fields'    => apply_filters('give_forms_offline_donations_metabox_fields', array()),
+			));
 		}
 
 		return $settings;
@@ -1123,37 +1123,33 @@ 
 	 *
 	 * @return mixed
 	 */
-	function sanitize_form_meta( $meta_value, $setting_field ) {
-		switch ( $setting_field['type'] ) {
+	function sanitize_form_meta($meta_value, $setting_field) {
+		switch ($setting_field['type']) {
 			case 'group':
-				if ( ! empty( $setting_field['fields'] ) ) {
-					foreach ( $setting_field['fields'] as $field ) {
-						if ( empty( $field['data_type'] ) || 'price' !== $field['data_type'] ) {
+				if ( ! empty($setting_field['fields'])) {
+					foreach ($setting_field['fields'] as $field) {
+						if (empty($field['data_type']) || 'price' !== $field['data_type']) {
 							continue;
 						}
 
-						foreach ( $meta_value as $index => $meta_data ) {
-							if ( ! isset( $meta_value[ $index ][ $field['id'] ] ) ) {
+						foreach ($meta_value as $index => $meta_data) {
+							if ( ! isset($meta_value[$index][$field['id']])) {
 								continue;
 							}
 
-							$meta_value[ $index ][ $field['id'] ] = ! empty( $meta_value[ $index ][ $field['id'] ] ) ?
-								give_sanitize_amount_for_db( $meta_value[ $index ][ $field['id'] ] ) :
-								( ( '_give_amount' === $field['id'] && empty( $field_value ) ) ?
-									give_sanitize_amount_for_db( '1.00' ) :
-									0 );
+							$meta_value[$index][$field['id']] = ! empty($meta_value[$index][$field['id']]) ?
+								give_sanitize_amount_for_db($meta_value[$index][$field['id']]) : (('_give_amount' === $field['id'] && empty($field_value)) ?
+									give_sanitize_amount_for_db('1.00') : 0);
 						}
 					}
 				}
 				break;
 
 			default:
-				if ( ! empty( $setting_field['data_type'] ) && 'price' === $setting_field['data_type'] ) {
+				if ( ! empty($setting_field['data_type']) && 'price' === $setting_field['data_type']) {
 					$meta_value = $meta_value ?
-						give_sanitize_amount_for_db( $meta_value ) :
-						( in_array( $setting_field['id'], array( '_give_set_price', '_give_custom_amount_minimum', '_give_set_goal' ) ) ?
-							give_sanitize_amount_for_db( '1.00' ) :
-							0 );
+						give_sanitize_amount_for_db($meta_value) : (in_array($setting_field['id'], array('_give_set_price', '_give_custom_amount_minimum', '_give_set_goal')) ?
+							give_sanitize_amount_for_db('1.00') : 0);
 				}
 		}
 
@@ -1171,12 +1167,12 @@ 
 	 *
 	 * @return string The URL after redirect.
 	 */
-	public function maintain_active_tab( $location, $post_id ) {
+	public function maintain_active_tab($location, $post_id) {
 		if (
-			'give_forms' === get_post_type( $post_id ) &&
-			! empty( $_POST['give_form_active_tab'] )
+			'give_forms' === get_post_type($post_id) &&
+			! empty($_POST['give_form_active_tab'])
 		) {
-			$location = add_query_arg( 'give_tab', give_clean( $_POST['give_form_active_tab'] ), $location );
+			$location = add_query_arg('give_tab', give_clean($_POST['give_form_active_tab']), $location);
 		}
 
 		return $location;

includes/admin/class-blank-slate.php

Spacing +55 added lines, -55 removed lines

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

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

Braces +4 added lines, -1 removed lines

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

Spacing +19 added lines, -19 removed lines

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

includes/admin/views/blank-slate.php

Spacing +10 added lines, -10 removed lines

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

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

Spacing +13 added lines, -13 removed lines

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

includes/payments/actions.php

Spacing +57 added lines, -57 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -28,21 +28,21 @@ 
  *
  * @return void
  */
-function give_complete_purchase( $payment_id, $new_status, $old_status ) {
+function give_complete_purchase($payment_id, $new_status, $old_status) {
 
 	// Make sure that payments are only completed once.
-	if ( $old_status == 'publish' || $old_status == 'complete' ) {
+	if ($old_status == 'publish' || $old_status == 'complete') {
 		return;
 	}
 
 	// Make sure the payment completion is only processed when new status is complete.
-	if ( $new_status != 'publish' && $new_status != 'complete' ) {
+	if ($new_status != 'publish' && $new_status != 'complete') {
 		return;
 	}
 
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	$creation_date  = get_post_field( 'post_date', $payment_id, 'raw' );
+	$creation_date  = get_post_field('post_date', $payment_id, 'raw');
 	$payment_meta   = $payment->payment_meta;
 	$completed_date = $payment->completed_date;
 	$user_info      = $payment->user_info;
@@ -58,12 +58,12 @@ 
 	 *
 	 * @param int $payment_id The ID of the payment.
 	 */
-	do_action( 'give_pre_complete_donation', $payment_id );
+	do_action('give_pre_complete_donation', $payment_id);
 
 	// Ensure these actions only run once, ever.
-	if ( empty( $completed_date ) ) {
+	if (empty($completed_date)) {
 
-		give_record_donation_in_log( $form_id, $payment_id, $price_id, $creation_date );
+		give_record_donation_in_log($form_id, $payment_id, $price_id, $creation_date);
 
 		/**
 		 * Fires after logging donation record.
@@ -74,29 +74,29 @@ 
 		 * @param int $payment_id The ID number of the payment.
 		 * @param array $payment_meta The payment meta.
 		 */
-		do_action( 'give_complete_form_donation', $form_id, $payment_id, $payment_meta );
+		do_action('give_complete_form_donation', $form_id, $payment_id, $payment_meta);
 
 	}
 
 	// Increase the earnings for this form ID.
-	give_increase_earnings( $form_id, $amount );
-	give_increase_donation_count( $form_id );
+	give_increase_earnings($form_id, $amount);
+	give_increase_donation_count($form_id);
 
 	// @todo: Refresh only range related stat cache
 	give_delete_donation_stats();
 
 	// Increase the donor's donation stats.
-	$donor = new Give_Donor( $donor_id );
+	$donor = new Give_Donor($donor_id);
 	$donor->increase_purchase_count();
-	$donor->increase_value( $amount );
+	$donor->increase_value($amount);
 
-	give_increase_total_earnings( $amount );
+	give_increase_total_earnings($amount);
 
 	// Ensure this action only runs once ever.
-	if ( empty( $completed_date ) ) {
+	if (empty($completed_date)) {
 
 		// Save the completed date.
-		$payment->completed_date = current_time( 'mysql' );
+		$payment->completed_date = current_time('mysql');
 		$payment->save();
 
 		/**
@@ -106,12 +106,12 @@ 
 		 *
 		 * @param int $payment_id The ID of the payment.
 		 */
-		do_action( 'give_complete_donation', $payment_id );
+		do_action('give_complete_donation', $payment_id);
 	}
 
 }
 
-add_action( 'give_update_payment_status', 'give_complete_purchase', 100, 3 );
+add_action('give_update_payment_status', 'give_complete_purchase', 100, 3);
 
 
 /**
@@ -125,20 +125,20 @@ 
  *
  * @return void
  */
-function give_record_status_change( $payment_id, $new_status, $old_status ) {
+function give_record_status_change($payment_id, $new_status, $old_status) {
 
 	// Get the list of statuses so that status in the payment note can be translated.
 	$stati      = give_get_payment_statuses();
-	$old_status = isset( $stati[ $old_status ] ) ? $stati[ $old_status ] : $old_status;
-	$new_status = isset( $stati[ $new_status ] ) ? $stati[ $new_status ] : $new_status;
+	$old_status = isset($stati[$old_status]) ? $stati[$old_status] : $old_status;
+	$new_status = isset($stati[$new_status]) ? $stati[$new_status] : $new_status;
 
 	// translators: 1: old status 2: new status.
-	$status_change = sprintf( esc_html__( 'Status changed from %1$s to %2$s.', 'give' ), $old_status, $new_status );
+	$status_change = sprintf(esc_html__('Status changed from %1$s to %2$s.', 'give'), $old_status, $new_status);
 
-	give_insert_payment_note( $payment_id, $status_change );
+	give_insert_payment_note($payment_id, $status_change);
 }
 
-add_action( 'give_update_payment_status', 'give_record_status_change', 100, 3 );
+add_action('give_update_payment_status', 'give_record_status_change', 100, 3);
 
 
 /**
@@ -154,25 +154,25 @@ 
  *
  * @return void
  */
-function give_update_old_payments_with_totals( $data ) {
-	if ( ! wp_verify_nonce( $data['_wpnonce'], 'give_upgrade_payments_nonce' ) ) {
+function give_update_old_payments_with_totals($data) {
+	if ( ! wp_verify_nonce($data['_wpnonce'], 'give_upgrade_payments_nonce')) {
 		return;
 	}
 
-	if ( get_option( 'give_payment_totals_upgraded' ) ) {
+	if (get_option('give_payment_totals_upgraded')) {
 		return;
 	}
 
-	$payments = give_get_payments( array(
+	$payments = give_get_payments(array(
 		'offset' => 0,
-		'number' => - 1,
+		'number' => -1,
 		'mode'   => 'all',
-	) );
+	));
 
-	if ( $payments ) {
-		foreach ( $payments as $payment ) {
+	if ($payments) {
+		foreach ($payments as $payment) {
 
-			$payment = new Give_Payment( $payment->ID );
+			$payment = new Give_Payment($payment->ID);
 			$meta    = $payment->get_meta();
 
 			$payment->total = $meta['amount'];
@@ -181,10 +181,10 @@ 
 		}
 	}
 
-	add_option( 'give_payment_totals_upgraded', 1 );
+	add_option('give_payment_totals_upgraded', 1);
 }
 
-add_action( 'give_upgrade_payments', 'give_update_old_payments_with_totals' );
+add_action('give_upgrade_payments', 'give_update_old_payments_with_totals');
 
 /**
  * Mark Abandoned Donations
@@ -198,17 +198,17 @@ 
 function give_mark_abandoned_donations() {
 	$args = array(
 		'status' => 'pending',
-		'number' => - 1,
+		'number' => -1,
 		'output' => 'give_payments',
 	);
 
-	add_filter( 'posts_where', 'give_filter_where_older_than_week' );
+	add_filter('posts_where', 'give_filter_where_older_than_week');
 
-	$payments = give_get_payments( $args );
+	$payments = give_get_payments($args);
 
-	remove_filter( 'posts_where', 'give_filter_where_older_than_week' );
+	remove_filter('posts_where', 'give_filter_where_older_than_week');
 
-	if ( $payments ) {
+	if ($payments) {
 		/**
 		 * Filter payment gateways:  Used to set payment gateways which can be skip while transferring pending payment to abandon.
 		 *
@@ -216,13 +216,13 @@ 
 		 *
 		 * @param array $skip_payment_gateways Array of payment gateways
 		 */
-		$skip_payment_gateways = apply_filters( 'give_mark_abandoned_donation_gateways', array( 'offline' ) );
+		$skip_payment_gateways = apply_filters('give_mark_abandoned_donation_gateways', array('offline'));
 
-		foreach ( $payments as $payment ) {
-			$gateway = give_get_payment_gateway( $payment );
+		foreach ($payments as $payment) {
+			$gateway = give_get_payment_gateway($payment);
 
 			// Skip payment gateways.
-			if ( in_array( $gateway, $skip_payment_gateways ) ) {
+			if (in_array($gateway, $skip_payment_gateways)) {
 				continue;
 			}
 
@@ -232,7 +232,7 @@ 
 	}
 }
 
-Give_Cron::add_weekly_event( 'give_mark_abandoned_donations' );
+Give_Cron::add_weekly_event('give_mark_abandoned_donations');
 
 
 /**
@@ -244,15 +244,15 @@ 
  *
  * @return void
  */
-function give_refresh_thismonth_stat_transients( $payment_ID ) {
+function give_refresh_thismonth_stat_transients($payment_ID) {
 	// Monthly stats.
-	Give_Cache::delete( Give_Cache::get_key( 'give_estimated_monthly_stats' ) );
+	Give_Cache::delete(Give_Cache::get_key('give_estimated_monthly_stats'));
 
 	// @todo: Refresh only range related stat cache
 	give_delete_donation_stats();
 }
 
-add_action( 'save_post_give_payment', 'give_refresh_thismonth_stat_transients' );
+add_action('save_post_give_payment', 'give_refresh_thismonth_stat_transients');
 
 /*
  * Add meta in payment that store page id and page url.
@@ -264,18 +264,18 @@ 
  *
  * @param {integer} $payment_id Payment id for which the meta value should be updated.
  */
-function give_payment_save_page_data( $payment_id ) {
-	$page_url = ( ! empty( $_REQUEST['give-current-url'] ) ? esc_url( $_REQUEST['give-current-url'] ) : false );
+function give_payment_save_page_data($payment_id) {
+	$page_url = ( ! empty($_REQUEST['give-current-url']) ? esc_url($_REQUEST['give-current-url']) : false);
 	// Check $page_url is not empty.
-	if ( $page_url ) {
-		update_post_meta( $payment_id, '_give_current_url', $page_url );
-		$page_id = url_to_postid( $page_url );
+	if ($page_url) {
+		update_post_meta($payment_id, '_give_current_url', $page_url);
+		$page_id = url_to_postid($page_url);
 		// Check $page_id is not empty.
-		if ( $page_id ) {
-			update_post_meta( $payment_id, '_give_current_page_id', $page_id );
+		if ($page_id) {
+			update_post_meta($payment_id, '_give_current_page_id', $page_id);
 		}
 	}
 }
 
 // Fire when payment is save.
-add_action( 'give_insert_payment', 'give_payment_save_page_data' );
\ No newline at end of file
+add_action('give_insert_payment', 'give_payment_save_page_data');
\ No newline at end of file

includes/payments/class-give-payment.php

Indentation +1 added lines, -1 removed lines

@@ -1012,7 +1012,7 @@
 					// Find a match between price_id and level_id.
 					// First verify array keys exists THEN make the match.
 					if ( ( isset( $args['price_id'] ) && isset( $price['_give_id']['level_id'] ) )
-					     && $args['price_id'] == $price['_give_id']['level_id']
+						 && $args['price_id'] == $price['_give_id']['level_id']
 					) {
 						$donation_amount = $price['_give_amount'];
 					}

Doc Comments +2 added lines, -2 removed lines

@@ -391,7 +391,7 @@ 
 	 *
 	 * @param  int|bool $payment_id A given payment
 	 *
-	 * @return mixed void|false
+	 * @return false|null void|false
 	 */
 	public function __construct( $payment_id = false ) {
 
@@ -1115,7 +1115,7 @@ 
 	 *
 	 * @param  string|bool $note The note to add
 	 *
-	 * @return bool           If the note was specified or not
+	 * @return false|null           If the note was specified or not
 	 */
 	public function add_note( $note = false ) {
 		// Bail if no note specified.

Spacing +287 added lines, -287 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -393,13 +393,13 @@ 
 	 *
 	 * @return mixed void|false
 	 */
-	public function __construct( $payment_id = false ) {
+	public function __construct($payment_id = false) {
 
-		if ( empty( $payment_id ) ) {
+		if (empty($payment_id)) {
 			return false;
 		}
 
-		$this->setup_payment( $payment_id );
+		$this->setup_payment($payment_id);
 	}
 
 	/**
@@ -412,11 +412,11 @@ 
 	 *
 	 * @return mixed        The value.
 	 */
-	public function __get( $key ) {
+	public function __get($key) {
 
-		if ( method_exists( $this, 'get_' . $key ) ) {
+		if (method_exists($this, 'get_'.$key)) {
 
-			$value = call_user_func( array( $this, 'get_' . $key ) );
+			$value = call_user_func(array($this, 'get_'.$key));
 
 		} else {
 
@@ -438,18 +438,18 @@ 
 	 * @param  string $key   The property name
 	 * @param  mixed  $value The value of the property
 	 */
-	public function __set( $key, $value ) {
-		$ignore = array( '_ID' );
+	public function __set($key, $value) {
+		$ignore = array('_ID');
 
-		if ( 'status' === $key ) {
+		if ('status' === $key) {
 			$this->old_status = $this->status;
 		}
 
-		if ( ! in_array( $key, $ignore ) ) {
-			$this->pending[ $key ] = $value;
+		if ( ! in_array($key, $ignore)) {
+			$this->pending[$key] = $value;
 		}
 
-		if ( '_ID' !== $key ) {
+		if ('_ID' !== $key) {
 			$this->$key = $value;
 		}
 	}
@@ -464,9 +464,9 @@ 
 	 *
 	 * @return boolean|null       If the item is set or not
 	 */
-	public function __isset( $name ) {
-		if ( property_exists( $this, $name ) ) {
-			return false === empty( $this->$name );
+	public function __isset($name) {
+		if (property_exists($this, $name)) {
+			return false === empty($this->$name);
 		} else {
 			return null;
 		}
@@ -482,20 +482,20 @@ 
 	 *
 	 * @return bool            If the setup was successful or not
 	 */
-	private function setup_payment( $payment_id ) {
+	private function setup_payment($payment_id) {
 		$this->pending = array();
 
-		if ( empty( $payment_id ) ) {
+		if (empty($payment_id)) {
 			return false;
 		}
 
-		$payment = get_post( $payment_id );
+		$payment = get_post($payment_id);
 
-		if ( ! $payment || is_wp_error( $payment ) ) {
+		if ( ! $payment || is_wp_error($payment)) {
 			return false;
 		}
 
-		if ( 'give_payment' !== $payment->post_type ) {
+		if ('give_payment' !== $payment->post_type) {
 			return false;
 		}
 
@@ -509,13 +509,13 @@ 
 		 * @param Give_Payment $this       Payment object.
 		 * @param int          $payment_id The ID of the payment.
 		 */
-		do_action( 'give_pre_setup_payment', $this, $payment_id );
+		do_action('give_pre_setup_payment', $this, $payment_id);
 
 		// Primary Identifier.
-		$this->ID = absint( $payment_id );
+		$this->ID = absint($payment_id);
 
 		// Protected ID that can never be changed.
-		$this->_ID = absint( $payment_id );
+		$this->_ID = absint($payment_id);
 
 		// We have a payment, get the generic payment_meta item to reduce calls to it.
 		$this->payment_meta = $this->get_meta();
@@ -531,7 +531,7 @@ 
 		$this->parent_payment = $payment->post_parent;
 
 		$all_payment_statuses  = give_get_payment_statuses();
-		$this->status_nicename = array_key_exists( $this->status, $all_payment_statuses ) ? $all_payment_statuses[ $this->status ] : ucfirst( $this->status );
+		$this->status_nicename = array_key_exists($this->status, $all_payment_statuses) ? $all_payment_statuses[$this->status] : ucfirst($this->status);
 
 		// Currency Based.
 		$this->total    = $this->setup_total();
@@ -570,7 +570,7 @@ 
 		 * @param Give_Payment $this       Payment object.
 		 * @param int          $payment_id The ID of the payment.
 		 */
-		do_action( 'give_setup_payment', $this, $payment_id );
+		do_action('give_setup_payment', $this, $payment_id);
 
 		return true;
 	}
@@ -588,8 +588,8 @@ 
 	 *
 	 * @return void
 	 */
-	public function update_payment_setup( $payment_id ) {
-		$this->setup_payment( $payment_id );
+	public function update_payment_setup($payment_id) {
+		$this->setup_payment($payment_id);
 	}
 
 	/**
@@ -604,24 +604,24 @@ 
 
 		// Construct the payment title.
 		$payment_title = '';
-		if ( ! empty( $this->first_name ) && ! empty( $this->last_name ) ) {
-			$payment_title = $this->first_name . ' ' . $this->last_name;
-		} elseif ( ! empty( $this->first_name ) && empty( $this->last_name ) ) {
+		if ( ! empty($this->first_name) && ! empty($this->last_name)) {
+			$payment_title = $this->first_name.' '.$this->last_name;
+		} elseif ( ! empty($this->first_name) && empty($this->last_name)) {
 			$payment_title = $this->first_name;
-		} elseif ( ! empty( $this->email ) && is_email( $this->email ) ) {
+		} elseif ( ! empty($this->email) && is_email($this->email)) {
 			$payment_title = $this->email;
 		}
 
 		// Set Key.
-		if ( empty( $this->key ) ) {
+		if (empty($this->key)) {
 
-			$auth_key             = defined( 'AUTH_KEY' ) ? AUTH_KEY : '';
-			$this->key            = strtolower( md5( $this->email . date( 'Y-m-d H:i:s' ) . $auth_key . uniqid( 'give', true ) ) );  // Unique key
+			$auth_key             = defined('AUTH_KEY') ? AUTH_KEY : '';
+			$this->key            = strtolower(md5($this->email.date('Y-m-d H:i:s').$auth_key.uniqid('give', true))); // Unique key
 			$this->pending['key'] = $this->key;
 		}
 
 		// Set IP.
-		if ( empty( $this->ip ) ) {
+		if (empty($this->ip)) {
 
 			$this->ip            = give_get_ip();
 			$this->pending['ip'] = $this->ip;
@@ -648,19 +648,19 @@ 
 			'status'       => $this->status,
 		);
 
-		$args = apply_filters( 'give_insert_payment_args', array(
+		$args = apply_filters('give_insert_payment_args', array(
 			'post_title'    => $payment_title,
 			'post_status'   => $this->status,
 			'post_type'     => 'give_payment',
-			'post_date'     => ! empty( $this->date ) ? $this->date : null,
-			'post_date_gmt' => ! empty( $this->date ) ? get_gmt_from_date( $this->date ) : null,
+			'post_date'     => ! empty($this->date) ? $this->date : null,
+			'post_date_gmt' => ! empty($this->date) ? get_gmt_from_date($this->date) : null,
 			'post_parent'   => $this->parent_payment,
-		), $payment_data );
+		), $payment_data);
 
 		// Create a blank payment
-		$payment_id = wp_insert_post( $args );
+		$payment_id = wp_insert_post($args);
 
-		if ( ! empty( $payment_id ) ) {
+		if ( ! empty($payment_id)) {
 
 			$this->ID  = $payment_id;
 			$this->_ID = $payment_id;
@@ -672,40 +672,40 @@ 
 			 *
 			 * @since 1.8.13
 			 */
-			$donor = apply_filters( 'give_update_donor_information', $donor, $payment_id, $payment_data, $args );
+			$donor = apply_filters('give_update_donor_information', $donor, $payment_id, $payment_data, $args);
 
-			if ( did_action( 'give_pre_process_donation' ) && is_user_logged_in() ) {
-				$donor = new Give_Donor( get_current_user_id(), true );
+			if (did_action('give_pre_process_donation') && is_user_logged_in()) {
+				$donor = new Give_Donor(get_current_user_id(), true);
 
 				// Donor is logged in but used a different email to purchase with so assign to their donor record.
-				if ( ! empty( $donor->id ) && $this->email !== $donor->email ) {
-					$donor->add_email( $this->email );
+				if ( ! empty($donor->id) && $this->email !== $donor->email) {
+					$donor->add_email($this->email);
 				}
 			}
 
-			if ( empty( $donor->id ) ) {
-				$donor = new Give_Donor( $this->email );
+			if (empty($donor->id)) {
+				$donor = new Give_Donor($this->email);
 			}
 
-			if ( empty( $donor->id ) ) {
+			if (empty($donor->id)) {
 
 				$donor_data = array(
-					'name'    => ! is_email( $payment_title ) ? $this->first_name . ' ' . $this->last_name : '',
+					'name'    => ! is_email($payment_title) ? $this->first_name.' '.$this->last_name : '',
 					'email'   => $this->email,
 					'user_id' => $this->user_id,
 				);
 
-				$donor->create( $donor_data );
+				$donor->create($donor_data);
 
 			}
 
 			$this->customer_id            = $donor->id;
 			$this->pending['customer_id'] = $this->customer_id;
-			$donor->attach_payment( $this->ID, false );
+			$donor->attach_payment($this->ID, false);
 
-			$this->payment_meta = apply_filters( 'give_payment_meta', $this->payment_meta, $payment_data );
+			$this->payment_meta = apply_filters('give_payment_meta', $this->payment_meta, $payment_data);
 
-			$this->update_meta( '_give_payment_meta', $this->payment_meta );
+			$this->update_meta('_give_payment_meta', $this->payment_meta);
 			$this->new = true;
 		}// End if().
 
@@ -727,11 +727,11 @@ 
 		$saved = false;
 
 		// Must have an ID.
-		if ( empty( $this->ID ) ) {
+		if (empty($this->ID)) {
 
 			$payment_id = $this->insert_payment();
 
-			if ( false === $payment_id ) {
+			if (false === $payment_id) {
 				$saved = false;
 			} else {
 				$this->ID = $payment_id;
@@ -739,42 +739,42 @@ 
 		}
 
 		// Set ID if not matching.
-		if ( $this->ID !== $this->_ID ) {
+		if ($this->ID !== $this->_ID) {
 			$this->ID = $this->_ID;
 		}
 
 		// If we have something pending, let's save it.
-		if ( ! empty( $this->pending ) ) {
+		if ( ! empty($this->pending)) {
 
 			$total_increase = 0;
 			$total_decrease = 0;
 
-			foreach ( $this->pending as $key => $value ) {
+			foreach ($this->pending as $key => $value) {
 
-				switch ( $key ) {
+				switch ($key) {
 
 					case 'donations':
 						// Update totals for pending donations.
-						foreach ( $this->pending[ $key ] as $item ) {
+						foreach ($this->pending[$key] as $item) {
 
-							$quantity = isset( $item['quantity'] ) ? $item['quantity'] : 1;
-							$price_id = isset( $item['price_id'] ) ? $item['price_id'] : 0;
+							$quantity = isset($item['quantity']) ? $item['quantity'] : 1;
+							$price_id = isset($item['price_id']) ? $item['price_id'] : 0;
 
-							switch ( $item['action'] ) {
+							switch ($item['action']) {
 
 								case 'add':
 
 									$price = $item['price'];
 
-									if ( 'publish' === $this->status || 'complete' === $this->status ) {
+									if ('publish' === $this->status || 'complete' === $this->status) {
 
 										// Add donation to logs.
-										$log_date = date_i18n( 'Y-m-d G:i:s', current_time( 'timestamp' ) );
-										give_record_donation_in_log( $item['id'], $this->ID, $price_id, $log_date );
+										$log_date = date_i18n('Y-m-d G:i:s', current_time('timestamp'));
+										give_record_donation_in_log($item['id'], $this->ID, $price_id, $log_date);
 
-										$form = new Give_Donate_Form( $item['id'] );
-										$form->increase_sales( $quantity );
-										$form->increase_earnings( $price );
+										$form = new Give_Donate_Form($item['id']);
+										$form->increase_sales($quantity);
+										$form->increase_earnings($price);
 
 										$total_increase += $price;
 									}
@@ -799,15 +799,15 @@ 
 										),
 									);
 
-									$found_logs = get_posts( $log_args );
-									foreach ( $found_logs as $log ) {
-										wp_delete_post( $log->ID, true );
+									$found_logs = get_posts($log_args);
+									foreach ($found_logs as $log) {
+										wp_delete_post($log->ID, true);
 									}
 
-									if ( 'publish' === $this->status || 'complete' === $this->status ) {
-										$form = new Give_Donate_Form( $item['id'] );
-										$form->decrease_sales( $quantity );
-										$form->decrease_earnings( $item['amount'] );
+									if ('publish' === $this->status || 'complete' === $this->status) {
+										$form = new Give_Donate_Form($item['id']);
+										$form->decrease_sales($quantity);
+										$form->decrease_earnings($item['amount']);
 
 										$total_decrease += $item['amount'];
 									}
@@ -818,44 +818,44 @@ 
 						break;
 
 					case 'status':
-						$this->update_status( $this->status );
+						$this->update_status($this->status);
 						break;
 
 					case 'gateway':
-						$this->update_meta( '_give_payment_gateway', $this->gateway );
+						$this->update_meta('_give_payment_gateway', $this->gateway);
 						break;
 
 					case 'mode':
-						$this->update_meta( '_give_payment_mode', $this->mode );
+						$this->update_meta('_give_payment_mode', $this->mode);
 						break;
 
 					case 'transaction_id':
-						$this->update_meta( '_give_payment_transaction_id', $this->transaction_id );
+						$this->update_meta('_give_payment_transaction_id', $this->transaction_id);
 						break;
 
 					case 'ip':
-						$this->update_meta( '_give_payment_user_ip', $this->ip );
+						$this->update_meta('_give_payment_user_ip', $this->ip);
 						break;
 
 					case 'customer_id':
-						$this->update_meta( '_give_payment_customer_id', $this->customer_id );
+						$this->update_meta('_give_payment_customer_id', $this->customer_id);
 						break;
 
 					case 'user_id':
-						$this->update_meta( '_give_payment_user_id', $this->user_id );
+						$this->update_meta('_give_payment_user_id', $this->user_id);
 						$this->user_info['id'] = $this->user_id;
 						break;
 
 					case 'form_title':
-						$this->update_meta( '_give_payment_form_title', $this->form_title );
+						$this->update_meta('_give_payment_form_title', $this->form_title);
 						break;
 
 					case 'form_id':
-						$this->update_meta( '_give_payment_form_id', $this->form_id );
+						$this->update_meta('_give_payment_form_id', $this->form_id);
 						break;
 
 					case 'price_id':
-						$this->update_meta( '_give_payment_price_id', $this->price_id );
+						$this->update_meta('_give_payment_price_id', $this->price_id);
 						break;
 
 					case 'first_name':
@@ -871,15 +871,15 @@ 
 						break;
 
 					case 'email':
-						$this->update_meta( '_give_payment_user_email', $this->email );
+						$this->update_meta('_give_payment_user_email', $this->email);
 						break;
 
 					case 'key':
-						$this->update_meta( '_give_payment_purchase_key', $this->key );
+						$this->update_meta('_give_payment_purchase_key', $this->key);
 						break;
 
 					case 'number':
-						$this->update_meta( '_give_payment_number', $this->number );
+						$this->update_meta('_give_payment_number', $this->number);
 						break;
 
 					case 'date':
@@ -889,11 +889,11 @@ 
 							'edit_date' => true,
 						);
 
-						wp_update_post( $args );
+						wp_update_post($args);
 						break;
 
 					case 'completed_date':
-						$this->update_meta( '_give_completed_date', $this->completed_date );
+						$this->update_meta('_give_completed_date', $this->completed_date);
 						break;
 
 					case 'parent_payment':
@@ -902,7 +902,7 @@ 
 							'post_parent' => $this->parent_payment,
 						);
 
-						wp_update_post( $args );
+						wp_update_post($args);
 						break;
 
 					default:
@@ -913,33 +913,33 @@ 
 						 *
 						 * @param Give_Payment $this Payment object.
 						 */
-						do_action( 'give_payment_save', $this, $key );
+						do_action('give_payment_save', $this, $key);
 						break;
 				}// End switch().
 			}// End foreach().
 
-			if ( 'pending' !== $this->status ) {
+			if ('pending' !== $this->status) {
 
-				$donor = new Give_Donor( $this->customer_id );
+				$donor = new Give_Donor($this->customer_id);
 
 				$total_change = $total_increase - $total_decrease;
-				if ( $total_change < 0 ) {
+				if ($total_change < 0) {
 
-					$total_change = - ( $total_change );
+					$total_change = - ($total_change);
 					// Decrease the donor's donation stats.
-					$donor->decrease_value( $total_change );
-					give_decrease_total_earnings( $total_change );
+					$donor->decrease_value($total_change);
+					give_decrease_total_earnings($total_change);
 
-				} elseif ( $total_change > 0 ) {
+				} elseif ($total_change > 0) {
 
 					// Increase the donor's donation stats.
-					$donor->increase_value( $total_change );
-					give_increase_total_earnings( $total_change );
+					$donor->increase_value($total_change);
+					give_increase_total_earnings($total_change);
 
 				}
 			}
 
-			$this->update_meta( '_give_payment_total', give_sanitize_amount_for_db( $this->total ) );
+			$this->update_meta('_give_payment_total', give_sanitize_amount_for_db($this->total));
 
 			$new_meta = array(
 				'form_title' => $this->form_title,
@@ -950,12 +950,12 @@ 
 			);
 
 			$meta        = $this->get_meta();
-			$merged_meta = array_merge( $meta, $new_meta );
+			$merged_meta = array_merge($meta, $new_meta);
 
 			// Only save the payment meta if it's changed.
-			if ( md5( serialize( $meta ) ) !== md5( serialize( $merged_meta ) ) ) {
-				$updated = $this->update_meta( '_give_payment_meta', $merged_meta );
-				if ( false !== $updated ) {
+			if (md5(serialize($meta)) !== md5(serialize($merged_meta))) {
+				$updated = $this->update_meta('_give_payment_meta', $merged_meta);
+				if (false !== $updated) {
 					$saved = true;
 				}
 			}
@@ -964,8 +964,8 @@ 
 			$saved         = true;
 		}// End if().
 
-		if ( true === $saved ) {
-			$this->setup_payment( $this->ID );
+		if (true === $saved) {
+			$this->setup_payment($this->ID);
 		}
 
 		return $saved;
@@ -983,12 +983,12 @@ 
 	 *
 	 * @return bool           True when successful, false otherwise
 	 */
-	public function add_donation( $form_id = 0, $args = array(), $options = array() ) {
+	public function add_donation($form_id = 0, $args = array(), $options = array()) {
 
-		$donation = new Give_Donate_Form( $form_id );
+		$donation = new Give_Donate_Form($form_id);
 
 		// Bail if this post isn't a give donation form.
-		if ( ! $donation || $donation->post_type !== 'give_forms' ) {
+		if ( ! $donation || $donation->post_type !== 'give_forms') {
 			return false;
 		}
 
@@ -998,59 +998,59 @@ 
 			'price_id' => false,
 		);
 
-		$args = wp_parse_args( apply_filters( 'give_payment_add_donation_args', $args, $donation->ID ), $defaults );
+		$args = wp_parse_args(apply_filters('give_payment_add_donation_args', $args, $donation->ID), $defaults);
 
 		// Allow overriding the price.
-		if ( false !== $args['price'] ) {
+		if (false !== $args['price']) {
 			$donation_amount = $args['price'];
 		} else {
 
 			// Deal with variable pricing.
-			if ( give_has_variable_prices( $donation->ID ) ) {
-				$prices          = give_get_meta( $form_id, '_give_donation_levels', true );
+			if (give_has_variable_prices($donation->ID)) {
+				$prices          = give_get_meta($form_id, '_give_donation_levels', true);
 				$donation_amount = '';
 				// Loop through prices.
-				foreach ( $prices as $price ) {
+				foreach ($prices as $price) {
 					// Find a match between price_id and level_id.
 					// First verify array keys exists THEN make the match.
-					if ( ( isset( $args['price_id'] ) && isset( $price['_give_id']['level_id'] ) )
+					if ((isset($args['price_id']) && isset($price['_give_id']['level_id']))
 					     && $args['price_id'] == $price['_give_id']['level_id']
 					) {
 						$donation_amount = $price['_give_amount'];
 					}
 				}
 				// Fallback to the lowest price point.
-				if ( $donation_amount == '' ) {
-					$donation_amount  = give_get_lowest_price_option( $donation->ID );
-					$args['price_id'] = give_get_lowest_price_id( $donation->ID );
+				if ($donation_amount == '') {
+					$donation_amount  = give_get_lowest_price_option($donation->ID);
+					$args['price_id'] = give_get_lowest_price_id($donation->ID);
 				}
 			} else {
 				// Simple form price.
-				$donation_amount = give_get_form_price( $donation->ID );
+				$donation_amount = give_get_form_price($donation->ID);
 			}
 		}
 
 		// Sanitizing the price here so we don't have a dozen calls later.
-		$donation_amount = give_maybe_sanitize_amount( $donation_amount );
-		$total           = round( $donation_amount, give_get_price_decimals( $this->ID ) );
+		$donation_amount = give_maybe_sanitize_amount($donation_amount);
+		$total           = round($donation_amount, give_get_price_decimals($this->ID));
 
 		// Add Options.
 		$default_options = array();
-		if ( false !== $args['price_id'] ) {
+		if (false !== $args['price_id']) {
 			$default_options['price_id'] = (int) $args['price_id'];
 		}
-		$options = wp_parse_args( $options, $default_options );
+		$options = wp_parse_args($options, $default_options);
 
 		// Do not allow totals to go negative.
-		if ( $total < 0 ) {
+		if ($total < 0) {
 			$total = 0;
 		}
 
 		$donation = array(
 			'name'     => $donation->post_title,
 			'id'       => $donation->ID,
-			'price'    => round( $total, give_get_price_decimals( $this->ID ) ),
-			'subtotal' => round( $total, give_get_price_decimals( $this->ID ) ),
+			'price'    => round($total, give_get_price_decimals($this->ID)),
+			'subtotal' => round($total, give_get_price_decimals($this->ID)),
 			'price_id' => $args['price_id'],
 			'action'   => 'add',
 			'options'  => $options,
@@ -1058,7 +1058,7 @@ 
 
 		$this->pending['donations'][] = $donation;
 
-		$this->increase_subtotal( $total );
+		$this->increase_subtotal($total);
 
 		return true;
 
@@ -1075,7 +1075,7 @@ 
 	 *
 	 * @return bool           If the item was removed or not
 	 */
-	public function remove_donation( $form_id, $args = array() ) {
+	public function remove_donation($form_id, $args = array()) {
 
 		// Set some defaults.
 		$defaults = array(
@@ -1083,12 +1083,12 @@ 
 			'price'    => false,
 			'price_id' => false,
 		);
-		$args     = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		$form = new Give_Donate_Form( $form_id );
+		$form = new Give_Donate_Form($form_id);
 
 		// Bail if this post isn't a valid give donation form.
-		if ( ! $form || $form->post_type !== 'give_forms' ) {
+		if ( ! $form || $form->post_type !== 'give_forms') {
 			return false;
 		}
 
@@ -1101,7 +1101,7 @@ 
 
 		$this->pending['donations'][] = $pending_args;
 
-		$this->decrease_subtotal( $this->total );
+		$this->decrease_subtotal($this->total);
 
 		return true;
 	}
@@ -1117,13 +1117,13 @@ 
 	 *
 	 * @return bool           If the note was specified or not
 	 */
-	public function add_note( $note = false ) {
+	public function add_note($note = false) {
 		// Bail if no note specified.
-		if ( ! $note ) {
+		if ( ! $note) {
 			return false;
 		}
 
-		give_insert_payment_note( $this->ID, $note );
+		give_insert_payment_note($this->ID, $note);
 	}
 
 	/**
@@ -1136,8 +1136,8 @@ 
 	 *
 	 * @return void
 	 */
-	private function increase_subtotal( $amount = 0.00 ) {
-		$amount         = (float) $amount;
+	private function increase_subtotal($amount = 0.00) {
+		$amount = (float) $amount;
 		$this->subtotal += $amount;
 
 		$this->recalculate_total();
@@ -1153,11 +1153,11 @@ 
 	 *
 	 * @return void
 	 */
-	private function decrease_subtotal( $amount = 0.00 ) {
-		$amount         = (float) $amount;
+	private function decrease_subtotal($amount = 0.00) {
+		$amount = (float) $amount;
 		$this->subtotal -= $amount;
 
-		if ( $this->subtotal < 0 ) {
+		if ($this->subtotal < 0) {
 			$this->subtotal = 0;
 		}
 
@@ -1186,24 +1186,24 @@ 
 	 *
 	 * @return bool   $updated Returns if the status was successfully updated.
 	 */
-	public function update_status( $status = false ) {
+	public function update_status($status = false) {
 
 		// standardize the 'complete(d)' status.
-		if ( $status == 'completed' || $status == 'complete' ) {
+		if ($status == 'completed' || $status == 'complete') {
 			$status = 'publish';
 		}
 
-		$old_status = ! empty( $this->old_status ) ? $this->old_status : false;
+		$old_status = ! empty($this->old_status) ? $this->old_status : false;
 
-		if ( $old_status === $status ) {
+		if ($old_status === $status) {
 			return false; // Don't permit status changes that aren't changes.
 		}
 
-		$do_change = apply_filters( 'give_should_update_payment_status', true, $this->ID, $status, $old_status );
+		$do_change = apply_filters('give_should_update_payment_status', true, $this->ID, $status, $old_status);
 
 		$updated = false;
 
-		if ( $do_change ) {
+		if ($do_change) {
 
 			/**
 			 * Fires before changing payment status.
@@ -1214,21 +1214,21 @@ 
 			 * @param string $status     The new status.
 			 * @param string $old_status The old status.
 			 */
-			do_action( 'give_before_payment_status_change', $this->ID, $status, $old_status );
+			do_action('give_before_payment_status_change', $this->ID, $status, $old_status);
 
 			$update_fields = array(
 				'ID'          => $this->ID,
 				'post_status' => $status,
-				'edit_date'   => current_time( 'mysql' ),
+				'edit_date'   => current_time('mysql'),
 			);
 
-			$updated = wp_update_post( apply_filters( 'give_update_payment_status_fields', $update_fields ) );
+			$updated = wp_update_post(apply_filters('give_update_payment_status_fields', $update_fields));
 
 			$all_payment_statuses  = give_get_payment_statuses();
-			$this->status_nicename = array_key_exists( $status, $all_payment_statuses ) ? $all_payment_statuses[ $status ] : ucfirst( $status );
+			$this->status_nicename = array_key_exists($status, $all_payment_statuses) ? $all_payment_statuses[$status] : ucfirst($status);
 
 			// Process any specific status functions.
-			switch ( $status ) {
+			switch ($status) {
 				case 'refunded':
 					$this->process_refund();
 					break;
@@ -1255,7 +1255,7 @@ 
 			 * @param string $status     The new status.
 			 * @param string $old_status The old status.
 			 */
-			do_action( 'give_update_payment_status', $this->ID, $status, $old_status );
+			do_action('give_update_payment_status', $this->ID, $status, $old_status);
 
 		}// End if().
 
@@ -1290,41 +1290,41 @@ 
 	 *
 	 * @return mixed             The value from the post meta
 	 */
-	public function get_meta( $meta_key = '_give_payment_meta', $single = true ) {
+	public function get_meta($meta_key = '_give_payment_meta', $single = true) {
 
-		$meta = give_get_meta( $this->ID, $meta_key, $single );
+		$meta = give_get_meta($this->ID, $meta_key, $single);
 
-		if ( $meta_key === '_give_payment_meta' ) {
+		if ($meta_key === '_give_payment_meta') {
 			$meta = (array) $meta;
 
-			if ( empty( $meta['key'] ) ) {
+			if (empty($meta['key'])) {
 				$meta['key'] = $this->setup_payment_key();
 			}
 
-			if ( empty( $meta['form_title'] ) ) {
+			if (empty($meta['form_title'])) {
 				$meta['form_title'] = $this->setup_form_title();
 			}
 
-			if ( empty( $meta['email'] ) ) {
+			if (empty($meta['email'])) {
 				$meta['email'] = $this->setup_email();
 			}
 
-			if ( empty( $meta['date'] ) ) {
-				$meta['date'] = get_post_field( 'post_date', $this->ID );
+			if (empty($meta['date'])) {
+				$meta['date'] = get_post_field('post_date', $this->ID);
 			}
 		}
 
-		$meta = apply_filters( "give_get_payment_meta_{$meta_key}", $meta, $this->ID );
+		$meta = apply_filters("give_get_payment_meta_{$meta_key}", $meta, $this->ID);
 
 		// Security check.
-		if ( is_serialized( $meta ) ) {
-			preg_match( '/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $meta, $matches );
-			if ( ! empty( $matches ) ) {
+		if (is_serialized($meta)) {
+			preg_match('/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $meta, $matches);
+			if ( ! empty($matches)) {
 				$meta = array();
 			}
 		}
 
-		return apply_filters( 'give_get_payment_meta', $meta, $this->ID, $meta_key );
+		return apply_filters('give_get_payment_meta', $meta, $this->ID, $meta_key);
 	}
 
 	/**
@@ -1339,23 +1339,23 @@ 
 	 *
 	 * @return int|bool           Meta ID if the key didn't exist, true on successful update, false on failure
 	 */
-	public function update_meta( $meta_key = '', $meta_value = '', $prev_value = '' ) {
-		if ( empty( $meta_key ) ) {
+	public function update_meta($meta_key = '', $meta_value = '', $prev_value = '') {
+		if (empty($meta_key)) {
 			return false;
 		}
 
-		if ( $meta_key == 'key' || $meta_key == 'date' ) {
+		if ($meta_key == 'key' || $meta_key == 'date') {
 
 			$current_meta              = $this->get_meta();
-			$current_meta[ $meta_key ] = $meta_value;
+			$current_meta[$meta_key] = $meta_value;
 
 			$meta_key   = '_give_payment_meta';
 			$meta_value = $current_meta;
 
-		} elseif ( $meta_key == 'email' || $meta_key == '_give_payment_user_email' ) {
+		} elseif ($meta_key == 'email' || $meta_key == '_give_payment_user_email') {
 
-			$meta_value = apply_filters( "give_update_payment_meta_{$meta_key}", $meta_value, $this->ID );
-			give_update_meta( $this->ID, '_give_payment_user_email', $meta_value );
+			$meta_value = apply_filters("give_update_payment_meta_{$meta_key}", $meta_value, $this->ID);
+			give_update_meta($this->ID, '_give_payment_user_email', $meta_value);
 
 			$current_meta                       = $this->get_meta();
 			$current_meta['user_info']['email'] = $meta_value;
@@ -1365,9 +1365,9 @@ 
 
 		}
 
-		$meta_value = apply_filters( "give_update_payment_meta_{$meta_key}", $meta_value, $this->ID );
+		$meta_value = apply_filters("give_update_payment_meta_{$meta_key}", $meta_value, $this->ID);
 
-		return give_update_meta( $this->ID, $meta_key, $meta_value, $prev_value );
+		return give_update_meta($this->ID, $meta_key, $meta_value, $prev_value);
 	}
 
 	/**
@@ -1382,14 +1382,14 @@ 
 		$process_refund = true;
 
 		// If the payment was not in publish or revoked status, don't decrement stats as they were never incremented.
-		if ( 'publish' != $this->old_status || 'refunded' != $this->status ) {
+		if ('publish' != $this->old_status || 'refunded' != $this->status) {
 			$process_refund = false;
 		}
 
 		// Allow extensions to filter for their own payment types, Example: Recurring Payments.
-		$process_refund = apply_filters( 'give_should_process_refund', $process_refund, $this );
+		$process_refund = apply_filters('give_should_process_refund', $process_refund, $this);
 
-		if ( false === $process_refund ) {
+		if (false === $process_refund) {
 			return;
 		}
 
@@ -1400,13 +1400,13 @@ 
 		 *
 		 * @param Give_Payment $this Payment object.
 		 */
-		do_action( 'give_pre_refund_payment', $this );
+		do_action('give_pre_refund_payment', $this);
 
-		$decrease_earnings       = apply_filters( 'give_decrease_store_earnings_on_refund', true, $this );
-		$decrease_customer_value = apply_filters( 'give_decrease_customer_value_on_refund', true, $this );
-		$decrease_purchase_count = apply_filters( 'give_decrease_customer_purchase_count_on_refund', true, $this );
+		$decrease_earnings       = apply_filters('give_decrease_store_earnings_on_refund', true, $this);
+		$decrease_customer_value = apply_filters('give_decrease_customer_value_on_refund', true, $this);
+		$decrease_purchase_count = apply_filters('give_decrease_customer_purchase_count_on_refund', true, $this);
 
-		$this->maybe_alter_stats( $decrease_earnings, $decrease_customer_value, $decrease_purchase_count );
+		$this->maybe_alter_stats($decrease_earnings, $decrease_customer_value, $decrease_purchase_count);
 		$this->delete_sales_logs();
 
 		// @todo: Refresh only range related stat cache
@@ -1419,7 +1419,7 @@ 
 		 *
 		 * @param Give_Payment $this Payment object.
 		 */
-		do_action( 'give_post_refund_payment', $this );
+		do_action('give_post_refund_payment', $this);
 	}
 
 	/**
@@ -1446,26 +1446,26 @@ 
 		$process_pending = true;
 
 		// If the payment was not in publish or revoked status, don't decrement stats as they were never incremented.
-		if ( 'publish' != $this->old_status || 'pending' != $this->status ) {
+		if ('publish' != $this->old_status || 'pending' != $this->status) {
 			$process_pending = false;
 		}
 
 		// Allow extensions to filter for their own payment types, Example: Recurring Payments.
-		$process_pending = apply_filters( 'give_should_process_pending', $process_pending, $this );
+		$process_pending = apply_filters('give_should_process_pending', $process_pending, $this);
 
-		if ( false === $process_pending ) {
+		if (false === $process_pending) {
 			return;
 		}
 
-		$decrease_earnings       = apply_filters( 'give_decrease_earnings_on_pending', true, $this );
-		$decrease_donor_value    = apply_filters( 'give_decrease_donor_value_on_pending', true, $this );
-		$decrease_donation_count = apply_filters( 'give_decrease_donors_donation_count_on_pending', true, $this );
+		$decrease_earnings       = apply_filters('give_decrease_earnings_on_pending', true, $this);
+		$decrease_donor_value    = apply_filters('give_decrease_donor_value_on_pending', true, $this);
+		$decrease_donation_count = apply_filters('give_decrease_donors_donation_count_on_pending', true, $this);
 
-		$this->maybe_alter_stats( $decrease_earnings, $decrease_donor_value, $decrease_donation_count );
+		$this->maybe_alter_stats($decrease_earnings, $decrease_donor_value, $decrease_donation_count);
 		$this->delete_sales_logs();
 
 		$this->completed_date = false;
-		$this->update_meta( '_give_completed_date', '' );
+		$this->update_meta('_give_completed_date', '');
 
 		// @todo: Refresh only range related stat cache
 		give_delete_donation_stats();
@@ -1483,26 +1483,26 @@ 
 		$process_cancelled = true;
 
 		// If the payment was not in publish or revoked status, don't decrement stats as they were never incremented.
-		if ( 'publish' != $this->old_status || 'cancelled' != $this->status ) {
+		if ('publish' != $this->old_status || 'cancelled' != $this->status) {
 			$process_cancelled = false;
 		}
 
 		// Allow extensions to filter for their own payment types, Example: Recurring Payments.
-		$process_cancelled = apply_filters( 'give_should_process_cancelled', $process_cancelled, $this );
+		$process_cancelled = apply_filters('give_should_process_cancelled', $process_cancelled, $this);
 
-		if ( false === $process_cancelled ) {
+		if (false === $process_cancelled) {
 			return;
 		}
 
-		$decrease_earnings       = apply_filters( 'give_decrease_earnings_on_cancelled', true, $this );
-		$decrease_donor_value    = apply_filters( 'give_decrease_donor_value_on_cancelled', true, $this );
-		$decrease_donation_count = apply_filters( 'give_decrease_donors_donation_count_on_cancelled', true, $this );
+		$decrease_earnings       = apply_filters('give_decrease_earnings_on_cancelled', true, $this);
+		$decrease_donor_value    = apply_filters('give_decrease_donor_value_on_cancelled', true, $this);
+		$decrease_donation_count = apply_filters('give_decrease_donors_donation_count_on_cancelled', true, $this);
 
-		$this->maybe_alter_stats( $decrease_earnings, $decrease_donor_value, $decrease_donation_count );
+		$this->maybe_alter_stats($decrease_earnings, $decrease_donor_value, $decrease_donation_count);
 		$this->delete_sales_logs();
 
 		$this->completed_date = false;
-		$this->update_meta( '_give_completed_date', '' );
+		$this->update_meta('_give_completed_date', '');
 
 		// @todo: Refresh only range related stat cache
 		give_delete_donation_stats();
@@ -1518,26 +1518,26 @@ 
 		$process_revoked = true;
 
 		// If the payment was not in publish, don't decrement stats as they were never incremented.
-		if ( 'publish' != $this->old_status || 'revoked' != $this->status ) {
+		if ('publish' != $this->old_status || 'revoked' != $this->status) {
 			$process_revoked = false;
 		}
 
 		// Allow extensions to filter for their own payment types, Example: Recurring Payments.
-		$process_revoked = apply_filters( 'give_should_process_revoked', $process_revoked, $this );
+		$process_revoked = apply_filters('give_should_process_revoked', $process_revoked, $this);
 
-		if ( false === $process_revoked ) {
+		if (false === $process_revoked) {
 			return;
 		}
 
-		$decrease_earnings       = apply_filters( 'give_decrease_earnings_on_revoked', true, $this );
-		$decrease_donor_value    = apply_filters( 'give_decrease_donor_value_on_revoked', true, $this );
-		$decrease_donation_count = apply_filters( 'give_decrease_donors_donation_count_on_revoked', true, $this );
+		$decrease_earnings       = apply_filters('give_decrease_earnings_on_revoked', true, $this);
+		$decrease_donor_value    = apply_filters('give_decrease_donor_value_on_revoked', true, $this);
+		$decrease_donation_count = apply_filters('give_decrease_donors_donation_count_on_revoked', true, $this);
 
-		$this->maybe_alter_stats( $decrease_earnings, $decrease_donor_value, $decrease_donation_count );
+		$this->maybe_alter_stats($decrease_earnings, $decrease_donor_value, $decrease_donation_count);
 		$this->delete_sales_logs();
 
 		$this->completed_date = false;
-		$this->update_meta( '_give_completed_date', '' );
+		$this->update_meta('_give_completed_date', '');
 
 		// @todo: Refresh only range related stat cache
 		give_delete_donation_stats();
@@ -1555,25 +1555,25 @@ 
 	 *
 	 * @return void
 	 */
-	private function maybe_alter_stats( $alter_store_earnings, $alter_customer_value, $alter_customer_purchase_count ) {
+	private function maybe_alter_stats($alter_store_earnings, $alter_customer_value, $alter_customer_purchase_count) {
 
-		give_undo_donation( $this->ID );
+		give_undo_donation($this->ID);
 
 		// Decrease store earnings.
-		if ( true === $alter_store_earnings ) {
-			give_decrease_total_earnings( $this->total );
+		if (true === $alter_store_earnings) {
+			give_decrease_total_earnings($this->total);
 		}
 
 		// Decrement the stats for the donor.
-		if ( ! empty( $this->customer_id ) ) {
+		if ( ! empty($this->customer_id)) {
 
-			$donor = new Give_Donor( $this->customer_id );
+			$donor = new Give_Donor($this->customer_id);
 
-			if ( true === $alter_customer_value ) {
-				$donor->decrease_value( $this->total );
+			if (true === $alter_customer_value) {
+				$donor->decrease_value($this->total);
 			}
 
-			if ( true === $alter_customer_purchase_count ) {
+			if (true === $alter_customer_purchase_count) {
 				$donor->decrease_donation_count();
 			}
 		}
@@ -1622,13 +1622,13 @@ 
 	 * @return string The date the payment was completed
 	 */
 	private function setup_completed_date() {
-		$payment = get_post( $this->ID );
+		$payment = get_post($this->ID);
 
-		if ( 'pending' == $payment->post_status || 'preapproved' == $payment->post_status ) {
+		if ('pending' == $payment->post_status || 'preapproved' == $payment->post_status) {
 			return false; // This payment was never completed.
 		}
 
-		$date = ( $date = $this->get_meta( '_give_completed_date', true ) ) ? $date : $payment->modified_date;
+		$date = ($date = $this->get_meta('_give_completed_date', true)) ? $date : $payment->modified_date;
 
 		return $date;
 	}
@@ -1642,7 +1642,7 @@ 
 	 * @return string The payment mode
 	 */
 	private function setup_mode() {
-		return $this->get_meta( '_give_payment_mode' );
+		return $this->get_meta('_give_payment_mode');
 	}
 
 	/**
@@ -1654,7 +1654,7 @@ 
 	 * @return bool The payment import
 	 */
 	private function setup_import() {
-		return (bool) $this->get_meta( '_give_payment_import' );
+		return (bool) $this->get_meta('_give_payment_import');
 	}
 
 	/**
@@ -1666,18 +1666,18 @@ 
 	 * @return float The payment total
 	 */
 	private function setup_total() {
-		$amount = $this->get_meta( '_give_payment_total', true );
+		$amount = $this->get_meta('_give_payment_total', true);
 
-		if ( empty( $amount ) && '0.00' != $amount ) {
-			$meta = $this->get_meta( '_give_payment_meta', true );
-			$meta = maybe_unserialize( $meta );
+		if (empty($amount) && '0.00' != $amount) {
+			$meta = $this->get_meta('_give_payment_meta', true);
+			$meta = maybe_unserialize($meta);
 
-			if ( isset( $meta['amount'] ) ) {
+			if (isset($meta['amount'])) {
 				$amount = $meta['amount'];
 			}
 		}
 
-		return round( (float) $amount, give_get_price_decimals( $this->ID ) );
+		return round((float) $amount, give_get_price_decimals($this->ID));
 	}
 
 	/**
@@ -1703,7 +1703,7 @@ 
 	 * @return string The currency for the payment
 	 */
 	private function setup_currency() {
-		$currency = ! empty( $this->payment_meta['currency'] ) ? $this->payment_meta['currency'] : apply_filters( 'give_payment_currency_default', give_get_currency( $this->form_id, $this ), $this );
+		$currency = ! empty($this->payment_meta['currency']) ? $this->payment_meta['currency'] : apply_filters('give_payment_currency_default', give_get_currency($this->form_id, $this), $this);
 
 		return $currency;
 	}
@@ -1717,7 +1717,7 @@ 
 	 * @return string The gateway
 	 */
 	private function setup_gateway() {
-		$gateway = $this->get_meta( '_give_payment_gateway', true );
+		$gateway = $this->get_meta('_give_payment_gateway', true);
 
 		return $gateway;
 	}
@@ -1731,11 +1731,11 @@ 
 	 * @return string The donation ID
 	 */
 	private function setup_transaction_id() {
-		$transaction_id = $this->get_meta( '_give_payment_transaction_id', true );
+		$transaction_id = $this->get_meta('_give_payment_transaction_id', true);
 
-		if ( empty( $transaction_id ) ) {
+		if (empty($transaction_id)) {
 			$gateway        = $this->gateway;
-			$transaction_id = apply_filters( "give_get_payment_transaction_id-{$gateway}", $this->ID );
+			$transaction_id = apply_filters("give_get_payment_transaction_id-{$gateway}", $this->ID);
 		}
 
 		return $transaction_id;
@@ -1750,7 +1750,7 @@ 
 	 * @return string The IP address for the payment
 	 */
 	private function setup_ip() {
-		$ip = $this->get_meta( '_give_payment_user_ip', true );
+		$ip = $this->get_meta('_give_payment_user_ip', true);
 
 		return $ip;
 	}
@@ -1764,7 +1764,7 @@ 
 	 * @return int The Donor ID.
 	 */
 	private function setup_donor_id() {
-		$customer_id = $this->get_meta( '_give_payment_customer_id', true );
+		$customer_id = $this->get_meta('_give_payment_customer_id', true);
 
 		return $customer_id;
 	}
@@ -1778,7 +1778,7 @@ 
 	 * @return int The User ID
 	 */
 	private function setup_user_id() {
-		$user_id = $this->get_meta( '_give_payment_user_id', true );
+		$user_id = $this->get_meta('_give_payment_user_id', true);
 
 		return $user_id;
 	}
@@ -1792,10 +1792,10 @@ 
 	 * @return string The email address for the payment.
 	 */
 	private function setup_email() {
-		$email = $this->get_meta( '_give_payment_user_email', true );
+		$email = $this->get_meta('_give_payment_user_email', true);
 
-		if ( empty( $email ) && $this->customer_id ) {
-			$email = Give()->donors->get_column( 'email', $this->customer_id );
+		if (empty($email) && $this->customer_id) {
+			$email = Give()->donors->get_column('email', $this->customer_id);
 		}
 
 		return $email;
@@ -1815,23 +1815,23 @@ 
 			'last_name'  => $this->last_name,
 		);
 
-		$user_info = isset( $this->payment_meta['user_info'] ) ? $this->payment_meta['user_info'] : array();
+		$user_info = isset($this->payment_meta['user_info']) ? $this->payment_meta['user_info'] : array();
 
-		if ( is_serialized( $user_info ) ) {
-			preg_match( '/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $user_info, $matches );
-			if ( ! empty( $matches ) ) {
+		if (is_serialized($user_info)) {
+			preg_match('/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $user_info, $matches);
+			if ( ! empty($matches)) {
 				$user_info = array();
 			}
 		}
 
-		$user_info = wp_parse_args( $user_info, $defaults );
+		$user_info = wp_parse_args($user_info, $defaults);
 
-		if ( empty( $user_info ) ) {
+		if (empty($user_info)) {
 			// Get the donor, but only if it's been created.
-			$donor = new Give_Donor( $this->customer_id );
+			$donor = new Give_Donor($this->customer_id);
 
-			if ( $donor->id > 0 ) {
-				$name      = explode( ' ', $donor->name, 2 );
+			if ($donor->id > 0) {
+				$name      = explode(' ', $donor->name, 2);
 				$user_info = array(
 					'first_name' => $name[0],
 					'last_name'  => $name[1],
@@ -1841,29 +1841,29 @@ 
 			}
 		} else {
 			// Get the donor, but only if it's been created.
-			$donor = new Give_Donor( $this->customer_id );
-			if ( $donor->id > 0 ) {
-				foreach ( $user_info as $key => $value ) {
-					if ( ! empty( $value ) ) {
+			$donor = new Give_Donor($this->customer_id);
+			if ($donor->id > 0) {
+				foreach ($user_info as $key => $value) {
+					if ( ! empty($value)) {
 						continue;
 					}
 
-					switch ( $key ) {
+					switch ($key) {
 						case 'first_name':
-							$name = explode( ' ', $donor->name, 2 );
+							$name = explode(' ', $donor->name, 2);
 
-							$user_info[ $key ] = $name[0];
+							$user_info[$key] = $name[0];
 							break;
 
 						case 'last_name':
-							$name      = explode( ' ', $donor->name, 2 );
-							$last_name = ! empty( $name[1] ) ? $name[1] : '';
+							$name      = explode(' ', $donor->name, 2);
+							$last_name = ! empty($name[1]) ? $name[1] : '';
 
-							$user_info[ $key ] = $last_name;
+							$user_info[$key] = $last_name;
 							break;
 
 						case 'email':
-							$user_info[ $key ] = $donor->email;
+							$user_info[$key] = $donor->email;
 							break;
 					}
 				}
@@ -1884,7 +1884,7 @@ 
 	 */
 	private function setup_address() {
 
-		$address = ! empty( $this->payment_meta['user_info']['address'] ) ? $this->payment_meta['user_info']['address'] : array(
+		$address = ! empty($this->payment_meta['user_info']['address']) ? $this->payment_meta['user_info']['address'] : array(
 			'line1'   => '',
 			'line2'   => '',
 			'city'    => '',
@@ -1906,7 +1906,7 @@ 
 	 */
 	private function setup_form_title() {
 
-		$form_id = $this->get_meta( '_give_payment_form_title', true );
+		$form_id = $this->get_meta('_give_payment_form_title', true);
 
 		return $form_id;
 	}
@@ -1921,7 +1921,7 @@ 
 	 */
 	private function setup_form_id() {
 
-		$form_id = $this->get_meta( '_give_payment_form_id', true );
+		$form_id = $this->get_meta('_give_payment_form_id', true);
 
 		return $form_id;
 	}
@@ -1935,7 +1935,7 @@ 
 	 * @return int The Form Price ID.
 	 */
 	private function setup_price_id() {
-		$price_id = $this->get_meta( '_give_payment_price_id', true );
+		$price_id = $this->get_meta('_give_payment_price_id', true);
 
 		return $price_id;
 	}
@@ -1949,7 +1949,7 @@ 
 	 * @return string The Payment Key.
 	 */
 	private function setup_payment_key() {
-		$key = $this->get_meta( '_give_payment_purchase_key', true );
+		$key = $this->get_meta('_give_payment_purchase_key', true);
 
 		return $key;
 	}
@@ -1965,11 +1965,11 @@ 
 	private function setup_payment_number() {
 		$number = $this->ID;
 
-		if ( give_get_option( 'enable_sequential' ) ) {
+		if (give_get_option('enable_sequential')) {
 
-			$number = $this->get_meta( '_give_payment_number', true );
+			$number = $this->get_meta('_give_payment_number', true);
 
-			if ( ! $number ) {
+			if ( ! $number) {
 
 				$number = $this->ID;
 
@@ -1987,7 +1987,7 @@ 
 	 * @return array The payment object as an array.
 	 */
 	public function array_convert() {
-		return get_object_vars( $this );
+		return get_object_vars($this);
 	}
 
 
@@ -2000,7 +2000,7 @@ 
 	 * @return bool
 	 */
 	public function is_completed() {
-		return ( 'publish' === $this->status && $this->completed_date );
+		return ('publish' === $this->status && $this->completed_date);
 	}
 
 	/**
@@ -2012,7 +2012,7 @@ 
 	 * @return string Date payment was completed.
 	 */
 	private function get_completed_date() {
-		return apply_filters( 'give_payment_completed_date', $this->completed_date, $this->ID, $this );
+		return apply_filters('give_payment_completed_date', $this->completed_date, $this->ID, $this);
 	}
 
 	/**
@@ -2024,7 +2024,7 @@ 
 	 * @return float Payment subtotal.
 	 */
 	private function get_subtotal() {
-		return apply_filters( 'give_get_payment_subtotal', $this->subtotal, $this->ID, $this );
+		return apply_filters('give_get_payment_subtotal', $this->subtotal, $this->ID, $this);
 	}
 
 	/**
@@ -2036,7 +2036,7 @@ 
 	 * @return string Payment currency code.
 	 */
 	private function get_currency() {
-		return apply_filters( 'give_payment_currency_code', $this->currency, $this->ID, $this );
+		return apply_filters('give_payment_currency_code', $this->currency, $this->ID, $this);
 	}
 
 	/**
@@ -2048,7 +2048,7 @@ 
 	 * @return string Gateway used.
 	 */
 	private function get_gateway() {
-		return apply_filters( 'give_payment_gateway', $this->gateway, $this->ID, $this );
+		return apply_filters('give_payment_gateway', $this->gateway, $this->ID, $this);
 	}
 
 	/**
@@ -2060,7 +2060,7 @@ 
 	 * @return string Donation ID from merchant processor.
 	 */
 	private function get_transaction_id() {
-		return apply_filters( 'give_get_payment_transaction_id', $this->transaction_id, $this->ID, $this );
+		return apply_filters('give_get_payment_transaction_id', $this->transaction_id, $this->ID, $this);
 	}
 
 	/**
@@ -2072,7 +2072,7 @@ 
 	 * @return string Payment IP address
 	 */
 	private function get_ip() {
-		return apply_filters( 'give_payment_user_ip', $this->ip, $this->ID, $this );
+		return apply_filters('give_payment_user_ip', $this->ip, $this->ID, $this);
 	}
 
 	/**
@@ -2084,7 +2084,7 @@ 
 	 * @return int Payment donor ID.
 	 */
 	private function get_donor_id() {
-		return apply_filters( 'give_payment_customer_id', $this->customer_id, $this->ID, $this );
+		return apply_filters('give_payment_customer_id', $this->customer_id, $this->ID, $this);
 	}
 
 	/**
@@ -2096,7 +2096,7 @@ 
 	 * @return int Payment user ID.
 	 */
 	private function get_user_id() {
-		return apply_filters( 'give_payment_user_id', $this->user_id, $this->ID, $this );
+		return apply_filters('give_payment_user_id', $this->user_id, $this->ID, $this);
 	}
 
 	/**
@@ -2108,7 +2108,7 @@ 
 	 * @return string Payment donor email.
 	 */
 	private function get_email() {
-		return apply_filters( 'give_payment_user_email', $this->email, $this->ID, $this );
+		return apply_filters('give_payment_user_email', $this->email, $this->ID, $this);
 	}
 
 	/**
@@ -2120,7 +2120,7 @@ 
 	 * @return array Payment user info.
 	 */
 	private function get_user_info() {
-		return apply_filters( 'give_payment_meta_user_info', $this->user_info, $this->ID, $this );
+		return apply_filters('give_payment_meta_user_info', $this->user_info, $this->ID, $this);
 	}
 
 	/**
@@ -2132,7 +2132,7 @@ 
 	 * @return array Payment billing address.
 	 */
 	private function get_address() {
-		return apply_filters( 'give_payment_address', $this->address, $this->ID, $this );
+		return apply_filters('give_payment_address', $this->address, $this->ID, $this);
 	}
 
 	/**
@@ -2144,7 +2144,7 @@ 
 	 * @return string Payment key.
 	 */
 	private function get_key() {
-		return apply_filters( 'give_payment_key', $this->key, $this->ID, $this );
+		return apply_filters('give_payment_key', $this->key, $this->ID, $this);
 	}
 
 	/**
@@ -2156,7 +2156,7 @@ 
 	 * @return string Payment form id
 	 */
 	private function get_form_id() {
-		return apply_filters( 'give_payment_form_id', $this->form_id, $this->ID, $this );
+		return apply_filters('give_payment_form_id', $this->form_id, $this->ID, $this);
 	}
 
 	/**
@@ -2168,7 +2168,7 @@ 
 	 * @return int|string Payment number
 	 */
 	private function get_number() {
-		return apply_filters( 'give_payment_number', $this->number, $this->ID, $this );
+		return apply_filters('give_payment_number', $this->number, $this->ID, $this);
 	}
 
 }

includes/class-give-cron.php

Spacing +37 added lines, -37 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -51,7 +51,7 @@ 
 	 * @return static
 	 */
 	public static function get_instance() {
-		if ( null === static::$instance ) {
+		if (null === static::$instance) {
 			self::$instance = new static();
 			self::$instance->setup();
 		}
@@ -66,12 +66,12 @@ 
 	 * @since 1.8.13
 	 */
 	private function setup() {
-		add_filter( 'cron_schedules', array( self::$instance, '__add_schedules' ) );
-		add_action( 'wp', array( self::$instance, '__schedule_events' ) );
+		add_filter('cron_schedules', array(self::$instance, '__add_schedules'));
+		add_action('wp', array(self::$instance, '__schedule_events'));
 
 		// Load async event only when cron is running.
-		if( defined( 'DOING_CRON' ) && DOING_CRON ) {
-			add_action( 'init', array( self::$instance, '__load_async_events' ) );
+		if (defined('DOING_CRON') && DOING_CRON) {
+			add_action('init', array(self::$instance, '__load_async_events'));
 		}
 	}
 
@@ -82,23 +82,23 @@ 
 	 * @since 1.8.13
 	 */
 	public function __load_async_events() {
-		$async_events = get_option( 'give_async_events', array() );
+		$async_events = get_option('give_async_events', array());
 
 		// Bailout.
-		if ( empty( $async_events ) ) {
+		if (empty($async_events)) {
 			return;
 		}
 
-		foreach ( $async_events as $index => $event ) {
+		foreach ($async_events as $index => $event) {
 			// Set cron name.
 			$cron_name = "give_async_scheduled_events_{$index}";
 
 			// Setup cron.
-			wp_schedule_single_event( current_time( 'timestamp', 1 ), $cron_name, $event['params'] );
+			wp_schedule_single_event(current_time('timestamp', 1), $cron_name, $event['params']);
 
 			// Add cron action.
-			add_action( $cron_name, $event['callback'], 10, count( $event['params'] ) );
-			add_action( $cron_name, array( $this, '__delete_async_events' ), 9999, 0 );
+			add_action($cron_name, $event['callback'], 10, count($event['params']));
+			add_action($cron_name, array($this, '__delete_async_events'), 9999, 0);
 		}
 	}
 
@@ -108,13 +108,13 @@ 
 	 * @since 1.8.13
 	 */
 	public function __delete_async_events() {
-		$async_events    = get_option( 'give_async_events', array() );
-		$cron_name_parts = explode( '_', current_action() );
-		$cron_id         = end( $cron_name_parts );
+		$async_events    = get_option('give_async_events', array());
+		$cron_name_parts = explode('_', current_action());
+		$cron_id         = end($cron_name_parts);
 
-		if ( ! empty( $async_events[ $cron_id ] ) ) {
-			unset( $async_events[ $cron_id ] );
-			update_option( 'give_async_events', $async_events );
+		if ( ! empty($async_events[$cron_id])) {
+			unset($async_events[$cron_id]);
+			update_option('give_async_events', $async_events);
 		}
 	}
 
@@ -128,11 +128,11 @@ 
 	 *
 	 * @return array            An array of non-default cron schedules.
 	 */
-	public function __add_schedules( $schedules = array() ) {
+	public function __add_schedules($schedules = array()) {
 		// Adds once weekly to the existing schedules.
 		$schedules['weekly'] = array(
 			'interval' => 604800,
-			'display'  => __( 'Once Weekly', 'give' ),
+			'display'  => __('Once Weekly', 'give'),
 		);
 
 		return $schedules;
@@ -160,8 +160,8 @@ 
 	 * @return void
 	 */
 	private function weekly_events() {
-		if ( ! wp_next_scheduled( 'give_weekly_scheduled_events' ) ) {
-			wp_schedule_event( current_time( 'timestamp' ), 'weekly', 'give_weekly_scheduled_events' );
+		if ( ! wp_next_scheduled('give_weekly_scheduled_events')) {
+			wp_schedule_event(current_time('timestamp'), 'weekly', 'give_weekly_scheduled_events');
 		}
 	}
 
@@ -174,8 +174,8 @@ 
 	 * @return void
 	 */
 	private function daily_events() {
-		if ( ! wp_next_scheduled( 'give_daily_scheduled_events' ) ) {
-			wp_schedule_event( current_time( 'timestamp' ), 'daily', 'give_daily_scheduled_events' );
+		if ( ! wp_next_scheduled('give_daily_scheduled_events')) {
+			wp_schedule_event(current_time('timestamp'), 'daily', 'give_daily_scheduled_events');
 		}
 	}
 
@@ -189,8 +189,8 @@ 
 	 *
 	 * @return string
 	 */
-	public static function get_cron_action( $type = 'weekly' ) {
-		switch ( $type ) {
+	public static function get_cron_action($type = 'weekly') {
+		switch ($type) {
 			case 'daily':
 				$cron_action = 'give_daily_scheduled_events';
 				break;
@@ -212,9 +212,9 @@ 
 	 * @param        $action
 	 * @param string $type
 	 */
-	private static function add_event( $action, $type = 'weekly' ) {
-		$cron_event = self::get_cron_action( $type );
-		add_action( $cron_event, $action );
+	private static function add_event($action, $type = 'weekly') {
+		$cron_event = self::get_cron_action($type);
+		add_action($cron_event, $action);
 	}
 
 	/**
@@ -225,8 +225,8 @@ 
 	 *
 	 * @param $action
 	 */
-	public static function add_weekly_event( $action ) {
-		self::add_event( $action, 'weekly' );
+	public static function add_weekly_event($action) {
+		self::add_event($action, 'weekly');
 	}
 
 	/**
@@ -237,8 +237,8 @@ 
 	 *
 	 * @param $action
 	 */
-	public static function add_daily_event( $action ) {
-		self::add_event( $action, 'daily' );
+	public static function add_daily_event($action) {
+		self::add_event($action, 'daily');
 	}
 
 	/**
@@ -251,17 +251,17 @@ 
 	 * @param string $action
 	 * @param array  $args
 	 */
-	public static function add_async_event( $action, $args = array() ) {
+	public static function add_async_event($action, $args = array()) {
 
 		// Cache async events.
-		$async_events             = get_option( 'give_async_events', array() );
-		$async_events[ uniqid() ] = array(
+		$async_events             = get_option('give_async_events', array());
+		$async_events[uniqid()] = array(
 			'callback' => $action,
 			'params'   => $args,
 		);
 
 
-		update_option( 'give_async_events', $async_events );
+		update_option('give_async_events', $async_events);
 	}
 }