WordImpress / Give

includes/install.php

Doc Comments +1 added lines, -1 removed lines

@@ -397,7 +397,7 @@
  *
  * @since 1.8.11
  *
- * @return void
+ * @return false|null
  */
 function give_create_pages() {
 

Spacing +60 added lines, -60 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -28,15 +28,15 @@ 
  * @global     $wpdb
  * @return void
  */
-function give_install( $network_wide = false ) {
+function give_install($network_wide = false) {
 
 	global $wpdb;
 
-	if ( is_multisite() && $network_wide ) {
+	if (is_multisite() && $network_wide) {
 
-		foreach ( $wpdb->get_col( "SELECT blog_id FROM $wpdb->blogs LIMIT 100" ) as $blog_id ) {
+		foreach ($wpdb->get_col("SELECT blog_id FROM $wpdb->blogs LIMIT 100") as $blog_id) {
 
-			switch_to_blog( $blog_id );
+			switch_to_blog($blog_id);
 			give_run_install();
 			restore_current_blog();
 
@@ -63,31 +63,31 @@ 
 	give_setup_post_types();
 
 	// Add Upgraded From Option.
-	$current_version = get_option( 'give_version' );
-	if ( $current_version ) {
-		update_option( 'give_version_upgraded_from', $current_version );
+	$current_version = get_option('give_version');
+	if ($current_version) {
+		update_option('give_version_upgraded_from', $current_version);
 	}
 
 	// Setup some default options.
 	$options = array();
 
 	//Fresh Install? Setup Test Mode, Base Country (US), Test Gateway, Currency.
-	if ( empty( $current_version ) ) {
-		$options = array_merge( $options, give_get_default_settings() );
+	if (empty($current_version)) {
+		$options = array_merge($options, give_get_default_settings());
 	}
 
 	// Populate the default values.
-	update_option( 'give_settings', array_merge( $give_options, $options ) );
+	update_option('give_settings', array_merge($give_options, $options));
 
 	/**
 	 * Run plugin upgrades.
 	 *
 	 * @since 1.8
 	 */
-	do_action( 'give_upgrades' );
+	do_action('give_upgrades');
 
-	if ( GIVE_VERSION !== get_option( 'give_version' ) ) {
-		update_option( 'give_version', GIVE_VERSION );
+	if (GIVE_VERSION !== get_option('give_version')) {
+		update_option('give_version', GIVE_VERSION);
 	}
 
 	// Create Give roles.
@@ -98,7 +98,7 @@ 
 	// Set api version, end point and refresh permalink.
 	$api = new Give_API();
 	$api->add_endpoint();
-	update_option( 'give_default_api_version', 'v' . $api->get_version() );
+	update_option('give_default_api_version', 'v'.$api->get_version());
 
 	flush_rewrite_rules();
 
@@ -113,11 +113,11 @@ 
 	$give_sessions->use_php_sessions();
 
 	// Add a temporary option to note that Give pages have been created.
-	Give_Cache::set( '_give_installed', $options, 30, true );
+	Give_Cache::set('_give_installed', $options, 30, true);
 
-	if ( ! $current_version ) {
+	if ( ! $current_version) {
 
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/upgrade-functions.php';
+		require_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/upgrade-functions.php';
 
 		// When new upgrade routines are added, mark them as complete on fresh install.
 		$upgrade_routines = array(
@@ -150,18 +150,18 @@ 
 			'v213_delete_donation_meta'
 		);
 
-		foreach ( $upgrade_routines as $upgrade ) {
-			give_set_upgrade_complete( $upgrade );
+		foreach ($upgrade_routines as $upgrade) {
+			give_set_upgrade_complete($upgrade);
 		}
 	}
 
 	// Bail if activating from network, or bulk.
-	if ( is_network_admin() || isset( $_GET['activate-multi'] ) ) {
+	if (is_network_admin() || isset($_GET['activate-multi'])) {
 		return;
 	}
 
 	// Add the transient to redirect.
-	Give_Cache::set( '_give_activation_redirect', true, 30, true );
+	Give_Cache::set('_give_activation_redirect', true, 30, true);
 }
 
 /**
@@ -179,11 +179,11 @@ 
  * @param  int    $site_id The Site ID.
  * @param  array  $meta    Blog Meta.
  */
-function give_on_create_blog( $blog_id, $user_id, $domain, $path, $site_id, $meta ) {
+function give_on_create_blog($blog_id, $user_id, $domain, $path, $site_id, $meta) {
 
-	if ( is_plugin_active_for_network( GIVE_PLUGIN_BASENAME ) ) {
+	if (is_plugin_active_for_network(GIVE_PLUGIN_BASENAME)) {
 
-		switch_to_blog( $blog_id );
+		switch_to_blog($blog_id);
 		give_install();
 		restore_current_blog();
 
@@ -191,7 +191,7 @@ 
 
 }
 
-add_action( 'wpmu_new_blog', 'give_on_create_blog', 10, 6 );
+add_action('wpmu_new_blog', 'give_on_create_blog', 10, 6);
 
 
 /**
@@ -204,13 +204,13 @@ 
  *
  * @return array          The tables to drop.
  */
-function give_wpmu_drop_tables( $tables, $blog_id ) {
+function give_wpmu_drop_tables($tables, $blog_id) {
 
-	switch_to_blog( $blog_id );
+	switch_to_blog($blog_id);
 	$donors_db     = new Give_DB_Donors();
 	$donor_meta_db = new Give_DB_Donor_Meta();
 
-	if ( $donors_db->installed() ) {
+	if ($donors_db->installed()) {
 		$tables[] = $donors_db->table_name;
 		$tables[] = $donor_meta_db->table_name;
 	}
@@ -220,7 +220,7 @@ 
 
 }
 
-add_filter( 'wpmu_drop_tables', 'give_wpmu_drop_tables', 10, 2 );
+add_filter('wpmu_drop_tables', 'give_wpmu_drop_tables', 10, 2);
 
 /**
  * Post-installation
@@ -232,16 +232,16 @@ 
  */
 function give_after_install() {
 
-	if ( ! is_admin() ) {
+	if ( ! is_admin()) {
 		return;
 	}
 
-	$give_options     = Give_Cache::get( '_give_installed', true );
-	$give_table_check = get_option( '_give_table_check', false );
+	$give_options     = Give_Cache::get('_give_installed', true);
+	$give_table_check = get_option('_give_table_check', false);
 
-	if ( false === $give_table_check || current_time( 'timestamp' ) > $give_table_check ) {
+	if (false === $give_table_check || current_time('timestamp') > $give_table_check) {
 
-		if ( ! @Give()->donor_meta->installed() ) {
+		if ( ! @Give()->donor_meta->installed()) {
 
 			// Create the donor meta database.
 			// (this ensures it creates it on multisite instances where it is network activated).
@@ -249,7 +249,7 @@ 
 
 		}
 
-		if ( ! @Give()->donors->installed() ) {
+		if ( ! @Give()->donors->installed()) {
 			// Create the donor database.
 			// (this ensures it creates it on multisite instances where it is network activated).
 			@Give()->donors->create_table();
@@ -261,22 +261,22 @@ 
 			 *
 			 * @param array $give_options Give plugin options.
 			 */
-			do_action( 'give_after_install', $give_options );
+			do_action('give_after_install', $give_options);
 		}
 
-		update_option( '_give_table_check', ( current_time( 'timestamp' ) + WEEK_IN_SECONDS ) );
+		update_option('_give_table_check', (current_time('timestamp') + WEEK_IN_SECONDS));
 
 	}
 
 	// Delete the transient
-	if ( false !== $give_options ) {
-		Give_Cache::delete( Give_Cache::get_key( '_give_installed' ) );
+	if (false !== $give_options) {
+		Give_Cache::delete(Give_Cache::get_key('_give_installed'));
 	}
 
 
 }
 
-add_action( 'admin_init', 'give_after_install' );
+add_action('admin_init', 'give_after_install');
 
 
 /**
@@ -291,11 +291,11 @@ 
 
 	global $wp_roles;
 
-	if ( ! is_object( $wp_roles ) ) {
+	if ( ! is_object($wp_roles)) {
 		return;
 	}
 
-	if ( ! array_key_exists( 'give_manager', $wp_roles->roles ) ) {
+	if ( ! array_key_exists('give_manager', $wp_roles->roles)) {
 
 		// Create Give plugin roles
 		$roles = new Give_Roles();
@@ -306,7 +306,7 @@ 
 
 }
 
-add_action( 'admin_init', 'give_install_roles_on_network' );
+add_action('admin_init', 'give_install_roles_on_network');
 
 /**
  * Default core setting values.
@@ -347,14 +347,14 @@ 
 		'uninstall_on_delete'                         => 'disabled',
 		'the_content_filter'                          => 'enabled',
 		'scripts_footer'                              => 'disabled',
-		'agree_to_terms_label'                        => __( 'Agree to Terms?', 'give' ),
+		'agree_to_terms_label'                        => __('Agree to Terms?', 'give'),
 		'agreement_text'                              => give_get_default_agreement_text(),
 
 		// Paypal IPN verification.
 		'paypal_verification'                         => 'enabled',
 
 		// Default is manual gateway.
-		'gateways'                                    => array( 'manual' => 1, 'offline' => 1 ),
+		'gateways'                                    => array('manual' => 1, 'offline' => 1),
 		'default_gateway'                             => 'manual',
 
 		// Offline gateway setup.
@@ -382,7 +382,7 @@ 
  */
 function give_get_default_agreement_text() {
 
-	$org_name = get_bloginfo( 'name' );
+	$org_name = get_bloginfo('name');
 
 	$agreement = sprintf(
 		'<p>Acceptance of any contribution, gift or grant is at the discretion of the %1$s. The  %1$s will not accept any gift unless it can be used or expended consistently with the purpose and mission of the  %1$s.</p>
@@ -396,7 +396,7 @@ 
 		$org_name
 	);
 
-	return apply_filters( 'give_get_default_agreement_text', $agreement, $org_name );
+	return apply_filters('give_get_default_agreement_text', $agreement, $org_name);
 }
 
 
@@ -410,19 +410,19 @@ 
 function give_create_pages() {
 
 	// Bailout if pages already created.
-	if ( get_option( 'give_install_pages_created' ) ) {
+	if (get_option('give_install_pages_created')) {
 		return false;
 	}
 
 	$options = array();
 
 	// Checks if the Success Page option exists AND that the page exists.
-	if ( ! get_post( give_get_option( 'success_page' ) ) ) {
+	if ( ! get_post(give_get_option('success_page'))) {
 
 		// Donation Confirmation (Success) Page
 		$success = wp_insert_post(
 			array(
-				'post_title'     => esc_html__( 'Donation Confirmation', 'give' ),
+				'post_title'     => esc_html__('Donation Confirmation', 'give'),
 				'post_content'   => '[give_receipt]',
 				'post_status'    => 'publish',
 				'post_author'    => 1,
@@ -436,13 +436,13 @@ 
 	}
 
 	// Checks if the Failure Page option exists AND that the page exists.
-	if ( ! get_post( give_get_option( 'failure_page' ) ) ) {
+	if ( ! get_post(give_get_option('failure_page'))) {
 
 		// Failed Donation Page
 		$failed = wp_insert_post(
 			array(
-				'post_title'     => esc_html__( 'Donation Failed', 'give' ),
-				'post_content'   => esc_html__( 'We\'re sorry, your donation failed to process. Please try again or contact site support.', 'give' ),
+				'post_title'     => esc_html__('Donation Failed', 'give'),
+				'post_content'   => esc_html__('We\'re sorry, your donation failed to process. Please try again or contact site support.', 'give'),
 				'post_status'    => 'publish',
 				'post_author'    => 1,
 				'post_type'      => 'page',
@@ -454,11 +454,11 @@ 
 	}
 
 	// Checks if the History Page option exists AND that the page exists.
-	if ( ! get_post( give_get_option( 'history_page' ) ) ) {
+	if ( ! get_post(give_get_option('history_page'))) {
 		// Donation History Page
 		$history = wp_insert_post(
 			array(
-				'post_title'     => esc_html__( 'Donation History', 'give' ),
+				'post_title'     => esc_html__('Donation History', 'give'),
 				'post_content'   => '[donation_history]',
 				'post_status'    => 'publish',
 				'post_author'    => 1,
@@ -470,11 +470,11 @@ 
 		$options['history_page'] = $history;
 	}
 
-	if ( ! empty( $options ) ) {
-		update_option( 'give_settings', array_merge( give_get_settings(), $options ) );
+	if ( ! empty($options)) {
+		update_option('give_settings', array_merge(give_get_settings(), $options));
 	}
 
-	add_option( 'give_install_pages_created', 1, '', 'no' );
+	add_option('give_install_pages_created', 1, '', 'no');
 }
 
-add_action( 'admin_init', 'give_create_pages', - 1 );
+add_action('admin_init', 'give_create_pages', - 1);

includes/payments/backward-compatibility.php

Doc Comments +1 added lines, -1 removed lines

@@ -207,7 +207,7 @@
  *                              metadata entries with the specified value.
  *                              Otherwise, update all entries.
  *
- * @return mixed
+ * @return null|boolean
  */
 function _give_20_bc_saving_old_payment_meta( $check, $object_id, $meta_key, $meta_value, $prev_value ) {
 	// Bailout.

Spacing +166 added lines, -171 removed lines

@@ -9,74 +9,74 @@ 
  *
  * @return void
  */
-function _give_20_bc_split_and_save_give_payment_meta( $object_id, $meta_value ) {
+function _give_20_bc_split_and_save_give_payment_meta($object_id, $meta_value) {
 	// Bailout
-	if ( empty( $meta_value ) ) {
+	if (empty($meta_value)) {
 		return;
-	} elseif ( ! is_array( $meta_value ) ) {
+	} elseif ( ! is_array($meta_value)) {
 		$meta_value = array();
 	}
 
-	remove_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10 );
+	remove_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10);
 
 	// Date payment meta.
-	if ( ! empty( $meta_value['date'] ) ) {
-		give_update_meta( $object_id, '_give_payment_date', $meta_value['date'] );
+	if ( ! empty($meta_value['date'])) {
+		give_update_meta($object_id, '_give_payment_date', $meta_value['date']);
 	}
 
 	// Currency payment meta.
-	if ( ! empty( $meta_value['currency'] ) ) {
-		give_update_meta( $object_id, '_give_payment_currency', $meta_value['currency'] );
+	if ( ! empty($meta_value['currency'])) {
+		give_update_meta($object_id, '_give_payment_currency', $meta_value['currency']);
 	}
 
 	// User information.
-	if ( ! empty( $meta_value['user_info'] ) ) {
+	if ( ! empty($meta_value['user_info'])) {
 		// Donor first name.
-		if ( ! empty( $meta_value['user_info']['first_name'] ) ) {
-			give_update_meta( $object_id, '_give_donor_billing_first_name', $meta_value['user_info']['first_name'] );
+		if ( ! empty($meta_value['user_info']['first_name'])) {
+			give_update_meta($object_id, '_give_donor_billing_first_name', $meta_value['user_info']['first_name']);
 		}
 
 		// Donor last name.
-		if ( ! empty( $meta_value['user_info']['last_name'] ) ) {
-			give_update_meta( $object_id, '_give_donor_billing_last_name', $meta_value['user_info']['last_name'] );
+		if ( ! empty($meta_value['user_info']['last_name'])) {
+			give_update_meta($object_id, '_give_donor_billing_last_name', $meta_value['user_info']['last_name']);
 		}
 
 		// Donor address payment meta.
-		if ( ! empty( $meta_value['user_info']['address'] ) ) {
+		if ( ! empty($meta_value['user_info']['address'])) {
 
 			// Address1.
-			if ( ! empty( $meta_value['user_info']['address']['line1'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_address1', $meta_value['user_info']['address']['line1'] );
+			if ( ! empty($meta_value['user_info']['address']['line1'])) {
+				give_update_meta($object_id, '_give_donor_billing_address1', $meta_value['user_info']['address']['line1']);
 			}
 
 			// Address2.
-			if ( ! empty( $meta_value['user_info']['address']['line2'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_address2', $meta_value['user_info']['address']['line2'] );
+			if ( ! empty($meta_value['user_info']['address']['line2'])) {
+				give_update_meta($object_id, '_give_donor_billing_address2', $meta_value['user_info']['address']['line2']);
 			}
 
 			// City.
-			if ( ! empty( $meta_value['user_info']['address']['city'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_city', $meta_value['user_info']['address']['city'] );
+			if ( ! empty($meta_value['user_info']['address']['city'])) {
+				give_update_meta($object_id, '_give_donor_billing_city', $meta_value['user_info']['address']['city']);
 			}
 
 			// Zip.
-			if ( ! empty( $meta_value['user_info']['address']['zip'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_zip', $meta_value['user_info']['address']['zip'] );
+			if ( ! empty($meta_value['user_info']['address']['zip'])) {
+				give_update_meta($object_id, '_give_donor_billing_zip', $meta_value['user_info']['address']['zip']);
 			}
 
 			// State.
-			if ( ! empty( $meta_value['user_info']['address']['state'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_state', $meta_value['user_info']['address']['state'] );
+			if ( ! empty($meta_value['user_info']['address']['state'])) {
+				give_update_meta($object_id, '_give_donor_billing_state', $meta_value['user_info']['address']['state']);
 			}
 
 			// Country.
-			if ( ! empty( $meta_value['user_info']['address']['country'] ) ) {
-				give_update_meta( $object_id, '_give_donor_billing_country', $meta_value['user_info']['address']['country'] );
+			if ( ! empty($meta_value['user_info']['address']['country'])) {
+				give_update_meta($object_id, '_give_donor_billing_country', $meta_value['user_info']['address']['country']);
 			}
 		}
 	}// End if().
 
-	add_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5 );
+	add_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5);
 }
 
 /**
@@ -89,103 +89,99 @@ 
  *
  * @return array
  */
-function _give_20_bc_give_payment_meta_value( $object_id, $meta_value ) {
+function _give_20_bc_give_payment_meta_value($object_id, $meta_value) {
 	$cache_key = "_give_payment_meta_{$object_id}";
-	$cache     = Give_Cache::get_db_query( $cache_key );
+	$cache     = Give_Cache::get_db_query($cache_key);
 
-	if ( ! is_null( $cache ) ) {
+	if ( ! is_null($cache)) {
 		return $cache;
 	}
 
 	// Set default value to array.
-	if ( ! is_array( $meta_value ) ) {
+	if ( ! is_array($meta_value)) {
 		$meta_value = array();
 	}
 
 	// Donation key.
-	$meta_value['key'] = give_get_meta( $object_id, '_give_payment_purchase_key', true );
+	$meta_value['key'] = give_get_meta($object_id, '_give_payment_purchase_key', true);
 
 	// Donation form.
-	$meta_value['form_title'] = give_get_meta( $object_id, '_give_payment_form_title', true );
+	$meta_value['form_title'] = give_get_meta($object_id, '_give_payment_form_title', true);
 
 	// Donor email.
-	$meta_value['email'] = give_get_meta( $object_id, '_give_payment_donor_email', true );
-	$meta_value['email'] = ! empty( $meta_value['email'] ) ?
-		$meta_value['email'] :
-		Give()->donors->get_column( 'email', give_get_payment_donor_id( $object_id ) );
+	$meta_value['email'] = give_get_meta($object_id, '_give_payment_donor_email', true);
+	$meta_value['email'] = ! empty($meta_value['email']) ?
+		$meta_value['email'] : Give()->donors->get_column('email', give_get_payment_donor_id($object_id));
 
 	// Form id.
-	$meta_value['form_id'] = give_get_meta( $object_id, '_give_payment_form_id', true );
+	$meta_value['form_id'] = give_get_meta($object_id, '_give_payment_form_id', true);
 
 	// Price id.
-	$meta_value['price_id'] = give_get_meta( $object_id, '_give_payment_price_id', true );
+	$meta_value['price_id'] = give_get_meta($object_id, '_give_payment_price_id', true);
 
 	// Date.
-	$meta_value['date'] = give_get_meta( $object_id, '_give_payment_date', true );
-	$meta_value['date'] = ! empty( $meta_value['date'] ) ?
-		$meta_value['date'] :
-		get_post_field( 'post_date', $object_id );
+	$meta_value['date'] = give_get_meta($object_id, '_give_payment_date', true);
+	$meta_value['date'] = ! empty($meta_value['date']) ?
+		$meta_value['date'] : get_post_field('post_date', $object_id);
 
 	// Currency.
-	$meta_value['currency'] = give_get_meta( $object_id, '_give_payment_currency', true );
+	$meta_value['currency'] = give_get_meta($object_id, '_give_payment_currency', true);
 
 	// Decode donor data.
-	$donor_names = give_get_donor_name_by( give_get_meta( $object_id, '_give_payment_donor_id', true ), 'donor' );
-	$donor_names = explode( ' ', $donor_names, 2 );
+	$donor_names = give_get_donor_name_by(give_get_meta($object_id, '_give_payment_donor_id', true), 'donor');
+	$donor_names = explode(' ', $donor_names, 2);
 
 	// Donor first name.
-	$donor_data['first_name'] = give_get_meta( $object_id, '_give_donor_billing_first_name', true );
-	$donor_data['first_name'] = ! empty( $donor_data['first_name'] ) ?
-		$donor_data['first_name'] :
-		$donor_names[0];
+	$donor_data['first_name'] = give_get_meta($object_id, '_give_donor_billing_first_name', true);
+	$donor_data['first_name'] = ! empty($donor_data['first_name']) ?
+		$donor_data['first_name'] : $donor_names[0];
 
 	// Donor last name.
-	$donor_data['last_name'] = give_get_meta( $object_id, '_give_donor_billing_last_name', true );
-	$donor_data['last_name'] = ! empty( $donor_data['last_name'] ) ?
-		$donor_data['last_name'] :
-		( isset( $donor_names[1] ) ? $donor_names[1] : '' );
+	$donor_data['last_name'] = give_get_meta($object_id, '_give_donor_billing_last_name', true);
+	$donor_data['last_name'] = ! empty($donor_data['last_name']) ?
+		$donor_data['last_name'] : (isset($donor_names[1]) ? $donor_names[1] : '');
 
 	// Donor email.
 	$donor_data['email'] = $meta_value['email'];
 
 	// User ID.
-	$donor_data['id'] = give_get_payment_user_id( $object_id );
+	$donor_data['id'] = give_get_payment_user_id($object_id);
 
 	$donor_data['address'] = false;
 
 	// Address1.
-	if ( $address1 = give_get_meta( $object_id, '_give_donor_billing_address1', true ) ) {
+	if ($address1 = give_get_meta($object_id, '_give_donor_billing_address1', true)) {
 		$donor_data['address']['line1'] = $address1;
 	}
 
 	// Address2.
-	if ( $address2 = give_get_meta( $object_id, '_give_donor_billing_address2', true ) ) {
+	if ($address2 = give_get_meta($object_id, '_give_donor_billing_address2', true)) {
 		$donor_data['address']['line2'] = $address2;
 	}
 
 	// City.
-	if ( $city = give_get_meta( $object_id, '_give_donor_billing_city', true ) ) {
+	if ($city = give_get_meta($object_id, '_give_donor_billing_city', true)) {
 		$donor_data['address']['city'] = $city;
 	}
 
 	// Zip.
-	if ( $zip = give_get_meta( $object_id, '_give_donor_billing_zip', true ) ) {
+	if ($zip = give_get_meta($object_id, '_give_donor_billing_zip', true)) {
 		$donor_data['address']['zip'] = $zip;
 	}
 
 	// State.
-	if ( $state = give_get_meta( $object_id, '_give_donor_billing_state', true ) ) {
+	if ($state = give_get_meta($object_id, '_give_donor_billing_state', true)) {
 		$donor_data['address']['state'] = $state;
 	}
 
 	// Country.
-	if ( $country = give_get_meta( $object_id, '_give_donor_billing_country', true ) ) {
+	if ($country = give_get_meta($object_id, '_give_donor_billing_country', true)) {
 		$donor_data['address']['country'] = $country;
 	}
 
-	$meta_value['user_info'] = maybe_unserialize( $donor_data );
+	$meta_value['user_info'] = maybe_unserialize($donor_data);
 
-	Give_Cache::set_db_query( $cache_key, $meta_value );
+	Give_Cache::set_db_query($cache_key, $meta_value);
 
 	return $meta_value;
 }
@@ -209,37 +205,37 @@ 
  *
  * @return mixed
  */
-function _give_20_bc_saving_old_payment_meta( $check, $object_id, $meta_key, $meta_value, $prev_value ) {
+function _give_20_bc_saving_old_payment_meta($check, $object_id, $meta_key, $meta_value, $prev_value) {
 	// Bailout.
 	if (
-		'give_payment' !== get_post_type( $object_id ) ||
-		! in_array( $meta_key, array(
+		'give_payment' !== get_post_type($object_id) ||
+		! in_array($meta_key, array(
 			'_give_payment_meta',
 			'_give_payment_user_email',
 			'_give_payment_customer_id',
 			'give_payment_user_ip',
-		) )
+		))
 	) {
 		return $check;
 	}
 
-	if ( '_give_payment_meta' === $meta_key ) {
-		_give_20_bc_split_and_save_give_payment_meta( $object_id, $meta_value );
-	} elseif ( '_give_payment_user_email' === $meta_key ) {
-		give_update_meta( $object_id, '_give_payment_donor_email', $meta_value );
+	if ('_give_payment_meta' === $meta_key) {
+		_give_20_bc_split_and_save_give_payment_meta($object_id, $meta_value);
+	} elseif ('_give_payment_user_email' === $meta_key) {
+		give_update_meta($object_id, '_give_payment_donor_email', $meta_value);
 		$check = true;
-	} elseif ( '_give_payment_customer_id' === $meta_key ) {
-		give_update_meta( $object_id, '_give_payment_donor_id', $meta_value );
+	} elseif ('_give_payment_customer_id' === $meta_key) {
+		give_update_meta($object_id, '_give_payment_donor_id', $meta_value);
 		$check = true;
-	} elseif ( 'give_payment_user_ip' === $meta_key ) {
-		give_update_meta( $object_id, '_give_payment_donor_ip', $meta_value );
+	} elseif ('give_payment_user_ip' === $meta_key) {
+		give_update_meta($object_id, '_give_payment_donor_ip', $meta_value);
 		$check = true;
 	}
 
 	return $check;
 }
 
-add_filter( 'update_post_metadata', '_give_20_bc_saving_old_payment_meta', 10, 5 );
+add_filter('update_post_metadata', '_give_20_bc_saving_old_payment_meta', 10, 5);
 
 
 /**
@@ -254,7 +250,7 @@ 
  *
  * @return mixed
  */
-function _give_20_bc_get_old_payment_meta( $check, $object_id, $meta_key, $single ) {
+function _give_20_bc_get_old_payment_meta($check, $object_id, $meta_key, $single) {
 	global $wpdb;
 
 	// Deprecated meta keys.
@@ -265,32 +261,31 @@ 
 	);
 
 	// Add _give_payment_meta to backward compatibility
-	if ( ! give_has_upgrade_completed( 'v20_upgrades_payment_metadata' ) ) {
+	if ( ! give_has_upgrade_completed('v20_upgrades_payment_metadata')) {
 		$old_meta_keys[] = '_give_payment_meta';
 	}
 
 	// Bailout.
 	if (
-		'give_payment' !== get_post_type( $object_id ) ||
-		! in_array( $meta_key, $old_meta_keys )
+		'give_payment' !== get_post_type($object_id) ||
+		! in_array($meta_key, $old_meta_keys)
 	) {
 		return $check;
 	}
 
 	$cache_key = "{$meta_key}_{$object_id}";
-	$check     = Give_Cache::get_db_query( $cache_key );
+	$check     = Give_Cache::get_db_query($cache_key);
 
-	if ( is_null( $check ) ) {
-		switch ( $meta_key ) {
+	if (is_null($check)) {
+		switch ($meta_key) {
 
 			// Handle old meta keys.
 			case '_give_payment_meta':
-				remove_filter( 'get_post_metadata', '_give_20_bc_get_old_payment_meta' );
+				remove_filter('get_post_metadata', '_give_20_bc_get_old_payment_meta');
 
 				// if ( $meta_value = give_get_meta( $object_id, '_give_payment_meta' ) ) {
-				$meta_value = ! empty( $meta_value ) ?
-					current( $meta_value ) :
-					(array) maybe_unserialize(
+				$meta_value = ! empty($meta_value) ?
+					current($meta_value) : (array) maybe_unserialize(
 						$wpdb->get_var(
 							$wpdb->prepare(
 								"
@@ -304,44 +299,44 @@ 
 							)
 						)
 					);
-				$check      = _give_20_bc_give_payment_meta_value( $object_id, $meta_value );
+				$check = _give_20_bc_give_payment_meta_value($object_id, $meta_value);
 				// }
 
-				add_filter( 'get_post_metadata', '_give_20_bc_get_old_payment_meta', 10, 5 );
+				add_filter('get_post_metadata', '_give_20_bc_get_old_payment_meta', 10, 5);
 
 				break;
 
 			case '_give_payment_customer_id':
-				if ( $donor_id = give_get_meta( $object_id, '_give_payment_donor_id', $single ) ) {
+				if ($donor_id = give_get_meta($object_id, '_give_payment_donor_id', $single)) {
 					$check = $donor_id;
 				}
 				break;
 
 			case '_give_payment_user_email':
-				if ( $donor_email = give_get_meta( $object_id, '_give_payment_donor_email', $single ) ) {
+				if ($donor_email = give_get_meta($object_id, '_give_payment_donor_email', $single)) {
 					$check = $donor_email;
 				}
 				break;
 
 			case '_give_payment_user_ip':
-				if ( $donor_ip = give_get_meta( $object_id, '_give_payment_donor_ip', $single ) ) {
+				if ($donor_ip = give_get_meta($object_id, '_give_payment_donor_ip', $single)) {
 					$check = $donor_ip;
 				}
 				break;
 		}// End switch().
 
-		Give_Cache::set_db_query( $cache_key, $check );
+		Give_Cache::set_db_query($cache_key, $check);
 	}
 
 	// Put result in an array on zero index.
-	if ( ! is_null( $check ) ) {
-		$check = array( $check );
+	if ( ! is_null($check)) {
+		$check = array($check);
 	}
 
 	return $check;
 }
 
-add_filter( 'get_post_metadata', '_give_20_bc_get_old_payment_meta', 10, 5 );
+add_filter('get_post_metadata', '_give_20_bc_get_old_payment_meta', 10, 5);
 
 
 /**
@@ -356,9 +351,9 @@ 
  *
  * @return mixed
  */
-function _give_20_bc_get_new_payment_meta( $check, $object_id, $meta_key, $single ) {
+function _give_20_bc_get_new_payment_meta($check, $object_id, $meta_key, $single) {
 	// Bailout: do not apply backward compatibility if upgrade done.
-	if ( give_has_upgrade_completed( 'v20_upgrades_payment_metadata' ) ) {
+	if (give_has_upgrade_completed('v20_upgrades_payment_metadata')) {
 		return $check;
 	}
 
@@ -380,26 +375,26 @@ 
 	);
 
 	// metadata_exists fx will cause of firing get_post_metadata filter again so remove it to prevent infinite loop.
-	remove_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta' );
+	remove_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta');
 
 	// Bailout.
 	if (
-		'give_payment' !== get_post_type( $object_id ) ||
-		! in_array( $meta_key, $new_meta_keys ) ||
-		metadata_exists( 'post', $object_id, $meta_key )
+		'give_payment' !== get_post_type($object_id) ||
+		! in_array($meta_key, $new_meta_keys) ||
+		metadata_exists('post', $object_id, $meta_key)
 	) {
-		add_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5 );
+		add_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5);
 
 		return $check;
 	}
 
-	add_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5 );
+	add_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5);
 
 	$cache_key = "{$meta_key}_{$object_id}";
-	$check    = Give_Cache::get_db_query( $cache_key );
+	$check = Give_Cache::get_db_query($cache_key);
 
-	if ( is_null( $check ) ) {
-		switch ( $meta_key ) {
+	if (is_null($check)) {
+		switch ($meta_key) {
 
 			// Handle new meta keys.
 			case '_give_payment_donor_id':
@@ -442,9 +437,9 @@ 
 			case '_give_donor_billing_country':
 			case '_give_payment_date':
 			case '_give_payment_currency':
-				$donation_meta = Give_Cache::get_db_query( "_give_payment_meta_{$object_id}" );
+				$donation_meta = Give_Cache::get_db_query("_give_payment_meta_{$object_id}");
 
-				if ( is_null( $donation_meta ) ) {
+				if (is_null($donation_meta)) {
 					$donation_meta = $wpdb->get_var(
 						$wpdb->prepare(
 							"SELECT meta_value FROM {$wpdb->postmeta} WHERE post_id=%d AND meta_key=%s",
@@ -452,51 +447,51 @@ 
 							'_give_payment_meta'
 						)
 					);
-					$donation_meta = maybe_unserialize( $donation_meta );
-					$donation_meta = ! is_array( $donation_meta ) ? array() : $donation_meta;
-					Give_Cache::set_db_query( "_give_payment_meta_{$object_id}", $donation_meta );
+					$donation_meta = maybe_unserialize($donation_meta);
+					$donation_meta = ! is_array($donation_meta) ? array() : $donation_meta;
+					Give_Cache::set_db_query("_give_payment_meta_{$object_id}", $donation_meta);
 				}
 
 				// Get results.
-				if ( empty( $donation_meta ) ) {
+				if (empty($donation_meta)) {
 					$check = '';
-				} elseif ( in_array( $meta_key, array( '_give_payment_date', '_give_payment_currency' ) ) ) {
-					$payment_meta_key = str_replace( '_give_payment_', '', $meta_key );
+				} elseif (in_array($meta_key, array('_give_payment_date', '_give_payment_currency'))) {
+					$payment_meta_key = str_replace('_give_payment_', '', $meta_key);
 
-					if ( isset( $donation_meta[ $payment_meta_key ] ) ) {
-						$check = $donation_meta[ $payment_meta_key ];
+					if (isset($donation_meta[$payment_meta_key])) {
+						$check = $donation_meta[$payment_meta_key];
 					}
 				} else {
-					$payment_meta_key = str_replace( '_give_donor_billing_', '', $meta_key );
+					$payment_meta_key = str_replace('_give_donor_billing_', '', $meta_key);
 
-					switch ( $payment_meta_key ) {
+					switch ($payment_meta_key) {
 						case 'address1':
-							if ( isset( $donation_meta['user_info']['address']['line1'] ) ) {
+							if (isset($donation_meta['user_info']['address']['line1'])) {
 								$check = $donation_meta['user_info']['address']['line1'];
 							}
 							break;
 
 						case 'address2':
-							if ( isset( $donation_meta['user_info']['address']['line2'] ) ) {
+							if (isset($donation_meta['user_info']['address']['line2'])) {
 								$check = $donation_meta['user_info']['address']['line2'];
 							}
 							break;
 
 						case 'first_name':
-							if ( isset( $donation_meta['user_info']['first_name'] ) ) {
+							if (isset($donation_meta['user_info']['first_name'])) {
 								$check = $donation_meta['user_info']['first_name'];
 							}
 							break;
 
 						case 'last_name':
-							if ( isset( $donation_meta['user_info']['last_name'] ) ) {
+							if (isset($donation_meta['user_info']['last_name'])) {
 								$check = $donation_meta['user_info']['last_name'];
 							}
 							break;
 
 						default:
-							if ( isset( $donation_meta['user_info']['address'][ $payment_meta_key ] ) ) {
-								$check = $donation_meta['user_info']['address'][ $payment_meta_key ];
+							if (isset($donation_meta['user_info']['address'][$payment_meta_key])) {
+								$check = $donation_meta['user_info']['address'][$payment_meta_key];
 							}
 					}
 				}
@@ -505,19 +500,19 @@ 
 		}// End switch().
 
 		// Set cache.
-		Give_Cache::set_db_query( $cache_key, $check );
+		Give_Cache::set_db_query($cache_key, $check);
 	}
 
 	// Put result in an array on zero index.
-	if ( ! $single ) {
-		$check = array( $check );
+	if ( ! $single) {
+		$check = array($check);
 	}
 
 
 	return $check;
 }
 
-add_filter( 'get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5 );
+add_filter('get_post_metadata', '_give_20_bc_get_new_payment_meta', 10, 5);
 
 
 /**
@@ -529,9 +524,9 @@ 
  *
  * @return void
  */
-function _give_20_bc_support_deprecated_meta_key_query( $query ) {
+function _give_20_bc_support_deprecated_meta_key_query($query) {
 	// Bailout.
-	if ( give_has_upgrade_completed( 'v20_upgrades_payment_metadata' ) ) {
+	if (give_has_upgrade_completed('v20_upgrades_payment_metadata')) {
 		return;
 	}
 
@@ -541,21 +536,21 @@ 
 		// '_give_payment_user_ip'     => '_give_payment_donor_ip',
 	);
 
-	$deprecated_meta_keys = array_flip( $new_meta_keys );
+	$deprecated_meta_keys = array_flip($new_meta_keys);
 
 	// Set meta keys.
 	$meta_keys = array();
 
 
 	// Bailout.
-	if ( ! empty( $query->query_vars['meta_key'] ) ) {
-		if ( in_array( $query->query_vars['meta_key'], $new_meta_keys ) ) {
+	if ( ! empty($query->query_vars['meta_key'])) {
+		if (in_array($query->query_vars['meta_key'], $new_meta_keys)) {
 			$meta_keys = $deprecated_meta_keys;
-		} elseif ( in_array( $query->query_vars['meta_key'], $deprecated_meta_keys ) ) {
+		} elseif (in_array($query->query_vars['meta_key'], $deprecated_meta_keys)) {
 			$meta_keys = $new_meta_keys;
 		}
 
-		if ( ! empty( $meta_keys ) ) {
+		if ( ! empty($meta_keys)) {
 			// Set meta_query
 			$query->set(
 				'meta_query',
@@ -566,35 +561,35 @@ 
 						'value' => $query->query_vars['meta_value'],
 					),
 					array(
-						'key'   => $meta_keys[ $query->query_vars['meta_key'] ],
+						'key'   => $meta_keys[$query->query_vars['meta_key']],
 						'value' => $query->query_vars['meta_value'],
 					),
 				)
 			);
 
 			// Unset single meta query.
-			unset( $query->query_vars['meta_key'] );
-			unset( $query->query_vars['meta_value'] );
+			unset($query->query_vars['meta_key']);
+			unset($query->query_vars['meta_value']);
 		}
 	} elseif (
-		! empty( $query->query_vars['meta_query'] ) &&
-		( 1 === count( $query->query_vars['meta_query'] ) )
+		! empty($query->query_vars['meta_query']) &&
+		(1 === count($query->query_vars['meta_query']))
 	) {
-		$meta_query = current( $query->query_vars['meta_query'] );
+		$meta_query = current($query->query_vars['meta_query']);
 
-		if ( empty( $meta_query[0]['key'] ) ) {
+		if (empty($meta_query[0]['key'])) {
 			return;
 		}
 
-		if ( in_array( $meta_query[0]['key'], $new_meta_keys ) ) {
+		if (in_array($meta_query[0]['key'], $new_meta_keys)) {
 			$meta_keys = $deprecated_meta_keys;
-		} elseif ( in_array( $meta_query[0]['key'], $deprecated_meta_keys ) ) {
+		} elseif (in_array($meta_query[0]['key'], $deprecated_meta_keys)) {
 			$meta_keys = $new_meta_keys;
 		} else {
 			return;
 		}
 
-		if ( ! empty( $meta_keys ) ) {
+		if ( ! empty($meta_keys)) {
 			// Set meta_query
 			$query->set(
 				'meta_query',
@@ -605,7 +600,7 @@ 
 						'value' => $query->query_vars['meta_query'][0]['value'],
 					),
 					array(
-						'key'   => $meta_keys[ $query->query_vars['meta_query'][0]['key'] ],
+						'key'   => $meta_keys[$query->query_vars['meta_query'][0]['key']],
 						'value' => $query->query_vars['meta_query'][0]['value'],
 					),
 				)
@@ -614,7 +609,7 @@ 
 	}
 }
 
-add_action( 'pre_get_posts', '_give_20_bc_support_deprecated_meta_key_query' );
+add_action('pre_get_posts', '_give_20_bc_support_deprecated_meta_key_query');
 
 
 /**
@@ -627,49 +622,49 @@ 
  * @param Give_Payment $payment
  * @param string       $key
  */
-function _give_20_bc_payment_save( $payment, $key ) {
+function _give_20_bc_payment_save($payment, $key) {
 	// Bailout.
-	if ( ! give_has_upgrade_completed( 'v20_upgrades_payment_metadata' ) ) {
+	if ( ! give_has_upgrade_completed('v20_upgrades_payment_metadata')) {
 		return;
 	}
 
-	switch ( $key ) {
+	switch ($key) {
 		case 'user_info':
-			if ( empty( $payment->user_info ) ) {
+			if (empty($payment->user_info)) {
 				// Bailout.
 				break;
-			} elseif ( is_string( $payment->user_info ) ) {
+			} elseif (is_string($payment->user_info)) {
 				// Check if value serialize.
-				$payment->user_info = maybe_unserialize( $payment->user_info );
+				$payment->user_info = maybe_unserialize($payment->user_info);
 			}
 
 
 			// Save first name.
-			if ( isset( $payment->user_info['first_name'] ) ) {
-				$payment->update_meta( '_give_donor_billing_first_name', $payment->user_info['first_name'] );
+			if (isset($payment->user_info['first_name'])) {
+				$payment->update_meta('_give_donor_billing_first_name', $payment->user_info['first_name']);
 			}
 
 
 			// Save last name.
-			if ( isset( $payment->user_info['last_name'] ) ) {
-				$payment->update_meta( '_give_donor_billing_last_name', $payment->user_info['last_name'] );
+			if (isset($payment->user_info['last_name'])) {
+				$payment->update_meta('_give_donor_billing_last_name', $payment->user_info['last_name']);
 			}
 
 
 			// Save address.
-			if ( ! empty( $payment->user_info['address'] ) ) {
-				foreach ( $payment->user_info['address'] as $address_name => $address ) {
-					switch ( $address_name ) {
+			if ( ! empty($payment->user_info['address'])) {
+				foreach ($payment->user_info['address'] as $address_name => $address) {
+					switch ($address_name) {
 						case 'line1':
-							$payment->update_meta( '_give_donor_billing_address1', $address );
+							$payment->update_meta('_give_donor_billing_address1', $address);
 							break;
 
 						case 'line2':
-							$payment->update_meta( '_give_donor_billing_address2', $address );
+							$payment->update_meta('_give_donor_billing_address2', $address);
 							break;
 
 						default:
-							$payment->update_meta( "_give_donor_billing_{$address_name}", $address );
+							$payment->update_meta("_give_donor_billing_{$address_name}", $address);
 					}
 				}
 			}
@@ -678,7 +673,7 @@ 
 	}
 }
 
-add_action( 'give_payment_save', '_give_20_bc_payment_save', 10, 2 );
+add_action('give_payment_save', '_give_20_bc_payment_save', 10, 2);
 
 
 /**
@@ -691,16 +686,16 @@ 
  *
  * @return mixed
  */
-function __give_20_bc_flush_cache( $check, $object_id ) {
+function __give_20_bc_flush_cache($check, $object_id) {
 	if (
-		! give_has_upgrade_completed( 'v20_move_metadata_into_new_table' ) &&
-		'give_payment' === get_post_type( $object_id )
+		! give_has_upgrade_completed('v20_move_metadata_into_new_table') &&
+		'give_payment' === get_post_type($object_id)
 	) {
-		Give_Cache::delete_group( $object_id, 'give-donations' );
+		Give_Cache::delete_group($object_id, 'give-donations');
 	}
 
 	return $check;
 }
 
-add_action( 'update_post_metadata', '__give_20_bc_flush_cache', 9999, 2 );
-add_action( 'add_post_metadata', '__give_20_bc_flush_cache', 9999, 2 );
+add_action('update_post_metadata', '__give_20_bc_flush_cache', 9999, 2);
+add_action('add_post_metadata', '__give_20_bc_flush_cache', 9999, 2);

includes/payments/class-payments-query.php

Doc Comments +3 added lines, -3 removed lines

@@ -107,7 +107,7 @@ 
 	 * @since  1.0
 	 * @access public
 	 *
-	 * @param $query_var
+	 * @param string $query_var
 	 * @param $value
 	 */
 	public function __set( $query_var, $value ) {
@@ -124,7 +124,7 @@ 
 	 * @since  1.0
 	 * @access public
 	 *
-	 * @param $query_var
+	 * @param string $query_var
 	 */
 	public function __unset( $query_var ) {
 		unset( $this->args[ $query_var ] );
@@ -468,7 +468,7 @@ 
 	 * @param string   $order
 	 * @param WP_Query $query
 	 *
-	 * @return mixed
+	 * @return string
 	 */
 	public function custom_orderby( $order, $query ) {
 

Spacing +144 added lines, -144 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -66,10 +66,10 @@ 
 	 *
 	 * @param  $args array The array of arguments that can be passed in and used for setting up this payment query.
 	 */
-	public function __construct( $args = array() ) {
+	public function __construct($args = array()) {
 		$defaults = array(
 			'output'          => 'payments',
-			'post_type'       => array( 'give_payment' ),
+			'post_type'       => array('give_payment'),
 			'start_date'      => false,
 			'end_date'        => false,
 			'number'          => 20,
@@ -96,7 +96,7 @@ 
 			'count'           => false,
 		);
 
-		$this->args = $this->_args = wp_parse_args( $args, $defaults );
+		$this->args = $this->_args = wp_parse_args($args, $defaults);
 
 		$this->init();
 	}
@@ -110,11 +110,11 @@ 
 	 * @param $query_var
 	 * @param $value
 	 */
-	public function __set( $query_var, $value ) {
-		if ( in_array( $query_var, array( 'meta_query', 'tax_query' ) ) ) {
-			$this->args[ $query_var ][] = $value;
+	public function __set($query_var, $value) {
+		if (in_array($query_var, array('meta_query', 'tax_query'))) {
+			$this->args[$query_var][] = $value;
 		} else {
-			$this->args[ $query_var ] = $value;
+			$this->args[$query_var] = $value;
 		}
 	}
 
@@ -126,8 +126,8 @@ 
 	 *
 	 * @param $query_var
 	 */
-	public function __unset( $query_var ) {
-		unset( $this->args[ $query_var ] );
+	public function __unset($query_var) {
+		unset($this->args[$query_var]);
 	}
 
 	/**
@@ -167,7 +167,7 @@ 
 		$this->give_forms();
 		$this->gateway_filter();
 
-		add_filter( 'posts_orderby', array( $this, 'custom_orderby' ), 10, 2 );
+		add_filter('posts_orderby', array($this, 'custom_orderby'), 10, 2);
 
 		/**
 		 * Fires after setup filters.
@@ -176,7 +176,7 @@ 
 		 *
 		 * @param Give_Payments_Query $this Payments query object.
 		 */
-		do_action( 'give_pre_get_payments', $this );
+		do_action('give_pre_get_payments', $this);
 	}
 
 	/**
@@ -186,7 +186,7 @@ 
 	 * @access private
 	 */
 	private function unset_filters() {
-		remove_filter( 'posts_orderby', array( $this, 'custom_orderby' ) );
+		remove_filter('posts_orderby', array($this, 'custom_orderby'));
 
 		/**
 		 * Fires after retrieving payments.
@@ -195,7 +195,7 @@ 
 		 *
 		 * @param Give_Payments_Query $this Payments query object.
 		 */
-		do_action( 'give_post_get_payments', $this );
+		do_action('give_post_get_payments', $this);
 	}
 
 
@@ -214,11 +214,11 @@ 
 	public function get_payments() {
 		global $post;
 
-		$cache_key      = Give_Cache::get_key( 'give_payment_query', $this->args, false );
-		$this->payments = Give_Cache::get_db_query( $cache_key );
+		$cache_key      = Give_Cache::get_key('give_payment_query', $this->args, false);
+		$this->payments = Give_Cache::get_db_query($cache_key);
 
 		// Return cached result.
-		if ( ! is_null( $this->payments ) ) {
+		if ( ! is_null($this->payments)) {
 			return $this->payments;
 		}
 
@@ -226,7 +226,7 @@ 
 		// Modify the query/query arguments before we retrieve payments.
 		$this->set_filters();
 
-		$query          = new WP_Query( $this->args );
+		$query          = new WP_Query($this->args);
 		$this->payments = array();
 
 		$custom_output = array(
@@ -234,32 +234,32 @@ 
 			'give_payments',
 		);
 
-		if ( ! in_array( $this->args['output'], $custom_output ) ) {
+		if ( ! in_array($this->args['output'], $custom_output)) {
 			return $query->posts;
 		}
 
-		if ( $query->have_posts() ) {
+		if ($query->have_posts()) {
 			$previous_post = $post;
 
-			while ( $query->have_posts() ) {
+			while ($query->have_posts()) {
 				$query->the_post();
 
 				$payment_id = get_post()->ID;
-				$payment    = new Give_Payment( $payment_id );
+				$payment    = new Give_Payment($payment_id);
 
-				$this->payments[] = apply_filters( 'give_payment', $payment, $payment_id, $this );
+				$this->payments[] = apply_filters('give_payment', $payment, $payment_id, $this);
 			}
 
 			wp_reset_postdata();
 
 			// Prevent nest loop from producing unexpected results.
-			if( $previous_post instanceof WP_Post ) {
+			if ($previous_post instanceof WP_Post) {
 				$post = $previous_post;
-				setup_postdata( $post );
+				setup_postdata($post);
 			}
 		}
 
-		Give_Cache::set_db_query( $cache_key, $this->payments );
+		Give_Cache::set_db_query($cache_key, $this->payments);
 
 		// Remove query filters after we retrieve payments.
 		$this->unset_filters();
@@ -278,31 +278,31 @@ 
 	public function get_payment_by_group() {
 		global $wpdb;
 
-		$allowed_groups = array( 'post_status' );
+		$allowed_groups = array('post_status');
 		$result         = array();
 
 
-		if ( in_array( $this->args['group_by'], $allowed_groups ) ) {
+		if (in_array($this->args['group_by'], $allowed_groups)) {
 			// Set only count in result.
-			if ( $this->args['count'] ) {
+			if ($this->args['count']) {
 
 				$this->set_filters();
 
-				$new_results = $wpdb->get_results( $this->get_sql(), ARRAY_N );
+				$new_results = $wpdb->get_results($this->get_sql(), ARRAY_N);
 
 				$this->unset_filters();
 
-				foreach ( $new_results as $results ) {
-					$result[ $results[0] ] = $results[1];
+				foreach ($new_results as $results) {
+					$result[$results[0]] = $results[1];
 				}
 
-				switch ( $this->args['group_by'] ) {
+				switch ($this->args['group_by']) {
 					case 'post_status':
 
 						/* @var Give_Payment $donation */
-						foreach ( give_get_payment_status_keys() as $status ) {
-							if ( ! isset( $result[ $status ] ) ) {
-								$result[ $status ] = 0;
+						foreach (give_get_payment_status_keys() as $status) {
+							if ( ! isset($result[$status])) {
+								$result[$status] = 0;
 							}
 						}
 
@@ -312,8 +312,8 @@ 
 				$donations = $this->get_payments();
 
 				/* @var $donation Give_Payment */
-				foreach ( $donations as $donation ) {
-					$result[ $donation->{$this->args['group_by']} ][] = $donation;
+				foreach ($donations as $donation) {
+					$result[$donation->{$this->args['group_by']}][] = $donation;
 				}
 			}
 		}
@@ -324,7 +324,7 @@ 
 		 *
 		 * @since 1.8.17
 		 */
-		return apply_filters( 'give_get_payment_by_group', $result, $this );
+		return apply_filters('give_get_payment_by_group', $result, $this);
 	}
 
 	/**
@@ -336,29 +336,29 @@ 
 	 * @return void
 	 */
 	public function date_filter_pre() {
-		if ( ! ( $this->args['start_date'] || $this->args['end_date'] ) ) {
+		if ( ! ($this->args['start_date'] || $this->args['end_date'])) {
 			return;
 		}
 
-		$this->setup_dates( $this->args['start_date'], $this->args['end_date'] );
-		$is_start_date = property_exists( __CLASS__, 'start_date' );
-		$is_end_date   = property_exists( __CLASS__, 'end_date' );
+		$this->setup_dates($this->args['start_date'], $this->args['end_date']);
+		$is_start_date = property_exists(__CLASS__, 'start_date');
+		$is_end_date   = property_exists(__CLASS__, 'end_date');
 
-		if ( $is_start_date || $is_end_date ) {
+		if ($is_start_date || $is_end_date) {
 			$date_query = array();
 
-			if ( $is_start_date && ! is_wp_error( $this->start_date ) ) {
-				$date_query['after'] = date( 'Y-m-d H:i:s', $this->start_date );
+			if ($is_start_date && ! is_wp_error($this->start_date)) {
+				$date_query['after'] = date('Y-m-d H:i:s', $this->start_date);
 			}
 
-			if ( $is_end_date && ! is_wp_error( $this->end_date ) ) {
-				$date_query['before'] = date( 'Y-m-d H:i:s', $this->end_date );
+			if ($is_end_date && ! is_wp_error($this->end_date)) {
+				$date_query['before'] = date('Y-m-d H:i:s', $this->end_date);
 			}
 
 			// Include Start Date and End Date while querying.
 			$date_query['inclusive'] = true;
 
-			$this->__set( 'date_query', $date_query );
+			$this->__set('date_query', $date_query);
 
 		}
 	}
@@ -372,12 +372,12 @@ 
 	 * @return void
 	 */
 	public function status() {
-		if ( ! isset( $this->args['status'] ) ) {
+		if ( ! isset($this->args['status'])) {
 			return;
 		}
 
-		$this->__set( 'post_status', $this->args['status'] );
-		$this->__unset( 'status' );
+		$this->__set('post_status', $this->args['status']);
+		$this->__unset('status');
 	}
 
 	/**
@@ -389,12 +389,12 @@ 
 	 * @return void
 	 */
 	public function page() {
-		if ( ! isset( $this->args['page'] ) ) {
+		if ( ! isset($this->args['page'])) {
 			return;
 		}
 
-		$this->__set( 'paged', $this->args['page'] );
-		$this->__unset( 'page' );
+		$this->__set('paged', $this->args['page']);
+		$this->__unset('page');
 	}
 
 	/**
@@ -407,17 +407,17 @@ 
 	 */
 	public function per_page() {
 
-		if ( ! isset( $this->args['number'] ) ) {
+		if ( ! isset($this->args['number'])) {
 			return;
 		}
 
-		if ( $this->args['number'] == - 1 ) {
-			$this->__set( 'nopaging', true );
+		if ($this->args['number'] == - 1) {
+			$this->__set('nopaging', true);
 		} else {
-			$this->__set( 'posts_per_page', $this->args['number'] );
+			$this->__set('posts_per_page', $this->args['number']);
 		}
 
-		$this->__unset( 'number' );
+		$this->__unset('number');
 	}
 
 	/**
@@ -429,12 +429,12 @@ 
 	 * @return void
 	 */
 	public function month() {
-		if ( ! isset( $this->args['month'] ) ) {
+		if ( ! isset($this->args['month'])) {
 			return;
 		}
 
-		$this->__set( 'monthnum', $this->args['month'] );
-		$this->__unset( 'month' );
+		$this->__set('monthnum', $this->args['month']);
+		$this->__unset('month');
 	}
 
 	/**
@@ -446,23 +446,23 @@ 
 	 * @return void
 	 */
 	public function orderby() {
-		switch ( $this->args['orderby'] ) {
+		switch ($this->args['orderby']) {
 			case 'amount':
-				$this->__set( 'orderby', 'meta_value_num' );
-				$this->__set( 'meta_key', '_give_payment_total' );
+				$this->__set('orderby', 'meta_value_num');
+				$this->__set('meta_key', '_give_payment_total');
 				break;
 
 			case 'status':
-				$this->__set( 'orderby', 'post_status' );
+				$this->__set('orderby', 'post_status');
 				break;
 
 			case 'donation_form':
-				$this->__set( 'orderby', 'meta_value' );
-				$this->__set( 'meta_key', '_give_payment_form_title' );
+				$this->__set('orderby', 'meta_value');
+				$this->__set('meta_key', '_give_payment_form_title');
 				break;
 
 			default:
-				$this->__set( 'orderby', $this->args['orderby'] );
+				$this->__set('orderby', $this->args['orderby']);
 				break;
 		}
 	}
@@ -479,19 +479,19 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function custom_orderby( $order, $query ) {
+	public function custom_orderby($order, $query) {
 
-		if ( ! empty( $query->query['post_type'] ) ) {
-			$post_types = is_array( $query->query['post_type'] ) ? $query->query['post_type'] : array( $query->query['post_type'] );
+		if ( ! empty($query->query['post_type'])) {
+			$post_types = is_array($query->query['post_type']) ? $query->query['post_type'] : array($query->query['post_type']);
 
-			if ( ! in_array( 'give_payment', $post_types ) || ! isset( $query->query['orderby'] ) || is_array( $query->query['orderby'] ) ) {
+			if ( ! in_array('give_payment', $post_types) || ! isset($query->query['orderby']) || is_array($query->query['orderby'])) {
 				return $order;
 			}
 
 			global $wpdb;
-			switch ( $query->query['orderby'] ) {
+			switch ($query->query['orderby']) {
 				case 'post_status':
-					$order = $wpdb->posts . '.post_status ' . strtoupper( $query->query['order'] );
+					$order = $wpdb->posts.'.post_status '.strtoupper($query->query['order']);
 					break;
 			}
 		}
@@ -508,11 +508,11 @@ 
 	 * @return void
 	 */
 	public function user() {
-		if ( is_null( $this->args['user'] ) ) {
+		if (is_null($this->args['user'])) {
 			return;
 		}
 
-		if ( is_numeric( $this->args['user'] ) ) {
+		if (is_numeric($this->args['user'])) {
 			$user_key = '_give_payment_donor_id';
 		} else {
 			$user_key = '_give_payment_donor_email';
@@ -534,16 +534,16 @@ 
 	 * @return  void
 	 */
 	public function donor() {
-		if ( is_null( $this->args['donor'] ) || ! is_numeric( $this->args['donor'] ) ) {
+		if (is_null($this->args['donor']) || ! is_numeric($this->args['donor'])) {
 			return;
 		}
 
 		$donor_meta_type = Give()->donor_meta->meta_type;
 
-		$this->__set( 'meta_query', array(
+		$this->__set('meta_query', array(
 			'key'   => "_give_payment_{$donor_meta_type}_id",
 			'value' => (int) $this->args['donor'],
-		) );
+		));
 	}
 
 	/**
@@ -556,33 +556,33 @@ 
 	 */
 	public function search() {
 
-		if ( ! isset( $this->args['s'] ) ) {
+		if ( ! isset($this->args['s'])) {
 			return;
 		}
 
-		$search = trim( $this->args['s'] );
+		$search = trim($this->args['s']);
 
-		if ( empty( $search ) ) {
+		if (empty($search)) {
 			return;
 		}
 
-		$is_email = is_email( $search ) || strpos( $search, '@' ) !== false;
-		$is_user  = strpos( $search, strtolower( 'user:' ) ) !== false;
+		$is_email = is_email($search) || strpos($search, '@') !== false;
+		$is_user  = strpos($search, strtolower('user:')) !== false;
 
-		if ( ! empty( $this->args['search_in_notes'] ) ) {
+		if ( ! empty($this->args['search_in_notes'])) {
 
-			$notes = give_get_payment_notes( 0, $search );
+			$notes = give_get_payment_notes(0, $search);
 
-			if ( ! empty( $notes ) ) {
+			if ( ! empty($notes)) {
 
-				$payment_ids = wp_list_pluck( (array) $notes, 'comment_post_ID' );
+				$payment_ids = wp_list_pluck((array) $notes, 'comment_post_ID');
 
-				$this->__set( 'post__in', $payment_ids );
+				$this->__set('post__in', $payment_ids);
 			}
 
-			$this->__unset( 's' );
+			$this->__unset('s');
 
-		} elseif ( $is_email || strlen( $search ) == 32 ) {
+		} elseif ($is_email || strlen($search) == 32) {
 
 			$key         = $is_email ? '_give_payment_donor_email' : '_give_payment_purchase_key';
 			$search_meta = array(
@@ -591,40 +591,40 @@ 
 				'compare' => 'LIKE',
 			);
 
-			$this->__set( 'meta_query', $search_meta );
-			$this->__unset( 's' );
+			$this->__set('meta_query', $search_meta);
+			$this->__unset('s');
 
-		} elseif ( $is_user ) {
+		} elseif ($is_user) {
 
 			$search_meta = array(
 				'key'   => '_give_payment_donor_id',
-				'value' => trim( str_replace( 'user:', '', strtolower( $search ) ) ),
+				'value' => trim(str_replace('user:', '', strtolower($search))),
 			);
 
-			$this->__set( 'meta_query', $search_meta );
+			$this->__set('meta_query', $search_meta);
 
-			$this->__unset( 's' );
+			$this->__unset('s');
 
-		} elseif ( is_numeric( $search ) ) {
+		} elseif (is_numeric($search)) {
 
-			$post = get_post( $search );
+			$post = get_post($search);
 
-			if ( is_object( $post ) && $post->post_type == 'give_payment' ) {
+			if (is_object($post) && $post->post_type == 'give_payment') {
 
 				$arr   = array();
 				$arr[] = $search;
-				$this->__set( 'post__in', $arr );
-				$this->__unset( 's' );
+				$this->__set('post__in', $arr);
+				$this->__unset('s');
 			}
-		} elseif ( '#' == substr( $search, 0, 1 ) ) {
+		} elseif ('#' == substr($search, 0, 1)) {
 
-			$search = str_replace( '#:', '', $search );
-			$search = str_replace( '#', '', $search );
-			$this->__set( 'give_forms', $search );
-			$this->__unset( 's' );
+			$search = str_replace('#:', '', $search);
+			$search = str_replace('#', '', $search);
+			$this->__set('give_forms', $search);
+			$this->__unset('s');
 
 		} else {
-			$this->__set( 's', $search );
+			$this->__set('s', $search);
 
 		}
 
@@ -639,8 +639,8 @@ 
 	 * @return void
 	 */
 	public function mode() {
-		if ( empty( $this->args['mode'] ) || $this->args['mode'] == 'all' ) {
-			$this->__unset( 'mode' );
+		if (empty($this->args['mode']) || $this->args['mode'] == 'all') {
+			$this->__unset('mode');
 
 			return;
 		}
@@ -662,10 +662,10 @@ 
 	 * @return void
 	 */
 	public function children() {
-		if ( empty( $this->args['children'] ) ) {
-			$this->__set( 'post_parent', 0 );
+		if (empty($this->args['children'])) {
+			$this->__set('post_parent', 0);
 		}
-		$this->__unset( 'children' );
+		$this->__unset('children');
 	}
 
 	/**
@@ -678,13 +678,13 @@ 
 	 */
 	public function give_forms() {
 
-		if ( empty( $this->args['give_forms'] ) ) {
+		if (empty($this->args['give_forms'])) {
 			return;
 		}
 
 		$compare = '=';
 
-		if ( is_array( $this->args['give_forms'] ) ) {
+		if (is_array($this->args['give_forms'])) {
 			$compare = 'IN';
 		}
 
@@ -697,7 +697,7 @@ 
 			)
 		);
 
-		$this->__unset( 'give_forms' );
+		$this->__unset('give_forms');
 
 	}
 
@@ -711,13 +711,13 @@ 
 	 */
 	public function gateway_filter() {
 
-		if ( empty( $this->args['gateway'] ) ) {
+		if (empty($this->args['gateway'])) {
 			return;
 		}
 
 		$compare = '=';
 
-		if ( is_array( $this->args['gateway'] ) ) {
+		if (is_array($this->args['gateway'])) {
 			$compare = 'IN';
 		}
 
@@ -731,7 +731,7 @@ 
 			)
 		);
 
-		$this->__unset( 'gateway' );
+		$this->__unset('gateway');
 
 	}
 
@@ -751,9 +751,9 @@ 
 		global $wpdb;
 
 		$where = "WHERE {$wpdb->posts}.post_type = 'give_payment'";
-		$where .= " AND {$wpdb->posts}.post_status IN ('" . implode( "','", $this->args['post_status'] ) . "')";
+		$where .= " AND {$wpdb->posts}.post_status IN ('".implode("','", $this->args['post_status'])."')";
 
-		if( is_numeric( $this->args['post_parent'] ) ) {
+		if (is_numeric($this->args['post_parent'])) {
 			$where .= " AND {$wpdb->posts}.post_parent={$this->args['post_parent']}";
 		}
 
@@ -762,42 +762,42 @@ 
 		$group_by = '';
 
 		// Set group by.
-		if ( ! empty( $this->args['group_by'] ) ) {
+		if ( ! empty($this->args['group_by'])) {
 			$group_by = "GROUP BY {$wpdb->posts}.{$this->args['group_by']}";
 		}
 
 		// Set offset.
 		if (
-			empty( $this->args['nopaging'] ) &&
-			empty( $this->args['offset'] ) &&
-			( ! empty( $this->args['page'] ) && 0 < $this->args['page'] )
+			empty($this->args['nopaging']) &&
+			empty($this->args['offset']) &&
+			( ! empty($this->args['page']) && 0 < $this->args['page'])
 		) {
-			$this->args['offset'] = $this->args['posts_per_page'] * ( $this->args['page'] - 1 );
+			$this->args['offset'] = $this->args['posts_per_page'] * ($this->args['page'] - 1);
 		}
 
 		// Set fields.
 		$fields = "{$wpdb->posts}.*";
-		if ( ! empty( $this->args['fields'] ) && 'all' !== $this->args['fields'] ) {
-			if ( is_string( $this->args['fields'] ) ) {
+		if ( ! empty($this->args['fields']) && 'all' !== $this->args['fields']) {
+			if (is_string($this->args['fields'])) {
 				$fields = "{$wpdb->posts}.{$this->args['fields']}";
-			} elseif ( is_array( $this->args['fields'] ) ) {
-				$fields = "{$wpdb->posts}." . implode( " , {$wpdb->posts}.", $this->args['fields'] );
+			} elseif (is_array($this->args['fields'])) {
+				$fields = "{$wpdb->posts}.".implode(" , {$wpdb->posts}.", $this->args['fields']);
 			}
 		}
 
 		// Set count.
-		if ( ! empty( $this->args['count'] ) ) {
+		if ( ! empty($this->args['count'])) {
 			$fields = "COUNT({$wpdb->posts}.ID)";
 
-			if ( ! empty( $this->args['group_by'] ) ) {
+			if ( ! empty($this->args['group_by'])) {
 				$fields = "{$wpdb->posts}.{$this->args['group_by']}, {$fields}";
 			}
 		}
 
 		// Date query.
-		if ( ! empty( $this->args['date_query'] ) ) {
-			$date_query_obj = new WP_Date_Query( $this->args['date_query'] );
-			$where          .= str_replace(
+		if ( ! empty($this->args['date_query'])) {
+			$date_query_obj = new WP_Date_Query($this->args['date_query']);
+			$where .= str_replace(
 				array(
 					"\n",
 					'(   (',
@@ -813,22 +813,22 @@ 
 		}
 
 		// Meta query.
-		if ( ! empty( $this->args['meta_query'] ) ) {
-			$meta_query_obj = new WP_Meta_Query( $this->args['meta_query'] );
-			$where          = implode( ' ', $meta_query_obj->get_sql( 'post', $wpdb->posts, 'ID' ) ) . " {$where}";
-			$where          = Give()->payment_meta->__rename_meta_table_name( $where, 'posts_where' );
+		if ( ! empty($this->args['meta_query'])) {
+			$meta_query_obj = new WP_Meta_Query($this->args['meta_query']);
+			$where          = implode(' ', $meta_query_obj->get_sql('post', $wpdb->posts, 'ID'))." {$where}";
+			$where          = Give()->payment_meta->__rename_meta_table_name($where, 'posts_where');
 		}
 
 		// Set sql query.
 		$sql = $wpdb->prepare(
 			"SELECT {$fields} FROM {$wpdb->posts} LIMIT %d,%d;",
-			absint( $this->args['offset'] ),
-			( empty( $this->args['nopaging'] ) ? absint( $this->args['posts_per_page'] ) : 99999999999 )
+			absint($this->args['offset']),
+			(empty($this->args['nopaging']) ? absint($this->args['posts_per_page']) : 99999999999)
 		);
 
 		// $where, $orderby and order already prepared query they can generate notice if you re prepare them in above.
 		// WordPress consider LIKE condition as placeholder if start with s,f, or d.
-		$sql = str_replace( 'LIMIT', "{$where} {$group_by} {$orderby} {$this->args['order']} LIMIT", $sql );
+		$sql = str_replace('LIMIT', "{$where} {$group_by} {$orderby} {$this->args['order']} LIMIT", $sql);
 
 		return $sql;
 	}

includes/admin/tools/logs/class-gateway-error-logs-list-table.php

Spacing +52 added lines, -52 removed lines

@@ -9,13 +9,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';
 }
 
 /**
@@ -44,11 +44,11 @@ 
 	 */
 	public function __construct() {
 		// Set parent defaults.
-		parent::__construct( array(
-			'singular' => give_get_forms_label_singular(),    // Singular name of the listed records.
-			'plural'   => give_get_forms_label_plural(),        // Plural name of the listed records.
-			'ajax'     => false,// Does this table support ajax?.
-		) );
+		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?.
+		));
 	}
 
 	/**
@@ -62,19 +62,19 @@ 
 	 *
 	 * @return string Column Name.
 	 */
-	public function column_default( $item, $column_name ) {
+	public function column_default($item, $column_name) {
 
-		switch ( $column_name ) {
+		switch ($column_name) {
 			case 'ID' :
 				return $item['ID_label'];
 			case 'payment_id' :
-				return empty( $item['payment_id'] ) ? esc_html__( 'n/a', 'give' ) : sprintf( "<a href=\"%s\" target=\"_blank\">{$item['payment_id']}</a>", get_edit_post_link( $item['payment_id'] ) );
+				return empty($item['payment_id']) ? esc_html__('n/a', 'give') : sprintf("<a href=\"%s\" target=\"_blank\">{$item['payment_id']}</a>", get_edit_post_link($item['payment_id']));
 			case 'gateway' :
-				return empty( $item['gateway'] ) ? esc_html__( 'n/a', 'give' ) : $item['gateway'];
+				return empty($item['gateway']) ? esc_html__('n/a', 'give') : $item['gateway'];
 			case 'error' :
-				return esc_html( $item['log_title'] );
+				return esc_html($item['log_title']);
 			default:
-				return $item[ $column_name ];
+				return $item[$column_name];
 		}
 	}
 
@@ -88,35 +88,35 @@ 
 	 *
 	 * @return void
 	 */
-	public function column_message( $item ) {
+	public function column_message($item) {
 		?>
 		<?php
-		echo Give()->tooltips->render_link( array(
-			'label'       => __( 'View Log Message', 'give' ),
+		echo Give()->tooltips->render_link(array(
+			'label'       => __('View Log Message', 'give'),
 			'tag_content' => '<span class="dashicons dashicons-visibility"></span>',
 			'link'        => "#TB_inline?width=640&amp;inlineId=log-message-{$item['ID']}",
 			'attributes'  => array(
 				'class' => 'thickbox give-error-log-details-link button button-small',
 			),
-		) );
+		));
 		?>
 		<div id="log-message-<?php echo $item['ID']; ?>" style="display:none;">
 			<?php
 
-			$serialized = strpos( $item['log_content'], '{"' );
+			$serialized = strpos($item['log_content'], '{"');
 
 			// Check to see if the log message contains serialized information
-			if ( $serialized !== false ) {
-				$length = strlen( $item['log_content'] ) - $serialized;
-				$intro  = substr( $item['log_content'], 0, - $length );
-				$data   = substr( $item['log_content'], $serialized, strlen( $item['log_content'] ) - 1 );
+			if ($serialized !== false) {
+				$length = strlen($item['log_content']) - $serialized;
+				$intro  = substr($item['log_content'], 0, - $length);
+				$data   = substr($item['log_content'], $serialized, strlen($item['log_content']) - 1);
 
-				echo wpautop( $intro );
-				echo wpautop( '<strong>' . esc_html__( 'Log data:', 'give' ) . '</strong>' );
-				echo '<div style="word-wrap: break-word;">' . wpautop( $data ) . '</div>';
+				echo wpautop($intro);
+				echo wpautop('<strong>'.esc_html__('Log data:', 'give').'</strong>');
+				echo '<div style="word-wrap: break-word;">'.wpautop($data).'</div>';
 			} else {
 				// No serialized data found
-				echo wpautop( $item['log_content'] );
+				echo wpautop($item['log_content']);
 			}
 			?>
 		</div>
@@ -132,12 +132,12 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'ID'         => esc_html__( 'Log ID', 'give' ),
-			'error'      => esc_html__( 'Error', 'give' ),
-			'gateway'    => esc_html__( 'Gateway', 'give' ),
-			'payment_id' => esc_html__( 'Donation ID', 'give' ),
-			'date'       => esc_html__( 'Date', 'give' ),
-			'message'    => esc_html__( 'Details', 'give' ),
+			'ID'         => esc_html__('Log ID', 'give'),
+			'error'      => esc_html__('Error', 'give'),
+			'gateway'    => esc_html__('Gateway', 'give'),
+			'payment_id' => esc_html__('Donation ID', 'give'),
+			'date'       => esc_html__('Date', 'give'),
+			'message'    => esc_html__('Details', 'give'),
 		);
 
 		return $columns;
@@ -151,7 +151,7 @@ 
 	 * @return int Current page number
 	 */
 	public function get_paged() {
-		return isset( $_GET['paged'] ) ? absint( $_GET['paged'] ) : 1;
+		return isset($_GET['paged']) ? absint($_GET['paged']) : 1;
 	}
 
 	/**
@@ -161,7 +161,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	public function bulk_actions( $which = '' ) {
+	public function bulk_actions($which = '') {
 		give_log_views();
 	}
 
@@ -176,7 +176,7 @@ 
 	public function get_logs() {
 		// Prevent the queries from getting cached.
 		// Without this there are occasional memory issues for some installs.
-		wp_suspend_cache_addition( true );
+		wp_suspend_cache_addition(true);
 
 		$logs_data = array();
 		$paged     = $this->get_paged();
@@ -186,17 +186,17 @@ 
 			'posts_per_page' => $this->per_page,
 		);
 
-		$logs = Give()->logs->get_connected_logs( $log_query );
+		$logs = Give()->logs->get_connected_logs($log_query);
 
-		if ( $logs ) {
-			foreach ( $logs as $log ) {
+		if ($logs) {
+			foreach ($logs as $log) {
 
 				$logs_data[] = array(
 					'ID'          => $log->ID,
-					'ID_label'    => '<span class=\'give-item-label give-item-label-gray\'>' . $log->ID . '</span>',
+					'ID_label'    => '<span class=\'give-item-label give-item-label-gray\'>'.$log->ID.'</span>',
 					'payment_id'  => $log->log_parent,
 					'error'       => 'error',
-					'gateway'     => give_get_payment_gateway( $log->log_parent ),
+					'gateway'     => give_get_payment_gateway($log->log_parent),
 					'date'        => $log->log_date,
 					'log_content' => $log->log_content,
 					'log_title'   => $log->log_title,
@@ -220,19 +220,19 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
-		if ( 'top' === $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+	protected function display_tablenav($which) {
+		if ('top' === $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav <?php echo esc_attr($which); ?>">
 
 			<div class="alignleft actions bulkactions">
-				<?php $this->bulk_actions( $which ); ?>
+				<?php $this->bulk_actions($which); ?>
 			</div>
 			<?php
-			$this->extra_tablenav( $which );
-			$this->pagination( $which );
+			$this->extra_tablenav($which);
+			$this->pagination($which);
 			?>
 
 			<br class="clear"/>
@@ -258,14 +258,14 @@ 
 		$columns               = $this->get_columns();
 		$hidden                = array(); // No hidden columns
 		$sortable              = $this->get_sortable_columns();
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 		$this->items           = $this->get_logs();
-		$total_items           = Give()->logs->get_log_count( 0, 'gateway_error' );
+		$total_items           = Give()->logs->get_log_count(0, 'gateway_error');
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 				'total_items' => $total_items,
 				'per_page'    => $this->per_page,
-				'total_pages' => ceil( $total_items / $this->per_page ),
+				'total_pages' => ceil($total_items / $this->per_page),
 			)
 		);
 	}

includes/admin/emails/filters.php

Spacing +12 added lines, -12 removed lines

@@ -19,28 +19,28 @@ 
  *
  * @return array
  */
-function give_email_notification_row_actions_callback( $row_actions, $email ) {
-	if( Give_Email_Notification_Util::is_email_preview( $email ) ) {
+function give_email_notification_row_actions_callback($row_actions, $email) {
+	if (Give_Email_Notification_Util::is_email_preview($email)) {
 		$preview_link = sprintf(
 			'<a href="%1$s" target="_blank">%2$s</a>',
 			wp_nonce_url(
 				add_query_arg(
-					array( 'give_action' => 'preview_email', 'email_type' => $email->config['id'] ),
+					array('give_action' => 'preview_email', 'email_type' => $email->config['id']),
 					home_url()
 				), 'give-preview-email'
 			),
-			__( 'Preview', 'give' )
+			__('Preview', 'give')
 		);
 
 		$send_preview_email_link = sprintf(
 			'<a href="%1$s">%2$s</a>',
 			wp_nonce_url(
-				add_query_arg( array(
+				add_query_arg(array(
 					'give_action'  => 'send_preview_email',
 					'email_type' => $email->config['id'],
 					'give-message' => 'sent-test-email',
-				) ), 'give-send-preview-email' ),
-			__( 'Send test email', 'give' )
+				)), 'give-send-preview-email' ),
+			__('Send test email', 'give')
 		);
 
 		$row_actions['email_preview'] = $preview_link;
@@ -49,7 +49,7 @@ 
 
 	return $row_actions;
 }
-add_filter( 'give_email_notification_row_actions', 'give_email_notification_row_actions_callback', 10, 2 );
+add_filter('give_email_notification_row_actions', 'give_email_notification_row_actions_callback', 10, 2);
 
 /**
  * This help to decode all email template tags.
@@ -61,12 +61,12 @@ 
  *
  * @return string
  */
-function give_decode_email_tags( $message, $email_obj ) {
-	if ( ! empty( $email_obj->tag_args ) ) {
-		$message = give_do_email_tags( $message, $email_obj->tag_args );
+function give_decode_email_tags($message, $email_obj) {
+	if ( ! empty($email_obj->tag_args)) {
+		$message = give_do_email_tags($message, $email_obj->tag_args);
 	}
 
 	return $message;
 }
 
-add_filter( 'give_email_message', 'give_decode_email_tags', 10, 2 );
\ No newline at end of file
+add_filter('give_email_message', 'give_decode_email_tags', 10, 2);
\ No newline at end of file

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

Indentation +4 added lines, -4 removed lines

@@ -175,10 +175,10 @@
 		 */
 		public function get_default_email_message() {
 			$message = sprintf(
-				           __( 'Please click the link to access your donation history on <a target="_blank" href="%1$s">%1$s</a>. If you did not request this email, please contact <a href="mailto:%2$s">%2$s</a>.', 'give' ),
-				           get_bloginfo( 'url' ),
-				           get_bloginfo( 'admin_email' )
-			           ) . "\n\n";
+						   __( 'Please click the link to access your donation history on <a target="_blank" href="%1$s">%1$s</a>. If you did not request this email, please contact <a href="mailto:%2$s">%2$s</a>.', 'give' ),
+						   get_bloginfo( 'url' ),
+						   get_bloginfo( 'admin_email' )
+					   ) . "\n\n";
 			$message .= '{email_access_link}' . "\n\n";
 			$message .= "\n\n";
 			$message .= __( 'Sincerely,', 'give' ) . "\n";

Spacing +52 added lines, -52 removed lines

@@ -11,11 +11,11 @@ 
  */
 
 // Exit if access directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! class_exists( 'Give_Email_Access_Email' ) ) :
+if ( ! class_exists('Give_Email_Access_Email')) :
 
 	/**
 	 * Give_Email_Access_Email
@@ -31,31 +31,31 @@ 
 		 * @since  2.0
 		 */
 		public function init() {
-			$this->load( array(
+			$this->load(array(
 				'id'                           => 'email-access',
-				'label'                        => __( 'Email access', 'give' ),
-				'description'                  => __( 'Sent when donors request access to their donation history using only their email as verification. (See Settings > General > Access Control)', 'give' ),
-				'notification_status'          => give_get_option( 'email_access', 'disabled' ),
+				'label'                        => __('Email access', 'give'),
+				'description'                  => __('Sent when donors request access to their donation history using only their email as verification. (See Settings > General > Access Control)', 'give'),
+				'notification_status'          => give_get_option('email_access', 'disabled'),
 				'form_metabox_setting'         => false,
 				'notification_status_editable' => false,
 				'email_tag_context'            => 'donor',
-				'recipient_group_name'         => __( 'Donor', 'give' ),
-				'default_email_subject'        => sprintf( __( 'Please confirm your email for %s', 'give' ), get_bloginfo( 'url' ) ),
+				'recipient_group_name'         => __('Donor', 'give'),
+				'default_email_subject'        => sprintf(__('Please confirm your email for %s', 'give'), get_bloginfo('url')),
 				'default_email_message'        => $this->get_default_email_message(),
-				'default_email_header'         => __( 'Confirm Email', 'give' ),
+				'default_email_header'         => __('Confirm Email', 'give'),
 				'notices' => array(
 					'non-notification-status-editable' => sprintf(
 						'%1$s <a href="%2$s">%3$s &raquo;</a>',
-						__( 'This notification is automatically toggled based on whether the email access is enabled or not.', 'give' ),
-						esc_url( admin_url('edit.php?post_type=give_forms&page=give-settings&tab=general&section=access-control') ),
-						__( 'Edit Setting', 'give' )
+						__('This notification is automatically toggled based on whether the email access is enabled or not.', 'give'),
+						esc_url(admin_url('edit.php?post_type=give_forms&page=give-settings&tab=general&section=access-control')),
+						__('Edit Setting', 'give')
 					)
 				),
-			) );
+			));
 
-			add_filter( "give_{$this->config['id']}_email_notification", array( $this, 'setup_email_notification' ), 10, 2 );
-			add_action( 'give_save_settings_give_settings', array( $this, 'set_notification_status' ), 10, 2 );
-			add_filter( 'give_email_preview_header', array( $this, 'email_preview_header' ), 10, 2 );
+			add_filter("give_{$this->config['id']}_email_notification", array($this, 'setup_email_notification'), 10, 2);
+			add_action('give_save_settings_give_settings', array($this, 'set_notification_status'), 10, 2);
+			add_filter('give_email_preview_header', array($this, 'email_preview_header'), 10, 2);
 		}
 
 		/**
@@ -68,11 +68,11 @@ 
 		 *
 		 * @return string
 		 */
-		public function get_email_subject( $form_id = null ) {
+		public 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']
 				)
@@ -84,14 +84,14 @@ 
 			 *
 			 * @since 1.0
 			 */
-			$subject = apply_filters( 'give_email_access_token_subject', $subject );
+			$subject = apply_filters('give_email_access_token_subject', $subject);
 
 			/**
 			 * Filters the donation notification subject.
 			 *
 			 * @since 2.0
 			 */
-			$subject = apply_filters( "give_{$this->config['id']}_get_email_subject", $subject, $this, $form_id );
+			$subject = apply_filters("give_{$this->config['id']}_get_email_subject", $subject, $this, $form_id);
 
 			return $subject;
 		}
@@ -107,10 +107,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']
 			);
@@ -121,14 +121,14 @@ 
 			 *
 			 * @since 1.0
 			 */
-			$message = apply_filters( 'give_email_access_token_message', $message );
+			$message = apply_filters('give_email_access_token_message', $message);
 
 			/**
 			 * Filter the email message
 			 *
 			 * @since 2.0
 			 */
-			$message = apply_filters( "give_{$this->config['id']}_get_default_email_message", $message, $this, $form_id );
+			$message = apply_filters("give_{$this->config['id']}_get_default_email_message", $message, $this, $form_id);
 
 			return $message;
 		}
@@ -143,7 +143,7 @@ 
 		 * @param int $form_id
 		 * @return array
 		 */
-		public function get_email_attachments( $form_id = null ) {
+		public function get_email_attachments($form_id = null) {
 			/**
 			 * Filters the donation notification email attachments.
 			 * By default, there is no attachment but plugins can hook in to provide one more multiple.
@@ -151,7 +151,7 @@ 
 			 *
 			 * @since 1.0
 			 */
-			$attachments = apply_filters( 'give_admin_donation_notification_attachments', array() );
+			$attachments = apply_filters('give_admin_donation_notification_attachments', array());
 
 			/**
 			 * Filters the donation notification email attachments.
@@ -159,7 +159,7 @@ 
 			 *
 			 * @since 2.0
 			 */
-			$attachments = apply_filters( "give_{$this->config['id']}_get_email_attachments", $attachments, $this, $form_id );
+			$attachments = apply_filters("give_{$this->config['id']}_get_email_attachments", $attachments, $this, $form_id);
 
 			return $attachments;
 		}
@@ -175,14 +175,14 @@ 
 		 */
 		public function get_default_email_message() {
 			$message = sprintf(
-				           __( 'Please click the link to access your donation history on <a target="_blank" href="%1$s">%1$s</a>. If you did not request this email, please contact <a href="mailto:%2$s">%2$s</a>.', 'give' ),
-				           get_bloginfo( 'url' ),
-				           get_bloginfo( 'admin_email' )
-			           ) . "\n\n";
-			$message .= '{email_access_link}' . "\n\n";
+				           __('Please click the link to access your donation history on <a target="_blank" href="%1$s">%1$s</a>. If you did not request this email, please contact <a href="mailto:%2$s">%2$s</a>.', 'give'),
+				           get_bloginfo('url'),
+				           get_bloginfo('admin_email')
+			           )."\n\n";
+			$message .= '{email_access_link}'."\n\n";
 			$message .= "\n\n";
-			$message .= __( 'Sincerely,', 'give' ) . "\n";
-			$message .= get_bloginfo( 'name' ) . "\n";
+			$message .= __('Sincerely,', 'give')."\n";
+			$message .= get_bloginfo('name')."\n";
 
 			/**
 			 * Filter the new donation email message
@@ -191,7 +191,7 @@ 
 			 *
 			 * @param string $message
 			 */
-			return apply_filters( "give_{$this->config['id']}_get_default_email_message", $message, $this );
+			return apply_filters("give_{$this->config['id']}_get_default_email_message", $message, $this);
 		}
 
 
@@ -207,7 +207,7 @@ 
 			 *
 			 * @since 1.0
 			 */
-			$from_name = apply_filters( 'give_donation_from_name', Give()->emails->get_from_name() );
+			$from_name = apply_filters('give_donation_from_name', Give()->emails->get_from_name());
 
 			/**
 			 * Filters the from email.
@@ -215,19 +215,19 @@ 
 			 *
 			 * @since 1.0
 			 */
-			$from_email = apply_filters( 'give_donation_from_address', Give()->emails->get_from_address() );
+			$from_email = apply_filters('give_donation_from_address', Give()->emails->get_from_address());
 
-			Give()->emails->__set( 'from_name', $from_name );
-			Give()->emails->__set( 'from_email', $from_email );
-			Give()->emails->__set( 'heading', apply_filters( 'give_email_access_token_heading', $this->get_email_header() ) );
+			Give()->emails->__set('from_name', $from_name);
+			Give()->emails->__set('from_email', $from_email);
+			Give()->emails->__set('heading', apply_filters('give_email_access_token_heading', $this->get_email_header()));
 			/**
 			 * Filters the donation notification email headers.
 			 *
 			 * @since 1.0
 			 */
-			$headers = apply_filters( 'give_admin_donation_notification_headers', Give()->emails->get_headers() );
+			$headers = apply_filters('give_admin_donation_notification_headers', Give()->emails->get_headers());
 
-			Give()->emails->__set( 'headers', $headers );
+			Give()->emails->__set('headers', $headers);
 		}
 
 		/**
@@ -241,8 +241,8 @@ 
 		 *
 		 * @return bool
 		 */
-		public function setup_email_notification( $donor_id, $email ) {
-			$donor = Give()->donors->get_donor_by( 'email', $email );
+		public function setup_email_notification($donor_id, $email) {
+			$donor = Give()->donors->get_donor_by('email', $email);
 			$this->recipient_email = $email;
 
 			// Set email data.
@@ -267,17 +267,17 @@ 
 		 * @param $update_options
 		 * @param $option_name
 		 */
-		public function set_notification_status( $update_options, $option_name ) {
+		public function set_notification_status($update_options, $option_name) {
 			// Get updated settings.
 			$update_options = give_get_settings();
 
 			if (
-				! empty( $update_options['email_access'] )
-				&& ! empty( $update_options[ "{$this->config['id']}_notification" ] )
-				&& $update_options['email_access'] !== $update_options[ "{$this->config['id']}_notification" ]
+				! empty($update_options['email_access'])
+				&& ! empty($update_options["{$this->config['id']}_notification"])
+				&& $update_options['email_access'] !== $update_options["{$this->config['id']}_notification"]
 			) {
-				$update_options[ "{$this->config['id']}_notification" ] = $update_options['email_access'];
-				update_option( $option_name, $update_options );
+				$update_options["{$this->config['id']}_notification"] = $update_options['email_access'];
+				update_option($option_name, $update_options);
 			}
 		}
 
@@ -292,8 +292,8 @@ 
 		 * @param Give_Email_Access_Email $email
 		 * @return string
 		 */
-		public function email_preview_header( $email_preview_header, $email ) {
-			if( $this->config['id'] === $email->config['id'] ) {
+		public function email_preview_header($email_preview_header, $email) {
+			if ($this->config['id'] === $email->config['id']) {
 				$email_preview_header = '';
 			}
 

includes/admin/emails/ajax-handler.php

Spacing +3 added lines, -3 removed lines

@@ -16,12 +16,12 @@
  * @since 2.0
  */
 function give_set_notification_status_handler() {
-	$notification_id = isset( $_POST['notification_id'] ) ? give_clean( $_POST['notification_id'] ) : '';
-	if ( ! empty( $notification_id ) && give_update_option( "{$notification_id}_notification", give_clean( $_POST['status'] ) ) ) {
+	$notification_id = isset($_POST['notification_id']) ? give_clean($_POST['notification_id']) : '';
+	if ( ! empty($notification_id) && give_update_option("{$notification_id}_notification", give_clean($_POST['status']))) {
 		wp_send_json_success();
 	}
 
 	wp_send_json_error();
 }
 
-add_action( 'wp_ajax_give_set_notification_status', 'give_set_notification_status_handler' );
+add_action('wp_ajax_give_set_notification_status', 'give_set_notification_status_handler');

includes/admin/emails/backward-compatibility.php

Spacing +27 added lines, -27 removed lines

@@ -11,15 +11,15 @@ 
  *
  * @return string
  */
-function _give_bc_offline_donation_instruction_notification_status( $notification_status, $email, $form_id ) {
+function _give_bc_offline_donation_instruction_notification_status($notification_status, $email, $form_id) {
 	// Bailout.
-	if ( ! $form_id ) {
+	if ( ! $form_id) {
 		return $notification_status;
 	}
 
-	if ( ! get_post_meta( $form_id, '_give_offline-donation-instruction_notification', true ) ) {
-		$old_value           = get_post_meta( $form_id, '_give_customize_offline_donations', true );
-		$notification_status = give_is_setting_enabled( $old_value, array( 'enabled', 'global' ) )
+	if ( ! get_post_meta($form_id, '_give_offline-donation-instruction_notification', true)) {
+		$old_value           = get_post_meta($form_id, '_give_customize_offline_donations', true);
+		$notification_status = give_is_setting_enabled($old_value, array('enabled', 'global'))
 			? $old_value
 			: 'global';
 	}
@@ -28,7 +28,7 @@ 
 	return $notification_status;
 }
 
-add_filter( 'give__give_offline-donation-instruction_get_notification_status', '_give_bc_offline_donation_instruction_notification_status', 10, 3 );
+add_filter('give__give_offline-donation-instruction_get_notification_status', '_give_bc_offline_donation_instruction_notification_status', 10, 3);
 
 
 /**
@@ -43,22 +43,22 @@ 
  *
  * @return mixed
  */
-function _give_bc_offline_donation_instruction_email_setting_values( $option_value, $option_name, $email, $form_id ) {
+function _give_bc_offline_donation_instruction_email_setting_values($option_value, $option_name, $email, $form_id) {
 	// Bailout.
-	if ( empty( $form_id ) || 'offline-donation-instruction' !== $email->config['id'] ) {
+	if (empty($form_id) || 'offline-donation-instruction' !== $email->config['id']) {
 		return $option_value;
 	}
 
-	switch ( $option_name ) {
+	switch ($option_name) {
 		case '_give_offline-donation-instruction_email_message':
-			if ( ! get_post_meta( $form_id, $option_name, true ) && give_is_setting_enabled( $email->get_notification_status( $form_id ) ) ) {
-				$option_value = get_post_meta( $form_id, '_give_offline_donation_email', true );
+			if ( ! get_post_meta($form_id, $option_name, true) && give_is_setting_enabled($email->get_notification_status($form_id))) {
+				$option_value = get_post_meta($form_id, '_give_offline_donation_email', true);
 			}
 			break;
 
 		case '_give_offline-donation-instruction_email_subject':
-			if ( ! get_post_meta( $form_id, $option_name, true ) && give_is_setting_enabled( $email->get_notification_status( $form_id ) ) ) {
-				$option_value = get_post_meta( $form_id, '_give_offline_donation_subject', true );
+			if ( ! get_post_meta($form_id, $option_name, true) && give_is_setting_enabled($email->get_notification_status($form_id))) {
+				$option_value = get_post_meta($form_id, '_give_offline_donation_subject', true);
 			}
 			break;
 	}
@@ -66,7 +66,7 @@ 
 	return $option_value;
 }
 
-add_filter( 'give_email_setting_value', '_give_bc_offline_donation_instruction_email_setting_values', 10, 4 );
+add_filter('give_email_setting_value', '_give_bc_offline_donation_instruction_email_setting_values', 10, 4);
 
 
 /**
@@ -80,10 +80,10 @@ 
  *
  * @return string
  */
-function _give_bc_offline_instruction_status_setting_value( $field_value, $field, $form_id ) {
-	if ( ! get_post_meta( $form_id, $field['id'], true ) ) {
-		$old_value   = get_post_meta( $form_id, '_give_customize_offline_donations', true );
-		$field_value = give_is_setting_enabled( $old_value, array( 'enabled', 'global' ) )
+function _give_bc_offline_instruction_status_setting_value($field_value, $field, $form_id) {
+	if ( ! get_post_meta($form_id, $field['id'], true)) {
+		$old_value   = get_post_meta($form_id, '_give_customize_offline_donations', true);
+		$field_value = give_is_setting_enabled($old_value, array('enabled', 'global'))
 			? $old_value
 			: 'global';
 	}
@@ -91,7 +91,7 @@ 
 	return $field_value;
 }
 
-add_filter( '_give_offline-donation-instruction_notification_field_value', '_give_bc_offline_instruction_status_setting_value', 10, 3 );
+add_filter('_give_offline-donation-instruction_notification_field_value', '_give_bc_offline_instruction_status_setting_value', 10, 3);
 
 
 /**
@@ -105,15 +105,15 @@ 
  *
  * @return string
  */
-function _offline_donation_instruction_email_subject_setting_value( $field_value, $field, $form_id ) {
-	if ( ! get_post_meta( $form_id, $field['id'], true ) ) {
-		$field_value = get_post_meta( $form_id, '_give_offline_donation_subject', true );
+function _offline_donation_instruction_email_subject_setting_value($field_value, $field, $form_id) {
+	if ( ! get_post_meta($form_id, $field['id'], true)) {
+		$field_value = get_post_meta($form_id, '_give_offline_donation_subject', true);
 	}
 
 	return $field_value;
 }
 
-add_filter( '_give_offline-donation-instruction_email_subject_field_value', '_offline_donation_instruction_email_subject_setting_value', 10, 3 );
+add_filter('_give_offline-donation-instruction_email_subject_field_value', '_offline_donation_instruction_email_subject_setting_value', 10, 3);
 
 
 /**
@@ -127,12 +127,12 @@ 
  *
  * @return string
  */
-function _give_bc_offline_donation_instruction_email_message_setting_value( $field_value, $field, $form_id ) {
-	if ( ! get_post_meta( $form_id, $field['id'], true ) ) {
-		$field_value = get_post_meta( $form_id, '_give_offline_donation_email', true );
+function _give_bc_offline_donation_instruction_email_message_setting_value($field_value, $field, $form_id) {
+	if ( ! get_post_meta($form_id, $field['id'], true)) {
+		$field_value = get_post_meta($form_id, '_give_offline_donation_email', true);
 	}
 
 	return $field_value;
 }
 
-add_filter( '_give_offline-donation-instruction_email_message_field_value', '_give_bc_offline_donation_instruction_email_message_setting_value', 10, 3 );
\ No newline at end of file
+add_filter('_give_offline-donation-instruction_email_message_field_value', '_give_bc_offline_donation_instruction_email_message_setting_value', 10, 3);
\ No newline at end of file

includes/admin/reports/reports.php

Spacing +59 added lines, -59 removed lines

@@ -16,7 +16,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -29,8 +29,8 @@ 
  * @return void
  */
 function give_reports_page() {
-	$current_page = admin_url( 'edit.php?post_type=give_forms&page=give-reports' );
-	$active_tab   = isset( $_GET['tab'] ) ? sanitize_text_field( $_GET['tab'] ) : 'earnings';
+	$current_page = admin_url('edit.php?post_type=give_forms&page=give-reports');
+	$active_tab   = isset($_GET['tab']) ? sanitize_text_field($_GET['tab']) : 'earnings';
 	$views        = give_reports_default_views();
 	?>
 	<div class="wrap give-settings-page">
@@ -38,17 +38,17 @@ 
 		<h1 class="screen-reader-text"><?php echo get_admin_page_title(); ?></h1>
 
 		<h2 class="nav-tab-wrapper">
-			<?php foreach ( $views as $tab => $label ) { ?>
-				<a href="<?php echo esc_url( add_query_arg( array(
+			<?php foreach ($views as $tab => $label) { ?>
+				<a href="<?php echo esc_url(add_query_arg(array(
 					'tab'              => $tab,
 					'settings-updated' => false,
-				), $current_page ) ); ?>" class="nav-tab <?php echo $tab === $active_tab ? esc_attr( 'nav-tab-active' ) : ''; ?>"><?php echo esc_html( $label ); ?></a>
+				), $current_page)); ?>" class="nav-tab <?php echo $tab === $active_tab ? esc_attr('nav-tab-active') : ''; ?>"><?php echo esc_html($label); ?></a>
 			<?php } ?>
-			<?php if ( current_user_can( 'export_give_reports' ) ) { ?>
-				<a href="<?php echo esc_url( add_query_arg( array(
+			<?php if (current_user_can('export_give_reports')) { ?>
+				<a href="<?php echo esc_url(add_query_arg(array(
 					'tab'              => 'export',
 					'settings-updated' => false,
-				), $current_page ) ); ?>" class="nav-tab <?php echo 'export' === $active_tab ? esc_attr( 'nav-tab-active' ) : ''; ?>"><?php esc_html_e( 'Export', 'give' ); ?></a>
+				), $current_page)); ?>" class="nav-tab <?php echo 'export' === $active_tab ? esc_attr('nav-tab-active') : ''; ?>"><?php esc_html_e('Export', 'give'); ?></a>
 			<?php }
 			/**
 			 * Fires in the report tabs.
@@ -57,7 +57,7 @@ 
 			 *
 			 * @since 1.0
 			 */
-			do_action( 'give_reports_tabs' );
+			do_action('give_reports_tabs');
 			?>
 		</h2>
 
@@ -67,10 +67,10 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_reports_page_top' );
+		do_action('give_reports_page_top');
 
 		// Set $active_tab prior to hook firing.
-		if ( in_array( $active_tab, array_keys( $views ) ) ) {
+		if (in_array($active_tab, array_keys($views))) {
 			$active_tab = 'reports';
 		}
 
@@ -79,14 +79,14 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( "give_reports_tab_{$active_tab}" );
+		do_action("give_reports_tab_{$active_tab}");
 
 		/**
 		 * Fires after the report page.
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_reports_page_bottom' );
+		do_action('give_reports_page_bottom');
 		?>
 	</div><!-- .wrap -->
 	<?php
@@ -100,12 +100,12 @@ 
  */
 function give_reports_default_views() {
 	$views = array(
-		'earnings' => esc_html__( 'Income', 'give' ),
-		'forms'    => esc_html__( 'Forms', 'give' ),
-		'gateways' => esc_html__( 'Donation Methods', 'give' ),
+		'earnings' => esc_html__('Income', 'give'),
+		'forms'    => esc_html__('Forms', 'give'),
+		'gateways' => esc_html__('Donation Methods', 'give'),
 	);
 
-	$views = apply_filters( 'give_report_views', $views );
+	$views = apply_filters('give_report_views', $views);
 
 	return $views;
 }
@@ -120,15 +120,15 @@ 
  * @since 1.0
  * @return string $view Report View
  */
-function give_get_reporting_view( $default = 'earnings' ) {
+function give_get_reporting_view($default = 'earnings') {
 
-	if ( ! isset( $_GET['view'] ) || ! in_array( $_GET['view'], array_keys( give_reports_default_views() ) ) ) {
+	if ( ! isset($_GET['view']) || ! in_array($_GET['view'], array_keys(give_reports_default_views()))) {
 		$view = $default;
 	} else {
 		$view = $_GET['view'];
 	}
 
-	return apply_filters( 'give_get_reporting_view', $view );
+	return apply_filters('give_get_reporting_view', $view);
 }
 
 /**
@@ -139,14 +139,14 @@ 
  */
 function give_reports_tab_reports() {
 
-	if( ! current_user_can( 'view_give_reports' ) ) {
-		wp_die( __( 'You do not have permission to access this report', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! current_user_can('view_give_reports')) {
+		wp_die(__('You do not have permission to access this report', 'give'), __('Error', 'give'), array('response' => 403));
 	}
 
 	$current_view = 'earnings';
 	$views        = give_reports_default_views();
 
-	if ( isset( $_GET['tab'] ) && array_key_exists( $_GET['tab'], $views ) ) {
+	if (isset($_GET['tab']) && array_key_exists($_GET['tab'], $views)) {
 		$current_view = $_GET['tab'];
 	}
 
@@ -155,10 +155,10 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( "give_reports_view_{$current_view}" );
+	do_action("give_reports_view_{$current_view}");
 }
 
-add_action( 'give_reports_tab_reports', 'give_reports_tab_reports' );
+add_action('give_reports_tab_reports', 'give_reports_tab_reports');
 
 /**
  * Renders the Reports Page Views Drop Downs
@@ -168,19 +168,19 @@ 
  */
 function give_report_views() {
 	$views        = give_reports_default_views();
-	$current_view = isset( $_GET['view'] ) ? $_GET['view'] : 'earnings';
+	$current_view = isset($_GET['view']) ? $_GET['view'] : 'earnings';
 	/**
 	 * Fires before the report page actions form.
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_report_view_actions_before' );
+	do_action('give_report_view_actions_before');
 	?>
 	<form id="give-reports-filter" method="get">
 		<select id="give-reports-view" name="view">
-			<option value="-1"><?php esc_html_e( 'Report Type', 'give' ); ?></option>
-			<?php foreach ( $views as $view_id => $label ) : ?>
-				<option value="<?php echo esc_attr( $view_id ); ?>" <?php selected( $view_id, $current_view ); ?>><?php echo $label; ?></option>
+			<option value="-1"><?php esc_html_e('Report Type', 'give'); ?></option>
+			<?php foreach ($views as $view_id => $label) : ?>
+				<option value="<?php echo esc_attr($view_id); ?>" <?php selected($view_id, $current_view); ?>><?php echo $label; ?></option>
 			<?php endforeach; ?>
 		</select>
 
@@ -192,12 +192,12 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_report_view_actions' );
+		do_action('give_report_view_actions');
 		?>
 
 		<input type="hidden" name="post_type" value="give_forms"/>
 		<input type="hidden" name="page" value="give-reports"/>
-		<?php submit_button( esc_html__( 'Show', 'give' ), 'secondary', 'submit', false ); ?>
+		<?php submit_button(esc_html__('Show', 'give'), 'secondary', 'submit', false); ?>
 	</form>
 	<?php
 	/**
@@ -205,7 +205,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_report_view_actions_after' );
+	do_action('give_report_view_actions_after');
 }
 
 /**
@@ -218,11 +218,11 @@ 
  */
 function give_reports_forms_table() {
 
-	if ( isset( $_GET['form-id'] ) ) {
+	if (isset($_GET['form-id'])) {
 		return;
 	}
 
-	include GIVE_PLUGIN_DIR . 'includes/admin/reports/class-form-reports-table.php';
+	include GIVE_PLUGIN_DIR.'includes/admin/reports/class-form-reports-table.php';
 
 	$give_table = new Give_Form_Reports_Table();
 	$give_table->prepare_items();
@@ -234,7 +234,7 @@ 
 	<?php
 }
 
-add_action( 'give_reports_view_forms', 'give_reports_forms_table' );
+add_action('give_reports_view_forms', 'give_reports_forms_table');
 
 /**
  * Renders the detailed report for a specific give form.
@@ -243,20 +243,20 @@ 
  * @return void
  */
 function give_reports_form_details() {
-	if ( ! isset( $_GET['form-id'] ) ) {
+	if ( ! isset($_GET['form-id'])) {
 		return;
 	}
 	?>
 	<div class="tablenav top reports-forms-details-wrap">
 		<div class="actions bulkactions">
-			<button onclick="history.go(-1);" class="button-secondary"><?php esc_html_e( 'Go Back', 'give' ); ?></button>
+			<button onclick="history.go(-1);" class="button-secondary"><?php esc_html_e('Go Back', 'give'); ?></button>
 		</div>
 	</div>
 	<?php
-	give_reports_graph_of_form( absint( $_GET['form-id'] ) );
+	give_reports_graph_of_form(absint($_GET['form-id']));
 }
 
-add_action( 'give_reports_view_forms', 'give_reports_form_details' );
+add_action('give_reports_view_forms', 'give_reports_form_details');
 
 /**
  * Renders the Gateways Table
@@ -267,14 +267,14 @@ 
  * @return void
  */
 function give_reports_gateways_table() {
-	include GIVE_PLUGIN_DIR . 'includes/admin/reports/class-gateways-reports-table.php';
+	include GIVE_PLUGIN_DIR.'includes/admin/reports/class-gateways-reports-table.php';
 
 	$give_table = new Give_Gateway_Reports_Table();
 	$give_table->prepare_items();
 	$give_table->display();
 }
 
-add_action( 'give_reports_view_gateways', 'give_reports_gateways_table' );
+add_action('give_reports_view_gateways', 'give_reports_gateways_table');
 
 /**
  * Renders the Reports Earnings Graphs
@@ -285,13 +285,13 @@ 
 function give_reports_earnings() {
 	?>
 	<div class="tablenav top reports-table-nav">
-		<h2 class="reports-earnings-title screen-reader-text"><?php _e( 'Income Report', 'give' ); ?></h2>
+		<h2 class="reports-earnings-title screen-reader-text"><?php _e('Income Report', 'give'); ?></h2>
 	</div>
 	<?php
 	give_reports_graph();
 }
 
-add_action( 'give_reports_view_earnings', 'give_reports_earnings' );
+add_action('give_reports_view_earnings', 'give_reports_earnings');
 
 
 /**
@@ -302,9 +302,9 @@ 
  */
 function give_estimated_monthly_stats() {
 
-	$estimated = Give_Cache::get( 'give_estimated_monthly_stats', true );
+	$estimated = Give_Cache::get('give_estimated_monthly_stats', true);
 
-	if ( false === $estimated ) {
+	if (false === $estimated) {
 
 		$estimated = array(
 			'earnings' => 0,
@@ -313,22 +313,22 @@ 
 
 		$stats = new Give_Payment_Stats;
 
-		$to_date_earnings = $stats->get_earnings( 0, 'this_month' );
-		$to_date_sales    = $stats->get_sales( 0, 'this_month' );
+		$to_date_earnings = $stats->get_earnings(0, 'this_month');
+		$to_date_sales    = $stats->get_sales(0, 'this_month');
 
-		$current_day   = date( 'd', current_time( 'timestamp' ) );
-		$current_month = date( 'n', current_time( 'timestamp' ) );
-		$current_year  = date( 'Y', current_time( 'timestamp' ) );
-		$days_in_month = cal_days_in_month( CAL_GREGORIAN, $current_month, $current_year );
+		$current_day   = date('d', current_time('timestamp'));
+		$current_month = date('n', current_time('timestamp'));
+		$current_year  = date('Y', current_time('timestamp'));
+		$days_in_month = cal_days_in_month(CAL_GREGORIAN, $current_month, $current_year);
 
-		$estimated['earnings'] = ( $to_date_earnings / $current_day ) * $days_in_month;
-		$estimated['sales']    = ( $to_date_sales / $current_day ) * $days_in_month;
+		$estimated['earnings'] = ($to_date_earnings / $current_day) * $days_in_month;
+		$estimated['sales']    = ($to_date_sales / $current_day) * $days_in_month;
 
 		// Cache for one day
-		Give_Cache::set( 'give_estimated_monthly_stats', $estimated, DAY_IN_SECONDS, true );
+		Give_Cache::set('give_estimated_monthly_stats', $estimated, DAY_IN_SECONDS, true);
 	}
 
-	return maybe_unserialize( $estimated );
+	return maybe_unserialize($estimated);
 }
 
 /**
@@ -341,7 +341,7 @@ 
 function give_reports_set_form_method() {
 	return 'get';
 }
-add_filter( 'give-reports_form_method_tab_forms', 'give_reports_set_form_method', 10 );
-add_filter( 'give-reports_form_method_tab_donors', 'give_reports_set_form_method', 10 );
+add_filter('give-reports_form_method_tab_forms', 'give_reports_set_form_method', 10);
+add_filter('give-reports_form_method_tab_donors', 'give_reports_set_form_method', 10);
 
 // @TODO: After release 1.8 Donations -> Reports generates with new setting api, so we can remove some old code from this file.