WordImpress / Give

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

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

Spacing +75 added lines, -75 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -62,26 +62,26 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 
 		// Set data from form submission
-		if ( isset( $_POST['form'] ) ) {
-			parse_str( $_POST['form'], $this->data );
+		if (isset($_POST['form'])) {
+			parse_str($_POST['form'], $this->data);
 		}
 
 		$this->form = $this->data['forms'];
 
 		// Setup donor ids cache.
-		if( ! empty( $this->form ) ) {
+		if ( ! empty($this->form)) {
 			// Cache donor ids to output unique list of donor.
-			$this->query_id = give_clean( $_REQUEST['give_export_option']['query_id'] );
-			if( ! ( $this->donor_ids = get_transient( $this->query_id ) ) ) {
+			$this->query_id = give_clean($_REQUEST['give_export_option']['query_id']);
+			if ( ! ($this->donor_ids = get_transient($this->query_id))) {
 				$this->donor_ids = array();
-				set_transient( $this->query_id, $this->donor_ids, HOUR_IN_SECONDS );
+				set_transient($this->query_id, $this->donor_ids, HOUR_IN_SECONDS);
 			}
 		}
 
-		$this->price_id = ! empty( $request['give_price_option'] ) && 0 !== $request['give_price_option'] ? absint( $request['give_price_option'] ) : null;
+		$this->price_id = ! empty($request['give_price_option']) && 0 !== $request['give_price_option'] ? absint($request['give_price_option']) : null;
 
 	}
 
@@ -96,14 +96,14 @@ 
 
 		$cols = array();
 
-		$columns = isset( $this->data['give_export_option'] ) ? $this->data['give_export_option'] : array();
+		$columns = isset($this->data['give_export_option']) ? $this->data['give_export_option'] : array();
 
 		// We need columns.
-		if ( empty( $columns ) ) {
+		if (empty($columns)) {
 			return false;
 		}
 
-		$cols = $this->get_cols( $columns );
+		$cols = $this->get_cols($columns);
 
 		return $cols;
 	}
@@ -115,38 +115,38 @@ 
 	 *
 	 * @return array
 	 */
-	private function get_cols( $columns ) {
+	private function get_cols($columns) {
 
 		$cols = array();
 
-		foreach ( $columns as $key => $value ) {
+		foreach ($columns as $key => $value) {
 
-			switch ( $key ) {
+			switch ($key) {
 				case 'full_name' :
-					$cols['full_name'] = esc_html__( 'Full Name', 'give' );
+					$cols['full_name'] = esc_html__('Full Name', 'give');
 					break;
 				case 'email' :
-					$cols['email'] = esc_html__( 'Email Address', 'give' );
+					$cols['email'] = esc_html__('Email Address', 'give');
 					break;
 				case 'address' :
-					$cols['address_line1']   = esc_html__( 'Address', 'give' );
-					$cols['address_line2']   = esc_html__( 'Address 2', 'give' );
-					$cols['address_city']    = esc_html__( 'City', 'give' );
-					$cols['address_state']   = esc_html__( 'State', 'give' );
-					$cols['address_zip']     = esc_html__( 'Zip', 'give' );
-					$cols['address_country'] = esc_html__( 'Country', 'give' );
+					$cols['address_line1']   = esc_html__('Address', 'give');
+					$cols['address_line2']   = esc_html__('Address 2', 'give');
+					$cols['address_city']    = esc_html__('City', 'give');
+					$cols['address_state']   = esc_html__('State', 'give');
+					$cols['address_zip']     = esc_html__('Zip', 'give');
+					$cols['address_country'] = esc_html__('Country', 'give');
 					break;
 				case 'userid' :
-					$cols['userid'] = esc_html__( 'User ID', 'give' );
+					$cols['userid'] = esc_html__('User ID', 'give');
 					break;
 				case 'date_first_donated' :
-					$cols['date_first_donated'] = esc_html__( 'First Donation Date', 'give' );
+					$cols['date_first_donated'] = esc_html__('First Donation Date', 'give');
 					break;
 				case 'donations' :
-					$cols['donations'] = esc_html__( 'Number of Donations', 'give' );
+					$cols['donations'] = esc_html__('Number of Donations', 'give');
 					break;
 				case 'donation_sum' :
-					$cols['donation_sum'] = esc_html__( 'Sum of Donations', 'give' );
+					$cols['donation_sum'] = esc_html__('Sum of Donations', 'give');
 					break;
 			}
 		}
@@ -169,20 +169,20 @@ 
 
 		$i = 0;
 
-		if ( ! empty( $this->form ) ) {
+		if ( ! empty($this->form)) {
 
 			// Export donors of a specific product
 			global $give_logs;
 
 			$args = array(
-				'post_parent'    => absint( $this->form ),
+				'post_parent'    => absint($this->form),
 				'log_type'       => 'sale',
 				'posts_per_page' => 30,
 				'paged'          => $this->step,
 			);
 
 			// Check for price option
-			if ( null !== $this->price_id ) {
+			if (null !== $this->price_id) {
 				$args['meta_query'] = array(
 					array(
 						'key'   => '_give_log_price_id',
@@ -191,44 +191,44 @@ 
 				);
 			}
 
-			$logs = $give_logs->get_connected_logs( $args );
+			$logs = $give_logs->get_connected_logs($args);
 
-			if ( $logs ) {
-				foreach ( $logs as $log ) {
-					$payment_id = get_post_meta( $log->ID, '_give_log_payment_id', true );
-					$payment    = new Give_Payment( $payment_id );
+			if ($logs) {
+				foreach ($logs as $log) {
+					$payment_id = get_post_meta($log->ID, '_give_log_payment_id', true);
+					$payment    = new Give_Payment($payment_id);
 
 					// Continue if donor already included.
-					if( in_array( $payment->customer_id , $this->donor_ids ) ) {
+					if (in_array($payment->customer_id, $this->donor_ids)) {
 						continue;
 					}
 
 					$this->donor_ids[] = $payment->customer_id;
 
-					$donor      = Give()->customers->get_customer_by( 'id', $payment->customer_id );
-					$data[]     = $this->set_donor_data( $i, $data, $donor );
-					$i ++;
+					$donor      = Give()->customers->get_customer_by('id', $payment->customer_id);
+					$data[]     = $this->set_donor_data($i, $data, $donor);
+					$i++;
 				}
 
 				// Cache donor ids only if admin export donor for specific form.
-				set_transient( $this->query_id, array_unique( $this->donor_ids ), HOUR_IN_SECONDS );
+				set_transient($this->query_id, array_unique($this->donor_ids), HOUR_IN_SECONDS);
 			}
 		} else {
 
 			// Export all customers
-			$offset = 30 * ( $this->step - 1 );
-			$donors = Give()->customers->get_customers( array( 'number' => 30, 'offset' => $offset ) );
+			$offset = 30 * ($this->step - 1);
+			$donors = Give()->customers->get_customers(array('number' => 30, 'offset' => $offset));
 
-			foreach ( $donors as $donor ) {
+			foreach ($donors as $donor) {
 
-				$data[] = $this->set_donor_data( $i, $data, $donor );
-				$i ++;
+				$data[] = $this->set_donor_data($i, $data, $donor);
+				$i++;
 			}
 		}
 
 
-		$data = apply_filters( 'give_export_get_data', $data );
-		$data = apply_filters( "give_export_get_data_{$this->export_type}", $data );
+		$data = apply_filters('give_export_get_data', $data);
+		$data = apply_filters("give_export_get_data_{$this->export_type}", $data);
 
 		return $data;
 	}
@@ -244,18 +244,18 @@ 
 		$percentage = 0;
 
 		// We can't count the number when getting them for a specific form
-		if ( empty( $this->form ) ) {
+		if (empty($this->form)) {
 
 			$total = Give()->customers->count();
 
-			if ( $total > 0 ) {
+			if ($total > 0) {
 
-				$percentage = ( ( 30 * $this->step ) / $total ) * 100;
+				$percentage = ((30 * $this->step) / $total) * 100;
 
 			}
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -271,46 +271,46 @@ 
 	 *
 	 * @return mixed
 	 */
-	private function set_donor_data( $i, $data, $donor ) {
+	private function set_donor_data($i, $data, $donor) {
 
 		$columns = $this->csv_cols();
 
 		// Set address variable
 		$address = '';
-		if ( isset( $donor->user_id ) && $donor->user_id > 0 ) {
-			$address = give_get_donor_address( $donor->user_id );
+		if (isset($donor->user_id) && $donor->user_id > 0) {
+			$address = give_get_donor_address($donor->user_id);
 		}
 
 		// Set columns
-		if ( ! empty( $columns['full_name'] ) ) {
-			$data[ $i ]['full_name'] = $donor->name;
+		if ( ! empty($columns['full_name'])) {
+			$data[$i]['full_name'] = $donor->name;
 		}
-		if ( ! empty( $columns['email'] ) ) {
-			$data[ $i ]['email'] = $donor->email;
+		if ( ! empty($columns['email'])) {
+			$data[$i]['email'] = $donor->email;
 		}
-		if ( ! empty( $columns['address_line1'] ) ) {
-
-			$data[ $i ]['address_line1']   = isset( $address['line1'] ) ? $address['line1'] : '';
-			$data[ $i ]['address_line2']   = isset( $address['line2'] ) ? $address['line2'] : '';
-			$data[ $i ]['address_city']    = isset( $address['city'] ) ? $address['city'] : '';
-			$data[ $i ]['address_state']   = isset( $address['state'] ) ? $address['state'] : '';
-			$data[ $i ]['address_zip']     = isset( $address['zip'] ) ? $address['zip'] : '';
-			$data[ $i ]['address_country'] = isset( $address['country'] ) ? $address['country'] : '';
+		if ( ! empty($columns['address_line1'])) {
+
+			$data[$i]['address_line1']   = isset($address['line1']) ? $address['line1'] : '';
+			$data[$i]['address_line2']   = isset($address['line2']) ? $address['line2'] : '';
+			$data[$i]['address_city']    = isset($address['city']) ? $address['city'] : '';
+			$data[$i]['address_state']   = isset($address['state']) ? $address['state'] : '';
+			$data[$i]['address_zip']     = isset($address['zip']) ? $address['zip'] : '';
+			$data[$i]['address_country'] = isset($address['country']) ? $address['country'] : '';
 		}
-		if ( ! empty( $columns['userid'] ) ) {
-			$data[ $i ]['userid'] = ! empty( $donor->user_id ) ? $donor->user_id : '';
+		if ( ! empty($columns['userid'])) {
+			$data[$i]['userid'] = ! empty($donor->user_id) ? $donor->user_id : '';
 		}
-		if ( ! empty( $columns['date_first_donated'] ) ) {
-			$data[ $i ]['date_first_donated'] = date_i18n( give_date_format(), strtotime( $donor->date_created ) );
+		if ( ! empty($columns['date_first_donated'])) {
+			$data[$i]['date_first_donated'] = date_i18n(give_date_format(), strtotime($donor->date_created));
 		}
-		if ( ! empty( $columns['donations'] ) ) {
-			$data[ $i ]['donations'] = $donor->purchase_count;
+		if ( ! empty($columns['donations'])) {
+			$data[$i]['donations'] = $donor->purchase_count;
 		}
-		if ( ! empty( $columns['donation_sum'] ) ) {
-			$data[ $i ]['donation_sum'] = give_format_amount( $donor->purchase_value );
+		if ( ! empty($columns['donation_sum'])) {
+			$data[$i]['donation_sum'] = give_format_amount($donor->purchase_value);
 		}
 
-		return $data[ $i ];
+		return $data[$i];
 
 	}
 

includes/admin/reporting/export/export-actions.php

Spacing +28 added lines, -28 removed lines

@@ -10,7 +10,7 @@ 
  * @license     https://opensource.org/licenses/gpl-license GNU Public License
  */
 
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -22,11 +22,11 @@ 
  */
 function give_process_batch_export_form() {
 
-	if ( ! wp_verify_nonce( $_REQUEST['nonce'], 'give-batch-export' ) ) {
-		wp_die( esc_html__( 'Nonce verification failed.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+	if ( ! wp_verify_nonce($_REQUEST['nonce'], 'give-batch-export')) {
+		wp_die(esc_html__('Nonce verification failed.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 	}
 
-	require_once GIVE_PLUGIN_DIR . 'includes/admin/reporting/export/class-batch-export.php';
+	require_once GIVE_PLUGIN_DIR.'includes/admin/reporting/export/class-batch-export.php';
 
 	/**
 	 * Fires before batch export.
@@ -35,14 +35,14 @@ 
 	 *
 	 * @param string $class Export class.
 	 */
-	do_action( 'give_batch_export_class_include', $_REQUEST['class'] );
+	do_action('give_batch_export_class_include', $_REQUEST['class']);
 
 	$export = new $_REQUEST['class'];
 	$export->export();
 
 }
 
-add_action( 'give_form_batch_export', 'give_process_batch_export_form' );
+add_action('give_form_batch_export', 'give_process_batch_export_form');
 
 /**
  * Exports earnings for a specified time period
@@ -53,14 +53,14 @@ 
  * @return void
  */
 function give_export_earnings() {
-	require_once GIVE_PLUGIN_DIR . 'includes/admin/reporting/class-export-earnings.php';
+	require_once GIVE_PLUGIN_DIR.'includes/admin/reporting/class-export-earnings.php';
 
 	$earnings_export = new Give_Earnings_Export();
 
 	$earnings_export->export();
 }
 
-add_action( 'give_earnings_export', 'give_export_earnings' );
+add_action('give_earnings_export', 'give_export_earnings');
 
 
 /**
@@ -73,14 +73,14 @@ 
  * @return void
  */
 function give_export_all_customers() {
-	require_once GIVE_PLUGIN_DIR . 'includes/admin/reporting/class-export-customers.php';
+	require_once GIVE_PLUGIN_DIR.'includes/admin/reporting/class-export-customers.php';
 
 	$customer_export = new Give_Donors_Export();
 
 	$customer_export->export();
 }
 
-add_action( 'give_email_export', 'give_export_all_customers' );
+add_action('give_email_export', 'give_export_all_customers');
 
 /**
  * Add a hook allowing extensions to register a hook on the batch export process
@@ -89,7 +89,7 @@ 
  * @return void
  */
 function give_register_batch_exporters() {
-	if ( is_admin() ) {
+	if (is_admin()) {
 		/**
 		 * Fires in the admin, while plugins loaded.
 		 *
@@ -99,21 +99,21 @@ 
 		 *
 		 * @param string $class Export class.
 		 */
-		do_action( 'give_register_batch_exporter' );
+		do_action('give_register_batch_exporter');
 	}
 }
 
-add_action( 'plugins_loaded', 'give_register_batch_exporters' );
+add_action('plugins_loaded', 'give_register_batch_exporters');
 
 /**
  * Register the payments batch exporter
  * @since  1.5
  */
 function give_register_payments_batch_export() {
-	add_action( 'give_batch_export_class_include', 'give_include_payments_batch_processor', 10, 1 );
+	add_action('give_batch_export_class_include', 'give_include_payments_batch_processor', 10, 1);
 }
 
-add_action( 'give_register_batch_exporter', 'give_register_payments_batch_export', 10 );
+add_action('give_register_batch_exporter', 'give_register_payments_batch_export', 10);
 
 /**
  * Loads the payments batch process if needed
@@ -124,10 +124,10 @@ 
  *
  * @return void
  */
-function give_include_payments_batch_processor( $class ) {
+function give_include_payments_batch_processor($class) {
 
-	if ( 'Give_Batch_Payments_Export' === $class ) {
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/reporting/export/class-batch-export-payments.php';
+	if ('Give_Batch_Payments_Export' === $class) {
+		require_once GIVE_PLUGIN_DIR.'includes/admin/reporting/export/class-batch-export-payments.php';
 	}
 
 }
@@ -137,10 +137,10 @@ 
  * @since  1.5.2
  */
 function give_register_customers_batch_export() {
-	add_action( 'give_batch_export_class_include', 'give_include_customers_batch_processor', 10, 1 );
+	add_action('give_batch_export_class_include', 'give_include_customers_batch_processor', 10, 1);
 }
 
-add_action( 'give_register_batch_exporter', 'give_register_customers_batch_export', 10 );
+add_action('give_register_batch_exporter', 'give_register_customers_batch_export', 10);
 
 /**
  * Loads the customers batch process if needed
@@ -151,10 +151,10 @@ 
  *
  * @return void
  */
-function give_include_customers_batch_processor( $class ) {
+function give_include_customers_batch_processor($class) {
 
-	if ( 'Give_Batch_Customers_Export' === $class ) {
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/reporting/export/class-batch-export-customers.php';
+	if ('Give_Batch_Customers_Export' === $class) {
+		require_once GIVE_PLUGIN_DIR.'includes/admin/reporting/export/class-batch-export-customers.php';
 	}
 
 }
@@ -165,10 +165,10 @@ 
  * @since  1.5
  */
 function give_register_forms_batch_export() {
-	add_action( 'give_batch_export_class_include', 'give_include_forms_batch_processor', 10, 1 );
+	add_action('give_batch_export_class_include', 'give_include_forms_batch_processor', 10, 1);
 }
 
-add_action( 'give_register_batch_exporter', 'give_register_forms_batch_export', 10 );
+add_action('give_register_batch_exporter', 'give_register_forms_batch_export', 10);
 
 /**
  * Loads the file downloads batch process if needed
@@ -179,10 +179,10 @@ 
  *
  * @return void
  */
-function give_include_forms_batch_processor( $class ) {
+function give_include_forms_batch_processor($class) {
 
-	if ( 'Give_Batch_Forms_Export' === $class ) {
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/reporting/export/class-batch-export-forms.php';
+	if ('Give_Batch_Forms_Export' === $class) {
+		require_once GIVE_PLUGIN_DIR.'includes/admin/reporting/export/class-batch-export-forms.php';
 	}
 
 }
\ No newline at end of file

includes/admin/reporting/class-settings-earnings.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_Earnings' ) ) :
+if ( ! class_exists('Give_Settings_Earnings')) :
 
 	/**
 	 * Give_Settings_Earnings.
@@ -43,16 +43,16 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'earnings';
-			$this->label = esc_html__( 'Income', 'give' );
+			$this->label = esc_html__('Income', '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_earnings', array( $this, 'render_report_earnings_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_earnings', array($this, 'render_report_earnings_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_tools_earnings',
@@ -95,7 +95,7 @@ 
 					),
 					array(
 						'id'   => 'earnings',
-						'name' => esc_html__( 'Income', 'give' ),
+						'name' => esc_html__('Income', 'give'),
 						'type' => 'report_earnings',
 					),
 					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_earnings_field( $field, $option_value ) {
-			do_action( 'give_reports_view_earnings' );
+		public function render_report_earnings_field($field, $option_value) {
+			do_action('give_reports_view_earnings');
 		}
 	}
 

includes/admin/views/html-admin-settings.php

Spacing +17 added lines, -17 removed lines

@@ -2,13 +2,13 @@ 
 /**
  * Admin View: Settings
  */
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 
 // Bailout: Do not output anything if setting tab is not defined.
-if( ! empty( $tabs ) && array_key_exists( give_get_current_setting_tab(), $tabs ) ) :
+if ( ! empty($tabs) && array_key_exists(give_get_current_setting_tab(), $tabs)) :
 	/**
 	 * Filter the form action.
 	 *
@@ -18,7 +18,7 @@ 
 	 *
 	 * @since 1.8
 	 */
-	$form_method    = apply_filters( self::$setting_filter_prefix . '_form_method_tab_' . $current_tab, 'post' );
+	$form_method = apply_filters(self::$setting_filter_prefix.'_form_method_tab_'.$current_tab, 'post');
 
 	/**
 	 * Filter the main form tab.
@@ -30,15 +30,15 @@ 
 	 *
 	 * @since 1.8
 	 */
-	$form_open_tag  = apply_filters( self::$setting_filter_prefix . '_open_form', '<form method="' . $form_method  . '" id="give-mainform" action="" enctype="multipart/form-data">' );
-	$form_close_tag = apply_filters( self::$setting_filter_prefix . '_close_form', '</form>' );
+	$form_open_tag  = apply_filters(self::$setting_filter_prefix.'_open_form', '<form method="'.$form_method.'" id="give-mainform" action="" enctype="multipart/form-data">');
+	$form_close_tag = apply_filters(self::$setting_filter_prefix.'_close_form', '</form>');
 	?>
-	<div class="wrap give-settings-page <?php echo self::$setting_filter_prefix . '-setting-page'; ?>">
+	<div class="wrap give-settings-page <?php echo self::$setting_filter_prefix.'-setting-page'; ?>">
 		<?php echo $form_open_tag; ?>
 			<h2 class="nav-tab-wrapper give-nav-tab-wrapper">
 				<?php
-				foreach ( $tabs as $name => $label ) {
-					echo '<a href="' . admin_url( "edit.php?post_type=give_forms&page=" . self::$setting_filter_prefix . "&tab={$name}" ) . '" class="nav-tab ' . ( $current_tab == $name ? 'nav-tab-active' : '' ) . '">' . $label . '</a>';
+				foreach ($tabs as $name => $label) {
+					echo '<a href="'.admin_url("edit.php?post_type=give_forms&page=".self::$setting_filter_prefix."&tab={$name}").'" class="nav-tab '.($current_tab == $name ? 'nav-tab-active' : '').'">'.$label.'</a>';
 				}
 
 				/**
@@ -50,14 +50,14 @@ 
 				 *
 				 * @since 1.8
 				 */
-				do_action( self::$setting_filter_prefix . '_tabs' );
+				do_action(self::$setting_filter_prefix.'_tabs');
 				?>
 			</h2>
 			<div class="give-sub-nav-tab-wrapper">
-				<a href="#" id="give-show-sub-nav" class="nav-tab give-not-tab" title="<?php _e( 'View remaining setting tabs', 'give' ); ?>"><span class="dashicons dashicons-arrow-down-alt2"></span></span></a>
+				<a href="#" id="give-show-sub-nav" class="nav-tab give-not-tab" title="<?php _e('View remaining setting tabs', 'give'); ?>"><span class="dashicons dashicons-arrow-down-alt2"></span></span></a>
 				<nav class="give-sub-nav-tab give-hidden"></nav>
 			</div>
-			<h1 class="screen-reader-text"><?php echo esc_html( $tabs[ $current_tab ] ); ?></h1>
+			<h1 class="screen-reader-text"><?php echo esc_html($tabs[$current_tab]); ?></h1>
 			<?php
 
 			/**
@@ -69,7 +69,7 @@ 
 			 *
 			 * @since 1.8
 			 */
-			do_action( self::$setting_filter_prefix . "_sections_{$current_tab}_page" );
+			do_action(self::$setting_filter_prefix."_sections_{$current_tab}_page");
 
 			
 			// Show messages.
@@ -85,15 +85,15 @@ 
 			 *
 			 * @since 1.8
 			 */
-			do_action( self::$setting_filter_prefix . "_settings_{$current_tab}_page" );
+			do_action(self::$setting_filter_prefix."_settings_{$current_tab}_page");
 
-			if ( empty( $GLOBALS['give_hide_save_button'] ) ) : ?>
+			if (empty($GLOBALS['give_hide_save_button'])) : ?>
 				<div class="give-submit-wrap">
-					<?php wp_nonce_field( 'give-save-settings', '_give-save-settings' ); ?>
-					<input name="save" class="button-primary give-save-button" type="submit" value="<?php esc_attr_e( 'Save changes', 'give' ); ?>" />
+					<?php wp_nonce_field('give-save-settings', '_give-save-settings'); ?>
+					<input name="save" class="button-primary give-save-button" type="submit" value="<?php esc_attr_e('Save changes', 'give'); ?>" />
 				</div>
 			<?php endif; ?>
 		<?php echo $form_close_tag; ?>
 	</div>
-	<?php else : echo '<div class="error"><p>' . __( 'Oops, this settings page does not exist.', 'give' ) . '</p></div>'; ?>
+	<?php else : echo '<div class="error"><p>'.__('Oops, this settings page does not exist.', 'give').'</p></div>'; ?>
 <?php endif; ?>
\ No newline at end of file

Braces +4 added lines, -1 removed lines

@@ -95,5 +95,8 @@
 			<?php endif; ?>
 		<?php echo $form_close_tag; ?>
 	</div>
-	<?php else : echo '<div class="error"><p>' . __( 'Oops, this settings page does not exist.', 'give' ) . '</p></div>'; ?>
+	<?php else {
+	: echo '<div class="error"><p>' . __( 'Oops, this settings page does not exist.', 'give' ) . '</p></div>';
+}
+?>
 <?php endif; ?>
\ No newline at end of file

includes/admin/admin-pages.php

Spacing +99 added lines, -99 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -33,17 +33,17 @@ 
 	global $give_settings_page, $give_payments_page, $give_reports_page, $give_add_ons_page, $give_upgrades_screen, $give_donors_page, $give_tools_page;
 
 	//Payments
-	$give_payment       = get_post_type_object( 'give_payment' );
-	$give_payments_page = add_submenu_page( 'edit.php?post_type=give_forms', $give_payment->labels->name, $give_payment->labels->menu_name, 'edit_give_payments', 'give-payment-history', 'give_payment_history_page' );
+	$give_payment       = get_post_type_object('give_payment');
+	$give_payments_page = add_submenu_page('edit.php?post_type=give_forms', $give_payment->labels->name, $give_payment->labels->menu_name, 'edit_give_payments', 'give-payment-history', 'give_payment_history_page');
 
 	//Donors
-	$give_donors_page = add_submenu_page( 'edit.php?post_type=give_forms', esc_html__( 'Donors', 'give' ), esc_html__( 'Donors', 'give' ), 'view_give_reports', 'give-donors', 'give_customers_page' );
+	$give_donors_page = add_submenu_page('edit.php?post_type=give_forms', esc_html__('Donors', 'give'), esc_html__('Donors', 'give'), 'view_give_reports', 'give-donors', 'give_customers_page');
 
 	//Reports`
 	$give_reports_page = add_submenu_page(
 		'edit.php?post_type=give_forms',
-		esc_html__( 'Donation Reports', 'give' ),
-		esc_html__( 'Reports', 'give' ),
+		esc_html__('Donation Reports', 'give'),
+		esc_html__('Reports', 'give'),
 		'view_give_reports',
 		'give-reports',
 		array(
@@ -55,8 +55,8 @@ 
 	//Settings
 	$give_settings_page = add_submenu_page(
 		'edit.php?post_type=give_forms',
-		esc_html__( 'Give Settings', 'give' ),
-		esc_html__( 'Settings', 'give' ),
+		esc_html__('Give Settings', 'give'),
+		esc_html__('Settings', 'give'),
 		'manage_give_settings',
 		'give-settings',
 		array(
@@ -66,21 +66,21 @@ 
 	);
 
 	//Tools.
-	$give_tools_page = add_submenu_page( 'edit.php?post_type=give_forms', esc_html__( 'Give Tools', 'give' ), esc_html__( 'Tools', 'give' ), 'manage_give_settings', 'give-tools', array(
+	$give_tools_page = add_submenu_page('edit.php?post_type=give_forms', esc_html__('Give Tools', 'give'), esc_html__('Tools', 'give'), 'manage_give_settings', 'give-tools', array(
 		Give()->give_settings,
 		'output'
-	) );
+	));
 
 	//Add-ons
-	$give_add_ons_page = add_submenu_page( 'edit.php?post_type=give_forms', esc_html__( 'Give Add-ons', 'give' ), esc_html__( 'Add-ons', 'give' ), 'install_plugins', 'give-addons', 'give_add_ons_page' );
+	$give_add_ons_page = add_submenu_page('edit.php?post_type=give_forms', esc_html__('Give Add-ons', 'give'), esc_html__('Add-ons', 'give'), 'install_plugins', 'give-addons', 'give_add_ons_page');
 
 	//Upgrades
-	$give_upgrades_screen = add_submenu_page( null, esc_html__( 'Give Upgrades', 'give' ), esc_html__( 'Give Upgrades', 'give' ), 'manage_give_settings', 'give-upgrades', 'give_upgrades_screen' );
+	$give_upgrades_screen = add_submenu_page(null, esc_html__('Give Upgrades', 'give'), esc_html__('Give Upgrades', 'give'), 'manage_give_settings', 'give-upgrades', 'give_upgrades_screen');
 
 
 }
 
-add_action( 'admin_menu', 'give_add_options_links', 10 );
+add_action('admin_menu', 'give_add_options_links', 10);
 
 /**
  *  Determines whether the current admin page is a Give admin page.
@@ -95,224 +95,224 @@ 
  *
  * @return bool True if Give admin page.
  */
-function give_is_admin_page( $passed_page = '', $passed_view = '' ) {
+function give_is_admin_page($passed_page = '', $passed_view = '') {
 
 	global $pagenow, $typenow;
 
 	$found     = false;
-	$post_type = isset( $_GET['post_type'] ) ? strtolower( $_GET['post_type'] ) : false;
-	$action    = isset( $_GET['action'] ) ? strtolower( $_GET['action'] ) : false;
-	$taxonomy  = isset( $_GET['taxonomy'] ) ? strtolower( $_GET['taxonomy'] ) : false;
-	$page      = isset( $_GET['page'] ) ? strtolower( $_GET['page'] ) : false;
-	$view      = isset( $_GET['view'] ) ? strtolower( $_GET['view'] ) : false;
-	$tab       = isset( $_GET['tab'] ) ? strtolower( $_GET['tab'] ) : false;
-
-	switch ( $passed_page ) {
+	$post_type = isset($_GET['post_type']) ? strtolower($_GET['post_type']) : false;
+	$action    = isset($_GET['action']) ? strtolower($_GET['action']) : false;
+	$taxonomy  = isset($_GET['taxonomy']) ? strtolower($_GET['taxonomy']) : false;
+	$page      = isset($_GET['page']) ? strtolower($_GET['page']) : false;
+	$view      = isset($_GET['view']) ? strtolower($_GET['view']) : false;
+	$tab       = isset($_GET['tab']) ? strtolower($_GET['tab']) : false;
+
+	switch ($passed_page) {
 		case 'give_forms':
-			switch ( $passed_view ) {
+			switch ($passed_view) {
 				case 'list-table':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php') {
 						$found = true;
 					}
 					break;
 				case 'edit':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'post.php' ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'post.php') {
 						$found = true;
 					}
 					break;
 				case 'new':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'post-new.php' ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'post-new.php') {
 						$found = true;
 					}
 					break;
 				default:
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) || 'give_forms' === $post_type || ( 'post-new.php' == $pagenow && 'give_forms' === $post_type ) ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) || 'give_forms' === $post_type || ('post-new.php' == $pagenow && 'give_forms' === $post_type)) {
 						$found = true;
 					}
 					break;
 			}
 			break;
 		case 'categories':
-			switch ( $passed_view ) {
+			switch ($passed_view) {
 				case 'list-table':
 				case 'new':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit-tags.php' && 'edit' !== $action && 'give_forms_category' === $taxonomy ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit-tags.php' && 'edit' !== $action && 'give_forms_category' === $taxonomy) {
 						$found = true;
 					}
 					break;
 				case 'edit':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit-tags.php' && 'edit' === $action && 'give_forms_category' === $taxonomy ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit-tags.php' && 'edit' === $action && 'give_forms_category' === $taxonomy) {
 						$found = true;
 					}
 					break;
 				default:
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit-tags.php' && 'give_forms_category' === $taxonomy ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit-tags.php' && 'give_forms_category' === $taxonomy) {
 						$found = true;
 					}
 					break;
 			}
 			break;
 		case 'tags':
-			switch ( $passed_view ) {
+			switch ($passed_view) {
 				case 'list-table':
 				case 'new':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit-tags.php' && 'edit' !== $action && 'give_forms_tag' === $taxonomy ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit-tags.php' && 'edit' !== $action && 'give_forms_tag' === $taxonomy) {
 						$found = true;
 					}
 					break;
 				case 'edit':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit-tags.php' && 'edit' === $action && 'give_forms_tag' === $taxonomy ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit-tags.php' && 'edit' === $action && 'give_forms_tag' === $taxonomy) {
 						$found = true;
 					}
 					break;
 				default:
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit-tags.php' && 'give_forms_tag' === $taxonomy ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit-tags.php' && 'give_forms_tag' === $taxonomy) {
 						$found = true;
 					}
 					break;
 			}
 			break;
 		case 'payments':
-			switch ( $passed_view ) {
+			switch ($passed_view) {
 				case 'list-table':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-payment-history' === $page && false === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-payment-history' === $page && false === $view) {
 						$found = true;
 					}
 					break;
 				case 'edit':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-payment-history' === $page && 'view-order-details' === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-payment-history' === $page && 'view-order-details' === $view) {
 						$found = true;
 					}
 					break;
 				default:
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-payment-history' === $page ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-payment-history' === $page) {
 						$found = true;
 					}
 					break;
 			}
 			break;
 		case 'reports':
-			switch ( $passed_view ) {
+			switch ($passed_view) {
 				// If you want to do something like enqueue a script on a particular report's duration, look at $_GET[ 'range' ]
 				case 'earnings':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-reports' === $page && ( 'earnings' === $view || '-1' === $view || false === $view ) ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-reports' === $page && ('earnings' === $view || '-1' === $view || false === $view)) {
 						$found = true;
 					}
 					break;
 				case 'donors':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-reports' === $page && 'customers' === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-reports' === $page && 'customers' === $view) {
 						$found = true;
 					}
 					break;
 				case 'gateways':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-reports' === $page && 'gateways' === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-reports' === $page && 'gateways' === $view) {
 						$found = true;
 					}
 					break;
 				case 'export':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-reports' === $page && 'export' === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-reports' === $page && 'export' === $view) {
 						$found = true;
 					}
 					break;
 				case 'logs':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-reports' === $page && 'logs' === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-reports' === $page && 'logs' === $view) {
 						$found = true;
 					}
 					break;
 				default:
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-reports' === $page ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-reports' === $page) {
 						$found = true;
 					}
 					break;
 			}
 			break;
 		case 'settings':
-			switch ( $passed_view ) {
+			switch ($passed_view) {
 				case 'general':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && ( 'general' === $tab || false === $tab ) ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && ('general' === $tab || false === $tab)) {
 						$found = true;
 					}
 					break;
 				case 'gateways':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && 'gateways' === $tab ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && 'gateways' === $tab) {
 						$found = true;
 					}
 					break;
 				case 'emails':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && 'emails' === $tab ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && 'emails' === $tab) {
 						$found = true;
 					}
 					break;
 				case 'display':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && 'display' === $tab ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && 'display' === $tab) {
 						$found = true;
 					}
 					break;
 				case 'licenses':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && 'licenses' === $tab ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && 'licenses' === $tab) {
 						$found = true;
 					}
 					break;
 				case 'api':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && 'api' === $tab ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && 'api' === $tab) {
 						$found = true;
 					}
 					break;
 				case 'advanced':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && 'advanced' === $tab ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && 'advanced' === $tab) {
 						$found = true;
 					}
 					break;
 				case 'system_info':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page && 'system_info' === $tab ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page && 'system_info' === $tab) {
 						$found = true;
 					}
 					break;
 				default:
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-settings' === $page ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-settings' === $page) {
 						$found = true;
 					}
 					break;
 			}
 			break;
 		case 'addons':
-			if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-addons' === $page ) {
+			if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-addons' === $page) {
 				$found = true;
 			}
 			break;
 		case 'donors':
-			switch ( $passed_view ) {
+			switch ($passed_view) {
 				case 'list-table':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-donors' === $page && false === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-donors' === $page && false === $view) {
 						$found = true;
 					}
 					break;
 				case 'overview':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-donors' === $page && 'overview' === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-donors' === $page && 'overview' === $view) {
 						$found = true;
 					}
 					break;
 				case 'notes':
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-donors' === $page && 'notes' === $view ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-donors' === $page && 'notes' === $view) {
 						$found = true;
 					}
 					break;
 				default:
-					if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-donors' === $page ) {
+					if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-donors' === $page) {
 						$found = true;
 					}
 					break;
 			}
 			break;
 		case 'reports':
-			if ( ( 'give_forms' == $typenow || 'give_forms' === $post_type ) && $pagenow == 'edit.php' && 'give-reports' === $page ) {
+			if (('give_forms' == $typenow || 'give_forms' === $post_type) && $pagenow == 'edit.php' && 'give-reports' === $page) {
 				$found = true;
 			}
 			break;
 		default:
 			global $give_payments_page, $give_settings_page, $give_reports_page, $give_system_info_page, $give_add_ons_page, $give_settings_export, $give_upgrades_screen, $give_customers_page, $give_tools_page;
 
-			$admin_pages = apply_filters( 'give_admin_pages', array(
+			$admin_pages = apply_filters('give_admin_pages', array(
 				$give_payments_page,
 				$give_settings_page,
 				$give_reports_page,
@@ -323,19 +323,19 @@ 
 				$give_customers_page,
 				$give_tools_page,
 				'widgets.php'
-		) );
-			if ( 'give_forms' == $typenow || 'index.php' == $pagenow || 'post-new.php' == $pagenow || 'post.php' == $pagenow ) {
+		));
+			if ('give_forms' == $typenow || 'index.php' == $pagenow || 'post-new.php' == $pagenow || 'post.php' == $pagenow) {
 				$found = true;
-				if ( 'give-upgrades' === $page ) {
+				if ('give-upgrades' === $page) {
 					$found = false;
 				}
-			} elseif ( in_array( $pagenow, $admin_pages ) ) {
+			} elseif (in_array($pagenow, $admin_pages)) {
 				$found = true;
 			}
 			break;
 	}
 
-	return (bool) apply_filters( 'give_is_admin_page', $found, $page, $view, $passed_page, $passed_view );
+	return (bool) apply_filters('give_is_admin_page', $found, $page, $view, $passed_page, $passed_view);
 
 }
 
@@ -347,37 +347,37 @@ 
  * @param  array $settings
  * @return array
  */
-function give_settings_page_pages( $settings ) {
-	include( 'abstract-admin-settings-page.php' );
-	include( 'settings/class-settings-cmb2-backward-compatibility.php' );
+function give_settings_page_pages($settings) {
+	include('abstract-admin-settings-page.php');
+	include('settings/class-settings-cmb2-backward-compatibility.php');
 
 	$settings = array(
 		// General settings.
-		include( 'settings/class-settings-general.php' ),
+		include('settings/class-settings-general.php'),
 
 		// Payment Gateways Settings.
-		include( 'settings/class-settings-gateways.php' ),
+		include('settings/class-settings-gateways.php'),
 
 		// Display settings.
-		include( 'settings/class-settings-display.php' ),
+		include('settings/class-settings-display.php'),
 
 		// Emails settings.
-		include( 'settings/class-settings-email.php' ),
+		include('settings/class-settings-email.php'),
 
 		// Addons settings.
-		include( 'settings/class-settings-addon.php' ),
+		include('settings/class-settings-addon.php'),
 
 		// License settings.
-		include( 'settings/class-settings-license.php' ),
+		include('settings/class-settings-license.php'),
 
 		// Advanced settinns.
-		include( 'settings/class-settings-advanced.php' )
+		include('settings/class-settings-advanced.php')
 	);
 
 	// Output.
 	return $settings;
 }
-add_filter( 'give-settings_get_settings_pages', 'give_settings_page_pages', 0, 1 );
+add_filter('give-settings_get_settings_pages', 'give_settings_page_pages', 0, 1);
 
 
 /**
@@ -387,30 +387,30 @@ 
  * @param  array $settings
  * @return array
  */
-function give_reports_page_pages( $settings ) {
-	include( 'abstract-admin-settings-page.php' );
+function give_reports_page_pages($settings) {
+	include('abstract-admin-settings-page.php');
 
 	$settings = array(
 		// Earnings.
-		include( 'reporting/class-settings-earnings.php' ),
+		include('reporting/class-settings-earnings.php'),
 
 		// Forms.
-		include( 'reporting/class-settings-forms.php' ),
+		include('reporting/class-settings-forms.php'),
 
 		// Donors.
-		include( 'reporting/class-settings-donors.php' ),
+		include('reporting/class-settings-donors.php'),
 
 		// Gateways.
-		include( 'reporting/class-settings-gateways.php' ),
+		include('reporting/class-settings-gateways.php'),
 
 		// Export.
-		include( 'reporting/class-settings-export.php' ),
+		include('reporting/class-settings-export.php'),
 	);
 
 	// Output.
 	return $settings;
 }
-add_filter( 'give-reports_get_settings_pages', 'give_reports_page_pages', 0, 1 );
+add_filter('give-reports_get_settings_pages', 'give_reports_page_pages', 0, 1);
 
 /**
  * Add setting tab to give-settings page
@@ -419,27 +419,27 @@ 
  * @param  array $settings
  * @return array
  */
-function give_tools_page_pages( $settings ) {
-	include( 'abstract-admin-settings-page.php' );
+function give_tools_page_pages($settings) {
+	include('abstract-admin-settings-page.php');
 
 	$settings = array(
 		// System Info.
-		include( 'tools/class-settings-system-info.php' ),
+		include('tools/class-settings-system-info.php'),
 
 		// Logs.
-		include( 'tools/class-settings-logs.php' ),
+		include('tools/class-settings-logs.php'),
 
 		// API.
-		include( 'tools/class-settings-api.php' ),
+		include('tools/class-settings-api.php'),
 
 		// Data.
-		include( 'tools/class-settings-data.php' ),
+		include('tools/class-settings-data.php'),
 	);
 
 	// Output.
 	return $settings;
 }
-add_filter( 'give-tools_get_settings_pages', 'give_tools_page_pages', 0, 1 );
+add_filter('give-tools_get_settings_pages', 'give_tools_page_pages', 0, 1);
 
 /**
  * Set default tools page tab.
@@ -448,10 +448,10 @@ 
  * @param  string $default_tab Default tab name.
  * @return string
  */
-function give_set_default_tab_form_tools_page( $default_tab ) {
+function give_set_default_tab_form_tools_page($default_tab) {
 	return 'system-info';
 }
-add_filter( 'give_default_setting_tab_give-tools', 'give_set_default_tab_form_tools_page', 10, 1 );
+add_filter('give_default_setting_tab_give-tools', 'give_set_default_tab_form_tools_page', 10, 1);
 
 
 /**
@@ -461,7 +461,7 @@ 
  * @param  string $default_tab Default tab name.
  * @return string
  */
-function give_set_default_tab_form_reports_page( $default_tab ) {
+function give_set_default_tab_form_reports_page($default_tab) {
 	return 'earnings';
 }
-add_filter( 'give_default_setting_tab_give-reports', 'give_set_default_tab_form_reports_page', 10, 1 );
\ No newline at end of file
+add_filter('give_default_setting_tab_give-reports', 'give_set_default_tab_form_reports_page', 10, 1);
\ No newline at end of file

includes/admin/settings/class-settings-advanced.php

Indentation +7 added lines, -7 removed lines

@@ -87,13 +87,13 @@
 								'disabled' => __( 'Head', 'give' ),
 							)
 						),
-                        array(
-                            'name'  => esc_html__( 'Advanced Settings Docs Link', 'give' ),
-                            'id'    => 'advanced_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-advanced' ),
-                            'title' => __( 'Advanced Settings', 'give' ),
-                            'type'  => 'give_docs_link',
-                        ),
+						array(
+							'name'  => esc_html__( 'Advanced Settings Docs Link', 'give' ),
+							'id'    => 'advanced_settings_docs_link',
+							'url'   => esc_url( 'http://docs.givewp.com/settings-advanced' ),
+							'title' => __( 'Advanced Settings', 'give' ),
+							'type'  => 'give_docs_link',
+						),
 						array(
 							'id'   => 'give_title_data_control_2',
 							'type' => 'sectionend'

Spacing +23 added lines, -23 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_Advanced' ) ) :
+if ( ! class_exists('Give_Settings_Advanced')) :
 
 	/**
 	 * Give_Settings_Advanced.
@@ -27,7 +27,7 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'advanced';
-			$this->label = esc_html__( 'Advanced', 'give' );
+			$this->label = esc_html__('Advanced', 'give');
 
 			$this->default_tab = 'advanced-options';
 
@@ -45,7 +45,7 @@ 
 
 			$current_section = give_get_current_setting_section();
 
-			switch ( $current_section ) {
+			switch ($current_section) {
 				case 'advanced-options':
 					$settings = array(
 						array(
@@ -53,45 +53,45 @@ 
 							'type' => 'title'
 						),
 						array(
-							'name'    => esc_html__( 'Remove Data on Uninstall', 'give' ),
-							'desc'    => esc_html__( 'When the plugin is deleted, completely remove all Give data. This includes all Give settings, forms, form meta, donor, donor data, donations. Everything.', 'give' ),
+							'name'    => esc_html__('Remove Data on Uninstall', 'give'),
+							'desc'    => esc_html__('When the plugin is deleted, completely remove all Give data. This includes all Give settings, forms, form meta, donor, donor data, donations. Everything.', 'give'),
 							'id'      => 'uninstall_on_delete',
 							'type'    => 'radio_inline',
 							'default' => 'disabled',
 							'options' => array(
-								'enabled'  => __( 'Yes, Remove all data', 'give' ),
-								'disabled' => __( 'No, keep my Give settings and donation data this time', 'give' ),
+								'enabled'  => __('Yes, Remove all data', 'give'),
+								'disabled' => __('No, keep my Give settings and donation data this time', 'give'),
 							)
 						),
 						array(
 							/* translators: %s: the_content */
-							'name'    => sprintf( __( '%s filter', 'give' ), '<code>the_content</code>' ),
+							'name'    => sprintf(__('%s filter', 'give'), '<code>the_content</code>'),
 							/* translators: 1: https://codex.wordpress.org/Plugin_API/Filter_Reference/the_content 2: the_content */
-							'desc'    => sprintf( __( 'If you are seeing extra social buttons, related posts, or other unwanted elements appearing within your forms then you can disable WordPress\' content filter. <a href="%1$s" target="_blank">Learn more</a> about %2$s filter.', 'give' ), esc_url( 'https://codex.wordpress.org/Plugin_API/Filter_Reference/the_content' ), '<code>the_content</code>' ),
+							'desc'    => sprintf(__('If you are seeing extra social buttons, related posts, or other unwanted elements appearing within your forms then you can disable WordPress\' content filter. <a href="%1$s" target="_blank">Learn more</a> about %2$s filter.', 'give'), esc_url('https://codex.wordpress.org/Plugin_API/Filter_Reference/the_content'), '<code>the_content</code>'),
 							'id'      => 'the_content_filter',
 							'default' => 'enabled',
 							'type'    => 'radio_inline',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							)
 						),
 						array(
-							'name'    => esc_html__( 'Script Loading Location', 'give' ),
-							'desc'    => __( 'This allows you to load your Give scripts either in the <code>&lt;head&gt;</code> or footer of your website.', 'give' ),
+							'name'    => esc_html__('Script Loading Location', 'give'),
+							'desc'    => __('This allows you to load your Give scripts either in the <code>&lt;head&gt;</code> or footer of your website.', 'give'),
 							'id'      => 'scripts_footer',
 							'type'    => 'radio_inline',
 							'default' => 'disabled',
 							'options' => array(
-								'enabled'  => __( 'Footer', 'give' ),
-								'disabled' => __( 'Head', 'give' ),
+								'enabled'  => __('Footer', 'give'),
+								'disabled' => __('Head', 'give'),
 							)
 						),
                         array(
-                            'name'  => esc_html__( 'Advanced Settings Docs Link', 'give' ),
+                            'name'  => esc_html__('Advanced Settings Docs Link', 'give'),
                             'id'    => 'advanced_settings_docs_link',
-                            'url'   => esc_url( 'http://docs.givewp.com/settings-advanced' ),
-                            'title' => __( 'Advanced Settings', 'give' ),
+                            'url'   => esc_url('http://docs.givewp.com/settings-advanced'),
+                            'title' => __('Advanced Settings', 'give'),
                             'type'  => 'give_docs_link',
                         ),
 						array(
@@ -107,7 +107,7 @@ 
 			 * Filter the advanced settings.
 			 * Backward compatibility: Please do not use this filter. This filter is deprecated in 1.8
 			 */
-			$settings = apply_filters( 'give_settings_advanced', $settings );
+			$settings = apply_filters('give_settings_advanced', $settings);
 
 			/**
 			 * Filter the settings.
@@ -116,7 +116,7 @@ 
 			 *
 			 * @param  array $settings
 			 */
-			$settings = apply_filters( 'give_get_settings_' . $this->id, $settings );
+			$settings = apply_filters('give_get_settings_'.$this->id, $settings);
 
 			// Output.
 			return $settings;
@@ -130,10 +130,10 @@ 
 		 */
 		public function get_sections() {
 			$sections = array(
-				'advanced-options' => esc_html__( 'Advanced Options', 'give' )
+				'advanced-options' => esc_html__('Advanced Options', 'give')
 			);
 
-			return apply_filters( 'give_get_sections_' . $this->id, $sections );
+			return apply_filters('give_get_sections_'.$this->id, $sections);
 		}
 	}