WordImpress / Give

includes/admin/donors/class-donor-table.php

Doc Comments +1 added lines, -1 removed lines

@@ -258,7 +258,7 @@
 	 * @access public
 	 * @since  1.0
 	 *
-	 * @return mixed string If search is present, false otherwise.
+	 * @return string|false string If search is present, false otherwise.
 	 */
 	public function get_search() {
 		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;

Spacing +83 added lines, -83 removed lines

@@ -12,13 +12,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded.
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -61,11 +61,11 @@ 
 	public function __construct() {
 
 		// Set parent defaults.
-		parent::__construct( array(
-			'singular' => __( 'Donor', 'give' ), // Singular name of the listed records.
-			'plural'   => __( 'Donors', 'give' ), // Plural name of the listed records.
+		parent::__construct(array(
+			'singular' => __('Donor', 'give'), // Singular name of the listed records.
+			'plural'   => __('Donors', 'give'), // Plural name of the listed records.
 			'ajax'     => false, // Does this table support ajax?.
-		) );
+		));
 
 	}
 
@@ -80,23 +80,23 @@ 
 	 *
 	 * @return void
 	 */
-	public function search_box( $text, $input_id ) {
-		$input_id = $input_id . '-search-input';
+	public function search_box($text, $input_id) {
+		$input_id = $input_id.'-search-input';
 
-		if ( ! empty( $_REQUEST['orderby'] ) ) {
-			echo sprintf( '<input type="hidden" name="orderby" value="%1$s" />', esc_attr( $_REQUEST['orderby'] ) );
+		if ( ! empty($_REQUEST['orderby'])) {
+			echo sprintf('<input type="hidden" name="orderby" value="%1$s" />', esc_attr($_REQUEST['orderby']));
 		}
 
-		if ( ! empty( $_REQUEST['order'] ) ) {
-			echo sprintf( '<input type="hidden" name="order" value="%1$s" />', esc_attr( $_REQUEST['order'] ) );
+		if ( ! empty($_REQUEST['order'])) {
+			echo sprintf('<input type="hidden" name="order" value="%1$s" />', esc_attr($_REQUEST['order']));
 		}
 		?>
 		<p class="search-box" role="search">
 			<label class="screen-reader-text" for="<?php echo $input_id ?>"><?php echo $text; ?>:</label>
 			<input type="search" id="<?php echo $input_id ?>" name="s" value="<?php _admin_search_query(); ?>"/>
-			<?php submit_button( $text, 'button', false, false, array(
+			<?php submit_button($text, 'button', false, false, array(
 				'ID' => 'search-submit',
-			) ); ?>
+			)); ?>
 		</p>
 		<?php
 	}
@@ -112,31 +112,31 @@ 
 	 *
 	 * @return string Column Name.
 	 */
-	public function column_default( $donor, $column_name ) {
-		switch ( $column_name ) {
+	public function column_default($donor, $column_name) {
+		switch ($column_name) {
 
 			case 'num_donations' :
 				$value = sprintf(
 					'<a href="%s">%s</a>',
-					admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&donor=' . absint( $donor['id'] ) ),
-					esc_html( $donor['num_donations'] )
+					admin_url('edit.php?post_type=give_forms&page=give-payment-history&donor='.absint($donor['id'])),
+					esc_html($donor['num_donations'])
 				);
 				break;
 
 			case 'amount_spent' :
-				$value = give_currency_filter( give_format_amount( $donor[ $column_name ], array( 'sanitize' => false ) ) );
+				$value = give_currency_filter(give_format_amount($donor[$column_name], array('sanitize' => false)));
 				break;
 
 			case 'date_created' :
-				$value = date_i18n( give_date_format(), strtotime( $donor['date_created'] ) );
+				$value = date_i18n(give_date_format(), strtotime($donor['date_created']));
 				break;
 
 			default:
-				$value = isset( $donor[ $column_name ] ) ? $donor[ $column_name ] : null;
+				$value = isset($donor[$column_name]) ? $donor[$column_name] : null;
 				break;
 		}
 
-		return apply_filters( "give_donors_column_{$column_name}", $value, $donor['id'] );
+		return apply_filters("give_donors_column_{$column_name}", $value, $donor['id']);
 
 	}
 
@@ -150,7 +150,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function column_cb( $donor ){
+	public function column_cb($donor) {
 		return sprintf(
 			'<input type="checkbox" name="%1$s[]" value="%2$s" />',
 			$this->_args['singular'],
@@ -168,13 +168,13 @@ 
 	 *
 	 * @return string
 	 */
-	public function column_name( $donor ) {
-		$name     = '#' . $donor['id'] . ' ';
-		$name     .= ! empty( $donor['name'] ) ? $donor['name'] : '<em>' . __( 'Unnamed Donor', 'give' ) . '</em>';
-		$view_url = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor['id'] );
-		$actions  = $this->get_row_actions( $donor );
+	public function column_name($donor) {
+		$name     = '#'.$donor['id'].' ';
+		$name .= ! empty($donor['name']) ? $donor['name'] : '<em>'.__('Unnamed Donor', 'give').'</em>';
+		$view_url = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor['id']);
+		$actions  = $this->get_row_actions($donor);
 
-		return '<a href="' . esc_url( $view_url ) . '">' . $name . '</a>' . $this->row_actions( $actions );
+		return '<a href="'.esc_url($view_url).'">'.$name.'</a>'.$this->row_actions($actions);
 	}
 
 	/**
@@ -188,14 +188,14 @@ 
 	public function get_columns() {
 		$columns = array(
 			'cb'            => '<input type="checkbox" />', // Render a checkbox instead of text.
-			'name'          => __( 'Name', 'give' ),
-			'email'         => __( 'Email', 'give' ),
-			'num_donations' => __( 'Donations', 'give' ),
-			'amount_spent'  => __( 'Total Donated', 'give' ),
-			'date_created'  => __( 'Date Created', 'give' ),
+			'name'          => __('Name', 'give'),
+			'email'         => __('Email', 'give'),
+			'num_donations' => __('Donations', 'give'),
+			'amount_spent'  => __('Total Donated', 'give'),
+			'date_created'  => __('Date Created', 'give'),
 		);
 
-		return apply_filters( 'give_list_donors_columns', $columns );
+		return apply_filters('give_list_donors_columns', $columns);
 
 	}
 
@@ -209,13 +209,13 @@ 
 	public function get_sortable_columns() {
 
 		$columns = array(
-			'date_created'  => array( 'date_created', true ),
-			'name'          => array( 'name', true ),
-			'num_donations' => array( 'purchase_count', false ),
-			'amount_spent'  => array( 'purchase_value', false ),
+			'date_created'  => array('date_created', true),
+			'name'          => array('name', true),
+			'num_donations' => array('purchase_count', false),
+			'amount_spent'  => array('purchase_value', false),
 		);
 
-		return apply_filters( 'give_list_donors_sortable_columns', $columns );
+		return apply_filters('give_list_donors_sortable_columns', $columns);
 	}
 
 	/**
@@ -228,15 +228,15 @@ 
 	 *
 	 * @return array An array of action links.
 	 */
-	public function get_row_actions( $donor ) {
+	public function get_row_actions($donor) {
 
 		$actions = array(
-			'view'   => sprintf( '<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor['id'] ), sprintf( esc_attr__( 'View "%s"', 'give' ), $donor['name'] ), __( 'View Donor', 'give' ) ),
-			'notes'  => sprintf( '<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=notes&id=' . $donor['id'] ), sprintf( esc_attr__( 'Notes for "%s"', 'give' ), $donor['name'] ), __( 'Notes', 'give' ) ),
-			'delete' => sprintf( '<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=delete&id=' . $donor['id'] ), sprintf( esc_attr__( 'Delete "%s"', 'give' ), $donor['name'] ), __( 'Delete', 'give' ) ),
+			'view'   => sprintf('<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor['id']), sprintf(esc_attr__('View "%s"', 'give'), $donor['name']), __('View Donor', 'give')),
+			'notes'  => sprintf('<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url('edit.php?post_type=give_forms&page=give-donors&view=notes&id='.$donor['id']), sprintf(esc_attr__('Notes for "%s"', 'give'), $donor['name']), __('Notes', 'give')),
+			'delete' => sprintf('<a href="%1$s" aria-label="%2$s">%3$s</a>', admin_url('edit.php?post_type=give_forms&page=give-donors&view=delete&id='.$donor['id']), sprintf(esc_attr__('Delete "%s"', 'give'), $donor['name']), __('Delete', 'give')),
 		);
 
-		return apply_filters( 'give_donor_row_actions', $actions, $donor );
+		return apply_filters('give_donor_row_actions', $actions, $donor);
 
 	}
 
@@ -249,7 +249,7 @@ 
 	 * @return int Current page number.
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 	/**
@@ -261,7 +261,7 @@ 
 	 * @return mixed string If search is present, false otherwise.
 	 */
 	public function get_search() {
-		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;
+		return ! empty($_GET['s']) ? urldecode(trim($_GET['s'])) : false;
 	}
 
 	/**
@@ -288,32 +288,32 @@ 
 	 * @return void
 	 */
 	public function process_bulk_action() {
-		$ids    = isset( $_GET['donor'] ) ? $_GET['donor'] : false;
+		$ids    = isset($_GET['donor']) ? $_GET['donor'] : false;
 		$action = $this->current_action();
 
-		if ( ! is_array( $ids ) ) {
-			$ids = array( $ids );
+		if ( ! is_array($ids)) {
+			$ids = array($ids);
 		}
 
 		// Bail Out, If Action is not set.
-		if ( empty( $action ) ) {
+		if (empty($action)) {
 			return;
 		}
 
-		foreach ( $ids as $id ) {
+		foreach ($ids as $id) {
 
 			// Detect when a bulk action is being triggered.
-			switch ( $this->current_action() ) {
+			switch ($this->current_action()) {
 
 				case 'delete':
 					$args = array(
-						'_wpnonce'                  => wp_create_nonce( 'delete-donor' ),
+						'_wpnonce'                  => wp_create_nonce('delete-donor'),
 						'customer_id'               => $id,
 						'give-donor-delete-confirm' => true,
 						'give-donor-delete-records' => true,
 						'redirect'                  => false,
 					);
-					give_donor_delete( $args );
+					give_donor_delete($args);
 					break;
 
 			} // End switch().
@@ -326,7 +326,7 @@ 
 			 *
 			 * @since 1.8.16
 			 */
-			do_action( 'give_donors_table_do_bulk_action', $id, $this->current_action() );
+			do_action('give_donors_table_do_bulk_action', $id, $this->current_action());
 		} // End foreach().
 
 	}
@@ -339,20 +339,20 @@ 
 	 * @access protected
 	 * @since  1.8.16
 	 */
-	protected function display_tablenav( $which ) {
-		if ( 'top' === $which ) {
-			$this->search_box( __( 'Search Donors', 'give' ), 'give-donors' );
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+	protected function display_tablenav($which) {
+		if ('top' === $which) {
+			$this->search_box(__('Search Donors', 'give'), 'give-donors');
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav <?php echo esc_attr( $which ); ?>">
-			<?php if ( $this->has_items() ): ?>
+		<div class="tablenav <?php echo esc_attr($which); ?>">
+			<?php if ($this->has_items()): ?>
 				<div class="alignleft actions bulkactions">
-					<?php $this->bulk_actions( $which ); ?>
+					<?php $this->bulk_actions($which); ?>
 				</div>
 			<?php endif;
-			$this->extra_tablenav( $which );
-			$this->pagination( $which );
+			$this->extra_tablenav($which);
+			$this->pagination($which);
 			?>
 			<br class="clear" />
 		</div>
@@ -373,13 +373,13 @@ 
 
 		// Get donor query.
 		$args   = $this->get_donor_query();
-		$donors = Give()->donors->get_donors( $args );
+		$donors = Give()->donors->get_donors($args);
 
-		if ( $donors ) {
+		if ($donors) {
 
-			foreach ( $donors as $donor ) {
+			foreach ($donors as $donor) {
 
-				$user_id = ! empty( $donor->user_id ) ? intval( $donor->user_id ) : 0;
+				$user_id = ! empty($donor->user_id) ? intval($donor->user_id) : 0;
 
 				$data[] = array(
 					'id'            => $donor->id,
@@ -393,7 +393,7 @@ 
 			}
 		}
 
-		return apply_filters( 'give_donors_column_query_data', $data );
+		return apply_filters('give_donors_column_query_data', $data);
 	}
 
 	/**
@@ -408,9 +408,9 @@ 
 
 		$_donor_query['number'] = - 1;
 		$_donor_query['offset'] = 0;
-		$donors                 = Give()->donors->get_donors( $_donor_query );
+		$donors                 = Give()->donors->get_donors($_donor_query);
 
-		return count( $donors );
+		return count($donors);
 	}
 
 	/**
@@ -423,10 +423,10 @@ 
 	 */
 	public function get_donor_query() {
 		$paged   = $this->get_paged();
-		$offset  = $this->per_page * ( $paged - 1 );
+		$offset  = $this->per_page * ($paged - 1);
 		$search  = $this->get_search();
-		$order   = isset( $_GET['order'] ) ? sanitize_text_field( $_GET['order'] ) : 'DESC';
-		$orderby = isset( $_GET['orderby'] ) ? sanitize_text_field( $_GET['orderby'] ) : 'id';
+		$order   = isset($_GET['order']) ? sanitize_text_field($_GET['order']) : 'DESC';
+		$orderby = isset($_GET['orderby']) ? sanitize_text_field($_GET['orderby']) : 'id';
 
 		$args = array(
 			'number'  => $this->per_page,
@@ -435,10 +435,10 @@ 
 			'orderby' => $orderby,
 		);
 
-		if ( $search ) {
-			if ( is_email( $search ) ) {
+		if ($search) {
+			if (is_email($search)) {
 				$args['email'] = $search;
-			} elseif ( is_numeric( $search ) ) {
+			} elseif (is_numeric($search)) {
 				$args['id'] = $search;
 			} else {
 				$args['name'] = $search;
@@ -462,7 +462,7 @@ 
 		$hidden   = array(); // No hidden columns.
 		$sortable = $this->get_sortable_columns();
 
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 
 		$this->process_bulk_action();
 
@@ -470,10 +470,10 @@ 
 
 		$this->total = $this->get_donor_count();
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 			'total_items' => $this->total,
 			'per_page'    => $this->per_page,
-			'total_pages' => ceil( $this->total / $this->per_page ),
-		) );
+			'total_pages' => ceil($this->total / $this->per_page),
+		));
 	}
 }

includes/admin/donors/donor-actions.php

Doc Comments +4 added lines, -4 removed lines

@@ -21,7 +21,7 @@ 
  *
  * @since 1.0
  *
- * @return array|bool $output Response messages
+ * @return false|null $output Response messages
  */
 function give_edit_donor( $args ) {
 
@@ -227,7 +227,7 @@ 
  *
  * @since 1.0
  *
- * @return int Whether it was a successful deletion.
+ * @return false|null Whether it was a successful deletion.
  */
 function give_donor_delete( $args ) {
 
@@ -521,7 +521,7 @@ 
  *
  * @since  1.7
  *
- * @return bool|null
+ * @return false|null
  */
 function give_remove_donor_email() {
 	if ( empty( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
@@ -565,7 +565,7 @@ 
  *
  * @since  1.7
  *
- * @return bool|null
+ * @return false|null
  */
 function give_set_donor_primary_email() {
 	if ( empty( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {

Spacing +194 added lines, -194 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,17 +23,17 @@ 
  *
  * @return array|bool $output Response messages
  */
-function give_edit_donor( $args ) {
+function give_edit_donor($args) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
-	if ( ! is_admin() || ! current_user_can( $donor_edit_role ) ) {
-		wp_die( __( 'You do not have permission to edit this donor.', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! is_admin() || ! current_user_can($donor_edit_role)) {
+		wp_die(__('You do not have permission to edit this donor.', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	if ( empty( $args ) ) {
+	if (empty($args)) {
 		return false;
 	}
 
@@ -41,15 +41,15 @@ 
 	$donor_id   = (int) $args['customerinfo']['id'];
 	$nonce      = $args['_wpnonce'];
 
-	if ( ! wp_verify_nonce( $nonce, 'edit-donor' ) ) {
-		wp_die( __( 'Cheatin’ uh?', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'edit-donor')) {
+		wp_die(__('Cheatin’ uh?', 'give'), __('Error', 'give'), array(
 			'response' => 400,
-		) );
+		));
 	}
 
-	$donor = new Give_Donor( $donor_id );
+	$donor = new Give_Donor($donor_id);
 
-	if ( empty( $donor->id ) ) {
+	if (empty($donor->id)) {
 		return false;
 	}
 
@@ -64,79 +64,79 @@ 
 		'country' => '',
 	);
 
-	$donor_info = wp_parse_args( $donor_info, $defaults );
+	$donor_info = wp_parse_args($donor_info, $defaults);
 
-	if ( (int) $donor_info['user_id'] !== (int) $donor->user_id ) {
+	if ((int) $donor_info['user_id'] !== (int) $donor->user_id) {
 
 		// Make sure we don't already have this user attached to a donor.
-		if ( ! empty( $donor_info['user_id'] ) && false !== Give()->donors->get_donor_by( 'user_id', $donor_info['user_id'] ) ) {
-			give_set_error( 'give-invalid-donor-user_id', sprintf( __( 'The User ID #%d is already associated with a different donor.', 'give' ), $donor_info['user_id'] ) );
+		if ( ! empty($donor_info['user_id']) && false !== Give()->donors->get_donor_by('user_id', $donor_info['user_id'])) {
+			give_set_error('give-invalid-donor-user_id', sprintf(__('The User ID #%d is already associated with a different donor.', 'give'), $donor_info['user_id']));
 		}
 
 		// Make sure it's actually a user.
-		$user = get_user_by( 'id', $donor_info['user_id'] );
-		if ( ! empty( $donor_info['user_id'] ) && false === $user ) {
-			give_set_error( 'give-invalid-user_id', sprintf( __( 'The User ID #%d does not exist. Please assign an existing user.', 'give' ), $donor_info['user_id'] ) );
+		$user = get_user_by('id', $donor_info['user_id']);
+		if ( ! empty($donor_info['user_id']) && false === $user) {
+			give_set_error('give-invalid-user_id', sprintf(__('The User ID #%d does not exist. Please assign an existing user.', 'give'), $donor_info['user_id']));
 		}
 	}
 
-	if ( give_get_errors() ) {
+	if (give_get_errors()) {
 		return false;
 	}
 
 	// Setup the donor address, if present.
 	$address = array();
-	if ( intval( $donor_info['user_id'] ) > 0 ) {
-
-		$current_address = (array) get_user_meta( $donor_info['user_id'], '_give_user_address', true );
-
-		if ( is_array( $current_address ) && 0 < count( $current_address ) ) {
-			$current_address    = wp_parse_args( $current_address, $defaults );
-			$address['line1']   = ! empty( $donor_info['line1'] ) ? $donor_info['line1'] : $current_address['line1'];
-			$address['line2']   = ! empty( $donor_info['line2'] ) ? $donor_info['line2'] : $current_address['line2'];
-			$address['city']    = ! empty( $donor_info['city'] ) ? $donor_info['city'] : $current_address['city'];
-			$address['country'] = ! empty( $donor_info['country'] ) ? $donor_info['country'] : $current_address['country'];
-			$address['zip']     = ! empty( $donor_info['zip'] ) ? $donor_info['zip'] : $current_address['zip'];
-			$address['state']   = ! empty( $donor_info['state'] ) ? $donor_info['state'] : $current_address['state'];
+	if (intval($donor_info['user_id']) > 0) {
+
+		$current_address = (array) get_user_meta($donor_info['user_id'], '_give_user_address', true);
+
+		if (is_array($current_address) && 0 < count($current_address)) {
+			$current_address    = wp_parse_args($current_address, $defaults);
+			$address['line1']   = ! empty($donor_info['line1']) ? $donor_info['line1'] : $current_address['line1'];
+			$address['line2']   = ! empty($donor_info['line2']) ? $donor_info['line2'] : $current_address['line2'];
+			$address['city']    = ! empty($donor_info['city']) ? $donor_info['city'] : $current_address['city'];
+			$address['country'] = ! empty($donor_info['country']) ? $donor_info['country'] : $current_address['country'];
+			$address['zip']     = ! empty($donor_info['zip']) ? $donor_info['zip'] : $current_address['zip'];
+			$address['state']   = ! empty($donor_info['state']) ? $donor_info['state'] : $current_address['state'];
 		} else {
-			$address['line1']   = ! empty( $donor_info['line1'] ) ? $donor_info['line1'] : '';
-			$address['line2']   = ! empty( $donor_info['line2'] ) ? $donor_info['line2'] : '';
-			$address['city']    = ! empty( $donor_info['city'] ) ? $donor_info['city'] : '';
-			$address['country'] = ! empty( $donor_info['country'] ) ? $donor_info['country'] : '';
-			$address['zip']     = ! empty( $donor_info['zip'] ) ? $donor_info['zip'] : '';
-			$address['state']   = ! empty( $donor_info['state'] ) ? $donor_info['state'] : '';
+			$address['line1']   = ! empty($donor_info['line1']) ? $donor_info['line1'] : '';
+			$address['line2']   = ! empty($donor_info['line2']) ? $donor_info['line2'] : '';
+			$address['city']    = ! empty($donor_info['city']) ? $donor_info['city'] : '';
+			$address['country'] = ! empty($donor_info['country']) ? $donor_info['country'] : '';
+			$address['zip']     = ! empty($donor_info['zip']) ? $donor_info['zip'] : '';
+			$address['state']   = ! empty($donor_info['state']) ? $donor_info['state'] : '';
 		}
 
 	}
 
 	// Sanitize the inputs.
 	$donor_data            = array();
-	$donor_data['name']    = strip_tags( stripslashes( $donor_info['name'] ) );
+	$donor_data['name']    = strip_tags(stripslashes($donor_info['name']));
 	$donor_data['user_id'] = $donor_info['user_id'];
 
-	$donor_data = apply_filters( 'give_edit_donor_info', $donor_data, $donor_id );
-	$address    = apply_filters( 'give_edit_donor_address', $address, $donor_id );
+	$donor_data = apply_filters('give_edit_donor_info', $donor_data, $donor_id);
+	$address    = apply_filters('give_edit_donor_address', $address, $donor_id);
 
-	$donor_data = array_map( 'sanitize_text_field', $donor_data );
-	$address    = array_map( 'sanitize_text_field', $address );
+	$donor_data = array_map('sanitize_text_field', $donor_data);
+	$address    = array_map('sanitize_text_field', $address);
 
-	$output = give_connect_user_donor_profile( $donor, $donor_data, $address );
+	$output = give_connect_user_donor_profile($donor, $donor_data, $address);
 
-	if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
-		header( 'Content-Type: application/json' );
-		echo json_encode( $output );
+	if (defined('DOING_AJAX') && DOING_AJAX) {
+		header('Content-Type: application/json');
+		echo json_encode($output);
 		wp_die();
 	}
 
-	if ( $output['success'] ) {
-		wp_redirect( admin_url( "edit.php?post_type=give_forms&page=give-donors&view=overview&id={$donor_id}&give-message=profile-updated" ) );
+	if ($output['success']) {
+		wp_redirect(admin_url("edit.php?post_type=give_forms&page=give-donors&view=overview&id={$donor_id}&give-message=profile-updated"));
 	}
 
 	exit;
 
 }
 
-add_action( 'give_edit-donor', 'give_edit_donor', 10, 1 );
+add_action('give_edit-donor', 'give_edit_donor', 10, 1);
 
 /**
  * Save a donor note.
@@ -147,40 +147,40 @@ 
  *
  * @return int The Note ID that was saved, or 0 if nothing was saved.
  */
-function give_donor_save_note( $args ) {
+function give_donor_save_note($args) {
 
-	$donor_view_role = apply_filters( 'give_view_donors_role', 'view_give_reports' );
+	$donor_view_role = apply_filters('give_view_donors_role', 'view_give_reports');
 
-	if ( ! is_admin() || ! current_user_can( $donor_view_role ) ) {
-		wp_die( __( 'You do not have permission to edit this donor.', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! is_admin() || ! current_user_can($donor_view_role)) {
+		wp_die(__('You do not have permission to edit this donor.', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	if ( empty( $args ) ) {
+	if (empty($args)) {
 		return false;
 	}
 
-	$donor_note = trim( sanitize_text_field( $args['donor_note'] ) );
+	$donor_note = trim(sanitize_text_field($args['donor_note']));
 	$donor_id   = (int) $args['customer_id'];
 	$nonce      = $args['add_donor_note_nonce'];
 
-	if ( ! wp_verify_nonce( $nonce, 'add-donor-note' ) ) {
-		wp_die( __( 'Cheatin&#8217; uh?', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'add-donor-note')) {
+		wp_die(__('Cheatin&#8217; uh?', 'give'), __('Error', 'give'), array(
 			'response' => 400,
-		) );
+		));
 	}
 
-	if ( empty( $donor_note ) ) {
-		give_set_error( 'empty-donor-note', __( 'A note is required.', 'give' ) );
+	if (empty($donor_note)) {
+		give_set_error('empty-donor-note', __('A note is required.', 'give'));
 	}
 
-	if ( give_get_errors() ) {
+	if (give_get_errors()) {
 		return false;
 	}
 
-	$donor    = new Give_Donor( $donor_id );
-	$new_note = $donor->add_note( $donor_note );
+	$donor    = new Give_Donor($donor_id);
+	$new_note = $donor->add_note($donor_note);
 
 	/**
 	 * Fires before inserting donor note.
@@ -190,22 +190,22 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_pre_insert_donor_note', $donor_id, $new_note );
+	do_action('give_pre_insert_donor_note', $donor_id, $new_note);
 
-	if ( ! empty( $new_note ) && ! empty( $donor->id ) ) {
+	if ( ! empty($new_note) && ! empty($donor->id)) {
 
 		ob_start();
 		?>
 		<div class="donor-note-wrapper dashboard-comment-wrap comment-item">
 			<span class="note-content-wrap">
-				<?php echo stripslashes( $new_note ); ?>
+				<?php echo stripslashes($new_note); ?>
 			</span>
 		</div>
 		<?php
 		$output = ob_get_contents();
 		ob_end_clean();
 
-		if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
+		if (defined('DOING_AJAX') && DOING_AJAX) {
 			echo $output;
 			exit;
 		}
@@ -218,7 +218,7 @@ 
 
 }
 
-add_action( 'give_add-donor-note', 'give_donor_save_note', 10, 1 );
+add_action('give_add-donor-note', 'give_donor_save_note', 10, 1);
 
 /**
  * Delete a donor.
@@ -229,46 +229,46 @@ 
  *
  * @return int Whether it was a successful deletion.
  */
-function give_donor_delete( $args ) {
+function give_donor_delete($args) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
-	if ( ! is_admin() || ! current_user_can( $donor_edit_role ) ) {
-		wp_die( __( 'You do not have permission to delete donors.', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! is_admin() || ! current_user_can($donor_edit_role)) {
+		wp_die(__('You do not have permission to delete donors.', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	if ( empty( $args ) ) {
+	if (empty($args)) {
 		return false;
 	}
 
 	$defaults = array(
 		'redirect' => true,
 	);
-	wp_parse_args( $args, $defaults );
+	wp_parse_args($args, $defaults);
 
 	$donor_id    = (int) $args['customer_id'];
-	$confirm     = ! empty( $args['give-donor-delete-confirm'] ) ? true : false;
-	$remove_data = ! empty( $args['give-donor-delete-records'] ) ? true : false;
+	$confirm     = ! empty($args['give-donor-delete-confirm']) ? true : false;
+	$remove_data = ! empty($args['give-donor-delete-records']) ? true : false;
 	$nonce       = $args['_wpnonce'];
 
-	if ( ! wp_verify_nonce( $nonce, 'delete-donor' ) ) {
-		wp_die( __( 'Cheatin&#8217; uh?', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'delete-donor')) {
+		wp_die(__('Cheatin&#8217; uh?', 'give'), __('Error', 'give'), array(
 			'response' => 400,
-		) );
+		));
 	}
 
-	if ( ! $confirm ) {
-		give_set_error( 'donor-delete-no-confirm', __( 'Please confirm you want to delete this donor.', 'give' ) );
+	if ( ! $confirm) {
+		give_set_error('donor-delete-no-confirm', __('Please confirm you want to delete this donor.', 'give'));
 	}
 
-	if ( give_get_errors() ) {
-		wp_redirect( admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor_id ) );
+	if (give_get_errors()) {
+		wp_redirect(admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor_id));
 		exit;
 	}
 
-	$donor = new Give_Donor( $donor_id );
+	$donor = new Give_Donor($donor_id);
 
 	/**
 	 * Fires before deleting donor.
@@ -279,53 +279,53 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_pre_delete_donor', $donor_id, $confirm, $remove_data );
+	do_action('give_pre_delete_donor', $donor_id, $confirm, $remove_data);
 
-	if ( $donor->id > 0 ) {
+	if ($donor->id > 0) {
 
-		$payments_array = explode( ',', $donor->payment_ids );
-		$success        = Give()->donors->delete( $donor->id );
+		$payments_array = explode(',', $donor->payment_ids);
+		$success        = Give()->donors->delete($donor->id);
 
-		if ( $success ) {
+		if ($success) {
 
-			if ( $remove_data ) {
+			if ($remove_data) {
 
 				// Remove all donations, logs, etc.
-				foreach ( $payments_array as $payment_id ) {
-					give_delete_donation( $payment_id );
+				foreach ($payments_array as $payment_id) {
+					give_delete_donation($payment_id);
 				}
 			} else {
 
 				// Just set the donations to customer_id of 0.
-				foreach ( $payments_array as $payment_id ) {
-					give_update_payment_meta( $payment_id, '_give_payment_customer_id', 0 );
+				foreach ($payments_array as $payment_id) {
+					give_update_payment_meta($payment_id, '_give_payment_customer_id', 0);
 				}
 			}
 
-			$redirect = admin_url( 'edit.php?post_type=give_forms&page=give-donors&give-message=donor-deleted' );
+			$redirect = admin_url('edit.php?post_type=give_forms&page=give-donors&give-message=donor-deleted');
 
 		} else {
 
-			give_set_error( 'give-donor-delete-failed', __( 'Error deleting donor.', 'give' ) );
-			$redirect = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=delete&id=' . $donor_id );
+			give_set_error('give-donor-delete-failed', __('Error deleting donor.', 'give'));
+			$redirect = admin_url('edit.php?post_type=give_forms&page=give-donors&view=delete&id='.$donor_id);
 
 		}
 	} else {
 
-		give_set_error( 'give-donor-delete-invalid-id', __( 'Invalid Donor ID.', 'give' ) );
-		$redirect = admin_url( 'edit.php?post_type=give_forms&page=give-donors' );
+		give_set_error('give-donor-delete-invalid-id', __('Invalid Donor ID.', 'give'));
+		$redirect = admin_url('edit.php?post_type=give_forms&page=give-donors');
 
 	}
 
 	// Proceed Redirect, only if redirect argument is set to true.
-	if( $args['redirect'] ) {
-		wp_redirect( $redirect );
+	if ($args['redirect']) {
+		wp_redirect($redirect);
 		exit;
 	}
 
 }
 
-add_action( 'give_delete-donor', 'give_donor_delete', 10, 1 );
+add_action('give_delete-donor', 'give_donor_delete', 10, 1);
 
 /**
  * Disconnect a user ID from a donor
@@ -336,17 +336,17 @@ 
  *
  * @return bool|array If the disconnect was successful.
  */
-function give_disconnect_donor_user_id( $args ) {
+function give_disconnect_donor_user_id($args) {
 
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
-	if ( ! is_admin() || ! current_user_can( $donor_edit_role ) ) {
-		wp_die( __( 'You do not have permission to edit this donor.', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! is_admin() || ! current_user_can($donor_edit_role)) {
+		wp_die(__('You do not have permission to edit this donor.', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	if ( empty( $args ) ) {
+	if (empty($args)) {
 		return false;
 	}
 
@@ -354,14 +354,14 @@ 
 
 	$nonce = $args['_wpnonce'];
 
-	if ( ! wp_verify_nonce( $nonce, 'edit-donor' ) ) {
-		wp_die( __( 'Cheatin&#8217; uh?', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'edit-donor')) {
+		wp_die(__('Cheatin&#8217; uh?', 'give'), __('Error', 'give'), array(
 			'response' => 400,
-		) );
+		));
 	}
 
-	$donor = new Give_Donor( $donor_id );
-	if ( empty( $donor->id ) ) {
+	$donor = new Give_Donor($donor_id);
+	if (empty($donor->id)) {
 		return false;
 	}
 
@@ -375,34 +375,34 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_pre_donor_disconnect_user_id', $donor_id, $user_id );
+	do_action('give_pre_donor_disconnect_user_id', $donor_id, $user_id);
 
 	$output     = array();
 	$donor_args = array(
 		'user_id' => 0,
 	);
 
-	if ( $donor->update( $donor_args ) ) {
+	if ($donor->update($donor_args)) {
 		global $wpdb;
 
-		if ( ! empty( $donor->payment_ids ) ) {
-			$wpdb->query( "UPDATE $wpdb->postmeta SET meta_value = 0 WHERE meta_key = '_give_payment_user_id' AND post_id IN ( $donor->payment_ids )" );
+		if ( ! empty($donor->payment_ids)) {
+			$wpdb->query("UPDATE $wpdb->postmeta SET meta_value = 0 WHERE meta_key = '_give_payment_user_id' AND post_id IN ( $donor->payment_ids )");
 		}
 
 		// Set Donor Disconnection status true, if user and donor are disconnected with each other.
-		update_user_meta( $user_id, '_give_is_donor_disconnected', true );
-		update_user_meta( $user_id, '_give_disconnected_donor_id', $donor->id );
-		$donor->update_meta( '_give_disconnected_user_id', $user_id );
+		update_user_meta($user_id, '_give_is_donor_disconnected', true);
+		update_user_meta($user_id, '_give_disconnected_donor_id', $donor->id);
+		$donor->update_meta('_give_disconnected_user_id', $user_id);
 
 		$output['success'] = true;
 
 	} else {
 
 		$output['success'] = false;
-		give_set_error( 'give-disconnect-user-fail', __( 'Failed to disconnect user from donor.', 'give' ) );
+		give_set_error('give-disconnect-user-fail', __('Failed to disconnect user from donor.', 'give'));
 	}
 
-	$output['redirect'] = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' ) . $donor_id;
+	$output['redirect'] = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id=').$donor_id;
 
 	/**
 	 * Fires after disconnecting user ID from a donor.
@@ -411,11 +411,11 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_post_donor_disconnect_user_id', $donor_id );
+	do_action('give_post_donor_disconnect_user_id', $donor_id);
 
-	if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
-		header( 'Content-Type: application/json' );
-		echo json_encode( $output );
+	if (defined('DOING_AJAX') && DOING_AJAX) {
+		header('Content-Type: application/json');
+		echo json_encode($output);
 		wp_die();
 	}
 
@@ -423,7 +423,7 @@ 
 
 }
 
-add_action( 'give_disconnect-userid', 'give_disconnect_donor_user_id', 10, 1 );
+add_action('give_disconnect-userid', 'give_disconnect_donor_user_id', 10, 1);
 
 /**
  * Add an email address to the donor from within the admin and log a donor note.
@@ -434,86 +434,86 @@ 
  *
  * @return mixed If DOING_AJAX echos out JSON, otherwise returns array of success (bool) and message (string).
  */
-function give_add_donor_email( $args ) {
+function give_add_donor_email($args) {
 
 	$donor_id = '';
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$donor_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
-	if ( ! is_admin() || ! current_user_can( $donor_edit_role ) ) {
-		wp_die( __( 'You do not have permission to edit this donor.', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! is_admin() || ! current_user_can($donor_edit_role)) {
+		wp_die(__('You do not have permission to edit this donor.', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
 	$output = array();
-	if ( empty( $args ) || empty( $args['email'] ) || empty( $args['customer_id'] ) ) {
+	if (empty($args) || empty($args['email']) || empty($args['customer_id'])) {
 		$output['success'] = false;
-		if ( empty( $args['email'] ) ) {
-			$output['message'] = __( 'Email address is required.', 'give' );
-		} elseif ( empty( $args['customer_id'] ) ) {
-			$output['message'] = __( 'Donor ID is required.', 'give' );
+		if (empty($args['email'])) {
+			$output['message'] = __('Email address is required.', 'give');
+		} elseif (empty($args['customer_id'])) {
+			$output['message'] = __('Donor ID is required.', 'give');
 		} else {
-			$output['message'] = __( 'An error has occurred. Please try again.', 'give' );
+			$output['message'] = __('An error has occurred. Please try again.', 'give');
 		}
-	} elseif ( ! wp_verify_nonce( $args['_wpnonce'], 'give_add_donor_email' ) ) {
+	} elseif ( ! wp_verify_nonce($args['_wpnonce'], 'give_add_donor_email')) {
 		$output = array(
 			'success' => false,
-			'message' => __( 'Nonce verification failed.', 'give' ),
+			'message' => __('Nonce verification failed.', 'give'),
 		);
-	} elseif ( ! is_email( $args['email'] ) ) {
+	} elseif ( ! is_email($args['email'])) {
 		$output = array(
 			'success' => false,
-			'message' => __( 'Invalid email.', 'give' ),
+			'message' => __('Invalid email.', 'give'),
 		);
 	} else {
-		$email    = sanitize_email( $args['email'] );
+		$email    = sanitize_email($args['email']);
 		$donor_id = (int) $args['customer_id'];
 		$primary  = 'true' === $args['primary'] ? true : false;
-		$donor    = new Give_Donor( $donor_id );
-		if ( false === $donor->add_email( $email, $primary ) ) {
-			if ( in_array( $email, $donor->emails ) ) {
+		$donor    = new Give_Donor($donor_id);
+		if (false === $donor->add_email($email, $primary)) {
+			if (in_array($email, $donor->emails)) {
 				$output = array(
 					'success' => false,
-					'message' => __( 'Email already associated with this donor.', 'give' ),
+					'message' => __('Email already associated with this donor.', 'give'),
 				);
 			} else {
 				$output = array(
 					'success' => false,
-					'message' => __( 'Email address is already associated with another donor.', 'give' ),
+					'message' => __('Email address is already associated with another donor.', 'give'),
 				);
 			}
 		} else {
-			$redirect = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor_id . '&give-message=email-added' );
+			$redirect = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor_id.'&give-message=email-added');
 			$output   = array(
 				'success'  => true,
-				'message'  => __( 'Email successfully added to donor.', 'give' ),
+				'message'  => __('Email successfully added to donor.', 'give'),
 				'redirect' => $redirect,
 			);
 
 			$user       = wp_get_current_user();
-			$user_login = ! empty( $user->user_login ) ? $user->user_login : __( 'System', 'give' );
-			$donor_note = sprintf( __( 'Email address %1$s added by %2$s', 'give' ), $email, $user_login );
-			$donor->add_note( $donor_note );
+			$user_login = ! empty($user->user_login) ? $user->user_login : __('System', 'give');
+			$donor_note = sprintf(__('Email address %1$s added by %2$s', 'give'), $email, $user_login);
+			$donor->add_note($donor_note);
 
-			if ( $primary ) {
-				$donor_note = sprintf( __( 'Email address %1$s set as primary by %2$s', 'give' ), $email, $user_login );
-				$donor->add_note( $donor_note );
+			if ($primary) {
+				$donor_note = sprintf(__('Email address %1$s set as primary by %2$s', 'give'), $email, $user_login);
+				$donor->add_note($donor_note);
 			}
 		}
 	} // End if().
 
-	do_action( 'give_post_add_donor_email', $donor_id, $args );
+	do_action('give_post_add_donor_email', $donor_id, $args);
 
-	if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
-		header( 'Content-Type: application/json' );
-		echo json_encode( $output );
+	if (defined('DOING_AJAX') && DOING_AJAX) {
+		header('Content-Type: application/json');
+		echo json_encode($output);
 		wp_die();
 	}
 
 	return $output;
 }
 
-add_action( 'give_add_donor_email', 'give_add_donor_email', 10, 1 );
+add_action('give_add_donor_email', 'give_add_donor_email', 10, 1);
 
 
 /**
@@ -524,39 +524,39 @@ 
  * @return bool|null
  */
 function give_remove_donor_email() {
-	if ( empty( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
+	if (empty($_GET['id']) || ! is_numeric($_GET['id'])) {
 		return false;
 	}
-	if ( empty( $_GET['email'] ) || ! is_email( $_GET['email'] ) ) {
+	if (empty($_GET['email']) || ! is_email($_GET['email'])) {
 		return false;
 	}
-	if ( empty( $_GET['_wpnonce'] ) ) {
+	if (empty($_GET['_wpnonce'])) {
 		return false;
 	}
 
 	$nonce = $_GET['_wpnonce'];
-	if ( ! wp_verify_nonce( $nonce, 'give-remove-donor-email' ) ) {
-		wp_die( __( 'Nonce verification failed', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'give-remove-donor-email')) {
+		wp_die(__('Nonce verification failed', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	$donor = new Give_Donor( $_GET['id'] );
-	if ( $donor->remove_email( $_GET['email'] ) ) {
-		$url        = add_query_arg( 'give-message', 'email-removed', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ) );
+	$donor = new Give_Donor($_GET['id']);
+	if ($donor->remove_email($_GET['email'])) {
+		$url        = add_query_arg('give-message', 'email-removed', admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id));
 		$user       = wp_get_current_user();
-		$user_login = ! empty( $user->user_login ) ? $user->user_login : __( 'System', 'give' );
-		$donor_note = sprintf( __( 'Email address %1$s removed by %2$s', 'give' ), $_GET['email'], $user_login );
-		$donor->add_note( $donor_note );
+		$user_login = ! empty($user->user_login) ? $user->user_login : __('System', 'give');
+		$donor_note = sprintf(__('Email address %1$s removed by %2$s', 'give'), $_GET['email'], $user_login);
+		$donor->add_note($donor_note);
 	} else {
-		$url = add_query_arg( 'give-message', 'email-remove-failed', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ) );
+		$url = add_query_arg('give-message', 'email-remove-failed', admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id));
 	}
 
-	wp_safe_redirect( $url );
+	wp_safe_redirect($url);
 	exit;
 }
 
-add_action( 'give_remove_donor_email', 'give_remove_donor_email', 10 );
+add_action('give_remove_donor_email', 'give_remove_donor_email', 10);
 
 
 /**
@@ -568,41 +568,41 @@ 
  * @return bool|null
  */
 function give_set_donor_primary_email() {
-	if ( empty( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
+	if (empty($_GET['id']) || ! is_numeric($_GET['id'])) {
 		return false;
 	}
 
-	if ( empty( $_GET['email'] ) || ! is_email( $_GET['email'] ) ) {
+	if (empty($_GET['email']) || ! is_email($_GET['email'])) {
 		return false;
 	}
 
-	if ( empty( $_GET['_wpnonce'] ) ) {
+	if (empty($_GET['_wpnonce'])) {
 		return false;
 	}
 
 	$nonce = $_GET['_wpnonce'];
 
-	if ( ! wp_verify_nonce( $nonce, 'give-set-donor-primary-email' ) ) {
-		wp_die( __( 'Nonce verification failed', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'give-set-donor-primary-email')) {
+		wp_die(__('Nonce verification failed', 'give'), __('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	$donor = new Give_Donor( $_GET['id'] );
+	$donor = new Give_Donor($_GET['id']);
 
-	if ( $donor->set_primary_email( $_GET['email'] ) ) {
-		$url        = add_query_arg( 'give-message', 'primary-email-updated', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ) );
+	if ($donor->set_primary_email($_GET['email'])) {
+		$url        = add_query_arg('give-message', 'primary-email-updated', admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id));
 		$user       = wp_get_current_user();
-		$user_login = ! empty( $user->user_login ) ? $user->user_login : __( 'System', 'give' );
-		$donor_note = sprintf( __( 'Email address %1$s set as primary by %2$s', 'give' ), $_GET['email'], $user_login );
+		$user_login = ! empty($user->user_login) ? $user->user_login : __('System', 'give');
+		$donor_note = sprintf(__('Email address %1$s set as primary by %2$s', 'give'), $_GET['email'], $user_login);
 
-		$donor->add_note( $donor_note );
+		$donor->add_note($donor_note);
 	} else {
-		$url = add_query_arg( 'give-message', 'primary-email-failed', admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ) );
+		$url = add_query_arg('give-message', 'primary-email-failed', admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id));
 	}
 
-	wp_safe_redirect( $url );
+	wp_safe_redirect($url);
 	exit;
 }
 
-add_action( 'give_set_donor_primary_email', 'give_set_donor_primary_email', 10 );
+add_action('give_set_donor_primary_email', 'give_set_donor_primary_email', 10);

includes/forms/functions.php

Doc Comments +4 added lines, -4 removed lines

@@ -155,7 +155,7 @@ 
  *
  * Used to redirect a user back to the donation form if there are errors present.
  *
- * @param array $args
+ * @param string $args
  *
  * @access public
  * @since  1.0
@@ -301,7 +301,7 @@ 
  * @since  1.0
  * @since  1.8.16 Add security check
  *
- * @return bool
+ * @return false|null
  */
 function give_listen_for_failed_payments() {
 
@@ -466,7 +466,7 @@ 
  *
  * @param int $form_id Give Form ID
  *
- * @return int $earnings Earnings for a certain form
+ * @return double $earnings Earnings for a certain form
  */
 function give_get_form_earnings_stats( $form_id = 0 ) {
 	$give_form = new Give_Donate_Form( $form_id );
@@ -736,7 +736,7 @@ 
  *
  * @param int $form_id ID number of the form to retrieve the minimum price for
  *
- * @return mixed string|int Minimum price of the form
+ * @return string string|int Minimum price of the form
  */
 function give_get_form_minimum_price( $form_id = 0 ) {
 

Spacing +215 added lines, -216 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,14 +23,14 @@ 
 
 	global $typenow;
 
-	if ( $typenow != 'give_forms' ) {
+	if ($typenow != 'give_forms') {
 		return true;
 	}
 
 	return false;
 }
 
-add_filter( 'give_shortcode_button_condition', 'give_shortcode_button_condition' );
+add_filter('give_shortcode_button_condition', 'give_shortcode_button_condition');
 
 
 /**
@@ -40,11 +40,11 @@ 
  *
  * @return int|false
  */
-function get_form_id_from_args( $args ) {
+function get_form_id_from_args($args) {
 
-	if ( isset( $args['form_id'] ) && $args['form_id'] != 0 ) {
+	if (isset($args['form_id']) && $args['form_id'] != 0) {
 
-		return intval( $args['form_id'] );
+		return intval($args['form_id']);
 	}
 
 	return false;
@@ -59,23 +59,23 @@ 
  *
  * @return bool
  */
-function give_is_float_labels_enabled( $args ) {
+function give_is_float_labels_enabled($args) {
 
 	$float_labels = '';
 
-	if ( ! empty( $args['float_labels'] ) ) {
+	if ( ! empty($args['float_labels'])) {
 		$float_labels = $args['float_labels'];
 	}
 
-	if ( empty( $float_labels ) ) {
-		$float_labels = give_get_meta( $args['form_id'], '_give_form_floating_labels', true );
+	if (empty($float_labels)) {
+		$float_labels = give_get_meta($args['form_id'], '_give_form_floating_labels', true);
 	}
 
-	if ( empty( $float_labels ) || ( 'global' === $float_labels ) ) {
-		$float_labels = give_get_option( 'floatlabels', 'disabled' );
+	if (empty($float_labels) || ('global' === $float_labels)) {
+		$float_labels = give_get_option('floatlabels', 'disabled');
 	}
 
-	return give_is_setting_enabled( $float_labels );
+	return give_is_setting_enabled($float_labels);
 }
 
 /**
@@ -91,7 +91,7 @@ 
 
 	$can_checkout = true;
 
-	return (bool) apply_filters( 'give_can_checkout', $can_checkout );
+	return (bool) apply_filters('give_can_checkout', $can_checkout);
 }
 
 /**
@@ -105,9 +105,9 @@ 
 function give_get_success_page_uri() {
 	$give_options = give_get_settings();
 
-	$success_page = isset( $give_options['success_page'] ) ? get_permalink( absint( $give_options['success_page'] ) ) : get_bloginfo( 'url' );
+	$success_page = isset($give_options['success_page']) ? get_permalink(absint($give_options['success_page'])) : get_bloginfo('url');
 
-	return apply_filters( 'give_get_success_page_uri', $success_page );
+	return apply_filters('give_get_success_page_uri', $success_page);
 }
 
 /**
@@ -119,9 +119,9 @@ 
  */
 function give_is_success_page() {
 	$give_options    = give_get_settings();
-	$is_success_page = isset( $give_options['success_page'] ) ? is_page( $give_options['success_page'] ) : false;
+	$is_success_page = isset($give_options['success_page']) ? is_page($give_options['success_page']) : false;
 
-	return apply_filters( 'give_is_success_page', $is_success_page );
+	return apply_filters('give_is_success_page', $is_success_page);
 }
 
 /**
@@ -135,17 +135,17 @@ 
  * @since       1.0
  * @return      void
  */
-function give_send_to_success_page( $query_string = null ) {
+function give_send_to_success_page($query_string = null) {
 
 	$redirect = give_get_success_page_uri();
 
-	if ( $query_string ) {
+	if ($query_string) {
 		$redirect .= $query_string;
 	}
 
-	$gateway = isset( $_REQUEST['give-gateway'] ) ? $_REQUEST['give-gateway'] : '';
+	$gateway = isset($_REQUEST['give-gateway']) ? $_REQUEST['give-gateway'] : '';
 
-	wp_redirect( apply_filters( 'give_success_page_redirect', $redirect, $gateway, $query_string ) );
+	wp_redirect(apply_filters('give_success_page_redirect', $redirect, $gateway, $query_string));
 	give_die();
 }
 
@@ -161,19 +161,19 @@ 
  * @since  1.0
  * @return Void
  */
-function give_send_back_to_checkout( $args = array() ) {
+function give_send_back_to_checkout($args = array()) {
 
-	$url     = isset( $_POST['give-current-url'] ) ? sanitize_text_field( $_POST['give-current-url'] ) : '';
+	$url     = isset($_POST['give-current-url']) ? sanitize_text_field($_POST['give-current-url']) : '';
 	$form_id = 0;
 
 	// Set the form_id.
-	if ( isset( $_POST['give-form-id'] ) ) {
-		$form_id = sanitize_text_field( $_POST['give-form-id'] );
+	if (isset($_POST['give-form-id'])) {
+		$form_id = sanitize_text_field($_POST['give-form-id']);
 	}
 
 	// Need a URL to continue. If none, redirect back to single form.
-	if ( empty( $url ) ) {
-		wp_safe_redirect( get_permalink( $form_id ) );
+	if (empty($url)) {
+		wp_safe_redirect(get_permalink($form_id));
 		give_die();
 	}
 
@@ -182,41 +182,41 @@ 
 	);
 
 	// Set the $level_id.
-	if ( isset( $_POST['give-price-id'] ) ) {
-		$defaults['level-id'] = sanitize_text_field( $_POST['give-price-id'] );
+	if (isset($_POST['give-price-id'])) {
+		$defaults['level-id'] = sanitize_text_field($_POST['give-price-id']);
 	}
 
 	// Check for backward compatibility.
-	if ( is_string( $args ) ) {
-		$args = str_replace( '?', '', $args );
+	if (is_string($args)) {
+		$args = str_replace('?', '', $args);
 	}
 
-	$args = wp_parse_args( $args, $defaults );
+	$args = wp_parse_args($args, $defaults);
 
 	// Merge URL query with $args to maintain third-party URL parameters after redirect.
-	$url_data = wp_parse_url( $url );
+	$url_data = wp_parse_url($url);
 
 	// Check if an array to prevent notices before parsing.
-	if ( isset( $url_data['query'] ) && ! empty( $url_data['query'] ) ) {
-		parse_str( $url_data['query'], $query );
+	if (isset($url_data['query']) && ! empty($url_data['query'])) {
+		parse_str($url_data['query'], $query);
 
 		// Precaution: don't allow any CC info.
-		unset( $query['card_number'] );
-		unset( $query['card_cvc'] );
+		unset($query['card_number']);
+		unset($query['card_cvc']);
 
 	} else {
 		// No $url_data so pass empty array.
 		$query = array();
 	}
 
-	$new_query        = array_merge( $args, $query );
-	$new_query_string = http_build_query( $new_query );
+	$new_query        = array_merge($args, $query);
+	$new_query_string = http_build_query($new_query);
 
 	// Assemble URL parts.
-	$redirect = home_url( '/' . $url_data['path'] . '?' . $new_query_string . '#give-form-' . $form_id . '-wrap' );
+	$redirect = home_url('/'.$url_data['path'].'?'.$new_query_string.'#give-form-'.$form_id.'-wrap');
 
 	// Redirect them.
-	wp_safe_redirect( apply_filters( 'give_send_back_to_checkout', $redirect, $args ) );
+	wp_safe_redirect(apply_filters('give_send_back_to_checkout', $redirect, $args));
 	give_die();
 
 }
@@ -232,16 +232,16 @@ 
  * @since       1.0
  * @return      string
  */
-function give_get_success_page_url( $query_string = null ) {
+function give_get_success_page_url($query_string = null) {
 
-	$success_page = give_get_option( 'success_page', 0 );
-	$success_page = get_permalink( $success_page );
+	$success_page = give_get_option('success_page', 0);
+	$success_page = get_permalink($success_page);
 
-	if ( $query_string ) {
+	if ($query_string) {
 		$success_page .= $query_string;
 	}
 
-	return apply_filters( 'give_success_page_url', $success_page );
+	return apply_filters('give_success_page_url', $success_page);
 
 }
 
@@ -254,32 +254,31 @@ 
  *
  * @return mixed Full URL to the Failed Donation Page, if present, home page if it doesn't exist.
  */
-function give_get_failed_transaction_uri( $extras = false ) {
+function give_get_failed_transaction_uri($extras = false) {
 	$give_options = give_get_settings();
 
 	// Remove question mark.
-	if ( 0 === strpos( $extras, '?' ) ) {
-		$extras = substr( $extras, 1 );
+	if (0 === strpos($extras, '?')) {
+		$extras = substr($extras, 1);
 	}
 
-	$extras_args = wp_parse_args( $extras );
+	$extras_args = wp_parse_args($extras);
 
 	// Set nonce if payment id exist in extra params.
-	if ( array_key_exists( 'payment-id', $extras_args ) ) {
-		$extras_args['_wpnonce'] = wp_create_nonce( "give-failed-donation-{$extras_args['payment-id']}" );
-		$extras                  = http_build_query( $extras_args );
+	if (array_key_exists('payment-id', $extras_args)) {
+		$extras_args['_wpnonce'] = wp_create_nonce("give-failed-donation-{$extras_args['payment-id']}");
+		$extras                  = http_build_query($extras_args);
 	}
 
-	$uri = ! empty( $give_options['failure_page'] ) ?
-		trailingslashit( get_permalink( $give_options['failure_page'] ) ) :
-		home_url();
+	$uri = ! empty($give_options['failure_page']) ?
+		trailingslashit(get_permalink($give_options['failure_page'])) : home_url();
 
 
-	if ( $extras ) {
+	if ($extras) {
 		$uri .= "?{$extras}";
 	}
 
-	return apply_filters( 'give_get_failed_transaction_uri', $uri );
+	return apply_filters('give_get_failed_transaction_uri', $uri);
 }
 
 /**
@@ -290,9 +289,9 @@ 
  */
 function give_is_failed_transaction_page() {
 	$give_options = give_get_settings();
-	$ret          = isset( $give_options['failure_page'] ) ? is_page( $give_options['failure_page'] ) : false;
+	$ret          = isset($give_options['failure_page']) ? is_page($give_options['failure_page']) : false;
 
-	return apply_filters( 'give_is_failure_page', $ret );
+	return apply_filters('give_is_failure_page', $ret);
 }
 
 /**
@@ -305,25 +304,25 @@ 
  */
 function give_listen_for_failed_payments() {
 
-	$failed_page = give_get_option( 'failure_page', 0 );
-	$payment_id  = ! empty( $_GET['payment-id'] ) ? absint( $_GET['payment-id'] ) : 0;
-	$nonce       = ! empty( $_GET['_wpnonce'] ) ? give_clean( $_GET['_wpnonce'] ) : false;
+	$failed_page = give_get_option('failure_page', 0);
+	$payment_id  = ! empty($_GET['payment-id']) ? absint($_GET['payment-id']) : 0;
+	$nonce       = ! empty($_GET['_wpnonce']) ? give_clean($_GET['_wpnonce']) : false;
 
 	// Bailout.
-	if ( ! $failed_page || ! is_page( $failed_page ) || ! $payment_id || ! $nonce ) {
+	if ( ! $failed_page || ! is_page($failed_page) || ! $payment_id || ! $nonce) {
 		return false;
 	}
 
 	// Security check.
-	if ( ! wp_verify_nonce( $nonce, "give-failed-donation-{$payment_id}" ) ) {
-		wp_die( __( 'Nonce verification failed.', 'give' ), __( 'Error', 'give' ) );
+	if ( ! wp_verify_nonce($nonce, "give-failed-donation-{$payment_id}")) {
+		wp_die(__('Nonce verification failed.', 'give'), __('Error', 'give'));
 	}
 
 	// Set payment status to failure
-	give_update_payment_status( $payment_id, 'failed' );
+	give_update_payment_status($payment_id, 'failed');
 }
 
-add_action( 'template_redirect', 'give_listen_for_failed_payments' );
+add_action('template_redirect', 'give_listen_for_failed_payments');
 
 /**
  * Retrieve the Donation History page URI
@@ -336,9 +335,9 @@ 
 function give_get_history_page_uri() {
 	$give_options = give_get_settings();
 
-	$history_page = isset( $give_options['history_page'] ) ? get_permalink( absint( $give_options['history_page'] ) ) : get_bloginfo( 'url' );
+	$history_page = isset($give_options['history_page']) ? get_permalink(absint($give_options['history_page'])) : get_bloginfo('url');
 
-	return apply_filters( 'give_get_history_page_uri', $history_page );
+	return apply_filters('give_get_history_page_uri', $history_page);
 }
 
 /**
@@ -351,11 +350,11 @@ 
  * @since       1.0
  * @return      bool
  */
-function give_field_is_required( $field = '', $form_id ) {
+function give_field_is_required($field = '', $form_id) {
 
-	$required_fields = give_get_required_fields( $form_id );
+	$required_fields = give_get_required_fields($form_id);
 
-	return array_key_exists( $field, $required_fields );
+	return array_key_exists($field, $required_fields);
 }
 
 /**
@@ -373,14 +372,14 @@ 
  *
  * @return void
  */
-function give_record_donation_in_log( $give_form_id = 0, $payment_id, $price_id = false, $donation_date = null ) {
+function give_record_donation_in_log($give_form_id = 0, $payment_id, $price_id = false, $donation_date = null) {
 	global $give_logs;
 
 	$log_data = array(
 		'post_parent'   => $give_form_id,
 		'log_type'      => 'sale',
-		'post_date'     => isset( $donation_date ) ? $donation_date : null,
-		'post_date_gmt' => isset( $donation_date ) ? $donation_date : null,
+		'post_date'     => isset($donation_date) ? $donation_date : null,
+		'post_date_gmt' => isset($donation_date) ? $donation_date : null,
 	);
 
 	$log_meta = array(
@@ -388,7 +387,7 @@ 
 		'price_id'   => (int) $price_id,
 	);
 
-	$give_logs->insert_log( $log_data, $log_meta );
+	$give_logs->insert_log($log_data, $log_meta);
 }
 
 
@@ -402,11 +401,11 @@ 
  *
  * @return bool|int
  */
-function give_increase_donation_count( $form_id = 0, $quantity = 1 ) {
+function give_increase_donation_count($form_id = 0, $quantity = 1) {
 	$quantity = (int) $quantity;
-	$form     = new Give_Donate_Form( $form_id );
+	$form     = new Give_Donate_Form($form_id);
 
-	return $form->increase_sales( $quantity );
+	return $form->increase_sales($quantity);
 }
 
 /**
@@ -419,11 +418,11 @@ 
  *
  * @return bool|int
  */
-function give_decrease_donation_count( $form_id = 0, $quantity = 1 ) {
+function give_decrease_donation_count($form_id = 0, $quantity = 1) {
 	$quantity = (int) $quantity;
-	$form     = new Give_Donate_Form( $form_id );
+	$form     = new Give_Donate_Form($form_id);
 
-	return $form->decrease_sales( $quantity );
+	return $form->decrease_sales($quantity);
 }
 
 /**
@@ -436,10 +435,10 @@ 
  *
  * @return bool|int
  */
-function give_increase_earnings( $give_form_id = 0, $amount ) {
-	$form = new Give_Donate_Form( $give_form_id );
+function give_increase_earnings($give_form_id = 0, $amount) {
+	$form = new Give_Donate_Form($give_form_id);
 
-	return $form->increase_earnings( $amount );
+	return $form->increase_earnings($amount);
 }
 
 /**
@@ -452,10 +451,10 @@ 
  *
  * @return bool|int
  */
-function give_decrease_earnings( $form_id = 0, $amount ) {
-	$form = new Give_Donate_Form( $form_id );
+function give_decrease_earnings($form_id = 0, $amount) {
+	$form = new Give_Donate_Form($form_id);
 
-	return $form->decrease_earnings( $amount );
+	return $form->decrease_earnings($amount);
 }
 
 
@@ -468,8 +467,8 @@ 
  *
  * @return int $earnings Earnings for a certain form
  */
-function give_get_form_earnings_stats( $form_id = 0 ) {
-	$give_form = new Give_Donate_Form( $form_id );
+function give_get_form_earnings_stats($form_id = 0) {
+	$give_form = new Give_Donate_Form($form_id);
 
 	return $give_form->earnings;
 }
@@ -484,8 +483,8 @@ 
  *
  * @return int $sales Amount of sales for a certain form
  */
-function give_get_form_sales_stats( $give_form_id = 0 ) {
-	$give_form = new Give_Donate_Form( $give_form_id );
+function give_get_form_sales_stats($give_form_id = 0) {
+	$give_form = new Give_Donate_Form($give_form_id);
 
 	return $give_form->sales;
 }
@@ -500,16 +499,16 @@ 
  *
  * @return float $sales Average monthly sales
  */
-function give_get_average_monthly_form_sales( $form_id = 0 ) {
-	$sales        = give_get_form_sales_stats( $form_id );
-	$release_date = get_post_field( 'post_date', $form_id );
+function give_get_average_monthly_form_sales($form_id = 0) {
+	$sales        = give_get_form_sales_stats($form_id);
+	$release_date = get_post_field('post_date', $form_id);
 
-	$diff = abs( current_time( 'timestamp' ) - strtotime( $release_date ) );
+	$diff = abs(current_time('timestamp') - strtotime($release_date));
 
-	$months = floor( $diff / ( 30 * 60 * 60 * 24 ) ); // Number of months since publication
+	$months = floor($diff / (30 * 60 * 60 * 24)); // Number of months since publication
 
-	if ( $months > 0 ) {
-		$sales = ( $sales / $months );
+	if ($months > 0) {
+		$sales = ($sales / $months);
 	}
 
 	return $sales;
@@ -525,16 +524,16 @@ 
  *
  * @return float $earnings Average monthly earnings
  */
-function give_get_average_monthly_form_earnings( $form_id = 0 ) {
-	$earnings     = give_get_form_earnings_stats( $form_id );
-	$release_date = get_post_field( 'post_date', $form_id );
+function give_get_average_monthly_form_earnings($form_id = 0) {
+	$earnings     = give_get_form_earnings_stats($form_id);
+	$release_date = get_post_field('post_date', $form_id);
 
-	$diff = abs( current_time( 'timestamp' ) - strtotime( $release_date ) );
+	$diff = abs(current_time('timestamp') - strtotime($release_date));
 
-	$months = floor( $diff / ( 30 * 60 * 60 * 24 ) ); // Number of months since publication
+	$months = floor($diff / (30 * 60 * 60 * 24)); // Number of months since publication
 
-	if ( $months > 0 ) {
-		$earnings = ( $earnings / $months );
+	if ($months > 0) {
+		$earnings = ($earnings / $months);
 	}
 
 	return $earnings < 0 ? 0 : $earnings;
@@ -554,23 +553,23 @@ 
  *
  * @return string $price_name Name of the price option
  */
-function give_get_price_option_name( $form_id = 0, $price_id = 0, $payment_id = 0 ) {
+function give_get_price_option_name($form_id = 0, $price_id = 0, $payment_id = 0) {
 
-	$prices     = give_get_variable_prices( $form_id );
+	$prices     = give_get_variable_prices($form_id);
 	$price_name = '';
 
-	foreach ( $prices as $price ) {
+	foreach ($prices as $price) {
 
-		if ( intval( $price['_give_id']['level_id'] ) == intval( $price_id ) ) {
+		if (intval($price['_give_id']['level_id']) == intval($price_id)) {
 
-			$price_text     = isset( $price['_give_text'] ) ? $price['_give_text'] : '';
-			$price_fallback = give_currency_filter( give_format_amount( $price['_give_amount'], array( 'sanitize' => false ) ), '', true );
-			$price_name     = ! empty( $price_text ) ? $price_text : $price_fallback;
+			$price_text     = isset($price['_give_text']) ? $price['_give_text'] : '';
+			$price_fallback = give_currency_filter(give_format_amount($price['_give_amount'], array('sanitize' => false)), '', true);
+			$price_name     = ! empty($price_text) ? $price_text : $price_fallback;
 
 		}
 	}
 
-	return apply_filters( 'give_get_price_option_name', $price_name, $form_id, $payment_id, $price_id );
+	return apply_filters('give_get_price_option_name', $price_name, $form_id, $payment_id, $price_id);
 }
 
 
@@ -584,25 +583,25 @@ 
  *
  * @return string $range A fully formatted price range
  */
-function give_price_range( $form_id = 0, $formatted = true ) {
-	$low        = give_get_lowest_price_option( $form_id );
-	$high       = give_get_highest_price_option( $form_id );
-	$order_type = ! empty( $_REQUEST['order'] ) ? $_REQUEST['order'] : 'asc';
+function give_price_range($form_id = 0, $formatted = true) {
+	$low        = give_get_lowest_price_option($form_id);
+	$high       = give_get_highest_price_option($form_id);
+	$order_type = ! empty($_REQUEST['order']) ? $_REQUEST['order'] : 'asc';
 
 	$range = sprintf(
 		'<span class="give_price_range_%1$s">%2$s</span><span class="give_price_range_sep">&nbsp;&ndash;&nbsp;</span><span class="give_price_range_%3$s">%4$s</span>',
 		'asc' === $order_type ? 'low' : 'high',
-		'asc' === $order_type ? give_currency_filter( give_format_amount( $low, array( 'sanitize' => false ) ) ) : give_currency_filter( give_format_amount( $high, array( 'sanitize' => false ) ) ),
+		'asc' === $order_type ? give_currency_filter(give_format_amount($low, array('sanitize' => false))) : give_currency_filter(give_format_amount($high, array('sanitize' => false))),
 		'asc' === $order_type ? 'high' : 'low',
-		'asc' === $order_type ? give_currency_filter( give_format_amount( $high, array( 'sanitize' => false ) ) ) : give_currency_filter( give_format_amount( $low, array( 'sanitize' => false ) ) )
+		'asc' === $order_type ? give_currency_filter(give_format_amount($high, array('sanitize' => false))) : give_currency_filter(give_format_amount($low, array('sanitize' => false)))
 
 	);
 
-	if( ! $formatted ) {
-		$range = wp_strip_all_tags( $range );
+	if ( ! $formatted) {
+		$range = wp_strip_all_tags($range);
 	}
 
-	return apply_filters( 'give_price_range', $range, $form_id, $low, $high );
+	return apply_filters('give_price_range', $range, $form_id, $low, $high);
 }
 
 
@@ -617,35 +616,35 @@ 
  *
  * @return int ID of the lowest price
  */
-function give_get_lowest_price_id( $form_id = 0 ) {
+function give_get_lowest_price_id($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	if ( ! give_has_variable_prices( $form_id ) ) {
-		return give_get_form_price( $form_id );
+	if ( ! give_has_variable_prices($form_id)) {
+		return give_get_form_price($form_id);
 	}
 
-	$prices = give_get_variable_prices( $form_id );
+	$prices = give_get_variable_prices($form_id);
 
 	$min = $min_id = 0;
 
-	if ( ! empty( $prices ) ) {
+	if ( ! empty($prices)) {
 
-		foreach ( $prices as $key => $price ) {
+		foreach ($prices as $key => $price) {
 
-			if ( empty( $price['_give_amount'] ) ) {
+			if (empty($price['_give_amount'])) {
 				continue;
 			}
 
-			if ( ! isset( $min ) ) {
+			if ( ! isset($min)) {
 				$min = $price['_give_amount'];
 			} else {
-				$min = min( $min, $price['_give_amount'] );
+				$min = min($min, $price['_give_amount']);
 			}
 
-			if ( $price['_give_amount'] == $min ) {
+			if ($price['_give_amount'] == $min) {
 				$min_id = $price['_give_id']['level_id'];
 			}
 		}
@@ -663,22 +662,22 @@ 
  *
  * @return float Amount of the lowest price
  */
-function give_get_lowest_price_option( $form_id = 0 ) {
-	if ( empty( $form_id ) ) {
+function give_get_lowest_price_option($form_id = 0) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	if ( ! give_has_variable_prices( $form_id ) ) {
-		return give_get_form_price( $form_id );
+	if ( ! give_has_variable_prices($form_id)) {
+		return give_get_form_price($form_id);
 	}
 
-	if ( ! ( $low = get_post_meta( $form_id, '_give_levels_minimum_amount', true ) ) ) {
+	if ( ! ($low = get_post_meta($form_id, '_give_levels_minimum_amount', true))) {
 		// Backward compatibility.
-		$prices = wp_list_pluck( give_get_variable_prices( $form_id ), '_give_amount' );
-		$low    = ! empty( $prices ) ? min( $prices ) : 0;
+		$prices = wp_list_pluck(give_get_variable_prices($form_id), '_give_amount');
+		$low    = ! empty($prices) ? min($prices) : 0;
 	}
 
-	return give_maybe_sanitize_amount( $low );
+	return give_maybe_sanitize_amount($low);
 }
 
 /**
@@ -690,23 +689,23 @@ 
  *
  * @return float Amount of the highest price
  */
-function give_get_highest_price_option( $form_id = 0 ) {
+function give_get_highest_price_option($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	if ( ! give_has_variable_prices( $form_id ) ) {
-		return give_get_form_price( $form_id );
+	if ( ! give_has_variable_prices($form_id)) {
+		return give_get_form_price($form_id);
 	}
 
-	if ( ! ( $high = get_post_meta( $form_id, '_give_levels_maximum_amount', true ) ) ) {
+	if ( ! ($high = get_post_meta($form_id, '_give_levels_maximum_amount', true))) {
 		// Backward compatibility.
-		$prices = wp_list_pluck( give_get_variable_prices( $form_id ), '_give_amount' );
-		$high   = ! empty( $prices ) ? max( $prices ) : 0;
+		$prices = wp_list_pluck(give_get_variable_prices($form_id), '_give_amount');
+		$high   = ! empty($prices) ? max($prices) : 0;
 	}
 
-	return give_maybe_sanitize_amount( $high );
+	return give_maybe_sanitize_amount($high);
 }
 
 /**
@@ -718,15 +717,15 @@ 
  *
  * @return mixed string|int Price of the form
  */
-function give_get_form_price( $form_id = 0 ) {
+function give_get_form_price($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
-	return $form->__get( 'price' );
+	return $form->__get('price');
 }
 
 /**
@@ -738,15 +737,15 @@ 
  *
  * @return mixed string|int Minimum price of the form
  */
-function give_get_form_minimum_price( $form_id = 0 ) {
+function give_get_form_minimum_price($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
-	return $form->__get( 'minimum_price' );
+	return $form->__get('minimum_price');
 
 }
 
@@ -761,48 +760,48 @@ 
  *
  * @return int $formatted_price
  */
-function give_price( $form_id = 0, $echo = true, $price_id = false ) {
+function give_price($form_id = 0, $echo = true, $price_id = false) {
 	$price = 0;
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	if ( give_has_variable_prices( $form_id ) ) {
+	if (give_has_variable_prices($form_id)) {
 
-		$prices = give_get_variable_prices( $form_id );
+		$prices = give_get_variable_prices($form_id);
 
-		if ( false !== $price_id ) {
+		if (false !== $price_id) {
 
 			// loop through multi-prices to see which is default
-			foreach ( $prices as $price ) {
+			foreach ($prices as $price) {
 				// this is the default price
-				if ( isset( $price['_give_default'] ) && $price['_give_default'] === 'default' ) {
+				if (isset($price['_give_default']) && $price['_give_default'] === 'default') {
 					$price = (float) $price['_give_amount'];
 				};
 			}
 		} else {
 
-			$price = give_get_lowest_price_option( $form_id );
+			$price = give_get_lowest_price_option($form_id);
 		}
 	} else {
 
-		$price = give_get_form_price( $form_id );
+		$price = give_get_form_price($form_id);
 	}
 
-	$price           = apply_filters( 'give_form_price', give_maybe_sanitize_amount( $price ), $form_id );
-	$formatted_price = '<span class="give_price" id="give_price_' . $form_id . '">' . $price . '</span>';
-	$formatted_price = apply_filters( 'give_form_price_after_html', $formatted_price, $form_id, $price );
+	$price           = apply_filters('give_form_price', give_maybe_sanitize_amount($price), $form_id);
+	$formatted_price = '<span class="give_price" id="give_price_'.$form_id.'">'.$price.'</span>';
+	$formatted_price = apply_filters('give_form_price_after_html', $formatted_price, $form_id, $price);
 
-	if ( $echo ) {
+	if ($echo) {
 		echo $formatted_price;
 	} else {
 		return $formatted_price;
 	}
 }
 
-add_filter( 'give_form_price', 'give_format_amount', 10 );
-add_filter( 'give_form_price', 'give_currency_filter', 20 );
+add_filter('give_form_price', 'give_format_amount', 10);
+add_filter('give_form_price', 'give_currency_filter', 20);
 
 
 /**
@@ -815,19 +814,19 @@ 
  *
  * @return float $amount Amount of the price option
  */
-function give_get_price_option_amount( $form_id = 0, $price_id = 0 ) {
-	$prices = give_get_variable_prices( $form_id );
+function give_get_price_option_amount($form_id = 0, $price_id = 0) {
+	$prices = give_get_variable_prices($form_id);
 
 	$amount = 0.00;
 
-	foreach ( $prices as $price ) {
-		if ( isset( $price['_give_id']['level_id'] ) && $price['_give_id']['level_id'] == $price_id ) {
-			$amount = isset( $price['_give_amount'] ) ? $price['_give_amount'] : 0.00;
+	foreach ($prices as $price) {
+		if (isset($price['_give_id']['level_id']) && $price['_give_id']['level_id'] == $price_id) {
+			$amount = isset($price['_give_amount']) ? $price['_give_amount'] : 0.00;
 			break;
 		};
 	}
 
-	return apply_filters( 'give_get_price_option_amount', give_maybe_sanitize_amount( $amount ), $form_id, $price_id );
+	return apply_filters('give_get_price_option_amount', give_maybe_sanitize_amount($amount), $form_id, $price_id);
 }
 
 /**
@@ -839,13 +838,13 @@ 
  *
  * @return mixed string|int Goal of the form
  */
-function give_get_form_goal( $form_id = 0 ) {
+function give_get_form_goal($form_id = 0) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $form_id );
+	$form = new Give_Donate_Form($form_id);
 
 	return $form->goal;
 
@@ -861,27 +860,27 @@ 
  *
  * @return string $formatted_goal
  */
-function give_goal( $form_id = 0, $echo = true ) {
+function give_goal($form_id = 0, $echo = true) {
 
-	if ( empty( $form_id ) ) {
+	if (empty($form_id)) {
 		$form_id = get_the_ID();
 	}
 
-	$goal = give_get_form_goal( $form_id );
+	$goal = give_get_form_goal($form_id);
 
-	$goal           = apply_filters( 'give_form_goal', give_maybe_sanitize_amount( $goal ), $form_id );
-	$formatted_goal = '<span class="give_price" id="give_price_' . $form_id . '">' . $goal . '</span>';
-	$formatted_goal = apply_filters( 'give_form_price_after_html', $formatted_goal, $form_id, $goal );
+	$goal           = apply_filters('give_form_goal', give_maybe_sanitize_amount($goal), $form_id);
+	$formatted_goal = '<span class="give_price" id="give_price_'.$form_id.'">'.$goal.'</span>';
+	$formatted_goal = apply_filters('give_form_price_after_html', $formatted_goal, $form_id, $goal);
 
-	if ( $echo ) {
+	if ($echo) {
 		echo $formatted_goal;
 	} else {
 		return $formatted_goal;
 	}
 }
 
-add_filter( 'give_form_goal', 'give_format_amount', 10 );
-add_filter( 'give_form_goal', 'give_currency_filter', 20 );
+add_filter('give_form_goal', 'give_format_amount', 10);
+add_filter('give_form_goal', 'give_currency_filter', 20);
 
 
 /**
@@ -893,15 +892,15 @@ 
  *
  * @return bool  $ret Whether or not the logged_in_only setting is set
  */
-function give_logged_in_only( $form_id ) {
+function give_logged_in_only($form_id) {
 	// If _give_logged_in_only is set to enable then guest can donate from that specific form.
 	// Otherwise it is member only donation form.
-	$val = give_get_meta( $form_id, '_give_logged_in_only', true );
-	$val = ! empty( $val ) ? $val : 'enabled';
+	$val = give_get_meta($form_id, '_give_logged_in_only', true);
+	$val = ! empty($val) ? $val : 'enabled';
 
-	$ret = ! give_is_setting_enabled( $val );
+	$ret = ! give_is_setting_enabled($val);
 
-	return (bool) apply_filters( 'give_logged_in_only', $ret, $form_id );
+	return (bool) apply_filters('give_logged_in_only', $ret, $form_id);
 }
 
 
@@ -914,11 +913,11 @@ 
  *
  * @return string
  */
-function give_show_login_register_option( $form_id ) {
+function give_show_login_register_option($form_id) {
 
-	$show_register_form = give_get_meta( $form_id, '_give_show_register_form', true );
+	$show_register_form = give_get_meta($form_id, '_give_show_register_form', true);
 
-	return apply_filters( 'give_show_register_form', $show_register_form, $form_id );
+	return apply_filters('give_show_register_form', $show_register_form, $form_id);
 
 }
 
@@ -934,12 +933,12 @@ 
  *
  * @return array
  */
-function _give_get_prefill_form_field_values( $form_id ) {
+function _give_get_prefill_form_field_values($form_id) {
 	$logged_in_donor_info = array();
 
-	if ( is_user_logged_in() ) :
-		$donor_data    = get_userdata( get_current_user_id() );
-		$donor_address = get_user_meta( get_current_user_id(), '_give_user_address', true );
+	if (is_user_logged_in()) :
+		$donor_data    = get_userdata(get_current_user_id());
+		$donor_address = get_user_meta(get_current_user_id(), '_give_user_address', true);
 
 		$logged_in_donor_info = array(
 			// First name.
@@ -952,42 +951,42 @@ 
 			'give_email'      => $donor_data->user_email,
 
 			// Street address 1.
-			'card_address'    => ( ! empty( $donor_address['line1'] ) ? $donor_address['line1'] : '' ),
+			'card_address'    => ( ! empty($donor_address['line1']) ? $donor_address['line1'] : ''),
 
 			// Street address 2.
-			'card_address_2'  => ( ! empty( $donor_address['line2'] ) ? $donor_address['line2'] : '' ),
+			'card_address_2'  => ( ! empty($donor_address['line2']) ? $donor_address['line2'] : ''),
 
 			// Country.
-			'billing_country' => ( ! empty( $donor_address['country'] ) ? $donor_address['country'] : '' ),
+			'billing_country' => ( ! empty($donor_address['country']) ? $donor_address['country'] : ''),
 
 			// State.
-			'card_state'      => ( ! empty( $donor_address['state'] ) ? $donor_address['state'] : '' ),
+			'card_state'      => ( ! empty($donor_address['state']) ? $donor_address['state'] : ''),
 
 			// City.
-			'card_city'       => ( ! empty( $donor_address['city'] ) ? $donor_address['city'] : '' ),
+			'card_city'       => ( ! empty($donor_address['city']) ? $donor_address['city'] : ''),
 
 			// Zipcode
-			'card_zip'        => ( ! empty( $donor_address['zip'] ) ? $donor_address['zip'] : '' ),
+			'card_zip'        => ( ! empty($donor_address['zip']) ? $donor_address['zip'] : ''),
 		);
 	endif;
 
 	// Bailout: Auto fill form field values only form form which donor is donating.
 	if (
-		empty( $_GET['form-id'] )
+		empty($_GET['form-id'])
 		|| ! $form_id
-		|| ( $form_id !== absint( $_GET['form-id'] ) )
+		|| ($form_id !== absint($_GET['form-id']))
 	) {
 		return $logged_in_donor_info;
 	}
 
 	// Get purchase data.
-	$give_purchase_data = Give()->session->get( 'give_purchase' );
+	$give_purchase_data = Give()->session->get('give_purchase');
 
 	// Get donor info from form data.
-	$give_donor_info_in_session = empty( $give_purchase_data['post_data'] )
+	$give_donor_info_in_session = empty($give_purchase_data['post_data'])
 		? array()
 		: $give_purchase_data['post_data'];
 
 	// Output.
-	return wp_parse_args( $give_donor_info_in_session, $logged_in_donor_info );
+	return wp_parse_args($give_donor_info_in_session, $logged_in_donor_info);
 }

includes/payments/functions.php

Doc Comments +3 added lines, -3 removed lines

@@ -705,7 +705,7 @@ 
  *
  * @since 1.0
  *
- * @return int $earnings Earnings
+ * @return double $earnings Earnings
  */
 function give_get_earnings_by_date( $day = null, $month_num, $year = null, $hour = null ) {
 
@@ -993,7 +993,7 @@ 
  *
  * @since 1.0
  *
- * @return array $user_info User Info Meta Values.
+ * @return string $user_info User Info Meta Values.
  */
 function give_get_payment_meta_user_info( $payment_id ) {
 	$payment = new Give_Payment( $payment_id );
@@ -1010,7 +1010,7 @@ 
  *
  * @since 1.0
  *
- * @return int $form_id Form ID.
+ * @return string $form_id Form ID.
  */
 function give_get_payment_form_id( $payment_id ) {
 	$payment = new Give_Payment( $payment_id );

Spacing +418 added lines, -418 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -43,15 +43,15 @@ 
  *
  * @return array $payments Payments retrieved from the database
  */
-function give_get_payments( $args = array() ) {
+function give_get_payments($args = array()) {
 
 	// Fallback to post objects to ensure backwards compatibility.
-	if ( ! isset( $args['output'] ) ) {
+	if ( ! isset($args['output'])) {
 		$args['output'] = 'posts';
 	}
 
-	$args     = apply_filters( 'give_get_payments_args', $args );
-	$payments = new Give_Payments_Query( $args );
+	$args     = apply_filters('give_get_payments_args', $args);
+	$payments = new Give_Payments_Query($args);
 
 	return $payments->get_payments();
 }
@@ -66,48 +66,48 @@ 
  *
  * @return mixed
  */
-function give_get_payment_by( $field = '', $value = '' ) {
+function give_get_payment_by($field = '', $value = '') {
 
-	if ( empty( $field ) || empty( $value ) ) {
+	if (empty($field) || empty($value)) {
 		return false;
 	}
 
-	switch ( strtolower( $field ) ) {
+	switch (strtolower($field)) {
 
 		case 'id':
-			$payment = new Give_Payment( $value );
+			$payment = new Give_Payment($value);
 			$id      = $payment->ID;
 
-			if ( empty( $id ) ) {
+			if (empty($id)) {
 				return false;
 			}
 
 			break;
 
 		case 'key':
-			$payment = give_get_payments( array(
+			$payment = give_get_payments(array(
 				'meta_key'       => '_give_payment_purchase_key',
 				'meta_value'     => $value,
 				'posts_per_page' => 1,
 				'fields'         => 'ids',
-			) );
+			));
 
-			if ( $payment ) {
-				$payment = new Give_Payment( $payment[0] );
+			if ($payment) {
+				$payment = new Give_Payment($payment[0]);
 			}
 
 			break;
 
 		case 'payment_number':
-			$payment = give_get_payments( array(
+			$payment = give_get_payments(array(
 				'meta_key'       => '_give_payment_number',
 				'meta_value'     => $value,
 				'posts_per_page' => 1,
 				'fields'         => 'ids',
-			) );
+			));
 
-			if ( $payment ) {
-				$payment = new Give_Payment( $payment[0] );
+			if ($payment) {
+				$payment = new Give_Payment($payment[0]);
 			}
 
 			break;
@@ -116,7 +116,7 @@ 
 			return false;
 	}// End switch().
 
-	if ( $payment ) {
+	if ($payment) {
 		return $payment;
 	}
 
@@ -132,9 +132,9 @@ 
  *
  * @return int|bool Payment ID if payment is inserted, false otherwise.
  */
-function give_insert_payment( $payment_data = array() ) {
+function give_insert_payment($payment_data = array()) {
 
-	if ( empty( $payment_data ) ) {
+	if (empty($payment_data)) {
 		return false;
 	}
 
@@ -145,25 +145,25 @@ 
 	 *
 	 * @param array $payment_data Arguments passed.
 	 */
-	$payment_data = apply_filters( 'give_pre_insert_payment', $payment_data );
+	$payment_data = apply_filters('give_pre_insert_payment', $payment_data);
 
 	$payment    = new Give_Payment();
-	$gateway    = ! empty( $payment_data['gateway'] ) ? $payment_data['gateway'] : '';
-	$gateway    = empty( $gateway ) && isset( $_POST['give-gateway'] ) ? $_POST['give-gateway'] : $gateway;
-	$form_id    = isset( $payment_data['give_form_id'] ) ? $payment_data['give_form_id'] : 0;
-	$price_id   = give_get_payment_meta_price_id( $payment_data );
-	$form_title = isset( $payment_data['give_form_title'] ) ? $payment_data['give_form_title'] : get_the_title( $form_id );
+	$gateway    = ! empty($payment_data['gateway']) ? $payment_data['gateway'] : '';
+	$gateway    = empty($gateway) && isset($_POST['give-gateway']) ? $_POST['give-gateway'] : $gateway;
+	$form_id    = isset($payment_data['give_form_id']) ? $payment_data['give_form_id'] : 0;
+	$price_id   = give_get_payment_meta_price_id($payment_data);
+	$form_title = isset($payment_data['give_form_title']) ? $payment_data['give_form_title'] : get_the_title($form_id);
 
 	// Set properties.
 	$payment->total          = $payment_data['price'];
-	$payment->status         = ! empty( $payment_data['status'] ) ? $payment_data['status'] : 'pending';
-	$payment->currency       = ! empty( $payment_data['currency'] ) ? $payment_data['currency'] : give_get_currency( $payment_data['give_form_id'], $payment_data );
+	$payment->status         = ! empty($payment_data['status']) ? $payment_data['status'] : 'pending';
+	$payment->currency       = ! empty($payment_data['currency']) ? $payment_data['currency'] : give_get_currency($payment_data['give_form_id'], $payment_data);
 	$payment->user_info      = $payment_data['user_info'];
 	$payment->gateway        = $gateway;
 	$payment->form_title     = $form_title;
 	$payment->form_id        = $form_id;
 	$payment->price_id       = $price_id;
-	$payment->donor_id       = ( ! empty( $payment_data['donor_id'] ) ? $payment_data['donor_id'] : '' );
+	$payment->donor_id       = ( ! empty($payment_data['donor_id']) ? $payment_data['donor_id'] : '');
 	$payment->user_id        = $payment_data['user_info']['id'];
 	$payment->email          = $payment_data['user_email'];
 	$payment->first_name     = $payment_data['user_info']['first_name'];
@@ -171,8 +171,8 @@ 
 	$payment->email          = $payment_data['user_info']['email'];
 	$payment->ip             = give_get_ip();
 	$payment->key            = $payment_data['purchase_key'];
-	$payment->mode           = ( ! empty( $payment_data['mode'] ) ? (string) $payment_data['mode'] : ( give_is_test_mode() ? 'test' : 'live' ) );
-	$payment->parent_payment = ! empty( $payment_data['parent'] ) ? absint( $payment_data['parent'] ) : '';
+	$payment->mode           = ( ! empty($payment_data['mode']) ? (string) $payment_data['mode'] : (give_is_test_mode() ? 'test' : 'live'));
+	$payment->parent_payment = ! empty($payment_data['parent']) ? absint($payment_data['parent']) : '';
 
 	// Add the donation.
 	$args = array(
@@ -180,19 +180,19 @@ 
 		'price_id' => $payment->price_id,
 	);
 
-	$payment->add_donation( $payment->form_id, $args );
+	$payment->add_donation($payment->form_id, $args);
 
 
 	// Set date if present.
-	if ( isset( $payment_data['post_date'] ) ) {
+	if (isset($payment_data['post_date'])) {
 		$payment->date = $payment_data['post_date'];
 	}
 
 	// Handle sequential payments.
-	if ( give_get_option( 'enable_sequential' ) ) {
+	if (give_get_option('enable_sequential')) {
 		$number          = give_get_next_payment_number();
-		$payment->number = give_format_payment_number( $number );
-		update_option( 'give_last_payment_number', $number );
+		$payment->number = give_format_payment_number($number);
+		update_option('give_last_payment_number', $number);
 	}
 
 	// Save payment.
@@ -206,10 +206,10 @@ 
 	 * @param int $payment_id The payment ID.
 	 * @param array $payment_data Arguments passed.
 	 */
-	do_action( 'give_insert_payment', $payment->ID, $payment_data );
+	do_action('give_insert_payment', $payment->ID, $payment_data);
 
 	// Return payment ID upon success.
-	if ( ! empty( $payment->ID ) ) {
+	if ( ! empty($payment->ID)) {
 		return $payment->ID;
 	}
 
@@ -225,10 +225,10 @@ 
  *
  * @return bool|int
  */
-function give_create_payment( $payment_data ) {
+function give_create_payment($payment_data) {
 
-	$form_id  = intval( $payment_data['post_data']['give-form-id'] );
-	$price_id = isset( $payment_data['post_data']['give-price-id'] ) ? $payment_data['post_data']['give-price-id'] : '';
+	$form_id  = intval($payment_data['post_data']['give-form-id']);
+	$price_id = isset($payment_data['post_data']['give-price-id']) ? $payment_data['post_data']['give-price-id'] : '';
 
 	// Collect payment data.
 	$insert_payment_data = array(
@@ -239,7 +239,7 @@ 
 		'date'            => $payment_data['date'],
 		'user_email'      => $payment_data['user_email'],
 		'purchase_key'    => $payment_data['purchase_key'],
-		'currency'        => give_get_currency( $form_id, $payment_data ),
+		'currency'        => give_get_currency($form_id, $payment_data),
 		'user_info'       => $payment_data['user_info'],
 		'status'          => 'pending',
 		'gateway'         => 'paypal',
@@ -252,10 +252,10 @@ 
 	 *
 	 * @param array $insert_payment_data
 	 */
-	$insert_payment_data = apply_filters( 'give_create_payment', $insert_payment_data );
+	$insert_payment_data = apply_filters('give_create_payment', $insert_payment_data);
 
 	// Record the pending payment.
-	return give_insert_payment( $insert_payment_data );
+	return give_insert_payment($insert_payment_data);
 }
 
 /**
@@ -268,12 +268,12 @@ 
  *
  * @return bool
  */
-function give_update_payment_status( $payment_id, $new_status = 'publish' ) {
+function give_update_payment_status($payment_id, $new_status = 'publish') {
 
 	$updated = false;
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	if ( $payment && $payment->ID > 0 ) {
+	if ($payment && $payment->ID > 0) {
 
 		$payment->status = $new_status;
 		$updated         = $payment->save();
@@ -295,38 +295,38 @@ 
  *
  * @return void
  */
-function give_delete_donation( $payment_id = 0, $update_donor = true ) {
+function give_delete_donation($payment_id = 0, $update_donor = true) {
 	global $give_logs;
 
-	$payment  = new Give_Payment( $payment_id );
-	$amount   = give_get_payment_amount( $payment_id );
+	$payment  = new Give_Payment($payment_id);
+	$amount   = give_get_payment_amount($payment_id);
 	$status   = $payment->post_status;
-	$donor_id = give_get_payment_donor_id( $payment_id );
-	$donor    = new Give_Donor( $donor_id );
+	$donor_id = give_get_payment_donor_id($payment_id);
+	$donor    = new Give_Donor($donor_id);
 
 	// Only undo donations that aren't these statuses.
-	$dont_undo_statuses = apply_filters( 'give_undo_donation_statuses', array(
+	$dont_undo_statuses = apply_filters('give_undo_donation_statuses', array(
 		'pending',
 		'cancelled',
-	) );
+	));
 
-	if ( ! in_array( $status, $dont_undo_statuses ) ) {
-		give_undo_donation( $payment_id );
+	if ( ! in_array($status, $dont_undo_statuses)) {
+		give_undo_donation($payment_id);
 	}
 
-	if ( $status == 'publish' ) {
+	if ($status == 'publish') {
 
 		// Only decrease earnings if they haven't already been decreased (or were never increased for this payment).
-		give_decrease_total_earnings( $amount );
+		give_decrease_total_earnings($amount);
 
 		// @todo: Refresh only range related stat cache
 		give_delete_donation_stats();
 
-		if ( $donor->id && $update_donor ) {
+		if ($donor->id && $update_donor) {
 
 			// Decrement the stats for the donor.
 			$donor->decrease_donation_count();
-			$donor->decrease_value( $amount );
+			$donor->decrease_value($amount);
 
 		}
 	}
@@ -338,25 +338,25 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_payment_delete', $payment_id );
+	do_action('give_payment_delete', $payment_id);
 
-	if ( $donor->id && $update_donor ) {
+	if ($donor->id && $update_donor) {
 
 		// Remove the payment ID from the donor.
-		$donor->remove_payment( $payment_id );
+		$donor->remove_payment($payment_id);
 
 	}
 
 	// Remove the payment.
-	wp_delete_post( $payment_id, true );
+	wp_delete_post($payment_id, true);
 
 	// Remove related sale log entries.
-	$give_logs->delete_logs( null, 'sale', array(
+	$give_logs->delete_logs(null, 'sale', array(
 		array(
 			'key'   => '_give_log_payment_id',
 			'value' => $payment_id,
 		),
-	) );
+	));
 
 	/**
 	 * Fires after payment deleted.
@@ -365,7 +365,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_payment_deleted', $payment_id );
+	do_action('give_payment_deleted', $payment_id);
 }
 
 /**
@@ -380,20 +380,20 @@ 
  *
  * @return void
  */
-function give_undo_donation( $payment_id ) {
+function give_undo_donation($payment_id) {
 
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	$maybe_decrease_earnings = apply_filters( 'give_decrease_earnings_on_undo', true, $payment, $payment->form_id );
-	if ( true === $maybe_decrease_earnings ) {
+	$maybe_decrease_earnings = apply_filters('give_decrease_earnings_on_undo', true, $payment, $payment->form_id);
+	if (true === $maybe_decrease_earnings) {
 		// Decrease earnings.
-		give_decrease_earnings( $payment->form_id, $payment->total );
+		give_decrease_earnings($payment->form_id, $payment->total);
 	}
 
-	$maybe_decrease_donations = apply_filters( 'give_decrease_donations_on_undo', true, $payment, $payment->form_id );
-	if ( true === $maybe_decrease_donations ) {
+	$maybe_decrease_donations = apply_filters('give_decrease_donations_on_undo', true, $payment, $payment->form_id);
+	if (true === $maybe_decrease_donations) {
 		// Decrease donation count.
-		give_decrease_donation_count( $payment->form_id );
+		give_decrease_donation_count($payment->form_id);
 	}
 
 }
@@ -410,7 +410,7 @@ 
  *
  * @return object $stats Contains the number of payments per payment status.
  */
-function give_count_payments( $args = array() ) {
+function give_count_payments($args = array()) {
 
 	global $wpdb;
 
@@ -422,18 +422,18 @@ 
 		'form_id'    => null,
 	);
 
-	$args = wp_parse_args( $args, $defaults );
+	$args = wp_parse_args($args, $defaults);
 
 	$select = 'SELECT p.post_status,count( * ) AS num_posts';
 	$join   = '';
-	$where  = "WHERE p.post_type = 'give_payment' AND p.post_status IN ('" . implode( "','", give_get_payment_status_keys() ) . "')";
+	$where  = "WHERE p.post_type = 'give_payment' AND p.post_status IN ('".implode("','", give_get_payment_status_keys())."')";
 
 	// Count payments for a specific user.
-	if ( ! empty( $args['user'] ) ) {
+	if ( ! empty($args['user'])) {
 
-		if ( is_email( $args['user'] ) ) {
+		if (is_email($args['user'])) {
 			$field = 'email';
-		} elseif ( is_numeric( $args['user'] ) ) {
+		} elseif (is_numeric($args['user'])) {
 			$field = 'id';
 		} else {
 			$field = '';
@@ -441,107 +441,107 @@ 
 
 		$join = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
 
-		if ( ! empty( $field ) ) {
+		if ( ! empty($field)) {
 			$where .= "
 				AND m.meta_key = '_give_payment_user_{$field}'
 				AND m.meta_value = '{$args['user']}'";
 		}
-	} elseif ( ! empty( $args['donor'] ) ) {
+	} elseif ( ! empty($args['donor'])) {
 
-		$join  = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
+		$join = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
 		$where .= "
 			AND m.meta_key = '_give_payment_customer_id'
 			AND m.meta_value = '{$args['donor']}'";
 
 		// Count payments for a search.
-	} elseif ( ! empty( $args['s'] ) ) {
+	} elseif ( ! empty($args['s'])) {
 
-		if ( is_email( $args['s'] ) || strlen( $args['s'] ) == 32 ) {
+		if (is_email($args['s']) || strlen($args['s']) == 32) {
 
-			if ( is_email( $args['s'] ) ) {
+			if (is_email($args['s'])) {
 				$field = '_give_payment_user_email';
 			} else {
 				$field = '_give_payment_purchase_key';
 			}
 
-			$join  = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
-			$where .= $wpdb->prepare( '
+			$join = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
+			$where .= $wpdb->prepare('
                 AND m.meta_key = %s
-                AND m.meta_value = %s', $field, $args['s'] );
+                AND m.meta_value = %s', $field, $args['s']);
 
-		} elseif ( '#' == substr( $args['s'], 0, 1 ) ) {
+		} elseif ('#' == substr($args['s'], 0, 1)) {
 
-			$search = str_replace( '#:', '', $args['s'] );
-			$search = str_replace( '#', '', $search );
+			$search = str_replace('#:', '', $args['s']);
+			$search = str_replace('#', '', $search);
 
 			$select = 'SELECT p.post_status,count( * ) AS num_posts ';
 			$join   = '';
-			$where  = $wpdb->prepare( 'WHERE p.post_type=%s  AND p.ID = %d ', 'give_payment', $search );
+			$where  = $wpdb->prepare('WHERE p.post_type=%s  AND p.ID = %d ', 'give_payment', $search);
 
-		} elseif ( is_numeric( $args['s'] ) ) {
+		} elseif (is_numeric($args['s'])) {
 
-			$join  = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
-			$where .= $wpdb->prepare( "
+			$join = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
+			$where .= $wpdb->prepare("
 				AND m.meta_key = '_give_payment_user_id'
-				AND m.meta_value = %d", $args['s'] );
+				AND m.meta_value = %d", $args['s']);
 
 		} else {
-			$search = $wpdb->esc_like( $args['s'] );
-			$search = '%' . $search . '%';
+			$search = $wpdb->esc_like($args['s']);
+			$search = '%'.$search.'%';
 
-			$where .= $wpdb->prepare( 'AND ((p.post_title LIKE %s) OR (p.post_content LIKE %s))', $search, $search );
+			$where .= $wpdb->prepare('AND ((p.post_title LIKE %s) OR (p.post_content LIKE %s))', $search, $search);
 		}// End if().
 	}// End if().
 
-	if ( ! empty( $args['form_id'] ) && is_numeric( $args['form_id'] ) ) {
+	if ( ! empty($args['form_id']) && is_numeric($args['form_id'])) {
 
-		$join  = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
-		$where .= $wpdb->prepare( '
+		$join = "LEFT JOIN $wpdb->postmeta m ON (p.ID = m.post_id)";
+		$where .= $wpdb->prepare('
                 AND m.meta_key = %s
-                AND m.meta_value = %s', '_give_payment_form_id', $args['form_id'] );
+                AND m.meta_value = %s', '_give_payment_form_id', $args['form_id']);
 	}
 
 	// Limit payments count by date.
-	if ( ! empty( $args['start-date'] ) && false !== strpos( $args['start-date'], '/' ) ) {
+	if ( ! empty($args['start-date']) && false !== strpos($args['start-date'], '/')) {
 
-		$date_parts = explode( '/', $args['start-date'] );
-		$month      = ! empty( $date_parts[0] ) && is_numeric( $date_parts[0] ) ? $date_parts[0] : 0;
-		$day        = ! empty( $date_parts[1] ) && is_numeric( $date_parts[1] ) ? $date_parts[1] : 0;
-		$year       = ! empty( $date_parts[2] ) && is_numeric( $date_parts[2] ) ? $date_parts[2] : 0;
+		$date_parts = explode('/', $args['start-date']);
+		$month      = ! empty($date_parts[0]) && is_numeric($date_parts[0]) ? $date_parts[0] : 0;
+		$day        = ! empty($date_parts[1]) && is_numeric($date_parts[1]) ? $date_parts[1] : 0;
+		$year       = ! empty($date_parts[2]) && is_numeric($date_parts[2]) ? $date_parts[2] : 0;
 
-		$is_date = checkdate( $month, $day, $year );
-		if ( false !== $is_date ) {
+		$is_date = checkdate($month, $day, $year);
+		if (false !== $is_date) {
 
-			$date  = new DateTime( $args['start-date'] );
-			$where .= $wpdb->prepare( " AND p.post_date >= '%s'", $date->format( 'Y-m-d' ) );
+			$date = new DateTime($args['start-date']);
+			$where .= $wpdb->prepare(" AND p.post_date >= '%s'", $date->format('Y-m-d'));
 
 		}
 
 		// Fixes an issue with the payments list table counts when no end date is specified (with stats class).
-		if ( empty( $args['end-date'] ) ) {
+		if (empty($args['end-date'])) {
 			$args['end-date'] = $args['start-date'];
 		}
 	}
 
-	if ( ! empty( $args['end-date'] ) && false !== strpos( $args['end-date'], '/' ) ) {
+	if ( ! empty($args['end-date']) && false !== strpos($args['end-date'], '/')) {
 
-		$date_parts = explode( '/', $args['end-date'] );
+		$date_parts = explode('/', $args['end-date']);
 
-		$month = ! empty( $date_parts[0] ) ? $date_parts[0] : 0;
-		$day   = ! empty( $date_parts[1] ) ? $date_parts[1] : 0;
-		$year  = ! empty( $date_parts[2] ) ? $date_parts[2] : 0;
+		$month = ! empty($date_parts[0]) ? $date_parts[0] : 0;
+		$day   = ! empty($date_parts[1]) ? $date_parts[1] : 0;
+		$year  = ! empty($date_parts[2]) ? $date_parts[2] : 0;
 
-		$is_date = checkdate( $month, $day, $year );
-		if ( false !== $is_date ) {
+		$is_date = checkdate($month, $day, $year);
+		if (false !== $is_date) {
 
-			$date  = new DateTime( $args['end-date'] );
-			$where .= $wpdb->prepare( " AND p.post_date <= '%s'", $date->format( 'Y-m-d' ) );
+			$date = new DateTime($args['end-date']);
+			$where .= $wpdb->prepare(" AND p.post_date <= '%s'", $date->format('Y-m-d'));
 
 		}
 	}
 
-	$where = apply_filters( 'give_count_payments_where', $where );
-	$join  = apply_filters( 'give_count_payments_join', $join );
+	$where = apply_filters('give_count_payments_where', $where);
+	$join  = apply_filters('give_count_payments_join', $join);
 
 	$query = "$select
 		FROM $wpdb->posts p
@@ -550,36 +550,36 @@ 
 		GROUP BY p.post_status
 	";
 
-	$cache_key = md5( $query );
+	$cache_key = md5($query);
 
-	$count = wp_cache_get( $cache_key, 'counts' );
-	if ( false !== $count ) {
+	$count = wp_cache_get($cache_key, 'counts');
+	if (false !== $count) {
 		return $count;
 	}
 
-	$count = $wpdb->get_results( $query, ARRAY_A );
+	$count = $wpdb->get_results($query, ARRAY_A);
 
 	$stats    = array();
 	$statuses = get_post_stati();
-	if ( isset( $statuses['private'] ) && empty( $args['s'] ) ) {
-		unset( $statuses['private'] );
+	if (isset($statuses['private']) && empty($args['s'])) {
+		unset($statuses['private']);
 	}
 
-	foreach ( $statuses as $state ) {
-		$stats[ $state ] = 0;
+	foreach ($statuses as $state) {
+		$stats[$state] = 0;
 	}
 
-	foreach ( (array) $count as $row ) {
+	foreach ((array) $count as $row) {
 
-		if ( 'private' == $row['post_status'] && empty( $args['s'] ) ) {
+		if ('private' == $row['post_status'] && empty($args['s'])) {
 			continue;
 		}
 
-		$stats[ $row['post_status'] ] = $row['num_posts'];
+		$stats[$row['post_status']] = $row['num_posts'];
 	}
 
 	$stats = (object) $stats;
-	wp_cache_set( $cache_key, $stats, 'counts' );
+	wp_cache_set($cache_key, $stats, 'counts');
 
 	return $stats;
 }
@@ -594,11 +594,11 @@ 
  *
  * @return bool $exists True if payment exists, false otherwise.
  */
-function give_check_for_existing_payment( $payment_id ) {
+function give_check_for_existing_payment($payment_id) {
 	$exists  = false;
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	if ( $payment_id === $payment->ID && 'publish' === $payment->status ) {
+	if ($payment_id === $payment->ID && 'publish' === $payment->status) {
 		$exists = true;
 	}
 
@@ -615,41 +615,41 @@ 
  *
  * @return bool|mixed True if payment status exists, false otherwise.
  */
-function give_get_payment_status( $payment, $return_label = false ) {
+function give_get_payment_status($payment, $return_label = false) {
 
-	if ( is_numeric( $payment ) ) {
+	if (is_numeric($payment)) {
 
-		$payment = new Give_Payment( $payment );
+		$payment = new Give_Payment($payment);
 
-		if ( ! $payment->ID > 0 ) {
+		if ( ! $payment->ID > 0) {
 			return false;
 		}
 
 	}
 
-	if ( ! is_object( $payment ) || ! isset( $payment->post_status ) ) {
+	if ( ! is_object($payment) || ! isset($payment->post_status)) {
 		return false;
 	}
 
 	$statuses = give_get_payment_statuses();
 
-	if ( ! is_array( $statuses ) || empty( $statuses ) ) {
+	if ( ! is_array($statuses) || empty($statuses)) {
 		return false;
 	}
 
 	// Get payment object if not already given.
-	$payment = $payment instanceof Give_Payment ? $payment : new Give_Payment( $payment->ID );
+	$payment = $payment instanceof Give_Payment ? $payment : new Give_Payment($payment->ID);
 
-	if ( array_key_exists( $payment->status, $statuses ) ) {
-		if ( true === $return_label ) {
+	if (array_key_exists($payment->status, $statuses)) {
+		if (true === $return_label) {
 			// Return translated status label.
-			return $statuses[ $payment->status ];
+			return $statuses[$payment->status];
 		} else {
 			// Account that our 'publish' status is labeled 'Complete'
 			$post_status = 'publish' === $payment->status ? 'Complete' : $payment->post_status;
 
 			// Make sure we're matching cases, since they matter
-			return array_search( strtolower( $post_status ), array_map( 'strtolower', $statuses ) );
+			return array_search(strtolower($post_status), array_map('strtolower', $statuses));
 		}
 	}
 
@@ -665,18 +665,18 @@ 
  */
 function give_get_payment_statuses() {
 	$payment_statuses = array(
-		'pending'     => __( 'Pending', 'give' ),
-		'publish'     => __( 'Complete', 'give' ),
-		'refunded'    => __( 'Refunded', 'give' ),
-		'failed'      => __( 'Failed', 'give' ),
-		'cancelled'   => __( 'Cancelled', 'give' ),
-		'abandoned'   => __( 'Abandoned', 'give' ),
-		'preapproval' => __( 'Pre-Approved', 'give' ),
-		'processing'  => __( 'Processing', 'give' ),
-		'revoked'     => __( 'Revoked', 'give' ),
+		'pending'     => __('Pending', 'give'),
+		'publish'     => __('Complete', 'give'),
+		'refunded'    => __('Refunded', 'give'),
+		'failed'      => __('Failed', 'give'),
+		'cancelled'   => __('Cancelled', 'give'),
+		'abandoned'   => __('Abandoned', 'give'),
+		'preapproval' => __('Pre-Approved', 'give'),
+		'processing'  => __('Processing', 'give'),
+		'revoked'     => __('Revoked', 'give'),
 	);
 
-	return apply_filters( 'give_payment_statuses', $payment_statuses );
+	return apply_filters('give_payment_statuses', $payment_statuses);
 }
 
 /**
@@ -689,10 +689,10 @@ 
  * @return array $payment_status All the available payment statuses.
  */
 function give_get_payment_status_keys() {
-	$statuses = array_keys( give_get_payment_statuses() );
-	asort( $statuses );
+	$statuses = array_keys(give_get_payment_statuses());
+	asort($statuses);
 
-	return array_values( $statuses );
+	return array_values($statuses);
 }
 
 /**
@@ -707,7 +707,7 @@ 
  *
  * @return int $earnings Earnings
  */
-function give_get_earnings_by_date( $day = null, $month_num, $year = null, $hour = null ) {
+function give_get_earnings_by_date($day = null, $month_num, $year = null, $hour = null) {
 
 	// This is getting deprecated soon. Use Give_Payment_Stats with the get_earnings() method instead.
 	global $wpdb;
@@ -717,41 +717,41 @@ 
 		'nopaging'               => true,
 		'year'                   => $year,
 		'monthnum'               => $month_num,
-		'post_status'            => array( 'publish' ),
+		'post_status'            => array('publish'),
 		'fields'                 => 'ids',
 		'update_post_term_cache' => false,
 	);
-	if ( ! empty( $day ) ) {
+	if ( ! empty($day)) {
 		$args['day'] = $day;
 	}
 
-	if ( isset( $hour ) ) {
+	if (isset($hour)) {
 		$args['hour'] = $hour;
 	}
 
-	$args = apply_filters( 'give_get_earnings_by_date_args', $args );
-	$key  = Give_Cache::get_key( 'give_stats', $args );
+	$args = apply_filters('give_get_earnings_by_date_args', $args);
+	$key  = Give_Cache::get_key('give_stats', $args);
 
-	if ( ! empty( $_GET['_wpnonce'] ) && wp_verify_nonce( $_GET['_wpnonce'], 'give-refresh-reports' ) ) {
+	if ( ! empty($_GET['_wpnonce']) && wp_verify_nonce($_GET['_wpnonce'], 'give-refresh-reports')) {
 		$earnings = false;
 	} else {
-		$earnings = Give_Cache::get( $key );
+		$earnings = Give_Cache::get($key);
 	}
 
-	if ( false === $earnings ) {
-		$donations = get_posts( $args );
+	if (false === $earnings) {
+		$donations = get_posts($args);
 		$earnings  = 0;
-		if ( $donations ) {
-			$donations = implode( ',', $donations );
+		if ($donations) {
+			$donations = implode(',', $donations);
 
-			$earnings = $wpdb->get_var( "SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN ({$donations})" );
+			$earnings = $wpdb->get_var("SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN ({$donations})");
 
 		}
 		// Cache the results for one hour.
-		Give_Cache::set( $key, $earnings, HOUR_IN_SECONDS );
+		Give_Cache::set($key, $earnings, HOUR_IN_SECONDS);
 	}
 
-	return round( $earnings, 2 );
+	return round($earnings, 2);
 }
 
 /**
@@ -766,7 +766,7 @@ 
  *
  * @return int $count Sales
  */
-function give_get_sales_by_date( $day = null, $month_num = null, $year = null, $hour = null ) {
+function give_get_sales_by_date($day = null, $month_num = null, $year = null, $hour = null) {
 
 	// This is getting deprecated soon. Use Give_Payment_Stats with the get_sales() method instead.
 	$args = array(
@@ -774,14 +774,14 @@ 
 		'nopaging'               => true,
 		'year'                   => $year,
 		'fields'                 => 'ids',
-		'post_status'            => array( 'publish' ),
+		'post_status'            => array('publish'),
 		'update_post_meta_cache' => false,
 		'update_post_term_cache' => false,
 	);
 
-	$show_free = apply_filters( 'give_sales_by_date_show_free', true, $args );
+	$show_free = apply_filters('give_sales_by_date_show_free', true, $args);
 
-	if ( false === $show_free ) {
+	if (false === $show_free) {
 		$args['meta_query'] = array(
 			array(
 				'key'     => '_give_payment_total',
@@ -792,33 +792,33 @@ 
 		);
 	}
 
-	if ( ! empty( $month_num ) ) {
+	if ( ! empty($month_num)) {
 		$args['monthnum'] = $month_num;
 	}
 
-	if ( ! empty( $day ) ) {
+	if ( ! empty($day)) {
 		$args['day'] = $day;
 	}
 
-	if ( isset( $hour ) ) {
+	if (isset($hour)) {
 		$args['hour'] = $hour;
 	}
 
-	$args = apply_filters( 'give_get_sales_by_date_args', $args );
+	$args = apply_filters('give_get_sales_by_date_args', $args);
 
-	$key = Give_Cache::get_key( 'give_stats', $args );
+	$key = Give_Cache::get_key('give_stats', $args);
 
-	if ( ! empty( $_GET['_wpnonce'] ) && wp_verify_nonce( $_GET['_wpnonce'], 'give-refresh-reports' ) ) {
+	if ( ! empty($_GET['_wpnonce']) && wp_verify_nonce($_GET['_wpnonce'], 'give-refresh-reports')) {
 		$count = false;
 	} else {
-		$count = Give_Cache::get( $key );
+		$count = Give_Cache::get($key);
 	}
 
-	if ( false === $count ) {
-		$donations = new WP_Query( $args );
+	if (false === $count) {
+		$donations = new WP_Query($args);
 		$count     = (int) $donations->post_count;
 		// Cache the results for one hour.
-		Give_Cache::set( $key, $count, HOUR_IN_SECONDS );
+		Give_Cache::set($key, $count, HOUR_IN_SECONDS);
 	}
 
 	return $count;
@@ -833,19 +833,19 @@ 
  *
  * @return bool $ret True if complete, false otherwise.
  */
-function give_is_payment_complete( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_is_payment_complete($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	$ret = false;
 
-	if ( $payment->ID > 0 ) {
+	if ($payment->ID > 0) {
 
-		if ( (int) $payment_id === (int) $payment->ID && 'publish' == $payment->status ) {
+		if ((int) $payment_id === (int) $payment->ID && 'publish' == $payment->status) {
 			$ret = true;
 		}
 	}
 
-	return apply_filters( 'give_is_payment_complete', $ret, $payment_id, $payment->post_status );
+	return apply_filters('give_is_payment_complete', $ret, $payment_id, $payment->post_status);
 }
 
 /**
@@ -871,49 +871,49 @@ 
  *
  * @return float $total Total earnings.
  */
-function give_get_total_earnings( $recalculate = false ) {
+function give_get_total_earnings($recalculate = false) {
 
-	$total = get_option( 'give_earnings_total', 0 );
+	$total = get_option('give_earnings_total', 0);
 
 	// Calculate total earnings.
-	if ( ! $total || $recalculate ) {
+	if ( ! $total || $recalculate) {
 		global $wpdb;
 
 		$total = (float) 0;
 
-		$args = apply_filters( 'give_get_total_earnings_args', array(
+		$args = apply_filters('give_get_total_earnings_args', array(
 			'offset' => 0,
-			'number' => - 1,
-			'status' => array( 'publish' ),
+			'number' => -1,
+			'status' => array('publish'),
 			'fields' => 'ids',
-		) );
+		));
 
-		$payments = give_get_payments( $args );
-		if ( $payments ) {
+		$payments = give_get_payments($args);
+		if ($payments) {
 
 			/**
 			 * If performing a donation, we need to skip the very last payment in the database,
 			 * since it calls give_increase_total_earnings() on completion,
 			 * which results in duplicated earnings for the very first donation.
 			 */
-			if ( did_action( 'give_update_payment_status' ) ) {
-				array_pop( $payments );
+			if (did_action('give_update_payment_status')) {
+				array_pop($payments);
 			}
 
-			if ( ! empty( $payments ) ) {
-				$payments = implode( ',', $payments );
-				$total    += $wpdb->get_var( "SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN({$payments})" );
+			if ( ! empty($payments)) {
+				$payments = implode(',', $payments);
+				$total += $wpdb->get_var("SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN({$payments})");
 			}
 		}
 
-		update_option( 'give_earnings_total', $total, 'no' );
+		update_option('give_earnings_total', $total, 'no');
 	}
 
-	if ( $total < 0 ) {
+	if ($total < 0) {
 		$total = 0; // Don't ever show negative earnings.
 	}
 
-	return apply_filters( 'give_total_earnings', round( $total, give_get_price_decimals() ), $total );
+	return apply_filters('give_total_earnings', round($total, give_get_price_decimals()), $total);
 }
 
 /**
@@ -925,10 +925,10 @@ 
  *
  * @return float $total Total earnings.
  */
-function give_increase_total_earnings( $amount = 0 ) {
+function give_increase_total_earnings($amount = 0) {
 	$total = give_get_total_earnings();
 	$total += $amount;
-	update_option( 'give_earnings_total', $total );
+	update_option('give_earnings_total', $total);
 
 	return $total;
 }
@@ -942,13 +942,13 @@ 
  *
  * @return float $total Total earnings.
  */
-function give_decrease_total_earnings( $amount = 0 ) {
+function give_decrease_total_earnings($amount = 0) {
 	$total = give_get_total_earnings();
 	$total -= $amount;
-	if ( $total < 0 ) {
+	if ($total < 0) {
 		$total = 0;
 	}
-	update_option( 'give_earnings_total', $total );
+	update_option('give_earnings_total', $total);
 
 	return $total;
 }
@@ -964,10 +964,10 @@ 
  *
  * @return mixed $meta Payment Meta.
  */
-function give_get_payment_meta( $payment_id = 0, $meta_key = '_give_payment_meta', $single = true ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_meta($payment_id = 0, $meta_key = '_give_payment_meta', $single = true) {
+	$payment = new Give_Payment($payment_id);
 
-	return $payment->get_meta( $meta_key, $single );
+	return $payment->get_meta($meta_key, $single);
 }
 
 /**
@@ -980,10 +980,10 @@ 
  *
  * @return mixed Meta ID if successful, false if unsuccessful.
  */
-function give_update_payment_meta( $payment_id = 0, $meta_key = '', $meta_value = '', $prev_value = '' ) {
-	$payment = new Give_Payment( $payment_id );
+function give_update_payment_meta($payment_id = 0, $meta_key = '', $meta_value = '', $prev_value = '') {
+	$payment = new Give_Payment($payment_id);
 
-	return $payment->update_meta( $meta_key, $meta_value, $prev_value );
+	return $payment->update_meta($meta_key, $meta_value, $prev_value);
 }
 
 /**
@@ -995,8 +995,8 @@ 
  *
  * @return array $user_info User Info Meta Values.
  */
-function give_get_payment_meta_user_info( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_meta_user_info($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->user_info;
 }
@@ -1012,8 +1012,8 @@ 
  *
  * @return int $form_id Form ID.
  */
-function give_get_payment_form_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_form_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->form_id;
 }
@@ -1027,8 +1027,8 @@ 
  *
  * @return string $email User email.
  */
-function give_get_payment_user_email( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_user_email($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->email;
 }
@@ -1042,11 +1042,11 @@ 
  *
  * @return bool $is_guest_payment If the payment is associated with a user (false) or not (true)
  */
-function give_is_guest_payment( $payment_id ) {
-	$payment_user_id  = give_get_payment_user_id( $payment_id );
-	$is_guest_payment = ! empty( $payment_user_id ) && $payment_user_id > 0 ? false : true;
+function give_is_guest_payment($payment_id) {
+	$payment_user_id  = give_get_payment_user_id($payment_id);
+	$is_guest_payment = ! empty($payment_user_id) && $payment_user_id > 0 ? false : true;
 
-	return (bool) apply_filters( 'give_is_guest_payment', $is_guest_payment, $payment_id );
+	return (bool) apply_filters('give_is_guest_payment', $is_guest_payment, $payment_id);
 }
 
 /**
@@ -1058,8 +1058,8 @@ 
  *
  * @return int $user_id User ID.
  */
-function give_get_payment_user_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_user_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->user_id;
 }
@@ -1073,8 +1073,8 @@ 
  *
  * @return int $payment->customer_id Donor ID.
  */
-function give_get_payment_donor_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_donor_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->customer_id;
 }
@@ -1088,8 +1088,8 @@ 
  *
  * @return string $ip User IP.
  */
-function give_get_payment_user_ip( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_user_ip($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->ip;
 }
@@ -1103,8 +1103,8 @@ 
  *
  * @return string $date The date the payment was completed.
  */
-function give_get_payment_completed_date( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_completed_date($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->completed_date;
 }
@@ -1118,8 +1118,8 @@ 
  *
  * @return string $gateway Gateway.
  */
-function give_get_payment_gateway( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_gateway($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->gateway;
 }
@@ -1133,8 +1133,8 @@ 
  *
  * @return string $currency The currency code.
  */
-function give_get_payment_currency_code( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_currency_code($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->currency;
 }
@@ -1148,10 +1148,10 @@ 
  *
  * @return string $currency The currency name.
  */
-function give_get_payment_currency( $payment_id = 0 ) {
-	$currency = give_get_payment_currency_code( $payment_id );
+function give_get_payment_currency($payment_id = 0) {
+	$currency = give_get_payment_currency_code($payment_id);
 
-	return apply_filters( 'give_payment_currency', give_get_currency_name( $currency ), $payment_id );
+	return apply_filters('give_payment_currency', give_get_currency_name($currency), $payment_id);
 }
 
 /**
@@ -1163,8 +1163,8 @@ 
  *
  * @return string $key Donation key.
  */
-function give_get_payment_key( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_key($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->key;
 }
@@ -1180,8 +1180,8 @@ 
  *
  * @return string $number Payment order number.
  */
-function give_get_payment_number( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_number($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->number;
 }
@@ -1195,23 +1195,23 @@ 
  *
  * @return string      The formatted payment number.
  */
-function give_format_payment_number( $number ) {
+function give_format_payment_number($number) {
 
-	if ( ! give_get_option( 'enable_sequential' ) ) {
+	if ( ! give_get_option('enable_sequential')) {
 		return $number;
 	}
 
-	if ( ! is_numeric( $number ) ) {
+	if ( ! is_numeric($number)) {
 		return $number;
 	}
 
-	$prefix  = give_get_option( 'sequential_prefix' );
-	$number  = absint( $number );
-	$postfix = give_get_option( 'sequential_postfix' );
+	$prefix  = give_get_option('sequential_prefix');
+	$number  = absint($number);
+	$postfix = give_get_option('sequential_postfix');
 
-	$formatted_number = $prefix . $number . $postfix;
+	$formatted_number = $prefix.$number.$postfix;
 
-	return apply_filters( 'give_format_payment_number', $formatted_number, $prefix, $number, $postfix );
+	return apply_filters('give_format_payment_number', $formatted_number, $prefix, $number, $postfix);
 }
 
 /**
@@ -1225,17 +1225,17 @@ 
  */
 function give_get_next_payment_number() {
 
-	if ( ! give_get_option( 'enable_sequential' ) ) {
+	if ( ! give_get_option('enable_sequential')) {
 		return false;
 	}
 
-	$number           = get_option( 'give_last_payment_number' );
-	$start            = give_get_option( 'sequential_start', 1 );
+	$number           = get_option('give_last_payment_number');
+	$start            = give_get_option('sequential_start', 1);
 	$increment_number = true;
 
-	if ( false !== $number ) {
+	if (false !== $number) {
 
-		if ( empty( $number ) ) {
+		if (empty($number)) {
 
 			$number           = $start;
 			$increment_number = false;
@@ -1244,24 +1244,24 @@ 
 	} else {
 
 		// This case handles the first addition of the new option, as well as if it get's deleted for any reason.
-		$payments     = new Give_Payments_Query( array(
+		$payments = new Give_Payments_Query(array(
 			'number'  => 1,
 			'order'   => 'DESC',
 			'orderby' => 'ID',
 			'output'  => 'posts',
 			'fields'  => 'ids',
-		) );
+		));
 		$last_payment = $payments->get_payments();
 
-		if ( ! empty( $last_payment ) ) {
+		if ( ! empty($last_payment)) {
 
-			$number = give_get_payment_number( $last_payment[0] );
+			$number = give_get_payment_number($last_payment[0]);
 
 		}
 
-		if ( ! empty( $number ) && $number !== (int) $last_payment[0] ) {
+		if ( ! empty($number) && $number !== (int) $last_payment[0]) {
 
-			$number = give_remove_payment_prefix_postfix( $number );
+			$number = give_remove_payment_prefix_postfix($number);
 
 		} else {
 
@@ -1270,13 +1270,13 @@ 
 		}
 	}// End if().
 
-	$increment_number = apply_filters( 'give_increment_payment_number', $increment_number, $number );
+	$increment_number = apply_filters('give_increment_payment_number', $increment_number, $number);
 
-	if ( $increment_number ) {
-		$number ++;
+	if ($increment_number) {
+		$number++;
 	}
 
-	return apply_filters( 'give_get_next_payment_number', $number );
+	return apply_filters('give_get_next_payment_number', $number);
 }
 
 /**
@@ -1288,25 +1288,25 @@ 
  *
  * @return string The new Payment number without prefix and postfix.
  */
-function give_remove_payment_prefix_postfix( $number ) {
+function give_remove_payment_prefix_postfix($number) {
 
-	$prefix  = give_get_option( 'sequential_prefix' );
-	$postfix = give_get_option( 'sequential_postfix' );
+	$prefix  = give_get_option('sequential_prefix');
+	$postfix = give_get_option('sequential_postfix');
 
 	// Remove prefix.
-	$number = preg_replace( '/' . $prefix . '/', '', $number, 1 );
+	$number = preg_replace('/'.$prefix.'/', '', $number, 1);
 
 	// Remove the postfix.
-	$length      = strlen( $number );
-	$postfix_pos = strrpos( $number, $postfix );
-	if ( false !== $postfix_pos ) {
-		$number = substr_replace( $number, '', $postfix_pos, $length );
+	$length      = strlen($number);
+	$postfix_pos = strrpos($number, $postfix);
+	if (false !== $postfix_pos) {
+		$number = substr_replace($number, '', $postfix_pos, $length);
 	}
 
 	// Ensure it's a whole number.
-	$number = intval( $number );
+	$number = intval($number);
 
-	return apply_filters( 'give_remove_payment_prefix_postfix', $number, $prefix, $postfix );
+	return apply_filters('give_remove_payment_prefix_postfix', $number, $prefix, $postfix);
 
 }
 
@@ -1322,10 +1322,10 @@ 
  *
  * @return string $amount Fully formatted payment amount.
  */
-function give_payment_amount( $payment_id = 0 ) {
-	$amount = give_get_payment_amount( $payment_id );
+function give_payment_amount($payment_id = 0) {
+	$amount = give_get_payment_amount($payment_id);
 
-	return give_currency_filter( give_format_amount( $amount, array( 'sanitize' => false ) ), give_get_payment_currency_code( $payment_id ) );
+	return give_currency_filter(give_format_amount($amount, array('sanitize' => false)), give_get_payment_currency_code($payment_id));
 }
 
 /**
@@ -1338,11 +1338,11 @@ 
  *
  * @return mixed
  */
-function give_get_payment_amount( $payment_id ) {
+function give_get_payment_amount($payment_id) {
 
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	return apply_filters( 'give_payment_amount', floatval( $payment->total ), $payment_id );
+	return apply_filters('give_payment_amount', floatval($payment->total), $payment_id);
 }
 
 /**
@@ -1359,10 +1359,10 @@ 
  *
  * @return array Fully formatted payment subtotal.
  */
-function give_payment_subtotal( $payment_id = 0 ) {
-	$subtotal = give_get_payment_subtotal( $payment_id );
+function give_payment_subtotal($payment_id = 0) {
+	$subtotal = give_get_payment_subtotal($payment_id);
 
-	return give_currency_filter( give_format_amount( $subtotal, array( 'sanitize' => false ) ), give_get_payment_currency_code( $payment_id ) );
+	return give_currency_filter(give_format_amount($subtotal, array('sanitize' => false)), give_get_payment_currency_code($payment_id));
 }
 
 /**
@@ -1376,8 +1376,8 @@ 
  *
  * @return float $subtotal Subtotal for payment (non formatted).
  */
-function give_get_payment_subtotal( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_subtotal($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->subtotal;
 }
@@ -1391,8 +1391,8 @@ 
  *
  * @return string The donation ID.
  */
-function give_get_payment_transaction_id( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_transaction_id($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->transaction_id;
 }
@@ -1407,15 +1407,15 @@ 
  *
  * @return bool|mixed
  */
-function give_set_payment_transaction_id( $payment_id = 0, $transaction_id = '' ) {
+function give_set_payment_transaction_id($payment_id = 0, $transaction_id = '') {
 
-	if ( empty( $payment_id ) || empty( $transaction_id ) ) {
+	if (empty($payment_id) || empty($transaction_id)) {
 		return false;
 	}
 
-	$transaction_id = apply_filters( 'give_set_payment_transaction_id', $transaction_id, $payment_id );
+	$transaction_id = apply_filters('give_set_payment_transaction_id', $transaction_id, $payment_id);
 
-	return give_update_payment_meta( $payment_id, '_give_payment_transaction_id', $transaction_id );
+	return give_update_payment_meta($payment_id, '_give_payment_transaction_id', $transaction_id);
 }
 
 /**
@@ -1428,12 +1428,12 @@ 
  *
  * @return int $purchase Donation ID.
  */
-function give_get_purchase_id_by_key( $key ) {
+function give_get_purchase_id_by_key($key) {
 	global $wpdb;
 
-	$purchase = $wpdb->get_var( $wpdb->prepare( "SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_purchase_key' AND meta_value = %s LIMIT 1", $key ) );
+	$purchase = $wpdb->get_var($wpdb->prepare("SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_purchase_key' AND meta_value = %s LIMIT 1", $key));
 
-	if ( $purchase != null ) {
+	if ($purchase != null) {
 		return $purchase;
 	}
 
@@ -1451,12 +1451,12 @@ 
  *
  * @return int $purchase Donation ID.
  */
-function give_get_purchase_id_by_transaction_id( $key ) {
+function give_get_purchase_id_by_transaction_id($key) {
 	global $wpdb;
 
-	$purchase = $wpdb->get_var( $wpdb->prepare( "SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_transaction_id' AND meta_value = %s LIMIT 1", $key ) );
+	$purchase = $wpdb->get_var($wpdb->prepare("SELECT post_id FROM $wpdb->postmeta WHERE meta_key = '_give_payment_transaction_id' AND meta_value = %s LIMIT 1", $key));
 
-	if ( $purchase != null ) {
+	if ($purchase != null) {
 		return $purchase;
 	}
 
@@ -1473,23 +1473,23 @@ 
  *
  * @return array $notes Donation Notes
  */
-function give_get_payment_notes( $payment_id = 0, $search = '' ) {
+function give_get_payment_notes($payment_id = 0, $search = '') {
 
-	if ( empty( $payment_id ) && empty( $search ) ) {
+	if (empty($payment_id) && empty($search)) {
 		return false;
 	}
 
-	remove_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
-	remove_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10 );
+	remove_action('pre_get_comments', 'give_hide_payment_notes', 10);
+	remove_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10);
 
-	$notes = get_comments( array(
+	$notes = get_comments(array(
 		'post_id' => $payment_id,
 		'order'   => 'ASC',
 		'search'  => $search,
-	) );
+	));
 
-	add_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
-	add_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2 );
+	add_action('pre_get_comments', 'give_hide_payment_notes', 10);
+	add_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2);
 
 	return $notes;
 }
@@ -1505,8 +1505,8 @@ 
  *
  * @return int The new note ID
  */
-function give_insert_payment_note( $payment_id = 0, $note = '' ) {
-	if ( empty( $payment_id ) ) {
+function give_insert_payment_note($payment_id = 0, $note = '') {
+	if (empty($payment_id)) {
 		return false;
 	}
 
@@ -1518,14 +1518,14 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_pre_insert_payment_note', $payment_id, $note );
+	do_action('give_pre_insert_payment_note', $payment_id, $note);
 
-	$note_id = wp_insert_comment( wp_filter_comment( array(
+	$note_id = wp_insert_comment(wp_filter_comment(array(
 		'comment_post_ID'      => $payment_id,
 		'comment_content'      => $note,
 		'user_id'              => is_admin() ? get_current_user_id() : 0,
-		'comment_date'         => current_time( 'mysql' ),
-		'comment_date_gmt'     => current_time( 'mysql', 1 ),
+		'comment_date'         => current_time('mysql'),
+		'comment_date_gmt'     => current_time('mysql', 1),
 		'comment_approved'     => 1,
 		'comment_parent'       => 0,
 		'comment_author'       => '',
@@ -1534,7 +1534,7 @@ 
 		'comment_author_email' => '',
 		'comment_type'         => 'give_payment_note',
 
-	) ) );
+	)));
 
 	/**
 	 * Fires after payment note inserted.
@@ -1545,7 +1545,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_insert_payment_note', $note_id, $payment_id, $note );
+	do_action('give_insert_payment_note', $note_id, $payment_id, $note);
 
 	return $note_id;
 }
@@ -1560,8 +1560,8 @@ 
  *
  * @return bool True on success, false otherwise.
  */
-function give_delete_payment_note( $comment_id = 0, $payment_id = 0 ) {
-	if ( empty( $comment_id ) ) {
+function give_delete_payment_note($comment_id = 0, $payment_id = 0) {
+	if (empty($comment_id)) {
 		return false;
 	}
 
@@ -1573,9 +1573,9 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_pre_delete_payment_note', $comment_id, $payment_id );
+	do_action('give_pre_delete_payment_note', $comment_id, $payment_id);
 
-	$ret = wp_delete_comment( $comment_id, true );
+	$ret = wp_delete_comment($comment_id, true);
 
 	/**
 	 * Fires after donation note deleted.
@@ -1585,7 +1585,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_post_delete_payment_note', $comment_id, $payment_id );
+	do_action('give_post_delete_payment_note', $comment_id, $payment_id);
 
 	return $ret;
 }
@@ -1600,32 +1600,32 @@ 
  *
  * @return string
  */
-function give_get_payment_note_html( $note, $payment_id = 0 ) {
+function give_get_payment_note_html($note, $payment_id = 0) {
 
-	if ( is_numeric( $note ) ) {
-		$note = get_comment( $note );
+	if (is_numeric($note)) {
+		$note = get_comment($note);
 	}
 
-	if ( ! empty( $note->user_id ) ) {
-		$user = get_userdata( $note->user_id );
+	if ( ! empty($note->user_id)) {
+		$user = get_userdata($note->user_id);
 		$user = $user->display_name;
 	} else {
-		$user = __( 'System', 'give' );
+		$user = __('System', 'give');
 	}
 
-	$date_format = give_date_format() . ', ' . get_option( 'time_format' );
+	$date_format = give_date_format().', '.get_option('time_format');
 
-	$delete_note_url = wp_nonce_url( add_query_arg( array(
+	$delete_note_url = wp_nonce_url(add_query_arg(array(
 		'give-action' => 'delete_payment_note',
 		'note_id'     => $note->comment_ID,
 		'payment_id'  => $payment_id,
-	) ), 'give_delete_payment_note_' . $note->comment_ID );
+	)), 'give_delete_payment_note_'.$note->comment_ID);
 
-	$note_html = '<div class="give-payment-note" id="give-payment-note-' . $note->comment_ID . '">';
+	$note_html = '<div class="give-payment-note" id="give-payment-note-'.$note->comment_ID.'">';
 	$note_html .= '<p>';
-	$note_html .= '<strong>' . $user . '</strong>&nbsp;&ndash;&nbsp;<span style="color:#aaa;font-style:italic;">' . date_i18n( $date_format, strtotime( $note->comment_date ) ) . '</span><br/>';
+	$note_html .= '<strong>'.$user.'</strong>&nbsp;&ndash;&nbsp;<span style="color:#aaa;font-style:italic;">'.date_i18n($date_format, strtotime($note->comment_date)).'</span><br/>';
 	$note_html .= $note->comment_content;
-	$note_html .= '&nbsp;&ndash;&nbsp;<a href="' . esc_url( $delete_note_url ) . '" class="give-delete-payment-note" data-note-id="' . absint( $note->comment_ID ) . '" data-payment-id="' . absint( $payment_id ) . '" aria-label="' . __( 'Delete this donation note.', 'give' ) . '">' . __( 'Delete', 'give' ) . '</a>';
+	$note_html .= '&nbsp;&ndash;&nbsp;<a href="'.esc_url($delete_note_url).'" class="give-delete-payment-note" data-note-id="'.absint($note->comment_ID).'" data-payment-id="'.absint($payment_id).'" aria-label="'.__('Delete this donation note.', 'give').'">'.__('Delete', 'give').'</a>';
 	$note_html .= '</p>';
 	$note_html .= '</div>';
 
@@ -1643,18 +1643,18 @@ 
  *
  * @return void
  */
-function give_hide_payment_notes( $query ) {
-	if ( version_compare( floatval( get_bloginfo( 'version' ) ), '4.1', '>=' ) ) {
-		$types = isset( $query->query_vars['type__not_in'] ) ? $query->query_vars['type__not_in'] : array();
-		if ( ! is_array( $types ) ) {
-			$types = array( $types );
+function give_hide_payment_notes($query) {
+	if (version_compare(floatval(get_bloginfo('version')), '4.1', '>=')) {
+		$types = isset($query->query_vars['type__not_in']) ? $query->query_vars['type__not_in'] : array();
+		if ( ! is_array($types)) {
+			$types = array($types);
 		}
 		$types[]                           = 'give_payment_note';
 		$query->query_vars['type__not_in'] = $types;
 	}
 }
 
-add_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
+add_action('pre_get_comments', 'give_hide_payment_notes', 10);
 
 /**
  * Exclude notes (comments) on give_payment post type from showing in Recent Comments widgets
@@ -1666,15 +1666,15 @@ 
  *
  * @return array $clauses Updated comment clauses.
  */
-function give_hide_payment_notes_pre_41( $clauses, $wp_comment_query ) {
-	if ( version_compare( floatval( get_bloginfo( 'version' ) ), '4.1', '<' ) ) {
+function give_hide_payment_notes_pre_41($clauses, $wp_comment_query) {
+	if (version_compare(floatval(get_bloginfo('version')), '4.1', '<')) {
 		$clauses['where'] .= ' AND comment_type != "give_payment_note"';
 	}
 
 	return $clauses;
 }
 
-add_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2 );
+add_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2);
 
 
 /**
@@ -1687,15 +1687,15 @@ 
  *
  * @return string $where
  */
-function give_hide_payment_notes_from_feeds( $where, $wp_comment_query ) {
+function give_hide_payment_notes_from_feeds($where, $wp_comment_query) {
 	global $wpdb;
 
-	$where .= $wpdb->prepare( ' AND comment_type != %s', 'give_payment_note' );
+	$where .= $wpdb->prepare(' AND comment_type != %s', 'give_payment_note');
 
 	return $where;
 }
 
-add_filter( 'comment_feed_where', 'give_hide_payment_notes_from_feeds', 10, 2 );
+add_filter('comment_feed_where', 'give_hide_payment_notes_from_feeds', 10, 2);
 
 
 /**
@@ -1709,32 +1709,32 @@ 
  *
  * @return array|object Array of comment counts.
  */
-function give_remove_payment_notes_in_comment_counts( $stats, $post_id ) {
+function give_remove_payment_notes_in_comment_counts($stats, $post_id) {
 	global $wpdb, $pagenow;
 
-	if ( 'index.php' != $pagenow ) {
+	if ('index.php' != $pagenow) {
 		return $stats;
 	}
 
 	$post_id = (int) $post_id;
 
-	if ( apply_filters( 'give_count_payment_notes_in_comments', false ) ) {
+	if (apply_filters('give_count_payment_notes_in_comments', false)) {
 		return $stats;
 	}
 
-	$stats = wp_cache_get( "comments-{$post_id}", 'counts' );
+	$stats = wp_cache_get("comments-{$post_id}", 'counts');
 
-	if ( false !== $stats ) {
+	if (false !== $stats) {
 		return $stats;
 	}
 
 	$where = 'WHERE comment_type != "give_payment_note"';
 
-	if ( $post_id > 0 ) {
-		$where .= $wpdb->prepare( ' AND comment_post_ID = %d', $post_id );
+	if ($post_id > 0) {
+		$where .= $wpdb->prepare(' AND comment_post_ID = %d', $post_id);
 	}
 
-	$count = $wpdb->get_results( "SELECT comment_approved, COUNT( * ) AS num_comments FROM {$wpdb->comments} {$where} GROUP BY comment_approved", ARRAY_A );
+	$count = $wpdb->get_results("SELECT comment_approved, COUNT( * ) AS num_comments FROM {$wpdb->comments} {$where} GROUP BY comment_approved", ARRAY_A);
 
 	$total    = 0;
 	$approved = array(
@@ -1744,30 +1744,30 @@ 
 		'trash'        => 'trash',
 		'post-trashed' => 'post-trashed',
 	);
-	foreach ( (array) $count as $row ) {
+	foreach ((array) $count as $row) {
 		// Don't count post-trashed toward totals.
-		if ( 'post-trashed' != $row['comment_approved'] && 'trash' != $row['comment_approved'] ) {
+		if ('post-trashed' != $row['comment_approved'] && 'trash' != $row['comment_approved']) {
 			$total += $row['num_comments'];
 		}
-		if ( isset( $approved[ $row['comment_approved'] ] ) ) {
-			$stats[ $approved[ $row['comment_approved'] ] ] = $row['num_comments'];
+		if (isset($approved[$row['comment_approved']])) {
+			$stats[$approved[$row['comment_approved']]] = $row['num_comments'];
 		}
 	}
 
 	$stats['total_comments'] = $total;
-	foreach ( $approved as $key ) {
-		if ( empty( $stats[ $key ] ) ) {
-			$stats[ $key ] = 0;
+	foreach ($approved as $key) {
+		if (empty($stats[$key])) {
+			$stats[$key] = 0;
 		}
 	}
 
 	$stats = (object) $stats;
-	wp_cache_set( "comments-{$post_id}", $stats, 'counts' );
+	wp_cache_set("comments-{$post_id}", $stats, 'counts');
 
 	return $stats;
 }
 
-add_filter( 'wp_count_comments', 'give_remove_payment_notes_in_comment_counts', 10, 2 );
+add_filter('wp_count_comments', 'give_remove_payment_notes_in_comment_counts', 10, 2);
 
 
 /**
@@ -1780,9 +1780,9 @@ 
  *
  * @return string $where Modified where clause.
  */
-function give_filter_where_older_than_week( $where = '' ) {
+function give_filter_where_older_than_week($where = '') {
 	// Payments older than one week.
-	$start = date( 'Y-m-d', strtotime( '-7 days' ) );
+	$start = date('Y-m-d', strtotime('-7 days'));
 	$where .= " AND post_date <= '{$start}'";
 
 	return $where;
@@ -1802,38 +1802,38 @@ 
  *
  * @return string $form_title Returns the full title if $only_level is false, otherwise returns the levels title.
  */
-function give_get_payment_form_title( $payment_meta, $only_level = false, $separator = '' ) {
+function give_get_payment_form_title($payment_meta, $only_level = false, $separator = '') {
 
-	$form_id    = isset( $payment_meta['form_id'] ) ? $payment_meta['form_id'] : 0;
-	$price_id   = isset( $payment_meta['price_id'] ) ? $payment_meta['price_id'] : null;
-	$form_title = isset( $payment_meta['form_title'] ) ? $payment_meta['form_title'] : '';
+	$form_id    = isset($payment_meta['form_id']) ? $payment_meta['form_id'] : 0;
+	$price_id   = isset($payment_meta['price_id']) ? $payment_meta['price_id'] : null;
+	$form_title = isset($payment_meta['form_title']) ? $payment_meta['form_title'] : '';
 
-	if ( $only_level == true ) {
+	if ($only_level == true) {
 		$form_title = '';
 	}
 
 	// If multi-level, append to the form title.
-	if ( give_has_variable_prices( $form_id ) ) {
+	if (give_has_variable_prices($form_id)) {
 
 		// Only add separator if there is a form title.
-		if ( ! empty( $form_title ) ) {
-			$form_title .= ' ' . $separator . ' ';
+		if ( ! empty($form_title)) {
+			$form_title .= ' '.$separator.' ';
 		}
 
 		$form_title .= '<span class="donation-level-text-wrap">';
 
-		if ( $price_id == 'custom' ) {
-			$custom_amount_text = give_get_meta( $form_id, '_give_custom_amount_text', true );
-			$form_title         .= ! empty( $custom_amount_text ) ? $custom_amount_text : __( 'Custom Amount', 'give' );
+		if ($price_id == 'custom') {
+			$custom_amount_text = give_get_meta($form_id, '_give_custom_amount_text', true);
+			$form_title .= ! empty($custom_amount_text) ? $custom_amount_text : __('Custom Amount', 'give');
 		} else {
-			$form_title .= give_get_price_option_name( $form_id, $price_id );
+			$form_title .= give_get_price_option_name($form_id, $price_id);
 		}
 
 		$form_title .= '</span>';
 
 	}
 
-	return apply_filters( 'give_get_payment_form_title', $form_title, $payment_meta );
+	return apply_filters('give_get_payment_form_title', $form_title, $payment_meta);
 
 }
 
@@ -1847,19 +1847,19 @@ 
  *
  * @return string $price_id
  */
-function give_get_price_id( $form_id, $price ) {
+function give_get_price_id($form_id, $price) {
 	$price_id = null;
 
-	if ( give_has_variable_prices( $form_id ) ) {
+	if (give_has_variable_prices($form_id)) {
 
-		$levels = give_get_meta( $form_id, '_give_donation_levels', true );
+		$levels = give_get_meta($form_id, '_give_donation_levels', true);
 
-		foreach ( $levels as $level ) {
+		foreach ($levels as $level) {
 
-			$level_amount = give_maybe_sanitize_amount( $level['_give_amount'] );
+			$level_amount = give_maybe_sanitize_amount($level['_give_amount']);
 
 			// Check that this indeed the recurring price.
-			if ( $level_amount == $price ) {
+			if ($level_amount == $price) {
 
 				$price_id = $level['_give_id']['level_id'];
 				break;
@@ -1867,13 +1867,13 @@ 
 			}
 		}
 
-		if( is_null( $price_id ) && give_is_custom_price_mode( $form_id ) ) {
+		if (is_null($price_id) && give_is_custom_price_mode($form_id)) {
 			$price_id = 'custom';
 		}
 	}
 
 	// Price ID must be numeric or string.
-	$price_id = ! is_numeric( $price_id ) && ! is_string( $price_id ) ? 0 : $price_id;
+	$price_id = ! is_numeric($price_id) && ! is_string($price_id) ? 0 : $price_id;
 
 	return $price_id;
 }
@@ -1891,10 +1891,10 @@ 
  *
  * @return string
  */
-function give_get_form_dropdown( $args = array(), $echo = false ) {
-	$form_dropdown_html = Give()->html->forms_dropdown( $args );
+function give_get_form_dropdown($args = array(), $echo = false) {
+	$form_dropdown_html = Give()->html->forms_dropdown($args);
 
-	if ( ! $echo ) {
+	if ( ! $echo) {
 		return $form_dropdown_html;
 	}
 
@@ -1911,17 +1911,17 @@ 
  *
  * @return string|bool
  */
-function give_get_form_variable_price_dropdown( $args = array(), $echo = false ) {
+function give_get_form_variable_price_dropdown($args = array(), $echo = false) {
 
 	// Check for give form id.
-	if ( empty( $args['id'] ) ) {
+	if (empty($args['id'])) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $args['id'] );
+	$form = new Give_Donate_Form($args['id']);
 
 	// Check if form has variable prices or not.
-	if ( ! $form->ID || ! $form->has_variable_prices() ) {
+	if ( ! $form->ID || ! $form->has_variable_prices()) {
 		return false;
 	}
 
@@ -1929,24 +1929,24 @@ 
 	$variable_price_options = array();
 
 	// Check if multi donation form support custom donation or not.
-	if ( $form->is_custom_price_mode() ) {
-		$variable_price_options['custom'] = _x( 'Custom', 'custom donation dropdown item', 'give' );
+	if ($form->is_custom_price_mode()) {
+		$variable_price_options['custom'] = _x('Custom', 'custom donation dropdown item', 'give');
 	}
 
 	// Get variable price and ID from variable price array.
-	foreach ( $variable_prices as $variable_price ) {
-		$variable_price_options[ $variable_price['_give_id']['level_id'] ] = ! empty( $variable_price['_give_text'] ) ? $variable_price['_give_text'] : give_currency_filter( give_format_amount( $variable_price['_give_amount'], array( 'sanitize' => false ) ) );
+	foreach ($variable_prices as $variable_price) {
+		$variable_price_options[$variable_price['_give_id']['level_id']] = ! empty($variable_price['_give_text']) ? $variable_price['_give_text'] : give_currency_filter(give_format_amount($variable_price['_give_amount'], array('sanitize' => false)));
 	}
 
 	// Update options.
-	$args = array_merge( $args, array(
+	$args = array_merge($args, array(
 		'options' => $variable_price_options,
-	) );
+	));
 
 	// Generate select html.
-	$form_dropdown_html = Give()->html->select( $args );
+	$form_dropdown_html = Give()->html->select($args);
 
-	if ( ! $echo ) {
+	if ( ! $echo) {
 		return $form_dropdown_html;
 	}
 
@@ -1965,16 +1965,16 @@ 
  *
  * @return string
  */
-function give_get_payment_meta_price_id( $payment_meta ) {
+function give_get_payment_meta_price_id($payment_meta) {
 
-	if ( isset( $payment_meta['give_price_id'] ) ) {
+	if (isset($payment_meta['give_price_id'])) {
 		$price_id = $payment_meta['give_price_id'];
-	} elseif ( isset( $payment_meta['price_id'] ) ) {
+	} elseif (isset($payment_meta['price_id'])) {
 		$price_id = $payment_meta['price_id'];
 	} else {
-		$price_id = give_get_price_id( $payment_meta['give_form_id'], $payment_meta['price'] );
+		$price_id = give_get_price_id($payment_meta['give_form_id'], $payment_meta['price']);
 	}
 
-	return apply_filters( 'give_get_payment_meta_price_id', $price_id );
+	return apply_filters('give_get_payment_meta_price_id', $price_id);
 
 }

give.php

Spacing +150 added lines, -150 removed lines

@@ -40,11 +40,11 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! class_exists( 'Give' ) ) :
+if ( ! class_exists('Give')) :
 
 	/**
 	 * Main Give Class
@@ -202,7 +202,7 @@ 
 		 * @return    Give
 		 */
 		public static function instance() {
-			if ( is_null( self::$_instance ) ) {
+			if (is_null(self::$_instance)) {
 				self::$_instance = new self();
 			}
 
@@ -214,13 +214,13 @@ 
 		 */
 		public function __construct() {
 			// PHP version
-			if ( ! defined( 'GIVE_REQUIRED_PHP_VERSION' ) ) {
-				define( 'GIVE_REQUIRED_PHP_VERSION', '5.3' );
+			if ( ! defined('GIVE_REQUIRED_PHP_VERSION')) {
+				define('GIVE_REQUIRED_PHP_VERSION', '5.3');
 			}
 
 			// Bailout: Need minimum php version to load plugin.
-			if ( function_exists( 'phpversion' ) && version_compare( GIVE_REQUIRED_PHP_VERSION, phpversion(), '>' ) ) {
-				add_action( 'admin_notices', array( $this, 'minmum_phpversion_notice' ) );
+			if (function_exists('phpversion') && version_compare(GIVE_REQUIRED_PHP_VERSION, phpversion(), '>')) {
+				add_action('admin_notices', array($this, 'minmum_phpversion_notice'));
 
 				return;
 			}
@@ -230,7 +230,7 @@ 
 			$this->includes();
 			$this->init_hooks();
 
-			do_action( 'give_loaded' );
+			do_action('give_loaded');
 		}
 
 		/**
@@ -239,8 +239,8 @@ 
 		 * @since  1.8.9
 		 */
 		private function init_hooks() {
-			register_activation_hook( __FILE__, 'give_install' );
-			add_action( 'plugins_loaded', array( $this, 'init' ), 0 );
+			register_activation_hook(__FILE__, 'give_install');
+			add_action('plugins_loaded', array($this, 'init'), 0);
 		}
 		/**
 		 * Init Give when WordPress Initializes.
@@ -254,7 +254,7 @@ 
 			 *
 			 * @since 1.8.9
 			 */
-			do_action( 'before_give_init' );
+			do_action('before_give_init');
 
 			// Set up localization.
 			$this->load_textdomain();
@@ -279,7 +279,7 @@ 
 			 *
 			 * @since 1.8.7
 			 */
-			do_action( 'give_init', $this );
+			do_action('give_init', $this);
 
 		}
 
@@ -296,7 +296,7 @@ 
 		 */
 		public function __clone() {
 			// Cloning instances of the class is forbidden.
-			_doing_it_wrong( __FUNCTION__, __( 'Cheatin’ huh?', 'give' ), '1.0' );
+			_doing_it_wrong(__FUNCTION__, __('Cheatin’ huh?', 'give'), '1.0');
 		}
 
 		/**
@@ -309,7 +309,7 @@ 
 		 */
 		public function __wakeup() {
 			// Unserializing instances of the class is forbidden.
-			_doing_it_wrong( __FUNCTION__, __( 'Cheatin’ huh?', 'give' ), '1.0' );
+			_doing_it_wrong(__FUNCTION__, __('Cheatin’ huh?', 'give'), '1.0');
 		}
 
 		/**
@@ -323,33 +323,33 @@ 
 		private function setup_constants() {
 
 			// Plugin version
-			if ( ! defined( 'GIVE_VERSION' ) ) {
-				define( 'GIVE_VERSION', '1.8.16' );
+			if ( ! defined('GIVE_VERSION')) {
+				define('GIVE_VERSION', '1.8.16');
 			}
 
 			// Plugin Folder Path
-			if ( ! defined( 'GIVE_PLUGIN_DIR' ) ) {
-				define( 'GIVE_PLUGIN_DIR', plugin_dir_path( __FILE__ ) );
+			if ( ! defined('GIVE_PLUGIN_DIR')) {
+				define('GIVE_PLUGIN_DIR', plugin_dir_path(__FILE__));
 			}
 
 			// Plugin Folder URL
-			if ( ! defined( 'GIVE_PLUGIN_URL' ) ) {
-				define( 'GIVE_PLUGIN_URL', plugin_dir_url( __FILE__ ) );
+			if ( ! defined('GIVE_PLUGIN_URL')) {
+				define('GIVE_PLUGIN_URL', plugin_dir_url(__FILE__));
 			}
 
 			// Plugin Basename aka: "give/give.php"
-			if ( ! defined( 'GIVE_PLUGIN_BASENAME' ) ) {
-				define( 'GIVE_PLUGIN_BASENAME', plugin_basename( __FILE__ ) );
+			if ( ! defined('GIVE_PLUGIN_BASENAME')) {
+				define('GIVE_PLUGIN_BASENAME', plugin_basename(__FILE__));
 			}
 
 			// Plugin Root File
-			if ( ! defined( 'GIVE_PLUGIN_FILE' ) ) {
-				define( 'GIVE_PLUGIN_FILE', __FILE__ );
+			if ( ! defined('GIVE_PLUGIN_FILE')) {
+				define('GIVE_PLUGIN_FILE', __FILE__);
 			}
 
 			// Make sure CAL_GREGORIAN is defined
-			if ( ! defined( 'CAL_GREGORIAN' ) ) {
-				define( 'CAL_GREGORIAN', 1 );
+			if ( ! defined('CAL_GREGORIAN')) {
+				define('CAL_GREGORIAN', 1);
 			}
 		}
 
@@ -364,132 +364,132 @@ 
 		private function includes() {
 			global $give_options;
 
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/class-admin-settings.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/class-give-settings.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/class-admin-settings.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/class-give-settings.php';
 			$give_options = give_get_settings();
 
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-cron.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/admin/give-metabox-functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-cache.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/post-types.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/scripts.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/ajax-functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/actions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/filters.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/api/class-give-api.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-notices.php';
-
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-roles.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-template-loader.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-donate-form.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-db.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-db-donors.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-db-donor-meta.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-donor.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-stats.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-session.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-html-elements.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-logging.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-license-handler.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/class-give-email-access.php';
-
-			require_once GIVE_PLUGIN_DIR . 'includes/country-functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/template-functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/misc-functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/import-functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/forms/functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/forms/template.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/forms/widget.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/shortcodes.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/formatting.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/price-functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/error-tracking.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/process-donation.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/login-register.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/user-functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/plugin-compatibility.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/deprecated/deprecated-classes.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/deprecated/deprecated-functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/deprecated/deprecated-actions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/deprecated/deprecated-filters.php';
-
-			require_once GIVE_PLUGIN_DIR . 'includes/payments/functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/payments/actions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/payments/class-payment-stats.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/payments/class-payments-query.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/payments/class-give-payment.php';
-
-			require_once GIVE_PLUGIN_DIR . 'includes/gateways/functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/gateways/actions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/gateways/paypal-standard.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/gateways/offline-donations.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/gateways/manual.php';
-
-			require_once GIVE_PLUGIN_DIR . 'includes/emails/class-give-emails.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/emails/class-give-email-tags.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/emails/functions.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/emails/template.php';
-			require_once GIVE_PLUGIN_DIR . 'includes/emails/actions.php';
-
-			require_once GIVE_PLUGIN_DIR . 'includes/donors/class-give-donors-query.php';
-
-			if ( defined( 'WP_CLI' ) && WP_CLI ) {
-				require_once GIVE_PLUGIN_DIR . 'includes/class-give-cli-commands.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-cron.php';
+			require_once GIVE_PLUGIN_DIR.'includes/admin/give-metabox-functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-cache.php';
+			require_once GIVE_PLUGIN_DIR.'includes/post-types.php';
+			require_once GIVE_PLUGIN_DIR.'includes/scripts.php';
+			require_once GIVE_PLUGIN_DIR.'includes/ajax-functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/actions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/filters.php';
+			require_once GIVE_PLUGIN_DIR.'includes/api/class-give-api.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-notices.php';
+
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-roles.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-template-loader.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-donate-form.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-db.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-db-donors.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-db-donor-meta.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-donor.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-stats.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-session.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-html-elements.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-logging.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-license-handler.php';
+			require_once GIVE_PLUGIN_DIR.'includes/class-give-email-access.php';
+
+			require_once GIVE_PLUGIN_DIR.'includes/country-functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/template-functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/misc-functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/import-functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/forms/functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/forms/template.php';
+			require_once GIVE_PLUGIN_DIR.'includes/forms/widget.php';
+			require_once GIVE_PLUGIN_DIR.'includes/shortcodes.php';
+			require_once GIVE_PLUGIN_DIR.'includes/formatting.php';
+			require_once GIVE_PLUGIN_DIR.'includes/price-functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/error-tracking.php';
+			require_once GIVE_PLUGIN_DIR.'includes/process-donation.php';
+			require_once GIVE_PLUGIN_DIR.'includes/login-register.php';
+			require_once GIVE_PLUGIN_DIR.'includes/user-functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/plugin-compatibility.php';
+			require_once GIVE_PLUGIN_DIR.'includes/deprecated/deprecated-classes.php';
+			require_once GIVE_PLUGIN_DIR.'includes/deprecated/deprecated-functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/deprecated/deprecated-actions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/deprecated/deprecated-filters.php';
+
+			require_once GIVE_PLUGIN_DIR.'includes/payments/functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/payments/actions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/payments/class-payment-stats.php';
+			require_once GIVE_PLUGIN_DIR.'includes/payments/class-payments-query.php';
+			require_once GIVE_PLUGIN_DIR.'includes/payments/class-give-payment.php';
+
+			require_once GIVE_PLUGIN_DIR.'includes/gateways/functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/gateways/actions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/gateways/paypal-standard.php';
+			require_once GIVE_PLUGIN_DIR.'includes/gateways/offline-donations.php';
+			require_once GIVE_PLUGIN_DIR.'includes/gateways/manual.php';
+
+			require_once GIVE_PLUGIN_DIR.'includes/emails/class-give-emails.php';
+			require_once GIVE_PLUGIN_DIR.'includes/emails/class-give-email-tags.php';
+			require_once GIVE_PLUGIN_DIR.'includes/emails/functions.php';
+			require_once GIVE_PLUGIN_DIR.'includes/emails/template.php';
+			require_once GIVE_PLUGIN_DIR.'includes/emails/actions.php';
+
+			require_once GIVE_PLUGIN_DIR.'includes/donors/class-give-donors-query.php';
+
+			if (defined('WP_CLI') && WP_CLI) {
+				require_once GIVE_PLUGIN_DIR.'includes/class-give-cli-commands.php';
 			}
 
-			if ( is_admin() || ( defined( 'WP_CLI' ) && WP_CLI ) ) {
+			if (is_admin() || (defined('WP_CLI') && WP_CLI)) {
 
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/admin-footer.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/welcome.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/admin-pages.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/class-api-keys-table.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/class-i18n-module.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/admin-actions.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/admin-filters.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/add-ons.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/plugins.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/dashboard-widgets.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/class-blank-slate.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/admin-footer.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/welcome.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/admin-pages.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/class-api-keys-table.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/class-i18n-module.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/admin-actions.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/admin-filters.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/add-ons.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/plugins.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/dashboard-widgets.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/class-blank-slate.php';
 
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/payments/actions.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/payments/payments-history.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/payments/actions.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/payments/payments-history.php';
 
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/donors/donors.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/donors/donor-functions.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/donors/donor-actions.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/donors/donors.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/donors/donor-functions.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/donors/donor-actions.php';
 
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/forms/metabox.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/forms/class-metabox-form-data.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/forms/dashboard-columns.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/forms/metabox.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/forms/class-metabox-form-data.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/forms/dashboard-columns.php';
 
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/export-functions.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/class-export.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/export-actions.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/export/pdf-reports.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/tools/export/export-functions.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/tools/export/class-export.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/tools/export/export-actions.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/tools/export/pdf-reports.php';
 
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/reports/reports.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/reports/class-give-graph.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/reports/graphing.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/reports/reports.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/reports/class-give-graph.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/reports/graphing.php';
 
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/logs/logs.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/tools/logs/logs.php';
 
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/tools/data/tools-actions.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/tools/data/tools-actions.php';
 
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/abstract-shortcode-generator.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/class-shortcode-button.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-form.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-goal.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-login.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-register.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-profile-editor.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-donation-history.php';
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/shortcodes/shortcode-give-receipt.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/shortcodes/abstract-shortcode-generator.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/shortcodes/class-shortcode-button.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/shortcodes/shortcode-give-form.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/shortcodes/shortcode-give-goal.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/shortcodes/shortcode-give-login.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/shortcodes/shortcode-give-register.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/shortcodes/shortcode-give-profile-editor.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/shortcodes/shortcode-give-donation-history.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/shortcodes/shortcode-give-receipt.php';
 
-				require_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/class-give-updates.php';
+				require_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/class-give-updates.php';
 
 			}// End if().
 
-			require_once GIVE_PLUGIN_DIR . 'includes/install.php';
+			require_once GIVE_PLUGIN_DIR.'includes/install.php';
 
 		}
 
@@ -504,16 +504,16 @@ 
 		public function load_textdomain() {
 
 			// Set filter for Give's languages directory
-			$give_lang_dir = dirname( plugin_basename( GIVE_PLUGIN_FILE ) ) . '/languages/';
-			$give_lang_dir = apply_filters( 'give_languages_directory', $give_lang_dir );
+			$give_lang_dir = dirname(plugin_basename(GIVE_PLUGIN_FILE)).'/languages/';
+			$give_lang_dir = apply_filters('give_languages_directory', $give_lang_dir);
 
 			// Traditional WordPress plugin locale filter.
-			$locale = is_admin() && function_exists( 'get_user_locale' ) ? get_user_locale() : get_locale();
-			$locale = apply_filters( 'plugin_locale', $locale, 'give' );
+			$locale = is_admin() && function_exists('get_user_locale') ? get_user_locale() : get_locale();
+			$locale = apply_filters('plugin_locale', $locale, 'give');
 
-			unload_textdomain( 'give' );
-			load_textdomain( 'give', WP_LANG_DIR . '/give/give-' . $locale . '.mo' );
-			load_plugin_textdomain( 'give', false, $give_lang_dir );
+			unload_textdomain('give');
+			load_textdomain('give', WP_LANG_DIR.'/give/give-'.$locale.'.mo');
+			load_plugin_textdomain('give', false, $give_lang_dir);
 
 		}
 
@@ -526,17 +526,17 @@ 
 		 */
 		public function minmum_phpversion_notice() {
 			// Bailout.
-			if ( ! is_admin() ) {
+			if ( ! is_admin()) {
 				return;
 			}
 
-			$notice_desc = '<p><strong>' . __( 'Your site could be faster and more secure with a newer PHP version.', 'give' ) . '</strong></p>';
-			$notice_desc .= '<p>' . __( 'Hey, we\'ve noticed that you\'re running an outdated version of PHP. PHP is the programming language that WordPress and Give are built on. The version that is currently used for your site is no longer supported. Newer versions of PHP are both faster and more secure. In fact, your version of PHP no longer receives security updates, which is why we\'re sending you this notice.', 'give' ) . '</p>';
-			$notice_desc .= '<p>' . __( 'Hosts have the ability to update your PHP version, but sometimes they don\'t dare to do that because they\'re afraid they\'ll break your site.', 'give' ) . '</p>';
-			$notice_desc .= '<p><strong>' . __( 'To which version should I update?', 'give' ) . '</strong></p>';
-			$notice_desc .= '<p>' . __( 'You should update your PHP version to either 5.6 or to 7.0 or 7.1. On a normal WordPress site, switching to PHP 5.6 should never cause issues. We would however actually recommend you switch to PHP7. There are some plugins that are not ready for PHP7 though, so do some testing first. PHP7 is much faster than PHP 5.6. It\'s also the only PHP version still in active development and therefore the better option for your site in the long run.', 'give' ) . '</p>';
-			$notice_desc .= '<p><strong>' . __( 'Can\'t update? Ask your host!', 'give' ) . '</strong></p>';
-			$notice_desc .= '<p>' . sprintf( __( 'If you cannot upgrade your PHP version yourself, you can send an email to your host. If they don\'t want to upgrade your PHP version, we would suggest you switch hosts. Have a look at one of the recommended %1$sWordPress hosting partners%2$s.', 'give' ), sprintf( '<a href="%1$s" target="_blank">', esc_url( 'https://wordpress.org/hosting/' ) ), '</a>' ) . '</p>';
+			$notice_desc = '<p><strong>'.__('Your site could be faster and more secure with a newer PHP version.', 'give').'</strong></p>';
+			$notice_desc .= '<p>'.__('Hey, we\'ve noticed that you\'re running an outdated version of PHP. PHP is the programming language that WordPress and Give are built on. The version that is currently used for your site is no longer supported. Newer versions of PHP are both faster and more secure. In fact, your version of PHP no longer receives security updates, which is why we\'re sending you this notice.', 'give').'</p>';
+			$notice_desc .= '<p>'.__('Hosts have the ability to update your PHP version, but sometimes they don\'t dare to do that because they\'re afraid they\'ll break your site.', 'give').'</p>';
+			$notice_desc .= '<p><strong>'.__('To which version should I update?', 'give').'</strong></p>';
+			$notice_desc .= '<p>'.__('You should update your PHP version to either 5.6 or to 7.0 or 7.1. On a normal WordPress site, switching to PHP 5.6 should never cause issues. We would however actually recommend you switch to PHP7. There are some plugins that are not ready for PHP7 though, so do some testing first. PHP7 is much faster than PHP 5.6. It\'s also the only PHP version still in active development and therefore the better option for your site in the long run.', 'give').'</p>';
+			$notice_desc .= '<p><strong>'.__('Can\'t update? Ask your host!', 'give').'</strong></p>';
+			$notice_desc .= '<p>'.sprintf(__('If you cannot upgrade your PHP version yourself, you can send an email to your host. If they don\'t want to upgrade your PHP version, we would suggest you switch hosts. Have a look at one of the recommended %1$sWordPress hosting partners%2$s.', 'give'), sprintf('<a href="%1$s" target="_blank">', esc_url('https://wordpress.org/hosting/')), '</a>').'</p>';
 
 			echo sprintf(
 				'<div class="notice notice-error">%1$s</div>',

templates/email-login-form.php

Spacing +37 added lines, -37 removed lines

@@ -7,24 +7,24 @@ 
 
 global $give_access_form_outputted;
 $show_form = true;
-$email     = isset( $_POST['give_email'] ) ? $_POST['give_email'] : '';
+$email     = isset($_POST['give_email']) ? $_POST['give_email'] : '';
 
 // Declare Variables.
-$recaptcha_key    = give_get_option( 'recaptcha_key' );
-$recaptcha_secret = give_get_option( 'recaptcha_secret' );
-$enable_recaptcha = ( ! empty( $recaptcha_key ) && ! empty( $recaptcha_secret ) ) ? true : false;
-$access_token     = ! empty( $_GET['payment_key'] ) ? $_GET['payment_key'] : '';
+$recaptcha_key    = give_get_option('recaptcha_key');
+$recaptcha_secret = give_get_option('recaptcha_secret');
+$enable_recaptcha = ( ! empty($recaptcha_key) && ! empty($recaptcha_secret)) ? true : false;
+$access_token     = ! empty($_GET['payment_key']) ? $_GET['payment_key'] : '';
 
 // Only output the form once.
-if ( $give_access_form_outputted ) {
+if ($give_access_form_outputted) {
 	return;
 }
 
 // Form submission.
-if ( is_email( $email ) && wp_verify_nonce( $_POST['_wpnonce'], 'give' ) ) {
+if (is_email($email) && wp_verify_nonce($_POST['_wpnonce'], 'give')) {
 
 	// Use reCAPTCHA
-	if ( $enable_recaptcha ) {
+	if ($enable_recaptcha) {
 
 		$args = array(
 			'secret'   => $recaptcha_secret,
@@ -32,87 +32,87 @@ 
 			'remoteip' => $_POST['give_ip'],
 		);
 
-		if ( ! empty( $args['response'] ) ) {
-			$request = wp_remote_post( 'https://www.google.com/recaptcha/api/siteverify', array(
+		if ( ! empty($args['response'])) {
+			$request = wp_remote_post('https://www.google.com/recaptcha/api/siteverify', array(
 				'body' => $args,
-			) );
-			if ( ! is_wp_error( $request ) || 200 == wp_remote_retrieve_response_code( $request ) ) {
+			));
+			if ( ! is_wp_error($request) || 200 == wp_remote_retrieve_response_code($request)) {
 
-				$response = json_decode( $request['body'], true );
+				$response = json_decode($request['body'], true);
 
 				// reCAPTCHA fail.
-				if ( ! $response['success'] ) {
-					give_set_error( 'give_recaptcha_test_failed', apply_filters( 'give_recaptcha_test_failed_message', __( 'reCAPTCHA test failed.', 'give' ) ) );
+				if ( ! $response['success']) {
+					give_set_error('give_recaptcha_test_failed', apply_filters('give_recaptcha_test_failed_message', __('reCAPTCHA test failed.', 'give')));
 				}
 			} else {
 
 				// Connection issue.
-				give_set_error( 'give_recaptcha_connection_issue', apply_filters( 'give_recaptcha_connection_issue_message', __( 'Unable to connect to reCAPTCHA server.', 'give' ) ) );
+				give_set_error('give_recaptcha_connection_issue', apply_filters('give_recaptcha_connection_issue_message', __('Unable to connect to reCAPTCHA server.', 'give')));
 
 			}
 		} // End if().
 		else {
 
-			give_set_error( 'give_recaptcha_failed', apply_filters( 'give_recaptcha_failed_message', __( 'It looks like the reCAPTCHA test has failed.', 'give' ) ) );
+			give_set_error('give_recaptcha_failed', apply_filters('give_recaptcha_failed_message', __('It looks like the reCAPTCHA test has failed.', 'give')));
 
 		}
 	}
 
 	// If no errors or only expired token key error - then send email.
-	if ( ! give_get_errors() ) {
+	if ( ! give_get_errors()) {
 
 		$payment_ids   = array();
 		$payment_match = false;
 
-		$donor = Give()->donors->get_donor_by( 'email', $email );
+		$donor = Give()->donors->get_donor_by('email', $email);
 
-		if( ! empty( $donor->payment_ids ) ) {
-			$payment_ids = explode( ',', $donor->payment_ids );
+		if ( ! empty($donor->payment_ids)) {
+			$payment_ids = explode(',', $donor->payment_ids);
 		}
 
-		foreach( $payment_ids AS $payment_id ) {
-			$payment = new Give_Payment( $payment_id );
+		foreach ($payment_ids AS $payment_id) {
+			$payment = new Give_Payment($payment_id);
 
 			// Make sure Donation Access Token matches with donation details of donor whose email is provided.
-			if ( $access_token === $payment->key ) {
+			if ($access_token === $payment->key) {
 				$payment_match = true;
 			}
 
 		}
 
-		if ( ! $payment_match ) {
-			give_set_error( 'give_email_access_token_not_match',  __( 'It looks like that email address provided and access token of the link does not match.', 'give' ) );
+		if ( ! $payment_match) {
+			give_set_error('give_email_access_token_not_match', __('It looks like that email address provided and access token of the link does not match.', 'give'));
 
 		} else {
 			// Set Verification for Access.
-			Give()->email_access->set_verify_key( $donor->id, $donor->email, $access_token );
+			Give()->email_access->set_verify_key($donor->id, $donor->email, $access_token);
 
-			wp_safe_redirect( esc_url( get_permalink( give_get_option( 'history_page' ) ) . '?give_nl=' . $access_token ) );
+			wp_safe_redirect(esc_url(get_permalink(give_get_option('history_page')).'?give_nl='.$access_token));
 		}
 
 	}
 } // End if().
 
 // Print any messages & errors.
-Give()->notices->render_frontend_notices( 0 );
+Give()->notices->render_frontend_notices(0);
 
 // Show the email login form?
-if ( $show_form ) { ?>
+if ($show_form) { ?>
 	<div class="give-form">
 
 		<?php
-		if ( ! give_get_errors() ) {
-			Give()->notices->print_frontend_notice( apply_filters( 'give_email_access_message', __( 'Please enter the email address you used for your donation.', 'give' ) ), true );
+		if ( ! give_get_errors()) {
+			Give()->notices->print_frontend_notice(apply_filters('give_email_access_message', __('Please enter the email address you used for your donation.', 'give')), true);
 		} ?>
 
 		<form method="post" action="" id="give-email-access-form">
-			<label for="give-email"><?php _e( 'Donation Email:', 'give' ); ?></label>
-			<input id="give-email" type="email" name="give_email" value="" placeholder="<?php _e( 'Email Address', 'give' ); ?>" />
-			<input type="hidden" name="_wpnonce" value="<?php echo wp_create_nonce( 'give' ); ?>" />
+			<label for="give-email"><?php _e('Donation Email:', 'give'); ?></label>
+			<input id="give-email" type="email" name="give_email" value="" placeholder="<?php _e('Email Address', 'give'); ?>" />
+			<input type="hidden" name="_wpnonce" value="<?php echo wp_create_nonce('give'); ?>" />
 
 			<?php
 			// Enable reCAPTCHA?
-			if ( $enable_recaptcha ) { ?>
+			if ($enable_recaptcha) { ?>
 
 				<script>
 					// IP verify for reCAPTCHA.
@@ -130,7 +130,7 @@ 
 				<input type="hidden" name="give_ip" class="give_ip" value="" />
 			<?php } ?>
 
-			<input type="submit" class="give-submit" value="<?php _e( 'Verify Email', 'give' ); ?>" />
+			<input type="submit" class="give-submit" value="<?php _e('Verify Email', 'give'); ?>" />
 		</form>
 	</div>
 

templates/history-donations.php

Spacing +51 added lines, -51 removed lines

@@ -3,33 +3,33 @@ 
  * This template is used to display the donation history of the current user.
  */
 
-$donation_history_args = Give()->session->get( 'give_donation_history_args' );
+$donation_history_args = Give()->session->get('give_donation_history_args');
 
 $donations = array();
 
 // User's Donations.
-if ( is_user_logged_in() ) {
-	$donations = give_get_users_donations( get_current_user_id(), 20, true, 'any' );
-} elseif ( Give()->email_access->token_exists ) {
+if (is_user_logged_in()) {
+	$donations = give_get_users_donations(get_current_user_id(), 20, true, 'any');
+} elseif (Give()->email_access->token_exists) {
 	// Email Access Token?
-	$donations = give_get_users_donations( 0, 20, true, 'any' );
-} elseif ( Give()->session->get_session_expiration() !== false ) {
+	$donations = give_get_users_donations(0, 20, true, 'any');
+} elseif (Give()->session->get_session_expiration() !== false) {
 	// Session active?
-	$email     = Give()->session->get( 'give_email' );
-	$donations = give_get_users_donations( $email, 20, true, 'any' );
+	$email     = Give()->session->get('give_email');
+	$donations = give_get_users_donations($email, 20, true, 'any');
 }
 
 
-if ( $donations ) : ?>
+if ($donations) : ?>
 	<?php
 	$table_headings = array(
-		'id'             => __( 'ID', 'give' ),
-		'date'           => __( 'Date', 'give' ),
-		'donor'          => __( 'Donor', 'give' ),
-		'amount'         => __( 'Amount', 'give' ),
-		'status'         => __( 'Status', 'give' ),
-		'payment_method' => __( 'Payment Method', 'give' ),
-		'details'        => __( 'Details', 'give' ),
+		'id'             => __('ID', 'give'),
+		'date'           => __('Date', 'give'),
+		'donor'          => __('Donor', 'give'),
+		'amount'         => __('Amount', 'give'),
+		'status'         => __('Status', 'give'),
+		'payment_method' => __('Payment Method', 'give'),
+		'details'        => __('Details', 'give'),
 	);
 	?>
 	<table id="give_user_history" class="give-table">
@@ -43,14 +43,14 @@ 
 				 *
 				 * @since 1.7
 				 */
-				do_action( 'give_donation_history_header_before' );
+				do_action('give_donation_history_header_before');
 
-				foreach ( $donation_history_args as $index => $value ) {
-					if ( filter_var( $donation_history_args[ $index ], FILTER_VALIDATE_BOOLEAN ) ) :
+				foreach ($donation_history_args as $index => $value) {
+					if (filter_var($donation_history_args[$index], FILTER_VALIDATE_BOOLEAN)) :
 						echo sprintf(
 							'<th scope="col" class="give-donation-%1$s>">%2$s</th>',
 							$index,
-							$table_headings[ $index ]
+							$table_headings[$index]
 						);
 					endif;
 				}
@@ -62,13 +62,13 @@ 
 				 *
 				 * @since 1.7
 				 */
-				do_action( 'give_donation_history_header_after' );
+				do_action('give_donation_history_header_after');
 				?>
 			</tr>
 		</thead>
-		<?php foreach ( $donations as $post ) :
-			setup_postdata( $post );
-			$donation_data = give_get_payment_meta( $post->ID ); ?>
+		<?php foreach ($donations as $post) :
+			setup_postdata($post);
+			$donation_data = give_get_payment_meta($post->ID); ?>
 			<tr class="give-donation-row">
 				<?php
 				/**
@@ -81,37 +81,37 @@ 
 				 * @param int   $post_id       The ID of the post.
 				 * @param mixed $donation_data Payment meta data.
 				 */
-				do_action( 'give_donation_history_row_start', $post->ID, $donation_data );
+				do_action('give_donation_history_row_start', $post->ID, $donation_data);
 
-				if ( filter_var( $donation_history_args['id'], FILTER_VALIDATE_BOOLEAN ) ) :
+				if (filter_var($donation_history_args['id'], FILTER_VALIDATE_BOOLEAN)) :
 					echo sprintf(
 						'<td class="give-donation-id">#%1$s</td>',
-						give_get_payment_number( $post->ID )
+						give_get_payment_number($post->ID)
 					);
 				endif;
 
-				if ( filter_var( $donation_history_args['date'], FILTER_VALIDATE_BOOLEAN ) ) :
+				if (filter_var($donation_history_args['date'], FILTER_VALIDATE_BOOLEAN)) :
 					echo sprintf(
 						'<td class="give-donation-date">%1$s</td>',
-						date_i18n( give_date_format(), strtotime( get_post_field( 'post_date', $post->ID ) ) )
+						date_i18n(give_date_format(), strtotime(get_post_field('post_date', $post->ID)))
 					);
 				endif;
 
-				if ( filter_var( $donation_history_args['donor'], FILTER_VALIDATE_BOOLEAN ) ) :
+				if (filter_var($donation_history_args['donor'], FILTER_VALIDATE_BOOLEAN)) :
 					echo sprintf(
 						'<td class="give-donation-donor">%1$s</td>',
-						give_get_donor_name_by( $post->ID )
+						give_get_donor_name_by($post->ID)
 					);
 				endif;
 				?>
 
-				<?php if ( filter_var( $donation_history_args['amount'], FILTER_VALIDATE_BOOLEAN ) ) : ?>
+				<?php if (filter_var($donation_history_args['amount'], FILTER_VALIDATE_BOOLEAN)) : ?>
 					<td class="give-donation-amount">
 						<span class="give-donation-amount">
 							<?php
-							$currency_code   = give_get_payment_currency_code( $post->ID );
+							$currency_code   = give_get_payment_currency_code($post->ID);
 							$donation_amount = give_currency_filter(
-								give_format_amount( give_get_payment_amount( $post->ID ), array(
+								give_format_amount(give_get_payment_amount($post->ID), array(
 									'sanitize'    => false,
 									'currency'    => $currency_code,
 								)),
@@ -128,42 +128,42 @@ 
 							 *
 							 * @return int
 							 */
-							echo apply_filters( 'give_donation_history_row_amount', $donation_amount, $post->ID );
+							echo apply_filters('give_donation_history_row_amount', $donation_amount, $post->ID);
 							?>
 						</span>
 					</td>
 				<?php endif; ?>
 
 				<?php
-				if ( filter_var( $donation_history_args['status'], FILTER_VALIDATE_BOOLEAN ) ) :
+				if (filter_var($donation_history_args['status'], FILTER_VALIDATE_BOOLEAN)) :
 					echo sprintf(
 						'<td class="give-donation-status">%1$s</td>',
-						give_get_payment_status( $post, true )
+						give_get_payment_status($post, true)
 					);
 				endif;
 
-				if ( filter_var( $donation_history_args['payment_method'], FILTER_VALIDATE_BOOLEAN ) ) :
+				if (filter_var($donation_history_args['payment_method'], FILTER_VALIDATE_BOOLEAN)) :
 					echo sprintf(
 						'<td class="give-donation-payment-method">%1$s</td>',
-						give_get_gateway_checkout_label( give_get_payment_gateway( $post->ID ) )
+						give_get_gateway_checkout_label(give_get_payment_gateway($post->ID))
 					);
 				endif;
 				?>
 				<td class="give-donation-details">
 					<?php
 					// Display View Receipt or.
-					if ( 'publish' !== $post->post_status && 'subscription' !== $post->post_status ) :
+					if ('publish' !== $post->post_status && 'subscription' !== $post->post_status) :
 						echo sprintf(
 							'<a href="%1$s"><span class="give-donation-status %2$s">%3$s</span></a>',
 							esc_url(
 								add_query_arg(
 									'payment_key',
-									give_get_payment_key( $post->ID ),
+									give_get_payment_key($post->ID),
 									give_get_history_page_uri()
 								)
 							),
 							$post->post_status,
-							__( 'View', 'give' ) . ' ' . give_get_payment_status( $post, true ) . ' &raquo;'
+							__('View', 'give').' '.give_get_payment_status($post, true).' &raquo;'
 						);
 
 					else :
@@ -172,11 +172,11 @@ 
 							esc_url(
 								add_query_arg(
 									'payment_key',
-									give_get_payment_key( $post->ID ),
+									give_get_payment_key($post->ID),
 									give_get_history_page_uri()
 								)
 							),
-							__( 'View Receipt &raquo;', 'give' )
+							__('View Receipt &raquo;', 'give')
 						);
 
 					endif;
@@ -193,7 +193,7 @@ 
 				 * @param int   $post_id       The ID of the post.
 				 * @param mixed $donation_data Payment meta data.
 				 */
-				do_action( 'give_donation_history_row_end', $post->ID, $donation_data );
+				do_action('give_donation_history_row_end', $post->ID, $donation_data);
 				?>
 			</tr>
 		<?php endforeach; ?>
@@ -201,15 +201,15 @@ 
 	<div id="give-donation-history-pagination" class="give_pagination navigation">
 		<?php
 		$big = 999999;
-		echo paginate_links( array(
-			'base'    => str_replace( $big, '%#%', esc_url( get_pagenum_link( $big ) ) ),
+		echo paginate_links(array(
+			'base'    => str_replace($big, '%#%', esc_url(get_pagenum_link($big))),
 			'format'  => '?paged=%#%',
-			'current' => max( 1, get_query_var( 'paged' ) ),
-			'total'   => ceil( give_count_donations_of_donor() / 20 ) // 20 items per page
-		) );
+			'current' => max(1, get_query_var('paged')),
+			'total'   => ceil(give_count_donations_of_donor() / 20) // 20 items per page
+		));
 		?>
 	</div>
 	<?php wp_reset_postdata(); ?>
 <?php else : ?>
-	<?php Give()->notices->print_frontend_notice( __( 'It looks like you haven\'t made any donations.', 'give' ), true, 'success' ); ?>
+	<?php Give()->notices->print_frontend_notice(__('It looks like you haven\'t made any donations.', 'give'), true, 'success'); ?>
 <?php endif;

includes/class-notices.php

Spacing +96 added lines, -96 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,29 +386,29 @@ 
 	 * @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);
 
 		wp_send_json_success();
 	}
@@ -426,18 +426,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;
 	}
@@ -450,11 +450,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,
@@ -481,31 +481,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;
 	}
@@ -519,9 +519,9 @@ 
 	 *
 	 * @param array $errors
 	 */
-	public static function print_frontend_errors( $errors ) {
+	public static function print_frontend_errors($errors) {
 		// Bailout.
-		if ( ! $errors ) {
+		if ( ! $errors) {
 			return;
 		}
 
@@ -536,37 +536,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']
 			);
 		}
@@ -588,8 +588,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 '';
 		}
 
@@ -603,14 +603,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(
@@ -620,12 +620,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;
 		}
 

includes/admin/class-addon-activation-banner.php

Spacing +49 added lines, -49 removed lines

@@ -8,7 +8,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -32,14 +32,14 @@ 
 	 *                               'testing'           => false, // (required) Never leave as "true" in production!!!
 	 *                               }
 	 */
-	function __construct( $_banner_details ) {
+	function __construct($_banner_details) {
 		$current_user = wp_get_current_user();
 
 		$this->plugin_activate_by   = 0;
 		$this->banner_details       = $_banner_details;
-		$this->test_mode            = ( $this->banner_details['testing'] == 'true' ) ? true : false;
-		$this->nag_meta_key         = 'give_addon_activation_ignore_' . sanitize_title( $this->banner_details['name'] );
-		$this->activate_by_meta_key = 'give_addon_' . sanitize_title( $this->banner_details['name'] ) . '_active_by_user';
+		$this->test_mode            = ($this->banner_details['testing'] == 'true') ? true : false;
+		$this->nag_meta_key         = 'give_addon_activation_ignore_'.sanitize_title($this->banner_details['name']);
+		$this->activate_by_meta_key = 'give_addon_'.sanitize_title($this->banner_details['name']).'_active_by_user';
 
 		//Get current user
 		$this->user_id = $current_user->ID;
@@ -62,19 +62,19 @@ 
 	public function init() {
 
 		//Testing?
-		if ( $this->test_mode ) {
-			delete_user_meta( $this->user_id, $this->nag_meta_key );
+		if ($this->test_mode) {
+			delete_user_meta($this->user_id, $this->nag_meta_key);
 		}
 
 		//Get the current page to add the notice to
-		add_action( 'current_screen', array( $this, 'give_addon_notice_ignore' ) );
-		add_action( 'admin_notices', array( $this, 'give_addon_activation_admin_notice' ) );
+		add_action('current_screen', array($this, 'give_addon_notice_ignore'));
+		add_action('admin_notices', array($this, 'give_addon_activation_admin_notice'));
 
 		// File path of addon must be included in banner detail other addon activate meta will not delete.
 		$file_name = $this->get_plugin_file_name();
 
-		if ( ! empty( $file_name ) ) {
-			add_action( 'deactivate_' . $file_name, array( $this, 'remove_addon_activate_meta' ) );
+		if ( ! empty($file_name)) {
+			add_action('deactivate_'.$file_name, array($this, 'remove_addon_activate_meta'));
 		}
 	}
 
@@ -89,7 +89,7 @@ 
 	private function is_plugin_page() {
 		$screen = get_current_screen();
 
-		return ( $screen->parent_file === 'plugins.php' );
+		return ($screen->parent_file === 'plugins.php');
 	}
 
 
@@ -102,12 +102,12 @@ 
 	public function give_addon_activation_admin_notice() {
 
 		// Bailout.
-		if ( ! $this->is_plugin_page() || $this->user_id !== $this->plugin_activate_by ) {
+		if ( ! $this->is_plugin_page() || $this->user_id !== $this->plugin_activate_by) {
 			return;
 		}
 
 		// If the user hasn't already dismissed the alert, output activation banner.
-		if ( ! get_user_meta( $this->user_id, $this->nag_meta_key ) ) {
+		if ( ! get_user_meta($this->user_id, $this->nag_meta_key)) {
 
 			$this->print_css();
 
@@ -125,44 +125,44 @@ 
 					<h3><?php
 						printf(
 						/* translators: %s: Add-on name */
-							esc_html__( "Thank you for installing Give's %s Add-on!", 'give' ),
-							'<span>' . $this->banner_details['name'] . '</span>'
+							esc_html__("Thank you for installing Give's %s Add-on!", 'give'),
+							'<span>'.$this->banner_details['name'].'</span>'
 						);
 						?></h3>
 
 					<a href="<?php
 					//The Dismiss Button.
-					$nag_admin_dismiss_url = 'plugins.php?' . $this->nag_meta_key . '=0';
-					echo admin_url( $nag_admin_dismiss_url ); ?>" class="dismiss"><span
+					$nag_admin_dismiss_url = 'plugins.php?'.$this->nag_meta_key.'=0';
+					echo admin_url($nag_admin_dismiss_url); ?>" class="dismiss"><span
 							class="dashicons dashicons-dismiss"></span></a>
 
 					<div class="alert-actions">
 
 						<?php //Point them to your settings page.
-						if ( isset( $this->banner_details['settings_url'] ) ) { ?>
+						if (isset($this->banner_details['settings_url'])) { ?>
 							<a href="<?php echo $this->banner_details['settings_url']; ?>">
-								<span class="dashicons dashicons-admin-settings"></span><?php esc_html_e( 'Go to Settings', 'give' ); ?>
+								<span class="dashicons dashicons-admin-settings"></span><?php esc_html_e('Go to Settings', 'give'); ?>
 							</a>
 						<?php } ?>
 
 						<?php
 						// Show them how to configure the Addon.
-						if ( isset( $this->banner_details['documentation_url'] ) ) { ?>
+						if (isset($this->banner_details['documentation_url'])) { ?>
 							<a href="<?php echo $this->banner_details['documentation_url'] ?>" target="_blank">
 								<span class="dashicons dashicons-media-text"></span><?php
 								printf(
 								/* translators: %s: Add-on name */
-									esc_html__( 'Documentation: %s Add-on', 'give' ),
+									esc_html__('Documentation: %s Add-on', 'give'),
 									$this->banner_details['name']
 								);
 								?></a>
 						<?php } ?>
 						<?php
 						//Let them signup for plugin updates
-						if ( isset( $this->banner_details['support_url'] ) ) { ?>
+						if (isset($this->banner_details['support_url'])) { ?>
 
 							<a href="<?php echo $this->banner_details['support_url'] ?>" target="_blank">
-								<span class="dashicons dashicons-sos"></span><?php esc_html_e( 'Get Support', 'give' ); ?>
+								<span class="dashicons dashicons-sos"></span><?php esc_html_e('Get Support', 'give'); ?>
 							</a>
 
 						<?php } ?>
@@ -176,12 +176,12 @@ 
 			
 			
 			// Register notice.
-			Give()->notices->register_notice( array(
-				'id'          => 'give_' . sanitize_title( $this->banner_details['name'] ) . '_notice',
+			Give()->notices->register_notice(array(
+				'id'          => 'give_'.sanitize_title($this->banner_details['name']).'_notice',
 				'type'        => 'updated',
 				'description_html' => $notice_html,
 				'show'        => true,
-			) );
+			));
 		}
 	}
 
@@ -192,7 +192,7 @@ 
 	 * @since 1.8.16
 	 * @access private
 	 */
-	private function print_css(){
+	private function print_css() {
 		?>
 		<style>
 			div.give-addon-alert.updated {
@@ -280,13 +280,13 @@ 
 		 * If user clicks to ignore the notice, add that to their user meta the banner then checks whether this tag exists already or not.
 		 * See here: http://codex.wordpress.org/Function_Reference/add_user_meta
 		 */
-		if ( isset( $_GET[ $this->nag_meta_key ] ) && '0' == $_GET[ $this->nag_meta_key ] ) {
+		if (isset($_GET[$this->nag_meta_key]) && '0' == $_GET[$this->nag_meta_key]) {
 
 			//Get the global user
 			$current_user = wp_get_current_user();
 			$user_id      = $current_user->ID;
 
-			add_user_meta( $user_id, $this->nag_meta_key, 'true', true );
+			add_user_meta($user_id, $this->nag_meta_key, 'true', true);
 		}
 	}
 
@@ -297,11 +297,11 @@ 
 	 * @access private
 	 */
 	private function add_addon_activate_meta() {
-		$user_id                  = get_option( $this->activate_by_meta_key );
+		$user_id                  = get_option($this->activate_by_meta_key);
 		$this->plugin_activate_by = (int) $user_id;
 
-		if ( ! $user_id ) {
-			add_option( $this->activate_by_meta_key, $this->user_id, '', 'no' );
+		if ( ! $user_id) {
+			add_option($this->activate_by_meta_key, $this->user_id, '', 'no');
 			$this->plugin_activate_by = (int) $this->user_id;
 		}
 	}
@@ -314,10 +314,10 @@ 
 	 * @access public
 	 */
 	public function remove_addon_activate_meta() {
-		$user_id = get_option( $this->activate_by_meta_key );
+		$user_id = get_option($this->activate_by_meta_key);
 
-		if ( $user_id ) {
-			delete_option( $this->activate_by_meta_key );
+		if ($user_id) {
+			delete_option($this->activate_by_meta_key);
 		}
 	}
 
@@ -330,39 +330,39 @@ 
 	 * @return mixed
 	 */
 	private function get_plugin_file_name() {
-		$active_plugins = get_option( 'active_plugins' );
+		$active_plugins = get_option('active_plugins');
 		$file_name      = '';
 
 		try {
 
 			// Check addon file path.
-			if ( ! empty( $this->banner_details['file'] ) ) {
+			if ( ! empty($this->banner_details['file'])) {
 				$file_name = '';
-				if ( $file_path = explode( '/plugins/', $this->banner_details['file'] ) ) {
-					$file_path = array_pop( $file_path );
-					$file_name = current( explode( '/', $file_path ) );
+				if ($file_path = explode('/plugins/', $this->banner_details['file'])) {
+					$file_path = array_pop($file_path);
+					$file_name = current(explode('/', $file_path));
 				}
 
-				if ( empty( $file_name ) ) {
+				if (empty($file_name)) {
 					return false;
 				}
 
-				foreach ( $active_plugins as $plugin ) {
-					if ( false !== strpos( $plugin, $file_name ) ) {
+				foreach ($active_plugins as $plugin) {
+					if (false !== strpos($plugin, $file_name)) {
 						$file_name = $plugin;
 						break;
 					}
 				}
-			} elseif ( WP_DEBUG ) {
-				throw new Exception( __( "File path must be added within the {$this->banner_details['name']} add-on in the banner details.", 'give' ) );
+			} elseif (WP_DEBUG) {
+				throw new Exception(__("File path must be added within the {$this->banner_details['name']} add-on in the banner details.", 'give'));
 			}
 
 			// Check plugin path calculated by addon file path.
-			if ( empty( $file_name ) && WP_DEBUG ) {
-				throw new Exception( __( "Empty add-on plugin path for {$this->banner_details['name']} add-on.", 'give' ) );
+			if (empty($file_name) && WP_DEBUG) {
+				throw new Exception(__("Empty add-on plugin path for {$this->banner_details['name']} add-on.", 'give'));
 			}
 
-		} catch ( Exception $e ) {
+		} catch (Exception $e) {
 			echo $e->getMessage();
 		}