WordImpress / Give

includes/install.php

Spacing +55 added lines, -55 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -26,15 +26,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();
 
@@ -48,7 +48,7 @@ 
 
 }
 
-register_activation_hook( GIVE_PLUGIN_FILE, 'give_install' );
+register_activation_hook(GIVE_PLUGIN_FILE, 'give_install');
 
 /**
  * Run the Give Install process.
@@ -64,24 +64,24 @@ 
 	give_setup_post_types();
 
 	// Clear the permalinks.
-	flush_rewrite_rules( false );
+	flush_rewrite_rules(false);
 
 	// 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();
 
 	// 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,
@@ -95,13 +95,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',
@@ -113,11 +113,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,
@@ -130,23 +130,23 @@ 
 	}
 
 	//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.
@@ -155,7 +155,7 @@ 
 	$roles->add_caps();
 
 	$api = new Give_API();
-	update_option( 'give_default_api_version', 'v' . $api->get_version() );
+	update_option('give_default_api_version', 'v'.$api->get_version());
 
 	// Create the customers databases.
 	@Give()->customers->create_table();
@@ -165,11 +165,11 @@ 
 	Give()->session->use_php_sessions();
 
 	// Add a temporary option to note that Give pages have been created.
-	set_transient( '_give_installed', $options, 30 );
+	set_transient('_give_installed', $options, 30);
 
-	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(
@@ -180,22 +180,22 @@ 
 			'v18_upgrades_form_metadata'
 		);
 
-		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.
-	set_transient( '_give_activation_redirect', true, 30 );
+	set_transient('_give_activation_redirect', true, 30);
 
 }
 
-register_activation_hook( GIVE_PLUGIN_FILE, 'give_install' );
+register_activation_hook(GIVE_PLUGIN_FILE, 'give_install');
 
 /**
  * Network Activated New Site Setup.
@@ -211,11 +211,11 @@ 
  * @param  int    $site_id The Site ID.
  * @param  array  $meta    Blog Meta.
  */
-function on_create_blog( $blog_id, $user_id, $domain, $path, $site_id, $meta ) {
+function 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();
 
@@ -223,7 +223,7 @@ 
 
 }
 
-add_action( 'wpmu_new_blog', 'on_create_blog', 10, 6 );
+add_action('wpmu_new_blog', 'on_create_blog', 10, 6);
 
 
 /**
@@ -236,13 +236,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);
 	$customers_db     = new Give_DB_Customers();
 	$customer_meta_db = new Give_DB_Customer_Meta();
 
-	if ( $customers_db->installed() ) {
+	if ($customers_db->installed()) {
 		$tables[] = $customers_db->table_name;
 		$tables[] = $customer_meta_db->table_name;
 	}
@@ -252,7 +252,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
@@ -264,16 +264,16 @@ 
  */
 function give_after_install() {
 
-	if ( ! is_admin() ) {
+	if ( ! is_admin()) {
 		return;
 	}
 
-	$give_options     = get_transient( '_give_installed' );
-	$give_table_check = get_option( '_give_table_check', false );
+	$give_options     = get_transient('_give_installed');
+	$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()->customer_meta->installed() ) {
+		if ( ! @Give()->customer_meta->installed()) {
 
 			// Create the customer meta database
 			// (this ensures it creates it on multisite instances where it is network activated).
@@ -281,7 +281,7 @@ 
 
 		}
 
-		if ( ! @Give()->customers->installed() ) {
+		if ( ! @Give()->customers->installed()) {
 			// Create the customers database
 			// (this ensures it creates it on multisite instances where it is network activated).
 			@Give()->customers->create_table();
@@ -293,22 +293,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 ) {
-		delete_transient( '_give_installed' );
+	if (false !== $give_options) {
+		delete_transient('_give_installed');
 	}
 
 
 }
 
-add_action( 'admin_init', 'give_after_install' );
+add_action('admin_init', 'give_after_install');
 
 
 /**
@@ -323,11 +323,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();
@@ -338,7 +338,7 @@ 
 
 }
 
-add_action( 'admin_init', 'give_install_roles_on_network' );
+add_action('admin_init', 'give_install_roles_on_network');
 
 /**
  * Default core setting values.
@@ -378,7 +378,7 @@ 
 		'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.

includes/class-give-cli-commands.php

Spacing +196 added lines, -196 removed lines

@@ -7,12 +7,12 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Add give command.
-WP_CLI::add_command( 'give', 'GIVE_CLI_COMMAND' );
+WP_CLI::add_command('give', 'GIVE_CLI_COMMAND');
 
 
 /**
@@ -68,8 +68,8 @@ 
 	 *
 	 * @subcommand    logo
 	 */
-	public function ascii( $args, $assoc_args ) {
-		WP_CLI::log( file_get_contents( GIVE_PLUGIN_DIR . 'assets/images/give-ascii-logo.txt' ) );
+	public function ascii($args, $assoc_args) {
+		WP_CLI::log(file_get_contents(GIVE_PLUGIN_DIR.'assets/images/give-ascii-logo.txt'));
 	}
 
 
@@ -94,54 +94,54 @@ 
 	 *
 	 * @subcommand    details
 	 */
-	public function details( $args, $assoc_args ) {
+	public function details($args, $assoc_args) {
 
 		/**
 		 * Plugin Information
 		 */
-		WP_CLI::log( $this->color_message( __( 'Give Version: ', 'give' ) ) . GIVE_VERSION );
+		WP_CLI::log($this->color_message(__('Give Version: ', 'give')).GIVE_VERSION);
 
 		/**
 		 * General Information.
 		 */
-		WP_CLI::log( "\n####   " . $this->color_message( __( 'General information', 'give' ) ) . '   ####' );
+		WP_CLI::log("\n####   ".$this->color_message(__('General information', 'give')).'   ####');
 
-		$success_page = give_get_option( 'success_page' );
-		$failure_page = give_get_option( 'failure_page' );
-		$history_page = give_get_option( 'history_page' );
+		$success_page = give_get_option('success_page');
+		$failure_page = give_get_option('failure_page');
+		$history_page = give_get_option('history_page');
 
-		WP_CLI::log( $this->color_message( sprintf( __( 'Success Page: ', 'give' ) ) ) . ( $success_page ? "[{$success_page}] " . get_permalink( $success_page ) : __( 'Not Set', 'give' ) ) );
-		WP_CLI::log( $this->color_message( __( 'Failed Donation Page: ', 'give' ) ) . ( $failure_page ? "[{$failure_page}] " . get_permalink( $failure_page ) : __( 'Not Set', 'give' ) ) );
-		WP_CLI::log( $this->color_message( __( 'Donation History Page: ', 'give' ) ) . ( $history_page ? "[{$history_page}] " . get_permalink( $history_page ) : __( 'Not Set', 'give' ) ) );
-		WP_CLI::log( $this->color_message( __( 'Country: ', 'give' ) ) . give_get_country() );
+		WP_CLI::log($this->color_message(sprintf(__('Success Page: ', 'give'))).($success_page ? "[{$success_page}] ".get_permalink($success_page) : __('Not Set', 'give')));
+		WP_CLI::log($this->color_message(__('Failed Donation Page: ', 'give')).($failure_page ? "[{$failure_page}] ".get_permalink($failure_page) : __('Not Set', 'give')));
+		WP_CLI::log($this->color_message(__('Donation History Page: ', 'give')).($history_page ? "[{$history_page}] ".get_permalink($history_page) : __('Not Set', 'give')));
+		WP_CLI::log($this->color_message(__('Country: ', 'give')).give_get_country());
 
 		/**
 		 * Currency Information.
 		 */
-		$default_gateway = give_get_option( 'default_gateway' );
+		$default_gateway = give_get_option('default_gateway');
 
-		WP_CLI::log( "\n####   " . $this->color_message( __( 'Currency Information', 'give' ) ) . '   ####' );
+		WP_CLI::log("\n####   ".$this->color_message(__('Currency Information', 'give')).'   ####');
 
-		WP_CLI::log( $this->color_message( __( 'Currency: ', 'give' ), give_get_currency() ) );
-		WP_CLI::log( $this->color_message( __( 'Currency Position: ', 'give' ), give_get_currency_position() ) );
-		WP_CLI::log( $this->color_message( __( 'Thousand Separator: ', 'give' ), give_get_price_thousand_separator() ) );
-		WP_CLI::log( $this->color_message( __( 'Decimal Separator: ', 'give' ), give_get_price_decimal_separator() ) );
-		WP_CLI::log( $this->color_message( __( 'Number of Decimals: ', 'give' ), give_get_price_decimals() ) );
-		WP_CLI::log( $this->color_message( __( 'Test Mode: ', 'give' ), ( give_get_option( 'test_mode' ) ? __( 'Yes', 'give' ) : __( 'No', 'give' ) ) ) );
-		WP_CLI::log( $this->color_message( __( 'Default Gateway: ', 'give' ), ( $default_gateway ? $default_gateway : __( 'Not Set', 'give' ) ) ) );
+		WP_CLI::log($this->color_message(__('Currency: ', 'give'), give_get_currency()));
+		WP_CLI::log($this->color_message(__('Currency Position: ', 'give'), give_get_currency_position()));
+		WP_CLI::log($this->color_message(__('Thousand Separator: ', 'give'), give_get_price_thousand_separator()));
+		WP_CLI::log($this->color_message(__('Decimal Separator: ', 'give'), give_get_price_decimal_separator()));
+		WP_CLI::log($this->color_message(__('Number of Decimals: ', 'give'), give_get_price_decimals()));
+		WP_CLI::log($this->color_message(__('Test Mode: ', 'give'), (give_get_option('test_mode') ? __('Yes', 'give') : __('No', 'give'))));
+		WP_CLI::log($this->color_message(__('Default Gateway: ', 'give'), ($default_gateway ? $default_gateway : __('Not Set', 'give'))));
 
 		// Payment gateways Information.
-		$gateways = give_get_ordered_payment_gateways( give_get_payment_gateways() );
-		WP_CLI::log( $this->color_message( __( 'Enabled Gateways: ', 'give' ) ) );
+		$gateways = give_get_ordered_payment_gateways(give_get_payment_gateways());
+		WP_CLI::log($this->color_message(__('Enabled Gateways: ', 'give')));
 
-		if ( ! empty( $gateways ) ) {
+		if ( ! empty($gateways)) {
 			self::$counter = 1;
-			foreach ( $gateways as $gateway ) {
-				WP_CLI::log( '  ' . $this->color_message( self::$counter, $gateway['admin_label'] ) );
-				self::$counter ++;
+			foreach ($gateways as $gateway) {
+				WP_CLI::log('  '.$this->color_message(self::$counter, $gateway['admin_label']));
+				self::$counter++;
 			}
 		} else {
-			WP_CLI::log( __( 'Not any payment gateways found', 'give' ) );
+			WP_CLI::log(__('Not any payment gateways found', 'give'));
 		}
 	}
 
@@ -173,15 +173,15 @@ 
 	 *
 	 * @subcommand    forms
 	 */
-	public function forms( $args, $assoc_args ) {
+	public function forms($args, $assoc_args) {
 		global $wp_query;
-		$form_id = isset( $assoc_args ) && array_key_exists( 'id', $assoc_args ) ? absint( $assoc_args['id'] ) : false;
-		$number  = isset( $assoc_args ) && array_key_exists( 'number', $assoc_args ) ? absint( $assoc_args['number'] ) : 10;
+		$form_id = isset($assoc_args) && array_key_exists('id', $assoc_args) ? absint($assoc_args['id']) : false;
+		$number  = isset($assoc_args) && array_key_exists('number', $assoc_args) ? absint($assoc_args['number']) : 10;
 		$start   = time();
 
 		// Cache previous number query var.
 		$is_set_number = $cache_per_page = false;
-		if ( isset( $wp_query->query_vars['number'] ) ) {
+		if (isset($wp_query->query_vars['number'])) {
 			$cache_per_page = $wp_query->query_vars['number'];
 			$is_set_number  = true;
 		}
@@ -190,22 +190,22 @@ 
 		$wp_query->query_vars['number'] = $number;
 
 		// Get forms.
-		$forms = $form_id ? $this->api->get_forms( $form_id ) : $this->api->get_forms();
+		$forms = $form_id ? $this->api->get_forms($form_id) : $this->api->get_forms();
 
 		// Reset number query var.
-		if ( $is_set_number ) {
+		if ($is_set_number) {
 			$wp_query->query_vars['number'] = $cache_per_page;
 		}
 
 		// Bailout.
-		if ( array_key_exists( 'error', $forms ) ) {
+		if (array_key_exists('error', $forms)) {
 
-			WP_CLI::warning( $forms['error'] );
+			WP_CLI::warning($forms['error']);
 
 			return;
-		} elseif ( empty( $forms['forms'] ) ) {
+		} elseif (empty($forms['forms'])) {
 
-			WP_CLI::error( __( 'No forms found.', 'give' ) );
+			WP_CLI::error(__('No forms found.', 'give'));
 
 			return;
 		}
@@ -213,25 +213,25 @@ 
 		// Param to check if form typeis already showed or not.
 		$is_show_form_type = false;
 
-		if ( 1 === count( $forms ) && $form_id ) {
+		if (1 === count($forms) && $form_id) {
 			// Show single form.
-			foreach ( $forms['forms'][0] as $key => $info ) {
-				switch ( $key ) {
+			foreach ($forms['forms'][0] as $key => $info) {
+				switch ($key) {
 					case 'stats':
-						$this->color_main_heading( ucfirst( $key ) );
+						$this->color_main_heading(ucfirst($key));
 
-						foreach ( $info as $heading => $data ) {
-							$this->color_sub_heading( ucfirst( $heading ) );
-							switch ( $heading ) {
+						foreach ($info as $heading => $data) {
+							$this->color_sub_heading(ucfirst($heading));
+							switch ($heading) {
 								default:
-									foreach ( $data as $subheading => $subdata ) {
+									foreach ($data as $subheading => $subdata) {
 
-										switch ( $subheading ) {
+										switch ($subheading) {
 											case 'earnings':
-												WP_CLI::log( $this->color_message( $subheading . ': ', give_currency_filter( $subdata ) ) );
+												WP_CLI::log($this->color_message($subheading.': ', give_currency_filter($subdata)));
 												break;
 											default:
-												WP_CLI::log( $this->color_message( $subheading . ': ', $subdata ) );
+												WP_CLI::log($this->color_message($subheading.': ', $subdata));
 										}
 									}
 							}
@@ -241,26 +241,26 @@ 
 					case 'pricing':
 					case 'info':
 					default:
-						$this->color_main_heading( ucfirst( $key ) );
+						$this->color_main_heading(ucfirst($key));
 
 						// Show form type.
-						if ( ! $is_show_form_type ) {
-							$form              = new Give_Donate_Form( $form_id );
+						if ( ! $is_show_form_type) {
+							$form              = new Give_Donate_Form($form_id);
 							$is_show_form_type = true;
 
-							WP_CLI::log( $this->color_message( __( 'form type', 'give' ), $form->get_type() ) );
+							WP_CLI::log($this->color_message(__('form type', 'give'), $form->get_type()));
 						}
 
-						foreach ( $info as $heading => $data ) {
+						foreach ($info as $heading => $data) {
 
-							switch ( $heading ) {
+							switch ($heading) {
 								case 'id':
-									WP_CLI::log( $this->color_message( $heading, $data ) );
+									WP_CLI::log($this->color_message($heading, $data));
 									break;
 
 								default:
-									$data = empty( $data ) ? __( 'Not set', 'give' ) : $data;
-									WP_CLI::log( $this->color_message( $heading, $data ) );
+									$data = empty($data) ? __('Not set', 'give') : $data;
+									WP_CLI::log($this->color_message($heading, $data));
 							}
 						}
 				}
@@ -271,37 +271,37 @@ 
 			$is_table_first_row_set = false;
 			$table_column_count     = 0;
 
-			WP_CLI::line( $this->color_message( sprintf( __( '%d donation forms found', 'give' ), count( $forms['forms'] ) ), '', false ) );
+			WP_CLI::line($this->color_message(sprintf(__('%d donation forms found', 'give'), count($forms['forms'])), '', false));
 
-			foreach ( $forms['forms'] as $index => $form_data ) {
+			foreach ($forms['forms'] as $index => $form_data) {
 
 				// Default table data.
 				$table_first_row = array();
 				$table_row       = array();
 
-				foreach ( $form_data['info'] as $key => $form ) {
+				foreach ($form_data['info'] as $key => $form) {
 
 					// Do not show thumbnail, content and link in table.
-					if ( in_array( $key, array( 'content', 'thumbnail', 'link' ), true ) ) {
+					if (in_array($key, array('content', 'thumbnail', 'link'), true)) {
 						continue;
 					}
 
-					if ( ! $is_table_first_row_set ) {
+					if ( ! $is_table_first_row_set) {
 						$table_first_row[] = $key;
 					}
 
 					$table_row[] = $form;
 
-					if ( 'status' === $key ) {
+					if ('status' === $key) {
 						// First array item will be an form id in our case.
-						$form = new Give_Donate_Form( absint( $table_row[0] ) );
+						$form = new Give_Donate_Form(absint($table_row[0]));
 
 						$table_row[] = $form->get_type();
 					}
 				}
 
 				// Set table first row.
-				if ( ! $is_table_first_row_set ) {
+				if ( ! $is_table_first_row_set) {
 
 					// Add extra column to table.
 					$table_first_row[] = 'type';
@@ -314,7 +314,7 @@ 
 				$table_data[] = $table_row;
 			}
 
-			$this->display_table( $table_data );
+			$this->display_table($table_data);
 		}
 	}
 
@@ -366,37 +366,37 @@ 
 	 *
 	 * @subcommand    donors
 	 */
-	public function donors( $args, $assoc_args ) {
+	public function donors($args, $assoc_args) {
 		global $wp_query;
-		$donor_id = isset( $assoc_args ) && array_key_exists( 'id', $assoc_args ) ? absint( $assoc_args['id'] ) : false;
-		$email    = isset( $assoc_args ) && array_key_exists( 'email', $assoc_args ) ? $assoc_args['email'] : false;
-		$name     = isset( $assoc_args ) && array_key_exists( 'name', $assoc_args ) ? $assoc_args['name'] : '';
-		$create   = isset( $assoc_args ) && array_key_exists( 'create', $assoc_args ) ? $assoc_args['create'] : false;
-		$number   = isset( $assoc_args ) && array_key_exists( 'number', $assoc_args ) ? $assoc_args['number'] : 10;
-		$form_id  = isset( $assoc_args ) && array_key_exists( 'form-id', $assoc_args ) ? $assoc_args['form-id'] : 0;
-		$format   = isset( $assoc_args ) && array_key_exists( 'format', $assoc_args ) ? $assoc_args['format'] : 'table';
+		$donor_id = isset($assoc_args) && array_key_exists('id', $assoc_args) ? absint($assoc_args['id']) : false;
+		$email    = isset($assoc_args) && array_key_exists('email', $assoc_args) ? $assoc_args['email'] : false;
+		$name     = isset($assoc_args) && array_key_exists('name', $assoc_args) ? $assoc_args['name'] : '';
+		$create   = isset($assoc_args) && array_key_exists('create', $assoc_args) ? $assoc_args['create'] : false;
+		$number   = isset($assoc_args) && array_key_exists('number', $assoc_args) ? $assoc_args['number'] : 10;
+		$form_id  = isset($assoc_args) && array_key_exists('form-id', $assoc_args) ? $assoc_args['form-id'] : 0;
+		$format   = isset($assoc_args) && array_key_exists('format', $assoc_args) ? $assoc_args['format'] : 'table';
 		$start    = time();
 
-		if ( $create ) {
+		if ($create) {
 			$number = 1;
 
-			if ( isset( $assoc_args['email'] ) && ! is_email( $email ) ) {
-				WP_CLI::warning( 'Wrong email address provided.', 'give' );
+			if (isset($assoc_args['email']) && ! is_email($email)) {
+				WP_CLI::warning('Wrong email address provided.', 'give');
 
 				return;
 			}
 
 			// Create one or more donors.
-			if ( ! $email ) {
+			if ( ! $email) {
 				// If no email is specified, look to see if we are generating arbitrary donor accounts.
-				$number = is_numeric( $create ) ? absint( $create ) : 1;
+				$number = is_numeric($create) ? absint($create) : 1;
 			}
 
-			for ( $i = 0; $i < $number; $i ++ ) {
-				if ( ! $email ) {
+			for ($i = 0; $i < $number; $i++) {
+				if ( ! $email) {
 
 					// Generate fake email.
-					$email = 'customer-' . uniqid() . '@test.com';
+					$email = 'customer-'.uniqid().'@test.com';
 				}
 
 				$args = array(
@@ -404,19 +404,19 @@ 
 					'name'  => $name,
 				);
 
-				$customer_id = Give()->customers->add( $args );
+				$customer_id = Give()->customers->add($args);
 
-				if ( $customer_id ) {
-					WP_CLI::line( $this->color_message( sprintf( __( 'Donor #%d created successfully', 'give' ), $customer_id ) ) );
+				if ($customer_id) {
+					WP_CLI::line($this->color_message(sprintf(__('Donor #%d created successfully', 'give'), $customer_id)));
 				} else {
-					WP_CLI::error( __( 'Failed to create donor', 'give' ) );
+					WP_CLI::error(__('Failed to create donor', 'give'));
 				}
 
 				// Reset email to false so it is generated on the next loop (if creating donors).
 				$email = false;
 			}
 
-			WP_CLI::line( $this->color_message( sprintf( __( '%1$d donors created in %2$d seconds', 'give' ), $number, time() - $start ) ) );
+			WP_CLI::line($this->color_message(sprintf(__('%1$d donors created in %2$d seconds', 'give'), $number, time() - $start)));
 
 		} else {
 			// Counter.
@@ -430,7 +430,7 @@ 
 			 */
 			// Cache previous number query var.
 			$is_set_number = $cache_per_page = false;
-			if ( isset( $wp_query->query_vars['number'] ) ) {
+			if (isset($wp_query->query_vars['number'])) {
 				$cache_per_page = $wp_query->query_vars['number'];
 				$is_set_number  = true;
 			}
@@ -439,24 +439,24 @@ 
 			$wp_query->query_vars['number'] = $number;
 
 			// Get donors.
-			if ( $form_id ) {
+			if ($form_id) {
 				// @TODO: Allow user to get a list of donors by donation status.
-				$donors = $this->get_donors_by_form_id( $form_id );
+				$donors = $this->get_donors_by_form_id($form_id);
 			} else {
-				$donors = $this->api->get_customers( $search );
+				$donors = $this->api->get_customers($search);
 			}
 
 			// Reset number query var.
-			if ( $is_set_number ) {
+			if ($is_set_number) {
 				$wp_query->query_vars['number'] = $cache_per_page;
 			}
 
-			if ( isset( $donors['error'] ) ) {
-				WP_CLI::error( $donors['error'] );
+			if (isset($donors['error'])) {
+				WP_CLI::error($donors['error']);
 			}
 
-			if ( empty( $donors ) ) {
-				WP_CLI::error( __( 'No donors found.', 'give' ) );
+			if (empty($donors)) {
+				WP_CLI::error(__('No donors found.', 'give'));
 
 				return;
 			}
@@ -464,24 +464,24 @@ 
 			$table_data             = array();
 			$is_table_first_row_set = false;
 
-			foreach ( $donors['donors'] as $donor_data ) {
+			foreach ($donors['donors'] as $donor_data) {
 				// Set default table row data.
-				$table_first_row = array( __( 's_no', 'give' ) );
-				$table_row       = array( self::$counter );
+				$table_first_row = array(__('s_no', 'give'));
+				$table_row       = array(self::$counter);
 
-				foreach ( $donor_data as $key => $donor ) {
-					switch ( $key ) {
+				foreach ($donor_data as $key => $donor) {
+					switch ($key) {
 						case 'stats':
-							foreach ( $donor as $heading => $data ) {
+							foreach ($donor as $heading => $data) {
 
 								// Get first row.
-								if ( ! $is_table_first_row_set ) {
+								if ( ! $is_table_first_row_set) {
 									$table_first_row[] = $heading;
 								}
 
-								switch ( $heading ) {
+								switch ($heading) {
 									case 'total_spent':
-										$table_row[] = give_currency_filter( $data );
+										$table_row[] = give_currency_filter($data);
 										break;
 
 									default:
@@ -492,10 +492,10 @@ 
 
 						case 'info':
 						default:
-							foreach ( $donor as $heading => $data ) {
+							foreach ($donor as $heading => $data) {
 
 								// Get first row.
-								if ( ! $is_table_first_row_set ) {
+								if ( ! $is_table_first_row_set) {
 									$table_first_row[] = $heading;
 								}
 
@@ -505,7 +505,7 @@ 
 				}
 
 				// Add first row data to table data.
-				if ( ! $is_table_first_row_set ) {
+				if ( ! $is_table_first_row_set) {
 					$table_data[]           = $table_first_row;
 					$is_table_first_row_set = true;
 				}
@@ -514,44 +514,44 @@ 
 				$table_data[] = $table_row;
 
 				// Increase counter.
-				self::$counter ++;
+				self::$counter++;
 			}
 
-			switch ( $format ) {
+			switch ($format) {
 				case 'json':
 					$table_column_name = $table_data[0];
-					unset( $table_data[0] );
+					unset($table_data[0]);
 
 					$new_table_data = array();
-					foreach ( $table_data as $index => $data ) {
-						foreach ( $data as $key => $value ) {
-							$new_table_data[ $index ][ $table_column_name[ $key ] ] = $value;
+					foreach ($table_data as $index => $data) {
+						foreach ($data as $key => $value) {
+							$new_table_data[$index][$table_column_name[$key]] = $value;
 						}
 					}
 
-					WP_CLI::log( json_encode( $new_table_data ) );
+					WP_CLI::log(json_encode($new_table_data));
 					break;
 
 				case 'csv':
-					$file_path = trailingslashit( WP_CONTENT_DIR ) . 'uploads/give_donors_' . date( 'Y_m_d_s', current_time( 'timestamp' ) ) . '.csv';
-					$fp        = fopen( $file_path, 'w' );
+					$file_path = trailingslashit(WP_CONTENT_DIR).'uploads/give_donors_'.date('Y_m_d_s', current_time('timestamp')).'.csv';
+					$fp        = fopen($file_path, 'w');
 
-					if ( is_writable( $file_path ) ) {
-						foreach ( $table_data as $fields ) {
-							fputcsv( $fp, $fields );
+					if (is_writable($file_path)) {
+						foreach ($table_data as $fields) {
+							fputcsv($fp, $fields);
 						}
 
-						fclose( $fp );
+						fclose($fp);
 
-						WP_CLI::success( "Donors list csv created successfully: {$file_path}" );
+						WP_CLI::success("Donors list csv created successfully: {$file_path}");
 					} else {
-						WP_CLI::warning( "Unable to create donors list csv file: {$file_path} (May folder do not have write permission)" );
+						WP_CLI::warning("Unable to create donors list csv file: {$file_path} (May folder do not have write permission)");
 					}
 
 					break;
 
 				default:
-					$this->display_table( $table_data );
+					$this->display_table($table_data);
 			}
 		}
 	}
@@ -581,13 +581,13 @@ 
 	 *
 	 * @subcommand    donations
 	 */
-	public function donations( $args, $assoc_args ) {
+	public function donations($args, $assoc_args) {
 		global $wp_query;
-		$number = isset( $assoc_args ) && array_key_exists( 'number', $assoc_args ) ? $assoc_args['number'] : 10;
+		$number = isset($assoc_args) && array_key_exists('number', $assoc_args) ? $assoc_args['number'] : 10;
 
 		// Cache previous number query var.
 		$is_set_number = $cache_per_page = false;
-		if ( isset( $wp_query->query_vars['number'] ) ) {
+		if (isset($wp_query->query_vars['number'])) {
 			$cache_per_page = $wp_query->query_vars['number'];
 			$is_set_number  = true;
 		}
@@ -599,46 +599,46 @@ 
 		$donations = $this->api->get_recent_donations();
 
 		// Reset number query var.
-		if ( $is_set_number ) {
+		if ($is_set_number) {
 			$wp_query->query_vars['number'] = $cache_per_page;
 		}
 
-		if ( empty( $donations ) ) {
-			WP_CLI::error( __( 'No donations found.', 'give' ) );
+		if (empty($donations)) {
+			WP_CLI::error(__('No donations found.', 'give'));
 
 			return;
 		}
 
 		self::$counter = 1;
 
-		foreach ( $donations['donations'] as $key => $donation ) {
-			$this->color_main_heading( sprintf( __( '%1$s. Donation #%2$s', 'give' ), self::$counter, $donation['ID'] ), 'Y' );
-			self::$counter ++;
+		foreach ($donations['donations'] as $key => $donation) {
+			$this->color_main_heading(sprintf(__('%1$s. Donation #%2$s', 'give'), self::$counter, $donation['ID']), 'Y');
+			self::$counter++;
 
-			foreach ( $donation as $column => $data ) {
+			foreach ($donation as $column => $data) {
 
-				if ( is_array( $data ) ) {
-					$this->color_sub_heading( $column );
-					foreach ( $data as $subcolumn => $subdata ) {
+				if (is_array($data)) {
+					$this->color_sub_heading($column);
+					foreach ($data as $subcolumn => $subdata) {
 
 						// Decode html codes.
-						switch ( $subcolumn ) {
+						switch ($subcolumn) {
 							case 'name':
-								$subdata = html_entity_decode( $subdata );
+								$subdata = html_entity_decode($subdata);
 								break;
 						}
 
 						// @TODO Check if multi dimension array information is importent to show or not. For example inside donation array we have array for fees data inside payment meta.
-						if ( is_array( $subdata ) ) {
+						if (is_array($subdata)) {
 							continue;
 						}
 
-						WP_CLI::log( $this->color_message( $subcolumn, $subdata ) );
+						WP_CLI::log($this->color_message($subcolumn, $subdata));
 					}
 					continue;
 				}
 
-				WP_CLI::log( $this->color_message( $column, $data ) );
+				WP_CLI::log($this->color_message($column, $data));
 			}
 		}
 	}
@@ -677,27 +677,27 @@ 
 	 *
 	 * @return        void
 	 */
-	public function report( $args, $assoc_args ) {
+	public function report($args, $assoc_args) {
 		$stats      = new Give_Payment_Stats();
-		$date       = isset( $assoc_args ) && array_key_exists( 'date', $assoc_args ) ? $assoc_args['date'] : false;
-		$start_date = isset( $assoc_args ) && array_key_exists( 'start-date', $assoc_args ) ? $assoc_args['start-date'] : false;
-		$end_date   = isset( $assoc_args ) && array_key_exists( 'end-date', $assoc_args ) ? $assoc_args['end-date'] : false;
-		$form_id    = isset( $assoc_args ) && array_key_exists( 'id', $assoc_args ) ? $assoc_args['id'] : 0;
+		$date       = isset($assoc_args) && array_key_exists('date', $assoc_args) ? $assoc_args['date'] : false;
+		$start_date = isset($assoc_args) && array_key_exists('start-date', $assoc_args) ? $assoc_args['start-date'] : false;
+		$end_date   = isset($assoc_args) && array_key_exists('end-date', $assoc_args) ? $assoc_args['end-date'] : false;
+		$form_id    = isset($assoc_args) && array_key_exists('id', $assoc_args) ? $assoc_args['id'] : 0;
 
-		if ( ! empty( $date ) ) {
+		if ( ! empty($date)) {
 			$start_date = $date;
 			$end_date   = false;
-		} elseif ( empty( $date ) && empty( $start_date ) ) {
+		} elseif (empty($date) && empty($start_date)) {
 			$start_date = 'this_month';
 			$end_date   = false;
 		}
 
 		// Get stats.
-		$earnings = $stats->get_earnings( $form_id, $start_date, $end_date );
-		$sales    = $stats->get_sales( $form_id, $start_date, $end_date );
+		$earnings = $stats->get_earnings($form_id, $start_date, $end_date);
+		$sales    = $stats->get_sales($form_id, $start_date, $end_date);
 
-		WP_CLI::line( $this->color_message( __( 'Earnings', 'give' ), give_currency_filter( $earnings ) ) );
-		WP_CLI::line( $this->color_message( __( 'Sales', 'give' ), $sales ) );
+		WP_CLI::line($this->color_message(__('Earnings', 'give'), give_currency_filter($earnings)));
+		WP_CLI::line($this->color_message(__('Sales', 'give'), $sales));
 	}
 
 
@@ -724,27 +724,27 @@ 
 	 *
 	 * @subcommand    cache
 	 */
-	public function cache( $args, $assoc_args ) {
-		$action = isset( $assoc_args ) && array_key_exists( 'action', $assoc_args ) ? $assoc_args['action'] : false;
+	public function cache($args, $assoc_args) {
+		$action = isset($assoc_args) && array_key_exists('action', $assoc_args) ? $assoc_args['action'] : false;
 
 		// Bailout.
-		if ( ! $action || ! in_array( $action, array( 'delete' ), true ) ) {
-			WP_CLI::warning( __( 'Type wp give cache --action=delete to delete all stat transients', 'give' ) );
+		if ( ! $action || ! in_array($action, array('delete'), true)) {
+			WP_CLI::warning(__('Type wp give cache --action=delete to delete all stat transients', 'give'));
 
 			return;
 		}
 
-		switch ( $action ) {
+		switch ($action) {
 			case 'delete' :
 				// Reset counter.
 				self::$counter = 1;
 
-				if ( $this->delete_stats_transients() ) {
+				if ($this->delete_stats_transients()) {
 					// Report .eading.
-					WP_CLI::success( 'Give cache deleted.' );
+					WP_CLI::success('Give cache deleted.');
 				} else {
 					// Report .eading.
-					WP_CLI::warning( 'We did not find any Give plugin cache to delete :)' );
+					WP_CLI::warning('We did not find any Give plugin cache to delete :)');
 				}
 				break;
 		}
@@ -773,27 +773,27 @@ 
 			ARRAY_A
 		);
 
-		if ( ! empty( $stat_option_names ) ) {
+		if ( ! empty($stat_option_names)) {
 
-			foreach ( $stat_option_names as $option_name ) {
+			foreach ($stat_option_names as $option_name) {
 				$error       = false;
 				$option_name = $option_name['option_name'];
 
-				switch ( true ) {
-					case ( false !== strpos( $option_name, 'transient' ) ):
-						$option_name = str_replace( '_transient_', '', $option_name );
-						$error       = delete_transient( $option_name );
+				switch (true) {
+					case (false !== strpos($option_name, 'transient')):
+						$option_name = str_replace('_transient_', '', $option_name);
+						$error       = delete_transient($option_name);
 						break;
 
 					default:
-						$error = delete_option( $option_name );
+						$error = delete_option($option_name);
 				}
 
-				if ( $error ) {
-					WP_CLI::log( $this->color_message( self::$counter, $option_name ) );
-					self::$counter ++;
+				if ($error) {
+					WP_CLI::log($this->color_message(self::$counter, $option_name));
+					self::$counter++;
 				} else {
-					WP_CLI::log( $this->color_message( __( 'Error while deleting this transient', 'give' ), $option_name ) );
+					WP_CLI::log($this->color_message(__('Error while deleting this transient', 'give'), $option_name));
 				}
 			}
 
@@ -814,13 +814,13 @@ 
 	 *
 	 * @return   string
 	 */
-	private function color_message( $heading, $message = '', $colon = true, $color = 'g' ) {
+	private function color_message($heading, $message = '', $colon = true, $color = 'g') {
 		// Add colon.
-		if ( $colon ) {
-			$heading = $heading . ': ';
+		if ($colon) {
+			$heading = $heading.': ';
 		}
 
-		return WP_CLI::colorize( "%{$color}" . $heading . '%n' ) . $message;
+		return WP_CLI::colorize("%{$color}".$heading.'%n').$message;
 	}
 
 
@@ -835,8 +835,8 @@ 
 	 *
 	 * @return    void
 	 */
-	private function color_main_heading( $heading, $color = 'g' ) {
-		WP_CLI::log( "\n######   " . $this->color_message( $heading, '', false, $color ) . '   ######' );
+	private function color_main_heading($heading, $color = 'g') {
+		WP_CLI::log("\n######   ".$this->color_message($heading, '', false, $color).'   ######');
 	}
 
 	/**
@@ -849,8 +849,8 @@ 
 	 *
 	 * @return    void
 	 */
-	private function color_sub_heading( $subheading ) {
-		WP_CLI::log( "\n--->" . $subheading . '', '', false );
+	private function color_sub_heading($subheading) {
+		WP_CLI::log("\n--->".$subheading.'', '', false);
 	}
 
 
@@ -864,17 +864,17 @@ 
 	 *
 	 * @return    void
 	 */
-	private function display_table( $data ) {
+	private function display_table($data) {
 		$table = new \cli\Table();
 
 		// Set table header.
-		$table->setHeaders( $data[0] );
+		$table->setHeaders($data[0]);
 
 		// Remove table header.
-		unset( $data[0] );
+		unset($data[0]);
 
 		// Set table data.
-		$table->setRows( $data );
+		$table->setRows($data);
 
 		// Display table.
 		$table->display();
@@ -891,14 +891,14 @@ 
 	 * @return array
 	 */
 
-	private function get_donors_by_form_id( $form_id ) {
+	private function get_donors_by_form_id($form_id) {
 		$donors = array();
 
 		$donations = new Give_Payments_Query(
 			array(
-				'give_forms' => array( $form_id ),
-				'number'     => - 1,
-				'status'     => array( 'publish' ),
+				'give_forms' => array($form_id),
+				'number'     => -1,
+				'status'     => array('publish'),
 			)
 		);
 
@@ -906,16 +906,16 @@ 
 		$skip_donors = array();
 
 		/* @var Give_Payment|object $donation Payment object. */
-		foreach ( $donations as $donation ) {
+		foreach ($donations as $donation) {
 
-			if ( in_array( $donation->customer_id, $skip_donors ) ) {
+			if (in_array($donation->customer_id, $skip_donors)) {
 				continue;
 			}
 
-			if ( ! empty( $donors ) ) {
-				$donors['donors'][] = current( current( $this->api->get_customers( (int) $donation->customer_id ) ) );
+			if ( ! empty($donors)) {
+				$donors['donors'][] = current(current($this->api->get_customers((int) $donation->customer_id)));
 			} else {
-				$donors = array_merge( $donors, $this->api->get_customers( (int) $donation->customer_id ) );
+				$donors = array_merge($donors, $this->api->get_customers((int) $donation->customer_id));
 			}
 
 			$skip_donors[] = $donation->customer_id;

includes/filters.php

Spacing +12 added lines, -12 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -26,29 +26,29 @@ 
  *
  * @return void
  */
-function give_set_settings_with_disable_prefix( $old_settings, $settings ) {
+function give_set_settings_with_disable_prefix($old_settings, $settings) {
 	// Get old setting names.
-	$old_settings   = array_flip( give_v18_renamed_core_settings() );
+	$old_settings   = array_flip(give_v18_renamed_core_settings());
 	$update_setting = false;
 
-	foreach ( $settings as $key => $value ) {
+	foreach ($settings as $key => $value) {
 
 		// Check 1. Check if new option is really updated or not.
 		// Check 2. Continue if key is not renamed.
-		if ( ! isset( $old_settings[ $key ] ) ) {
+		if ( ! isset($old_settings[$key])) {
 			continue;
 		}
 
 		// Set old setting.
-		$settings[ $old_settings[ $key ] ] = 'on';
+		$settings[$old_settings[$key]] = 'on';
 
 		// Do not need to set old setting if new setting is not set.
 		if (
-			( give_is_setting_enabled( $value ) && ( false !== strpos( $old_settings[ $key ], 'disable_' ) ) )
-			|| ( ! give_is_setting_enabled( $value ) && ( false !== strpos( $old_settings[ $key ], 'enable_' ) ) )
+			(give_is_setting_enabled($value) && (false !== strpos($old_settings[$key], 'disable_')))
+			|| ( ! give_is_setting_enabled($value) && (false !== strpos($old_settings[$key], 'enable_')))
 
 		) {
-			unset( $settings[ $old_settings[ $key ] ] );
+			unset($settings[$old_settings[$key]]);
 		}
 
 		// Tell bot to update setting.
@@ -56,8 +56,8 @@ 
 	}
 
 	// Update setting if any old setting set.
-	if ( $update_setting ) {
-		update_option( 'give_settings', $settings );
+	if ($update_setting) {
+		update_option('give_settings', $settings);
 	}
 }
-add_action( 'update_option_give_settings', 'give_set_settings_with_disable_prefix', 10, 2 );
+add_action('update_option_give_settings', 'give_set_settings_with_disable_prefix', 10, 2);

includes/gateways/actions.php

Spacing +13 added lines, -13 removed lines

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

includes/admin/reporting/class-donor-reports-table.php

Doc Comments +1 added lines, -1 removed lines

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

Indentation +2 added lines, -2 removed lines

@@ -185,8 +185,8 @@
 				
 			case 'num_donations' :
 				$value = '<a href="' .
-				         admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&user=' . urlencode( $item['email'] )
-				         ) . '">' . esc_html( $item['num_donations'] ) . '</a>';
+						 admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&user=' . urlencode( $item['email'] )
+						 ) . '">' . esc_html( $item['num_donations'] ) . '</a>';
 				break;
 
 			case 'amount_spent' :

Spacing +68 added lines, -68 removed lines

@@ -10,13 +10,13 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -62,11 +62,11 @@ 
 		global $status, $page;
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => esc_html__( 'Donor', 'give' ),     // Singular name of the listed records
-			'plural'   => esc_html__( 'Donors', 'give' ),    // Plural name of the listed records
-			'ajax'     => false,// Does this table support ajax?
-		) );
+		parent::__construct(array(
+			'singular' => esc_html__('Donor', 'give'), // Singular name of the listed records
+			'plural'   => esc_html__('Donors', 'give'), // Plural name of the listed records
+			'ajax'     => false, // Does this table support ajax?
+		));
 
 	}
 
@@ -83,7 +83,7 @@ 
 	 *
 	 * @return false
 	 */
-	public function search_box( $text, $input_id ) {
+	public function search_box($text, $input_id) {
 		return false;
 	}
 
@@ -98,20 +98,20 @@ 
 	 *
 	 * @return void
 	 */
-	public function give_search_box( $text, $input_id ) {
-		$input_id = $input_id . '-search-input';
+	public function give_search_box($text, $input_id) {
+		$input_id = $input_id.'-search-input';
 
-		if ( ! empty( $_REQUEST['orderby'] ) ) {
-			echo '<input type="hidden" name="orderby" value="' . esc_attr( $_REQUEST['orderby'] ) . '" />';
+		if ( ! empty($_REQUEST['orderby'])) {
+			echo '<input type="hidden" name="orderby" value="'.esc_attr($_REQUEST['orderby']).'" />';
 		}
-		if ( ! empty( $_REQUEST['order'] ) ) {
-			echo '<input type="hidden" name="order" value="' . esc_attr( $_REQUEST['order'] ) . '" />';
+		if ( ! empty($_REQUEST['order'])) {
+			echo '<input type="hidden" name="order" value="'.esc_attr($_REQUEST['order']).'" />';
 		}
 		?>
 		<p class="search-box donor-search" role="search">
 			<label class="screen-reader-text" for="<?php echo $input_id ?>"><?php echo $text; ?>:</label>
 			<input type="search" id="<?php echo $input_id ?>" name="s" value="<?php _admin_search_query(); ?>" />
-			<?php submit_button( $text, 'button', false, false, array( 'ID' => 'search-submit' ) ); ?>
+			<?php submit_button($text, 'button', false, false, array('ID' => 'search-submit')); ?>
 		</p>
 	<?php
 	}
@@ -124,33 +124,33 @@ 
 	 *
 	 * @param string $which
 	 */
-	protected function display_tablenav( $which ) {
+	protected function display_tablenav($which) {
 
-		if ( 'top' === $which ) {
-			wp_nonce_field( 'bulk-' . $this->_args['plural'] );
+		if ('top' === $which) {
+			wp_nonce_field('bulk-'.$this->_args['plural']);
 		}
 		?>
-		<div class="tablenav give-clearfix <?php echo esc_attr( $which ); ?>">
+		<div class="tablenav give-clearfix <?php echo esc_attr($which); ?>">
 
-			<?php if ( 'top' === $which ) { ?>
+			<?php if ('top' === $which) { ?>
 				<h3 class="alignleft reports-earnings-title">
-					<span><?php esc_html_e( 'Donors Report', 'give' ); ?></span>
+					<span><?php esc_html_e('Donors Report', 'give'); ?></span>
 				</h3>
 			<?php } ?>
 
 			<div class="alignright tablenav-right">
 				<div class="actions bulkactions">
 					<?php
-					if ( 'top' === $which ) {
-						$this->give_search_box( esc_html__( 'Search Donors', 'give' ), 'give-donors-report-search' );
+					if ('top' === $which) {
+						$this->give_search_box(esc_html__('Search Donors', 'give'), 'give-donors-report-search');
 					}
 
-					$this->bulk_actions( $which ); ?>
+					$this->bulk_actions($which); ?>
 
 				</div>
 				<?php
-				$this->extra_tablenav( $which );
-				$this->pagination( $which );
+				$this->extra_tablenav($which);
+				$this->pagination($which);
 				?>
 			</div>
 
@@ -172,33 +172,33 @@ 
 	 *
 	 * @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 'name' :
-				$name = '#' . $item['id'] . ' ';
-				$name .= ! empty( $item['name'] ) ? $item['name'] : '<em>' . esc_html__( 'Unnamed Donor', 'give' ) . '</em>';
-				$view_url = admin_url( 'edit.php?post_type=give_forms&page=give-donors&view=overview&id=' . $item['id'] );
-				$value    = '<a href="' . esc_url( $view_url ) . '">' . $name . '</a>';
+				$name = '#'.$item['id'].' ';
+				$name .= ! empty($item['name']) ? $item['name'] : '<em>'.esc_html__('Unnamed Donor', 'give').'</em>';
+				$view_url = admin_url('edit.php?post_type=give_forms&page=give-donors&view=overview&id='.$item['id']);
+				$value    = '<a href="'.esc_url($view_url).'">'.$name.'</a>';
 				break;
 
 			case 'num_donations' :
-				$value = '<a href="' .
-				         admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&user=' . urlencode( $item['email'] )
-				         ) . '">' . esc_html( $item['num_donations'] ) . '</a>';
+				$value = '<a href="'.
+				         admin_url('edit.php?post_type=give_forms&page=give-payment-history&user='.urlencode($item['email'])
+				         ).'">'.esc_html($item['num_donations']).'</a>';
 				break;
 
 			case 'amount_spent' :
-				$value = give_currency_filter( give_format_amount( $item[ $column_name ] ) );
+				$value = give_currency_filter(give_format_amount($item[$column_name]));
 				break;
 
 			default:
-				$value = isset( $item[ $column_name ] ) ? $item[ $column_name ] : null;
+				$value = isset($item[$column_name]) ? $item[$column_name] : null;
 				break;
 		}
 
-		return apply_filters( "give_report_column_{$column_name}", $value, $item['id'] );
+		return apply_filters("give_report_column_{$column_name}", $value, $item['id']);
 	}
 
 	/**
@@ -210,13 +210,13 @@ 
 	 */
 	public function get_columns() {
 		$columns = array(
-			'name'          => esc_html__( 'Name', 'give' ),
-			'email'         => esc_html__( 'Email', 'give' ),
-			'num_donations' => esc_html__( 'Donations', 'give' ),
-			'amount_spent'  => esc_html__( 'Total Donated', 'give' ),
+			'name'          => esc_html__('Name', 'give'),
+			'email'         => esc_html__('Email', 'give'),
+			'num_donations' => esc_html__('Donations', 'give'),
+			'amount_spent'  => esc_html__('Total Donated', 'give'),
 		);
 
-		return apply_filters( 'give_report_donor_columns', $columns );
+		return apply_filters('give_report_donor_columns', $columns);
 
 	}
 
@@ -229,10 +229,10 @@ 
 	 */
 	public function get_sortable_columns() {
 		return array(
-			'id'            => array( 'id', true ),
-			'name'          => array( 'name', true ),
-			'num_donations' => array( 'purchase_count', false ),
-			'amount_spent'  => array( 'purchase_value', false ),
+			'id'            => array('id', true),
+			'name'          => array('name', true),
+			'num_donations' => array('purchase_count', false),
+			'amount_spent'  => array('purchase_value', false),
 		);
 	}
 
@@ -243,7 +243,7 @@ 
 	 * @since  1.0
 	 * @return void
 	 */
-	public function bulk_actions( $which = '' ) {
+	public function bulk_actions($which = '') {
 
 	}
 
@@ -255,7 +255,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;
 	}
 
 	/**
@@ -266,7 +266,7 @@ 
 	 * @return mixed string If search is present, false otherwise
 	 */
 	public function get_search() {
-		return ! empty( $_GET['s'] ) ? urldecode( trim( $_GET['s'] ) ) : false;
+		return ! empty($_GET['s']) ? urldecode(trim($_GET['s'])) : false;
 	}
 
 	/**
@@ -285,15 +285,15 @@ 
 
 		// Get donor query.
 		$args   = $this->get_donor_query();
-		$donors = Give()->customers->get_customers( $args );
+		$donors = Give()->customers->get_customers($args);
 
-		if ( $donors ) {
+		if ($donors) {
 
-			$this->count = count( $donors );
+			$this->count = count($donors);
 
-			foreach ( $donors as $donor ) {
+			foreach ($donors as $donor) {
 
-				$user_id = ! empty( $donor->user_id ) ? absint( $donor->user_id ) : 0;
+				$user_id = ! empty($donor->user_id) ? absint($donor->user_id) : 0;
 
 				$data[] = array(
 					'id'            => $donor->id,
@@ -320,9 +320,9 @@ 
 		$_donor_query = $this->get_donor_query();
 
 		$_donor_query['number'] = -1;
-		$donors = Give()->customers->get_customers( $_donor_query );
+		$donors = Give()->customers->get_customers($_donor_query);
 
-		return count( $donors );
+		return count($donors);
 	}
 
 	/**
@@ -334,10 +334,10 @@ 
 	 */
 	public function get_donor_query() {
 		$paged   = $this->get_paged();
-		$offset  = $this->per_page * ( $paged - 1 );
+		$offset  = $this->per_page * ($paged - 1);
 		$search  = $this->get_search();
-		$order   = isset( $_GET['order'] ) ? sanitize_text_field( $_GET['order'] ) : 'DESC';
-		$orderby = isset( $_GET['orderby'] ) ? sanitize_text_field( $_GET['orderby'] ) : 'id';
+		$order   = isset($_GET['order']) ? sanitize_text_field($_GET['order']) : 'DESC';
+		$orderby = isset($_GET['orderby']) ? sanitize_text_field($_GET['orderby']) : 'id';
 
 		$args = array(
 			'number'  => $this->per_page,
@@ -346,10 +346,10 @@ 
 			'orderby' => $orderby,
 		);
 
-		if( $search ) {
-			if ( is_email( $search ) ) {
+		if ($search) {
+			if (is_email($search)) {
 				$args['email'] = $search;
-			} elseif ( is_numeric( $search ) ) {
+			} elseif (is_numeric($search)) {
 				$args['id'] = $search;
 			} else {
 				$args['name'] = $search;
@@ -376,16 +376,16 @@ 
 		$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->reports_data();
 
 		$this->total = $this->get_donor_count();
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 			'total_items' => $this->total,
 			'per_page'    => $this->per_page,
-			'total_pages' => ceil( $this->total / $this->per_page ),
-		) );
+			'total_pages' => ceil($this->total / $this->per_page),
+		));
 	}
 }

includes/actions.php

Spacing +39 added lines, -39 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -25,14 +25,14 @@ 
  */
 function give_get_actions() {
 
-	$_get_action = ! empty( $_GET['give_action'] ) ? $_GET['give_action'] : null;
+	$_get_action = ! empty($_GET['give_action']) ? $_GET['give_action'] : null;
 
 	// Add backward compatibility to give-action param ( $_GET or $_POST )
-	if(  doing_action( 'admin_init' ) && empty( $_get_action ) ) {
-		$_get_action = ! empty( $_GET['give-action'] ) ? $_GET['give-action'] : null;
+	if (doing_action('admin_init') && empty($_get_action)) {
+		$_get_action = ! empty($_GET['give-action']) ? $_GET['give-action'] : null;
 	}
 
-	if ( isset( $_get_action ) ) {
+	if (isset($_get_action)) {
 		/**
 		 * Fires in WordPress init or admin init, when give_action is present in $_GET.
 		 *
@@ -40,13 +40,13 @@ 
 		 *
 		 * @param array $_GET Array of HTTP GET variables.
 		 */
-		do_action( "give_{$_get_action}", $_GET );
+		do_action("give_{$_get_action}", $_GET);
 	}
 
 }
 
-add_action( 'init', 'give_get_actions' );
-add_action( 'admin_init', 'give_get_actions' );
+add_action('init', 'give_get_actions');
+add_action('admin_init', 'give_get_actions');
 
 /**
  * Hooks Give actions, when present in the $_POST superglobal. Every give_action
@@ -59,15 +59,15 @@ 
  */
 function give_post_actions() {
 
-	$_post_action = ! empty( $_POST['give_action'] ) ? $_POST['give_action'] : null;
+	$_post_action = ! empty($_POST['give_action']) ? $_POST['give_action'] : null;
 
 
 	// Add backward compatibility to give-action param ( $_GET or $_POST )
-	if(  doing_action( 'admin_init' ) && empty( $_post_action ) ) {
-		$_post_action = ! empty( $_POST['give-action'] ) ? $_POST['give-action'] : null;
+	if (doing_action('admin_init') && empty($_post_action)) {
+		$_post_action = ! empty($_POST['give-action']) ? $_POST['give-action'] : null;
 	}
 
-	if ( isset( $_post_action ) ) {
+	if (isset($_post_action)) {
 		/**
 		 * Fires in WordPress init or admin init, when give_action is present in $_POST.
 		 *
@@ -75,13 +75,13 @@ 
 		 *
 		 * @param array $_POST Array of HTTP POST variables.
 		 */
-		do_action( "give_{$_post_action}", $_POST );
+		do_action("give_{$_post_action}", $_POST);
 	}
 
 }
 
-add_action( 'init', 'give_post_actions' );
-add_action( 'admin_init', 'give_post_actions' );
+add_action('init', 'give_post_actions');
+add_action('admin_init', 'give_post_actions');
 
 /**
  * Connect WordPress user with Donor.
@@ -91,27 +91,27 @@ 
  * @param  array $user_data User Data
  * @return void
  */
-function give_connect_donor_to_wpuser( $user_id, $user_data ){
-	$donor = new Give_Customer( $user_data['user_email'] );
+function give_connect_donor_to_wpuser($user_id, $user_data) {
+	$donor = new Give_Customer($user_data['user_email']);
 
 	// Validate donor id and check if do nor is already connect to wp user or not.
-	if( $donor->id && ! $donor->user_id ) {
+	if ($donor->id && ! $donor->user_id) {
 
 		// Update donor user_id.
-		if( $donor->update( array( 'user_id' => $user_id ) ) ) {
-			$donor_note = sprintf( esc_html__( 'WordPress user #%d is connected to #%d', 'give' ), $user_id, $donor->id );
-			$donor->add_note( $donor_note );
+		if ($donor->update(array('user_id' => $user_id))) {
+			$donor_note = sprintf(esc_html__('WordPress user #%d is connected to #%d', 'give'), $user_id, $donor->id);
+			$donor->add_note($donor_note);
 
 			// Update user_id meta in payments.
-			if( ! empty( $donor->payment_ids ) && ( $donations = explode( ',', $donor->payment_ids ) ) ) {
-				foreach ( $donations as $donation  ) {
-					update_post_meta( $donation, '_give_payment_user_id', $user_id );
+			if ( ! empty($donor->payment_ids) && ($donations = explode(',', $donor->payment_ids))) {
+				foreach ($donations as $donation) {
+					update_post_meta($donation, '_give_payment_user_id', $user_id);
 				}
 			}
 		}
 	}
 }
-add_action( 'give_insert_user', 'give_connect_donor_to_wpuser', 10, 2 );
+add_action('give_insert_user', 'give_connect_donor_to_wpuser', 10, 2);
 
 
 /**
@@ -125,24 +125,24 @@ 
 function give_validate_license_when_site_migrated() {
 	// Store current site address if not already stored.
 	$homeurl = home_url();
-	if( ! get_option( 'give_site_address_before_migrate' ) ) {
+	if ( ! get_option('give_site_address_before_migrate')) {
 		// Update site address.
-		update_option( 'give_site_address_before_migrate', $homeurl );
+		update_option('give_site_address_before_migrate', $homeurl);
 
 		return;
 	}
 
-	if( $homeurl !== get_option( 'give_site_address_before_migrate' ) ) {
+	if ($homeurl !== get_option('give_site_address_before_migrate')) {
 		// Immediately run cron.
-		wp_schedule_single_event( time() , 'give_validate_license_when_site_migrated' );
+		wp_schedule_single_event(time(), 'give_validate_license_when_site_migrated');
 
 		// Update site address.
-		update_option( 'give_site_address_before_migrate', home_url() );
+		update_option('give_site_address_before_migrate', home_url());
 	}
 
 }
-add_action( 'init', 'give_validate_license_when_site_migrated' );
-add_action( 'admin_init', 'give_validate_license_when_site_migrated' );
+add_action('init', 'give_validate_license_when_site_migrated');
+add_action('admin_init', 'give_validate_license_when_site_migrated');
 
 
 /**
@@ -151,15 +151,15 @@ 
  * @since 1.8
  * @param $data
  */
-function give_donor_batch_export_complete( $data ) {
+function give_donor_batch_export_complete($data) {
 	// Remove donor ids cache.
-	if(
-		isset( $data['class'] )
+	if (
+		isset($data['class'])
 		&& 'Give_Batch_Customers_Export' === $data['class']
-		&& ! empty( $data['forms'] )
-		&& isset( $data['give_export_option']['query_id'] )
+		&& ! empty($data['forms'])
+		&& isset($data['give_export_option']['query_id'])
 	) {
-		delete_transient( $data['give_export_option']['query_id'] );
+		delete_transient($data['give_export_option']['query_id']);
 	}
 }
-add_action('give_file_export_complete', 'give_donor_batch_export_complete' );
+add_action('give_file_export_complete', 'give_donor_batch_export_complete');

includes/admin/abstract-admin-settings-page.php

Spacing +22 added lines, -22 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_Page' ) ) :
+if ( ! class_exists('Give_Settings_Page')) :
 
 	/**
 	 * Give_Settings_Page.
@@ -62,11 +62,11 @@ 
 			// Get current setting page.
 			$this->current_setting_page = give_get_current_setting_page();
 
-			add_filter( "give_default_setting_tab_section_{$this->id}", array( $this, 'set_default_setting_tab' ), 10 );
-			add_filter( "{$this->current_setting_page}_tabs_array", array( $this, 'add_settings_page' ), 20 );
-			add_action( "{$this->current_setting_page}_sections_{$this->id}_page", array( $this, 'output_sections' ) );
-			add_action( "{$this->current_setting_page}_settings_{$this->id}_page", array( $this, 'output' ) );
-			add_action( "{$this->current_setting_page}_save_{$this->id}", array( $this, 'save' ) );
+			add_filter("give_default_setting_tab_section_{$this->id}", array($this, 'set_default_setting_tab'), 10);
+			add_filter("{$this->current_setting_page}_tabs_array", array($this, 'add_settings_page'), 20);
+			add_action("{$this->current_setting_page}_sections_{$this->id}_page", array($this, 'output_sections'));
+			add_action("{$this->current_setting_page}_settings_{$this->id}_page", array($this, 'output'));
+			add_action("{$this->current_setting_page}_save_{$this->id}", array($this, 'save'));
 		}
 
 		/**
@@ -78,7 +78,7 @@ 
 		 *
 		 * @return string
 		 */
-		function set_default_setting_tab( $setting_tab ) {
+		function set_default_setting_tab($setting_tab) {
 			return $this->default_tab;
 		}
 
@@ -91,8 +91,8 @@ 
 		 *
 		 * @return array
 		 */
-		public function add_settings_page( $pages ) {
-			$pages[ $this->id ] = $this->label;
+		public function add_settings_page($pages) {
+			$pages[$this->id] = $this->label;
 
 			return $pages;
 		}
@@ -111,7 +111,7 @@ 
 			 *
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, array() );
+			$settings = apply_filters('give_get_settings_'.$this->id, array());
 
 			// Output.
 			return $settings;
@@ -124,7 +124,7 @@ 
 		 * @return array
 		 */
 		public function get_sections() {
-			return apply_filters( 'give_get_sections_' . $this->id, array() );
+			return apply_filters('give_get_sections_'.$this->id, array());
 		}
 
 		/**
@@ -141,29 +141,29 @@ 
 			$sections = $this->get_sections();
 
 			// Show section settings only if setting section exist.
-			if ( $current_section && ! in_array( $current_section, array_keys( $sections ) ) ) {
-				echo '<div class="error"><p>' . __( 'Oops, this settings page does not exist.', 'give' ) . '</p></div>';
+			if ($current_section && ! in_array($current_section, array_keys($sections))) {
+				echo '<div class="error"><p>'.__('Oops, this settings page does not exist.', 'give').'</p></div>';
 				$GLOBALS['give_hide_save_button'] = true;
 
 				return;
 			}
 
 			// Bailout.
-			if ( empty( $sections ) ) {
+			if (empty($sections)) {
 				return;
 			}
 
-			if ( is_null( $this->current_setting_page ) ) {
+			if (is_null($this->current_setting_page)) {
 				$this->current_setting_page = give_get_current_setting_page();
 			}
 
 			echo '<ul class="subsubsub">';
 
 			// Get section keys.
-			$array_keys = array_keys( $sections );
+			$array_keys = array_keys($sections);
 
-			foreach ( $sections as $id => $label ) {
-				echo '<li><a href="' . admin_url( 'edit.php?post_type=give_forms&page=' . $this->current_setting_page . '&tab=' . $this->id . '&section=' . sanitize_title( $id ) ) . '" class="' . ( $current_section == $id ? 'current' : '' ) . '">' . $label . '</a> ' . ( end( $array_keys ) == $id ? '' : '|' ) . ' </li>';
+			foreach ($sections as $id => $label) {
+				echo '<li><a href="'.admin_url('edit.php?post_type=give_forms&page='.$this->current_setting_page.'&tab='.$this->id.'&section='.sanitize_title($id)).'" class="'.($current_section == $id ? 'current' : '').'">'.$label.'</a> '.(end($array_keys) == $id ? '' : '|').' </li>';
 			}
 
 			echo '</ul><br class="clear" /><hr>';
@@ -178,7 +178,7 @@ 
 		public function output() {
 			$settings = $this->get_settings();
 
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
 		}
 
 		/**
@@ -191,14 +191,14 @@ 
 			$settings        = $this->get_settings();
 			$current_section = give_get_current_setting_section();
 
-			Give_Admin_Settings::save_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::save_fields($settings, 'give_settings');
 
 			/**
 			 * Trigger Action
 			 *
 			 * @since 1.8
 			 */
-			do_action( 'give_update_options_' . $this->id . '_' . $current_section );
+			do_action('give_update_options_'.$this->id.'_'.$current_section);
 		}
 	}
 

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

Spacing +16 added lines, -16 removed lines

@@ -9,11 +9,11 @@ 
  * @since       1.8
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit; // Exit if accessed directly
 }
 
-if ( ! class_exists( 'Give_Settings_Export' ) ) :
+if ( ! class_exists('Give_Settings_Export')) :
 
 	/**
 	 * Give_Settings_Export.
@@ -43,16 +43,16 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'export';
-			$this->label = esc_html__( 'Export', 'give' );
+			$this->label = esc_html__('Export', 'give');
 
-			add_filter( 'give-reports_tabs_array', array( $this, 'add_settings_page' ), 20 );
-			add_action( "give-reports_settings_{$this->id}_page", array( $this, 'output' ) );
-			add_action( 'give_admin_field_report_export', array( $this, 'render_report_export_field' ), 10, 2 );
+			add_filter('give-reports_tabs_array', array($this, 'add_settings_page'), 20);
+			add_action("give-reports_settings_{$this->id}_page", array($this, 'output'));
+			add_action('give_admin_field_report_export', array($this, 'render_report_export_field'), 10, 2);
 
 			// Do not use main donor for this tab.
-			if( give_get_current_setting_tab() === $this->id ) {
-				add_action( 'give-reports_open_form', '__return_empty_string' );
-				add_action( 'give-reports_close_form', '__return_empty_string' );
+			if (give_get_current_setting_tab() === $this->id) {
+				add_action('give-reports_open_form', '__return_empty_string');
+				add_action('give-reports_close_form', '__return_empty_string');
 			}
 		}
 
@@ -63,8 +63,8 @@ 
 		 * @param  array $pages Lst of pages.
 		 * @return array
 		 */
-		public function add_settings_page( $pages ) {
-			$pages[ $this->id ] = $this->label;
+		public function add_settings_page($pages) {
+			$pages[$this->id] = $this->label;
 
 			return $pages;
 		}
@@ -86,7 +86,7 @@ 
 			 * @param  array $settings
 			 */
 			$settings = apply_filters(
-				'give_get_settings_' . $this->id,
+				'give_get_settings_'.$this->id,
 				array(
 					array(
 						'id'   => 'give_reports_export',
@@ -95,7 +95,7 @@ 
 					),
 					array(
 						'id'   => 'export',
-						'name' => esc_html__( 'Export', 'give' ),
+						'name' => esc_html__('Export', 'give'),
 						'type' => 'report_export',
 					),
 					array(
@@ -119,7 +119,7 @@ 
 		public function output() {
 			$settings = $this->get_settings();
 
-			Give_Admin_Settings::output_fields( $settings, 'give_settings' );
+			Give_Admin_Settings::output_fields($settings, 'give_settings');
 		}
 
 		/**
@@ -131,8 +131,8 @@ 
 		 * @param $field
 		 * @param $option_value
 		 */
-		public function render_report_export_field( $field, $option_value ) {
-			do_action( 'give_reports_tab_export');
+		public function render_report_export_field($field, $option_value) {
+			do_action('give_reports_tab_export');
 		}
 	}
 

includes/admin/reporting/reports.php

Spacing +108 added lines, -108 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,13 +100,13 @@ 
  */
 function give_reports_default_views() {
 	$views = array(
-		'earnings' => esc_html__( 'Income', 'give' ),
-		'forms'    => esc_html__( 'Forms', 'give' ),
-		'donors'   => esc_html__( 'Donors', 'give' ),
-		'gateways' => esc_html__( 'Donation Methods', 'give' ),
+		'earnings' => esc_html__('Income', 'give'),
+		'forms'    => esc_html__('Forms', 'give'),
+		'donors'   => esc_html__('Donors', 'give'),
+		'gateways' => esc_html__('Donation Methods', 'give'),
 	);
 
-	$views = apply_filters( 'give_report_views', $views );
+	$views = apply_filters('give_report_views', $views);
 
 	return $views;
 }
@@ -121,15 +121,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);
 }
 
 /**
@@ -142,7 +142,7 @@ 
 	$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'];
 	}
 
@@ -151,10 +151,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
@@ -164,19 +164,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>
 
@@ -188,12 +188,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
 	/**
@@ -201,7 +201,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_report_view_actions_after' );
+	do_action('give_report_view_actions_after');
 }
 
 /**
@@ -214,18 +214,18 @@ 
  */
 function give_reports_forms_table() {
 
-	if ( isset( $_GET['form-id'] ) ) {
+	if (isset($_GET['form-id'])) {
 		return;
 	}
 
-	include( dirname( __FILE__ ) . '/class-form-reports-table.php' );
+	include(dirname(__FILE__).'/class-form-reports-table.php');
 
 	$give_table = new Give_Form_Reports_Table();
 	$give_table->prepare_items();
 	$give_table->display();
 }
 
-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
@@ -234,20 +234,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 Reports Donors Table
@@ -258,7 +258,7 @@ 
  * @return void
  */
 function give_reports_donors_table() {
-	include( dirname( __FILE__ ) . '/class-donor-reports-table.php' );
+	include(dirname(__FILE__).'/class-donor-reports-table.php');
 
 	$give_table = new Give_Donor_Reports_Table();
 	$give_table->prepare_items();
@@ -270,11 +270,11 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_logs_donors_table_top' );
+		do_action('give_logs_donors_table_top');
 		?>
 		<form id="give-donors-filter" method="get">
 			<?php
-			$give_table->search_box( esc_html__( 'Search', 'give' ), 'give-donors' );
+			$give_table->search_box(esc_html__('Search', 'give'), 'give-donors');
 			$give_table->display();
 			?>
 			<input type="hidden" name="post_type" value="give_forms"/>
@@ -287,13 +287,13 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_logs_donors_table_bottom' );
+		do_action('give_logs_donors_table_bottom');
 		?>
 	</div>
 	<?php
 }
 
-add_action( 'give_reports_view_donors', 'give_reports_donors_table' );
+add_action('give_reports_view_donors', 'give_reports_donors_table');
 
 
 /**
@@ -305,14 +305,14 @@ 
  * @return void
  */
 function give_reports_gateways_table() {
-	include( dirname( __FILE__ ) . '/class-gateways-reports-table.php' );
+	include(dirname(__FILE__).'/class-gateways-reports-table.php');
 
 	$give_table = new Give_Gateawy_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');
 
 
 /**
@@ -324,13 +324,13 @@ 
 function give_reports_earnings() {
 	?>
 	<div class="tablenav top reports-table-nav">
-		<h3 class="alignleft reports-earnings-title"><span><?php esc_html_e( 'Income Report', 'give' ); ?></span></h3>
+		<h3 class="alignleft reports-earnings-title"><span><?php esc_html_e('Income Report', 'give'); ?></span></h3>
 	</div>
 	<?php
 	give_reports_graph();
 }
 
-add_action( 'give_reports_view_earnings', 'give_reports_earnings' );
+add_action('give_reports_view_earnings', 'give_reports_earnings');
 
 
 /**
@@ -351,14 +351,14 @@ 
 				 *
 				 * @since 1.0
 				 */
-				do_action( 'give_reports_tab_export_content_top' );
+				do_action('give_reports_tab_export_content_top');
 				?>
 
 				<table class="widefat export-options-table give-table">
 					<thead>
 						<tr>
-							<th scope="col"><?php esc_html_e( 'Export Type', 'give' ); ?></th>
-							<th scope="col"><?php esc_html_e( 'Export Options', 'give' ); ?></th>
+							<th scope="col"><?php esc_html_e('Export Type', 'give'); ?></th>
+							<th scope="col"><?php esc_html_e('Export Options', 'give'); ?></th>
 						</tr>
 					</thead>
 					<tbody>
@@ -370,42 +370,42 @@ 
 						 *
 						 * @since 1.0
 						 */
-						do_action( 'give_reports_tab_export_table_top' );
+						do_action('give_reports_tab_export_table_top');
 						?>
 						<tr class="give-export-pdf-sales-earnings">
 							<td scope="row" class="row-title">
-								<h3><span><?php esc_html_e( 'Export PDF of Donations and Income', 'give' ); ?></span>
+								<h3><span><?php esc_html_e('Export PDF of Donations and Income', 'give'); ?></span>
 								</h3>
-								<p><?php esc_html_e( 'Download a PDF of Donations and Income reports for all forms for the current year.', 'give' ); ?></p>
+								<p><?php esc_html_e('Download a PDF of Donations and Income reports for all forms for the current year.', 'give'); ?></p>
 							</td>
 							<td>
-								<a class="button" href="<?php echo wp_nonce_url( add_query_arg( array( 'give-action' => 'generate_pdf' ) ), 'give_generate_pdf' ); ?>"><?php esc_html_e( 'Generate PDF', 'give' ); ?></a>
+								<a class="button" href="<?php echo wp_nonce_url(add_query_arg(array('give-action' => 'generate_pdf')), 'give_generate_pdf'); ?>"><?php esc_html_e('Generate PDF', 'give'); ?></a>
 							</td>
 						</tr>
 						<tr class="alternate give-export-sales-earnings">
 							<td scope="row" class="row-title">
-								<h3><span><?php esc_html_e( 'Export Income and Donation Stats', 'give' ); ?></span></h3>
-								<p><?php esc_html_e( 'Download a CSV of income and donations over time.', 'give' ); ?></p>
+								<h3><span><?php esc_html_e('Export Income and Donation Stats', 'give'); ?></span></h3>
+								<p><?php esc_html_e('Download a CSV of income and donations over time.', 'give'); ?></p>
 							</td>
 							<td>
 								<form method="post">
 									<?php
 									printf(
 									/* translators: 1: start date dropdown 2: end date dropdown */
-										esc_html__( '%1$s to %2$s', 'give' ),
-										Give()->html->year_dropdown( 'start_year' ) . ' ' . Give()->html->month_dropdown( 'start_month' ),
-										Give()->html->year_dropdown( 'end_year' ) . ' ' . Give()->html->month_dropdown( 'end_month' )
+										esc_html__('%1$s to %2$s', 'give'),
+										Give()->html->year_dropdown('start_year').' '.Give()->html->month_dropdown('start_month'),
+										Give()->html->year_dropdown('end_year').' '.Give()->html->month_dropdown('end_month')
 									);
 									?>
 									<input type="hidden" name="give-action" value="earnings_export"/>
-									<input type="submit" value="<?php esc_attr_e( 'Generate CSV', 'give' ); ?>" class="button-secondary"/>
+									<input type="submit" value="<?php esc_attr_e('Generate CSV', 'give'); ?>" class="button-secondary"/>
 								</form>
 							</td>
 						</tr>
 						<tr class="give-export-payment-history">
 							<td scope="row" class="row-title">
-								<h3><span><?php esc_html_e( 'Export Donation History', 'give' ); ?></span></h3>
-								<p><?php esc_html_e( 'Download a CSV of all donations recorded.', 'give' ); ?></p>
+								<h3><span><?php esc_html_e('Export Donation History', 'give'); ?></span></h3>
+								<p><?php esc_html_e('Download a CSV of all donations recorded.', 'give'); ?></p>
 							</td>
 							<td>
 								<form id="give-export-payments" class="give-export-form" method="post">
@@ -413,27 +413,27 @@ 
 									$args = array(
 										'id'          => 'give-payment-export-start',
 										'name'        => 'start',
-										'placeholder' => esc_attr__( 'Start date', 'give' ),
+										'placeholder' => esc_attr__('Start date', 'give'),
 									);
-									echo Give()->html->date_field( $args ); ?>
+									echo Give()->html->date_field($args); ?>
 									<?php
 									$args = array(
 										'id'          => 'give-payment-export-end',
 										'name'        => 'end',
-										'placeholder' => esc_attr__( 'End date', 'give' ),
+										'placeholder' => esc_attr__('End date', 'give'),
 									);
-									echo Give()->html->date_field( $args ); ?>
+									echo Give()->html->date_field($args); ?>
 									<select name="status">
-										<option value="any"><?php esc_html_e( 'All Statuses', 'give' ); ?></option>
+										<option value="any"><?php esc_html_e('All Statuses', 'give'); ?></option>
 										<?php
 										$statuses = give_get_payment_statuses();
-										foreach ( $statuses as $status => $label ) {
-											echo '<option value="' . $status . '">' . $label . '</option>';
+										foreach ($statuses as $status => $label) {
+											echo '<option value="'.$status.'">'.$label.'</option>';
 										}
 										?>
 									</select>
 									<?php
-									if ( give_is_setting_enabled( give_get_option( 'categories' ) ) ) {
+									if (give_is_setting_enabled(give_get_option('categories'))) {
 										echo Give()->html->category_dropdown(
 											'give_forms_categories[]',
 											0,
@@ -443,13 +443,13 @@ 
 												'multiple'        => true,
 												'selected'        => array(),
 												'show_option_all' => false,
-												'placeholder'     => __( 'Choose one or more from categories', 'give' ),
+												'placeholder'     => __('Choose one or more from categories', 'give'),
 											)
 										);
 										$add_break = true;
 									}
 
-									if ( give_is_setting_enabled( give_get_option( 'tags' ) ) ) {
+									if (give_is_setting_enabled(give_get_option('tags'))) {
 										echo Give()->html->tags_dropdown(
 											'give_forms_tags[]',
 											0,
@@ -459,17 +459,17 @@ 
 												'multiple'        => true,
 												'selected'        => array(),
 												'show_option_all' => false,
-												'placeholder'     => __( 'Choose one or more from tags', 'give' ),
+												'placeholder'     => __('Choose one or more from tags', 'give'),
 											)
 										);
 										$add_break = true;
 									}
 
-									wp_nonce_field( 'give_ajax_export', 'give_ajax_export' );
+									wp_nonce_field('give_ajax_export', 'give_ajax_export');
 									?>
 									<input type="hidden" name="give-export-class" value="Give_Batch_Payments_Export"/>
 									<span>
-									<input type="submit" value="<?php esc_attr_e( 'Generate CSV', 'give' ); ?>" class="button-secondary"/>
+									<input type="submit" value="<?php esc_attr_e('Generate CSV', 'give'); ?>" class="button-secondary"/>
 									<span class="spinner"></span>
 								</span>
 								</form>
@@ -477,8 +477,8 @@ 
 						</tr>
 						<tr class="alternate give-export-donors">
 							<td scope="row" class="row-title">
-								<h3><span><?php esc_html_e( 'Export Donors in CSV', 'give' ); ?></span></h3>
-								<p><?php esc_html_e( 'Download an export of donors for all donation forms or only those who have given to a particular form.', 'give' ); ?></p>
+								<h3><span><?php esc_html_e('Export Donors in CSV', 'give'); ?></span></h3>
+								<p><?php esc_html_e('Download an export of donors for all donation forms or only those who have given to a particular form.', 'give'); ?></p>
 							</td>
 							<td>
 								<form method="post" id="give_donor_export" class="give-export-form">
@@ -489,46 +489,46 @@ 
 										'id'     => 'give_customer_export_form',
 										'chosen' => true,
 									);
-									echo Give()->html->forms_dropdown( $args ); ?>
+									echo Give()->html->forms_dropdown($args); ?>
 
-									<input type="submit" value="<?php esc_attr_e( 'Generate CSV', 'give' ); ?>" class="button-secondary"/>
+									<input type="submit" value="<?php esc_attr_e('Generate CSV', 'give'); ?>" class="button-secondary"/>
 
 									<div id="export-donor-options-wrap" class="give-clearfix">
-										<p><?php esc_html_e( 'Export Columns:', 'give' ); ?></p>
+										<p><?php esc_html_e('Export Columns:', 'give'); ?></p>
 										<ul id="give-export-option-ul">
 											<li>
-												<label for="give-export-fullname"><input type="checkbox" checked name="give_export_option[full_name]" id="give-export-fullname"><?php esc_html_e( 'Name', 'give' ); ?>
+												<label for="give-export-fullname"><input type="checkbox" checked name="give_export_option[full_name]" id="give-export-fullname"><?php esc_html_e('Name', 'give'); ?>
 												</label>
 											</li>
 											<li>
-												<label for="give-export-email"><input type="checkbox" checked name="give_export_option[email]" id="give-export-email"><?php esc_html_e( 'Email', 'give' ); ?>
+												<label for="give-export-email"><input type="checkbox" checked name="give_export_option[email]" id="give-export-email"><?php esc_html_e('Email', 'give'); ?>
 												</label>
 											</li>
 											<li>
-												<label for="give-export-address"><input type="checkbox" checked name="give_export_option[address]" id="give-export-address"><?php esc_html_e( 'Address', 'give' ); ?>
+												<label for="give-export-address"><input type="checkbox" checked name="give_export_option[address]" id="give-export-address"><?php esc_html_e('Address', 'give'); ?>
 												</label>
 											</li>
 											<li>
-												<label for="give-export-userid"><input type="checkbox" checked name="give_export_option[userid]" id="give-export-userid"><?php esc_html_e( 'User ID', 'give' ); ?>
+												<label for="give-export-userid"><input type="checkbox" checked name="give_export_option[userid]" id="give-export-userid"><?php esc_html_e('User ID', 'give'); ?>
 												</label>
 											</li>
 											<li>
-												<label for="give-export-first-donation-date"><input type="checkbox" checked name="give_export_option[date_first_donated]" id="give-export-first-donation-date"><?php esc_html_e( 'First Donation Date', 'give' ); ?>
+												<label for="give-export-first-donation-date"><input type="checkbox" checked name="give_export_option[date_first_donated]" id="give-export-first-donation-date"><?php esc_html_e('First Donation Date', 'give'); ?>
 												</label>
 											</li>
 											<li>
-												<label for="give-export-donation-number"><input type="checkbox" checked name="give_export_option[donations]" id="give-export-donation-number"><?php esc_html_e( 'Number of Donations', 'give' ); ?>
+												<label for="give-export-donation-number"><input type="checkbox" checked name="give_export_option[donations]" id="give-export-donation-number"><?php esc_html_e('Number of Donations', 'give'); ?>
 												</label>
 											</li>
 											<li>
-												<label for="give-export-donation-sum"><input type="checkbox" checked name="give_export_option[donation_sum]" id="give-export-donation-sum"><?php esc_html_e( 'Total Donated', 'give' ); ?>
+												<label for="give-export-donation-sum"><input type="checkbox" checked name="give_export_option[donation_sum]" id="give-export-donation-sum"><?php esc_html_e('Total Donated', 'give'); ?>
 												</label>
 											</li>
 										</ul>
 									</div>
-									<?php wp_nonce_field( 'give_ajax_export', 'give_ajax_export' ); ?>
+									<?php wp_nonce_field('give_ajax_export', 'give_ajax_export'); ?>
 									<input type="hidden" name="give-export-class" value="Give_Batch_Customers_Export"/>
-									<input type="hidden" name="give_export_option[query_id]" value="<?php echo uniqid( 'give_' ); ?>"/>
+									<input type="hidden" name="give_export_option[query_id]" value="<?php echo uniqid('give_'); ?>"/>
 									<input type="hidden" name="give_action" value="email_export"/>
 								</form>
 							</td>
@@ -541,7 +541,7 @@ 
 						 *
 						 * @since 1.0
 						 */
-						do_action( 'give_reports_tab_export_table_bottom' );
+						do_action('give_reports_tab_export_table_bottom');
 						?>
 					</tbody>
 				</table>
@@ -552,7 +552,7 @@ 
 				 *
 				 * @since 1.0
 				 */
-				do_action( 'give_reports_tab_export_content_bottom' );
+				do_action('give_reports_tab_export_content_bottom');
 				?>
 
 			</div>
@@ -563,7 +563,7 @@ 
 	<?php
 }
 
-add_action( 'give_reports_tab_export', 'give_reports_tab_export' );
+add_action('give_reports_tab_export', 'give_reports_tab_export');
 
 /**
  * Renders the Reports page
@@ -573,7 +573,7 @@ 
  */
 function give_reports_tab_logs() {
 
-	require( GIVE_PLUGIN_DIR . 'includes/admin/reporting/logs.php' );
+	require(GIVE_PLUGIN_DIR.'includes/admin/reporting/logs.php');
 
 	// Get current section.
 	$current_section = $_GET['section'] = give_get_current_setting_section();
@@ -583,7 +583,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( "give_logs_view_{$current_section}" );
+	do_action("give_logs_view_{$current_section}");
 }
 
 /**
@@ -594,9 +594,9 @@ 
  */
 function give_estimated_monthly_stats() {
 
-	$estimated = get_transient( 'give_estimated_monthly_stats' );
+	$estimated = get_transient('give_estimated_monthly_stats');
 
-	if ( false === $estimated ) {
+	if (false === $estimated) {
 
 		$estimated = array(
 			'earnings' => 0,
@@ -605,22 +605,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
-		set_transient( 'give_estimated_monthly_stats', $estimated, 86400 );
+		set_transient('give_estimated_monthly_stats', $estimated, 86400);
 	}
 
-	return maybe_unserialize( $estimated );
+	return maybe_unserialize($estimated);
 }
 
 // @TODO: After release 1.8 Donations -> Reports generates with new setting api, so we can remove some old code from this file.