WordImpress / Give

includes/admin/reporting/export/class-batch-export-customers.php

Doc Comments +1 added lines

@@ -206,6 +206,7 @@
 	 * Set Donor Data
 	 *
 	 * @param $donor
+	 * @param integer $i
 	 */
 	private function set_donor_data( $i, $data, $donor ) {
 

Spacing +71 added lines, -71 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -46,16 +46,16 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 
 		//Set data from form submission
-		if ( isset( $_POST['form'] ) ) {
-			parse_str( $_POST['form'], $this->data );
+		if (isset($_POST['form'])) {
+			parse_str($_POST['form'], $this->data);
 		}
 
 		$this->form = $this->data['forms'];
 
-		$this->price_id = ! empty( $request['give_price_option'] ) && 0 !== $request['give_price_option'] ? absint( $request['give_price_option'] ) : null;
+		$this->price_id = ! empty($request['give_price_option']) && 0 !== $request['give_price_option'] ? absint($request['give_price_option']) : null;
 
 	}
 
@@ -70,36 +70,36 @@ 
 
 		$cols = array();
 
-		$columns = isset( $this->data['give_export_option'] ) ? $this->data['give_export_option'] : array();
+		$columns = isset($this->data['give_export_option']) ? $this->data['give_export_option'] : array();
 
-		if ( empty( $columns ) ) {
+		if (empty($columns)) {
 			return false;
 		}
-		if ( ! empty( $columns['full_name'] ) ) {
-			$cols['full_name'] = esc_html__( 'Full Name', 'give' );
+		if ( ! empty($columns['full_name'])) {
+			$cols['full_name'] = esc_html__('Full Name', 'give');
 		}
-		if ( ! empty( $columns['email'] ) ) {
-			$cols['email'] = esc_html__( 'Email Address', 'give' );
+		if ( ! empty($columns['email'])) {
+			$cols['email'] = esc_html__('Email Address', 'give');
 		}
-		if ( ! empty( $columns['address'] ) ) {
-			$cols['address_line1']   = esc_html__( 'Address', 'give' );
-			$cols['address_line2']   = esc_html__( 'Address (Line 2)', 'give' );
-			$cols['address_city']    = esc_html__( 'City', 'give' );
-			$cols['address_state']   = esc_html__( 'State', 'give' );
-			$cols['address_zip']     = esc_html__( 'Zip', 'give' );
-			$cols['address_country'] = esc_html__( 'Country', 'give' );
+		if ( ! empty($columns['address'])) {
+			$cols['address_line1']   = esc_html__('Address', 'give');
+			$cols['address_line2']   = esc_html__('Address (Line 2)', 'give');
+			$cols['address_city']    = esc_html__('City', 'give');
+			$cols['address_state']   = esc_html__('State', 'give');
+			$cols['address_zip']     = esc_html__('Zip', 'give');
+			$cols['address_country'] = esc_html__('Country', 'give');
 		}
-		if ( ! empty( $columns['userid'] ) ) {
-			$cols['userid'] = esc_html__( 'User ID', 'give' );
+		if ( ! empty($columns['userid'])) {
+			$cols['userid'] = esc_html__('User ID', 'give');
 		}
-		if ( ! empty( $columns['date_first_donated'] ) ) {
-			$cols['date_first_donated'] = esc_html__( 'First Donation Date', 'give' );
+		if ( ! empty($columns['date_first_donated'])) {
+			$cols['date_first_donated'] = esc_html__('First Donation Date', 'give');
 		}
-		if ( ! empty( $columns['donations'] ) ) {
-			$cols['donations'] = esc_html__( 'Number of Donations', 'give' );
+		if ( ! empty($columns['donations'])) {
+			$cols['donations'] = esc_html__('Number of Donations', 'give');
 		}
-		if ( ! empty( $columns['donation_sum'] ) ) {
-			$cols['donation_sum'] = esc_html__( 'Sum of Donations', 'give' );
+		if ( ! empty($columns['donation_sum'])) {
+			$cols['donation_sum'] = esc_html__('Sum of Donations', 'give');
 		}
 
 		return $cols;
@@ -119,20 +119,20 @@ 
 
 		$i = 0;
 
-		if ( ! empty( $this->form ) ) {
+		if ( ! empty($this->form)) {
 
 			// Export donors of a specific product
 			global $give_logs;
 
 			$args = array(
-				'post_parent'    => absint( $this->form ),
+				'post_parent'    => absint($this->form),
 				'log_type'       => 'sale',
 				'posts_per_page' => 30,
 				'paged'          => $this->step
 			);
 
 			//Check for price option
-			if ( null !== $this->price_id ) {
+			if (null !== $this->price_id) {
 				$args['meta_query'] = array(
 					array(
 						'key'   => '_give_log_price_id',
@@ -141,33 +141,33 @@ 
 				);
 			}
 
-			$logs = $give_logs->get_connected_logs( $args );
+			$logs = $give_logs->get_connected_logs($args);
 
-			if ( $logs ) {
-				foreach ( $logs as $log ) {
-					$payment_id = get_post_meta( $log->ID, '_give_log_payment_id', true );
-					$payment    = new Give_Payment( $payment_id );
-					$donor      = Give()->customers->get_customer_by( 'id', $payment->customer_id );
-					$data[]     = $this->set_donor_data( $i, $data, $donor );
-					$i ++;
+			if ($logs) {
+				foreach ($logs as $log) {
+					$payment_id = get_post_meta($log->ID, '_give_log_payment_id', true);
+					$payment    = new Give_Payment($payment_id);
+					$donor      = Give()->customers->get_customer_by('id', $payment->customer_id);
+					$data[]     = $this->set_donor_data($i, $data, $donor);
+					$i++;
 				}
 			}
 
 		} else {
 
 			// Export all customers
-			$offset = 30 * ( $this->step - 1 );
-			$donors = Give()->customers->get_customers( array( 'number' => 30, 'offset' => $offset ) );
+			$offset = 30 * ($this->step - 1);
+			$donors = Give()->customers->get_customers(array('number' => 30, 'offset' => $offset));
 
-			foreach ( $donors as $donor ) {
+			foreach ($donors as $donor) {
 
-				$data[] = $this->set_donor_data( $i, $data, $donor );
-				$i ++;
+				$data[] = $this->set_donor_data($i, $data, $donor);
+				$i++;
 			}
 		}
 
-		$data = apply_filters( 'give_export_get_data', $data );
-		$data = apply_filters( 'give_export_get_data_' . $this->export_type, $data );
+		$data = apply_filters('give_export_get_data', $data);
+		$data = apply_filters('give_export_get_data_'.$this->export_type, $data);
 
 		return $data;
 	}
@@ -183,19 +183,19 @@ 
 		$percentage = 0;
 
 		// We can't count the number when getting them for a specific form
-		if ( empty( $this->form ) ) {
+		if (empty($this->form)) {
 
 			$total = Give()->customers->count();
 
-			if ( $total > 0 ) {
+			if ($total > 0) {
 
-				$percentage = ( ( 30 * $this->step ) / $total ) * 100;
+				$percentage = ((30 * $this->step) / $total) * 100;
 
 			}
 
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -207,46 +207,46 @@ 
 	 *
 	 * @param $donor
 	 */
-	private function set_donor_data( $i, $data, $donor ) {
+	private function set_donor_data($i, $data, $donor) {
 
 		$columns = $this->csv_cols();
 
 		//Set address variable
 		$address = '';
-		if ( isset( $donor->user_id ) && $donor->user_id > 0 ) {
-			$address = give_get_donor_address( $donor->user_id );
+		if (isset($donor->user_id) && $donor->user_id > 0) {
+			$address = give_get_donor_address($donor->user_id);
 		}
 
 		//Set columns
-		if ( ! empty( $columns['full_name'] ) ) {
-			$data[ $i ]['full_name'] = $donor->name;
+		if ( ! empty($columns['full_name'])) {
+			$data[$i]['full_name'] = $donor->name;
 		}
-		if ( ! empty( $columns['email'] ) ) {
-			$data[ $i ]['email'] = $donor->email;
+		if ( ! empty($columns['email'])) {
+			$data[$i]['email'] = $donor->email;
 		}
-		if ( ! empty( $columns['address_line1'] ) ) {
+		if ( ! empty($columns['address_line1'])) {
 
-			$data[ $i ]['address_line1']   = isset( $address['line1'] ) ? $address['line1'] : '';
-			$data[ $i ]['address_line2']   = isset( $address['line2'] ) ? $address['line2'] : '';
-			$data[ $i ]['address_city']    = isset( $address['city'] ) ? $address['city'] : '';
-			$data[ $i ]['address_state']   = isset( $address['state'] ) ? $address['state'] : '';
-			$data[ $i ]['address_zip']     = isset( $address['zip'] ) ? $address['zip'] : '';
-			$data[ $i ]['address_country'] = isset( $address['country'] ) ? $address['country'] : '';
+			$data[$i]['address_line1']   = isset($address['line1']) ? $address['line1'] : '';
+			$data[$i]['address_line2']   = isset($address['line2']) ? $address['line2'] : '';
+			$data[$i]['address_city']    = isset($address['city']) ? $address['city'] : '';
+			$data[$i]['address_state']   = isset($address['state']) ? $address['state'] : '';
+			$data[$i]['address_zip']     = isset($address['zip']) ? $address['zip'] : '';
+			$data[$i]['address_country'] = isset($address['country']) ? $address['country'] : '';
 		}
-		if ( ! empty( $columns['userid'] ) ) {
-			$data[ $i ]['userid'] = ! empty( $donor->user_id ) ? $donor->user_id : '';
+		if ( ! empty($columns['userid'])) {
+			$data[$i]['userid'] = ! empty($donor->user_id) ? $donor->user_id : '';
 		}
-		if ( ! empty( $columns['date_first_donated'] ) ) {
-			$data[ $i ]['date_first_donated'] = date_i18n( get_option( 'date_format' ), strtotime( $donor->date_created ) );
+		if ( ! empty($columns['date_first_donated'])) {
+			$data[$i]['date_first_donated'] = date_i18n(get_option('date_format'), strtotime($donor->date_created));
 		}
-		if ( ! empty( $columns['donations'] ) ) {
-			$data[ $i ]['donations'] = $donor->purchase_count;
+		if ( ! empty($columns['donations'])) {
+			$data[$i]['donations'] = $donor->purchase_count;
 		}
-		if ( ! empty( $columns['donation_sum'] ) ) {
-			$data[ $i ]['donation_sum'] = give_format_amount( $donor->purchase_value );
+		if ( ! empty($columns['donation_sum'])) {
+			$data[$i]['donation_sum'] = give_format_amount($donor->purchase_value);
 		}
 
-		return $data[ $i ];
+		return $data[$i];
 
 	}
 

includes/forms/functions.php

Spacing +183 added lines, -183 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,14 +23,14 @@ 
 
 	global $typenow;
 
-	if ( $typenow != 'give_forms' ) {
+	if ($typenow != 'give_forms') {
 		return true;
 	}
 
 	return false;
 }
 
-add_filter( 'give_shortcode_button_condition', 'give_shortcode_button_condition' );
+add_filter('give_shortcode_button_condition', 'give_shortcode_button_condition');
 
 
 /**
@@ -40,11 +40,11 @@ 
  *
  * @return int|false
  */
-function get_form_id_from_args( $args ) {
+function get_form_id_from_args($args) {
 
-	if ( isset( $args['form_id'] ) && $args['form_id'] != 0 ) {
+	if (isset($args['form_id']) && $args['form_id'] != 0) {
 
-		return intval( $args['form_id'] );
+		return intval($args['form_id']);
 	}
 
 	return false;
@@ -59,23 +59,23 @@ 
  *
  * @return bool
  */
-function give_is_float_labels_enabled( $args ) {
+function give_is_float_labels_enabled($args) {
 
 	$float_labels = '';
 
-	if ( ! empty( $args['float_labels'] ) ) {
+	if ( ! empty($args['float_labels'])) {
 		$float_labels = $args['float_labels'];
 	}
 
-	if ( empty( $float_labels ) ) {
-		$float_labels = get_post_meta( $args['form_id'], '_give_form_floating_labels', true );
+	if (empty($float_labels)) {
+		$float_labels = get_post_meta($args['form_id'], '_give_form_floating_labels', true);
 	}
 
-	if ( empty( $float_labels ) ) {
-		$float_labels = give_get_option( 'enable_floatlabels' ) ? 'enabled' : 'disabled';
+	if (empty($float_labels)) {
+		$float_labels = give_get_option('enable_floatlabels') ? 'enabled' : 'disabled';
 	}
 
-	return ( $float_labels == 'enabled' ) ? true : false;
+	return ($float_labels == 'enabled') ? true : false;
 }
 
 /**
@@ -90,7 +90,7 @@ 
 
 	$can_checkout = true;
 
-	return (bool) apply_filters( 'give_can_checkout', $can_checkout );
+	return (bool) apply_filters('give_can_checkout', $can_checkout);
 }
 
 /**
@@ -103,9 +103,9 @@ 
 function give_get_success_page_uri() {
 	global $give_options;
 
-	$success_page = isset( $give_options['success_page'] ) ? get_permalink( absint( $give_options['success_page'] ) ) : get_bloginfo( 'url' );
+	$success_page = isset($give_options['success_page']) ? get_permalink(absint($give_options['success_page'])) : get_bloginfo('url');
 
-	return apply_filters( 'give_get_success_page_uri', $success_page );
+	return apply_filters('give_get_success_page_uri', $success_page);
 }
 
 /**
@@ -116,9 +116,9 @@ 
  */
 function give_is_success_page() {
 	global $give_options;
-	$is_success_page = isset( $give_options['success_page'] ) ? is_page( $give_options['success_page'] ) : false;
+	$is_success_page = isset($give_options['success_page']) ? is_page($give_options['success_page']) : false;
 
-	return apply_filters( 'give_is_success_page', $is_success_page );
+	return apply_filters('give_is_success_page', $is_success_page);
 }
 
 /**
@@ -132,17 +132,17 @@ 
  * @since       1.0
  * @return      void
  */
-function give_send_to_success_page( $query_string = null ) {
+function give_send_to_success_page($query_string = null) {
 
 	$redirect = give_get_success_page_uri();
 
-	if ( $query_string ) {
+	if ($query_string) {
 		$redirect .= $query_string;
 	}
 
-	$gateway = isset( $_REQUEST['give-gateway'] ) ? $_REQUEST['give-gateway'] : '';
+	$gateway = isset($_REQUEST['give-gateway']) ? $_REQUEST['give-gateway'] : '';
 
-	wp_redirect( apply_filters( 'give_success_page_redirect', $redirect, $gateway, $query_string ) );
+	wp_redirect(apply_filters('give_success_page_redirect', $redirect, $gateway, $query_string));
 	give_die();
 }
 
@@ -159,25 +159,25 @@ 
  * @since  1.0
  * @return Void
  */
-function give_send_back_to_checkout( $args = array() ) {
+function give_send_back_to_checkout($args = array()) {
 
-	$redirect = ( isset( $_POST['give-current-url'] ) ) ? $_POST['give-current-url'] : '';
-	$form_id  = isset( $_POST['give-form-id'] ) ? $_POST['give-form-id'] : 0;
+	$redirect = (isset($_POST['give-current-url'])) ? $_POST['give-current-url'] : '';
+	$form_id  = isset($_POST['give-form-id']) ? $_POST['give-form-id'] : 0;
 
 	$defaults = array(
 		'form-id' => (int) $form_id
 	);
 
 	// Check for backward compatibility
-	if ( is_string( $args ) ) {
-		$args = str_replace( '?', '', $args );
+	if (is_string($args)) {
+		$args = str_replace('?', '', $args);
 	}
 
-	$args = wp_parse_args( $args, $defaults );
+	$args = wp_parse_args($args, $defaults);
 
-	$redirect = add_query_arg( $args, $redirect ) . '#give-form-' . $form_id . '-wrap';
+	$redirect = add_query_arg($args, $redirect).'#give-form-'.$form_id.'-wrap';
 
-	wp_redirect( apply_filters( 'give_send_back_to_checkout', $redirect, $args ) );
+	wp_redirect(apply_filters('give_send_back_to_checkout', $redirect, $args));
 	give_die();
 }
 
@@ -192,16 +192,16 @@ 
  * @since       1.0
  * @return      string
  */
-function give_get_success_page_url( $query_string = null ) {
+function give_get_success_page_url($query_string = null) {
 
-	$success_page = give_get_option( 'success_page', 0 );
-	$success_page = get_permalink( $success_page );
+	$success_page = give_get_option('success_page', 0);
+	$success_page = get_permalink($success_page);
 
-	if ( $query_string ) {
+	if ($query_string) {
 		$success_page .= $query_string;
 	}
 
-	return apply_filters( 'give_success_page_url', $success_page );
+	return apply_filters('give_success_page_url', $success_page);
 
 }
 
@@ -215,15 +215,15 @@ 
  *
  * @return mixed|void Full URL to the Transaction Failed page, if present, home page if it doesn't exist
  */
-function give_get_failed_transaction_uri( $extras = false ) {
+function give_get_failed_transaction_uri($extras = false) {
 	global $give_options;
 
-	$uri = ! empty( $give_options['failure_page'] ) ? trailingslashit( get_permalink( $give_options['failure_page'] ) ) : home_url();
-	if ( $extras ) {
+	$uri = ! empty($give_options['failure_page']) ? trailingslashit(get_permalink($give_options['failure_page'])) : home_url();
+	if ($extras) {
 		$uri .= $extras;
 	}
 
-	return apply_filters( 'give_get_failed_transaction_uri', $uri );
+	return apply_filters('give_get_failed_transaction_uri', $uri);
 }
 
 /**
@@ -234,9 +234,9 @@ 
  */
 function give_is_failed_transaction_page() {
 	global $give_options;
-	$ret = isset( $give_options['failure_page'] ) ? is_page( $give_options['failure_page'] ) : false;
+	$ret = isset($give_options['failure_page']) ? is_page($give_options['failure_page']) : false;
 
-	return apply_filters( 'give_is_failure_page', $ret );
+	return apply_filters('give_is_failure_page', $ret);
 }
 
 /**
@@ -248,18 +248,18 @@ 
  */
 function give_listen_for_failed_payments() {
 
-	$failed_page = give_get_option( 'failure_page', 0 );
+	$failed_page = give_get_option('failure_page', 0);
 
-	if ( ! empty( $failed_page ) && is_page( $failed_page ) && ! empty( $_GET['payment-id'] ) ) {
+	if ( ! empty($failed_page) && is_page($failed_page) && ! empty($_GET['payment-id'])) {
 
-		$payment_id = absint( $_GET['payment-id'] );
-		give_update_payment_status( $payment_id, 'failed' );
+		$payment_id = absint($_GET['payment-id']);
+		give_update_payment_status($payment_id, 'failed');
 
 	}
 
 }
 
-add_action( 'template_redirect', 'give_listen_for_failed_payments' );
+add_action('template_redirect', 'give_listen_for_failed_payments');
 
 
 /**
@@ -272,11 +272,11 @@ 
  * @since       1.0
  * @return      bool
  */
-function give_field_is_required( $field = '', $form_id ) {
+function give_field_is_required($field = '', $form_id) {
 
-	$required_fields = give_purchase_form_required_fields( $form_id );
+	$required_fields = give_purchase_form_required_fields($form_id);
 
-	return array_key_exists( $field, $required_fields );
+	return array_key_exists($field, $required_fields);
 }
 
 /**
@@ -294,14 +294,14 @@ 
  *
  * @return void
  */
-function give_record_sale_in_log( $give_form_id = 0, $payment_id, $price_id = false, $sale_date = null ) {
+function give_record_sale_in_log($give_form_id = 0, $payment_id, $price_id = false, $sale_date = null) {
 	global $give_logs;
 
 	$log_data = array(
 		'post_parent'   => $give_form_id,
 		'log_type'      => 'sale',
-		'post_date'     => isset( $sale_date ) ? $sale_date : null,
-		'post_date_gmt' => isset( $sale_date ) ? $sale_date : null
+		'post_date'     => isset($sale_date) ? $sale_date : null,
+		'post_date_gmt' => isset($sale_date) ? $sale_date : null
 	);
 
 	$log_meta = array(
@@ -309,7 +309,7 @@ 
 		'price_id'   => (int) $price_id
 	);
 
-	$give_logs->insert_log( $log_data, $log_meta );
+	$give_logs->insert_log($log_data, $log_meta);
 }
 
 
@@ -323,11 +323,11 @@ 
  *
  * @return bool|int
  */
-function give_increase_purchase_count( $form_id = 0, $quantity = 1 ) {
+function give_increase_purchase_count($form_id = 0, $quantity = 1) {
 	$quantity = (int) $quantity;
-	$form     = new Give_Donate_Form( $form_id );
+	$form     = new Give_Donate_Form($form_id);
 
-	return $form->increase_sales( $quantity );
+	return $form->increase_sales($quantity);
 }
 
 /**
@@ -340,11 +340,11 @@ 
  *
  * @return bool|int
  */
-function give_decrease_purchase_count( $form_id = 0, $quantity = 1 ) {
+function give_decrease_purchase_count($form_id = 0, $quantity = 1) {
 	$quantity = (int) $quantity;
-	$form     = new Give_Donate_Form( $form_id );
+	$form     = new Give_Donate_Form($form_id);
 
-	return $form->decrease_sales( $quantity );
+	return $form->decrease_sales($quantity);
 }
 
 /**
@@ -357,10 +357,10 @@ 
  *
  * @return bool|int
  */
-function give_increase_earnings( $give_form_id = 0, $amount ) {
-	$form = new Give_Donate_Form( $give_form_id );
+function give_increase_earnings($give_form_id = 0, $amount) {
+	$form = new Give_Donate_Form($give_form_id);
 
-	return $form->increase_earnings( $amount );
+	return $form->increase_earnings($amount);
 }
 
 /**
@@ -373,10 +373,10 @@ 
  *
  * @return bool|int
  */
-function give_decrease_earnings( $form_id = 0, $amount ) {
-	$form = new Give_Donate_Form( $form_id );
+function give_decrease_earnings($form_id = 0, $amount) {
+	$form = new Give_Donate_Form($form_id);
 
-	return $form->decrease_earnings( $amount );
+	return $form->decrease_earnings($amount);
 }
 
 
@@ -389,8 +389,8 @@ 
  *
  * @return int $earnings Earnings for a certain form
  */
-function give_get_form_earnings_stats( $form_id = 0 ) {
-	$give_form = new Give_Donate_Form( $form_id );
+function give_get_form_earnings_stats($form_id = 0) {
+	$give_form = new Give_Donate_Form($form_id);
 
 	return $give_form->earnings;
 }
@@ -405,8 +405,8 @@ 
  *
  * @return int $sales Amount of sales for a certain form
  */
-function give_get_form_sales_stats( $give_form_id = 0 ) {
-	$give_form = new Give_Donate_Form( $give_form_id );
+function give_get_form_sales_stats($give_form_id = 0) {
+	$give_form = new Give_Donate_Form($give_form_id);
 
 	return $give_form->sales;
 }
@@ -421,16 +421,16 @@ 
  *
  * @return float $sales Average monthly sales
  */
-function give_get_average_monthly_form_sales( $form_id = 0 ) {
-	$sales        = give_get_form_sales_stats( $form_id );
-	$release_date = get_post_field( 'post_date', $form_id );
+function give_get_average_monthly_form_sales($form_id = 0) {
+	$sales        = give_get_form_sales_stats($form_id);
+	$release_date = get_post_field('post_date', $form_id);
 
-	$diff = abs( current_time( 'timestamp' ) - strtotime( $release_date ) );
+	$diff = abs(current_time('timestamp') - strtotime($release_date));
 
-	$months = floor( $diff / ( 30 * 60 * 60 * 24 ) ); // Number of months since publication
+	$months = floor($diff / (30 * 60 * 60 * 24)); // Number of months since publication
 
-	if ( $months > 0 ) {
-		$sales = ( $sales / $months );
+	if ($months > 0) {
+		$sales = ($sales / $months);
 	}
 
 	return $sales;
@@ -446,16 +446,16 @@ 
  *
  * @return float $earnings Average monthly earnings
  */
-function give_get_average_monthly_form_earnings( $form_id = 0 ) {
-	$earnings     = give_get_form_earnings_stats( $form_id );
-	$release_date = get_post_field( 'post_date', $form_id );
+function give_get_average_monthly_form_earnings($form_id = 0) {
+	$earnings     = give_get_form_earnings_stats($form_id);
+	$release_date = get_post_field('post_date', $form_id);
 
-	$diff = abs( current_time( 'timestamp' ) - strtotime( $release_date ) );
+	$diff = abs(current_time('timestamp') - strtotime($release_date));
 
-	$months = floor( $diff / ( 30 * 60 * 60 * 24 ) ); // Number of months since publication
+	$months = floor($diff / (30 * 60 * 60 * 24)); // Number of months since publication
 
-	if ( $months > 0 ) {
-		$earnings = ( $earnings / $months );
+	if ($months > 0) {
+		$earnings = ($earnings / $months);
 	}
 
 	return $earnings < 0 ? 0 : $earnings;
@@ -475,25 +475,25 @@ 
  *
  * @return string $price_name Name of the price option
  */
-function give_get_price_option_name( $form_id = 0, $price_id = 0, $payment_id = 0 ) {
+function give_get_price_option_name($form_id = 0, $price_id = 0, $payment_id = 0) {
 
-	$prices     = give_get_variable_prices( $form_id );
+	$prices     = give_get_variable_prices($form_id);
 	$price_name = '';
 
-	foreach ( $prices as $price ) {
+	foreach ($prices as $price) {
 
-		if ( intval( $price['_give_id']['level_id'] ) == intval( $price_id ) ) {
+		if (intval($price['_give_id']['level_id']) == intval($price_id)) {
 
-			$price_text     = isset( $price['_give_text'] ) ? $price['_give_text'] : '';
-			$price_fallback = give_currency_filter( give_format_amount( $price['_give_amount'] ) );
-			$price_name     = ! empty( $price_text ) ? $price_text : $price_fallback;
+			$price_text     = isset($price['_give_text']) ? $price['_give_text'] : '';
+			$price_fallback = give_currency_filter(give_format_amount($price['_give_amount']));
+			$price_name     = ! empty($price_text) ? $price_text : $price_fallback;
 
 		}
 
 	}
 
 
-	return apply_filters( 'give_get_price_option_name', $price_name, $form_id, $payment_id, $price_id );
+	return apply_filters('give_get_price_option_name', $price_name, $form_id, $payment_id, $price_id);
 }
 
 
@@ -506,14 +506,14 @@ 
  *
  * @return string $range A fully formatted price range
  */
-function give_price_range( $form_id = 0 ) {
-	$low   = give_get_lowest_price_option( $form_id );
-	$high  = give_get_highest_price_option( $form_id );
-	$range = '<span class="give_price_range_low">' . give_currency_filter( give_format_amount( $low ) ) . '</span>';
+function give_price_range($form_id = 0) {
+	$low   = give_get_lowest_price_option($form_id);
+	$high  = give_get_highest_price_option($form_id);
+	$range = '<span class="give_price_range_low">'.give_currency_filter(give_format_amount($low)).'</span>';
 	$range .= '<span class="give_price_range_sep">&nbsp;&ndash;&nbsp;</span>';
-	$range .= '<span class="give_price_range_high">' . give_currency_filter( give_format_amount( $high ) ) . '</span>';
+	$range .= '<span class="give_price_range_high">'.give_currency_filter(give_format_amount($high)).'</span>';
 
-	return apply_filters( 'give_price_range', $range, $form_id, $low, $high );
+	return apply_filters('give_price_range', $range, $form_id, $low, $high);
 }
 
 
@@ -528,36 +528,36 @@ 
  *
  * @return int ID of the lowest price
  */
-function give_get_lowest_price_id( $form_id = 0 ) {
+function give_get_lowest_price_id($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	if ( ! give_has_variable_prices( $form_id ) ) {
-		return give_get_form_price( $form_id );
+	if ( ! give_has_variable_prices($form_id)) {
+		return give_get_form_price($form_id);
 	}
 
-	$prices = give_get_variable_prices( $form_id );
+	$prices = give_get_variable_prices($form_id);
 
 	$low    = 0.00;
 	$min_id = 1;
 
-	if ( ! empty( $prices ) ) {
+	if ( ! empty($prices)) {
 
-		foreach ( $prices as $key => $price ) {
+		foreach ($prices as $key => $price) {
 
-			if ( empty( $price['_give_amount'] ) ) {
+			if (empty($price['_give_amount'])) {
 				continue;
 			}
 
-			if ( ! isset( $min ) ) {
+			if ( ! isset($min)) {
 				$min = $price['_give_amount'];
 			} else {
-				$min = min( $min, $price['_give_amount'] );
+				$min = min($min, $price['_give_amount']);
 			}
 
-			if ( $price['_give_amount'] == $min ) {
+			if ($price['_give_amount'] == $min) {
 				$min_id = $price['_give_id']['level_id'];
 			}
 		}
@@ -575,43 +575,43 @@ 
  *
  * @return float Amount of the lowest price
  */
-function give_get_lowest_price_option( $form_id = 0 ) {
-	if ( empty( $form_id ) ) {
+function give_get_lowest_price_option($form_id = 0) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	if ( ! give_has_variable_prices( $form_id ) ) {
-		return give_get_form_price( $form_id );
+	if ( ! give_has_variable_prices($form_id)) {
+		return give_get_form_price($form_id);
 	}
 
-	$prices = give_get_variable_prices( $form_id );
+	$prices = give_get_variable_prices($form_id);
 
 	$low = 0.00;
 
-	if ( ! empty( $prices ) ) {
+	if ( ! empty($prices)) {
 
-		foreach ( $prices as $key => $price ) {
+		foreach ($prices as $key => $price) {
 
-			if ( empty( $price['_give_amount'] ) ) {
+			if (empty($price['_give_amount'])) {
 				continue;
 			}
 
-			if ( ! isset( $min ) ) {
+			if ( ! isset($min)) {
 				$min = $price['_give_amount'];
 			} else {
-				$min = min( $min, give_sanitize_amount( $price['_give_amount'] ) );
+				$min = min($min, give_sanitize_amount($price['_give_amount']));
 			}
 
-			if ( $price['_give_amount'] == $min ) {
+			if ($price['_give_amount'] == $min) {
 				$min_id = $key;
 			}
 		}
 
-		$low = $prices[ $min_id ]['_give_amount'];
+		$low = $prices[$min_id]['_give_amount'];
 
 	}
 
-	return give_sanitize_amount( $low );
+	return give_sanitize_amount($low);
 }
 
 /**
@@ -623,41 +623,41 @@ 
  *
  * @return float Amount of the highest price
  */
-function give_get_highest_price_option( $form_id = 0 ) {
+function give_get_highest_price_option($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	if ( ! give_has_variable_prices( $form_id ) ) {
-		return give_get_form_price( $form_id );
+	if ( ! give_has_variable_prices($form_id)) {
+		return give_get_form_price($form_id);
 	}
 
-	$prices = give_get_variable_prices( $form_id );
+	$prices = give_get_variable_prices($form_id);
 
 	$high = 0.00;
 
-	if ( ! empty( $prices ) ) {
+	if ( ! empty($prices)) {
 
 		$max = 0;
 
-		foreach ( $prices as $key => $price ) {
-			if ( empty( $price['_give_amount'] ) ) {
+		foreach ($prices as $key => $price) {
+			if (empty($price['_give_amount'])) {
 				continue;
 			}
-			$give_amount = give_sanitize_amount( $price['_give_amount'] );
+			$give_amount = give_sanitize_amount($price['_give_amount']);
 
-			$max = max( $max, $give_amount );
+			$max = max($max, $give_amount);
 
-			if ( $give_amount == $max ) {
+			if ($give_amount == $max) {
 				$max_id = $key;
 			}
 		}
 
-		$high = $prices[ $max_id ]['_give_amount'];
+		$high = $prices[$max_id]['_give_amount'];
 	}
 
-	return give_sanitize_amount( $high );
+	return give_sanitize_amount($high);
 }
 
 /**
@@ -669,15 +669,15 @@ 
  *
  * @return mixed string|int Price of the form
  */
-function give_get_form_price( $form_id = 0 ) {
+function give_get_form_price($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
-	return $form->__get( 'price' );
+	return $form->__get('price');
 }
 
 /**
@@ -689,15 +689,15 @@ 
  *
  * @return mixed string|int Minimum price of the form
  */
-function give_get_form_minimum_price( $form_id = 0 ) {
+function give_get_form_minimum_price($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
-	return $form->__get( 'minimum_price' );
+	return $form->__get('minimum_price');
 
 }
 
@@ -712,52 +712,52 @@ 
  *
  * @return int $formatted_price
  */
-function give_price( $form_id = 0, $echo = true, $price_id = false ) {
+function give_price($form_id = 0, $echo = true, $price_id = false) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	if ( give_has_variable_prices( $form_id ) ) {
+	if (give_has_variable_prices($form_id)) {
 
-		$prices = give_get_variable_prices( $form_id );
+		$prices = give_get_variable_prices($form_id);
 
-		if ( false !== $price_id ) {
+		if (false !== $price_id) {
 
 			//loop through multi-prices to see which is default
-			foreach ( $prices as $price ) {
+			foreach ($prices as $price) {
 				//this is the default price
-				if ( isset( $price['_give_default'] ) && $price['_give_default'] === 'default' ) {
+				if (isset($price['_give_default']) && $price['_give_default'] === 'default') {
 					$price = (float) $price['_give_amount'];
 				};
 			}
 
 		} else {
 
-			$price = give_get_lowest_price_option( $form_id );
+			$price = give_get_lowest_price_option($form_id);
 		}
 
-		$price = give_sanitize_amount( $price );
+		$price = give_sanitize_amount($price);
 
 	} else {
 
-		$price = give_get_form_price( $form_id );
+		$price = give_get_form_price($form_id);
 
 	}
 
-	$price           = apply_filters( 'give_form_price', give_sanitize_amount( $price ), $form_id );
-	$formatted_price = '<span class="give_price" id="give_price_' . $form_id . '">' . $price . '</span>';
-	$formatted_price = apply_filters( 'give_form_price_after_html', $formatted_price, $form_id, $price );
+	$price           = apply_filters('give_form_price', give_sanitize_amount($price), $form_id);
+	$formatted_price = '<span class="give_price" id="give_price_'.$form_id.'">'.$price.'</span>';
+	$formatted_price = apply_filters('give_form_price_after_html', $formatted_price, $form_id, $price);
 
-	if ( $echo ) {
+	if ($echo) {
 		echo $formatted_price;
 	} else {
 		return $formatted_price;
 	}
 }
 
-add_filter( 'give_form_price', 'give_format_amount', 10 );
-add_filter( 'give_form_price', 'give_currency_filter', 20 );
+add_filter('give_form_price', 'give_format_amount', 10);
+add_filter('give_form_price', 'give_currency_filter', 20);
 
 
 /**
@@ -771,18 +771,18 @@ 
  *
  * @return float $amount Amount of the price option
  */
-function give_get_price_option_amount( $form_id = 0, $price_id = 0 ) {
-	$prices = give_get_variable_prices( $form_id );
+function give_get_price_option_amount($form_id = 0, $price_id = 0) {
+	$prices = give_get_variable_prices($form_id);
 
 	$amount = 0.00;
 
-	foreach ( $prices as $price ) {
-		if ( isset( $price['_give_id']['level_id'] ) && $price['_give_id']['level_id'] == $price_id ) {
-			$amount = isset( $price['_give_amount'] ) ? $price['_give_amount'] : 0.00;
+	foreach ($prices as $price) {
+		if (isset($price['_give_id']['level_id']) && $price['_give_id']['level_id'] == $price_id) {
+			$amount = isset($price['_give_amount']) ? $price['_give_amount'] : 0.00;
 		};
 	}
 
-	return apply_filters( 'give_get_price_option_amount', give_sanitize_amount( $amount ), $form_id, $price_id );
+	return apply_filters('give_get_price_option_amount', give_sanitize_amount($amount), $form_id, $price_id);
 }
 
 /**
@@ -794,13 +794,13 @@ 
  *
  * @return mixed string|int Goal of the form
  */
-function give_get_form_goal( $form_id = 0 ) {
+function give_get_form_goal($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
 	return $form->goal;
 
@@ -816,27 +816,27 @@ 
  *
  * @return string $formatted_goal
  */
-function give_goal( $form_id = 0, $echo = true ) {
+function give_goal($form_id = 0, $echo = true) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	$goal = give_get_form_goal( $form_id );
+	$goal = give_get_form_goal($form_id);
 
-	$goal           = apply_filters( 'give_form_goal', give_sanitize_amount( $goal ), $form_id );
-	$formatted_goal = '<span class="give_price" id="give_price_' . $form_id . '">' . $goal . '</span>';
-	$formatted_goal = apply_filters( 'give_form_price_after_html', $formatted_goal, $form_id, $goal );
+	$goal           = apply_filters('give_form_goal', give_sanitize_amount($goal), $form_id);
+	$formatted_goal = '<span class="give_price" id="give_price_'.$form_id.'">'.$goal.'</span>';
+	$formatted_goal = apply_filters('give_form_price_after_html', $formatted_goal, $form_id, $goal);
 
-	if ( $echo ) {
+	if ($echo) {
 		echo $formatted_goal;
 	} else {
 		return $formatted_goal;
 	}
 }
 
-add_filter( 'give_form_goal', 'give_format_amount', 10 );
-add_filter( 'give_form_goal', 'give_currency_filter', 20 );
+add_filter('give_form_goal', 'give_format_amount', 10);
+add_filter('give_form_goal', 'give_currency_filter', 20);
 
 
 /**
@@ -846,13 +846,13 @@ 
  * @global $give_options
  * @return bool $ret Whether or not the logged_in_only setting is set
  */
-function give_logged_in_only( $form_id ) {
+function give_logged_in_only($form_id) {
 
-	$form_option = get_post_meta( $form_id, '_give_logged_in_only', true );
+	$form_option = get_post_meta($form_id, '_give_logged_in_only', true);
 
-	$ret = ! empty( $form_option ) ? $form_option : false;
+	$ret = ! empty($form_option) ? $form_option : false;
 
-	return (bool) apply_filters( 'give_logged_in_only', $ret, $form_id );
+	return (bool) apply_filters('give_logged_in_only', $ret, $form_id);
 
 }
 
@@ -864,10 +864,10 @@ 
  *
  * @param $form_id
  */
-function give_show_login_register_option( $form_id ) {
+function give_show_login_register_option($form_id) {
 
-	$show_register_form = get_post_meta( $form_id, '_give_show_register_form', true );
+	$show_register_form = get_post_meta($form_id, '_give_show_register_form', true);
 
-	return apply_filters( 'give_show_register_form', $show_register_form, $form_id );
+	return apply_filters('give_show_register_form', $show_register_form, $form_id);
 
 }
\ No newline at end of file

Doc Comments +2 added lines, -2 removed lines

@@ -156,7 +156,7 @@ 
  * Used to redirect a user back to the purchase
  * page if there are errors present.
  *
- * @param array $args
+ * @param string $args
  *
  * @access public
  * @since  1.0
@@ -690,7 +690,7 @@ 
  *
  * @param int $form_id ID number of the form to retrieve the minimum price for
  *
- * @return mixed string|int Minimum price of the form
+ * @return string string|int Minimum price of the form
  */
 function give_get_form_minimum_price( $form_id = 0 ) {
 

Indentation +1 added lines, -1 removed lines

@@ -781,7 +781,7 @@
 	foreach ( $prices as $price ) {
 		if ( isset( $price['_give_id']['level_id'] ) && $price['_give_id']['level_id'] == $price_id ) {
 			$amount = isset( $price['_give_amount'] ) ? $price['_give_amount'] : 0.00;
-            break;
+			break;
 		};
 	}
 

includes/error-tracking.php

Doc Comments +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
  *
  * @since 1.0
  * @uses  Give_Session::get()
- * @return mixed array if errors are present, false if none found
+ * @return string array if errors are present, false if none found
  */
 function give_get_errors() {
 	return Give()->session->get( 'give_errors' );

Spacing +30 added lines, -30 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -29,33 +29,33 @@ 
  *
  * @return void
  */
-function give_print_errors( $form_id ) {
+function give_print_errors($form_id) {
 
 	$errors = give_get_errors();
 
-	$request_form_id = isset( $_REQUEST['form-id'] ) ? intval( $_REQUEST['form-id'] ) : 0;
+	$request_form_id = isset($_REQUEST['form-id']) ? intval($_REQUEST['form-id']) : 0;
 
 	//Sanity checks first: Ensure that gateway returned errors display on the appropriate form
-	if ( ! isset( $_POST['give_ajax'] ) && $request_form_id !== $form_id ) {
+	if ( ! isset($_POST['give_ajax']) && $request_form_id !== $form_id) {
 		return;
 	}
 
-	if ( $errors ) {
-		$classes = apply_filters( 'give_error_class', array(
+	if ($errors) {
+		$classes = apply_filters('give_error_class', array(
 			'give_errors'
-		) );
-		echo '<div class="' . implode( ' ', $classes ) . '">';
+		));
+		echo '<div class="'.implode(' ', $classes).'">';
 		// Loop error codes and display errors
-		foreach ( $errors as $error_id => $error ) {
-			echo '<div class="give_error" id="give_error_' . $error_id . '"><p><strong>' . esc_html__( 'Error', 'give' ) . '</strong>: ' . $error . '</p></div>';
+		foreach ($errors as $error_id => $error) {
+			echo '<div class="give_error" id="give_error_'.$error_id.'"><p><strong>'.esc_html__('Error', 'give').'</strong>: '.$error.'</p></div>';
 		}
 		echo '</div>';
 		give_clear_errors();
 	}
 }
 
-add_action( 'give_purchase_form_before_personal_info', 'give_print_errors' );
-add_action( 'give_ajax_checkout_errors', 'give_print_errors' );
+add_action('give_purchase_form_before_personal_info', 'give_print_errors');
+add_action('give_ajax_checkout_errors', 'give_print_errors');
 
 /**
  * Get Errors
@@ -68,7 +68,7 @@ 
  * @return mixed array if errors are present, false if none found
  */
 function give_get_errors() {
-	return Give()->session->get( 'give_errors' );
+	return Give()->session->get('give_errors');
 }
 
 /**
@@ -84,13 +84,13 @@ 
  *
  * @return void
  */
-function give_set_error( $error_id, $error_message ) {
+function give_set_error($error_id, $error_message) {
 	$errors = give_get_errors();
-	if ( ! $errors ) {
+	if ( ! $errors) {
 		$errors = array();
 	}
-	$errors[ $error_id ] = $error_message;
-	Give()->session->set( 'give_errors', $errors );
+	$errors[$error_id] = $error_message;
+	Give()->session->set('give_errors', $errors);
 }
 
 /**
@@ -101,7 +101,7 @@ 
  * @return void
  */
 function give_clear_errors() {
-	Give()->session->set( 'give_errors', null );
+	Give()->session->set('give_errors', null);
 }
 
 /**
@@ -114,11 +114,11 @@ 
  *
  * @return string
  */
-function give_unset_error( $error_id ) {
+function give_unset_error($error_id) {
 	$errors = give_get_errors();
-	if ( $errors ) {
-		unset( $errors[ $error_id ] );
-		Give()->session->set( 'give_errors', $errors );
+	if ($errors) {
+		unset($errors[$error_id]);
+		Give()->session->set('give_errors', $errors);
 	}
 }
 
@@ -129,7 +129,7 @@ 
  * @return void
  */
 function _give_die_handler() {
-	if ( defined( 'GIVE_UNIT_TESTS' ) ) {
+	if (defined('GIVE_UNIT_TESTS')) {
 		return '_give_die_handler';
 	} else {
 		die();
@@ -144,10 +144,10 @@ 
  * @since  1.0
  * @return void
  */
-function give_die( $message = '', $title = '', $status = 400 ) {
-	add_filter( 'wp_die_ajax_handler', '_give_die_handler', 10, 3 );
-	add_filter( 'wp_die_handler', '_give_die_handler', 10, 3 );
-	wp_die( $message, $title, array( 'response' => $status ) );
+function give_die($message = '', $title = '', $status = 400) {
+	add_filter('wp_die_ajax_handler', '_give_die_handler', 10, 3);
+	add_filter('wp_die_handler', '_give_die_handler', 10, 3);
+	wp_die($message, $title, array('response' => $status));
 }
 
 /**
@@ -162,10 +162,10 @@ 
  *
  * @return   string  $error
  */
-function give_output_error( $message, $echo = true, $error_id = 'warning' ) {
-	$error = '<div class="give_errors" id="give_error_' . $error_id . '"><p class="give_error  give_' . $error_id . '">' . $message . '</p></div>';
+function give_output_error($message, $echo = true, $error_id = 'warning') {
+	$error = '<div class="give_errors" id="give_error_'.$error_id.'"><p class="give_error  give_'.$error_id.'">'.$message.'</p></div>';
 
-	if ( $echo ) {
+	if ($echo) {
 		echo $error;
 	} else {
 		return $error;

includes/login-register.php

Indentation +24 added lines, -24 removed lines

@@ -33,20 +33,20 @@ 
 		$login_redirect = add_query_arg('give-login-success', 'true', give_get_current_page_url());
 	}
 
-    if ( empty( $logout_redirect ) ) {
-        $logout_redirect = add_query_arg( 'give-logout-success', 'true', give_get_current_page_url() );
-    }
+	if ( empty( $logout_redirect ) ) {
+		$logout_redirect = add_query_arg( 'give-logout-success', 'true', give_get_current_page_url() );
+	}
 
 
-    // Add user_logout action to logout url.
-    $logout_redirect = add_query_arg(
-        array(
-            'give_action'       => 'user_logout',
-            'give_logout_nonce' => wp_create_nonce( 'give-logout-nonce' ),
-            'give_logout_redirect' => urlencode( $logout_redirect )
-        ),
-        home_url('/')
-    );
+	// Add user_logout action to logout url.
+	$logout_redirect = add_query_arg(
+		array(
+			'give_action'       => 'user_logout',
+			'give_logout_nonce' => wp_create_nonce( 'give-logout-nonce' ),
+			'give_logout_redirect' => urlencode( $logout_redirect )
+		),
+		home_url('/')
+	);
 
 	$give_login_redirect = $login_redirect;
 	$give_logout_redirect = $logout_redirect;
@@ -136,23 +136,23 @@ 
  * @return void
  */
 function give_process_user_logout( $data ) {
-    if ( wp_verify_nonce( $data['give_logout_nonce'], 'give-logout-nonce' ) && is_user_logged_in() ) {
+	if ( wp_verify_nonce( $data['give_logout_nonce'], 'give-logout-nonce' ) && is_user_logged_in() ) {
 
-        // Prevent occurring of any custom action on wp_logout.
-        remove_all_actions( 'wp_logout' );
+		// Prevent occurring of any custom action on wp_logout.
+		remove_all_actions( 'wp_logout' );
 
-        // Before logout give action.
-        do_action( 'give_before_user_logout' );
+		// Before logout give action.
+		do_action( 'give_before_user_logout' );
 
-        // Logout user.
-        wp_logout();
+		// Logout user.
+		wp_logout();
 
-        // After logout give action.
-        do_action( 'give_after_user_logout' );
+		// After logout give action.
+		do_action( 'give_after_user_logout' );
 
-        wp_redirect( $data['give_logout_redirect'] );
-        give_die();
-    }
+		wp_redirect( $data['give_logout_redirect'] );
+		give_die();
+	}
 }
 
 add_action( 'give_user_logout', 'give_process_user_logout' );

Spacing +71 added lines, -71 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -26,15 +26,15 @@ 
  *
  * @return string Login form
  */
-function give_login_form( $login_redirect = '', $logout_redirect = '' ) {
-	global $give_login_redirect, $give_logout_redirect;;
+function give_login_form($login_redirect = '', $logout_redirect = '') {
+	global $give_login_redirect, $give_logout_redirect; ;
 
-	if ( empty( $login_redirect ) ) {
+	if (empty($login_redirect)) {
 		$login_redirect = add_query_arg('give-login-success', 'true', give_get_current_page_url());
 	}
 
-    if ( empty( $logout_redirect ) ) {
-        $logout_redirect = add_query_arg( 'give-logout-success', 'true', give_get_current_page_url() );
+    if (empty($logout_redirect)) {
+        $logout_redirect = add_query_arg('give-logout-success', 'true', give_get_current_page_url());
     }
 
 
@@ -42,8 +42,8 @@ 
     $logout_redirect = add_query_arg(
         array(
             'give_action'       => 'user_logout',
-            'give_logout_nonce' => wp_create_nonce( 'give-logout-nonce' ),
-            'give_logout_redirect' => urlencode( $logout_redirect )
+            'give_logout_nonce' => wp_create_nonce('give-logout-nonce'),
+            'give_logout_redirect' => urlencode($logout_redirect)
         ),
         home_url('/')
     );
@@ -53,9 +53,9 @@ 
 
 	ob_start();
 
-	give_get_template_part( 'shortcode', 'login' );
+	give_get_template_part('shortcode', 'login');
 
-	return apply_filters( 'give_login_form', ob_get_clean() );
+	return apply_filters('give_login_form', ob_get_clean());
 }
 
 /**
@@ -69,10 +69,10 @@ 
  *
  * @return string Register form
  */
-function give_register_form( $redirect = '' ) {
+function give_register_form($redirect = '') {
 	global $give_options, $give_register_redirect;
 
-	if ( empty( $redirect ) ) {
+	if (empty($redirect)) {
 		$redirect = give_get_current_page_url();
 	}
 
@@ -80,11 +80,11 @@ 
 
 	ob_start();
 
-	if ( ! is_user_logged_in() ) {
-		give_get_template_part( 'shortcode', 'register' );
+	if ( ! is_user_logged_in()) {
+		give_get_template_part('shortcode', 'register');
 	}
 
-	return apply_filters( 'give_register_form', ob_get_clean() );
+	return apply_filters('give_register_form', ob_get_clean());
 }
 
 /**
@@ -96,34 +96,34 @@ 
  *
  * @return void
  */
-function give_process_login_form( $data ) {
-	if ( wp_verify_nonce( $data['give_login_nonce'], 'give-login-nonce' ) ) {
-		$user_data = get_user_by( 'login', $data['give_user_login'] );
-		if ( ! $user_data ) {
-			$user_data = get_user_by( 'email', $data['give_user_login'] );
+function give_process_login_form($data) {
+	if (wp_verify_nonce($data['give_login_nonce'], 'give-login-nonce')) {
+		$user_data = get_user_by('login', $data['give_user_login']);
+		if ( ! $user_data) {
+			$user_data = get_user_by('email', $data['give_user_login']);
 		}
-		if ( $user_data ) {
+		if ($user_data) {
 			$user_ID    = $user_data->ID;
 			$user_email = $user_data->user_email;
-			if ( wp_check_password( $data['give_user_pass'], $user_data->user_pass, $user_data->ID ) ) {
-				give_log_user_in( $user_data->ID, $data['give_user_login'], $data['give_user_pass'] );
+			if (wp_check_password($data['give_user_pass'], $user_data->user_pass, $user_data->ID)) {
+				give_log_user_in($user_data->ID, $data['give_user_login'], $data['give_user_pass']);
 			} else {
-				give_set_error( 'password_incorrect', esc_html__( 'The password you entered is incorrect.', 'give' ) );
+				give_set_error('password_incorrect', esc_html__('The password you entered is incorrect.', 'give'));
 			}
 		} else {
-			give_set_error( 'username_incorrect', esc_html__( 'The username you entered does not exist.', 'give' ) );
+			give_set_error('username_incorrect', esc_html__('The username you entered does not exist.', 'give'));
 		}
 		// Check for errors and redirect if none present
 		$errors = give_get_errors();
-		if ( ! $errors ) {
-			$redirect = apply_filters( 'give_login_redirect', $data['give_login_redirect'], $user_ID );
-			wp_redirect( $redirect );
+		if ( ! $errors) {
+			$redirect = apply_filters('give_login_redirect', $data['give_login_redirect'], $user_ID);
+			wp_redirect($redirect);
 			give_die();
 		}
 	}
 }
 
-add_action( 'give_user_login', 'give_process_login_form' );
+add_action('give_user_login', 'give_process_login_form');
 
 
 /**
@@ -135,27 +135,27 @@ 
  *
  * @return void
  */
-function give_process_user_logout( $data ) {
-    if ( wp_verify_nonce( $data['give_logout_nonce'], 'give-logout-nonce' ) && is_user_logged_in() ) {
+function give_process_user_logout($data) {
+    if (wp_verify_nonce($data['give_logout_nonce'], 'give-logout-nonce') && is_user_logged_in()) {
 
         // Prevent occurring of any custom action on wp_logout.
-        remove_all_actions( 'wp_logout' );
+        remove_all_actions('wp_logout');
 
         // Before logout give action.
-        do_action( 'give_before_user_logout' );
+        do_action('give_before_user_logout');
 
         // Logout user.
         wp_logout();
 
         // After logout give action.
-        do_action( 'give_after_user_logout' );
+        do_action('give_after_user_logout');
 
-        wp_redirect( $data['give_logout_redirect'] );
+        wp_redirect($data['give_logout_redirect']);
         give_die();
     }
 }
 
-add_action( 'give_user_logout', 'give_process_user_logout' );
+add_action('give_user_logout', 'give_process_user_logout');
 
 /**
  * Log User In
@@ -168,15 +168,15 @@ 
  *
  * @return void
  */
-function give_log_user_in( $user_id, $user_login, $user_pass ) {
-	if ( $user_id < 1 ) {
+function give_log_user_in($user_id, $user_login, $user_pass) {
+	if ($user_id < 1) {
 		return;
 	}
 
-	wp_set_auth_cookie( $user_id );
-	wp_set_current_user( $user_id, $user_login );
-	do_action( 'wp_login', $user_login, get_userdata( $user_id ) );
-	do_action( 'give_log_user_in', $user_id, $user_login, $user_pass );
+	wp_set_auth_cookie($user_id);
+	wp_set_current_user($user_id, $user_login);
+	do_action('wp_login', $user_login, get_userdata($user_id));
+	do_action('give_log_user_in', $user_id, $user_login, $user_pass);
 }
 
 
@@ -189,70 +189,70 @@ 
  *
  * @return void
  */
-function give_process_register_form( $data ) {
+function give_process_register_form($data) {
 
-	if ( is_user_logged_in() ) {
+	if (is_user_logged_in()) {
 		return;
 	}
 
-	if ( empty( $_POST['give_register_submit'] ) ) {
+	if (empty($_POST['give_register_submit'])) {
 		return;
 	}
 
-	do_action( 'give_pre_process_register_form' );
+	do_action('give_pre_process_register_form');
 
-	if ( empty( $data['give_user_login'] ) ) {
-		give_set_error( 'empty_username', esc_html__( 'Invalid username.', 'give' ) );
+	if (empty($data['give_user_login'])) {
+		give_set_error('empty_username', esc_html__('Invalid username.', 'give'));
 	}
 
-	if ( username_exists( $data['give_user_login'] ) ) {
-		give_set_error( 'username_unavailable', esc_html__( 'Username already taken.', 'give' ) );
+	if (username_exists($data['give_user_login'])) {
+		give_set_error('username_unavailable', esc_html__('Username already taken.', 'give'));
 	}
 
-	if ( ! validate_username( $data['give_user_login'] ) ) {
-		give_set_error( 'username_invalid', esc_html__( 'Invalid username.', 'give' ) );
+	if ( ! validate_username($data['give_user_login'])) {
+		give_set_error('username_invalid', esc_html__('Invalid username.', 'give'));
 	}
 
-	if ( email_exists( $data['give_user_email'] ) ) {
-		give_set_error( 'email_unavailable', esc_html__( 'Email address already taken.', 'give' ) );
+	if (email_exists($data['give_user_email'])) {
+		give_set_error('email_unavailable', esc_html__('Email address already taken.', 'give'));
 	}
 
-	if ( empty( $data['give_user_email'] ) || ! is_email( $data['give_user_email'] ) ) {
-		give_set_error( 'email_invalid', esc_html__( 'Invalid email.', 'give' ) );
+	if (empty($data['give_user_email']) || ! is_email($data['give_user_email'])) {
+		give_set_error('email_invalid', esc_html__('Invalid email.', 'give'));
 	}
 
-	if ( ! empty( $data['give_payment_email'] ) && $data['give_payment_email'] != $data['give_user_email'] && ! is_email( $data['give_payment_email'] ) ) {
-		give_set_error( 'payment_email_invalid', esc_html__( 'Invalid payment email.', 'give' ) );
+	if ( ! empty($data['give_payment_email']) && $data['give_payment_email'] != $data['give_user_email'] && ! is_email($data['give_payment_email'])) {
+		give_set_error('payment_email_invalid', esc_html__('Invalid payment email.', 'give'));
 	}
 
-	if ( empty( $_POST['give_user_pass'] ) ) {
-		give_set_error( 'empty_password', esc_html__( 'Please enter a password.', 'give' ) );
+	if (empty($_POST['give_user_pass'])) {
+		give_set_error('empty_password', esc_html__('Please enter a password.', 'give'));
 	}
 
-	if ( ( ! empty( $_POST['give_user_pass'] ) && empty( $_POST['give_user_pass2'] ) ) || ( $_POST['give_user_pass'] !== $_POST['give_user_pass2'] ) ) {
-		give_set_error( 'password_mismatch', esc_html__( 'Passwords don\'t match.', 'give' ) );
+	if (( ! empty($_POST['give_user_pass']) && empty($_POST['give_user_pass2'])) || ($_POST['give_user_pass'] !== $_POST['give_user_pass2'])) {
+		give_set_error('password_mismatch', esc_html__('Passwords don\'t match.', 'give'));
 	}
 
-	do_action( 'give_process_register_form' );
+	do_action('give_process_register_form');
 
 	// Check for errors and redirect if none present
 	$errors = give_get_errors();
 
-	if ( empty( $errors ) ) {
+	if (empty($errors)) {
 
-		$redirect = apply_filters( 'give_register_redirect', $data['give_redirect'] );
+		$redirect = apply_filters('give_register_redirect', $data['give_redirect']);
 
-		give_register_and_login_new_user( array(
+		give_register_and_login_new_user(array(
 			'user_login'      => $data['give_user_login'],
 			'user_pass'       => $data['give_user_pass'],
 			'user_email'      => $data['give_user_email'],
-			'user_registered' => date( 'Y-m-d H:i:s' ),
-			'role'            => get_option( 'default_role' )
-		) );
+			'user_registered' => date('Y-m-d H:i:s'),
+			'role'            => get_option('default_role')
+		));
 
-		wp_redirect( $redirect );
+		wp_redirect($redirect);
 		give_die();
 	}
 }
 
-add_action( 'give_user_register', 'give_process_register_form' );
\ No newline at end of file
+add_action('give_user_register', 'give_process_register_form');
\ No newline at end of file

includes/shortcodes.php

Indentation +5 added lines, -5 removed lines

@@ -91,7 +91,7 @@ 
 
 		//validate display_style and float_labels value
 		if ( ( $key == 'display_style' && ! in_array( $value, array( 'onpage', 'reveal', 'modal' ) ) )
-		     || ( $key == 'float_labels' && ! in_array( $value, array( 'enabled', 'disabled' ) ) )
+			 || ( $key == 'float_labels' && ! in_array( $value, array( 'enabled', 'disabled' ) ) )
 		) {
 
 			$atts[ $key ] = '';
@@ -169,15 +169,15 @@ 
  */
 function give_login_form_shortcode( $atts, $content = null ) {
 	$atts = shortcode_atts( array(
-        // Add backward compatibility for redirect attribute.
-        'redirect'          => '',
+		// Add backward compatibility for redirect attribute.
+		'redirect'          => '',
 
 		'login-redirect'    => '',
 		'logout-redirect'   => '',
 	), $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'] : '' );
+	// 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'] : '' );
 
 	return give_login_form( $atts['login-redirect'], $atts['logout-redirect'] );
 }

Spacing +109 added lines, -109 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,28 +25,28 @@ 
  */
 function give_donation_history() {
 
-	$email_access = give_get_option( 'email_access' );
+	$email_access = give_get_option('email_access');
 
 	//Is user logged in? Does a session exist? Does an email-access token exist?
-	if ( is_user_logged_in() || Give()->session->get_session_expiration() !== false || ( $email_access == 'on' && Give()->email_access->token_exists ) ) {
+	if (is_user_logged_in() || Give()->session->get_session_expiration() !== false || ($email_access == 'on' && Give()->email_access->token_exists)) {
 		ob_start();
-		give_get_template_part( 'history', 'donations' );
+		give_get_template_part('history', 'donations');
 
 		return ob_get_clean();
 	} //Is Email-based access enabled?
-	elseif ( $email_access == 'on' ) {
+	elseif ($email_access == 'on') {
 
 		ob_start();
-		give_get_template_part( 'email', 'login-form' );
+		give_get_template_part('email', 'login-form');
 
 		return ob_get_clean();
 	} else {
-		$message = esc_html__( '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' );
-		echo apply_filters( 'give_donation_history_nonuser_message', give_output_error( $message, false ), $message );
+		$message = esc_html__('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');
+		echo apply_filters('give_donation_history_nonuser_message', give_output_error($message, false), $message);
 	}
 }
 
-add_shortcode( 'donation_history', 'give_donation_history' );
+add_shortcode('donation_history', 'give_donation_history');
 
 /**
  * Donation Form Shortcode
@@ -60,53 +60,53 @@ 
  *
  * @return string
  */
-function give_form_shortcode( $atts, $content = null ) {
-	$atts = shortcode_atts( array(
+function give_form_shortcode($atts, $content = null) {
+	$atts = shortcode_atts(array(
 		'id'            => '',
 		'show_title'    => true,
 		'show_goal'     => true,
 		'show_content'  => '',
 		'float_labels'  => '',
 		'display_style' => '',
-	), $atts, 'give_form' );
+	), $atts, 'give_form');
 
-	foreach ( $atts as $key => $value ) {
+	foreach ($atts as $key => $value) {
 		//convert shortcode_atts values to booleans
-		if ( $key == 'show_title' ) {
-			$atts[ $key ] = filter_var( $atts[ $key ], FILTER_VALIDATE_BOOLEAN );
-		} elseif ( $key == 'show_goal' ) {
-			$atts[ $key ] = filter_var( $atts[ $key ], FILTER_VALIDATE_BOOLEAN );
+		if ($key == 'show_title') {
+			$atts[$key] = filter_var($atts[$key], FILTER_VALIDATE_BOOLEAN);
+		} elseif ($key == 'show_goal') {
+			$atts[$key] = filter_var($atts[$key], FILTER_VALIDATE_BOOLEAN);
 		}
 
 		//validate show_content value
-		if ( $key == 'show_content' ) {
-			if ( ! in_array( $value, array( 'none', 'above', 'below' ) ) ) {
-				$atts[ $key ] = '';
-			} else if ( $value == 'above' ) {
-				$atts[ $key ] = 'give_pre_form';
-			} else if ( $value == 'below' ) {
-				$atts[ $key ] = 'give_post_form';
+		if ($key == 'show_content') {
+			if ( ! in_array($value, array('none', 'above', 'below'))) {
+				$atts[$key] = '';
+			} else if ($value == 'above') {
+				$atts[$key] = 'give_pre_form';
+			} else if ($value == 'below') {
+				$atts[$key] = 'give_post_form';
 			}
 		}
 
 		//validate display_style and float_labels value
-		if ( ( $key == 'display_style' && ! in_array( $value, array( 'onpage', 'reveal', 'modal' ) ) )
-		     || ( $key == 'float_labels' && ! in_array( $value, array( 'enabled', 'disabled' ) ) )
+		if (($key == 'display_style' && ! in_array($value, array('onpage', 'reveal', 'modal')))
+		     || ($key == 'float_labels' && ! in_array($value, array('enabled', 'disabled')))
 		) {
 
-			$atts[ $key ] = '';
+			$atts[$key] = '';
 		}
 	}
 
 	//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
@@ -120,37 +120,37 @@ 
  *
  * @return string
  */
-function give_goal_shortcode( $atts, $content = null ) {
-	$atts = shortcode_atts( array(
+function give_goal_shortcode($atts, $content = null) {
+	$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_output_error( esc_html__( 'Error: No Donation form ID for the shortcode provided.', 'give' ), true );
+	if (empty($atts['id'])) {
+		give_output_error(esc_html__('Error: No Donation form ID for the shortcode provided.', 'give'), true);
 	}
 
 	//Sanity check 2: Check that this form even has Goals enabled
-	$goal_option = get_post_meta( $atts['id'], '_give_goal_option', true );
-	if ( empty( $goal_option ) || $goal_option !== 'yes' ) {
-		give_output_error( esc_html__( 'Error: This form does not have Goals enabled.', 'give' ), true );
+	$goal_option = get_post_meta($atts['id'], '_give_goal_option', true);
+	if (empty($goal_option) || $goal_option !== 'yes') {
+		give_output_error(esc_html__('Error: This 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');
 
 
 /**
@@ -167,22 +167,22 @@ 
  * @uses  give_login_form()
  * @return string
  */
-function give_login_form_shortcode( $atts, $content = null ) {
-	$atts = shortcode_atts( array(
+function give_login_form_shortcode($atts, $content = null) {
+	$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
@@ -197,15 +197,15 @@ 
  * @uses  give_register_form()
  * @return string
  */
-function give_register_form_shortcode( $atts, $content = null ) {
-	$atts = shortcode_atts( array(
+function give_register_form_shortcode($atts, $content = null) {
+	$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');
 
 
 /**
@@ -220,62 +220,62 @@ 
  *
  * @return string
  */
-function give_receipt_shortcode( $atts, $content = null ) {
+function give_receipt_shortcode($atts, $content = null) {
 
 	global $give_receipt_args, $payment;
 
-	$give_receipt_args = shortcode_atts( array(
-		'error'          => esc_html__( 'Sorry, you are missing the payment key to view this donation receipt.', 'give' ),
+	$give_receipt_args = shortcode_atts(array(
+		'error'          => esc_html__('Sorry, you are missing the payment key to view this donation receipt.', 'give'),
 		'price'          => true,
 		'donor'          => true,
 		'date'           => true,
 		'payment_key'    => false,
 		'payment_method' => true,
 		'payment_id'     => true
-	), $atts, 'give_receipt' );
+	), $atts, 'give_receipt');
 
 	//set $session var
 	$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 ) && $email_access == 'on' && ! Give()->email_access->token_exists ) {
+	if ( ! isset($payment_key) && $email_access == 'on' && ! 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_output_error( $give_receipt_args['error'], false, 'error' );
+		return give_output_error($give_receipt_args['error'], false, 'error');
 
 	}
 
-	$payment_id    = give_get_purchase_id_by_key( $payment_key );
-	$user_can_view = give_can_view_receipt( $payment_key );
+	$payment_id    = give_get_purchase_id_by_key($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 && $email_access == 'on' && ! Give()->email_access->token_exists ) {
+	if ( ! $user_can_view && $email_access == 'on' && ! 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;
 
@@ -283,9 +283,9 @@ 
 
 		ob_start();
 
-		give_output_error( apply_filters( 'give_must_be_logged_in_error_message', esc_html__( 'You must be logged in to view this donation payment receipt.', 'give' ) ) );
+		give_output_error(apply_filters('give_must_be_logged_in_error_message', esc_html__('You must be logged in to view this donation payment receipt.', 'give')));
 
-		give_get_template_part( 'shortcode', 'login' );
+		give_get_template_part('shortcode', 'login');
 
 		$login_form = ob_get_clean();
 
@@ -302,13 +302,13 @@ 
 	 * 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_output_error( $give_receipt_args['error'], false, 'error' );
+	if ( ! apply_filters('give_user_can_view_receipt', $user_can_view, $give_receipt_args)) {
+		return give_output_error($give_receipt_args['error'], false, 'error');
 	}
 
 	ob_start();
 
-	give_get_template_part( 'shortcode', 'receipt' );
+	give_get_template_part('shortcode', 'receipt');
 
 	$display = ob_get_clean();
 
@@ -317,7 +317,7 @@ 
 
 }
 
-add_shortcode( 'give_receipt', 'give_receipt_shortcode' );
+add_shortcode('give_receipt', 'give_receipt_shortcode');
 
 /**
  * Profile Editor Shortcode
@@ -337,18 +337,18 @@ 
  *
  * @return string Output generated from the profile editor
  */
-function give_profile_editor_shortcode( $atts, $content = null ) {
+function give_profile_editor_shortcode($atts, $content = null) {
 
 	ob_start();
 
-	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
@@ -361,30 +361,30 @@ 
  *
  * @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 );
-
-	$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;
-	$email        = isset( $data['give_email'] ) ? sanitize_email( $data['give_email'] ) : $old_user_data->user_email;
-	$line1        = ( isset( $data['give_address_line1'] ) ? sanitize_text_field( $data['give_address_line1'] ) : '' );
-	$line2        = ( isset( $data['give_address_line2'] ) ? sanitize_text_field( $data['give_address_line2'] ) : '' );
-	$city         = ( isset( $data['give_address_city'] ) ? sanitize_text_field( $data['give_address_city'] ) : '' );
-	$state        = ( isset( $data['give_address_state'] ) ? sanitize_text_field( $data['give_address_state'] ) : '' );
-	$zip          = ( isset( $data['give_address_zip'] ) ? sanitize_text_field( $data['give_address_zip'] ) : '' );
-	$country      = ( isset( $data['give_address_country'] ) ? sanitize_text_field( $data['give_address_country'] ) : '' );
+	$old_user_data = get_userdata($user_id);
+
+	$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;
+	$email        = isset($data['give_email']) ? sanitize_email($data['give_email']) : $old_user_data->user_email;
+	$line1        = (isset($data['give_address_line1']) ? sanitize_text_field($data['give_address_line1']) : '');
+	$line2        = (isset($data['give_address_line2']) ? sanitize_text_field($data['give_address_line2']) : '');
+	$city         = (isset($data['give_address_city']) ? sanitize_text_field($data['give_address_city']) : '');
+	$state        = (isset($data['give_address_state']) ? sanitize_text_field($data['give_address_state']) : '');
+	$zip          = (isset($data['give_address_zip']) ? sanitize_text_field($data['give_address_zip']) : '');
+	$country      = (isset($data['give_address_country']) ? sanitize_text_field($data['give_address_country']) : '');
 
 	$userdata = array(
 		'ID'           => $user_id,
@@ -404,45 +404,45 @@ 
 		'country' => $country
 	);
 
-	do_action( 'give_pre_update_user_profile', $user_id, $userdata );
+	do_action('give_pre_update_user_profile', $user_id, $userdata);
 
 	// New password
-	if ( ! empty( $data['give_new_user_pass1'] ) ) {
-		if ( $data['give_new_user_pass1'] !== $data['give_new_user_pass2'] ) {
-			give_set_error( 'password_mismatch', esc_html__( 'The passwords you entered do not match. Please try again.', 'give' ) );
+	if ( ! empty($data['give_new_user_pass1'])) {
+		if ($data['give_new_user_pass1'] !== $data['give_new_user_pass2']) {
+			give_set_error('password_mismatch', esc_html__('The passwords you entered do not match. Please try again.', 'give'));
 		} else {
 			$userdata['user_pass'] = $data['give_new_user_pass1'];
 		}
 	}
 
 	// Make sure the new email doesn't belong to another user
-	if ( $email != $old_user_data->user_email ) {
-		if ( email_exists( $email ) ) {
-			give_set_error( 'email_exists', esc_html__( 'The email you entered belongs to another user. Please use another.', 'give' ) );
+	if ($email != $old_user_data->user_email) {
+		if (email_exists($email)) {
+			give_set_error('email_exists', esc_html__('The email you entered belongs to another user. 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 the user
-	$meta    = update_user_meta( $user_id, '_give_user_address', $address );
-	$updated = wp_update_user( $userdata );
+	$meta    = update_user_meta($user_id, '_give_user_address', $address);
+	$updated = wp_update_user($userdata);
 
-	if ( $updated ) {
-		do_action( 'give_user_profile_updated', $user_id, $userdata );
-		wp_redirect( add_query_arg( 'updated', 'true', $data['give_redirect'] ) );
+	if ($updated) {
+		do_action('give_user_profile_updated', $user_id, $userdata);
+		wp_redirect(add_query_arg('updated', 'true', $data['give_redirect']));
 		give_die();
 	}
 
 	return false;
 }
 
-add_action( 'give_edit_user_profile', 'give_process_profile_editor_updates' );
+add_action('give_edit_user_profile', 'give_process_profile_editor_updates');
 

includes/class-give-donate-form.php

Doc Comments +1 added lines, -1 removed lines

@@ -182,7 +182,7 @@
 	 *
 	 * @param  array $data Array of attributes for a donation form
 	 *
-	 * @return mixed  false if data isn't passed and class not instantiated for creation, or New Form ID
+	 * @return boolean  false if data isn't passed and class not instantiated for creation, or New Form ID
 	 */
 	public function create( $data = array() ) {
 

Indentation +25 added lines, -25 removed lines

@@ -463,31 +463,31 @@
 
 	}
 
-    /**
-     * Get if form type set or not.
-     *
-     * @since 1.6
-     * @return bool true if form type is 'multi' and false otherwise.
-     */
-    public function is_set_type_donation_form() {
-        $form_type = $this->get_type();
-
-        return ( 'set' === $form_type ? true : false );
-
-    }
-
-    /**
-     * Get if form type multi or not.
-     *
-     * @since 1.6
-     * @return bool true if form type is 'multi' and false otherwise.
-     */
-    public function is_multi_type_donation_form() {
-        $form_type = $this->get_type();
-
-        return ( 'multi' === $form_type ? true : false );
-
-    }
+	/**
+	 * Get if form type set or not.
+	 *
+	 * @since 1.6
+	 * @return bool true if form type is 'multi' and false otherwise.
+	 */
+	public function is_set_type_donation_form() {
+		$form_type = $this->get_type();
+
+		return ( 'set' === $form_type ? true : false );
+
+	}
+
+	/**
+	 * Get if form type multi or not.
+	 *
+	 * @since 1.6
+	 * @return bool true if form type is 'multi' and false otherwise.
+	 */
+	public function is_multi_type_donation_form() {
+		$form_type = $this->get_type();
+
+		return ( 'multi' === $form_type ? true : false );
+
+	}
 
 	/**
 	 * Retrieve the sale count for the donation form

Spacing +102 added lines, -102 removed lines

@@ -100,12 +100,12 @@ 
 	 * @param bool $_id
 	 * @param array $_args
 	 */
-	public function __construct( $_id = false, $_args = array() ) {
+	public function __construct($_id = false, $_args = array()) {
 
 
-		$donation_form = WP_Post::get_instance( $_id );
+		$donation_form = WP_Post::get_instance($_id);
 
-		return $this->setup_donation_form( $donation_form );
+		return $this->setup_donation_form($donation_form);
 	}
 
 	/**
@@ -119,23 +119,23 @@ 
 	 *
 	 * @return bool             If the setup was successful or not
 	 */
-	private function setup_donation_form( $donation_form ) {
+	private function setup_donation_form($donation_form) {
 
-		if ( ! is_object( $donation_form ) ) {
+		if ( ! is_object($donation_form)) {
 			return false;
 		}
 
-		if ( ! is_a( $donation_form, 'WP_Post' ) ) {
+		if ( ! is_a($donation_form, 'WP_Post')) {
 			return false;
 		}
 
-		if ( 'give_forms' !== $donation_form->post_type ) {
+		if ('give_forms' !== $donation_form->post_type) {
 			return false;
 		}
 
-		foreach ( $donation_form as $key => $value ) {
+		foreach ($donation_form as $key => $value) {
 
-			switch ( $key ) {
+			switch ($key) {
 
 				default:
 					$this->$key = $value;
@@ -159,16 +159,16 @@ 
 	 * @return mixed
 	 * @throws Exception
 	 */
-	public function __get( $key ) {
+	public function __get($key) {
 
-		if ( method_exists( $this, 'get_' . $key ) ) {
+		if (method_exists($this, 'get_'.$key)) {
 
-			return call_user_func( array( $this, 'get_' . $key ) );
+			return call_user_func(array($this, 'get_'.$key));
 
 		} else {
 
 			/* translators: %s: property key */
-			return new WP_Error( 'give-form-invalid-property', sprintf( esc_html__( 'Can\'t get property %s.', 'give' ), $key ) );
+			return new WP_Error('give-form-invalid-property', sprintf(esc_html__('Can\'t get property %s.', 'give'), $key));
 
 		}
 
@@ -184,30 +184,30 @@ 
 	 *
 	 * @return mixed  false if data isn't passed and class not instantiated for creation, or New Form ID
 	 */
-	public function create( $data = array() ) {
+	public function create($data = array()) {
 
-		if ( $this->id != 0 ) {
+		if ($this->id != 0) {
 			return false;
 		}
 
 		$defaults = array(
 			'post_type'   => 'give_forms',
 			'post_status' => 'draft',
-			'post_title'  => esc_html__( 'New Donation Form', 'give' )
+			'post_title'  => esc_html__('New Donation Form', 'give')
 		);
 
-		$args = wp_parse_args( $data, $defaults );
+		$args = wp_parse_args($data, $defaults);
 
 		/**
 		 * Fired before a donation form is created
 		 *
 		 * @param array $args The post object arguments used for creation.
 		 */
-		do_action( 'give_form_pre_create', $args );
+		do_action('give_form_pre_create', $args);
 
-		$id = wp_insert_post( $args, true );
+		$id = wp_insert_post($args, true);
 
-		$donation_form = WP_Post::get_instance( $id );
+		$donation_form = WP_Post::get_instance($id);
 
 		/**
 		 * Fired after a donation form is created
@@ -215,9 +215,9 @@ 
 		 * @param int $id The post ID of the created item.
 		 * @param array $args The post object arguments used for creation.
 		 */
-		do_action( 'give_form_post_create', $id, $args );
+		do_action('give_form_post_create', $id, $args);
 
-		return $this->setup_donation_form( $donation_form );
+		return $this->setup_donation_form($donation_form);
 
 	}
 
@@ -240,7 +240,7 @@ 
 	 * @return string Name of the donation form
 	 */
 	public function get_name() {
-		return get_the_title( $this->ID );
+		return get_the_title($this->ID);
 	}
 
 	/**
@@ -251,13 +251,13 @@ 
 	 */
 	public function get_price() {
 
-		if ( ! isset( $this->price ) ) {
+		if ( ! isset($this->price)) {
 
-			$this->price = get_post_meta( $this->ID, '_give_set_price', true );
+			$this->price = get_post_meta($this->ID, '_give_set_price', true);
 
-			if ( $this->price ) {
+			if ($this->price) {
 
-				$this->price = give_sanitize_amount( $this->price );
+				$this->price = give_sanitize_amount($this->price);
 
 			} else {
 
@@ -275,7 +275,7 @@ 
 		 * @param string $price The donation form price.
 		 * @param string|int $id The form ID.
 		 */
-		return apply_filters( 'give_get_set_price', $this->price, $this->ID );
+		return apply_filters('give_get_set_price', $this->price, $this->ID);
 	}
 
 	/**
@@ -286,14 +286,14 @@ 
 	 */
 	public function get_minimum_price() {
 
-		if ( ! isset( $this->minimum_price ) ) {
+		if ( ! isset($this->minimum_price)) {
 
-			$allow_custom_amount = get_post_meta( $this->ID, '_give_custom_amount', true );
-			$this->minimum_price = get_post_meta( $this->ID, '_give_custom_amount_minimum', true );
+			$allow_custom_amount = get_post_meta($this->ID, '_give_custom_amount', true);
+			$this->minimum_price = get_post_meta($this->ID, '_give_custom_amount_minimum', true);
 
-			if ( $allow_custom_amount != 'no' && $this->minimum_price ) {
+			if ($allow_custom_amount != 'no' && $this->minimum_price) {
 
-				$this->minimum_price = give_sanitize_amount( $this->minimum_price );
+				$this->minimum_price = give_sanitize_amount($this->minimum_price);
 
 			} else {
 
@@ -303,7 +303,7 @@ 
 
 		}
 
-		return apply_filters( 'give_get_set_minimum_price', $this->minimum_price, $this->ID );
+		return apply_filters('give_get_set_minimum_price', $this->minimum_price, $this->ID);
 	}
 
 	/**
@@ -314,9 +314,9 @@ 
 	 */
 	public function get_prices() {
 
-		if ( ! isset( $this->prices ) ) {
+		if ( ! isset($this->prices)) {
 
-			$this->prices = get_post_meta( $this->ID, '_give_donation_levels', true );
+			$this->prices = get_post_meta($this->ID, '_give_donation_levels', true);
 
 		}
 
@@ -328,7 +328,7 @@ 
 		 * @param array $prices The array of mulit-level prices.
 		 * @param int|string The ID of the form.
 		 */
-		return apply_filters( 'give_get_donation_levels', $this->prices, $this->ID );
+		return apply_filters('give_get_donation_levels', $this->prices, $this->ID);
 
 	}
 
@@ -340,13 +340,13 @@ 
 	 */
 	public function get_goal() {
 
-		if ( ! isset( $this->goal ) ) {
+		if ( ! isset($this->goal)) {
 
-			$this->goal = get_post_meta( $this->ID, '_give_set_goal', true );
+			$this->goal = get_post_meta($this->ID, '_give_set_goal', true);
 
-			if ( $this->goal ) {
+			if ($this->goal) {
 
-				$this->goal = give_sanitize_amount( $this->goal );
+				$this->goal = give_sanitize_amount($this->goal);
 
 			} else {
 
@@ -356,7 +356,7 @@ 
 
 		}
 
-		return apply_filters( 'give_get_set_goal', $this->goal, $this->ID );
+		return apply_filters('give_get_set_goal', $this->goal, $this->ID);
 
 	}
 
@@ -368,10 +368,10 @@ 
 	 */
 	public function is_single_price_mode() {
 
-		$option = get_post_meta( $this->ID, '_give_price_options_mode', true );
+		$option = get_post_meta($this->ID, '_give_price_options_mode', true);
 		$ret    = 0;
 
-		if ( empty( $option ) || $option === 'set' ) {
+		if (empty($option) || $option === 'set') {
 			$ret = 1;
 		}
 		
@@ -383,7 +383,7 @@ 
 		 * @param bool $ret Is donation form in single price mode?
 		 * @param int|string The ID of the donation form.
 		 */
-		return (bool) apply_filters( 'give_single_price_option_mode', $ret, $this->ID );
+		return (bool) apply_filters('give_single_price_option_mode', $ret, $this->ID);
 
 	}
 
@@ -395,10 +395,10 @@ 
 	 */
 	public function is_custom_price_mode() {
 
-		$option = get_post_meta( $this->ID, '_give_custom_amount', true );
+		$option = get_post_meta($this->ID, '_give_custom_amount', true);
 		$ret    = 0;
 
-		if ( $option === 'yes' ) {
+		if ($option === 'yes') {
 			$ret = 1;
 		}
 
@@ -410,7 +410,7 @@ 
 		 * @param bool $ret Is donation form in custom price mode?
 		 * @param int|string The ID of the donation form.
 		 */
-		return (bool) apply_filters( 'give_custom_price_option_mode', $ret, $this->ID );
+		return (bool) apply_filters('give_custom_price_option_mode', $ret, $this->ID);
 
 	}
 
@@ -424,10 +424,10 @@ 
 	 */
 	public function has_variable_prices() {
 
-		$option = get_post_meta( $this->ID, '_give_price_option', true );
+		$option = get_post_meta($this->ID, '_give_price_option', true);
 		$ret    = 0;
 
-		if ( $option === 'multi' ) {
+		if ($option === 'multi') {
 			$ret = 1;
 		}
 
@@ -437,7 +437,7 @@ 
 		 * @param bool $ret Does donation form have variable prices?
 		 * @param int|string The ID of the donation form.
 		 */
-		return (bool) apply_filters( 'give_has_variable_prices', $ret, $this->ID );
+		return (bool) apply_filters('give_has_variable_prices', $ret, $this->ID);
 
 	}
 
@@ -449,17 +449,17 @@ 
 	 */
 	public function get_type() {
 
-		if ( ! isset( $this->type ) ) {
+		if ( ! isset($this->type)) {
 
-			$this->type = get_post_meta( $this->ID, '_give_price_option', true );
+			$this->type = get_post_meta($this->ID, '_give_price_option', true);
 
-			if ( empty( $this->type ) ) {
+			if (empty($this->type)) {
 				$this->type = 'set';
 			}
 
 		}
 
-		return apply_filters( 'give_get_form_type', $this->type, $this->ID );
+		return apply_filters('give_get_form_type', $this->type, $this->ID);
 
 	}
 
@@ -472,7 +472,7 @@ 
     public function is_set_type_donation_form() {
         $form_type = $this->get_type();
 
-        return ( 'set' === $form_type ? true : false );
+        return ('set' === $form_type ? true : false);
 
     }
 
@@ -485,7 +485,7 @@ 
     public function is_multi_type_donation_form() {
         $form_type = $this->get_type();
 
-        return ( 'multi' === $form_type ? true : false );
+        return ('multi' === $form_type ? true : false);
 
     }
 
@@ -497,15 +497,15 @@ 
 	 */
 	public function get_sales() {
 
-		if ( ! isset( $this->sales ) ) {
+		if ( ! isset($this->sales)) {
 
-			if ( '' == get_post_meta( $this->ID, '_give_form_sales', true ) ) {
-				add_post_meta( $this->ID, '_give_form_sales', 0 );
+			if ('' == get_post_meta($this->ID, '_give_form_sales', true)) {
+				add_post_meta($this->ID, '_give_form_sales', 0);
 			} // End if
 
-			$this->sales = get_post_meta( $this->ID, '_give_form_sales', true );
+			$this->sales = get_post_meta($this->ID, '_give_form_sales', true);
 
-			if ( $this->sales < 0 ) {
+			if ($this->sales < 0) {
 				// Never let sales be less than zero
 				$this->sales = 0;
 			}
@@ -525,13 +525,13 @@ 
 	 *
 	 * @return int|false  New number of total sales
 	 */
-	public function increase_sales( $quantity = 1 ) {
+	public function increase_sales($quantity = 1) {
 
-		$sales       = give_get_form_sales_stats( $this->ID );
-		$quantity    = absint( $quantity );
+		$sales       = give_get_form_sales_stats($this->ID);
+		$quantity    = absint($quantity);
 		$total_sales = $sales + $quantity;
 
-		if ( $this->update_meta( '_give_form_sales', $total_sales ) ) {
+		if ($this->update_meta('_give_form_sales', $total_sales)) {
 
 			$this->sales = $total_sales;
 
@@ -551,17 +551,17 @@ 
 	 *
 	 * @return int|false  New number of total sales
 	 */
-	public function decrease_sales( $quantity = 1 ) {
+	public function decrease_sales($quantity = 1) {
 
-		$sales = give_get_form_sales_stats( $this->ID );
+		$sales = give_get_form_sales_stats($this->ID);
 
 		// Only decrease if not already zero
-		if ( $sales > 0 ) {
+		if ($sales > 0) {
 
-			$quantity    = absint( $quantity );
+			$quantity    = absint($quantity);
 			$total_sales = $sales - $quantity;
 
-			if ( $this->update_meta( '_give_form_sales', $total_sales ) ) {
+			if ($this->update_meta('_give_form_sales', $total_sales)) {
 
 				$this->sales = $sales;
 
@@ -583,15 +583,15 @@ 
 	 */
 	public function get_earnings() {
 
-		if ( ! isset( $this->earnings ) ) {
+		if ( ! isset($this->earnings)) {
 
-			if ( '' == get_post_meta( $this->ID, '_give_form_earnings', true ) ) {
-				add_post_meta( $this->ID, '_give_form_earnings', 0 );
+			if ('' == get_post_meta($this->ID, '_give_form_earnings', true)) {
+				add_post_meta($this->ID, '_give_form_earnings', 0);
 			}
 
-			$this->earnings = get_post_meta( $this->ID, '_give_form_earnings', true );
+			$this->earnings = get_post_meta($this->ID, '_give_form_earnings', true);
 
-			if ( $this->earnings < 0 ) {
+			if ($this->earnings < 0) {
 				// Never let earnings be less than zero
 				$this->earnings = 0;
 			}
@@ -611,12 +611,12 @@ 
 	 * 
 	 * @return float|false
 	 */
-	public function increase_earnings( $amount = 0 ) {
+	public function increase_earnings($amount = 0) {
 
-		$earnings   = give_get_form_earnings_stats( $this->ID );
+		$earnings   = give_get_form_earnings_stats($this->ID);
 		$new_amount = $earnings + (float) $amount;
 
-		if ( $this->update_meta( '_give_form_earnings', $new_amount ) ) {
+		if ($this->update_meta('_give_form_earnings', $new_amount)) {
 
 			$this->earnings = $new_amount;
 
@@ -634,16 +634,16 @@ 
 	 * @since 1.0
 	 * @return float|false
 	 */
-	public function decrease_earnings( $amount ) {
+	public function decrease_earnings($amount) {
 
-		$earnings = give_get_form_earnings_stats( $this->ID );
+		$earnings = give_get_form_earnings_stats($this->ID);
 
-		if ( $earnings > 0 ) {
+		if ($earnings > 0) {
 			// Only decrease if greater than zero
 			$new_amount = $earnings - (float) $amount;
 
 
-			if ( $this->update_meta( '_give_form_earnings', $new_amount ) ) {
+			if ($this->update_meta('_give_form_earnings', $new_amount)) {
 
 				$this->earnings = $new_amount;
 
@@ -663,22 +663,22 @@ 
 	 * @since 1.0
 	 * @return bool
 	 */
-	public function is_free( $price_id = false ) {
+	public function is_free($price_id = false) {
 
 		$is_free          = false;
-		$variable_pricing = give_has_variable_prices( $this->ID );
+		$variable_pricing = give_has_variable_prices($this->ID);
 
-		if ( $variable_pricing && ! is_null( $price_id ) && $price_id !== false ) {
-			$price = give_get_price_option_amount( $this->ID, $price_id );
-		} elseif ( ! $variable_pricing ) {
-			$price = get_post_meta( $this->ID, '_give_set_price', true );
+		if ($variable_pricing && ! is_null($price_id) && $price_id !== false) {
+			$price = give_get_price_option_amount($this->ID, $price_id);
+		} elseif ( ! $variable_pricing) {
+			$price = get_post_meta($this->ID, '_give_set_price', true);
 		}
 
-		if ( isset( $price ) && (float) $price == 0 ) {
+		if (isset($price) && (float) $price == 0) {
 			$is_free = true;
 		}
 
-		return (bool) apply_filters( 'give_is_free_donation', $is_free, $this->ID, $price_id );
+		return (bool) apply_filters('give_is_free_donation', $is_free, $this->ID, $price_id);
 
 	}
 
@@ -695,9 +695,9 @@ 
 	 */
 	public function is_close_donation_form() {
 		return (
-				'yes' === get_post_meta( $this->ID, '_give_goal_option', true ) )
-				&& ( 'yes' === get_post_meta( $this->ID, '_give_close_form_when_goal_achieved', true ) )
-				&& ( $this->get_goal() <= $this->get_earnings()
+				'yes' === get_post_meta($this->ID, '_give_goal_option', true) )
+				&& ('yes' === get_post_meta($this->ID, '_give_close_form_when_goal_achieved', true))
+				&& ($this->get_goal() <= $this->get_earnings()
 		);
 	}
 
@@ -713,29 +713,29 @@ 
 	 *
 	 * @return bool                             The result of the update query
 	 */
-	private function update_meta( $meta_key = '', $meta_value = '' ) {
+	private function update_meta($meta_key = '', $meta_value = '') {
 
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
-		if ( empty( $meta_key ) || empty( $meta_value ) ) {
+		if (empty($meta_key) || empty($meta_value)) {
 			return false;
 		}
 
 		// Make sure if it needs to be serialized, we do
-		$meta_value = maybe_serialize( $meta_value );
+		$meta_value = maybe_serialize($meta_value);
 
-		if ( is_numeric( $meta_value ) ) {
-			$value_type = is_float( $meta_value ) ? '%f' : '%d';
+		if (is_numeric($meta_value)) {
+			$value_type = is_float($meta_value) ? '%f' : '%d';
 		} else {
 			$value_type = "'%s'";
 		}
 
-		$sql = $wpdb->prepare( "UPDATE $wpdb->postmeta SET meta_value = $value_type WHERE post_id = $this->ID AND meta_key = '%s'", $meta_value, $meta_key );
+		$sql = $wpdb->prepare("UPDATE $wpdb->postmeta SET meta_value = $value_type WHERE post_id = $this->ID AND meta_key = '%s'", $meta_value, $meta_key);
 
-		if ( $wpdb->query( $sql ) ) {
+		if ($wpdb->query($sql)) {
 
-			clean_post_cache( $this->ID );
+			clean_post_cache($this->ID);
 
 			return true;
 

includes/class-give-logging.php

Doc Comments +3 added lines, -3 removed lines

@@ -214,7 +214,7 @@ 
 	 * @param array $log_data Log entry data
 	 * @param array $log_meta Log entry meta
 	 *
-	 * @return bool True if successful, false otherwise
+	 * @return boolean|null True if successful, false otherwise
 	 */
 	public function update_log( $log_data = array(), $log_meta = array() ) {
 
@@ -392,13 +392,13 @@ 
  * @param string $title
  * @param string $message
  * @param int $parent
- * @param null $type
+ * @param string $type
  *
  * @global $give_logs GIVE Logs Object
  *
  * @uses Give_Logging::add()
  *
- * @return mixed ID of the new log entry
+ * @return integer ID of the new log entry
  */
 function give_record_log( $title = '', $message = '', $parent = 0, $type = null ) {
 	/* @var Give_Logging $give_logs */

Spacing +56 added lines, -56 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -30,10 +30,10 @@ 
 	 */
 	public function __construct() {
 		// Create the log post type
-		add_action( 'init', array( $this, 'register_post_type' ), 1 );
+		add_action('init', array($this, 'register_post_type'), 1);
 
 		// Create types taxonomy and default types
-		add_action( 'init', array( $this, 'register_taxonomy' ), 1 );
+		add_action('init', array($this, 'register_taxonomy'), 1);
 
 	}
 
@@ -47,7 +47,7 @@ 
 	public function register_post_type() {
 		/* Logs post type */
 		$log_args = array(
-			'labels'              => array( 'name' => esc_html__( 'Logs', 'give' ) ),
+			'labels'              => array('name' => esc_html__('Logs', 'give')),
 			'public'              => false,
 			'exclude_from_search' => true,
 			'publicly_queryable'  => false,
@@ -55,11 +55,11 @@ 
 			'query_var'           => false,
 			'rewrite'             => false,
 			'capability_type'     => 'post',
-			'supports'            => array( 'title', 'editor' ),
+			'supports'            => array('title', 'editor'),
 			'can_export'          => true
 		);
 
-		register_post_type( 'give_log', $log_args );
+		register_post_type('give_log', $log_args);
 	}
 
 	/**
@@ -72,7 +72,7 @@ 
 	 * @return void
 	 */
 	public function register_taxonomy() {
-		register_taxonomy( 'give_log_type', 'give_log', array( 'public' => false ) );
+		register_taxonomy('give_log_type', 'give_log', array('public' => false));
 	}
 
 	/**
@@ -91,7 +91,7 @@ 
 			'api_request'
 		);
 
-		return apply_filters( 'give_log_types', $terms );
+		return apply_filters('give_log_types', $terms);
 	}
 
 	/**
@@ -107,8 +107,8 @@ 
 	 *
 	 * @return bool Whether log type is valid
 	 */
-	function valid_type( $type ) {
-		return in_array( $type, $this->log_types() );
+	function valid_type($type) {
+		return in_array($type, $this->log_types());
 	}
 
 	/**
@@ -128,7 +128,7 @@ 
 	 *
 	 * @return int Log ID
 	 */
-	public function add( $title = '', $message = '', $parent = 0, $type = null ) {
+	public function add($title = '', $message = '', $parent = 0, $type = null) {
 		$log_data = array(
 			'post_title'   => $title,
 			'post_content' => $message,
@@ -136,7 +136,7 @@ 
 			'log_type'     => $type
 		);
 
-		return $this->insert_log( $log_data );
+		return $this->insert_log($log_data);
 	}
 
 	/**
@@ -152,12 +152,12 @@ 
 	 *
 	 * @return array Array of the connected logs
 	 */
-	public function get_logs( $object_id = 0, $type = null, $paged = null ) {
-		return $this->get_connected_logs( array(
+	public function get_logs($object_id = 0, $type = null, $paged = null) {
+		return $this->get_connected_logs(array(
 			'post_parent' => $object_id,
 			'paged'       => $paged,
 			'log_type'    => $type
-		) );
+		));
 	}
 
 	/**
@@ -172,7 +172,7 @@ 
 	 *
 	 * @return int The ID of the newly created log item
 	 */
-	function insert_log( $log_data = array(), $log_meta = array() ) {
+	function insert_log($log_data = array(), $log_meta = array()) {
 		$defaults = array(
 			'post_type'    => 'give_log',
 			'post_status'  => 'publish',
@@ -181,26 +181,26 @@ 
 			'log_type'     => false
 		);
 
-		$args = wp_parse_args( $log_data, $defaults );
+		$args = wp_parse_args($log_data, $defaults);
 
-		do_action( 'give_pre_insert_log', $log_data, $log_meta );
+		do_action('give_pre_insert_log', $log_data, $log_meta);
 
 		// Store the log entry
-		$log_id = wp_insert_post( $args );
+		$log_id = wp_insert_post($args);
 
 		// Set the log type, if any
-		if ( $log_data['log_type'] && $this->valid_type( $log_data['log_type'] ) ) {
-			wp_set_object_terms( $log_id, $log_data['log_type'], 'give_log_type', false );
+		if ($log_data['log_type'] && $this->valid_type($log_data['log_type'])) {
+			wp_set_object_terms($log_id, $log_data['log_type'], 'give_log_type', false);
 		}
 
 		// Set log meta, if any
-		if ( $log_id && ! empty( $log_meta ) ) {
-			foreach ( (array) $log_meta as $key => $meta ) {
-				update_post_meta( $log_id, '_give_log_' . sanitize_key( $key ), $meta );
+		if ($log_id && ! empty($log_meta)) {
+			foreach ((array) $log_meta as $key => $meta) {
+				update_post_meta($log_id, '_give_log_'.sanitize_key($key), $meta);
 			}
 		}
 
-		do_action( 'give_post_insert_log', $log_id, $log_data, $log_meta );
+		do_action('give_post_insert_log', $log_id, $log_data, $log_meta);
 
 		return $log_id;
 	}
@@ -216,9 +216,9 @@ 
 	 *
 	 * @return bool True if successful, false otherwise
 	 */
-	public function update_log( $log_data = array(), $log_meta = array() ) {
+	public function update_log($log_data = array(), $log_meta = array()) {
 
-		do_action( 'give_pre_update_log', $log_data, $log_meta );
+		do_action('give_pre_update_log', $log_data, $log_meta);
 
 		$defaults = array(
 			'post_type'   => 'give_log',
@@ -226,20 +226,20 @@ 
 			'post_parent' => 0
 		);
 
-		$args = wp_parse_args( $log_data, $defaults );
+		$args = wp_parse_args($log_data, $defaults);
 
 		// Store the log entry
-		$log_id = wp_update_post( $args );
+		$log_id = wp_update_post($args);
 
-		if ( $log_id && ! empty( $log_meta ) ) {
-			foreach ( (array) $log_meta as $key => $meta ) {
-				if ( ! empty( $meta ) ) {
-					update_post_meta( $log_id, '_give_log_' . sanitize_key( $key ), $meta );
+		if ($log_id && ! empty($log_meta)) {
+			foreach ((array) $log_meta as $key => $meta) {
+				if ( ! empty($meta)) {
+					update_post_meta($log_id, '_give_log_'.sanitize_key($key), $meta);
 				}
 			}
 		}
 
-		do_action( 'give_post_update_log', $log_id, $log_data, $log_meta );
+		do_action('give_post_update_log', $log_id, $log_data, $log_meta);
 	}
 
 	/**
@@ -254,19 +254,19 @@ 
 	 *
 	 * @return mixed array if logs were found, false otherwise
 	 */
-	public function get_connected_logs( $args = array() ) {
+	public function get_connected_logs($args = array()) {
 
 		$defaults = array(
 			'post_type'      => 'give_log',
 			'posts_per_page' => 20,
 			'post_status'    => 'publish',
-			'paged'          => get_query_var( 'paged' ),
+			'paged'          => get_query_var('paged'),
 			'log_type'       => false
 		);
 
-		$query_args = wp_parse_args( $args, $defaults );
+		$query_args = wp_parse_args($args, $defaults);
 
-		if ( $query_args['log_type'] && $this->valid_type( $query_args['log_type'] ) ) {
+		if ($query_args['log_type'] && $this->valid_type($query_args['log_type'])) {
 			$query_args['tax_query'] = array(
 				array(
 					'taxonomy' => 'give_log_type',
@@ -276,9 +276,9 @@ 
 			);
 		}
 		
-		$logs = get_posts( $query_args );
+		$logs = get_posts($query_args);
 	
-		if ( $logs ) {
+		if ($logs) {
 			return $logs;
 		}
 
@@ -299,17 +299,17 @@ 
 	 *
 	 * @return int Log count
 	 */
-	public function get_log_count( $object_id = 0, $type = null, $meta_query = null, $date_query = null ) {
+	public function get_log_count($object_id = 0, $type = null, $meta_query = null, $date_query = null) {
 
 		$query_args = array(
 			'post_parent'    => $object_id,
 			'post_type'      => 'give_log',
-			'posts_per_page' => - 1,
+			'posts_per_page' => -1,
 			'post_status'    => 'publish',
 			'fields'         => 'ids',
 		);
 
-		if ( ! empty( $type ) && $this->valid_type( $type ) ) {
+		if ( ! empty($type) && $this->valid_type($type)) {
 			$query_args['tax_query'] = array(
 				array(
 					'taxonomy' => 'give_log_type',
@@ -319,15 +319,15 @@ 
 			);
 		}
 
-		if ( ! empty( $meta_query ) ) {
+		if ( ! empty($meta_query)) {
 			$query_args['meta_query'] = $meta_query;
 		}
 
-		if ( ! empty( $date_query ) ) {
+		if ( ! empty($date_query)) {
 			$query_args['date_query'] = $date_query;
 		}
 
-		$logs = new WP_Query( $query_args );
+		$logs = new WP_Query($query_args);
 
 		return (int) $logs->post_count;
 	}
@@ -345,16 +345,16 @@ 
 	 *
 	 * @return void
 	 */
-	public function delete_logs( $object_id = 0, $type = null, $meta_query = null ) {
+	public function delete_logs($object_id = 0, $type = null, $meta_query = null) {
 		$query_args = array(
 			'post_parent'    => $object_id,
 			'post_type'      => 'give_log',
-			'posts_per_page' => - 1,
+			'posts_per_page' => -1,
 			'post_status'    => 'publish',
 			'fields'         => 'ids'
 		);
 
-		if ( ! empty( $type ) && $this->valid_type( $type ) ) {
+		if ( ! empty($type) && $this->valid_type($type)) {
 			$query_args['tax_query'] = array(
 				array(
 					'taxonomy' => 'give_log_type',
@@ -364,15 +364,15 @@ 
 			);
 		}
 
-		if ( ! empty( $meta_query ) ) {
+		if ( ! empty($meta_query)) {
 			$query_args['meta_query'] = $meta_query;
 		}
 
-		$logs = get_posts( $query_args );
+		$logs = get_posts($query_args);
 
-		if ( $logs ) {
-			foreach ( $logs as $log ) {
-				wp_delete_post( $log, true );
+		if ($logs) {
+			foreach ($logs as $log) {
+				wp_delete_post($log, true);
 			}
 		}
 	}
@@ -400,9 +400,9 @@ 
  *
  * @return mixed ID of the new log entry
  */
-function give_record_log( $title = '', $message = '', $parent = 0, $type = null ) {
+function give_record_log($title = '', $message = '', $parent = 0, $type = null) {
 	global $give_logs;
-	$log = $give_logs->add( $title, $message, $parent, $type );
+	$log = $give_logs->add($title, $message, $parent, $type);
 
 	return $log;
 }
\ No newline at end of file

includes/price-functions.php

Spacing +18 added lines, -18 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,13 +23,13 @@ 
  *
  * @return bool true if has variable prices, false otherwise
  */
-function give_has_variable_prices( $form_id = 0 ) {
+function give_has_variable_prices($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
 	return $form->has_variable_prices();
 }
@@ -44,13 +44,13 @@ 
  *
  * @return array Variable prices
  */
-function give_get_variable_prices( $form_id = 0 ) {
+function give_get_variable_prices($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
 	return $form->prices;
 
@@ -67,13 +67,13 @@ 
  *
  * @return string $default_price
  */
-function give_get_default_multilevel_amount( $form_id ) {
+function give_get_default_multilevel_amount($form_id) {
 	$default_price = '1.00';
-	$prices        = apply_filters( 'give_form_variable_prices', give_get_variable_prices( $form_id ), $form_id );
+	$prices        = apply_filters('give_form_variable_prices', give_get_variable_prices($form_id), $form_id);
 
-	foreach ( $prices as $price ) {
+	foreach ($prices as $price) {
 
-		if ( isset( $price['_give_default'] ) && $price['_give_default'] === 'default' ) {
+		if (isset($price['_give_default']) && $price['_give_default'] === 'default') {
 			$default_price = $price['_give_amount'];
 		}
 
@@ -94,15 +94,15 @@ 
  * @return string $default_price
  * @since      1.0
  */
-function give_get_default_form_amount( $form_id ) {
+function give_get_default_form_amount($form_id) {
 
-	if ( give_has_variable_prices( $form_id ) ) {
+	if (give_has_variable_prices($form_id)) {
 
-		$default_amount = give_get_default_multilevel_amount( $form_id );
+		$default_amount = give_get_default_multilevel_amount($form_id);
 
 	} else {
 
-		$default_amount = get_post_meta( $form_id, '_give_set_price', true );
+		$default_amount = get_post_meta($form_id, '_give_set_price', true);
 
 	}
 
@@ -123,13 +123,13 @@ 
  * 
  * @return bool
  */
-function give_is_custom_price_mode( $form_id = 0 ) {
+function give_is_custom_price_mode($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
 	return $form->is_custom_price_mode();
 }
\ No newline at end of file

includes/admin/payments/actions.php

Indentation +58 added lines, -58 removed lines

@@ -37,7 +37,7 @@ 
 	// Retrieve the payment ID
 	$payment_id = absint( $data['give_payment_id'] );
 
-    /* @var Give_Payment $payment */
+	/* @var Give_Payment $payment */
 	$payment    = new Give_Payment( $payment_id );
 
 	// Retrieve existing payment meta
@@ -151,9 +151,9 @@ 
 		$previous_customer->remove_payment( $payment_id, false );
 		$customer->attach_payment( $payment_id, false );
 
-        // Reduce previous user donation count and amoount.
-        $previous_customer->decrease_purchase_count();
-        $previous_customer->decrease_value( $curr_total );
+		// Reduce previous user donation count and amoount.
+		$previous_customer->decrease_purchase_count();
+		$previous_customer->decrease_value( $curr_total );
 
 		// If purchase was completed and not ever refunded, adjust stats of new customers.
 		if ( 'revoked' == $status || 'publish' == $status ) {
@@ -165,7 +165,7 @@ 
 	} else{
 		// Update user donation stat.
 		$customer->update_donation_value( $curr_total, $new_total );
-    }
+	}
 
 	// Set new meta values
 	$payment->user_id    = $customer->user_id;
@@ -206,76 +206,76 @@ 
 
 	$payment->save();
 
-    // Get new give form ID.
-    $new_form_id = absint( $data['forms'] );
-    $current_form_id = absint( $payment->get_meta( '_give_payment_form_id' ) );
+	// Get new give form ID.
+	$new_form_id = absint( $data['forms'] );
+	$current_form_id = absint( $payment->get_meta( '_give_payment_form_id' ) );
 
-    // We are adding payment transfer code in last to remove any conflict with above functionality.
-    // For example: above code will automatically handle form stat (increase/decrease) when payment status changes.
-    /* Check if user want to transfer current payment to new give form id. */
-    if( $new_form_id != $current_form_id  ) {
+	// We are adding payment transfer code in last to remove any conflict with above functionality.
+	// For example: above code will automatically handle form stat (increase/decrease) when payment status changes.
+	/* Check if user want to transfer current payment to new give form id. */
+	if( $new_form_id != $current_form_id  ) {
 
-        // Get new give form title.
-        $new_form_title = get_the_title( $new_form_id );
+		// Get new give form title.
+		$new_form_title = get_the_title( $new_form_id );
 
-        // Update new give form data in payment data.
-        $payment_meta = $payment->get_meta();
-        $payment_meta['form_title'] = $new_form_title;
-        $payment_meta['form_id']    = $new_form_id;
+		// Update new give form data in payment data.
+		$payment_meta = $payment->get_meta();
+		$payment_meta['form_title'] = $new_form_title;
+		$payment_meta['form_id']    = $new_form_id;
 
-        // Update price id post meta data for set donation form.
-        if( ! give_has_variable_prices( $new_form_id ) ) {
-            $payment_meta['price_id'] = '';
-        }
+		// Update price id post meta data for set donation form.
+		if( ! give_has_variable_prices( $new_form_id ) ) {
+			$payment_meta['price_id'] = '';
+		}
 
-        // Update payment give form meta data.
-        $payment->update_meta( '_give_payment_form_id', $new_form_id );
-        $payment->update_meta( '_give_payment_form_title', $new_form_title );
-        $payment->update_meta( '_give_payment_meta', $payment_meta );
+		// Update payment give form meta data.
+		$payment->update_meta( '_give_payment_form_id', $new_form_id );
+		$payment->update_meta( '_give_payment_form_title', $new_form_title );
+		$payment->update_meta( '_give_payment_meta', $payment_meta );
 
-        // Update price id payment metadata.
-        if( ! give_has_variable_prices( $new_form_id ) ) {
-            $payment->update_meta( '_give_payment_price_id', '' );
-        }
+		// Update price id payment metadata.
+		if( ! give_has_variable_prices( $new_form_id ) ) {
+			$payment->update_meta( '_give_payment_price_id', '' );
+		}
 
 
-        // If purchase was completed and not ever refunded, adjust stats of forms
-        if ( 'revoked' == $status || 'publish' == $status ) {
+		// If purchase was completed and not ever refunded, adjust stats of forms
+		if ( 'revoked' == $status || 'publish' == $status ) {
 
-            // Decrease sale of old give form. For other payment status 
-            $current_form = new Give_Donate_Form( $current_form_id );
-            $current_form->decrease_sales();
-            $current_form->decrease_earnings( $curr_total );
+			// Decrease sale of old give form. For other payment status 
+			$current_form = new Give_Donate_Form( $current_form_id );
+			$current_form->decrease_sales();
+			$current_form->decrease_earnings( $curr_total );
             
-            // Increase sale of new give form.
-            $new_form = new Give_Donate_Form($new_form_id);
-            $new_form->increase_sales();
-            $new_form->increase_earnings($new_total);
-        }
+			// Increase sale of new give form.
+			$new_form = new Give_Donate_Form($new_form_id);
+			$new_form->increase_sales();
+			$new_form->increase_earnings($new_total);
+		}
 
-        // Re setup payment to update new meta value in object.
-        $payment->update_payment_setup( $payment->ID );
-    }
+		// Re setup payment to update new meta value in object.
+		$payment->update_payment_setup( $payment->ID );
+	}
 
-    // Update price id if current form is variable form.
-    if( ! empty( $data['give-variable-price'] ) && give_has_variable_prices( $payment->form_id ) ) {
+	// Update price id if current form is variable form.
+	if( ! empty( $data['give-variable-price'] ) && give_has_variable_prices( $payment->form_id ) ) {
 
-        // Get payment meta data.
-        $payment_meta = $payment->get_meta();
+		// Get payment meta data.
+		$payment_meta = $payment->get_meta();
 
-        // Set payment id to empty string if variable price id is negative ( i.e. custom amount feature enabled ).
-        $data['give-variable-price'] = ( 'custom' === $data['give-variable-price'] ) ? 'custom' : ( 0 < $data['give-variable-price'] ) ? $data['give-variable-price'] : '';
+		// Set payment id to empty string if variable price id is negative ( i.e. custom amount feature enabled ).
+		$data['give-variable-price'] = ( 'custom' === $data['give-variable-price'] ) ? 'custom' : ( 0 < $data['give-variable-price'] ) ? $data['give-variable-price'] : '';
 
-        // Update payment meta data.
-        $payment_meta['price_id'] = $data['give-variable-price'];
+		// Update payment meta data.
+		$payment_meta['price_id'] = $data['give-variable-price'];
 
-        // Update payment give form meta data.
-        $payment->update_meta( '_give_payment_price_id', $data['give-variable-price'] );
-        $payment->update_meta( '_give_payment_meta', $payment_meta );
+		// Update payment give form meta data.
+		$payment->update_meta( '_give_payment_price_id', $data['give-variable-price'] );
+		$payment->update_meta( '_give_payment_meta', $payment_meta );
 
-        // Re setup payment to update new meta value in object.
-        $payment->update_payment_setup( $payment->ID );
-    }
+		// Re setup payment to update new meta value in object.
+		$payment->update_payment_setup( $payment->ID );
+	}
 
 	do_action( 'give_updated_edited_purchase', $payment_id );
 

Spacing +117 added lines, -117 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -26,109 +26,109 @@ 
  * @return      void
  *
  */
-function give_update_payment_details( $data ) {
+function give_update_payment_details($data) {
 
-	if ( ! current_user_can( 'edit_give_payments', $data['give_payment_id'] ) ) {
-		wp_die( esc_html__( 'You do not have permission to edit payment records.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! current_user_can('edit_give_payments', $data['give_payment_id'])) {
+		wp_die(esc_html__('You do not have permission to edit payment records.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 	}
 
-	check_admin_referer( 'give_update_payment_details_nonce' );
+	check_admin_referer('give_update_payment_details_nonce');
 
 	// Retrieve the payment ID
-	$payment_id = absint( $data['give_payment_id'] );
+	$payment_id = absint($data['give_payment_id']);
 
     /* @var Give_Payment $payment */
-	$payment    = new Give_Payment( $payment_id );
+	$payment    = new Give_Payment($payment_id);
 
 	// Retrieve existing payment meta
 	$meta      = $payment->get_meta();
 	$user_info = $payment->user_info;
 
 	$status = $data['give-payment-status'];
-	$date   = sanitize_text_field( $data['give-payment-date'] );
-	$hour   = sanitize_text_field( $data['give-payment-time-hour'] );
+	$date   = sanitize_text_field($data['give-payment-date']);
+	$hour   = sanitize_text_field($data['give-payment-time-hour']);
 
 	// Restrict to our high and low
-	if ( $hour > 23 ) {
+	if ($hour > 23) {
 		$hour = 23;
-	} elseif ( $hour < 0 ) {
+	} elseif ($hour < 0) {
 		$hour = 00;
 	}
 
-	$minute = sanitize_text_field( $data['give-payment-time-min'] );
+	$minute = sanitize_text_field($data['give-payment-time-min']);
 
 	// Restrict to our high and low
-	if ( $minute > 59 ) {
+	if ($minute > 59) {
 		$minute = 59;
-	} elseif ( $minute < 0 ) {
+	} elseif ($minute < 0) {
 		$minute = 00;
 	}
 
-	$address = array_map( 'trim', $data['give-payment-address'][0] );
+	$address = array_map('trim', $data['give-payment-address'][0]);
 
-	$curr_total = give_sanitize_amount( $payment->total );
-	$new_total  = give_sanitize_amount( $data['give-payment-total'] );
-	$date       = date( 'Y-m-d', strtotime( $date ) ) . ' ' . $hour . ':' . $minute . ':00';
+	$curr_total = give_sanitize_amount($payment->total);
+	$new_total  = give_sanitize_amount($data['give-payment-total']);
+	$date       = date('Y-m-d', strtotime($date)).' '.$hour.':'.$minute.':00';
 
-	$curr_customer_id = sanitize_text_field( $data['give-current-customer'] );
-	$new_customer_id  = sanitize_text_field( $data['customer-id'] );
+	$curr_customer_id = sanitize_text_field($data['give-current-customer']);
+	$new_customer_id  = sanitize_text_field($data['customer-id']);
 
-	do_action( 'give_update_edited_purchase', $payment_id );
+	do_action('give_update_edited_purchase', $payment_id);
 
 	$payment->date = $date;
 	$updated       = $payment->save();
 
-	if ( 0 === $updated ) {
-		wp_die( esc_html__( 'Error Updating Payment.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 400 ) );
+	if (0 === $updated) {
+		wp_die(esc_html__('Error Updating Payment.', 'give'), esc_html__('Error', 'give'), array('response' => 400));
 	}
 
 
 	$customer_changed = false;
 
-	if ( isset( $data['give-new-customer'] ) && $data['give-new-customer'] == '1' ) {
+	if (isset($data['give-new-customer']) && $data['give-new-customer'] == '1') {
 
-		$email = isset( $data['give-new-customer-email'] ) ? sanitize_text_field( $data['give-new-customer-email'] ) : '';
-		$names = isset( $data['give-new-customer-name'] ) ? sanitize_text_field( $data['give-new-customer-name'] ) : '';
+		$email = isset($data['give-new-customer-email']) ? sanitize_text_field($data['give-new-customer-email']) : '';
+		$names = isset($data['give-new-customer-name']) ? sanitize_text_field($data['give-new-customer-name']) : '';
 
-		if ( empty( $email ) || empty( $names ) ) {
-			wp_die( esc_html__( 'New Customers require a name and email address.', 'give' ) );
+		if (empty($email) || empty($names)) {
+			wp_die(esc_html__('New Customers require a name and email address.', 'give'));
 		}
 
-		$customer = new Give_Customer( $email );
-		if ( empty( $customer->id ) ) {
-			$customer_data = array( 'name' => $names, 'email' => $email );
-			$user_id       = email_exists( $email );
-			if ( false !== $user_id ) {
+		$customer = new Give_Customer($email);
+		if (empty($customer->id)) {
+			$customer_data = array('name' => $names, 'email' => $email);
+			$user_id       = email_exists($email);
+			if (false !== $user_id) {
 				$customer_data['user_id'] = $user_id;
 			}
 
-			if ( ! $customer->create( $customer_data ) ) {
+			if ( ! $customer->create($customer_data)) {
 				// Failed to crete the new donor, assume the previous donor
 				$customer_changed = false;
-				$customer         = new Give_Customer( $curr_customer_id );
-				give_set_error( 'give-payment-new-customer-fail', esc_html__( 'Error creating new donor.', 'give' ) );
+				$customer         = new Give_Customer($curr_customer_id);
+				give_set_error('give-payment-new-customer-fail', esc_html__('Error creating new donor.', 'give'));
 			}
 		}
 
 		$new_customer_id = $customer->id;
 
-		$previous_customer = new Give_Customer( $curr_customer_id );
+		$previous_customer = new Give_Customer($curr_customer_id);
 
 		$customer_changed = true;
 
-	} elseif ( $curr_customer_id !== $new_customer_id ) {
+	} elseif ($curr_customer_id !== $new_customer_id) {
 
-		$customer = new Give_Customer( $new_customer_id );
+		$customer = new Give_Customer($new_customer_id);
 		$email    = $customer->email;
 		$names    = $customer->name;
 
-		$previous_customer = new Give_Customer( $curr_customer_id );
+		$previous_customer = new Give_Customer($curr_customer_id);
 
 		$customer_changed = true;
 
 	} else {
 
-		$customer = new Give_Customer( $curr_customer_id );
+		$customer = new Give_Customer($curr_customer_id);
 		$email    = $customer->email;
 		$names    = $customer->name;
 
@@ -136,35 +136,35 @@ 
 
 
 	// Setup first and last name from input values
-	$names      = explode( ' ', $names );
-	$first_name = ! empty( $names[0] ) ? $names[0] : '';
+	$names      = explode(' ', $names);
+	$first_name = ! empty($names[0]) ? $names[0] : '';
 	$last_name  = '';
-	if ( ! empty( $names[1] ) ) {
-		unset( $names[0] );
-		$last_name = implode( ' ', $names );
+	if ( ! empty($names[1])) {
+		unset($names[0]);
+		$last_name = implode(' ', $names);
 	}
 
 
-	if ( $customer_changed ) {
+	if ($customer_changed) {
 
 		// Remove the stats and payment from the previous customer and attach it to the new customer
-		$previous_customer->remove_payment( $payment_id, false );
-		$customer->attach_payment( $payment_id, false );
+		$previous_customer->remove_payment($payment_id, false);
+		$customer->attach_payment($payment_id, false);
 
         // Reduce previous user donation count and amoount.
         $previous_customer->decrease_purchase_count();
-        $previous_customer->decrease_value( $curr_total );
+        $previous_customer->decrease_value($curr_total);
 
 		// If purchase was completed and not ever refunded, adjust stats of new customers.
-		if ( 'revoked' == $status || 'publish' == $status ) {
+		if ('revoked' == $status || 'publish' == $status) {
 			$customer->increase_purchase_count();
-			$customer->increase_value( $new_total );
+			$customer->increase_value($new_total);
 		}
 
 		$payment->customer_id = $customer->id;
-	} else{
+	} else {
 		// Update user donation stat.
-		$customer->update_donation_value( $curr_total, $new_total );
+		$customer->update_donation_value($curr_total, $new_total);
     }
 
 	// Set new meta values
@@ -177,10 +177,10 @@ 
 
 
 	// Check for payment notes
-	if ( ! empty( $data['give-payment-note'] ) ) {
+	if ( ! empty($data['give-payment-note'])) {
 
-		$note = wp_kses( $data['give-payment-note'], array() );
-		give_insert_payment_note( $payment_id, $note );
+		$note = wp_kses($data['give-payment-note'], array());
+		give_insert_payment_note($payment_id, $note);
 
 	}
 
@@ -188,17 +188,17 @@ 
 	$payment->status = $status;
 
 	// Adjust total store earnings if the payment total has been changed
-	if ( $new_total !== $curr_total && ( 'publish' == $status || 'revoked' == $status ) ) {
+	if ($new_total !== $curr_total && ('publish' == $status || 'revoked' == $status)) {
 
-		if ( $new_total > $curr_total ) {
+		if ($new_total > $curr_total) {
 			// Increase if our new total is higher
 			$difference = $new_total - $curr_total;
-			give_increase_total_earnings( $difference );
+			give_increase_total_earnings($difference);
 
-		} elseif ( $curr_total > $new_total ) {
+		} elseif ($curr_total > $new_total) {
 			// Decrease if our new total is lower
 			$difference = $curr_total - $new_total;
-			give_decrease_total_earnings( $difference );
+			give_decrease_total_earnings($difference);
 
 		}
 
@@ -207,16 +207,16 @@ 
 	$payment->save();
 
     // Get new give form ID.
-    $new_form_id = absint( $data['forms'] );
-    $current_form_id = absint( $payment->get_meta( '_give_payment_form_id' ) );
+    $new_form_id = absint($data['forms']);
+    $current_form_id = absint($payment->get_meta('_give_payment_form_id'));
 
     // We are adding payment transfer code in last to remove any conflict with above functionality.
     // For example: above code will automatically handle form stat (increase/decrease) when payment status changes.
     /* Check if user want to transfer current payment to new give form id. */
-    if( $new_form_id != $current_form_id  ) {
+    if ($new_form_id != $current_form_id) {
 
         // Get new give form title.
-        $new_form_title = get_the_title( $new_form_id );
+        $new_form_title = get_the_title($new_form_id);
 
         // Update new give form data in payment data.
         $payment_meta = $payment->get_meta();
@@ -224,28 +224,28 @@ 
         $payment_meta['form_id']    = $new_form_id;
 
         // Update price id post meta data for set donation form.
-        if( ! give_has_variable_prices( $new_form_id ) ) {
+        if ( ! give_has_variable_prices($new_form_id)) {
             $payment_meta['price_id'] = '';
         }
 
         // Update payment give form meta data.
-        $payment->update_meta( '_give_payment_form_id', $new_form_id );
-        $payment->update_meta( '_give_payment_form_title', $new_form_title );
-        $payment->update_meta( '_give_payment_meta', $payment_meta );
+        $payment->update_meta('_give_payment_form_id', $new_form_id);
+        $payment->update_meta('_give_payment_form_title', $new_form_title);
+        $payment->update_meta('_give_payment_meta', $payment_meta);
 
         // Update price id payment metadata.
-        if( ! give_has_variable_prices( $new_form_id ) ) {
-            $payment->update_meta( '_give_payment_price_id', '' );
+        if ( ! give_has_variable_prices($new_form_id)) {
+            $payment->update_meta('_give_payment_price_id', '');
         }
 
 
         // If purchase was completed and not ever refunded, adjust stats of forms
-        if ( 'revoked' == $status || 'publish' == $status ) {
+        if ('revoked' == $status || 'publish' == $status) {
 
             // Decrease sale of old give form. For other payment status 
-            $current_form = new Give_Donate_Form( $current_form_id );
+            $current_form = new Give_Donate_Form($current_form_id);
             $current_form->decrease_sales();
-            $current_form->decrease_earnings( $curr_total );
+            $current_form->decrease_earnings($curr_total);
             
             // Increase sale of new give form.
             $new_form = new Give_Donate_Form($new_form_id);
@@ -254,36 +254,36 @@ 
         }
 
         // Re setup payment to update new meta value in object.
-        $payment->update_payment_setup( $payment->ID );
+        $payment->update_payment_setup($payment->ID);
     }
 
     // Update price id if current form is variable form.
-    if( ! empty( $data['give-variable-price'] ) && give_has_variable_prices( $payment->form_id ) ) {
+    if ( ! empty($data['give-variable-price']) && give_has_variable_prices($payment->form_id)) {
 
         // Get payment meta data.
         $payment_meta = $payment->get_meta();
 
         // Set payment id to empty string if variable price id is negative ( i.e. custom amount feature enabled ).
-        $data['give-variable-price'] = ( 'custom' === $data['give-variable-price'] ) ? 'custom' : ( 0 < $data['give-variable-price'] ) ? $data['give-variable-price'] : '';
+        $data['give-variable-price'] = ('custom' === $data['give-variable-price']) ? 'custom' : (0 < $data['give-variable-price']) ? $data['give-variable-price'] : '';
 
         // Update payment meta data.
         $payment_meta['price_id'] = $data['give-variable-price'];
 
         // Update payment give form meta data.
-        $payment->update_meta( '_give_payment_price_id', $data['give-variable-price'] );
-        $payment->update_meta( '_give_payment_meta', $payment_meta );
+        $payment->update_meta('_give_payment_price_id', $data['give-variable-price']);
+        $payment->update_meta('_give_payment_meta', $payment_meta);
 
         // Re setup payment to update new meta value in object.
-        $payment->update_payment_setup( $payment->ID );
+        $payment->update_payment_setup($payment->ID);
     }
 
-	do_action( 'give_updated_edited_purchase', $payment_id );
+	do_action('give_updated_edited_purchase', $payment_id);
 
-	wp_safe_redirect( admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-order-details&give-message=payment-updated&id=' . $payment_id ) );
+	wp_safe_redirect(admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-order-details&give-message=payment-updated&id='.$payment_id));
 	exit;
 }
 
-add_action( 'give_update_payment_details', 'give_update_payment_details' );
+add_action('give_update_payment_details', 'give_update_payment_details');
 
 /**
  * Trigger a Purchase Deletion
@@ -294,48 +294,48 @@ 
  *
  * @return void
  */
-function give_trigger_purchase_delete( $data ) {
-	if ( wp_verify_nonce( $data['_wpnonce'], 'give_payment_nonce' ) ) {
+function give_trigger_purchase_delete($data) {
+	if (wp_verify_nonce($data['_wpnonce'], 'give_payment_nonce')) {
 
-		$payment_id = absint( $data['purchase_id'] );
+		$payment_id = absint($data['purchase_id']);
 
-		if ( ! current_user_can( 'edit_give_payments', $payment_id ) ) {
-			wp_die( esc_html__( 'You do not have permission to edit payment records.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! current_user_can('edit_give_payments', $payment_id)) {
+			wp_die(esc_html__('You do not have permission to edit payment records.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 		}
 
-		give_delete_purchase( $payment_id );
-		wp_redirect( admin_url( '/edit.php?post_type=give_forms&page=give-payment-history&give-message=payment_deleted' ) );
+		give_delete_purchase($payment_id);
+		wp_redirect(admin_url('/edit.php?post_type=give_forms&page=give-payment-history&give-message=payment_deleted'));
 		give_die();
 	}
 }
 
-add_action( 'give_delete_payment', 'give_trigger_purchase_delete' );
+add_action('give_delete_payment', 'give_trigger_purchase_delete');
 
 /**
  * AJAX Store Payment Note
  */
 function give_ajax_store_payment_note() {
 
-	$payment_id = absint( $_POST['payment_id'] );
-	$note       = wp_kses( $_POST['note'], array() );
+	$payment_id = absint($_POST['payment_id']);
+	$note       = wp_kses($_POST['note'], array());
 
-	if ( ! current_user_can( 'edit_give_payments', $payment_id ) ) {
-		wp_die( esc_html__( 'You do not have permission to edit payment records.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! current_user_can('edit_give_payments', $payment_id)) {
+		wp_die(esc_html__('You do not have permission to edit payment records.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 	}
 
-	if ( empty( $payment_id ) ) {
-		die( '-1' );
+	if (empty($payment_id)) {
+		die('-1');
 	}
 
-	if ( empty( $note ) ) {
-		die( '-1' );
+	if (empty($note)) {
+		die('-1');
 	}
 
-	$note_id = give_insert_payment_note( $payment_id, $note );
-	die( give_get_payment_note_html( $note_id ) );
+	$note_id = give_insert_payment_note($payment_id, $note);
+	die(give_get_payment_note_html($note_id));
 }
 
-add_action( 'wp_ajax_give_insert_payment_note', 'give_ajax_store_payment_note' );
+add_action('wp_ajax_give_insert_payment_note', 'give_ajax_store_payment_note');
 
 /**
  * Triggers a payment note deletion without ajax
@@ -346,24 +346,24 @@ 
  *
  * @return void
  */
-function give_trigger_payment_note_deletion( $data ) {
+function give_trigger_payment_note_deletion($data) {
 
-	if ( ! wp_verify_nonce( $data['_wpnonce'], 'give_delete_payment_note_' . $data['note_id'] ) ) {
+	if ( ! wp_verify_nonce($data['_wpnonce'], 'give_delete_payment_note_'.$data['note_id'])) {
 		return;
 	}
 
-	if ( ! current_user_can( 'edit_give_payments', $data['payment_id'] ) ) {
-		wp_die( esc_html__( 'You do not have permission to edit payment records.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! current_user_can('edit_give_payments', $data['payment_id'])) {
+		wp_die(esc_html__('You do not have permission to edit payment records.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 	}
 
-	$edit_order_url = admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-order-details&give-message=payment-note-deleted&id=' . absint( $data['payment_id'] ) );
+	$edit_order_url = admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-order-details&give-message=payment-note-deleted&id='.absint($data['payment_id']));
 
-	give_delete_payment_note( $data['note_id'], $data['payment_id'] );
+	give_delete_payment_note($data['note_id'], $data['payment_id']);
 
-	wp_redirect( $edit_order_url );
+	wp_redirect($edit_order_url);
 }
 
-add_action( 'give_delete_payment_note', 'give_trigger_payment_note_deletion' );
+add_action('give_delete_payment_note', 'give_trigger_payment_note_deletion');
 
 /**
  * Delete a payment note deletion with ajax
@@ -374,16 +374,16 @@ 
  */
 function give_ajax_delete_payment_note() {
 
-	if ( ! current_user_can( 'edit_give_payments', $_POST['payment_id'] ) ) {
-		wp_die( esc_html__( 'You do not have permission to edit payment records.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! current_user_can('edit_give_payments', $_POST['payment_id'])) {
+		wp_die(esc_html__('You do not have permission to edit payment records.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 	}
 
-	if ( give_delete_payment_note( $_POST['note_id'], $_POST['payment_id'] ) ) {
-		die( '1' );
+	if (give_delete_payment_note($_POST['note_id'], $_POST['payment_id'])) {
+		die('1');
 	} else {
-		die( '-1' );
+		die('-1');
 	}
 
 }
 
-add_action( 'wp_ajax_give_delete_payment_note', 'give_ajax_delete_payment_note' );
\ No newline at end of file
+add_action('wp_ajax_give_delete_payment_note', 'give_ajax_delete_payment_note');
\ No newline at end of file