WordImpress / Give

includes/admin/customers/customer-actions.php

Doc Comments +2 added lines, -2 removed lines

@@ -527,7 +527,7 @@ 
  * and redirect back to the donor interface for feedback
  *
  * @since  1.7
- * @return bool|null
+ * @return false|null
  */
 function give_remove_donor_email() {
 	if ( empty( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {
@@ -568,7 +568,7 @@ 
  * and redirect back to the donor interface for feedback
  *
  * @since  1.7
- * @return bool|null
+ * @return false|null
  */
 function give_set_donor_primary_email() {
 	if ( empty( $_GET['id'] ) || ! is_numeric( $_GET['id'] ) ) {

Indentation +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
 
 	$donor = new Give_Customer( $donor_id );
 
-    if ( empty( $donor->id ) ) {
+	if ( empty( $donor->id ) ) {
 		return false;
 	}
 

Spacing +195 added lines, -195 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-customer' ) ) {
-		wp_die( __( 'Cheatin’ uh?', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'edit-customer')) {
+		wp_die(__('Cheatin’ uh?', 'give'), __('Error', 'give'), array(
 			'response' => 400,
-		) );
+		));
 	}
 
-	$donor = new Give_Customer( $donor_id );
+	$donor = new Give_Customer($donor_id);
 
-    if ( empty( $donor->id ) ) {
+    if (empty($donor->id)) {
 		return false;
 	}
 
@@ -58,70 +58,70 @@ 
 		'user_id' => 0,
 	);
 
-	$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()->customers->get_customer_by( 'user_id', $donor_info['user_id'] ) ) {
-			give_set_error( 'give-invalid-customer-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()->customers->get_customer_by('user_id', $donor_info['user_id'])) {
+			give_set_error('give-invalid-customer-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']));
 		}
 	}
 
 	// Record this for later.
 	$previous_user_id = $donor->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 ) {
+	if (intval($donor_info['user_id']) > 0) {
 
-		$current_address = get_user_meta( $donor_info['user_id'], '_give_user_address', true );
+		$current_address = get_user_meta($donor_info['user_id'], '_give_user_address', true);
 
-		if ( false === $current_address ) {
-			$address['line1']   = isset( $donor_info['line1'] ) ? $donor_info['line1'] : '';
-			$address['line2']   = isset( $donor_info['line2'] ) ? $donor_info['line2'] : '';
-			$address['city']    = isset( $donor_info['city'] ) ? $donor_info['city'] : '';
-			$address['country'] = isset( $donor_info['country'] ) ? $donor_info['country'] : '';
-			$address['zip']     = isset( $donor_info['zip'] ) ? $donor_info['zip'] : '';
-			$address['state']   = isset( $donor_info['state'] ) ? $donor_info['state'] : '';
+		if (false === $current_address) {
+			$address['line1']   = isset($donor_info['line1']) ? $donor_info['line1'] : '';
+			$address['line2']   = isset($donor_info['line2']) ? $donor_info['line2'] : '';
+			$address['city']    = isset($donor_info['city']) ? $donor_info['city'] : '';
+			$address['country'] = isset($donor_info['country']) ? $donor_info['country'] : '';
+			$address['zip']     = isset($donor_info['zip']) ? $donor_info['zip'] : '';
+			$address['state']   = isset($donor_info['state']) ? $donor_info['state'] : '';
 		} else {
-			$current_address    = wp_parse_args( $current_address, array(
+			$current_address    = wp_parse_args($current_address, array(
 				'line1',
 				'line2',
 				'city',
 				'zip',
 				'state',
 				'country',
-			) );
-			$address['line1']   = isset( $donor_info['line1'] ) ? $donor_info['line1'] : $current_address['line1'];
-			$address['line2']   = isset( $donor_info['line2'] ) ? $donor_info['line2'] : $current_address['line2'];
-			$address['city']    = isset( $donor_info['city'] ) ? $donor_info['city'] : $current_address['city'];
-			$address['country'] = isset( $donor_info['country'] ) ? $donor_info['country'] : $current_address['country'];
-			$address['zip']     = isset( $donor_info['zip'] ) ? $donor_info['zip'] : $current_address['zip'];
-			$address['state']   = isset( $donor_info['state'] ) ? $donor_info['state'] : $current_address['state'];
+			));
+			$address['line1']   = isset($donor_info['line1']) ? $donor_info['line1'] : $current_address['line1'];
+			$address['line2']   = isset($donor_info['line2']) ? $donor_info['line2'] : $current_address['line2'];
+			$address['city']    = isset($donor_info['city']) ? $donor_info['city'] : $current_address['city'];
+			$address['country'] = isset($donor_info['country']) ? $donor_info['country'] : $current_address['country'];
+			$address['zip']     = isset($donor_info['zip']) ? $donor_info['zip'] : $current_address['zip'];
+			$address['state']   = isset($donor_info['state']) ? $donor_info['state'] : $current_address['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);
 
 	/**
 	 * Fires before editing a donor.
@@ -132,27 +132,27 @@ 
 	 * @param array $donor_data The donor data.
 	 * @param array $address The donor's address.
 	 */
-	do_action( 'give_pre_edit_donor', $donor_id, $donor_data, $address );
+	do_action('give_pre_edit_donor', $donor_id, $donor_data, $address);
 
 	$output = array();
 
-	if ( $donor->update( $donor_data ) ) {
+	if ($donor->update($donor_data)) {
 
-		if ( ! empty( $donor->user_id ) && $donor->user_id > 0 ) {
-			update_user_meta( $donor->user_id, '_give_user_address', $address );
+		if ( ! empty($donor->user_id) && $donor->user_id > 0) {
+			update_user_meta($donor->user_id, '_give_user_address', $address);
 		}
 
 		// Update some donation meta if we need to.
-		$payments_array = explode( ',', $donor->payment_ids );
+		$payments_array = explode(',', $donor->payment_ids);
 
-		if ( $donor->user_id != $previous_user_id ) {
-			foreach ( $payments_array as $payment_id ) {
-				give_update_payment_meta( $payment_id, '_give_payment_user_id', $donor->user_id );
+		if ($donor->user_id != $previous_user_id) {
+			foreach ($payments_array as $payment_id) {
+				give_update_payment_meta($payment_id, '_give_payment_user_id', $donor->user_id);
 			}
 		}
 
 		$output['success']       = true;
-		$donor_data              = array_merge( $donor_data, $address );
+		$donor_data              = array_merge($donor_data, $address);
 		$output['customer_info'] = $donor_data;
 
 	} else {
@@ -169,11 +169,11 @@ 
 	 * @param int $donor_id The ID of the donor.
 	 * @param array $donor_data The donor data.
 	 */
-	do_action( 'give_post_edit_donor', $donor_id, $donor_data );
+	do_action('give_post_edit_donor', $donor_id, $donor_data);
 
-	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();
 	}
 
@@ -181,7 +181,7 @@ 
 
 }
 
-add_action( 'give_edit-customer', 'give_edit_donor', 10, 1 );
+add_action('give_edit-customer', 'give_edit_donor', 10, 1);
 
 /**
  * Save a customer note being added
@@ -192,40 +192,40 @@ 
  *
  * @return int         The Note ID that was saved, or 0 if nothing was saved
  */
-function give_customer_save_note( $args ) {
+function give_customer_save_note($args) {
 
-	$customer_view_role = apply_filters( 'give_view_customers_role', 'view_give_reports' );
+	$customer_view_role = apply_filters('give_view_customers_role', 'view_give_reports');
 
-	if ( ! is_admin() || ! current_user_can( $customer_view_role ) ) {
-		wp_die( esc_html__( 'You do not have permission to edit this donor.', 'give' ), esc_html__( 'Error', 'give' ), array(
+	if ( ! is_admin() || ! current_user_can($customer_view_role)) {
+		wp_die(esc_html__('You do not have permission to edit this donor.', 'give'), esc_html__('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	if ( empty( $args ) ) {
+	if (empty($args)) {
 		return;
 	}
 
-	$customer_note = trim( sanitize_text_field( $args['customer_note'] ) );
+	$customer_note = trim(sanitize_text_field($args['customer_note']));
 	$customer_id   = (int) $args['customer_id'];
 	$nonce         = $args['add_customer_note_nonce'];
 
-	if ( ! wp_verify_nonce( $nonce, 'add-customer-note' ) ) {
-		wp_die( esc_html__( 'Cheatin’ uh?', 'give' ), esc_html__( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'add-customer-note')) {
+		wp_die(esc_html__('Cheatin’ uh?', 'give'), esc_html__('Error', 'give'), array(
 			'response' => 400,
-		) );
+		));
 	}
 
-	if ( empty( $customer_note ) ) {
-		give_set_error( 'empty-customer-note', esc_html__( 'A note is required.', 'give' ) );
+	if (empty($customer_note)) {
+		give_set_error('empty-customer-note', esc_html__('A note is required.', 'give'));
 	}
 
-	if ( give_get_errors() ) {
+	if (give_get_errors()) {
 		return;
 	}
 
-	$customer = new Give_Customer( $customer_id );
-	$new_note = $customer->add_note( $customer_note );
+	$customer = new Give_Customer($customer_id);
+	$new_note = $customer->add_note($customer_note);
 
 	/**
 	 * Fires before inserting customer note.
@@ -235,22 +235,22 @@ 
 	 * @param int $customer_id The ID of the customer.
 	 * @param string $new_note Note content.
 	 */
-	do_action( 'give_pre_insert_customer_note', $customer_id, $new_note );
+	do_action('give_pre_insert_customer_note', $customer_id, $new_note);
 
-	if ( ! empty( $new_note ) && ! empty( $customer->id ) ) {
+	if ( ! empty($new_note) && ! empty($customer->id)) {
 
 		ob_start();
 		?>
 		<div class="customer-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;
 		}
@@ -263,7 +263,7 @@ 
 
 }
 
-add_action( 'give_add-customer-note', 'give_customer_save_note', 10, 1 );
+add_action('give_add-customer-note', 'give_customer_save_note', 10, 1);
 
 /**
  * Delete a customer
@@ -274,41 +274,41 @@ 
  *
  * @return int Whether it was a successful deletion
  */
-function give_customer_delete( $args ) {
+function give_customer_delete($args) {
 
-	$customer_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+	$customer_edit_role = apply_filters('give_edit_donors_role', 'edit_give_payments');
 
-	if ( ! is_admin() || ! current_user_can( $customer_edit_role ) ) {
-		wp_die( esc_html__( 'You do not have permission to delete donors.', 'give' ), esc_html__( 'Error', 'give' ), array(
+	if ( ! is_admin() || ! current_user_can($customer_edit_role)) {
+		wp_die(esc_html__('You do not have permission to delete donors.', 'give'), esc_html__('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	if ( empty( $args ) ) {
+	if (empty($args)) {
 		return;
 	}
 
 	$customer_id = (int) $args['customer_id'];
-	$confirm     = ! empty( $args['give-customer-delete-confirm'] ) ? true : false;
-	$remove_data = ! empty( $args['give-customer-delete-records'] ) ? true : false;
+	$confirm     = ! empty($args['give-customer-delete-confirm']) ? true : false;
+	$remove_data = ! empty($args['give-customer-delete-records']) ? true : false;
 	$nonce       = $args['_wpnonce'];
 
-	if ( ! wp_verify_nonce( $nonce, 'delete-customer' ) ) {
-		wp_die( esc_html__( 'Cheatin&#8217; uh?', 'give' ), esc_html__( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'delete-customer')) {
+		wp_die(esc_html__('Cheatin&#8217; uh?', 'give'), esc_html__('Error', 'give'), array(
 			'response' => 400,
-		) );
+		));
 	}
 
-	if ( ! $confirm ) {
-		give_set_error( 'customer-delete-no-confirm', esc_html__( 'Please confirm you want to delete this donor.', 'give' ) );
+	if ( ! $confirm) {
+		give_set_error('customer-delete-no-confirm', esc_html__('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=' . $customer_id ) );
+	if (give_get_errors()) {
+		wp_redirect(admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$customer_id));
 		exit;
 	}
 
-	$customer = new Give_Customer( $customer_id );
+	$customer = new Give_Customer($customer_id);
 
 	/**
 	 * Fires before deleting customer.
@@ -319,50 +319,50 @@ 
 	 * @param bool $confirm Delete confirmation.
 	 * @param bool $remove_data Records delete confirmation.
 	 */
-	do_action( 'give_pre_delete_customer', $customer_id, $confirm, $remove_data );
+	do_action('give_pre_delete_customer', $customer_id, $confirm, $remove_data);
 
-	if ( $customer->id > 0 ) {
+	if ($customer->id > 0) {
 
-		$payments_array = explode( ',', $customer->payment_ids );
-		$success        = Give()->customers->delete( $customer->id );
+		$payments_array = explode(',', $customer->payment_ids);
+		$success        = Give()->customers->delete($customer->id);
 
-		if ( $success ) {
+		if ($success) {
 
-			if ( $remove_data ) {
+			if ($remove_data) {
 
 				// Remove all donations, logs, etc
-				foreach ( $payments_array as $payment_id ) {
-					give_delete_purchase( $payment_id );
+				foreach ($payments_array as $payment_id) {
+					give_delete_purchase($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=customer-deleted' );
+			$redirect = admin_url('edit.php?post_type=give_forms&page=give-donors&give-message=customer-deleted');
 
 		} else {
 
-			give_set_error( 'give-donor-delete-failed', esc_html__( 'Error deleting donor.', 'give' ) );
-			$redirect = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=delete&id=' . $customer_id );
+			give_set_error('give-donor-delete-failed', esc_html__('Error deleting donor.', 'give'));
+			$redirect = admin_url('edit.php?post_type=give_forms&page=give-donors&view=delete&id='.$customer_id);
 
 		}
 	} else {
 
-		give_set_error( 'give-customer-delete-invalid-id', esc_html__( 'Invalid Donor ID.', 'give' ) );
-		$redirect = admin_url( 'edit.php?post_type=give_forms&page=give-donors' );
+		give_set_error('give-customer-delete-invalid-id', esc_html__('Invalid Donor ID.', 'give'));
+		$redirect = admin_url('edit.php?post_type=give_forms&page=give-donors');
 
 	}
 
-	wp_redirect( $redirect );
+	wp_redirect($redirect);
 	exit;
 
 }
 
-add_action( 'give_delete-customer', 'give_customer_delete', 10, 1 );
+add_action('give_delete-customer', 'give_customer_delete', 10, 1);
 
 /**
  * Disconnect a user ID from a donor
@@ -373,32 +373,32 @@ 
  *
  * @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;
 	}
 
 	$donor_id = (int) $args['customer_id'];
 
-	$nonce       = $args['_wpnonce'];
+	$nonce = $args['_wpnonce'];
 
-	if ( ! wp_verify_nonce( $nonce, 'edit-customer' ) ) {
-		wp_die( __( 'Cheatin&#8217; uh?', 'give' ), __( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'edit-customer')) {
+		wp_die(__('Cheatin&#8217; uh?', 'give'), __('Error', 'give'), array(
 			'response' => 400,
-		) );
+		));
 	}
 
-	$donor = new Give_Customer( $donor_id );
-	if ( empty( $donor->id ) ) {
+	$donor = new Give_Customer($donor_id);
+	if (empty($donor->id)) {
 		return false;
 	}
 
@@ -412,18 +412,18 @@ 
 	 * @param int $donor_id The ID of the donor.
 	 * @param int $user_id The ID of the user.
 	 */
-	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();
+	$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 )");
 		}
 
 		$output['success'] = true;
@@ -431,7 +431,7 @@ 
 	} 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'));
 	}
 
 	/**
@@ -441,11 +441,11 @@ 
 	 *
 	 * @param int $donor_id The ID of the donor.
 	 */
-	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();
 	}
 
@@ -453,7 +453,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
@@ -464,82 +464,82 @@ 
  *
  * @return mixed        If DOING_AJAX echos out JSON, otherwise returns array of success (bool) and message (string)
  */
-function give_add_donor_email( $args ) {
-	$donor_edit_role = apply_filters( 'give_edit_donors_role', 'edit_give_payments' );
+function give_add_donor_email($args) {
+	$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.', 'edit' ) );
+	if ( ! is_admin() || ! current_user_can($donor_edit_role)) {
+		wp_die(__('You do not have permission to edit this donor.', 'edit'));
 	}
 
 	$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' => esc_html__( 'Nonce verification failed.', 'give' ),
+			'message' => esc_html__('Nonce verification failed.', 'give'),
 		);
-	} elseif ( ! is_email( $args['email'] ) ) {
+	} elseif ( ! is_email($args['email'])) {
 		$output = array(
 			'success' => false,
-			'message' => esc_html__( 'Invalid email.', 'give' ),
+			'message' => esc_html__('Invalid email.', 'give'),
 		);
 	} else {
-		$email       = sanitize_email( $args['email'] );
+		$email       = sanitize_email($args['email']);
 		$customer_id = (int) $args['customer_id'];
 		$primary     = 'true' === $args['primary'] ? true : false;
-		$customer    = new Give_Customer( $customer_id );
-		if ( false === $customer->add_email( $email, $primary ) ) {
-			if ( in_array( $email, $customer->emails ) ) {
+		$customer    = new Give_Customer($customer_id);
+		if (false === $customer->add_email($email, $primary)) {
+			if (in_array($email, $customer->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=' . $customer_id . '&give-message=email-added' );
+			$redirect = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$customer_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' );
-			$customer_note = sprintf( __( 'Email address %1$s added by %2$s', 'give' ), $email, $user_login );
-			$customer->add_note( $customer_note );
+			$user_login    = ! empty($user->user_login) ? $user->user_login : __('System', 'give');
+			$customer_note = sprintf(__('Email address %1$s added by %2$s', 'give'), $email, $user_login);
+			$customer->add_note($customer_note);
 
-			if ( $primary ) {
-				$customer_note = sprintf( __( 'Email address %1$s set as primary by %2$s', 'give' ), $email, $user_login );
-				$customer->add_note( $customer_note );
+			if ($primary) {
+				$customer_note = sprintf(__('Email address %1$s set as primary by %2$s', 'give'), $email, $user_login);
+				$customer->add_note($customer_note);
 			}
 		}
 	}// End if().
 
-	do_action( 'give_post_add_customer_email', $customer_id, $args );
+	do_action('give_post_add_customer_email', $customer_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);
 
 
 /**
@@ -550,39 +550,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( esc_html__( 'Nonce verification failed', 'give' ), esc_html__( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'give-remove-donor-email')) {
+		wp_die(esc_html__('Nonce verification failed', 'give'), esc_html__('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	$customer = new Give_Customer( $_GET['id'] );
-	if ( $customer->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=' . $customer->id ) );
+	$customer = new Give_Customer($_GET['id']);
+	if ($customer->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='.$customer->id));
 		$user          = wp_get_current_user();
-		$user_login    = ! empty( $user->user_login ) ? $user->user_login : esc_html__( 'System', 'give' );
-		$customer_note = sprintf( __( 'Email address %1$s removed by %2$s', 'give' ), $_GET['email'], $user_login );
-		$customer->add_note( $customer_note );
+		$user_login    = ! empty($user->user_login) ? $user->user_login : esc_html__('System', 'give');
+		$customer_note = sprintf(__('Email address %1$s removed by %2$s', 'give'), $_GET['email'], $user_login);
+		$customer->add_note($customer_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=' . $customer->id ) );
+		$url = add_query_arg('give-message', 'email-remove-failed', admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$customer->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);
 
 
 /**
@@ -593,41 +593,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( esc_html__( 'Nonce verification failed', 'give' ), esc_html__( 'Error', 'give' ), array(
+	if ( ! wp_verify_nonce($nonce, 'give-set-donor-primary-email')) {
+		wp_die(esc_html__('Nonce verification failed', 'give'), esc_html__('Error', 'give'), array(
 			'response' => 403,
-		) );
+		));
 	}
 
-	$donor = new Give_Customer( $_GET['id'] );
+	$donor = new Give_Customer($_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 : esc_html__( '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 : esc_html__('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/gateways/paypal-standard.php

Doc Comments +2 added lines, -2 removed lines

@@ -565,7 +565,7 @@ 
  *
  * @since 1.6.3
  *
- * @param $pending_reason
+ * @param string $pending_reason
  *
  * @return string
  */
@@ -642,7 +642,7 @@ 
  * @param int   $payment_id   Payment ID
  * @param array $payment_data Array of payment data.
  *
- * @return mixed|string
+ * @return string
  */
 function give_build_paypal_url( $payment_id, $payment_data ) {
 	// Only send to PayPal if the pending payment is created successfully.

Spacing +168 added lines, -168 removed lines

@@ -9,7 +9,7 @@ 
  * @since       1.0
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -22,10 +22,10 @@ 
  *
  * @return bool
  */
-function give_paypal_standard_billing_fields( $form_id ) {
+function give_paypal_standard_billing_fields($form_id) {
 
-	if ( give_is_setting_enabled( give_get_option( 'paypal_standard_billing_details' ) ) ) {
-		give_default_cc_address_fields( $form_id );
+	if (give_is_setting_enabled(give_get_option('paypal_standard_billing_details'))) {
+		give_default_cc_address_fields($form_id);
 
 		return true;
 	}
@@ -34,7 +34,7 @@ 
 
 }
 
-add_action( 'give_paypal_cc_form', 'give_paypal_standard_billing_fields');
+add_action('give_paypal_cc_form', 'give_paypal_standard_billing_fields');
 
 /**
  * Process PayPal Payment.
@@ -45,34 +45,34 @@ 
  *
  * @return void
  */
-function give_process_paypal_payment( $payment_data ) {
+function give_process_paypal_payment($payment_data) {
 
 	// Validate nonce.
-	give_validate_nonce( $payment_data['gateway_nonce'], 'give-gateway' );
-	$payment_id = give_create_payment( $payment_data );
+	give_validate_nonce($payment_data['gateway_nonce'], 'give-gateway');
+	$payment_id = give_create_payment($payment_data);
 
 	// Check payment.
-	if ( empty( $payment_id ) ) {
+	if (empty($payment_id)) {
 		// Record the error.
 		give_record_gateway_error(
-			esc_html__( 'Payment Error', 'give' ),
+			esc_html__('Payment Error', 'give'),
 			sprintf(
 			/* translators: %s: payment data */
-				esc_html__( 'Payment creation failed before sending donor to PayPal. Payment data: %s', 'give' ),
-				json_encode( $payment_data )
+				esc_html__('Payment creation failed before sending donor to PayPal. Payment data: %s', 'give'),
+				json_encode($payment_data)
 			),
 			$payment_id
 		);
 		// Problems? Send back.
-		give_send_back_to_checkout( '?payment-mode=' . $payment_data['post_data']['give-gateway'] );
+		give_send_back_to_checkout('?payment-mode='.$payment_data['post_data']['give-gateway']);
 	}
 
 	// Redirect to PayPal.
-	wp_redirect( give_build_paypal_url( $payment_id, $payment_data ) );
+	wp_redirect(give_build_paypal_url($payment_id, $payment_data));
 	exit;
 }
 
-add_action( 'give_gateway_paypal', 'give_process_paypal_payment' );
+add_action('give_gateway_paypal', 'give_process_paypal_payment');
 
 /**
  * Listens for a PayPal IPN requests and then sends to the processing function
@@ -82,17 +82,17 @@ 
  */
 function give_listen_for_paypal_ipn() {
 	// Regular PayPal IPN
-	if ( isset( $_GET['give-listener'] ) && $_GET['give-listener'] == 'IPN' ) {
+	if (isset($_GET['give-listener']) && $_GET['give-listener'] == 'IPN') {
 		/**
 		 * Fires while verifying PayPal IPN
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_verify_paypal_ipn' );
+		do_action('give_verify_paypal_ipn');
 	}
 }
 
-add_action( 'init', 'give_listen_for_paypal_ipn' );
+add_action('init', 'give_listen_for_paypal_ipn');
 
 /**
  * Process PayPal IPN
@@ -103,7 +103,7 @@ 
 function give_process_paypal_ipn() {
 
 	// Check the request method is POST
-	if ( isset( $_SERVER['REQUEST_METHOD'] ) && $_SERVER['REQUEST_METHOD'] != 'POST' ) {
+	if (isset($_SERVER['REQUEST_METHOD']) && $_SERVER['REQUEST_METHOD'] != 'POST') {
 		return;
 	}
 
@@ -111,11 +111,11 @@ 
 	$post_data = '';
 
 	// Fallback just in case post_max_size is lower than needed
-	if ( ini_get( 'allow_url_fopen' ) ) {
-		$post_data = file_get_contents( 'php://input' );
+	if (ini_get('allow_url_fopen')) {
+		$post_data = file_get_contents('php://input');
 	} else {
 		// If allow_url_fopen is not enabled, then make sure that post_max_size is large enough
-		ini_set( 'post_max_size', '12M' );
+		ini_set('post_max_size', '12M');
 	}
 	// Start the encoded data collection with notification command
 	$encoded_data = 'cmd=_notify-validate';
@@ -124,39 +124,39 @@ 
 	$arg_separator = give_get_php_arg_separator_output();
 
 	// Verify there is a post_data
-	if ( $post_data || strlen( $post_data ) > 0 ) {
+	if ($post_data || strlen($post_data) > 0) {
 		// Append the data
-		$encoded_data .= $arg_separator . $post_data;
+		$encoded_data .= $arg_separator.$post_data;
 	} else {
 		// Check if POST is empty
-		if ( empty( $_POST ) ) {
+		if (empty($_POST)) {
 			// Nothing to do
 			return;
 		} else {
 			// Loop through each POST
-			foreach ( $_POST as $key => $value ) {
+			foreach ($_POST as $key => $value) {
 				// Encode the value and append the data.
-				$encoded_data .= $arg_separator . "$key=" . urlencode( $value );
+				$encoded_data .= $arg_separator."$key=".urlencode($value);
 			}
 		}
 	}
 
 	// Convert collected post data to an array.
-	parse_str( $encoded_data, $encoded_data_array );
+	parse_str($encoded_data, $encoded_data_array);
 
-	foreach ( $encoded_data_array as $key => $value ) {
+	foreach ($encoded_data_array as $key => $value) {
 
-		if ( false !== strpos( $key, 'amp;' ) ) {
-			$new_key = str_replace( '&', '&', $key );
-			$new_key = str_replace( 'amp;', '&', $new_key );
+		if (false !== strpos($key, 'amp;')) {
+			$new_key = str_replace('&', '&', $key);
+			$new_key = str_replace('amp;', '&', $new_key);
 
-			unset( $encoded_data_array[ $key ] );
-			$encoded_data_array[ $new_key ] = $value;
+			unset($encoded_data_array[$key]);
+			$encoded_data_array[$new_key] = $value;
 		}
 	}
 
 	// Validate IPN request w/ PayPal if user hasn't disabled this security measure
-	if ( give_is_setting_enabled( give_get_option( 'paypal_verification' ) ) ) {
+	if (give_is_setting_enabled(give_get_option('paypal_verification'))) {
 
 		$remote_post_vars = array(
 			'method'      => 'POST',
@@ -176,28 +176,28 @@ 
 		);
 
 		// Validate the IPN.
-		$api_response = wp_remote_post( give_get_paypal_redirect(), $remote_post_vars );
+		$api_response = wp_remote_post(give_get_paypal_redirect(), $remote_post_vars);
 
-		if ( is_wp_error( $api_response ) ) {
+		if (is_wp_error($api_response)) {
 			give_record_gateway_error(
-				esc_html__( 'IPN Error', 'give' ),
+				esc_html__('IPN Error', 'give'),
 				sprintf(
 				/* translators: %s: Paypal IPN response */
-					esc_html__( 'Invalid IPN verification response. IPN data: %s', 'give' ),
-					json_encode( $api_response )
+					esc_html__('Invalid IPN verification response. IPN data: %s', 'give'),
+					json_encode($api_response)
 				)
 			);
 
 			return; // Something went wrong
 		}
 
-		if ( $api_response['body'] !== 'VERIFIED' ) {
+		if ($api_response['body'] !== 'VERIFIED') {
 			give_record_gateway_error(
-				esc_html__( 'IPN Error', 'give' ),
+				esc_html__('IPN Error', 'give'),
 				sprintf(
 				/* translators: %s: Paypal IPN response */
-					esc_html__( 'Invalid IPN verification response. IPN data: %s', 'give' ),
-					json_encode( $api_response )
+					esc_html__('Invalid IPN verification response. IPN data: %s', 'give'),
+					json_encode($api_response)
 				)
 			);
 
@@ -206,7 +206,7 @@ 
 	}
 
 	// Check if $post_data_array has been populated
-	if ( ! is_array( $encoded_data_array ) && ! empty( $encoded_data_array ) ) {
+	if ( ! is_array($encoded_data_array) && ! empty($encoded_data_array)) {
 		return;
 	}
 
@@ -215,12 +215,12 @@ 
 		'payment_status' => '',
 	);
 
-	$encoded_data_array = wp_parse_args( $encoded_data_array, $defaults );
+	$encoded_data_array = wp_parse_args($encoded_data_array, $defaults);
 
-	$payment_id = isset( $encoded_data_array['custom'] ) ? absint( $encoded_data_array['custom'] ) : 0;
+	$payment_id = isset($encoded_data_array['custom']) ? absint($encoded_data_array['custom']) : 0;
 	$txn_type   = $encoded_data_array['txn_type'];
 
-	if ( has_action( 'give_paypal_' . $txn_type ) ) {
+	if (has_action('give_paypal_'.$txn_type)) {
 		/**
 		 * Fires while processing PayPal IPN $txn_type.
 		 *
@@ -231,7 +231,7 @@ 
 		 * @param array $encoded_data_array Encoded data.
 		 * @param int   $payment_id         Payment id.
 		 */
-		do_action( "give_paypal_{$txn_type}", $encoded_data_array, $payment_id );
+		do_action("give_paypal_{$txn_type}", $encoded_data_array, $payment_id);
 	} else {
 		/**
 		 * Fires while process PayPal IPN.
@@ -243,12 +243,12 @@ 
 		 * @param array $encoded_data_array Encoded data.
 		 * @param int   $payment_id         Payment id.
 		 */
-		do_action( 'give_paypal_web_accept', $encoded_data_array, $payment_id );
+		do_action('give_paypal_web_accept', $encoded_data_array, $payment_id);
 	}
 	exit;
 }
 
-add_action( 'give_verify_paypal_ipn', 'give_process_paypal_ipn' );
+add_action('give_verify_paypal_ipn', 'give_process_paypal_ipn');
 
 /**
  * Process web accept (one time) payment IPNs.
@@ -260,128 +260,128 @@ 
  *
  * @return void
  */
-function give_process_paypal_web_accept_and_cart( $data, $payment_id ) {
+function give_process_paypal_web_accept_and_cart($data, $payment_id) {
 
 	// Only allow through these transaction types.
-	if ( $data['txn_type'] != 'web_accept' && $data['txn_type'] != 'cart' && strtolower( $data['payment_status'] ) != 'refunded' ) {
+	if ($data['txn_type'] != 'web_accept' && $data['txn_type'] != 'cart' && strtolower($data['payment_status']) != 'refunded') {
 		return;
 	}
 
 	// Need $payment_id to continue.
-	if ( empty( $payment_id ) ) {
+	if (empty($payment_id)) {
 		return;
 	}
 
 	// Collect donation payment details.
 	$paypal_amount  = $data['mc_gross'];
-	$payment_status = strtolower( $data['payment_status'] );
-	$currency_code  = strtolower( $data['mc_currency'] );
-	$business_email = isset( $data['business'] ) && is_email( $data['business'] ) ? trim( $data['business'] ) : trim( $data['receiver_email'] );
-	$payment_meta   = give_get_payment_meta( $payment_id );
+	$payment_status = strtolower($data['payment_status']);
+	$currency_code  = strtolower($data['mc_currency']);
+	$business_email = isset($data['business']) && is_email($data['business']) ? trim($data['business']) : trim($data['receiver_email']);
+	$payment_meta   = give_get_payment_meta($payment_id);
 
 	// Must be a PayPal standard IPN.
-	if ( give_get_payment_gateway( $payment_id ) != 'paypal' ) {
+	if (give_get_payment_gateway($payment_id) != 'paypal') {
 		return;
 	}
 
 	// Verify payment recipient
-	if ( strcasecmp( $business_email, trim( give_get_option( 'paypal_email' ) ) ) != 0 ) {
+	if (strcasecmp($business_email, trim(give_get_option('paypal_email'))) != 0) {
 
 		give_record_gateway_error(
-			esc_html__( 'IPN Error', 'give' ),
+			esc_html__('IPN Error', 'give'),
 			sprintf(
 			/* translators: %s: Paypal IPN response */
-				esc_html__( 'Invalid business email in IPN response. IPN data: %s', 'give' ),
-				json_encode( $data )
+				esc_html__('Invalid business email in IPN response. IPN data: %s', 'give'),
+				json_encode($data)
 			),
 			$payment_id
 		);
-		give_update_payment_status( $payment_id, 'failed' );
-		give_insert_payment_note( $payment_id, esc_html__( 'Payment failed due to invalid PayPal business email.', 'give' ) );
+		give_update_payment_status($payment_id, 'failed');
+		give_insert_payment_note($payment_id, esc_html__('Payment failed due to invalid PayPal business email.', 'give'));
 
 		return;
 	}
 
 	// Verify payment currency.
-	if ( $currency_code != strtolower( $payment_meta['currency'] ) ) {
+	if ($currency_code != strtolower($payment_meta['currency'])) {
 
 		give_record_gateway_error(
-			esc_html__( 'IPN Error', 'give' ),
+			esc_html__('IPN Error', 'give'),
 			sprintf(
 			/* translators: %s: Paypal IPN response */
-				esc_html__( 'Invalid currency in IPN response. IPN data: %s', 'give' ),
-				json_encode( $data )
+				esc_html__('Invalid currency in IPN response. IPN data: %s', 'give'),
+				json_encode($data)
 			),
 			$payment_id
 		);
-		give_update_payment_status( $payment_id, 'failed' );
-		give_insert_payment_note( $payment_id, esc_html__( 'Payment failed due to invalid currency in PayPal IPN.', 'give' ) );
+		give_update_payment_status($payment_id, 'failed');
+		give_insert_payment_note($payment_id, esc_html__('Payment failed due to invalid currency in PayPal IPN.', 'give'));
 
 		return;
 	}
 
 	// Process refunds & reversed.
-	if ( $payment_status == 'refunded' || $payment_status == 'reversed' ) {
-		give_process_paypal_refund( $data, $payment_id );
+	if ($payment_status == 'refunded' || $payment_status == 'reversed') {
+		give_process_paypal_refund($data, $payment_id);
 
 		return;
 	}
 
 	// Only complete payments once.
-	if ( get_post_status( $payment_id ) == 'publish' ) {
+	if (get_post_status($payment_id) == 'publish') {
 		return;
 	}
 
 	// Retrieve the total donation amount (before PayPal).
-	$payment_amount = give_get_payment_amount( $payment_id );
+	$payment_amount = give_get_payment_amount($payment_id);
 
 	// Check that the donation PP and local db amounts match.
-	if ( number_format( (float) $paypal_amount, 2 ) < number_format( (float) $payment_amount, 2 ) ) {
+	if (number_format((float) $paypal_amount, 2) < number_format((float) $payment_amount, 2)) {
 		// The prices don't match
 		give_record_gateway_error(
-			esc_html__( 'IPN Error', 'give' ),
+			esc_html__('IPN Error', 'give'),
 			sprintf(
 			/* translators: %s: Paypal IPN response */
-				esc_html__( 'Invalid payment amount in IPN response. IPN data: %s', 'give' ),
-				json_encode( $data )
+				esc_html__('Invalid payment amount in IPN response. IPN data: %s', 'give'),
+				json_encode($data)
 			),
 			$payment_id
 		);
-		give_update_payment_status( $payment_id, 'failed' );
-		give_insert_payment_note( $payment_id, esc_html__( 'Payment failed due to invalid amount in PayPal IPN.', 'give' ) );
+		give_update_payment_status($payment_id, 'failed');
+		give_insert_payment_note($payment_id, esc_html__('Payment failed due to invalid amount in PayPal IPN.', 'give'));
 
 		return;
 	}
 
 	// Process completed donations.
-	if ( $payment_status == 'completed' || give_is_test_mode() ) {
+	if ($payment_status == 'completed' || give_is_test_mode()) {
 
 		give_insert_payment_note(
 			$payment_id,
 			sprintf(
 			/* translators: %s: Paypal transaction ID */
-				esc_html__( 'PayPal Transaction ID: %s', 'give' ),
+				esc_html__('PayPal Transaction ID: %s', 'give'),
 				$data['txn_id']
 			)
 		);
-		give_set_payment_transaction_id( $payment_id, $data['txn_id'] );
-		give_update_payment_status( $payment_id, 'publish' );
+		give_set_payment_transaction_id($payment_id, $data['txn_id']);
+		give_update_payment_status($payment_id, 'publish');
 
-	} elseif ( 'pending' == $payment_status && isset( $data['pending_reason'] ) ) {
+	} elseif ('pending' == $payment_status && isset($data['pending_reason'])) {
 
 		// Look for possible pending reasons, such as an echeck.
-		$note = give_paypal_get_pending_donation_note( strtolower( $data['pending_reason'] ) );
+		$note = give_paypal_get_pending_donation_note(strtolower($data['pending_reason']));
 
-		if ( ! empty( $note ) ) {
+		if ( ! empty($note)) {
 
-			give_insert_payment_note( $payment_id, $note );
+			give_insert_payment_note($payment_id, $note);
 
 		}
 	}
 
 }
 
-add_action( 'give_paypal_web_accept', 'give_process_paypal_web_accept_and_cart', 10, 2 );
+add_action('give_paypal_web_accept', 'give_process_paypal_web_accept_and_cart', 10, 2);
 
 /**
  * Process PayPal IPN Refunds
@@ -393,27 +393,27 @@ 
  *
  * @return void
  */
-function give_process_paypal_refund( $data, $payment_id = 0 ) {
+function give_process_paypal_refund($data, $payment_id = 0) {
 
 	// Collect payment details
-	if ( empty( $payment_id ) ) {
+	if (empty($payment_id)) {
 		return;
 	}
 
-	if ( get_post_status( $payment_id ) == 'refunded' ) {
+	if (get_post_status($payment_id) == 'refunded') {
 		return; // Only refund payments once
 	}
 
-	$payment_amount = give_get_payment_amount( $payment_id );
+	$payment_amount = give_get_payment_amount($payment_id);
 	$refund_amount  = $data['payment_gross'] * - 1;
 
-	if ( number_format( (float) $refund_amount, 2 ) < number_format( (float) $payment_amount, 2 ) ) {
+	if (number_format((float) $refund_amount, 2) < number_format((float) $payment_amount, 2)) {
 
 		give_insert_payment_note(
 			$payment_id,
 			sprintf(
 			/* translators: %s: Paypal parent transaction ID */
-				esc_html__( 'Partial PayPal refund processed: %s', 'give' ),
+				esc_html__('Partial PayPal refund processed: %s', 'give'),
 				$data['parent_txn_id']
 			)
 		);
@@ -426,7 +426,7 @@ 
 		$payment_id,
 		sprintf(
 		/* translators: 1: Paypal parent transaction ID 2. Paypal reason code */
-			esc_html__( 'PayPal Payment #%1$s Refunded for reason: %2$s', 'give' ),
+			esc_html__('PayPal Payment #%1$s Refunded for reason: %2$s', 'give'),
 			$data['parent_txn_id'],
 			$data['reason_code']
 		)
@@ -435,11 +435,11 @@ 
 		$payment_id,
 		sprintf(
 		/* translators: %s: Paypal transaction ID */
-			esc_html__( 'PayPal Refund Transaction ID: %s', 'give' ),
+			esc_html__('PayPal Refund Transaction ID: %s', 'give'),
 			$data['txn_id']
 		)
 	);
-	give_update_payment_status( $payment_id, 'refunded' );
+	give_update_payment_status($payment_id, 'refunded');
 }
 
 /**
@@ -451,24 +451,24 @@ 
  *
  * @return string
  */
-function give_get_paypal_redirect( $ssl_check = false ) {
+function give_get_paypal_redirect($ssl_check = false) {
 
-	if ( is_ssl() || ! $ssl_check ) {
+	if (is_ssl() || ! $ssl_check) {
 		$protocol = 'https://';
 	} else {
 		$protocol = 'http://';
 	}
 
 	// Check the current payment mode
-	if ( give_is_test_mode() ) {
+	if (give_is_test_mode()) {
 		// Test mode
-		$paypal_uri = $protocol . 'www.sandbox.paypal.com/cgi-bin/webscr';
+		$paypal_uri = $protocol.'www.sandbox.paypal.com/cgi-bin/webscr';
 	} else {
 		// Live mode
-		$paypal_uri = $protocol . 'www.paypal.com/cgi-bin/webscr';
+		$paypal_uri = $protocol.'www.paypal.com/cgi-bin/webscr';
 	}
 
-	return apply_filters( 'give_paypal_uri', $paypal_uri );
+	return apply_filters('give_paypal_uri', $paypal_uri);
 }
 
 /**
@@ -478,9 +478,9 @@ 
  * @return string
  */
 function give_get_paypal_page_style() {
-	$page_style = trim( give_get_option( 'paypal_page_style', 'PayPal' ) );
+	$page_style = trim(give_get_option('paypal_page_style', 'PayPal'));
 
-	return apply_filters( 'give_paypal_page_style', $page_style );
+	return apply_filters('give_paypal_page_style', $page_style);
 }
 
 /**
@@ -494,26 +494,26 @@ 
  *
  * @return string
  */
-function give_paypal_success_page_content( $content ) {
+function give_paypal_success_page_content($content) {
 
-	if ( ! isset( $_GET['payment-id'] ) && ! give_get_purchase_session() ) {
+	if ( ! isset($_GET['payment-id']) && ! give_get_purchase_session()) {
 		return $content;
 	}
 
-	$payment_id = isset( $_GET['payment-id'] ) ? absint( $_GET['payment-id'] ) : false;
+	$payment_id = isset($_GET['payment-id']) ? absint($_GET['payment-id']) : false;
 
-	if ( ! $payment_id ) {
+	if ( ! $payment_id) {
 		$session    = give_get_purchase_session();
-		$payment_id = give_get_purchase_id_by_key( $session['purchase_key'] );
+		$payment_id = give_get_purchase_id_by_key($session['purchase_key']);
 	}
 
-	$payment = get_post( $payment_id );
-	if ( $payment && 'pending' == $payment->post_status ) {
+	$payment = get_post($payment_id);
+	if ($payment && 'pending' == $payment->post_status) {
 
 		// Payment is still pending so show processing indicator to fix the race condition.
 		ob_start();
 
-		give_get_template_part( 'payment', 'processing' );
+		give_get_template_part('payment', 'processing');
 
 		$content = ob_get_clean();
 
@@ -523,7 +523,7 @@ 
 
 }
 
-add_filter( 'give_payment_confirm_paypal', 'give_paypal_success_page_content' );
+add_filter('give_payment_confirm_paypal', 'give_paypal_success_page_content');
 
 /**
  * Given a Payment ID, extract the transaction ID
@@ -534,22 +534,22 @@ 
  *
  * @return string                   Transaction ID
  */
-function give_paypal_get_payment_transaction_id( $payment_id ) {
+function give_paypal_get_payment_transaction_id($payment_id) {
 
 	$transaction_id = '';
-	$notes          = give_get_payment_notes( $payment_id );
+	$notes          = give_get_payment_notes($payment_id);
 
-	foreach ( $notes as $note ) {
-		if ( preg_match( '/^PayPal Transaction ID: ([^\s]+)/', $note->comment_content, $match ) ) {
+	foreach ($notes as $note) {
+		if (preg_match('/^PayPal Transaction ID: ([^\s]+)/', $note->comment_content, $match)) {
 			$transaction_id = $match[1];
 			continue;
 		}
 	}
 
-	return apply_filters( 'give_paypal_set_payment_transaction_id', $transaction_id, $payment_id );
+	return apply_filters('give_paypal_set_payment_transaction_id', $transaction_id, $payment_id);
 }
 
-add_filter( 'give_get_payment_transaction_id-paypal', 'give_paypal_get_payment_transaction_id', 10, 1 );
+add_filter('give_get_payment_transaction_id-paypal', 'give_paypal_get_payment_transaction_id', 10, 1);
 
 /**
  * Given a transaction ID, generate a link to the PayPal transaction ID details
@@ -561,16 +561,16 @@ 
  *
  * @return string                 A link to the PayPal transaction details
  */
-function give_paypal_link_transaction_id( $transaction_id, $payment_id ) {
+function give_paypal_link_transaction_id($transaction_id, $payment_id) {
 
 	$paypal_base_url = 'https://history.paypal.com/cgi-bin/webscr?cmd=_history-details-from-hub&id=';
-	$transaction_url = '<a href="' . esc_url( $paypal_base_url . $transaction_id ) . '" target="_blank">' . $transaction_id . '</a>';
+	$transaction_url = '<a href="'.esc_url($paypal_base_url.$transaction_id).'" target="_blank">'.$transaction_id.'</a>';
 
-	return apply_filters( 'give_paypal_link_payment_details_transaction_id', $transaction_url );
+	return apply_filters('give_paypal_link_payment_details_transaction_id', $transaction_url);
 
 }
 
-add_filter( 'give_payment_details_transaction_id-paypal', 'give_paypal_link_transaction_id', 10, 2 );
+add_filter('give_payment_details_transaction_id-paypal', 'give_paypal_link_transaction_id', 10, 2);
 
 
 /**
@@ -582,64 +582,64 @@ 
  *
  * @return string
  */
-function give_paypal_get_pending_donation_note( $pending_reason ) {
+function give_paypal_get_pending_donation_note($pending_reason) {
 
 	$note = '';
 
-	switch ( $pending_reason ) {
+	switch ($pending_reason) {
 
 		case 'echeck' :
 
-			$note = esc_html__( 'Payment made via eCheck and will clear automatically in 5-8 days.', 'give' );
+			$note = esc_html__('Payment made via eCheck and will clear automatically in 5-8 days.', 'give');
 
 			break;
 
 		case 'address' :
 
-			$note = esc_html__( 'Payment requires a confirmed donor address and must be accepted manually through PayPal.', 'give' );
+			$note = esc_html__('Payment requires a confirmed donor address and must be accepted manually through PayPal.', 'give');
 
 			break;
 
 		case 'intl' :
 
-			$note = esc_html__( 'Payment must be accepted manually through PayPal due to international account regulations.', 'give' );
+			$note = esc_html__('Payment must be accepted manually through PayPal due to international account regulations.', 'give');
 
 			break;
 
 		case 'multi-currency' :
 
-			$note = esc_html__( 'Payment received in non-shop currency and must be accepted manually through PayPal.', 'give' );
+			$note = esc_html__('Payment received in non-shop currency and must be accepted manually through PayPal.', 'give');
 
 			break;
 
 		case 'paymentreview' :
 		case 'regulatory_review' :
 
-			$note = esc_html__( 'Payment is being reviewed by PayPal staff as high-risk or in possible violation of government regulations.', 'give' );
+			$note = esc_html__('Payment is being reviewed by PayPal staff as high-risk or in possible violation of government regulations.', 'give');
 
 			break;
 
 		case 'unilateral' :
 
-			$note = esc_html__( 'Payment was sent to non-confirmed or non-registered email address.', 'give' );
+			$note = esc_html__('Payment was sent to non-confirmed or non-registered email address.', 'give');
 
 			break;
 
 		case 'upgrade' :
 
-			$note = esc_html__( 'PayPal account must be upgraded before this payment can be accepted.', 'give' );
+			$note = esc_html__('PayPal account must be upgraded before this payment can be accepted.', 'give');
 
 			break;
 
 		case 'verify' :
 
-			$note = esc_html__( 'PayPal account is not verified. Verify account in order to accept this donation.', 'give' );
+			$note = esc_html__('PayPal account is not verified. Verify account in order to accept this donation.', 'give');
 
 			break;
 
 		case 'other' :
 
-			$note = esc_html__( 'Payment is pending for unknown reasons. Contact PayPal support for assistance.', 'give' );
+			$note = esc_html__('Payment is pending for unknown reasons. Contact PayPal support for assistance.', 'give');
 
 			break;
 
@@ -657,49 +657,49 @@ 
  *
  * @return mixed|string
  */
-function give_build_paypal_url( $payment_id, $payment_data ) {
+function give_build_paypal_url($payment_id, $payment_data) {
 	// Only send to PayPal if the pending payment is created successfully.
-	$listener_url = add_query_arg( 'give-listener', 'IPN', home_url( 'index.php' ) );
+	$listener_url = add_query_arg('give-listener', 'IPN', home_url('index.php'));
 
 	// Get the success url.
-	$return_url = add_query_arg( array(
+	$return_url = add_query_arg(array(
 		'payment-confirmation' => 'paypal',
 		'payment-id'           => $payment_id,
 
-	), get_permalink( give_get_option( 'success_page' ) ) );
+	), get_permalink(give_get_option('success_page')));
 
 	// Get the PayPal redirect uri.
-	$paypal_redirect = trailingslashit( give_get_paypal_redirect() ) . '?';
+	$paypal_redirect = trailingslashit(give_get_paypal_redirect()).'?';
 
 	// Item name.
-	$item_name = give_build_paypal_item_title( $payment_data );
+	$item_name = give_build_paypal_item_title($payment_data);
 
 	// Setup PayPal API params.
 	$paypal_args = array(
-		'business'      => give_get_option( 'paypal_email', false ),
+		'business'      => give_get_option('paypal_email', false),
 		'first_name'    => $payment_data['user_info']['first_name'],
 		'last_name'     => $payment_data['user_info']['last_name'],
 		'email'         => $payment_data['user_email'],
 		'invoice'       => $payment_data['purchase_key'],
 		'amount'        => $payment_data['price'],
-		'item_name'     => stripslashes( $item_name ),
+		'item_name'     => stripslashes($item_name),
 		'no_shipping'   => '1',
 		'shipping'      => '0',
 		'no_note'       => '1',
 		'currency_code' => give_get_currency(),
-		'charset'       => get_bloginfo( 'charset' ),
+		'charset'       => get_bloginfo('charset'),
 		'custom'        => $payment_id,
 		'rm'            => '2',
 		'return'        => $return_url,
-		'cancel_return' => give_get_failed_transaction_uri( '?payment-id=' . $payment_id ),
+		'cancel_return' => give_get_failed_transaction_uri('?payment-id='.$payment_id),
 		'notify_url'    => $listener_url,
 		'page_style'    => give_get_paypal_page_style(),
-		'cbt'           => get_bloginfo( 'name' ),
+		'cbt'           => get_bloginfo('name'),
 		'bn'            => 'givewp_SP',
 	);
 
 	// Add user address if present.
-	if ( ! empty( $payment_data['user_info']['address'] ) ) {
+	if ( ! empty($payment_data['user_info']['address'])) {
 		$default_address = array(
 			'line1'   => '',
 			'line2'   => '',
@@ -709,13 +709,13 @@ 
 			'country' => '',
 		);
 
-		$address = wp_parse_args( $payment_data['user_info']['address'], $default_address );
+		$address = wp_parse_args($payment_data['user_info']['address'], $default_address);
 
 		$paypal_args['address1'] = $address['line1'];
 		$paypal_args['address2'] = $address['line2'];
 		$paypal_args['city']     = $address['city'];
 		$paypal_args['state']    = $address['state'];
-		$paypal_args['zip']    = $address['zip'];
+		$paypal_args['zip'] = $address['zip'];
 		$paypal_args['country']  = $address['country'];
 	}
 
@@ -730,13 +730,13 @@ 
 	 * @param array $paypal_args
 	 * @param array $payment_data
 	 */
-	$paypal_args = apply_filters( 'give_paypal_redirect_args', $paypal_args, $payment_data );
+	$paypal_args = apply_filters('give_paypal_redirect_args', $paypal_args, $payment_data);
 
 	// Build query.
-	$paypal_redirect .= http_build_query( $paypal_args );
+	$paypal_redirect .= http_build_query($paypal_args);
 
 	// Fix for some sites that encode the entities.
-	$paypal_redirect = str_replace( '&amp;', '&', $paypal_redirect );
+	$paypal_redirect = str_replace('&amp;', '&', $paypal_redirect);
 
 	return $paypal_redirect;
 }
@@ -751,7 +751,7 @@ 
 function give_get_paypal_button_type() {
 	// paypal_button_type can be donation or standard.
 	$paypal_button_type = '_donations';
-	if ( give_get_option( 'paypal_button_type' ) === 'standard' ) {
+	if (give_get_option('paypal_button_type') === 'standard') {
 		$paypal_button_type = '_xclick';
 	}
 
@@ -768,32 +768,32 @@ 
  *
  * @return string
  */
-function give_build_paypal_item_title( $payment_data ) {
-	$form_id   = intval( $payment_data['post_data']['give-form-id'] );
+function give_build_paypal_item_title($payment_data) {
+	$form_id   = intval($payment_data['post_data']['give-form-id']);
 	$item_name = $payment_data['post_data']['give-form-title'];
 
 	// Verify has variable prices.
-	if ( give_has_variable_prices( $form_id ) && isset( $payment_data['post_data']['give-price-id'] ) ) {
+	if (give_has_variable_prices($form_id) && isset($payment_data['post_data']['give-price-id'])) {
 
-		$item_price_level_text = give_get_price_option_name( $form_id, $payment_data['post_data']['give-price-id'] );
-		$price_level_amount    = give_get_price_option_amount( $form_id, $payment_data['post_data']['give-price-id'] );
+		$item_price_level_text = give_get_price_option_name($form_id, $payment_data['post_data']['give-price-id']);
+		$price_level_amount    = give_get_price_option_amount($form_id, $payment_data['post_data']['give-price-id']);
 
 		// Donation given doesn't match selected level (must be a custom amount).
-		if ( $price_level_amount != give_sanitize_amount( $payment_data['price'] ) ) {
-			$custom_amount_text = give_get_meta( $form_id, '_give_custom_amount_text', true );
+		if ($price_level_amount != give_sanitize_amount($payment_data['price'])) {
+			$custom_amount_text = give_get_meta($form_id, '_give_custom_amount_text', true);
 			// user custom amount text if any, fallback to default if not.
-			$item_name .= ' - ' . give_check_variable( $custom_amount_text, 'empty', esc_html__( 'Custom Amount', 'give' ) );
+			$item_name .= ' - '.give_check_variable($custom_amount_text, 'empty', esc_html__('Custom Amount', 'give'));
 
 		} //Is there any donation level text?
-		elseif ( ! empty( $item_price_level_text ) ) {
-			$item_name .= ' - ' . $item_price_level_text;
+		elseif ( ! empty($item_price_level_text)) {
+			$item_name .= ' - '.$item_price_level_text;
 		}
 
 	} //Single donation: Custom Amount.
-	elseif ( give_get_form_price( $form_id ) !== give_sanitize_amount( $payment_data['price'] ) ) {
-		$custom_amount_text = give_get_meta( $form_id, '_give_custom_amount_text', true );
+	elseif (give_get_form_price($form_id) !== give_sanitize_amount($payment_data['price'])) {
+		$custom_amount_text = give_get_meta($form_id, '_give_custom_amount_text', true);
 		// user custom amount text if any, fallback to default if not.
-		$item_name .= ' - ' . give_check_variable( $custom_amount_text, 'empty', esc_html__( 'Custom Amount', 'give' ) );
+		$item_name .= ' - '.give_check_variable($custom_amount_text, 'empty', esc_html__('Custom Amount', 'give'));
 	}
 
 	return $item_name;

includes/admin/settings/class-settings-license.php

Spacing +11 added lines, -11 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_License' ) ) :
+if ( ! class_exists('Give_Settings_License')) :
 
 	/**
 	 * Give_Settings_License.
@@ -43,10 +43,10 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'licenses';
-			$this->label = esc_html__( 'Licenses', 'give' );
+			$this->label = esc_html__('Licenses', 'give');
 
-			add_filter( 'give-settings_tabs_array', array( $this, 'add_settings_page' ), 20 );
-			add_action( "give-settings_settings_{$this->id}_page", array( $this, 'output' ) );
+			add_filter('give-settings_tabs_array', array($this, 'add_settings_page'), 20);
+			add_action("give-settings_settings_{$this->id}_page", array($this, 'output'));
 		}
 
 		/**
@@ -56,11 +56,11 @@ 
 		 * @param  array $pages Lst of pages.
 		 * @return array
 		 */
-		public function add_settings_page( $pages ) {
+		public function add_settings_page($pages) {
 			$setting = $this->get_settings();
 			// Bailout: Do not add licenses setting tab if it does not contain any setting fields.
-			if( ! empty( $setting ) ) {
-				$pages[ $this->id ] = $this->label;
+			if ( ! empty($setting)) {
+				$pages[$this->id] = $this->label;
 			}
 
 			return $pages;
@@ -79,7 +79,7 @@ 
 			 * Filter the licenses settings.
 			 * Backward compatibility: Please do not use this filter. This filter is deprecated in 1.8
 			 */
-			$settings = apply_filters( 'give_settings_licenses', $settings );
+			$settings = apply_filters('give_settings_licenses', $settings);
 
 			/**
 			 * Filter the settings.
@@ -87,7 +87,7 @@ 
 			 * @since  1.8
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, $settings );
+			$settings = apply_filters('give_get_settings_'.$this->id, $settings);
 
 			// Output.
 			return $settings;
@@ -102,7 +102,7 @@ 
 		public function output() {
 			$settings = $this->get_settings();
 
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
 		}
 	}
 

includes/admin/settings/class-settings-addon.php

Spacing +11 added lines, -11 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_Addon' ) ) :
+if ( ! class_exists('Give_Settings_Addon')) :
 
 	/**
 	 * Give_Settings_Addon.
@@ -26,7 +26,7 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'addons';
-			$this->label = esc_html__( 'Add-ons', 'give' );
+			$this->label = esc_html__('Add-ons', 'give');
 
 			parent::__construct();
 		}
@@ -38,12 +38,12 @@ 
 		 * @param  $setting_tab
 		 * @return string
 		 */
-		function set_default_setting_tab( $setting_tab ) {
+		function set_default_setting_tab($setting_tab) {
 			$default_tab = '';
 
 			// Set default tab to first setting tab.
-			if( $sections = array_keys( $this->get_sections() ) ) {
-				$default_tab = current( $sections );
+			if ($sections = array_keys($this->get_sections())) {
+				$default_tab = current($sections);
 			}
 			return $default_tab;
 		}
@@ -55,11 +55,11 @@ 
 		 * @param  array $pages Lst of pages.
 		 * @return array
 		 */
-		public function add_settings_page( $pages ) {
+		public function add_settings_page($pages) {
 			$setting = $this->get_settings();
 			// Bailout: Do not add addons setting tab if it does not contain any setting fields.
-			if( ! empty( $setting ) ) {
-				$pages[ $this->id ] = $this->label;
+			if ( ! empty($setting)) {
+				$pages[$this->id] = $this->label;
 			}
 
 			return $pages;
@@ -78,7 +78,7 @@ 
 			 * Filter the addons settings.
 			 * Backward compatibility: Please do not use this filter. This filter is deprecated in 1.8
 			 */
-			$settings = apply_filters( 'give_settings_addons', $settings );
+			$settings = apply_filters('give_settings_addons', $settings);
 
 			/**
 			 * Filter the settings.
@@ -86,7 +86,7 @@ 
 			 * @since  1.8
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, $settings );
+			$settings = apply_filters('give_get_settings_'.$this->id, $settings);
 
 			// Output.
 			return $settings;

includes/admin/admin-filters.php

Spacing +4 added lines, -4 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -26,7 +26,7 @@ 
  *
  * @return  mixed
  */
-function _give_sanitize_number_decimals_setting_field( $value ) {
-	return absint( $value );
+function _give_sanitize_number_decimals_setting_field($value) {
+	return absint($value);
 }
-add_filter( 'give_admin_settings_sanitize_option_number_decimals', '_give_sanitize_number_decimals_setting_field', 10 );
\ No newline at end of file
+add_filter('give_admin_settings_sanitize_option_number_decimals', '_give_sanitize_number_decimals_setting_field', 10);
\ No newline at end of file

includes/admin/forms/metabox.php

Spacing +6 added lines, -6 removed lines

@@ -10,7 +10,7 @@ 
  */

 

 // Exit if accessed directly.

-if ( ! defined( 'ABSPATH' ) ) {

+if ( ! defined('ABSPATH')) {

 	exit;

 }

 

@@ -22,19 +22,19 @@ 
  */

 function give_add_shortcode_to_publish_metabox() {

 

-	if ( 'give_forms' !== get_post_type() ) {

+	if ('give_forms' !== get_post_type()) {

 		return false;

 	}

 	global $post;

 

 	//Only enqueue scripts for CPT on post type screen

-	if ( 'give_forms' === $post->post_type ) {

+	if ('give_forms' === $post->post_type) {

 		//Shortcode column with select all input

-		$shortcode = htmlentities( '[give_form id="' . $post->ID . '"]' );

-		echo '<div class="shortcode-wrap box-sizing"><label for="shortcode-input">' . esc_html__( 'Give Form Shortcode:', 'give' ) . '</label><input onClick="this.setSelectionRange(0, this.value.length)" type="text" name="shortcode-input" id="shortcode-input" class="shortcode-input" readonly value="' . $shortcode . '"></div>';

+		$shortcode = htmlentities('[give_form id="'.$post->ID.'"]');

+		echo '<div class="shortcode-wrap box-sizing"><label for="shortcode-input">'.esc_html__('Give Form Shortcode:', 'give').'</label><input onClick="this.setSelectionRange(0, this.value.length)" type="text" name="shortcode-input" id="shortcode-input" class="shortcode-input" readonly value="'.$shortcode.'"></div>';

 

 	}

 

 }

 

-add_action( 'post_submitbox_misc_actions', 'give_add_shortcode_to_publish_metabox' );

+add_action('post_submitbox_misc_actions', 'give_add_shortcode_to_publish_metabox');

includes/admin/admin-footer.php

Spacing +6 added lines, -6 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,14 +24,14 @@ 
  *
  * @return      string
  */
-function give_admin_rate_us( $footer_text ) {
+function give_admin_rate_us($footer_text) {
 	global $typenow;
 
-	if ( $typenow == 'give_forms' ) {
+	if ($typenow == 'give_forms') {
 		$rate_text = sprintf(
 			/* translators: %s: Link to 5 star rating */
-			__( 'If you like <strong>Give</strong> please leave us a %s rating. It takes a minute and helps a lot. Thanks in advance!', 'give' ),
-			'<a href="https://wordpress.org/support/view/plugin-reviews/give?filter=5#postform" target="_blank" class="give-rating-link" style="text-decoration:none;" data-rated="' . esc_attr__( 'Thanks :)', 'give' ) . '">&#9733;&#9733;&#9733;&#9733;&#9733;</a>'
+			__('If you like <strong>Give</strong> please leave us a %s rating. It takes a minute and helps a lot. Thanks in advance!', 'give'),
+			'<a href="https://wordpress.org/support/view/plugin-reviews/give?filter=5#postform" target="_blank" class="give-rating-link" style="text-decoration:none;" data-rated="'.esc_attr__('Thanks :)', 'give').'">&#9733;&#9733;&#9733;&#9733;&#9733;</a>'
 		);
 
 		return $rate_text;
@@ -40,4 +40,4 @@ 
 	}
 }
 
-add_filter( 'admin_footer_text', 'give_admin_rate_us' );
+add_filter('admin_footer_text', 'give_admin_rate_us');

includes/admin/admin-actions.php

Doc Comments +1 added lines, -1 removed lines

@@ -98,7 +98,7 @@
  *
  * @since 1.8
  *
- * @return bool
+ * @return false|null
  */
 function give_redirect_to_clean_url_admin_pages() {
 	// Give admin pages.

Indentation +10 added lines, -10 removed lines

@@ -158,20 +158,20 @@
 
 	$current_user = wp_get_current_user();
 
-    // Get notice id.
-    $notice_id = sanitize_text_field( $_POST['_give_hide_license_notices_shortly'] );
+	// Get notice id.
+	$notice_id = sanitize_text_field( $_POST['_give_hide_license_notices_shortly'] );
 
-    // Transient key name.
-    $transient_key = "_give_hide_license_notices_shortly_{$current_user->ID}_{$notice_id}";
+	// Transient key name.
+	$transient_key = "_give_hide_license_notices_shortly_{$current_user->ID}_{$notice_id}";
 
-    if ( Give_Cache::get( $transient_key, true ) ) {
-        return;
-    }
+	if ( Give_Cache::get( $transient_key, true ) ) {
+		return;
+	}
 
-    // Hide notice for 24 hours.
-    Give_Cache::set( $transient_key, true, DAY_IN_SECONDS, true );
+	// Hide notice for 24 hours.
+	Give_Cache::set( $transient_key, true, DAY_IN_SECONDS, true );
 
-    die();
+	die();
 
 }
 

Spacing +31 added lines, -31 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,53 +24,53 @@ 
 function give_hide_subscription_notices() {
 
 	// Hide subscription notices permanently.
-	if ( ! empty( $_GET['_give_hide_license_notices_permanently'] ) ) {
+	if ( ! empty($_GET['_give_hide_license_notices_permanently'])) {
 		$current_user = wp_get_current_user();
 
 		// check previously disabled notice ids.
-		$already_dismiss_notices = ( $already_dismiss_notices = get_user_meta( $current_user->ID, '_give_hide_license_notices_permanently', true ) )
+		$already_dismiss_notices = ($already_dismiss_notices = get_user_meta($current_user->ID, '_give_hide_license_notices_permanently', true))
 			? $already_dismiss_notices
 			: array();
 
 		// Get notice id.
-		$notice_id = sanitize_text_field( $_GET['_give_hide_license_notices_permanently'] );
+		$notice_id = sanitize_text_field($_GET['_give_hide_license_notices_permanently']);
 
-		if ( ! in_array( $notice_id, $already_dismiss_notices ) ) {
+		if ( ! in_array($notice_id, $already_dismiss_notices)) {
 			$already_dismiss_notices[] = $notice_id;
 		}
 
 		// Store subscription ids.
-		update_user_meta( $current_user->ID, '_give_hide_license_notices_permanently', $already_dismiss_notices );
+		update_user_meta($current_user->ID, '_give_hide_license_notices_permanently', $already_dismiss_notices);
 
 		// Redirect user.
-		wp_safe_redirect( remove_query_arg( '_give_hide_license_notices_permanently', $_SERVER['REQUEST_URI'] ) );
+		wp_safe_redirect(remove_query_arg('_give_hide_license_notices_permanently', $_SERVER['REQUEST_URI']));
 		exit();
 	}
 
 	// Hide subscription notices shortly.
-	if ( ! empty( $_GET['_give_hide_license_notices_shortly'] ) ) {
+	if ( ! empty($_GET['_give_hide_license_notices_shortly'])) {
 		$current_user = wp_get_current_user();
 
 		// Get notice id.
-		$notice_id = sanitize_text_field( $_GET['_give_hide_license_notices_shortly'] );
+		$notice_id = sanitize_text_field($_GET['_give_hide_license_notices_shortly']);
 
 		// Transient key name.
 		$transient_key = "_give_hide_license_notices_shortly_{$current_user->ID}_{$notice_id}";
 
-		if ( Give_Cache::get( $transient_key, true ) ) {
+		if (Give_Cache::get($transient_key, true)) {
 			return;
 		}
 
 		// Hide notice for 24 hours.
-		Give_Cache::set( $transient_key, true, DAY_IN_SECONDS, true );
+		Give_Cache::set($transient_key, true, DAY_IN_SECONDS, true);
 
 		// Redirect user.
-		wp_safe_redirect( remove_query_arg( '_give_hide_license_notices_shortly', $_SERVER['REQUEST_URI'] ) );
+		wp_safe_redirect(remove_query_arg('_give_hide_license_notices_shortly', $_SERVER['REQUEST_URI']));
 		exit();
 	}
 }
 
-add_action( 'admin_init', 'give_hide_subscription_notices' );
+add_action('admin_init', 'give_hide_subscription_notices');
 
 /**
  * Load wp editor by ajax.
@@ -78,19 +78,19 @@ 
  * @since 1.8
  */
 function give_load_wp_editor() {
-	if ( ! isset( $_POST['wp_editor'] ) ) {
+	if ( ! isset($_POST['wp_editor'])) {
 		die();
 	}
 
-	$wp_editor                     = json_decode( base64_decode( $_POST['wp_editor'] ), true );
+	$wp_editor                     = json_decode(base64_decode($_POST['wp_editor']), true);
 	$wp_editor[2]['textarea_name'] = $_POST['textarea_name'];
 
-	wp_editor( $wp_editor[0], $_POST['wp_editor_id'], $wp_editor[2] );
+	wp_editor($wp_editor[0], $_POST['wp_editor_id'], $wp_editor[2]);
 
 	die();
 }
 
-add_action( 'wp_ajax_give_load_wp_editor', 'give_load_wp_editor' );
+add_action('wp_ajax_give_load_wp_editor', 'give_load_wp_editor');
 
 
 /**
@@ -109,13 +109,13 @@ 
 	);
 
 	// Get current page.
-	$current_page = isset( $_GET['page'] ) ? esc_attr( $_GET['page'] ) : '';
+	$current_page = isset($_GET['page']) ? esc_attr($_GET['page']) : '';
 
 	// Bailout.
 	if (
-		empty( $current_page )
-		|| empty( $_GET['_wp_http_referer'] )
-		|| ! in_array( $current_page, $give_pages )
+		empty($current_page)
+		|| empty($_GET['_wp_http_referer'])
+		|| ! in_array($current_page, $give_pages)
 	) {
 		return false;
 	}
@@ -125,21 +125,21 @@ 
 	 *
 	 * @since 1.8
 	 */
-	$redirect = apply_filters( "give_validate_{$current_page}", true );
+	$redirect = apply_filters("give_validate_{$current_page}", true);
 
-	if ( $redirect ) {
+	if ($redirect) {
 		// Redirect.
 		wp_redirect(
 			remove_query_arg(
-				array( '_wp_http_referer', '_wpnonce' ),
-				wp_unslash( $_SERVER['REQUEST_URI'] )
+				array('_wp_http_referer', '_wpnonce'),
+				wp_unslash($_SERVER['REQUEST_URI'])
 			)
 		);
 		exit;
 	}
 }
 
-add_action( 'admin_init', 'give_redirect_to_clean_url_admin_pages' );
+add_action('admin_init', 'give_redirect_to_clean_url_admin_pages');
 
 /**
  * Hide License Notice Shortly.
@@ -152,27 +152,27 @@ 
  */
 function give_hide_license_notice() {
 
-	if ( ! isset( $_POST['_give_hide_license_notices_shortly'] ) ) {
+	if ( ! isset($_POST['_give_hide_license_notices_shortly'])) {
 		die();
 	}
 
 	$current_user = wp_get_current_user();
 
     // Get notice id.
-    $notice_id = sanitize_text_field( $_POST['_give_hide_license_notices_shortly'] );
+    $notice_id = sanitize_text_field($_POST['_give_hide_license_notices_shortly']);
 
     // Transient key name.
     $transient_key = "_give_hide_license_notices_shortly_{$current_user->ID}_{$notice_id}";
 
-    if ( Give_Cache::get( $transient_key, true ) ) {
+    if (Give_Cache::get($transient_key, true)) {
         return;
     }
 
     // Hide notice for 24 hours.
-    Give_Cache::set( $transient_key, true, DAY_IN_SECONDS, true );
+    Give_Cache::set($transient_key, true, DAY_IN_SECONDS, true);
 
     die();
 
 }
 
-add_action( 'wp_ajax_give_hide_license_notice', 'give_hide_license_notice' );
+add_action('wp_ajax_give_hide_license_notice', 'give_hide_license_notice');

includes/admin/class-admin-notices.php

Doc Comments +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
 	 *
 	 * @param WP_Admin_Bar $wp_admin_bar WP_Admin_Bar instance, passed by reference.
 	 *
-	 * @return bool
+	 * @return false|null
 	 */
 	public function give_admin_bar_menu( $wp_admin_bar ) {
 

Spacing +68 added lines, -68 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -33,9 +33,9 @@ 
 	 * @since 1.0
 	 */
 	public function __construct() {
-		add_action( 'admin_notices', array( $this, 'show_notices' ) );
-		add_action( 'give_dismiss_notices', array( $this, 'dismiss_notices' ) );
-		add_action( 'admin_bar_menu', array( $this, 'give_admin_bar_menu' ), 1000, 1 );
+		add_action('admin_notices', array($this, 'show_notices'));
+		add_action('give_dismiss_notices', array($this, 'dismiss_notices'));
+		add_action('admin_bar_menu', array($this, 'give_admin_bar_menu'), 1000, 1);
 
 		$this->notices = array(
 			'updated' => array(),
@@ -51,20 +51,20 @@ 
 	 *
 	 * @return bool
 	 */
-	public function give_admin_bar_menu( $wp_admin_bar ) {
+	public function give_admin_bar_menu($wp_admin_bar) {
 
-		if ( ! give_is_test_mode() || ! current_user_can( 'view_give_reports' ) ) {
+		if ( ! give_is_test_mode() || ! current_user_can('view_give_reports')) {
 			return false;
 		}
 
 		// Add the main siteadmin menu item.
-		$wp_admin_bar->add_menu( array(
+		$wp_admin_bar->add_menu(array(
 			'id'     => 'give-test-notice',
-			'href'   => admin_url( 'edit.php?post_type=give_forms&page=give-settings&tab=gateways' ),
+			'href'   => admin_url('edit.php?post_type=give_forms&page=give-settings&tab=gateways'),
 			'parent' => 'top-secondary',
-			'title'  => esc_html__( 'Give Test Mode Active', 'give' ),
-			'meta'   => array( 'class' => 'give-test-mode-active' ),
-		) );
+			'title'  => esc_html__('Give Test Mode Active', 'give'),
+			'meta'   => array('class' => 'give-test-mode-active'),
+		));
 
 	}
 
@@ -75,100 +75,100 @@ 
 	 */
 	public function show_notices() {
 
-		if ( ! give_test_ajax_works() && ! get_user_meta( get_current_user_id(), '_give_admin_ajax_inaccessible_dismissed', true ) && current_user_can( 'manage_give_settings' ) ) {
+		if ( ! give_test_ajax_works() && ! get_user_meta(get_current_user_id(), '_give_admin_ajax_inaccessible_dismissed', true) && current_user_can('manage_give_settings')) {
 			echo '<div class="error">';
-			echo '<p>' . __( 'Your site appears to be blocking the WordPress ajax interface. This may cause issues with Give.', 'give' ) . '</p>';
+			echo '<p>'.__('Your site appears to be blocking the WordPress ajax interface. This may cause issues with Give.', 'give').'</p>';
 			/* translators: %s: http://docs.givewp.com/ajax-blocked */
-			echo '<p>' . sprintf( __( 'Please see <a href="%s" target="_blank">this reference</a> for possible solutions.', 'give' ), esc_url( 'http://docs.givewp.com/ajax-blocked' ) ) . '</p>';
-			echo '<p><a href="' . add_query_arg( array(
+			echo '<p>'.sprintf(__('Please see <a href="%s" target="_blank">this reference</a> for possible solutions.', 'give'), esc_url('http://docs.givewp.com/ajax-blocked')).'</p>';
+			echo '<p><a href="'.add_query_arg(array(
 					'give_action' => 'dismiss_notices',
 					'give_notice' => 'admin_ajax_inaccessible',
-				) ) . '">' . __( 'Dismiss Notice', 'give' ) . '</a></p>';
+				)).'">'.__('Dismiss Notice', 'give').'</a></p>';
 			echo '</div>';
 		}
 
-		if ( isset( $_GET['give-message'] ) ) {
+		if (isset($_GET['give-message'])) {
 
 			// Donation reports errors.
-			if ( current_user_can( 'view_give_reports' ) ) {
-				switch ( $_GET['give-message'] ) {
+			if (current_user_can('view_give_reports')) {
+				switch ($_GET['give-message']) {
 					case 'donation_deleted' :
-						$this->notices['updated']['give-donation-deleted'] = __( 'The donation has been deleted.', 'give' );
+						$this->notices['updated']['give-donation-deleted'] = __('The donation has been deleted.', 'give');
 						break;
 					case 'email_sent' :
-						$this->notices['updated']['give-payment-sent'] = __( 'The donation receipt has been resent.', 'give' );
+						$this->notices['updated']['give-payment-sent'] = __('The donation receipt has been resent.', 'give');
 						break;
 					case 'refreshed-reports' :
-						$this->notices['updated']['give-refreshed-reports'] = __( 'The reports cache has been cleared.', 'give' );
+						$this->notices['updated']['give-refreshed-reports'] = __('The reports cache has been cleared.', 'give');
 						break;
 					case 'donation-note-deleted' :
-						$this->notices['updated']['give-donation-note-deleted'] = __( 'The donation note has been deleted.', 'give' );
+						$this->notices['updated']['give-donation-note-deleted'] = __('The donation note has been deleted.', 'give');
 						break;
 				}
 			}
 
 			// Give settings notices and errors.
-			if ( current_user_can( 'manage_give_settings' ) ) {
-				switch ( $_GET['give-message'] ) {
+			if (current_user_can('manage_give_settings')) {
+				switch ($_GET['give-message']) {
 					case 'settings-imported' :
-						$this->notices['updated']['give-settings-imported'] = __( 'The settings have been imported.', 'give' );
+						$this->notices['updated']['give-settings-imported'] = __('The settings have been imported.', 'give');
 						break;
 					case 'api-key-generated' :
-						$this->notices['updated']['give-api-key-generated'] = __( 'API keys have been generated.', 'give' );
+						$this->notices['updated']['give-api-key-generated'] = __('API keys have been generated.', 'give');
 						break;
 					case 'api-key-exists' :
-						$this->notices['error']['give-api-key-exists'] = __( 'The specified user already has API keys.', 'give' );
+						$this->notices['error']['give-api-key-exists'] = __('The specified user already has API keys.', 'give');
 						break;
 					case 'api-key-regenerated' :
-						$this->notices['updated']['give-api-key-regenerated'] = __( 'API keys have been regenerated.', 'give' );
+						$this->notices['updated']['give-api-key-regenerated'] = __('API keys have been regenerated.', 'give');
 						break;
 					case 'api-key-revoked' :
-						$this->notices['updated']['give-api-key-revoked'] = __( 'API keys have been revoked.', 'give' );
+						$this->notices['updated']['give-api-key-revoked'] = __('API keys have been revoked.', 'give');
 						break;
 					case 'sent-test-email' :
-						$this->notices['updated']['give-sent-test-email'] = __( 'The test email has been sent.', 'give' );
+						$this->notices['updated']['give-sent-test-email'] = __('The test email has been sent.', 'give');
 						break;
 					case 'matched-success-failure-page':
-						$this->notices['updated']['give-matched-success-failure-page'] = __( 'You cannot set the success and failed pages to the same page', 'give' );
+						$this->notices['updated']['give-matched-success-failure-page'] = __('You cannot set the success and failed pages to the same page', 'give');
 				}
 			}
 			// Payments errors.
-			if ( current_user_can( 'edit_give_payments' ) ) {
-				switch ( $_GET['give-message'] ) {
+			if (current_user_can('edit_give_payments')) {
+				switch ($_GET['give-message']) {
 					case 'note-added' :
-						$this->notices['updated']['give-note-added'] = __( 'The donation note has been added.', 'give' );
+						$this->notices['updated']['give-note-added'] = __('The donation note has been added.', 'give');
 						break;
 					case 'payment-updated' :
-						$this->notices['updated']['give-payment-updated'] = __( 'The donation has been updated.', 'give' );
+						$this->notices['updated']['give-payment-updated'] = __('The donation has been updated.', 'give');
 						break;
 				}
 			}
 
 			// Customer Notices.
-			if ( current_user_can( 'edit_give_payments' ) ) {
-				switch ( $_GET['give-message'] ) {
+			if (current_user_can('edit_give_payments')) {
+				switch ($_GET['give-message']) {
 					case 'customer-deleted' :
-						$this->notices['updated']['give-customer-deleted'] = __( 'The donor has been deleted.', 'give' );
+						$this->notices['updated']['give-customer-deleted'] = __('The donor has been deleted.', 'give');
 						break;
 
 					case 'email-added' :
-						$this->notices['updated']['give-customer-email-added'] = __( 'Donor email added', 'give' );
+						$this->notices['updated']['give-customer-email-added'] = __('Donor email added', 'give');
 						break;
 
 					case 'email-removed' :
-						$this->notices['updated']['give-customer-email-removed'] = __( 'Donor email removed', 'give' );
+						$this->notices['updated']['give-customer-email-removed'] = __('Donor email removed', 'give');
 						break;
 
 					case 'email-remove-failed' :
-						$this->notices['error']['give-customer-email-remove-failed'] = __( 'Failed to remove donor email', 'give' );
+						$this->notices['error']['give-customer-email-remove-failed'] = __('Failed to remove donor email', 'give');
 						break;
 
 					case 'primary-email-updated' :
-						$this->notices['updated']['give-customer-primary-email-updated'] = __( 'Primary email updated for donor', 'give' );
+						$this->notices['updated']['give-customer-primary-email-updated'] = __('Primary email updated for donor', 'give');
 						break;
 
 					case 'primary-email-failed' :
-						$this->notices['error']['give-customer-primary-email-failed'] = __( 'Failed to set primary email', 'give' );
+						$this->notices['error']['give-customer-primary-email-failed'] = __('Failed to set primary email', 'give');
 
 				}
 			}
@@ -176,19 +176,19 @@ 
 
 		$this->add_payment_bulk_action_notice();
 
-		if ( count( $this->notices['updated'] ) > 0 ) {
-			foreach ( $this->notices['updated'] as $notice => $message ) {
-				add_settings_error( 'give-notices', $notice, $message, 'updated' );
+		if (count($this->notices['updated']) > 0) {
+			foreach ($this->notices['updated'] as $notice => $message) {
+				add_settings_error('give-notices', $notice, $message, 'updated');
 			}
 		}
 
-		if ( count( $this->notices['error'] ) > 0 ) {
-			foreach ( $this->notices['error'] as $notice => $message ) {
-				add_settings_error( 'give-notices', $notice, $message, 'error' );
+		if (count($this->notices['error']) > 0) {
+			foreach ($this->notices['error'] as $notice => $message) {
+				add_settings_error('give-notices', $notice, $message, 'error');
 			}
 		}
 
-		settings_errors( 'give-notices' );
+		settings_errors('give-notices');
 
 	}
 
@@ -200,8 +200,8 @@ 
 	 * @return void
 	 */
 	function give_admin_addons_notices() {
-		add_settings_error( 'give-notices', 'give-addons-feed-error', __( 'There seems to be an issue with the server. Please try again in a few minutes.', 'give' ), 'error' );
-		settings_errors( 'give-notices' );
+		add_settings_error('give-notices', 'give-addons-feed-error', __('There seems to be an issue with the server. Please try again in a few minutes.', 'give'), 'error');
+		settings_errors('give-notices');
 	}
 
 
@@ -212,9 +212,9 @@ 
 	 * @return void
 	 */
 	function dismiss_notices() {
-		if ( isset( $_GET['give_notice'] ) ) {
-			update_user_meta( get_current_user_id(), '_give_' . $_GET['give_notice'] . '_dismissed', 1 );
-			wp_redirect( remove_query_arg( array( 'give_action', 'give_notice' ) ) );
+		if (isset($_GET['give_notice'])) {
+			update_user_meta(get_current_user_id(), '_give_'.$_GET['give_notice'].'_dismissed', 1);
+			wp_redirect(remove_query_arg(array('give_action', 'give_notice')));
 			exit;
 		}
 	}
@@ -229,24 +229,24 @@ 
 	 */
 	function add_payment_bulk_action_notice() {
 		if (
-			current_user_can( 'edit_give_payments' )
-			&& isset( $_GET['action'] )
-			&& ! empty( $_GET['action'] )
-			&& isset( $_GET['payment'] )
-			&& ! empty( $_GET['payment'] )
+			current_user_can('edit_give_payments')
+			&& isset($_GET['action'])
+			&& ! empty($_GET['action'])
+			&& isset($_GET['payment'])
+			&& ! empty($_GET['payment'])
 		) {
-			$payment_count = isset( $_GET['payment'] ) ? count( $_GET['payment'] ) : 0;
+			$payment_count = isset($_GET['payment']) ? count($_GET['payment']) : 0;
 
-			switch ( $_GET['action'] ) {
+			switch ($_GET['action']) {
 				case 'delete':
-					if ( $payment_count ) {
-						$this->notices['updated']['bulk_action_delete'] = sprintf( _n( 'Successfully deleted only one transaction.', 'Successfully deleted %d number of transactions.', $payment_count, 'give' ), $payment_count );
+					if ($payment_count) {
+						$this->notices['updated']['bulk_action_delete'] = sprintf(_n('Successfully deleted only one transaction.', 'Successfully deleted %d number of transactions.', $payment_count, 'give'), $payment_count);
 					}
 					break;
 
 				case 'resend-receipt':
-					if ( $payment_count ) {
-						$this->notices['updated']['bulk_action_resend_receipt'] = sprintf( _n( 'Successfully send email receipt to only one recipient.', 'Successfully send email receipts to %d recipients.', $payment_count, 'give' ), $payment_count );
+					if ($payment_count) {
+						$this->notices['updated']['bulk_action_resend_receipt'] = sprintf(_n('Successfully send email receipt to only one recipient.', 'Successfully send email receipts to %d recipients.', $payment_count, 'give'), $payment_count);
 					}
 					break;
 			}
@@ -267,7 +267,7 @@ 
 	 *
 	 * @return string
 	 */
-	public static function notice_html( $message, $type = 'updated' ) {
+	public static function notice_html($message, $type = 'updated') {
 		ob_start();
 		?>
 		<div class="<?php echo $type; ?> notice">