WordImpress / Give

includes/class-give-db-logs.php

Spacing +71 added lines, -73 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -35,7 +35,7 @@ 
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
-		$this->table_name  = $wpdb->prefix . 'give_logs';
+		$this->table_name  = $wpdb->prefix.'give_logs';
 		$this->primary_key = 'ID';
 		$this->version     = '1.0';
 
@@ -73,8 +73,8 @@ 
 	 * @return array  Default column values.
 	 */
 	public function get_column_defaults() {
-		$log_create_date     = current_time( 'mysql', 0 );
-		$log_create_date_gmt = get_gmt_from_date( $log_create_date );
+		$log_create_date     = current_time('mysql', 0);
+		$log_create_date_gmt = get_gmt_from_date($log_create_date);
 
 		return array(
 			'ID'           => 0,
@@ -97,39 +97,39 @@ 
 	 *
 	 * @return bool|int
 	 */
-	public function add( $data = array() ) {
+	public function add($data = array()) {
 		// Valid table columns.
-		$table_columns = array_keys( $this->get_columns() );
+		$table_columns = array_keys($this->get_columns());
 
 		// Filter data.
-		foreach ( $data as $table_column => $column_data ) {
-			if ( ! in_array( $table_column, $table_columns ) ) {
-				unset( $data[ $table_column ] );
+		foreach ($data as $table_column => $column_data) {
+			if ( ! in_array($table_column, $table_columns)) {
+				unset($data[$table_column]);
 			}
 		}
 
 		// Set default values.
-		$current_log_data = wp_parse_args( $data, $this->get_column_defaults() );
+		$current_log_data = wp_parse_args($data, $this->get_column_defaults());
 
 		// Log parent should be an int.
-		$current_log_data['log_parent'] = absint( $current_log_data['log_parent'] );
+		$current_log_data['log_parent'] = absint($current_log_data['log_parent']);
 
 		// Get log.
-		$existing_log = $this->get_log_by( $current_log_data['ID'] );
+		$existing_log = $this->get_log_by($current_log_data['ID']);
 
 		// Update an existing log.
-		if ( $existing_log ) {
+		if ($existing_log) {
 
 			// Create new log data from existing and new log data.
-			$current_log_data = array_merge( $current_log_data, $existing_log );
+			$current_log_data = array_merge($current_log_data, $existing_log);
 
 			// Update log data.
-			$this->update( $current_log_data['ID'], $current_log_data );
+			$this->update($current_log_data['ID'], $current_log_data);
 
 			$log_id = $current_log_data['ID'];
 
 		} else {
-			$log_id = $this->insert( $current_log_data, 'log' );
+			$log_id = $this->insert($current_log_data, 'log');
 		}
 
 		return $log_id;
@@ -147,20 +147,20 @@ 
 	 *
 	 * @return bool|null|array
 	 */
-	public function get_log_by( $log_id = 0, $by = 'id' ) {
+	public function get_log_by($log_id = 0, $by = 'id') {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 		$log = null;
 
 		// Make sure $log_id is int.
-		$log_id = absint( $log_id );
+		$log_id = absint($log_id);
 
 		// Bailout.
-		if ( empty( $log_id ) ) {
+		if (empty($log_id)) {
 			return null;
 		}
 
-		switch ( $by ) {
+		switch ($by) {
 			case 'id':
 				$log = $wpdb->get_row(
 					$wpdb->prepare(
@@ -172,7 +172,7 @@ 
 				break;
 
 			default:
-				$log = apply_filters( "give_get_log_by_{$by}", $log, $log_id );
+				$log = apply_filters("give_get_log_by_{$by}", $log, $log_id);
 		}
 
 		return $log;
@@ -188,14 +188,14 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function get_logs( $args = array() ) {
+	public function get_logs($args = array()) {
 		global $wpdb;
-		$sql_query = $this->get_sql( $args );
+		$sql_query = $this->get_sql($args);
 		
 		// Get log.
-		if ( ! ( $logs = Give_Cache::get( 'give_logs', true, $sql_query ) ) ) {
-			$logs = $wpdb->get_results( $sql_query );
-			Give_Cache::set( 'give_logs', $logs, 3600, true, $sql_query );
+		if ( ! ($logs = Give_Cache::get('give_logs', true, $sql_query))) {
+			$logs = $wpdb->get_results($sql_query);
+			Give_Cache::set('give_logs', $logs, 3600, true, $sql_query);
 		}
 
 		return $logs;
@@ -212,21 +212,21 @@ 
 	 *
 	 * @return int
 	 */
-	public function count( $args = array() ) {
+	public function count($args = array()) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 		$args['number'] = - 1;
 		$args['fields'] = 'ID';
 		$args['count']  = true;
 
-		$sql_query = $this->get_sql( $args );
+		$sql_query = $this->get_sql($args);
 
-		if ( ! ( $count = Give_Cache::get( 'give_logs_count', true, $sql_query ) ) ) {
-			$count = $wpdb->get_var( $sql_query );
-			Give_Cache::set( 'give_logs_count', $count, 3600, true, $args );
+		if ( ! ($count = Give_Cache::get('give_logs_count', true, $sql_query))) {
+			$count = $wpdb->get_var($sql_query);
+			Give_Cache::set('give_logs_count', $count, 3600, true, $args);
 		}
 		
-		return absint( $count );
+		return absint($count);
 	}
 
 	/**
@@ -252,10 +252,10 @@ 
         PRIMARY KEY  (ID)
         ) {$charset_collate};";
 
-		require_once( ABSPATH . 'wp-admin/includes/upgrade.php' );
-		dbDelta( $sql );
+		require_once(ABSPATH.'wp-admin/includes/upgrade.php');
+		dbDelta($sql);
 
-		update_option( $this->table_name . '_db_version', $this->version );
+		update_option($this->table_name.'_db_version', $this->version);
 	}
 
 
@@ -269,7 +269,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_sql( $args = array() ) {
+	public function get_sql($args = array()) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
@@ -283,12 +283,12 @@ 
 			'count'   => false,
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
 		// validate params.
-		$this->validate_params( $args );
+		$this->validate_params($args);
 
-		if ( $args['number'] < 1 ) {
+		if ($args['number'] < 1) {
 			$args['number'] = 999999999999;
 		}
 
@@ -296,78 +296,78 @@ 
 		$where = '';
 
 		// Get sql query for meta.
-		if ( ! empty( $args['meta_query'] ) ) {
-			$meta_query_object = new WP_Meta_Query( $args['meta_query'] );
-			$meta_query        = $meta_query_object->get_sql( 'log', $this->table_name, 'id' );
-			$where             = implode( '', $meta_query );
+		if ( ! empty($args['meta_query'])) {
+			$meta_query_object = new WP_Meta_Query($args['meta_query']);
+			$meta_query        = $meta_query_object->get_sql('log', $this->table_name, 'id');
+			$where             = implode('', $meta_query);
 		}
 
 		$where .= ' WHERE 1=1 ';
 
 		// Set offset.
-		if ( empty( $args['offset'] ) && ( 0 < $args['paged'] ) ) {
-			$args['offset'] = $args['number'] * ( $args['paged'] - 1 );
+		if (empty($args['offset']) && (0 < $args['paged'])) {
+			$args['offset'] = $args['number'] * ($args['paged'] - 1);
 		}
 
 		// Set fields.
 		$fields = "{$this->table_name}.*";
-		if ( is_string( $args['fields'] ) && ( 'all' !== $args['fields'] ) ) {
+		if (is_string($args['fields']) && ('all' !== $args['fields'])) {
 			$fields = "{$this->table_name}.{$args['fields']}";
 		}
 
 		// Set count.
-		if ( $args['count'] ) {
+		if ($args['count']) {
 			$fields = "COUNT({$fields})";
 		}
 
 		// Specific logs.
-		if ( ! empty( $args['ID'] ) ) {
+		if ( ! empty($args['ID'])) {
 
-			if ( ! is_array( $args['ID'] ) ) {
-				$args['ID'] = explode( ',', $args['ID'] );
+			if ( ! is_array($args['ID'])) {
+				$args['ID'] = explode(',', $args['ID']);
 			}
-			$log_ids = implode( ',', array_map( 'intval', $args['ID'] ) );
+			$log_ids = implode(',', array_map('intval', $args['ID']));
 
 			$where .= " AND {$this->table_name}.ID IN( {$log_ids} ) ";
 		}
 
 		// Logs created for a specific date or in a date range
-		if ( ! empty( $args['date_query'] ) ) {
-			$date_query_object = new WP_Date_Query( $args['date_query'], "{$this->table_name}.log_date" );
-			$where             .= $date_query_object->get_sql();
+		if ( ! empty($args['date_query'])) {
+			$date_query_object = new WP_Date_Query($args['date_query'], "{$this->table_name}.log_date");
+			$where .= $date_query_object->get_sql();
 		}
 
 		// Logs create for specific parent.
-		if ( ! empty( $args['log_parent'] ) ) {
-			if ( ! is_array( $args['log_parent'] ) ) {
-				$args['log_parent'] = explode( ',', $args['log_parent'] );
+		if ( ! empty($args['log_parent'])) {
+			if ( ! is_array($args['log_parent'])) {
+				$args['log_parent'] = explode(',', $args['log_parent']);
 			}
-			$parent_ids = implode( ',', array_map( 'intval', $args['log_parent'] ) );
+			$parent_ids = implode(',', array_map('intval', $args['log_parent']));
 
 			$where .= " AND {$this->table_name}.log_parent IN( {$parent_ids} ) ";
 		}
 
 		// Logs create for specific type.
 		// is_array check is for backward compatibility.
-		if ( ! empty( $args['log_type'] ) && ! is_array( $args['log_type'] ) ) {
-			if ( ! is_array( $args['log_type'] ) ) {
-				$args['log_type'] = explode( ',', $args['log_type'] );
+		if ( ! empty($args['log_type']) && ! is_array($args['log_type'])) {
+			if ( ! is_array($args['log_type'])) {
+				$args['log_type'] = explode(',', $args['log_type']);
 			}
 
-			$log_types = implode( '\',\'', array_map( 'trim', $args['log_type'] ) );
+			$log_types = implode('\',\'', array_map('trim', $args['log_type']));
 
 			$where .= " AND {$this->table_name}.log_type IN( '{$log_types}' ) ";
 		}
 
-		$args['orderby'] = ! array_key_exists( $args['orderby'], $this->get_columns() ) ? 'log_date' : $args['orderby'];
+		$args['orderby'] = ! array_key_exists($args['orderby'], $this->get_columns()) ? 'log_date' : $args['orderby'];
 
-		$args['orderby'] = esc_sql( $args['orderby'] );
-		$args['order']   = esc_sql( $args['order'] );
+		$args['orderby'] = esc_sql($args['orderby']);
+		$args['order']   = esc_sql($args['order']);
 
 		return $wpdb->prepare(
 			"SELECT {$fields} FROM {$this->table_name} {$where} ORDER BY {$this->table_name}.{$args['orderby']} {$args['order']} LIMIT %d,%d;",
-			absint( $args['offset'] ),
-			absint( $args['number'] )
+			absint($args['offset']),
+			absint($args['number'])
 		);
 	}
 
@@ -382,13 +382,11 @@ 
 	 *
 	 * @return mixed
 	 */
-	private function validate_params( &$args ) {
+	private function validate_params(&$args) {
 		// fields params
 		$args['fields'] = 'ids' === $args['fields'] ?
-			'ID' :
-			$args['fields'];
-		$args['fields'] = array_key_exists( $args['fields'], $this->get_columns() ) ?
-			$args['fields'] :
-			'all';
+			'ID' : $args['fields'];
+		$args['fields'] = array_key_exists($args['fields'], $this->get_columns()) ?
+			$args['fields'] : 'all';
 	}
 }

includes/error-tracking.php

Spacing +19 added lines, -19 removed lines

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

includes/payments/actions.php

Spacing +106 added lines, -110 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -28,21 +28,21 @@ 
  *
  * @return void
  */
-function give_complete_purchase( $payment_id, $new_status, $old_status ) {
+function give_complete_purchase($payment_id, $new_status, $old_status) {
 
 	// Make sure that payments are only completed once.
-	if ( $old_status == 'publish' || $old_status == 'complete' ) {
+	if ($old_status == 'publish' || $old_status == 'complete') {
 		return;
 	}
 
 	// Make sure the payment completion is only processed when new status is complete.
-	if ( $new_status != 'publish' && $new_status != 'complete' ) {
+	if ($new_status != 'publish' && $new_status != 'complete') {
 		return;
 	}
 
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	$creation_date  = get_post_field( 'post_date', $payment_id, 'raw' );
+	$creation_date  = get_post_field('post_date', $payment_id, 'raw');
 	$payment_meta   = $payment->payment_meta;
 	$completed_date = $payment->completed_date;
 	$user_info      = $payment->user_info;
@@ -58,12 +58,12 @@ 
 	 *
 	 * @param int $payment_id The ID of the payment.
 	 */
-	do_action( 'give_pre_complete_donation', $payment_id );
+	do_action('give_pre_complete_donation', $payment_id);
 
 	// Ensure these actions only run once, ever.
-	if ( empty( $completed_date ) ) {
+	if (empty($completed_date)) {
 
-		give_record_donation_in_log( $form_id, $payment_id, $price_id, $creation_date );
+		give_record_donation_in_log($form_id, $payment_id, $price_id, $creation_date);
 
 		/**
 		 * Fires after logging donation record.
@@ -74,29 +74,29 @@ 
 		 * @param int   $payment_id   The ID number of the payment.
 		 * @param array $payment_meta The payment meta.
 		 */
-		do_action( 'give_complete_form_donation', $form_id, $payment_id, $payment_meta );
+		do_action('give_complete_form_donation', $form_id, $payment_id, $payment_meta);
 
 	}
 
 	// Increase the earnings for this form ID.
-	give_increase_earnings( $form_id, $amount );
-	give_increase_donation_count( $form_id );
+	give_increase_earnings($form_id, $amount);
+	give_increase_donation_count($form_id);
 
 	// @todo: Refresh only range related stat cache
 	give_delete_donation_stats();
 
 	// Increase the donor's donation stats.
-	$donor = new Give_Donor( $donor_id );
+	$donor = new Give_Donor($donor_id);
 	$donor->increase_purchase_count();
-	$donor->increase_value( $amount );
+	$donor->increase_value($amount);
 
-	give_increase_total_earnings( $amount );
+	give_increase_total_earnings($amount);
 
 	// Ensure this action only runs once ever.
-	if ( empty( $completed_date ) ) {
+	if (empty($completed_date)) {
 
 		// Save the completed date.
-		$payment->completed_date = current_time( 'mysql' );
+		$payment->completed_date = current_time('mysql');
 		$payment->save();
 
 		/**
@@ -106,12 +106,12 @@ 
 		 *
 		 * @param int $payment_id The ID of the payment.
 		 */
-		do_action( 'give_complete_donation', $payment_id );
+		do_action('give_complete_donation', $payment_id);
 	}
 
 }
 
-add_action( 'give_update_payment_status', 'give_complete_purchase', 100, 3 );
+add_action('give_update_payment_status', 'give_complete_purchase', 100, 3);
 
 
 /**
@@ -125,20 +125,20 @@ 
  *
  * @return void
  */
-function give_record_status_change( $payment_id, $new_status, $old_status ) {
+function give_record_status_change($payment_id, $new_status, $old_status) {
 
 	// Get the list of statuses so that status in the payment note can be translated.
 	$stati      = give_get_payment_statuses();
-	$old_status = isset( $stati[ $old_status ] ) ? $stati[ $old_status ] : $old_status;
-	$new_status = isset( $stati[ $new_status ] ) ? $stati[ $new_status ] : $new_status;
+	$old_status = isset($stati[$old_status]) ? $stati[$old_status] : $old_status;
+	$new_status = isset($stati[$new_status]) ? $stati[$new_status] : $new_status;
 
 	// translators: 1: old status 2: new status.
-	$status_change = sprintf( esc_html__( 'Status changed from %1$s to %2$s.', 'give' ), $old_status, $new_status );
+	$status_change = sprintf(esc_html__('Status changed from %1$s to %2$s.', 'give'), $old_status, $new_status);
 
-	give_insert_payment_note( $payment_id, $status_change );
+	give_insert_payment_note($payment_id, $status_change);
 }
 
-add_action( 'give_update_payment_status', 'give_record_status_change', 100, 3 );
+add_action('give_update_payment_status', 'give_record_status_change', 100, 3);
 
 
 /**
@@ -154,25 +154,25 @@ 
  *
  * @return void
  */
-function give_update_old_payments_with_totals( $data ) {
-	if ( ! wp_verify_nonce( $data['_wpnonce'], 'give_upgrade_payments_nonce' ) ) {
+function give_update_old_payments_with_totals($data) {
+	if ( ! wp_verify_nonce($data['_wpnonce'], 'give_upgrade_payments_nonce')) {
 		return;
 	}
 
-	if ( get_option( 'give_payment_totals_upgraded' ) ) {
+	if (get_option('give_payment_totals_upgraded')) {
 		return;
 	}
 
-	$payments = give_get_payments( array(
+	$payments = give_get_payments(array(
 		'offset' => 0,
-		'number' => - 1,
+		'number' => -1,
 		'mode'   => 'all',
-	) );
+	));
 
-	if ( $payments ) {
-		foreach ( $payments as $payment ) {
+	if ($payments) {
+		foreach ($payments as $payment) {
 
-			$payment = new Give_Payment( $payment->ID );
+			$payment = new Give_Payment($payment->ID);
 			$meta    = $payment->get_meta();
 
 			$payment->total = $meta['amount'];
@@ -181,10 +181,10 @@ 
 		}
 	}
 
-	add_option( 'give_payment_totals_upgraded', 1 );
+	add_option('give_payment_totals_upgraded', 1);
 }
 
-add_action( 'give_upgrade_payments', 'give_update_old_payments_with_totals' );
+add_action('give_upgrade_payments', 'give_update_old_payments_with_totals');
 
 /**
  * Mark Abandoned Donations
@@ -198,17 +198,17 @@ 
 function give_mark_abandoned_donations() {
 	$args = array(
 		'status' => 'pending',
-		'number' => - 1,
+		'number' => -1,
 		'output' => 'give_payments',
 	);
 
-	add_filter( 'posts_where', 'give_filter_where_older_than_week' );
+	add_filter('posts_where', 'give_filter_where_older_than_week');
 
-	$payments = give_get_payments( $args );
+	$payments = give_get_payments($args);
 
-	remove_filter( 'posts_where', 'give_filter_where_older_than_week' );
+	remove_filter('posts_where', 'give_filter_where_older_than_week');
 
-	if ( $payments ) {
+	if ($payments) {
 		/**
 		 * Filter payment gateways:  Used to set payment gateways which can be skip while transferring pending payment to abandon.
 		 *
@@ -216,13 +216,13 @@ 
 		 *
 		 * @param array $skip_payment_gateways Array of payment gateways
 		 */
-		$skip_payment_gateways = apply_filters( 'give_mark_abandoned_donation_gateways', array( 'offline' ) );
+		$skip_payment_gateways = apply_filters('give_mark_abandoned_donation_gateways', array('offline'));
 
-		foreach ( $payments as $payment ) {
-			$gateway = give_get_payment_gateway( $payment );
+		foreach ($payments as $payment) {
+			$gateway = give_get_payment_gateway($payment);
 
 			// Skip payment gateways.
-			if ( in_array( $gateway, $skip_payment_gateways ) ) {
+			if (in_array($gateway, $skip_payment_gateways)) {
 				continue;
 			}
 
@@ -232,7 +232,7 @@ 
 	}
 }
 
-Give_Cron::add_weekly_event( 'give_mark_abandoned_donations' );
+Give_Cron::add_weekly_event('give_mark_abandoned_donations');
 
 
 /**
@@ -244,15 +244,15 @@ 
  *
  * @return void
  */
-function give_refresh_thismonth_stat_transients( $payment_ID ) {
+function give_refresh_thismonth_stat_transients($payment_ID) {
 	// Monthly stats.
-	Give_Cache::delete( Give_Cache::get_key( 'give_estimated_monthly_stats' ) );
+	Give_Cache::delete(Give_Cache::get_key('give_estimated_monthly_stats'));
 
 	// @todo: Refresh only range related stat cache
 	give_delete_donation_stats();
 }
 
-add_action( 'save_post_give_payment', 'give_refresh_thismonth_stat_transients' );
+add_action('save_post_give_payment', 'give_refresh_thismonth_stat_transients');
 
 
 /**
@@ -268,12 +268,12 @@ 
  *
  * @return array
  */
-function give_bc_v20_get_payment_meta( $check, $object_id, $meta_key, $single ) {
+function give_bc_v20_get_payment_meta($check, $object_id, $meta_key, $single) {
 	// Bailout.
 	if (
-		'give_payment' !== get_post_type( $object_id ) ||
+		'give_payment' !== get_post_type($object_id) ||
 		'_give_payment_meta' !== $meta_key ||
-		! give_has_upgrade_completed( 'v20_upgrades_payment_metadata' )
+		! give_has_upgrade_completed('v20_upgrades_payment_metadata')
 	) {
 		return $check;
 	}
@@ -281,73 +281,69 @@ 
 	$cache_key = "_give_payment_meta_{$object_id}";
 
 	// Get already calculate payment meta from cache.
-	$payment_meta = Give_Cache::get_db_query( $cache_key );
+	$payment_meta = Give_Cache::get_db_query($cache_key);
 
-	if ( is_null( $payment_meta ) ) {
+	if (is_null($payment_meta)) {
 		// Remove filter.
-		remove_filter( 'get_post_metadata', 'give_bc_v20_get_payment_meta', 999 );
+		remove_filter('get_post_metadata', 'give_bc_v20_get_payment_meta', 999);
 
-		$donation = new Give_Payment( $object_id );
+		$donation = new Give_Payment($object_id);
 
 		// Get all payment meta.
-		$payment_meta = give_get_meta( $object_id );
+		$payment_meta = give_get_meta($object_id);
 
 		// Set default value to array.
-		if ( empty( $payment_meta ) ) {
+		if (empty($payment_meta)) {
 			return $check;
 		}
 
 		// Convert all meta key value to string instead of array
-		array_walk( $payment_meta, function ( &$meta, $key ) {
-			$meta = current( $meta );
+		array_walk($payment_meta, function(&$meta, $key) {
+			$meta = current($meta);
 		} );
 
 		/**
 		 * Add backward compatibility to old meta keys.
 		 */
 		// Donation key.
-		$payment_meta['key'] = ! empty( $payment_meta['_give_payment_purchase_key'] ) ? $payment_meta['_give_payment_purchase_key'] : '';
+		$payment_meta['key'] = ! empty($payment_meta['_give_payment_purchase_key']) ? $payment_meta['_give_payment_purchase_key'] : '';
 
 		// Donation form.
-		$payment_meta['form_title'] = ! empty( $payment_meta['_give_payment_form_title'] ) ? $payment_meta['_give_payment_form_title'] : '';
+		$payment_meta['form_title'] = ! empty($payment_meta['_give_payment_form_title']) ? $payment_meta['_give_payment_form_title'] : '';
 
 		// Donor email.
-		$payment_meta['email'] = ! empty( $payment_meta['_give_payment_donor_email'] ) ? $payment_meta['_give_payment_donor_email'] : '';
-		$payment_meta['email'] = ! empty( $payment_meta['email'] ) ?
-			$payment_meta['email'] :
-			Give()->donors->get_column( 'email', $donation->donor_id );
+		$payment_meta['email'] = ! empty($payment_meta['_give_payment_donor_email']) ? $payment_meta['_give_payment_donor_email'] : '';
+		$payment_meta['email'] = ! empty($payment_meta['email']) ?
+			$payment_meta['email'] : Give()->donors->get_column('email', $donation->donor_id);
 
 		// Form id.
-		$payment_meta['form_id'] = ! empty( $payment_meta['_give_payment_form_id'] ) ? $payment_meta['_give_payment_form_id'] : '';
+		$payment_meta['form_id'] = ! empty($payment_meta['_give_payment_form_id']) ? $payment_meta['_give_payment_form_id'] : '';
 
 		// Price id.
-		$payment_meta['price_id'] = ! empty( $payment_meta['_give_payment_price_id'] ) ? $payment_meta['_give_payment_price_id'] : '';
+		$payment_meta['price_id'] = ! empty($payment_meta['_give_payment_price_id']) ? $payment_meta['_give_payment_price_id'] : '';
 
 		// Date.
-		$payment_meta['date'] = ! empty( $payment_meta['_give_payment_date'] ) ? $payment_meta['_give_payment_date'] : '';
-		$payment_meta['date'] = ! empty( $payment_meta['date'] ) ?
-			$payment_meta['date'] :
-			get_post_field( 'post_date', $object_id );
+		$payment_meta['date'] = ! empty($payment_meta['_give_payment_date']) ? $payment_meta['_give_payment_date'] : '';
+		$payment_meta['date'] = ! empty($payment_meta['date']) ?
+			$payment_meta['date'] : get_post_field('post_date', $object_id);
 
 
 		// Currency.
-		$payment_meta['currency'] = ! empty( $payment_meta['_give_payment_currency'] ) ? $payment_meta['_give_payment_currency'] : '';
+		$payment_meta['currency'] = ! empty($payment_meta['_give_payment_currency']) ? $payment_meta['_give_payment_currency'] : '';
 
 		// Decode donor data.
-		$donor_id = ! empty( $payment_meta['_give_payment_donor_id'] ) ? $payment_meta['_give_payment_donor_id'] : 0;
-		$donor    = new Give_Donor( $donor_id );
+		$donor_id = ! empty($payment_meta['_give_payment_donor_id']) ? $payment_meta['_give_payment_donor_id'] : 0;
+		$donor    = new Give_Donor($donor_id);
 
 		// Donor first name.
-		$donor_data['first_name'] = ! empty( $payment_meta['_give_donor_billing_first_name'] ) ? $payment_meta['_give_donor_billing_first_name'] : '';
-		$donor_data['first_name'] = ! empty( $donor_data['first_name'] ) ?
-			$donor_data['first_name'] :
-			$donor->get_first_name();
+		$donor_data['first_name'] = ! empty($payment_meta['_give_donor_billing_first_name']) ? $payment_meta['_give_donor_billing_first_name'] : '';
+		$donor_data['first_name'] = ! empty($donor_data['first_name']) ?
+			$donor_data['first_name'] : $donor->get_first_name();
 
 		// Donor last name.
-		$donor_data['last_name'] = ! empty( $payment_meta['_give_donor_billing_last_name'] ) ? $payment_meta['_give_donor_billing_last_name'] : '';
-		$donor_data['last_name'] = ! empty( $donor_data['last_name'] ) ?
-			$donor_data['last_name'] :
-			$donor->get_last_name();
+		$donor_data['last_name'] = ! empty($payment_meta['_give_donor_billing_last_name']) ? $payment_meta['_give_donor_billing_last_name'] : '';
+		$donor_data['last_name'] = ! empty($donor_data['last_name']) ?
+			$donor_data['last_name'] : $donor->get_last_name();
 
 		// Donor email.
 		$donor_data['email'] = $payment_meta['email'];
@@ -358,63 +354,63 @@ 
 		$donor_data['address'] = false;
 
 		// Address1.
-		$address1 = ! empty( $payment_meta['_give_payment_billing_address1'] ) ? $payment_meta['_give_payment_billing_address1'] : '';
-		if ( $address1 ) {
+		$address1 = ! empty($payment_meta['_give_payment_billing_address1']) ? $payment_meta['_give_payment_billing_address1'] : '';
+		if ($address1) {
 			$donor_data['address']['line1'] = $address1;
 		}
 
 		// Address2.
-		$address2 = ! empty( $payment_meta['_give_payment_billing_address2'] ) ? $payment_meta['_give_payment_billing_address2'] : '';
-		if ( $address2 ) {
+		$address2 = ! empty($payment_meta['_give_payment_billing_address2']) ? $payment_meta['_give_payment_billing_address2'] : '';
+		if ($address2) {
 			$donor_data['address']['line2'] = $address2;
 		}
 
 		// City.
-		$city = ! empty( $payment_meta['_give_payment_billing_city'] ) ? $payment_meta['_give_payment_billing_city'] : '';
-		if ( $city ) {
+		$city = ! empty($payment_meta['_give_payment_billing_city']) ? $payment_meta['_give_payment_billing_city'] : '';
+		if ($city) {
 			$donor_data['address']['city'] = $city;
 		}
 
 		// Zip.
-		$zip = ! empty( $payment_meta['_give_payment_billing_zip'] ) ? $payment_meta['_give_payment_billing_zip'] : '';
-		if ( $zip ) {
+		$zip = ! empty($payment_meta['_give_payment_billing_zip']) ? $payment_meta['_give_payment_billing_zip'] : '';
+		if ($zip) {
 			$donor_data['address']['zip'] = $zip;
 		}
 
 		// State.
-		$state = ! empty( $payment_meta['_give_payment_billing_state'] ) ? $payment_meta['_give_payment_billing_state'] : '';
-		if ( $state ) {
+		$state = ! empty($payment_meta['_give_payment_billing_state']) ? $payment_meta['_give_payment_billing_state'] : '';
+		if ($state) {
 			$donor_data['address']['state'] = $state;
 		}
 
 		// Country.
-		$country = ! empty( $payment_meta['_give_payment_billing_country'] ) ? $payment_meta['_give_payment_billing_country'] : '';
-		if ( $country ) {
+		$country = ! empty($payment_meta['_give_payment_billing_country']) ? $payment_meta['_give_payment_billing_country'] : '';
+		if ($country) {
 			$donor_data['address']['country'] = $country;
 		}
 
 		$payment_meta['user_info'] = $donor_data;
 
 		// Add filter
-		add_filter( 'get_post_metadata', 'give_bc_v20_get_payment_meta', 999, 4 );
+		add_filter('get_post_metadata', 'give_bc_v20_get_payment_meta', 999, 4);
 
 		// Set custom meta key into payment meta.
-		if ( ! empty( $payment_meta['_give_payment_meta'] ) ) {
-			$payment_meta = array_merge( maybe_unserialize( $payment_meta['_give_payment_meta'] ), $payment_meta );
+		if ( ! empty($payment_meta['_give_payment_meta'])) {
+			$payment_meta = array_merge(maybe_unserialize($payment_meta['_give_payment_meta']), $payment_meta);
 		}
 
 		// Set cache.
-		Give_Cache::set_db_query( $cache_key, $payment_meta );
+		Give_Cache::set_db_query($cache_key, $payment_meta);
 	}
 
-	if ( $single ) {
+	if ($single) {
 		/**
 		 * Filter the payment meta
 		 * Add custom meta key to payment meta
 		 *
 		 * @since 2.0
 		 */
-		$new_payment_meta[0] = apply_filters( 'give_get_payment_meta', $payment_meta, $object_id, $meta_key );
+		$new_payment_meta[0] = apply_filters('give_get_payment_meta', $payment_meta, $object_id, $meta_key);
 
 		$payment_meta = $new_payment_meta;
 	}
@@ -422,7 +418,7 @@ 
 	return $payment_meta;
 }
 
-add_filter( 'get_post_metadata', 'give_bc_v20_get_payment_meta', 999, 4 );
+add_filter('get_post_metadata', 'give_bc_v20_get_payment_meta', 999, 4);
 
 /**
  * Add meta in payment that store page id and page url.
@@ -434,19 +430,19 @@ 
  *
  * @param int $payment_id Payment id for which the meta value should be updated.
  */
-function give_payment_save_page_data( $payment_id ) {
-	$page_url = ( ! empty( $_REQUEST['give-current-url'] ) ? esc_url( $_REQUEST['give-current-url'] ) : false );
+function give_payment_save_page_data($payment_id) {
+	$page_url = ( ! empty($_REQUEST['give-current-url']) ? esc_url($_REQUEST['give-current-url']) : false);
 
 	// Check $page_url is not empty.
-	if ( $page_url ) {
-		update_post_meta( $payment_id, '_give_current_url', $page_url );
-		$page_id = url_to_postid( $page_url );
+	if ($page_url) {
+		update_post_meta($payment_id, '_give_current_url', $page_url);
+		$page_id = url_to_postid($page_url);
 		// Check $page_id is not empty.
-		if ( $page_id ) {
-			update_post_meta( $payment_id, '_give_current_page_id', $page_id );
+		if ($page_id) {
+			update_post_meta($payment_id, '_give_current_page_id', $page_id);
 		}
 	}
 }
 
 // Fire when payment is save.
-add_action( 'give_insert_payment', 'give_payment_save_page_data' );
\ No newline at end of file
+add_action('give_insert_payment', 'give_payment_save_page_data');
\ No newline at end of file

includes/class-give-async-process.php

Spacing +4 added lines, -4 removed lines

@@ -11,7 +11,7 @@ 
  * @category Class
  * @author   WordImpress
  */
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -52,16 +52,16 @@ 
 		 * )
 		 */
 
-		$_post = give_clean( $_POST );
+		$_post = give_clean($_POST);
 
-		if ( empty( $_post ) || empty( $_post['data'] ) || empty( $_post['hook'] ) ) {
+		if (empty($_post) || empty($_post['data']) || empty($_post['hook'])) {
 			exit();
 		}
 
 		/**
 		 * Fire the hook.
 		 */
-		do_action( $_post['hook'], $_post['data'] );
+		do_action($_post['hook'], $_post['data']);
 
 		exit();
 	}

includes/shortcodes.php

Spacing +112 added lines, -112 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,41 +23,41 @@ 
  *
  * @return string|bool
  */
-function give_donation_history( $atts ) {
+function give_donation_history($atts) {
 
-	$donation_history_args = shortcode_atts( array(
+	$donation_history_args = shortcode_atts(array(
 		'id'             => true,
 		'date'           => true,
 		'donor'          => false,
 		'amount'         => true,
 		'status'         => false,
 		'payment_method' => false,
-	), $atts, 'donation_history' );
+	), $atts, 'donation_history');
 
 	// Always show receipt link.
 	$donation_history_args['details'] = true;
 
 	// Set Donation History Shortcode Arguments in session variable.
-	Give()->session->set( 'give_donation_history_args', $donation_history_args );
+	Give()->session->set('give_donation_history_args', $donation_history_args);
 
 	// If payment_key query arg exists, return receipt instead of donation history.
-	if ( isset( $_GET['payment_key'] ) ) {
+	if (isset($_GET['payment_key'])) {
 		ob_start();
 
-		echo give_receipt_shortcode( array() );
+		echo give_receipt_shortcode(array());
 
 		// Display donation history link only if Receipt Access Session is available.
-		if ( give_get_receipt_session() ) {
+		if (give_get_receipt_session()) {
 			echo sprintf(
 				'<a href="%s">%s</a>',
-				esc_url( give_get_history_page_uri() ),
-				__( '&laquo; Return to All Donations', 'give' )
+				esc_url(give_get_history_page_uri()),
+				__('&laquo; Return to All Donations', 'give')
 			);
 		}
 		return ob_get_clean();
 	}
 
-	$email_access = give_get_option( 'email_access' );
+	$email_access = give_get_option('email_access');
 
 	/**
 	 * Determine access
@@ -68,31 +68,31 @@ 
 	if (
 		is_user_logged_in() ||
 		false !== Give()->session->get_session_expiration() ||
-		( give_is_setting_enabled( $email_access ) && Give()->email_access->token_exists ) ||
+		(give_is_setting_enabled($email_access) && Give()->email_access->token_exists) ||
 		true === give_get_history_session()
 	) {
 		ob_start();
-		give_get_template_part( 'history', 'donations' );
+		give_get_template_part('history', 'donations');
 
 		return ob_get_clean();
 
-	} elseif ( give_is_setting_enabled( $email_access ) ) {
+	} elseif (give_is_setting_enabled($email_access)) {
 		// Is Email-based access enabled?
 		ob_start();
-		give_get_template_part( 'email', 'login-form' );
+		give_get_template_part('email', 'login-form');
 
 		return ob_get_clean();
 
 	} else {
 
-		$output = apply_filters( 'give_donation_history_nonuser_message', Give()->notices->print_frontend_notice( __( 'You must be logged in to view your donation history. Please login using your account or create an account using the same email you used to donate with.', 'give' ), false ) );
-		$output .= do_shortcode( '[give_login]' );
+		$output = apply_filters('give_donation_history_nonuser_message', Give()->notices->print_frontend_notice(__('You must be logged in to view your donation history. Please login using your account or create an account using the same email you used to donate with.', 'give'), false));
+		$output .= do_shortcode('[give_login]');
 
 		return $output;
 	}
 }
 
-add_shortcode( 'donation_history', 'give_donation_history' );
+add_shortcode('donation_history', 'give_donation_history');
 
 /**
  * Donation Form Shortcode
@@ -105,8 +105,8 @@ 
  *
  * @return string
  */
-function give_form_shortcode( $atts ) {
-	$atts = shortcode_atts( array(
+function give_form_shortcode($atts) {
+	$atts = shortcode_atts(array(
 		'id'                    => '',
 		'show_title'            => true,
 		'show_goal'             => true,
@@ -114,21 +114,21 @@ 
 		'float_labels'          => '',
 		'display_style'         => '',
 		'continue_button_title' => '',
-	), $atts, 'give_form' );
+	), $atts, 'give_form');
 
 	// Convert string to bool.
-	$atts['show_title'] = filter_var( $atts['show_title'], FILTER_VALIDATE_BOOLEAN );
-	$atts['show_goal']  = filter_var( $atts['show_goal'], FILTER_VALIDATE_BOOLEAN );
+	$atts['show_title'] = filter_var($atts['show_title'], FILTER_VALIDATE_BOOLEAN);
+	$atts['show_goal']  = filter_var($atts['show_goal'], FILTER_VALIDATE_BOOLEAN);
 
 	// get the Give Form
 	ob_start();
-	give_get_donation_form( $atts );
+	give_get_donation_form($atts);
 	$final_output = ob_get_clean();
 
-	return apply_filters( 'give_donate_form', $final_output, $atts );
+	return apply_filters('give_donate_form', $final_output, $atts);
 }
 
-add_shortcode( 'give_form', 'give_form_shortcode' );
+add_shortcode('give_form', 'give_form_shortcode');
 
 /**
  * Donation Form Goal Shortcode.
@@ -141,36 +141,36 @@ 
  *
  * @return string
  */
-function give_goal_shortcode( $atts ) {
-	$atts = shortcode_atts( array(
+function give_goal_shortcode($atts) {
+	$atts = shortcode_atts(array(
 		'id'        => '',
 		'show_text' => true,
 		'show_bar'  => true,
-	), $atts, 'give_goal' );
+	), $atts, 'give_goal');
 
 	// get the Give Form.
 	ob_start();
 
 	// Sanity check 1: ensure there is an ID Provided.
-	if ( empty( $atts['id'] ) ) {
-		Give()->notices->print_frontend_notice( __( 'The shortcode is missing Donation Form ID attribute.', 'give' ), true );
+	if (empty($atts['id'])) {
+		Give()->notices->print_frontend_notice(__('The shortcode is missing Donation Form ID attribute.', 'give'), true);
 	}
 
 	// Sanity check 2: Check the form even has Goals enabled.
-	if ( ! give_is_setting_enabled( give_get_meta( $atts['id'], '_give_goal_option', true ) ) ) {
+	if ( ! give_is_setting_enabled(give_get_meta($atts['id'], '_give_goal_option', true))) {
 
-		Give()->notices->print_frontend_notice( __( 'The form does not have Goals enabled.', 'give' ), true );
+		Give()->notices->print_frontend_notice(__('The form does not have Goals enabled.', 'give'), true);
 	} else {
 		// Passed all sanity checks: output Goal.
-		give_show_goal_progress( $atts['id'], $atts );
+		give_show_goal_progress($atts['id'], $atts);
 	}
 
 	$final_output = ob_get_clean();
 
-	return apply_filters( 'give_goal_shortcode_output', $final_output, $atts );
+	return apply_filters('give_goal_shortcode_output', $final_output, $atts);
 }
 
-add_shortcode( 'give_goal', 'give_goal_shortcode' );
+add_shortcode('give_goal', 'give_goal_shortcode');
 
 
 /**
@@ -187,22 +187,22 @@ 
  *
  * @return string
  */
-function give_login_form_shortcode( $atts ) {
+function give_login_form_shortcode($atts) {
 
-	$atts = shortcode_atts( array(
+	$atts = shortcode_atts(array(
 		// Add backward compatibility for redirect attribute.
 		'redirect' => '',
 		'login-redirect'  => '',
 		'logout-redirect' => '',
-	), $atts, 'give_login' );
+	), $atts, 'give_login');
 
 	// Check login-redirect attribute first, if it empty or not found then check for redirect attribute and add value of this to login-redirect attribute.
-	$atts['login-redirect'] = ! empty( $atts['login-redirect'] ) ? $atts['login-redirect'] : ( ! empty( $atts['redirect'] ) ? $atts['redirect'] : '' );
+	$atts['login-redirect'] = ! empty($atts['login-redirect']) ? $atts['login-redirect'] : ( ! empty($atts['redirect']) ? $atts['redirect'] : '');
 
-	return give_login_form( $atts['login-redirect'], $atts['logout-redirect'] );
+	return give_login_form($atts['login-redirect'], $atts['logout-redirect']);
 }
 
-add_shortcode( 'give_login', 'give_login_form_shortcode' );
+add_shortcode('give_login', 'give_login_form_shortcode');
 
 /**
  * Register Shortcode.
@@ -217,15 +217,15 @@ 
  *
  * @return string
  */
-function give_register_form_shortcode( $atts ) {
-	$atts = shortcode_atts( array(
+function give_register_form_shortcode($atts) {
+	$atts = shortcode_atts(array(
 		'redirect' => '',
-	), $atts, 'give_register' );
+	), $atts, 'give_register');
 
-	return give_register_form( $atts['redirect'] );
+	return give_register_form($atts['redirect']);
 }
 
-add_shortcode( 'give_register', 'give_register_form_shortcode' );
+add_shortcode('give_register', 'give_register_form_shortcode');
 
 /**
  * Receipt Shortcode.
@@ -238,12 +238,12 @@ 
  *
  * @return string
  */
-function give_receipt_shortcode( $atts ) {
+function give_receipt_shortcode($atts) {
 
 	global $give_receipt_args;
 
-	$give_receipt_args = shortcode_atts( array(
-		'error'          => __( 'You are missing the payment key to view this donation receipt.', 'give' ),
+	$give_receipt_args = shortcode_atts(array(
+		'error'          => __('You are missing the payment key to view this donation receipt.', 'give'),
 		'price'          => true,
 		'donor'          => true,
 		'date'           => true,
@@ -252,49 +252,49 @@ 
 		'payment_id'     => true,
 		'payment_status' => false,
 		'status_notice'  => 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 ) && give_is_setting_enabled( $email_access ) && ! Give()->email_access->token_exists ) {
+	if ( ! isset($payment_key) && give_is_setting_enabled($email_access) && ! Give()->email_access->token_exists) {
 
 		ob_start();
 
-		give_get_template_part( 'email-login-form' );
+		give_get_template_part('email-login-form');
 
 		return ob_get_clean();
 
-	} elseif ( ! isset( $payment_key ) ) {
+	} elseif ( ! isset($payment_key)) {
 
-		return Give()->notices->print_frontend_notice( $give_receipt_args['error'], false, 'error' );
+		return Give()->notices->print_frontend_notice($give_receipt_args['error'], false, 'error');
 
 	}
 
-	$user_can_view = give_can_view_receipt( $payment_key );
+	$user_can_view = give_can_view_receipt($payment_key);
 
 	// Key was provided, but user is logged out. Offer them the ability to login and view the receipt.
-	if ( ! $user_can_view && give_is_setting_enabled( $email_access ) && ! Give()->email_access->token_exists ) {
+	if ( ! $user_can_view && give_is_setting_enabled($email_access) && ! Give()->email_access->token_exists) {
 
 		ob_start();
 
-		give_get_template_part( 'email-login-form' );
+		give_get_template_part('email-login-form');
 
 		return ob_get_clean();
 
-	} elseif ( ! $user_can_view ) {
+	} elseif ( ! $user_can_view) {
 
 		global $give_login_redirect;
 
@@ -302,9 +302,9 @@ 
 
 		ob_start();
 
-		Give()->notices->print_frontend_notice( apply_filters( 'give_must_be_logged_in_error_message', __( 'You must be logged in to view this donation receipt.', 'give' ) ) );
+		Give()->notices->print_frontend_notice(apply_filters('give_must_be_logged_in_error_message', __('You must be logged in to view this donation receipt.', 'give')));
 
-		give_get_template_part( 'shortcode', 'login' );
+		give_get_template_part('shortcode', 'login');
 
 		$login_form = ob_get_clean();
 
@@ -318,20 +318,20 @@ 
 	 * or if user is logged out and donation was made as a guest, the donation session is checked for
 	 * or if user is logged in and the user can view sensitive shop data.
 	 */
-	if ( ! apply_filters( 'give_user_can_view_receipt', $user_can_view, $give_receipt_args ) ) {
-		return Give()->notices->print_frontend_notice( $give_receipt_args['error'], false, 'error' );
+	if ( ! apply_filters('give_user_can_view_receipt', $user_can_view, $give_receipt_args)) {
+		return Give()->notices->print_frontend_notice($give_receipt_args['error'], false, 'error');
 	}
 
 	ob_start();
 
-	give_get_template_part( 'shortcode', 'receipt' );
+	give_get_template_part('shortcode', 'receipt');
 
 	$display = ob_get_clean();
 
 	return $display;
 }
 
-add_shortcode( 'give_receipt', 'give_receipt_shortcode' );
+add_shortcode('give_receipt', 'give_receipt_shortcode');
 
 /**
  * Profile Editor Shortcode.
@@ -350,25 +350,25 @@ 
  *
  * @return string Output generated from the profile editor
  */
-function give_profile_editor_shortcode( $atts ) {
+function give_profile_editor_shortcode($atts) {
 
 	ob_start();
 
 	// Restrict access to donor profile, if donor and user are disconnected.
-	$is_donor_disconnected = get_user_meta( get_current_user_id(), '_give_is_donor_disconnected', true );
-	if ( is_user_logged_in() && $is_donor_disconnected ) {
-		Give()->notices->print_frontend_notice( __( 'Your Donor and User profile are no longer connected. Please contact the site administrator.', 'give' ), true, 'error' );
+	$is_donor_disconnected = get_user_meta(get_current_user_id(), '_give_is_donor_disconnected', true);
+	if (is_user_logged_in() && $is_donor_disconnected) {
+		Give()->notices->print_frontend_notice(__('Your Donor and User profile are no longer connected. Please contact the site administrator.', 'give'), true, 'error');
 		return false;
 	}
 
-	give_get_template_part( 'shortcode', 'profile-editor' );
+	give_get_template_part('shortcode', 'profile-editor');
 
 	$display = ob_get_clean();
 
 	return $display;
 }
 
-add_shortcode( 'give_profile_editor', 'give_profile_editor_shortcode' );
+add_shortcode('give_profile_editor', 'give_profile_editor_shortcode');
 
 /**
  * Process Profile Updater Form.
@@ -381,29 +381,29 @@ 
  *
  * @return bool
  */
-function give_process_profile_editor_updates( $data ) {
+function give_process_profile_editor_updates($data) {
 	// Profile field change request.
-	if ( empty( $_POST['give_profile_editor_submit'] ) && ! is_user_logged_in() ) {
+	if (empty($_POST['give_profile_editor_submit']) && ! is_user_logged_in()) {
 		return false;
 	}
 
 	// Nonce security.
-	if ( ! wp_verify_nonce( $data['give_profile_editor_nonce'], 'give-profile-editor-nonce' ) ) {
+	if ( ! wp_verify_nonce($data['give_profile_editor_nonce'], 'give-profile-editor-nonce')) {
 		return false;
 	}
 
 	$user_id       = get_current_user_id();
-	$old_user_data = get_userdata( $user_id );
+	$old_user_data = get_userdata($user_id);
 
 	/* @var Give_Donor $donor */
-	$donor = new Give_Donor( $user_id, true );
+	$donor = new Give_Donor($user_id, true);
 
-	$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;
-	$password         = ! empty( $data['give_new_user_pass1'] ) ? $data['give_new_user_pass1'] : '';
-	$confirm_password = ! empty( $data['give_new_user_pass2'] ) ? $data['give_new_user_pass2'] : '';
+	$display_name     = isset($data['give_display_name']) ? sanitize_text_field($data['give_display_name']) : $old_user_data->display_name;
+	$first_name       = isset($data['give_first_name']) ? sanitize_text_field($data['give_first_name']) : $old_user_data->first_name;
+	$last_name        = isset($data['give_last_name']) ? sanitize_text_field($data['give_last_name']) : $old_user_data->last_name;
+	$email            = isset($data['give_email']) ? sanitize_email($data['give_email']) : $old_user_data->user_email;
+	$password         = ! empty($data['give_new_user_pass1']) ? $data['give_new_user_pass1'] : '';
+	$confirm_password = ! empty($data['give_new_user_pass2']) ? $data['give_new_user_pass2'] : '';
 
 	$userdata = array(
 		'ID'           => $user_id,
@@ -422,55 +422,55 @@ 
 	 * @param int $user_id The ID of the user.
 	 * @param array $userdata User info, including ID, first name, last name, display name and email.
 	 */
-	do_action( 'give_pre_update_user_profile', $user_id, $userdata );
+	do_action('give_pre_update_user_profile', $user_id, $userdata);
 
 	// Make sure to validate first name of existing donors.
-	if ( empty( $first_name ) ) {
+	if (empty($first_name)) {
 		// Empty First Name.
-		give_set_error( 'empty_first_name', __( 'Please enter your first name.', 'give' ) );
+		give_set_error('empty_first_name', __('Please enter your first name.', 'give'));
 	}
 
 	// Make sure to validate passwords for existing Donors.
-	give_validate_user_password( $password, $confirm_password );
+	give_validate_user_password($password, $confirm_password);
 
-	if ( empty( $email ) ) {
+	if (empty($email)) {
 		// Make sure email should not be empty.
-		give_set_error( 'email_empty', __( 'The email you entered is empty.', 'give' ) );
+		give_set_error('email_empty', __('The email you entered is empty.', 'give'));
 
-	} elseif ( ! is_email( $email ) ) {
+	} elseif ( ! is_email($email)) {
 		// Make sure email should be valid.
-		give_set_error( 'email_not_valid', __( 'The email you entered is not valid. Please use another', 'give' ) );
+		give_set_error('email_not_valid', __('The email you entered is not valid. Please use another', 'give'));
 
-	} elseif ( $email != $old_user_data->user_email ) {
+	} elseif ($email != $old_user_data->user_email) {
 		// Make sure the new email doesn't belong to another user
-		if ( email_exists( $email ) ) {
-			give_set_error( 'user_email_exists', __( 'The email you entered belongs to another user. Please use another.', 'give' ) );
-		} elseif ( Give()->donors->get_donor_by( 'email', $email ) ) {
+		if (email_exists($email)) {
+			give_set_error('user_email_exists', __('The email you entered belongs to another user. Please use another.', 'give'));
+		} elseif (Give()->donors->get_donor_by('email', $email)) {
 			// Make sure the new email doesn't belong to another user
-			give_set_error( 'donor_email_exists', __( 'The email you entered belongs to another donor. Please use another.', 'give' ) );
+			give_set_error('donor_email_exists', __('The email you entered belongs to another donor. Please use another.', 'give'));
 		}
 	}
 
 	// Check for errors.
 	$errors = give_get_errors();
 
-	if ( $errors ) {
+	if ($errors) {
 		// Send back to the profile editor if there are errors.
-		wp_redirect( $data['give_redirect'] );
+		wp_redirect($data['give_redirect']);
 		give_die();
 	}
 
 	// Update Donor First Name and Last Name.
-	Give()->donors->update( $donor->id, array(
-		'name' => trim( "{$first_name} {$last_name}" ),
-	) );
-	Give()->donor_meta->update_meta( $donor->id, '_give_donor_first_name', $first_name );
-	Give()->donor_meta->update_meta( $donor->id, '_give_donor_last_name', $last_name );
+	Give()->donors->update($donor->id, array(
+		'name' => trim("{$first_name} {$last_name}"),
+	));
+	Give()->donor_meta->update_meta($donor->id, '_give_donor_first_name', $first_name);
+	Give()->donor_meta->update_meta($donor->id, '_give_donor_last_name', $last_name);
 
 	// Update the user.
-	$updated = wp_update_user( $userdata );
+	$updated = wp_update_user($userdata);
 
-	if ( $updated ) {
+	if ($updated) {
 
 		/**
 		 * Fires after updating user profile.
@@ -480,12 +480,12 @@ 
 		 * @param int $user_id The ID of the user.
 		 * @param array $userdata User info, including ID, first name, last name, display name and email.
 		 */
-		do_action( 'give_user_profile_updated', $user_id, $userdata );
-		wp_redirect( add_query_arg( 'updated', 'true', $data['give_redirect'] ) );
+		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-notices.php

Spacing +104 added lines, -104 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -53,21 +53,21 @@ 
 	 * @since 1.8.9
 	 */
 	public function __construct() {
-		add_action( 'admin_notices', array( $this, 'render_admin_notices' ), 999 );
-		add_action( 'give_dismiss_notices', array( $this, 'dismiss_notices' ) );
+		add_action('admin_notices', array($this, 'render_admin_notices'), 999);
+		add_action('give_dismiss_notices', array($this, 'dismiss_notices'));
 
-		add_action( 'give_frontend_notices', array( $this, 'render_frontend_notices' ), 999 );
-		add_action( 'give_donation_form_before_personal_info', array( $this, 'render_frontend_notices' ) );
-		add_action( 'give_ajax_donation_errors', array( $this, 'render_frontend_notices' ) );
+		add_action('give_frontend_notices', array($this, 'render_frontend_notices'), 999);
+		add_action('give_donation_form_before_personal_info', array($this, 'render_frontend_notices'));
+		add_action('give_ajax_donation_errors', array($this, 'render_frontend_notices'));
 
 		/**
 		 * Backward compatibility for deprecated params.
 		 *
 		 * @since 1.8.14
 		 */
-		add_filter( 'give_register_notice_args', array( $this, 'bc_deprecated_params' ) );
-		add_filter( 'give_frontend_errors_args', array( $this, 'bc_deprecated_params' ) );
-		add_filter( 'give_frontend_notice_args', array( $this, 'bc_deprecated_params' ) );
+		add_filter('give_register_notice_args', array($this, 'bc_deprecated_params'));
+		add_filter('give_frontend_errors_args', array($this, 'bc_deprecated_params'));
+		add_filter('give_frontend_notice_args', array($this, 'bc_deprecated_params'));
 	}
 
 	/**
@@ -80,19 +80,19 @@ 
 	 *
 	 * @return array
 	 */
-	public function bc_deprecated_params( $args ) {
+	public function bc_deprecated_params($args) {
 		/**
 		 *  Param: auto_dismissible
 		 *  deprecated in 1.8.14
 		 *
 		 *  Check if auto_dismissible is set and it true then unset and change dismissible parameter value to auto
 		 */
-		if ( isset( $args['auto_dismissible'] ) ) {
-			if ( ! empty( $args['auto_dismissible'] ) ) {
+		if (isset($args['auto_dismissible'])) {
+			if ( ! empty($args['auto_dismissible'])) {
 				$args['dismissible'] = 'auto';
 			}
 			// unset auto_dismissible as it has been deprecated.
-			unset( $args['auto_dismissible'] );
+			unset($args['auto_dismissible']);
 		}
 
 		return $args;
@@ -108,9 +108,9 @@ 
 	 *
 	 * @return bool
 	 */
-	public function register_notice( $notice_args ) {
+	public function register_notice($notice_args) {
 		// Bailout.
-		if ( empty( $notice_args['id'] ) || array_key_exists( $notice_args['id'], self::$notices ) ) {
+		if (empty($notice_args['id']) || array_key_exists($notice_args['id'], self::$notices)) {
 			return false;
 		}
 
@@ -159,39 +159,39 @@ 
 		 *
 		 * @since 1.8.14
 		 */
-		$notice_args = apply_filters( 'give_register_notice_args', $notice_args );
+		$notice_args = apply_filters('give_register_notice_args', $notice_args);
 
 		// Set extra dismiss links if any.
-		if ( false !== strpos( $notice_args['description'], 'data-dismiss-interval' ) ) {
+		if (false !== strpos($notice_args['description'], 'data-dismiss-interval')) {
 
-			preg_match_all( "/data-([^\"]*)=\"([^\"]*)\"/", $notice_args['description'], $extra_notice_dismiss_link );
+			preg_match_all("/data-([^\"]*)=\"([^\"]*)\"/", $notice_args['description'], $extra_notice_dismiss_link);
 
-			if ( ! empty( $extra_notice_dismiss_link ) ) {
-				$extra_notice_dismiss_links = array_chunk( current( $extra_notice_dismiss_link ), 3 );
-				foreach ( $extra_notice_dismiss_links as $extra_notice_dismiss_link ) {
+			if ( ! empty($extra_notice_dismiss_link)) {
+				$extra_notice_dismiss_links = array_chunk(current($extra_notice_dismiss_link), 3);
+				foreach ($extra_notice_dismiss_links as $extra_notice_dismiss_link) {
 					// Create array og key ==> value by parsing query string created after renaming data attributes.
-					$data_attribute_query_str = str_replace( array( 'data-', '-', '"' ), array(
+					$data_attribute_query_str = str_replace(array('data-', '-', '"'), array(
 						'',
 						'_',
 						'',
-					), implode( '&', $extra_notice_dismiss_link ) );
+					), implode('&', $extra_notice_dismiss_link));
 
-					$notice_args['extra_links'][] = wp_parse_args( $data_attribute_query_str );
+					$notice_args['extra_links'][] = wp_parse_args($data_attribute_query_str);
 				}
 			}
 		}
 
 
-		self::$notices[ $notice_args['id'] ] = $notice_args;
+		self::$notices[$notice_args['id']] = $notice_args;
 
 		// Auto set show param if not already set.
-		if ( ! isset( self::$notices[ $notice_args['id'] ]['show'] ) ) {
-			self::$notices[ $notice_args['id'] ]['show'] = $this->is_notice_dismissed( $notice_args ) ? false : true;
+		if ( ! isset(self::$notices[$notice_args['id']]['show'])) {
+			self::$notices[$notice_args['id']]['show'] = $this->is_notice_dismissed($notice_args) ? false : true;
 		}
 
 		// Auto set time interval for shortly.
-		if ( 'shortly' === self::$notices[ $notice_args['id'] ]['dismiss_interval'] ) {
-			self::$notices[ $notice_args['id'] ]['dismiss_interval_time'] = DAY_IN_SECONDS;
+		if ('shortly' === self::$notices[$notice_args['id']]['dismiss_interval']) {
+			self::$notices[$notice_args['id']]['dismiss_interval_time'] = DAY_IN_SECONDS;
 		}
 
 		return true;
@@ -205,51 +205,51 @@ 
 	 */
 	public function render_admin_notices() {
 		// Bailout.
-		if ( empty( self::$notices ) ) {
+		if (empty(self::$notices)) {
 			return;
 		}
 
 		$output = '';
 
-		foreach ( self::$notices as $notice_id => $notice ) {
+		foreach (self::$notices as $notice_id => $notice) {
 			// Check flag set to true to show notice.
-			if ( ! $notice['show'] ) {
+			if ( ! $notice['show']) {
 				continue;
 			}
 
 
 			// Render custom html.
-			if( ! empty( $notice['description_html'] ) ) {
+			if ( ! empty($notice['description_html'])) {
 				$output .= "{$notice['description_html']} \n";
 				continue;
 			}
 
 			// Check if notice dismissible or not.
-			if ( ! self::$has_auto_dismissible_notice ) {
-				self::$has_auto_dismissible_notice = ( 'auto' === $notice['dismissible'] );
+			if ( ! self::$has_auto_dismissible_notice) {
+				self::$has_auto_dismissible_notice = ('auto' === $notice['dismissible']);
 			}
 
 			// Check if notice dismissible or not.
-			if ( ! self::$has_dismiss_interval_notice ) {
+			if ( ! self::$has_dismiss_interval_notice) {
 				self::$has_dismiss_interval_notice = $notice['dismiss_interval'];
 			}
 
-			$css_id = ( false === strpos( $notice['id'], 'give' ) ? "give-{$notice['id']}" : $notice['id'] );
+			$css_id = (false === strpos($notice['id'], 'give') ? "give-{$notice['id']}" : $notice['id']);
 
-			$css_class = 'give-notice notice ' . ( empty( $notice['dismissible'] ) ? 'non' : 'is' ) . "-dismissible {$notice['type']} notice-{$notice['type']}";
-			$output    .= sprintf(
-				'<div id="%1$s" class="%2$s" data-dismissible="%3$s" data-dismissible-type="%4$s" data-dismiss-interval="%5$s" data-notice-id="%6$s" data-security="%7$s" data-dismiss-interval-time="%8$s" style="display: none">' . " \n",
+			$css_class = 'give-notice notice '.(empty($notice['dismissible']) ? 'non' : 'is')."-dismissible {$notice['type']} notice-{$notice['type']}";
+			$output .= sprintf(
+				'<div id="%1$s" class="%2$s" data-dismissible="%3$s" data-dismissible-type="%4$s" data-dismiss-interval="%5$s" data-notice-id="%6$s" data-security="%7$s" data-dismiss-interval-time="%8$s" style="display: none">'." \n",
 				$css_id,
 				$css_class,
-				give_clean( $notice['dismissible'] ),
+				give_clean($notice['dismissible']),
 				$notice['dismissible_type'],
 				$notice['dismiss_interval'],
 				$notice['id'],
-				empty( $notice['dismissible_type'] ) ? '' : wp_create_nonce( "give_edit_{$notice_id}_notice" ),
+				empty($notice['dismissible_type']) ? '' : wp_create_nonce("give_edit_{$notice_id}_notice"),
 				$notice['dismiss_interval_time']
 			);
 
-			$output .= ( 0 === strpos( $notice['description'], '<div' ) || 0 === strpos( $notice['description'], '<p' ) ? $notice['description'] : "<p>{$notice['description']}</p>" );
+			$output .= (0 === strpos($notice['description'], '<div') || 0 === strpos($notice['description'], '<p') ? $notice['description'] : "<p>{$notice['description']}</p>");
 			$output .= "</div> \n";
 		}
 
@@ -267,18 +267,18 @@ 
 	 *
 	 * @param int $form_id
 	 */
-	public function render_frontend_notices( $form_id = 0 ) {
+	public function render_frontend_notices($form_id = 0) {
 		$errors = give_get_errors();
 
-		$request_form_id = isset( $_REQUEST['form-id'] ) ? absint( $_REQUEST['form-id'] ) : 0;
+		$request_form_id = isset($_REQUEST['form-id']) ? absint($_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 ) {
-			self::print_frontend_errors( $errors );
+		if ($errors) {
+			self::print_frontend_errors($errors);
 
 			give_clear_errors();
 		}
@@ -291,7 +291,7 @@ 
 	 * @access private
 	 */
 	private function print_js() {
-		if ( self::$has_auto_dismissible_notice ) :
+		if (self::$has_auto_dismissible_notice) :
 			?>
 			<script>
 				jQuery(document).ready(function () {
@@ -307,7 +307,7 @@ 
 			<?php
 		endif;
 
-		if ( self::$has_dismiss_interval_notice ) :
+		if (self::$has_dismiss_interval_notice) :
 			?>
 			<script>
 				jQuery(document).ready(function () {
@@ -386,36 +386,36 @@ 
 	 * @access public
 	 */
 	public function dismiss_notices() {
-		$_post     = give_clean( $_POST );
-		$notice_id = esc_attr( $_post['notice_id'] );
+		$_post     = give_clean($_POST);
+		$notice_id = esc_attr($_post['notice_id']);
 
 		// Bailout.
 		if (
-			empty( $notice_id ) ||
-			empty( $_post['dismissible_type'] ) ||
-			empty( $_post['dismiss_interval'] ) ||
-			! check_ajax_referer( "give_edit_{$notice_id}_notice", '_wpnonce' )
+			empty($notice_id) ||
+			empty($_post['dismissible_type']) ||
+			empty($_post['dismiss_interval']) ||
+			! check_ajax_referer("give_edit_{$notice_id}_notice", '_wpnonce')
 		) {
 			wp_send_json_error();
 		}
 
-		$notice_key = Give()->notices->get_notice_key( $notice_id, $_post['dismiss_interval'] );
-		if ( 'user' === $_post['dismissible_type'] ) {
+		$notice_key = Give()->notices->get_notice_key($notice_id, $_post['dismiss_interval']);
+		if ('user' === $_post['dismissible_type']) {
 			$current_user = wp_get_current_user();
-			$notice_key   = Give()->notices->get_notice_key( $notice_id, $_post['dismiss_interval'], $current_user->ID );
+			$notice_key   = Give()->notices->get_notice_key($notice_id, $_post['dismiss_interval'], $current_user->ID);
 		}
 
-		$notice_dismiss_time = ! empty( $_post['dismiss_interval_time'] ) ? $_post['dismiss_interval_time'] : null;
+		$notice_dismiss_time = ! empty($_post['dismiss_interval_time']) ? $_post['dismiss_interval_time'] : null;
 
 		// Save option to hide notice.
-		Give_Cache::set( $notice_key, true, $notice_dismiss_time, true );
+		Give_Cache::set($notice_key, true, $notice_dismiss_time, true);
 
 		/**
 		 * Fire the action when notice dismissed
 		 *
 		 * @since 2.0
 		 */
-		do_action( 'give_dismiss_notices', $_post );
+		do_action('give_dismiss_notices', $_post);
 
 		wp_send_json_success();
 	}
@@ -433,18 +433,18 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_notice_key( $notice_id, $dismiss_interval = null, $user_id = 0 ) {
+	public function get_notice_key($notice_id, $dismiss_interval = null, $user_id = 0) {
 		$notice_key = "_give_notice_{$notice_id}";
 
-		if ( ! empty( $dismiss_interval ) ) {
+		if ( ! empty($dismiss_interval)) {
 			$notice_key .= "_{$dismiss_interval}";
 		}
 
-		if ( $user_id ) {
+		if ($user_id) {
 			$notice_key .= "_{$user_id}";
 		}
 
-		$notice_key = sanitize_key( $notice_key );
+		$notice_key = sanitize_key($notice_key);
 
 		return $notice_key;
 	}
@@ -457,11 +457,11 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_dismiss_link( $notice_args ) {
+	public function get_dismiss_link($notice_args) {
 		$notice_args = wp_parse_args(
 			$notice_args,
 			array(
-				'title'                 => __( 'Click here', 'give' ),
+				'title'                 => __('Click here', 'give'),
 				'dismissible_type'      => '',
 				'dismiss_interval'      => '',
 				'dismiss_interval_time' => null,
@@ -488,31 +488,31 @@ 
 	 *
 	 * @return bool|null
 	 */
-	public function is_notice_dismissed( $notice ) {
-		$notice_key          = $this->get_notice_key( $notice['id'], $notice['dismiss_interval'] );
+	public function is_notice_dismissed($notice) {
+		$notice_key          = $this->get_notice_key($notice['id'], $notice['dismiss_interval']);
 		$is_notice_dismissed = false;
 
-		if ( 'user' === $notice['dismissible_type'] ) {
+		if ('user' === $notice['dismissible_type']) {
 			$current_user = wp_get_current_user();
-			$notice_key   = Give()->notices->get_notice_key( $notice['id'], $notice['dismiss_interval'], $current_user->ID );
+			$notice_key   = Give()->notices->get_notice_key($notice['id'], $notice['dismiss_interval'], $current_user->ID);
 		}
 
-		$notice_data = Give_Cache::get( $notice_key, true );
+		$notice_data = Give_Cache::get($notice_key, true);
 
 		// Find notice dismiss link status if notice has extra dismissible links.
-		if ( ( empty( $notice_data ) || is_wp_error( $notice_data ) ) && ! empty( $notice['extra_links'] ) ) {
+		if ((empty($notice_data) || is_wp_error($notice_data)) && ! empty($notice['extra_links'])) {
 
-			foreach ( $notice['extra_links'] as $extra_link ) {
-				$new_notice_data = wp_parse_args( $extra_link, $notice );
-				unset( $new_notice_data['extra_links'] );
+			foreach ($notice['extra_links'] as $extra_link) {
+				$new_notice_data = wp_parse_args($extra_link, $notice);
+				unset($new_notice_data['extra_links']);
 
-				if ( $is_notice_dismissed = $this->is_notice_dismissed( $new_notice_data ) ) {
+				if ($is_notice_dismissed = $this->is_notice_dismissed($new_notice_data)) {
 					return $is_notice_dismissed;
 				}
 			}
 		}
 
-		$is_notice_dismissed = ! empty( $notice_data ) && ! is_wp_error( $notice_data );
+		$is_notice_dismissed = ! empty($notice_data) && ! is_wp_error($notice_data);
 
 		return $is_notice_dismissed;
 	}
@@ -526,9 +526,9 @@ 
 	 *
 	 * @param array $errors
 	 */
-	public static function print_frontend_errors( $errors ) {
+	public static function print_frontend_errors($errors) {
 		// Bailout.
-		if ( ! $errors ) {
+		if ( ! $errors) {
 			return;
 		}
 
@@ -543,37 +543,37 @@ 
 		);
 
 		// Note: we will remove give_errors class in future.
-		$classes = apply_filters( 'give_error_class', array( 'give_notices', 'give_errors' ) );
+		$classes = apply_filters('give_error_class', array('give_notices', 'give_errors'));
 
-		echo sprintf( '<div class="%s">', implode( ' ', $classes ) );
+		echo sprintf('<div class="%s">', implode(' ', $classes));
 
 		// Loop error codes and display errors.
-		foreach ( $errors as $error_id => $error ) {
+		foreach ($errors as $error_id => $error) {
 			// Backward compatibility v<1.8.11
-			if ( is_string( $error ) ) {
+			if (is_string($error)) {
 				$error = array(
 					'message'     => $error,
 					'notice_args' => array(),
 				);
 			}
 
-			$notice_args = wp_parse_args( $error['notice_args'], $default_notice_args );
+			$notice_args = wp_parse_args($error['notice_args'], $default_notice_args);
 
 			/**
 			 * Filter to modify Frontend Errors args before errors is display.
 			 *
 			 * @since 1.8.14
 			 */
-			$notice_args = apply_filters( 'give_frontend_errors_args', $notice_args );
+			$notice_args = apply_filters('give_frontend_errors_args', $notice_args);
 
 			echo sprintf(
 				'<div class="give_error give_notice" id="give_error_%1$s" data-dismissible="%2$s" data-dismiss-interval="%3$d">
 						<p><strong>%4$s</strong>: %5$s</p>
 					</div>',
 				$error_id,
-				give_clean( $notice_args['dismissible'] ),
-				absint( $notice_args['dismiss_interval'] ),
-				esc_html__( 'Error', 'give' ),
+				give_clean($notice_args['dismissible']),
+				absint($notice_args['dismiss_interval']),
+				esc_html__('Error', 'give'),
 				$error['message']
 			);
 		}
@@ -595,8 +595,8 @@ 
 	 *
 	 * @return  string
 	 */
-	public static function print_frontend_notice( $message, $echo = true, $notice_type = 'warning', $notice_args = array() ) {
-		if ( empty( $message ) ) {
+	public static function print_frontend_notice($message, $echo = true, $notice_type = 'warning', $notice_args = array()) {
+		if (empty($message)) {
 			return '';
 		}
 
@@ -610,14 +610,14 @@ 
 			'dismiss_interval' => 5000,
 		);
 
-		$notice_args = wp_parse_args( $notice_args, $default_notice_args );
+		$notice_args = wp_parse_args($notice_args, $default_notice_args);
 
 		/**
 		 * Filter to modify Frontend notice args before notices is display.
 		 *
 		 * @since 1.8.14
 		 */
-		$notice_args = apply_filters( 'give_frontend_notice_args', $notice_args );
+		$notice_args = apply_filters('give_frontend_notice_args', $notice_args);
 
 		// Note: we will remove give_errors class in future.
 		$error = sprintf(
@@ -627,12 +627,12 @@ 
 				</p>
 			</div>',
 			$notice_type,
-			give_clean( $notice_args['dismissible'] ),
-			absint( $notice_args['dismiss_interval'] ),
+			give_clean($notice_args['dismissible']),
+			absint($notice_args['dismiss_interval']),
 			$message
 		);
 
-		if ( ! $echo ) {
+		if ( ! $echo) {
 			return $error;
 		}
 
@@ -652,24 +652,24 @@ 
 	 *
 	 * @return string
 	 */
-	public function print_admin_notices( $notice_args = array() ) {
+	public function print_admin_notices($notice_args = array()) {
 		// Bailout.
-		if ( empty( $notice_args['description'] ) ) {
+		if (empty($notice_args['description'])) {
 			return '';
 		}
 
-		$defaults    = array(
+		$defaults = array(
 			'id'          => '',
 			'echo'        => true,
 			'notice_type' => 'warning',
 			'dismissible' => true,
 		);
-		$notice_args = wp_parse_args( $notice_args, $defaults );
+		$notice_args = wp_parse_args($notice_args, $defaults);
 
 		$output    = '';
-		$css_id    = ! empty( $notice_args['id'] ) ? $notice_args['id'] : uniqid( 'give-inline-notice-' );
+		$css_id    = ! empty($notice_args['id']) ? $notice_args['id'] : uniqid('give-inline-notice-');
 		$css_class = "notice-{$notice_args['notice_type']} give-notice notice inline";
-		$css_class .= ( $notice_args['dismissible'] ) ? ' is-dismissible' : '';
+		$css_class .= ($notice_args['dismissible']) ? ' is-dismissible' : '';
 		$output    .= sprintf(
 			'<div id="%1$s" class="%2$s"><p>%3$s</p></div>',
 			$css_id,
@@ -677,7 +677,7 @@ 
 			$notice_args['description']
 		);
 
-		if ( ! $notice_args['echo'] ) {
+		if ( ! $notice_args['echo']) {
 			return $output;
 		}
 

includes/gateways/actions.php

Spacing +18 added lines, -18 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,14 +21,14 @@ 
  *
  * @param $data
  */
-function give_process_gateway_select( $data ) {
-	if ( isset( $_POST['gateway_submit'] ) ) {
-		wp_redirect( esc_url( add_query_arg( 'payment-mode', $_POST['payment-mode'] ) ) );
+function give_process_gateway_select($data) {
+	if (isset($_POST['gateway_submit'])) {
+		wp_redirect(esc_url(add_query_arg('payment-mode', $_POST['payment-mode'])));
 		exit;
 	}
 }
 
-add_action( 'give_gateway_select', 'give_process_gateway_select' );
+add_action('give_gateway_select', 'give_process_gateway_select');
 
 /**
  * Loads a payment gateway via AJAX.
@@ -38,20 +38,20 @@ 
  * @return void
  */
 function give_load_ajax_gateway() {
-	if ( isset( $_POST['give_payment_mode'] ) ) {
+	if (isset($_POST['give_payment_mode'])) {
 		/**
 		 * Fire to render donation form.
 		 *
 		 * @since 1.7
 		 */
-		do_action( 'give_donation_form', $_POST['give_form_id'] );
+		do_action('give_donation_form', $_POST['give_form_id']);
 
 		exit();
 	}
 }
 
-add_action( 'wp_ajax_give_load_gateway', 'give_load_ajax_gateway' );
-add_action( 'wp_ajax_nopriv_give_load_gateway', 'give_load_ajax_gateway' );
+add_action('wp_ajax_give_load_gateway', 'give_load_ajax_gateway');
+add_action('wp_ajax_nopriv_give_load_gateway', 'give_load_ajax_gateway');
 
 /**
  * Create wp nonce using Ajax call.
@@ -63,18 +63,18 @@ 
  * @return void
  */
 function give_donation_form_nonce() {
-	if ( isset( $_POST['give_form_id'] ) ) {
+	if (isset($_POST['give_form_id'])) {
 
 		// Get donation form id.
-		$form_id = is_numeric( $_POST['give_form_id'] ) ? absint( $_POST['give_form_id'] ) : 0;
+		$form_id = is_numeric($_POST['give_form_id']) ? absint($_POST['give_form_id']) : 0;
 
 		// Send nonce json data.
-		wp_send_json_success( wp_create_nonce( "donation_form_nonce_{$form_id}" ) );
+		wp_send_json_success(wp_create_nonce("donation_form_nonce_{$form_id}"));
 	}
 }
 
-add_action( 'wp_ajax_give_donation_form_nonce', 'give_donation_form_nonce' );
-add_action( 'wp_ajax_nopriv_give_donation_form_nonce', 'give_donation_form_nonce' );
+add_action('wp_ajax_give_donation_form_nonce', 'give_donation_form_nonce');
+add_action('wp_ajax_nopriv_give_donation_form_nonce', 'give_donation_form_nonce');
 
 /**
  * Sets an error within the donation form if no gateways are enabled.
@@ -86,11 +86,11 @@ 
 function give_no_gateway_error() {
 	$gateways = give_get_enabled_payment_gateways();
 
-	if ( empty( $gateways ) ) {
-		give_set_error( 'no_gateways', esc_html__( 'You must enable a payment gateway to use Give.', 'give' ) );
+	if (empty($gateways)) {
+		give_set_error('no_gateways', esc_html__('You must enable a payment gateway to use Give.', 'give'));
 	} else {
-		give_unset_error( 'no_gateways' );
+		give_unset_error('no_gateways');
 	}
 }
 
-add_action( 'init', 'give_no_gateway_error' );
+add_action('init', 'give_no_gateway_error');

includes/gateways/functions.php

Spacing +70 added lines, -70 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,16 +24,16 @@ 
 	// Default, built-in gateways
 	$gateways = array(
 		'paypal' => array(
-			'admin_label'    => __( 'PayPal Standard', 'give' ),
-			'checkout_label' => __( 'PayPal', 'give' ),
+			'admin_label'    => __('PayPal Standard', 'give'),
+			'checkout_label' => __('PayPal', 'give'),
 		),
 		'manual' => array(
-			'admin_label'    => __( 'Test Donation', 'give' ),
-			'checkout_label' => __( 'Test Donation', 'give' )
+			'admin_label'    => __('Test Donation', 'give'),
+			'checkout_label' => __('Test Donation', 'give')
 		),
 	);
 
-	return apply_filters( 'give_payment_gateways', $gateways );
+	return apply_filters('give_payment_gateways', $gateways);
 
 }
 
@@ -46,24 +46,24 @@ 
  *
  * @return array $gateway_list All the available gateways
  */
-function give_get_enabled_payment_gateways( $form_id = 0 ) {
+function give_get_enabled_payment_gateways($form_id = 0) {
 
 	$gateways = give_get_payment_gateways();
 
-	$enabled = isset( $_POST['gateways'] ) ? $_POST['gateways'] : give_get_option( 'gateways' );
+	$enabled = isset($_POST['gateways']) ? $_POST['gateways'] : give_get_option('gateways');
 
 	$gateway_list = array();
 
-	foreach ( $gateways as $key => $gateway ) {
-		if ( isset( $enabled[ $key ] ) && $enabled[ $key ] == 1 ) {
-			$gateway_list[ $key ] = $gateway;
+	foreach ($gateways as $key => $gateway) {
+		if (isset($enabled[$key]) && $enabled[$key] == 1) {
+			$gateway_list[$key] = $gateway;
 		}
 	}
 
 	// Set order of payment gateway in list.
-	$gateway_list = give_get_ordered_payment_gateways( $gateway_list );
+	$gateway_list = give_get_ordered_payment_gateways($gateway_list);
 
-	return apply_filters( 'give_enabled_payment_gateways', $gateway_list, $form_id );
+	return apply_filters('give_enabled_payment_gateways', $gateway_list, $form_id);
 }
 
 /**
@@ -75,12 +75,12 @@ 
  *
  * @return boolean true if enabled, false otherwise
  */
-function give_is_gateway_active( $gateway ) {
+function give_is_gateway_active($gateway) {
 	$gateways = give_get_enabled_payment_gateways();
 
-	$ret = array_key_exists( $gateway, $gateways );
+	$ret = array_key_exists($gateway, $gateways);
 
-	return apply_filters( 'give_is_gateway_active', $ret, $gateway, $gateways );
+	return apply_filters('give_is_gateway_active', $ret, $gateway, $gateways);
 }
 
 /**
@@ -92,25 +92,25 @@ 
  *
  * @return string Gateway ID
  */
-function give_get_default_gateway( $form_id ) {
+function give_get_default_gateway($form_id) {
 
-	$enabled_gateways = array_keys( give_get_enabled_payment_gateways() );
+	$enabled_gateways = array_keys(give_get_enabled_payment_gateways());
 	$default_gateway  = give_get_option('default_gateway');
-	$default          = ! empty( $default_gateway ) && give_is_gateway_active( $default_gateway ) ? $default_gateway : $enabled_gateways[0];
-	$form_default     = give_get_meta( $form_id, '_give_default_gateway', true );
+	$default          = ! empty($default_gateway) && give_is_gateway_active($default_gateway) ? $default_gateway : $enabled_gateways[0];
+	$form_default     = give_get_meta($form_id, '_give_default_gateway', true);
 
 	// Single Form settings varies compared to the Global default settings.
 	if (
-		! empty( $form_default ) &&
+		! empty($form_default) &&
 		$form_id !== null &&
 		$default !== $form_default &&
 		'global' !== $form_default &&
-		give_is_gateway_active( $form_default )
+		give_is_gateway_active($form_default)
 	) {
 		$default = $form_default;
 	}
 
-	return apply_filters( 'give_default_gateway', $default );
+	return apply_filters('give_default_gateway', $default);
 }
 
 /**
@@ -122,18 +122,18 @@ 
  *
  * @return string Gateway admin label
  */
-function give_get_gateway_admin_label( $gateway ) {
+function give_get_gateway_admin_label($gateway) {
 	$gateways = give_get_payment_gateways();
-	$label    = isset( $gateways[ $gateway ] ) ? $gateways[ $gateway ]['admin_label'] : $gateway;
-	$payment  = isset( $_GET['id'] ) ? absint( $_GET['id'] ) : false;
+	$label    = isset($gateways[$gateway]) ? $gateways[$gateway]['admin_label'] : $gateway;
+	$payment  = isset($_GET['id']) ? absint($_GET['id']) : false;
 
-	if ( $gateway == 'manual' && $payment ) {
-		if ( give_donation_amount( $payment ) == 0 ) {
-			$label = __( 'Test Donation', 'give' );
+	if ($gateway == 'manual' && $payment) {
+		if (give_donation_amount($payment) == 0) {
+			$label = __('Test Donation', 'give');
 		}
 	}
 
-	return apply_filters( 'give_gateway_admin_label', $label, $gateway );
+	return apply_filters('give_gateway_admin_label', $label, $gateway);
 }
 
 /**
@@ -145,15 +145,15 @@ 
  *
  * @return string Checkout label for the gateway
  */
-function give_get_gateway_checkout_label( $gateway ) {
+function give_get_gateway_checkout_label($gateway) {
 	$gateways = give_get_payment_gateways();
-	$label    = isset( $gateways[ $gateway ] ) ? $gateways[ $gateway ]['checkout_label'] : $gateway;
+	$label    = isset($gateways[$gateway]) ? $gateways[$gateway]['checkout_label'] : $gateway;
 
-	if ( $gateway == 'manual' ) {
-		$label = __( 'Test Donation', 'give' );
+	if ($gateway == 'manual') {
+		$label = __('Test Donation', 'give');
 	}
 
-	return apply_filters( 'give_gateway_checkout_label', $label, $gateway );
+	return apply_filters('give_gateway_checkout_label', $label, $gateway);
 }
 
 /**
@@ -165,11 +165,11 @@ 
  *
  * @return array Options the gateway supports
  */
-function give_get_gateway_supports( $gateway ) {
+function give_get_gateway_supports($gateway) {
 	$gateways = give_get_enabled_payment_gateways();
-	$supports = isset( $gateways[ $gateway ]['supports'] ) ? $gateways[ $gateway ]['supports'] : array();
+	$supports = isset($gateways[$gateway]['supports']) ? $gateways[$gateway]['supports'] : array();
 
-	return apply_filters( 'give_gateway_supports', $supports, $gateway );
+	return apply_filters('give_gateway_supports', $supports, $gateway);
 }
 
 /**
@@ -182,9 +182,9 @@ 
  *
  * @return void
  */
-function give_send_to_gateway( $gateway, $payment_data ) {
+function give_send_to_gateway($gateway, $payment_data) {
 
-	$payment_data['gateway_nonce'] = wp_create_nonce( 'give-gateway' );
+	$payment_data['gateway_nonce'] = wp_create_nonce('give-gateway');
 
 	/**
 	 * Fires while loading payment gateway via AJAX.
@@ -195,7 +195,7 @@ 
 	 *
 	 * @param array $payment_data All the payment data to be sent to the gateway.
 	 */
-	do_action( "give_gateway_{$gateway}", $payment_data );
+	do_action("give_gateway_{$gateway}", $payment_data);
 }
 
 
@@ -209,34 +209,34 @@ 
  *
  * @return string $enabled_gateway The slug of the gateway
  */
-function give_get_chosen_gateway( $form_id ) {
+function give_get_chosen_gateway($form_id) {
 
-	$request_form_id = isset( $_REQUEST['give_form_id'] ) ? $_REQUEST['give_form_id'] : 0;
+	$request_form_id = isset($_REQUEST['give_form_id']) ? $_REQUEST['give_form_id'] : 0;
 
 	// Back to check if 'form-id' is present.
-	if ( empty( $request_form_id ) ) {
-		$request_form_id = isset( $_REQUEST['form-id'] ) ? $_REQUEST['form-id'] : 0;
+	if (empty($request_form_id)) {
+		$request_form_id = isset($_REQUEST['form-id']) ? $_REQUEST['form-id'] : 0;
 	}
 
-	$request_payment_mode = isset( $_REQUEST['payment-mode'] ) ? $_REQUEST['payment-mode'] : '';
+	$request_payment_mode = isset($_REQUEST['payment-mode']) ? $_REQUEST['payment-mode'] : '';
 	$chosen               = false;
 
 	// If both 'payment-mode' and 'form-id' then set for only this form.
-	if ( ! empty( $request_form_id ) && $form_id == $request_form_id ) {
+	if ( ! empty($request_form_id) && $form_id == $request_form_id) {
 		$chosen = $request_payment_mode;
-	} elseif ( empty( $request_form_id ) && $request_payment_mode ) {
+	} elseif (empty($request_form_id) && $request_payment_mode) {
 		// If no 'form-id' but there is 'payment-mode'.
 		$chosen = $request_payment_mode;
 	}
 
 	// Get the enable gateway based of chosen var.
-	if ( $chosen && give_is_gateway_active( $chosen ) ) {
-		$enabled_gateway = urldecode( $chosen );
+	if ($chosen && give_is_gateway_active($chosen)) {
+		$enabled_gateway = urldecode($chosen);
 	} else {
-		$enabled_gateway = give_get_default_gateway( $form_id );
+		$enabled_gateway = give_get_default_gateway($form_id);
 	}
 
-	return apply_filters( 'give_chosen_gateway', $enabled_gateway );
+	return apply_filters('give_chosen_gateway', $enabled_gateway);
 
 }
 
@@ -255,8 +255,8 @@ 
  *
  * @return int             ID of the new log entry.
  */
-function give_record_log( $title = '', $message = '', $parent = 0, $type = null ) {
-	return Give()->logs->add( $title, $message, $parent, $type );
+function give_record_log($title = '', $message = '', $parent = 0, $type = null) {
+	return Give()->logs->add($title, $message, $parent, $type);
 }
 
 /**
@@ -273,10 +273,10 @@ 
  *
  * @return int ID of the new log entry
  */
-function give_record_gateway_error( $title = '', $message = '', $parent = 0 ) {
-	$title = empty( $title ) ? esc_html__( 'Payment Error', 'give' ) : $title;
+function give_record_gateway_error($title = '', $message = '', $parent = 0) {
+	$title = empty($title) ? esc_html__('Payment Error', 'give') : $title;
 
-	return give_record_log( $title, $message, $parent, 'gateway_error' );
+	return give_record_log($title, $message, $parent, 'gateway_error');
 }
 
 /**
@@ -289,7 +289,7 @@ 
  *
  * @return int
  */
-function give_count_sales_by_gateway( $gateway_id = 'paypal', $status = 'publish' ) {
+function give_count_sales_by_gateway($gateway_id = 'paypal', $status = 'publish') {
 
 	$ret  = 0;
 	$args = array(
@@ -301,9 +301,9 @@ 
 		'fields'      => 'ids',
 	);
 
-	$payments = new WP_Query( $args );
+	$payments = new WP_Query($args);
 
-	if ( $payments ) {
+	if ($payments) {
 		$ret = $payments->post_count;
 	}
 
@@ -320,27 +320,27 @@ 
  *
  * @return array $gateways All the available gateways
  */
-function give_get_ordered_payment_gateways( $gateways ) {
+function give_get_ordered_payment_gateways($gateways) {
 
 	// Get gateways setting.
-	$gateways_setting = isset( $_POST['gateways'] ) ? $_POST['gateways'] : give_get_option( 'gateways' );
+	$gateways_setting = isset($_POST['gateways']) ? $_POST['gateways'] : give_get_option('gateways');
 
 	// Return from here if we do not have gateways setting.
-	if ( empty( $gateways_setting ) ) {
+	if (empty($gateways_setting)) {
 		return $gateways;
 	}
 
 	// Reverse array to order payment gateways.
-	$gateways_setting = array_reverse( $gateways_setting );
+	$gateways_setting = array_reverse($gateways_setting);
 
 	// Reorder gateways array
-	foreach ( $gateways_setting as $gateway_key => $value ) {
+	foreach ($gateways_setting as $gateway_key => $value) {
 
-		$new_gateway_value = isset( $gateways[ $gateway_key ] ) ? $gateways[ $gateway_key ] : '';
-		unset( $gateways[ $gateway_key ] );
+		$new_gateway_value = isset($gateways[$gateway_key]) ? $gateways[$gateway_key] : '';
+		unset($gateways[$gateway_key]);
 
-		if ( ! empty( $new_gateway_value ) ) {
-			$gateways = array_merge( array( $gateway_key => $new_gateway_value ), $gateways );
+		if ( ! empty($new_gateway_value)) {
+			$gateways = array_merge(array($gateway_key => $new_gateway_value), $gateways);
 		}
 	}
 
@@ -351,5 +351,5 @@ 
 	 *
 	 * @param array $gateways All the available gateways
 	 */
-	return apply_filters( 'give_payment_gateways_order', $gateways );
+	return apply_filters('give_payment_gateways_order', $gateways);
 }
\ No newline at end of file

includes/class-give-html-elements.php

Spacing +156 added lines, -156 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -35,7 +35,7 @@ 
 	 *
 	 * @return string       Donations dropdown.
 	 */
-	public function donations_dropdown( $args = array() ) {
+	public function donations_dropdown($args = array()) {
 
 		$defaults = array(
 			'name'        => 'donations',
@@ -45,32 +45,32 @@ 
 			'selected'    => 0,
 			'chosen'      => false,
 			'number'      => 30,
-			'placeholder' => __( 'Select a donation', 'give' ),
+			'placeholder' => __('Select a donation', 'give'),
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		$payments = new Give_Payments_Query( array(
+		$payments = new Give_Payments_Query(array(
 			'number' => $args['number'],
-		) );
+		));
 
 		$payments = $payments->get_payments();
 
 		$options = array();
 
 		// Provide nice human readable options.
-		if ( $payments ) {
+		if ($payments) {
 			$options[0] = $args['placeholder'];
-			foreach ( $payments as $payment ) {
+			foreach ($payments as $payment) {
 
-				$options[ absint( $payment->ID ) ] = esc_html( '#' . $payment->ID . ' - ' . $payment->email . ' - ' . $payment->form_title );
+				$options[absint($payment->ID)] = esc_html('#'.$payment->ID.' - '.$payment->email.' - '.$payment->form_title);
 
 			}
 		} else {
-			$options[0] = __( 'No donations found.', 'give' );
+			$options[0] = __('No donations found.', 'give');
 		}
 
-		$output = $this->select( array(
+		$output = $this->select(array(
 			'name'             => $args['name'],
 			'selected'         => $args['selected'],
 			'id'               => $args['id'],
@@ -82,7 +82,7 @@ 
 			'select_atts'      => $args['select_atts'],
 			'show_option_all'  => false,
 			'show_option_none' => false,
-		) );
+		));
 
 		return $output;
 	}
@@ -99,7 +99,7 @@ 
 	 *
 	 * @return string      Give forms dropdown.
 	 */
-	public function forms_dropdown( $args = array() ) {
+	public function forms_dropdown($args = array()) {
 
 		$defaults = array(
 			'name'        => 'forms',
@@ -109,13 +109,13 @@ 
 			'selected'    => 0,
 			'chosen'      => false,
 			'number'      => 30,
-			'placeholder' => esc_attr__( 'All Forms', 'give' ),
+			'placeholder' => esc_attr__('All Forms', 'give'),
 			'data'        => array(
 				'search-type' => 'form',
 			),
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
 		$form_args = array(
 			'post_type'      => 'give_forms',
@@ -124,34 +124,34 @@ 
 			'posts_per_page' => $args['number'],
 		);
 
-		$cache_key   = Give_Cache::get_key( 'give_forms', $form_args, false );
+		$cache_key = Give_Cache::get_key('give_forms', $form_args, false);
 
 		// Get forms from cache.
-		$forms = Give_Cache::get_db_query( $cache_key );
+		$forms = Give_Cache::get_db_query($cache_key);
 
-		if ( is_null( $forms ) ) {
-			$forms = get_posts( $form_args );
-			Give_Cache::set_db_query( $cache_key, $forms );
+		if (is_null($forms)) {
+			$forms = get_posts($form_args);
+			Give_Cache::set_db_query($cache_key, $forms);
 		}
 
 		$options = array();
 
 		// Ensure the selected.
-		if ( false !== $args['selected'] && $args['selected'] !== 0 ) {
-			$options[ $args['selected'] ] = get_the_title( $args['selected'] );
+		if (false !== $args['selected'] && $args['selected'] !== 0) {
+			$options[$args['selected']] = get_the_title($args['selected']);
 		}
 
-		if ( $forms ) {
+		if ($forms) {
 			$options[0] = $args['placeholder'];
-			foreach ( $forms as $form ) {
-				$form_title                     = empty( $form->post_title ) ? sprintf( __( 'Untitled (#%s)', 'give' ), $form->ID ) : $form->post_title;
-				$options[ absint( $form->ID ) ] = esc_html( $form_title );
+			foreach ($forms as $form) {
+				$form_title                     = empty($form->post_title) ? sprintf(__('Untitled (#%s)', 'give'), $form->ID) : $form->post_title;
+				$options[absint($form->ID)] = esc_html($form_title);
 			}
 		} else {
-			$options[0] = esc_html__( 'No forms found.', 'give' );
+			$options[0] = esc_html__('No forms found.', 'give');
 		}
 
-		$output = $this->select( array(
+		$output = $this->select(array(
 			'name'             => $args['name'],
 			'selected'         => $args['selected'],
 			'id'               => $args['id'],
@@ -163,7 +163,7 @@ 
 			'show_option_all'  => false,
 			'show_option_none' => false,
 			'data'             => $args['data'],
-		) );
+		));
 
 		return $output;
 	}
@@ -180,7 +180,7 @@ 
 	 *
 	 * @return string      Donors dropdown.
 	 */
-	public function donor_dropdown( $args = array() ) {
+	public function donor_dropdown($args = array()) {
 
 		$defaults = array(
 			'name'        => 'donors',
@@ -189,57 +189,57 @@ 
 			'multiple'    => false,
 			'selected'    => 0,
 			'chosen'      => true,
-			'placeholder' => esc_attr__( 'Select a Donor', 'give' ),
+			'placeholder' => esc_attr__('Select a Donor', 'give'),
 			'number'      => 30,
 			'data'        => array(
 				'search-type' => 'donor',
 			),
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		$donors = Give()->donors->get_donors( array(
+		$donors = Give()->donors->get_donors(array(
 			'number' => $args['number']
-		) );
+		));
 
 		$options = array();
 
-		if ( $donors ) {
-			$options[0] = esc_html__( 'No donor attached', 'give' );
-			foreach ( $donors as $donor ) {
-				$options[ absint( $donor->id ) ] = esc_html( $donor->name . ' (' . $donor->email . ')' );
+		if ($donors) {
+			$options[0] = esc_html__('No donor attached', 'give');
+			foreach ($donors as $donor) {
+				$options[absint($donor->id)] = esc_html($donor->name.' ('.$donor->email.')');
 			}
 		} else {
-			$options[0] = esc_html__( 'No donors found.', 'give' );
+			$options[0] = esc_html__('No donors found.', 'give');
 		}
 
-		if ( ! empty( $args['selected'] ) ) {
+		if ( ! empty($args['selected'])) {
 
 			// If a selected customer has been specified, we need to ensure it's in the initial list of customers displayed.
-			if ( ! array_key_exists( $args['selected'], $options ) ) {
+			if ( ! array_key_exists($args['selected'], $options)) {
 
-				$donor = new Give_Donor( $args['selected'] );
+				$donor = new Give_Donor($args['selected']);
 
-				if ( $donor ) {
+				if ($donor) {
 
-					$options[ absint( $args['selected'] ) ] = esc_html( $donor->name . ' (' . $donor->email . ')' );
+					$options[absint($args['selected'])] = esc_html($donor->name.' ('.$donor->email.')');
 
 				}
 			}
 		}
 
-		$output = $this->select( array(
+		$output = $this->select(array(
 			'name'             => $args['name'],
 			'selected'         => $args['selected'],
 			'id'               => $args['id'],
-			'class'            => $args['class'] . ' give-customer-select',
+			'class'            => $args['class'].' give-customer-select',
 			'options'          => $options,
 			'multiple'         => $args['multiple'],
 			'chosen'           => $args['chosen'],
 			'show_option_all'  => false,
 			'show_option_none' => false,
 			'data'             => $args['data'],
-		) );
+		));
 
 		return $output;
 	}
@@ -258,21 +258,21 @@ 
 	 *
 	 * @return string           Categories dropdown.
 	 */
-	public function category_dropdown( $name = 'give_forms_categories', $selected = 0, $args = array() ) {
-		$categories = get_terms( 'give_forms_category', apply_filters( 'give_forms_category_dropdown', array() ) );
+	public function category_dropdown($name = 'give_forms_categories', $selected = 0, $args = array()) {
+		$categories = get_terms('give_forms_category', apply_filters('give_forms_category_dropdown', array()));
 		$options    = array();
 
-		foreach ( $categories as $category ) {
-			$options[ absint( $category->term_id ) ] = esc_html( $category->name );
+		foreach ($categories as $category) {
+			$options[absint($category->term_id)] = esc_html($category->name);
 		}
 
-		$output = $this->select( wp_parse_args( $args, array(
+		$output = $this->select(wp_parse_args($args, array(
 			'name'             => $name,
 			'selected'         => $selected,
 			'options'          => $options,
-			'show_option_all'  => esc_html__( 'All Categories', 'give' ),
+			'show_option_all'  => esc_html__('All Categories', 'give'),
 			'show_option_none' => false,
-		) ) );
+		)));
 
 		return $output;
 	}
@@ -291,21 +291,21 @@ 
 	 *
 	 * @return string           Tags dropdown.
 	 */
-	public function tags_dropdown( $name = 'give_forms_tags', $selected = 0, $args = array() ) {
-		$tags    = get_terms( 'give_forms_tag', apply_filters( 'give_forms_tag_dropdown', array() ) );
+	public function tags_dropdown($name = 'give_forms_tags', $selected = 0, $args = array()) {
+		$tags    = get_terms('give_forms_tag', apply_filters('give_forms_tag_dropdown', array()));
 		$options = array();
 
-		foreach ( $tags as $tag ) {
-			$options[ absint( $tag->term_id ) ] = esc_html( $tag->name );
+		foreach ($tags as $tag) {
+			$options[absint($tag->term_id)] = esc_html($tag->name);
 		}
 
-		$output = $this->select( wp_parse_args( $args, array(
+		$output = $this->select(wp_parse_args($args, array(
 			'name'             => $name,
 			'selected'         => $selected,
 			'options'          => $options,
-			'show_option_all'  => esc_html__( 'All Tags', 'give' ),
+			'show_option_all'  => esc_html__('All Tags', 'give'),
 			'show_option_none' => false,
-		) ) );
+		)));
 
 		return $output;
 	}
@@ -325,25 +325,25 @@ 
 	 *
 	 * @return string               Years dropdown.
 	 */
-	public function year_dropdown( $name = 'year', $selected = 0, $years_before = 5, $years_after = 0 ) {
-		$current    = date( 'Y' );
-		$start_year = $current - absint( $years_before );
-		$end_year   = $current + absint( $years_after );
-		$selected   = empty( $selected ) ? date( 'Y' ) : $selected;
+	public function year_dropdown($name = 'year', $selected = 0, $years_before = 5, $years_after = 0) {
+		$current    = date('Y');
+		$start_year = $current - absint($years_before);
+		$end_year   = $current + absint($years_after);
+		$selected   = empty($selected) ? date('Y') : $selected;
 		$options    = array();
 
-		while ( $start_year <= $end_year ) {
-			$options[ absint( $start_year ) ] = $start_year;
-			$start_year ++;
+		while ($start_year <= $end_year) {
+			$options[absint($start_year)] = $start_year;
+			$start_year++;
 		}
 
-		$output = $this->select( array(
+		$output = $this->select(array(
 			'name'             => $name,
 			'selected'         => $selected,
 			'options'          => $options,
 			'show_option_all'  => false,
 			'show_option_none' => false,
-		) );
+		));
 
 		return $output;
 	}
@@ -361,23 +361,23 @@ 
 	 *
 	 * @return string           Months dropdown.
 	 */
-	public function month_dropdown( $name = 'month', $selected = 0 ) {
+	public function month_dropdown($name = 'month', $selected = 0) {
 		$month    = 1;
 		$options  = array();
-		$selected = empty( $selected ) ? date( 'n' ) : $selected;
+		$selected = empty($selected) ? date('n') : $selected;
 
-		while ( $month <= 12 ) {
-			$options[ absint( $month ) ] = give_month_num_to_name( $month );
-			$month ++;
+		while ($month <= 12) {
+			$options[absint($month)] = give_month_num_to_name($month);
+			$month++;
 		}
 
-		$output = $this->select( array(
+		$output = $this->select(array(
 			'name'             => $name,
 			'selected'         => $selected,
 			'options'          => $options,
 			'show_option_all'  => false,
 			'show_option_none' => false,
-		) );
+		));
 
 		return $output;
 	}
@@ -394,7 +394,7 @@ 
 	 *
 	 * @return string      The dropdown.
 	 */
-	public function select( $args = array() ) {
+	public function select($args = array()) {
 		$defaults = array(
 			'options'          => array(),
 			'name'             => null,
@@ -405,74 +405,74 @@ 
 			'placeholder'      => null,
 			'multiple'         => false,
 			'select_atts'      => false,
-			'show_option_all'  => __( 'All', 'give' ),
-			'show_option_none' => __( 'None', 'give' ),
+			'show_option_all'  => __('All', 'give'),
+			'show_option_none' => __('None', 'give'),
 			'data'             => array(),
 			'readonly'         => false,
 			'disabled'         => false,
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
 		$data_elements = '';
-		foreach ( $args['data'] as $key => $value ) {
-			$data_elements .= ' data-' . esc_attr( $key ) . '="' . esc_attr( $value ) . '"';
+		foreach ($args['data'] as $key => $value) {
+			$data_elements .= ' data-'.esc_attr($key).'="'.esc_attr($value).'"';
 		}
 
 		$multiple = '';
-		if ( $args['multiple'] ) {
+		if ($args['multiple']) {
 			$multiple = 'MULTIPLE';
 		}
 
-		if ( $args['chosen'] ) {
+		if ($args['chosen']) {
 			$args['class'] .= ' give-select-chosen';
 		}
 
 		$placeholder = '';
-		if ( $args['placeholder'] ) {
+		if ($args['placeholder']) {
 			$placeholder = $args['placeholder'];
 		}
 
 		$output = sprintf(
 			'<select name="%1$s" id="%2$s" class="give-select %3$s" %4$s %5$s placeholder="%6$s" data-placeholder="%6$s" %7$s>',
-			esc_attr( $args['name'] ),
-			esc_attr( sanitize_key( str_replace( '-', '_', $args['id'] ) ) ),
-			esc_attr( $args['class'] ),
+			esc_attr($args['name']),
+			esc_attr(sanitize_key(str_replace('-', '_', $args['id']))),
+			esc_attr($args['class']),
 			$multiple,
 			$args['select_atts'],
 			$placeholder,
 			$data_elements
 		);
 
-		if ( $args['show_option_all'] ) {
-			if ( $args['multiple'] ) {
-				$selected = selected( true, in_array( 0, $args['selected'] ), false );
+		if ($args['show_option_all']) {
+			if ($args['multiple']) {
+				$selected = selected(true, in_array(0, $args['selected']), false);
 			} else {
-				$selected = selected( $args['selected'], 0, false );
+				$selected = selected($args['selected'], 0, false);
 			}
-			$output .= '<option value="all"' . $selected . '>' . esc_html( $args['show_option_all'] ) . '</option>';
+			$output .= '<option value="all"'.$selected.'>'.esc_html($args['show_option_all']).'</option>';
 		}
 
-		if ( ! empty( $args['options'] ) ) {
+		if ( ! empty($args['options'])) {
 
-			if ( $args['show_option_none'] ) {
-				if ( $args['multiple'] ) {
-					$selected = selected( true, in_array( - 1, $args['selected'] ), false );
+			if ($args['show_option_none']) {
+				if ($args['multiple']) {
+					$selected = selected(true, in_array( -1, $args['selected'] ), false);
 				} else {
-					$selected = selected( $args['selected'], - 1, false );
+					$selected = selected($args['selected'], - 1, false);
 				}
-				$output .= '<option value="-1"' . $selected . '>' . esc_html( $args['show_option_none'] ) . '</option>';
+				$output .= '<option value="-1"'.$selected.'>'.esc_html($args['show_option_none']).'</option>';
 			}
 
-			foreach ( $args['options'] as $key => $option ) {
+			foreach ($args['options'] as $key => $option) {
 
-				if ( $args['multiple'] && is_array( $args['selected'] ) ) {
-					$selected = selected( true, in_array( $key, $args['selected'] ), false );
+				if ($args['multiple'] && is_array($args['selected'])) {
+					$selected = selected(true, in_array($key, $args['selected']), false);
 				} else {
-					$selected = selected( $args['selected'], $key, false );
+					$selected = selected($args['selected'], $key, false);
 				}
 
-				$output .= '<option value="' . esc_attr( $key ) . '"' . $selected . '>' . esc_html( $option ) . '</option>';
+				$output .= '<option value="'.esc_attr($key).'"'.$selected.'>'.esc_html($option).'</option>';
 			}
 		}
 
@@ -493,7 +493,7 @@ 
 	 *
 	 * @return string      The checkbox.
 	 */
-	public function checkbox( $args = array() ) {
+	public function checkbox($args = array()) {
 		$defaults = array(
 			'name'    => null,
 			'current' => null,
@@ -504,16 +504,16 @@ 
 			),
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
 		$options = '';
-		if ( ! empty( $args['options']['disabled'] ) ) {
+		if ( ! empty($args['options']['disabled'])) {
 			$options .= ' disabled="disabled"';
-		} elseif ( ! empty( $args['options']['readonly'] ) ) {
+		} elseif ( ! empty($args['options']['readonly'])) {
 			$options .= ' readonly';
 		}
 
-		$output = '<input type="checkbox"' . $options . ' name="' . esc_attr( $args['name'] ) . '" id="' . esc_attr( $args['name'] ) . '" class="' . $args['class'] . ' ' . esc_attr( $args['name'] ) . '" ' . checked( 1, $args['current'], false ) . ' />';
+		$output = '<input type="checkbox"'.$options.' name="'.esc_attr($args['name']).'" id="'.esc_attr($args['name']).'" class="'.$args['class'].' '.esc_attr($args['name']).'" '.checked(1, $args['current'], false).' />';
 
 		return $output;
 	}
@@ -530,22 +530,22 @@ 
 	 *
 	 * @return string      The text field.
 	 */
-	public function text( $args = array() ) {
+	public function text($args = array()) {
 		// Backwards compatibility.
-		if ( func_num_args() > 1 ) {
+		if (func_num_args() > 1) {
 			$args = func_get_args();
 
 			$name  = $args[0];
-			$value = isset( $args[1] ) ? $args[1] : '';
-			$label = isset( $args[2] ) ? $args[2] : '';
-			$desc  = isset( $args[3] ) ? $args[3] : '';
+			$value = isset($args[1]) ? $args[1] : '';
+			$label = isset($args[2]) ? $args[2] : '';
+			$desc  = isset($args[3]) ? $args[3] : '';
 		}
 
 		$defaults = array(
-			'name'         => isset( $name ) ? $name : 'text',
-			'value'        => isset( $value ) ? $value : null,
-			'label'        => isset( $label ) ? $label : null,
-			'desc'         => isset( $desc ) ? $desc : null,
+			'name'         => isset($name) ? $name : 'text',
+			'value'        => isset($value) ? $value : null,
+			'label'        => isset($label) ? $label : null,
+			'desc'         => isset($desc) ? $desc : null,
 			'placeholder'  => '',
 			'class'        => 'regular-text',
 			'disabled'     => false,
@@ -553,29 +553,29 @@ 
 			'data'         => false,
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
 		$disabled = '';
-		if ( $args['disabled'] ) {
+		if ($args['disabled']) {
 			$disabled = ' disabled="disabled"';
 		}
 
 		$data = '';
-		if ( ! empty( $args['data'] ) ) {
-			foreach ( $args['data'] as $key => $value ) {
-				$data .= 'data-' . $key . '="' . $value . '" ';
+		if ( ! empty($args['data'])) {
+			foreach ($args['data'] as $key => $value) {
+				$data .= 'data-'.$key.'="'.$value.'" ';
 			}
 		}
 
-		$output = '<span id="give-' . sanitize_key( $args['name'] ) . '-wrap">';
+		$output = '<span id="give-'.sanitize_key($args['name']).'-wrap">';
 
-		$output .= '<label class="give-label" for="give-' . sanitize_key( $args['name'] ) . '">' . esc_html( $args['label'] ) . '</label>';
+		$output .= '<label class="give-label" for="give-'.sanitize_key($args['name']).'">'.esc_html($args['label']).'</label>';
 
-		if ( ! empty( $args['desc'] ) ) {
-			$output .= '<span class="give-description">' . esc_html( $args['desc'] ) . '</span>';
+		if ( ! empty($args['desc'])) {
+			$output .= '<span class="give-description">'.esc_html($args['desc']).'</span>';
 		}
 
-		$output .= '<input type="text" name="' . esc_attr( $args['name'] ) . '" id="' . esc_attr( $args['name'] ) . '" autocomplete="' . esc_attr( $args['autocomplete'] ) . '" value="' . esc_attr( $args['value'] ) . '" placeholder="' . esc_attr( $args['placeholder'] ) . '" class="' . $args['class'] . '" ' . $data . '' . $disabled . '/>';
+		$output .= '<input type="text" name="'.esc_attr($args['name']).'" id="'.esc_attr($args['name']).'" autocomplete="'.esc_attr($args['autocomplete']).'" value="'.esc_attr($args['value']).'" placeholder="'.esc_attr($args['placeholder']).'" class="'.$args['class'].'" '.$data.''.$disabled.'/>';
 
 		$output .= '</span>';
 
@@ -594,15 +594,15 @@ 
 	 *
 	 * @return string      The date picker.
 	 */
-	public function date_field( $args = array() ) {
+	public function date_field($args = array()) {
 
-		if ( empty( $args['class'] ) ) {
+		if (empty($args['class'])) {
 			$args['class'] = 'give_datepicker';
-		} elseif ( ! strpos( $args['class'], 'give_datepicker' ) ) {
+		} elseif ( ! strpos($args['class'], 'give_datepicker')) {
 			$args['class'] .= ' give_datepicker';
 		}
 
-		return $this->text( $args );
+		return $this->text($args);
 	}
 
 	/**
@@ -617,7 +617,7 @@ 
 	 *
 	 * @return string      The textarea.
 	 */
-	public function textarea( $args = array() ) {
+	public function textarea($args = array()) {
 		$defaults = array(
 			'name'     => 'textarea',
 			'value'    => null,
@@ -627,21 +627,21 @@ 
 			'disabled' => false,
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
 		$disabled = '';
-		if ( $args['disabled'] ) {
+		if ($args['disabled']) {
 			$disabled = ' disabled="disabled"';
 		}
 
-		$output = '<span id="give-' . sanitize_key( $args['name'] ) . '-wrap">';
+		$output = '<span id="give-'.sanitize_key($args['name']).'-wrap">';
 
-		$output .= '<label class="give-label" for="give-' . sanitize_key( $args['name'] ) . '">' . esc_html( $args['label'] ) . '</label>';
+		$output .= '<label class="give-label" for="give-'.sanitize_key($args['name']).'">'.esc_html($args['label']).'</label>';
 
-		$output .= '<textarea name="' . esc_attr( $args['name'] ) . '" id="' . esc_attr( $args['name'] ) . '" class="' . $args['class'] . '"' . $disabled . '>' . esc_attr( $args['value'] ) . '</textarea>';
+		$output .= '<textarea name="'.esc_attr($args['name']).'" id="'.esc_attr($args['name']).'" class="'.$args['class'].'"'.$disabled.'>'.esc_attr($args['value']).'</textarea>';
 
-		if ( ! empty( $args['desc'] ) ) {
-			$output .= '<span class="give-description">' . esc_html( $args['desc'] ) . '</span>';
+		if ( ! empty($args['desc'])) {
+			$output .= '<span class="give-description">'.esc_html($args['desc']).'</span>';
 		}
 
 		$output .= '</span>';
@@ -661,7 +661,7 @@ 
 	 *
 	 * @return string      The text field with ajax search.
 	 */
-	public function ajax_user_search( $args = array() ) {
+	public function ajax_user_search($args = array()) {
 
 		$defaults = array(
 			'name'        => 'users',
@@ -672,13 +672,13 @@ 
 			'chosen'      => true,
 			'number'      => 30,
 			'select_atts' => '',
-			'placeholder' => __( 'Select a user', 'give' ),
+			'placeholder' => __('Select a user', 'give'),
 			'data'        => array(
 				'search-type' => 'user',
 			),
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
 		// Set initial args.
 		$get_users_args = array(
@@ -687,31 +687,31 @@ 
 
 		// Ensure selected user is not included in initial query.
 		// This is because sites with many users, it's not a guarantee the selected user will be returned.
-		if ( ! empty( $args['selected'] ) ) {
+		if ( ! empty($args['selected'])) {
 			$get_users_args['exclude'] = $args['selected'];
 		}
 
 		// Initial users array.
-		$users = apply_filters( 'give_ajax_user_search_initial_results', get_users( $get_users_args ), $args );
+		$users = apply_filters('give_ajax_user_search_initial_results', get_users($get_users_args), $args);
 
 		// Now add the selected user to the $users array if the arg is present.
-		if ( ! empty( $args['selected'] ) ) {
-			$selected_user =  apply_filters( 'give_ajax_user_search_selected_results', get_users( "include={$args['selected']}" ), $args );;
-			$users         = array_merge( $users, $selected_user );
+		if ( ! empty($args['selected'])) {
+			$selected_user = apply_filters('give_ajax_user_search_selected_results', get_users("include={$args['selected']}"), $args); ;
+			$users         = array_merge($users, $selected_user);
 		}
 
 		$options = array();
 
-		if ( $users ) {
+		if ($users) {
 			$options[0] = $args['placeholder'];
-			foreach ( $users as $user ) {
-				$options[ absint( $user->ID ) ] = esc_html( $user->user_login . ' (' . $user->user_email . ')' );
+			foreach ($users as $user) {
+				$options[absint($user->ID)] = esc_html($user->user_login.' ('.$user->user_email.')');
 			}
 		} else {
-			$options[0] = __( 'No users found.', 'give' );
+			$options[0] = __('No users found.', 'give');
 		}
 
-		$output = $this->select( array(
+		$output = $this->select(array(
 			'name'             => $args['name'],
 			'selected'         => $args['selected'],
 			'id'               => $args['id'],
@@ -724,7 +724,7 @@ 
 			'show_option_all'  => false,
 			'show_option_none' => false,
 			'data'             => $args['data'],
-		) );
+		));
 
 		return $output;