WordImpress / Give

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.

includes/admin/reporting/class-settings-gateways.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_Gateways' ) ) :
+if ( ! class_exists('Give_Settings_Gateways')) :
 
 	/**
 	 * Give_Settings_Gateways.
@@ -43,16 +43,16 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'gateways';
-			$this->label = esc_html__( 'Donation Methods', 'give' );
+			$this->label = esc_html__('Donation Methods', '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_gateways', array( $this, 'render_report_gateways_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_gateways', array($this, 'render_report_gateways_field'), 10, 2);
 
 			// Do not use main form 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_gateways',
@@ -95,7 +95,7 @@ 
 					),
 					array(
 						'id'   => 'gateways',
-						'name' => esc_html__( 'Gateways', 'give' ),
+						'name' => esc_html__('Gateways', 'give'),
 						'type' => 'report_gateways',
 					),
 					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_gateways_field( $field, $option_value ) {
-			do_action( 'give_reports_view_gateways');
+		public function render_report_gateways_field($field, $option_value) {
+			do_action('give_reports_view_gateways');
 		}
 	}
 

includes/admin/reporting/class-settings-donors.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_Donors' ) ) :
+if ( ! class_exists('Give_Settings_Donors')) :
 
 	/**
 	 * Give_Settings_Donors.
@@ -43,16 +43,16 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'donors';
-			$this->label = esc_html__( 'Donors', 'give' );
+			$this->label = esc_html__('Donors', '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_donors', array( $this, 'render_report_donors_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_donors', array($this, 'render_report_donors_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_donors',
@@ -95,7 +95,7 @@ 
 					),
 					array(
 						'id'   => 'donors',
-						'name' => esc_html__( 'Donors', 'give' ),
+						'name' => esc_html__('Donors', 'give'),
 						'type' => 'report_donors',
 					),
 					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_donors_field( $field, $option_value ) {
-			do_action( 'give_reports_view_donors');
+		public function render_report_donors_field($field, $option_value) {
+			do_action('give_reports_view_donors');
 		}
 	}
 

includes/admin/reporting/class-settings-forms.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_Forms' ) ) :
+if ( ! class_exists('Give_Settings_Forms')) :
 
 	/**
 	 * Give_Settings_Forms.
@@ -43,16 +43,16 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'forms';
-			$this->label = esc_html__( 'Forms', 'give' );
+			$this->label = esc_html__('Forms', '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_forms', array( $this, 'render_report_forms_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_forms', array($this, 'render_report_forms_field'), 10, 2);
 
 			// Do not use main form 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_forms',
@@ -95,7 +95,7 @@ 
 					),
 					array(
 						'id'   => 'forms',
-						'name' => esc_html__( 'Forms', 'give' ),
+						'name' => esc_html__('Forms', 'give'),
 						'type' => 'report_forms',
 					),
 					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_forms_field( $field, $option_value ) {
-			do_action( 'give_reports_view_forms');
+		public function render_report_forms_field($field, $option_value) {
+			do_action('give_reports_view_forms');
 		}
 	}
 

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

Spacing +34 added lines, -34 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -106,14 +106,14 @@ 
 	 * @param $_step int The step to process
 	 * @since 1.5
 	 */
-	public function __construct( $_step = 1 ) {
+	public function __construct($_step = 1) {
 
 		$upload_dir       = wp_upload_dir();
 		$this->filetype   = '.csv';
-		$this->filename   = 'give-' . $this->export_type . $this->filetype;
-		$this->file       = trailingslashit( $upload_dir['basedir'] ) . $this->filename;
+		$this->filename   = 'give-'.$this->export_type.$this->filetype;
+		$this->file       = trailingslashit($upload_dir['basedir']).$this->filename;
 
-		if ( ! is_writeable( $upload_dir['basedir'] ) ) {
+		if ( ! is_writeable($upload_dir['basedir'])) {
 			$this->is_writable = false;
 		}
 
@@ -129,20 +129,20 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( esc_html__( 'You do not have permission to export data.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(esc_html__('You do not have permission to export data.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 		}
 
-		if( $this->step < 2 ) {
+		if ($this->step < 2) {
 
 			// Make sure we start with a fresh file on step 1
-			@unlink( $this->file );
+			@unlink($this->file);
 			$this->print_csv_cols();
 		}
 
 		$rows = $this->print_csv_rows();
 
-		if( $rows ) {
+		if ($rows) {
 			return true;
 		} else {
 			return false;
@@ -162,14 +162,14 @@ 
 		$col_data = '';
 		$cols = $this->get_csv_cols();
 		$i = 1;
-		foreach( $cols as $col_id => $column ) {
-			$col_data .= '"' . addslashes( $column ) . '"';
-			$col_data .= $i == count( $cols ) ? '' : ',';
+		foreach ($cols as $col_id => $column) {
+			$col_data .= '"'.addslashes($column).'"';
+			$col_data .= $i == count($cols) ? '' : ',';
 			$i++;
 		}
 		$col_data .= "\r\n";
 
-		$this->stash_step_data( $col_data );
+		$this->stash_step_data($col_data);
 
 		return $col_data;
 
@@ -188,23 +188,23 @@ 
 		$data     = $this->get_data();
 		$cols     = $this->get_csv_cols();
 
-		if( $data ) {
+		if ($data) {
 
 			// Output each row
-			foreach ( $data as $row ) {
+			foreach ($data as $row) {
 				$i = 1;
-				foreach ( $row as $col_id => $column ) {
+				foreach ($row as $col_id => $column) {
 					// Make sure the column is valid
-					if ( array_key_exists( $col_id, $cols ) ) {
-						$row_data .= '"' . addslashes( preg_replace( "/\"/","'", $column ) ) . '"';
-						$row_data .= $i == count( $cols ) ? '' : ',';
+					if (array_key_exists($col_id, $cols)) {
+						$row_data .= '"'.addslashes(preg_replace("/\"/", "'", $column)).'"';
+						$row_data .= $i == count($cols) ? '' : ',';
 						$i++;
 					}
 				}
 				$row_data .= "\r\n";
 			}
 
-			$this->stash_step_data( $row_data );
+			$this->stash_step_data($row_data);
 
 			return $row_data;
 		}
@@ -232,18 +232,18 @@ 
 
 		$file = '';
 
-		if ( @file_exists( $this->file ) ) {
+		if (@file_exists($this->file)) {
 
-			if ( ! is_writeable( $this->file ) ) {
+			if ( ! is_writeable($this->file)) {
 				$this->is_writable = false;
 			}
 
-			$file = @file_get_contents( $this->file );
+			$file = @file_get_contents($this->file);
 
 		} else {
 
-			@file_put_contents( $this->file, '' );
-			@chmod( $this->file, 0664 );
+			@file_put_contents($this->file, '');
+			@chmod($this->file, 0664);
 
 		}
 
@@ -257,18 +257,18 @@ 
 	 * @param $data string The data to add to the file
 	 * @return void
 	 */
-	protected function stash_step_data( $data = '' ) {
+	protected function stash_step_data($data = '') {
 
 		$file = $this->get_file();
 		$file .= $data;
-		@file_put_contents( $this->file, $file );
+		@file_put_contents($this->file, $file);
 
 		// If we have no rows after this step, mark it as an empty export
-		$file_rows    = file( $this->file, FILE_SKIP_EMPTY_LINES);
+		$file_rows    = file($this->file, FILE_SKIP_EMPTY_LINES);
 		$default_cols = $this->get_csv_cols();
-		$default_cols = empty( $default_cols ) ? 0 : 1;
+		$default_cols = empty($default_cols) ? 0 : 1;
 
-		$this->is_empty = count( $file_rows ) == $default_cols ? true : false;
+		$this->is_empty = count($file_rows) == $default_cols ? true : false;
 
 	}
 
@@ -286,7 +286,7 @@ 
 
 		$file = $this->get_file();
 
-		@unlink( $this->file );
+		@unlink($this->file);
 
 		echo $file;
 
@@ -295,7 +295,7 @@ 
 		 *
 		 * @since 1.8
 		 */
-		do_action( 'give_file_export_complete', $_REQUEST );
+		do_action('give_file_export_complete', $_REQUEST);
 
 		give_die();
 	}
@@ -306,7 +306,7 @@ 
 	 * @since 1.5
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {}
+	public function set_properties($request) {}
 
 	/**
 	 * Allow for prefetching of data for the remainder of the exporter