WordImpress / Give

includes/error-tracking.php

Spacing +19 added lines, -19 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,7 +25,7 @@ 
  * @return mixed array if errors are present, false if none found
  */
 function give_get_errors() {
-	return Give()->session->get( 'give_errors' );
+	return Give()->session->get('give_errors');
 }
 
 /**
@@ -42,23 +42,23 @@ 
  *
  * @return void
  */
-function give_set_error( $error_id, $error_message, $notice_args = array() ) {
+function give_set_error($error_id, $error_message, $notice_args = array()) {
 	$errors = give_get_errors();
-	if ( ! $errors ) {
+	if ( ! $errors) {
 		$errors = array();
 	}
 
-	if( is_array( $notice_args ) && ! empty( $notice_args ) ) {
-		$errors[ $error_id ] = array(
+	if (is_array($notice_args) && ! empty($notice_args)) {
+		$errors[$error_id] = array(
 			'message'     => $error_message,
 			'notice_args' => $notice_args,
 		);
 	} else {
 		// Backward compatibility v<1.8.11.
-		$errors[ $error_id ] = $error_message;
+		$errors[$error_id] = $error_message;
 	}
 
-	Give()->session->set( 'give_errors', $errors );
+	Give()->session->set('give_errors', $errors);
 }
 
 /**
@@ -69,7 +69,7 @@ 
  * @return void
  */
 function give_clear_errors() {
-	Give()->session->set( 'give_errors', null );
+	Give()->session->set('give_errors', null);
 }
 
 /**
@@ -82,19 +82,19 @@ 
  *
  * @return void
  */
-function give_unset_error( $error_id ) {
+function give_unset_error($error_id) {
 	$errors = give_get_errors();
-	if ( $errors ) {
+	if ($errors) {
 		/**
 		 * Check If $error_id exists in the array.
 		 * If exists then unset it.
 		 *
 		 * @since 1.8.13
 		 */
-		if ( isset( $errors[ $error_id ] ) ) {
-			unset( $errors[ $error_id ] );
+		if (isset($errors[$error_id])) {
+			unset($errors[$error_id]);
 		}
-		Give()->session->set( 'give_errors', $errors );
+		Give()->session->set('give_errors', $errors);
 	}
 }
 
@@ -105,7 +105,7 @@ 
  * @return string
  */
 function _give_die_handler() {
-	if ( defined( 'GIVE_UNIT_TESTS' ) ) {
+	if (defined('GIVE_UNIT_TESTS')) {
 		return '_give_die_handler';
 	} else {
 		die();
@@ -125,8 +125,8 @@ 
  *
  * @return void
  */
-function give_die( $message = '', $title = '', $status = 400 ) {
-	add_filter( 'wp_die_ajax_handler', '_give_die_handler', 10, 3 );
-	add_filter( 'wp_die_handler', '_give_die_handler', 10, 3 );
-	wp_die( $message, $title, array( 'response' => $status ) );
+function give_die($message = '', $title = '', $status = 400) {
+	add_filter('wp_die_ajax_handler', '_give_die_handler', 10, 3);
+	add_filter('wp_die_handler', '_give_die_handler', 10, 3);
+	wp_die($message, $title, array('response' => $status));
 }

includes/class-give-donor.php

Spacing +170 added lines, -170 removed lines

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

Doc Comments +2 added lines, -2 removed lines

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

includes/class-give-db.php

Indentation +28 added lines, -28 removed lines

@@ -165,16 +165,16 @@ 
 	 *
 	 * @since  1.0
 	 * @access public
-     *
-     * @param  int    $column       Column ID.
-     * @param  string $column_where Column name.
-     * @param  string $column_value Column value.
-     *
+	 *
+	 * @param  int    $column       Column ID.
+	 * @param  string $column_where Column name.
+	 * @param  string $column_value Column value.
+	 *
 	 * @return string
 	 */
 	public function get_column_by( $column, $column_where, $column_value ) {
-        /* @var WPDB $wpdb */
-        global $wpdb;
+		/* @var WPDB $wpdb */
+		global $wpdb;
 
 		// Bailout.
 		if ( empty( $column ) || empty( $column_where ) || empty( $column_value ) ) {
@@ -191,15 +191,15 @@ 
 	 *
 	 * @since  1.0
 	 * @access public
-     *
-     * @param  array  $data
-     * @param  string $type
-     *
+	 *
+	 * @param  array  $data
+	 * @param  string $type
+	 *
 	 * @return int
 	 */
 	public function insert( $data, $type = '' ) {
-        /* @var WPDB $wpdb */
-        global $wpdb;
+		/* @var WPDB $wpdb */
+		global $wpdb;
 
 		// Set default values.
 		$data = wp_parse_args( $data, $this->get_column_defaults() );
@@ -246,16 +246,16 @@ 
 	 *
 	 * @since  1.0
 	 * @access public
-     *
-     * @param  int    $row_id Column ID
-     * @param  array  $data
-     * @param  string $where  Column value
-     *
+	 *
+	 * @param  int    $row_id Column ID
+	 * @param  array  $data
+	 * @param  string $where  Column value
+	 *
 	 * @return bool
 	 */
 	public function update( $row_id, $data = array(), $where = '' ) {
-        /* @var WPDB $wpdb */
-        global $wpdb;
+		/* @var WPDB $wpdb */
+		global $wpdb;
 
 		// Row ID must be positive integer
 		$row_id = absint( $row_id );
@@ -293,14 +293,14 @@ 
 	 *
 	 * @since  1.0
 	 * @access public
-     *
-     * @param  int $row_id Column ID.
-     *
+	 *
+	 * @param  int $row_id Column ID.
+	 *
 	 * @return bool
 	 */
 	public function delete( $row_id = 0 ) {
-        /* @var WPDB $wpdb */
-        global $wpdb;
+		/* @var WPDB $wpdb */
+		global $wpdb;
 
 		// Row ID must be positive integer
 		$row_id = absint( $row_id );
@@ -321,13 +321,13 @@ 
 	 *
 	 * @since  1.3.2
 	 * @access public
-     *
+	 *
 	 * @param  string $table The table name.
-     *
+	 *
 	 * @return bool          If the table name exists.
 	 */
 	public function table_exists( $table ) {
-        /* @var WPDB $wpdb */
+		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		$table = sanitize_text_field( $table );

Spacing +46 added lines, -46 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -98,16 +98,16 @@ 
 	 *
 	 * @return object
 	 */
-	public function get( $row_id ) {
+	public function get($row_id) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Bailout.
-		if ( empty( $row_id ) ) {
+		if (empty($row_id)) {
 			return null;
 		}
 
-		return $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id ) );
+		return $wpdb->get_row($wpdb->prepare("SELECT * FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id));
 	}
 
 	/**
@@ -121,18 +121,18 @@ 
 	 *
 	 * @return object
 	 */
-	public function get_by( $column, $row_id ) {
+	public function get_by($column, $row_id) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Bailout.
-		if ( empty( $column ) || empty( $row_id ) ) {
+		if (empty($column) || empty($row_id)) {
 			return null;
 		}
 
-		$column = esc_sql( $column );
+		$column = esc_sql($column);
 
-		return $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $this->table_name WHERE $column = %s LIMIT 1;", $row_id ) );
+		return $wpdb->get_row($wpdb->prepare("SELECT * FROM $this->table_name WHERE $column = %s LIMIT 1;", $row_id));
 	}
 
 	/**
@@ -146,18 +146,18 @@ 
 	 *
 	 * @return string      Column value.
 	 */
-	public function get_column( $column, $row_id ) {
+	public function get_column($column, $row_id) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Bailout.
-		if ( empty( $column ) || empty( $row_id ) ) {
+		if (empty($column) || empty($row_id)) {
 			return null;
 		}
 
-		$column = esc_sql( $column );
+		$column = esc_sql($column);
 
-		return $wpdb->get_var( $wpdb->prepare( "SELECT $column FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id ) );
+		return $wpdb->get_var($wpdb->prepare("SELECT $column FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id));
 	}
 
 	/**
@@ -172,18 +172,18 @@ 
      *
 	 * @return string
 	 */
-	public function get_column_by( $column, $column_where, $column_value ) {
+	public function get_column_by($column, $column_where, $column_value) {
         /* @var WPDB $wpdb */
         global $wpdb;
 
 		// Bailout.
-		if ( empty( $column ) || empty( $column_where ) || empty( $column_value ) ) {
+		if (empty($column) || empty($column_where) || empty($column_value)) {
 			return null;
 		}
 
-		$column_where = esc_sql( $column_where );
-		$column       = esc_sql( $column );
-		return $wpdb->get_var( $wpdb->prepare( "SELECT $column FROM $this->table_name WHERE $column_where = %s LIMIT 1;", $column_value ) );
+		$column_where = esc_sql($column_where);
+		$column       = esc_sql($column);
+		return $wpdb->get_var($wpdb->prepare("SELECT $column FROM $this->table_name WHERE $column_where = %s LIMIT 1;", $column_value));
 	}
 
 	/**
@@ -197,12 +197,12 @@ 
      *
 	 * @return int
 	 */
-	public function insert( $data, $type = '' ) {
+	public function insert($data, $type = '') {
         /* @var WPDB $wpdb */
         global $wpdb;
 
 		// Set default values.
-		$data = wp_parse_args( $data, $this->get_column_defaults() );
+		$data = wp_parse_args($data, $this->get_column_defaults());
 
 		/**
 		 * Fires before inserting data to the database.
@@ -211,22 +211,22 @@ 
 		 *
 		 * @param array $data
 		 */
-		do_action( "give_pre_insert_{$type}", $data );
+		do_action("give_pre_insert_{$type}", $data);
 
 		// Initialise column format array
 		$column_formats = $this->get_columns();
 
 		// Force fields to lower case
-		$data = array_change_key_case( $data );
+		$data = array_change_key_case($data);
 
 		// White list columns
-		$data = array_intersect_key( $data, $column_formats );
+		$data = array_intersect_key($data, $column_formats);
 
 		// Reorder $column_formats to match the order of columns given in $data
-		$data_keys      = array_keys( $data );
-		$column_formats = array_merge( array_flip( $data_keys ), $column_formats );
+		$data_keys      = array_keys($data);
+		$column_formats = array_merge(array_flip($data_keys), $column_formats);
 
-		$wpdb->insert( $this->table_name, $data, $column_formats );
+		$wpdb->insert($this->table_name, $data, $column_formats);
 
 		/**
 		 * Fires after inserting data to the database.
@@ -236,7 +236,7 @@ 
 		 * @param int   $insert_id
 		 * @param array $data
 		 */
-		do_action( "give_post_insert_{$type}", $wpdb->insert_id, $data );
+		do_action("give_post_insert_{$type}", $wpdb->insert_id, $data);
 
 		return $wpdb->insert_id;
 	}
@@ -253,18 +253,18 @@ 
      *
 	 * @return bool
 	 */
-	public function update( $row_id, $data = array(), $where = '' ) {
+	public function update($row_id, $data = array(), $where = '') {
         /* @var WPDB $wpdb */
         global $wpdb;
 
 		// Row ID must be positive integer
-		$row_id = absint( $row_id );
+		$row_id = absint($row_id);
 
-		if ( empty( $row_id ) ) {
+		if (empty($row_id)) {
 			return false;
 		}
 
-		if ( empty( $where ) ) {
+		if (empty($where)) {
 			$where = $this->primary_key;
 		}
 
@@ -272,16 +272,16 @@ 
 		$column_formats = $this->get_columns();
 
 		// Force fields to lower case
-		$data = array_change_key_case( $data );
+		$data = array_change_key_case($data);
 
 		// White list columns
-		$data = array_intersect_key( $data, $column_formats );
+		$data = array_intersect_key($data, $column_formats);
 
 		// Reorder $column_formats to match the order of columns given in $data
-		$data_keys      = array_keys( $data );
-		$column_formats = array_merge( array_flip( $data_keys ), $column_formats );
+		$data_keys      = array_keys($data);
+		$column_formats = array_merge(array_flip($data_keys), $column_formats);
 
-		if ( false === $wpdb->update( $this->table_name, $data, array( $where => $row_id ), $column_formats ) ) {
+		if (false === $wpdb->update($this->table_name, $data, array($where => $row_id), $column_formats)) {
 			return false;
 		}
 
@@ -298,18 +298,18 @@ 
      *
 	 * @return bool
 	 */
-	public function delete( $row_id = 0 ) {
+	public function delete($row_id = 0) {
         /* @var WPDB $wpdb */
         global $wpdb;
 
 		// Row ID must be positive integer
-		$row_id = absint( $row_id );
+		$row_id = absint($row_id);
 
-		if ( empty( $row_id ) ) {
+		if (empty($row_id)) {
 			return false;
 		}
 
-		if ( false === $wpdb->query( $wpdb->prepare( "DELETE FROM $this->table_name WHERE $this->primary_key = %d", $row_id ) ) ) {
+		if (false === $wpdb->query($wpdb->prepare("DELETE FROM $this->table_name WHERE $this->primary_key = %d", $row_id))) {
 			return false;
 		}
 
@@ -326,13 +326,13 @@ 
      *
 	 * @return bool          If the table name exists.
 	 */
-	public function table_exists( $table ) {
+	public function table_exists($table) {
         /* @var WPDB $wpdb */
 		global $wpdb;
 
-		$table = sanitize_text_field( $table );
+		$table = sanitize_text_field($table);
 
-		return $wpdb->get_var( $wpdb->prepare( "SHOW TABLES LIKE '%s'", $table ) ) === $table;
+		return $wpdb->get_var($wpdb->prepare("SHOW TABLES LIKE '%s'", $table)) === $table;
 	}
 
 	/**
@@ -344,7 +344,7 @@ 
 	 * @return bool Returns if the customers table was installed and upgrade routine run.
 	 */
 	public function installed() {
-		return $this->table_exists( $this->table_name );
+		return $this->table_exists($this->table_name);
 	}
 
 	/**
@@ -354,8 +354,8 @@ 
 	 * @access public
 	 */
 	public function register_table() {
-		$current_version = get_option( $this->table_name . '_db_version' );
-		if ( ! $current_version || version_compare( $current_version, $this->version, '<' ) ) {
+		$current_version = get_option($this->table_name.'_db_version');
+		if ( ! $current_version || version_compare($current_version, $this->version, '<')) {
 			$this->create_table();
 		}
 	}
@@ -366,5 +366,5 @@ 
 	 * @since  1.8.9
 	 * @access public
 	 */
-	public function create_table(){}
+	public function create_table() {}
 }

includes/class-give-session.php

Spacing +76 added lines, -76 removed lines

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

includes/admin/upgrades/class-give-updates.php

Spacing +79 added lines, -79 removed lines

@@ -70,24 +70,24 @@ 
 	 *
 	 * @param array $args
 	 */
-	public function register( $args ) {
+	public function register($args) {
 		$args_default = array(
 			'id'       => '',
 			'version'  => '',
 			'callback' => '',
 		);
 
-		$args = wp_parse_args( $args, $args_default );
+		$args = wp_parse_args($args, $args_default);
 
 		// You can only register database upgrade.
 		$args['type'] = 'database';
 
 		// Bailout.
-		if ( empty( $args['id'] ) || empty( $args['version'] ) || empty( $args['callback'] ) || ! is_callable( $args['callback'] ) ) {
+		if (empty($args['id']) || empty($args['version']) || empty($args['callback']) || ! is_callable($args['callback'])) {
 			return;
 		}
 
-		$this->updates[ $args['type'] ][] = $args;
+		$this->updates[$args['type']][] = $args;
 	}
 
 
@@ -102,32 +102,32 @@ 
 	 *
 	 * @return array
 	 */
-	public function get_updates( $update_type = '', $status = 'all' ) {
+	public function get_updates($update_type = '', $status = 'all') {
 		// return all updates.
-		if ( empty( $update_type ) ) {
+		if (empty($update_type)) {
 			return $this->updates;
 		}
 
 		// Get specific update.
-		$updates = ! empty( $this->updates[ $update_type ] ) ? $this->updates[ $update_type ] : array();
+		$updates = ! empty($this->updates[$update_type]) ? $this->updates[$update_type] : array();
 
 		// Bailout.
-		if ( empty( $updates ) ) {
+		if (empty($updates)) {
 			return $updates;
 		}
 
-		switch ( $status ) {
+		switch ($status) {
 			case 'new':
 				// Remove already completed updates.
 				$completed_updates = give_get_completed_upgrades();
 
-				if ( ! empty( $completed_updates ) ) {
-					foreach ( $updates as $index => $update ) {
-						if ( in_array( $update['id'], $completed_updates ) ) {
-							unset( $updates[ $index ] );
+				if ( ! empty($completed_updates)) {
+					foreach ($updates as $index => $update) {
+						if (in_array($update['id'], $completed_updates)) {
+							unset($updates[$index]);
 						}
 					}
-					$updates = array_values( $updates );
+					$updates = array_values($updates);
 				}
 
 				break;
@@ -144,7 +144,7 @@ 
 	 * @return static
 	 */
 	static function get_instance() {
-		if ( is_null( self::$instance ) ) {
+		if (is_null(self::$instance)) {
 			self::$instance = new self();
 		}
 
@@ -162,16 +162,16 @@ 
 		/**
 		 * Setup hooks.
 		 */
-		add_action( 'init', array( $this, '__register_upgrade' ), 9999 );
-		add_action( 'admin_init', array( $this, '__change_donations_label' ), 9999 );
-		add_action( 'admin_menu', array( $this, '__register_menu' ), 9999 );
-		add_action( 'give_set_upgrade_completed', array( $this, '__flush_resume_updates' ), 9999 );
-		add_action( 'wp_ajax_give_do_ajax_updates', array( $this, '__give_ajax_updates' ) );
+		add_action('init', array($this, '__register_upgrade'), 9999);
+		add_action('admin_init', array($this, '__change_donations_label'), 9999);
+		add_action('admin_menu', array($this, '__register_menu'), 9999);
+		add_action('give_set_upgrade_completed', array($this, '__flush_resume_updates'), 9999);
+		add_action('wp_ajax_give_do_ajax_updates', array($this, '__give_ajax_updates'));
 
 		/**
 		 * Load file
 		 */
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/upgrade-functions.php';
+		require_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/upgrade-functions.php';
 	}
 
 	/**
@@ -184,12 +184,12 @@ 
 		$addons         = give_get_plugins();
 		$plugin_updates = get_plugin_updates();
 
-		foreach ( $addons as $key => $info ) {
-			if ( 'active' != $info['Status'] || 'add-on' != $info['Type'] || empty( $plugin_updates[ $key ] ) ) {
+		foreach ($addons as $key => $info) {
+			if ('active' != $info['Status'] || 'add-on' != $info['Type'] || empty($plugin_updates[$key])) {
 				continue;
 			}
 
-			$this->updates['plugin'][] = array_merge( $info, (array) $plugin_updates[ $key ] );
+			$this->updates['plugin'][] = array_merge($info, (array) $plugin_updates[$key]);
 		}
 	}
 
@@ -201,7 +201,7 @@ 
 	 * @access public
 	 */
 	public function __register_upgrade() {
-		if ( ! is_admin() ) {
+		if ( ! is_admin()) {
 			return;
 		}
 
@@ -210,7 +210,7 @@ 
 		 *
 		 * @since 1.8.12
 		 */
-		do_action( 'give_register_updates', $this );
+		do_action('give_register_updates', $this);
 	}
 
 	/**
@@ -224,17 +224,17 @@ 
 		global $submenu;
 
 		// Bailout.
-		if ( empty( $menu ) || ! $this->get_update_count() ) {
+		if (empty($menu) || ! $this->get_update_count()) {
 			return;
 		}
 
-		foreach ( $menu as $index => $menu_item ) {
-			if ( 'edit.php?post_type=give_forms' !== $menu_item[2] ) {
+		foreach ($menu as $index => $menu_item) {
+			if ('edit.php?post_type=give_forms' !== $menu_item[2]) {
 				continue;
 			}
 
-			$menu[ $index ][0] = sprintf(
-				__( 'Donations %s', 'give' ),
+			$menu[$index][0] = sprintf(
+				__('Donations %s', 'give'),
 				sprintf(
 					'<span class="update-plugins count-%1$d"><span class="plugin-count">%1$d</span></span>',
 					$this->get_update_count()
@@ -259,15 +259,15 @@ 
 		// Bailout.
 		if ( ! $this->get_update_count()) {
 			// Show complete update message if still on update setting page.
-			if ( isset($_GET['page']) && 'give-updates' === $_GET['page'] ) {
+			if (isset($_GET['page']) && 'give-updates' === $_GET['page']) {
 				// Upgrades
 				add_submenu_page(
 					'edit.php?post_type=give_forms',
-					esc_html__( 'Give Updates Complete', 'give' ),
-					__( 'Updates', 'give' ),
+					esc_html__('Give Updates Complete', 'give'),
+					__('Updates', 'give'),
 					'manage_give_settings',
 					'give-updates',
-					array( $this, 'render_complete_page' )
+					array($this, 'render_complete_page')
 				);
 			}
 
@@ -277,15 +277,15 @@ 
 		// Upgrades
 		add_submenu_page(
 			'edit.php?post_type=give_forms',
-			esc_html__( 'Give Updates', 'give' ),
+			esc_html__('Give Updates', 'give'),
 			sprintf(
 				'%1$s <span class="update-plugins count-%2$d"><span class="plugin-count">%2$d</span></span>',
-				__( 'Updates', 'give' ),
+				__('Updates', 'give'),
 				$this->get_update_count()
 			),
 			'manage_give_settings',
 			'give-updates',
-			array( $this, 'render_page' )
+			array($this, 'render_page')
 		);
 	}
 
@@ -297,7 +297,7 @@ 
 	 * @return int
 	 */
 	public function get_db_update_count() {
-		return count( $this->get_updates( 'database', 'new' ) );
+		return count($this->get_updates('database', 'new'));
 	}
 
 	/**
@@ -307,7 +307,7 @@ 
 	 * @access public
 	 */
 	public function render_complete_page() {
-		include_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/views/upgrades-complete.php';
+		include_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/views/upgrades-complete.php';
 	}
 
 	/**
@@ -317,7 +317,7 @@ 
 	 * @access public
 	 */
 	public function render_page() {
-		include_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/views/upgrades.php';
+		include_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/views/upgrades.php';
 	}
 
 	/**
@@ -328,7 +328,7 @@ 
 	 * @return int
 	 */
 	public function get_plugin_update_count() {
-		return count( $this->get_updates( 'plugin' ) );
+		return count($this->get_updates('plugin'));
 	}
 
 	/**
@@ -343,7 +343,7 @@ 
 		$db_update_count     = $this->get_db_update_count();
 		$plugin_update_count = $this->get_plugin_update_count();
 
-		return ( $db_update_count + $plugin_update_count );
+		return ($db_update_count + $plugin_update_count);
 	}
 
 
@@ -354,12 +354,12 @@ 
 	 * @access public
 	 */
 	public function __flush_resume_updates() {
-		delete_option( 'give_doing_upgrade' );
-		update_option( 'give_version', preg_replace( '/[^0-9.].*/', '', GIVE_VERSION ) );
+		delete_option('give_doing_upgrade');
+		update_option('give_version', preg_replace('/[^0-9.].*/', '', GIVE_VERSION));
 
 		// Reset counter.
 		$this->step = $this->percentage = 0;
-		++ $this->update;
+		++$this->update;
 	}
 
 	/**
@@ -370,24 +370,24 @@ 
 	 */
 	public function __give_ajax_updates() {
 		// Check permission.
-		if ( ! current_user_can( 'manage_give_settings' ) ) {
+		if ( ! current_user_can('manage_give_settings')) {
 			$this->send_ajax_response(
 				array(
-					'message' => esc_html__( 'You do not have permission to do Give upgrades.', 'give' ),
+					'message' => esc_html__('You do not have permission to do Give upgrades.', 'give'),
 				),
 				'error'
 			);
 		}
 
 		// Set params.
-		$this->step   = absint( $_POST['step'] );
-		$this->update = absint( $_POST['update'] );
+		$this->step   = absint($_POST['step']);
+		$this->update = absint($_POST['update']);
 
 		// Bailout: step and update must be positive and greater then zero.
-		if ( ! $this->step ) {
+		if ( ! $this->step) {
 			$this->send_ajax_response(
 				array(
-					'message'    => __( 'Please reload this page  and try again', 'give' ),
+					'message'    => __('Please reload this page  and try again', 'give'),
 					'heading'    => '',
 					'percentage' => 0,
 				),
@@ -396,14 +396,14 @@ 
 		}
 
 		// Get updates.
-		$updates = $this->get_updates( 'database', 'new' );
+		$updates = $this->get_updates('database', 'new');
 
 		// Bailout if we do not have nay updates.
-		if ( empty( $updates ) ) {
+		if (empty($updates)) {
 			$this->send_ajax_response(
 				array(
-					'message'    => __( 'The database is already up to date.', 'give' ),
-					'heading'    => __( 'Updates Completed.', 'give' ),
+					'message'    => __('The database is already up to date.', 'give'),
+					'heading'    => __('Updates Completed.', 'give'),
 					'percentage' => 0,
 				),
 				'success'
@@ -411,26 +411,26 @@ 
 		}
 
 		// Process update.
-		foreach ( $updates as $index => $update ) {
+		foreach ($updates as $index => $update) {
 			// Check if update depend upon any other update.
-			if ( ! empty( $update['depend'] ) && ! give_has_upgrade_completed( $update['depend'] ) ) {
+			if ( ! empty($update['depend']) && ! give_has_upgrade_completed($update['depend'])) {
 				continue;
 			}
 
 			// Run update.
-			if ( is_array( $update['callback'] ) ) {
+			if (is_array($update['callback'])) {
 				$update['callback'][0]->$update['callback'][1]();
 			} else {
 				$update['callback']();
 			}
 
 			// Check if current update completed or not.
-			if ( give_has_upgrade_completed( $update['id'] ) ) {
-				if ( 1 === count( $updates ) ) {
+			if (give_has_upgrade_completed($update['id'])) {
+				if (1 === count($updates)) {
 					$this->send_ajax_response(
 						array(
-							'message'    => __( 'Database updated successfully.', 'give' ),
-							'heading'    => __( 'Updates Completed.', 'give' ),
+							'message'    => __('Database updated successfully.', 'give'),
+							'heading'    => __('Updates Completed.', 'give'),
 							'percentage' => 0,
 						),
 						'success'
@@ -440,16 +440,16 @@ 
 
 			$doing_upgrade_args = array(
 				'update_info' => $update,
-				'step'        => ++ $this->step,
+				'step'        => ++$this->step,
 				'update'      => $this->update,
-				'heading'     => sprintf( 'Update %s of {update_count}', $this->update ),
+				'heading'     => sprintf('Update %s of {update_count}', $this->update),
 				'percentage'  => $this->percentage,
 			);
 
 			// Cache upgrade.
-			update_option( 'give_doing_upgrade', $doing_upgrade_args );
+			update_option('give_doing_upgrade', $doing_upgrade_args);
 
-			$this->send_ajax_response( $doing_upgrade_args );
+			$this->send_ajax_response($doing_upgrade_args);
 		}// End foreach().
 	}
 
@@ -462,7 +462,7 @@ 
 	 * @param        $data
 	 * @param string $type
 	 */
-	public function send_ajax_response( $data, $type = '' ) {
+	public function send_ajax_response($data, $type = '') {
 		$default = array(
 			'message'    => '',
 			'heading'    => '',
@@ -472,21 +472,21 @@ 
 		);
 
 		// Set data.
-		$data = wp_parse_args( $data, $default );
+		$data = wp_parse_args($data, $default);
 
-		switch ( $type ) {
+		switch ($type) {
 			case 'success':
-				wp_send_json_success( $data );
+				wp_send_json_success($data);
 				break;
 
 			case 'error':
-				wp_send_json_error( $data );
+				wp_send_json_error($data);
 				break;
 
 			default:
-				wp_send_json( array(
+				wp_send_json(array(
 					'data' => $data,
-				) );
+				));
 				break;
 		}
 	}
@@ -503,8 +503,8 @@ 
 	public function resume_updates() {
 		$status = false;
 
-		if ( $update = get_option( 'give_doing_upgrade' ) ) {
-			$status = ! empty( $update['step'] ) ? $update['step'] : $status;
+		if ($update = get_option('give_doing_upgrade')) {
+			$status = ! empty($update['step']) ? $update['step'] : $status;
 		}
 
 		return $status;
@@ -520,12 +520,12 @@ 
 	 * @param $total
 	 * @param $current_total
 	 */
-	public function set_percentage( $total, $current_total ) {
+	public function set_percentage($total, $current_total) {
 		// Set percentage.
-		$this->percentage = $total ? ( ( $current_total ) / $total ) * 100 : 0;
+		$this->percentage = $total ? (($current_total) / $total) * 100 : 0;
 
 		// Verify percentage.
-		$this->percentage = ( 100 < $this->percentage ) ? 100 : $this->percentage;
+		$this->percentage = (100 < $this->percentage) ? 100 : $this->percentage;
 	}
 }
 

includes/admin/upgrades/upgrade-functions.php

Spacing +199 added lines, -199 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,56 +25,56 @@ 
  */
 function give_do_automatic_upgrades() {
 	$did_upgrade  = false;
-	$give_version = preg_replace( '/[^0-9.].*/', '', get_option( 'give_version' ) );
+	$give_version = preg_replace('/[^0-9.].*/', '', get_option('give_version'));
 
-	if ( ! $give_version ) {
+	if ( ! $give_version) {
 		// 1.0 is the first version to use this option so we must add it.
 		$give_version = '1.0';
 	}
 
-	switch ( true ) {
+	switch (true) {
 
-		case version_compare( $give_version, '1.6', '<' ) :
+		case version_compare($give_version, '1.6', '<') :
 			give_v16_upgrades();
 			$did_upgrade = true;
 
-		case version_compare( $give_version, '1.7', '<' ) :
+		case version_compare($give_version, '1.7', '<') :
 			give_v17_upgrades();
 			$did_upgrade = true;
 
-		case version_compare( $give_version, '1.8', '<' ) :
+		case version_compare($give_version, '1.8', '<') :
 			give_v18_upgrades();
 			$did_upgrade = true;
 
-		case version_compare( $give_version, '1.8.7', '<' ) :
+		case version_compare($give_version, '1.8.7', '<') :
 			give_v187_upgrades();
 			$did_upgrade = true;
 
-		case version_compare( $give_version, '1.8.8', '<' ) :
+		case version_compare($give_version, '1.8.8', '<') :
 			give_v188_upgrades();
 			$did_upgrade = true;
 
-		case version_compare( $give_version, '1.8.9', '<' ) :
+		case version_compare($give_version, '1.8.9', '<') :
 			give_v189_upgrades();
 			$did_upgrade = true;
 
-		case version_compare( $give_version, '1.8.12', '<' ) :
+		case version_compare($give_version, '1.8.12', '<') :
 			give_v1812_upgrades();
 			$did_upgrade = true;
 
-		case version_compare( $give_version, '1.8.13', '<' ) :
+		case version_compare($give_version, '1.8.13', '<') :
 			give_v1813_upgrades();
 			$did_upgrade = true;
 
 	}
 
-	if ( $did_upgrade ) {
-		update_option( 'give_version', preg_replace( '/[^0-9.].*/', '', GIVE_VERSION ) );
+	if ($did_upgrade) {
+		update_option('give_version', preg_replace('/[^0-9.].*/', '', GIVE_VERSION));
 	}
 }
 
-add_action( 'admin_init', 'give_do_automatic_upgrades' );
-add_action( 'give_upgrades', 'give_do_automatic_upgrades' );
+add_action('admin_init', 'give_do_automatic_upgrades');
+add_action('give_upgrades', 'give_do_automatic_upgrades');
 
 /**
  * Display Upgrade Notices
@@ -86,7 +86,7 @@ 
  *
  * @return void
  */
-function give_show_upgrade_notices( $give_updates ) {
+function give_show_upgrade_notices($give_updates) {
 	// v1.3.2 Upgrades
 	$give_updates->register(
 		array(
@@ -152,7 +152,7 @@ 
 	);
 }
 
-add_action( 'give_register_updates', 'give_show_upgrade_notices' );
+add_action('give_register_updates', 'give_show_upgrade_notices');
 
 /**
  * Triggers all upgrade functions
@@ -164,29 +164,29 @@ 
  */
 function give_trigger_upgrades() {
 
-	if ( ! current_user_can( 'manage_give_settings' ) ) {
-		wp_die( esc_html__( 'You do not have permission to do Give upgrades.', 'give' ), esc_html__( 'Error', 'give' ), array(
+	if ( ! current_user_can('manage_give_settings')) {
+		wp_die(esc_html__('You do not have permission to do Give upgrades.', 'give'), esc_html__('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	$give_version = get_option( 'give_version' );
+	$give_version = get_option('give_version');
 
-	if ( ! $give_version ) {
+	if ( ! $give_version) {
 		// 1.0 is the first version to use this option so we must add it.
 		$give_version = '1.0';
-		add_option( 'give_version', $give_version );
+		add_option('give_version', $give_version);
 	}
 
-	update_option( 'give_version', GIVE_VERSION );
-	delete_option( 'give_doing_upgrade' );
+	update_option('give_version', GIVE_VERSION);
+	delete_option('give_doing_upgrade');
 
-	if ( DOING_AJAX ) {
-		die( 'complete' );
+	if (DOING_AJAX) {
+		die('complete');
 	} // End if().
 }
 
-add_action( 'wp_ajax_give_trigger_upgrades', 'give_trigger_upgrades' );
+add_action('wp_ajax_give_trigger_upgrades', 'give_trigger_upgrades');
 
 
 /**
@@ -202,24 +202,24 @@ 
 	/* @var Give_Updates $give_updates */
 	$give_updates = Give_Updates::get_instance();
 
-	if ( ! current_user_can( 'manage_give_settings' ) ) {
-		wp_die( esc_html__( 'You do not have permission to do Give upgrades.', 'give' ), esc_html__( 'Error', 'give' ), array(
+	if ( ! current_user_can('manage_give_settings')) {
+		wp_die(esc_html__('You do not have permission to do Give upgrades.', 'give'), esc_html__('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	ignore_user_abort( true );
+	ignore_user_abort(true);
 
-	if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-		@set_time_limit( 0 );
+	if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+		@set_time_limit(0);
 	}
 
 	// UPDATE DB METAKEYS.
 	$sql   = "UPDATE $wpdb->postmeta SET meta_key = '_give_payment_customer_id' WHERE meta_key = '_give_payment_donor_id'";
-	$query = $wpdb->query( $sql );
+	$query = $wpdb->query($sql);
 
 	$give_updates->percentage = 100;
-	give_set_upgrade_complete( 'upgrade_give_payment_customer_id' );
+	give_set_upgrade_complete('upgrade_give_payment_customer_id');
 }
 
 
@@ -243,24 +243,24 @@ 
 	$where  .= "AND ( p.post_status = 'abandoned' )";
 	$where  .= "AND ( m.meta_key = '_give_payment_gateway' AND m.meta_value = 'offline' )";
 
-	$sql            = $select . $join . $where;
-	$found_payments = $wpdb->get_col( $sql );
+	$sql            = $select.$join.$where;
+	$found_payments = $wpdb->get_col($sql);
 
-	foreach ( $found_payments as $payment ) {
+	foreach ($found_payments as $payment) {
 
 		// Only change ones marked abandoned since our release last week because the admin may have marked some abandoned themselves.
-		$modified_time = get_post_modified_time( 'U', false, $payment );
+		$modified_time = get_post_modified_time('U', false, $payment);
 
 		// 1450124863 =  12/10/2015 20:42:25.
-		if ( $modified_time >= 1450124863 ) {
+		if ($modified_time >= 1450124863) {
 
-			give_update_payment_status( $payment, 'pending' );
+			give_update_payment_status($payment, 'pending');
 
 		}
 	}
 
 	$give_updates->percentage = 100;
-	give_set_upgrade_complete( 'upgrade_give_offline_status' );
+	give_set_upgrade_complete('upgrade_give_offline_status');
 }
 
 
@@ -273,17 +273,17 @@ 
  */
 function give_v152_cleanup_users() {
 
-	$give_version = get_option( 'give_version' );
+	$give_version = get_option('give_version');
 
-	if ( ! $give_version ) {
+	if ( ! $give_version) {
 		// 1.0 is the first version to use this option so we must add it.
 		$give_version = '1.0';
 	}
 
-	$give_version = preg_replace( '/[^0-9.].*/', '', $give_version );
+	$give_version = preg_replace('/[^0-9.].*/', '', $give_version);
 
 	// v1.5.2 Upgrades
-	if ( version_compare( $give_version, '1.5.2', '<' ) || ! give_has_upgrade_completed( 'upgrade_give_user_caps_cleanup' ) ) {
+	if (version_compare($give_version, '1.5.2', '<') || ! give_has_upgrade_completed('upgrade_give_user_caps_cleanup')) {
 
 		// Delete all caps with "ss".
 		// Also delete all unused "campaign" roles.
@@ -330,9 +330,9 @@ 
 		);
 
 		global $wp_roles;
-		foreach ( $delete_caps as $cap ) {
-			foreach ( array_keys( $wp_roles->roles ) as $role ) {
-				$wp_roles->remove_cap( $role, $cap );
+		foreach ($delete_caps as $cap) {
+			foreach (array_keys($wp_roles->roles) as $role) {
+				$wp_roles->remove_cap($role, $cap);
 			}
 		}
 
@@ -342,15 +342,15 @@ 
 		$roles->add_caps();
 
 		// The Update Ran.
-		update_option( 'give_version', preg_replace( '/[^0-9.].*/', '', GIVE_VERSION ) );
-		give_set_upgrade_complete( 'upgrade_give_user_caps_cleanup' );
-		delete_option( 'give_doing_upgrade' );
+		update_option('give_version', preg_replace('/[^0-9.].*/', '', GIVE_VERSION));
+		give_set_upgrade_complete('upgrade_give_user_caps_cleanup');
+		delete_option('give_doing_upgrade');
 
 	}// End if().
 
 }
 
-add_action( 'admin_init', 'give_v152_cleanup_users' );
+add_action('admin_init', 'give_v152_cleanup_users');
 
 /**
  * 1.6 Upgrade routine to create the customer meta table.
@@ -393,53 +393,53 @@ 
 
 	// Get addons license key.
 	$addons = array();
-	foreach ( $give_options as $key => $value ) {
-		if ( false !== strpos( $key, '_license_key' ) ) {
-			$addons[ $key ] = $value;
+	foreach ($give_options as $key => $value) {
+		if (false !== strpos($key, '_license_key')) {
+			$addons[$key] = $value;
 		}
 	}
 
 	// Bailout: We do not have any addon license data to upgrade.
-	if ( empty( $addons ) ) {
+	if (empty($addons)) {
 		return false;
 	}
 
-	foreach ( $addons as $key => $addon_license ) {
+	foreach ($addons as $key => $addon_license) {
 
 		// Get addon shortname.
-		$shortname = str_replace( '_license_key', '', $key );
+		$shortname = str_replace('_license_key', '', $key);
 
 		// Addon license option name.
-		$addon_license_option_name = $shortname . '_license_active';
+		$addon_license_option_name = $shortname.'_license_active';
 
 		// bailout if license is empty.
-		if ( empty( $addon_license ) ) {
-			delete_option( $addon_license_option_name );
+		if (empty($addon_license)) {
+			delete_option($addon_license_option_name);
 			continue;
 		}
 
 		// Get addon name.
 		$addon_name       = array();
-		$addon_name_parts = explode( '_', str_replace( 'give_', '', $shortname ) );
-		foreach ( $addon_name_parts as $name_part ) {
+		$addon_name_parts = explode('_', str_replace('give_', '', $shortname));
+		foreach ($addon_name_parts as $name_part) {
 
 			// Fix addon name
-			switch ( $name_part ) {
+			switch ($name_part) {
 				case 'authorizenet' :
 					$name_part = 'authorize.net';
 					break;
 			}
 
-			$addon_name[] = ucfirst( $name_part );
+			$addon_name[] = ucfirst($name_part);
 		}
 
-		$addon_name = implode( ' ', $addon_name );
+		$addon_name = implode(' ', $addon_name);
 
 		// Data to send to the API
 		$api_params = array(
 			'edd_action' => 'activate_license', // never change from "edd_" to "give_"!
 			'license'    => $addon_license,
-			'item_name'  => urlencode( $addon_name ),
+			'item_name'  => urlencode($addon_name),
 			'url'        => home_url(),
 		);
 
@@ -454,17 +454,17 @@ 
 		);
 
 		// Make sure there are no errors.
-		if ( is_wp_error( $response ) ) {
-			delete_option( $addon_license_option_name );
+		if (is_wp_error($response)) {
+			delete_option($addon_license_option_name);
 			continue;
 		}
 
 		// Tell WordPress to look for updates.
-		set_site_transient( 'update_plugins', null );
+		set_site_transient('update_plugins', null);
 
 		// Decode license data.
-		$license_data = json_decode( wp_remote_retrieve_body( $response ) );
-		update_option( $addon_license_option_name, $license_data );
+		$license_data = json_decode(wp_remote_retrieve_body($response));
+		update_option($addon_license_option_name, $license_data);
 	}// End foreach().
 }
 
@@ -494,9 +494,9 @@ 
 	);
 
 	global $wp_roles;
-	foreach ( $delete_caps as $cap ) {
-		foreach ( array_keys( $wp_roles->roles ) as $role ) {
-			$wp_roles->remove_cap( $role, $cap );
+	foreach ($delete_caps as $cap) {
+		foreach (array_keys($wp_roles->roles) as $role) {
+			$wp_roles->remove_cap($role, $cap);
 		}
 	}
 
@@ -530,7 +530,7 @@ 
 function give_v18_upgrades_core_setting() {
 	// Core settings which changes from checkbox to radio.
 	$core_setting_names = array_merge(
-		array_keys( give_v18_renamed_core_settings() ),
+		array_keys(give_v18_renamed_core_settings()),
 		array(
 			'uninstall_on_delete',
 			'scripts_footer',
@@ -542,48 +542,48 @@ 
 	);
 
 	// Bailout: If not any setting define.
-	if ( $give_settings = get_option( 'give_settings' ) ) {
+	if ($give_settings = get_option('give_settings')) {
 
 		$setting_changed = false;
 
 		// Loop: check each setting field.
-		foreach ( $core_setting_names as $setting_name ) {
+		foreach ($core_setting_names as $setting_name) {
 			// New setting name.
-			$new_setting_name = preg_replace( '/^(enable_|disable_)/', '', $setting_name );
+			$new_setting_name = preg_replace('/^(enable_|disable_)/', '', $setting_name);
 
 			// Continue: If setting already set.
 			if (
-				array_key_exists( $new_setting_name, $give_settings )
-				&& in_array( $give_settings[ $new_setting_name ], array( 'enabled', 'disabled' ) )
+				array_key_exists($new_setting_name, $give_settings)
+				&& in_array($give_settings[$new_setting_name], array('enabled', 'disabled'))
 			) {
 				continue;
 			}
 
 			// Set checkbox value to radio value.
-			$give_settings[ $setting_name ] = ( ! empty( $give_settings[ $setting_name ] ) && 'on' === $give_settings[ $setting_name ] ? 'enabled' : 'disabled' );
+			$give_settings[$setting_name] = ( ! empty($give_settings[$setting_name]) && 'on' === $give_settings[$setting_name] ? 'enabled' : 'disabled');
 
 			// @see https://github.com/WordImpress/Give/issues/1063
-			if ( false !== strpos( $setting_name, 'disable_' ) ) {
+			if (false !== strpos($setting_name, 'disable_')) {
 
-				$give_settings[ $new_setting_name ] = ( give_is_setting_enabled( $give_settings[ $setting_name ] ) ? 'disabled' : 'enabled' );
-			} elseif ( false !== strpos( $setting_name, 'enable_' ) ) {
+				$give_settings[$new_setting_name] = (give_is_setting_enabled($give_settings[$setting_name]) ? 'disabled' : 'enabled');
+			} elseif (false !== strpos($setting_name, 'enable_')) {
 
-				$give_settings[ $new_setting_name ] = ( give_is_setting_enabled( $give_settings[ $setting_name ] ) ? 'enabled' : 'disabled' );
+				$give_settings[$new_setting_name] = (give_is_setting_enabled($give_settings[$setting_name]) ? 'enabled' : 'disabled');
 			}
 
 			// Tell bot to update core setting to db.
-			if ( ! $setting_changed ) {
+			if ( ! $setting_changed) {
 				$setting_changed = true;
 			}
 		}
 
 		// Update setting only if they changed.
-		if ( $setting_changed ) {
-			update_option( 'give_settings', $give_settings );
+		if ($setting_changed) {
+			update_option('give_settings', $give_settings);
 		}
 	}// End if().
 
-	give_set_upgrade_complete( 'v18_upgrades_core_setting' );
+	give_set_upgrade_complete('v18_upgrades_core_setting');
 }
 
 /**
@@ -597,7 +597,7 @@ 
 	$give_updates = Give_Updates::get_instance();
 
 	// form query
-	$forms = new WP_Query( array(
+	$forms = new WP_Query(array(
 			'paged'          => $give_updates->step,
 			'status'         => 'any',
 			'order'          => 'ASC',
@@ -606,41 +606,41 @@ 
 		)
 	);
 
-	if ( $forms->have_posts() ) {
-		$give_updates->set_percentage( $forms->found_posts, ( $give_updates->step * 20 ) );
+	if ($forms->have_posts()) {
+		$give_updates->set_percentage($forms->found_posts, ($give_updates->step * 20));
 
-		while ( $forms->have_posts() ) {
+		while ($forms->have_posts()) {
 			$forms->the_post();
 
 			// Form content.
 			// Note in version 1.8 display content setting split into display content and content placement setting.
 			// You can delete _give_content_option in future
-			$show_content = give_get_meta( get_the_ID(), '_give_content_option', true );
-			if ( $show_content && ! give_get_meta( get_the_ID(), '_give_display_content', true ) ) {
-				$field_value = ( 'none' !== $show_content ? 'enabled' : 'disabled' );
-				give_update_meta( get_the_ID(), '_give_display_content', $field_value );
+			$show_content = give_get_meta(get_the_ID(), '_give_content_option', true);
+			if ($show_content && ! give_get_meta(get_the_ID(), '_give_display_content', true)) {
+				$field_value = ('none' !== $show_content ? 'enabled' : 'disabled');
+				give_update_meta(get_the_ID(), '_give_display_content', $field_value);
 
-				$field_value = ( 'none' !== $show_content ? $show_content : 'give_pre_form' );
-				give_update_meta( get_the_ID(), '_give_content_placement', $field_value );
+				$field_value = ('none' !== $show_content ? $show_content : 'give_pre_form');
+				give_update_meta(get_the_ID(), '_give_content_placement', $field_value);
 			}
 
 			// "Disable" Guest Donation. Checkbox
 			// See: https://github.com/WordImpress/Give/issues/1470
-			$guest_donation        = give_get_meta( get_the_ID(), '_give_logged_in_only', true );
-			$guest_donation_newval = ( in_array( $guest_donation, array( 'yes', 'on' ) ) ? 'disabled' : 'enabled' );
-			give_update_meta( get_the_ID(), '_give_logged_in_only', $guest_donation_newval );
+			$guest_donation        = give_get_meta(get_the_ID(), '_give_logged_in_only', true);
+			$guest_donation_newval = (in_array($guest_donation, array('yes', 'on')) ? 'disabled' : 'enabled');
+			give_update_meta(get_the_ID(), '_give_logged_in_only', $guest_donation_newval);
 
 			// Offline Donations
 			// See: https://github.com/WordImpress/Give/issues/1579
-			$offline_donation = give_get_meta( get_the_ID(), '_give_customize_offline_donations', true );
-			if ( 'no' === $offline_donation ) {
+			$offline_donation = give_get_meta(get_the_ID(), '_give_customize_offline_donations', true);
+			if ('no' === $offline_donation) {
 				$offline_donation_newval = 'global';
-			} elseif ( 'yes' === $offline_donation ) {
+			} elseif ('yes' === $offline_donation) {
 				$offline_donation_newval = 'enabled';
 			} else {
 				$offline_donation_newval = 'disabled';
 			}
-			give_update_meta( get_the_ID(), '_give_customize_offline_donations', $offline_donation_newval );
+			give_update_meta(get_the_ID(), '_give_customize_offline_donations', $offline_donation_newval);
 
 			// Convert yes/no setting field to enabled/disabled.
 			$form_radio_settings = array(
@@ -660,15 +660,15 @@ 
 				'_give_offline_donation_enable_billing_fields_single',
 			);
 
-			foreach ( $form_radio_settings as $meta_key ) {
+			foreach ($form_radio_settings as $meta_key) {
 				// Get value.
-				$field_value = give_get_meta( get_the_ID(), $meta_key, true );
+				$field_value = give_get_meta(get_the_ID(), $meta_key, true);
 
 				// Convert meta value only if it is in yes/no/none.
-				if ( in_array( $field_value, array( 'yes', 'on', 'no', 'none' ) ) ) {
+				if (in_array($field_value, array('yes', 'on', 'no', 'none'))) {
 
-					$field_value = ( in_array( $field_value, array( 'yes', 'on' ) ) ? 'enabled' : 'disabled' );
-					give_update_meta( get_the_ID(), $meta_key, $field_value );
+					$field_value = (in_array($field_value, array('yes', 'on')) ? 'enabled' : 'disabled');
+					give_update_meta(get_the_ID(), $meta_key, $field_value);
 				}
 			}
 		}// End while().
@@ -677,7 +677,7 @@ 
 
 	} else {
 		// No more forms found, finish up.
-		give_set_upgrade_complete( 'v18_upgrades_form_metadata' );
+		give_set_upgrade_complete('v18_upgrades_form_metadata');
 	}
 }
 
@@ -726,7 +726,7 @@ 
 				'_transient_give_stats_',
 				'give_cache',
 				'_transient_give_add_ons_feed',
-				'_transient__give_ajax_works' .
+				'_transient__give_ajax_works'.
 				'_transient_give_total_api_keys',
 				'_transient_give_i18n_give_promo_hide',
 				'_transient_give_contributors',
@@ -753,24 +753,24 @@ 
 		ARRAY_A
 	);
 
-	if ( ! empty( $user_apikey_options ) ) {
-		foreach ( $user_apikey_options as $user ) {
-			$cached_options[] = '_transient_' . md5( 'give_api_user_' . $user['meta_key'] );
-			$cached_options[] = '_transient_' . md5( 'give_api_user_public_key' . $user['user_id'] );
-			$cached_options[] = '_transient_' . md5( 'give_api_user_secret_key' . $user['user_id'] );
+	if ( ! empty($user_apikey_options)) {
+		foreach ($user_apikey_options as $user) {
+			$cached_options[] = '_transient_'.md5('give_api_user_'.$user['meta_key']);
+			$cached_options[] = '_transient_'.md5('give_api_user_public_key'.$user['user_id']);
+			$cached_options[] = '_transient_'.md5('give_api_user_secret_key'.$user['user_id']);
 		}
 	}
 
-	if ( ! empty( $cached_options ) ) {
-		foreach ( $cached_options as $option ) {
-			switch ( true ) {
-				case ( false !== strpos( $option, 'transient' ) ):
-					$option = str_replace( '_transient_', '', $option );
-					delete_transient( $option );
+	if ( ! empty($cached_options)) {
+		foreach ($cached_options as $option) {
+			switch (true) {
+				case (false !== strpos($option, 'transient')):
+					$option = str_replace('_transient_', '', $option);
+					delete_transient($option);
 					break;
 
 				default:
-					delete_option( $option );
+					delete_option($option);
 			}
 		}
 	}
@@ -788,7 +788,7 @@ 
 	global $wp_roles;
 
 	// Get the role object.
-	$give_worker = get_role( 'give_worker' );
+	$give_worker = get_role('give_worker');
 
 	// A list of capabilities to add for give workers.
 	$caps_to_add = array(
@@ -796,9 +796,9 @@ 
 		'edit_pages',
 	);
 
-	foreach ( $caps_to_add as $cap ) {
+	foreach ($caps_to_add as $cap) {
 		// Add the capability.
-		$give_worker->add_cap( $cap );
+		$give_worker->add_cap($cap);
 	}
 
 }
@@ -815,7 +815,7 @@ 
 	$give_updates = Give_Updates::get_instance();
 
 	// form query
-	$donation_forms = new WP_Query( array(
+	$donation_forms = new WP_Query(array(
 			'paged'          => $give_updates->step,
 			'status'         => 'any',
 			'order'          => 'ASC',
@@ -824,10 +824,10 @@ 
 		)
 	);
 
-	if ( $donation_forms->have_posts() ) {
-		$give_updates->set_percentage( $donation_forms->found_posts, ( $give_updates->step * 20 ) );
+	if ($donation_forms->have_posts()) {
+		$give_updates->set_percentage($donation_forms->found_posts, ($give_updates->step * 20));
 
-		while ( $donation_forms->have_posts() ) {
+		while ($donation_forms->have_posts()) {
 			$donation_forms->the_post();
 			$form_id = get_the_ID();
 
@@ -835,41 +835,41 @@ 
 			update_post_meta(
 				$form_id,
 				'_give_set_price',
-				give_sanitize_amount( get_post_meta( $form_id, '_give_set_price', true ) )
+				give_sanitize_amount(get_post_meta($form_id, '_give_set_price', true))
 			);
 
 			// Remove formatting from _give_custom_amount_minimum
 			update_post_meta(
 				$form_id,
 				'_give_custom_amount_minimum',
-				give_sanitize_amount( get_post_meta( $form_id, '_give_custom_amount_minimum', true ) )
+				give_sanitize_amount(get_post_meta($form_id, '_give_custom_amount_minimum', true))
 			);
 
 			// Bailout.
-			if ( 'set' === get_post_meta( $form_id, '_give_price_option', true ) ) {
+			if ('set' === get_post_meta($form_id, '_give_price_option', true)) {
 				continue;
 			}
 
-			$donation_levels = get_post_meta( $form_id, '_give_donation_levels', true );
+			$donation_levels = get_post_meta($form_id, '_give_donation_levels', true);
 
-			if ( ! empty( $donation_levels ) ) {
+			if ( ! empty($donation_levels)) {
 
-				foreach ( $donation_levels as $index => $donation_level ) {
-					if ( isset( $donation_level['_give_amount'] ) ) {
-						$donation_levels[ $index ]['_give_amount'] = give_sanitize_amount( $donation_level['_give_amount'] );
+				foreach ($donation_levels as $index => $donation_level) {
+					if (isset($donation_level['_give_amount'])) {
+						$donation_levels[$index]['_give_amount'] = give_sanitize_amount($donation_level['_give_amount']);
 					}
 				}
 
-				update_post_meta( $form_id, '_give_donation_levels', $donation_levels );
+				update_post_meta($form_id, '_give_donation_levels', $donation_levels);
 
-				$donation_levels_amounts = wp_list_pluck( $donation_levels, '_give_amount' );
+				$donation_levels_amounts = wp_list_pluck($donation_levels, '_give_amount');
 
-				$min_amount = min( $donation_levels_amounts );
-				$max_amount = max( $donation_levels_amounts );
+				$min_amount = min($donation_levels_amounts);
+				$max_amount = max($donation_levels_amounts);
 
 				// Set Minimum and Maximum amount for Multi Level Donation Forms
-				give_update_meta( $form_id, '_give_levels_minimum_amount', $min_amount ? give_sanitize_amount( $min_amount ) : 0 );
-				give_update_meta( $form_id, '_give_levels_maximum_amount', $max_amount ? give_sanitize_amount( $max_amount ) : 0 );
+				give_update_meta($form_id, '_give_levels_minimum_amount', $min_amount ? give_sanitize_amount($min_amount) : 0);
+				give_update_meta($form_id, '_give_levels_maximum_amount', $max_amount ? give_sanitize_amount($max_amount) : 0);
 			}
 
 		}
@@ -878,7 +878,7 @@ 
 		wp_reset_postdata();
 	} else {
 		// The Update Ran.
-		give_set_upgrade_complete( 'v189_upgrades_levels_post_meta' );
+		give_set_upgrade_complete('v189_upgrades_levels_post_meta');
 	}
 
 }
@@ -934,22 +934,22 @@ 
 	$give_settings        = give_get_settings();
 	$give_setting_updated = false;
 
-	if ( $give_settings['thousands_separator'] === $give_settings['decimal_separator'] ) {
+	if ($give_settings['thousands_separator'] === $give_settings['decimal_separator']) {
 		$give_settings['number_decimals']   = 0;
 		$give_settings['decimal_separator'] = '';
 		$give_setting_updated               = true;
 
-	} elseif ( empty( $give_settings['decimal_separator'] ) ) {
+	} elseif (empty($give_settings['decimal_separator'])) {
 		$give_settings['number_decimals'] = 0;
 		$give_setting_updated             = true;
 
-	} elseif ( 6 < absint( $give_settings['number_decimals'] ) ) {
+	} elseif (6 < absint($give_settings['number_decimals'])) {
 		$give_settings['number_decimals'] = 5;
 		$give_setting_updated             = true;
 	}
 
-	if ( $give_setting_updated ) {
-		update_option( 'give_settings', $give_settings );
+	if ($give_setting_updated) {
+		update_option('give_settings', $give_settings);
 	}
 }
 
@@ -968,70 +968,70 @@ 
 	$give_updates = Give_Updates::get_instance();
 
 	// form query
-	$donation_forms = new WP_Query( array(
+	$donation_forms = new WP_Query(array(
 			'paged'          => $give_updates->step,
 			'status'         => 'any',
 			'order'          => 'ASC',
-			'post_type'      => array( 'give_forms', 'give_payment' ),
+			'post_type'      => array('give_forms', 'give_payment'),
 			'posts_per_page' => 20,
 		)
 	);
 
-	if ( $donation_forms->have_posts() ) {
-		$give_updates->set_percentage( $donation_forms->found_posts, ( $give_updates->step * 20 ) );
+	if ($donation_forms->have_posts()) {
+		$give_updates->set_percentage($donation_forms->found_posts, ($give_updates->step * 20));
 
-		while ( $donation_forms->have_posts() ) {
+		while ($donation_forms->have_posts()) {
 			$donation_forms->the_post();
 			global $post;
 
-			$meta = get_post_meta( $post->ID );
+			$meta = get_post_meta($post->ID);
 
-			switch ( $post->post_type ) {
+			switch ($post->post_type) {
 				case 'give_forms':
 					// _give_set_price
-					if ( ! empty( $meta['_give_set_price'][0] ) ) {
-						update_post_meta( $post->ID, '_give_set_price', give_sanitize_amount_for_db( $meta['_give_set_price'][0] ) );
+					if ( ! empty($meta['_give_set_price'][0])) {
+						update_post_meta($post->ID, '_give_set_price', give_sanitize_amount_for_db($meta['_give_set_price'][0]));
 					}
 
 					// _give_custom_amount_minimum
-					if ( ! empty( $meta['_give_custom_amount_minimum'][0] ) ) {
-						update_post_meta( $post->ID, '_give_custom_amount_minimum', give_sanitize_amount_for_db( $meta['_give_custom_amount_minimum'][0] ) );
+					if ( ! empty($meta['_give_custom_amount_minimum'][0])) {
+						update_post_meta($post->ID, '_give_custom_amount_minimum', give_sanitize_amount_for_db($meta['_give_custom_amount_minimum'][0]));
 					}
 
 					// _give_levels_minimum_amount
-					if ( ! empty( $meta['_give_levels_minimum_amount'][0] ) ) {
-						update_post_meta( $post->ID, '_give_levels_minimum_amount', give_sanitize_amount_for_db( $meta['_give_levels_minimum_amount'][0] ) );
+					if ( ! empty($meta['_give_levels_minimum_amount'][0])) {
+						update_post_meta($post->ID, '_give_levels_minimum_amount', give_sanitize_amount_for_db($meta['_give_levels_minimum_amount'][0]));
 					}
 
 					// _give_levels_maximum_amount
-					if ( ! empty( $meta['_give_levels_maximum_amount'][0] ) ) {
-						update_post_meta( $post->ID, '_give_levels_maximum_amount', give_sanitize_amount_for_db( $meta['_give_levels_maximum_amount'][0] ) );
+					if ( ! empty($meta['_give_levels_maximum_amount'][0])) {
+						update_post_meta($post->ID, '_give_levels_maximum_amount', give_sanitize_amount_for_db($meta['_give_levels_maximum_amount'][0]));
 					}
 
 					// _give_set_goal
-					if ( ! empty( $meta['_give_set_goal'][0] ) ) {
-						update_post_meta( $post->ID, '_give_set_goal', give_sanitize_amount_for_db( $meta['_give_set_goal'][0] ) );
+					if ( ! empty($meta['_give_set_goal'][0])) {
+						update_post_meta($post->ID, '_give_set_goal', give_sanitize_amount_for_db($meta['_give_set_goal'][0]));
 					}
 
 					// _give_form_earnings
-					if ( ! empty( $meta['_give_form_earnings'][0] ) ) {
-						update_post_meta( $post->ID, '_give_form_earnings', give_sanitize_amount_for_db( $meta['_give_form_earnings'][0] ) );
+					if ( ! empty($meta['_give_form_earnings'][0])) {
+						update_post_meta($post->ID, '_give_form_earnings', give_sanitize_amount_for_db($meta['_give_form_earnings'][0]));
 					}
 
 					// _give_custom_amount_minimum
-					if ( ! empty( $meta['_give_donation_levels'][0] ) ) {
-						$donation_levels = unserialize( $meta['_give_donation_levels'][0] );
+					if ( ! empty($meta['_give_donation_levels'][0])) {
+						$donation_levels = unserialize($meta['_give_donation_levels'][0]);
 
-						foreach ( $donation_levels as $index => $level ) {
-							if ( empty( $level['_give_amount'] ) ) {
+						foreach ($donation_levels as $index => $level) {
+							if (empty($level['_give_amount'])) {
 								continue;
 							}
 
-							$donation_levels[ $index ]['_give_amount'] = give_sanitize_amount_for_db( $level['_give_amount'] );
+							$donation_levels[$index]['_give_amount'] = give_sanitize_amount_for_db($level['_give_amount']);
 						}
 
 						$meta['_give_donation_levels'] = $donation_levels;
-						update_post_meta( $post->ID, '_give_donation_levels', $meta['_give_donation_levels'] );
+						update_post_meta($post->ID, '_give_donation_levels', $meta['_give_donation_levels']);
 					}
 
 
@@ -1039,8 +1039,8 @@ 
 
 				case 'give_payment':
 					// _give_payment_total
-					if ( ! empty( $meta['_give_payment_total'][0] ) ) {
-						update_post_meta( $post->ID, '_give_payment_total', give_sanitize_amount_for_db( $meta['_give_payment_total'][0] ) );
+					if ( ! empty($meta['_give_payment_total'][0])) {
+						update_post_meta($post->ID, '_give_payment_total', give_sanitize_amount_for_db($meta['_give_payment_total'][0]));
 					}
 
 					break;
@@ -1051,7 +1051,7 @@ 
 		wp_reset_postdata();
 	} else {
 		// The Update Ran.
-		give_set_upgrade_complete( 'v1812_update_amount_values' );
+		give_set_upgrade_complete('v1812_update_amount_values');
 	}
 }
 
@@ -1071,22 +1071,22 @@ 
 	$offset       = 1 === $give_updates->step ? 0 : $give_updates->step * 20;
 
 	// form query
-	$donors = Give()->donors->get_donors( array(
+	$donors = Give()->donors->get_donors(array(
 			'number' => 20,
 			'offset' => $offset,
 		)
 	);
 
-	if ( ! empty( $donors ) ) {
-		$give_updates->set_percentage( Give()->donors->count(), ( $give_updates->step * 20 ) );
+	if ( ! empty($donors)) {
+		$give_updates->set_percentage(Give()->donors->count(), ($give_updates->step * 20));
 
 		/* @var Object $donor */
-		foreach ( $donors as $donor ) {
-			Give()->donors->update( $donor->id, array( 'purchase_value' => give_sanitize_amount_for_db( $donor->purchase_value ) ) );
+		foreach ($donors as $donor) {
+			Give()->donors->update($donor->id, array('purchase_value' => give_sanitize_amount_for_db($donor->purchase_value)));
 		}
 	} else {
 		// The Update Ran.
-		give_set_upgrade_complete( 'v1812_update_donor_purchase_values' );
+		give_set_upgrade_complete('v1812_update_donor_purchase_values');
 	}
 }
 
@@ -1101,25 +1101,25 @@ 
 	$offset       = 1 === $give_updates->step ? 0 : $give_updates->step * 20;
 
 	// Fetch all the existing donors.
-	$donors = Give()->donors->get_donors( array(
+	$donors = Give()->donors->get_donors(array(
 			'number' => 20,
 			'offset' => $offset,
 		)
 	);
 
-	if ( ! empty( $donors ) ) {
-		$give_updates->set_percentage( Give()->donors->count(), ( $give_updates->step * 20 ) );
+	if ( ! empty($donors)) {
+		$give_updates->set_percentage(Give()->donors->count(), ($give_updates->step * 20));
 
 		/* @var Object $donor */
-		foreach ( $donors as $donor ) {
+		foreach ($donors as $donor) {
 			$user_id = $donor->user_id;
 
 			// Proceed, if donor is attached with user.
-			if ( $user_id ) {
-				$user = get_userdata( $user_id );
+			if ($user_id) {
+				$user = get_userdata($user_id);
 
 				// Update user role, if user has subscriber role.
-				if ( is_array( $user->roles ) && in_array( 'subscriber', $user->roles ) ) {
+				if (is_array($user->roles) && in_array('subscriber', $user->roles)) {
 					wp_update_user(
 						array(
 							'ID'   => $user_id,
@@ -1131,7 +1131,7 @@ 
 		}
 	} else {
 		// The Update Ran.
-		give_set_upgrade_complete( 'v1813_update_donor_user_roles' );
+		give_set_upgrade_complete('v1813_update_donor_user_roles');
 	}
 }
 
@@ -1143,7 +1143,7 @@ 
  */
 function give_v1813_upgrades() {
 	// Update admin setting.
-	give_update_option( 'donor_default_user_role', 'give_donor' );
+	give_update_option('donor_default_user_role', 'give_donor');
 
 	// Update Give roles.
 	$roles = new Give_Roles();

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

Spacing +303 added lines, -303 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);
 	}
 
 	/**
@@ -85,24 +85,24 @@ 
 	 */
 	function get_settings() {
 		$post_id               = give_get_admin_post_id();
-		$price                 = give_get_form_price( $post_id );
-		$custom_amount_minimum = give_get_form_minimum_price( $post_id );
-		$goal                  = give_format_amount( give_get_form_goal( $post_id ), array( 'sanitize' => false ) );
-		$price_placeholder     = give_format_decimal( '1.00', false, false );
+		$price                 = give_get_form_price($post_id);
+		$custom_amount_minimum = give_get_form_minimum_price($post_id);
+		$goal                  = give_format_amount(give_get_form_goal($post_id), array('sanitize' => false));
+		$price_placeholder     = give_format_decimal('1.00', false, false);
 
 		// No empty prices - min. 1.00 for new forms
-		if ( empty( $price ) && is_null( $post_id ) ) {
+		if (empty($price) && is_null($post_id)) {
 			$price = '1.00';
 		}
 
 		// Min. $1.00 for new forms
-		if ( empty( $custom_amount_minimum ) ) {
+		if (empty($custom_amount_minimum)) {
 			$custom_amount_minimum = '1.00';
 		}
 
 		// Format amounts.
-		$price                 = give_format_amount( $price, array( 'sanitize' => false ) );
-		$custom_amount_minimum = give_format_amount( $custom_amount_minimum, array( 'sanitize' => false ) );
+		$price                 = give_format_amount($price, array('sanitize' => false));
+		$custom_amount_minimum = give_format_amount($custom_amount_minimum, array('sanitize' => false));
 
 		// Start with an underscore to hide fields from custom fields list
 		$prefix = '_give_';
@@ -111,27 +111,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'     => 'set',
-						'options'     => apply_filters( 'give_forms_price_options', array(
-							'set'   => __( 'Set Donation', 'give' ),
-							'multi' => __( 'Multi-level Donation', 'give' ),
-						) ),
+						'options'     => apply_filters('give_forms_price_options', array(
+							'set'   => __('Set Donation', 'give'),
+							'multi' => __('Multi-level 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(
@@ -142,34 +142,34 @@ 
 					),
 					// Display Style
 					array(
-						'name'          => __( 'Display Style', 'give' ),
-						'description'   => __( 'Set how the donations levels will display on the form.', 'give' ),
-						'id'            => $prefix . 'display_style',
+						'name'          => __('Display Style', 'give'),
+						'description'   => __('Set how the donations levels will display on the form.', 'give'),
+						'id'            => $prefix.'display_style',
 						'type'          => 'radio_inline',
 						'default'       => 'buttons',
 						'options'       => array(
-							'buttons'  => __( 'Buttons', 'give' ),
-							'radios'   => __( 'Radios', 'give' ),
-							'dropdown' => __( 'Dropdown', 'give' ),
+							'buttons'  => __('Buttons', 'give'),
+							'radios'   => __('Radios', 'give'),
+							'dropdown' => __('Dropdown', 'give'),
 						),
 						'wrapper_class' => 'give-hidden',
 					),
 					// Custom Amount
 					array(
-						'name'        => __( 'Custom Amount', 'give' ),
-						'description' => __( 'Do you want the user to be able to input their own donation amount?', 'give' ),
-						'id'          => $prefix . 'custom_amount',
+						'name'        => __('Custom Amount', 'give'),
+						'description' => __('Do you want the user to be able to input their own donation amount?', 'give'),
+						'id'          => $prefix.'custom_amount',
 						'type'        => 'radio_inline',
 						'default'     => 'disabled',
 						'options'     => array(
-							'enabled'  => __( 'Enabled', 'give' ),
-							'disabled' => __( 'Disabled', 'give' ),
+							'enabled'  => __('Enabled', 'give'),
+							'disabled' => __('Disabled', 'give'),
 						),
 					),
 					array(
-						'name'          => __( 'Minimum Amount', 'give' ),
-						'description'   => __( 'Enter the minimum custom donation amount.', 'give' ),
-						'id'            => $prefix . 'custom_amount_minimum',
+						'name'          => __('Minimum Amount', 'give'),
+						'description'   => __('Enter the minimum custom donation amount.', 'give'),
+						'id'            => $prefix.'custom_amount_minimum',
 						'type'          => 'text_small',
 						'data_type'     => 'price',
 						'attributes'    => array(
@@ -180,37 +180,37 @@ 
 						'wrapper_class' => 'give-hidden',
 					),
 					array(
-						'name'          => __( 'Custom Amount Text', 'give' ),
-						'description'   => __( 'This text appears as a label below the custom amount field for set donation forms. For multi-level forms the text will appear as it\'s own level (ie button, radio, or select option).', 'give' ),
-						'id'            => $prefix . 'custom_amount_text',
+						'name'          => __('Custom Amount Text', 'give'),
+						'description'   => __('This text appears as a label below the custom amount field for set donation forms. For multi-level forms the text will appear as it\'s own level (ie button, radio, or select option).', 'give'),
+						'id'            => $prefix.'custom_amount_text',
 						'type'          => 'text_medium',
 						'attributes'    => array(
 							'rows'        => 3,
-							'placeholder' => esc_attr__( 'Give a Custom Amount', 'give' ),
+							'placeholder' => esc_attr__('Give a Custom Amount', 'give'),
 						),
 						'wrapper_class' => 'give-hidden',
 					),
 					// Donation Levels: Repeatable CMB2 Group
 					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(
@@ -219,112 +219,112 @@ 
 								),
 							),
 							array(
-								'name'       => __( 'Text', 'give' ),
-								'id'         => $prefix . 'text',
+								'name'       => __('Text', 'give'),
+								'id'         => $prefix.'text',
 								'type'       => 'text',
 								'attributes' => array(
-									'placeholder' => __( 'Donation Level', 'give' ),
+									'placeholder' => __('Donation Level', 'give'),
 									'class'       => 'give-multilevel-text-field',
 								),
 							),
 							array(
-								'name' => __( 'Default', 'give' ),
-								'id'   => $prefix . 'default',
+								'name' => __('Default', 'give'),
+								'id'   => $prefix.'default',
 								'type' => 'give_default_radio_inline',
 							),
-						) ),
+						)),
 					),
 					array(
 						'name'  => 'donation_options_docs',
 						'type'  => 'docs_link',
 						'url'   => 'http://docs.givewp.com/form-donation-options',
-						'title' => __( 'Donation Options', 'give' ),
+						'title' => __('Donation Options', 'give'),
 					),
 				),
 					$post_id
 				),
-			) ),
+			)),
 
 			/**
 			 * Display Options
 			 */
-			'form_display_options'  => apply_filters( 'give_form_display_options', array(
+			'form_display_options'  => apply_filters('give_form_display_options', array(
 					'id'        => 'form_display_options',
-					'title'     => __( 'Form Display', 'give' ),
+					'title'     => __('Form Display', 'give'),
 					'icon-html' => '<span class="give-icon give-icon-display"></span>',
-					'fields'    => apply_filters( 'give_forms_display_options_metabox_fields', array(
+					'fields'    => apply_filters('give_forms_display_options_metabox_fields', array(
 						array(
-							'name'    => __( 'Display Options', 'give' ),
-							'desc'    => sprintf( __( 'How would you like to display donation information for this form?', 'give' ), '#' ),
-							'id'      => $prefix . 'payment_display',
+							'name'    => __('Display Options', 'give'),
+							'desc'    => sprintf(__('How would you like to display donation information for this form?', 'give'), '#'),
+							'id'      => $prefix.'payment_display',
 							'type'    => 'radio_inline',
 							'options' => array(
-								'onpage' => __( 'All Fields', 'give' ),
-								'modal'  => __( 'Modal', 'give' ),
-								'reveal' => __( 'Reveal', 'give' ),
-								'button' => __( 'Button', 'give' ),
+								'onpage' => __('All Fields', 'give'),
+								'modal'  => __('Modal', 'give'),
+								'reveal' => __('Reveal', 'give'),
+								'button' => __('Button', 'give'),
 							),
 							'default' => 'onpage',
 						),
 						array(
-							'id'            => $prefix . 'reveal_label',
-							'name'          => __( 'Continue Button', 'give' ),
-							'desc'          => __( 'The button label for displaying the additional payment fields.', 'give' ),
+							'id'            => $prefix.'reveal_label',
+							'name'          => __('Continue Button', 'give'),
+							'desc'          => __('The button label for displaying the additional payment fields.', 'give'),
 							'type'          => 'text_small',
 							'attributes'    => array(
-								'placeholder' => esc_attr__( 'Donate Now', 'give' ),
+								'placeholder' => esc_attr__('Donate Now', 'give'),
 							),
 							'wrapper_class' => 'give-hidden',
 						),
 						array(
-							'id'         => $prefix . 'checkout_label',
-							'name'       => __( 'Submit Button', 'give' ),
-							'desc'       => __( 'The button label for completing a donation.', 'give' ),
+							'id'         => $prefix.'checkout_label',
+							'name'       => __('Submit Button', 'give'),
+							'desc'       => __('The button label for completing a donation.', 'give'),
 							'type'       => 'text_small',
 							'attributes' => array(
-								'placeholder' => __( 'Donate Now', 'give' ),
+								'placeholder' => __('Donate Now', 'give'),
 							),
 						),
 						array(
-							'name' => __( 'Default Gateway', 'give' ),
-							'desc' => __( 'By default, the gateway for this form will inherit the global default gateway (set under Give > Settings > Payment Gateways). This option allows you to customize the default gateway for this form only.', 'give' ),
-							'id'   => $prefix . 'default_gateway',
+							'name' => __('Default Gateway', 'give'),
+							'desc' => __('By default, the gateway for this form will inherit the global default gateway (set under Give > Settings > Payment Gateways). This option allows you to customize the default gateway for this form only.', 'give'),
+							'id'   => $prefix.'default_gateway',
 							'type' => 'default_gateway',
 						),
 						array(
-							'name'    => __( 'Guest Donations', 'give' ),
-							'desc'    => __( 'Do you want to allow non-logged-in users to make donations?', 'give' ),
-							'id'      => $prefix . 'logged_in_only',
+							'name'    => __('Guest Donations', 'give'),
+							'desc'    => __('Do you want to allow non-logged-in users to make donations?', 'give'),
+							'id'      => $prefix.'logged_in_only',
 							'type'    => 'radio_inline',
 							'default' => 'enabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							),
 						),
 						array(
-							'name'    => __( 'Registration', 'give' ),
-							'desc'    => __( 'Display the registration and login forms in the payment section for non-logged-in users.', 'give' ),
-							'id'      => $prefix . 'show_register_form',
+							'name'    => __('Registration', 'give'),
+							'desc'    => __('Display the registration and login forms in the payment section for non-logged-in users.', 'give'),
+							'id'      => $prefix.'show_register_form',
 							'type'    => 'radio',
 							'options' => array(
-								'none'         => __( 'None', 'give' ),
-								'registration' => __( 'Registration', 'give' ),
-								'login'        => __( 'Login', 'give' ),
-								'both'         => __( 'Registration + Login', 'give' ),
+								'none'         => __('None', 'give'),
+								'registration' => __('Registration', 'give'),
+								'login'        => __('Login', 'give'),
+								'both'         => __('Registration + Login', 'give'),
 							),
 							'default' => 'none',
 						),
 						array(
-							'name'    => __( 'Floating Labels', 'give' ),
+							'name'    => __('Floating Labels', 'give'),
 							/* translators: %s: forms http://docs.givewp.com/form-floating-labels */
-							'desc'    => sprintf( __( 'Select the <a href="%s" target="_blank">floating labels</a> setting for this Give form. Be aware that if you have the "Disable CSS" option enabled, you will need to style the floating labels yourself.', 'give' ), esc_url( 'http://docs.givewp.com/form-floating-labels' ) ),
-							'id'      => $prefix . 'form_floating_labels',
+							'desc'    => sprintf(__('Select the <a href="%s" target="_blank">floating labels</a> setting for this Give form. Be aware that if you have the "Disable CSS" option enabled, you will need to style the floating labels yourself.', 'give'), esc_url('http://docs.givewp.com/form-floating-labels')),
+							'id'      => $prefix.'form_floating_labels',
 							'type'    => 'radio_inline',
 							'options' => array(
-								'global'   => __( 'Global Option', 'give' ),
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'global'   => __('Global Option', 'give'),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							),
 							'default' => 'global',
 						),
@@ -332,7 +332,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
@@ -343,31 +343,31 @@ 
 			/**
 			 * Donation Goals
 			 */
-			'donation_goal_options' => apply_filters( 'give_donation_goal_options', array(
+			'donation_goal_options' => apply_filters('give_donation_goal_options', array(
 				'id'        => 'donation_goal_options',
-				'title'     => __( 'Donation Goal', 'give' ),
+				'title'     => __('Donation Goal', 'give'),
 				'icon-html' => '<span class="give-icon give-icon-target"></span>',
-				'fields'    => apply_filters( 'give_forms_donation_goal_metabox_fields', array(
+				'fields'    => apply_filters('give_forms_donation_goal_metabox_fields', array(
 					// Goals
 					array(
-						'name'        => __( 'Donation Goal', 'give' ),
-						'description' => __( 'Do you want to set a donation goal for this form?', 'give' ),
-						'id'          => $prefix . 'goal_option',
+						'name'        => __('Donation Goal', 'give'),
+						'description' => __('Do you want to set a donation goal for this form?', 'give'),
+						'id'          => $prefix.'goal_option',
 						'type'        => 'radio_inline',
 						'default'     => 'disabled',
 						'options'     => array(
-							'enabled'  => __( 'Enabled', 'give' ),
-							'disabled' => __( 'Disabled', 'give' ),
+							'enabled'  => __('Enabled', 'give'),
+							'disabled' => __('Disabled', 'give'),
 						),
 					),
 					array(
-						'name'          => __( 'Goal Amount', 'give' ),
-						'description'   => __( 'This is the monetary goal amount you want to reach for this form.', 'give' ),
-						'id'            => $prefix . 'set_goal',
+						'name'          => __('Goal Amount', 'give'),
+						'description'   => __('This is the monetary goal amount you want to reach for this form.', 'give'),
+						'id'            => $prefix.'set_goal',
 						'type'          => 'text_small',
 						'data_type'     => 'price',
 						'attributes'    => array(
-							'placeholder' => give_format_decimal( '0.00', false, false ),
+							'placeholder' => give_format_decimal('0.00', false, false),
 							'value'       => $goal,
 							'class'       => 'give-money-field',
 						),
@@ -375,97 +375,97 @@ 
 					),
 
 					array(
-						'name'          => __( 'Goal Format', 'give' ),
-						'description'   => __( 'Do you want to display the total amount raised based on your monetary goal or a percentage? For instance, "$500 of $1,000 raised" or "50% funded".', 'give' ),
-						'id'            => $prefix . 'goal_format',
+						'name'          => __('Goal Format', 'give'),
+						'description'   => __('Do you want to display the total amount raised based on your monetary goal or a percentage? For instance, "$500 of $1,000 raised" or "50% funded".', 'give'),
+						'id'            => $prefix.'goal_format',
 						'type'          => 'radio_inline',
 						'default'       => 'amount',
 						'options'       => array(
-							'amount'     => __( 'Amount', 'give' ),
-							'percentage' => __( 'Percentage', 'give' ),
+							'amount'     => __('Amount', 'give'),
+							'percentage' => __('Percentage', 'give'),
 						),
 						'wrapper_class' => 'give-hidden',
 					),
 					array(
-						'name'          => __( 'Progress Bar Color', 'give' ),
-						'desc'          => __( 'Customize the color of the goal progress bar.', 'give' ),
-						'id'            => $prefix . 'goal_color',
+						'name'          => __('Progress Bar Color', 'give'),
+						'desc'          => __('Customize the color of the goal progress bar.', 'give'),
+						'id'            => $prefix.'goal_color',
 						'type'          => 'colorpicker',
 						'default'       => '#2bc253',
 						'wrapper_class' => 'give-hidden',
 					),
 
 					array(
-						'name'          => __( 'Close Form', 'give' ),
-						'desc'          => __( 'Do you want to close the donation forms and stop accepting donations once this goal has been met?', 'give' ),
-						'id'            => $prefix . 'close_form_when_goal_achieved',
+						'name'          => __('Close Form', 'give'),
+						'desc'          => __('Do you want to close the donation forms and stop accepting donations once this goal has been met?', 'give'),
+						'id'            => $prefix.'close_form_when_goal_achieved',
 						'type'          => 'radio_inline',
 						'default'       => 'disabled',
 						'options'       => array(
-							'enabled'  => __( 'Enabled', 'give' ),
-							'disabled' => __( 'Disabled', 'give' ),
+							'enabled'  => __('Enabled', 'give'),
+							'disabled' => __('Disabled', 'give'),
 						),
 						'wrapper_class' => 'give-hidden',
 					),
 					array(
-						'name'          => __( 'Goal Achieved Message', 'give' ),
-						'desc'          => __( 'Do you want to display a custom message when the goal is closed?', 'give' ),
-						'id'            => $prefix . 'form_goal_achieved_message',
+						'name'          => __('Goal Achieved Message', 'give'),
+						'desc'          => __('Do you want to display a custom message when the goal is closed?', 'give'),
+						'id'            => $prefix.'form_goal_achieved_message',
 						'type'          => 'wysiwyg',
-						'default'       => __( 'Thank you to all our donors, we have met our fundraising goal.', 'give' ),
+						'default'       => __('Thank you to all our donors, we have met our fundraising goal.', 'give'),
 						'wrapper_class' => 'give-hidden',
 					),
 					array(
 						'name'  => 'donation_goal_docs',
 						'type'  => 'docs_link',
 						'url'   => 'http://docs.givewp.com/form-donation-goal',
-						'title' => __( 'Donation Goal', 'give' ),
+						'title' => __('Donation Goal', 'give'),
 					),
 				),
 					$post_id
 				),
-			) ),
+			)),
 
 			/**
 			 * Content Field
 			 */
-			'form_content_options'  => apply_filters( 'give_forms_content_options', array(
+			'form_content_options'  => apply_filters('give_forms_content_options', array(
 				'id'        => 'form_content_options',
-				'title'     => __( 'Form Content', 'give' ),
+				'title'     => __('Form Content', 'give'),
 				'icon-html' => '<span class="give-icon give-icon-edit"></span>',
-				'fields'    => apply_filters( 'give_forms_content_options_metabox_fields', array(
+				'fields'    => apply_filters('give_forms_content_options_metabox_fields', array(
 
 					// Donation content.
 					array(
-						'name'        => __( 'Display Content', 'give' ),
-						'description' => __( 'Do you want to add custom content to this form?', 'give' ),
-						'id'          => $prefix . 'display_content',
+						'name'        => __('Display Content', 'give'),
+						'description' => __('Do you want to add custom content to this form?', 'give'),
+						'id'          => $prefix.'display_content',
 						'type'        => 'radio_inline',
 						'options'     => array(
-							'enabled'  => __( 'Enabled', 'give' ),
-							'disabled' => __( 'Disabled', 'give' ),
+							'enabled'  => __('Enabled', 'give'),
+							'disabled' => __('Disabled', 'give'),
 						),
 						'default'     => 'disabled',
 					),
 
 					// Content placement.
 					array(
-						'name'          => __( 'Content Placement', 'give' ),
-						'description'   => __( 'This option controls where the content appears within the donation form.', 'give' ),
-						'id'            => $prefix . 'content_placement',
+						'name'          => __('Content Placement', 'give'),
+						'description'   => __('This option controls where the content appears within the donation form.', 'give'),
+						'id'            => $prefix.'content_placement',
 						'type'          => 'radio_inline',
-						'options'       => apply_filters( 'give_forms_content_options_select', array(
-								'give_pre_form'  => __( 'Above fields', 'give' ),
-								'give_post_form' => __( 'Below fields', 'give' ),
+						'options'       => apply_filters('give_forms_content_options_select', array(
+								'give_pre_form'  => __('Above fields', 'give'),
+								'give_post_form' => __('Below fields', 'give'),
 							)
 						),
 						'default'       => 'give_pre_form',
 						'wrapper_class' => 'give-hidden',
 					),
 					array(
-						'name'          => __( 'Content', 'give' ),
-						'description'   => __( 'This content will display on the single give form page.', 'give' ),
-						'id'            => $prefix . 'form_content',
+						'name'          => __('Content', 'give'),
+						'description'   => __('This content will display on the single give form page.', 'give'),
+						'id'            => $prefix.'form_content',
 						'type'          => 'wysiwyg',
 						'wrapper_class' => 'give-hidden',
 					),
@@ -473,51 +473,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' => esc_attr__( 'Agree to Terms?', 'give' ),
+							'placeholder' => esc_attr__('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',
 					),
@@ -525,18 +525,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;
@@ -552,8 +552,8 @@ 
 		add_meta_box(
 			$this->get_metabox_ID(),
 			$this->get_metabox_label(),
-			array( $this, 'output' ),
-			array( 'give_forms' ),
+			array($this, 'output'),
+			array('give_forms'),
 			'normal',
 			'high'
 		);
@@ -569,7 +569,7 @@ 
 	function enqueue_script() {
 		global $post;
 
-		if ( is_object( $post ) && 'give_forms' === $post->post_type ) {
+		if (is_object($post) && 'give_forms' === $post->post_type) {
 
 		}
 	}
@@ -604,32 +604,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;
 				}
 			}
@@ -646,33 +646,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>
@@ -687,30 +687,30 @@ 
 					<?php endforeach; ?>
 				</ul>
 
-				<?php foreach ( $this->settings as $setting ) : ?>
-					<?php if ( ! $this->has_sub_tab( $setting ) ) : ?>
-						<?php do_action( "give_before_{$setting['id']}_settings" ); ?>
+				<?php foreach ($this->settings as $setting) : ?>
+					<?php if ( ! $this->has_sub_tab($setting)) : ?>
+						<?php do_action("give_before_{$setting['id']}_settings"); ?>
 						<?php
 						// Determine if current panel is active.
 						$is_active = $active_tab === $setting['id'] ? true : false;
 						?>
-						<div id="<?php echo $setting['id']; ?>" class="panel give_options_panel<?php echo( $is_active ? ' active' : '' ); ?>">
-							<?php if ( ! empty( $setting['fields'] ) ) : ?>
-								<?php foreach ( $setting['fields'] as $field ) : ?>
-									<?php give_render_field( $field ); ?>
+						<div id="<?php echo $setting['id']; ?>" class="panel give_options_panel<?php echo($is_active ? ' active' : ''); ?>">
+							<?php if ( ! empty($setting['fields'])) : ?>
+								<?php foreach ($setting['fields'] as $field) : ?>
+									<?php give_render_field($field); ?>
 								<?php endforeach; ?>
 							<?php endif; ?>
 						</div>
 
-						<?php do_action( "give_after_{$setting['id']}_settings" ); ?>
+						<?php do_action("give_after_{$setting['id']}_settings"); ?>
 					<?php else : ?>
-						<?php if ( $this->has_sub_tab( $setting ) ) : ?>
-							<?php if ( ! empty( $setting['sub-fields'] ) ) : ?>
-								<?php foreach ( $setting['sub-fields'] as $index => $sub_fields ) : ?>
+						<?php if ($this->has_sub_tab($setting)) : ?>
+							<?php if ( ! empty($setting['sub-fields'])) : ?>
+								<?php foreach ($setting['sub-fields'] as $index => $sub_fields) : ?>
 									<div id="<?php echo $sub_fields['id']; ?>" class="panel give_options_panel give-hidden">
-										<?php if ( ! empty( $sub_fields['fields'] ) ) : ?>
-											<?php foreach ( $sub_fields['fields'] as $sub_field ) : ?>
-												<?php give_render_field( $sub_field ); ?>
+										<?php if ( ! empty($sub_fields['fields'])) : ?>
+											<?php foreach ($sub_fields['fields'] as $sub_field) : ?>
+												<?php give_render_field($sub_field); ?>
 											<?php endforeach; ?>
 										<?php endif; ?>
 									</div>
@@ -734,9 +734,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;
 		}
 
@@ -750,13 +750,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]);
 			}
 		}
 
@@ -774,96 +774,96 @@ 
 	 *
 	 * @return void
 	 */
-	public function save( $post_id, $post ) {
+	public function save($post_id, $post) {
 
 		// $post_id and $post are required.
-		if ( empty( $post_id ) || empty( $post ) ) {
+		if (empty($post_id) || empty($post)) {
 			return;
 		}
 
 		// Don't save meta boxes for revisions or autosaves.
-		if ( defined( 'DOING_AUTOSAVE' ) || is_int( wp_is_post_revision( $post ) ) || is_int( wp_is_post_autosave( $post ) ) ) {
+		if (defined('DOING_AUTOSAVE') || is_int(wp_is_post_revision($post)) || is_int(wp_is_post_autosave($post))) {
 			return;
 		}
 
 		// Check the nonce.
-		if ( empty( $_POST['give_form_meta_nonce'] ) || ! wp_verify_nonce( $_POST['give_form_meta_nonce'], 'give_save_form_meta' ) ) {
+		if (empty($_POST['give_form_meta_nonce']) || ! wp_verify_nonce($_POST['give_form_meta_nonce'], 'give_save_form_meta')) {
 			return;
 		}
 
 		// Check the post being saved == the $post_id to prevent triggering this call for other save_post events.
-		if ( empty( $_POST['post_ID'] ) || $_POST['post_ID'] != $post_id ) {
+		if (empty($_POST['post_ID']) || $_POST['post_ID'] != $post_id) {
 			return;
 		}
 
 		// Check user has permission to edit.
-		if ( ! current_user_can( 'edit_post', $post_id ) ) {
+		if ( ! current_user_can('edit_post', $post_id)) {
 			return;
 		}
 
 		// Fire action before saving form meta.
-		do_action( 'give_pre_process_give_forms_meta', $post_id, $post );
+		do_action('give_pre_process_give_forms_meta', $post_id, $post);
 
 		/**
 		 * Filter the meta key to save.
 		 * Third party addon developer can remove there meta keys from this array to handle saving data on there own.
 		 */
-		$form_meta_keys = apply_filters( 'give_process_form_meta_keys', $this->get_meta_keys_from_settings() );
+		$form_meta_keys = apply_filters('give_process_form_meta_keys', $this->get_meta_keys_from_settings());
 
 		// Save form meta data.
-		if ( ! empty( $form_meta_keys ) ) {
-			foreach ( $form_meta_keys as $form_meta_key ) {
+		if ( ! empty($form_meta_keys)) {
+			foreach ($form_meta_keys as $form_meta_key) {
 
 				// Set default value for checkbox fields.
 				if (
-					! isset( $_POST[ $form_meta_key ] )
-					&& ( 'checkbox' === $this->get_field_type( $form_meta_key ) )
+					! isset($_POST[$form_meta_key])
+					&& ('checkbox' === $this->get_field_type($form_meta_key))
 				) {
-					$_POST[ $form_meta_key ] = '';
+					$_POST[$form_meta_key] = '';
 				}
 
-				if ( isset( $_POST[ $form_meta_key ] ) ) {
-					$setting_field = $this->get_setting_field( $form_meta_key );
-					if ( ! empty( $setting_field['type'] ) ) {
-						switch ( $setting_field['type'] ) {
+				if (isset($_POST[$form_meta_key])) {
+					$setting_field = $this->get_setting_field($form_meta_key);
+					if ( ! empty($setting_field['type'])) {
+						switch ($setting_field['type']) {
 							case 'textarea':
 							case 'wysiwyg':
-								$form_meta_value = wp_kses_post( $_POST[ $form_meta_key ] );
+								$form_meta_value = wp_kses_post($_POST[$form_meta_key]);
 								break;
 
 							case 'group':
 								$form_meta_value = array();
 
-								foreach ( $_POST[ $form_meta_key ] as $index => $group ) {
+								foreach ($_POST[$form_meta_key] as $index => $group) {
 
 									// Do not save template input field values.
-									if ( '{{row-count-placeholder}}' === $index ) {
+									if ('{{row-count-placeholder}}' === $index) {
 										continue;
 									}
 
 									$group_meta_value = array();
-									foreach ( $group as $field_id => $field_value ) {
-										switch ( $this->get_field_type( $field_id, $form_meta_key ) ) {
+									foreach ($group as $field_id => $field_value) {
+										switch ($this->get_field_type($field_id, $form_meta_key)) {
 											case 'wysiwyg':
-												$group_meta_value[ $field_id ] = wp_kses_post( $field_value );
+												$group_meta_value[$field_id] = wp_kses_post($field_value);
 												break;
 
 											default:
-												$group_meta_value[ $field_id ] = give_clean( $field_value );
+												$group_meta_value[$field_id] = give_clean($field_value);
 										}
 									}
 
-									if ( ! empty( $group_meta_value ) ) {
-										$form_meta_value[ $index ] = $group_meta_value;
+									if ( ! empty($group_meta_value)) {
+										$form_meta_value[$index] = $group_meta_value;
 									}
 								}
 
 								// Arrange repeater field keys in order.
-								$form_meta_value = array_values( $form_meta_value );
+								$form_meta_value = array_values($form_meta_value);
 								break;
 
 							default:
-								$form_meta_value = give_clean( $_POST[ $form_meta_key ] );
+								$form_meta_value = give_clean($_POST[$form_meta_key]);
 						}// End switch().
 
 						/**
@@ -873,24 +873,24 @@ 
 						 */
 						$form_meta_value = apply_filters(
 							'give_pre_save_form_meta_value',
-							$this->sanitize_form_meta( $form_meta_value, $setting_field ),
+							$this->sanitize_form_meta($form_meta_value, $setting_field),
 							$form_meta_key,
 							$this,
 							$post_id
 						);
 
 						// Save data.
-						give_update_meta( $post_id, $form_meta_key, $form_meta_value );
+						give_update_meta($post_id, $form_meta_key, $form_meta_value);
 
 						// Fire after saving form meta key.
-						do_action( "give_save_{$form_meta_key}", $form_meta_key, $form_meta_value, $post_id, $post );
+						do_action("give_save_{$form_meta_key}", $form_meta_key, $form_meta_value, $post_id, $post);
 					}// End if().
 				}// End if().
 			}// End foreach().
 		}// End if().
 
 		// Fire action after saving form meta.
-		do_action( 'give_post_process_give_forms_meta', $post_id, $post );
+		do_action('give_post_process_give_forms_meta', $post_id, $post);
 	}
 
 
@@ -903,10 +903,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'];
 
 		}
@@ -923,12 +923,12 @@ 
 	 *
 	 * @return array
 	 */
-	private function get_fields_id( $setting ) {
+	private function get_fields_id($setting) {
 		$meta_keys = array();
 
-		if ( ! empty( $setting ) ) {
-			foreach ( $setting['fields'] as $field ) {
-				if ( $field_id = $this->get_field_id( $field ) ) {
+		if ( ! empty($setting)) {
+			foreach ($setting['fields'] as $field) {
+				if ($field_id = $this->get_field_id($field)) {
 					$meta_keys[] = $field_id;
 				}
 			}
@@ -946,14 +946,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;
 						}
 					}
@@ -974,14 +974,14 @@ 
 	private function get_meta_keys_from_settings() {
 		$meta_keys = array();
 
-		foreach ( $this->settings as $setting ) {
-			if ( $this->has_sub_tab( $setting ) ) {
-				$meta_key = $this->get_sub_fields_id( $setting );
+		foreach ($this->settings as $setting) {
+			if ($this->has_sub_tab($setting)) {
+				$meta_key = $this->get_sub_fields_id($setting);
 			} else {
-				$meta_key = $this->get_fields_id( $setting );
+				$meta_key = $this->get_fields_id($setting);
 			}
 
-			$meta_keys = array_merge( $meta_keys, $meta_key );
+			$meta_keys = array_merge($meta_keys, $meta_key);
 		}
 
 		return $meta_keys;
@@ -998,10 +998,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']
 			: '';
 
@@ -1019,12 +1019,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;
 				}
@@ -1044,12 +1044,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;
 				}
@@ -1069,17 +1069,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;
 				}
@@ -1087,9 +1087,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;
 				}
 			}
@@ -1108,14 +1108,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;
@@ -1133,22 +1133,22 @@ 
 	 *
 	 * @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'] ] )
+							$meta_value[$index][$field['id']] = ! empty($meta_value[$index][$field['id']])
+								? give_sanitize_amount_for_db($meta_value[$index][$field['id']])
 								: 0;
 						}
 					}
@@ -1156,8 +1156,8 @@ 
 				break;
 
 			default:
-				if ( ! empty( $setting_field['data_type'] ) && 'price' === $setting_field['data_type'] ) {
-					$meta_value = $meta_value ? give_sanitize_amount_for_db( $meta_value ) : 0;
+				if ( ! empty($setting_field['data_type']) && 'price' === $setting_field['data_type']) {
+					$meta_value = $meta_value ? give_sanitize_amount_for_db($meta_value) : 0;
 				}
 		}
 
@@ -1175,12 +1175,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;

Braces +15 added lines, -6 removed lines

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

includes/admin/class-blank-slate.php

Spacing +55 added lines, -55 removed lines

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

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

Braces +4 added lines, -1 removed lines

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

Spacing +17 added lines, -17 removed lines

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