WordImpress / Give

includes/payments/backward-compatibility.php

Doc Comments +1 added lines, -1 removed lines

@@ -207,7 +207,7 @@
  *                              metadata entries with the specified value.
  *                              Otherwise, update all entries.
  *
- * @return mixed
+ * @return null|boolean
  */
 function _give_20_bc_saving_old_payment_meta( $check, $object_id, $meta_key, $meta_value, $prev_value ) {
 	// Bailout.

Spacing +166 added lines, -171 removed lines

@@ -9,74 +9,74 @@ 
  *
  * @return void
  */
-function _give_20_bc_split_and_save_give_payment_meta( $object_id, $meta_value ) {
+function _give_20_bc_split_and_save_give_payment_meta($object_id, $meta_value) {
 	// Bailout
-	if ( empty( $meta_value ) ) {
+	if (empty($meta_value)) {
 		return;
-	} elseif ( ! is_array( $meta_value ) ) {
+	} elseif ( ! is_array($meta_value)) {
 		$meta_value = array();
 	}
 
-	remove_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10 );
+	remove_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10);
 
 	// Date payment meta.
-	if ( ! empty( $meta_value['date'] ) ) {
-		give_update_meta( $object_id, '_give_payment_date', $meta_value['date'] );
+	if ( ! empty($meta_value['date'])) {
+		give_update_meta($object_id, '_give_payment_date', $meta_value['date']);
 	}
 
 	// Currency payment meta.
-	if ( ! empty( $meta_value['currency'] ) ) {
-		give_update_meta( $object_id, '_give_payment_currency', $meta_value['currency'] );
+	if ( ! empty($meta_value['currency'])) {
+		give_update_meta($object_id, '_give_payment_currency', $meta_value['currency']);
 	}
 
 	// User information.
-	if ( ! empty( $meta_value['user_info'] ) ) {
+	if ( ! empty($meta_value['user_info'])) {
 		// Donor first name.
-		if ( ! empty( $meta_value['user_info']['first_name'] ) ) {
-			give_update_meta( $object_id, '_give_donor_billing_first_name', $meta_value['user_info']['first_name'] );
+		if ( ! empty($meta_value['user_info']['first_name'])) {
+			give_update_meta($object_id, '_give_donor_billing_first_name', $meta_value['user_info']['first_name']);
 		}
 
 		// Donor last name.
-		if ( ! empty( $meta_value['user_info']['last_name'] ) ) {
-			give_update_meta( $object_id, '_give_donor_billing_last_name', $meta_value['user_info']['last_name'] );
+		if ( ! empty($meta_value['user_info']['last_name'])) {
+			give_update_meta($object_id, '_give_donor_billing_last_name', $meta_value['user_info']['last_name']);
 		}
 
 		// Donor address payment meta.
-		if ( ! empty( $meta_value['user_info']['address'] ) ) {
+		if ( ! empty($meta_value['user_info']['address'])) {
 
 			// Address1.
-			if ( ! empty( $meta_value['user_info']['address']['line1'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_address1', $meta_value['user_info']['address']['line1'] );
+			if ( ! empty($meta_value['user_info']['address']['line1'])) {
+				give_update_meta($object_id, '_give_donor_billing_address1', $meta_value['user_info']['address']['line1']);
 			}
 
 			// Address2.
-			if ( ! empty( $meta_value['user_info']['address']['line2'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_address2', $meta_value['user_info']['address']['line2'] );
+			if ( ! empty($meta_value['user_info']['address']['line2'])) {
+				give_update_meta($object_id, '_give_donor_billing_address2', $meta_value['user_info']['address']['line2']);
 			}
 
 			// City.
-			if ( ! empty( $meta_value['user_info']['address']['city'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_city', $meta_value['user_info']['address']['city'] );
+			if ( ! empty($meta_value['user_info']['address']['city'])) {
+				give_update_meta($object_id, '_give_donor_billing_city', $meta_value['user_info']['address']['city']);
 			}
 
 			// Zip.
-			if ( ! empty( $meta_value['user_info']['address']['zip'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_zip', $meta_value['user_info']['address']['zip'] );
+			if ( ! empty($meta_value['user_info']['address']['zip'])) {
+				give_update_meta($object_id, '_give_donor_billing_zip', $meta_value['user_info']['address']['zip']);
 			}
 
 			// State.
-			if ( ! empty( $meta_value['user_info']['address']['state'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_state', $meta_value['user_info']['address']['state'] );
+			if ( ! empty($meta_value['user_info']['address']['state'])) {
+				give_update_meta($object_id, '_give_donor_billing_state', $meta_value['user_info']['address']['state']);
 			}
 
 			// Country.
-			if ( ! empty( $meta_value['user_info']['address']['country'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_country', $meta_value['user_info']['address']['country'] );
+			if ( ! empty($meta_value['user_info']['address']['country'])) {
+				give_update_meta($object_id, '_give_donor_billing_country', $meta_value['user_info']['address']['country']);
 			}
 		}
 	}// End if().
 
-	add_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5 );
+	add_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5);
 }
 
 /**
@@ -89,103 +89,99 @@ 
  *
  * @return array
  */
-function _give_20_bc_give_payment_meta_value( $object_id, $meta_value ) {
+function _give_20_bc_give_payment_meta_value($object_id, $meta_value) {
 	$cache_key = "_give_payment_meta_{$object_id}";
-	$cache     = Give_Cache::get_db_query( $cache_key );
+	$cache     = Give_Cache::get_db_query($cache_key);
 
-	if ( ! is_null( $cache ) ) {
+	if ( ! is_null($cache)) {
 		return $cache;
 	}
 
 	// Set default value to array.
-	if ( ! is_array( $meta_value ) ) {
+	if ( ! is_array($meta_value)) {
 		$meta_value = array();
 	}
 
 	// Donation key.
-	$meta_value['key'] = give_get_meta( $object_id, '_give_payment_purchase_key', true );
+	$meta_value['key'] = give_get_meta($object_id, '_give_payment_purchase_key', true);
 
 	// Donation form.
-	$meta_value['form_title'] = give_get_meta( $object_id, '_give_payment_form_title', true );
+	$meta_value['form_title'] = give_get_meta($object_id, '_give_payment_form_title', true);
 
 	// Donor email.
-	$meta_value['email'] = give_get_meta( $object_id, '_give_payment_donor_email', true );
-	$meta_value['email'] = ! empty( $meta_value['email'] ) ?
-		$meta_value['email'] :
-		Give()->donors->get_column( 'email', give_get_payment_donor_id( $object_id ) );
+	$meta_value['email'] = give_get_meta($object_id, '_give_payment_donor_email', true);
+	$meta_value['email'] = ! empty($meta_value['email']) ?
+		$meta_value['email'] : Give()->donors->get_column('email', give_get_payment_donor_id($object_id));
 
 	// Form id.
-	$meta_value['form_id'] = give_get_meta( $object_id, '_give_payment_form_id', true );
+	$meta_value['form_id'] = give_get_meta($object_id, '_give_payment_form_id', true);
 
 	// Price id.
-	$meta_value['price_id'] = give_get_meta( $object_id, '_give_payment_price_id', true );
+	$meta_value['price_id'] = give_get_meta($object_id, '_give_payment_price_id', true);
 
 	// Date.
-	$meta_value['date'] = give_get_meta( $object_id, '_give_payment_date', true );
-	$meta_value['date'] = ! empty( $meta_value['date'] ) ?
-		$meta_value['date'] :
-		get_post_field( 'post_date', $object_id );
+	$meta_value['date'] = give_get_meta($object_id, '_give_payment_date', true);
+	$meta_value['date'] = ! empty($meta_value['date']) ?
+		$meta_value['date'] : get_post_field('post_date', $object_id);
 
 	// Currency.
-	$meta_value['currency'] = give_get_meta( $object_id, '_give_payment_currency', true );
+	$meta_value['currency'] = give_get_meta($object_id, '_give_payment_currency', true);
 
 	// Decode donor data.
-	$donor_names = give_get_donor_name_by( give_get_meta( $object_id, '_give_payment_donor_id', true ), 'donor' );
-	$donor_names = explode( ' ', $donor_names, 2 );
+	$donor_names = give_get_donor_name_by(give_get_meta($object_id, '_give_payment_donor_id', true), 'donor');
+	$donor_names = explode(' ', $donor_names, 2);
 
 	// Donor first name.
-	$donor_data['first_name'] = give_get_meta( $object_id, '_give_donor_billing_first_name', true );
-	$donor_data['first_name'] = ! empty( $donor_data['first_name'] ) ?
-		$donor_data['first_name'] :
-		$donor_names[0];
+	$donor_data['first_name'] = give_get_meta($object_id, '_give_donor_billing_first_name', true);
+	$donor_data['first_name'] = ! empty($donor_data['first_name']) ?
+		$donor_data['first_name'] : $donor_names[0];
 
 	// Donor last name.
-	$donor_data['last_name'] = give_get_meta( $object_id, '_give_donor_billing_last_name', true );
-	$donor_data['last_name'] = ! empty( $donor_data['last_name'] ) ?
-		$donor_data['last_name'] :
-		( isset( $donor_names[1] ) ? $donor_names[1] : '' );
+	$donor_data['last_name'] = give_get_meta($object_id, '_give_donor_billing_last_name', true);
+	$donor_data['last_name'] = ! empty($donor_data['last_name']) ?
+		$donor_data['last_name'] : (isset($donor_names[1]) ? $donor_names[1] : '');
 
 	// Donor email.
 	$donor_data['email'] = $meta_value['email'];
 
 	// User ID.
-	$donor_data['id'] = give_get_payment_user_id( $object_id );
+	$donor_data['id'] = give_get_payment_user_id($object_id);
 
 	$donor_data['address'] = false;
 
 	// Address1.
-	if ( $address1 = give_get_meta( $object_id, '_give_donor_billing_address1', true ) ) {
+	if ($address1 = give_get_meta($object_id, '_give_donor_billing_address1', true)) {
 		$donor_data['address']['line1'] = $address1;
 	}
 
 	// Address2.
-	if ( $address2 = give_get_meta( $object_id, '_give_donor_billing_address2', true ) ) {
+	if ($address2 = give_get_meta($object_id, '_give_donor_billing_address2', true)) {
 		$donor_data['address']['line2'] = $address2;
 	}
 
 	// City.
-	if ( $city = give_get_meta( $object_id, '_give_donor_billing_city', true ) ) {
+	if ($city = give_get_meta($object_id, '_give_donor_billing_city', true)) {
 		$donor_data['address']['city'] = $city;
 	}
 
 	// Zip.
-	if ( $zip = give_get_meta( $object_id, '_give_donor_billing_zip', true ) ) {
+	if ($zip = give_get_meta($object_id, '_give_donor_billing_zip', true)) {
 		$donor_data['address']['zip'] = $zip;
 	}
 
 	// State.
-	if ( $state = give_get_meta( $object_id, '_give_donor_billing_state', true ) ) {
+	if ($state = give_get_meta($object_id, '_give_donor_billing_state', true)) {
 		$donor_data['address']['state'] = $state;
 	}
 
 	// Country.
-	if ( $country = give_get_meta( $object_id, '_give_donor_billing_country', true ) ) {
+	if ($country = give_get_meta($object_id, '_give_donor_billing_country', true)) {
 		$donor_data['address']['country'] = $country;
 	}
 
-	$meta_value['user_info'] = maybe_unserialize( $donor_data );
+	$meta_value['user_info'] = maybe_unserialize($donor_data);
 
-	Give_Cache::set_db_query( $cache_key, $meta_value );
+	Give_Cache::set_db_query($cache_key, $meta_value);
 
 	return $meta_value;
 }
@@ -209,37 +205,37 @@ 
  *
  * @return mixed
  */
-function _give_20_bc_saving_old_payment_meta( $check, $object_id, $meta_key, $meta_value, $prev_value ) {
+function _give_20_bc_saving_old_payment_meta($check, $object_id, $meta_key, $meta_value, $prev_value) {
 	// Bailout.
 	if (
-		'give_payment' !== get_post_type( $object_id ) ||
-		! in_array( $meta_key, array(
+		'give_payment' !== get_post_type($object_id) ||
+		! in_array($meta_key, array(
 			'_give_payment_meta',
 			'_give_payment_user_email',
 			'_give_payment_customer_id',
 			'give_payment_user_ip',
-		) )
+		))
 	) {
 		return $check;
 	}
 
-	if ( '_give_payment_meta' === $meta_key ) {
-		_give_20_bc_split_and_save_give_payment_meta( $object_id, $meta_value );
-	} elseif ( '_give_payment_user_email' === $meta_key ) {
-		give_update_meta( $object_id, '_give_payment_donor_email', $meta_value );
+	if ('_give_payment_meta' === $meta_key) {
+		_give_20_bc_split_and_save_give_payment_meta($object_id, $meta_value);
+	} elseif ('_give_payment_user_email' === $meta_key) {
+		give_update_meta($object_id, '_give_payment_donor_email', $meta_value);
 		$check = true;
-	} elseif ( '_give_payment_customer_id' === $meta_key ) {
-		give_update_meta( $object_id, '_give_payment_donor_id', $meta_value );
+	} elseif ('_give_payment_customer_id' === $meta_key) {
+		give_update_meta($object_id, '_give_payment_donor_id', $meta_value);
 		$check = true;
-	} elseif ( 'give_payment_user_ip' === $meta_key ) {
-		give_update_meta( $object_id, '_give_payment_donor_ip', $meta_value );
+	} elseif ('give_payment_user_ip' === $meta_key) {
+		give_update_meta($object_id, '_give_payment_donor_ip', $meta_value);
 		$check = true;
 	}
 
 	return $check;
 }
 
-add_filter( 'update_post_metadata', '_give_20_bc_saving_old_payment_meta', 10, 5 );
+add_filter('update_post_metadata', '_give_20_bc_saving_old_payment_meta', 10, 5);
 
 
 /**
@@ -254,7 +250,7 @@ 
  *
  * @return mixed
  */
-function _give_20_bc_get_old_payment_meta( $check, $object_id, $meta_key, $single ) {
+function _give_20_bc_get_old_payment_meta($check, $object_id, $meta_key, $single) {
 	global $wpdb;
 
 	// Deprecated meta keys.
@@ -265,32 +261,31 @@ 
 	);
 
 	// Add _give_payment_meta to backward compatibility
-	if ( ! give_has_upgrade_completed( 'v20_upgrades_payment_metadata' ) ) {
+	if ( ! give_has_upgrade_completed('v20_upgrades_payment_metadata')) {
 		$old_meta_keys[] = '_give_payment_meta';
 	}
 
 	// Bailout.
 	if (
-		'give_payment' !== get_post_type( $object_id ) ||
-		! in_array( $meta_key, $old_meta_keys )
+		'give_payment' !== get_post_type($object_id) ||
+		! in_array($meta_key, $old_meta_keys)
 	) {
 		return $check;
 	}
 
 	$cache_key = "{$meta_key}_{$object_id}";
-	$check     = Give_Cache::get_db_query( $cache_key );
+	$check     = Give_Cache::get_db_query($cache_key);
 
-	if ( is_null( $check ) ) {
-		switch ( $meta_key ) {
+	if (is_null($check)) {
+		switch ($meta_key) {
 
 			// Handle old meta keys.
 			case '_give_payment_meta':
-				remove_filter( 'get_post_metadata', '_give_20_bc_get_old_payment_meta' );
+				remove_filter('get_post_metadata', '_give_20_bc_get_old_payment_meta');
 
 				// if ( $meta_value = give_get_meta( $object_id, '_give_payment_meta' ) ) {
-				$meta_value = ! empty( $meta_value ) ?
-					current( $meta_value ) :
-					(array) maybe_unserialize(
+				$meta_value = ! empty($meta_value) ?
+					current($meta_value) : (array) maybe_unserialize(
 						$wpdb->get_var(
 							$wpdb->prepare(
 								"
@@ -304,44 +299,44 @@ 
 							)
 						)
 					);
-				$check      = _give_20_bc_give_payment_meta_value( $object_id, $meta_value );
+				$check = _give_20_bc_give_payment_meta_value($object_id, $meta_value);
 				// }
 
-				add_filter( 'get_post_metadata', '_give_20_bc_get_old_payment_meta', 10, 5 );
+				add_filter('get_post_metadata', '_give_20_bc_get_old_payment_meta', 10, 5);
 
 				break;
 
 			case '_give_payment_customer_id':
-				if ( $donor_id = give_get_meta( $object_id, '_give_payment_donor_id', $single ) ) {
+				if ($donor_id = give_get_meta($object_id, '_give_payment_donor_id', $single)) {
 					$check = $donor_id;
 				}
 				break;
 
 			case '_give_payment_user_email':
-				if ( $donor_email = give_get_meta( $object_id, '_give_payment_donor_email', $single ) ) {
+				if ($donor_email = give_get_meta($object_id, '_give_payment_donor_email', $single)) {
 					$check = $donor_email;
 				}
 				break;
 
 			case '_give_payment_user_ip':
-				if ( $donor_ip = give_get_meta( $object_id, '_give_payment_donor_ip', $single ) ) {
+				if ($donor_ip = give_get_meta($object_id, '_give_payment_donor_ip', $single)) {
 					$check = $donor_ip;
 				}
 				break;
 		}// End switch().
 
-		Give_Cache::set_db_query( $cache_key, $check );
+		Give_Cache::set_db_query($cache_key, $check);
 	}
 
 	// Put result in an array on zero index.
-	if ( ! is_null( $check ) ) {
-		$check = array( $check );
+	if ( ! is_null($check)) {
+		$check = array($check);
 	}
 
 	return $check;
 }
 
-add_filter( 'get_post_metadata', '_give_20_bc_get_old_payment_meta', 10, 5 );
+add_filter('get_post_metadata', '_give_20_bc_get_old_payment_meta', 10, 5);
 
 
 /**
@@ -356,9 +351,9 @@ 
  *
  * @return mixed
  */
-function _give_20_bc_get_new_payment_meta( $check, $object_id, $meta_key, $single ) {
+function _give_20_bc_get_new_payment_meta($check, $object_id, $meta_key, $single) {
 	// Bailout: do not apply backward compatibility if upgrade done.
-	if ( give_has_upgrade_completed( 'v20_upgrades_payment_metadata' ) ) {
+	if (give_has_upgrade_completed('v20_upgrades_payment_metadata')) {
 		return $check;
 	}
 
@@ -380,26 +375,26 @@ 
 	);
 
 	// metadata_exists fx will cause of firing get_post_metadata filter again so remove it to prevent infinite loop.
-	remove_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta' );
+	remove_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta');
 
 	// Bailout.
 	if (
-		'give_payment' !== get_post_type( $object_id ) ||
-		! in_array( $meta_key, $new_meta_keys ) ||
-		metadata_exists( 'post', $object_id, $meta_key )
+		'give_payment' !== get_post_type($object_id) ||
+		! in_array($meta_key, $new_meta_keys) ||
+		metadata_exists('post', $object_id, $meta_key)
 	) {
-		add_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5 );
+		add_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5);
 
 		return $check;
 	}
 
-	add_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5 );
+	add_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5);
 
 	$cache_key = "{$meta_key}_{$object_id}";
-	$check    = Give_Cache::get_db_query( $cache_key );
+	$check = Give_Cache::get_db_query($cache_key);
 
-	if ( is_null( $check ) ) {
-		switch ( $meta_key ) {
+	if (is_null($check)) {
+		switch ($meta_key) {
 
 			// Handle new meta keys.
 			case '_give_payment_donor_id':
@@ -442,9 +437,9 @@ 
 			case '_give_donor_billing_country':
 			case '_give_payment_date':
 			case '_give_payment_currency':
-				$donation_meta = Give_Cache::get_db_query( "_give_payment_meta_{$object_id}" );
+				$donation_meta = Give_Cache::get_db_query("_give_payment_meta_{$object_id}");
 
-				if ( is_null( $donation_meta ) ) {
+				if (is_null($donation_meta)) {
 					$donation_meta = $wpdb->get_var(
 						$wpdb->prepare(
 							"SELECT meta_value FROM {$wpdb->postmeta} WHERE post_id=%d AND meta_key=%s",
@@ -452,51 +447,51 @@ 
 							'_give_payment_meta'
 						)
 					);
-					$donation_meta = maybe_unserialize( $donation_meta );
-					$donation_meta = ! is_array( $donation_meta ) ? array() : $donation_meta;
-					Give_Cache::set_db_query( "_give_payment_meta_{$object_id}", $donation_meta );
+					$donation_meta = maybe_unserialize($donation_meta);
+					$donation_meta = ! is_array($donation_meta) ? array() : $donation_meta;
+					Give_Cache::set_db_query("_give_payment_meta_{$object_id}", $donation_meta);
 				}
 
 				// Get results.
-				if ( empty( $donation_meta ) ) {
+				if (empty($donation_meta)) {
 					$check = '';
-				} elseif ( in_array( $meta_key, array( '_give_payment_date', '_give_payment_currency' ) ) ) {
-					$payment_meta_key = str_replace( '_give_payment_', '', $meta_key );
+				} elseif (in_array($meta_key, array('_give_payment_date', '_give_payment_currency'))) {
+					$payment_meta_key = str_replace('_give_payment_', '', $meta_key);
 
-					if ( isset( $donation_meta[ $payment_meta_key ] ) ) {
-						$check = $donation_meta[ $payment_meta_key ];
+					if (isset($donation_meta[$payment_meta_key])) {
+						$check = $donation_meta[$payment_meta_key];
 					}
 				} else {
-					$payment_meta_key = str_replace( '_give_donor_billing_', '', $meta_key );
+					$payment_meta_key = str_replace('_give_donor_billing_', '', $meta_key);
 
-					switch ( $payment_meta_key ) {
+					switch ($payment_meta_key) {
 						case 'address1':
-							if ( isset( $donation_meta['user_info']['address']['line1'] ) ) {
+							if (isset($donation_meta['user_info']['address']['line1'])) {
 								$check = $donation_meta['user_info']['address']['line1'];
 							}
 							break;
 
 						case 'address2':
-							if ( isset( $donation_meta['user_info']['address']['line2'] ) ) {
+							if (isset($donation_meta['user_info']['address']['line2'])) {
 								$check = $donation_meta['user_info']['address']['line2'];
 							}
 							break;
 
 						case 'first_name':
-							if ( isset( $donation_meta['user_info']['first_name'] ) ) {
+							if (isset($donation_meta['user_info']['first_name'])) {
 								$check = $donation_meta['user_info']['first_name'];
 							}
 							break;
 
 						case 'last_name':
-							if ( isset( $donation_meta['user_info']['last_name'] ) ) {
+							if (isset($donation_meta['user_info']['last_name'])) {
 								$check = $donation_meta['user_info']['last_name'];
 							}
 							break;
 
 						default:
-							if ( isset( $donation_meta['user_info']['address'][ $payment_meta_key ] ) ) {
-								$check = $donation_meta['user_info']['address'][ $payment_meta_key ];
+							if (isset($donation_meta['user_info']['address'][$payment_meta_key])) {
+								$check = $donation_meta['user_info']['address'][$payment_meta_key];
 							}
 					}
 				}
@@ -505,19 +500,19 @@ 
 		}// End switch().
 
 		// Set cache.
-		Give_Cache::set_db_query( $cache_key, $check );
+		Give_Cache::set_db_query($cache_key, $check);
 	}
 
 	// Put result in an array on zero index.
-	if ( ! $single ) {
-		$check = array( $check );
+	if ( ! $single) {
+		$check = array($check);
 	}
 
 
 	return $check;
 }
 
-add_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5 );
+add_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5);
 
 
 /**
@@ -529,9 +524,9 @@ 
  *
  * @return void
  */
-function _give_20_bc_support_deprecated_meta_key_query( $query ) {
+function _give_20_bc_support_deprecated_meta_key_query($query) {
 	// Bailout.
-	if ( give_has_upgrade_completed( 'v20_upgrades_payment_metadata' ) ) {
+	if (give_has_upgrade_completed('v20_upgrades_payment_metadata')) {
 		return;
 	}
 
@@ -541,21 +536,21 @@ 
 		// '_give_payment_user_ip'     => '_give_payment_donor_ip',
 	);
 
-	$deprecated_meta_keys = array_flip( $new_meta_keys );
+	$deprecated_meta_keys = array_flip($new_meta_keys);
 
 	// Set meta keys.
 	$meta_keys = array();
 
 
 	// Bailout.
-	if ( ! empty( $query->query_vars['meta_key'] ) ) {
-		if ( in_array( $query->query_vars['meta_key'], $new_meta_keys ) ) {
+	if ( ! empty($query->query_vars['meta_key'])) {
+		if (in_array($query->query_vars['meta_key'], $new_meta_keys)) {
 			$meta_keys = $deprecated_meta_keys;
-		} elseif ( in_array( $query->query_vars['meta_key'], $deprecated_meta_keys ) ) {
+		} elseif (in_array($query->query_vars['meta_key'], $deprecated_meta_keys)) {
 			$meta_keys = $new_meta_keys;
 		}
 
-		if ( ! empty( $meta_keys ) ) {
+		if ( ! empty($meta_keys)) {
 			// Set meta_query
 			$query->set(
 				'meta_query',
@@ -566,35 +561,35 @@ 
 						'value' => $query->query_vars['meta_value'],
 					),
 					array(
-						'key'   => $meta_keys[ $query->query_vars['meta_key'] ],
+						'key'   => $meta_keys[$query->query_vars['meta_key']],
 						'value' => $query->query_vars['meta_value'],
 					),
 				)
 			);
 
 			// Unset single meta query.
-			unset( $query->query_vars['meta_key'] );
-			unset( $query->query_vars['meta_value'] );
+			unset($query->query_vars['meta_key']);
+			unset($query->query_vars['meta_value']);
 		}
 	} elseif (
-		! empty( $query->query_vars['meta_query'] ) &&
-		( 1 === count( $query->query_vars['meta_query'] ) )
+		! empty($query->query_vars['meta_query']) &&
+		(1 === count($query->query_vars['meta_query']))
 	) {
-		$meta_query = current( $query->query_vars['meta_query'] );
+		$meta_query = current($query->query_vars['meta_query']);
 
-		if ( empty( $meta_query[0]['key'] ) ) {
+		if (empty($meta_query[0]['key'])) {
 			return;
 		}
 
-		if ( in_array( $meta_query[0]['key'], $new_meta_keys ) ) {
+		if (in_array($meta_query[0]['key'], $new_meta_keys)) {
 			$meta_keys = $deprecated_meta_keys;
-		} elseif ( in_array( $meta_query[0]['key'], $deprecated_meta_keys ) ) {
+		} elseif (in_array($meta_query[0]['key'], $deprecated_meta_keys)) {
 			$meta_keys = $new_meta_keys;
 		} else {
 			return;
 		}
 
-		if ( ! empty( $meta_keys ) ) {
+		if ( ! empty($meta_keys)) {
 			// Set meta_query
 			$query->set(
 				'meta_query',
@@ -605,7 +600,7 @@ 
 						'value' => $query->query_vars['meta_query'][0]['value'],
 					),
 					array(
-						'key'   => $meta_keys[ $query->query_vars['meta_query'][0]['key'] ],
+						'key'   => $meta_keys[$query->query_vars['meta_query'][0]['key']],
 						'value' => $query->query_vars['meta_query'][0]['value'],
 					),
 				)
@@ -614,7 +609,7 @@ 
 	}
 }
 
-add_action( 'pre_get_posts', '_give_20_bc_support_deprecated_meta_key_query' );
+add_action('pre_get_posts', '_give_20_bc_support_deprecated_meta_key_query');
 
 
 /**
@@ -627,49 +622,49 @@ 
  * @param Give_Payment $payment
  * @param string       $key
  */
-function _give_20_bc_payment_save( $payment, $key ) {
+function _give_20_bc_payment_save($payment, $key) {
 	// Bailout.
-	if ( ! give_has_upgrade_completed( 'v20_upgrades_payment_metadata' ) ) {
+	if ( ! give_has_upgrade_completed('v20_upgrades_payment_metadata')) {
 		return;
 	}
 
-	switch ( $key ) {
+	switch ($key) {
 		case 'user_info':
-			if ( empty( $payment->user_info ) ) {
+			if (empty($payment->user_info)) {
 				// Bailout.
 				break;
-			} elseif ( is_string( $payment->user_info ) ) {
+			} elseif (is_string($payment->user_info)) {
 				// Check if value serialize.
-				$payment->user_info = maybe_unserialize( $payment->user_info );
+				$payment->user_info = maybe_unserialize($payment->user_info);
 			}
 
 
 			// Save first name.
-			if ( isset( $payment->user_info['first_name'] ) ) {
-				$payment->update_meta( '_give_donor_billing_first_name', $payment->user_info['first_name'] );
+			if (isset($payment->user_info['first_name'])) {
+				$payment->update_meta('_give_donor_billing_first_name', $payment->user_info['first_name']);
 			}
 
 
 			// Save last name.
-			if ( isset( $payment->user_info['last_name'] ) ) {
-				$payment->update_meta( '_give_donor_billing_last_name', $payment->user_info['last_name'] );
+			if (isset($payment->user_info['last_name'])) {
+				$payment->update_meta('_give_donor_billing_last_name', $payment->user_info['last_name']);
 			}
 
 
 			// Save address.
-			if ( ! empty( $payment->user_info['address'] ) ) {
-				foreach ( $payment->user_info['address'] as $address_name => $address ) {
-					switch ( $address_name ) {
+			if ( ! empty($payment->user_info['address'])) {
+				foreach ($payment->user_info['address'] as $address_name => $address) {
+					switch ($address_name) {
 						case 'line1':
-							$payment->update_meta( '_give_donor_billing_address1', $address );
+							$payment->update_meta('_give_donor_billing_address1', $address);
 							break;
 
 						case 'line2':
-							$payment->update_meta( '_give_donor_billing_address2', $address );
+							$payment->update_meta('_give_donor_billing_address2', $address);
 							break;
 
 						default:
-							$payment->update_meta( "_give_donor_billing_{$address_name}", $address );
+							$payment->update_meta("_give_donor_billing_{$address_name}", $address);
 					}
 				}
 			}
@@ -678,7 +673,7 @@ 
 	}
 }
 
-add_action( 'give_payment_save', '_give_20_bc_payment_save', 10, 2 );
+add_action('give_payment_save', '_give_20_bc_payment_save', 10, 2);
 
 
 /**
@@ -691,16 +686,16 @@ 
  *
  * @return mixed
  */
-function __give_20_bc_flush_cache( $check, $object_id ) {
+function __give_20_bc_flush_cache($check, $object_id) {
 	if (
-		! give_has_upgrade_completed( 'v20_move_metadata_into_new_table' ) &&
-		'give_payment' === get_post_type( $object_id )
+		! give_has_upgrade_completed('v20_move_metadata_into_new_table') &&
+		'give_payment' === get_post_type($object_id)
 	) {
-		Give_Cache::delete_group( $object_id, 'give-donations' );
+		Give_Cache::delete_group($object_id, 'give-donations');
 	}
 
 	return $check;
 }
 
-add_action( 'update_post_metadata', '__give_20_bc_flush_cache', 9999, 2 );
-add_action( 'add_post_metadata', '__give_20_bc_flush_cache', 9999, 2 );
+add_action('update_post_metadata', '__give_20_bc_flush_cache', 9999, 2);
+add_action('add_post_metadata', '__give_20_bc_flush_cache', 9999, 2);

includes/payments/class-payments-query.php

Doc Comments +3 added lines, -3 removed lines

@@ -107,7 +107,7 @@ 
 	 * @since  1.0
 	 * @access public
 	 *
-	 * @param $query_var
+	 * @param string $query_var
 	 * @param $value
 	 */
 	public function __set( $query_var, $value ) {
@@ -124,7 +124,7 @@ 
 	 * @since  1.0
 	 * @access public
 	 *
-	 * @param $query_var
+	 * @param string $query_var
 	 */
 	public function __unset( $query_var ) {
 		unset( $this->args[ $query_var ] );
@@ -468,7 +468,7 @@ 
 	 * @param string   $order
 	 * @param WP_Query $query
 	 *
-	 * @return mixed
+	 * @return string
 	 */
 	public function custom_orderby( $order, $query ) {
 

Spacing +144 added lines, -144 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -66,10 +66,10 @@ 
 	 *
 	 * @param  $args array The array of arguments that can be passed in and used for setting up this payment query.
 	 */
-	public function __construct( $args = array() ) {
+	public function __construct($args = array()) {
 		$defaults = array(
 			'output'          => 'payments',
-			'post_type'       => array( 'give_payment' ),
+			'post_type'       => array('give_payment'),
 			'start_date'      => false,
 			'end_date'        => false,
 			'number'          => 20,
@@ -96,7 +96,7 @@ 
 			'count'           => false,
 		);
 
-		$this->args = $this->_args = wp_parse_args( $args, $defaults );
+		$this->args = $this->_args = wp_parse_args($args, $defaults);
 
 		$this->init();
 	}
@@ -110,11 +110,11 @@ 
 	 * @param $query_var
 	 * @param $value
 	 */
-	public function __set( $query_var, $value ) {
-		if ( in_array( $query_var, array( 'meta_query', 'tax_query' ) ) ) {
-			$this->args[ $query_var ][] = $value;
+	public function __set($query_var, $value) {
+		if (in_array($query_var, array('meta_query', 'tax_query'))) {
+			$this->args[$query_var][] = $value;
 		} else {
-			$this->args[ $query_var ] = $value;
+			$this->args[$query_var] = $value;
 		}
 	}
 
@@ -126,8 +126,8 @@ 
 	 *
 	 * @param $query_var
 	 */
-	public function __unset( $query_var ) {
-		unset( $this->args[ $query_var ] );
+	public function __unset($query_var) {
+		unset($this->args[$query_var]);
 	}
 
 	/**
@@ -167,7 +167,7 @@ 
 		$this->give_forms();
 		$this->gateway_filter();
 
-		add_filter( 'posts_orderby', array( $this, 'custom_orderby' ), 10, 2 );
+		add_filter('posts_orderby', array($this, 'custom_orderby'), 10, 2);
 
 		/**
 		 * Fires after setup filters.
@@ -176,7 +176,7 @@ 
 		 *
 		 * @param Give_Payments_Query $this Payments query object.
 		 */
-		do_action( 'give_pre_get_payments', $this );
+		do_action('give_pre_get_payments', $this);
 	}
 
 	/**
@@ -186,7 +186,7 @@ 
 	 * @access private
 	 */
 	private function unset_filters() {
-		remove_filter( 'posts_orderby', array( $this, 'custom_orderby' ) );
+		remove_filter('posts_orderby', array($this, 'custom_orderby'));
 
 		/**
 		 * Fires after retrieving payments.
@@ -195,7 +195,7 @@ 
 		 *
 		 * @param Give_Payments_Query $this Payments query object.
 		 */
-		do_action( 'give_post_get_payments', $this );
+		do_action('give_post_get_payments', $this);
 	}
 
 
@@ -214,11 +214,11 @@ 
 	public function get_payments() {
 		global $post;
 
-		$cache_key      = Give_Cache::get_key( 'give_payment_query', $this->args, false );
-		$this->payments = Give_Cache::get_db_query( $cache_key );
+		$cache_key      = Give_Cache::get_key('give_payment_query', $this->args, false);
+		$this->payments = Give_Cache::get_db_query($cache_key);
 
 		// Return cached result.
-		if ( ! is_null( $this->payments ) ) {
+		if ( ! is_null($this->payments)) {
 			return $this->payments;
 		}
 
@@ -226,7 +226,7 @@ 
 		// Modify the query/query arguments before we retrieve payments.
 		$this->set_filters();
 
-		$query          = new WP_Query( $this->args );
+		$query          = new WP_Query($this->args);
 		$this->payments = array();
 
 		$custom_output = array(
@@ -234,32 +234,32 @@ 
 			'give_payments',
 		);
 
-		if ( ! in_array( $this->args['output'], $custom_output ) ) {
+		if ( ! in_array($this->args['output'], $custom_output)) {
 			return $query->posts;
 		}
 
-		if ( $query->have_posts() ) {
+		if ($query->have_posts()) {
 			$previous_post = $post;
 
-			while ( $query->have_posts() ) {
+			while ($query->have_posts()) {
 				$query->the_post();
 
 				$payment_id = get_post()->ID;
-				$payment    = new Give_Payment( $payment_id );
+				$payment    = new Give_Payment($payment_id);
 
-				$this->payments[] = apply_filters( 'give_payment', $payment, $payment_id, $this );
+				$this->payments[] = apply_filters('give_payment', $payment, $payment_id, $this);
 			}
 
 			wp_reset_postdata();
 
 			// Prevent nest loop from producing unexpected results.
-			if( $previous_post instanceof WP_Post ) {
+			if ($previous_post instanceof WP_Post) {
 				$post = $previous_post;
-				setup_postdata( $post );
+				setup_postdata($post);
 			}
 		}
 
-		Give_Cache::set_db_query( $cache_key, $this->payments );
+		Give_Cache::set_db_query($cache_key, $this->payments);
 
 		// Remove query filters after we retrieve payments.
 		$this->unset_filters();
@@ -278,31 +278,31 @@ 
 	public function get_payment_by_group() {
 		global $wpdb;
 
-		$allowed_groups = array( 'post_status' );
+		$allowed_groups = array('post_status');
 		$result         = array();
 
 
-		if ( in_array( $this->args['group_by'], $allowed_groups ) ) {
+		if (in_array($this->args['group_by'], $allowed_groups)) {
 			// Set only count in result.
-			if ( $this->args['count'] ) {
+			if ($this->args['count']) {
 
 				$this->set_filters();
 
-				$new_results = $wpdb->get_results( $this->get_sql(), ARRAY_N );
+				$new_results = $wpdb->get_results($this->get_sql(), ARRAY_N);
 
 				$this->unset_filters();
 
-				foreach ( $new_results as $results ) {
-					$result[ $results[0] ] = $results[1];
+				foreach ($new_results as $results) {
+					$result[$results[0]] = $results[1];
 				}
 
-				switch ( $this->args['group_by'] ) {
+				switch ($this->args['group_by']) {
 					case 'post_status':
 
 						/* @var Give_Payment $donation */
-						foreach ( give_get_payment_status_keys() as $status ) {
-							if ( ! isset( $result[ $status ] ) ) {
-								$result[ $status ] = 0;
+						foreach (give_get_payment_status_keys() as $status) {
+							if ( ! isset($result[$status])) {
+								$result[$status] = 0;
 							}
 						}
 
@@ -312,8 +312,8 @@ 
 				$donations = $this->get_payments();
 
 				/* @var $donation Give_Payment */
-				foreach ( $donations as $donation ) {
-					$result[ $donation->{$this->args['group_by']} ][] = $donation;
+				foreach ($donations as $donation) {
+					$result[$donation->{$this->args['group_by']}][] = $donation;
 				}
 			}
 		}
@@ -324,7 +324,7 @@ 
 		 *
 		 * @since 1.8.17
 		 */
-		return apply_filters( 'give_get_payment_by_group', $result, $this );
+		return apply_filters('give_get_payment_by_group', $result, $this);
 	}
 
 	/**
@@ -336,29 +336,29 @@ 
 	 * @return void
 	 */
 	public function date_filter_pre() {
-		if ( ! ( $this->args['start_date'] || $this->args['end_date'] ) ) {
+		if ( ! ($this->args['start_date'] || $this->args['end_date'])) {
 			return;
 		}
 
-		$this->setup_dates( $this->args['start_date'], $this->args['end_date'] );
-		$is_start_date = property_exists( __CLASS__, 'start_date' );
-		$is_end_date   = property_exists( __CLASS__, 'end_date' );
+		$this->setup_dates($this->args['start_date'], $this->args['end_date']);
+		$is_start_date = property_exists(__CLASS__, 'start_date');
+		$is_end_date   = property_exists(__CLASS__, 'end_date');
 
-		if ( $is_start_date || $is_end_date ) {
+		if ($is_start_date || $is_end_date) {
 			$date_query = array();
 
-			if ( $is_start_date && ! is_wp_error( $this->start_date ) ) {
-				$date_query['after'] = date( 'Y-m-d H:i:s', $this->start_date );
+			if ($is_start_date && ! is_wp_error($this->start_date)) {
+				$date_query['after'] = date('Y-m-d H:i:s', $this->start_date);
 			}
 
-			if ( $is_end_date && ! is_wp_error( $this->end_date ) ) {
-				$date_query['before'] = date( 'Y-m-d H:i:s', $this->end_date );
+			if ($is_end_date && ! is_wp_error($this->end_date)) {
+				$date_query['before'] = date('Y-m-d H:i:s', $this->end_date);
 			}
 
 			// Include Start Date and End Date while querying.
 			$date_query['inclusive'] = true;
 
-			$this->__set( 'date_query', $date_query );
+			$this->__set('date_query', $date_query);
 
 		}
 	}
@@ -372,12 +372,12 @@ 
 	 * @return void
 	 */
 	public function status() {
-		if ( ! isset( $this->args['status'] ) ) {
+		if ( ! isset($this->args['status'])) {
 			return;
 		}
 
-		$this->__set( 'post_status', $this->args['status'] );
-		$this->__unset( 'status' );
+		$this->__set('post_status', $this->args['status']);
+		$this->__unset('status');
 	}
 
 	/**
@@ -389,12 +389,12 @@ 
 	 * @return void
 	 */
 	public function page() {
-		if ( ! isset( $this->args['page'] ) ) {
+		if ( ! isset($this->args['page'])) {
 			return;
 		}
 
-		$this->__set( 'paged', $this->args['page'] );
-		$this->__unset( 'page' );
+		$this->__set('paged', $this->args['page']);
+		$this->__unset('page');
 	}
 
 	/**
@@ -407,17 +407,17 @@ 
 	 */
 	public function per_page() {
 
-		if ( ! isset( $this->args['number'] ) ) {
+		if ( ! isset($this->args['number'])) {
 			return;
 		}
 
-		if ( $this->args['number'] == - 1 ) {
-			$this->__set( 'nopaging', true );
+		if ($this->args['number'] == - 1) {
+			$this->__set('nopaging', true);
 		} else {
-			$this->__set( 'posts_per_page', $this->args['number'] );
+			$this->__set('posts_per_page', $this->args['number']);
 		}
 
-		$this->__unset( 'number' );
+		$this->__unset('number');
 	}
 
 	/**
@@ -429,12 +429,12 @@ 
 	 * @return void
 	 */
 	public function month() {
-		if ( ! isset( $this->args['month'] ) ) {
+		if ( ! isset($this->args['month'])) {
 			return;
 		}
 
-		$this->__set( 'monthnum', $this->args['month'] );
-		$this->__unset( 'month' );
+		$this->__set('monthnum', $this->args['month']);
+		$this->__unset('month');
 	}
 
 	/**
@@ -446,23 +446,23 @@ 
 	 * @return void
 	 */
 	public function orderby() {
-		switch ( $this->args['orderby'] ) {
+		switch ($this->args['orderby']) {
 			case 'amount':
-				$this->__set( 'orderby', 'meta_value_num' );
-				$this->__set( 'meta_key', '_give_payment_total' );
+				$this->__set('orderby', 'meta_value_num');
+				$this->__set('meta_key', '_give_payment_total');
 				break;
 
 			case 'status':
-				$this->__set( 'orderby', 'post_status' );
+				$this->__set('orderby', 'post_status');
 				break;
 
 			case 'donation_form':
-				$this->__set( 'orderby', 'meta_value' );
-				$this->__set( 'meta_key', '_give_payment_form_title' );
+				$this->__set('orderby', 'meta_value');
+				$this->__set('meta_key', '_give_payment_form_title');
 				break;
 
 			default:
-				$this->__set( 'orderby', $this->args['orderby'] );
+				$this->__set('orderby', $this->args['orderby']);
 				break;
 		}
 	}
@@ -479,19 +479,19 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function custom_orderby( $order, $query ) {
+	public function custom_orderby($order, $query) {
 
-		if ( ! empty( $query->query['post_type'] ) ) {
-			$post_types = is_array( $query->query['post_type'] ) ? $query->query['post_type'] : array( $query->query['post_type'] );
+		if ( ! empty($query->query['post_type'])) {
+			$post_types = is_array($query->query['post_type']) ? $query->query['post_type'] : array($query->query['post_type']);
 
-			if ( ! in_array( 'give_payment', $post_types ) || ! isset( $query->query['orderby'] ) || is_array( $query->query['orderby'] ) ) {
+			if ( ! in_array('give_payment', $post_types) || ! isset($query->query['orderby']) || is_array($query->query['orderby'])) {
 				return $order;
 			}
 
 			global $wpdb;
-			switch ( $query->query['orderby'] ) {
+			switch ($query->query['orderby']) {
 				case 'post_status':
-					$order = $wpdb->posts . '.post_status ' . strtoupper( $query->query['order'] );
+					$order = $wpdb->posts.'.post_status '.strtoupper($query->query['order']);
 					break;
 			}
 		}
@@ -508,11 +508,11 @@ 
 	 * @return void
 	 */
 	public function user() {
-		if ( is_null( $this->args['user'] ) ) {
+		if (is_null($this->args['user'])) {
 			return;
 		}
 
-		if ( is_numeric( $this->args['user'] ) ) {
+		if (is_numeric($this->args['user'])) {
 			$user_key = '_give_payment_donor_id';
 		} else {
 			$user_key = '_give_payment_donor_email';
@@ -534,16 +534,16 @@ 
 	 * @return  void
 	 */
 	public function donor() {
-		if ( is_null( $this->args['donor'] ) || ! is_numeric( $this->args['donor'] ) ) {
+		if (is_null($this->args['donor']) || ! is_numeric($this->args['donor'])) {
 			return;
 		}
 
 		$donor_meta_type = Give()->donor_meta->meta_type;
 
-		$this->__set( 'meta_query', array(
+		$this->__set('meta_query', array(
 			'key'   => "_give_payment_{$donor_meta_type}_id",
 			'value' => (int) $this->args['donor'],
-		) );
+		));
 	}
 
 	/**
@@ -556,33 +556,33 @@ 
 	 */
 	public function search() {
 
-		if ( ! isset( $this->args['s'] ) ) {
+		if ( ! isset($this->args['s'])) {
 			return;
 		}
 
-		$search = trim( $this->args['s'] );
+		$search = trim($this->args['s']);
 
-		if ( empty( $search ) ) {
+		if (empty($search)) {
 			return;
 		}
 
-		$is_email = is_email( $search ) || strpos( $search, '@' ) !== false;
-		$is_user  = strpos( $search, strtolower( 'user:' ) ) !== false;
+		$is_email = is_email($search) || strpos($search, '@') !== false;
+		$is_user  = strpos($search, strtolower('user:')) !== false;
 
-		if ( ! empty( $this->args['search_in_notes'] ) ) {
+		if ( ! empty($this->args['search_in_notes'])) {
 
-			$notes = give_get_payment_notes( 0, $search );
+			$notes = give_get_payment_notes(0, $search);
 
-			if ( ! empty( $notes ) ) {
+			if ( ! empty($notes)) {
 
-				$payment_ids = wp_list_pluck( (array) $notes, 'comment_post_ID' );
+				$payment_ids = wp_list_pluck((array) $notes, 'comment_post_ID');
 
-				$this->__set( 'post__in', $payment_ids );
+				$this->__set('post__in', $payment_ids);
 			}
 
-			$this->__unset( 's' );
+			$this->__unset('s');
 
-		} elseif ( $is_email || strlen( $search ) == 32 ) {
+		} elseif ($is_email || strlen($search) == 32) {
 
 			$key         = $is_email ? '_give_payment_donor_email' : '_give_payment_purchase_key';
 			$search_meta = array(
@@ -591,40 +591,40 @@ 
 				'compare' => 'LIKE',
 			);
 
-			$this->__set( 'meta_query', $search_meta );
-			$this->__unset( 's' );
+			$this->__set('meta_query', $search_meta);
+			$this->__unset('s');
 
-		} elseif ( $is_user ) {
+		} elseif ($is_user) {
 
 			$search_meta = array(
 				'key'   => '_give_payment_donor_id',
-				'value' => trim( str_replace( 'user:', '', strtolower( $search ) ) ),
+				'value' => trim(str_replace('user:', '', strtolower($search))),
 			);
 
-			$this->__set( 'meta_query', $search_meta );
+			$this->__set('meta_query', $search_meta);
 
-			$this->__unset( 's' );
+			$this->__unset('s');
 
-		} elseif ( is_numeric( $search ) ) {
+		} elseif (is_numeric($search)) {
 
-			$post = get_post( $search );
+			$post = get_post($search);
 
-			if ( is_object( $post ) && $post->post_type == 'give_payment' ) {
+			if (is_object($post) && $post->post_type == 'give_payment') {
 
 				$arr   = array();
 				$arr[] = $search;
-				$this->__set( 'post__in', $arr );
-				$this->__unset( 's' );
+				$this->__set('post__in', $arr);
+				$this->__unset('s');
 			}
-		} elseif ( '#' == substr( $search, 0, 1 ) ) {
+		} elseif ('#' == substr($search, 0, 1)) {
 
-			$search = str_replace( '#:', '', $search );
-			$search = str_replace( '#', '', $search );
-			$this->__set( 'give_forms', $search );
-			$this->__unset( 's' );
+			$search = str_replace('#:', '', $search);
+			$search = str_replace('#', '', $search);
+			$this->__set('give_forms', $search);
+			$this->__unset('s');
 
 		} else {
-			$this->__set( 's', $search );
+			$this->__set('s', $search);
 
 		}
 
@@ -639,8 +639,8 @@ 
 	 * @return void
 	 */
 	public function mode() {
-		if ( empty( $this->args['mode'] ) || $this->args['mode'] == 'all' ) {
-			$this->__unset( 'mode' );
+		if (empty($this->args['mode']) || $this->args['mode'] == 'all') {
+			$this->__unset('mode');
 
 			return;
 		}
@@ -662,10 +662,10 @@ 
 	 * @return void
 	 */
 	public function children() {
-		if ( empty( $this->args['children'] ) ) {
-			$this->__set( 'post_parent', 0 );
+		if (empty($this->args['children'])) {
+			$this->__set('post_parent', 0);
 		}
-		$this->__unset( 'children' );
+		$this->__unset('children');
 	}
 
 	/**
@@ -678,13 +678,13 @@ 
 	 */
 	public function give_forms() {
 
-		if ( empty( $this->args['give_forms'] ) ) {
+		if (empty($this->args['give_forms'])) {
 			return;
 		}
 
 		$compare = '=';
 
-		if ( is_array( $this->args['give_forms'] ) ) {
+		if (is_array($this->args['give_forms'])) {
 			$compare = 'IN';
 		}
 
@@ -697,7 +697,7 @@ 
 			)
 		);
 
-		$this->__unset( 'give_forms' );
+		$this->__unset('give_forms');
 
 	}
 
@@ -711,13 +711,13 @@ 
 	 */
 	public function gateway_filter() {
 
-		if ( empty( $this->args['gateway'] ) ) {
+		if (empty($this->args['gateway'])) {
 			return;
 		}
 
 		$compare = '=';
 
-		if ( is_array( $this->args['gateway'] ) ) {
+		if (is_array($this->args['gateway'])) {
 			$compare = 'IN';
 		}
 
@@ -731,7 +731,7 @@ 
 			)
 		);
 
-		$this->__unset( 'gateway' );
+		$this->__unset('gateway');
 
 	}
 
@@ -751,9 +751,9 @@ 
 		global $wpdb;
 
 		$where = "WHERE {$wpdb->posts}.post_type = 'give_payment'";
-		$where .= " AND {$wpdb->posts}.post_status IN ('" . implode( "','", $this->args['post_status'] ) . "')";
+		$where .= " AND {$wpdb->posts}.post_status IN ('".implode("','", $this->args['post_status'])."')";
 
-		if( is_numeric( $this->args['post_parent'] ) ) {
+		if (is_numeric($this->args['post_parent'])) {
 			$where .= " AND {$wpdb->posts}.post_parent={$this->args['post_parent']}";
 		}
 
@@ -762,42 +762,42 @@ 
 		$group_by = '';
 
 		// Set group by.
-		if ( ! empty( $this->args['group_by'] ) ) {
+		if ( ! empty($this->args['group_by'])) {
 			$group_by = "GROUP BY {$wpdb->posts}.{$this->args['group_by']}";
 		}
 
 		// Set offset.
 		if (
-			empty( $this->args['nopaging'] ) &&
-			empty( $this->args['offset'] ) &&
-			( ! empty( $this->args['page'] ) && 0 < $this->args['page'] )
+			empty($this->args['nopaging']) &&
+			empty($this->args['offset']) &&
+			( ! empty($this->args['page']) && 0 < $this->args['page'])
 		) {
-			$this->args['offset'] = $this->args['posts_per_page'] * ( $this->args['page'] - 1 );
+			$this->args['offset'] = $this->args['posts_per_page'] * ($this->args['page'] - 1);
 		}
 
 		// Set fields.
 		$fields = "{$wpdb->posts}.*";
-		if ( ! empty( $this->args['fields'] ) && 'all' !== $this->args['fields'] ) {
-			if ( is_string( $this->args['fields'] ) ) {
+		if ( ! empty($this->args['fields']) && 'all' !== $this->args['fields']) {
+			if (is_string($this->args['fields'])) {
 				$fields = "{$wpdb->posts}.{$this->args['fields']}";
-			} elseif ( is_array( $this->args['fields'] ) ) {
-				$fields = "{$wpdb->posts}." . implode( " , {$wpdb->posts}.", $this->args['fields'] );
+			} elseif (is_array($this->args['fields'])) {
+				$fields = "{$wpdb->posts}.".implode(" , {$wpdb->posts}.", $this->args['fields']);
 			}
 		}
 
 		// Set count.
-		if ( ! empty( $this->args['count'] ) ) {
+		if ( ! empty($this->args['count'])) {
 			$fields = "COUNT({$wpdb->posts}.ID)";
 
-			if ( ! empty( $this->args['group_by'] ) ) {
+			if ( ! empty($this->args['group_by'])) {
 				$fields = "{$wpdb->posts}.{$this->args['group_by']}, {$fields}";
 			}
 		}
 
 		// Date query.
-		if ( ! empty( $this->args['date_query'] ) ) {
-			$date_query_obj = new WP_Date_Query( $this->args['date_query'] );
-			$where          .= str_replace(
+		if ( ! empty($this->args['date_query'])) {
+			$date_query_obj = new WP_Date_Query($this->args['date_query']);
+			$where .= str_replace(
 				array(
 					"\n",
 					'(   (',
@@ -813,22 +813,22 @@ 
 		}
 
 		// Meta query.
-		if ( ! empty( $this->args['meta_query'] ) ) {
-			$meta_query_obj = new WP_Meta_Query( $this->args['meta_query'] );
-			$where          = implode( ' ', $meta_query_obj->get_sql( 'post', $wpdb->posts, 'ID' ) ) . " {$where}";
-			$where          = Give()->payment_meta->__rename_meta_table_name( $where, 'posts_where' );
+		if ( ! empty($this->args['meta_query'])) {
+			$meta_query_obj = new WP_Meta_Query($this->args['meta_query']);
+			$where          = implode(' ', $meta_query_obj->get_sql('post', $wpdb->posts, 'ID'))." {$where}";
+			$where          = Give()->payment_meta->__rename_meta_table_name($where, 'posts_where');
 		}
 
 		// Set sql query.
 		$sql = $wpdb->prepare(
 			"SELECT {$fields} FROM {$wpdb->posts} LIMIT %d,%d;",
-			absint( $this->args['offset'] ),
-			( empty( $this->args['nopaging'] ) ? absint( $this->args['posts_per_page'] ) : 99999999999 )
+			absint($this->args['offset']),
+			(empty($this->args['nopaging']) ? absint($this->args['posts_per_page']) : 99999999999)
 		);
 
 		// $where, $orderby and order already prepared query they can generate notice if you re prepare them in above.
 		// WordPress consider LIKE condition as placeholder if start with s,f, or d.
-		$sql = str_replace( 'LIMIT', "{$where} {$group_by} {$orderby} {$this->args['order']} LIMIT", $sql );
+		$sql = str_replace('LIMIT', "{$where} {$group_by} {$orderby} {$this->args['order']} LIMIT", $sql);
 
 		return $sql;
 	}

includes/admin/tools/logs/class-gateway-error-logs-list-table.php

Spacing +52 added lines, -52 removed lines

@@ -9,13 +9,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded.
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -44,11 +44,11 @@ 
 	 */
 	public function __construct() {
 		// Set parent defaults.
-		parent::__construct( array(
-			'singular' => give_get_forms_label_singular(),    // Singular name of the listed records.
-			'plural'   => give_get_forms_label_plural(),        // Plural name of the listed records.
-			'ajax'     => false,// Does this table support ajax?.
-		) );
+		parent::__construct(array(
+			'singular' => give_get_forms_label_singular(), // Singular name of the listed records.
+			'plural'   => give_get_forms_label_plural(), // Plural name of the listed records.
+			'ajax'     => false, // Does this table support ajax?.
+		));
 	}
 
 	/**
@@ -62,19 +62,19 @@ 
 	 *
 	 * @return string Column Name.
 	 */
-	public function column_default( $item, $column_name ) {
+	public function column_default($item, $column_name) {
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 			case 'ID' :
 				return $item['ID_label'];
 			case 'payment_id' :
-				return empty( $item['payment_id'] ) ? esc_html__( 'n/a', 'give' ) : sprintf( "<a href=\"%s\" target=\"_blank\">{$item['payment_id']}</a>", get_edit_post_link( $item['payment_id'] ) );
+				return empty($item['payment_id']) ? esc_html__('n/a', 'give') : sprintf("<a href=\"%s\" target=\"_blank\">{$item['payment_id']}</a>", get_edit_post_link($item['payment_id']));
 			case 'gateway' :
-				return empty( $item['gateway'] ) ? esc_html__( 'n/a', 'give' ) : $item['gateway'];
+				return empty($item['gateway']) ? esc_html__('n/a', 'give') : $item['gateway'];
 			case 'error' :
-				return esc_html( $item['log_title'] );
+				return esc_html($item['log_title']);
 			default:
-				return $item[ $column_name ];
+				return $item[$column_name];
 		}
 	}
 
@@ -88,35 +88,35 @@ 
 	 *
 	 * @return void
 	 */
-	public function column_message( $item ) {
+	public function column_message($item) {
 		?>
 		<?php
-		echo Give()->tooltips->render_link( array(
-			'label'       => __( 'View Log Message', 'give' ),
+		echo Give()->tooltips->render_link(array(
+			'label'       => __('View Log Message', 'give'),
 			'tag_content' => '<span class="dashicons dashicons-visibility"></span>',
 			'link'        => "#TB_inline?width=640&amp;inlineId=log-message-{$item['ID']}",
 			'attributes'  => array(
 				'class' => 'thickbox give-error-log-details-link button button-small',
 			),
-		) );
+		));
 		?>
 		<div id="log-message-<?php echo $item['ID']; ?>" style="display:none;">
 			<?php
 
-			$serialized = strpos( $item['log_content'], '{"' );
+			$serialized = strpos($item['log_content'], '{"');
 
 			// Check to see if the log message contains serialized information
-			if ( $serialized !== false ) {
-				$length = strlen( $item['log_content'] ) - $serialized;
-				$intro  = substr( $item['log_content'], 0, - $length );
-				$data   = substr( $item['log_content'], $serialized, strlen( $item['log_content'] ) - 1 );
+			if ($serialized !== false) {
+				$length = strlen($item['log_content']) - $serialized;
+				$intro  = substr($item['log_content'], 0, - $length);
+				$data   = substr($item['log_content'], $serialized, strlen($item['log_content']) - 1);
 
-				echo wpautop( $intro );
-				echo wpautop( '<strong>' . esc_html__( 'Log data:', 'give' ) . '</strong>' );
-				echo '<div style="word-wrap: break-word;">' . wpautop( $data ) . '</div>';
+				echo wpautop($intro);
+				echo wpautop('<strong>'.esc_html__('Log data:', 'give').'</strong>');
+				echo '<div style="word-wrap: break-word;">'.wpautop($data).'</div>';
 			} else {
 				// No serialized data found
-				echo wpautop( $item['log_content'] );
+				echo wpautop($item['log_content']);
 			}
 			?>
 		</div>
@@ -132,12 +132,12 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'ID'         => esc_html__( 'Log ID', 'give' ),
-			'error'      => esc_html__( 'Error', 'give' ),
-			'gateway'    => esc_html__( 'Gateway', 'give' ),
-			'payment_id' => esc_html__( 'Donation ID', 'give' ),
-			'date'       => esc_html__( 'Date', 'give' ),
-			'message'    => esc_html__( 'Details', 'give' ),
+			'ID'         => esc_html__('Log ID', 'give'),
+			'error'      => esc_html__('Error', 'give'),
+			'gateway'    => esc_html__('Gateway', 'give'),
+			'payment_id' => esc_html__('Donation ID', 'give'),
+			'date'       => esc_html__('Date', 'give'),
+			'message'    => esc_html__('Details', 'give'),
 		);
 
 		return $columns;
@@ -151,7 +151,7 @@ 
 	 * @return int Current page number
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 	/**
@@ -161,7 +161,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	public function bulk_actions( $which = '' ) {
+	public function bulk_actions($which = '') {
 		give_log_views();
 	}
 
@@ -176,7 +176,7 @@ 
 	public function get_logs() {
 		// Prevent the queries from getting cached.
 		// Without this there are occasional memory issues for some installs.
-		wp_suspend_cache_addition( true );
+		wp_suspend_cache_addition(true);
 
 		$logs_data = array();
 		$paged     = $this->get_paged();
@@ -186,17 +186,17 @@ 
 			'posts_per_page' => $this->per_page,
 		);
 
-		$logs = Give()->logs->get_connected_logs( $log_query );
+		$logs = Give()->logs->get_connected_logs($log_query);
 
-		if ( $logs ) {
-			foreach ( $logs as $log ) {
+		if ($logs) {
+			foreach ($logs as $log) {
 
 				$logs_data[] = array(
 					'ID'          => $log->ID,
-					'ID_label'    => '<span class=\'give-item-label give-item-label-gray\'>' . $log->ID . '</span>',
+					'ID_label'    => '<span class=\'give-item-label give-item-label-gray\'>'.$log->ID.'</span>',
 					'payment_id'  => $log->log_parent,
 					'error'       => 'error',
-					'gateway'     => give_get_payment_gateway( $log->log_parent ),
+					'gateway'     => give_get_payment_gateway($log->log_parent),
 					'date'        => $log->log_date,
 					'log_content' => $log->log_content,
 					'log_title'   => $log->log_title,
@@ -220,19 +220,19 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
-		if ( 'top' === $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+	protected function display_tablenav($which) {
+		if ('top' === $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav <?php echo esc_attr($which); ?>">
 
 			<div class="alignleft actions bulkactions">
-				<?php $this->bulk_actions( $which ); ?>
+				<?php $this->bulk_actions($which); ?>
 			</div>
 			<?php
-			$this->extra_tablenav( $which );
-			$this->pagination( $which );
+			$this->extra_tablenav($which);
+			$this->pagination($which);
 			?>
 
 			<br class="clear"/>
@@ -258,14 +258,14 @@ 
 		$columns               = $this->get_columns();
 		$hidden                = array(); // No hidden columns
 		$sortable              = $this->get_sortable_columns();
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 		$this->items           = $this->get_logs();
-		$total_items           = Give()->logs->get_log_count( 0, 'gateway_error' );
+		$total_items           = Give()->logs->get_log_count(0, 'gateway_error');
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 				'total_items' => $total_items,
 				'per_page'    => $this->per_page,
-				'total_pages' => ceil( $total_items / $this->per_page ),
+				'total_pages' => ceil($total_items / $this->per_page),
 			)
 		);
 	}

includes/admin/emails/class-email-access-email.php

Indentation +4 added lines, -4 removed lines

@@ -175,10 +175,10 @@
 		 */
 		public function get_default_email_message() {
 			$message = sprintf(
-				           __( 'Please click the link to access your donation history on <a target="_blank" href="%1$s">%1$s</a>. If you did not request this email, please contact <a href="mailto:%2$s">%2$s</a>.', 'give' ),
-				           get_bloginfo( 'url' ),
-				           get_bloginfo( 'admin_email' )
-			           ) . "\n\n";
+						   __( 'Please click the link to access your donation history on <a target="_blank" href="%1$s">%1$s</a>. If you did not request this email, please contact <a href="mailto:%2$s">%2$s</a>.', 'give' ),
+						   get_bloginfo( 'url' ),
+						   get_bloginfo( 'admin_email' )
+					   ) . "\n\n";
 			$message .= '{email_access_link}' . "\n\n";
 			$message .= "\n\n";
 			$message .= __( 'Sincerely,', 'give' ) . "\n";

Spacing +52 added lines, -52 removed lines

@@ -11,11 +11,11 @@ 
  */
 
 // Exit if access directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! class_exists( 'Give_Email_Access_Email' ) ) :
+if ( ! class_exists('Give_Email_Access_Email')) :
 
 	/**
 	 * Give_Email_Access_Email
@@ -31,31 +31,31 @@ 
 		 * @since  2.0
 		 */
 		public function init() {
-			$this->load( array(
+			$this->load(array(
 				'id'                           => 'email-access',
-				'label'                        => __( 'Email access', 'give' ),
-				'description'                  => __( 'Sent when donors request access to their donation history using only their email as verification. (See Settings > General > Access Control)', 'give' ),
-				'notification_status'          => give_get_option( 'email_access', 'disabled' ),
+				'label'                        => __('Email access', 'give'),
+				'description'                  => __('Sent when donors request access to their donation history using only their email as verification. (See Settings > General > Access Control)', 'give'),
+				'notification_status'          => give_get_option('email_access', 'disabled'),
 				'form_metabox_setting'         => false,
 				'notification_status_editable' => false,
 				'email_tag_context'            => 'donor',
-				'recipient_group_name'         => __( 'Donor', 'give' ),
-				'default_email_subject'        => sprintf( __( 'Please confirm your email for %s', 'give' ), get_bloginfo( 'url' ) ),
+				'recipient_group_name'         => __('Donor', 'give'),
+				'default_email_subject'        => sprintf(__('Please confirm your email for %s', 'give'), get_bloginfo('url')),
 				'default_email_message'        => $this->get_default_email_message(),
-				'default_email_header'         => __( 'Confirm Email', 'give' ),
+				'default_email_header'         => __('Confirm Email', 'give'),
 				'notices' => array(
 					'non-notification-status-editable' => sprintf(
 						'%1$s <a href="%2$s">%3$s &raquo;</a>',
-						__( 'This notification is automatically toggled based on whether the email access is enabled or not.', 'give' ),
-						esc_url( admin_url('edit.php?post_type=give_forms&page=give-settings&tab=general&section=access-control') ),
-						__( 'Edit Setting', 'give' )
+						__('This notification is automatically toggled based on whether the email access is enabled or not.', 'give'),
+						esc_url(admin_url('edit.php?post_type=give_forms&page=give-settings&tab=general&section=access-control')),
+						__('Edit Setting', 'give')
 					)
 				),
-			) );
+			));
 
-			add_filter( "give_{$this->config['id']}_email_notification", array( $this, 'setup_email_notification' ), 10, 2 );
-			add_action( 'give_save_settings_give_settings', array( $this, 'set_notification_status' ), 10, 2 );
-			add_filter( 'give_email_preview_header', array( $this, 'email_preview_header' ), 10, 2 );
+			add_filter("give_{$this->config['id']}_email_notification", array($this, 'setup_email_notification'), 10, 2);
+			add_action('give_save_settings_give_settings', array($this, 'set_notification_status'), 10, 2);
+			add_filter('give_email_preview_header', array($this, 'email_preview_header'), 10, 2);
 		}
 
 		/**
@@ -68,11 +68,11 @@ 
 		 *
 		 * @return string
 		 */
-		public function get_email_subject( $form_id = null ) {
+		public function get_email_subject($form_id = null) {
 			$subject = wp_strip_all_tags(
 				Give_Email_Notification_Util::get_value(
 					$this,
-					Give_Email_Setting_Field::get_prefix( $this, $form_id ) . 'email_subject',
+					Give_Email_Setting_Field::get_prefix($this, $form_id).'email_subject',
 					$form_id,
 					$this->config['default_email_subject']
 				)
@@ -84,14 +84,14 @@ 
 			 *
 			 * @since 1.0
 			 */
-			$subject = apply_filters( 'give_email_access_token_subject', $subject );
+			$subject = apply_filters('give_email_access_token_subject', $subject);
 
 			/**
 			 * Filters the donation notification subject.
 			 *
 			 * @since 2.0
 			 */
-			$subject = apply_filters( "give_{$this->config['id']}_get_email_subject", $subject, $this, $form_id );
+			$subject = apply_filters("give_{$this->config['id']}_get_email_subject", $subject, $this, $form_id);
 
 			return $subject;
 		}
@@ -107,10 +107,10 @@ 
 		 *
 		 * @return string
 		 */
-		public function get_email_message( $form_id = null ) {
+		public function get_email_message($form_id = null) {
 			$message = Give_Email_Notification_Util::get_value(
 				$this,
-				Give_Email_Setting_Field::get_prefix( $this, $form_id ) . 'email_message',
+				Give_Email_Setting_Field::get_prefix($this, $form_id).'email_message',
 				$form_id,
 				$this->config['default_email_message']
 			);
@@ -121,14 +121,14 @@ 
 			 *
 			 * @since 1.0
 			 */
-			$message = apply_filters( 'give_email_access_token_message', $message );
+			$message = apply_filters('give_email_access_token_message', $message);
 
 			/**
 			 * Filter the email message
 			 *
 			 * @since 2.0
 			 */
-			$message = apply_filters( "give_{$this->config['id']}_get_default_email_message", $message, $this, $form_id );
+			$message = apply_filters("give_{$this->config['id']}_get_default_email_message", $message, $this, $form_id);
 
 			return $message;
 		}
@@ -143,7 +143,7 @@ 
 		 * @param int $form_id
 		 * @return array
 		 */
-		public function get_email_attachments( $form_id = null ) {
+		public function get_email_attachments($form_id = null) {
 			/**
 			 * Filters the donation notification email attachments.
 			 * By default, there is no attachment but plugins can hook in to provide one more multiple.
@@ -151,7 +151,7 @@ 
 			 *
 			 * @since 1.0
 			 */
-			$attachments = apply_filters( 'give_admin_donation_notification_attachments', array() );
+			$attachments = apply_filters('give_admin_donation_notification_attachments', array());
 
 			/**
 			 * Filters the donation notification email attachments.
@@ -159,7 +159,7 @@ 
 			 *
 			 * @since 2.0
 			 */
-			$attachments = apply_filters( "give_{$this->config['id']}_get_email_attachments", $attachments, $this, $form_id );
+			$attachments = apply_filters("give_{$this->config['id']}_get_email_attachments", $attachments, $this, $form_id);
 
 			return $attachments;
 		}
@@ -175,14 +175,14 @@ 
 		 */
 		public function get_default_email_message() {
 			$message = sprintf(
-				           __( 'Please click the link to access your donation history on <a target="_blank" href="%1$s">%1$s</a>. If you did not request this email, please contact <a href="mailto:%2$s">%2$s</a>.', 'give' ),
-				           get_bloginfo( 'url' ),
-				           get_bloginfo( 'admin_email' )
-			           ) . "\n\n";
-			$message .= '{email_access_link}' . "\n\n";
+				           __('Please click the link to access your donation history on <a target="_blank" href="%1$s">%1$s</a>. If you did not request this email, please contact <a href="mailto:%2$s">%2$s</a>.', 'give'),
+				           get_bloginfo('url'),
+				           get_bloginfo('admin_email')
+			           )."\n\n";
+			$message .= '{email_access_link}'."\n\n";
 			$message .= "\n\n";
-			$message .= __( 'Sincerely,', 'give' ) . "\n";
-			$message .= get_bloginfo( 'name' ) . "\n";
+			$message .= __('Sincerely,', 'give')."\n";
+			$message .= get_bloginfo('name')."\n";
 
 			/**
 			 * Filter the new donation email message
@@ -191,7 +191,7 @@ 
 			 *
 			 * @param string $message
 			 */
-			return apply_filters( "give_{$this->config['id']}_get_default_email_message", $message, $this );
+			return apply_filters("give_{$this->config['id']}_get_default_email_message", $message, $this);
 		}
 
 
@@ -207,7 +207,7 @@ 
 			 *
 			 * @since 1.0
 			 */
-			$from_name = apply_filters( 'give_donation_from_name', Give()->emails->get_from_name() );
+			$from_name = apply_filters('give_donation_from_name', Give()->emails->get_from_name());
 
 			/**
 			 * Filters the from email.
@@ -215,19 +215,19 @@ 
 			 *
 			 * @since 1.0
 			 */
-			$from_email = apply_filters( 'give_donation_from_address', Give()->emails->get_from_address() );
+			$from_email = apply_filters('give_donation_from_address', Give()->emails->get_from_address());
 
-			Give()->emails->__set( 'from_name', $from_name );
-			Give()->emails->__set( 'from_email', $from_email );
-			Give()->emails->__set( 'heading', apply_filters( 'give_email_access_token_heading', $this->get_email_header() ) );
+			Give()->emails->__set('from_name', $from_name);
+			Give()->emails->__set('from_email', $from_email);
+			Give()->emails->__set('heading', apply_filters('give_email_access_token_heading', $this->get_email_header()));
 			/**
 			 * Filters the donation notification email headers.
 			 *
 			 * @since 1.0
 			 */
-			$headers = apply_filters( 'give_admin_donation_notification_headers', Give()->emails->get_headers() );
+			$headers = apply_filters('give_admin_donation_notification_headers', Give()->emails->get_headers());
 
-			Give()->emails->__set( 'headers', $headers );
+			Give()->emails->__set('headers', $headers);
 		}
 
 		/**
@@ -241,8 +241,8 @@ 
 		 *
 		 * @return bool
 		 */
-		public function setup_email_notification( $donor_id, $email ) {
-			$donor = Give()->donors->get_donor_by( 'email', $email );
+		public function setup_email_notification($donor_id, $email) {
+			$donor = Give()->donors->get_donor_by('email', $email);
 			$this->recipient_email = $email;
 
 			// Set email data.
@@ -267,17 +267,17 @@ 
 		 * @param $update_options
 		 * @param $option_name
 		 */
-		public function set_notification_status( $update_options, $option_name ) {
+		public function set_notification_status($update_options, $option_name) {
 			// Get updated settings.
 			$update_options = give_get_settings();
 
 			if (
-				! empty( $update_options['email_access'] )
-				&& ! empty( $update_options[ "{$this->config['id']}_notification" ] )
-				&& $update_options['email_access'] !== $update_options[ "{$this->config['id']}_notification" ]
+				! empty($update_options['email_access'])
+				&& ! empty($update_options["{$this->config['id']}_notification"])
+				&& $update_options['email_access'] !== $update_options["{$this->config['id']}_notification"]
 			) {
-				$update_options[ "{$this->config['id']}_notification" ] = $update_options['email_access'];
-				update_option( $option_name, $update_options );
+				$update_options["{$this->config['id']}_notification"] = $update_options['email_access'];
+				update_option($option_name, $update_options);
 			}
 		}
 
@@ -292,8 +292,8 @@ 
 		 * @param Give_Email_Access_Email $email
 		 * @return string
 		 */
-		public function email_preview_header( $email_preview_header, $email ) {
-			if( $this->config['id'] === $email->config['id'] ) {
+		public function email_preview_header($email_preview_header, $email) {
+			if ($this->config['id'] === $email->config['id']) {
 				$email_preview_header = '';
 			}
 

includes/admin/emails/ajax-handler.php

Spacing +3 added lines, -3 removed lines

@@ -16,12 +16,12 @@
  * @since 2.0
  */
 function give_set_notification_status_handler() {
-	$notification_id = isset( $_POST['notification_id'] ) ? give_clean( $_POST['notification_id'] ) : '';
-	if ( ! empty( $notification_id ) && give_update_option( "{$notification_id}_notification", give_clean( $_POST['status'] ) ) ) {
+	$notification_id = isset($_POST['notification_id']) ? give_clean($_POST['notification_id']) : '';
+	if ( ! empty($notification_id) && give_update_option("{$notification_id}_notification", give_clean($_POST['status']))) {
 		wp_send_json_success();
 	}
 
 	wp_send_json_error();
 }
 
-add_action( 'wp_ajax_give_set_notification_status', 'give_set_notification_status_handler' );
+add_action('wp_ajax_give_set_notification_status', 'give_set_notification_status_handler');

includes/admin/emails/backward-compatibility.php

Spacing +27 added lines, -27 removed lines

@@ -11,15 +11,15 @@ 
  *
  * @return string
  */
-function _give_bc_offline_donation_instruction_notification_status( $notification_status, $email, $form_id ) {
+function _give_bc_offline_donation_instruction_notification_status($notification_status, $email, $form_id) {
 	// Bailout.
-	if ( ! $form_id ) {
+	if ( ! $form_id) {
 		return $notification_status;
 	}
 
-	if ( ! get_post_meta( $form_id, '_give_offline-donation-instruction_notification', true ) ) {
-		$old_value           = get_post_meta( $form_id, '_give_customize_offline_donations', true );
-		$notification_status = give_is_setting_enabled( $old_value, array( 'enabled', 'global' ) )
+	if ( ! get_post_meta($form_id, '_give_offline-donation-instruction_notification', true)) {
+		$old_value           = get_post_meta($form_id, '_give_customize_offline_donations', true);
+		$notification_status = give_is_setting_enabled($old_value, array('enabled', 'global'))
 			? $old_value
 			: 'global';
 	}
@@ -28,7 +28,7 @@ 
 	return $notification_status;
 }
 
-add_filter( 'give__give_offline-donation-instruction_get_notification_status', '_give_bc_offline_donation_instruction_notification_status', 10, 3 );
+add_filter('give__give_offline-donation-instruction_get_notification_status', '_give_bc_offline_donation_instruction_notification_status', 10, 3);
 
 
 /**
@@ -43,22 +43,22 @@ 
  *
  * @return mixed
  */
-function _give_bc_offline_donation_instruction_email_setting_values( $option_value, $option_name, $email, $form_id ) {
+function _give_bc_offline_donation_instruction_email_setting_values($option_value, $option_name, $email, $form_id) {
 	// Bailout.
-	if ( empty( $form_id ) || 'offline-donation-instruction' !== $email->config['id'] ) {
+	if (empty($form_id) || 'offline-donation-instruction' !== $email->config['id']) {
 		return $option_value;
 	}
 
-	switch ( $option_name ) {
+	switch ($option_name) {
 		case '_give_offline-donation-instruction_email_message':
-			if ( ! get_post_meta( $form_id, $option_name, true ) && give_is_setting_enabled( $email->get_notification_status( $form_id ) ) ) {
-				$option_value = get_post_meta( $form_id, '_give_offline_donation_email', true );
+			if ( ! get_post_meta($form_id, $option_name, true) && give_is_setting_enabled($email->get_notification_status($form_id))) {
+				$option_value = get_post_meta($form_id, '_give_offline_donation_email', true);
 			}
 			break;
 
 		case '_give_offline-donation-instruction_email_subject':
-			if ( ! get_post_meta( $form_id, $option_name, true ) && give_is_setting_enabled( $email->get_notification_status( $form_id ) ) ) {
-				$option_value = get_post_meta( $form_id, '_give_offline_donation_subject', true );
+			if ( ! get_post_meta($form_id, $option_name, true) && give_is_setting_enabled($email->get_notification_status($form_id))) {
+				$option_value = get_post_meta($form_id, '_give_offline_donation_subject', true);
 			}
 			break;
 	}
@@ -66,7 +66,7 @@ 
 	return $option_value;
 }
 
-add_filter( 'give_email_setting_value', '_give_bc_offline_donation_instruction_email_setting_values', 10, 4 );
+add_filter('give_email_setting_value', '_give_bc_offline_donation_instruction_email_setting_values', 10, 4);
 
 
 /**
@@ -80,10 +80,10 @@ 
  *
  * @return string
  */
-function _give_bc_offline_instruction_status_setting_value( $field_value, $field, $form_id ) {
-	if ( ! get_post_meta( $form_id, $field['id'], true ) ) {
-		$old_value   = get_post_meta( $form_id, '_give_customize_offline_donations', true );
-		$field_value = give_is_setting_enabled( $old_value, array( 'enabled', 'global' ) )
+function _give_bc_offline_instruction_status_setting_value($field_value, $field, $form_id) {
+	if ( ! get_post_meta($form_id, $field['id'], true)) {
+		$old_value   = get_post_meta($form_id, '_give_customize_offline_donations', true);
+		$field_value = give_is_setting_enabled($old_value, array('enabled', 'global'))
 			? $old_value
 			: 'global';
 	}
@@ -91,7 +91,7 @@ 
 	return $field_value;
 }
 
-add_filter( '_give_offline-donation-instruction_notification_field_value', '_give_bc_offline_instruction_status_setting_value', 10, 3 );
+add_filter('_give_offline-donation-instruction_notification_field_value', '_give_bc_offline_instruction_status_setting_value', 10, 3);
 
 
 /**
@@ -105,15 +105,15 @@ 
  *
  * @return string
  */
-function _offline_donation_instruction_email_subject_setting_value( $field_value, $field, $form_id ) {
-	if ( ! get_post_meta( $form_id, $field['id'], true ) ) {
-		$field_value = get_post_meta( $form_id, '_give_offline_donation_subject', true );
+function _offline_donation_instruction_email_subject_setting_value($field_value, $field, $form_id) {
+	if ( ! get_post_meta($form_id, $field['id'], true)) {
+		$field_value = get_post_meta($form_id, '_give_offline_donation_subject', true);
 	}
 
 	return $field_value;
 }
 
-add_filter( '_give_offline-donation-instruction_email_subject_field_value', '_offline_donation_instruction_email_subject_setting_value', 10, 3 );
+add_filter('_give_offline-donation-instruction_email_subject_field_value', '_offline_donation_instruction_email_subject_setting_value', 10, 3);
 
 
 /**
@@ -127,12 +127,12 @@ 
  *
  * @return string
  */
-function _give_bc_offline_donation_instruction_email_message_setting_value( $field_value, $field, $form_id ) {
-	if ( ! get_post_meta( $form_id, $field['id'], true ) ) {
-		$field_value = get_post_meta( $form_id, '_give_offline_donation_email', true );
+function _give_bc_offline_donation_instruction_email_message_setting_value($field_value, $field, $form_id) {
+	if ( ! get_post_meta($form_id, $field['id'], true)) {
+		$field_value = get_post_meta($form_id, '_give_offline_donation_email', true);
 	}
 
 	return $field_value;
 }
 
-add_filter( '_give_offline-donation-instruction_email_message_field_value', '_give_bc_offline_donation_instruction_email_message_setting_value', 10, 3 );
\ No newline at end of file
+add_filter('_give_offline-donation-instruction_email_message_field_value', '_give_bc_offline_donation_instruction_email_message_setting_value', 10, 3);
\ No newline at end of file

includes/admin/reports/reports.php

Spacing +59 added lines, -59 removed lines

@@ -16,7 +16,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -29,8 +29,8 @@ 
  * @return void
  */
 function give_reports_page() {
-	$current_page = admin_url( 'edit.php?post_type=give_forms&page=give-reports' );
-	$active_tab   = isset( $_GET['tab'] ) ? sanitize_text_field( $_GET['tab'] ) : 'earnings';
+	$current_page = admin_url('edit.php?post_type=give_forms&page=give-reports');
+	$active_tab   = isset($_GET['tab']) ? sanitize_text_field($_GET['tab']) : 'earnings';
 	$views        = give_reports_default_views();
 	?>
 	<div class="wrap give-settings-page">
@@ -38,17 +38,17 @@ 
 		<h1 class="screen-reader-text"><?php echo get_admin_page_title(); ?></h1>
 
 		<h2 class="nav-tab-wrapper">
-			<?php foreach ( $views as $tab => $label ) { ?>
-				<a href="<?php echo esc_url( add_query_arg( array(
+			<?php foreach ($views as $tab => $label) { ?>
+				<a href="<?php echo esc_url(add_query_arg(array(
 					'tab'              => $tab,
 					'settings-updated' => false,
-				), $current_page ) ); ?>" class="nav-tab <?php echo $tab === $active_tab ? esc_attr( 'nav-tab-active' ) : ''; ?>"><?php echo esc_html( $label ); ?></a>
+				), $current_page)); ?>" class="nav-tab <?php echo $tab === $active_tab ? esc_attr('nav-tab-active') : ''; ?>"><?php echo esc_html($label); ?></a>
 			<?php } ?>
-			<?php if ( current_user_can( 'export_give_reports' ) ) { ?>
-				<a href="<?php echo esc_url( add_query_arg( array(
+			<?php if (current_user_can('export_give_reports')) { ?>
+				<a href="<?php echo esc_url(add_query_arg(array(
 					'tab'              => 'export',
 					'settings-updated' => false,
-				), $current_page ) ); ?>" class="nav-tab <?php echo 'export' === $active_tab ? esc_attr( 'nav-tab-active' ) : ''; ?>"><?php esc_html_e( 'Export', 'give' ); ?></a>
+				), $current_page)); ?>" class="nav-tab <?php echo 'export' === $active_tab ? esc_attr('nav-tab-active') : ''; ?>"><?php esc_html_e('Export', 'give'); ?></a>
 			<?php }
 			/**
 			 * Fires in the report tabs.
@@ -57,7 +57,7 @@ 
 			 *
 			 * @since 1.0
 			 */
-			do_action( 'give_reports_tabs' );
+			do_action('give_reports_tabs');
 			?>
 		</h2>
 
@@ -67,10 +67,10 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_reports_page_top' );
+		do_action('give_reports_page_top');
 
 		// Set $active_tab prior to hook firing.
-		if ( in_array( $active_tab, array_keys( $views ) ) ) {
+		if (in_array($active_tab, array_keys($views))) {
 			$active_tab = 'reports';
 		}
 
@@ -79,14 +79,14 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( "give_reports_tab_{$active_tab}" );
+		do_action("give_reports_tab_{$active_tab}");
 
 		/**
 		 * Fires after the report page.
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_reports_page_bottom' );
+		do_action('give_reports_page_bottom');
 		?>
 	</div><!-- .wrap -->
 	<?php
@@ -100,12 +100,12 @@ 
  */
 function give_reports_default_views() {
 	$views = array(
-		'earnings' => esc_html__( 'Income', 'give' ),
-		'forms'    => esc_html__( 'Forms', 'give' ),
-		'gateways' => esc_html__( 'Donation Methods', 'give' ),
+		'earnings' => esc_html__('Income', 'give'),
+		'forms'    => esc_html__('Forms', 'give'),
+		'gateways' => esc_html__('Donation Methods', 'give'),
 	);
 
-	$views = apply_filters( 'give_report_views', $views );
+	$views = apply_filters('give_report_views', $views);
 
 	return $views;
 }
@@ -120,15 +120,15 @@ 
  * @since 1.0
  * @return string $view Report View
  */
-function give_get_reporting_view( $default = 'earnings' ) {
+function give_get_reporting_view($default = 'earnings') {
 
-	if ( ! isset( $_GET['view'] ) || ! in_array( $_GET['view'], array_keys( give_reports_default_views() ) ) ) {
+	if ( ! isset($_GET['view']) || ! in_array($_GET['view'], array_keys(give_reports_default_views()))) {
 		$view = $default;
 	} else {
 		$view = $_GET['view'];
 	}
 
-	return apply_filters( 'give_get_reporting_view', $view );
+	return apply_filters('give_get_reporting_view', $view);
 }
 
 /**
@@ -139,14 +139,14 @@ 
  */
 function give_reports_tab_reports() {
 
-	if( ! current_user_can( 'view_give_reports' ) ) {
-		wp_die( __( 'You do not have permission to access this report', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! current_user_can('view_give_reports')) {
+		wp_die(__('You do not have permission to access this report', 'give'), __('Error', 'give'), array('response' => 403));
 	}
 
 	$current_view = 'earnings';
 	$views        = give_reports_default_views();
 
-	if ( isset( $_GET['tab'] ) && array_key_exists( $_GET['tab'], $views ) ) {
+	if (isset($_GET['tab']) && array_key_exists($_GET['tab'], $views)) {
 		$current_view = $_GET['tab'];
 	}
 
@@ -155,10 +155,10 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( "give_reports_view_{$current_view}" );
+	do_action("give_reports_view_{$current_view}");
 }
 
-add_action( 'give_reports_tab_reports', 'give_reports_tab_reports' );
+add_action('give_reports_tab_reports', 'give_reports_tab_reports');
 
 /**
  * Renders the Reports Page Views Drop Downs
@@ -168,19 +168,19 @@ 
  */
 function give_report_views() {
 	$views        = give_reports_default_views();
-	$current_view = isset( $_GET['view'] ) ? $_GET['view'] : 'earnings';
+	$current_view = isset($_GET['view']) ? $_GET['view'] : 'earnings';
 	/**
 	 * Fires before the report page actions form.
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_report_view_actions_before' );
+	do_action('give_report_view_actions_before');
 	?>
 	<form id="give-reports-filter" method="get">
 		<select id="give-reports-view" name="view">
-			<option value="-1"><?php esc_html_e( 'Report Type', 'give' ); ?></option>
-			<?php foreach ( $views as $view_id => $label ) : ?>
-				<option value="<?php echo esc_attr( $view_id ); ?>" <?php selected( $view_id, $current_view ); ?>><?php echo $label; ?></option>
+			<option value="-1"><?php esc_html_e('Report Type', 'give'); ?></option>
+			<?php foreach ($views as $view_id => $label) : ?>
+				<option value="<?php echo esc_attr($view_id); ?>" <?php selected($view_id, $current_view); ?>><?php echo $label; ?></option>
 			<?php endforeach; ?>
 		</select>
 
@@ -192,12 +192,12 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_report_view_actions' );
+		do_action('give_report_view_actions');
 		?>
 
 		<input type="hidden" name="post_type" value="give_forms"/>
 		<input type="hidden" name="page" value="give-reports"/>
-		<?php submit_button( esc_html__( 'Show', 'give' ), 'secondary', 'submit', false ); ?>
+		<?php submit_button(esc_html__('Show', 'give'), 'secondary', 'submit', false); ?>
 	</form>
 	<?php
 	/**
@@ -205,7 +205,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_report_view_actions_after' );
+	do_action('give_report_view_actions_after');
 }
 
 /**
@@ -218,11 +218,11 @@ 
  */
 function give_reports_forms_table() {
 
-	if ( isset( $_GET['form-id'] ) ) {
+	if (isset($_GET['form-id'])) {
 		return;
 	}
 
-	include GIVE_PLUGIN_DIR . 'includes/admin/reports/class-form-reports-table.php';
+	include GIVE_PLUGIN_DIR.'includes/admin/reports/class-form-reports-table.php';
 
 	$give_table = new Give_Form_Reports_Table();
 	$give_table->prepare_items();
@@ -234,7 +234,7 @@ 
 	<?php
 }
 
-add_action( 'give_reports_view_forms', 'give_reports_forms_table' );
+add_action('give_reports_view_forms', 'give_reports_forms_table');
 
 /**
  * Renders the detailed report for a specific give form.
@@ -243,20 +243,20 @@ 
  * @return void
  */
 function give_reports_form_details() {
-	if ( ! isset( $_GET['form-id'] ) ) {
+	if ( ! isset($_GET['form-id'])) {
 		return;
 	}
 	?>
 	<div class="tablenav top reports-forms-details-wrap">
 		<div class="actions bulkactions">
-			<button onclick="history.go(-1);" class="button-secondary"><?php esc_html_e( 'Go Back', 'give' ); ?></button>
+			<button onclick="history.go(-1);" class="button-secondary"><?php esc_html_e('Go Back', 'give'); ?></button>
 		</div>
 	</div>
 	<?php
-	give_reports_graph_of_form( absint( $_GET['form-id'] ) );
+	give_reports_graph_of_form(absint($_GET['form-id']));
 }
 
-add_action( 'give_reports_view_forms', 'give_reports_form_details' );
+add_action('give_reports_view_forms', 'give_reports_form_details');
 
 /**
  * Renders the Gateways Table
@@ -267,14 +267,14 @@ 
  * @return void
  */
 function give_reports_gateways_table() {
-	include GIVE_PLUGIN_DIR . 'includes/admin/reports/class-gateways-reports-table.php';
+	include GIVE_PLUGIN_DIR.'includes/admin/reports/class-gateways-reports-table.php';
 
 	$give_table = new Give_Gateway_Reports_Table();
 	$give_table->prepare_items();
 	$give_table->display();
 }
 
-add_action( 'give_reports_view_gateways', 'give_reports_gateways_table' );
+add_action('give_reports_view_gateways', 'give_reports_gateways_table');
 
 /**
  * Renders the Reports Earnings Graphs
@@ -285,13 +285,13 @@ 
 function give_reports_earnings() {
 	?>
 	<div class="tablenav top reports-table-nav">
-		<h2 class="reports-earnings-title screen-reader-text"><?php _e( 'Income Report', 'give' ); ?></h2>
+		<h2 class="reports-earnings-title screen-reader-text"><?php _e('Income Report', 'give'); ?></h2>
 	</div>
 	<?php
 	give_reports_graph();
 }
 
-add_action( 'give_reports_view_earnings', 'give_reports_earnings' );
+add_action('give_reports_view_earnings', 'give_reports_earnings');
 
 
 /**
@@ -302,9 +302,9 @@ 
  */
 function give_estimated_monthly_stats() {
 
-	$estimated = Give_Cache::get( 'give_estimated_monthly_stats', true );
+	$estimated = Give_Cache::get('give_estimated_monthly_stats', true);
 
-	if ( false === $estimated ) {
+	if (false === $estimated) {
 
 		$estimated = array(
 			'earnings' => 0,
@@ -313,22 +313,22 @@ 
 
 		$stats = new Give_Payment_Stats;
 
-		$to_date_earnings = $stats->get_earnings( 0, 'this_month' );
-		$to_date_sales    = $stats->get_sales( 0, 'this_month' );
+		$to_date_earnings = $stats->get_earnings(0, 'this_month');
+		$to_date_sales    = $stats->get_sales(0, 'this_month');
 
-		$current_day   = date( 'd', current_time( 'timestamp' ) );
-		$current_month = date( 'n', current_time( 'timestamp' ) );
-		$current_year  = date( 'Y', current_time( 'timestamp' ) );
-		$days_in_month = cal_days_in_month( CAL_GREGORIAN, $current_month, $current_year );
+		$current_day   = date('d', current_time('timestamp'));
+		$current_month = date('n', current_time('timestamp'));
+		$current_year  = date('Y', current_time('timestamp'));
+		$days_in_month = cal_days_in_month(CAL_GREGORIAN, $current_month, $current_year);
 
-		$estimated['earnings'] = ( $to_date_earnings / $current_day ) * $days_in_month;
-		$estimated['sales']    = ( $to_date_sales / $current_day ) * $days_in_month;
+		$estimated['earnings'] = ($to_date_earnings / $current_day) * $days_in_month;
+		$estimated['sales']    = ($to_date_sales / $current_day) * $days_in_month;
 
 		// Cache for one day
-		Give_Cache::set( 'give_estimated_monthly_stats', $estimated, DAY_IN_SECONDS, true );
+		Give_Cache::set('give_estimated_monthly_stats', $estimated, DAY_IN_SECONDS, true);
 	}
 
-	return maybe_unserialize( $estimated );
+	return maybe_unserialize($estimated);
 }
 
 /**
@@ -341,7 +341,7 @@ 
 function give_reports_set_form_method() {
 	return 'get';
 }
-add_filter( 'give-reports_form_method_tab_forms', 'give_reports_set_form_method', 10 );
-add_filter( 'give-reports_form_method_tab_donors', 'give_reports_set_form_method', 10 );
+add_filter('give-reports_form_method_tab_forms', 'give_reports_set_form_method', 10);
+add_filter('give-reports_form_method_tab_donors', 'give_reports_set_form_method', 10);
 
 // @TODO: After release 1.8 Donations -> Reports generates with new setting api, so we can remove some old code from this file.

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

Braces +10 added lines, -4 removed lines

@@ -663,9 +663,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 ) ) : ?>
@@ -675,9 +678,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>

Spacing +326 added lines, -330 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'          => __( 'Donation Limit', 'give' ),
-						'description'   => __( 'Set the minimum and maximum amount for all gateways.', 'give' ),
-						'id'            => $prefix . 'custom_amount_range',
+						'name'          => __('Donation Limit', 'give'),
+						'description'   => __('Set the minimum and maximum amount for all gateways.', 'give'),
+						'id'            => $prefix.'custom_amount_range',
 						'type'          => 'donation_limit',
 						'wrapper_class' => 'give-hidden',
 						'data_type'     => 'price',
@@ -162,43 +162,43 @@ 
 							'class'       => 'give-money-field',
 						),
 						'options'       => array(
-							'display_label' => __( 'Donation Limits: ', 'give' ),
+							'display_label' => __('Donation Limits: ', 'give'),
 							'minimum'       => 1.00,
 							'maximum'       => 999999.99,
 						),
 					),
 					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(
@@ -207,126 +207,126 @@ 
 								),
 							),
 							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'    => __( 'Company Donations', 'give' ),
-							'desc'    => __( 'Do you want a Company field to appear after First Name and Last Name?', 'give' ),
-							'id'      => $prefix . 'company_field',
+							'name'    => __('Company Donations', 'give'),
+							'desc'    => __('Do you want a Company field to appear after First Name and Last Name?', 'give'),
+							'id'      => $prefix.'company_field',
 							'type'    => 'radio_inline',
 							'default' => 'global',
 							'options' => array(
-								'global' => __( 'Global Option', 'give' ),
-								'required' => __( 'Required', 'give' ),
-								'optional' => __( 'Optional', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'global' => __('Global Option', 'give'),
+								'required' => __('Required', 'give'),
+								'optional' => __('Optional', 'give'),
+								'disabled' => __('Disabled', 'give'),
 
 							),
 						),
 						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',
 						),
@@ -334,7 +334,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
@@ -345,42 +345,42 @@ 
 			/**
 			 * 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 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" or "1 of 5 donations". You can also display a donor-based goal, such as "100 of 1,000 donors have given".', '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" or "1 of 5 donations". You can also display a donor-based goal, such as "100 of 1,000 donors have given".', 'give'),
+						'id'          => $prefix.'goal_format',
 						'type'        => 'donation_form_goal',
 						'default'     => 'amount',
 						'options'     => array(
-							'amount'     => __( 'Amount Raised', 'give' ),
-							'percentage' => __( 'Percentage Raised', 'give' ),
-							'donation'   => __( 'Number of Donations', 'give' ),
-							'donors'     => __( 'Number of Donors', 'give' ),
+							'amount'     => __('Amount Raised', 'give'),
+							'percentage' => __('Percentage Raised', 'give'),
+							'donation'   => __('Number of Donations', 'give'),
+							'donors'     => __('Number of Donors', '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(
@@ -390,9 +390,9 @@ 
 						'wrapper_class' => 'give-hidden',
 					),
 					array(
-						'id'         => $prefix . 'number_of_donation_goal',
-						'name'       => __( 'Donation Goal', 'give' ),
-						'desc'       => __( 'Set the total number of donations as a goal.', 'give' ),
+						'id'         => $prefix.'number_of_donation_goal',
+						'name'       => __('Donation Goal', 'give'),
+						'desc'       => __('Set the total number of donations as a goal.', 'give'),
 						'type'       => 'number',
 						'default'    => 1,
 						'attributes' => array(
@@ -400,9 +400,9 @@ 
 						),
 					),
 					array(
-						'id'         => $prefix . 'number_of_donor_goal',
-						'name'       => __( 'Donor Goal', 'give' ),
-						'desc'       => __( 'Set the total number of donors as a goal.', 'give' ),
+						'id'         => $prefix.'number_of_donor_goal',
+						'name'       => __('Donor Goal', 'give'),
+						'desc'       => __('Set the total number of donors as a goal.', 'give'),
 						'type'       => 'number',
 						'default'    => 1,
 						'attributes' => array(
@@ -410,85 +410,85 @@ 
 						),
 					),
 					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',
 					),
@@ -496,51 +496,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',
 					),
@@ -548,18 +548,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;
@@ -576,19 +576,19 @@ 
 		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'
 		);
 
 		// Show Goal Metabox only if goal is enabled.
-		if ( give_is_setting_enabled( give_get_meta( give_get_admin_post_id(), '_give_goal_option', true ) ) ) {
+		if (give_is_setting_enabled(give_get_meta(give_get_admin_post_id(), '_give_goal_option', true))) {
 			add_meta_box(
 				'give-form-goal-stats',
-				__( 'Goal Statistics', 'give' ),
-				array( $this, 'output_goal' ),
-				array( 'give_forms' ),
+				__('Goal Statistics', 'give'),
+				array($this, 'output_goal'),
+				array('give_forms'),
 				'side',
 				'low'
 			);
@@ -607,7 +607,7 @@ 
 	function enqueue_script() {
 		global $post;
 
-		if ( is_object( $post ) && 'give_forms' === $post->post_type ) {
+		if (is_object($post) && 'give_forms' === $post->post_type) {
 
 		}
 	}
@@ -645,32 +645,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;
 				}
 			}
@@ -688,33 +688,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>
@@ -729,29 +729,29 @@ 
 					<?php endforeach; ?>
 				</ul>
 
-				<?php foreach ( $this->settings as $setting ) : ?>
-					<?php do_action( "give_before_{$setting['id']}_settings" ); ?>
+				<?php foreach ($this->settings as $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 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>
@@ -774,9 +774,9 @@ 
 	 *
 	 * @return void
 	 */
-	public function output_goal( $post ) {
+	public function output_goal($post) {
 
-		echo give_admin_form_goal_stats( $post->ID );
+		echo give_admin_form_goal_stats($post->ID);
 
 	}
 
@@ -789,9 +789,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;
 		}
 
@@ -806,13 +806,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]);
 			}
 		}
 
@@ -830,100 +830,100 @@ 
 	 *
 	 * @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 'donation_limit' :
-								$form_meta_value = $_POST[ $form_meta_key ];
+								$form_meta_value = $_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().
 
 						/**
@@ -933,38 +933,38 @@ 
 						 */
 						$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
 						);
 
 						// Range slider.
-						if ( 'donation_limit' === $setting_field['type'] ) {
+						if ('donation_limit' === $setting_field['type']) {
 
 							// Sanitize amount for db.
-							$form_meta_value = array_map( 'give_sanitize_amount_for_db', $form_meta_value );
+							$form_meta_value = array_map('give_sanitize_amount_for_db', $form_meta_value);
 
 							// Store it to form meta.
-							give_update_meta( $post_id, $form_meta_key . '_minimum', $form_meta_value['minimum'] );
-							give_update_meta( $post_id, $form_meta_key . '_maximum', $form_meta_value['maximum'] );
+							give_update_meta($post_id, $form_meta_key.'_minimum', $form_meta_value['minimum']);
+							give_update_meta($post_id, $form_meta_key.'_maximum', $form_meta_value['maximum']);
 						} else {
 							// Save data.
-							give_update_meta( $post_id, $form_meta_key, $form_meta_value );
+							give_update_meta($post_id, $form_meta_key, $form_meta_value);
 						}
 
 						// Verify and delete form meta based on the form status.
-						give_set_form_closed_status( $post_id );
+						give_set_form_closed_status($post_id);
 
 						// 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);
 	}
 
 
@@ -977,10 +977,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'];
 
 		}
@@ -997,16 +997,16 @@ 
 	 *
 	 * @return array
 	 */
-	private function get_fields_id( $setting ) {
+	private function get_fields_id($setting) {
 		$meta_keys = array();
 
 		if (
-			! empty( $setting )
-			&& array_key_exists( 'fields', $setting )
-			&& ! empty( $setting['fields'] )
+			! empty($setting)
+			&& array_key_exists('fields', $setting)
+			&& ! empty($setting['fields'])
 		) {
-			foreach ( $setting['fields'] as $field ) {
-				if ( $field_id = $this->get_field_id( $field ) ) {
+			foreach ($setting['fields'] as $field) {
+				if ($field_id = $this->get_field_id($field)) {
 					$meta_keys[] = $field_id;
 				}
 			}
@@ -1024,14 +1024,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;
 						}
 					}
@@ -1053,14 +1053,14 @@ 
 	private function get_meta_keys_from_settings() {
 		$meta_keys = array();
 
-		foreach ( $this->settings as $setting ) {
-			$meta_key = $this->get_fields_id( $setting );
+		foreach ($this->settings as $setting) {
+			$meta_key = $this->get_fields_id($setting);
 
-			if ( $this->has_sub_tab( $setting ) ) {
-				$meta_key = array_merge( $meta_key, $this->get_sub_fields_id( $setting ) );
+			if ($this->has_sub_tab($setting)) {
+				$meta_key = array_merge($meta_key, $this->get_sub_fields_id($setting));
 			}
 
-			$meta_keys = array_merge( $meta_keys, $meta_key );
+			$meta_keys = array_merge($meta_keys, $meta_key);
 		}
 
 		return $meta_keys;
@@ -1077,10 +1077,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']
 			: '';
 
@@ -1098,12 +1098,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;
 				}
@@ -1123,12 +1123,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;
 				}
@@ -1148,17 +1148,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;
 				}
@@ -1166,9 +1166,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;
 				}
 			}
@@ -1187,14 +1187,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;
@@ -1212,37 +1212,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);
 				}
 		}
 
@@ -1260,12 +1256,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/class-give-cron.php

Spacing +19 added lines, -19 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,8 +66,8 @@ 
 	 * @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'));
 	}
 
 	/**
@@ -80,11 +80,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;
@@ -112,8 +112,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');
 		}
 	}
 
@@ -126,8 +126,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');
 		}
 	}
 
@@ -141,8 +141,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;
@@ -164,9 +164,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);
 	}
 
 	/**
@@ -177,8 +177,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');
 	}
 
 	/**
@@ -189,8 +189,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');
 	}
 }