WordImpress / Give

includes/donors/frontend-donor-functions.php

Doc Comments +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
  *
  * @since 2.2.0
  *
- * @param string|int $id_or_email
+ * @param string $id_or_email
  *
  * @return bool
  */

Spacing +42 added lines, -42 removed lines

@@ -21,15 +21,15 @@ 
  *
  * @return string HTML output.
  */
-function give_get_donor_avatar( $donor, $size = 60 ) {
+function give_get_donor_avatar($donor, $size = 60) {
 	ob_start();
 	?>
 	<div class="give-donor__image">
 		<?php
 		// Check if gravatar exists.
-		if ( give_validate_gravatar( $donor->email ) ) {
+		if (give_validate_gravatar($donor->email)) {
 			// Return avatar.
-			echo get_avatar( $donor->email, $size );
+			echo get_avatar($donor->email, $size);
 		} else {
 			// No gravatar = output initials.
 			echo $donor->get_donor_initals();
@@ -37,7 +37,7 @@ 
 	</div>
 	<?php
 
-	return apply_filters( 'give_get_donor_avatar', ob_get_clean() );
+	return apply_filters('give_get_donor_avatar', ob_get_clean());
 
 }
 
@@ -50,51 +50,51 @@ 
  *
  * @return bool
  */
-function give_validate_gravatar( $id_or_email ) {
+function give_validate_gravatar($id_or_email) {
 
 	//id or email code borrowed from wp-includes/pluggable.php
 	$email = '';
-	if ( is_numeric( $id_or_email ) ) {
+	if (is_numeric($id_or_email)) {
 		$id   = (int) $id_or_email;
-		$user = get_userdata( $id );
-		if ( $user ) {
+		$user = get_userdata($id);
+		if ($user) {
 			$email = $user->user_email;
 		}
-	} elseif ( is_object( $id_or_email ) ) {
+	} elseif (is_object($id_or_email)) {
 		// No avatar for pingbacks or trackbacks
-		$allowed_comment_types = apply_filters( 'get_avatar_comment_types', array( 'comment' ) );
-		if ( ! empty( $id_or_email->comment_type ) && ! in_array( $id_or_email->comment_type, (array) $allowed_comment_types ) ) {
+		$allowed_comment_types = apply_filters('get_avatar_comment_types', array('comment'));
+		if ( ! empty($id_or_email->comment_type) && ! in_array($id_or_email->comment_type, (array) $allowed_comment_types)) {
 			return false;
 		}
 
-		if ( ! empty( $id_or_email->user_id ) ) {
+		if ( ! empty($id_or_email->user_id)) {
 			$id   = (int) $id_or_email->user_id;
-			$user = get_userdata( $id );
-			if ( $user ) {
+			$user = get_userdata($id);
+			if ($user) {
 				$email = $user->user_email;
 			}
-		} elseif ( ! empty( $id_or_email->comment_author_email ) ) {
+		} elseif ( ! empty($id_or_email->comment_author_email)) {
 			$email = $id_or_email->comment_author_email;
 		}
 	} else {
 		$email = $id_or_email;
 	}
 
-	$hashkey = md5( strtolower( trim( $email ) ) );
-	$uri     = 'http://www.gravatar.com/avatar/' . $hashkey . '?d=404';
+	$hashkey = md5(strtolower(trim($email)));
+	$uri     = 'http://www.gravatar.com/avatar/'.$hashkey.'?d=404';
 
-	$data = wp_cache_get( $hashkey );
-	if ( false === $data ) {
-		$response = wp_remote_head( $uri );
-		if ( is_wp_error( $response ) ) {
+	$data = wp_cache_get($hashkey);
+	if (false === $data) {
+		$response = wp_remote_head($uri);
+		if (is_wp_error($response)) {
 			$data = 'not200';
 		} else {
 			$data = $response['response']['code'];
 		}
-		wp_cache_set( $hashkey, $data, $group = '', $expire = 60 * 5 );
+		wp_cache_set($hashkey, $data, $group = '', $expire = 60 * 5);
 
 	}
-	if ( $data == '200' ) {
+	if ($data == '200') {
 		return true;
 	} else {
 		return false;
@@ -114,18 +114,18 @@ 
  *
  * @return int The new note ID
  */
-function give_insert_donor_donation_comment( $donation_id, $donor, $note, $comment_args = array() ) {
+function give_insert_donor_donation_comment($donation_id, $donor, $note, $comment_args = array()) {
 	$comment_args = wp_parse_args(
 		$comment_args,
 		array(
 			'comment_approved' => 0,
-			'comment_parent'   => give_get_payment_form_id( $donation_id )
+			'comment_parent'   => give_get_payment_form_id($donation_id)
 		)
 	);
 
-	$comment_id = Give_Comment::add( $donation_id, $note, 'payment', $comment_args );
+	$comment_id = Give_Comment::add($donation_id, $note, 'payment', $comment_args);
 
-	update_comment_meta( $comment_id, '_give_donor_id', $donor );
+	update_comment_meta($comment_id, '_give_donor_id', $donor);
 
 	return $comment_id;
 }
@@ -144,7 +144,7 @@ 
  *
  * @return WP_Comment|array
  */
-function give_get_donor_donation_comment( $donation_id, $donor_id, $search = '' ) {
+function give_get_donor_donation_comment($donation_id, $donor_id, $search = '') {
 	$comments = Give_Comment::get(
 		$donation_id,
 		'payment',
@@ -160,7 +160,7 @@ 
 		$search
 	);
 
-	return ( ! empty( $comments ) ? current( $comments ) : array() );
+	return ( ! empty($comments) ? current($comments) : array());
 }
 
 /**
@@ -176,9 +176,9 @@ 
  *
  * @return int
  */
-function give_get_donor_donation_comment_id( $donation_id, $donor_id, $search = '' ) {
+function give_get_donor_donation_comment_id($donation_id, $donor_id, $search = '') {
 	/* @var WP_Comment|array $comment */
-	$comment    = give_get_donor_donation_comment( $donation_id, $donor_id, $search );
+	$comment    = give_get_donor_donation_comment($donation_id, $donor_id, $search);
 	$comment_id = $comment instanceof WP_Comment ? $comment->comment_ID : 0;
 
 	return $comment_id;
@@ -197,7 +197,7 @@ 
  *
  * @return array
  */
-function give_get_donor_donation_comments( $donor_id, $comment_args = array(), $search = '' ) {
+function give_get_donor_donation_comments($donor_id, $comment_args = array(), $search = '') {
 	$comments = Give_Comment::get(
 		$donor_id,
 		'payment',
@@ -205,7 +205,7 @@ 
 		$search
 	);
 
-	return ( ! empty( $comments ) ? $comments : array() );
+	return ( ! empty($comments) ? $comments : array());
 }
 
 
@@ -219,19 +219,19 @@ 
  *
  * @return string
  */
-function give_get_donor_donation_comment_html( $comment, $payment_id = 0 ) {
+function give_get_donor_donation_comment_html($comment, $payment_id = 0) {
 
-	if ( is_numeric( $comment ) ) {
-		$comment = get_comment( $comment );
+	if (is_numeric($comment)) {
+		$comment = get_comment($comment);
 	}
 
-	$date_format = give_date_format() . ', ' . get_option( 'time_format' );
+	$date_format = give_date_format().', '.get_option('time_format');
 
 	$comment_html = sprintf(
 		'<div class="give-payment-note" id="give-payment-note-%s"><p><strong>%s</strong>&nbsp;&ndash;&nbsp;<span style="color:#aaa;font-style:italic;">%s</span><br/>%s</p></div>',
 		$comment->comment_ID,
-		get_comment_author( $comment->comment_ID ),
-		date_i18n( $date_format, strtotime( $comment->comment_date ) ),
+		get_comment_author($comment->comment_ID),
+		date_i18n($date_format, strtotime($comment->comment_date)),
 		$comment->comment_content
 	);
 
@@ -250,7 +250,7 @@ 
  *
  * @return WP_Comment/array
  */
-function give_get_donor_latest_comment( $donor_id, $form_id = 0 ) {
+function give_get_donor_latest_comment($donor_id, $form_id = 0) {
 	$comment_args = array(
 		'post_id'    => 0,
 		'orderby'    => 'comment_ID',
@@ -270,11 +270,11 @@ 
 	);
 
 	// Get donor donation comment for specific form.
-	if ( $form_id ) {
+	if ($form_id) {
 		$comment_args['parent'] = $form_id;
 	}
 
-	$comment = current( give_get_donor_donation_comments( $donor_id, $comment_args ) );
+	$comment = current(give_get_donor_donation_comments($donor_id, $comment_args));
 
 	return $comment;
 }

includes/payments/class-give-payment.php

Doc Comments +2 added lines, -2 removed lines

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

Spacing +280 added lines, -281 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -407,13 +407,13 @@ 
 	 *
 	 * @return mixed void|false
 	 */
-	public function __construct( $payment_id = false ) {
+	public function __construct($payment_id = false) {
 
-		if ( empty( $payment_id ) ) {
+		if (empty($payment_id)) {
 			return false;
 		}
 
-		$this->setup_payment( $payment_id );
+		$this->setup_payment($payment_id);
 	}
 
 	/**
@@ -426,11 +426,11 @@ 
 	 *
 	 * @return mixed        The value.
 	 */
-	public function __get( $key ) {
+	public function __get($key) {
 
-		if ( method_exists( $this, 'get_' . $key ) ) {
+		if (method_exists($this, 'get_'.$key)) {
 
-			$value = call_user_func( array( $this, 'get_' . $key ) );
+			$value = call_user_func(array($this, 'get_'.$key));
 
 		} else {
 
@@ -452,18 +452,18 @@ 
 	 * @param  string $key   The property name.
 	 * @param  mixed  $value The value of the property.
 	 */
-	public function __set( $key, $value ) {
-		$ignore = array( '_ID' );
+	public function __set($key, $value) {
+		$ignore = array('_ID');
 
-		if ( 'status' === $key ) {
+		if ('status' === $key) {
 			$this->old_status = $this->status;
 		}
 
-		if ( ! in_array( $key, $ignore ) ) {
-			$this->pending[ $key ] = $value;
+		if ( ! in_array($key, $ignore)) {
+			$this->pending[$key] = $value;
 		}
 
-		if ( '_ID' !== $key ) {
+		if ('_ID' !== $key) {
 			$this->$key = $value;
 		}
 	}
@@ -478,9 +478,9 @@ 
 	 *
 	 * @return boolean|null       If the item is set or not
 	 */
-	public function __isset( $name ) {
-		if ( property_exists( $this, $name ) ) {
-			return false === empty( $this->$name );
+	public function __isset($name) {
+		if (property_exists($this, $name)) {
+			return false === empty($this->$name);
 		} else {
 			return null;
 		}
@@ -496,20 +496,20 @@ 
 	 *
 	 * @return bool            If the setup was successful or not
 	 */
-	private function setup_payment( $payment_id ) {
+	private function setup_payment($payment_id) {
 		$this->pending = array();
 
-		if ( empty( $payment_id ) ) {
+		if (empty($payment_id)) {
 			return false;
 		}
 
-		$payment = get_post( absint( $payment_id ) );
+		$payment = get_post(absint($payment_id));
 
-		if ( ! $payment || is_wp_error( $payment ) ) {
+		if ( ! $payment || is_wp_error($payment)) {
 			return false;
 		}
 
-		if ( 'give_payment' !== $payment->post_type ) {
+		if ('give_payment' !== $payment->post_type) {
 			return false;
 		}
 
@@ -523,17 +523,17 @@ 
 		 * @param Give_Payment $this       Payment object.
 		 * @param int          $payment_id The ID of the payment.
 		 */
-		do_action( 'give_pre_setup_payment', $this, $payment_id );
+		do_action('give_pre_setup_payment', $this, $payment_id);
 
 		// Get payment from cache.
-		$donation_vars = Give_Cache::get_group( $payment_id, 'give-donations' );
+		$donation_vars = Give_Cache::get_group($payment_id, 'give-donations');
 
-		if ( is_null( $donation_vars ) ) {
+		if (is_null($donation_vars)) {
 			// Primary Identifier.
-			$this->ID = absint( $payment_id );
+			$this->ID = absint($payment_id);
 
 			// Protected ID that can never be changed.
-			$this->_ID = absint( $payment_id );
+			$this->_ID = absint($payment_id);
 
 			// We have a payment, get the generic payment_meta item to reduce calls to it.
 			$this->payment_meta = $this->get_meta();
@@ -549,7 +549,7 @@ 
 			$this->parent_payment = $payment->post_parent;
 
 			$all_payment_statuses  = give_get_payment_statuses();
-			$this->status_nicename = array_key_exists( $this->status, $all_payment_statuses ) ? $all_payment_statuses[ $this->status ] : ucfirst( $this->status );
+			$this->status_nicename = array_key_exists($this->status, $all_payment_statuses) ? $all_payment_statuses[$this->status] : ucfirst($this->status);
 
 			// Currency Based.
 			$this->total    = $this->setup_total();
@@ -570,7 +570,7 @@ 
 			$this->address      = $this->setup_address();
 			$this->first_name   = $this->user_info['first_name'];
 			$this->last_name    = $this->user_info['last_name'];
-			$this->title_prefix = isset( $this->user_info['title'] ) ? $this->user_info['title'] : '';
+			$this->title_prefix = isset($this->user_info['title']) ? $this->user_info['title'] : '';
 
 			// Other Identifiers.
 			$this->form_title = $this->setup_form_title();
@@ -579,10 +579,10 @@ 
 			$this->key        = $this->setup_payment_key();
 			$this->number     = $this->setup_payment_number();
 
-			Give_Cache::set_group( $this->ID, get_object_vars( $this ), 'give-donations' );
+			Give_Cache::set_group($this->ID, get_object_vars($this), 'give-donations');
 		} else {
 
-			foreach ( $donation_vars as $donation_var => $value ) {
+			foreach ($donation_vars as $donation_var => $value) {
 				$this->$donation_var = $value;
 			}
 		} // End if().
@@ -597,7 +597,7 @@ 
 		 * @param Give_Payment $this       Payment object.
 		 * @param int          $payment_id The ID of the payment.
 		 */
-		do_action( 'give_setup_payment', $this, $payment_id );
+		do_action('give_setup_payment', $this, $payment_id);
 
 		return true;
 	}
@@ -616,11 +616,11 @@ 
 	 *
 	 * @return void
 	 */
-	public function update_payment_setup( $payment_id ) {
+	public function update_payment_setup($payment_id) {
 		// Delete cache.
-		Give_Cache::delete_group( $this->ID,'give-donations' );
+		Give_Cache::delete_group($this->ID, 'give-donations');
 
-		$this->setup_payment( $payment_id );
+		$this->setup_payment($payment_id);
 	}
 
 	/**
@@ -635,24 +635,24 @@ 
 
 		// Construct the payment title.
 		$payment_title = '';
-		if ( ! empty( $this->first_name ) && ! empty( $this->last_name ) ) {
-			$payment_title = $this->first_name . ' ' . $this->last_name;
-		} elseif ( ! empty( $this->first_name ) && empty( $this->last_name ) ) {
+		if ( ! empty($this->first_name) && ! empty($this->last_name)) {
+			$payment_title = $this->first_name.' '.$this->last_name;
+		} elseif ( ! empty($this->first_name) && empty($this->last_name)) {
 			$payment_title = $this->first_name;
-		} elseif ( ! empty( $this->email ) && is_email( $this->email ) ) {
+		} elseif ( ! empty($this->email) && is_email($this->email)) {
 			$payment_title = $this->email;
 		}
 
 		// Set Key.
-		if ( empty( $this->key ) ) {
+		if (empty($this->key)) {
 
-			$auth_key             = defined( 'AUTH_KEY' ) ? AUTH_KEY : '';
-			$this->key            = strtolower( md5( $this->email . date( 'Y-m-d H:i:s' ) . $auth_key . uniqid( 'give', true ) ) );  // Unique key.
+			$auth_key             = defined('AUTH_KEY') ? AUTH_KEY : '';
+			$this->key            = strtolower(md5($this->email.date('Y-m-d H:i:s').$auth_key.uniqid('give', true))); // Unique key.
 			$this->pending['key'] = $this->key;
 		}
 
 		// Set IP.
-		if ( empty( $this->ip ) ) {
+		if (empty($this->ip)) {
 
 			$this->ip            = give_get_ip();
 			$this->pending['ip'] = $this->ip;
@@ -682,19 +682,19 @@ 
 			'status'       => $this->status,
 		);
 
-		$args = apply_filters( 'give_insert_payment_args', array(
+		$args = apply_filters('give_insert_payment_args', array(
 			'post_title'    => $payment_title,
 			'post_status'   => $this->status,
 			'post_type'     => 'give_payment',
-			'post_date'     => ! empty( $this->date ) ? $this->date : null,
-			'post_date_gmt' => ! empty( $this->date ) ? get_gmt_from_date( $this->date ) : null,
+			'post_date'     => ! empty($this->date) ? $this->date : null,
+			'post_date_gmt' => ! empty($this->date) ? get_gmt_from_date($this->date) : null,
 			'post_parent'   => $this->parent_payment,
-		), $payment_data );
+		), $payment_data);
 
 		// Create a blank payment.
-		$payment_id = wp_insert_post( $args );
+		$payment_id = wp_insert_post($args);
 
-		if ( ! empty( $payment_id ) ) {
+		if ( ! empty($payment_id)) {
 
 			$this->ID  = $payment_id;
 			$this->_ID = $payment_id;
@@ -706,43 +706,43 @@ 
 			 *
 			 * @since 1.8.13
 			 */
-			$donor = apply_filters( 'give_update_donor_information', $donor, $payment_id, $payment_data, $args );
+			$donor = apply_filters('give_update_donor_information', $donor, $payment_id, $payment_data, $args);
 
-			if ( did_action( 'give_pre_process_donation' ) && is_user_logged_in() ) {
-				$donor = new Give_Donor( get_current_user_id(), true );
+			if (did_action('give_pre_process_donation') && is_user_logged_in()) {
+				$donor = new Give_Donor(get_current_user_id(), true);
 
 				// Donor is logged in but used a different email to purchase with so assign to their donor record.
-				if ( ! empty( $donor->id ) && $this->email !== $donor->email ) {
-					$donor->add_email( $this->email );
+				if ( ! empty($donor->id) && $this->email !== $donor->email) {
+					$donor->add_email($this->email);
 				}
 			}
 
-			if ( empty( $donor->id ) ) {
-				$donor = new Give_Donor( $this->email );
+			if (empty($donor->id)) {
+				$donor = new Give_Donor($this->email);
 			}
 
-			if ( empty( $donor->id ) ) {
+			if (empty($donor->id)) {
 
 				$donor_data = array(
-					'name'    => ! is_email( $payment_title ) ? $this->first_name . ' ' . $this->last_name : '',
+					'name'    => ! is_email($payment_title) ? $this->first_name.' '.$this->last_name : '',
 					'email'   => $this->email,
 					'user_id' => $this->user_id,
 				);
 
-				$donor->create( $donor_data );
+				$donor->create($donor_data);
 
 			}
 
 			// Update Donor Meta once donor is created.
-			$donor->update_meta( '_give_donor_first_name', $this->first_name );
-			$donor->update_meta( '_give_donor_last_name', $this->last_name );
-			$donor->update_meta( '_give_donor_title_prefix', $this->title_prefix );
+			$donor->update_meta('_give_donor_first_name', $this->first_name);
+			$donor->update_meta('_give_donor_last_name', $this->last_name);
+			$donor->update_meta('_give_donor_title_prefix', $this->title_prefix);
 
 			$this->customer_id            = $donor->id;
 			$this->pending['customer_id'] = $this->customer_id;
-			$donor->attach_payment( $this->ID, false );
+			$donor->attach_payment($this->ID, false);
 
-			$this->payment_meta = apply_filters( 'give_payment_meta', $this->payment_meta, $payment_data );
+			$this->payment_meta = apply_filters('give_payment_meta', $this->payment_meta, $payment_data);
 
 			/*
 			 * _give_payment_meta backward compatibility.
@@ -750,25 +750,25 @@ 
 			 * @since 2.0.1
 			 */
 			$custom_payment_meta = array_diff(
-				array_map( 'maybe_serialize', $this->payment_meta ),
-				array_map( 'maybe_serialize', $payment_data )
+				array_map('maybe_serialize', $this->payment_meta),
+				array_map('maybe_serialize', $payment_data)
 			);
 
-			if ( ! empty( $custom_payment_meta ) ) {
-				give_doing_it_wrong( '_give_payment_meta', __( 'This custom meta key deprecated. We are not using this meta key for storing payment meta but your custom meta data will be store because we added backward compatibility. Please change your logic because in future we can remove it.', 'give' ), '2.0.0' );
+			if ( ! empty($custom_payment_meta)) {
+				give_doing_it_wrong('_give_payment_meta', __('This custom meta key deprecated. We are not using this meta key for storing payment meta but your custom meta data will be store because we added backward compatibility. Please change your logic because in future we can remove it.', 'give'), '2.0.0');
 
-				$this->update_meta( '_give_payment_meta', array_map( 'maybe_unserialize', $custom_payment_meta ) );
+				$this->update_meta('_give_payment_meta', array_map('maybe_unserialize', $custom_payment_meta));
 			}
 
-			$give_company = ( ! empty( $_REQUEST['give_company_name'] ) ? give_clean( $_REQUEST['give_company_name'] ) : '' );
+			$give_company = ( ! empty($_REQUEST['give_company_name']) ? give_clean($_REQUEST['give_company_name']) : '');
 
 			// Check $page_url is not empty.
-			if ( $give_company ) {
-				give_update_meta( $payment_id, '_give_donation_company', $give_company );
+			if ($give_company) {
+				give_update_meta($payment_id, '_give_donation_company', $give_company);
 
-				$donor_id = absint( $donor->id );
-				if ( ! empty( $donor_id ) ) {
-					Give()->donor_meta->update_meta( $donor_id, '_give_donor_company', $give_company );
+				$donor_id = absint($donor->id);
+				if ( ! empty($donor_id)) {
+					Give()->donor_meta->update_meta($donor_id, '_give_donor_company', $give_company);
 				}
 			}
 
@@ -792,11 +792,11 @@ 
 		$saved = false;
 
 		// Must have an ID.
-		if ( empty( $this->ID ) ) {
+		if (empty($this->ID)) {
 
 			$payment_id = $this->insert_payment();
 
-			if ( false === $payment_id ) {
+			if (false === $payment_id) {
 				$saved = false;
 			} else {
 				$this->ID = $payment_id;
@@ -804,42 +804,42 @@ 
 		}
 
 		// Set ID if not matching.
-		if ( $this->ID !== $this->_ID ) {
+		if ($this->ID !== $this->_ID) {
 			$this->ID = $this->_ID;
 		}
 
 		// If we have something pending, let's save it.
-		if ( ! empty( $this->pending ) ) {
+		if ( ! empty($this->pending)) {
 
 			$total_increase = 0;
 			$total_decrease = 0;
 
-			foreach ( $this->pending as $key => $value ) {
+			foreach ($this->pending as $key => $value) {
 
-				switch ( $key ) {
+				switch ($key) {
 
 					case 'donations':
 						// Update totals for pending donations.
-						foreach ( $this->pending[ $key ] as $item ) {
+						foreach ($this->pending[$key] as $item) {
 
-							$quantity = isset( $item['quantity'] ) ? $item['quantity'] : 1;
-							$price_id = isset( $item['price_id'] ) ? $item['price_id'] : 0;
+							$quantity = isset($item['quantity']) ? $item['quantity'] : 1;
+							$price_id = isset($item['price_id']) ? $item['price_id'] : 0;
 
-							switch ( $item['action'] ) {
+							switch ($item['action']) {
 
 								case 'add':
 
 									$price = $item['price'];
 
-									if ( 'publish' === $this->status || 'complete' === $this->status ) {
+									if ('publish' === $this->status || 'complete' === $this->status) {
 
 										// Add donation to logs.
-										$log_date = date_i18n( 'Y-m-d G:i:s', current_time( 'timestamp' ) );
-										give_record_donation_in_log( $item['id'], $this->ID, $price_id, $log_date );
+										$log_date = date_i18n('Y-m-d G:i:s', current_time('timestamp'));
+										give_record_donation_in_log($item['id'], $this->ID, $price_id, $log_date);
 
-										$form = new Give_Donate_Form( $item['id'] );
-										$form->increase_sales( $quantity );
-										$form->increase_earnings( $price, $this->ID );
+										$form = new Give_Donate_Form($item['id']);
+										$form->increase_sales($quantity);
+										$form->increase_earnings($price, $this->ID);
 
 										$total_increase += $price;
 									}
@@ -847,10 +847,10 @@ 
 
 								case 'remove':
 									$this->delete_sales_logs();
-									if ( 'publish' === $this->status || 'complete' === $this->status ) {
-										$form = new Give_Donate_Form( $item['id'] );
-										$form->decrease_sales( $quantity );
-										$form->decrease_earnings( $item['amount'], $this->ID );
+									if ('publish' === $this->status || 'complete' === $this->status) {
+										$form = new Give_Donate_Form($item['id']);
+										$form->decrease_sales($quantity);
+										$form->decrease_earnings($item['amount'], $this->ID);
 
 										$total_decrease += $item['amount'];
 									}
@@ -861,89 +861,89 @@ 
 						break;
 
 					case 'status':
-						$this->update_status( $this->status );
+						$this->update_status($this->status);
 						break;
 
 					case 'gateway':
-						$this->update_meta( '_give_payment_gateway', $this->gateway );
+						$this->update_meta('_give_payment_gateway', $this->gateway);
 						break;
 
 					case 'mode':
-						$this->update_meta( '_give_payment_mode', $this->mode );
+						$this->update_meta('_give_payment_mode', $this->mode);
 						break;
 
 					case 'transaction_id':
-						$this->update_meta( '_give_payment_transaction_id', $this->transaction_id );
+						$this->update_meta('_give_payment_transaction_id', $this->transaction_id);
 						break;
 
 					case 'ip':
-						$this->update_meta( '_give_payment_donor_ip', $this->ip );
+						$this->update_meta('_give_payment_donor_ip', $this->ip);
 						break;
 
 					case 'customer_id':
-						$this->update_meta( '_give_payment_donor_id', $this->customer_id );
+						$this->update_meta('_give_payment_donor_id', $this->customer_id);
 						break;
 
 					case 'form_title':
-						$this->update_meta( '_give_payment_form_title', $this->form_title );
+						$this->update_meta('_give_payment_form_title', $this->form_title);
 						break;
 
 					case 'form_id':
-						$this->update_meta( '_give_payment_form_id', $this->form_id );
+						$this->update_meta('_give_payment_form_id', $this->form_id);
 						break;
 
 					case 'price_id':
-						$this->update_meta( '_give_payment_price_id', $this->price_id );
+						$this->update_meta('_give_payment_price_id', $this->price_id);
 						break;
 
 					case 'first_name':
-						$this->update_meta( '_give_donor_billing_first_name', $this->first_name );
+						$this->update_meta('_give_donor_billing_first_name', $this->first_name);
 						break;
 
 					case 'last_name':
-						$this->update_meta( '_give_donor_billing_last_name', $this->last_name );
+						$this->update_meta('_give_donor_billing_last_name', $this->last_name);
 						break;
 
 					case 'currency':
-						$this->update_meta( '_give_payment_currency', $this->currency );
+						$this->update_meta('_give_payment_currency', $this->currency);
 						break;
 
 					case 'address':
-						if ( ! empty( $this->address ) ) {
-							foreach ( $this->address as $address_name => $address ) {
-								switch ( $address_name ) {
+						if ( ! empty($this->address)) {
+							foreach ($this->address as $address_name => $address) {
+								switch ($address_name) {
 									case 'line1':
-										$this->update_meta( '_give_donor_billing_address1', $address );
+										$this->update_meta('_give_donor_billing_address1', $address);
 										break;
 
 									case 'line2':
-										$this->update_meta( '_give_donor_billing_address2', $address );
+										$this->update_meta('_give_donor_billing_address2', $address);
 										break;
 
 									default:
-										$this->update_meta( "_give_donor_billing_{$address_name}", $address );
+										$this->update_meta("_give_donor_billing_{$address_name}", $address);
 								}
 							}
 						}
 						break;
 
 					case 'email':
-						$this->update_meta( '_give_payment_donor_email', $this->email );
+						$this->update_meta('_give_payment_donor_email', $this->email);
 						break;
 
 					case 'title_prefix':
-						$this->update_meta( '_give_payment_donor_title_prefix', $this->title_prefix );
+						$this->update_meta('_give_payment_donor_title_prefix', $this->title_prefix);
 						break;
 
 					case 'key':
-						$this->update_meta( '_give_payment_purchase_key', $this->key );
+						$this->update_meta('_give_payment_purchase_key', $this->key);
 						break;
 
 					case 'number':
 						// @todo: remove unused meta data.
 						// Core is using post_title to store donation serial code ( fi enabled ) instead this meta key.
 						// Do not use this meta key in your logic, can be remove in future
-						$this->update_meta( '_give_payment_number', $this->number );
+						$this->update_meta('_give_payment_number', $this->number);
 						break;
 
 					case 'date':
@@ -953,11 +953,11 @@ 
 							'edit_date' => true,
 						);
 
-						wp_update_post( $args );
+						wp_update_post($args);
 						break;
 
 					case 'completed_date':
-						$this->update_meta( '_give_completed_date', $this->completed_date );
+						$this->update_meta('_give_completed_date', $this->completed_date);
 						break;
 
 					case 'parent_payment':
@@ -966,11 +966,11 @@ 
 							'post_parent' => $this->parent_payment,
 						);
 
-						wp_update_post( $args );
+						wp_update_post($args);
 						break;
 
 					case 'total':
-						$this->update_meta( '_give_payment_total', give_sanitize_amount_for_db( $this->total ) );
+						$this->update_meta('_give_payment_total', give_sanitize_amount_for_db($this->total));
 						break;
 
 					default:
@@ -981,46 +981,46 @@ 
 						 *
 						 * @param Give_Payment $this Payment object.
 						 */
-						do_action( 'give_payment_save', $this, $key );
+						do_action('give_payment_save', $this, $key);
 						break;
 				} // End switch().
 			} // End foreach().
 
-			if ( 'pending' !== $this->status ) {
+			if ('pending' !== $this->status) {
 
-				$donor = new Give_Donor( $this->customer_id );
+				$donor = new Give_Donor($this->customer_id);
 
 				$total_change = $total_increase - $total_decrease;
-				if ( $total_change < 0 ) {
+				if ($total_change < 0) {
 
-					$total_change = - ( $total_change );
+					$total_change = - ($total_change);
 
 					// Decrease the donor's donation stats.
-					$donor->decrease_value( $total_change );
-					give_decrease_total_earnings( $total_change );
+					$donor->decrease_value($total_change);
+					give_decrease_total_earnings($total_change);
 
 					$donor->decrease_donation_count();
 
-				} elseif ( $total_change > 0 ) {
+				} elseif ($total_change > 0) {
 
 					// Increase the donor's donation stats.
-					$donor->increase_value( $total_change );
-					give_increase_total_earnings( $total_change );
+					$donor->increase_value($total_change);
+					give_increase_total_earnings($total_change);
 
 					$donor->increase_purchase_count();
 
 				}
 
 				// Verify and update form meta based on the form status.
-				give_set_form_closed_status( $this->form_id );
+				give_set_form_closed_status($this->form_id);
 			}
 
 			$this->pending = array();
 			$saved         = true;
 		} // End if().
 
-		if ( true === $saved ) {
-			$this->setup_payment( $this->ID );
+		if (true === $saved) {
+			$this->setup_payment($this->ID);
 		}
 
 		return $saved;
@@ -1038,12 +1038,12 @@ 
 	 *
 	 * @return bool           True when successful, false otherwise
 	 */
-	public function add_donation( $form_id = 0, $args = array(), $options = array() ) {
+	public function add_donation($form_id = 0, $args = array(), $options = array()) {
 
-		$donation = new Give_Donate_Form( $form_id );
+		$donation = new Give_Donate_Form($form_id);
 
 		// Bail if this post isn't a give donation form.
-		if ( ! $donation || 'give_forms' !== $donation->post_type ) {
+		if ( ! $donation || 'give_forms' !== $donation->post_type) {
 			return false;
 		}
 
@@ -1053,25 +1053,25 @@ 
 			'price_id' => false,
 		);
 
-		$args = wp_parse_args( apply_filters( 'give_payment_add_donation_args', $args, $donation->ID ), $defaults );
+		$args = wp_parse_args(apply_filters('give_payment_add_donation_args', $args, $donation->ID), $defaults);
 
 		// Allow overriding the price.
-		if ( false !== $args['price'] ) {
+		if (false !== $args['price']) {
 			$donation_amount = $args['price'];
 		} else {
 
 			// Deal with variable pricing.
-			if ( give_has_variable_prices( $donation->ID ) ) {
-				$prices          = give_get_meta( $form_id, '_give_donation_levels', true );
+			if (give_has_variable_prices($donation->ID)) {
+				$prices          = give_get_meta($form_id, '_give_donation_levels', true);
 				$donation_amount = '';
 
 				// Loop through prices.
-				foreach ( $prices as $price ) {
+				foreach ($prices as $price) {
 					// Find a match between price_id and level_id.
 					// First verify array keys exists THEN make the match.
 					if (
-						isset( $args['price_id'] ) &&
-						isset( $price['_give_id']['level_id'] ) &&
+						isset($args['price_id']) &&
+						isset($price['_give_id']['level_id']) &&
 						$args['price_id'] === (int) $price['_give_id']['level_id']
 					) {
 						$donation_amount = $price['_give_amount'];
@@ -1079,37 +1079,37 @@ 
 				}
 
 				// Fallback to the lowest price point.
-				if ( '' === $donation_amount ) {
-					$donation_amount  = give_get_lowest_price_option( $donation->ID );
-					$args['price_id'] = give_get_lowest_price_id( $donation->ID );
+				if ('' === $donation_amount) {
+					$donation_amount  = give_get_lowest_price_option($donation->ID);
+					$args['price_id'] = give_get_lowest_price_id($donation->ID);
 				}
 			} else {
 				// Simple form price.
-				$donation_amount = give_get_form_price( $donation->ID );
+				$donation_amount = give_get_form_price($donation->ID);
 			}
 		}
 
 		// Sanitizing the price here so we don't have a dozen calls later.
-		$donation_amount = give_maybe_sanitize_amount( $donation_amount );
-		$total           = round( $donation_amount, give_get_price_decimals( $this->ID ) );
+		$donation_amount = give_maybe_sanitize_amount($donation_amount);
+		$total           = round($donation_amount, give_get_price_decimals($this->ID));
 
 		// Add Options.
 		$default_options = array();
-		if ( false !== $args['price_id'] ) {
+		if (false !== $args['price_id']) {
 			$default_options['price_id'] = (int) $args['price_id'];
 		}
-		$options = wp_parse_args( $options, $default_options );
+		$options = wp_parse_args($options, $default_options);
 
 		// Do not allow totals to go negative.
-		if ( $total < 0 ) {
+		if ($total < 0) {
 			$total = 0;
 		}
 
 		$donation = array(
 			'name'     => $donation->post_title,
 			'id'       => $donation->ID,
-			'price'    => round( $total, give_get_price_decimals( $this->ID ) ),
-			'subtotal' => round( $total, give_get_price_decimals( $this->ID ) ),
+			'price'    => round($total, give_get_price_decimals($this->ID)),
+			'subtotal' => round($total, give_get_price_decimals($this->ID)),
 			'price_id' => $args['price_id'],
 			'action'   => 'add',
 			'options'  => $options,
@@ -1117,7 +1117,7 @@ 
 
 		$this->pending['donations'][] = $donation;
 
-		$this->increase_subtotal( $total );
+		$this->increase_subtotal($total);
 
 		return true;
 
@@ -1134,7 +1134,7 @@ 
 	 *
 	 * @return bool           If the item was removed or not
 	 */
-	public function remove_donation( $form_id, $args = array() ) {
+	public function remove_donation($form_id, $args = array()) {
 
 		// Set some defaults.
 		$defaults = array(
@@ -1142,12 +1142,12 @@ 
 			'price'    => false,
 			'price_id' => false,
 		);
-		$args     = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		$form = new Give_Donate_Form( $form_id );
+		$form = new Give_Donate_Form($form_id);
 
 		// Bail if this post isn't a valid give donation form.
-		if ( ! $form || 'give_forms' !== $form->post_type ) {
+		if ( ! $form || 'give_forms' !== $form->post_type) {
 			return false;
 		}
 
@@ -1160,7 +1160,7 @@ 
 
 		$this->pending['donations'][] = $pending_args;
 
-		$this->decrease_subtotal( $this->total );
+		$this->decrease_subtotal($this->total);
 
 		return true;
 	}
@@ -1176,13 +1176,13 @@ 
 	 *
 	 * @return bool           If the note was specified or not
 	 */
-	public function add_note( $note = false ) {
+	public function add_note($note = false) {
 		// Bail if no note specified.
-		if ( ! $note ) {
+		if ( ! $note) {
 			return false;
 		}
 
-		give_insert_payment_note( $this->ID, $note );
+		give_insert_payment_note($this->ID, $note);
 	}
 
 	/**
@@ -1195,8 +1195,8 @@ 
 	 *
 	 * @return void
 	 */
-	private function increase_subtotal( $amount = 0.00 ) {
-		$amount         = (float) $amount;
+	private function increase_subtotal($amount = 0.00) {
+		$amount = (float) $amount;
 		$this->subtotal += $amount;
 
 		$this->recalculate_total();
@@ -1212,11 +1212,11 @@ 
 	 *
 	 * @return void
 	 */
-	private function decrease_subtotal( $amount = 0.00 ) {
-		$amount         = (float) $amount;
+	private function decrease_subtotal($amount = 0.00) {
+		$amount = (float) $amount;
 		$this->subtotal -= $amount;
 
-		if ( $this->subtotal < 0 ) {
+		if ($this->subtotal < 0) {
 			$this->subtotal = 0;
 		}
 
@@ -1246,24 +1246,24 @@ 
 	 *
 	 * @return bool   $updated Returns if the status was successfully updated.
 	 */
-	public function update_status( $status = false ) {
+	public function update_status($status = false) {
 
 		// standardize the 'complete(d)' status.
-		if ( 'completed' === $status || 'complete' === $status ) {
+		if ('completed' === $status || 'complete' === $status) {
 			$status = 'publish';
 		}
 
-		$old_status = ! empty( $this->old_status ) ? $this->old_status : false;
+		$old_status = ! empty($this->old_status) ? $this->old_status : false;
 
-		if ( $old_status === $status ) {
+		if ($old_status === $status) {
 			return false; // Don't permit status changes that aren't changes.
 		}
 
-		$do_change = apply_filters( 'give_should_update_payment_status', true, $this->ID, $status, $old_status );
+		$do_change = apply_filters('give_should_update_payment_status', true, $this->ID, $status, $old_status);
 
 		$updated = false;
 
-		if ( $do_change ) {
+		if ($do_change) {
 
 			/**
 			 * Fires before changing payment status.
@@ -1274,21 +1274,21 @@ 
 			 * @param string $status     The new status.
 			 * @param string $old_status The old status.
 			 */
-			do_action( 'give_before_payment_status_change', $this->ID, $status, $old_status );
+			do_action('give_before_payment_status_change', $this->ID, $status, $old_status);
 
 			$update_fields = array(
 				'ID'          => $this->ID,
 				'post_status' => $status,
-				'edit_date'   => current_time( 'mysql' ),
+				'edit_date'   => current_time('mysql'),
 			);
 
-			$updated = wp_update_post( apply_filters( 'give_update_payment_status_fields', $update_fields ) );
+			$updated = wp_update_post(apply_filters('give_update_payment_status_fields', $update_fields));
 
 			$all_payment_statuses  = give_get_payment_statuses();
-			$this->status_nicename = array_key_exists( $status, $all_payment_statuses ) ? $all_payment_statuses[ $status ] : ucfirst( $status );
+			$this->status_nicename = array_key_exists($status, $all_payment_statuses) ? $all_payment_statuses[$status] : ucfirst($status);
 
 			// Process any specific status functions.
-			$this->process_status( $status );
+			$this->process_status($status);
 
 			/**
 			 * Fires after changing payment status.
@@ -1299,7 +1299,7 @@ 
 			 * @param string $status     The new status.
 			 * @param string $old_status The old status.
 			 */
-			do_action( 'give_update_payment_status', $this->ID, $status, $old_status );
+			do_action('give_update_payment_status', $this->ID, $status, $old_status);
 
 		} // End if().
 
@@ -1334,27 +1334,27 @@ 
 	 *
 	 * @return mixed             The value from the post meta
 	 */
-	public function get_meta( $meta_key = '_give_payment_meta', $single = true ) {
+	public function get_meta($meta_key = '_give_payment_meta', $single = true) {
 		if (
-			! has_filter( 'get_post_metadata', 'give_bc_v20_get_payment_meta' ) &&
-			! doing_filter( 'get_post_metadata' )
+			! has_filter('get_post_metadata', 'give_bc_v20_get_payment_meta') &&
+			! doing_filter('get_post_metadata')
 		) {
-			add_filter( 'get_post_metadata', 'give_bc_v20_get_payment_meta', 999, 4 );
+			add_filter('get_post_metadata', 'give_bc_v20_get_payment_meta', 999, 4);
 		}
 
-		$meta = give_get_meta( $this->ID, $meta_key, $single );
+		$meta = give_get_meta($this->ID, $meta_key, $single);
 
 		/**
 		 * Filter the specific meta key value.
 		 *
 		 * @since 1.5
 		 */
-		$meta = apply_filters( "give_get_payment_meta_{$meta_key}", $meta, $this->ID );
+		$meta = apply_filters("give_get_payment_meta_{$meta_key}", $meta, $this->ID);
 
 		// Security check.
-		if ( is_serialized( $meta ) ) {
-			preg_match( '/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $meta, $matches );
-			if ( ! empty( $matches ) ) {
+		if (is_serialized($meta)) {
+			preg_match('/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $meta, $matches);
+			if ( ! empty($matches)) {
 				$meta = array();
 			}
 		}
@@ -1364,7 +1364,7 @@ 
 		 *
 		 * @since 1.5
 		 */
-		return apply_filters( 'give_get_payment_meta', $meta, $this->ID, $meta_key );
+		return apply_filters('give_get_payment_meta', $meta, $this->ID, $meta_key);
 	}
 
 	/**
@@ -1379,8 +1379,8 @@ 
 	 *
 	 * @return int|bool           Meta ID if the key didn't exist, true on successful update, false on failure
 	 */
-	public function update_meta( $meta_key = '', $meta_value = '', $prev_value = '' ) {
-		if ( empty( $meta_key ) ) {
+	public function update_meta($meta_key = '', $meta_value = '', $prev_value = '') {
+		if (empty($meta_key)) {
 			return false;
 		}
 
@@ -1390,9 +1390,9 @@ 
 		 *
 		 * @since 1.5
 		 */
-		$meta_value = apply_filters( "give_update_payment_meta_{$meta_key}", $meta_value, $this->ID );
+		$meta_value = apply_filters("give_update_payment_meta_{$meta_key}", $meta_value, $this->ID);
 
-		return give_update_meta( $this->ID, $meta_key, $meta_value, $prev_value );
+		return give_update_meta($this->ID, $meta_key, $meta_value, $prev_value);
 	}
 
 	/**
@@ -1405,7 +1405,7 @@ 
 	 *
 	 * @return void
 	 */
-	private function process_status( $status ) {
+	private function process_status($status) {
 		$process = true;
 
 		// Bailout, if changed from completed to preapproval/processing.
@@ -1420,9 +1420,9 @@ 
 		}
 
 		// Allow extensions to filter for their own payment types, Example: Recurring Payments.
-		$process = apply_filters( "give_should_process_{$status}", $process, $this );
+		$process = apply_filters("give_should_process_{$status}", $process, $this);
 
-		if ( false === $process ) {
+		if (false === $process) {
 			return;
 		}
 
@@ -1433,13 +1433,13 @@ 
 		 *
 		 * @since 1.5
 		 */
-		do_action( "give_pre_{$status}_payment", $this );
+		do_action("give_pre_{$status}_payment", $this);
 
-		$decrease_earnings       = apply_filters( "give_decrease_earnings_on_{$status}", true, $this );
-		$decrease_donor_value    = apply_filters( "give_decrease_donor_value_on_{$status}", true, $this );
-		$decrease_donation_count = apply_filters( "give_decrease_donors_donation_count_on_{$status}", true, $this );
+		$decrease_earnings       = apply_filters("give_decrease_earnings_on_{$status}", true, $this);
+		$decrease_donor_value    = apply_filters("give_decrease_donor_value_on_{$status}", true, $this);
+		$decrease_donation_count = apply_filters("give_decrease_donors_donation_count_on_{$status}", true, $this);
 
-		$this->maybe_alter_stats( $decrease_earnings, $decrease_donor_value, $decrease_donation_count );
+		$this->maybe_alter_stats($decrease_earnings, $decrease_donor_value, $decrease_donation_count);
 		$this->delete_sales_logs();
 
 		// @todo: Refresh only range related stat cache
@@ -1452,7 +1452,7 @@ 
 		 *
 		 * @since 1.5
 		 */
-		do_action( "give_post_{$status}_payment", $this );
+		do_action("give_post_{$status}_payment", $this);
 	}
 
 	/**
@@ -1467,25 +1467,25 @@ 
 	 *
 	 * @return void
 	 */
-	private function maybe_alter_stats( $alter_store_earnings, $alter_customer_value, $alter_customer_purchase_count ) {
+	private function maybe_alter_stats($alter_store_earnings, $alter_customer_value, $alter_customer_purchase_count) {
 
-		give_undo_donation( $this->ID );
+		give_undo_donation($this->ID);
 
 		// Decrease store earnings.
-		if ( true === $alter_store_earnings ) {
-			give_decrease_total_earnings( $this->total );
+		if (true === $alter_store_earnings) {
+			give_decrease_total_earnings($this->total);
 		}
 
 		// Decrement the stats for the donor.
-		if ( ! empty( $this->customer_id ) ) {
+		if ( ! empty($this->customer_id)) {
 
-			$donor = new Give_Donor( $this->customer_id );
+			$donor = new Give_Donor($this->customer_id);
 
-			if ( true === $alter_customer_value ) {
-				$donor->decrease_value( $this->total );
+			if (true === $alter_customer_value) {
+				$donor->decrease_value($this->total);
 			}
 
-			if ( true === $alter_customer_purchase_count ) {
+			if (true === $alter_customer_purchase_count) {
 				$donor->decrease_donation_count();
 			}
 		}
@@ -1502,7 +1502,7 @@ 
 	 */
 	private function delete_sales_logs() {
 		// Remove related sale log entries.
-		Give()->logs->delete_logs( $this->ID );
+		Give()->logs->delete_logs($this->ID);
 	}
 
 	/**
@@ -1523,13 +1523,13 @@ 
 	 * @return string The date the payment was completed
 	 */
 	private function setup_completed_date() {
-		$payment = get_post( $this->ID );
+		$payment = get_post($this->ID);
 
-		if ( 'pending' === $payment->post_status || 'preapproved' === $payment->post_status ) {
+		if ('pending' === $payment->post_status || 'preapproved' === $payment->post_status) {
 			return false; // This payment was never completed.
 		}
 
-		$date = ( $date = $this->get_meta( '_give_completed_date', true ) ) ? $date : $payment->modified_date;
+		$date = ($date = $this->get_meta('_give_completed_date', true)) ? $date : $payment->modified_date;
 
 		return $date;
 	}
@@ -1543,7 +1543,7 @@ 
 	 * @return string The payment mode
 	 */
 	private function setup_mode() {
-		return $this->get_meta( '_give_payment_mode' );
+		return $this->get_meta('_give_payment_mode');
 	}
 
 	/**
@@ -1555,7 +1555,7 @@ 
 	 * @return bool The payment import
 	 */
 	private function setup_import() {
-		return (bool) $this->get_meta( '_give_payment_import' );
+		return (bool) $this->get_meta('_give_payment_import');
 	}
 
 	/**
@@ -1567,9 +1567,9 @@ 
 	 * @return float The payment total
 	 */
 	private function setup_total() {
-		$amount = $this->get_meta( '_give_payment_total', true );
+		$amount = $this->get_meta('_give_payment_total', true);
 
-		return round( floatval( $amount ), give_get_price_decimals( $this->ID ) );
+		return round(floatval($amount), give_get_price_decimals($this->ID));
 	}
 
 	/**
@@ -1596,17 +1596,16 @@ 
 	 * @return string The currency for the payment
 	 */
 	private function setup_currency() {
-		$currency = $this->get_meta( '_give_payment_currency', true );
-		$currency = ! empty( $currency ) ?
-			$currency :
-			/**
+		$currency = $this->get_meta('_give_payment_currency', true);
+		$currency = ! empty($currency) ?
+			$currency : /**
 			 * Filter the default donation currency
 			 *
 			 * @since 1.5
 			 */
 			apply_filters(
 				'give_payment_currency_default',
-				give_get_currency( $this->form_id, $this ),
+				give_get_currency($this->form_id, $this),
 				$this
 			);
 
@@ -1622,7 +1621,7 @@ 
 	 * @return string The gateway
 	 */
 	private function setup_gateway() {
-		$gateway = $this->get_meta( '_give_payment_gateway', true );
+		$gateway = $this->get_meta('_give_payment_gateway', true);
 
 		return $gateway;
 	}
@@ -1636,11 +1635,11 @@ 
 	 * @return string The donation ID
 	 */
 	private function setup_transaction_id() {
-		$transaction_id = $this->get_meta( '_give_payment_transaction_id', true );
+		$transaction_id = $this->get_meta('_give_payment_transaction_id', true);
 
-		if ( empty( $transaction_id ) ) {
+		if (empty($transaction_id)) {
 			$gateway        = $this->gateway;
-			$transaction_id = apply_filters( "give_get_payment_transaction_id-{$gateway}", $this->ID );
+			$transaction_id = apply_filters("give_get_payment_transaction_id-{$gateway}", $this->ID);
 		}
 
 		return $transaction_id;
@@ -1656,7 +1655,7 @@ 
 	 * @return string The IP address for the payment
 	 */
 	private function setup_ip() {
-		$ip = $this->get_meta( '_give_payment_donor_ip', true );
+		$ip = $this->get_meta('_give_payment_donor_ip', true);
 
 		return $ip;
 	}
@@ -1671,7 +1670,7 @@ 
 	 * @return int The Donor ID.
 	 */
 	private function setup_donor_id() {
-		$donor_id = $this->get_meta( '_give_payment_donor_id', true );
+		$donor_id = $this->get_meta('_give_payment_donor_id', true);
 
 		return $donor_id;
 	}
@@ -1688,8 +1687,8 @@ 
 	 */
 	private function setup_user_id() {
 
-		$donor   = Give()->customers->get_customer_by( 'id', $this->customer_id );
-		$user_id = $donor ? absint( $donor->user_id ) : 0;
+		$donor   = Give()->customers->get_customer_by('id', $this->customer_id);
+		$user_id = $donor ? absint($donor->user_id) : 0;
 
 
 		return $user_id;
@@ -1706,10 +1705,10 @@ 
 	 * @return string The email address for the payment.
 	 */
 	private function setup_email() {
-		$email = $this->get_meta( '_give_payment_donor_email', true );
+		$email = $this->get_meta('_give_payment_donor_email', true);
 
-		if ( empty( $email ) && $this->customer_id ) {
-			$email = Give()->donors->get_column( 'email', $this->customer_id );
+		if (empty($email) && $this->customer_id) {
+			$email = Give()->donors->get_column('email', $this->customer_id);
 		}
 
 		return $email;
@@ -1730,22 +1729,22 @@ 
 			'last_name'  => $this->last_name,
 		);
 
-		$user_info = isset( $this->payment_meta['user_info'] ) ? $this->payment_meta['user_info'] : array();
+		$user_info = isset($this->payment_meta['user_info']) ? $this->payment_meta['user_info'] : array();
 
-		if ( is_serialized( $user_info ) ) {
-			preg_match( '/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $user_info, $matches );
-			if ( ! empty( $matches ) ) {
+		if (is_serialized($user_info)) {
+			preg_match('/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $user_info, $matches);
+			if ( ! empty($matches)) {
 				$user_info = array();
 			}
 		}
 
-		$user_info = wp_parse_args( $user_info, $defaults );
+		$user_info = wp_parse_args($user_info, $defaults);
 
-		if ( empty( $user_info ) ) {
+		if (empty($user_info)) {
 			// Get the donor, but only if it's been created.
-			$donor = new Give_Donor( $this->customer_id );
+			$donor = new Give_Donor($this->customer_id);
 
-			if ( $donor->id > 0 ) {
+			if ($donor->id > 0) {
 				$user_info = array(
 					'first_name' => $donor->get_first_name(),
 					'last_name'  => $donor->get_last_name(),
@@ -1755,29 +1754,29 @@ 
 			}
 		} else {
 			// Get the donor, but only if it's been created.
-			$donor = new Give_Donor( $this->customer_id );
+			$donor = new Give_Donor($this->customer_id);
 
-			if ( $donor->id > 0 ) {
-				foreach ( $user_info as $key => $value ) {
-					if ( ! empty( $value ) ) {
+			if ($donor->id > 0) {
+				foreach ($user_info as $key => $value) {
+					if ( ! empty($value)) {
 						continue;
 					}
 
-					switch ( $key ) {
+					switch ($key) {
 						case 'title':
-							$user_info[ $key ] = Give()->donor_meta->get_meta( $donor->id, '_give_donor_title_prefix', true );
+							$user_info[$key] = Give()->donor_meta->get_meta($donor->id, '_give_donor_title_prefix', true);
 							break;
 
 						case 'first_name':
-							$user_info[ $key ] = $donor->get_first_name();
+							$user_info[$key] = $donor->get_first_name();
 							break;
 
 						case 'last_name':
-							$user_info[ $key ] = $donor->get_last_name();
+							$user_info[$key] = $donor->get_last_name();
 							break;
 
 						case 'email':
-							$user_info[ $key ] = $donor->email;
+							$user_info[$key] = $donor->email;
 							break;
 					}
 				}
@@ -1797,12 +1796,12 @@ 
 	 * @return array The Address information for the payment.
 	 */
 	private function setup_address() {
-		$address['line1']   = give_get_meta( $this->ID, '_give_donor_billing_address1', true, '' );
-		$address['line2']   = give_get_meta( $this->ID, '_give_donor_billing_address2', true, '' );
-		$address['city']    = give_get_meta( $this->ID, '_give_donor_billing_city', true, '' );
-		$address['state']   = give_get_meta( $this->ID, '_give_donor_billing_state', true, '' );
-		$address['zip']     = give_get_meta( $this->ID, '_give_donor_billing_zip', true, '' );
-		$address['country'] = give_get_meta( $this->ID, '_give_donor_billing_country', true, '' );
+		$address['line1']   = give_get_meta($this->ID, '_give_donor_billing_address1', true, '');
+		$address['line2']   = give_get_meta($this->ID, '_give_donor_billing_address2', true, '');
+		$address['city']    = give_get_meta($this->ID, '_give_donor_billing_city', true, '');
+		$address['state']   = give_get_meta($this->ID, '_give_donor_billing_state', true, '');
+		$address['zip']     = give_get_meta($this->ID, '_give_donor_billing_zip', true, '');
+		$address['country'] = give_get_meta($this->ID, '_give_donor_billing_country', true, '');
 
 		return $address;
 	}
@@ -1817,7 +1816,7 @@ 
 	 */
 	private function setup_form_title() {
 
-		$form_id = $this->get_meta( '_give_payment_form_title', true );
+		$form_id = $this->get_meta('_give_payment_form_title', true);
 
 		return $form_id;
 	}
@@ -1832,7 +1831,7 @@ 
 	 */
 	private function setup_form_id() {
 
-		$form_id = $this->get_meta( '_give_payment_form_id', true );
+		$form_id = $this->get_meta('_give_payment_form_id', true);
 
 		return $form_id;
 	}
@@ -1846,7 +1845,7 @@ 
 	 * @return int The Form Price ID.
 	 */
 	private function setup_price_id() {
-		$price_id = $this->get_meta( '_give_payment_price_id', true );
+		$price_id = $this->get_meta('_give_payment_price_id', true);
 
 		return $price_id;
 	}
@@ -1860,7 +1859,7 @@ 
 	 * @return string The Payment Key.
 	 */
 	private function setup_payment_key() {
-		$key = $this->get_meta( '_give_payment_purchase_key', true );
+		$key = $this->get_meta('_give_payment_purchase_key', true);
 
 		return $key;
 	}
@@ -1885,7 +1884,7 @@ 
 	 * @return array The payment object as an array.
 	 */
 	public function array_convert() {
-		return get_object_vars( $this );
+		return get_object_vars($this);
 	}
 
 
@@ -1898,7 +1897,7 @@ 
 	 * @return bool
 	 */
 	public function is_completed() {
-		return ( 'publish' === $this->status && $this->completed_date );
+		return ('publish' === $this->status && $this->completed_date);
 	}
 
 	/**
@@ -1910,7 +1909,7 @@ 
 	 * @return string Date payment was completed.
 	 */
 	private function get_completed_date() {
-		return apply_filters( 'give_payment_completed_date', $this->completed_date, $this->ID, $this );
+		return apply_filters('give_payment_completed_date', $this->completed_date, $this->ID, $this);
 	}
 
 	/**
@@ -1922,7 +1921,7 @@ 
 	 * @return float Payment subtotal.
 	 */
 	private function get_subtotal() {
-		return apply_filters( 'give_get_payment_subtotal', $this->subtotal, $this->ID, $this );
+		return apply_filters('give_get_payment_subtotal', $this->subtotal, $this->ID, $this);
 	}
 
 	/**
@@ -1934,7 +1933,7 @@ 
 	 * @return string Payment currency code.
 	 */
 	private function get_currency() {
-		return apply_filters( 'give_payment_currency_code', $this->currency, $this->ID, $this );
+		return apply_filters('give_payment_currency_code', $this->currency, $this->ID, $this);
 	}
 
 	/**
@@ -1946,7 +1945,7 @@ 
 	 * @return string Gateway used.
 	 */
 	private function get_gateway() {
-		return apply_filters( 'give_payment_gateway', $this->gateway, $this->ID, $this );
+		return apply_filters('give_payment_gateway', $this->gateway, $this->ID, $this);
 	}
 
 	/**
@@ -1958,7 +1957,7 @@ 
 	 * @return string Donation ID from merchant processor.
 	 */
 	private function get_transaction_id() {
-		return apply_filters( 'give_get_payment_transaction_id', $this->transaction_id, $this->ID, $this );
+		return apply_filters('give_get_payment_transaction_id', $this->transaction_id, $this->ID, $this);
 	}
 
 	/**
@@ -1970,7 +1969,7 @@ 
 	 * @return string Payment IP address
 	 */
 	private function get_ip() {
-		return apply_filters( 'give_payment_user_ip', $this->ip, $this->ID, $this );
+		return apply_filters('give_payment_user_ip', $this->ip, $this->ID, $this);
 	}
 
 	/**
@@ -1982,7 +1981,7 @@ 
 	 * @return int Payment donor ID.
 	 */
 	private function get_donor_id() {
-		return apply_filters( 'give_payment_customer_id', $this->customer_id, $this->ID, $this );
+		return apply_filters('give_payment_customer_id', $this->customer_id, $this->ID, $this);
 	}
 
 	/**
@@ -1994,7 +1993,7 @@ 
 	 * @return int Payment user ID.
 	 */
 	private function get_user_id() {
-		return apply_filters( 'give_payment_user_id', $this->user_id, $this->ID, $this );
+		return apply_filters('give_payment_user_id', $this->user_id, $this->ID, $this);
 	}
 
 	/**
@@ -2006,7 +2005,7 @@ 
 	 * @return string Payment donor email.
 	 */
 	private function get_email() {
-		return apply_filters( 'give_payment_user_email', $this->email, $this->ID, $this );
+		return apply_filters('give_payment_user_email', $this->email, $this->ID, $this);
 	}
 
 	/**
@@ -2018,7 +2017,7 @@ 
 	 * @return array Payment user info.
 	 */
 	private function get_user_info() {
-		return apply_filters( 'give_payment_meta_user_info', $this->user_info, $this->ID, $this );
+		return apply_filters('give_payment_meta_user_info', $this->user_info, $this->ID, $this);
 	}
 
 	/**
@@ -2030,7 +2029,7 @@ 
 	 * @return array Payment billing address.
 	 */
 	private function get_address() {
-		return apply_filters( 'give_payment_address', $this->address, $this->ID, $this );
+		return apply_filters('give_payment_address', $this->address, $this->ID, $this);
 	}
 
 	/**
@@ -2042,7 +2041,7 @@ 
 	 * @return string Payment key.
 	 */
 	private function get_key() {
-		return apply_filters( 'give_payment_key', $this->key, $this->ID, $this );
+		return apply_filters('give_payment_key', $this->key, $this->ID, $this);
 	}
 
 	/**
@@ -2054,7 +2053,7 @@ 
 	 * @return string Payment form id
 	 */
 	private function get_form_id() {
-		return apply_filters( 'give_payment_form_id', $this->form_id, $this->ID, $this );
+		return apply_filters('give_payment_form_id', $this->form_id, $this->ID, $this);
 	}
 
 	/**
@@ -2066,7 +2065,7 @@ 
 	 * @return int|string Payment number
 	 */
 	private function get_number() {
-		return apply_filters( 'give_payment_number', $this->number, $this->ID, $this );
+		return apply_filters('give_payment_number', $this->number, $this->ID, $this);
 	}
 
 	/**
@@ -2078,7 +2077,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_serial_code( $args = array() ) {
-		return Give()->seq_donation_number->get_serial_code( $this, $args );
+	public function get_serial_code($args = array()) {
+		return Give()->seq_donation_number->get_serial_code($this, $args);
 	}
 }

includes/shortcodes.php

Doc Comments +1 added lines, -1 removed lines

@@ -789,7 +789,7 @@
  * @type string $display_style       How the form is displayed, either in new page or modal popup.
  *                                       Default 'redirect'. Accepts 'redirect', 'modal'.
  * }
- * @return string|bool The markup of the form grid or false.
+ * @return string|null The markup of the form grid or false.
  */
 function give_form_grid_shortcode( $atts ) {
 

Spacing +213 added lines, -213 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -26,42 +26,42 @@ 
  *
  * @return string|bool
  */
-function give_donation_history( $atts, $content = false ) {
+function give_donation_history($atts, $content = false) {
 
-	$donation_history_args = shortcode_atts( array(
+	$donation_history_args = shortcode_atts(array(
 		'id'             => true,
 		'date'           => true,
 		'donor'          => false,
 		'amount'         => true,
 		'status'         => false,
 		'payment_method' => false,
-	), $atts, 'donation_history' );
+	), $atts, 'donation_history');
 
 	// Always show receipt link.
 	$donation_history_args['details'] = true;
 
 	// Set Donation History Shortcode Arguments in session variable.
-	Give()->session->set( 'give_donation_history_args', $donation_history_args );
+	Give()->session->set('give_donation_history_args', $donation_history_args);
 
 	// If payment_key query arg exists, return receipt instead of donation history.
-	if ( isset( $_GET['payment_key'] ) ) {
+	if (isset($_GET['payment_key'])) {
 		ob_start();
 
-		echo give_receipt_shortcode( array() );
+		echo give_receipt_shortcode(array());
 
 		// Display donation history link only if Receipt Access Session is available.
-		if ( give_get_receipt_session() ) {
+		if (give_get_receipt_session()) {
 			echo sprintf(
 				'<a href="%s">%s</a>',
-				esc_url( give_get_history_page_uri() ),
-				__( '&laquo; Return to All Donations', 'give' )
+				esc_url(give_get_history_page_uri()),
+				__('&laquo; Return to All Donations', 'give')
 			);
 		}
 
 		return ob_get_clean();
 	}
 
-	$email_access = give_get_option( 'email_access' );
+	$email_access = give_get_option('email_access');
 
 	ob_start();
 
@@ -74,23 +74,23 @@ 
 	if (
 		is_user_logged_in()
 		|| false !== Give()->session->get_session_expiration()
-		|| ( give_is_setting_enabled( $email_access ) && Give()->email_access->token_exists )
+		|| (give_is_setting_enabled($email_access) && Give()->email_access->token_exists)
 		|| true === give_get_history_session()
 	) {
-		give_get_template_part( 'history', 'donations' );
+		give_get_template_part('history', 'donations');
 
-		if ( ! empty( $content ) ) {
-			echo do_shortcode( $content );
+		if ( ! empty($content)) {
+			echo do_shortcode($content);
 		}
 
-	} elseif ( give_is_setting_enabled( $email_access ) ) {
+	} elseif (give_is_setting_enabled($email_access)) {
 		// Is Email-based access enabled?
-		give_get_template_part( 'email', 'login-form' );
+		give_get_template_part('email', 'login-form');
 
 	} else {
 
-		echo apply_filters( 'give_donation_history_nonuser_message', Give()->notices->print_frontend_notice( __( 'You must be logged in to view your donation history. Please login using your account or create an account using the same email you used to donate with.', 'give' ), false ) );
-		echo do_shortcode( '[give_login]' );
+		echo apply_filters('give_donation_history_nonuser_message', Give()->notices->print_frontend_notice(__('You must be logged in to view your donation history. Please login using your account or create an account using the same email you used to donate with.', 'give'), false));
+		echo do_shortcode('[give_login]');
 	}
 
 	/**
@@ -104,10 +104,10 @@ 
 	 *
 	 * @return string HTML content
 	 */
-	return apply_filters( 'give_donation_history_shortcode_html', ob_get_clean(), $atts, $content );
+	return apply_filters('give_donation_history_shortcode_html', ob_get_clean(), $atts, $content);
 }
 
-add_shortcode( 'donation_history', 'give_donation_history' );
+add_shortcode('donation_history', 'give_donation_history');
 
 /**
  * Donation Form Shortcode
@@ -120,8 +120,8 @@ 
  *
  * @return string
  */
-function give_form_shortcode( $atts ) {
-	$atts = shortcode_atts( array(
+function give_form_shortcode($atts) {
+	$atts = shortcode_atts(array(
 		'id'                    => '',
 		'show_title'            => true,
 		'show_goal'             => true,
@@ -129,21 +129,21 @@ 
 		'float_labels'          => '',
 		'display_style'         => '',
 		'continue_button_title' => '',
-	), $atts, 'give_form' );
+	), $atts, 'give_form');
 
 	// Convert string to bool.
-	$atts['show_title'] = filter_var( $atts['show_title'], FILTER_VALIDATE_BOOLEAN );
-	$atts['show_goal']  = filter_var( $atts['show_goal'], FILTER_VALIDATE_BOOLEAN );
+	$atts['show_title'] = filter_var($atts['show_title'], FILTER_VALIDATE_BOOLEAN);
+	$atts['show_goal']  = filter_var($atts['show_goal'], FILTER_VALIDATE_BOOLEAN);
 
 	// get the Give Form
 	ob_start();
-	give_get_donation_form( $atts );
+	give_get_donation_form($atts);
 	$final_output = ob_get_clean();
 
-	return apply_filters( 'give_donate_form', $final_output, $atts );
+	return apply_filters('give_donate_form', $final_output, $atts);
 }
 
-add_shortcode( 'give_form', 'give_form_shortcode' );
+add_shortcode('give_form', 'give_form_shortcode');
 
 /**
  * Donation Form Goal Shortcode.
@@ -156,36 +156,36 @@ 
  *
  * @return string
  */
-function give_goal_shortcode( $atts ) {
-	$atts = shortcode_atts( array(
+function give_goal_shortcode($atts) {
+	$atts = shortcode_atts(array(
 		'id'        => '',
 		'show_text' => true,
 		'show_bar'  => true,
-	), $atts, 'give_goal' );
+	), $atts, 'give_goal');
 
 	// get the Give Form.
 	ob_start();
 
 	// Sanity check 1: ensure there is an ID Provided.
-	if ( empty( $atts['id'] ) ) {
-		Give()->notices->print_frontend_notice( __( 'The shortcode is missing Donation Form ID attribute.', 'give' ), true );
+	if (empty($atts['id'])) {
+		Give()->notices->print_frontend_notice(__('The shortcode is missing Donation Form ID attribute.', 'give'), true);
 	}
 
 	// Sanity check 2: Check the form even has Goals enabled.
-	if ( ! give_is_setting_enabled( give_get_meta( $atts['id'], '_give_goal_option', true ) ) ) {
+	if ( ! give_is_setting_enabled(give_get_meta($atts['id'], '_give_goal_option', true))) {
 
-		Give()->notices->print_frontend_notice( __( 'The form does not have Goals enabled.', 'give' ), true );
+		Give()->notices->print_frontend_notice(__('The form does not have Goals enabled.', 'give'), true);
 	} else {
 		// Passed all sanity checks: output Goal.
-		give_show_goal_progress( $atts['id'], $atts );
+		give_show_goal_progress($atts['id'], $atts);
 	}
 
 	$final_output = ob_get_clean();
 
-	return apply_filters( 'give_goal_shortcode_output', $final_output, $atts );
+	return apply_filters('give_goal_shortcode_output', $final_output, $atts);
 }
 
-add_shortcode( 'give_goal', 'give_goal_shortcode' );
+add_shortcode('give_goal', 'give_goal_shortcode');
 
 
 /**
@@ -202,22 +202,22 @@ 
  *
  * @return string
  */
-function give_login_form_shortcode( $atts ) {
+function give_login_form_shortcode($atts) {
 
-	$atts = shortcode_atts( array(
+	$atts = shortcode_atts(array(
 		// Add backward compatibility for redirect attribute.
 		'redirect'        => '',
 		'login-redirect'  => '',
 		'logout-redirect' => '',
-	), $atts, 'give_login' );
+	), $atts, 'give_login');
 
 	// Check login-redirect attribute first, if it empty or not found then check for redirect attribute and add value of this to login-redirect attribute.
-	$atts['login-redirect'] = ! empty( $atts['login-redirect'] ) ? $atts['login-redirect'] : ( ! empty( $atts['redirect'] ) ? $atts['redirect'] : '' );
+	$atts['login-redirect'] = ! empty($atts['login-redirect']) ? $atts['login-redirect'] : ( ! empty($atts['redirect']) ? $atts['redirect'] : '');
 
-	return give_login_form( $atts['login-redirect'], $atts['logout-redirect'] );
+	return give_login_form($atts['login-redirect'], $atts['logout-redirect']);
 }
 
-add_shortcode( 'give_login', 'give_login_form_shortcode' );
+add_shortcode('give_login', 'give_login_form_shortcode');
 
 /**
  * Register Shortcode.
@@ -232,15 +232,15 @@ 
  *
  * @return string
  */
-function give_register_form_shortcode( $atts ) {
-	$atts = shortcode_atts( array(
+function give_register_form_shortcode($atts) {
+	$atts = shortcode_atts(array(
 		'redirect' => '',
-	), $atts, 'give_register' );
+	), $atts, 'give_register');
 
-	return give_register_form( $atts['redirect'] );
+	return give_register_form($atts['redirect']);
 }
 
-add_shortcode( 'give_register', 'give_register_form_shortcode' );
+add_shortcode('give_register', 'give_register_form_shortcode');
 
 /**
  * Receipt Shortcode.
@@ -253,12 +253,12 @@ 
  *
  * @return string
  */
-function give_receipt_shortcode( $atts ) {
+function give_receipt_shortcode($atts) {
 
 	global $give_receipt_args;
 
-	$give_receipt_args = shortcode_atts( array(
-		'error'          => __( 'You are missing the payment key to view this donation receipt.', 'give' ),
+	$give_receipt_args = shortcode_atts(array(
+		'error'          => __('You are missing the payment key to view this donation receipt.', 'give'),
 		'price'          => true,
 		'donor'          => true,
 		'date'           => true,
@@ -268,16 +268,16 @@ 
 		'payment_status' => false,
 		'company_name'   => false,
 		'status_notice'  => true,
-	), $atts, 'give_receipt' );
+	), $atts, 'give_receipt');
 
-	if( ! wp_doing_ajax() ) {
+	if ( ! wp_doing_ajax()) {
 		ob_start();
-		give_get_template_part( 'receipt/placeholder' );
+		give_get_template_part('receipt/placeholder');
 		$placeholder = ob_get_clean();
 
 		return sprintf(
 			'<div id="give-receipt" data-shortcode="%s">%s</div>',
-			urlencode_deep( wp_json_encode( $atts ) ),
+			urlencode_deep(wp_json_encode($atts)),
 			$placeholder
 		);
 	}
@@ -286,43 +286,43 @@ 
 	$session = give_get_purchase_session();
 
 	// set payment key var
-	if ( isset( $_GET['payment_key'] ) ) {
-		$payment_key = urldecode( $_GET['payment_key'] );
-	} elseif ( $session ) {
+	if (isset($_GET['payment_key'])) {
+		$payment_key = urldecode($_GET['payment_key']);
+	} elseif ($session) {
 		$payment_key = $session['purchase_key'];
-	} elseif ( $give_receipt_args['payment_key'] ) {
+	} elseif ($give_receipt_args['payment_key']) {
 		$payment_key = $give_receipt_args['payment_key'];
 	}
 
-	$email_access = give_get_option( 'email_access' );
+	$email_access = give_get_option('email_access');
 
 	// No payment_key found & Email Access is Turned on.
-	if ( ! isset( $payment_key ) && give_is_setting_enabled( $email_access ) && ! Give()->email_access->token_exists ) {
+	if ( ! isset($payment_key) && give_is_setting_enabled($email_access) && ! Give()->email_access->token_exists) {
 
 		ob_start();
 
-		give_get_template_part( 'email-login-form' );
+		give_get_template_part('email-login-form');
 
 		return ob_get_clean();
 
-	} elseif ( ! isset( $payment_key ) ) {
+	} elseif ( ! isset($payment_key)) {
 
-		return Give()->notices->print_frontend_notice( $give_receipt_args['error'], false, 'error' );
+		return Give()->notices->print_frontend_notice($give_receipt_args['error'], false, 'error');
 
 	}
 
-	$user_can_view = give_can_view_receipt( $payment_key );
+	$user_can_view = give_can_view_receipt($payment_key);
 
 	// Key was provided, but user is logged out. Offer them the ability to login and view the receipt.
-	if ( ! $user_can_view && give_is_setting_enabled( $email_access ) && ! Give()->email_access->token_exists ) {
+	if ( ! $user_can_view && give_is_setting_enabled($email_access) && ! Give()->email_access->token_exists) {
 
 		ob_start();
 
-		give_get_template_part( 'email-login-form' );
+		give_get_template_part('email-login-form');
 
 		return ob_get_clean();
 
-	} elseif ( ! $user_can_view ) {
+	} elseif ( ! $user_can_view) {
 
 		global $give_login_redirect;
 
@@ -330,9 +330,9 @@ 
 
 		ob_start();
 
-		Give()->notices->print_frontend_notice( apply_filters( 'give_must_be_logged_in_error_message', __( 'You must be logged in to view this donation receipt.', 'give' ) ) );
+		Give()->notices->print_frontend_notice(apply_filters('give_must_be_logged_in_error_message', __('You must be logged in to view this donation receipt.', 'give')));
 
-		give_get_template_part( 'shortcode', 'login' );
+		give_get_template_part('shortcode', 'login');
 
 		$login_form = ob_get_clean();
 
@@ -346,20 +346,20 @@ 
 	 * or if user is logged out and donation was made as a guest, the donation session is checked for
 	 * or if user is logged in and the user can view sensitive shop data.
 	 */
-	if ( ! apply_filters( 'give_user_can_view_receipt', $user_can_view, $give_receipt_args ) ) {
-		return Give()->notices->print_frontend_notice( $give_receipt_args['error'], false, 'error' );
+	if ( ! apply_filters('give_user_can_view_receipt', $user_can_view, $give_receipt_args)) {
+		return Give()->notices->print_frontend_notice($give_receipt_args['error'], false, 'error');
 	}
 
 	ob_start();
 
-	give_get_template_part( 'shortcode', 'receipt' );
+	give_get_template_part('shortcode', 'receipt');
 
 	$display = ob_get_clean();
 
 	return $display;
 }
 
-add_shortcode( 'give_receipt', 'give_receipt_shortcode' );
+add_shortcode('give_receipt', 'give_receipt_shortcode');
 
 /**
  * Profile Editor Shortcode.
@@ -378,26 +378,26 @@ 
  *
  * @return string Output generated from the profile editor
  */
-function give_profile_editor_shortcode( $atts ) {
+function give_profile_editor_shortcode($atts) {
 
 	ob_start();
 
 	// Restrict access to donor profile, if donor and user are disconnected.
-	$is_donor_disconnected = get_user_meta( get_current_user_id(), '_give_is_donor_disconnected', true );
-	if ( is_user_logged_in() && $is_donor_disconnected ) {
-		Give()->notices->print_frontend_notice( __( 'Your Donor and User profile are no longer connected. Please contact the site administrator.', 'give' ), true, 'error' );
+	$is_donor_disconnected = get_user_meta(get_current_user_id(), '_give_is_donor_disconnected', true);
+	if (is_user_logged_in() && $is_donor_disconnected) {
+		Give()->notices->print_frontend_notice(__('Your Donor and User profile are no longer connected. Please contact the site administrator.', 'give'), true, 'error');
 
 		return false;
 	}
 
-	give_get_template_part( 'shortcode', 'profile-editor' );
+	give_get_template_part('shortcode', 'profile-editor');
 
 	$display = ob_get_clean();
 
 	return $display;
 }
 
-add_shortcode( 'give_profile_editor', 'give_profile_editor_shortcode' );
+add_shortcode('give_profile_editor', 'give_profile_editor_shortcode');
 
 /**
  * Process Profile Updater Form.
@@ -410,31 +410,31 @@ 
  *
  * @return bool
  */
-function give_process_profile_editor_updates( $data ) {
+function give_process_profile_editor_updates($data) {
 	// Profile field change request.
-	if ( empty( $_POST['give_profile_editor_submit'] ) && ! is_user_logged_in() ) {
+	if (empty($_POST['give_profile_editor_submit']) && ! is_user_logged_in()) {
 		return false;
 	}
 
 	// Nonce security.
-	if ( ! wp_verify_nonce( $data['give_profile_editor_nonce'], 'give-profile-editor-nonce' ) ) {
+	if ( ! wp_verify_nonce($data['give_profile_editor_nonce'], 'give-profile-editor-nonce')) {
 		return false;
 	}
 
 	$user_id       = get_current_user_id();
-	$old_user_data = get_userdata( $user_id );
+	$old_user_data = get_userdata($user_id);
 
 	/* @var Give_Donor $donor */
-	$donor            = new Give_Donor( $user_id, true );
+	$donor            = new Give_Donor($user_id, true);
 	$old_company_name = $donor->get_company_name();
 
-	$display_name     = isset( $data['give_display_name'] ) ? sanitize_text_field( $data['give_display_name'] ) : $old_user_data->display_name;
-	$first_name       = isset( $data['give_first_name'] ) ? sanitize_text_field( $data['give_first_name'] ) : $old_user_data->first_name;
-	$last_name        = isset( $data['give_last_name'] ) ? sanitize_text_field( $data['give_last_name'] ) : $old_user_data->last_name;
-	$company_name     = ! empty( $data['give_company_name'] ) ? sanitize_text_field( $data['give_company_name'] ) : $old_company_name;
-	$email            = isset( $data['give_email'] ) ? sanitize_email( $data['give_email'] ) : $old_user_data->user_email;
-	$password         = ! empty( $data['give_new_user_pass1'] ) ? $data['give_new_user_pass1'] : '';
-	$confirm_password = ! empty( $data['give_new_user_pass2'] ) ? $data['give_new_user_pass2'] : '';
+	$display_name     = isset($data['give_display_name']) ? sanitize_text_field($data['give_display_name']) : $old_user_data->display_name;
+	$first_name       = isset($data['give_first_name']) ? sanitize_text_field($data['give_first_name']) : $old_user_data->first_name;
+	$last_name        = isset($data['give_last_name']) ? sanitize_text_field($data['give_last_name']) : $old_user_data->last_name;
+	$company_name     = ! empty($data['give_company_name']) ? sanitize_text_field($data['give_company_name']) : $old_company_name;
+	$email            = isset($data['give_email']) ? sanitize_email($data['give_email']) : $old_user_data->user_email;
+	$password         = ! empty($data['give_new_user_pass1']) ? $data['give_new_user_pass1'] : '';
+	$confirm_password = ! empty($data['give_new_user_pass2']) ? $data['give_new_user_pass2'] : '';
 
 	$userdata = array(
 		'ID'           => $user_id,
@@ -454,60 +454,60 @@ 
 	 * @param int   $user_id  The ID of the user.
 	 * @param array $userdata User info, including ID, first name, last name, display name and email.
 	 */
-	do_action( 'give_pre_update_user_profile', $user_id, $userdata );
+	do_action('give_pre_update_user_profile', $user_id, $userdata);
 
 	// Make sure to validate first name of existing donors.
-	if ( empty( $first_name ) ) {
+	if (empty($first_name)) {
 		// Empty First Name.
-		give_set_error( 'empty_first_name', __( 'Please enter your first name.', 'give' ) );
+		give_set_error('empty_first_name', __('Please enter your first name.', 'give'));
 	}
 
 	// Make sure to validate passwords for existing Donors.
-	give_validate_user_password( $password, $confirm_password );
+	give_validate_user_password($password, $confirm_password);
 
-	if ( empty( $email ) ) {
+	if (empty($email)) {
 		// Make sure email should not be empty.
-		give_set_error( 'email_empty', __( 'The email you entered is empty.', 'give' ) );
+		give_set_error('email_empty', __('The email you entered is empty.', 'give'));
 
-	} elseif ( ! is_email( $email ) ) {
+	} elseif ( ! is_email($email)) {
 		// Make sure email should be valid.
-		give_set_error( 'email_not_valid', __( 'The email you entered is not valid. Please use another', 'give' ) );
+		give_set_error('email_not_valid', __('The email you entered is not valid. Please use another', 'give'));
 
-	} elseif ( $email != $old_user_data->user_email ) {
+	} elseif ($email != $old_user_data->user_email) {
 		// Make sure the new email doesn't belong to another user.
-		if ( email_exists( $email ) ) {
-			give_set_error( 'user_email_exists', __( 'The email you entered belongs to another user. Please use another.', 'give' ) );
-		} elseif ( Give()->donors->get_donor_by( 'email', $email ) ) {
+		if (email_exists($email)) {
+			give_set_error('user_email_exists', __('The email you entered belongs to another user. Please use another.', 'give'));
+		} elseif (Give()->donors->get_donor_by('email', $email)) {
 			// Make sure the new email doesn't belong to another user.
-			give_set_error( 'donor_email_exists', __( 'The email you entered belongs to another donor. Please use another.', 'give' ) );
+			give_set_error('donor_email_exists', __('The email you entered belongs to another donor. Please use another.', 'give'));
 		}
 	}
 
 	// Check for errors.
 	$errors = give_get_errors();
 
-	if ( $errors ) {
+	if ($errors) {
 		// Send back to the profile editor if there are errors.
-		wp_redirect( $data['give_redirect'] );
+		wp_redirect($data['give_redirect']);
 		give_die();
 	}
 
 	// Update Donor First Name and Last Name.
-	Give()->donors->update( $donor->id, array(
-		'name' => trim( "{$first_name} {$last_name}" ),
-	) );
-	Give()->donor_meta->update_meta( $donor->id, '_give_donor_first_name', $first_name );
-	Give()->donor_meta->update_meta( $donor->id, '_give_donor_last_name', $last_name );
-	Give()->donor_meta->update_meta( $donor->id, '_give_donor_company', $company_name );
+	Give()->donors->update($donor->id, array(
+		'name' => trim("{$first_name} {$last_name}"),
+	));
+	Give()->donor_meta->update_meta($donor->id, '_give_donor_first_name', $first_name);
+	Give()->donor_meta->update_meta($donor->id, '_give_donor_last_name', $last_name);
+	Give()->donor_meta->update_meta($donor->id, '_give_donor_company', $company_name);
 
 	$current_user = wp_get_current_user();
 
 	// Compares new values with old values to detect change in values.
-	$email_update        = ( $email !== $current_user->user_email ) ? true : false;
-	$display_name_update = ( $display_name !== $current_user->display_name ) ? true : false;
-	$first_name_update   = ( $first_name !== $current_user->first_name ) ? true : false;
-	$last_name_update    = ( $last_name !== $current_user->last_name ) ? true : false;
-	$company_name_update = ( $company_name !== $old_company_name ) ? true : false;
+	$email_update        = ($email !== $current_user->user_email) ? true : false;
+	$display_name_update = ($display_name !== $current_user->display_name) ? true : false;
+	$first_name_update   = ($first_name !== $current_user->first_name) ? true : false;
+	$last_name_update    = ($last_name !== $current_user->last_name) ? true : false;
+	$company_name_update = ($company_name !== $old_company_name) ? true : false;
 	$update_code         = 0;
 
 	/**
@@ -515,35 +515,35 @@ 
 	 *
 	 * @var boolean
 	 */
-	$profile_update = ( $email_update || $display_name_update || $first_name_update || $last_name_update || $company_name_update );
+	$profile_update = ($email_update || $display_name_update || $first_name_update || $last_name_update || $company_name_update);
 
 	/**
 	 * True if password fields are filled.
 	 *
 	 * @var boolean
 	 */
-	$password_update = ( ! empty( $password ) && ! empty( $confirm_password ) );
+	$password_update = ( ! empty($password) && ! empty($confirm_password));
 
-	if ( $profile_update ) {
+	if ($profile_update) {
 
 		// If only profile fields are updated.
 		$update_code = '1';
 
-		if ( $password_update ) {
+		if ($password_update) {
 
 			// If profile fields AND password both are updated.
 			$update_code = '2';
 		}
-	} elseif ( $password_update ) {
+	} elseif ($password_update) {
 
 		// If only password is updated.
 		$update_code = '3';
 	}
 
 	// Update the user.
-	$updated = wp_update_user( $userdata );
+	$updated = wp_update_user($userdata);
 
-	if ( $updated ) {
+	if ($updated) {
 
 		/**
 		 * Fires after updating user profile.
@@ -553,7 +553,7 @@ 
 		 * @param int   $user_id  The ID of the user.
 		 * @param array $userdata User info, including ID, first name, last name, display name and email.
 		 */
-		do_action( 'give_user_profile_updated', $user_id, $userdata );
+		do_action('give_user_profile_updated', $user_id, $userdata);
 
 		$profile_edit_redirect_args = array(
 			'updated'     => 'true',
@@ -564,10 +564,10 @@ 
 		 * Update codes '2' and '3' indicate a password change.
 		 * If the password is changed, then logout and redirect to the same page.
 		 */
-		if ( '2' === $update_code || '3' === $update_code ) {
-			wp_logout( wp_redirect( add_query_arg( $profile_edit_redirect_args, $data['give_redirect'] ) ) );
+		if ('2' === $update_code || '3' === $update_code) {
+			wp_logout(wp_redirect(add_query_arg($profile_edit_redirect_args, $data['give_redirect'])));
 		} else {
-			wp_redirect( add_query_arg( $profile_edit_redirect_args, $data['give_redirect'] ) );
+			wp_redirect(add_query_arg($profile_edit_redirect_args, $data['give_redirect']));
 		}
 
 		give_die();
@@ -576,7 +576,7 @@ 
 	return false;
 }
 
-add_action( 'give_edit_user_profile', 'give_process_profile_editor_updates' );
+add_action('give_edit_user_profile', 'give_process_profile_editor_updates');
 
 /**
  * Give totals Shortcode.
@@ -589,24 +589,24 @@ 
  *
  * @return string
  */
-function give_totals_shortcode( $atts ) {
-	$total = get_option( 'give_earnings_total', false );
+function give_totals_shortcode($atts) {
+	$total = get_option('give_earnings_total', false);
 
-	$message = apply_filters( 'give_totals_message', __( 'Hey! We\'ve raised {total} of the {total_goal} we are trying to raise for this campaign!', 'give' ) );
+	$message = apply_filters('give_totals_message', __('Hey! We\'ve raised {total} of the {total_goal} we are trying to raise for this campaign!', 'give'));
 
-	$atts = shortcode_atts( array(
+	$atts = shortcode_atts(array(
 		'total_goal'   => 0, // integer
 		'ids'          => 0, // integer|array
 		'cats'         => 0, // integer|array
 		'tags'         => 0, // integer|array
 		'message'      => $message,
 		'link'         => '', // URL
-		'link_text'    => __( 'Donate Now', 'give' ), // string,
+		'link_text'    => __('Donate Now', 'give'), // string,
 		'progress_bar' => true, // boolean
-	), $atts, 'give_totals' );
+	), $atts, 'give_totals');
 
 	// Total Goal.
-	$total_goal = give_maybe_sanitize_amount( $atts['total_goal'] );
+	$total_goal = give_maybe_sanitize_amount($atts['total_goal']);
 
 	/**
 	 * Give Action fire before the shortcode is rendering is started.
@@ -615,14 +615,14 @@ 
 	 *
 	 * @param array $atts shortcode attribute.
 	 */
-	do_action( 'give_totals_goal_shortcode_before_render', $atts );
+	do_action('give_totals_goal_shortcode_before_render', $atts);
 
 	// Build query based on cat, tag and Form ids.
-	if ( ! empty( $atts['cats'] ) || ! empty( $atts['tags'] ) || ! empty( $atts['ids'] ) ) {
+	if ( ! empty($atts['cats']) || ! empty($atts['tags']) || ! empty($atts['ids'])) {
 
 		$form_ids = array();
-		if ( ! empty( $atts['ids'] ) ) {
-			$form_ids = array_filter( array_map( 'trim', explode( ',', $atts['ids'] ) ) );
+		if ( ! empty($atts['ids'])) {
+			$form_ids = array_filter(array_map('trim', explode(',', $atts['ids'])));
 		}
 
 		/**
@@ -636,23 +636,23 @@ 
 			'post_type'        => 'give_forms',
 			'post_status'      => 'publish',
 			'post__in'         => $form_ids,
-			'posts_per_page'   => - 1,
+			'posts_per_page'   => -1,
 			'fields'           => 'ids',
 			'tax_query'        => array(
 				'relation' => 'AND',
 			),
 		);
 
-		if ( ! empty( $atts['cats'] ) ) {
-			$cats                     = array_filter( array_map( 'trim', explode( ',', $atts['cats'] ) ) );
+		if ( ! empty($atts['cats'])) {
+			$cats                     = array_filter(array_map('trim', explode(',', $atts['cats'])));
 			$form_args['tax_query'][] = array(
 				'taxonomy' => 'give_forms_category',
 				'terms'    => $cats,
 			);
 		}
 
-		if ( ! empty( $atts['tags'] ) ) {
-			$tags                     = array_filter( array_map( 'trim', explode( ',', $atts['tags'] ) ) );
+		if ( ! empty($atts['tags'])) {
+			$tags                     = array_filter(array_map('trim', explode(',', $atts['tags'])));
 			$form_args['tax_query'][] = array(
 				'taxonomy' => 'give_forms_tag',
 				'terms'    => $tags,
@@ -668,15 +668,15 @@ 
 		 *
 		 * @return array $form_args WP query argument for Total Goal.
 		 */
-		$form_args = (array) apply_filters( 'give_totals_goal_shortcode_query_args', $form_args );
+		$form_args = (array) apply_filters('give_totals_goal_shortcode_query_args', $form_args);
 
-		$forms = new WP_Query( $form_args );
+		$forms = new WP_Query($form_args);
 
-		if ( isset( $forms->posts ) ) {
+		if (isset($forms->posts)) {
 			$total = 0;
-			foreach ( $forms->posts as $post ) {
-				$form_earning = give_get_meta( $post, '_give_form_earnings', true );
-				$form_earning = ! empty( $form_earning ) ? $form_earning : 0;
+			foreach ($forms->posts as $post) {
+				$form_earning = give_get_meta($post, '_give_form_earnings', true);
+				$form_earning = ! empty($form_earning) ? $form_earning : 0;
 
 				/**
 				 * Update Form earnings.
@@ -686,7 +686,7 @@ 
 				 * @param int $post Form ID.
 				 * @param string $form_earning Total earning of Form.
 				 */
-				$total += apply_filters( 'give_totals_form_earning', $form_earning, $post );
+				$total += apply_filters('give_totals_form_earning', $form_earning, $post);
 			}
 		}
 
@@ -694,23 +694,23 @@ 
 
 	// Append link with text.
 	$donate_link = '';
-	if ( ! empty( $atts['link'] ) ) {
-		$donate_link = sprintf( ' <a class="give-totals-text-link" href="%1$s">%2$s</a>', esc_url( $atts['link'] ), esc_html( $atts['link_text'] ) );
+	if ( ! empty($atts['link'])) {
+		$donate_link = sprintf(' <a class="give-totals-text-link" href="%1$s">%2$s</a>', esc_url($atts['link']), esc_html($atts['link_text']));
 	}
 
 	// Replace {total} in message.
-	$message = str_replace( '{total}', give_currency_filter(
-		give_format_amount( $total,
-			array( 'sanitize' => false )
+	$message = str_replace('{total}', give_currency_filter(
+		give_format_amount($total,
+			array('sanitize' => false)
 		)
-	), esc_html( $atts['message'] ) );
+	), esc_html($atts['message']));
 
 	// Replace {total_goal} in message.
-	$message = str_replace( '{total_goal}', give_currency_filter(
-		give_format_amount( $total_goal,
-			array( 'sanitize' => true )
+	$message = str_replace('{total_goal}', give_currency_filter(
+		give_format_amount($total_goal,
+			array('sanitize' => true)
 		)
-	), $message );
+	), $message);
 
 	/**
 	 * Update Give totals shortcode output.
@@ -720,19 +720,19 @@ 
 	 * @param string $message Shortcode Message.
 	 * @param array $atts ShortCode attributes.
 	 */
-	$message = apply_filters( 'give_totals_shortcode_message', $message, $atts );
+	$message = apply_filters('give_totals_shortcode_message', $message, $atts);
 
 	ob_start();
 	?>
 	<div class="give-totals-shortcode-wrap">
 		<?php
 		// Show Progress Bar if progress_bar set true.
-		$show_progress_bar = isset( $atts['progress_bar'] ) ? filter_var( $atts['progress_bar'], FILTER_VALIDATE_BOOLEAN ) : true;
-		if ( $show_progress_bar ) {
-			give_show_goal_totals_progress( $total, $total_goal );
+		$show_progress_bar = isset($atts['progress_bar']) ? filter_var($atts['progress_bar'], FILTER_VALIDATE_BOOLEAN) : true;
+		if ($show_progress_bar) {
+			give_show_goal_totals_progress($total, $total_goal);
 		}
 
-		echo sprintf( $message ) . $donate_link;
+		echo sprintf($message).$donate_link;
 		?>
 	</div>
 	<?php
@@ -747,7 +747,7 @@ 
 	 * @param array  $atts               shortcode attribute.
 	 * @param string $give_totals_output shortcode output.
 	 */
-	do_action( 'give_totals_goal_shortcode_after_render', $atts, $give_totals_output );
+	do_action('give_totals_goal_shortcode_after_render', $atts, $give_totals_output);
 
 	/**
 	 * Give Totals Shortcode output.
@@ -756,11 +756,11 @@ 
 	 *
 	 * @param string $give_totals_output
 	 */
-	return apply_filters( 'give_totals_shortcode_output', $give_totals_output );
+	return apply_filters('give_totals_shortcode_output', $give_totals_output);
 
 }
 
-add_shortcode( 'give_totals', 'give_totals_shortcode' );
+add_shortcode('give_totals', 'give_totals_shortcode');
 
 
 /**
@@ -791,11 +791,11 @@ 
  * }
  * @return string|bool The markup of the form grid or false.
  */
-function give_form_grid_shortcode( $atts ) {
+function give_form_grid_shortcode($atts) {
 
 	$give_settings = give_get_settings();
 
-	$atts = shortcode_atts( array(
+	$atts = shortcode_atts(array(
 		'forms_per_page'      => 12,
 		'paged'               => true,
 		'ids'                 => '',
@@ -812,11 +812,11 @@ 
 		'excerpt_length'      => 16,
 		'display_style'       => 'modal_reveal',
 		'status'              => '', // open or closed
-	), $atts );
+	), $atts);
 
 	// Validate integer attributes.
-	$atts['forms_per_page'] = intval( $atts['forms_per_page'] );
-	$atts['excerpt_length'] = intval( $atts['excerpt_length'] );
+	$atts['forms_per_page'] = intval($atts['forms_per_page']);
+	$atts['excerpt_length'] = intval($atts['excerpt_length']);
 
 	// Validate boolean attributes.
 	$boolean_attributes = array(
@@ -827,8 +827,8 @@ 
 		'show_featured_image',
 	);
 
-	foreach ( $boolean_attributes as $att ) {
-		$atts[ $att ] = filter_var( $atts[ $att ], FILTER_VALIDATE_BOOLEAN );
+	foreach ($boolean_attributes as $att) {
+		$atts[$att] = filter_var($atts[$att], FILTER_VALIDATE_BOOLEAN);
 	}
 
 	// Set default form query args.
@@ -842,9 +842,9 @@ 
 	);
 
 	// Filter results of form grid based on form status.
-	$form_closed_status = trim( $atts['status'] );
+	$form_closed_status = trim($atts['status']);
 
-	if ( ! empty( $form_closed_status ) ) {
+	if ( ! empty($form_closed_status)) {
 		$form_args['meta_query'] = array(
 			array(
 				'key'   => '_give_form_status',
@@ -854,25 +854,25 @@ 
 	}
 
 	// Maybe add pagination.
-	if ( true === $atts['paged'] ) {
-		$form_args['paged'] = get_query_var( 'paged' ) ? get_query_var( 'paged' ) : 1;
+	if (true === $atts['paged']) {
+		$form_args['paged'] = get_query_var('paged') ? get_query_var('paged') : 1;
 	}
 
 	// Maybe filter forms by IDs.
-	if ( ! empty( $atts['ids'] ) ) {
-		$form_args['post__in'] = array_filter( array_map( 'trim', explode( ',', $atts['ids'] ) ) );
+	if ( ! empty($atts['ids'])) {
+		$form_args['post__in'] = array_filter(array_map('trim', explode(',', $atts['ids'])));
 	}
 
 	// Convert comma-separated form IDs into array.
-	if ( ! empty( $atts['exclude'] ) ) {
-		$form_args['post__not_in'] = array_filter( array_map( function( $item ) {
-			return intval( trim( $item ) );
-		}, explode( ',', $atts['exclude'] ) ) );
+	if ( ! empty($atts['exclude'])) {
+		$form_args['post__not_in'] = array_filter(array_map(function($item) {
+			return intval(trim($item));
+		}, explode(',', $atts['exclude'])));
 	}
 
 	// Maybe filter by form category.
-	if ( ! empty( $atts['cats'] ) ) {
-		$cats                     = array_filter( array_map( 'trim', explode( ',', $atts['cats'] ) ) );
+	if ( ! empty($atts['cats'])) {
+		$cats                     = array_filter(array_map('trim', explode(',', $atts['cats'])));
 		$tax_query                = array(
 			'taxonomy' => 'give_forms_category',
 			'terms'    => $cats,
@@ -881,8 +881,8 @@ 
 	}
 
 	// Maybe filter by form tag.
-	if ( ! empty( $atts['tags'] ) ) {
-		$tags                     = array_filter( array_map( 'trim', explode( ',', $atts['tags'] ) ) );
+	if ( ! empty($atts['tags'])) {
+		$tags                     = array_filter(array_map('trim', explode(',', $atts['tags'])));
 		$tax_query                = array(
 			'taxonomy' => 'give_forms_tag',
 			'terms'    => $tags,
@@ -891,24 +891,24 @@ 
 	}
 
 	// Query to output donation forms.
-	$form_query = new WP_Query( $form_args );
+	$form_query = new WP_Query($form_args);
 
-	if ( $form_query->have_posts() ) {
+	if ($form_query->have_posts()) {
 		ob_start();
 
-		add_filter( 'add_give_goal_progress_class', 'add_give_goal_progress_class', 10, 1 );
-		add_filter( 'add_give_goal_progress_bar_class', 'add_give_goal_progress_bar_class', 10, 1 );
-		add_filter( 'give_form_wrap_classes', 'add_class_for_form_grid', 10, 3 );
-		add_action( 'give_donation_form_top', 'give_is_form_grid_page_hidden_field', 10, 3 );
+		add_filter('add_give_goal_progress_class', 'add_give_goal_progress_class', 10, 1);
+		add_filter('add_give_goal_progress_bar_class', 'add_give_goal_progress_bar_class', 10, 1);
+		add_filter('give_form_wrap_classes', 'add_class_for_form_grid', 10, 3);
+		add_action('give_donation_form_top', 'give_is_form_grid_page_hidden_field', 10, 3);
 
 		echo '<div class="give-wrap">';
-		echo '<div class="give-grid give-grid--' . esc_attr( $atts['columns'] ) . '">';
+		echo '<div class="give-grid give-grid--'.esc_attr($atts['columns']).'">';
 
-		while ( $form_query->have_posts() ) {
+		while ($form_query->have_posts()) {
 			$form_query->the_post();
 
 			// Give/templates/shortcode-form-grid.php.
-			give_get_template( 'shortcode-form-grid', array( $give_settings, $atts ) );
+			give_get_template('shortcode-form-grid', array($give_settings, $atts));
 
 		}
 
@@ -916,28 +916,28 @@ 
 
 		echo '</div><!-- .give-grid -->';
 
-		remove_filter( 'add_give_goal_progress_class', 'add_give_goal_progress_class' );
-		remove_filter( 'add_give_goal_progress_bar_class', 'add_give_goal_progress_bar_class' );
-		remove_filter( 'give_form_wrap_classes', 'add_class_for_form_grid', 10 );
-		remove_action( 'give_donation_form_top', 'give_is_form_grid_page_hidden_field', 10 );
+		remove_filter('add_give_goal_progress_class', 'add_give_goal_progress_class');
+		remove_filter('add_give_goal_progress_bar_class', 'add_give_goal_progress_bar_class');
+		remove_filter('give_form_wrap_classes', 'add_class_for_form_grid', 10);
+		remove_action('give_donation_form_top', 'give_is_form_grid_page_hidden_field', 10);
 
-		if ( false !== $atts['paged'] ) {
+		if (false !== $atts['paged']) {
 			$paginate_args = array(
-				'current'   => max( 1, get_query_var( 'paged' ) ),
+				'current'   => max(1, get_query_var('paged')),
 				'total'     => $form_query->max_num_pages,
 				'show_all'  => false,
 				'end_size'  => 1,
 				'mid_size'  => 2,
 				'prev_next' => true,
-				'prev_text' => __( '« Previous', 'give' ),
-				'next_text' => __( 'Next »', 'give' ),
+				'prev_text' => __('« Previous', 'give'),
+				'next_text' => __('Next »', 'give'),
 				'type'      => 'plain',
 				'add_args'  => false,
 			);
 
 			printf(
 				'<div class="give-page-numbers">%s</div>',
-				paginate_links( $paginate_args )
+				paginate_links($paginate_args)
 			);
 		}
 		echo '</div><!-- .give-wrap -->';
@@ -946,4 +946,4 @@ 
 	}
 }
 
-add_shortcode( 'give_form_grid', 'give_form_grid_shortcode' );
+add_shortcode('give_form_grid', 'give_form_grid_shortcode');

templates/shortcode-receipt.php

Spacing +57 added lines, -57 removed lines

@@ -6,29 +6,29 @@ 
 global $give_receipt_args, $payment;
 
 // Validation: Ensure $payment var is set.
-if ( empty( $payment ) ) {
-	$payment = ! empty( $give_receipt_args['id'] ) ? get_post( $give_receipt_args['id'] ) : 0;
+if (empty($payment)) {
+	$payment = ! empty($give_receipt_args['id']) ? get_post($give_receipt_args['id']) : 0;
 }
 
 // Double-Validation: Check for $payment global.
-if ( empty( $payment ) ) {
-	Give()->notices->print_frontend_notice( __( 'The specified receipt ID appears to be invalid.', 'give' ) );
+if (empty($payment)) {
+	Give()->notices->print_frontend_notice(__('The specified receipt ID appears to be invalid.', 'give'));
 
 	return;
 }
 
 $donation_id  = $payment->ID;
-$donation_number = Give()->seq_donation_number->get_serial_code( $payment->ID );
-$form_id      = give_get_payment_meta( $donation_id, '_give_payment_form_id', true );
-$donation     = give_get_donation_form_title( $donation_id );
-$user         = give_get_payment_meta_user_info( $donation_id );
-$email        = give_get_payment_user_email( $donation_id );
+$donation_number = Give()->seq_donation_number->get_serial_code($payment->ID);
+$form_id      = give_get_payment_meta($donation_id, '_give_payment_form_id', true);
+$donation     = give_get_donation_form_title($donation_id);
+$user         = give_get_payment_meta_user_info($donation_id);
+$email        = give_get_payment_user_email($donation_id);
 $status       = $payment->post_status;
-$status_label = give_get_payment_status( $payment, true );
-$company_name = give_get_payment_meta( $donation_id, '_give_donation_company', true );
+$status_label = give_get_payment_status($payment, true);
+$company_name = give_get_payment_meta($donation_id, '_give_donation_company', true);
 
 // Update donor name, if title prefix is set.
-$full_name    = give_get_donor_name_with_title_prefixes( $user['title'], "{$user['first_name']} {$user['last_name']}" );
+$full_name    = give_get_donor_name_with_title_prefixes($user['title'], "{$user['first_name']} {$user['last_name']}");
 
 /**
  * Generate Donation Receipt Arguments.
@@ -39,7 +39,7 @@ 
  * @since 1.8.8
  */
 $give_receipt_args['donation_receipt']['donor'] = array(
-	'name'    => __( 'Donor', 'give' ),
+	'name'    => __('Donor', 'give'),
 	'value'   => $full_name,
 	'display' => $give_receipt_args['donor'],
 );
@@ -54,51 +54,51 @@ 
  * @return bool show/hide company name in donation receipt page.
  */
 $give_receipt_args['donation_receipt']['company_name'] = array(
-	'name'    => __( 'Company Name', 'give' ),
-	'value'   => esc_attr( $company_name ),
+	'name'    => __('Company Name', 'give'),
+	'value'   => esc_attr($company_name),
 	// Do not show company field if empty
-	'display' => empty( $company_name ) ? false : $give_receipt_args['company_name'],
+	'display' => empty($company_name) ? false : $give_receipt_args['company_name'],
 );
 
 $give_receipt_args['donation_receipt']['date'] = array(
-	'name'    => __( 'Date', 'give' ),
-	'value'   => date_i18n( give_date_format(), strtotime( give_get_payment_completed_date( $donation_id ) ) ),
+	'name'    => __('Date', 'give'),
+	'value'   => date_i18n(give_date_format(), strtotime(give_get_payment_completed_date($donation_id))),
 	'display' => $give_receipt_args['date'],
 );
 
 $give_receipt_args['donation_receipt']['total_donation'] = array(
-	'name'    => __( 'Total Donation', 'give' ),
-	'value'   => give_donation_amount( $donation_id, array( 'currency' => true, 'amount' => true, 'type' => 'receipt' ) ),
+	'name'    => __('Total Donation', 'give'),
+	'value'   => give_donation_amount($donation_id, array('currency' => true, 'amount' => true, 'type' => 'receipt')),
 	'display' => $give_receipt_args['price'],
 );
 
 $give_receipt_args['donation_receipt']['donation'] = array(
-	'name'    => __( 'Donation', 'give' ),
+	'name'    => __('Donation', 'give'),
 	'value'   => $donation,
 	'display' => true,
 );
 
 $give_receipt_args['donation_receipt']['donation_status'] = array(
-	'name'    => __( 'Donation Status', 'give' ),
-	'value'   => esc_attr( $status_label ),
+	'name'    => __('Donation Status', 'give'),
+	'value'   => esc_attr($status_label),
 	'display' => $give_receipt_args['payment_status'],
 );
 
 $give_receipt_args['donation_receipt']['donation_id'] = array(
-	'name'    => __( 'Donation ID', 'give' ),
+	'name'    => __('Donation ID', 'give'),
 	'value'   => $donation_number,
 	'display' => $give_receipt_args['payment_id'],
 );
 
 $give_receipt_args['donation_receipt']['payment_key'] = array(
-	'name'    => __( 'Payment Key', 'give' ),
-	'value'   => get_post_meta( $donation_id, '_give_payment_purchase_key', true ),
+	'name'    => __('Payment Key', 'give'),
+	'value'   => get_post_meta($donation_id, '_give_payment_purchase_key', true),
 	'display' => $give_receipt_args['payment_key'],
 );
 
 $give_receipt_args['donation_receipt']['payment_method'] = array(
-	'name'    => __( 'Payment Method', 'give' ),
-	'value'   => give_get_gateway_checkout_label( give_get_payment_gateway( $donation_id ) ),
+	'name'    => __('Payment Method', 'give'),
+	'value'   => give_get_gateway_checkout_label(give_get_payment_gateway($donation_id)),
 	'display' => $give_receipt_args['payment_method'],
 );
 
@@ -113,10 +113,10 @@ 
  *
  * @since 1.8.8
  */
-$give_receipt_args['donation_receipt'] = apply_filters( 'give_donation_receipt_args', $give_receipt_args['donation_receipt'], $donation_id, $form_id );
+$give_receipt_args['donation_receipt'] = apply_filters('give_donation_receipt_args', $give_receipt_args['donation_receipt'], $donation_id, $form_id);
 
 // When the donation were made through offline donation, We won't show receipt and payment status though.
-if ( 'offline' === give_get_payment_gateway( $payment->ID ) && 'pending' === $status ) {
+if ('offline' === give_get_payment_gateway($payment->ID) && 'pending' === $status) {
 
 	/**
 	 * Before the offline donation receipt content starts.
@@ -126,13 +126,13 @@ 
 	 * @param Give_Payment $payment           Donation payment object.
 	 * @param array        $give_receipt_args Receipt Arguments.
 	 */
-	do_action( 'give_receipt_before_offline_payment', $payment, $give_receipt_args );
+	do_action('give_receipt_before_offline_payment', $payment, $give_receipt_args);
 	?>
-	<h2><?php echo apply_filters( 'give_receipt_offline_payment_heading', __( 'Your Donation is Almost Complete!', 'give' ) ); ?></h2>
-	<div id="give_donation_receipt" class="<?php echo esc_attr( apply_filters( 'give_receipt_offline_payment_classes', 'give_receipt_offline_payment' ) ); ?>">
+	<h2><?php echo apply_filters('give_receipt_offline_payment_heading', __('Your Donation is Almost Complete!', 'give')); ?></h2>
+	<div id="give_donation_receipt" class="<?php echo esc_attr(apply_filters('give_receipt_offline_payment_classes', 'give_receipt_offline_payment')); ?>">
 		<?php
 		// Instruction for offline donation.
-		$offline_instruction = give_get_offline_payment_instruction( $form_id, true );
+		$offline_instruction = give_get_offline_payment_instruction($form_id, true);
 
 		/**
 		 * Instruction for the offline donation.
@@ -143,7 +143,7 @@ 
 		 * @param Give_Payment $payment             Payment object.
 		 * @param integer      $form_id             Donation form id.
 		 */
-		echo apply_filters( 'give_receipt_offline_payment_instruction', $offline_instruction, $payment, $form_id );
+		echo apply_filters('give_receipt_offline_payment_instruction', $offline_instruction, $payment, $form_id);
 		?>
 	</div>
 	<?php
@@ -155,52 +155,52 @@ 
 	 * @param Give_Payment $payment           Donation payment object.
 	 * @param array        $give_receipt_args Receipt Arguments.
 	 */
-	do_action( 'give_receipt_after_offline_payment', $payment, $give_receipt_args );
+	do_action('give_receipt_after_offline_payment', $payment, $give_receipt_args);
 
 	return;
 }
 
 // Show payment status notice based on shortcode attribute.
-if ( filter_var( $give_receipt_args['status_notice'], FILTER_VALIDATE_BOOLEAN ) ) {
+if (filter_var($give_receipt_args['status_notice'], FILTER_VALIDATE_BOOLEAN)) {
 	$notice_message = '';
 	$notice_type    = 'warning';
 
-	switch ( $status ) {
+	switch ($status) {
 		case 'publish':
-			$notice_message = __( 'Payment Complete: Thank you for your donation.', 'give' );
+			$notice_message = __('Payment Complete: Thank you for your donation.', 'give');
 			$notice_type    = 'success';
 			break;
 		case 'pending':
-			$notice_message = __( 'Payment Pending: Your donation is currently processing.', 'give' );
+			$notice_message = __('Payment Pending: Your donation is currently processing.', 'give');
 			$notice_type    = 'warning';
 			break;
 		case 'refunded':
-			$notice_message = __( 'Payment Refunded: Your donation has been refunded.', 'give' );
+			$notice_message = __('Payment Refunded: Your donation has been refunded.', 'give');
 			$notice_type    = 'warning';
 			break;
 		case 'preapproval':
-			$notice_message = __( 'Payment Preapproved: Thank you for your donation.', 'give' );
+			$notice_message = __('Payment Preapproved: Thank you for your donation.', 'give');
 			$notice_type    = 'warning';
 			break;
 		case 'failed':
-			$notice_message = __( 'Payment Failed: Please contact the site owner for assistance.', 'give' );
+			$notice_message = __('Payment Failed: Please contact the site owner for assistance.', 'give');
 			$notice_type    = 'error';
 			break;
 		case 'cancelled':
-			$notice_message = __( 'Payment Cancelled: Your donation has been cancelled.', 'give' );
+			$notice_message = __('Payment Cancelled: Your donation has been cancelled.', 'give');
 			$notice_type    = 'error';
 			break;
 		case 'abandoned':
-			$notice_message = __( 'Payment Abandoned: This donation has not been completed.', 'give' );
+			$notice_message = __('Payment Abandoned: This donation has not been completed.', 'give');
 			$notice_type    = 'error';
 			break;
 		case 'revoked':
-			$notice_message = __( 'Payment Revoked: Please contact the site owner for assistance.', 'give' );
+			$notice_message = __('Payment Revoked: Please contact the site owner for assistance.', 'give');
 			$notice_type    = 'error';
 			break;
 	}
 
-	if ( ! empty( $notice_message ) ) {
+	if ( ! empty($notice_message)) {
 		/**
 		 * Filters payment status notice for receipts.
 		 *
@@ -215,7 +215,7 @@ 
 		 * @param string $status Payment status.
 		 * @param int $donation_id Donation ID.
 		 */
-		echo apply_filters( 'give_receipt_status_notice', Give()->notices->print_frontend_notice( $notice_message, false, $notice_type ), $id, $status, $donation_id );
+		echo apply_filters('give_receipt_status_notice', Give()->notices->print_frontend_notice($notice_message, false, $notice_type), $id, $status, $donation_id);
 	}
 }// End if().
 
@@ -229,7 +229,7 @@ 
  * @param object $payment           The payment object.
  * @param array  $give_receipt_args Receipt_argument.
  */
-do_action( 'give_payment_receipt_before_table', $payment, $give_receipt_args );
+do_action('give_payment_receipt_before_table', $payment, $give_receipt_args);
 ?>
 
 <table id="give_donation_receipt" class="give-table">
@@ -245,11 +245,11 @@ 
 	 * @param object $payment           The payment object.
 	 * @param array  $give_receipt_args Receipt_argument.
 	 */
-	do_action( 'give_payment_receipt_header_before', $payment, $give_receipt_args );
+	do_action('give_payment_receipt_header_before', $payment, $give_receipt_args);
 	?>
 	<tr>
 		<th scope="colgroup" colspan="2">
-			<span class="give-receipt-thead-text"><?php esc_html_e( 'Donation Receipt', 'give' ) ?></span>
+			<span class="give-receipt-thead-text"><?php esc_html_e('Donation Receipt', 'give') ?></span>
 		</th>
 	</tr>
 	<?php
@@ -263,7 +263,7 @@ 
 	 * @param object $payment           The payment object.
 	 * @param array  $give_receipt_args Receipt_argument.
 	 */
-	do_action( 'give_payment_receipt_header_after', $payment, $give_receipt_args );
+	do_action('give_payment_receipt_header_after', $payment, $give_receipt_args);
 	?>
 	</thead>
 
@@ -279,11 +279,11 @@ 
 	 * @param object $payment           The payment object.
 	 * @param array  $give_receipt_args Receipt_argument.
 	 */
-	do_action( 'give_payment_receipt_before', $payment, $give_receipt_args );
+	do_action('give_payment_receipt_before', $payment, $give_receipt_args);
 	?>
 
-	<?php foreach ( $give_receipt_args['donation_receipt'] as $receipt_item ) { ?>
-		<?php if ( filter_var( $receipt_item['display'], FILTER_VALIDATE_BOOLEAN ) ) : ?>
+	<?php foreach ($give_receipt_args['donation_receipt'] as $receipt_item) { ?>
+		<?php if (filter_var($receipt_item['display'], FILTER_VALIDATE_BOOLEAN)) : ?>
 			<tr>
 				<td scope="row"><strong><?php echo $receipt_item['name']; ?></strong></td>
 				<td><?php echo $receipt_item['value']; ?></td>
@@ -302,7 +302,7 @@ 
 	 * @param object $payment           The payment object.
 	 * @param array  $give_receipt_args Receipt_argument.
 	 */
-	do_action( 'give_payment_receipt_after', $payment, $give_receipt_args );
+	do_action('give_payment_receipt_after', $payment, $give_receipt_args);
 	?>
 	</tbody>
 </table>
@@ -318,5 +318,5 @@ 
  * @param object $payment           The payment object.
  * @param array  $give_receipt_args Receipt_argument.
  */
-do_action( 'give_payment_receipt_after_table', $payment, $give_receipt_args );
+do_action('give_payment_receipt_after_table', $payment, $give_receipt_args);
 ?>

includes/emails/functions.php

Spacing +74 added lines, -74 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -26,14 +26,14 @@ 
  *
  * @return void
  */
-function give_email_donation_receipt( $payment_id, $admin_notice = true ) {
+function give_email_donation_receipt($payment_id, $admin_notice = true) {
 	/**
 	 * Fire the action
 	 */
-	do_action( 'give_donation-receipt_email_notification', $payment_id );
+	do_action('give_donation-receipt_email_notification', $payment_id);
 
 	// If admin notifications are on, send the admin notice.
-	if ( $admin_notice && give_is_setting_enabled( Give_Email_Notification::get_instance('new-donation' )->get_notification_status() ) ) {
+	if ($admin_notice && give_is_setting_enabled(Give_Email_Notification::get_instance('new-donation')->get_notification_status())) {
 		/**
 		 * Fires in the donation email receipt.
 		 *
@@ -44,7 +44,7 @@ 
 		 * @param int   $payment_id   Payment id.
 		 * @param mixed $payment_data Payment meta data.
 		 */
-		do_action( 'give_new-donation_email_notification', $payment_id, give_get_payment_meta( $payment_id ) );
+		do_action('give_new-donation_email_notification', $payment_id, give_get_payment_meta($payment_id));
 	}
 }
 
@@ -57,7 +57,7 @@ 
  *
  * @return void
  */
-function give_admin_email_notice( $payment_id ) {
+function give_admin_email_notice($payment_id) {
 	/**
 	 * Fires in the donation email receipt.
 	 *
@@ -68,10 +68,10 @@ 
 	 * @param int   $payment_id   Payment id.
 	 * @param mixed $payment_data Payment meta data.
 	 */
-	do_action( 'give_new-donation_email_notification', $payment_id );
+	do_action('give_new-donation_email_notification', $payment_id);
 }
 
-add_action( 'give_admin_donation_email', 'give_admin_email_notice' );
+add_action('give_admin_donation_email', 'give_admin_email_notice');
 
 
 /**
@@ -84,16 +84,16 @@ 
  */
 function give_get_default_donation_notification_email() {
 
-	$default_email_body = __( 'Hi there,', 'give' ) . "\n\n";
-	$default_email_body .= __( 'This email is to inform you that a new donation has been made on your website:', 'give' ) . ' {site_url}' . ".\n\n";
-	$default_email_body .= '<strong>' . __( 'Donor:', 'give' ) . '</strong> {name}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Donation:', 'give' ) . '</strong> {donation}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Amount:', 'give' ) . '</strong> {amount}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Payment Method:', 'give' ) . '</strong> {payment_method}' . "\n\n";
-	$default_email_body .= __( 'Thank you,', 'give' ) . "\n\n";
-	$default_email_body .= '{sitename}' . "\n";
+	$default_email_body = __('Hi there,', 'give')."\n\n";
+	$default_email_body .= __('This email is to inform you that a new donation has been made on your website:', 'give').' {site_url}'.".\n\n";
+	$default_email_body .= '<strong>'.__('Donor:', 'give').'</strong> {name}'."\n";
+	$default_email_body .= '<strong>'.__('Donation:', 'give').'</strong> {donation}'."\n";
+	$default_email_body .= '<strong>'.__('Amount:', 'give').'</strong> {amount}'."\n";
+	$default_email_body .= '<strong>'.__('Payment Method:', 'give').'</strong> {payment_method}'."\n\n";
+	$default_email_body .= __('Thank you,', 'give')."\n\n";
+	$default_email_body .= '{sitename}'."\n";
 
-	return apply_filters( 'give_default_donation_notification_email', $default_email_body );
+	return apply_filters('give_default_donation_notification_email', $default_email_body);
 }
 
 
@@ -107,21 +107,21 @@ 
  */
 function give_get_default_donation_receipt_email() {
 
-	$default_email_body = __( 'Dear', 'give' ) . " {name},\n\n";
-	$default_email_body .= __( 'Thank you for your donation. Your generosity is appreciated! Here are the details of your donation:', 'give' ) . "\n\n";
-	$default_email_body .= '<strong>' . __( 'Donor:', 'give' ) . '</strong> {fullname}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Donation:', 'give' ) . '</strong> {donation}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Donation Date:', 'give' ) . '</strong> {date}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Amount:', 'give' ) . '</strong> {amount}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Payment Method:', 'give' ) . '</strong> {payment_method}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Payment ID:', 'give' ) . '</strong> {payment_id}' . "\n";
-	$default_email_body .= '<strong>' . __( 'Receipt ID:', 'give' ) . '</strong> {receipt_id}' . "\n\n";
-	$default_email_body .= '{receipt_link}' . "\n\n";
+	$default_email_body = __('Dear', 'give')." {name},\n\n";
+	$default_email_body .= __('Thank you for your donation. Your generosity is appreciated! Here are the details of your donation:', 'give')."\n\n";
+	$default_email_body .= '<strong>'.__('Donor:', 'give').'</strong> {fullname}'."\n";
+	$default_email_body .= '<strong>'.__('Donation:', 'give').'</strong> {donation}'."\n";
+	$default_email_body .= '<strong>'.__('Donation Date:', 'give').'</strong> {date}'."\n";
+	$default_email_body .= '<strong>'.__('Amount:', 'give').'</strong> {amount}'."\n";
+	$default_email_body .= '<strong>'.__('Payment Method:', 'give').'</strong> {payment_method}'."\n";
+	$default_email_body .= '<strong>'.__('Payment ID:', 'give').'</strong> {payment_id}'."\n";
+	$default_email_body .= '<strong>'.__('Receipt ID:', 'give').'</strong> {receipt_id}'."\n\n";
+	$default_email_body .= '{receipt_link}'."\n\n";
 	$default_email_body .= "\n\n";
-	$default_email_body .= __( 'Sincerely,', 'give' ) . "\n";
-	$default_email_body .= '{sitename}' . "\n";
+	$default_email_body .= __('Sincerely,', 'give')."\n";
+	$default_email_body .= '{sitename}'."\n";
 
-	return apply_filters( 'give_default_donation_receipt_email', $default_email_body );
+	return apply_filters('give_default_donation_receipt_email', $default_email_body);
 }
 
 /**
@@ -134,22 +134,22 @@ 
  *
  * @return array $email_names
  */
-function give_get_email_names( $user_info, $payment = false ) {
+function give_get_email_names($user_info, $payment = false) {
 	$email_names = array();
 
-	if ( is_a( $payment, 'Give_Payment' ) ) {
+	if (is_a($payment, 'Give_Payment')) {
 
-		if ( $payment->user_id > 0 ) {
+		if ($payment->user_id > 0) {
 
-			$user_data               = get_userdata( $payment->user_id );
+			$user_data               = get_userdata($payment->user_id);
 			$email_names['name']     = $payment->first_name;
-			$email_names['fullname'] = trim( $payment->first_name . ' ' . $payment->last_name );
+			$email_names['fullname'] = trim($payment->first_name.' '.$payment->last_name);
 			$email_names['username'] = $user_data->user_login;
 
-		} elseif ( ! empty( $payment->first_name ) ) {
+		} elseif ( ! empty($payment->first_name)) {
 
 			$email_names['name']     = $payment->first_name;
-			$email_names['fullname'] = trim( $payment->first_name . ' ' . $payment->last_name );
+			$email_names['fullname'] = trim($payment->first_name.' '.$payment->last_name);
 			$email_names['username'] = $payment->first_name;
 
 		} else {
@@ -161,29 +161,29 @@ 
 	} else {
 
 		// Support for old serialized data.
-		if ( is_serialized( $user_info ) ) {
+		if (is_serialized($user_info)) {
 
 			// Security check.
-			preg_match( '/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $user_info, $matches );
-			if ( ! empty( $matches ) ) {
+			preg_match('/[oO]\s*:\s*\d+\s*:\s*"\s*(?!(?i)(stdClass))/', $user_info, $matches);
+			if ( ! empty($matches)) {
 				return array(
 					'name'     => '',
 					'fullname' => '',
 					'username' => '',
 				);
 			} else {
-				$user_info = maybe_unserialize( $user_info );
+				$user_info = maybe_unserialize($user_info);
 			}
 		}
 
-		if ( isset( $user_info['id'] ) && $user_info['id'] > 0 && isset( $user_info['first_name'] ) ) {
-			$user_data               = get_userdata( $user_info['id'] );
+		if (isset($user_info['id']) && $user_info['id'] > 0 && isset($user_info['first_name'])) {
+			$user_data               = get_userdata($user_info['id']);
 			$email_names['name']     = $user_info['first_name'];
-			$email_names['fullname'] = $user_info['first_name'] . ' ' . $user_info['last_name'];
+			$email_names['fullname'] = $user_info['first_name'].' '.$user_info['last_name'];
 			$email_names['username'] = $user_data->user_login;
-		} elseif ( isset( $user_info['first_name'] ) ) {
+		} elseif (isset($user_info['first_name'])) {
 			$email_names['name']     = $user_info['first_name'];
-			$email_names['fullname'] = $user_info['first_name'] . ' ' . $user_info['last_name'];
+			$email_names['fullname'] = $user_info['first_name'].' '.$user_info['last_name'];
 			$email_names['username'] = $user_info['first_name'];
 		} else {
 			$email_names['name']     = $user_info['email'];
@@ -192,13 +192,13 @@ 
 	} // End if().
 
 	// Set title prefix to name, if non empty.
-	if ( ! empty( $user_info['title'] ) && ! empty( $user_info['last_name'] ) ) {
-		$email_names['name'] = give_get_donor_name_with_title_prefixes( $user_info['title'], $user_info['last_name'] );
+	if ( ! empty($user_info['title']) && ! empty($user_info['last_name'])) {
+		$email_names['name'] = give_get_donor_name_with_title_prefixes($user_info['title'], $user_info['last_name']);
 	}
 
 	// Set title prefix to fullname, if non empty.
-	if ( ! empty( $user_info['title'] ) && ! empty( $email_names['fullname'] ) ) {
-		$email_names['fullname'] = give_get_donor_name_with_title_prefixes( $user_info['title'], $email_names['fullname'] );
+	if ( ! empty($user_info['title']) && ! empty($email_names['fullname'])) {
+		$email_names['fullname'] = give_get_donor_name_with_title_prefixes($user_info['title'], $email_names['fullname']);
 	}
 
 	return $email_names;
@@ -212,37 +212,37 @@ 
  *
  * @since 1.8.14
  */
-function give_admin_email_user_donor_disconnection( $user_id, $donor_id ) {
+function give_admin_email_user_donor_disconnection($user_id, $donor_id) {
 
-	$user_id  = absint( $user_id );
-	$donor_id = absint( $donor_id );
+	$user_id  = absint($user_id);
+	$donor_id = absint($donor_id);
 
 	// Bail Out, if user id doesn't exists.
-	if ( empty( $user_id ) ) {
+	if (empty($user_id)) {
 		return;
 	}
 
 	// Bail Out, if donor id doesn't exists.
-	if ( empty( $donor_id ) ) {
+	if (empty($donor_id)) {
 		return;
 	}
 
-	$from_name = give_get_option( 'from_name', wp_specialchars_decode( get_bloginfo( 'name' ), ENT_QUOTES ) );
+	$from_name = give_get_option('from_name', wp_specialchars_decode(get_bloginfo('name'), ENT_QUOTES));
 
-	$from_email = give_get_option( 'from_email', get_bloginfo( 'admin_email' ) );
+	$from_email = give_get_option('from_email', get_bloginfo('admin_email'));
 
 	/* translators: %s: payment id */
-	$subject = __( 'Attention: User tries to login whose Donor profile is disconnected!', 'give' );
+	$subject = __('Attention: User tries to login whose Donor profile is disconnected!', 'give');
 
 	/**
 	 * Filters the Donor-User Disconnection notification subject.
 	 *
 	 * @since 1.8.14
 	 */
-	$subject = apply_filters( 'give_admin_donor_user_disconnection_notification_subject', wp_strip_all_tags( $subject ) );
+	$subject = apply_filters('give_admin_donor_user_disconnection_notification_subject', wp_strip_all_tags($subject));
 
-	$headers = "From: " . stripslashes_deep( html_entity_decode( $from_name, ENT_COMPAT, 'UTF-8' ) ) . " <$from_email>\r\n";
-	$headers .= "Reply-To: " . $from_email . "\r\n";
+	$headers = "From: ".stripslashes_deep(html_entity_decode($from_name, ENT_COMPAT, 'UTF-8'))." <$from_email>\r\n";
+	$headers .= "Reply-To: ".$from_email."\r\n";
 	$headers .= "Content-Type: text/html; charset=utf-8\r\n";
 
 	/**
@@ -250,25 +250,25 @@ 
 	 *
 	 * @since 1.8.14
 	 */
-	$headers = apply_filters( 'give_admin_donor_user_disconnection_notification_headers', $headers );
+	$headers = apply_filters('give_admin_donor_user_disconnection_notification_headers', $headers);
 
-	$message = __( 'Hi Admin,', 'give' ) . "\n\n";
-	$message .= __( 'This email is to inform you that a user has tried logging in. But, User was unable to login due to User-Donor profile disconnection.', 'give' ) . "\n\n";
-	$message .= __( 'Do you want to reconnect User and Donor profile again?', 'give' ) . "\n\n";
+	$message = __('Hi Admin,', 'give')."\n\n";
+	$message .= __('This email is to inform you that a user has tried logging in. But, User was unable to login due to User-Donor profile disconnection.', 'give')."\n\n";
+	$message .= __('Do you want to reconnect User and Donor profile again?', 'give')."\n\n";
 	$message .= sprintf(
 		'<a href="%1$s">%2$s</a>',
-		esc_url( admin_url() . 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor_id . '&user_id=' . $user_id . '&give-messages[]=reconnect-user' ),
-		__( 'Reconnect User', 'give' ) . "\n\n"
+		esc_url(admin_url().'edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor_id.'&user_id='.$user_id.'&give-messages[]=reconnect-user'),
+		__('Reconnect User', 'give')."\n\n"
 	);
-	$message .= __( 'Thank you,', 'give' ) . "\n\n";
-	$message .= '{sitename}' . "\n";
+	$message .= __('Thank you,', 'give')."\n\n";
+	$message .= '{sitename}'."\n";
 
 	$emails = Give()->emails;
-	$emails->__set( 'from_name', $from_name );
-	$emails->__set( 'from_email', $from_email );
-	$emails->__set( 'headers', $headers );
-	$emails->__set( 'heading', __( 'User - Donor Profile Disconnection', 'give' ) );
+	$emails->__set('from_name', $from_name);
+	$emails->__set('from_email', $from_email);
+	$emails->__set('headers', $headers);
+	$emails->__set('heading', __('User - Donor Profile Disconnection', 'give'));
 
-	$emails->send( give_get_admin_notice_emails(), $subject, give_do_email_tags( $message ) );
+	$emails->send(give_get_admin_notice_emails(), $subject, give_do_email_tags($message));
 
 }

includes/filters.php

Spacing +54 added lines, -56 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -26,34 +26,34 @@ 
  *
  * @return void
  */
-function give_set_settings_with_disable_prefix( $old_settings, $settings ) {
+function give_set_settings_with_disable_prefix($old_settings, $settings) {
 	// Bailout.
-	if ( ! function_exists( 'give_v18_renamed_core_settings' ) ) {
+	if ( ! function_exists('give_v18_renamed_core_settings')) {
 		return;
 	}
 
 	// Get old setting names.
-	$old_settings   = array_flip( give_v18_renamed_core_settings() );
+	$old_settings   = array_flip(give_v18_renamed_core_settings());
 	$update_setting = false;
 
-	foreach ( $settings as $key => $value ) {
+	foreach ($settings as $key => $value) {
 
 		// Check 1. Check if new option is really updated or not.
 		// Check 2. Continue if key is not renamed.
-		if ( ! isset( $old_settings[ $key ] ) ) {
+		if ( ! isset($old_settings[$key])) {
 			continue;
 		}
 
 		// Set old setting.
-		$settings[ $old_settings[ $key ] ] = 'on';
+		$settings[$old_settings[$key]] = 'on';
 
 		// Do not need to set old setting if new setting is not set.
 		if (
-			( give_is_setting_enabled( $value ) && ( false !== strpos( $old_settings[ $key ], 'disable_' ) ) )
-			|| ( ! give_is_setting_enabled( $value ) && ( false !== strpos( $old_settings[ $key ], 'enable_' ) ) )
+			(give_is_setting_enabled($value) && (false !== strpos($old_settings[$key], 'disable_')))
+			|| ( ! give_is_setting_enabled($value) && (false !== strpos($old_settings[$key], 'enable_')))
 
 		) {
-			unset( $settings[ $old_settings[ $key ] ] );
+			unset($settings[$old_settings[$key]]);
 		}
 
 		// Tell bot to update setting.
@@ -61,12 +61,12 @@ 
 	}
 
 	// Update setting if any old setting set.
-	if ( $update_setting ) {
-		update_option( 'give_settings', $settings, false );
+	if ($update_setting) {
+		update_option('give_settings', $settings, false);
 	}
 }
 
-add_action( 'update_option_give_settings', 'give_set_settings_with_disable_prefix', 10, 2 );
+add_action('update_option_give_settings', 'give_set_settings_with_disable_prefix', 10, 2);
 
 /**
  * Check spam through Akismet.
@@ -80,45 +80,45 @@ 
  *
  * @return bool|mixed
  */
-function give_akismet( $spam ) {
+function give_akismet($spam) {
 
 	// Bail out, If spam.
-	if ( $spam ) {
+	if ($spam) {
 		return $spam;
 	}
 
 	// Bail out, if Akismet key not exist.
-	if ( ! give_check_akismet_key() ) {
+	if ( ! give_check_akismet_key()) {
 		return false;
 	}
 
 	// Build args array.
 	$args = array();
 
-	$args['comment_author']       = isset( $_POST['give_first'] ) ? strip_tags( trim( $_POST['give_first'] ) ) : '';
-	$args['comment_author_email'] = isset( $_POST['give_email'] ) ? $_POST['give_email'] : false;
-	$args['blog']                 = get_option( 'home' );
+	$args['comment_author']       = isset($_POST['give_first']) ? strip_tags(trim($_POST['give_first'])) : '';
+	$args['comment_author_email'] = isset($_POST['give_email']) ? $_POST['give_email'] : false;
+	$args['blog']                 = get_option('home');
 	$args['blog_lang']            = get_locale();
-	$args['blog_charset']         = get_option( 'blog_charset' );
+	$args['blog_charset']         = get_option('blog_charset');
 	$args['user_ip']              = $_SERVER['REMOTE_ADDR'];
 	$args['user_agent']           = $_SERVER['HTTP_USER_AGENT'];
 	$args['referrer']             = $_SERVER['HTTP_REFERER'];
 	$args['comment_type']         = 'contact-form';
 
-	$ignore = array( 'HTTP_COOKIE', 'HTTP_COOKIE2', 'PHP_AUTH_PW' );
+	$ignore = array('HTTP_COOKIE', 'HTTP_COOKIE2', 'PHP_AUTH_PW');
 
-	foreach ( $_SERVER as $key => $value ) {
-		if ( ! in_array( $key, (array) $ignore ) ) {
+	foreach ($_SERVER as $key => $value) {
+		if ( ! in_array($key, (array) $ignore)) {
 			$args["$key"] = $value;
 		}
 	}
 
 	// It will return Akismet spam detect API response.
-	return give_akismet_spam_check( $args );
+	return give_akismet_spam_check($args);
 
 }
 
-add_filter( 'give_spam', 'give_akismet' );
+add_filter('give_spam', 'give_akismet');
 
 /**
  * Check Akismet API Key.
@@ -128,11 +128,11 @@ 
  * @return bool
  */
 function give_check_akismet_key() {
-	if ( is_callable( array( 'Akismet', 'get_api_key' ) ) ) { // Akismet v3.0+
+	if (is_callable(array('Akismet', 'get_api_key'))) { // Akismet v3.0+
 		return (bool) Akismet::get_api_key();
 	}
 
-	if ( function_exists( 'akismet_get_key' ) ) {
+	if (function_exists('akismet_get_key')) {
 		return (bool) akismet_get_key();
 	}
 
@@ -148,26 +148,26 @@ 
  *
  * @return bool|mixed
  */
-function give_akismet_spam_check( $args ) {
+function give_akismet_spam_check($args) {
 	global $akismet_api_host, $akismet_api_port;
 
 	$spam         = false;
-	$query_string = http_build_query( $args );
+	$query_string = http_build_query($args);
 
-	if ( is_callable( array( 'Akismet', 'http_post' ) ) ) { // Akismet v3.0+
-		$response = Akismet::http_post( $query_string, 'comment-check' );
+	if (is_callable(array('Akismet', 'http_post'))) { // Akismet v3.0+
+		$response = Akismet::http_post($query_string, 'comment-check');
 	} else {
-		$response = akismet_http_post( $query_string, $akismet_api_host,
-			'/1.1/comment-check', $akismet_api_port );
+		$response = akismet_http_post($query_string, $akismet_api_host,
+			'/1.1/comment-check', $akismet_api_port);
 	}
 
 	// It's spam if response status is true.
-	if ( 'true' === $response[1] ) {
+	if ('true' === $response[1]) {
 		$spam = true;
 	}
 
 	// Allow developer to modified Akismet spam detection response.
-	return apply_filters( 'give_akismet_spam_check', $spam, $args );
+	return apply_filters('give_akismet_spam_check', $spam, $args);
 }
 
 /**
@@ -180,15 +180,15 @@ 
  *
  * @return array
  */
-function give_bc_v1817_iranian_currency_code( $currencies ) {
-	if ( ! give_has_upgrade_completed( 'v1817_update_donation_iranian_currency_code' ) ) {
+function give_bc_v1817_iranian_currency_code($currencies) {
+	if ( ! give_has_upgrade_completed('v1817_update_donation_iranian_currency_code')) {
 		$currencies['RIAL'] = $currencies['IRR'];
 	}
 
 	return $currencies;
 }
 
-add_filter( 'give_currencies', 'give_bc_v1817_iranian_currency_code', 0 );
+add_filter('give_currencies', 'give_bc_v1817_iranian_currency_code', 0);
 
 
 /**
@@ -202,25 +202,23 @@ 
  *
  * @return string
  */
-function give_format_price_for_right_to_left_supported_currency( $formatted_amount, $currency_args, $price ) {
-	if ( ! give_is_right_to_left_supported_currency( $currency_args['currency_code'] ) ) {
+function give_format_price_for_right_to_left_supported_currency($formatted_amount, $currency_args, $price) {
+	if ( ! give_is_right_to_left_supported_currency($currency_args['currency_code'])) {
 		return $formatted_amount;
 	}
 
 	$formatted_amount = (
 	'before' === (string) $currency_args['position'] ?
-		'‫' . $price . $currency_args['symbol'] . '‬' :
-		'‪' . $price . $currency_args['symbol'] . '‬'
+		'‫'.$price.$currency_args['symbol'].'‬' : '‪'.$price.$currency_args['symbol'].'‬'
 	);
 
 	$formatted_amount = $currency_args['decode_currency'] ?
-		html_entity_decode( $formatted_amount, ENT_COMPAT, 'UTF-8' ) :
-		$formatted_amount;
+		html_entity_decode($formatted_amount, ENT_COMPAT, 'UTF-8') : $formatted_amount;
 
 	return $formatted_amount;
 }
 
-add_filter( 'give_currency_filter', 'give_format_price_for_right_to_left_supported_currency', 10, 3 );
+add_filter('give_currency_filter', 'give_format_price_for_right_to_left_supported_currency', 10, 3);
 
 /**
  * Validate active gateway value before returning result.
@@ -231,31 +229,31 @@ 
  *
  * @return array
  */
-function __give_validate_active_gateways( $value ) {
-	$gateways = array_keys( give_get_payment_gateways() );
-	$active_gateways = is_array( $value ) ? array_keys( $value ) : array();
+function __give_validate_active_gateways($value) {
+	$gateways = array_keys(give_get_payment_gateways());
+	$active_gateways = is_array($value) ? array_keys($value) : array();
 
 	// Remove deactivated payment gateways.
-	if( ! empty( $active_gateways ) ) {
-		foreach ( $active_gateways as $index => $gateway_id ) {
-			if( ! in_array( $gateway_id, $gateways ) ) {
-				unset( $value[$gateway_id] );
+	if ( ! empty($active_gateways)) {
+		foreach ($active_gateways as $index => $gateway_id) {
+			if ( ! in_array($gateway_id, $gateways)) {
+				unset($value[$gateway_id]);
 			}
 		}
 	}
 
-	if ( empty( $value ) ) {
+	if (empty($value)) {
 		/**
 		 * Filter the default active gateway
 		 *
 		 * @since 2.1.0
 		 */
-		$value = apply_filters( 'give_default_active_gateways', array(
+		$value = apply_filters('give_default_active_gateways', array(
 			'manual' => 1,
-		) );
+		));
 	}
 
 	return $value;
 }
 
-add_filter( 'give_get_option_gateways', '__give_validate_active_gateways', 10, 1 );
+add_filter('give_get_option_gateways', '__give_validate_active_gateways', 10, 1);

includes/gateways/actions.php

Spacing +28 added lines, -28 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,14 +21,14 @@ 
  *
  * @param $data
  */
-function give_process_gateway_select( $data ) {
-	if ( isset( $_POST['gateway_submit'] ) ) {
-		wp_redirect( esc_url( add_query_arg( 'payment-mode', $_POST['payment-mode'] ) ) );
+function give_process_gateway_select($data) {
+	if (isset($_POST['gateway_submit'])) {
+		wp_redirect(esc_url(add_query_arg('payment-mode', $_POST['payment-mode'])));
 		exit;
 	}
 }
 
-add_action( 'give_gateway_select', 'give_process_gateway_select' );
+add_action('give_gateway_select', 'give_process_gateway_select');
 
 /**
  * Loads a payment gateway via AJAX.
@@ -39,11 +39,11 @@ 
  */
 function give_load_ajax_gateway() {
 
-	$post_data = give_clean( $_POST ); // WPCS: input var ok, CSRF ok.
+	$post_data = give_clean($_POST); // WPCS: input var ok, CSRF ok.
 
-	if ( isset( $post_data['give_payment_mode'] ) ) {
+	if (isset($post_data['give_payment_mode'])) {
 
-		$form_id_prefix = ! empty( $post_data['give_form_id_prefix'] ) ? $post_data['give_form_id_prefix'] : '';
+		$form_id_prefix = ! empty($post_data['give_form_id_prefix']) ? $post_data['give_form_id_prefix'] : '';
 
 		$args = array(
 			'id_prefix' => $form_id_prefix,
@@ -54,14 +54,14 @@ 
 		 *
 		 * @since 1.7
 		 */
-		do_action( 'give_donation_form', $post_data['give_form_id'], $args );
+		do_action('give_donation_form', $post_data['give_form_id'], $args);
 
 		exit();
 	}
 }
 
-add_action( 'wp_ajax_give_load_gateway', 'give_load_ajax_gateway' );
-add_action( 'wp_ajax_nopriv_give_load_gateway', 'give_load_ajax_gateway' );
+add_action('wp_ajax_give_load_gateway', 'give_load_ajax_gateway');
+add_action('wp_ajax_nopriv_give_load_gateway', 'give_load_ajax_gateway');
 
 /**
  * Create wp nonce using Ajax call.
@@ -73,18 +73,18 @@ 
  * @return void
  */
 function give_donation_form_nonce() {
-	if ( isset( $_POST['give_form_id'] ) ) {
+	if (isset($_POST['give_form_id'])) {
 
 		// Get donation form id.
-		$form_id = is_numeric( $_POST['give_form_id'] ) ? absint( $_POST['give_form_id'] ) : 0;
+		$form_id = is_numeric($_POST['give_form_id']) ? absint($_POST['give_form_id']) : 0;
 
 		// Send nonce json data.
-		wp_send_json_success( wp_create_nonce( "give_donation_form_nonce_{$form_id}" ) );
+		wp_send_json_success(wp_create_nonce("give_donation_form_nonce_{$form_id}"));
 	}
 }
 
-add_action( 'wp_ajax_give_donation_form_nonce', 'give_donation_form_nonce' );
-add_action( 'wp_ajax_nopriv_give_donation_form_nonce', 'give_donation_form_nonce' );
+add_action('wp_ajax_give_donation_form_nonce', 'give_donation_form_nonce');
+add_action('wp_ajax_nopriv_give_donation_form_nonce', 'give_donation_form_nonce');
 
 
 /**
@@ -96,14 +96,14 @@ 
  * @return void
  */
 function __give_donation_form_reset_all_nonce() {
-	if ( isset( $_POST['give_form_id'] ) ) {
+	if (isset($_POST['give_form_id'])) {
 
 		// Get donation form id.
-		$form_id = is_numeric( $_POST['give_form_id'] ) ? absint( $_POST['give_form_id'] ) : 0;
+		$form_id = is_numeric($_POST['give_form_id']) ? absint($_POST['give_form_id']) : 0;
 
 		$data = array(
-			'give_form_hash'               => wp_create_nonce( "give_donation_form_nonce_{$form_id}" ),
-			'give_form_user_register_hash' => wp_create_nonce( "give_form_create_user_nonce_{$form_id}" )
+			'give_form_hash'               => wp_create_nonce("give_donation_form_nonce_{$form_id}"),
+			'give_form_user_register_hash' => wp_create_nonce("give_form_create_user_nonce_{$form_id}")
 		);
 
 		/**
@@ -112,15 +112,15 @@ 
 		 * @since  2.2.0
 		 *
 		 */
-		$data = apply_filters( 'give_donation_form_reset_all_nonce_data', $data );
+		$data = apply_filters('give_donation_form_reset_all_nonce_data', $data);
 
 		// Send nonce json data.
-		wp_send_json_success( $data );
+		wp_send_json_success($data);
 	}
 }
 
-add_action( 'wp_ajax_give_donation_form_reset_all_nonce', '__give_donation_form_reset_all_nonce' );
-add_action( 'wp_ajax_nopriv_give_donation_form_reset_all_nonce', '__give_donation_form_reset_all_nonce' );
+add_action('wp_ajax_give_donation_form_reset_all_nonce', '__give_donation_form_reset_all_nonce');
+add_action('wp_ajax_nopriv_give_donation_form_reset_all_nonce', '__give_donation_form_reset_all_nonce');
 
 /**
  * Sets an error within the donation form if no gateways are enabled.
@@ -133,11 +133,11 @@ 
 function give_no_gateway_error() {
 	$gateways = give_get_enabled_payment_gateways();
 
-	if ( empty( $gateways ) ) {
-		give_set_error( 'no_gateways', esc_html__( 'You must enable a payment gateway to use Give.', 'give' ) );
+	if (empty($gateways)) {
+		give_set_error('no_gateways', esc_html__('You must enable a payment gateway to use Give.', 'give'));
 	} else {
-		give_unset_error( 'no_gateways' );
+		give_unset_error('no_gateways');
 	}
 }
 
-add_action( 'init', 'give_no_gateway_error' );
+add_action('init', 'give_no_gateway_error');

includes/class-give-db-sessions.php

Spacing +18 added lines, -18 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -54,8 +54,8 @@ 
 		// Set cache group id.
 		$current_blog_id        = get_current_blog_id();
 		$this->incrementer_name = "give-cache-incrementer-sessions-{$current_blog_id}";
-		$incrementer_value      = wp_cache_get( $this->incrementer_name );
-		$incrementer_value      = ! empty( $incrementer_value ) ? $incrementer_value : microtime( true );
+		$incrementer_value      = wp_cache_get($this->incrementer_name);
+		$incrementer_value      = ! empty($incrementer_value) ? $incrementer_value : microtime(true);
 		$this->cache_group      = "{$this->cache_group}_{$current_blog_id}_{$incrementer_value}";
 
 		$this->register_table();
@@ -101,10 +101,10 @@ 
   				UNIQUE KEY session_id (session_id)
 			) {$charset_collate};";
 
-		require_once ABSPATH . 'wp-admin/includes/upgrade.php';
-		dbDelta( $sql );
+		require_once ABSPATH.'wp-admin/includes/upgrade.php';
+		dbDelta($sql);
 
-		update_option( $this->table_name . '_db_version', $this->version, false );
+		update_option($this->table_name.'_db_version', $this->version, false);
 	}
 
 
@@ -118,14 +118,14 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function get_session( $donor_id, $default = false ) {
+	public function get_session($donor_id, $default = false) {
 		global $wpdb;
 
-		if ( defined( 'WP_SETUP_CONFIG' ) ) {
+		if (defined('WP_SETUP_CONFIG')) {
 			return false;
 		}
 
-		if ( ! ( $value = wp_cache_get( $donor_id, $this->cache_group ) ) ) { // @codingStandardsIgnoreLine
+		if ( ! ($value = wp_cache_get($donor_id, $this->cache_group))) { // @codingStandardsIgnoreLine
 
 			// @codingStandardsIgnoreStart
 			$value = $wpdb->get_var(
@@ -140,14 +140,14 @@ 
 			);
 			// @codingStandardsIgnoreEnd
 
-			if ( is_null( $value ) ) {
+			if (is_null($value)) {
 				$value = $default;
 			}
 
-			wp_cache_add( $donor_id, $value, $this->cache_group );
+			wp_cache_add($donor_id, $value, $this->cache_group);
 		}
 
-		return maybe_unserialize( $value );
+		return maybe_unserialize($value);
 	}
 
 	/**
@@ -156,7 +156,7 @@ 
 	 * @param string $donor_id  Donor ID.
 	 * @param int    $timestamp Timestamp to expire the cookie.
 	 */
-	public function update_session_timestamp( $donor_id, $timestamp ) {
+	public function update_session_timestamp($donor_id, $timestamp) {
 		global $wpdb;
 
 		// @codingStandardsIgnoreStart.
@@ -183,10 +183,10 @@ 
 	 *
 	 * @param int $donor_id Customer ID.
 	 */
-	public function delete_session( $donor_id ) {
+	public function delete_session($donor_id) {
 		global $wpdb;
 
-		wp_cache_delete( $donor_id, $this->cache_group );
+		wp_cache_delete($donor_id, $this->cache_group);
 
 		// @codingStandardsIgnoreStart
 		$wpdb->delete(
@@ -209,7 +209,7 @@ 
 	public function delete_expired_sessions() {
 		global $wpdb;
 
-		wp_cache_set( $this->incrementer_name, microtime( true ) );
+		wp_cache_set($this->incrementer_name, microtime(true));
 
 		// @codingStandardsIgnoreStart
 		$wpdb->query(
@@ -232,10 +232,10 @@ 
 	 * @param array  $data       Data.
 	 * @param array  $format     Array for data format of each key:value in data.
 	 */
-	public function __replace( $table_name, $data, $format = null ) {
+	public function __replace($table_name, $data, $format = null) {
 		global $wpdb;
 
-		wp_cache_set( $data['session_key'], $data['session_value'], $this->cache_group, $data['session_expiry'] - time() );
+		wp_cache_set($data['session_key'], $data['session_value'], $this->cache_group, $data['session_expiry'] - time());
 
 
 		// @codingStandardsIgnoreStart

includes/class-give-session.php

Spacing +67 added lines, -67 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -135,7 +135,7 @@ 
 	 * @return Give_Session
 	 */
 	public static function get_instance() {
-		if ( null === static::$instance ) {
+		if (null === static::$instance) {
 			self::$instance = new static();
 			self::$instance->__setup();
 		}
@@ -150,25 +150,25 @@ 
 	 * @access public
 	 */
 	private function __setup() {  // @codingStandardsIgnoreLine
-		$this->exp_option = give_get_option( 'session_lifetime' );
-		$this->exp_option = ! empty( $this->exp_option )
+		$this->exp_option = give_get_option('session_lifetime');
+		$this->exp_option = ! empty($this->exp_option)
 			? $this->exp_option
 			: 30 * 60 * 24; // Default expiration time is 12 hours
 
 		$this->set_cookie_name();
-		$this->cookie_name = $this->get_cookie_name( 'session' );
+		$this->cookie_name = $this->get_cookie_name('session');
 		$cookie            = $this->get_session_cookie();
 
-		if ( ! empty( $cookie ) ) {
+		if ( ! empty($cookie)) {
 			$this->donor_id           = $cookie[0];
 			$this->session_expiration = $cookie[1];
 			$this->session_expiring   = $cookie[2];
 			$this->has_cookie         = true;
 
 			// Update session if its close to expiring.
-			if ( time() > $this->session_expiring ) {
+			if (time() > $this->session_expiring) {
 				$this->set_expiration_time();
-				Give()->session_db->update_session_timestamp( $this->donor_id, $this->session_expiration );
+				Give()->session_db->update_session_timestamp($this->donor_id, $this->session_expiration);
 			}
 
 			// Load session.
@@ -178,17 +178,17 @@ 
 			$this->generate_donor_id();
 		}
 
-		add_action( 'give_process_donation_after_validation', array( $this, 'maybe_start_session' ) );
+		add_action('give_process_donation_after_validation', array($this, 'maybe_start_session'));
 
-		add_action( 'shutdown', array( $this, 'save_data' ), 20 );
-		add_action( 'wp_logout', array( $this, 'destroy_session' ) );
+		add_action('shutdown', array($this, 'save_data'), 20);
+		add_action('wp_logout', array($this, 'destroy_session'));
 
-		if ( ! is_user_logged_in() ) {
-			add_filter( 'nonce_user_logged_out', array( $this, '__nonce_user_logged_out' ) );
+		if ( ! is_user_logged_in()) {
+			add_filter('nonce_user_logged_out', array($this, '__nonce_user_logged_out'));
 		}
 
 		// Remove old sessions.
-		Give_Cron::add_daily_event( array( $this, '__cleanup_sessions' ) );
+		Give_Cron::add_daily_event(array($this, '__cleanup_sessions'));
 	}
 
 	/**
@@ -200,7 +200,7 @@ 
 	 * @return array
 	 */
 	public function get_session_data() {
-		return $this->has_session() ? (array) Give()->session_db->get_session( $this->donor_id, array() ) : array();
+		return $this->has_session() ? (array) Give()->session_db->get_session($this->donor_id, array()) : array();
 	}
 
 
@@ -214,27 +214,27 @@ 
 	 */
 	public function get_session_cookie() {
 		$session      = array();
-		$cookie_value = isset( $_COOKIE[ $this->cookie_name ] ) ? give_clean( $_COOKIE[ $this->cookie_name ] ) : false; // @codingStandardsIgnoreLine.
+		$cookie_value = isset($_COOKIE[$this->cookie_name]) ? give_clean($_COOKIE[$this->cookie_name]) : false; // @codingStandardsIgnoreLine.
 
-		if ( empty( $cookie_value ) || ! is_string( $cookie_value ) ) {
+		if (empty($cookie_value) || ! is_string($cookie_value)) {
 			return $session;
 		}
 
-		list( $donor_id, $session_expiration, $session_expiring, $cookie_hash ) = explode( '||', $cookie_value );
+		list($donor_id, $session_expiration, $session_expiring, $cookie_hash) = explode('||', $cookie_value);
 
-		if ( empty( $donor_id ) ) {
+		if (empty($donor_id)) {
 			return $session;
 		}
 
 		// Validate hash.
-		$to_hash = $donor_id . '|' . $session_expiration;
-		$hash    = hash_hmac( 'md5', $to_hash, wp_hash( $to_hash ) );
+		$to_hash = $donor_id.'|'.$session_expiration;
+		$hash    = hash_hmac('md5', $to_hash, wp_hash($to_hash));
 
-		if ( empty( $cookie_hash ) || ! hash_equals( $hash, $cookie_hash ) ) {
+		if (empty($cookie_hash) || ! hash_equals($hash, $cookie_hash)) {
 			return $session;
 		}
 
-		return array( $donor_id, $session_expiration, $session_expiring, $cookie_hash );
+		return array($donor_id, $session_expiration, $session_expiring, $cookie_hash);
 	}
 
 
@@ -259,8 +259,8 @@ 
 	 * @return string Cookie name.
 	 */
 	private function set_cookie_name() {
-		$this->cookie_name       = apply_filters( 'give_session_cookie', 'wp_give_session_' . COOKIEHASH );
-		$this->nonce_cookie_name = 'wp_give_session_reset_nonce_' . COOKIEHASH;
+		$this->cookie_name       = apply_filters('give_session_cookie', 'wp_give_session_'.COOKIEHASH);
+		$this->nonce_cookie_name = 'wp_give_session_reset_nonce_'.COOKIEHASH;
 	}
 
 	/**
@@ -276,10 +276,10 @@ 
 	 *
 	 * @return string|array      Session variable.
 	 */
-	public function get( $key, $default = false ) {
-		$key = sanitize_key( $key );
+	public function get($key, $default = false) {
+		$key = sanitize_key($key);
 
-		return isset( $this->session[ $key ] ) ? maybe_unserialize( $this->session[ $key ] ) : $default;
+		return isset($this->session[$key]) ? maybe_unserialize($this->session[$key]) : $default;
 	}
 
 	/**
@@ -293,13 +293,13 @@ 
 	 *
 	 * @return string        Session variable.
 	 */
-	public function set( $key, $value ) {
-		if ( $value !== $this->get( $key ) ) {
-			$this->session[ sanitize_key( $key ) ] = maybe_serialize( $value );
+	public function set($key, $value) {
+		if ($value !== $this->get($key)) {
+			$this->session[sanitize_key($key)] = maybe_serialize($value);
 			$this->session_data_changed            = true;
 		}
 
-		return $this->session[ $key ];
+		return $this->session[$key];
 	}
 
 	/**
@@ -313,17 +313,17 @@ 
 	 *
 	 * @param bool $set Flag to check if set cookie or not.
 	 */
-	public function set_session_cookies( $set ) {
-		if ( $set ) {
+	public function set_session_cookies($set) {
+		if ($set) {
 			$this->set_expiration_time();
 
-			$to_hash          = $this->donor_id . '|' . $this->session_expiration;
-			$cookie_hash      = hash_hmac( 'md5', $to_hash, wp_hash( $to_hash ) );
-			$cookie_value     = $this->donor_id . '||' . $this->session_expiration . '||' . $this->session_expiring . '||' . $cookie_hash;
+			$to_hash          = $this->donor_id.'|'.$this->session_expiration;
+			$cookie_hash      = hash_hmac('md5', $to_hash, wp_hash($to_hash));
+			$cookie_value     = $this->donor_id.'||'.$this->session_expiration.'||'.$this->session_expiring.'||'.$cookie_hash;
 			$this->has_cookie = true;
 
-			give_setcookie( $this->cookie_name, $cookie_value, $this->session_expiration, apply_filters( 'give_session_use_secure_cookie', false ) );
-			give_setcookie( $this->nonce_cookie_name, '1', $this->session_expiration, apply_filters( 'give_session_use_secure_cookie', false ) );
+			give_setcookie($this->cookie_name, $cookie_value, $this->session_expiration, apply_filters('give_session_use_secure_cookie', false));
+			give_setcookie($this->nonce_cookie_name, '1', $this->session_expiration, apply_filters('give_session_use_secure_cookie', false));
 		}
 	}
 
@@ -338,8 +338,8 @@ 
 	 * @return int
 	 */
 	public function set_expiration_time() {
-		$this->session_expiring   = time() + intval( apply_filters( 'give_session_expiring', ( $this->exp_option - 3600 ) ) ); // Default 11 Hours.
-		$this->session_expiration = time() + intval( apply_filters( 'give_session_expiration', $this->exp_option ) ); // Default 12 Hours.
+		$this->session_expiring   = time() + intval(apply_filters('give_session_expiring', ($this->exp_option - 3600))); // Default 11 Hours.
+		$this->session_expiration = time() + intval(apply_filters('give_session_expiration', $this->exp_option)); // Default 12 Hours.
 
 		return $this->session_expiration;
 	}
@@ -371,10 +371,10 @@ 
 	public function maybe_start_session() {
 		if (
 			! headers_sent()
-			&& empty( $this->session )
+			&& empty($this->session)
 			&& ! $this->has_cookie
 		) {
-			$this->set_session_cookies( true );
+			$this->set_session_cookies(true);
 		}
 	}
 
@@ -387,10 +387,10 @@ 
 	 * @access public
 	 */
 	public function generate_donor_id() {
-		require_once ABSPATH . 'wp-includes/class-phpass.php';
+		require_once ABSPATH.'wp-includes/class-phpass.php';
 
-		$hasher         = new PasswordHash( 8, false );
-		$this->donor_id = md5( $hasher->get_random_bytes( 32 ) );
+		$hasher         = new PasswordHash(8, false);
+		$this->donor_id = md5($hasher->get_random_bytes(32));
 	}
 
 	/**
@@ -401,14 +401,14 @@ 
 	 */
 	public function save_data() {
 		// Dirty if something changed - prevents saving nothing new.
-		if ( $this->session_data_changed && $this->has_session() ) {
+		if ($this->session_data_changed && $this->has_session()) {
 			global $wpdb;
 
 			Give()->session_db->__replace(
 				Give()->session_db->table_name,
 				array(
 					'session_key'    => $this->donor_id,
-					'session_value'  => maybe_serialize( $this->session ),
+					'session_value'  => maybe_serialize($this->session),
 					'session_expiry' => $this->session_expiration,
 				),
 				array(
@@ -429,10 +429,10 @@ 
 	 * @access public
 	 */
 	public function destroy_session() {
-		give_setcookie( $this->cookie_name, '', time() - YEAR_IN_SECONDS, apply_filters( 'give_session_use_secure_cookie', false ) );
-		give_setcookie( $this->nonce_cookie_name, '', time() - YEAR_IN_SECONDS, apply_filters( 'give_session_use_secure_cookie', false ) );
+		give_setcookie($this->cookie_name, '', time() - YEAR_IN_SECONDS, apply_filters('give_session_use_secure_cookie', false));
+		give_setcookie($this->nonce_cookie_name, '', time() - YEAR_IN_SECONDS, apply_filters('give_session_use_secure_cookie', false));
 
-		Give()->session_db->delete_session( $this->donor_id );
+		Give()->session_db->delete_session($this->donor_id);
 
 		$this->session              = array();
 		$this->session_data_changed = false;
@@ -448,10 +448,10 @@ 
 	 *
 	 * @return bool
 	 */
-	public function is_delete_nonce_cookie(){
+	public function is_delete_nonce_cookie() {
 		$value = false;
 
-		if ( Give()->session->has_session() ) {
+		if (Give()->session->has_session()) {
 			$value = true;
 		}
 
@@ -468,10 +468,10 @@ 
 	 *
 	 * @return string Cookie name
 	 */
-	public function get_cookie_name( $type = '' ) {
+	public function get_cookie_name($type = '') {
 		$name = '';
 
-		switch ( $type ) {
+		switch ($type) {
 			case 'nonce':
 				$name = $this->nonce_cookie_name;
 				break;
@@ -495,7 +495,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function __nonce_user_logged_out( $uid ) {
+	public function __nonce_user_logged_out($uid) {
 		return $this->has_session() && $this->donor_id ? $this->donor_id : $uid;
 	}
 
@@ -541,7 +541,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);
 	}
 
 	/**
@@ -558,9 +558,9 @@ 
 	public function use_php_sessions() {
 		$ret = false;
 
-		give_doing_it_wrong( __FUNCTION__, __( 'We are using database session logic instead of PHP session', 'give' ), '2.2.0' );
+		give_doing_it_wrong(__FUNCTION__, __('We are using database session logic instead of PHP session', 'give'), '2.2.0');
 
-		return (bool) apply_filters( 'give_use_php_sessions', $ret );
+		return (bool) apply_filters('give_use_php_sessions', $ret);
 	}
 
 	/**
@@ -578,10 +578,10 @@ 
 
 		$start_session = true;
 
-		give_doing_it_wrong( __FUNCTION__, __( 'We are using database session logic instead of PHP session', 'give' ), '2.2.0' );
+		give_doing_it_wrong(__FUNCTION__, __('We are using database session logic instead of PHP session', 'give'), '2.2.0');
 
 
-		if ( ! empty( $_SERVER['REQUEST_URI'] ) ) {  // @codingStandardsIgnoreLine
+		if ( ! empty($_SERVER['REQUEST_URI'])) {  // @codingStandardsIgnoreLine
 
 			$blacklist = apply_filters(
 				'give_session_start_uri_blacklist', array(
@@ -594,19 +594,19 @@ 
 					'comments/feed/',
 				)
 			);
-			$uri       = ltrim( $_SERVER['REQUEST_URI'], '/' ); // // @codingStandardsIgnoreLine
-			$uri       = untrailingslashit( $uri );
-			if ( in_array( $uri, $blacklist, true ) ) {
+			$uri       = ltrim($_SERVER['REQUEST_URI'], '/'); // // @codingStandardsIgnoreLine
+			$uri       = untrailingslashit($uri);
+			if (in_array($uri, $blacklist, true)) {
 				$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);
 	}
 }