WordImpress / Give

includes/admin/payments/actions.php

Indentation +64 added lines, -64 removed lines

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

Spacing +118 added lines, -118 removed lines

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

includes/admin/reporting/class-export-earnings.php

Spacing +28 added lines, -28 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -38,16 +38,16 @@ 
 	 * @return void
 	 */
 	public function headers() {
-		ignore_user_abort( true );
+		ignore_user_abort(true);
 
-		if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-			set_time_limit( 0 );
+		if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+			set_time_limit(0);
 		}
 
 		nocache_headers();
-		header( 'Content-Type: text/csv; charset=utf-8' );
-		header( 'Content-Disposition: attachment; filename=' . apply_filters( 'give_earnings_export_filename', 'give-export-' . $this->export_type . '-' . date( 'n' ) . '-' . date( 'Y' ) ) . '.csv' );
-		header( "Expires: 0" );
+		header('Content-Type: text/csv; charset=utf-8');
+		header('Content-Disposition: attachment; filename='.apply_filters('give_earnings_export_filename', 'give-export-'.$this->export_type.'-'.date('n').'-'.date('Y')).'.csv');
+		header("Expires: 0");
 
 	}
 
@@ -61,10 +61,10 @@ 
 	public function csv_cols() {
 
 		$cols = array(
-			'date'      => esc_html__( 'Date', 'give' ),
-			'donations' => esc_html__( 'Donations', 'give' ),
+			'date'      => esc_html__('Date', 'give'),
+			'donations' => esc_html__('Donations', 'give'),
 			/* translators: %s: currency */
-			'earnings'  => sprintf( esc_html__( 'Income (%s)', 'give' ), html_entity_decode( give_currency_filter( '' ) ) )
+			'earnings'  => sprintf(esc_html__('Income (%s)', 'give'), html_entity_decode(give_currency_filter('')))
 		);
 
 		return $cols;
@@ -79,28 +79,28 @@ 
 	 */
 	public function get_data() {
 
-		$start_year  = isset( $_POST['start_year'] ) ? absint( $_POST['start_year'] ) : date( 'Y' );
-		$end_year    = isset( $_POST['end_year'] ) ? absint( $_POST['end_year'] ) : date( 'Y' );
-		$start_month = isset( $_POST['start_month'] ) ? absint( $_POST['start_month'] ) : date( 'n' );
-		$end_month   = isset( $_POST['end_month'] ) ? absint( $_POST['end_month'] ) : date( 'n' );
+		$start_year  = isset($_POST['start_year']) ? absint($_POST['start_year']) : date('Y');
+		$end_year    = isset($_POST['end_year']) ? absint($_POST['end_year']) : date('Y');
+		$start_month = isset($_POST['start_month']) ? absint($_POST['start_month']) : date('n');
+		$end_month   = isset($_POST['end_month']) ? absint($_POST['end_month']) : date('n');
 
 		$data  = array();
 		$year  = $start_year;
 		$stats = new Give_Payment_Stats;
 
-		while ( $year <= $end_year ) {
+		while ($year <= $end_year) {
 
-			if ( $year == $start_year && $year == $end_year ) {
+			if ($year == $start_year && $year == $end_year) {
 
 				$m1 = $start_month;
 				$m2 = $end_month;
 
-			} elseif ( $year == $start_year ) {
+			} elseif ($year == $start_year) {
 
 				$m1 = $start_month;
 				$m2 = 12;
 
-			} elseif ( $year == $end_year ) {
+			} elseif ($year == $end_year) {
 
 				$m1 = 1;
 				$m2 = $end_month;
@@ -112,28 +112,28 @@ 
 
 			}
 
-			while ( $m1 <= $m2 ) {
+			while ($m1 <= $m2) {
 
-				$date1 = mktime( 0, 0, 0, $m1, 1, $year );
-				$date2 = mktime( 0, 0, 0, $m1, cal_days_in_month( CAL_GREGORIAN, $m1, $year ), $year );
+				$date1 = mktime(0, 0, 0, $m1, 1, $year);
+				$date2 = mktime(0, 0, 0, $m1, cal_days_in_month(CAL_GREGORIAN, $m1, $year), $year);
 
 				$data[] = array(
-					'date'      => date_i18n( 'F Y', $date1 ),
-					'donations' => $stats->get_sales( 0, $date1, $date2 ),
-					'earnings'  => give_format_amount( $stats->get_earnings( 0, $date1, $date2 ) ),
+					'date'      => date_i18n('F Y', $date1),
+					'donations' => $stats->get_sales(0, $date1, $date2),
+					'earnings'  => give_format_amount($stats->get_earnings(0, $date1, $date2)),
 				);
 
-				$m1 ++;
+				$m1++;
 
 			}
 
 
-			$year ++;
+			$year++;
 
 		}
 
-		$data = apply_filters( 'give_export_get_data', $data );
-		$data = apply_filters( 'give_export_get_data_' . $this->export_type, $data );
+		$data = apply_filters('give_export_get_data', $data);
+		$data = apply_filters('give_export_get_data_'.$this->export_type, $data);
 
 		return $data;
 	}

includes/admin/customers/customer-actions.php

Spacing +128 added lines, -128 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,15 +23,15 @@ 
  *
  * @return array $output Response messages
  */
-function give_edit_customer( $args ) {
+function give_edit_customer($args) {
 	
-	$customer_edit_role = apply_filters( 'give_edit_customers_role', 'edit_give_payments' );
+	$customer_edit_role = apply_filters('give_edit_customers_role', 'edit_give_payments');
 
-	if ( ! is_admin() || ! current_user_can( $customer_edit_role ) ) {
-		wp_die( esc_html__( 'You do not have permission to edit this donor.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! is_admin() || ! current_user_can($customer_edit_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;
 	}
 
@@ -39,12 +39,12 @@ 
 	$customer_id   = (int) $args['customerinfo']['id'];
 	$nonce         = $args['_wpnonce'];
 
-	if ( ! wp_verify_nonce( $nonce, 'edit-customer' ) ) {
-		wp_die( esc_html__( 'Cheatin\' eh?!', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 400 ) );
+	if ( ! wp_verify_nonce($nonce, 'edit-customer')) {
+		wp_die(esc_html__('Cheatin\' eh?!', 'give'), esc_html__('Error', 'give'), array('response' => 400));
 	}
 
-	$customer = new Give_Customer( $customer_id );
-	if ( empty( $customer->id ) ) {
+	$customer = new Give_Customer($customer_id);
+	if (empty($customer->id)) {
 		return false;
 	}
 
@@ -54,23 +54,23 @@ 
 		'user_id' => 0
 	);
 
-	$customer_info = wp_parse_args( $customer_info, $defaults );
+	$customer_info = wp_parse_args($customer_info, $defaults);
 
-	if ( ! is_email( $customer_info['email'] ) ) {
-		give_set_error( 'give-invalid-email', esc_html__( 'Please enter a valid email address.', 'give' ) );
+	if ( ! is_email($customer_info['email'])) {
+		give_set_error('give-invalid-email', esc_html__('Please enter a valid email address.', 'give'));
 	}
 
-	if ( (int) $customer_info['user_id'] != (int) $customer->user_id ) {
+	if ((int) $customer_info['user_id'] != (int) $customer->user_id) {
 
 		// Make sure we don't already have this user attached to a customer
-		if ( ! empty( $customer_info['user_id'] ) && false !== Give()->customers->get_customer_by( 'user_id', $customer_info['user_id'] ) ) {
-			give_set_error( 'give-invalid-customer-user_id', sprintf( esc_html__( 'The User ID %d is already associated with a different donor.', 'give' ), $customer_info['user_id'] ) );
+		if ( ! empty($customer_info['user_id']) && false !== Give()->customers->get_customer_by('user_id', $customer_info['user_id'])) {
+			give_set_error('give-invalid-customer-user_id', sprintf(esc_html__('The User ID %d is already associated with a different donor.', 'give'), $customer_info['user_id']));
 		}
 
 		// Make sure it's actually a user
-		$user = get_user_by( 'id', $customer_info['user_id'] );
-		if ( ! empty( $customer_info['user_id'] ) && false === $user ) {
-			give_set_error( 'give-invalid-user_id', sprintf( esc_html__( 'The User ID %d does not exist. Please assign an existing user.', 'give' ), $customer_info['user_id'] ) );
+		$user = get_user_by('id', $customer_info['user_id']);
+		if ( ! empty($customer_info['user_id']) && false === $user) {
+			give_set_error('give-invalid-user_id', sprintf(esc_html__('The User ID %d does not exist. Please assign an existing user.', 'give'), $customer_info['user_id']));
 		}
 
 	}
@@ -78,53 +78,53 @@ 
 	// Record this for later
 	$previous_user_id = $customer->user_id;
 
-	if ( give_get_errors() ) {
+	if (give_get_errors()) {
 		return;
 	}
 
 	// Setup the customer address, if present
 	$address = array();
-	if ( intval( $customer_info['user_id'] ) > 0 ) {
+	if (intval($customer_info['user_id']) > 0) {
 
-		$current_address = get_user_meta( $customer_info['user_id'], '_give_user_address', true );
+		$current_address = get_user_meta($customer_info['user_id'], '_give_user_address', true);
 
-		if ( false === $current_address ) {
-			$address['line1']   = isset( $customer_info['line1'] ) ? $customer_info['line1'] : '';
-			$address['line2']   = isset( $customer_info['line2'] ) ? $customer_info['line2'] : '';
-			$address['city']    = isset( $customer_info['city'] ) ? $customer_info['city'] : '';
-			$address['country'] = isset( $customer_info['country'] ) ? $customer_info['country'] : '';
-			$address['zip']     = isset( $customer_info['zip'] ) ? $customer_info['zip'] : '';
-			$address['state']   = isset( $customer_info['state'] ) ? $customer_info['state'] : '';
+		if (false === $current_address) {
+			$address['line1']   = isset($customer_info['line1']) ? $customer_info['line1'] : '';
+			$address['line2']   = isset($customer_info['line2']) ? $customer_info['line2'] : '';
+			$address['city']    = isset($customer_info['city']) ? $customer_info['city'] : '';
+			$address['country'] = isset($customer_info['country']) ? $customer_info['country'] : '';
+			$address['zip']     = isset($customer_info['zip']) ? $customer_info['zip'] : '';
+			$address['state']   = isset($customer_info['state']) ? $customer_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( $customer_info['line1'] ) ? $customer_info['line1'] : $current_address['line1'];
-			$address['line2']   = isset( $customer_info['line2'] ) ? $customer_info['line2'] : $current_address['line2'];
-			$address['city']    = isset( $customer_info['city'] ) ? $customer_info['city'] : $current_address['city'];
-			$address['country'] = isset( $customer_info['country'] ) ? $customer_info['country'] : $current_address['country'];
-			$address['zip']     = isset( $customer_info['zip'] ) ? $customer_info['zip'] : $current_address['zip'];
-			$address['state']   = isset( $customer_info['state'] ) ? $customer_info['state'] : $current_address['state'];
+			));
+			$address['line1']   = isset($customer_info['line1']) ? $customer_info['line1'] : $current_address['line1'];
+			$address['line2']   = isset($customer_info['line2']) ? $customer_info['line2'] : $current_address['line2'];
+			$address['city']    = isset($customer_info['city']) ? $customer_info['city'] : $current_address['city'];
+			$address['country'] = isset($customer_info['country']) ? $customer_info['country'] : $current_address['country'];
+			$address['zip']     = isset($customer_info['zip']) ? $customer_info['zip'] : $current_address['zip'];
+			$address['state']   = isset($customer_info['state']) ? $customer_info['state'] : $current_address['state'];
 		}
 
 	}
 
 	// Sanitize the inputs
 	$customer_data            = array();
-	$customer_data['name']    = strip_tags( stripslashes( $customer_info['name'] ) );
+	$customer_data['name']    = strip_tags(stripslashes($customer_info['name']));
 	$customer_data['email']   = $customer_info['email'];
 	$customer_data['user_id'] = $customer_info['user_id'];
 
-	$customer_data = apply_filters( 'give_edit_customer_info', $customer_data, $customer_id );
-	$address       = apply_filters( 'give_edit_customer_address', $address, $customer_id );
+	$customer_data = apply_filters('give_edit_customer_info', $customer_data, $customer_id);
+	$address       = apply_filters('give_edit_customer_address', $address, $customer_id);
 
-	$customer_data = array_map( 'sanitize_text_field', $customer_data );
-	$address       = array_map( 'sanitize_text_field', $address );
+	$customer_data = array_map('sanitize_text_field', $customer_data);
+	$address       = array_map('sanitize_text_field', $address);
 
 
 	/**
@@ -136,34 +136,34 @@ 
 	 * @param array $customer_data The customer data.
 	 * @param array $address       The customer address.
 	 */
-	do_action( 'give_pre_edit_customer', $customer_id, $customer_data, $address );
+	do_action('give_pre_edit_customer', $customer_id, $customer_data, $address);
 
 	$output         = array();
 	$previous_email = $customer->email;
 
-	if ( $customer->update( $customer_data ) ) {
+	if ($customer->update($customer_data)) {
 
-		if ( ! empty( $customer->user_id ) && $customer->user_id > 0 ) {
-			update_user_meta( $customer->user_id, '_give_user_address', $address );
+		if ( ! empty($customer->user_id) && $customer->user_id > 0) {
+			update_user_meta($customer->user_id, '_give_user_address', $address);
 		}
 
 		// Update some payment meta if we need to
-		$payments_array = explode( ',', $customer->payment_ids );
+		$payments_array = explode(',', $customer->payment_ids);
 
-		if ( $customer->email != $previous_email ) {
-			foreach ( $payments_array as $payment_id ) {
-				give_update_payment_meta( $payment_id, 'email', $customer->email );
+		if ($customer->email != $previous_email) {
+			foreach ($payments_array as $payment_id) {
+				give_update_payment_meta($payment_id, 'email', $customer->email);
 			}
 		}
 
-		if ( $customer->user_id != $previous_user_id ) {
-			foreach ( $payments_array as $payment_id ) {
-				give_update_payment_meta( $payment_id, '_give_payment_user_id', $customer->user_id );
+		if ($customer->user_id != $previous_user_id) {
+			foreach ($payments_array as $payment_id) {
+				give_update_payment_meta($payment_id, '_give_payment_user_id', $customer->user_id);
 			}
 		}
 
 		$output['success']       = true;
-		$customer_data           = array_merge( $customer_data, $address );
+		$customer_data           = array_merge($customer_data, $address);
 		$output['customer_info'] = $customer_data;
 
 	} else {
@@ -180,11 +180,11 @@ 
 	 * @param int   $customer_id   The ID of the customer.
 	 * @param array $customer_data The customer data.
 	 */
-	do_action( 'give_post_edit_customer', $customer_id, $customer_data );
+	do_action('give_post_edit_customer', $customer_id, $customer_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();
 	}
 
@@ -192,7 +192,7 @@ 
 
 }
 
-add_action( 'give_edit-customer', 'give_edit_customer', 10, 1 );
+add_action('give_edit-customer', 'give_edit_customer', 10, 1);
 
 /**
  * Save a customer note being added
@@ -203,36 +203,36 @@ 
  *
  * @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( 'response' => 403 ) );
+	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\' eh?!', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 400 ) );
+	if ( ! wp_verify_nonce($nonce, 'add-customer-note')) {
+		wp_die(esc_html__('Cheatin\' eh?!', '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.
@@ -242,22 +242,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;
 		}
@@ -270,7 +270,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
@@ -281,37 +281,37 @@ 
  *
  * @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_customers_role', 'edit_give_payments' );
+	$customer_edit_role = apply_filters('give_edit_customers_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( 'response' => 403 ) );
+	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\' eh?!', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 400 ) );
+	if ( ! wp_verify_nonce($nonce, 'delete-customer')) {
+		wp_die(esc_html__('Cheatin\' eh?!', '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.
@@ -322,55 +322,55 @@ 
 	 * @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);
 
 	$success = false;
 
-	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 payments, 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 payments 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
@@ -381,27 +381,27 @@ 
  *
  * @return bool        If the disconnect was successful
  */
-function give_disconnect_customer_user_id( $args ) {
+function give_disconnect_customer_user_id($args) {
 
-	$customer_edit_role = apply_filters( 'give_edit_customers_role', 'edit_give_payments' );
+	$customer_edit_role = apply_filters('give_edit_customers_role', 'edit_give_payments');
 
-	if ( ! is_admin() || ! current_user_can( $customer_edit_role ) ) {
-		wp_die( esc_html__( 'You do not have permission to edit this donor.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! is_admin() || ! current_user_can($customer_edit_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_id = (int) $args['customer_id'];
 	$nonce       = $args['_wpnonce'];
 
-	if ( ! wp_verify_nonce( $nonce, 'edit-customer' ) ) {
-		wp_die( esc_html__( 'Cheatin\' eh?!', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 400 ) );
+	if ( ! wp_verify_nonce($nonce, 'edit-customer')) {
+		wp_die(esc_html__('Cheatin\' eh?!', 'give'), esc_html__('Error', 'give'), array('response' => 400));
 	}
 
-	$customer = new Give_Customer( $customer_id );
-	if ( empty( $customer->id ) ) {
+	$customer = new Give_Customer($customer_id);
+	if (empty($customer->id)) {
 		return false;
 	}
 
@@ -415,15 +415,15 @@ 
 	 * @param int $customer_id The ID of the customer.
 	 * @param int $user_id     The ID of the user.
 	 */
-	do_action( 'give_pre_customer_disconnect_user_id', $customer_id, $user_id );
+	do_action('give_pre_customer_disconnect_user_id', $customer_id, $user_id);
 
-	$customer_args = array( 'user_id' => 0 );
+	$customer_args = array('user_id' => 0);
 
-	if ( $customer->update( $customer_args ) ) {
+	if ($customer->update($customer_args)) {
 		global $wpdb;
 
-		if ( ! empty( $customer->payment_ids ) ) {
-			$wpdb->query( "UPDATE $wpdb->postmeta SET meta_value = 0 WHERE meta_key = '_give_payment_user_id' AND post_id IN ( $customer->payment_ids )" );
+		if ( ! empty($customer->payment_ids)) {
+			$wpdb->query("UPDATE $wpdb->postmeta SET meta_value = 0 WHERE meta_key = '_give_payment_user_id' AND post_id IN ( $customer->payment_ids )");
 		}
 
 		$output['success'] = true;
@@ -431,7 +431,7 @@ 
 	} else {
 
 		$output['success'] = false;
-		give_set_error( 'give-disconnect-user-fail', esc_html__( 'Failed to disconnect user from donor.', 'give' ) );
+		give_set_error('give-disconnect-user-fail', esc_html__('Failed to disconnect user from donor.', 'give'));
 	}
 
 	/**
@@ -441,11 +441,11 @@ 
 	 *
 	 * @param int $customer_id The ID of the customer.
 	 */
-	do_action( 'give_post_customer_disconnect_user_id', $customer_id );
+	do_action('give_post_customer_disconnect_user_id', $customer_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,4 +453,4 @@ 
 
 }
 
-add_action( 'give_disconnect-userid', 'give_disconnect_customer_user_id', 10, 1 );
+add_action('give_disconnect-userid', 'give_disconnect_customer_user_id', 10, 1);

includes/admin/dashboard-widgets.php

Spacing +33 added lines, -33 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,12 +21,12 @@ 
  * @return void
  */
 function give_register_dashboard_widgets() {
-	if ( current_user_can( apply_filters( 'give_dashboard_stats_cap', 'view_give_reports' ) ) ) {
-		wp_add_dashboard_widget( 'give_dashboard_sales', esc_html__( 'Give: Donation Statistics', 'give' ), 'give_dashboard_sales_widget' );
+	if (current_user_can(apply_filters('give_dashboard_stats_cap', 'view_give_reports'))) {
+		wp_add_dashboard_widget('give_dashboard_sales', esc_html__('Give: Donation Statistics', 'give'), 'give_dashboard_sales_widget');
 	}
 }
 
-add_action( 'wp_dashboard_setup', 'give_register_dashboard_widgets', 10 );
+add_action('wp_dashboard_setup', 'give_register_dashboard_widgets', 10);
 
 /**
  * Sales Summary Dashboard Widget
@@ -38,7 +38,7 @@ 
  */
 function give_dashboard_sales_widget() {
 
-	if ( ! current_user_can( apply_filters( 'give_dashboard_stats_cap', 'view_give_reports' ) ) ) {
+	if ( ! current_user_can(apply_filters('give_dashboard_stats_cap', 'view_give_reports'))) {
 		return;
 	}
 	$stats = new Give_Payment_Stats; ?>
@@ -46,24 +46,24 @@ 
 	<div class="give-dashboard-widget">
 
 		<div class="give-dashboard-today give-clearfix">
-			<h3 class="give-dashboard-date-today"><?php echo date( 'F j, Y' ); ?></h3>
+			<h3 class="give-dashboard-date-today"><?php echo date('F j, Y'); ?></h3>
 
 			<p class="give-dashboard-happy-day"><?php
 				printf(
 				/* translators: %s: day of the week */
-					esc_html__( 'Happy %s!', 'give' ),
-					date( 'l', current_time( 'timestamp' ) )
+					esc_html__('Happy %s!', 'give'),
+					date('l', current_time('timestamp'))
 				);
 				?></p>
 
-			<?php $earnings_today = $stats->get_earnings( 0, 'today', false ); ?>
+			<?php $earnings_today = $stats->get_earnings(0, 'today', false); ?>
 
-			<p class="give-dashboard-today-earnings"><?php echo give_currency_filter( give_format_amount( $earnings_today ) ); ?></p>
+			<p class="give-dashboard-today-earnings"><?php echo give_currency_filter(give_format_amount($earnings_today)); ?></p>
 
 			<p class="give-orders-today">
-				<?php $donations_today = $stats->get_sales( 0, 'today', false ); ?>
-				<?php echo give_format_amount( $donations_today, false ); ?>
-				<span><?php echo _x( 'donations today', 'Displays in WP admin dashboard widget after the day\'s total donations', 'give' ); ?></span>
+				<?php $donations_today = $stats->get_sales(0, 'today', false); ?>
+				<?php echo give_format_amount($donations_today, false); ?>
+				<span><?php echo _x('donations today', 'Displays in WP admin dashboard widget after the day\'s total donations', 'give'); ?></span>
 			</p>
 
 
@@ -73,34 +73,34 @@ 
 		<table class="give-table-stats">
 			<thead style="display: none;">
 			<tr>
-				<th><?php esc_html_e( 'This Week', 'give' ); ?></th>
-				<th><?php esc_html_e( 'This Month', 'give' ); ?></th>
-				<th><?php esc_html_e( 'Past 30 Days', 'give' ); ?></th>
+				<th><?php esc_html_e('This Week', 'give'); ?></th>
+				<th><?php esc_html_e('This Month', 'give'); ?></th>
+				<th><?php esc_html_e('Past 30 Days', 'give'); ?></th>
 			</tr>
 			</thead>
 			<tbody>
 			<tr id="give-table-stats-tr-1">
 				<td>
-					<p class="give-dashboard-stat-total"><?php echo give_currency_filter( give_format_amount( $stats->get_earnings( 0, 'this_week' ) ) ); ?></p>
+					<p class="give-dashboard-stat-total"><?php echo give_currency_filter(give_format_amount($stats->get_earnings(0, 'this_week'))); ?></p>
 
-					<p class="give-dashboard-stat-total-label"><?php esc_html_e( 'this week', 'give' ); ?></p>
+					<p class="give-dashboard-stat-total-label"><?php esc_html_e('this week', 'give'); ?></p>
 				</td>
 				<td>
-					<p class="give-dashboard-stat-total"><?php echo give_currency_filter( give_format_amount( $stats->get_earnings( 0, 'this_month' ) ) ); ?></p>
+					<p class="give-dashboard-stat-total"><?php echo give_currency_filter(give_format_amount($stats->get_earnings(0, 'this_month'))); ?></p>
 
-					<p class="give-dashboard-stat-total-label"><?php esc_html_e( 'this month', 'give' ); ?></p>
+					<p class="give-dashboard-stat-total-label"><?php esc_html_e('this month', 'give'); ?></p>
 				</td>
 			</tr>
 			<tr id="give-table-stats-tr-2">
 				<td>
-					<p class="give-dashboard-stat-total"><?php echo give_currency_filter( give_format_amount( $stats->get_earnings( 0, 'last_month' ) ) ) ?></p>
+					<p class="give-dashboard-stat-total"><?php echo give_currency_filter(give_format_amount($stats->get_earnings(0, 'last_month'))) ?></p>
 
-					<p class="give-dashboard-stat-total-label"><?php esc_html_e( 'last month', 'give' ); ?></p>
+					<p class="give-dashboard-stat-total-label"><?php esc_html_e('last month', 'give'); ?></p>
 				</td>
 				<td>
-					<p class="give-dashboard-stat-total"><?php echo give_currency_filter( give_format_amount( $stats->get_earnings( 0, 'this_year', false ) ) ) ?></p>
+					<p class="give-dashboard-stat-total"><?php echo give_currency_filter(give_format_amount($stats->get_earnings(0, 'this_year', false))) ?></p>
 
-					<p class="give-dashboard-stat-total-label"><?php esc_html_e( 'this year', 'give' ); ?></p>
+					<p class="give-dashboard-stat-total-label"><?php esc_html_e('this year', 'give'); ?></p>
 				</td>
 			</tr>
 			</tbody>
@@ -120,25 +120,25 @@ 
  *
  * @return array
  */
-function give_dashboard_at_a_glance_widget( $items ) {
-	$num_posts = wp_count_posts( 'give_forms' );
+function give_dashboard_at_a_glance_widget($items) {
+	$num_posts = wp_count_posts('give_forms');
 
-	if ( $num_posts && $num_posts->publish ) {
+	if ($num_posts && $num_posts->publish) {
 
 		$text = sprintf(
 		/* translators: 1: number of posts published 2: forms singular label 3: forms plural label */
-			_n( '%1$s Give %2$s', '%1$s Give %3$s', $num_posts->publish, 'give' ),
+			_n('%1$s Give %2$s', '%1$s Give %3$s', $num_posts->publish, 'give'),
 			$num_posts->publish,
 			give_get_forms_label_singular(),
 			give_get_forms_label_plural()
 		);
 
-		$text = sprintf( $text, number_format_i18n( $num_posts->publish ) );
+		$text = sprintf($text, number_format_i18n($num_posts->publish));
 
-		if ( current_user_can( 'edit_give_forms', get_current_user_id() ) ) {
-			$text = sprintf( '<a class="give-forms-count" href="edit.php?post_type=give_forms">%1$s</a>', $text );
+		if (current_user_can('edit_give_forms', get_current_user_id())) {
+			$text = sprintf('<a class="give-forms-count" href="edit.php?post_type=give_forms">%1$s</a>', $text);
 		} else {
-			$text = sprintf( '<span class="give-forms-count">%1$s</span>', $text );
+			$text = sprintf('<span class="give-forms-count">%1$s</span>', $text);
 		}
 
 		$items[] = $text;
@@ -147,4 +147,4 @@ 
 	return $items;
 }
 
-add_filter( 'dashboard_glance_items', 'give_dashboard_at_a_glance_widget', 1, 1 );
+add_filter('dashboard_glance_items', 'give_dashboard_at_a_glance_widget', 1, 1);

includes/gateways/actions.php

Spacing +13 added lines, -13 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -21,14 +21,14 @@ 
  *
  * @param $data
  */
-function give_process_gateway_select( $data ) {
-	if ( isset( $_POST['gateway_submit'] ) ) {
-		wp_redirect( esc_url( add_query_arg( 'payment-mode', $_POST['payment-mode'] ) ) );
+function give_process_gateway_select($data) {
+	if (isset($_POST['gateway_submit'])) {
+		wp_redirect(esc_url(add_query_arg('payment-mode', $_POST['payment-mode'])));
 		exit;
 	}
 }
 
-add_action( 'give_gateway_select', 'give_process_gateway_select' );
+add_action('give_gateway_select', 'give_process_gateway_select');
 
 /**
  * Loads a payment gateway via AJAX
@@ -38,14 +38,14 @@ 
  * @return void
  */
 function give_load_ajax_gateway() {
-	if ( isset( $_POST['give_payment_mode'] ) ) {
-		do_action( 'give_purchase_form', $_POST['give_form_id'] );
+	if (isset($_POST['give_payment_mode'])) {
+		do_action('give_purchase_form', $_POST['give_form_id']);
 		exit();
 	}
 }
 
-add_action( 'wp_ajax_give_load_gateway', 'give_load_ajax_gateway' );
-add_action( 'wp_ajax_nopriv_give_load_gateway', 'give_load_ajax_gateway' );
+add_action('wp_ajax_give_load_gateway', 'give_load_ajax_gateway');
+add_action('wp_ajax_nopriv_give_load_gateway', 'give_load_ajax_gateway');
 
 /**
  * Sets an error on checkout if no gateways are enabled
@@ -57,11 +57,11 @@ 
 function give_no_gateway_error() {
 	$gateways = give_get_enabled_payment_gateways();
 
-	if ( empty( $gateways ) ) {
-		give_set_error( 'no_gateways', esc_html__( 'You must enable a payment gateway to use Give.', 'give' ) );
+	if (empty($gateways)) {
+		give_set_error('no_gateways', esc_html__('You must enable a payment gateway to use Give.', 'give'));
 	} else {
-		give_unset_error( 'no_gateways' );
+		give_unset_error('no_gateways');
 	}
 }
 
-add_action( 'init', 'give_no_gateway_error' );
+add_action('init', 'give_no_gateway_error');

includes/class-give-db.php

Indentation +40 added lines, -40 removed lines

@@ -111,14 +111,14 @@ 
 	 * @since  1.0
 	 * @access public
 	 *
-     * @param  int $column Column ID.
-     * @param  int $row_id Row ID.
-     *
-     * @return object
+	 * @param  int $column Column ID.
+	 * @param  int $row_id Row ID.
+	 *
+	 * @return object
 	 */
 	public function get_by( $column, $row_id ) {
-        /* @var WPDB $wpdb */
-        global $wpdb;
+		/* @var WPDB $wpdb */
+		global $wpdb;
 
 		$column = esc_sql( $column );
 		return $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $this->table_name WHERE $column = %s LIMIT 1;", $row_id ) );
@@ -129,15 +129,15 @@ 
 	 *
 	 * @since  1.0
 	 * @access public
-     *
-     * @param  int $column Column ID.
-     * @param  int $row_id Row ID.
-     *
+	 *
+	 * @param  int $column Column ID.
+	 * @param  int $row_id Row ID.
+	 *
 	 * @return string      Column value.
 	 */
 	public function get_column( $column, $row_id ) {
-        /* @var WPDB $wpdb */
-        global $wpdb;
+		/* @var WPDB $wpdb */
+		global $wpdb;
 
 		$column = esc_sql( $column );
 		return $wpdb->get_var( $wpdb->prepare( "SELECT $column FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id ) );
@@ -148,16 +148,16 @@ 
 	 *
 	 * @since  1.0
 	 * @access public
-     *
-     * @param  int    $column       Column ID.
-     * @param  string $column_where Column name.
-     * @param  string $column_value Column value.
-     *
+	 *
+	 * @param  int    $column       Column ID.
+	 * @param  string $column_where Column name.
+	 * @param  string $column_value Column value.
+	 *
 	 * @return string
 	 */
 	public function get_column_by( $column, $column_where, $column_value ) {
-        /* @var WPDB $wpdb */
-        global $wpdb;
+		/* @var WPDB $wpdb */
+		global $wpdb;
 
 		$column_where = esc_sql( $column_where );
 		$column       = esc_sql( $column );
@@ -169,15 +169,15 @@ 
 	 *
 	 * @since  1.0
 	 * @access public
-     *
-     * @param  array  $data
-     * @param  string $type
-     *
+	 *
+	 * @param  array  $data
+	 * @param  string $type
+	 *
 	 * @return int
 	 */
 	public function insert( $data, $type = '' ) {
-        /* @var WPDB $wpdb */
-        global $wpdb;
+		/* @var WPDB $wpdb */
+		global $wpdb;
 
 		// Set default values
 		$data = wp_parse_args( $data, $this->get_column_defaults() );
@@ -209,16 +209,16 @@ 
 	 *
 	 * @since  1.0
 	 * @access public
-     *
-     * @param  int    $row_id Column ID
-     * @param  array  $data
-     * @param  string $where  Column value
-     *
+	 *
+	 * @param  int    $row_id Column ID
+	 * @param  array  $data
+	 * @param  string $where  Column value
+	 *
 	 * @return bool
 	 */
 	public function update( $row_id, $data = array(), $where = '' ) {
-        /* @var WPDB $wpdb */
-        global $wpdb;
+		/* @var WPDB $wpdb */
+		global $wpdb;
 
 		// Row ID must be positive integer
 		$row_id = absint( $row_id );
@@ -256,14 +256,14 @@ 
 	 *
 	 * @since  1.0
 	 * @access public
-     *
-     * @param  int $row_id Column ID.
-     *
+	 *
+	 * @param  int $row_id Column ID.
+	 *
 	 * @return bool
 	 */
 	public function delete( $row_id = 0 ) {
-        /* @var WPDB $wpdb */
-        global $wpdb;
+		/* @var WPDB $wpdb */
+		global $wpdb;
 
 		// Row ID must be positive integer
 		$row_id = absint( $row_id );
@@ -284,13 +284,13 @@ 
 	 *
 	 * @since  1.3.2
 	 * @access public
-     *
+	 *
 	 * @param  string $table The table name.
-     *
+	 *
 	 * @return bool          If the table name exists.
 	 */
 	public function table_exists( $table ) {
-        /* @var WPDB $wpdb */
+		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		$table = sanitize_text_field( $table );

Spacing +39 added lines, -39 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -98,11 +98,11 @@ 
 	 *
 	 * @return object
 	 */
-	public function get( $row_id ) {
+	public function get($row_id) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
-		return $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id ) );
+		return $wpdb->get_row($wpdb->prepare("SELECT * FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id));
 	}
 
 	/**
@@ -116,12 +116,12 @@ 
      *
      * @return object
 	 */
-	public function get_by( $column, $row_id ) {
+	public function get_by($column, $row_id) {
         /* @var WPDB $wpdb */
         global $wpdb;
 
-		$column = esc_sql( $column );
-		return $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $this->table_name WHERE $column = %s LIMIT 1;", $row_id ) );
+		$column = esc_sql($column);
+		return $wpdb->get_row($wpdb->prepare("SELECT * FROM $this->table_name WHERE $column = %s LIMIT 1;", $row_id));
 	}
 
 	/**
@@ -135,12 +135,12 @@ 
      *
 	 * @return string      Column value.
 	 */
-	public function get_column( $column, $row_id ) {
+	public function get_column($column, $row_id) {
         /* @var WPDB $wpdb */
         global $wpdb;
 
-		$column = esc_sql( $column );
-		return $wpdb->get_var( $wpdb->prepare( "SELECT $column FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id ) );
+		$column = esc_sql($column);
+		return $wpdb->get_var($wpdb->prepare("SELECT $column FROM $this->table_name WHERE $this->primary_key = %s LIMIT 1;", $row_id));
 	}
 
 	/**
@@ -155,13 +155,13 @@ 
      *
 	 * @return string
 	 */
-	public function get_column_by( $column, $column_where, $column_value ) {
+	public function get_column_by($column, $column_where, $column_value) {
         /* @var WPDB $wpdb */
         global $wpdb;
 
-		$column_where = esc_sql( $column_where );
-		$column       = esc_sql( $column );
-		return $wpdb->get_var( $wpdb->prepare( "SELECT $column FROM $this->table_name WHERE $column_where = %s LIMIT 1;", $column_value ) );
+		$column_where = esc_sql($column_where);
+		$column       = esc_sql($column);
+		return $wpdb->get_var($wpdb->prepare("SELECT $column FROM $this->table_name WHERE $column_where = %s LIMIT 1;", $column_value));
 	}
 
 	/**
@@ -175,31 +175,31 @@ 
      *
 	 * @return int
 	 */
-	public function insert( $data, $type = '' ) {
+	public function insert($data, $type = '') {
         /* @var WPDB $wpdb */
         global $wpdb;
 
 		// Set default values
-		$data = wp_parse_args( $data, $this->get_column_defaults() );
+		$data = wp_parse_args($data, $this->get_column_defaults());
 
-		do_action( 'give_pre_insert_' . $type, $data );
+		do_action('give_pre_insert_'.$type, $data);
 
 		// Initialise column format array
 		$column_formats = $this->get_columns();
 
 		// Force fields to lower case
-		$data = array_change_key_case( $data );
+		$data = array_change_key_case($data);
 
 		// White list columns
-		$data = array_intersect_key( $data, $column_formats );
+		$data = array_intersect_key($data, $column_formats);
 
 		// Reorder $column_formats to match the order of columns given in $data
-		$data_keys      = array_keys( $data );
-		$column_formats = array_merge( array_flip( $data_keys ), $column_formats );
+		$data_keys      = array_keys($data);
+		$column_formats = array_merge(array_flip($data_keys), $column_formats);
 
-		$wpdb->insert( $this->table_name, $data, $column_formats );
+		$wpdb->insert($this->table_name, $data, $column_formats);
 
-		do_action( 'give_post_insert_' . $type, $wpdb->insert_id, $data );
+		do_action('give_post_insert_'.$type, $wpdb->insert_id, $data);
 
 		return $wpdb->insert_id;
 	}
@@ -216,18 +216,18 @@ 
      *
 	 * @return bool
 	 */
-	public function update( $row_id, $data = array(), $where = '' ) {
+	public function update($row_id, $data = array(), $where = '') {
         /* @var WPDB $wpdb */
         global $wpdb;
 
 		// Row ID must be positive integer
-		$row_id = absint( $row_id );
+		$row_id = absint($row_id);
 
-		if ( empty( $row_id ) ) {
+		if (empty($row_id)) {
 			return false;
 		}
 
-		if ( empty( $where ) ) {
+		if (empty($where)) {
 			$where = $this->primary_key;
 		}
 
@@ -235,16 +235,16 @@ 
 		$column_formats = $this->get_columns();
 
 		// Force fields to lower case
-		$data = array_change_key_case( $data );
+		$data = array_change_key_case($data);
 
 		// White list columns
-		$data = array_intersect_key( $data, $column_formats );
+		$data = array_intersect_key($data, $column_formats);
 
 		// Reorder $column_formats to match the order of columns given in $data
-		$data_keys      = array_keys( $data );
-		$column_formats = array_merge( array_flip( $data_keys ), $column_formats );
+		$data_keys      = array_keys($data);
+		$column_formats = array_merge(array_flip($data_keys), $column_formats);
 
-		if ( false === $wpdb->update( $this->table_name, $data, array( $where => $row_id ), $column_formats ) ) {
+		if (false === $wpdb->update($this->table_name, $data, array($where => $row_id), $column_formats)) {
 			return false;
 		}
 
@@ -261,18 +261,18 @@ 
      *
 	 * @return bool
 	 */
-	public function delete( $row_id = 0 ) {
+	public function delete($row_id = 0) {
         /* @var WPDB $wpdb */
         global $wpdb;
 
 		// Row ID must be positive integer
-		$row_id = absint( $row_id );
+		$row_id = absint($row_id);
 
-		if ( empty( $row_id ) ) {
+		if (empty($row_id)) {
 			return false;
 		}
 
-		if ( false === $wpdb->query( $wpdb->prepare( "DELETE FROM $this->table_name WHERE $this->primary_key = %d", $row_id ) ) ) {
+		if (false === $wpdb->query($wpdb->prepare("DELETE FROM $this->table_name WHERE $this->primary_key = %d", $row_id))) {
 			return false;
 		}
 
@@ -289,13 +289,13 @@ 
      *
 	 * @return bool          If the table name exists.
 	 */
-	public function table_exists( $table ) {
+	public function table_exists($table) {
         /* @var WPDB $wpdb */
 		global $wpdb;
 
-		$table = sanitize_text_field( $table );
+		$table = sanitize_text_field($table);
 
-		return $wpdb->get_var( $wpdb->prepare( "SHOW TABLES LIKE '%s'", $table ) ) === $table;
+		return $wpdb->get_var($wpdb->prepare("SHOW TABLES LIKE '%s'", $table)) === $table;
 	}
 
 	/**
@@ -307,7 +307,7 @@ 
 	 * @return bool Returns if the customers table was installed and upgrade routine run.
 	 */
 	public function installed() {
-		return $this->table_exists( $this->table_name );
+		return $this->table_exists($this->table_name);
 	}
 
 }

includes/actions.php

Spacing +7 added lines, -7 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,12 +24,12 @@ 
  * @return void
  */
 function give_get_actions() {
-	if ( isset( $_GET['give_action'] ) ) {
-		do_action( 'give_' . $_GET['give_action'], $_GET );
+	if (isset($_GET['give_action'])) {
+		do_action('give_'.$_GET['give_action'], $_GET);
 	}
 }
 
-add_action( 'init', 'give_get_actions' );
+add_action('init', 'give_get_actions');
 
 /**
  * Hooks Give actions, when present in the $_POST superglobal. Every give_action
@@ -41,9 +41,9 @@ 
  * @return void
  */
 function give_post_actions() {
-	if ( isset( $_POST['give_action'] ) ) {
-		do_action( 'give_' . $_POST['give_action'], $_POST );
+	if (isset($_POST['give_action'])) {
+		do_action('give_'.$_POST['give_action'], $_POST);
 	}
 }
 
-add_action( 'init', 'give_post_actions' );
+add_action('init', 'give_post_actions');

includes/admin/payments/class-payments-table.php

Doc Comments +1 added lines

@@ -398,6 +398,7 @@
 	 *
 	 * @since 1.6
 	 *
+	 * @param Give_Payment $payment
 	 * @return mixed|void
 	 */
 	function get_row_actions( $payment ) {

Spacing +180 added lines, -180 removed lines

@@ -10,13 +10,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';
 }
 
 /**
@@ -118,37 +118,37 @@ 
 	public function __construct() {
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => give_get_forms_label_singular(),    // Singular name of the listed records
-			'plural'   => give_get_forms_label_plural(),        // Plural name of the listed records
+		parent::__construct(array(
+			'singular' => give_get_forms_label_singular(), // Singular name of the listed records
+			'plural'   => give_get_forms_label_plural(), // Plural name of the listed records
 			'ajax'     => false                        // Does this table support ajax?
-		) );
+		));
 
 		$this->get_payment_counts();
 		$this->process_bulk_action();
-		$this->base_url = admin_url( 'edit.php?post_type=give_forms&page=give-payment-history' );
+		$this->base_url = admin_url('edit.php?post_type=give_forms&page=give-payment-history');
 	}
 
 	public function advanced_filters() {
-		$start_date = isset( $_GET['start-date'] ) ? sanitize_text_field( $_GET['start-date'] ) : null;
-		$end_date   = isset( $_GET['end-date'] ) ? sanitize_text_field( $_GET['end-date'] ) : null;
-		$status     = isset( $_GET['status'] ) ? $_GET['status'] : '';
+		$start_date = isset($_GET['start-date']) ? sanitize_text_field($_GET['start-date']) : null;
+		$end_date   = isset($_GET['end-date']) ? sanitize_text_field($_GET['end-date']) : null;
+		$status     = isset($_GET['status']) ? $_GET['status'] : '';
 		?>
 		<div id="give-payment-filters">
 			<span id="give-payment-date-filters">
-				<label for="start-date" class="give-start-date-label"><?php esc_html_e( 'Start Date:', 'give' ); ?></label>
+				<label for="start-date" class="give-start-date-label"><?php esc_html_e('Start Date:', 'give'); ?></label>
 				<input type="text" id="start-date" name="start-date" class="give_datepicker" value="<?php echo $start_date; ?>" placeholder="mm/dd/yyyy"/>
-				<label for="end-date" class="give-end-date-label"><?php esc_html_e( 'End Date:', 'give' ); ?></label>
+				<label for="end-date" class="give-end-date-label"><?php esc_html_e('End Date:', 'give'); ?></label>
 				<input type="text" id="end-date" name="end-date" class="give_datepicker" value="<?php echo $end_date; ?>" placeholder="mm/dd/yyyy"/>
-				<input type="submit" class="button-secondary" value="<?php esc_attr_e( 'Apply', 'give' ); ?>"/>
+				<input type="submit" class="button-secondary" value="<?php esc_attr_e('Apply', 'give'); ?>"/>
 			</span>
-			<?php if ( ! empty( $status ) ) : ?>
-				<input type="hidden" name="status" value="<?php echo esc_attr( $status ); ?>"/>
+			<?php if ( ! empty($status)) : ?>
+				<input type="hidden" name="status" value="<?php echo esc_attr($status); ?>"/>
 			<?php endif; ?>
-			<?php if ( ! empty( $start_date ) || ! empty( $end_date ) ) : ?>
-				<a href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-payment-history' ); ?>" class="button-secondary"><?php esc_html_e( 'Clear Filter', 'give' ); ?></a>
+			<?php if ( ! empty($start_date) || ! empty($end_date)) : ?>
+				<a href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-payment-history'); ?>" class="button-secondary"><?php esc_html_e('Clear Filter', 'give'); ?></a>
 			<?php endif; ?>
-			<?php $this->search_box( esc_html__( 'Search', 'give' ), 'give-payments' ); ?>
+			<?php $this->search_box(esc_html__('Search', 'give'), 'give-payments'); ?>
 		</div>
 
 		<?php
@@ -165,18 +165,18 @@ 
 	 *
 	 * @return void
 	 */
-	public function search_box( $text, $input_id ) {
-		if ( empty( $_REQUEST['s'] ) && ! $this->has_items() ) {
+	public function search_box($text, $input_id) {
+		if (empty($_REQUEST['s']) && ! $this->has_items()) {
 			return;
 		}
 
-		$input_id = $input_id . '-search-input';
+		$input_id = $input_id.'-search-input';
 
-		if ( ! empty( $_REQUEST['orderby'] ) ) {
-			echo '<input type="hidden" name="orderby" value="' . esc_attr( $_REQUEST['orderby'] ) . '" />';
+		if ( ! empty($_REQUEST['orderby'])) {
+			echo '<input type="hidden" name="orderby" value="'.esc_attr($_REQUEST['orderby']).'" />';
 		}
-		if ( ! empty( $_REQUEST['order'] ) ) {
-			echo '<input type="hidden" name="order" value="' . esc_attr( $_REQUEST['order'] ) . '" />';
+		if ( ! empty($_REQUEST['order'])) {
+			echo '<input type="hidden" name="order" value="'.esc_attr($_REQUEST['order']).'" />';
 		}
 		?>
 		<p class="search-box">
@@ -188,11 +188,11 @@ 
 			 *
 			 * @since 1.0
 			 */
-			do_action( 'give_payment_history_search' );
+			do_action('give_payment_history_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( 'ID' => 'search-submit' ) ); ?><br/>
+			<?php submit_button($text, 'button', false, false, array('ID' => 'search-submit')); ?><br/>
 		</p>
 		<?php
 	}
@@ -206,52 +206,52 @@ 
 	 */
 	public function get_views() {
 
-		$current         = isset( $_GET['status'] ) ? $_GET['status'] : '';
-		$total_count     = '&nbsp;<span class="count">(' . $this->total_count . ')</span>';
-		$complete_count  = '&nbsp;<span class="count">(' . $this->complete_count . ')</span>';
-		$cancelled_count = '&nbsp;<span class="count">(' . $this->cancelled_count . ')</span>';
-		$pending_count   = '&nbsp;<span class="count">(' . $this->pending_count . ')</span>';
-		$refunded_count  = '&nbsp;<span class="count">(' . $this->refunded_count . ')</span>';
-		$failed_count    = '&nbsp;<span class="count">(' . $this->failed_count . ')</span>';
-		$abandoned_count = '&nbsp;<span class="count">(' . $this->abandoned_count . ')</span>';
-		$revoked_count   = '&nbsp;<span class="count">(' . $this->revoked_count . ')</span>';
+		$current         = isset($_GET['status']) ? $_GET['status'] : '';
+		$total_count     = '&nbsp;<span class="count">('.$this->total_count.')</span>';
+		$complete_count  = '&nbsp;<span class="count">('.$this->complete_count.')</span>';
+		$cancelled_count = '&nbsp;<span class="count">('.$this->cancelled_count.')</span>';
+		$pending_count   = '&nbsp;<span class="count">('.$this->pending_count.')</span>';
+		$refunded_count  = '&nbsp;<span class="count">('.$this->refunded_count.')</span>';
+		$failed_count    = '&nbsp;<span class="count">('.$this->failed_count.')</span>';
+		$abandoned_count = '&nbsp;<span class="count">('.$this->abandoned_count.')</span>';
+		$revoked_count   = '&nbsp;<span class="count">('.$this->revoked_count.')</span>';
 
 		$views = array(
-			'all'       => sprintf( '<a href="%s"%s>%s</a>', remove_query_arg( array(
+			'all'       => sprintf('<a href="%s"%s>%s</a>', remove_query_arg(array(
 				'status',
 				'paged'
-			) ), $current === 'all' || $current == '' ? ' class="current"' : '', esc_html__( 'All', 'give' ) . $total_count ),
-			'publish'   => sprintf( '<a href="%s"%s>%s</a>', esc_url( add_query_arg( array(
+			)), $current === 'all' || $current == '' ? ' class="current"' : '', esc_html__('All', 'give').$total_count),
+			'publish'   => sprintf('<a href="%s"%s>%s</a>', esc_url(add_query_arg(array(
 				'status' => 'publish',
 				'paged'  => false
-			) ) ), $current === 'publish' ? ' class="current"' : '', esc_html__( 'Completed', 'give' ) . $complete_count ),
-			'pending'   => sprintf( '<a href="%s"%s>%s</a>', esc_url( add_query_arg( array(
+			))), $current === 'publish' ? ' class="current"' : '', esc_html__('Completed', 'give').$complete_count),
+			'pending'   => sprintf('<a href="%s"%s>%s</a>', esc_url(add_query_arg(array(
 				'status' => 'pending',
 				'paged'  => false
-			) ) ), $current === 'pending' ? ' class="current"' : '', esc_html__( 'Pending', 'give' ) . $pending_count ),
-			'refunded'  => sprintf( '<a href="%s"%s>%s</a>', esc_url( add_query_arg( array(
+			))), $current === 'pending' ? ' class="current"' : '', esc_html__('Pending', 'give').$pending_count),
+			'refunded'  => sprintf('<a href="%s"%s>%s</a>', esc_url(add_query_arg(array(
 				'status' => 'refunded',
 				'paged'  => false
-			) ) ), $current === 'refunded' ? ' class="current"' : '', esc_html__( 'Refunded', 'give' ) . $refunded_count ),
-			'revoked'   => sprintf( '<a href="%s"%s>%s</a>', esc_url( add_query_arg( array(
+			))), $current === 'refunded' ? ' class="current"' : '', esc_html__('Refunded', 'give').$refunded_count),
+			'revoked'   => sprintf('<a href="%s"%s>%s</a>', esc_url(add_query_arg(array(
 				'status' => 'revoked',
 				'paged'  => false
-			) ) ), $current === 'revoked' ? ' class="current"' : '', esc_html__( 'Revoked', 'give' ) . $revoked_count ),
-			'failed'    => sprintf( '<a href="%s"%s>%s</a>', esc_url( add_query_arg( array(
+			))), $current === 'revoked' ? ' class="current"' : '', esc_html__('Revoked', 'give').$revoked_count),
+			'failed'    => sprintf('<a href="%s"%s>%s</a>', esc_url(add_query_arg(array(
 				'status' => 'failed',
 				'paged'  => false
-			) ) ), $current === 'failed' ? ' class="current"' : '', esc_html__( 'Failed', 'give' ) . $failed_count ),
-			'cancelled' => sprintf( '<a href="%s"%s>%s</a>', esc_url( add_query_arg( array(
+			))), $current === 'failed' ? ' class="current"' : '', esc_html__('Failed', 'give').$failed_count),
+			'cancelled' => sprintf('<a href="%s"%s>%s</a>', esc_url(add_query_arg(array(
 				'status' => 'cancelled',
 				'paged'  => false
-			) ) ), $current === 'cancelled' ? ' class="current"' : '', esc_html__( 'Cancelled', 'give' ) . $cancelled_count ),
-			'abandoned' => sprintf( '<a href="%s"%s>%s</a>', esc_url( add_query_arg( array(
+			))), $current === 'cancelled' ? ' class="current"' : '', esc_html__('Cancelled', 'give').$cancelled_count),
+			'abandoned' => sprintf('<a href="%s"%s>%s</a>', esc_url(add_query_arg(array(
 				'status' => 'abandoned',
 				'paged'  => false
-			) ) ), $current === 'abandoned' ? ' class="current"' : '', esc_html__( 'Abandoned', 'give' ) . $abandoned_count )
+			))), $current === 'abandoned' ? ' class="current"' : '', esc_html__('Abandoned', 'give').$abandoned_count)
 		);
 
-		return apply_filters( 'give_payments_table_views', $views );
+		return apply_filters('give_payments_table_views', $views);
 	}
 
 	/**
@@ -264,15 +264,15 @@ 
 	public function get_columns() {
 		$columns = array(
 			'cb'            => '<input type="checkbox" />', //Render a checkbox instead of text
-			'donation'      => esc_html__( 'Donation', 'give' ),
-			'donation_form' => esc_html__( 'Donation Form', 'give' ),
-			'status'        => esc_html__( 'Status', 'give' ),
-			'date'          => esc_html__( 'Date', 'give' ),
-			'amount'        => esc_html__( 'Amount', 'give' ),
-			'details'       => esc_html__( 'Details', 'give' ),
+			'donation'      => esc_html__('Donation', 'give'),
+			'donation_form' => esc_html__('Donation Form', 'give'),
+			'status'        => esc_html__('Status', 'give'),
+			'date'          => esc_html__('Date', 'give'),
+			'amount'        => esc_html__('Amount', 'give'),
+			'details'       => esc_html__('Details', 'give'),
 		);
 
-		return apply_filters( 'give_payments_table_columns', $columns );
+		return apply_filters('give_payments_table_columns', $columns);
 	}
 
 	/**
@@ -284,12 +284,12 @@ 
 	 */
 	public function get_sortable_columns() {
 		$columns = array(
-			'donation' => array( 'ID', true ),
-			'amount'   => array( 'amount', false ),
-			'date'     => array( 'date', false )
+			'donation' => array('ID', true),
+			'amount'   => array('amount', false),
+			'date'     => array('date', false)
 		);
 
-		return apply_filters( 'give_payments_table_sortable_columns', $columns );
+		return apply_filters('give_payments_table_sortable_columns', $columns);
 	}
 
 	/**
@@ -315,59 +315,59 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $payment, $column_name ) {
+	public function column_default($payment, $column_name) {
 
-		$single_donation_url = esc_url( add_query_arg( 'id', $payment->ID, admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-order-details' ) ) );
-		$row_actions         = $this->get_row_actions( $payment );
+		$single_donation_url = esc_url(add_query_arg('id', $payment->ID, admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-order-details')));
+		$row_actions         = $this->get_row_actions($payment);
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 			case 'donation' :
 				ob_start();
 				?>
-				<a href="<?php echo $single_donation_url; ?>" data-tooltip="<?php esc_html_e( 'View details', 'give' ) ?>">#<?php echo $payment->ID; ?></a>&nbsp;<?php _e( 'by', 'give' ); ?>&nbsp;<?php echo $this->get_donor( $payment ); ?>
+				<a href="<?php echo $single_donation_url; ?>" data-tooltip="<?php esc_html_e('View details', 'give') ?>">#<?php echo $payment->ID; ?></a>&nbsp;<?php _e('by', 'give'); ?>&nbsp;<?php echo $this->get_donor($payment); ?>
 				<br>
-				<?php echo $this->get_donor_email( $payment ); ?>
-				<?php echo $this->row_actions( $row_actions ); ?>
+				<?php echo $this->get_donor_email($payment); ?>
+				<?php echo $this->row_actions($row_actions); ?>
 				<?php
 				$value = ob_get_clean();
 				break;
 
 			case 'amount' :
-				$amount = ! empty( $payment->total ) ? $payment->total : 0;
-				$value  = give_currency_filter( give_format_amount( $amount ), give_get_payment_currency_code( $payment->ID ) );
+				$amount = ! empty($payment->total) ? $payment->total : 0;
+				$value  = give_currency_filter(give_format_amount($amount), give_get_payment_currency_code($payment->ID));
 				break;
 
 			case 'donation_form' :
 
-				$value = '<a href="' . admin_url( 'post.php?post=' . $payment->form_id . '&action=edit' ) . '">' . $payment->form_title . '</a>';
-				$level = give_get_payment_form_title( $payment->meta, true );
+				$value = '<a href="'.admin_url('post.php?post='.$payment->form_id.'&action=edit').'">'.$payment->form_title.'</a>';
+				$level = give_get_payment_form_title($payment->meta, true);
 
-				if ( ! empty( $level ) ) {
+				if ( ! empty($level)) {
 					$value .= $level;
 				}
 
 				break;
 
 			case 'date' :
-				$date  = strtotime( $payment->date );
-				$value = date_i18n( get_option( 'date_format' ), $date );
+				$date  = strtotime($payment->date);
+				$value = date_i18n(get_option('date_format'), $date);
 				break;
 
 			case 'status' :
-				$value = $this->get_payment_status( $payment );
+				$value = $this->get_payment_status($payment);
 				break;
 
 			case 'details' :
-				$value = '<div class="give-payment-details-link-wrap"><a href="' . $single_donation_url . '" data-tooltip="' . __( 'View details', 'give' ) . '" class="give-payment-details-link button button-small" title="' . __( 'View Details', 'give' ) . '"><span class="dashicons dashicons-visibility"></span></a></div>';
+				$value = '<div class="give-payment-details-link-wrap"><a href="'.$single_donation_url.'" data-tooltip="'.__('View details', 'give').'" class="give-payment-details-link button button-small" title="'.__('View Details', 'give').'"><span class="dashicons dashicons-visibility"></span></a></div>';
 				break;
 
 			default:
-				$value = isset( $payment->$column_name ) ? $payment->$column_name : '';
+				$value = isset($payment->$column_name) ? $payment->$column_name : '';
 				break;
 
 		}
 
-		return apply_filters( 'give_payments_table_column', $value, $payment->ID, $column_name );
+		return apply_filters('give_payments_table_column', $value, $payment->ID, $column_name);
 	}
 
 	/**
@@ -380,17 +380,17 @@ 
 	 *
 	 * @return string                Data shown in the Email column
 	 */
-	public function get_donor_email( $payment ) {
+	public function get_donor_email($payment) {
 
-		$email = give_get_payment_user_email( $payment->ID );
+		$email = give_get_payment_user_email($payment->ID);
 
-		if ( empty( $email ) ) {
-			$email = esc_html__( '(unknown)', 'give' );
+		if (empty($email)) {
+			$email = esc_html__('(unknown)', 'give');
 		}
 
-		$value = '<a href="mailto:' . $email . '" data-tooltip="' . __( 'Email donor', 'give' ) . '">' . $email . '</a>';
+		$value = '<a href="mailto:'.$email.'" data-tooltip="'.__('Email donor', 'give').'">'.$email.'</a>';
 
-		return apply_filters( 'give_payments_table_column', $value, $payment->ID, 'email' );
+		return apply_filters('give_payments_table_column', $value, $payment->ID, 'email');
 	}
 
 	/**
@@ -400,31 +400,31 @@ 
 	 *
 	 * @return mixed|void
 	 */
-	function get_row_actions( $payment ) {
+	function get_row_actions($payment) {
 
 		$row_actions = array();
-		$email       = give_get_payment_user_email( $payment->ID );
+		$email       = give_get_payment_user_email($payment->ID);
 
 		// Add search term string back to base URL
-		$search_terms = ( isset( $_GET['s'] ) ? trim( $_GET['s'] ) : '' );
-		if ( ! empty( $search_terms ) ) {
-			$this->base_url = add_query_arg( 's', $search_terms, $this->base_url );
+		$search_terms = (isset($_GET['s']) ? trim($_GET['s']) : '');
+		if ( ! empty($search_terms)) {
+			$this->base_url = add_query_arg('s', $search_terms, $this->base_url);
 		}
 
-		if ( give_is_payment_complete( $payment->ID ) && ! empty( $email ) ) {
-			$row_actions['email_links'] = '<a href="' . add_query_arg( array(
+		if (give_is_payment_complete($payment->ID) && ! empty($email)) {
+			$row_actions['email_links'] = '<a href="'.add_query_arg(array(
 					'give-action' => 'email_links',
 					'purchase_id' => $payment->ID
-				), $this->base_url ) . '">' . esc_html__( 'Resend Donation Receipt', 'give' ) . '</a>';
+				), $this->base_url).'">'.esc_html__('Resend Donation Receipt', 'give').'</a>';
 
 		}
 
-		$row_actions['delete'] = '<a href="' . wp_nonce_url( add_query_arg( array(
+		$row_actions['delete'] = '<a href="'.wp_nonce_url(add_query_arg(array(
 				'give-action' => 'delete_payment',
 				'purchase_id' => $payment->ID
-			), $this->base_url ), 'give_payment_nonce' ) . '">' . esc_html__( 'Delete', 'give' ) . '</a>';
+			), $this->base_url), 'give_payment_nonce').'">'.esc_html__('Delete', 'give').'</a>';
 
-		return apply_filters( 'give_payment_row_actions', $row_actions, $payment );
+		return apply_filters('give_payment_row_actions', $row_actions, $payment);
 	}
 
 	/**
@@ -437,10 +437,10 @@ 
 	 *
 	 * @return string                Data shown in the Email column
 	 */
-	function get_payment_status( $payment ) {
-		$value = '<div class="give-donation-status status-' . sanitize_title( give_get_payment_status( $payment, true ) ) . '"><span class="give-donation-status-icon"></span> ' . give_get_payment_status( $payment, true ) . '</div>';
-		if ( $payment->mode == 'test' ) {
-			$value .= ' <span class="give-item-label give-item-label-orange give-test-mode-transactions-label" data-tooltip="' . esc_attr__( 'This payment was made in test mode', 'give' ) . '">' . esc_html__( 'Test', 'give' ) . '</span>';
+	function get_payment_status($payment) {
+		$value = '<div class="give-donation-status status-'.sanitize_title(give_get_payment_status($payment, true)).'"><span class="give-donation-status-icon"></span> '.give_get_payment_status($payment, true).'</div>';
+		if ($payment->mode == 'test') {
+			$value .= ' <span class="give-item-label give-item-label-orange give-test-mode-transactions-label" data-tooltip="'.esc_attr__('This payment was made in test mode', 'give').'">'.esc_html__('Test', 'give').'</span>';
 		}
 
 		return $value;
@@ -456,7 +456,7 @@ 
 	 *
 	 * @return string Displays a checkbox
 	 */
-	public function column_cb( $payment ) {
+	public function column_cb($payment) {
 		return sprintf(
 			'<input type="checkbox" name="%1$s[]" value="%2$s" />',
 			'payment',
@@ -474,8 +474,8 @@ 
 	 *
 	 * @return string Displays a checkbox
 	 */
-	public function get_payment_id( $payment ) {
-		return '<span class="give-payment-id">' . give_get_payment_number( $payment->ID ) . '</span>';
+	public function get_payment_id($payment) {
+		return '<span class="give-payment-id">'.give_get_payment_number($payment->ID).'</span>';
 	}
 
 	/**
@@ -488,19 +488,19 @@ 
 	 *
 	 * @return string Data shown in the User column
 	 */
-	public function get_donor( $payment ) {
+	public function get_donor($payment) {
 
-		$customer_id = give_get_payment_customer_id( $payment->ID );
+		$customer_id = give_get_payment_customer_id($payment->ID);
 
-		if ( ! empty( $customer_id ) ) {
-			$customer = new Give_Customer( $customer_id );
-			$value    = '<a href="' . esc_url( admin_url( "edit.php?post_type=give_forms&page=give-donors&view=overview&id=$customer_id" ) ) . '">' . $customer->name . '</a>';
+		if ( ! empty($customer_id)) {
+			$customer = new Give_Customer($customer_id);
+			$value    = '<a href="'.esc_url(admin_url("edit.php?post_type=give_forms&page=give-donors&view=overview&id=$customer_id")).'">'.$customer->name.'</a>';
 		} else {
-			$email = give_get_payment_user_email( $payment->ID );
-			$value = '<a href="' . esc_url( admin_url( "edit.php?post_type=give_forms&page=give-payment-history&s=$email" ) ) . '">' . esc_html__( '(donor missing)', 'give' ) . '</a>';
+			$email = give_get_payment_user_email($payment->ID);
+			$value = '<a href="'.esc_url(admin_url("edit.php?post_type=give_forms&page=give-payment-history&s=$email")).'">'.esc_html__('(donor missing)', 'give').'</a>';
 		}
 
-		return apply_filters( 'give_payments_table_column', $value, $payment->ID, 'donor' );
+		return apply_filters('give_payments_table_column', $value, $payment->ID, 'donor');
 	}
 
 	/**
@@ -512,18 +512,18 @@ 
 	 */
 	public function get_bulk_actions() {
 		$actions = array(
-			'delete'               => esc_html__( 'Delete', 'give' ),
-			'set-status-publish'   => esc_html__( 'Set To Completed', 'give' ),
-			'set-status-pending'   => esc_html__( 'Set To Pending', 'give' ),
-			'set-status-refunded'  => esc_html__( 'Set To Refunded', 'give' ),
-			'set-status-revoked'   => esc_html__( 'Set To Revoked', 'give' ),
-			'set-status-failed'    => esc_html__( 'Set To Failed', 'give' ),
-			'set-status-cancelled' => esc_html__( 'Set To Cancelled', 'give' ),
-			'set-status-abandoned' => esc_html__( 'Set To Abandoned', 'give' ),
-			'resend-receipt'       => esc_html__( 'Resend Email Receipts', 'give' )
+			'delete'               => esc_html__('Delete', 'give'),
+			'set-status-publish'   => esc_html__('Set To Completed', 'give'),
+			'set-status-pending'   => esc_html__('Set To Pending', 'give'),
+			'set-status-refunded'  => esc_html__('Set To Refunded', 'give'),
+			'set-status-revoked'   => esc_html__('Set To Revoked', 'give'),
+			'set-status-failed'    => esc_html__('Set To Failed', 'give'),
+			'set-status-cancelled' => esc_html__('Set To Cancelled', 'give'),
+			'set-status-abandoned' => esc_html__('Set To Abandoned', 'give'),
+			'resend-receipt'       => esc_html__('Resend Email Receipts', 'give')
 		);
 
-		return apply_filters( 'give_payments_table_bulk_actions', $actions );
+		return apply_filters('give_payments_table_bulk_actions', $actions);
 	}
 
 	/**
@@ -534,59 +534,59 @@ 
 	 * @return void
 	 */
 	public function process_bulk_action() {
-		$ids    = isset( $_GET['payment'] ) ? $_GET['payment'] : false;
+		$ids    = isset($_GET['payment']) ? $_GET['payment'] : false;
 		$action = $this->current_action();
 
-		if ( ! is_array( $ids ) ) {
-			$ids = array( $ids );
+		if ( ! is_array($ids)) {
+			$ids = array($ids);
 		}
 
-		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':
-					give_delete_purchase( $id );
+					give_delete_purchase($id);
 					break;
 
 				case 'set-status-publish':
-					give_update_payment_status( $id, 'publish' );
+					give_update_payment_status($id, 'publish');
 					break;
 
 				case 'set-status-pending':
-					give_update_payment_status( $id, 'pending' );
+					give_update_payment_status($id, 'pending');
 					break;
 
 				case 'set-status-refunded':
-					give_update_payment_status( $id, 'refunded' );
+					give_update_payment_status($id, 'refunded');
 					break;
 				case 'set-status-revoked':
-					give_update_payment_status( $id, 'revoked' );
+					give_update_payment_status($id, 'revoked');
 					break;
 
 				case 'set-status-failed':
-					give_update_payment_status( $id, 'failed' );
+					give_update_payment_status($id, 'failed');
 					break;
 
 				case 'set-status-cancelled':
-					give_update_payment_status( $id, 'cancelled' );
+					give_update_payment_status($id, 'cancelled');
 					break;
 
 				case 'set-status-abandoned':
-					give_update_payment_status( $id, 'abandoned' );
+					give_update_payment_status($id, 'abandoned');
 					break;
 
 				case 'set-status-preapproval':
-					give_update_payment_status( $id, 'preapproval' );
+					give_update_payment_status($id, 'preapproval');
 					break;
 
 				case 'resend-receipt':
-					give_email_donation_receipt( $id, false );
+					give_email_donation_receipt($id, false);
 					break;
 			}
 
@@ -598,7 +598,7 @@ 
 			 * @param int    $id The ID of the payment.
 			 * @param string $current_action The action that is being triggered.
 			 */
-			do_action( 'give_payments_table_do_bulk_action', $id, $this->current_action() );
+			do_action('give_payments_table_do_bulk_action', $id, $this->current_action());
 		}
 
 	}
@@ -614,27 +614,27 @@ 
 
 		$args = array();
 
-		if ( isset( $_GET['user'] ) ) {
-			$args['user'] = urldecode( $_GET['user'] );
-		} elseif ( isset( $_GET['s'] ) ) {
-			$is_user = strpos( $_GET['s'], strtolower( 'user:' ) ) !== false;
-			if ( $is_user ) {
-				$args['user'] = absint( trim( str_replace( 'user:', '', strtolower( $_GET['s'] ) ) ) );
-				unset( $args['s'] );
+		if (isset($_GET['user'])) {
+			$args['user'] = urldecode($_GET['user']);
+		} elseif (isset($_GET['s'])) {
+			$is_user = strpos($_GET['s'], strtolower('user:')) !== false;
+			if ($is_user) {
+				$args['user'] = absint(trim(str_replace('user:', '', strtolower($_GET['s']))));
+				unset($args['s']);
 			} else {
-				$args['s'] = sanitize_text_field( $_GET['s'] );
+				$args['s'] = sanitize_text_field($_GET['s']);
 			}
 		}
 
-		if ( ! empty( $_GET['start-date'] ) ) {
-			$args['start-date'] = urldecode( $_GET['start-date'] );
+		if ( ! empty($_GET['start-date'])) {
+			$args['start-date'] = urldecode($_GET['start-date']);
 		}
 
-		if ( ! empty( $_GET['end-date'] ) ) {
-			$args['end-date'] = urldecode( $_GET['end-date'] );
+		if ( ! empty($_GET['end-date'])) {
+			$args['end-date'] = urldecode($_GET['end-date']);
 		}
 
-		$payment_count         = give_count_payments( $args );
+		$payment_count         = give_count_payments($args);
 		$this->complete_count  = $payment_count->publish;
 		$this->pending_count   = $payment_count->pending;
 		$this->refunded_count  = $payment_count->refunded;
@@ -643,7 +643,7 @@ 
 		$this->cancelled_count = $payment_count->cancelled;
 		$this->abandoned_count = $payment_count->abandoned;
 
-		foreach ( $payment_count as $count ) {
+		foreach ($payment_count as $count) {
 			$this->total_count += $count;
 		}
 	}
@@ -658,26 +658,26 @@ 
 	public function payments_data() {
 
 		$per_page   = $this->per_page;
-		$orderby    = isset( $_GET['orderby'] ) ? urldecode( $_GET['orderby'] ) : 'ID';
-		$order      = isset( $_GET['order'] ) ? $_GET['order'] : 'DESC';
-		$user       = isset( $_GET['user'] ) ? $_GET['user'] : null;
-		$status     = isset( $_GET['status'] ) ? $_GET['status'] : give_get_payment_status_keys();
-		$meta_key   = isset( $_GET['meta_key'] ) ? $_GET['meta_key'] : null;
-		$year       = isset( $_GET['year'] ) ? $_GET['year'] : null;
-		$month      = isset( $_GET['m'] ) ? $_GET['m'] : null;
-		$day        = isset( $_GET['day'] ) ? $_GET['day'] : null;
-		$search     = isset( $_GET['s'] ) ? sanitize_text_field( $_GET['s'] ) : null;
-		$start_date = isset( $_GET['start-date'] ) ? sanitize_text_field( $_GET['start-date'] ) : null;
-		$end_date   = isset( $_GET['end-date'] ) ? sanitize_text_field( $_GET['end-date'] ) : $start_date;
-
-		if ( ! empty( $search ) ) {
+		$orderby    = isset($_GET['orderby']) ? urldecode($_GET['orderby']) : 'ID';
+		$order      = isset($_GET['order']) ? $_GET['order'] : 'DESC';
+		$user       = isset($_GET['user']) ? $_GET['user'] : null;
+		$status     = isset($_GET['status']) ? $_GET['status'] : give_get_payment_status_keys();
+		$meta_key   = isset($_GET['meta_key']) ? $_GET['meta_key'] : null;
+		$year       = isset($_GET['year']) ? $_GET['year'] : null;
+		$month      = isset($_GET['m']) ? $_GET['m'] : null;
+		$day        = isset($_GET['day']) ? $_GET['day'] : null;
+		$search     = isset($_GET['s']) ? sanitize_text_field($_GET['s']) : null;
+		$start_date = isset($_GET['start-date']) ? sanitize_text_field($_GET['start-date']) : null;
+		$end_date   = isset($_GET['end-date']) ? sanitize_text_field($_GET['end-date']) : $start_date;
+
+		if ( ! empty($search)) {
 			$status = 'any'; // Force all payment statuses when searching
 		}
 
 		$args = array(
 			'output'     => 'payments',
 			'number'     => $per_page,
-			'page'       => isset( $_GET['paged'] ) ? $_GET['paged'] : null,
+			'page'       => isset($_GET['paged']) ? $_GET['paged'] : null,
 			'orderby'    => $orderby,
 			'order'      => $order,
 			'user'       => $user,
@@ -691,14 +691,14 @@ 
 			'end_date'   => $end_date,
 		);
 
-		if ( is_string( $search ) && false !== strpos( $search, 'txn:' ) ) {
+		if (is_string($search) && false !== strpos($search, 'txn:')) {
 
 			$args['search_in_notes'] = true;
-			$args['s']               = trim( str_replace( 'txn:', '', $args['s'] ) );
+			$args['s']               = trim(str_replace('txn:', '', $args['s']));
 
 		}
 
-		$p_query = new Give_Payments_Query( $args );
+		$p_query = new Give_Payments_Query($args);
 
 		return $p_query->get_payments();
 
@@ -718,17 +718,17 @@ 
 	 */
 	public function prepare_items() {
 
-		wp_reset_vars( array( 'action', 'payment', 'orderby', 'order', 's' ) );
+		wp_reset_vars(array('action', 'payment', 'orderby', 'order', 's'));
 
 		$columns  = $this->get_columns();
 		$hidden   = array(); // No hidden columns
 		$sortable = $this->get_sortable_columns();
 		$data     = $this->payments_data();
-		$status   = isset( $_GET['status'] ) ? $_GET['status'] : 'any';
+		$status   = isset($_GET['status']) ? $_GET['status'] : 'any';
 
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 
-		switch ( $status ) {
+		switch ($status) {
 			case 'publish':
 				$total_items = $this->complete_count;
 				break;
@@ -755,19 +755,19 @@ 
 				break;
 			default:
 				// Retrieve the count of the non-default-Give status
-				$count       = wp_count_posts( 'give_payment' );
+				$count       = wp_count_posts('give_payment');
 				$total_items = $count->{$status};
 				break;
 		}
 
 		$this->items = $data;
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 				'total_items' => $total_items,
 				// WE have to calculate the total number of items
 				'per_page'    => $this->per_page,
 				// WE have to determine how many items to show on a page
-				'total_pages' => ceil( $total_items / $this->per_page )
+				'total_pages' => ceil($total_items / $this->per_page)
 				// WE have to calculate the total number of pages
 			)
 		);

includes/forms/widget.php

Spacing +51 added lines, -51 removed lines

@@ -10,14 +10,14 @@ 
  */
 
 // Exit if accessed directly
-defined( 'ABSPATH' ) or exit;
+defined('ABSPATH') or exit;
 
 /**
  * Give Form widget
  *
  * @since 1.0
  */
-class Give_Forms_Widget extends WP_Widget{
+class Give_Forms_Widget extends WP_Widget {
 	/**
 	 * The widget class name
 	 *
@@ -28,19 +28,19 @@ 
 	/**
 	 * Instantiate the class
 	 */
-	public function __construct(){
-		$this->self = get_class( $this );
+	public function __construct() {
+		$this->self = get_class($this);
 
 		parent::__construct(
-			strtolower( $this->self ),
-			esc_html__( 'Give - Donation Form', 'give' ),
+			strtolower($this->self),
+			esc_html__('Give - Donation Form', 'give'),
 			array(
-				'description' => esc_html__( 'Display a Give Donation Form in your theme\'s widget powered sidebar.', 'give' )
+				'description' => esc_html__('Display a Give Donation Form in your theme\'s widget powered sidebar.', 'give')
 			)
 		);
 
-		add_action( 'widgets_init',          array( $this, 'widget_init' ) );
-		add_action( 'admin_enqueue_scripts', array( $this, 'admin_widget_scripts' ) );
+		add_action('widgets_init', array($this, 'widget_init'));
+		add_action('admin_enqueue_scripts', array($this, 'admin_widget_scripts'));
 	}
 
 	/**
@@ -50,23 +50,23 @@ 
 	 *
 	 * @return void
 	 */
-	public function admin_widget_scripts( $hook ){
+	public function admin_widget_scripts($hook) {
 		// Directories of assets
-		$js_dir     = GIVE_PLUGIN_URL . 'assets/js/admin/';
-		$js_plugins = GIVE_PLUGIN_URL . 'assets/js/plugins/';
-		$css_dir    = GIVE_PLUGIN_URL . 'assets/css/';
+		$js_dir     = GIVE_PLUGIN_URL.'assets/js/admin/';
+		$js_plugins = GIVE_PLUGIN_URL.'assets/js/plugins/';
+		$css_dir    = GIVE_PLUGIN_URL.'assets/css/';
 
 		// Use minified libraries if SCRIPT_DEBUG is turned off
-		$suffix = ( defined( 'SCRIPT_DEBUG' ) && SCRIPT_DEBUG ) ? '' : '.min';
+		$suffix = (defined('SCRIPT_DEBUG') && SCRIPT_DEBUG) ? '' : '.min';
 
 		// Widget Script
-		if ( $hook == 'widgets.php' ) {
+		if ($hook == 'widgets.php') {
 
-			wp_enqueue_style( 'give-qtip-css', $css_dir . 'jquery.qtip' . $suffix . '.css' );
+			wp_enqueue_style('give-qtip-css', $css_dir.'jquery.qtip'.$suffix.'.css');
 
-			wp_enqueue_script( 'give-qtip', $js_plugins . 'jquery.qtip' . $suffix . '.js', array( 'jquery' ), GIVE_VERSION );
+			wp_enqueue_script('give-qtip', $js_plugins.'jquery.qtip'.$suffix.'.js', array('jquery'), GIVE_VERSION);
 
-			wp_enqueue_script( 'give-admin-widgets-scripts', $js_dir . 'admin-widgets' . $suffix . '.js', array( 'jquery' ), GIVE_VERSION, false );
+			wp_enqueue_script('give-admin-widgets-scripts', $js_dir.'admin-widgets'.$suffix.'.js', array('jquery'), GIVE_VERSION, false);
 		}
 	}
 
@@ -77,21 +77,21 @@ 
 	 *                        before_widget, and after_widget.
 	 * @param array $instance The settings for the particular instance of the widget.
 	 */
-	public function widget( $args, $instance ){
-		$title = !empty( $instance['title'] ) ? $instance['title'] : '';
-		$title = apply_filters( 'widget_title', $title, $instance, $this->id_base );
+	public function widget($args, $instance) {
+		$title = ! empty($instance['title']) ? $instance['title'] : '';
+		$title = apply_filters('widget_title', $title, $instance, $this->id_base);
 
 		echo $args['before_widget'];
 
-		do_action( 'give_before_forms_widget' );
+		do_action('give_before_forms_widget');
 
-		echo $title ? $args['before_title'] . $title . $args['after_title'] : '';
+		echo $title ? $args['before_title'].$title.$args['after_title'] : '';
 
-		give_get_donation_form( $instance );
+		give_get_donation_form($instance);
 
 		echo $args['after_widget'];
 
-		do_action( 'give_after_forms_widget' );
+		do_action('give_after_forms_widget');
 	}
 
 	/**
@@ -101,65 +101,65 @@ 
 	 *
 	 * @return string
 	 */
-	public function form( $instance ){
+	public function form($instance) {
 		$defaults = array(
 			'title'        => '',
 			'id'           => '',
 			'float_labels' => '',
 		);
 
-		$instance = wp_parse_args( (array) $instance, $defaults );
+		$instance = wp_parse_args((array) $instance, $defaults);
 
 		// Query Give Forms
 		$args = array(
 			'post_type'      => 'give_forms',
-			'posts_per_page' => - 1,
+			'posts_per_page' => -1,
 			'post_status'    => 'publish',
 		);
 
-		$give_forms = get_posts( $args );
+		$give_forms = get_posts($args);
 
 		// Widget: Title
 
 		?><p>
-			<label for="<?php echo $this->get_field_id( 'title' ); ?>"><?php esc_html_e( 'Title:', 'give' ); ?></label>
-			<input type="text" class="widefat" id="<?php echo $this->get_field_id( 'title' ); ?>" name="<?php echo $this->get_field_name( 'title' ); ?>" value="<?php esc_attr_e( $instance['title'] ); ?>" /><br>
-			<small><?php esc_html_e( 'Leave blank to hide the widget title.', 'give' ); ?></small>
+			<label for="<?php echo $this->get_field_id('title'); ?>"><?php esc_html_e('Title:', 'give'); ?></label>
+			<input type="text" class="widefat" id="<?php echo $this->get_field_id('title'); ?>" name="<?php echo $this->get_field_name('title'); ?>" value="<?php esc_attr_e($instance['title']); ?>" /><br>
+			<small><?php esc_html_e('Leave blank to hide the widget title.', 'give'); ?></small>
 		</p><?php
 
 		// Widget: Give Form
 
 		?><p>
-			<label for="<?php echo esc_attr( $this->get_field_id( 'id' ) ); ?>"><?php
+			<label for="<?php echo esc_attr($this->get_field_id('id')); ?>"><?php
 				printf(
 					/* translators: %s: form singular label */
-					esc_html__( 'Give %s:', 'give' ),
+					esc_html__('Give %s:', 'give'),
 					give_get_forms_label_singular()
 				);
 			?></label>
-			<select class="widefat" name="<?php echo esc_attr( $this->get_field_name( 'id' ) ); ?>" id="<?php echo esc_attr( $this->get_field_id( 'id' ) ); ?>">
-				<option value="current"><?php esc_html_e( '— Select —', 'give' ); ?></option>
-				<?php foreach ( $give_forms as $give_form ) { ?>
-					<option <?php selected( absint( $instance['id'] ), $give_form->ID ); ?> value="<?php echo esc_attr( $give_form->ID ); ?>"><?php echo $give_form->post_title; ?></option>
+			<select class="widefat" name="<?php echo esc_attr($this->get_field_name('id')); ?>" id="<?php echo esc_attr($this->get_field_id('id')); ?>">
+				<option value="current"><?php esc_html_e('— Select —', 'give'); ?></option>
+				<?php foreach ($give_forms as $give_form) { ?>
+					<option <?php selected(absint($instance['id']), $give_form->ID); ?> value="<?php echo esc_attr($give_form->ID); ?>"><?php echo $give_form->post_title; ?></option>
 				<?php } ?>
 			</select><br>
-			<small><?php esc_html_e( 'Select a Give Form to embed in this widget.', 'give' ); ?></small>
+			<small><?php esc_html_e('Select a Give Form to embed in this widget.', 'give'); ?></small>
 		</p><?php
 
 		// Widget: Floating Labels
 
 		?><p>
-			<label for="<?php echo esc_attr( $this->get_field_id( 'float_labels' ) ); ?>"><?php esc_html_e( 'Floating Labels (optional):', 'give' ); ?></label>
-			<select class="widefat" name="<?php echo esc_attr( $this->get_field_name( 'float_labels' ) ); ?>" id="<?php echo esc_attr( $this->get_field_id( 'float_labels' ) ); ?>">
-				<option value="" <?php selected( esc_attr( $instance['float_labels'] ), '' ) ?>><?php esc_html_e( '- Select -', 'give' ); ?></option>
-				<option value="enabled" <?php selected( esc_attr( $instance['float_labels'] ), 'enabled' ) ?>><?php esc_html_e( 'Enabled', 'give' ); ?></option>
-				<option value="disabled" <?php selected( esc_attr( $instance['float_labels'] ), 'disabled' ) ?>><?php esc_html_e( 'Disabled', 'give' ); ?></option>
+			<label for="<?php echo esc_attr($this->get_field_id('float_labels')); ?>"><?php esc_html_e('Floating Labels (optional):', 'give'); ?></label>
+			<select class="widefat" name="<?php echo esc_attr($this->get_field_name('float_labels')); ?>" id="<?php echo esc_attr($this->get_field_id('float_labels')); ?>">
+				<option value="" <?php selected(esc_attr($instance['float_labels']), '') ?>><?php esc_html_e('- Select -', 'give'); ?></option>
+				<option value="enabled" <?php selected(esc_attr($instance['float_labels']), 'enabled') ?>><?php esc_html_e('Enabled', 'give'); ?></option>
+				<option value="disabled" <?php selected(esc_attr($instance['float_labels']), 'disabled') ?>><?php esc_html_e('Disabled', 'give'); ?></option>
 			</select><br>
 			<small><?php
 				printf(
 					/* translators: %s: https://givewp.com/documentation/core/give-forms/creating-give-forms/#floating-labels */
-					__( 'Override the <a href="%s" target="_blank">floating labels</a> setting for this Give form.', 'give' ),
-					esc_url( 'https://givewp.com/documentation/core/give-forms/creating-give-forms/#floating-labels' )
+					__('Override the <a href="%s" target="_blank">floating labels</a> setting for this Give form.', 'give'),
+					esc_url('https://givewp.com/documentation/core/give-forms/creating-give-forms/#floating-labels')
 				);
 			?></small>
 		</p><?php
@@ -170,8 +170,8 @@ 
 	 *
 	 * @return void
 	 */
-	function widget_init(){
-		register_widget( $this->self );
+	function widget_init() {
+		register_widget($this->self);
 	}
 
 	/**
@@ -182,7 +182,7 @@ 
 	 *
 	 * @return array
 	 */
-	public function update( $new_instance, $old_instance ){
+	public function update($new_instance, $old_instance) {
 		$this->flush_widget_cache();
 
 		return $new_instance;
@@ -193,8 +193,8 @@ 
 	 *
 	 * @return void
 	 */
-	public function flush_widget_cache(){
-		wp_cache_delete( $this->self, 'widget' );
+	public function flush_widget_cache() {
+		wp_cache_delete($this->self, 'widget');
 	}
 }