WordImpress / Give

includes/currency-functions.php

Spacing +138 added lines, -138 removed lines

@@ -20,19 +20,19 @@ 
  *
  * @return string The currency code
  */
-function give_get_currency( $donation_or_form_id = null, $args = array() ) {
+function give_get_currency($donation_or_form_id = null, $args = array()) {
 
 	// Get currency from donation
-	if ( is_numeric( $donation_or_form_id ) && 'give_payment' === get_post_type( $donation_or_form_id ) ) {
-		$donation_meta = give_get_meta( $donation_or_form_id, '_give_payment_meta', true );
+	if (is_numeric($donation_or_form_id) && 'give_payment' === get_post_type($donation_or_form_id)) {
+		$donation_meta = give_get_meta($donation_or_form_id, '_give_payment_meta', true);
 
-		if ( ! empty( $donation_meta['currency'] ) ) {
+		if ( ! empty($donation_meta['currency'])) {
 			$currency = $donation_meta['currency'];
 		} else {
-			$currency = give_get_option( 'currency', 'USD' );
+			$currency = give_get_option('currency', 'USD');
 		}
 	} else {
-		$currency = give_get_option( 'currency', 'USD' );
+		$currency = give_get_option('currency', 'USD');
 	}
 
 	/**
@@ -40,7 +40,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	return apply_filters( 'give_currency', $currency, $donation_or_form_id, $args );
+	return apply_filters('give_currency', $currency, $donation_or_form_id, $args);
 }
 
 /**
@@ -52,9 +52,9 @@ 
  */
 function give_get_currency_position() {
 
-	$currency_pos = give_get_option( 'currency_position', 'before' );
+	$currency_pos = give_get_option('currency_position', 'before');
 
-	return apply_filters( 'give_currency_position', $currency_pos );
+	return apply_filters('give_currency_position', $currency_pos);
 }
 
 /**
@@ -67,7 +67,7 @@ 
 function give_get_currencies_list() {
 	$currencies = array(
 		'USD' => array(
-			'admin_label' => __( 'US Dollars ($)', 'give' ),
+			'admin_label' => __('US Dollars ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -77,7 +77,7 @@ 
 			),
 		),
 		'EUR' => array(
-			'admin_label' => __( 'Euros (€)', 'give' ),
+			'admin_label' => __('Euros (€)', 'give'),
 			'symbol'      => '€',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -87,7 +87,7 @@ 
 			),
 		),
 		'GBP' => array(
-			'admin_label' => __( 'Pounds Sterling (£)', 'give' ),
+			'admin_label' => __('Pounds Sterling (£)', 'give'),
 			'symbol'      => '£',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -97,7 +97,7 @@ 
 			),
 		),
 		'AUD' => array(
-			'admin_label' => __( 'Australian Dollars ($)', 'give' ),
+			'admin_label' => __('Australian Dollars ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -107,7 +107,7 @@ 
 			),
 		),
 		'BRL' => array(
-			'admin_label' => __( 'Brazilian Real (R$)', 'give' ),
+			'admin_label' => __('Brazilian Real (R$)', 'give'),
 			'symbol'      => 'R$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -117,7 +117,7 @@ 
 			),
 		),
 		'CAD' => array(
-			'admin_label' => __( 'Canadian Dollars ($)', 'give' ),
+			'admin_label' => __('Canadian Dollars ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -127,7 +127,7 @@ 
 			),
 		),
 		'CZK' => array(
-			'admin_label' => __( 'Czech Koruna (Kč)', 'give' ),
+			'admin_label' => __('Czech Koruna (Kč)', 'give'),
 			'symbol'      => 'Kč',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -137,7 +137,7 @@ 
 			),
 		),
 		'DKK' => array(
-			'admin_label' => __( 'Danish Krone (kr.)', 'give' ),
+			'admin_label' => __('Danish Krone (kr.)', 'give'),
 			'symbol'      => ' kr. ',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -147,7 +147,7 @@ 
 			),
 		),
 		'HKD' => array(
-			'admin_label' => __( 'Hong Kong Dollar ($)', 'give' ),
+			'admin_label' => __('Hong Kong Dollar ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -157,7 +157,7 @@ 
 			),
 		),
 		'HUF' => array(
-			'admin_label' => __( 'Hungarian Forint (Ft)', 'give' ),
+			'admin_label' => __('Hungarian Forint (Ft)', 'give'),
 			'symbol'      => 'Ft',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -167,7 +167,7 @@ 
 			),
 		),
 		'ILS' => array(
-			'admin_label' => __( 'Israeli Shekel (₪)', 'give' ),
+			'admin_label' => __('Israeli Shekel (₪)', 'give'),
 			'symbol'      => '₪',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -177,7 +177,7 @@ 
 			),
 		),
 		'JPY' => array(
-			'admin_label' => __( 'Japanese Yen (¥)', 'give' ),
+			'admin_label' => __('Japanese Yen (¥)', 'give'),
 			'symbol'      => '¥',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -187,7 +187,7 @@ 
 			),
 		),
 		'MYR' => array(
-			'admin_label' => __( 'Malaysian Ringgits (RM)', 'give' ),
+			'admin_label' => __('Malaysian Ringgits (RM)', 'give'),
 			'symbol'      => 'RM',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -197,7 +197,7 @@ 
 			),
 		),
 		'MXN' => array(
-			'admin_label' => __( 'Mexican Peso ($)', 'give' ),
+			'admin_label' => __('Mexican Peso ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -207,7 +207,7 @@ 
 			),
 		),
 		'MAD' => array(
-			'admin_label' => __( 'Moroccan Dirham (.د.م)', 'give' ),
+			'admin_label' => __('Moroccan Dirham (.د.م)', 'give'),
 			'symbol'      => '.د.م',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -217,7 +217,7 @@ 
 			),
 		),
 		'NZD' => array(
-			'admin_label' => __( 'New Zealand Dollar ($)', 'give' ),
+			'admin_label' => __('New Zealand Dollar ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -227,7 +227,7 @@ 
 			),
 		),
 		'NOK' => array(
-			'admin_label' => __( 'Norwegian Krone (Kr.)', 'give' ),
+			'admin_label' => __('Norwegian Krone (Kr.)', 'give'),
 			'symbol'      => 'kr.',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -237,7 +237,7 @@ 
 			),
 		),
 		'PHP' => array(
-			'admin_label' => __( 'Philippine Pesos (₱)', 'give' ),
+			'admin_label' => __('Philippine Pesos (₱)', 'give'),
 			'symbol'      => '₱',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -247,7 +247,7 @@ 
 			),
 		),
 		'PLN' => array(
-			'admin_label' => __( 'Polish Zloty (zł)', 'give' ),
+			'admin_label' => __('Polish Zloty (zł)', 'give'),
 			'symbol'      => 'zł',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -257,7 +257,7 @@ 
 			),
 		),
 		'SGD' => array(
-			'admin_label' => __( 'Singapore Dollar ($)', 'give' ),
+			'admin_label' => __('Singapore Dollar ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -267,7 +267,7 @@ 
 			),
 		),
 		'KRW' => array(
-			'admin_label' => __( 'South Korean Won (₩)', 'give' ),
+			'admin_label' => __('South Korean Won (₩)', 'give'),
 			'symbol'      => '₩',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -277,7 +277,7 @@ 
 			),
 		),
 		'ZAR' => array(
-			'admin_label' => __( 'South African Rand (R)', 'give' ),
+			'admin_label' => __('South African Rand (R)', 'give'),
 			'symbol'      => 'R',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -287,7 +287,7 @@ 
 			),
 		),
 		'SEK' => array(
-			'admin_label' => __( 'Swedish Krona (kr)', 'give' ),
+			'admin_label' => __('Swedish Krona (kr)', 'give'),
 			'symbol'      => ' kr. ',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -297,7 +297,7 @@ 
 			),
 		),
 		'CHF' => array(
-			'admin_label' => __( 'Swiss Franc (Fr)', 'give' ),
+			'admin_label' => __('Swiss Franc (Fr)', 'give'),
 			'symbol'      => 'Fr',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -307,7 +307,7 @@ 
 			),
 		),
 		'TWD' => array(
-			'admin_label' => __( 'Taiwan New Dollars (NT$)', 'give' ),
+			'admin_label' => __('Taiwan New Dollars (NT$)', 'give'),
 			'symbol'      => 'NT$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -317,7 +317,7 @@ 
 			),
 		),
 		'THB' => array(
-			'admin_label' => __( 'Thai Baht (฿)', 'give' ),
+			'admin_label' => __('Thai Baht (฿)', 'give'),
 			'symbol'      => '฿',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -327,7 +327,7 @@ 
 			),
 		),
 		'INR' => array(
-			'admin_label' => __( 'Indian Rupee (₹)', 'give' ),
+			'admin_label' => __('Indian Rupee (₹)', 'give'),
 			'symbol'      => '₹',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -337,7 +337,7 @@ 
 			),
 		),
 		'TRY' => array(
-			'admin_label' => __( 'Turkish Lira (₺)', 'give' ),
+			'admin_label' => __('Turkish Lira (₺)', 'give'),
 			'symbol'      => '₺',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -347,7 +347,7 @@ 
 			),
 		),
 		'IRR' => array(
-			'admin_label' => __( 'Iranian Rial (﷼)', 'give' ),
+			'admin_label' => __('Iranian Rial (﷼)', 'give'),
 			'symbol'      => '﷼',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -357,7 +357,7 @@ 
 			),
 		),
 		'RUB' => array(
-			'admin_label' => __( 'Russian Rubles (руб)', 'give' ),
+			'admin_label' => __('Russian Rubles (руб)', 'give'),
 			'symbol'      => '₽',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -367,7 +367,7 @@ 
 			),
 		),
 		'AED' => array(
-			'admin_label' => __( 'United Arab Emirates dirham (د.إ)', 'give' ),
+			'admin_label' => __('United Arab Emirates dirham (د.إ)', 'give'),
 			'symbol'      => 'د.إ',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -377,7 +377,7 @@ 
 			),
 		),
 		'AMD' => array(
-			'admin_label' => __( 'Armenian dram (AMD)', 'give' ),
+			'admin_label' => __('Armenian dram (AMD)', 'give'),
 			'symbol'      => 'AMD', // Add backward compatibility. Using AMD in place of ֏
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -387,7 +387,7 @@ 
 			),
 		),
 		'ANG' => array(
-			'admin_label' => __( 'Netherlands Antillean guilder (ƒ)', 'give' ),
+			'admin_label' => __('Netherlands Antillean guilder (ƒ)', 'give'),
 			'symbol'      => 'ƒ',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -397,7 +397,7 @@ 
 			),
 		),
 		'ARS' => array(
-			'admin_label' => __( 'Argentine peso ($)', 'give' ),
+			'admin_label' => __('Argentine peso ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -407,7 +407,7 @@ 
 			),
 		),
 		'AWG' => array(
-			'admin_label' => __( 'Aruban florin (ƒ)', 'give' ),
+			'admin_label' => __('Aruban florin (ƒ)', 'give'),
 			'symbol'      => 'ƒ',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -417,7 +417,7 @@ 
 			),
 		),
 		'BAM' => array(
-			'admin_label' => __( 'Bosnia and Herzegovina convertible mark (KM)', 'give' ),
+			'admin_label' => __('Bosnia and Herzegovina convertible mark (KM)', 'give'),
 			'symbol'      => 'KM',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -427,7 +427,7 @@ 
 			),
 		),
 		'BDT' => array(
-			'admin_label' => __( 'Bangladeshi taka (৳)', 'give' ),
+			'admin_label' => __('Bangladeshi taka (৳)', 'give'),
 			'symbol'      => '৳',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -437,7 +437,7 @@ 
 			),
 		),
 		'BHD' => array(
-			'admin_label' => __( 'Bahraini dinar (.د.ب)', 'give' ),
+			'admin_label' => __('Bahraini dinar (.د.ب)', 'give'),
 			'symbol'      => '.د.ب',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -447,7 +447,7 @@ 
 			),
 		),
 		'BMD' => array(
-			'admin_label' => __( 'Bermudian dollar (BD$)', 'give' ),
+			'admin_label' => __('Bermudian dollar (BD$)', 'give'),
 			'symbol'      => 'BD$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -457,7 +457,7 @@ 
 			),
 		),
 		'BND' => array(
-			'admin_label' => __( 'Brunei dollar (B$)', 'give' ),
+			'admin_label' => __('Brunei dollar (B$)', 'give'),
 			'symbol'      => 'B$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -467,7 +467,7 @@ 
 			),
 		),
 		'BOB' => array(
-			'admin_label' => __( 'Bolivian boliviano (Bs.)', 'give' ),
+			'admin_label' => __('Bolivian boliviano (Bs.)', 'give'),
 			'symbol'      => 'Bs.',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -477,7 +477,7 @@ 
 			),
 		),
 		'BSD' => array(
-			'admin_label' => __( 'Bahamian dollar (B$)', 'give' ),
+			'admin_label' => __('Bahamian dollar (B$)', 'give'),
 			'symbol'      => 'B$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -487,7 +487,7 @@ 
 			),
 		),
 		'BWP' => array(
-			'admin_label' => __( 'Botswana pula (P)', 'give' ),
+			'admin_label' => __('Botswana pula (P)', 'give'),
 			'symbol'      => 'P',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -497,7 +497,7 @@ 
 			),
 		),
 		'BZD' => array(
-			'admin_label' => __( 'Belizean dollar (BZ$)', 'give' ),
+			'admin_label' => __('Belizean dollar (BZ$)', 'give'),
 			'symbol'      => 'BZ$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -507,7 +507,7 @@ 
 			),
 		),
 		'CLP' => array(
-			'admin_label' => __( 'Chilean peso ($)', 'give' ),
+			'admin_label' => __('Chilean peso ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -517,7 +517,7 @@ 
 			),
 		),
 		'CNY' => array(
-			'admin_label' => __( 'Chinese yuan (¥)', 'give' ),
+			'admin_label' => __('Chinese yuan (¥)', 'give'),
 			'symbol'      => '¥',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -527,7 +527,7 @@ 
 			),
 		),
 		'COP' => array(
-			'admin_label' => __( 'Colombian peso ($)', 'give' ),
+			'admin_label' => __('Colombian peso ($)', 'give'),
 			'symbol'      => '$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -537,7 +537,7 @@ 
 			),
 		),
 		'CRC' => array(
-			'admin_label' => __( 'Costa Rican colón (₡)', 'give' ),
+			'admin_label' => __('Costa Rican colón (₡)', 'give'),
 			'symbol'      => '₡',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -547,7 +547,7 @@ 
 			),
 		),
 		'CUC' => array(
-			'admin_label' => __( 'Cuban convertible peso (₱)', 'give' ),
+			'admin_label' => __('Cuban convertible peso (₱)', 'give'),
 			'symbol'      => '₱',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -557,7 +557,7 @@ 
 			),
 		),
 		'CUP' => array(
-			'admin_label' => __( 'Cuban convertible peso (₱)', 'give' ),
+			'admin_label' => __('Cuban convertible peso (₱)', 'give'),
 			'symbol'      => '₱',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -567,7 +567,7 @@ 
 			),
 		),
 		'DOP' => array(
-			'admin_label' => __( 'Dominican peso (RD$)', 'give' ),
+			'admin_label' => __('Dominican peso (RD$)', 'give'),
 			'symbol'      => 'RD$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -577,7 +577,7 @@ 
 			),
 		),
 		'EGP' => array(
-			'admin_label' => __( 'Egyptian pound (E£)', 'give' ),
+			'admin_label' => __('Egyptian pound (E£)', 'give'),
 			'symbol'      => 'E£',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -587,7 +587,7 @@ 
 			),
 		),
 		'GIP' => array(
-			'admin_label' => __( 'Gibraltar pound (£)', 'give' ),
+			'admin_label' => __('Gibraltar pound (£)', 'give'),
 			'symbol'      => '£',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -597,7 +597,7 @@ 
 			),
 		),
 		'GTQ' => array(
-			'admin_label' => __( 'Guatemalan quetzal (Q)', 'give' ),
+			'admin_label' => __('Guatemalan quetzal (Q)', 'give'),
 			'symbol'      => 'Q',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -607,7 +607,7 @@ 
 			),
 		),
 		'HNL' => array(
-			'admin_label' => __( 'Honduran lempira (L)', 'give' ),
+			'admin_label' => __('Honduran lempira (L)', 'give'),
 			'symbol'      => 'L',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -617,7 +617,7 @@ 
 			),
 		),
 		'HRK' => array(
-			'admin_label' => __( 'Croatian kuna (kn)', 'give' ),
+			'admin_label' => __('Croatian kuna (kn)', 'give'),
 			'symbol'      => 'kn',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -627,7 +627,7 @@ 
 			),
 		),
 		'IDR' => array(
-			'admin_label' => __( 'Indonesian rupiah (Rp)', 'give' ),
+			'admin_label' => __('Indonesian rupiah (Rp)', 'give'),
 			'symbol'      => 'Rp',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -637,7 +637,7 @@ 
 			),
 		),
 		'ISK' => array(
-			'admin_label' => __( 'Icelandic króna (kr)', 'give' ),
+			'admin_label' => __('Icelandic króna (kr)', 'give'),
 			'symbol'      => 'kr',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -647,7 +647,7 @@ 
 			),
 		),
 		'JMD' => array(
-			'admin_label' => __( 'Jamaican dollar (J$)', 'give' ),
+			'admin_label' => __('Jamaican dollar (J$)', 'give'),
 			'symbol'      => 'j$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -657,7 +657,7 @@ 
 			),
 		),
 		'JOD' => array(
-			'admin_label' => __( 'Jordanian dinar (د.ا)', 'give' ),
+			'admin_label' => __('Jordanian dinar (د.ا)', 'give'),
 			'symbol'      => 'د.ا',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -667,7 +667,7 @@ 
 			),
 		),
 		'KES' => array(
-			'admin_label' => __( 'Kenyan shilling (KSh)', 'give' ),
+			'admin_label' => __('Kenyan shilling (KSh)', 'give'),
 			'symbol'      => 'KSh',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -677,7 +677,7 @@ 
 			),
 		),
 		'KWD' => array(
-			'admin_label' => __( 'Kuwaiti dinar (د.ك)', 'give' ),
+			'admin_label' => __('Kuwaiti dinar (د.ك)', 'give'),
 			'symbol'      => 'د.ك',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -687,7 +687,7 @@ 
 			),
 		),
 		'KYD' => array(
-			'admin_label' => __( 'Cayman Islands dollar (KY$)', 'give' ),
+			'admin_label' => __('Cayman Islands dollar (KY$)', 'give'),
 			'symbol'      => 'KY$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -697,7 +697,7 @@ 
 			),
 		),
 		'MKD' => array(
-			'admin_label' => __( 'Macedonian denar (ден)', 'give' ),
+			'admin_label' => __('Macedonian denar (ден)', 'give'),
 			'symbol'      => 'ден',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -707,7 +707,7 @@ 
 			),
 		),
 		'NPR' => array(
-			'admin_label' => __( 'Nepalese rupee (NɌs)', 'give' ),
+			'admin_label' => __('Nepalese rupee (NɌs)', 'give'),
 			'symbol'      => '₨',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -717,7 +717,7 @@ 
 			),
 		),
 		'OMR' => array(
-			'admin_label' => __( 'Omani rial (ر.ع.)', 'give' ),
+			'admin_label' => __('Omani rial (ر.ع.)', 'give'),
 			'symbol'      => 'ر.ع.',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -727,7 +727,7 @@ 
 			),
 		),
 		'PEN' => array(
-			'admin_label' => __( 'Peruvian nuevo sol (S/.)', 'give' ),
+			'admin_label' => __('Peruvian nuevo sol (S/.)', 'give'),
 			'symbol'      => 'S/.',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -737,7 +737,7 @@ 
 			),
 		),
 		'PKR' => array(
-			'admin_label' => __( 'Pakistani rupee (Ɍs)', 'give' ),
+			'admin_label' => __('Pakistani rupee (Ɍs)', 'give'),
 			'symbol'      => '₨',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -747,7 +747,7 @@ 
 			),
 		),
 		'RON' => array(
-			'admin_label' => __( 'Romanian leu (L)', 'give' ),
+			'admin_label' => __('Romanian leu (L)', 'give'),
 			'symbol'      => 'L',
 			'setting'     => array(
 				'currency_position'   => 'after',
@@ -757,7 +757,7 @@ 
 			),
 		),
 		'SAR' => array(
-			'admin_label' => __( 'Saudi riyal (ر.س)', 'give' ),
+			'admin_label' => __('Saudi riyal (ر.س)', 'give'),
 			'symbol'      => 'ر.س',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -767,7 +767,7 @@ 
 			),
 		),
 		'SZL' => array(
-			'admin_label' => __( 'Swazi lilangeni (L)', 'give' ),
+			'admin_label' => __('Swazi lilangeni (L)', 'give'),
 			'symbol'      => 'L',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -777,7 +777,7 @@ 
 			),
 		),
 		'TOP' => array(
-			'admin_label' => __( 'Tongan paʻanga (T$)', 'give' ),
+			'admin_label' => __('Tongan paʻanga (T$)', 'give'),
 			'symbol'      => 'T$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -787,7 +787,7 @@ 
 			),
 		),
 		'TZS' => array(
-			'admin_label' => __( 'Tanzanian shilling (TSh)', 'give' ),
+			'admin_label' => __('Tanzanian shilling (TSh)', 'give'),
 			'symbol'      => 'TSh',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -797,7 +797,7 @@ 
 			),
 		),
 		'UAH' => array(
-			'admin_label' => __( 'Ukrainian hryvnia (₴)', 'give' ),
+			'admin_label' => __('Ukrainian hryvnia (₴)', 'give'),
 			'symbol'      => '₴',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -807,7 +807,7 @@ 
 			),
 		),
 		'UYU' => array(
-			'admin_label' => __( 'Uruguayan peso ($U)', 'give' ),
+			'admin_label' => __('Uruguayan peso ($U)', 'give'),
 			'symbol'      => '$U',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -817,7 +817,7 @@ 
 			),
 		),
 		'VEF' => array(
-			'admin_label' => __( 'Venezuelan bolívar (Bs)', 'give' ),
+			'admin_label' => __('Venezuelan bolívar (Bs)', 'give'),
 			'symbol'      => 'Bs',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -827,7 +827,7 @@ 
 			),
 		),
 		'XCD' => array(
-			'admin_label' => __( 'East Caribbean dollar (EC$)', 'give' ),
+			'admin_label' => __('East Caribbean dollar (EC$)', 'give'),
 			'symbol'      => 'EC$',
 			'setting'     => array(
 				'currency_position'   => 'before',
@@ -852,7 +852,7 @@ 
 	 *
 	 * @param array $currencies
 	 */
-	return (array) apply_filters( 'give_currencies', $currencies );
+	return (array) apply_filters('give_currencies', $currencies);
 }
 
 /**
@@ -864,22 +864,22 @@ 
  *
  * @return array $currencies A list of the available currencies
  */
-function give_get_currencies( $info = 'admin_label' ) {
+function give_get_currencies($info = 'admin_label') {
 
 	$currencies = give_get_currencies_list();
 
 	// Backward compatibility: handle old way of currency registration.
 	// Backward compatibility: Return desired result.
-	if ( ! empty( $currencies ) ) {
-		foreach ( $currencies as $currency_code => $currency_setting ) {
-			if ( is_string( $currency_setting ) ) {
-				$currencies[ $currency_code ] = array(
+	if ( ! empty($currencies)) {
+		foreach ($currencies as $currency_code => $currency_setting) {
+			if (is_string($currency_setting)) {
+				$currencies[$currency_code] = array(
 					'admin_label' => $currency_setting,
 				);
 			}
 
-			$currencies[ $currency_code ] = wp_parse_args(
-				$currencies[ $currency_code ],
+			$currencies[$currency_code] = wp_parse_args(
+				$currencies[$currency_code],
 				array(
 					'admin_label' => '',
 					'symbol'      => $currency_code,
@@ -888,8 +888,8 @@ 
 			);
 		}
 
-		if ( ! empty( $info ) && is_string( $info ) && 'all' !== $info ) {
-			$currencies = wp_list_pluck( $currencies, $info );
+		if ( ! empty($info) && is_string($info) && 'all' !== $info) {
+			$currencies = wp_list_pluck($currencies, $info);
 		}
 	}
 
@@ -906,12 +906,12 @@ 
  *
  * @return array
  */
-function give_currency_symbols( $decode_currencies = false ) {
-	$currencies = give_get_currencies( 'symbol' );
+function give_currency_symbols($decode_currencies = false) {
+	$currencies = give_get_currencies('symbol');
 
-	if ( $decode_currencies ) {
-		array_walk( $currencies, function ( &$currency_symbol ) {
-			$currency_symbol = html_entity_decode( $currency_symbol, ENT_COMPAT, 'UTF-8' );
+	if ($decode_currencies) {
+		array_walk($currencies, function(&$currency_symbol) {
+			$currency_symbol = html_entity_decode($currency_symbol, ENT_COMPAT, 'UTF-8');
 		} );
 	}
 
@@ -922,7 +922,7 @@ 
 	 *
 	 * @param array $currencies
 	 */
-	return apply_filters( 'give_currency_symbols', $currencies );
+	return apply_filters('give_currency_symbols', $currencies);
 }
 
 
@@ -939,14 +939,14 @@ 
  *
  * @return string           The symbol to use for the currency
  */
-function give_currency_symbol( $currency = '', $decode_currency = false ) {
+function give_currency_symbol($currency = '', $decode_currency = false) {
 
-	if ( empty( $currency ) ) {
+	if (empty($currency)) {
 		$currency = give_get_currency();
 	}
 
-	$currencies = give_currency_symbols( $decode_currency );
-	$symbol     = array_key_exists( $currency, $currencies ) ? $currencies[ $currency ] : $currency;
+	$currencies = give_currency_symbols($decode_currency);
+	$symbol     = array_key_exists($currency, $currencies) ? $currencies[$currency] : $currency;
 
 	/**
 	 * Filter the currency symbol
@@ -956,7 +956,7 @@ 
 	 * @param string $symbol
 	 * @param string $currency
 	 */
-	return apply_filters( 'give_currency_symbol', $symbol, $currency );
+	return apply_filters('give_currency_symbol', $symbol, $currency);
 }
 
 
@@ -969,13 +969,13 @@ 
  *
  * @return string
  */
-function give_get_currency_name( $currency_code ) {
+function give_get_currency_name($currency_code) {
 	$currency_name  = '';
 	$currency_names = give_get_currencies();
 
-	if ( $currency_code && array_key_exists( $currency_code, $currency_names ) ) {
-		$currency_name = explode( '(', $currency_names[ $currency_code ] );
-		$currency_name = trim( current( $currency_name ) );
+	if ($currency_code && array_key_exists($currency_code, $currency_names)) {
+		$currency_name = explode('(', $currency_names[$currency_code]);
+		$currency_name = trim(current($currency_name));
 	}
 
 	/**
@@ -986,7 +986,7 @@ 
 	 * @param string $currency_name
 	 * @param string $currency_code
 	 */
-	return apply_filters( 'give_currency_name', $currency_name, $currency_code );
+	return apply_filters('give_currency_name', $currency_name, $currency_code);
 }
 
 /**
@@ -999,20 +999,20 @@ 
  *
  * @return mixed|string
  */
-function give_currency_filter( $price = '', $args = array() ) {
+function give_currency_filter($price = '', $args = array()) {
 
 	// Get functions arguments.
 	$func_args = func_get_args();
 
 	// Backward compatibility: modify second param to array
-	if ( isset( $func_args[1] ) && is_string( $func_args[1] ) ) {
+	if (isset($func_args[1]) && is_string($func_args[1])) {
 		$args = array(
-			'currency_code'   => isset( $func_args[1] ) ? $func_args[1] : '',
-			'decode_currency' => isset( $func_args[2] ) ? $func_args[2] : false,
-			'form_id'         => isset( $func_args[3] ) ? $func_args[3] : '',
+			'currency_code'   => isset($func_args[1]) ? $func_args[1] : '',
+			'decode_currency' => isset($func_args[2]) ? $func_args[2] : false,
+			'form_id'         => isset($func_args[3]) ? $func_args[3] : '',
 		);
 
-		give_doing_it_wrong( __FUNCTION__, 'Pass second argument as Array.', GIVE_VERSION );
+		give_doing_it_wrong(__FUNCTION__, 'Pass second argument as Array.', GIVE_VERSION);
 	}
 
 	// Set default values.
@@ -1025,22 +1025,22 @@ 
 		)
 	);
 
-	if ( empty( $args['currency_code'] ) || ! array_key_exists( (string) $args['currency_code'], give_get_currencies() ) ) {
-		$args['currency_code'] = give_get_currency( $args['form_id'] );
+	if (empty($args['currency_code']) || ! array_key_exists((string) $args['currency_code'], give_get_currencies())) {
+		$args['currency_code'] = give_get_currency($args['form_id']);
 	}
 
-	$args['position'] = give_get_option( 'currency_position', 'before' );
+	$args['position'] = give_get_option('currency_position', 'before');
 
 	$negative = $price < 0;
 
-	if ( $negative ) {
+	if ($negative) {
 		// Remove proceeding "-".
-		$price = substr( $price, 1 );
+		$price = substr($price, 1);
 	}
 
-	$args['symbol'] = give_currency_symbol( $args['currency_code'], $args['decode_currency'] );
+	$args['symbol'] = give_currency_symbol($args['currency_code'], $args['decode_currency']);
 
-	switch ( $args['currency_code'] ) :
+	switch ($args['currency_code']) :
 		case 'GBP' :
 		case 'BRL' :
 		case 'EUR' :
@@ -1070,13 +1070,13 @@ 
 		case 'MAD' :
 		case 'KRW' :
 		case 'ZAR' :
-			$formatted = ( 'before' === $args['position'] ? $args['symbol'] . $price : $price . $args['symbol'] );
+			$formatted = ('before' === $args['position'] ? $args['symbol'].$price : $price.$args['symbol']);
 			break;
 		case 'NOK':
-			$formatted = ( 'before' === $args['position'] ? $args['symbol'] . ' ' . $price : $price . ' ' . $args['symbol'] );
+			$formatted = ('before' === $args['position'] ? $args['symbol'].' '.$price : $price.' '.$args['symbol']);
 			break;
 		default:
-			$formatted = ( 'before' === $args['position'] ? $args['symbol'] . ' ' . $price : $price . ' ' . $args['symbol'] );
+			$formatted = ('before' === $args['position'] ? $args['symbol'].' '.$price : $price.' '.$args['symbol']);
 			break;
 	endswitch;
 
@@ -1085,7 +1085,7 @@ 
 	 *
 	 * @since 1.8.17
 	 */
-	$formatted = apply_filters( 'give_currency_filter', $formatted, $args, $price );
+	$formatted = apply_filters('give_currency_filter', $formatted, $args, $price);
 
 	/**
 	 * Filter formatted amount with currency
@@ -1099,16 +1099,16 @@ 
 	 *           filter name will be give_usd_currency_filter_after
 	 */
 	$formatted = apply_filters(
-		'give_' . strtolower( $args['currency_code'] ) . "_currency_filter_{$args['position']}",
+		'give_'.strtolower($args['currency_code'])."_currency_filter_{$args['position']}",
 		$formatted,
 		$args['currency_code'],
 		$price,
 		$args
 	);
 
-	if ( $negative ) {
+	if ($negative) {
 		// Prepend the minus sign before the currency sign.
-		$formatted = '-' . $formatted;
+		$formatted = '-'.$formatted;
 	}
 
 	return $formatted;
@@ -1125,7 +1125,7 @@ 
  *
  * @return bool
  */
-function give_is_zero_based_currency( $currency = '' ) {
+function give_is_zero_based_currency($currency = '') {
 	$zero_based_currency = array(
 		'PYG', // Paraguayan Guarani.
 		'GNF', // Guinean Franc.
@@ -1145,12 +1145,12 @@ 
 	);
 
 	// Set default currency.
-	if ( empty( $currency ) ) {
+	if (empty($currency)) {
 		$currency = give_get_currency();
 	}
 
 	// Check for Zero Based Currency.
-	if ( in_array( $currency, $zero_based_currency ) ) {
+	if (in_array($currency, $zero_based_currency)) {
 		return true;
 	}
 
@@ -1165,7 +1165,7 @@ 
  *
  * @return bool
  */
-function give_is_right_to_left_supported_currency( $currency = '' ) {
+function give_is_right_to_left_supported_currency($currency = '') {
 	$zero_based_currency = apply_filters(
 		'give_right_to_left_supported_currency',
 		array(
@@ -1181,12 +1181,12 @@ 
 	);
 
 	// Set default currency.
-	if ( empty( $currency ) ) {
+	if (empty($currency)) {
 		$currency = give_get_currency();
 	}
 
 	// Check for Zero Based Currency.
-	if ( in_array( $currency, $zero_based_currency ) ) {
+	if (in_array($currency, $zero_based_currency)) {
 		return true;
 	}
 

templates/shortcode-profile-editor.php

Braces +3 added lines, -1 removed lines

@@ -182,7 +182,9 @@
 	?>
 
 	<?php
-else:
+else {
+	:
 	_e( 'You need to login to edit your profile.', 'give' );
+}
 	echo give_login_form();
 endif;

Spacing +41 added lines, -41 removed lines

@@ -7,22 +7,22 @@ 
  * @copyright    Copyright (c) 2016, WordImpress
  * @license      https://opensource.org/licenses/gpl-license GNU Public License
  */
-$current_user     = wp_get_current_user();
+$current_user = wp_get_current_user();
 
-if ( is_user_logged_in() ):
+if (is_user_logged_in()):
 	$user_id = get_current_user_id();
-	$first_name   = get_user_meta( $user_id, 'first_name', true );
-	$last_name    = get_user_meta( $user_id, 'last_name', true );
+	$first_name   = get_user_meta($user_id, 'first_name', true);
+	$last_name    = get_user_meta($user_id, 'last_name', true);
 	$display_name = $current_user->display_name;
-	$address      = give_get_donor_address( $user_id, array( 'address_type' => 'personal' ) );
+	$address      = give_get_donor_address($user_id, array('address_type' => 'personal'));
 
-	if ( isset( $_GET['updated'] ) && $_GET['updated'] == true && ! give_get_errors() ): ?>
+	if (isset($_GET['updated']) && $_GET['updated'] == true && ! give_get_errors()): ?>
 		<p class="give_success">
-			<strong><?php esc_html_e( 'Success:', 'give' ); ?></strong> <?php esc_html_e( 'Your profile has been updated.', 'give' ); ?>
+			<strong><?php esc_html_e('Success:', 'give'); ?></strong> <?php esc_html_e('Your profile has been updated.', 'give'); ?>
 		</p>
 	<?php endif; ?>
 
-	<?php Give()->notices->render_frontend_notices( 0 ); ?>
+	<?php Give()->notices->render_frontend_notices(0); ?>
 
 	<?php
 	/**
@@ -32,42 +32,42 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_profile_editor_before' );
+	do_action('give_profile_editor_before');
 	?>
 
 	<form id="give_profile_editor_form" class="give-form" action="<?php echo give_get_current_page_url(); ?>" method="post">
 
 		<fieldset>
-			<legend id="give_profile_name_label"><?php _e( 'Profile', 'give' ); ?></legend>
+			<legend id="give_profile_name_label"><?php _e('Profile', 'give'); ?></legend>
 
-			<h3 id="give_personal_information_label" class="give-section-break"><?php _e( 'Change your Name', 'give' ); ?></h3>
+			<h3 id="give_personal_information_label" class="give-section-break"><?php _e('Change your Name', 'give'); ?></h3>
 
 			<p id="give_profile_first_name_wrap" class="form-row form-row-first form-row-responsive">
 				<label for="give_first_name">
-					<?php _e( 'First Name', 'give' ); ?>
+					<?php _e('First Name', 'give'); ?>
 					<span class="give-required-indicator  ">*</span>
 				</label>
-				<input name="give_first_name" id="give_first_name" class="text give-input" type="text" value="<?php echo esc_attr( $first_name ); ?>"/>
+				<input name="give_first_name" id="give_first_name" class="text give-input" type="text" value="<?php echo esc_attr($first_name); ?>"/>
 			</p>
 
 			<p id="give_profile_last_name_wrap" class="form-row form-row-last form-row-responsive">
-				<label for="give_last_name"><?php _e( 'Last Name', 'give' ); ?></label>
-				<input name="give_last_name" id="give_last_name" class="text give-input" type="text" value="<?php echo esc_attr( $last_name ); ?>"/>
+				<label for="give_last_name"><?php _e('Last Name', 'give'); ?></label>
+				<input name="give_last_name" id="give_last_name" class="text give-input" type="text" value="<?php echo esc_attr($last_name); ?>"/>
 			</p>
 
 			<p id="give_profile_display_name_wrap" class="form-row form-row-first form-row-responsive">
-				<label for="give_display_name"><?php _e( 'Display Name', 'give' ); ?></label>
+				<label for="give_display_name"><?php _e('Display Name', 'give'); ?></label>
 				<select name="give_display_name" id="give_display_name" class="select give-select">
-					<?php if ( ! empty( $current_user->first_name ) ): ?>
-						<option <?php selected( $display_name, $current_user->first_name ); ?> value="<?php echo esc_attr( $current_user->first_name ); ?>"><?php echo esc_html( $current_user->first_name ); ?></option>
+					<?php if ( ! empty($current_user->first_name)): ?>
+						<option <?php selected($display_name, $current_user->first_name); ?> value="<?php echo esc_attr($current_user->first_name); ?>"><?php echo esc_html($current_user->first_name); ?></option>
 					<?php endif; ?>
-					<option <?php selected( $display_name, $current_user->user_nicename ); ?> value="<?php echo esc_attr( $current_user->user_nicename ); ?>"><?php echo esc_html( $current_user->user_nicename ); ?></option>
-					<?php if ( ! empty( $current_user->last_name ) ): ?>
-						<option <?php selected( $display_name, $current_user->last_name ); ?> value="<?php echo esc_attr( $current_user->last_name ); ?>"><?php echo esc_html( $current_user->last_name ); ?></option>
+					<option <?php selected($display_name, $current_user->user_nicename); ?> value="<?php echo esc_attr($current_user->user_nicename); ?>"><?php echo esc_html($current_user->user_nicename); ?></option>
+					<?php if ( ! empty($current_user->last_name)): ?>
+						<option <?php selected($display_name, $current_user->last_name); ?> value="<?php echo esc_attr($current_user->last_name); ?>"><?php echo esc_html($current_user->last_name); ?></option>
 					<?php endif; ?>
-					<?php if ( ! empty( $current_user->first_name ) && ! empty( $current_user->last_name ) ): ?>
-						<option <?php selected( $display_name, $current_user->first_name . ' ' . $current_user->last_name ); ?> value="<?php echo esc_attr( $current_user->first_name . ' ' . $current_user->last_name ); ?>"><?php echo esc_html( $current_user->first_name . ' ' . $current_user->last_name ); ?></option>
-						<option <?php selected( $display_name, $current_user->last_name . ' ' . $current_user->first_name ); ?> value="<?php echo esc_attr( $current_user->last_name . ' ' . $current_user->first_name ); ?>"><?php echo esc_html( $current_user->last_name . ' ' . $current_user->first_name ); ?></option>
+					<?php if ( ! empty($current_user->first_name) && ! empty($current_user->last_name)): ?>
+						<option <?php selected($display_name, $current_user->first_name.' '.$current_user->last_name); ?> value="<?php echo esc_attr($current_user->first_name.' '.$current_user->last_name); ?>"><?php echo esc_html($current_user->first_name.' '.$current_user->last_name); ?></option>
+						<option <?php selected($display_name, $current_user->last_name.' '.$current_user->first_name); ?> value="<?php echo esc_attr($current_user->last_name.' '.$current_user->first_name); ?>"><?php echo esc_html($current_user->last_name.' '.$current_user->first_name); ?></option>
 					<?php endif; ?>
 				</select>
 				<?php
@@ -78,7 +78,7 @@ 
 				 *
 				 * @since 1.0
 				 */
-				do_action( 'give_profile_editor_name' );
+				do_action('give_profile_editor_name');
 				?>
 			</p>
 
@@ -90,15 +90,15 @@ 
 			 *
 			 * @since 1.0
 			 */
-			do_action( 'give_profile_editor_after_name' );
+			do_action('give_profile_editor_after_name');
 			?>
 
 			<p class="form-row form-row-last form-row-responsive">
 				<label for="give_email">
-					<?php _e( 'Email Address', 'give' ); ?>
+					<?php _e('Email Address', 'give'); ?>
 					<span class="give-required-indicator  ">*</span>
 				</label>
-				<input name="give_email" id="give_email" class="text give-input required" type="email" value="<?php echo esc_attr( $current_user->user_email ); ?>" required aria-required="true"/>
+				<input name="give_email" id="give_email" class="text give-input required" type="email" value="<?php echo esc_attr($current_user->user_email); ?>" required aria-required="true"/>
 				<?php
 				/**
 				 * Fires in the profile editor shortcode, to the email section.
@@ -107,7 +107,7 @@ 
 				 *
 				 * @since 1.0
 				 */
-				do_action( 'give_profile_editor_email' );
+				do_action('give_profile_editor_email');
 				?>
 			</p>
 
@@ -119,19 +119,19 @@ 
 			 *
 			 * @since 1.0
 			 */
-			do_action( 'give_profile_editor_after_email' );
+			do_action('give_profile_editor_after_email');
 			?>
 
-			<h3 id="give_profile_password_label" class="give-section-break"><?php _e( 'Change your Password', 'give' ); ?></h3>
+			<h3 id="give_profile_password_label" class="give-section-break"><?php _e('Change your Password', 'give'); ?></h3>
 
 			<div id="give_profile_password_wrap" class="give-clearfix">
 				<p id="give_profile_password_wrap_1" class="form-row form-row-first form-row-responsive">
-					<label for="give_new_user_pass1"><?php _e( 'New Password', 'give' ); ?></label>
+					<label for="give_new_user_pass1"><?php _e('New Password', 'give'); ?></label>
 					<input name="give_new_user_pass1" id="give_new_user_pass1" class="password give-input" type="password"/>
 				</p>
 
 				<p id="give_profile_password_wrap_2" class="form-row form-row-last form-row-responsive">
-					<label for="give_new_user_pass2"><?php _e( 'Re-enter Password', 'give' ); ?></label>
+					<label for="give_new_user_pass2"><?php _e('Re-enter Password', 'give'); ?></label>
 					<input name="give_new_user_pass2" id="give_new_user_pass2" class="password give-input" type="password"/>
 					<?php
 					/**
@@ -141,12 +141,12 @@ 
 					 *
 					 * @since 1.0
 					 */
-					do_action( 'give_profile_editor_password' );
+					do_action('give_profile_editor_password');
 					?>
 				</p>
 			</div>
 
-			<p class="give_password_change_notice"><?php _e( 'Please note after changing your password, you must log back in.', 'give' ); ?></p>
+			<p class="give_password_change_notice"><?php _e('Please note after changing your password, you must log back in.', 'give'); ?></p>
 
 			<?php
 			/**
@@ -156,14 +156,14 @@ 
 			 *
 			 * @since 1.0
 			 */
-			do_action( 'give_profile_editor_after_password' );
+			do_action('give_profile_editor_after_password');
 			?>
 
 			<p id="give_profile_submit_wrap">
-				<input type="hidden" name="give_profile_editor_nonce" value="<?php echo wp_create_nonce( 'give-profile-editor-nonce' ); ?>"/>
+				<input type="hidden" name="give_profile_editor_nonce" value="<?php echo wp_create_nonce('give-profile-editor-nonce'); ?>"/>
 				<input type="hidden" name="give_action" value="edit_user_profile"/>
-				<input type="hidden" name="give_redirect" value="<?php echo esc_url( give_get_current_page_url() ); ?>"/>
-				<input name="give_profile_editor_submit" id="give_profile_editor_submit" type="submit" class="give_submit" value="<?php _e( 'Save Changes', 'give' ); ?>"/>
+				<input type="hidden" name="give_redirect" value="<?php echo esc_url(give_get_current_page_url()); ?>"/>
+				<input name="give_profile_editor_submit" id="give_profile_editor_submit" type="submit" class="give_submit" value="<?php _e('Save Changes', 'give'); ?>"/>
 			</p>
 
 		</fieldset>
@@ -178,11 +178,11 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_profile_editor_after' );
+	do_action('give_profile_editor_after');
 	?>
 
 	<?php
 else:
-	_e( 'You need to login to edit your profile.', 'give' );
+	_e('You need to login to edit your profile.', 'give');
 	echo give_login_form();
 endif;

includes/admin/admin-actions.php

Doc Comments +3 added lines, -3 removed lines

@@ -40,7 +40,7 @@ 
  *
  * @since 1.8
  *
- * @return bool
+ * @return false|null
  */
 function give_redirect_to_clean_url_admin_pages() {
 	// Give admin pages.
@@ -992,7 +992,7 @@ 
  * @access public
  * @since  2.0
  *
- * @return bool
+ * @return false|null
  */
 function give_update_donor_name_on_user_update( $user_id = 0 ) {
 
@@ -1032,7 +1032,7 @@ 
  * @param  int          $user_id       User ID.
  * @param  WP_User|bool $old_user_data User data.
  *
- * @return bool
+ * @return false|null
  */
 function give_update_donor_email_on_user_update( $user_id = 0, $old_user_data = false ) {
 

Spacing +279 added lines, -285 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -20,19 +20,19 @@ 
  * @since 1.8
  */
 function give_load_wp_editor() {
-	if ( ! isset( $_POST['wp_editor'] ) ) {
+	if ( ! isset($_POST['wp_editor'])) {
 		die();
 	}
 
-	$wp_editor                     = json_decode( base64_decode( $_POST['wp_editor'] ), true );
+	$wp_editor                     = json_decode(base64_decode($_POST['wp_editor']), true);
 	$wp_editor[2]['textarea_name'] = $_POST['textarea_name'];
 
-	wp_editor( $wp_editor[0], $_POST['wp_editor_id'], $wp_editor[2] );
+	wp_editor($wp_editor[0], $_POST['wp_editor_id'], $wp_editor[2]);
 
 	die();
 }
 
-add_action( 'wp_ajax_give_load_wp_editor', 'give_load_wp_editor' );
+add_action('wp_ajax_give_load_wp_editor', 'give_load_wp_editor');
 
 
 /**
@@ -52,13 +52,13 @@ 
 	);
 
 	// Get current page.
-	$current_page = isset( $_GET['page'] ) ? esc_attr( $_GET['page'] ) : '';
+	$current_page = isset($_GET['page']) ? esc_attr($_GET['page']) : '';
 
 	// Bailout.
 	if (
-		empty( $current_page )
-		|| empty( $_GET['_wp_http_referer'] )
-		|| ! in_array( $current_page, $give_pages )
+		empty($current_page)
+		|| empty($_GET['_wp_http_referer'])
+		|| ! in_array($current_page, $give_pages)
 	) {
 		return false;
 	}
@@ -68,21 +68,21 @@ 
 	 *
 	 * @since 1.8
 	 */
-	$redirect = apply_filters( "give_validate_{$current_page}", true );
+	$redirect = apply_filters("give_validate_{$current_page}", true);
 
-	if ( $redirect ) {
+	if ($redirect) {
 		// Redirect.
 		wp_redirect(
 			remove_query_arg(
-				array( '_wp_http_referer', '_wpnonce' ),
-				wp_unslash( $_SERVER['REQUEST_URI'] )
+				array('_wp_http_referer', '_wpnonce'),
+				wp_unslash($_SERVER['REQUEST_URI'])
 			)
 		);
 		exit;
 	}
 }
 
-add_action( 'admin_init', 'give_redirect_to_clean_url_admin_pages' );
+add_action('admin_init', 'give_redirect_to_clean_url_admin_pages');
 
 
 /**
@@ -96,25 +96,25 @@ 
  */
 function give_hide_outdated_php_notice() {
 
-	if ( ! isset( $_POST['_give_hide_outdated_php_notices_shortly'] ) ) {
+	if ( ! isset($_POST['_give_hide_outdated_php_notices_shortly'])) {
 		give_die();
 	}
 
 	// Transient key name.
 	$transient_key = "_give_hide_outdated_php_notices_shortly";
 
-	if ( Give_Cache::get( $transient_key, true ) ) {
+	if (Give_Cache::get($transient_key, true)) {
 		return;
 	}
 
 	// Hide notice for 24 hours.
-	Give_Cache::set( $transient_key, true, DAY_IN_SECONDS, true );
+	Give_Cache::set($transient_key, true, DAY_IN_SECONDS, true);
 
 	give_die();
 
 }
 
-add_action( 'wp_ajax_give_hide_outdated_php_notice', 'give_hide_outdated_php_notice' );
+add_action('wp_ajax_give_hide_outdated_php_notice', 'give_hide_outdated_php_notice');
 
 /**
  * Register admin notices.
@@ -123,27 +123,27 @@ 
  */
 function _give_register_admin_notices() {
 	// Bailout.
-	if ( ! is_admin() ) {
+	if ( ! is_admin()) {
 		return;
 	}
 
 	// Bulk action notices.
 	if (
-		isset( $_GET['action'] ) &&
-		! empty( $_GET['action'] )
+		isset($_GET['action']) &&
+		! empty($_GET['action'])
 	) {
 
 		// Add payment bulk notice.
 		if (
-			current_user_can( 'edit_give_payments' ) &&
-			isset( $_GET['payment'] ) &&
-			! empty( $_GET['payment'] )
+			current_user_can('edit_give_payments') &&
+			isset($_GET['payment']) &&
+			! empty($_GET['payment'])
 		) {
-			$payment_count = isset( $_GET['payment'] ) ? count( $_GET['payment'] ) : 0;
+			$payment_count = isset($_GET['payment']) ? count($_GET['payment']) : 0;
 
-			switch ( $_GET['action'] ) {
+			switch ($_GET['action']) {
 				case 'delete':
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'bulk_action_delete',
 						'type'        => 'updated',
 						'description' => sprintf(
@@ -155,12 +155,12 @@ 
 							),
 							$payment_count ),
 						'show'        => true,
-					) );
+					));
 
 					break;
 
 				case 'resend-receipt':
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'bulk_action_resend_receipt',
 						'type'        => 'updated',
 						'description' => sprintf(
@@ -173,7 +173,7 @@ 
 							$payment_count
 						),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'set-status-publish' :
@@ -185,7 +185,7 @@ 
 				case 'set-status-cancelled' :
 				case 'set-status-abandoned' :
 				case 'set-status-preapproval' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'bulk_action_status_change',
 						'type'        => 'updated',
 						'description' => _n(
@@ -195,251 +195,251 @@ 
 							'give'
 						),
 						'show'        => true,
-					) );
+					));
 					break;
 			}
 		}
 	}
 
 	// Add give message notices.
-	if ( ! empty( $_GET['give-message'] ) ) {
+	if ( ! empty($_GET['give-message'])) {
 		// Donation reports errors.
-		if ( current_user_can( 'view_give_reports' ) ) {
-			switch ( $_GET['give-message'] ) {
+		if (current_user_can('view_give_reports')) {
+			switch ($_GET['give-message']) {
 				case 'donation_deleted' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donation-deleted',
 						'type'        => 'updated',
-						'description' => __( 'The donation has been deleted.', 'give' ),
+						'description' => __('The donation has been deleted.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'email_sent' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-payment-sent',
 						'type'        => 'updated',
-						'description' => __( 'The donation receipt has been resent.', 'give' ),
+						'description' => __('The donation receipt has been resent.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'refreshed-reports' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-refreshed-reports',
 						'type'        => 'updated',
-						'description' => __( 'The reports cache has been cleared.', 'give' ),
+						'description' => __('The reports cache has been cleared.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'donation-note-deleted' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donation-note-deleted',
 						'type'        => 'updated',
-						'description' => __( 'The donation note has been deleted.', 'give' ),
+						'description' => __('The donation note has been deleted.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 			}
 		}
 
 		// Give settings notices and errors.
-		if ( current_user_can( 'manage_give_settings' ) ) {
-			switch ( $_GET['give-message'] ) {
+		if (current_user_can('manage_give_settings')) {
+			switch ($_GET['give-message']) {
 				case 'settings-imported' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-settings-imported',
 						'type'        => 'updated',
-						'description' => __( 'The settings have been imported.', 'give' ),
+						'description' => __('The settings have been imported.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'api-key-generated' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-api-key-generated',
 						'type'        => 'updated',
-						'description' => __( 'API keys have been generated.', 'give' ),
+						'description' => __('API keys have been generated.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'api-key-exists' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-api-key-exists',
 						'type'        => 'updated',
-						'description' => __( 'The specified user already has API keys.', 'give' ),
+						'description' => __('The specified user already has API keys.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'api-key-regenerated' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-api-key-regenerated',
 						'type'        => 'updated',
-						'description' => __( 'API keys have been regenerated.', 'give' ),
+						'description' => __('API keys have been regenerated.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'api-key-revoked' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-api-key-revoked',
 						'type'        => 'updated',
-						'description' => __( 'API keys have been revoked.', 'give' ),
+						'description' => __('API keys have been revoked.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'sent-test-email' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-sent-test-email',
 						'type'        => 'updated',
-						'description' => __( 'The test email has been sent.', 'give' ),
+						'description' => __('The test email has been sent.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'matched-success-failure-page':
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-matched-success-failure-page',
 						'type'        => 'updated',
-						'description' => __( 'You cannot set the success and failed pages to the same page', 'give' ),
+						'description' => __('You cannot set the success and failed pages to the same page', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 			}
 		}
 		// Payments errors.
-		if ( current_user_can( 'edit_give_payments' ) ) {
-			switch ( $_GET['give-message'] ) {
+		if (current_user_can('edit_give_payments')) {
+			switch ($_GET['give-message']) {
 				case 'note-added' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-note-added',
 						'type'        => 'updated',
-						'description' => __( 'The donation note has been added.', 'give' ),
+						'description' => __('The donation note has been added.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 				case 'payment-updated' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-payment-updated',
 						'type'        => 'updated',
-						'description' => __( 'The donation has been updated.', 'give' ),
+						'description' => __('The donation has been updated.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 			}
 		}
 
 		// Donor Notices.
-		if ( current_user_can( 'edit_give_payments' ) ) {
-			switch ( $_GET['give-message'] ) {
+		if (current_user_can('edit_give_payments')) {
+			switch ($_GET['give-message']) {
 				case 'donor-deleted' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-deleted',
 						'type'        => 'updated',
-						'description' => __( 'The selected donor(s) has been deleted.', 'give' ),
+						'description' => __('The selected donor(s) has been deleted.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'donor-donations-deleted' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-donations-deleted',
 						'type'        => 'updated',
-						'description' => __( 'The selected donor(s) and its associated donations has been deleted.', 'give' ),
+						'description' => __('The selected donor(s) and its associated donations has been deleted.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'confirm-delete-donor' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-confirm-delete-donor',
 						'type'        => 'updated',
-						'description' => __( 'You must confirm to delete the selected donor(s).', 'give' ),
+						'description' => __('You must confirm to delete the selected donor(s).', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'invalid-donor-id' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-invalid-donor-id',
 						'type'        => 'updated',
-						'description' => __( 'Invalid Donor ID.', 'give' ),
+						'description' => __('Invalid Donor ID.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'donor-delete-failed' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-delete-failed',
 						'type'        => 'error',
-						'description' => __( 'Unable to delete selected donor(s).', 'give' ),
+						'description' => __('Unable to delete selected donor(s).', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'email-added' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-email-added',
 						'type'        => 'updated',
-						'description' => __( 'Donor email added.', 'give' ),
+						'description' => __('Donor email added.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'email-removed' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-email-removed',
 						'type'        => 'updated',
-						'description' => __( 'Donor email removed.', 'give' ),
+						'description' => __('Donor email removed.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'email-remove-failed' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-email-remove-failed',
 						'type'        => 'updated',
-						'description' => __( 'Failed to remove donor email.', 'give' ),
+						'description' => __('Failed to remove donor email.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'primary-email-updated' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-primary-email-updated',
 						'type'        => 'updated',
-						'description' => __( 'Primary email updated for donor.', 'give' ),
+						'description' => __('Primary email updated for donor.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'primary-email-failed' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-primary-email-failed',
 						'type'        => 'updated',
-						'description' => __( 'Failed to set primary email.', 'give' ),
+						'description' => __('Failed to set primary email.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'reconnect-user' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-reconnect-user',
 						'type'        => 'updated',
-						'description' => __( 'User has been successfully connected with Donor.', 'give' ),
+						'description' => __('User has been successfully connected with Donor.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 
 				case 'profile-updated' :
-					Give()->notices->register_notice( array(
+					Give()->notices->register_notice(array(
 						'id'          => 'give-donor-profile-updated',
 						'type'        => 'updated',
-						'description' => __( 'Donor information updated successfully.', 'give' ),
+						'description' => __('Donor information updated successfully.', 'give'),
 						'show'        => true,
-					) );
+					));
 					break;
 			}
 		}
 	}
 }
 
-add_action( 'admin_notices', '_give_register_admin_notices', - 1 );
+add_action('admin_notices', '_give_register_admin_notices', - 1);
 
 
 /**
@@ -449,31 +449,30 @@ 
  *
  * @return bool
  */
-function _give_show_test_mode_notice_in_admin_bar( $wp_admin_bar ) {
-	$is_test_mode = ! empty( $_POST['test_mode'] ) ?
-		give_is_setting_enabled( $_POST['test_mode'] ) :
-		give_is_test_mode();
+function _give_show_test_mode_notice_in_admin_bar($wp_admin_bar) {
+	$is_test_mode = ! empty($_POST['test_mode']) ?
+		give_is_setting_enabled($_POST['test_mode']) : give_is_test_mode();
 
 	if (
-		! current_user_can( 'view_give_reports' ) ||
+		! current_user_can('view_give_reports') ||
 		! $is_test_mode
 	) {
 		return false;
 	}
 
 	// Add the main site admin menu item.
-	$wp_admin_bar->add_menu( array(
+	$wp_admin_bar->add_menu(array(
 		'id'     => 'give-test-notice',
-		'href'   => admin_url( 'edit.php?post_type=give_forms&page=give-settings&tab=gateways' ),
+		'href'   => admin_url('edit.php?post_type=give_forms&page=give-settings&tab=gateways'),
 		'parent' => 'top-secondary',
-		'title'  => __( 'Give Test Mode Active', 'give' ),
-		'meta'   => array( 'class' => 'give-test-mode-active' ),
-	) );
+		'title'  => __('Give Test Mode Active', 'give'),
+		'meta'   => array('class' => 'give-test-mode-active'),
+	));
 
 	return true;
 }
 
-add_action( 'admin_bar_menu', '_give_show_test_mode_notice_in_admin_bar', 1000, 1 );
+add_action('admin_bar_menu', '_give_show_test_mode_notice_in_admin_bar', 1000, 1);
 
 /**
  * Add Link to Import page in from donation archive and donation single page
@@ -482,12 +481,12 @@ 
  */
 function give_import_page_link_callback() {
 	?>
-	<a href="<?php echo esc_url( give_import_page_url() ); ?>"
-	   class="page-import-action page-title-action"><?php _e( 'Import Donations', 'give' ); ?></a>
+	<a href="<?php echo esc_url(give_import_page_url()); ?>"
+	   class="page-import-action page-title-action"><?php _e('Import Donations', 'give'); ?></a>
 
 	<?php
 	// Check if view donation single page only.
-	if ( ! empty( $_REQUEST['view'] ) && 'view-payment-details' === (string) give_clean( $_REQUEST['view'] ) && 'give-payment-history' === give_clean( $_REQUEST['page'] ) ) {
+	if ( ! empty($_REQUEST['view']) && 'view-payment-details' === (string) give_clean($_REQUEST['view']) && 'give-payment-history' === give_clean($_REQUEST['page'])) {
 		?>
 		<style type="text/css">
 			.wrap #transaction-details-heading {
@@ -498,7 +497,7 @@ 
 	}
 }
 
-add_action( 'give_payments_page_top', 'give_import_page_link_callback', 11 );
+add_action('give_payments_page_top', 'give_import_page_link_callback', 11);
 
 /**
  * Load donation import ajax callback
@@ -510,9 +509,9 @@ 
  */
 function give_donation_import_callback() {
 	$import_setting = array();
-	$fields         = isset( $_POST['fields'] ) ? $_POST['fields'] : null;
+	$fields         = isset($_POST['fields']) ? $_POST['fields'] : null;
 
-	parse_str( $fields );
+	parse_str($fields);
 
 	$import_setting['create_user'] = $create_user;
 	$import_setting['mode']        = $mode;
@@ -521,57 +520,57 @@ 
 	$import_setting['delete_csv']  = $delete_csv;
 
 	// Parent key id.
-	$main_key = maybe_unserialize( $main_key );
-
-	$current    = absint( $_REQUEST['current'] );
-	$total_ajax = absint( $_REQUEST['total_ajax'] );
-	$start      = absint( $_REQUEST['start'] );
-	$end        = absint( $_REQUEST['end'] );
-	$next       = absint( $_REQUEST['next'] );
-	$total      = absint( $_REQUEST['total'] );
-	$per_page   = absint( $_REQUEST['per_page'] );
-	if ( empty( $delimiter ) ) {
+	$main_key = maybe_unserialize($main_key);
+
+	$current    = absint($_REQUEST['current']);
+	$total_ajax = absint($_REQUEST['total_ajax']);
+	$start      = absint($_REQUEST['start']);
+	$end        = absint($_REQUEST['end']);
+	$next       = absint($_REQUEST['next']);
+	$total      = absint($_REQUEST['total']);
+	$per_page   = absint($_REQUEST['per_page']);
+	if (empty($delimiter)) {
 		$delimiter = ',';
 	}
 
 	// Processing done here.
-	$raw_data = give_get_donation_data_from_csv( $csv, $start, $end, $delimiter );
-	$raw_key  = maybe_unserialize( $mapto );
+	$raw_data = give_get_donation_data_from_csv($csv, $start, $end, $delimiter);
+	$raw_key  = maybe_unserialize($mapto);
 
 	// Prevent normal emails.
-	remove_action( 'give_complete_donation', 'give_trigger_donation_receipt', 999 );
-	remove_action( 'give_insert_user', 'give_new_user_notification', 10 );
-	remove_action( 'give_insert_payment', 'give_payment_save_page_data' );
-
-	$current_key  = $start;
-	foreach ( $raw_data as $row_data ) {
-		$import_setting['donation_key']  = $current_key;
-		give_save_import_donation_to_db( $raw_key, $row_data, $main_key, $import_setting );
+	remove_action('give_complete_donation', 'give_trigger_donation_receipt', 999);
+	remove_action('give_insert_user', 'give_new_user_notification', 10);
+	remove_action('give_insert_payment', 'give_payment_save_page_data');
+
+	$current_key = $start;
+	foreach ($raw_data as $row_data) {
+		$import_setting['donation_key'] = $current_key;
+		give_save_import_donation_to_db($raw_key, $row_data, $main_key, $import_setting);
 		$current_key++;
 	}
 
 	// Check if function exists or not.
-	if ( function_exists( 'give_payment_save_page_data' ) ) {
-		add_action( 'give_insert_payment', 'give_payment_save_page_data' );
+	if (function_exists('give_payment_save_page_data')) {
+		add_action('give_insert_payment', 'give_payment_save_page_data');
 	}
-	add_action( 'give_insert_user', 'give_new_user_notification', 10, 2 );
-	add_action( 'give_complete_donation', 'give_trigger_donation_receipt', 999 );
+	add_action('give_insert_user', 'give_new_user_notification', 10, 2);
+	add_action('give_complete_donation', 'give_trigger_donation_receipt', 999);
 
-	if ( $next == false ) {
+	if ($next == false) {
 		$json_data = array(
 			'success' => true,
-			'message' => __( 'All donation uploaded successfully!', 'give' ),
+			'message' => __('All donation uploaded successfully!', 'give'),
 		);
 	} else {
 		$index_start = $start;
 		$index_end   = $end;
 		$last        = false;
 		$next        = true;
-		if ( $next ) {
+		if ($next) {
 			$index_start = $index_start + $per_page;
-			$index_end   = $per_page + ( $index_start - 1 );
+			$index_end   = $per_page + ($index_start - 1);
 		}
-		if ( $index_end >= $total ) {
+		if ($index_end >= $total) {
 			$index_end = $total;
 			$last      = true;
 		}
@@ -585,27 +584,27 @@ 
 		);
 	}
 
-	$url              = give_import_page_url( array(
+	$url = give_import_page_url(array(
 		'step'          => '4',
 		'importer-type' => 'import_donations',
 		'csv'           => $csv,
 		'total'         => $total,
 		'delete_csv'    => $import_setting['delete_csv'],
-		'success'       => ( isset( $json_data['success'] ) ? $json_data['success'] : '' ),
-	) );
+		'success'       => (isset($json_data['success']) ? $json_data['success'] : ''),
+	));
 	$json_data['url'] = $url;
 
-	$current ++;
+	$current++;
 	$json_data['current'] = $current;
 
-	$percentage              = ( 100 / ( $total_ajax + 1 ) ) * $current;
+	$percentage              = (100 / ($total_ajax + 1)) * $current;
 	$json_data['percentage'] = $percentage;
 
-	$json_data = apply_filters( 'give_import_ajax_responces', $json_data, $fields );
-	wp_die( json_encode( $json_data ) );
+	$json_data = apply_filters('give_import_ajax_responces', $json_data, $fields);
+	wp_die(json_encode($json_data));
 }
 
-add_action( 'wp_ajax_give_donation_import', 'give_donation_import_callback' );
+add_action('wp_ajax_give_donation_import', 'give_donation_import_callback');
 
 /**
  * Load core settings import ajax callback
@@ -617,8 +616,8 @@ 
  */
 
 function give_core_settings_import_callback() {
-	$fields = isset( $_POST['fields'] ) ? $_POST['fields'] : null;
-	parse_str( $fields, $fields );
+	$fields = isset($_POST['fields']) ? $_POST['fields'] : null;
+	parse_str($fields, $fields);
 
 	$json_data['success'] = false;
 
@@ -633,22 +632,22 @@ 
 	 *
 	 * @return array $fields
 	 */
-	$fields = (array) apply_filters( 'give_import_core_settings_fields', $fields );
+	$fields = (array) apply_filters('give_import_core_settings_fields', $fields);
 
-	$file_name = ( ! empty( $fields['file_name'] ) ? give_clean( $fields['file_name'] ) : false );
+	$file_name = ( ! empty($fields['file_name']) ? give_clean($fields['file_name']) : false);
 
-	if ( ! empty( $file_name ) ) {
-		$type = ( ! empty( $fields['type'] ) ? give_clean( $fields['type'] ) : 'merge' );
+	if ( ! empty($file_name)) {
+		$type = ( ! empty($fields['type']) ? give_clean($fields['type']) : 'merge');
 
 		// Get the json data from the file and then alter it in array format
-		$json_string   = give_get_core_settings_json( $file_name );
-		$json_to_array = json_decode( $json_string, true );
+		$json_string   = give_get_core_settings_json($file_name);
+		$json_to_array = json_decode($json_string, true);
 
 		// get the current settign from the options table.
-		$host_give_options = get_option( 'give_settings', array() );
+		$host_give_options = get_option('give_settings', array());
 
 		// Save old settins for backup.
-		update_option( 'give_settings_old', $host_give_options );
+		update_option('give_settings_old', $host_give_options);
 
 		/**
 		 * Filter to Modify Core Settings that are being going to get import in options table as give settings.
@@ -664,9 +663,9 @@ 
 		 *
 		 * @return array $json_to_array Setting that are being going to get imported
 		 */
-		$json_to_array = (array) apply_filters( 'give_import_core_settings_data', $json_to_array, $type, $host_give_options, $fields );
+		$json_to_array = (array) apply_filters('give_import_core_settings_data', $json_to_array, $type, $host_give_options, $fields);
 
-		update_option( 'give_settings', $json_to_array );
+		update_option('give_settings', $json_to_array);
 
 		$json_data['success'] = true;
 	}
@@ -682,16 +681,16 @@ 
 	 *
 	 * @return array $url
 	 */
-	$json_data['url'] = give_import_page_url( (array) apply_filters( 'give_import_core_settings_success_url', array(
-		'step'          => ( empty( $json_data['success'] ) ? '1' : '3' ),
+	$json_data['url'] = give_import_page_url((array) apply_filters('give_import_core_settings_success_url', array(
+		'step'          => (empty($json_data['success']) ? '1' : '3'),
 		'importer-type' => 'import_core_setting',
-		'success'       => ( empty( $json_data['success'] ) ? '0' : '1' ),
-	) ) );
+		'success'       => (empty($json_data['success']) ? '0' : '1'),
+	)));
 
-	wp_send_json( $json_data );
+	wp_send_json($json_data);
 }
 
-add_action( 'wp_ajax_give_core_settings_import', 'give_core_settings_import_callback' );
+add_action('wp_ajax_give_core_settings_import', 'give_core_settings_import_callback');
 
 /**
  * Initializes blank slate content if a list table is empty.
@@ -703,7 +702,7 @@ 
 	$blank_slate->init();
 }
 
-add_action( 'current_screen', 'give_blank_slate' );
+add_action('current_screen', 'give_blank_slate');
 
 /**
  * Validate Fields of User Profile
@@ -716,24 +715,24 @@ 
  *
  * @return mixed
  */
-function give_validate_user_profile( $errors, $update, $user ) {
+function give_validate_user_profile($errors, $update, $user) {
 
-	if ( ! empty( $_POST['action'] ) && ( 'adduser' === $_POST['action'] || 'createuser' === $_POST['action'] ) ) {
+	if ( ! empty($_POST['action']) && ('adduser' === $_POST['action'] || 'createuser' === $_POST['action'])) {
 		return;
 	}
 
-	if ( ! empty( $user->ID ) ) {
-		$donor = Give()->donors->get_donor_by( 'user_id', $user->ID );
+	if ( ! empty($user->ID)) {
+		$donor = Give()->donors->get_donor_by('user_id', $user->ID);
 
-		if ( $donor ) {
+		if ($donor) {
 			// If Donor is attached with User, then validate first name.
-			if ( empty( $_POST['first_name'] ) ) {
+			if (empty($_POST['first_name'])) {
 				$errors->add(
 					'empty_first_name',
 					sprintf(
 						'<strong>%1$s:</strong> %2$s',
-						__( 'ERROR', 'give' ),
-						__( 'Please enter your first name.', 'give' )
+						__('ERROR', 'give'),
+						__('Please enter your first name.', 'give')
 					)
 				);
 			}
@@ -742,7 +741,7 @@ 
 
 }
 
-add_action( 'user_profile_update_errors', 'give_validate_user_profile', 10, 3 );
+add_action('user_profile_update_errors', 'give_validate_user_profile', 10, 3);
 
 /**
  * Show Donor Information on User Profile Page.
@@ -751,19 +750,19 @@ 
  *
  * @since 2.0
  */
-function give_donor_information_profile_fields( $user ) {
-	$donor = Give()->donors->get_donor_by( 'user_id', $user->ID );
+function give_donor_information_profile_fields($user) {
+	$donor = Give()->donors->get_donor_by('user_id', $user->ID);
 
 	// Display Donor Information, only if donor is attached with User.
-	if ( ! empty( $donor->user_id ) ) {
+	if ( ! empty($donor->user_id)) {
 		?>
 		<table class="form-table">
 			<tbody>
 			<tr>
-				<th scope="row"><?php _e( 'Donor', 'give' ); ?></th>
+				<th scope="row"><?php _e('Donor', 'give'); ?></th>
 				<td>
-					<a href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $donor->id ); ?>">
-						<?php _e( 'View Donor Information', 'give' ); ?>
+					<a href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$donor->id); ?>">
+						<?php _e('View Donor Information', 'give'); ?>
 					</a>
 				</td>
 			</tr>
@@ -773,7 +772,7 @@ 
 	}
 }
 
-add_action( 'personal_options', 'give_donor_information_profile_fields' );
+add_action('personal_options', 'give_donor_information_profile_fields');
 /**
  * Get Array of WP User Roles.
  *
@@ -785,8 +784,8 @@ 
 	$user_roles = array();
 
 	// Loop through User Roles.
-	foreach ( get_editable_roles() as $role_name => $role_info ):
-		$user_roles[ $role_name ] = $role_info['name'];
+	foreach (get_editable_roles() as $role_name => $role_info):
+		$user_roles[$role_name] = $role_info['name'];
 	endforeach;
 
 	return $user_roles;
@@ -803,44 +802,42 @@ 
 function __give_ajax_donor_manage_addresses() {
 	// Bailout.
 	if (
-		empty( $_POST['form'] ) ||
-		empty( $_POST['donorID'] )
+		empty($_POST['form']) ||
+		empty($_POST['donorID'])
 	) {
-		wp_send_json_error( array( 'error' => 1 ) );
+		wp_send_json_error(array('error' => 1));
 	}
 
-	$post                  = give_clean( wp_parse_args( $_POST ) );
-	$donorID               = absint( $post['donorID'] );
-	$form_data             = give_clean( wp_parse_args( $post['form'] ) );
-	$is_multi_address_type = ( 'billing' === $form_data['address-id'] || false !== strpos( $form_data['address-id'], '_' ) );
-	$address_type          = false !== strpos( $form_data['address-id'], '_' ) ?
-		array_shift( explode( '_', $form_data['address-id'] ) ) :
-		$form_data['address-id'];
-	$address_id            = false !== strpos( $form_data['address-id'], '_' ) ?
-		array_pop( explode( '_', $form_data['address-id'] ) ) :
-		null;
+	$post                  = give_clean(wp_parse_args($_POST));
+	$donorID               = absint($post['donorID']);
+	$form_data             = give_clean(wp_parse_args($post['form']));
+	$is_multi_address_type = ('billing' === $form_data['address-id'] || false !== strpos($form_data['address-id'], '_'));
+	$address_type          = false !== strpos($form_data['address-id'], '_') ?
+		array_shift(explode('_', $form_data['address-id'])) : $form_data['address-id'];
+	$address_id            = false !== strpos($form_data['address-id'], '_') ?
+		array_pop(explode('_', $form_data['address-id'])) : null;
 	$response_data         = array(
 		'action' => $form_data['address-action'],
 		'id'     => $form_data['address-id'],
 	);
 
 	// Security check.
-	if ( ! wp_verify_nonce( $form_data['_wpnonce'], 'give-manage-donor-addresses' ) ) {
-		wp_send_json_error( array(
+	if ( ! wp_verify_nonce($form_data['_wpnonce'], 'give-manage-donor-addresses')) {
+		wp_send_json_error(array(
 				'error'     => 1,
 				'error_msg' => wp_sprintf(
 					'<div class="notice notice-error"><p>%s</p></div>',
-					__( 'Error: Security issue.', 'give' )
+					__('Error: Security issue.', 'give')
 				)
 			)
 		);
 	}
 
-	$donor = new Give_Donor( $donorID );
+	$donor = new Give_Donor($donorID);
 
 	// Verify donor.
-	if ( ! $donor->id ) {
-		wp_send_json_error( array( 'error' => 3 ) );
+	if ( ! $donor->id) {
+		wp_send_json_error(array('error' => 3));
 	}
 
 	// Unset all data except address.
@@ -851,57 +848,55 @@ 
 	);
 
 	// Process action.
-	switch ( $response_data['action'] ) {
+	switch ($response_data['action']) {
 
 		case 'add':
-			if ( ! $donor->add_address( "{$address_type}[]", $form_data ) ) {
-				wp_send_json_error( array(
+			if ( ! $donor->add_address("{$address_type}[]", $form_data)) {
+				wp_send_json_error(array(
 						'error'     => 1,
 						'error_msg' => wp_sprintf(
 							'<div class="notice notice-error"><p>%s</p></div>',
-							__( 'Error: Unable to save the address. Please check if address already exist.', 'give' )
+							__('Error: Unable to save the address. Please check if address already exist.', 'give')
 						)
 					)
 				);
 			}
 
-			$total_addresses = count( $donor->address[ $address_type ] );
+			$total_addresses = count($donor->address[$address_type]);
 
 			$address_index = $is_multi_address_type ?
-				$total_addresses - 1 :
-				$address_type;
+				$total_addresses - 1 : $address_type;
 
 			$address_id = $is_multi_address_type ?
-				end( array_keys( $donor->address[ $address_type ] ) ) :
-				$address_type;
+				end(array_keys($donor->address[$address_type])) : $address_type;
 
 			$response_data['address_html'] = __give_get_format_address(
-				end( $donor->address['billing'] ),
+				end($donor->address['billing']),
 				array(
 					// We can add only billing address from donor screen.
 					'type'  => 'billing',
 					'id'    => $address_id,
-					'index' => ++ $address_index,
+					'index' => ++$address_index,
 				)
 			);
-			$response_data['success_msg']  = wp_sprintf(
+			$response_data['success_msg'] = wp_sprintf(
 				'<div class="notice updated"><p>%s</p></div>',
-				__( 'Successfully added a new address to the donor.', 'give' )
+				__('Successfully added a new address to the donor.', 'give')
 			);
 
-			if ( $is_multi_address_type ) {
+			if ($is_multi_address_type) {
 				$response_data['id'] = "{$response_data['id']}_{$address_index}";
 			}
 
 			break;
 
 		case 'remove':
-			if ( ! $donor->remove_address( $response_data['id'] ) ) {
-				wp_send_json_error( array(
+			if ( ! $donor->remove_address($response_data['id'])) {
+				wp_send_json_error(array(
 						'error'     => 2,
 						'error_msg' => wp_sprintf(
 							'<div class="notice notice-error"><p>%s</p></div>',
-							__( 'Error: Unable to delete address.', 'give' )
+							__('Error: Unable to delete address.', 'give')
 						)
 					)
 				);
@@ -909,18 +904,18 @@ 
 
 			$response_data['success_msg'] = wp_sprintf(
 				'<div class="notice updated"><p>%s</p></div>',
-				__( 'Successfully removed a address of donor.', 'give' )
+				__('Successfully removed a address of donor.', 'give')
 			);
 
 			break;
 
 		case 'update':
-			if ( ! $donor->update_address( $response_data['id'], $form_data ) ) {
-				wp_send_json_error( array(
+			if ( ! $donor->update_address($response_data['id'], $form_data)) {
+				wp_send_json_error(array(
 						'error'     => 3,
 						'error_msg' => wp_sprintf(
 							'<div class="notice notice-error"><p>%s</p></div>',
-							__( 'Error: Unable to update address. Please check if address already exist.', 'give' )
+							__('Error: Unable to update address. Please check if address already exist.', 'give')
 						)
 					)
 				);
@@ -928,26 +923,25 @@ 
 
 			$response_data['address_html'] = __give_get_format_address(
 				$is_multi_address_type ?
-					$donor->address[ $address_type ][ $address_id ] :
-					$donor->address[ $address_type ],
+					$donor->address[$address_type][$address_id] : $donor->address[$address_type],
 				array(
 					'type'  => $address_type,
 					'id'    => $address_id,
 					'index' => $address_id,
 				)
 			);
-			$response_data['success_msg']  = wp_sprintf(
+			$response_data['success_msg'] = wp_sprintf(
 				'<div class="notice updated"><p>%s</p></div>',
-				__( 'Successfully updated a address of donor', 'give' )
+				__('Successfully updated a address of donor', 'give')
 			);
 
 			break;
 	}
 
-	wp_send_json_success( $response_data );
+	wp_send_json_success($response_data);
 }
 
-add_action( 'wp_ajax_donor_manage_addresses', '__give_ajax_donor_manage_addresses' );
+add_action('wp_ajax_donor_manage_addresses', '__give_ajax_donor_manage_addresses');
 
 /**
  * Admin donor billing address label
@@ -958,13 +952,13 @@ 
  *
  * @return string
  */
-function __give_donor_billing_address_label( $address_label ) {
-	$address_label = __( 'Billing Address', 'give' );
+function __give_donor_billing_address_label($address_label) {
+	$address_label = __('Billing Address', 'give');
 
 	return $address_label;
 }
 
-add_action( 'give_donor_billing_address_label', '__give_donor_billing_address_label' );
+add_action('give_donor_billing_address_label', '__give_donor_billing_address_label');
 
 /**
  * Admin donor personal address label
@@ -975,13 +969,13 @@ 
  *
  * @return string
  */
-function __give_donor_personal_address_label( $address_label ) {
-	$address_label = __( 'Personal Address', 'give' );
+function __give_donor_personal_address_label($address_label) {
+	$address_label = __('Personal Address', 'give');
 
 	return $address_label;
 }
 
-add_action( 'give_donor_personal_address_label', '__give_donor_personal_address_label' );
+add_action('give_donor_personal_address_label', '__give_donor_personal_address_label');
 
 /**
  * Update Donor Information when User Profile is updated from admin.
@@ -994,32 +988,32 @@ 
  *
  * @return bool
  */
-function give_update_donor_name_on_user_update( $user_id = 0 ) {
+function give_update_donor_name_on_user_update($user_id = 0) {
 
-	if ( current_user_can( 'edit_user', $user_id ) ) {
+	if (current_user_can('edit_user', $user_id)) {
 
-		$donor = new Give_Donor( $user_id, true );
+		$donor = new Give_Donor($user_id, true);
 
 		// Bailout, if donor doesn't exists.
-		if ( ! $donor ) {
+		if ( ! $donor) {
 			return false;
 		}
 
 		// Get User First name and Last name.
-		$first_name = ( $_POST['first_name'] ) ? give_clean( $_POST['first_name'] ) : get_user_meta( $user_id, 'first_name', true );
-		$last_name  = ( $_POST['last_name'] ) ? give_clean( $_POST['last_name'] ) : get_user_meta( $user_id, 'last_name', true );
-		$full_name  = strip_tags( wp_unslash( trim( "{$first_name} {$last_name}" ) ) );
+		$first_name = ($_POST['first_name']) ? give_clean($_POST['first_name']) : get_user_meta($user_id, 'first_name', true);
+		$last_name  = ($_POST['last_name']) ? give_clean($_POST['last_name']) : get_user_meta($user_id, 'last_name', true);
+		$full_name  = strip_tags(wp_unslash(trim("{$first_name} {$last_name}")));
 
 		// Assign User First name and Last name to Donor.
-		Give()->donors->update( $donor->id, array( 'name' => $full_name ) );
-		Give()->donor_meta->update_meta( $donor->id, '_give_donor_first_name', $first_name );
-		Give()->donor_meta->update_meta( $donor->id, '_give_donor_last_name', $last_name );
+		Give()->donors->update($donor->id, array('name' => $full_name));
+		Give()->donor_meta->update_meta($donor->id, '_give_donor_first_name', $first_name);
+		Give()->donor_meta->update_meta($donor->id, '_give_donor_last_name', $last_name);
 
 	}
 }
 
-add_action( 'edit_user_profile_update', 'give_update_donor_name_on_user_update', 10 );
-add_action( 'personal_options_update', 'give_update_donor_name_on_user_update', 10 );
+add_action('edit_user_profile_update', 'give_update_donor_name_on_user_update', 10);
+add_action('personal_options_update', 'give_update_donor_name_on_user_update', 10);
 
 
 /**
@@ -1034,29 +1028,29 @@ 
  *
  * @return bool
  */
-function give_update_donor_email_on_user_update( $user_id = 0, $old_user_data = false ) {
+function give_update_donor_email_on_user_update($user_id = 0, $old_user_data = false) {
 
-	$donor = new Give_Donor( $user_id, true );
+	$donor = new Give_Donor($user_id, true);
 
-	if ( ! $donor ) {
+	if ( ! $donor) {
 		return false;
 	}
 
-	$user = get_userdata( $user_id );
+	$user = get_userdata($user_id);
 
-	if ( ! empty( $user ) && $user->user_email !== $donor->email ) {
+	if ( ! empty($user) && $user->user_email !== $donor->email) {
 
-		$success = Give()->donors->update( $donor->id, array( 'email' => $user->user_email ) );
+		$success = Give()->donors->update($donor->id, array('email' => $user->user_email));
 
-		if ( $success ) {
+		if ($success) {
 			// Update some payment meta if we need to
-			$payments_array = explode( ',', $donor->payment_ids );
+			$payments_array = explode(',', $donor->payment_ids);
 
-			if ( ! empty( $payments_array ) ) {
+			if ( ! empty($payments_array)) {
 
-				foreach ( $payments_array as $payment_id ) {
+				foreach ($payments_array as $payment_id) {
 
-					give_update_payment_meta( $payment_id, 'email', $user->user_email );
+					give_update_payment_meta($payment_id, 'email', $user->user_email);
 
 				}
 
@@ -1070,7 +1064,7 @@ 
 			 * @param  WP_User    $user  WordPress User object.
 			 * @param  Give_Donor $donor Give donor object.
 			 */
-			do_action( 'give_update_donor_email_on_user_update', $user, $donor );
+			do_action('give_update_donor_email_on_user_update', $user, $donor);
 
 		}
 
@@ -1078,4 +1072,4 @@ 
 
 }
 
-add_action( 'profile_update', 'give_update_donor_email_on_user_update', 10, 2 );
\ No newline at end of file
+add_action('profile_update', 'give_update_donor_email_on_user_update', 10, 2);
\ No newline at end of file

includes/admin/donors/donor-functions.php

Doc Comments +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
  * Connect and Reconnect Donor with User profile.
  * @todo $address is unnecessary param because we are store address to user.
  *
- * @param object $donor      Donor Object.
+ * @param Give_Donor $donor      Donor Object.
  * @param array  $donor_data Donor Post Variables.
  * @param array  $address    Address Information.
  *

Spacing +24 added lines, -24 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,7 +23,7 @@ 
  *
  * @return array        The altered list of views.
  */
-function give_register_default_donor_views( $views ) {
+function give_register_default_donor_views($views) {
 
 	$default_views = array(
 		'overview' => 'give_donor_view',
@@ -31,11 +31,11 @@ 
 		'notes'    => 'give_donor_notes_view',
 	);
 
-	return array_merge( $views, $default_views );
+	return array_merge($views, $default_views);
 
 }
 
-add_filter( 'give_donor_views', 'give_register_default_donor_views', 1, 1 );
+add_filter('give_donor_views', 'give_register_default_donor_views', 1, 1);
 
 /**
  * Register a tab for the single donor view.
@@ -46,23 +46,23 @@ 
  *
  * @return array       The altered list of tabs
  */
-function give_register_default_donor_tabs( $tabs ) {
+function give_register_default_donor_tabs($tabs) {
 
 	$default_tabs = array(
 		'overview' => array(
 			'dashicon' => 'dashicons-admin-users',
-			'title' => __( 'Donor Profile', 'give' ),
+			'title' => __('Donor Profile', 'give'),
 		),
 		'notes'    => array(
 			'dashicon' => 'dashicons-admin-comments',
-			'title' => __( 'Donor Notes', 'give' ),
+			'title' => __('Donor Notes', 'give'),
 		),
 	);
 
-	return array_merge( $tabs, $default_tabs );
+	return array_merge($tabs, $default_tabs);
 }
 
-add_filter( 'give_donor_tabs', 'give_register_default_donor_tabs', 1, 1 );
+add_filter('give_donor_tabs', 'give_register_default_donor_tabs', 1, 1);
 
 /**
  * Register the Delete icon as late as possible so it's at the bottom.
@@ -73,17 +73,17 @@ 
  *
  * @return array       The altered list of tabs, with 'delete' at the bottom.
  */
-function give_register_delete_donor_tab( $tabs ) {
+function give_register_delete_donor_tab($tabs) {
 
 	$tabs['delete'] = array(
 		'dashicon' => 'dashicons-trash',
-		'title'    => __( 'Delete Donor', 'give' ),
+		'title'    => __('Delete Donor', 'give'),
 	);
 
 	return $tabs;
 }
 
-add_filter( 'give_donor_tabs', 'give_register_delete_donor_tab', PHP_INT_MAX, 1 );
+add_filter('give_donor_tabs', 'give_register_delete_donor_tab', PHP_INT_MAX, 1);
 
 /**
  * Connect and Reconnect Donor with User profile.
@@ -97,7 +97,7 @@ 
  *
  * @return array
  */
-function give_connect_user_donor_profile( $donor, $donor_data, $address ) {
+function give_connect_user_donor_profile($donor, $donor_data, $address) {
 
 	$donor_id         = $donor->id;
 	$previous_user_id = $donor->user_id;
@@ -111,31 +111,31 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_pre_edit_donor', $donor_id, $donor_data, $address );
+	do_action('give_pre_edit_donor', $donor_id, $donor_data, $address);
 
 	$output = array();
 
-	if ( $donor->update( $donor_data ) ) {
+	if ($donor->update($donor_data)) {
 
 		// Create and Update Donor First Name and Last Name in Meta Fields.
-		$donor->update_meta( '_give_donor_first_name', $donor_data['first_name'] );
-		$donor->update_meta( '_give_donor_last_name', $donor_data['last_name'] );
+		$donor->update_meta('_give_donor_first_name', $donor_data['first_name']);
+		$donor->update_meta('_give_donor_last_name', $donor_data['last_name']);
 
 		// Fetch disconnected user id, if exists.
-		$disconnected_user_id = $donor->get_meta( '_give_disconnected_user_id', true );
+		$disconnected_user_id = $donor->get_meta('_give_disconnected_user_id', true);
 
 		// Flag User and Donor Disconnection.
-		delete_user_meta( $disconnected_user_id, '_give_is_donor_disconnected' );
+		delete_user_meta($disconnected_user_id, '_give_is_donor_disconnected');
 
 		// Check whether the disconnected user id and the reconnected user id are same or not.
 		// If both are same then delete user id store in donor meta.
-		if( $donor_data['user_id'] === $disconnected_user_id ) {
-			delete_user_meta( $disconnected_user_id, '_give_disconnected_donor_id' );
-			$donor->delete_meta( '_give_disconnected_user_id' );
+		if ($donor_data['user_id'] === $disconnected_user_id) {
+			delete_user_meta($disconnected_user_id, '_give_disconnected_donor_id');
+			$donor->delete_meta('_give_disconnected_user_id');
 		}
 
 		$output['success']       = true;
-		$donor_data              = array_merge( $donor_data, $address );
+		$donor_data              = array_merge($donor_data, $address);
 		$output['customer_info'] = $donor_data;
 
 	} else {
@@ -152,7 +152,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_post_edit_donor', $donor_id, $donor_data );
+	do_action('give_post_edit_donor', $donor_id, $donor_data);
 
 
 	return $output;

includes/admin/emails/abstract-email-notification.php

Doc Comments +1 added lines, -1 removed lines

@@ -600,7 +600,7 @@
 		 *
 		 * @param int $form_id
 		 *
-		 * @return array|string
+		 * @return string
 		 */
 		public function get_preview_email_recipient( $form_id = null ) {
 			$recipients = $this->get_recipient( $form_id );

Spacing +156 added lines, -157 removed lines

@@ -12,11 +12,11 @@ 
  */
 
 // Exit if access directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! class_exists( 'Give_Email_Notification' ) ) :
+if ( ! class_exists('Give_Email_Notification')) :
 
 	/**
 	 * Give_Email_Notification
@@ -93,14 +93,14 @@ 
 		 *
 		 * @return Give_Email_Notification
 		 */
-		public static function get_instance( $email_notification_id = '' ) {
+		public static function get_instance($email_notification_id = '') {
 			$class = '';
 
-			if ( ! empty( $email_notification_id ) ) {
+			if ( ! empty($email_notification_id)) {
 				/* @var Give_Email_Notification $class */
-				foreach ( self::$singleton as $class ) {
-					if ( $email_notification_id === $class->config['id'] ) {
-						$class = get_class( $class );
+				foreach (self::$singleton as $class) {
+					if ($email_notification_id === $class->config['id']) {
+						$class = get_class($class);
 						break;
 					}
 				}
@@ -108,11 +108,11 @@ 
 				$class = get_called_class();
 			}
 
-			if ( ! empty( $class ) && ( ! array_key_exists( $class, self::$singleton ) || is_null( self::$singleton[ $class ] ) ) ) {
-				self::$singleton[ $class ] = new $class();
+			if ( ! empty($class) && ( ! array_key_exists($class, self::$singleton) || is_null(self::$singleton[$class]))) {
+				self::$singleton[$class] = new $class();
 			}
 
-			return ( isset( self::$singleton[ $class ] ) ? self::$singleton[ $class ] : null );
+			return (isset(self::$singleton[$class]) ? self::$singleton[$class] : null);
 		}
 
 		/**
@@ -123,35 +123,35 @@ 
 		 *
 		 * @param array $config
 		 */
-		public function load( $config ) {
+		public function load($config) {
 			// Set notification configuration.
-			$this->config = wp_parse_args( $config, $this->config );
+			$this->config = wp_parse_args($config, $this->config);
 
 			// Set email preview header status.
 			$this->config['has_preview_header'] = $this->config['has_preview'] && $this->config['has_preview_header'] ? true : false;
 
 			// Set email content type
-			$this->config['content_type'] = empty( $this->config['content_type'] ) || ! in_array( $this->config['content_type'], array(
+			$this->config['content_type'] = empty($this->config['content_type']) || ! in_array($this->config['content_type'], array(
 				'text/html',
 				'text/plain',
-			) )
+			))
 				? Give()->emails->get_content_type()
 				: $this->config['content_type'];
-			$this->config['content_type'] = give_get_option( Give_Email_Setting_Field::get_prefix( $this ) . 'email_content_type', $this->config['content_type'] );
+			$this->config['content_type'] = give_get_option(Give_Email_Setting_Field::get_prefix($this).'email_content_type', $this->config['content_type']);
 
 			// Set email template type.
-			$this->config['email_template'] = empty( $this->config['email_template'] )
-				? give_get_option( 'email_template' )
+			$this->config['email_template'] = empty($this->config['email_template'])
+				? give_get_option('email_template')
 				: $this->config['email_template'];
 
 			// Set recipient group name.
-			$this->config['recipient_group_name'] = empty( $this->config['recipient_group_name'] )
-				? ( ! Give_Email_Notification_Util::has_recipient_field( $this ) ? __( 'Donor', 'give' ) : '' )
+			$this->config['recipient_group_name'] = empty($this->config['recipient_group_name'])
+				? ( ! Give_Email_Notification_Util::has_recipient_field($this) ? __('Donor', 'give') : '')
 				: $this->config['recipient_group_name'];
 
 			// Non notification status editable notice.
-			$this->config['notices']['non-notification-status-editable'] = empty( $this->config['notices']['non-notification-status-editable'] )
-				? __( 'You can not edit notification status from here.', 'give' )
+			$this->config['notices']['non-notification-status-editable'] = empty($this->config['notices']['non-notification-status-editable'])
+				? __('You can not edit notification status from here.', 'give')
 				: $this->config['notices']['non-notification-status-editable'];
 
 			/**
@@ -162,7 +162,7 @@ 
 			 * @param                         array                   Give_Email_Notification::config
 			 * @param Give_Email_Notification $this
 			 */
-			$this->config = apply_filters( 'give_email_api_notification_config', $this->config, $this );
+			$this->config = apply_filters('give_email_api_notification_config', $this->config, $this);
 
 			// Setup filters.
 			$this->setup_filters();
@@ -177,7 +177,7 @@ 
 		 */
 		private function setup_filters() {
 			// Apply filter only for current email notification section.
-			if ( give_get_current_setting_section() === $this->config['id'] ) {
+			if (give_get_current_setting_section() === $this->config['id']) {
 				// Initialize email context for email notification.
 				$this->config['email_tag_context'] = apply_filters(
 					"give_{$this->config['id']}_email_tag_context",
@@ -187,23 +187,23 @@ 
 			}
 
 			// Setup setting fields.
-			if( $this->config['show_on_emails_setting_page'] ) {
-				add_filter( 'give_get_settings_emails', array( $this, 'add_setting_fields' ), 10, 2 );
+			if ($this->config['show_on_emails_setting_page']) {
+				add_filter('give_get_settings_emails', array($this, 'add_setting_fields'), 10, 2);
 			}
 
-			if ( $this->config['form_metabox_setting'] ) {
+			if ($this->config['form_metabox_setting']) {
 				add_filter(
 					'give_email_notification_options_metabox_fields',
-					array( $this, 'add_metabox_setting_field' ),
+					array($this, 'add_metabox_setting_field'),
 					10,
 					2
 				);
 			}
 
-			if( $this->config['has_recipient_field'] ) {
+			if ($this->config['has_recipient_field']) {
 				add_action(
 						"give_save__give_{$this->config['id']}_recipient",
-						array( $this, 'validate_form_recipient_field_value' ),
+						array($this, 'validate_form_recipient_field_value'),
 						10,
 						3
 				);
@@ -241,8 +241,8 @@ 
 		 *
 		 * @return array
 		 */
-		public function add_section( $sections ) {
-			$sections[ $this->config['id'] ] = $this->config['label'];
+		public function add_section($sections) {
+			$sections[$this->config['id']] = $this->config['label'];
 
 			return $sections;
 		}
@@ -256,7 +256,7 @@ 
 		 *
 		 * @return bool
 		 */
-		public function hide_section( $hide_section ) {
+		public function hide_section($hide_section) {
 			$hide_section = true;
 
 			return $hide_section;
@@ -272,8 +272,8 @@ 
 		 *
 		 * @return  array
 		 */
-		public function add_setting_fields( $settings ) {
-			if ( $this->config['id'] === give_get_current_setting_section() ) {
+		public function add_setting_fields($settings) {
+			if ($this->config['id'] === give_get_current_setting_section()) {
 				$settings = $this->get_setting_fields();
 			}
 
@@ -291,8 +291,8 @@ 
 		 *
 		 * @return array
 		 */
-		public function get_setting_fields( $form_id = null ) {
-			return Give_Email_Setting_Field::get_setting_fields( $this, $form_id );
+		public function get_setting_fields($form_id = null) {
+			return Give_Email_Setting_Field::get_setting_fields($this, $form_id);
 		}
 
 
@@ -307,13 +307,13 @@ 
 		 *
 		 * @return array
 		 */
-		public function add_metabox_setting_field( $settings, $form_id ) {
+		public function add_metabox_setting_field($settings, $form_id) {
 
-			if( Give_Email_Notification_Util::is_email_notification_active( $this ) ) {
+			if (Give_Email_Notification_Util::is_email_notification_active($this)) {
 				$settings[] = array(
 					'id'     => $this->config['id'],
 					'title'  => $this->config['label'],
-					'fields' => $this->get_setting_fields( $form_id ),
+					'fields' => $this->get_setting_fields($form_id),
 				);
 			}
 
@@ -331,7 +331,7 @@ 
 		 *
 		 * @return array
 		 */
-		public function get_extra_setting_fields( $form_id = null ) {
+		public function get_extra_setting_fields($form_id = null) {
 			return array();
 		}
 
@@ -348,11 +348,11 @@ 
 		 *
 		 * @return string|array
 		 */
-		public function get_recipient( $form_id = null ) {
-			if ( empty( $this->recipient_email ) && $this->config['has_recipient_field'] ) {
+		public function get_recipient($form_id = null) {
+			if (empty($this->recipient_email) && $this->config['has_recipient_field']) {
 				$this->recipient_email = Give_Email_Notification_Util::get_value(
 						$this,
-						Give_Email_Setting_Field::get_prefix( $this, $form_id ) . 'recipient',
+						Give_Email_Setting_Field::get_prefix($this, $form_id).'recipient',
 						$form_id
 				);
 
@@ -363,7 +363,7 @@ 
 				 * @since 1.0
 				 * @deprecated 2.0
 				 */
-				$this->recipient_email = apply_filters( 'give_admin_notice_emails', $this->recipient_email, $this, $form_id );
+				$this->recipient_email = apply_filters('give_admin_notice_emails', $this->recipient_email, $this, $form_id);
 			}
 
 			/**
@@ -393,10 +393,10 @@ 
 		 *
 		 * @return bool
 		 */
-		public function get_notification_status( $form_id = null ) {
-			$notification_status = empty( $form_id )
-				? give_get_option( "{$this->config['id']}_notification", $this->config['notification_status'] )
-				: give_get_meta( $form_id, Give_Email_Setting_Field::get_prefix( $this, $form_id ) . 'notification', true, 'global' );
+		public function get_notification_status($form_id = null) {
+			$notification_status = empty($form_id)
+				? give_get_option("{$this->config['id']}_notification", $this->config['notification_status'])
+				: give_get_meta($form_id, Give_Email_Setting_Field::get_prefix($this, $form_id).'notification', true, 'global');
 
 			/**
 			 * Filter the notification status.
@@ -421,11 +421,11 @@ 
 		 *
 		 * @return string
 		 */
-		function get_email_subject( $form_id = null ) {
+		function get_email_subject($form_id = null) {
 			$subject = wp_strip_all_tags(
 				Give_Email_Notification_Util::get_value(
 					$this,
-					Give_Email_Setting_Field::get_prefix( $this, $form_id ) . 'email_subject',
+					Give_Email_Setting_Field::get_prefix($this, $form_id).'email_subject',
 					$form_id,
 					$this->config['default_email_subject']
 				)
@@ -454,10 +454,10 @@ 
 		 *
 		 * @return string
 		 */
-		public function get_email_message( $form_id = null ) {
+		public function get_email_message($form_id = null) {
 			$message = Give_Email_Notification_Util::get_value(
 				$this,
-				Give_Email_Setting_Field::get_prefix( $this, $form_id ) . 'email_message',
+				Give_Email_Setting_Field::get_prefix($this, $form_id).'email_message',
 				$form_id,
 				$this->config['default_email_message']
 			);
@@ -486,10 +486,10 @@ 
 		 *
 		 * @return string
 		 */
-		public function get_email_content_type( $form_id ) {
+		public function get_email_content_type($form_id) {
 			$content_type = Give_Email_Notification_Util::get_value(
 				$this,
-				Give_Email_Setting_Field::get_prefix( $this, $form_id ) . 'email_content_type',
+				Give_Email_Setting_Field::get_prefix($this, $form_id).'email_content_type',
 				$form_id,
 				$this->config['content_type']
 			);
@@ -517,15 +517,14 @@ 
 		 *
 		 * @return string
 		 */
-		public function get_email_template( $form_id ) {
-			$email_template = give_get_meta( $form_id, '_give_email_template', true );
+		public function get_email_template($form_id) {
+			$email_template = give_get_meta($form_id, '_give_email_template', true);
 			$email_template = Give_Email_Notification_Util::get_value(
 				$this,
-				Give_Email_Setting_Field::get_prefix( $this, $form_id ) .'email_template',
+				Give_Email_Setting_Field::get_prefix($this, $form_id).'email_template',
 				$form_id,
-				! empty( $email_template ) && Give_Email_Notification_Util::can_use_form_email_options( $this, $form_id ) ?
-					$email_template :
-					$this->config['email_template']
+				! empty($email_template) && Give_Email_Notification_Util::can_use_form_email_options($this, $form_id) ?
+					$email_template : $this->config['email_template']
 			);
 
 			/**
@@ -552,34 +551,34 @@ 
 		 *
 		 * @return array
 		 */
-		public function get_allowed_email_tags( $formatted = false ) {
+		public function get_allowed_email_tags($formatted = false) {
 			// Get all email tags.
 			$email_tags = Give()->email_tags->get_tags();
 
 			// Skip if all email template tags context setup exit.
-			if ( $this->config['email_tag_context'] && 'all' !== $this->config['email_tag_context'] ) {
-				if ( is_array( $this->config['email_tag_context'] ) ) {
-					foreach ( $email_tags as $index => $email_tag ) {
-						if ( in_array( $email_tag['context'], $this->config['email_tag_context'] ) ) {
+			if ($this->config['email_tag_context'] && 'all' !== $this->config['email_tag_context']) {
+				if (is_array($this->config['email_tag_context'])) {
+					foreach ($email_tags as $index => $email_tag) {
+						if (in_array($email_tag['context'], $this->config['email_tag_context'])) {
 							continue;
 						}
 
-						unset( $email_tags[ $index ] );
+						unset($email_tags[$index]);
 					}
 				} else {
-					foreach ( $email_tags as $index => $email_tag ) {
-						if ( $this->config['email_tag_context'] === $email_tag['context'] ) {
+					foreach ($email_tags as $index => $email_tag) {
+						if ($this->config['email_tag_context'] === $email_tag['context']) {
 							continue;
 						}
 
-						unset( $email_tags[ $index ] );
+						unset($email_tags[$index]);
 					}
 				}
 			}
 
-			if ( count( $email_tags ) && $formatted ) : ob_start() ?>
+			if (count($email_tags) && $formatted) : ob_start() ?>
 				<div class="give-email-tags-wrap">
-					<?php foreach ( $email_tags as $email_tag ) : ?>
+					<?php foreach ($email_tags as $email_tag) : ?>
 						<span class="give_<?php echo $email_tag['tag']; ?>_tag">
 							<code>{<?php echo $email_tag['tag']; ?>}</code> - <?php echo $email_tag['description']; ?>
 						</span>
@@ -602,8 +601,8 @@ 
 		 *
 		 * @return array|string
 		 */
-		public function get_preview_email_recipient( $form_id = null ) {
-			$recipients = $this->get_recipient( $form_id );
+		public function get_preview_email_recipient($form_id = null) {
+			$recipients = $this->get_recipient($form_id);
 
 			/**
 			 * Filter the preview email recipients.
@@ -613,7 +612,7 @@ 
 			 * @param string|array            $recipients List of recipients.
 			 * @param Give_Email_Notification $this
 			 */
-			$recipients = apply_filters( 'give_get_preview_email_recipient', $recipients, $this, $form_id );
+			$recipients = apply_filters('give_get_preview_email_recipient', $recipients, $this, $form_id);
 
 			return $recipients;
 		}
@@ -628,13 +627,13 @@ 
 		 *
 		 * @return array
 		 */
-		public function get_email_attachments( $form_id = null ) {
+		public function get_email_attachments($form_id = null) {
 			/**
 			 * Filter the attachment.
 			 *
 			 * @since 2.0
 			 */
-			return apply_filters( "give_{$this->config['id']}_get_email_attachments", array(), $this, $form_id );
+			return apply_filters("give_{$this->config['id']}_get_email_attachments", array(), $this, $form_id);
 		}
 
 
@@ -646,37 +645,37 @@ 
 		 */
 		public function send_preview_email() {
 			// Get form id
-			$form_id = ! empty( $_GET['form_id'] ) ? absint( $_GET['form_id'] ) : null;
+			$form_id = ! empty($_GET['form_id']) ? absint($_GET['form_id']) : null;
 
 			// setup email data.
 			$this->setup_email_data();
 
 			$attachments  = $this->get_email_attachments();
-			$message      = $this->preview_email_template_tags( $this->get_email_message( $form_id ) );
-			$subject      = $this->preview_email_template_tags( $this->get_email_subject( $form_id ) );
-			$content_type = $this->get_email_content_type( $form_id );
+			$message      = $this->preview_email_template_tags($this->get_email_message($form_id));
+			$subject      = $this->preview_email_template_tags($this->get_email_subject($form_id));
+			$content_type = $this->get_email_content_type($form_id);
 
 			// Setup email content type.
-			Give()->emails->__set( 'content_type', $content_type );
-			Give()->emails->__set( 'html', true );
+			Give()->emails->__set('content_type', $content_type);
+			Give()->emails->__set('html', true);
 
 			// Setup email template
-			Give()->emails->__set( 'template', $this->get_email_template( $form_id ) );
+			Give()->emails->__set('template', $this->get_email_template($form_id));
 
 			// Format plain content type email.
-			if ( 'text/plain' === $content_type ) {
-				Give()->emails->__set( 'html', false );
-				Give()->emails->__set( 'template', 'none' );
-				$message = strip_tags( $message );
+			if ('text/plain' === $content_type) {
+				Give()->emails->__set('html', false);
+				Give()->emails->__set('template', 'none');
+				$message = strip_tags($message);
 			}
 
-			if ( Give_Email_Notification_Util::can_use_form_email_options( $this, $form_id ) ) {
+			if (Give_Email_Notification_Util::can_use_form_email_options($this, $form_id)) {
 				Give()->emails->form_id      = $form_id;
-				Give()->emails->from_name    = give_get_meta( $form_id, '_give_from_name', true );
-				Give()->emails->from_address = give_get_meta( $form_id, '_give_from_email', true );
+				Give()->emails->from_name    = give_get_meta($form_id, '_give_from_name', true);
+				Give()->emails->from_address = give_get_meta($form_id, '_give_from_email', true);
 			}
 
-			return Give()->emails->send( $this->get_preview_email_recipient( $form_id ), $subject, $message, $attachments );
+			return Give()->emails->send($this->get_preview_email_recipient($form_id), $subject, $message, $attachments);
 		}
 
 
@@ -697,7 +696,7 @@ 
 		 *
 		 * @return bool
 		 */
-		public function send_email_notification( $email_tag_args = array() ) {
+		public function send_email_notification($email_tag_args = array()) {
 			// Add email content type email tags.
 			$email_tag_args['email_content_type'] = $this->config['content_type'];
 
@@ -706,16 +705,16 @@ 
 			 *
 			 * @since 2.0
 			 */
-			$email_tag_args = apply_filters( "give_{$this->config['id']}_email_tag_args", $email_tag_args, $this );
+			$email_tag_args = apply_filters("give_{$this->config['id']}_email_tag_args", $email_tag_args, $this);
 
 			// Get form id.
-			$form_id = ! empty( $email_tag_args['form_id'] )
-				? absint( $email_tag_args['form_id'] )
-				: ( ! empty( $email_tag_args['payment_id'] ) ? give_get_payment_form_id( $email_tag_args['payment_id'] ) : null );
+			$form_id = ! empty($email_tag_args['form_id'])
+				? absint($email_tag_args['form_id'])
+				: ( ! empty($email_tag_args['payment_id']) ? give_get_payment_form_id($email_tag_args['payment_id']) : null);
 
 
 			// Do not send email if notification is disable.
-			if ( ! Give_Email_Notification_Util::is_email_notification_active( $this, $form_id ) ) {
+			if ( ! Give_Email_Notification_Util::is_email_notification_active($this, $form_id)) {
 				return false;
 			}
 
@@ -724,41 +723,41 @@ 
 			 *
 			 * @since 2.0
 			 */
-			do_action( "give_{$this->config['id']}_email_send_before", $this, $form_id );
+			do_action("give_{$this->config['id']}_email_send_before", $this, $form_id);
 
 			$attachments  = $this->get_email_attachments();
-			$message      = give_do_email_tags( $this->get_email_message( $form_id ), $email_tag_args );
-			$subject      = give_do_email_tags( $this->get_email_subject( $form_id ), $email_tag_args );
-			$content_type = $this->get_email_content_type( $form_id );
+			$message      = give_do_email_tags($this->get_email_message($form_id), $email_tag_args);
+			$subject      = give_do_email_tags($this->get_email_subject($form_id), $email_tag_args);
+			$content_type = $this->get_email_content_type($form_id);
 
 			// Setup email content type.
-			Give()->emails->__set( 'content_type', $content_type );
-			Give()->emails->__set( 'html', true );
+			Give()->emails->__set('content_type', $content_type);
+			Give()->emails->__set('html', true);
 
 			// Set email template.
-			Give()->emails->__set( 'template', $this->get_email_template( $form_id ) );
+			Give()->emails->__set('template', $this->get_email_template($form_id));
 
-			if ( 'text/plain' === $content_type ) {
-				Give()->emails->__set( 'html', false );
-				Give()->emails->__set( 'template', 'none' );
-				$message = strip_tags( $message );
+			if ('text/plain' === $content_type) {
+				Give()->emails->__set('html', false);
+				Give()->emails->__set('template', 'none');
+				$message = strip_tags($message);
 			}
 
-			if ( Give_Email_Notification_Util::can_use_form_email_options( $this, $form_id ) ) {
+			if (Give_Email_Notification_Util::can_use_form_email_options($this, $form_id)) {
 				Give()->emails->form_id      = $form_id;
-				Give()->emails->from_name    = give_get_meta( $form_id, '_give_from_name', true );
-				Give()->emails->from_address = give_get_meta( $form_id, '_give_from_email', true );
+				Give()->emails->from_name    = give_get_meta($form_id, '_give_from_name', true);
+				Give()->emails->from_address = give_get_meta($form_id, '_give_from_email', true);
 			}
 
 			// Send email.
-			$email_status = Give()->emails->send( $this->get_recipient( $form_id ), $subject, $message, $attachments );
+			$email_status = Give()->emails->send($this->get_recipient($form_id), $subject, $message, $attachments);
 
 			/**
 			 * Fire action after after email send.
 			 *
 			 * @since 2.0
 			 */
-			do_action( "give_{$this->config['id']}_email_send_after", $email_status, $this, $form_id );
+			do_action("give_{$this->config['id']}_email_send_after", $email_status, $this, $form_id);
 
 			return $email_status;
 		}
@@ -773,67 +772,67 @@ 
 		 *
 		 * @return string
 		 */
-		public function preview_email_template_tags( $message ) {
+		public function preview_email_template_tags($message) {
 			// Set Payment.
-			$payment_id = give_check_variable( give_clean( $_GET ), 'isset_empty', 0, 'preview_id' );
-			$payment    = $payment_id ? new Give_Payment( $payment_id ) : new stdClass();
+			$payment_id = give_check_variable(give_clean($_GET), 'isset_empty', 0, 'preview_id');
+			$payment    = $payment_id ? new Give_Payment($payment_id) : new stdClass();
 
 			// Set donor.
 			$user_id = $payment_id
 				? $payment->user_id
-				: give_check_variable( give_clean( $_GET ), 'isset_empty', 0, 'user_id' );
+				: give_check_variable(give_clean($_GET), 'isset_empty', 0, 'user_id');
 			$user_id = $user_id ? $user_id : wp_get_current_user()->ID;
 
 			// Set receipt.
-			$receipt_id = strtolower( md5( uniqid() ) );
+			$receipt_id = strtolower(md5(uniqid()));
 
-			$receipt_link_url = esc_url( add_query_arg( array(
+			$receipt_link_url = esc_url(add_query_arg(array(
 				'payment_key' => $receipt_id,
-			), give_get_history_page_uri() ) );
+			), give_get_history_page_uri()));
 
 			$receipt_link = sprintf(
 				'<a href="%1$s">%2$s</a>',
 				$receipt_link_url,
-				esc_html__( 'View the receipt in your browser &raquo;', 'give' )
+				esc_html__('View the receipt in your browser &raquo;', 'give')
 			);
 
 			// Set default values for tags.
 			$this->config['preview_email_tags_values'] = wp_parse_args(
 				$this->config['preview_email_tags_values'],
 				array(
-					'name'              => give_email_tag_first_name( array(
+					'name'              => give_email_tag_first_name(array(
 						'payment_id' => $payment_id,
 						'user_id'    => $user_id,
-					) ),
-					'fullname'          => give_email_tag_fullname( array(
+					)),
+					'fullname'          => give_email_tag_fullname(array(
 						'payment_id' => $payment_id,
 						'user_id'    => $user_id,
-					) ),
-					'username'          => give_email_tag_username( array(
+					)),
+					'username'          => give_email_tag_username(array(
 						'payment_id' => $payment_id,
 						'user_id'    => $user_id,
-					) ),
-					'user_email'        => give_email_tag_user_email( array(
+					)),
+					'user_email'        => give_email_tag_user_email(array(
 						'payment_id' => $payment_id,
 						'user_id'    => $user_id,
-					) ),
-					'payment_total'     => $payment_id ? give_email_tag_payment_total( array( 'payment_id' => $payment_id ) ) : give_currency_filter( '10.50' ),
-					'amount'            => $payment_id ? give_email_tag_amount( array( 'payment_id' => $payment_id ) ) : give_currency_filter( '10.50' ),
-					'price'             => $payment_id ? give_email_tag_price( array( 'payment_id' => $payment_id ) ) : give_currency_filter( '10.50' ),
-					'payment_method'    => $payment_id ? give_email_tag_payment_method( array( 'payment_id' => $payment_id ) ) : __( 'PayPal', 'give' ),
+					)),
+					'payment_total'     => $payment_id ? give_email_tag_payment_total(array('payment_id' => $payment_id)) : give_currency_filter('10.50'),
+					'amount'            => $payment_id ? give_email_tag_amount(array('payment_id' => $payment_id)) : give_currency_filter('10.50'),
+					'price'             => $payment_id ? give_email_tag_price(array('payment_id' => $payment_id)) : give_currency_filter('10.50'),
+					'payment_method'    => $payment_id ? give_email_tag_payment_method(array('payment_id' => $payment_id)) : __('PayPal', 'give'),
 					'receipt_id'        => $receipt_id,
-					'payment_id'        => $payment_id ? $payment_id : rand( 2000, 2050 ),
+					'payment_id'        => $payment_id ? $payment_id : rand(2000, 2050),
 					'receipt_link_url'  => $receipt_link_url,
 					'receipt_link'      => $receipt_link,
-					'date'              => $payment_id ? date( give_date_format(), strtotime( $payment->date ) ) : date( give_date_format(), current_time( 'timestamp' ) ),
-					'donation'          => $payment_id ? give_email_tag_donation( array( 'payment_id' => $payment_id ) ) : esc_html__( 'Sample Donation Form Title', 'give' ),
-					'form_title'        => $payment_id ? give_email_tag_form_title( array( 'payment_id' => $payment_id ) ) : esc_html__( 'Sample Donation Form Title - Sample Donation Level', 'give' ),
-					'sitename'          => $payment_id ? give_email_tag_sitename( array( 'payment_id' => $payment_id ) ) : get_bloginfo( 'name' ),
+					'date'              => $payment_id ? date(give_date_format(), strtotime($payment->date)) : date(give_date_format(), current_time('timestamp')),
+					'donation'          => $payment_id ? give_email_tag_donation(array('payment_id' => $payment_id)) : esc_html__('Sample Donation Form Title', 'give'),
+					'form_title'        => $payment_id ? give_email_tag_form_title(array('payment_id' => $payment_id)) : esc_html__('Sample Donation Form Title - Sample Donation Level', 'give'),
+					'sitename'          => $payment_id ? give_email_tag_sitename(array('payment_id' => $payment_id)) : get_bloginfo('name'),
 					'pdf_receipt'       => sprintf(
 						'<a href="#">%s</a>',
-						__( 'Download Receipt', 'give' )
+						__('Download Receipt', 'give')
 					),
-					'billing_address'   => $payment_id ? give_email_tag_billing_address( array( 'payment_id' => $payment_id ) ) : '',
+					'billing_address'   => $payment_id ? give_email_tag_billing_address(array('payment_id' => $payment_id)) : '',
 					'email_access_link' => sprintf(
 						'<a href="%1$s">%2$s</a>',
 						add_query_arg(
@@ -842,20 +841,20 @@ 
 							),
 							give_get_history_page_uri()
 						),
-						__( 'View your donation history &raquo;', 'give' )
+						__('View your donation history &raquo;', 'give')
 					),
-					'reset_password_link' => $user_id ? give_email_tag_reset_password_link( array( 'user_id' => $user_id ), $payment_id ) : '',
+					'reset_password_link' => $user_id ? give_email_tag_reset_password_link(array('user_id' => $user_id), $payment_id) : '',
 				)
 			);
 
 			// Decode tags.
-			foreach ( $this->config['preview_email_tags_values'] as $preview_tag => $value ) {
-				if ( isset( $this->config['preview_email_tags_values'][ $preview_tag ] ) ) {
-					$message = str_replace( "{{$preview_tag}}", $this->config['preview_email_tags_values'][ $preview_tag ], $message );
+			foreach ($this->config['preview_email_tags_values'] as $preview_tag => $value) {
+				if (isset($this->config['preview_email_tags_values'][$preview_tag])) {
+					$message = str_replace("{{$preview_tag}}", $this->config['preview_email_tags_values'][$preview_tag], $message);
 				}
 			}
 
-			return apply_filters( 'give_email_preview_template_tags', $message );
+			return apply_filters('give_email_preview_template_tags', $message);
 		}
 
 		/**
@@ -880,17 +879,17 @@ 
 		 * @param $post_id
 		 *
 		 */
-		public function validate_form_recipient_field_value( $form_meta_key, $form_meta_value, $post_id ) {
+		public function validate_form_recipient_field_value($form_meta_key, $form_meta_value, $post_id) {
 			// Get valid emails.
-			$new_form_meta_value = array_filter( $form_meta_value, function ( $value ) {
-				return ! empty( $value['email'] ) && is_email( $value['email'] );
+			$new_form_meta_value = array_filter($form_meta_value, function($value) {
+				return ! empty($value['email']) && is_email($value['email']);
 			} );
 
 			// Remove duplicate emails from array.
 			$email_arr = array();
-			foreach ( $new_form_meta_value as $index => $email ) {
-				if( in_array( $email['email'], $email_arr  ) ) {
-					unset( $new_form_meta_value[$index] );
+			foreach ($new_form_meta_value as $index => $email) {
+				if (in_array($email['email'], $email_arr)) {
+					unset($new_form_meta_value[$index]);
 					continue;
 				}
 
@@ -899,25 +898,25 @@ 
 
 			$update = false;
 
-			if ( empty( $new_form_meta_value ) ) {
+			if (empty($new_form_meta_value)) {
 				// Set default recipient.
 				$form_meta_value = array(
 					array(
-						'email' => get_bloginfo( 'admin_email' )
+						'email' => get_bloginfo('admin_email')
 					),
 				);
 
 				$update = true;
 
-			} elseif ( count( $new_form_meta_value ) !== count( $form_meta_value ) ) {
+			} elseif (count($new_form_meta_value) !== count($form_meta_value)) {
 				// Filter recipient emails.
 				$form_meta_value = $new_form_meta_value;
 
 				$update = true;
 			}
 
-			if( $update ) {
-				give_update_meta( $post_id, $form_meta_key, $form_meta_value );
+			if ($update) {
+				give_update_meta($post_id, $form_meta_key, $form_meta_value);
 			}
 		}
 	}

includes/admin/emails/class-donor-register-email.php

Doc Comments +1 added lines, -1 removed lines

@@ -120,7 +120,7 @@
 		 * @param string                    $email_preview_header
 		 * @param Give_Donor_Register_Email $email
 		 *
-		 * @return bool
+		 * @return string|null
 		 */
 		public function email_preview_header( $email_preview_header, $email ) {
 			// Bailout.

Spacing +37 added lines, -37 removed lines

@@ -10,11 +10,11 @@ 
  */
 
 // Exit if access directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! class_exists( 'Give_Donor_Register_Email' ) ) :
+if ( ! class_exists('Give_Donor_Register_Email')) :
 
 	/**
 	 * Give_Donor_Register_Email
@@ -31,34 +31,34 @@ 
 		 * @since   2.0
 		 */
 		public function init() {
-			$this->load( array(
+			$this->load(array(
 				'id'                    => 'donor-register',
-				'label'                 => __( 'User Registration Information', 'give' ),
-				'description'           => __( 'Sent to the donor when they register for an account on the site.', 'give' ),
+				'label'                 => __('User Registration Information', 'give'),
+				'description'           => __('Sent to the donor when they register for an account on the site.', 'give'),
 				'notification_status'   => 'enabled',
 				'email_tag_contex'      => 'donor',
 				'form_metabox_setting'  => false,
-				'recipient_group_name'  => __( 'Donor', 'give' ),
-				'email_tag_context'     => array( 'donor', 'general' ),
+				'recipient_group_name'  => __('Donor', 'give'),
+				'email_tag_context'     => array('donor', 'general'),
 				'default_email_subject' => sprintf(
 				/* translators: %s: site name */
-					esc_attr__( '[%s] Your username and password', 'give' ),
-					get_bloginfo( 'name' )
+					esc_attr__('[%s] Your username and password', 'give'),
+					get_bloginfo('name')
 				),
 				'default_email_message' => $this->get_default_email_message(),
-			) );
+			));
 
 			// Setup action hook.
 			add_action(
 				"give_{$this->config['id']}_email_notification",
-				array( $this, 'setup_email_notification' ),
+				array($this, 'setup_email_notification'),
 				10,
 				2
 			);
 
 			add_filter(
 				'give_email_preview_header',
-				array( $this, 'email_preview_header' ),
+				array($this, 'email_preview_header'),
 				10,
 				2
 			);
@@ -73,13 +73,13 @@ 
 		 * @return string
 		 */
 		function get_default_email_message() {
-			$message = esc_attr__( 'Username: {username}', 'give' ) . "\r\n\r\n";
+			$message = esc_attr__('Username: {username}', 'give')."\r\n\r\n";
 
-			$message .= __( 'To reset your password, simply click the link below which will take you to a web page where you can create a new password.', 'give' ) . "\r\n";
-			$message .= '{reset_password_link}' . "\r\n\r\n";
+			$message .= __('To reset your password, simply click the link below which will take you to a web page where you can create a new password.', 'give')."\r\n";
+			$message .= '{reset_password_link}'."\r\n\r\n";
 
-			$message .= __( 'After resetting password, Please login to your account with link below.', 'give' ) . "\r\n";
-			$message .= '<a href="' . wp_login_url() . '"> ' . esc_attr__( 'Click Here to Login &raquo;', 'give' ) . '</a>' . "\r\n";
+			$message .= __('After resetting password, Please login to your account with link below.', 'give')."\r\n";
+			$message .= '<a href="'.wp_login_url().'"> '.esc_attr__('Click Here to Login &raquo;', 'give').'</a>'."\r\n";
 
 			/**
 			 * Filter the default email message
@@ -98,7 +98,7 @@ 
 		 * @since 2.0
 		 */
 		public function setup_email_data() {
-			Give()->emails->__set( 'heading', esc_html__( 'New User Registration', 'give' ) );
+			Give()->emails->__set('heading', esc_html__('New User Registration', 'give'));
 		}
 
 		/**
@@ -112,13 +112,13 @@ 
 		 *
 		 * @return string
 		 */
-		public function setup_email_notification( $user_id, $user_data ) {
+		public function setup_email_notification($user_id, $user_data) {
 			$this->setup_email_data();
 
 			$this->recipient_email = $user_data['user_email'];
-			$this->send_email_notification( array(
+			$this->send_email_notification(array(
 				'user_id' => $user_id,
-			) );
+			));
 		}
 
 		/**
@@ -132,42 +132,42 @@ 
 		 *
 		 * @return bool
 		 */
-		public function email_preview_header( $email_preview_header, $email ) {
+		public function email_preview_header($email_preview_header, $email) {
 			// Bailout.
-			if ( $this->config['id'] !== $email->config['id'] ) {
+			if ($this->config['id'] !== $email->config['id']) {
 				return $email_preview_header;
 			}
 
 			// Payment receipt switcher
-			$user_id = give_check_variable( give_clean( $_GET ), 'isset', 0, 'user_id' );
+			$user_id = give_check_variable(give_clean($_GET), 'isset', 0, 'user_id');
 
 			// Get payments.
 			$donors  = new Give_API();
-			$donors  = give_check_variable( $donors->get_donors(), 'empty', array(), 'donors' );
+			$donors  = give_check_variable($donors->get_donors(), 'empty', array(), 'donors');
 			$options = array();
 
 			// Default option.
-			$options[0] = esc_html__( 'No donor(s) found.', 'give' );
+			$options[0] = esc_html__('No donor(s) found.', 'give');
 
 			// Provide nice human readable options.
-			if ( $donors ) {
-				$options[0] = esc_html__( '- Select a donor -', 'give' );
-				foreach ( $donors as $donor ) {
+			if ($donors) {
+				$options[0] = esc_html__('- Select a donor -', 'give');
+				foreach ($donors as $donor) {
 					// Exclude customers for which wp user not exist.
-					if ( ! $donor['info']['user_id'] ) {
+					if ( ! $donor['info']['user_id']) {
 						continue;
 					}
-					$options[ $donor['info']['user_id'] ] = esc_html( '#' . $donor['info']['donor_id'] . ' - ' . $donor['info']['email'] );
+					$options[$donor['info']['user_id']] = esc_html('#'.$donor['info']['donor_id'].' - '.$donor['info']['email']);
 				}
 			}
 
-			$request_url_data = wp_parse_url( $_SERVER['REQUEST_URI'] );
+			$request_url_data = wp_parse_url($_SERVER['REQUEST_URI']);
 			$query            = $request_url_data['query'];
 
 			// Remove user id query param if set from request url.
-			$query = remove_query_arg( array( 'user_id' ), $query );
+			$query = remove_query_arg(array('user_id'), $query);
 
-			$request_url = home_url( '/?' . str_replace( '', '', $query ) );
+			$request_url = home_url('/?'.str_replace('', '', $query));
 			?>
 
 			<!-- Start constructing HTML output.-->
@@ -185,12 +185,12 @@ 
 				</script>
 
 				<label for="give_preview_email_user_id" style="font-size:12px;color:#333;margin:0 4px 0 0;">
-					<?php echo esc_html__( 'Preview email with a donor:', 'give' ); ?>
+					<?php echo esc_html__('Preview email with a donor:', 'give'); ?>
 				</label>
 
 				<?php
 				// The select field with 100 latest transactions
-				echo Give()->html->select( array(
+				echo Give()->html->select(array(
 					'name'             => 'preview_email_user_id',
 					'selected'         => $user_id,
 					'id'               => 'give_preview_email_user_id',
@@ -200,7 +200,7 @@ 
 					'select_atts'      => 'onchange="change_preview()"',
 					'show_option_all'  => false,
 					'show_option_none' => false,
-				) );
+				));
 				?>
 			</div>
 			<?php

includes/admin/emails/class-email-notification-util.php

Doc Comments +1 added lines, -1 removed lines

@@ -178,7 +178,7 @@
 	 * @param Give_Email_Notification $email
 	 * @param int $form_id
 	 *
-	 * @return string
+	 * @return boolean
 	 */
 	public static function is_email_notification_active( Give_Email_Notification $email, $form_id = null ) {
 		$notification_status = $email->get_notification_status( $form_id );

Spacing +36 added lines, -36 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if access directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -46,7 +46,7 @@ 
 	 * @return static
 	 */
 	static function get_instance() {
-		if ( null === static::$instance ) {
+		if (null === static::$instance) {
 			self::$instance = new static();
 		}
 
@@ -64,7 +64,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function has_preview( Give_Email_Notification $email ) {
+	public static function has_preview(Give_Email_Notification $email) {
 		return $email->config['has_preview'];
 	}
 
@@ -78,7 +78,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function has_recipient_field( Give_Email_Notification $email ) {
+	public static function has_recipient_field(Give_Email_Notification $email) {
 		return $email->config['has_recipient_field'];
 	}
 
@@ -92,7 +92,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function is_notification_status_editable( Give_Email_Notification $email ) {
+	public static function is_notification_status_editable(Give_Email_Notification $email) {
 		$user_can_edit = $email->config['notification_status_editable'];
 
 		return (bool) $user_can_edit;
@@ -108,7 +108,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function is_content_type_editable( Give_Email_Notification $email ) {
+	public static function is_content_type_editable(Give_Email_Notification $email) {
 		return $email->config['content_type_editable'];
 	}
 
@@ -122,7 +122,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function is_email_preview_has_header( Give_Email_Notification $email ) {
+	public static function is_email_preview_has_header(Give_Email_Notification $email) {
 		return $email->config['has_preview_header'];
 	}
 
@@ -136,7 +136,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function is_email_preview( Give_Email_Notification $email ) {
+	public static function is_email_preview(Give_Email_Notification $email) {
 		return $email->config['has_preview'];
 	}
 
@@ -150,7 +150,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function is_show_on_emails_setting_page( Give_Email_Notification $email ){
+	public static function is_show_on_emails_setting_page(Give_Email_Notification $email) {
 		return $email->config['show_on_emails_setting_page'];
 	}
 
@@ -165,8 +165,8 @@ 
 	 *
 	 * @return bool
 	 */
-	public static function can_use_form_email_options( Give_Email_Notification $email, $form_id = null ){
-		return give_is_setting_enabled( give_get_meta( $form_id, '_give_email_options', true ) );
+	public static function can_use_form_email_options(Give_Email_Notification $email, $form_id = null) {
+		return give_is_setting_enabled(give_get_meta($form_id, '_give_email_options', true));
 	}
 
 	/**
@@ -180,13 +180,13 @@ 
 	 *
 	 * @return string
 	 */
-	public static function is_email_notification_active( Give_Email_Notification $email, $form_id = null ) {
-		$notification_status = $email->get_notification_status( $form_id );
+	public static function is_email_notification_active(Give_Email_Notification $email, $form_id = null) {
+		$notification_status = $email->get_notification_status($form_id);
 
 
-		$notification_status = empty( $form_id )
-			? give_is_setting_enabled( $notification_status )
-			: give_is_setting_enabled( give_get_option( "{$email->config['id']}_notification", $email->config['notification_status'] ) ) && give_is_setting_enabled( $notification_status, array( 'enabled', 'global' ) );
+		$notification_status = empty($form_id)
+			? give_is_setting_enabled($notification_status)
+			: give_is_setting_enabled(give_get_option("{$email->config['id']}_notification", $email->config['notification_status'])) && give_is_setting_enabled($notification_status, array('enabled', 'global'));
 			// To check if email notification is active or not on a per-form basis, email notification must be globally active—otherwise it will be considered disabled.
 
 		return $notification_status;
@@ -203,8 +203,8 @@ 
 		$is_preview = false;
 
 		if (
-			current_user_can( 'manage_give_settings' )
-			&& ! empty( $_GET['give_action'] )
+			current_user_can('manage_give_settings')
+			&& ! empty($_GET['give_action'])
 			&& 'preview_email' === $_GET['give_action']
 		) {
 			$is_preview = true;
@@ -224,8 +224,8 @@ 
 		$is_preview = false;
 
 		if (
-			current_user_can( 'manage_give_settings' )
-			&& ! empty( $_GET['give_action'] )
+			current_user_can('manage_give_settings')
+			&& ! empty($_GET['give_action'])
 			&& 'send_preview_email' === $_GET['give_action']
 		) {
 			$is_preview = true;
@@ -245,13 +245,13 @@ 
 	 *
 	 * @return string
 	 */
-	public static function get_formatted_email_type( $content_type ) {
+	public static function get_formatted_email_type($content_type) {
 		$email_contents = array(
-			'text/html'  => __( 'HTML', 'give' ),
-			'text/plain' => __( 'Plain', 'give' ),
+			'text/html'  => __('HTML', 'give'),
+			'text/plain' => __('Plain', 'give'),
 		);
 
-		return $email_contents[ $content_type ];
+		return $email_contents[$content_type];
 	}
 
 
@@ -268,40 +268,40 @@ 
 	 *
 	 * @return mixed
 	 */
-	public static function get_value( Give_Email_Notification $email, $option_name, $form_id = null, $default = false ) {
+	public static function get_value(Give_Email_Notification $email, $option_name, $form_id = null, $default = false) {
 		// If form id set then option name can be contain _give_ prefix which is only used for meta key,
 		// So make sure you are using correct option name.
-		$global_option_name = ( 0 === strpos( $option_name, '_give_' )
-			? str_replace( '_give_', '', $option_name )
-			: $option_name );
-		$option_value = give_get_option( $global_option_name, $default );
+		$global_option_name = (0 === strpos($option_name, '_give_')
+			? str_replace('_give_', '', $option_name)
+			: $option_name);
+		$option_value = give_get_option($global_option_name, $default);
 
 		if (
-			! empty( $form_id )
+			! empty($form_id)
 			&& give_is_setting_enabled(
 				give_get_meta(
 					$form_id,
-					Give_Email_Setting_Field::get_prefix( $email, $form_id ) . 'notification',
+					Give_Email_Setting_Field::get_prefix($email, $form_id).'notification',
 					true,
 					'global'
 				)
 			)
 		) {
-			$option_value = get_post_meta( $form_id, $option_name, true );
+			$option_value = get_post_meta($form_id, $option_name, true);
 
 			// Get only email field value from recipients setting.
-			if( Give_Email_Setting_Field::get_prefix( $email, $form_id ) . 'recipient' === $option_name ) {
-				$option_value = wp_list_pluck( $option_value, 'email' );
+			if (Give_Email_Setting_Field::get_prefix($email, $form_id).'recipient' === $option_name) {
+				$option_value = wp_list_pluck($option_value, 'email');
 			}
 		}
 
-		$option_value = empty( $option_value ) ? $default : $option_value;
+		$option_value = empty($option_value) ? $default : $option_value;
 
 		/**
 		 * Filter the setting value
 		 *
 		 * @since 2.0
 		 */
-		return apply_filters( 'give_email_setting_value', $option_value, $option_name, $email, $form_id, $default );
+		return apply_filters('give_email_setting_value', $option_value, $option_name, $email, $form_id, $default);
 	}
 }

includes/admin/emails/class-email-notifications.php

Doc Comments +2 added lines, -2 removed lines

@@ -231,7 +231,7 @@ 
 	 *
 	 * @since  2.0
 	 * @access public
-	 * @return bool|null
+	 * @return false|null
 	 */
 	public function preview_email() {
 		// Bailout.
@@ -369,7 +369,7 @@ 
 	 *
 	 * @since  2.0
 	 * @access public
-	 * @return bool|null
+	 * @return false|null
 	 */
 	public function send_preview_email() {
 		// Bailout.

Spacing +82 added lines, -82 removed lines

@@ -52,7 +52,7 @@ 
 	 * @return static
 	 */
 	static function get_instance() {
-		if ( null === static::$instance ) {
+		if (null === static::$instance) {
 			self::$instance = new static();
 		}
 
@@ -66,31 +66,31 @@ 
 	 */
 	public function init() {
 		// Load files.
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/emails/ajax-handler.php';
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/emails/class-email-setting-field.php';
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/emails/filters.php';
+		require_once GIVE_PLUGIN_DIR.'includes/admin/emails/ajax-handler.php';
+		require_once GIVE_PLUGIN_DIR.'includes/admin/emails/class-email-setting-field.php';
+		require_once GIVE_PLUGIN_DIR.'includes/admin/emails/filters.php';
 
 		// Load email notifications.
 		$this->add_emails_notifications();
 
-		add_filter( 'give_metabox_form_data_settings', array( $this, 'add_metabox_setting_fields' ), 10, 2 );
-		add_action( 'init', array( $this, 'preview_email' ) );
-		add_action( 'init', array( $this, 'send_preview_email' ) );
-		add_action( 'init', array( $this, 'validate_settings' ) );
+		add_filter('give_metabox_form_data_settings', array($this, 'add_metabox_setting_fields'), 10, 2);
+		add_action('init', array($this, 'preview_email'));
+		add_action('init', array($this, 'send_preview_email'));
+		add_action('init', array($this, 'validate_settings'));
 
 		/* @var Give_Email_Notification $email */
-		foreach ( $this->get_email_notifications() as $email ) {
+		foreach ($this->get_email_notifications() as $email) {
 			// Setup email section.
-			if( Give_Email_Notification_Util::is_show_on_emails_setting_page( $email ) ) {
-				add_filter( 'give_get_sections_emails', array( $email, 'add_section' ) );
-				add_filter( "give_hide_section_{$email->config['id']}_on_emails_page", array( $email, 'hide_section' ) );
+			if (Give_Email_Notification_Util::is_show_on_emails_setting_page($email)) {
+				add_filter('give_get_sections_emails', array($email, 'add_section'));
+				add_filter("give_hide_section_{$email->config['id']}_on_emails_page", array($email, 'hide_section'));
 			}
 
 			// Setup email preview.
-			if ( Give_Email_Notification_Util::is_email_preview_has_header( $email ) ) {
-				add_action( "give_{$email->config['id']}_email_preview", array( $this, 'email_preview_header' ) );
-				add_filter( "give_{$email->config['id']}_email_preview_data", array( $this, 'email_preview_data' ) );
-				add_filter( "give_{$email->config['id']}_email_preview_message", array( $this, 'email_preview_message' ), 1, 2 );
+			if (Give_Email_Notification_Util::is_email_preview_has_header($email)) {
+				add_action("give_{$email->config['id']}_email_preview", array($this, 'email_preview_header'));
+				add_filter("give_{$email->config['id']}_email_preview_data", array($this, 'email_preview_data'));
+				add_filter("give_{$email->config['id']}_email_preview_message", array($this, 'email_preview_message'), 1, 2);
 			}
 		}
 	}
@@ -107,63 +107,63 @@ 
 	 *
 	 * @return array
 	 */
-	public function add_metabox_setting_fields( $settings, $post_id ) {
+	public function add_metabox_setting_fields($settings, $post_id) {
 		$emails = $this->get_email_notifications();
 
 		// Bailout.
-		if ( empty( $emails ) ) {
+		if (empty($emails)) {
 			return $settings;
 		}
 
 		// Email notification setting.
 		$settings['email_notification_options'] = array(
 			'id'         => 'email_notification_options',
-			'title'      => __( 'Email Notification', 'give' ),
+			'title'      => __('Email Notification', 'give'),
 			'fields'     => array(
 				array(
-					'name'        => __( 'Email Options', 'give' ),
-					'description' => __( '', 'give' ),
+					'name'        => __('Email Options', 'give'),
+					'description' => __('', 'give'),
 					'id'          => '_give_email_options',
 					'type'        => 'radio_inline',
 					'default'     => 'global',
 					'options'     => array(
-						'global'   => __( 'Global Options' ),
-						'enabled'  => __( 'Customize', 'give' ),
+						'global'   => __('Global Options'),
+						'enabled'  => __('Customize', 'give'),
 					),
 				),
 				array(
 					'id'      => '_give_email_template',
-					'name'    => esc_html__( 'Email Template', 'give' ),
-					'desc'    => esc_html__( 'Choose your template from the available registered template types.', 'give' ),
+					'name'    => esc_html__('Email Template', 'give'),
+					'desc'    => esc_html__('Choose your template from the available registered template types.', 'give'),
 					'type'    => 'select',
 					'default' => 'default',
 					'options' => give_get_email_templates(),
 				),
 				array(
 					'id'   => '_give_email_logo',
-					'name' => esc_html__( 'Logo', 'give' ),
-					'desc' => esc_html__( 'Upload or choose a logo to be displayed at the top of the donation receipt emails. Displayed on HTML emails only.', 'give' ),
+					'name' => esc_html__('Logo', 'give'),
+					'desc' => esc_html__('Upload or choose a logo to be displayed at the top of the donation receipt emails. Displayed on HTML emails only.', 'give'),
 					'type' => 'file',
 				),
 				array(
 					'id'      => '_give_from_name',
-					'name'    => esc_html__( 'From Name', 'give' ),
-					'desc'    => esc_html__( 'The name which appears in the "From" field in all Give donation emails.', 'give' ),
-					'default' => get_bloginfo( 'name' ),
+					'name'    => esc_html__('From Name', 'give'),
+					'desc'    => esc_html__('The name which appears in the "From" field in all Give donation emails.', 'give'),
+					'default' => get_bloginfo('name'),
 					'type'    => 'text',
 				),
 				array(
 					'id'      => '_give_from_email',
-					'name'    => esc_html__( 'From Email', 'give' ),
-					'desc'    => esc_html__( 'Email address from which all Give emails are sent from. This will act as the "from" and "reply-to" email address.', 'give' ),
-					'default' => get_bloginfo( 'admin_email' ),
+					'name'    => esc_html__('From Email', 'give'),
+					'desc'    => esc_html__('Email address from which all Give emails are sent from. This will act as the "from" and "reply-to" email address.', 'give'),
+					'default' => get_bloginfo('admin_email'),
 					'type'    => 'text',
 				),
 				array(
 					'name'  => 'email_notification_docs',
 					'type'  => 'docs_link',
 					'url'   => 'http://docs.givewp.com/email-notification',
-					'title' => __( 'Email Notification', 'give' ),
+					'title' => __('Email Notification', 'give'),
 				),
 			),
 
@@ -172,7 +172,7 @@ 
 			 *
 			 * @since 2.0
 			 */
-			'sub-fields' => apply_filters( 'give_email_notification_options_metabox_fields', array(), $post_id ),
+			'sub-fields' => apply_filters('give_email_notification_options_metabox_fields', array(), $post_id),
 		);
 
 		return $settings;
@@ -186,13 +186,13 @@ 
 	 */
 	private function add_emails_notifications() {
 		$this->emails = array(
-			include GIVE_PLUGIN_DIR . 'includes/admin/emails/class-new-donation-email.php',
-			include GIVE_PLUGIN_DIR . 'includes/admin/emails/class-donation-receipt-email.php',
-			include GIVE_PLUGIN_DIR . 'includes/admin/emails/class-new-offline-donation-email.php',
-			include GIVE_PLUGIN_DIR . 'includes/admin/emails/class-offline-donation-instruction-email.php',
-			include GIVE_PLUGIN_DIR . 'includes/admin/emails/class-new-donor-register-email.php',
-			include GIVE_PLUGIN_DIR . 'includes/admin/emails/class-donor-register-email.php',
-			include GIVE_PLUGIN_DIR . 'includes/admin/emails/class-email-access-email.php',
+			include GIVE_PLUGIN_DIR.'includes/admin/emails/class-new-donation-email.php',
+			include GIVE_PLUGIN_DIR.'includes/admin/emails/class-donation-receipt-email.php',
+			include GIVE_PLUGIN_DIR.'includes/admin/emails/class-new-offline-donation-email.php',
+			include GIVE_PLUGIN_DIR.'includes/admin/emails/class-offline-donation-instruction-email.php',
+			include GIVE_PLUGIN_DIR.'includes/admin/emails/class-new-donor-register-email.php',
+			include GIVE_PLUGIN_DIR.'includes/admin/emails/class-donor-register-email.php',
+			include GIVE_PLUGIN_DIR.'includes/admin/emails/class-email-access-email.php',
 		);
 
 		/**
@@ -200,15 +200,15 @@ 
 		 *
 		 * @since 2.0
 		 */
-		$this->emails = apply_filters( 'give_email_notifications', $this->emails, $this );
+		$this->emails = apply_filters('give_email_notifications', $this->emails, $this);
 
 		// Bailout.
-		if ( empty( $this->emails ) ) {
+		if (empty($this->emails)) {
 			return;
 		}
 
 		// Initiate email notifications.
-		foreach ( $this->emails as $email ) {
+		foreach ($this->emails as $email) {
 			$email->init();
 		}
 	}
@@ -235,41 +235,41 @@ 
 	 */
 	public function preview_email() {
 		// Bailout.
-		if ( ! Give_Email_Notification_Util::can_preview_email() ) {
+		if ( ! Give_Email_Notification_Util::can_preview_email()) {
 			return false;
 		}
 
 		// Security check.
-		give_validate_nonce( $_GET['_wpnonce'], 'give-preview-email' );
+		give_validate_nonce($_GET['_wpnonce'], 'give-preview-email');
 
 		// Get email type.
-		$email_type = isset( $_GET['email_type'] ) ? esc_attr( $_GET['email_type'] ) : '';
+		$email_type = isset($_GET['email_type']) ? esc_attr($_GET['email_type']) : '';
 
 		/* @var Give_Email_Notification $email */
-		foreach ( $this->get_email_notifications() as $email ) {
-			if ( $email_type !== $email->config['id'] ) {
+		foreach ($this->get_email_notifications() as $email) {
+			if ($email_type !== $email->config['id']) {
 				continue;
 			}
 
 			// Set form id.
-			$form_id = empty( $_GET['form_id']  ) ? null : absint( $_GET['form_id'] );
+			$form_id = empty($_GET['form_id']) ? null : absint($_GET['form_id']);
 
 			// Call setup email data to apply filter and other thing to email.
 			$email->setup_email_data();
 
 			// Decode message.
-			$email_message = $email->preview_email_template_tags( $email->get_email_message( $form_id ) );
+			$email_message = $email->preview_email_template_tags($email->get_email_message($form_id));
 
 			// Set email template.
-			Give()->emails->html    = true;
-			Give()->emails->__set( 'template', $email->get_email_template( $form_id ) );
+			Give()->emails->html = true;
+			Give()->emails->__set('template', $email->get_email_template($form_id));
 
-			if ( 'text/plain' === $email->config['content_type'] ) {
+			if ('text/plain' === $email->config['content_type']) {
 				// Give()->emails->__set( 'html', false );
-				Give()->emails->__set( 'template', 'none' );
+				Give()->emails->__set('template', 'none');
 			}
 
-			if ( $email_message = Give()->emails->build_email( $email_message ) ) {
+			if ($email_message = Give()->emails->build_email($email_message)) {
 
 				/**
 				 * Filter the email preview data
@@ -278,14 +278,14 @@ 
 				 *
 				 * @param array
 				 */
-				$email_preview_data = apply_filters( "give_{$email_type}_email_preview_data", array() );
+				$email_preview_data = apply_filters("give_{$email_type}_email_preview_data", array());
 
 				/**
 				 * Fire the give_{$email_type}_email_preview action
 				 *
 				 * @since 2.0
 				 */
-				do_action( "give_{$email_type}_email_preview", $email );
+				do_action("give_{$email_type}_email_preview", $email);
 
 				/**
 				 * Filter the email message
@@ -296,7 +296,7 @@ 
 				 * @param array                   $email_preview_data
 				 * @param Give_Email_Notification $email
 				 */
-				echo apply_filters( "give_{$email_type}_email_preview_message", $email_message, $email_preview_data, $email );
+				echo apply_filters("give_{$email_type}_email_preview_message", $email_message, $email_preview_data, $email);
 
 				exit();
 			}
@@ -312,7 +312,7 @@ 
 	 *
 	 * @param Give_Email_Notification $email
 	 */
-	public function email_preview_header( $email ) {
+	public function email_preview_header($email) {
 		/**
 		 * Filter the all email preview headers.
 		 *
@@ -320,7 +320,7 @@ 
 		 *
 		 * @param Give_Email_Notification $email
 		 */
-		$email_preview_header = apply_filters( 'give_email_preview_header', give_get_preview_email_header(), $email );
+		$email_preview_header = apply_filters('give_email_preview_header', give_get_preview_email_header(), $email);
 
 		echo $email_preview_header;
 	}
@@ -335,9 +335,9 @@ 
 	 *
 	 * @return array
 	 */
-	public function email_preview_data( $email_preview_data ) {
-		$email_preview_data['payment_id'] = absint( give_check_variable( give_clean( $_GET ), 'isset', 0, 'preview_id' ) );
-		$email_preview_data['user_id']    = absint( give_check_variable( give_clean( $_GET ), 'isset', 0, 'user_id' ) );
+	public function email_preview_data($email_preview_data) {
+		$email_preview_data['payment_id'] = absint(give_check_variable(give_clean($_GET), 'isset', 0, 'preview_id'));
+		$email_preview_data['user_id']    = absint(give_check_variable(give_clean($_GET), 'isset', 0, 'user_id'));
 
 		return $email_preview_data;
 	}
@@ -353,12 +353,12 @@ 
 	 *
 	 * @return string
 	 */
-	public function email_preview_message( $email_message, $email_preview_data ) {
+	public function email_preview_message($email_message, $email_preview_data) {
 		if (
-			! empty( $email_preview_data['payment_id'] )
-			|| ! empty( $email_preview_data['user_id'] )
+			! empty($email_preview_data['payment_id'])
+			|| ! empty($email_preview_data['user_id'])
 		) {
-			$email_message = give_do_email_tags( $email_message, $email_preview_data );
+			$email_message = give_do_email_tags($email_message, $email_preview_data);
 		}
 
 		return $email_message;
@@ -373,26 +373,26 @@ 
 	 */
 	public function send_preview_email() {
 		// Bailout.
-		if ( ! Give_Email_Notification_Util::can_send_preview_email() ) {
+		if ( ! Give_Email_Notification_Util::can_send_preview_email()) {
 			return false;
 		}
 
 		// Security check.
-		give_validate_nonce( $_GET['_wpnonce'], 'give-send-preview-email' );
+		give_validate_nonce($_GET['_wpnonce'], 'give-send-preview-email');
 
 		// Get email type.
-		$email_type = give_check_variable( give_clean( $_GET ), 'isset', '', 'email_type' );
+		$email_type = give_check_variable(give_clean($_GET), 'isset', '', 'email_type');
 
 		/* @var Give_Email_Notification $email */
-		foreach ( $this->get_email_notifications() as $email ) {
-			if ( $email_type === $email->config['id'] && Give_Email_Notification_Util::is_email_preview( $email ) ) {
+		foreach ($this->get_email_notifications() as $email) {
+			if ($email_type === $email->config['id'] && Give_Email_Notification_Util::is_email_preview($email)) {
 				$email->send_preview_email();
 				break;
 			}
 		}
 
 		// Remove the test email query arg.
-		wp_redirect( remove_query_arg( 'give_action' ) );
+		wp_redirect(remove_query_arg('give_action'));
 		exit;
 	}
 
@@ -404,7 +404,7 @@ 
 	 * @access public
 	 */
 	public function load() {
-		add_action( 'init', array( $this, 'init' ), -1 );
+		add_action('init', array($this, 'init'), -1);
 	}
 
 
@@ -419,7 +419,7 @@ 
 		if (
 			! Give_Admin_Settings::is_saving_settings() ||
 			'emails' !== give_get_current_setting_tab() ||
-			! isset( $_GET['section'] )
+			! isset($_GET['section'])
 		) {
 			return;
 		}
@@ -427,18 +427,18 @@ 
 		// Get email type.
 		$email_type = give_get_current_setting_section();
 
-		if ( ! empty( $_POST["{$email_type}_recipient"] ) ) {
-			$_POST["{$email_type}_recipient"] = array_unique( array_filter( $_POST["{$email_type}_recipient"] ) );
+		if ( ! empty($_POST["{$email_type}_recipient"])) {
+			$_POST["{$email_type}_recipient"] = array_unique(array_filter($_POST["{$email_type}_recipient"]));
 		}
 	}
 }
 
 // Helper class.
-require_once GIVE_PLUGIN_DIR . 'includes/admin/emails/abstract-email-notification.php';
-require_once GIVE_PLUGIN_DIR . 'includes/admin/emails/class-email-notification-util.php';
+require_once GIVE_PLUGIN_DIR.'includes/admin/emails/abstract-email-notification.php';
+require_once GIVE_PLUGIN_DIR.'includes/admin/emails/class-email-notification-util.php';
 
 // Add backward compatibility.
-require_once GIVE_PLUGIN_DIR . 'includes/admin/emails/backward-compatibility.php';
+require_once GIVE_PLUGIN_DIR.'includes/admin/emails/backward-compatibility.php';
 
 /**
  * Initialize functionality.

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

Doc Comments +3 added lines, -3 removed lines

@@ -528,7 +528,7 @@ 
 	/**
 	 * Get donor email html.
 	 *
-	 * @param object $payment Contains all the data of the payment.
+	 * @param Give_Payment $payment Contains all the data of the payment.
 	 *
 	 * @access public
 	 * @since  1.0
@@ -556,7 +556,7 @@ 
 	/**
 	 * Get Row Actions
 	 *
-	 * @param object $payment Payment Data.
+	 * @param Give_Payment $payment Payment Data.
 	 *
 	 * @since 1.6
 	 *
@@ -677,7 +677,7 @@ 
 	/**
 	 * Get donor html.
 	 *
-	 * @param object $payment Contains all the data of the payment.
+	 * @param Give_Payment $payment Contains all the data of the payment.
 	 *
 	 * @access public
 	 * @since  1.0

Spacing +216 added lines, -217 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';
 }
 
 /**
@@ -136,15 +136,15 @@ 
 		// 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.
-				'ajax'     => false,                              // Does this table support ajax?
+				'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->process_bulk_action();
 		$this->get_payment_counts();
-		$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');
 	}
 
 	/**
@@ -153,31 +153,31 @@ 
 	 * @return void
 	 */
 	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'] ) ? sanitize_text_field( $_GET['status'] ) : '';
-		$donor      = isset( $_GET['donor'] ) ? sanitize_text_field( $_GET['donor'] ) : '';
-		$search     = isset( $_GET['s'] ) ? sanitize_text_field( $_GET['s'] ) : '';
-		$form_id    = ! empty( $_GET['form_id'] ) ? absint( $_GET['form_id'] ) : 0;
+		$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']) ? sanitize_text_field($_GET['status']) : '';
+		$donor      = isset($_GET['donor']) ? sanitize_text_field($_GET['donor']) : '';
+		$search     = isset($_GET['s']) ? sanitize_text_field($_GET['s']) : '';
+		$form_id    = ! empty($_GET['form_id']) ? absint($_GET['form_id']) : 0;
 		?>
 		<div id="give-payment-filters" class="give-filters">
-			<?php $this->search_box( __( 'Search', 'give' ), 'give-payments' ); ?>
+			<?php $this->search_box(__('Search', 'give'), 'give-payments'); ?>
 			<div id="give-payment-date-filters">
 				<div class="give-filter give-filter-half">
 					<label for="start-date"
-					       class="give-start-date-label"><?php _e( 'Start Date', 'give' ); ?></label>
+					       class="give-start-date-label"><?php _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"/>
 				</div>
 				<div class="give-filter give-filter-half">
-					<label for="end-date" class="give-end-date-label"><?php _e( 'End Date', 'give' ); ?></label>
+					<label for="end-date" class="give-end-date-label"><?php _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"/>
 				</div>
 			</div>
 			<div id="give-payment-form-filter" class="give-filter">
 				<label for="give-donation-forms-filter"
-				       class="give-donation-forms-filter-label"><?php _e( 'Form', 'give' ); ?></label>
+				       class="give-donation-forms-filter-label"><?php _e('Form', 'give'); ?></label>
 				<?php
 				// Filter Donations by Donation Forms.
 				echo Give()->html->forms_dropdown(
@@ -187,7 +187,7 @@ 
 						'class'    => 'give-donation-forms-filter',
 						'selected' => $form_id, // Make sure to have $form_id set to 0, if there is no selection.
 						'chosen'   => true,
-						'number'   => - 1,
+						'number'   => -1,
 					)
 				);
 				?>
@@ -199,21 +199,21 @@ 
 			 *
 			 * @since 1.8.18
 			 */
-			do_action( 'give_payment_table_advanced_filters' );
+			do_action('give_payment_table_advanced_filters');
 			?>
 
-			<?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; ?>
 
 			<div class="give-filter">
-				<?php submit_button( __( 'Apply', 'give' ), 'secondary', '', false ); ?>
+				<?php submit_button(__('Apply', 'give'), 'secondary', '', false); ?>
 				<?php
 				// Clear active filters button.
-				if ( ! empty( $start_date ) || ! empty( $end_date ) || ! empty( $donor ) || ! empty( $search ) || ! empty( $status ) || ! empty( $form_id ) ) :
+				if ( ! empty($start_date) || ! empty($end_date) || ! empty($donor) || ! empty($search) || ! empty($status) || ! empty($form_id)) :
 					?>
-					<a href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-payment-history' ); ?>"
-					   class="button give-clear-filters-button"><?php _e( 'Clear Filters', 'give' ); ?></a>
+					<a href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-payment-history'); ?>"
+					   class="button give-clear-filters-button"><?php _e('Clear Filters', 'give'); ?></a>
 				<?php endif; ?>
 			</div>
 		</div>
@@ -232,18 +232,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']).'" />';
 		}
 		?>
 		<div class="give-filter give-filter-search" role="search">
@@ -255,13 +255,13 @@ 
 			 *
 			 * @since 1.7
 			 */
-			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(
+			<?php submit_button($text, 'button', false, false, array(
 				'ID' => 'search-submit',
-			) ); ?><br/>
+			)); ?><br/>
 		</div>
 		<?php
 	}
@@ -276,48 +276,48 @@ 
 	 */
 	public function get_views() {
 
-		$current = isset( $_GET['status'] ) ? $_GET['status'] : '';
+		$current = isset($_GET['status']) ? $_GET['status'] : '';
 		$views   = array();
 		$tabs    = array(
 			'all'         => array(
 				'total_count',
-				__( 'All', 'give' ),
+				__('All', 'give'),
 			),
 			'publish'     => array(
 				'complete_count',
-				__( 'Completed', 'give' ),
+				__('Completed', 'give'),
 			),
 			'pending'     => array(
 				'pending_count',
-				__( 'Pending', 'give' ),
+				__('Pending', 'give'),
 			),
 			'processing'  => array(
 				'processing_count',
-				__( 'Processing', 'give' ),
+				__('Processing', 'give'),
 			),
 			'refunded'    => array(
 				'refunded_count',
-				__( 'Refunded', 'give' ),
+				__('Refunded', 'give'),
 			),
 			'revoked'     => array(
 				'revoked_count',
-				__( 'Revoked', 'give' ),
+				__('Revoked', 'give'),
 			),
 			'failed'      => array(
 				'failed_count',
-				__( 'Failed', 'give' ),
+				__('Failed', 'give'),
 			),
 			'cancelled'   => array(
 				'cancelled_count',
-				__( 'Cancelled', 'give' ),
+				__('Cancelled', 'give'),
 			),
 			'abandoned'   => array(
 				'abandoned_count',
-				__( 'Abandoned', 'give' ),
+				__('Abandoned', 'give'),
 			),
 			'preapproval' => array(
 				'preapproval_count',
-				__( 'Preapproval Pending', 'give' ),
+				__('Preapproval Pending', 'give'),
 			),
 		);
 
@@ -326,12 +326,12 @@ 
 		 *
 		 * @since 1.8.18
 		 */
-		$args = (array) apply_filters( 'give_payments_table_status_remove_query_arg', array( 'paged', '_wpnonce', '_wp_http_referer' ) );
+		$args = (array) apply_filters('give_payments_table_status_remove_query_arg', array('paged', '_wpnonce', '_wp_http_referer'));
 
 		// Build URL.
-		$staus_url = remove_query_arg( $args );
+		$staus_url = remove_query_arg($args);
 
-		foreach ( $tabs as $key => $tab ) {
+		foreach ($tabs as $key => $tab) {
 			$count_key = $tab[0];
 			$name      = $tab[1];
 			$count     = $this->$count_key;
@@ -346,16 +346,15 @@ 
 			 *
 			 * @since 1.8.12
 			 */
-			if ( 'all' === $key || $key === $current || apply_filters( 'give_payments_table_show_all_status', 0 < $count, $key, $count ) ) {
+			if ('all' === $key || $key === $current || apply_filters('give_payments_table_show_all_status', 0 < $count, $key, $count)) {
 
 				$staus_url = 'all' === $key ?
-					add_query_arg( array( 'status' => false ), $staus_url ) :
-					add_query_arg( array( 'status' => $key ), $staus_url );
+					add_query_arg(array('status' => false), $staus_url) : add_query_arg(array('status' => $key), $staus_url);
 
-				$views[ $key ] = sprintf(
+				$views[$key] = sprintf(
 					'<a href="%s"%s>%s&nbsp;<span class="count">(%s)</span></a>',
-					esc_url( $staus_url ),
-					( ( 'all' === $key && empty( $current ) ) ) ? ' class="current"' : ( $current == $key ? 'class="current"' : '' ),
+					esc_url($staus_url),
+					(('all' === $key && empty($current))) ? ' class="current"' : ($current == $key ? 'class="current"' : ''),
 					$name,
 					$count
 				);
@@ -370,7 +369,7 @@ 
 		 * @param array $views
 		 * @param Give_Payment_History_Table
 		 */
-		return apply_filters( 'give_payments_table_views', $views, $this );
+		return apply_filters('give_payments_table_views', $views, $this);
 	}
 
 	/**
@@ -384,18 +383,18 @@ 
 	public function get_columns() {
 		$columns = array(
 			'cb'            => '<input type="checkbox" />', // Render a checkbox instead of text.
-			'donation'      => __( 'Donation', 'give' ),
-			'donation_form' => __( 'Donation Form', 'give' ),
-			'status'        => __( 'Status', 'give' ),
-			'date'          => __( 'Date', 'give' ),
-			'amount'        => __( 'Amount', 'give' ),
+			'donation'      => __('Donation', 'give'),
+			'donation_form' => __('Donation Form', 'give'),
+			'status'        => __('Status', 'give'),
+			'date'          => __('Date', 'give'),
+			'amount'        => __('Amount', 'give'),
 		);
 
-		if ( current_user_can( 'view_give_payments' ) ) {
-			$columns['details'] = __( 'Details', 'give' );
+		if (current_user_can('view_give_payments')) {
+			$columns['details'] = __('Details', 'give');
 		}
 
-		return apply_filters( 'give_payments_table_columns', $columns );
+		return apply_filters('give_payments_table_columns', $columns);
 	}
 
 	/**
@@ -408,14 +407,14 @@ 
 	 */
 	public function get_sortable_columns() {
 		$columns = array(
-			'donation'      => array( 'ID', true ),
-			'donation_form' => array( 'donation_form', false ),
-			'status'        => array( 'status', false ),
-			'amount'        => array( 'amount', false ),
-			'date'          => array( 'date', false ),
+			'donation'      => array('ID', true),
+			'donation_form' => array('donation_form', false),
+			'status'        => array('status', false),
+			'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);
 	}
 
 	/**
@@ -441,43 +440,43 @@ 
 	 *
 	 * @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-payment-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-payment-details')));
+		$row_actions         = $this->get_row_actions($payment);
 		$value               = '';
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 			case 'donation' :
-				if ( current_user_can( 'view_give_payments' ) ) {
-					$value = Give()->tooltips->render_link( array(
-						'label'       => sprintf( __( 'View Donation #%s', 'give' ), $payment->ID ),
+				if (current_user_can('view_give_payments')) {
+					$value = Give()->tooltips->render_link(array(
+						'label'       => sprintf(__('View Donation #%s', 'give'), $payment->ID),
 						'tag_content' => "#$payment->ID",
 						'link'        => $single_donation_url,
-					) );
+					));
 				} else {
 					$value = "#{$payment->ID}";
 				}
 
 				$value .= sprintf(
 					'&nbsp;%1$s&nbsp;%2$s<br>',
-					__( 'by', 'give' ),
-					$this->get_donor( $payment )
+					__('by', 'give'),
+					$this->get_donor($payment)
 				);
 
-				$value .= $this->get_donor_email( $payment );
-				$value .= $this->row_actions( $row_actions );
+				$value .= $this->get_donor_email($payment);
+				$value .= $this->row_actions($row_actions);
 				break;
 
 			case 'amount':
-				$amount = ! empty( $payment->total ) ? $payment->total : 0;
-				$value  = give_donation_amount( $payment, true );
-				$value .= sprintf( '<br><small>%1$s %2$s</small>', __( 'via', 'give' ), give_get_gateway_admin_label( $payment->gateway ) );
+				$amount = ! empty($payment->total) ? $payment->total : 0;
+				$value  = give_donation_amount($payment, true);
+				$value .= sprintf('<br><small>%1$s %2$s</small>', __('via', 'give'), give_get_gateway_admin_label($payment->gateway));
 				break;
 
 			case 'donation_form':
-				$form_title = empty( $payment->form_title ) ? sprintf( __( 'Untitled (#%s)', 'give' ), $payment->form_id ) : $payment->form_title;
-				$value      = '<a href="' . admin_url( 'post.php?post=' . $payment->form_id . '&action=edit' ) . '">' . $form_title . '</a>';
+				$form_title = empty($payment->form_title) ? sprintf(__('Untitled (#%s)', 'give'), $payment->form_id) : $payment->form_title;
+				$value      = '<a href="'.admin_url('post.php?post='.$payment->form_id.'&action=edit').'">'.$form_title.'</a>';
 				$level      = give_get_donation_form_title(
 					$payment,
 					array(
@@ -485,44 +484,44 @@ 
 					)
 				);
 
-				if ( ! empty( $level ) ) {
+				if ( ! empty($level)) {
 					$value .= $level;
 				}
 
 				break;
 
 			case 'date':
-				$date  = strtotime( $payment->date );
-				$value = date_i18n( give_date_format(), $date );
+				$date  = strtotime($payment->date);
+				$value = date_i18n(give_date_format(), $date);
 				break;
 
 			case 'status':
-				$value = $this->get_payment_status( $payment );
+				$value = $this->get_payment_status($payment);
 				break;
 
 
 			case 'details' :
-				if ( current_user_can( 'view_give_payments' ) ) {
-					$value = Give()->tooltips->render_link( array(
-						'label'       => sprintf( __( 'View Donation #%s', 'give' ), $payment->ID ),
+				if (current_user_can('view_give_payments')) {
+					$value = Give()->tooltips->render_link(array(
+						'label'       => sprintf(__('View Donation #%s', 'give'), $payment->ID),
 						'tag_content' => '<span class="dashicons dashicons-visibility"></span>',
 						'link'        => $single_donation_url,
 						'attributes'  => array(
 							'class' => 'give-payment-details-link button button-small',
 						),
-					) );
+					));
 
 					$value = "<div class=\"give-payment-details-link-wrap\">{$value}</div>";
 				}
 				break;
 
 			default:
-				$value = isset( $payment->$column_name ) ? $payment->$column_name : '';
+				$value = isset($payment->$column_name) ? $payment->$column_name : '';
 				break;
 
 		}// End switch().
 
-		return apply_filters( 'give_payments_table_column', $value, $payment->ID, $column_name );
+		return apply_filters('give_payments_table_column', $value, $payment->ID, $column_name);
 	}
 
 	/**
@@ -535,22 +534,22 @@ 
 	 *
 	 * @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 = __( '(unknown)', 'give' );
+		if (empty($email)) {
+			$email = __('(unknown)', 'give');
 		}
 
 
-		$value = Give()->tooltips->render_link( array(
+		$value = Give()->tooltips->render_link(array(
 			'link'        => "mailto:{$email}",
-			'label'       => __( 'Email donor', 'give' ),
+			'label'       => __('Email donor', 'give'),
 			'tag_content' => $email,
-		) );
+		));
 
-		return apply_filters( 'give_payments_table_column', $value, $payment->ID, 'email' );
+		return apply_filters('give_payments_table_column', $value, $payment->ID, 'email');
 	}
 
 	/**
@@ -562,18 +561,18 @@ 
 	 *
 	 * @return array $actions
 	 */
-	function get_row_actions( $payment ) {
+	function get_row_actions($payment) {
 
 		$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 ) ) {
+		if (give_is_payment_complete($payment->ID) && ! empty($email)) {
 
 			$actions['email_links'] = sprintf(
 				'<a class="resend-single-donation-receipt" href="%1$s" aria-label="%2$s">%3$s</a>', wp_nonce_url(
@@ -583,12 +582,12 @@ 
 						'purchase_id' => $payment->ID,
 					), $this->base_url
 				), 'give_payment_nonce'
-			), sprintf( __( 'Resend Donation %s Receipt', 'give' ), $payment->ID ), __( 'Resend Receipt', 'give' )
+			), sprintf(__('Resend Donation %s Receipt', 'give'), $payment->ID), __('Resend Receipt', 'give')
 			);
 
 		}
 
-		if ( current_user_can( 'view_give_payments' ) ) {
+		if (current_user_can('view_give_payments')) {
 			$actions['delete'] = sprintf(
 				'<a class="delete-single-donation" href="%1$s" aria-label="%2$s">%3$s</a>',
 				wp_nonce_url(
@@ -598,11 +597,11 @@ 
 							'purchase_id' => $payment->ID,
 						), $this->base_url
 					), 'give_donation_nonce'
-				), sprintf( __( 'Delete Donation %s', 'give' ), $payment->ID ), __( 'Delete', 'give' )
+				), sprintf(__('Delete Donation %s', 'give'), $payment->ID), __('Delete', 'give')
 			);
 		}
 
-		return apply_filters( 'give_payment_row_actions', $actions, $payment );
+		return apply_filters('give_payment_row_actions', $actions, $payment);
 	}
 
 
@@ -616,30 +615,30 @@ 
 	 *
 	 * @return string Data shown in the Email column
 	 */
-	function get_payment_status( $payment ) {
+	function get_payment_status($payment) {
 		$value = sprintf(
 			'<div class="give-donation-status status-%1$s"><span class="give-donation-status-icon"></span>&nbsp;%2$s</div>',
-			sanitize_title( give_get_payment_status( $payment, true ) ),
-			give_get_payment_status( $payment, true )
+			sanitize_title(give_get_payment_status($payment, true)),
+			give_get_payment_status($payment, true)
 		);
 
-		if ( $payment->mode == 'test' ) {
-			$value .= Give()->tooltips->render_span( array(
-				'label'       => __( 'This donation was made in test mode.', 'give' ),
-				'tag_content' => __( 'Test', 'give' ),
+		if ($payment->mode == 'test') {
+			$value .= Give()->tooltips->render_span(array(
+				'label'       => __('This donation was made in test mode.', 'give'),
+				'tag_content' => __('Test', 'give'),
 				'attributes'  => array(
 					'class' => 'give-item-label give-item-label-orange give-test-mode-transactions-label',
 				),
 
 
-			) );
+			));
 		}
 
-		if ( true === $payment->import && true === (bool) apply_filters( 'give_payment_show_importer_label', false ) ) {
+		if (true === $payment->import && true === (bool) apply_filters('give_payment_show_importer_label', false)) {
 			$value .= sprintf(
 				'&nbsp;<span class="give-item-label give-item-label-orange give-test-mode-transactions-label" data-tooltip="%1$s">%2$s</span>',
-				__( 'This donation was imported.', 'give' ),
-				__( 'Import', 'give' )
+				__('This donation was imported.', 'give'),
+				__('Import', 'give')
 			);
 		}
 
@@ -656,8 +655,8 @@ 
 	 *
 	 * @return string Displays a checkbox.
 	 */
-	public function column_cb( $payment ) {
-		return sprintf( '<input type="checkbox" name="%1$s[]" value="%2$s" />', 'payment', $payment->ID );
+	public function column_cb($payment) {
+		return sprintf('<input type="checkbox" name="%1$s[]" value="%2$s" />', 'payment', $payment->ID);
 	}
 
 	/**
@@ -670,8 +669,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>';
 	}
 
 	/**
@@ -684,32 +683,32 @@ 
 	 *
 	 * @return string Data shown in the User column
 	 */
-	public function get_donor( $payment ) {
+	public function get_donor($payment) {
 
-		$donor_id           = give_get_payment_donor_id( $payment->ID );
-		$donor_billing_name = give_get_donor_name_by( $payment->ID, 'donation' );
-		$donor_name         = give_get_donor_name_by( $donor_id, 'donor' );
+		$donor_id           = give_get_payment_donor_id($payment->ID);
+		$donor_billing_name = give_get_donor_name_by($payment->ID, 'donation');
+		$donor_name         = give_get_donor_name_by($donor_id, 'donor');
 
 		$value = '';
-		if ( ! empty( $donor_id ) ) {
+		if ( ! empty($donor_id)) {
 
 			// Check whether the donor name and WP_User name is same or not.
-			if ( sanitize_title( $donor_billing_name ) !== sanitize_title( $donor_name ) ) {
-				$value .= $donor_billing_name . ' (';
+			if (sanitize_title($donor_billing_name) !== sanitize_title($donor_name)) {
+				$value .= $donor_billing_name.' (';
 			}
 
-			$value .= '<a href="' . esc_url( admin_url( "edit.php?post_type=give_forms&page=give-donors&view=overview&id=$donor_id" ) ) . '">' . $donor_name . '</a>';
+			$value .= '<a href="'.esc_url(admin_url("edit.php?post_type=give_forms&page=give-donors&view=overview&id=$donor_id")).'">'.$donor_name.'</a>';
 
 			// Check whether the donor name and WP_User name is same or not.
-			if ( sanitize_title( $donor_billing_name ) != sanitize_title( $donor_name ) ) {
+			if (sanitize_title($donor_billing_name) != sanitize_title($donor_name)) {
 				$value .= ')';
 			}
 		} 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" ) ) . '">' . __( '(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")).'">'.__('(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');
 	}
 
 	/**
@@ -722,20 +721,20 @@ 
 	 */
 	public function get_bulk_actions() {
 		$actions = array(
-			'delete'                 => __( 'Delete', 'give' ),
-			'set-status-publish'     => __( 'Set To Completed', 'give' ),
-			'set-status-pending'     => __( 'Set To Pending', 'give' ),
-			'set-status-processing'  => __( 'Set To Processing', 'give' ),
-			'set-status-refunded'    => __( 'Set To Refunded', 'give' ),
-			'set-status-revoked'     => __( 'Set To Revoked', 'give' ),
-			'set-status-failed'      => __( 'Set To Failed', 'give' ),
-			'set-status-cancelled'   => __( 'Set To Cancelled', 'give' ),
-			'set-status-abandoned'   => __( 'Set To Abandoned', 'give' ),
-			'set-status-preapproval' => __( 'Set To Preapproval', 'give' ),
-			'resend-receipt'         => __( 'Resend Email Receipts', 'give' ),
+			'delete'                 => __('Delete', 'give'),
+			'set-status-publish'     => __('Set To Completed', 'give'),
+			'set-status-pending'     => __('Set To Pending', 'give'),
+			'set-status-processing'  => __('Set To Processing', 'give'),
+			'set-status-refunded'    => __('Set To Refunded', 'give'),
+			'set-status-revoked'     => __('Set To Revoked', 'give'),
+			'set-status-failed'      => __('Set To Failed', 'give'),
+			'set-status-cancelled'   => __('Set To Cancelled', 'give'),
+			'set-status-abandoned'   => __('Set To Abandoned', 'give'),
+			'set-status-preapproval' => __('Set To Preapproval', 'give'),
+			'resend-receipt'         => __('Resend Email Receipts', 'give'),
 		);
 
-		return apply_filters( 'give_payments_table_bulk_actions', $actions );
+		return apply_filters('give_payments_table_bulk_actions', $actions);
 	}
 
 	/**
@@ -747,59 +746,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_donation( $id );
+					give_delete_donation($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-processing':
-					give_update_payment_status( $id, 'processing' );
+					give_update_payment_status($id, 'processing');
 					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':
@@ -808,7 +807,7 @@ 
 					 *
 					 * @since 2.0
 					 */
-					do_action( 'give_donation-receipt_email_notification', $id );
+					do_action('give_donation-receipt_email_notification', $id);
 					break;
 			}// End switch().
 
@@ -820,7 +819,7 @@ 
 			 *
 			 * @since 1.7
 			 */
-			do_action( 'give_payments_table_do_bulk_action', $id, $this->current_action() );
+			do_action('give_payments_table_do_bulk_action', $id, $this->current_action());
 		}// End foreach().
 
 	}
@@ -837,32 +836,32 @@ 
 
 		$args = array();
 
-		if ( isset( $_GET['user'] ) ) {
-			$args['user'] = urldecode( $_GET['user'] );
-		} elseif ( isset( $_GET['donor'] ) ) {
-			$args['donor'] = absint( $_GET['donor'] );
-		} 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['donor'])) {
+			$args['donor'] = absint($_GET['donor']);
+		} 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']);
 		}
 
-		$args['form_id'] = ! empty( $_GET['form_id'] ) ? absint( $_GET['form_id'] ) : null;
-		$args['gateway'] = ! empty( $_GET['gateway'] ) ? give_clean( $_GET['gateway'] ) : null;
+		$args['form_id'] = ! empty($_GET['form_id']) ? absint($_GET['form_id']) : null;
+		$args['gateway'] = ! empty($_GET['gateway']) ? give_clean($_GET['gateway']) : null;
 
-		$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->processing_count  = $payment_count->processing;
@@ -873,7 +872,7 @@ 
 		$this->abandoned_count   = $payment_count->abandoned;
 		$this->preapproval_count = $payment_count->preapproval;
 
-		foreach ( $payment_count as $count ) {
+		foreach ($payment_count as $count) {
 			$this->total_count += $count;
 		}
 
@@ -891,25 +890,25 @@ 
 	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;
-		$donor      = isset( $_GET['donor'] ) ? $_GET['donor'] : 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;
-		$form_id    = ! empty( $_GET['form_id'] ) ? absint( $_GET['form_id'] ) : null;
-		$gateway    = ! empty( $_GET['gateway'] ) ? give_clean( $_GET['gateway'] ) : null;
+		$orderby    = isset($_GET['orderby']) ? urldecode($_GET['orderby']) : 'ID';
+		$order      = isset($_GET['order']) ? $_GET['order'] : 'DESC';
+		$user       = isset($_GET['user']) ? $_GET['user'] : null;
+		$donor      = isset($_GET['donor']) ? $_GET['donor'] : 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;
+		$form_id    = ! empty($_GET['form_id']) ? absint($_GET['form_id']) : null;
+		$gateway    = ! empty($_GET['gateway']) ? give_clean($_GET['gateway']) : null;
 
 		$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,
@@ -926,9 +925,9 @@ 
 			'give_forms' => $form_id,
 		);
 
-		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']));
 		}
 
 		/**
@@ -936,9 +935,9 @@ 
 		 *
 		 * @since 1.8.18
 		 */
-		$args = (array) apply_filters( 'give_payment_table_payments_query', $args );
+		$args = (array) apply_filters('give_payment_table_payments_query', $args);
 
-		$p_query = new Give_Payments_Query( $args );
+		$p_query = new Give_Payments_Query($args);
 
 		return $p_query->get_payments();
 
@@ -959,17 +958,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;
@@ -1002,8 +1001,8 @@ 
 				break;
 			default:
 				// Retrieve the count of the non-default-Give status.
-				$count       = wp_count_posts( 'give_payment' );
-				$total_items = isset( $count->{$status} ) ? $count->{$status} : 0;
+				$count       = wp_count_posts('give_payment');
+				$total_items = isset($count->{$status} ) ? $count->{$status} : 0;
 				break;
 		}
 
@@ -1014,7 +1013,7 @@ 
 		 *
 		 * @since 1.8.19
 		 */
-		$total_items = (int) apply_filters( 'give_payment_table_pagination_total_count', $total_items, $this );
+		$total_items = (int) apply_filters('give_payment_table_pagination_total_count', $total_items, $this);
 
 		$this->set_pagination_args(
 			array(
@@ -1022,7 +1021,7 @@ 
 				// 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.
 			)
 		);