ravinderk / Give

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

Spacing +71 added lines, -71 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -39,30 +39,30 @@ 
 	 */
 	public function csv_cols() {
 		$cols = array(
-			'id'        => esc_html__( 'ID', 'give' ), // unaltered payment ID (use for querying).
-			'seq_id'    => esc_html__( 'Payment Number', 'give' ), // sequential payment ID.
-			'email'     => esc_html__( 'Email', 'give' ),
-			'first'     => esc_html__( 'First Name', 'give' ),
-			'last'      => esc_html__( 'Last Name', 'give' ),
-			'address1'  => esc_html__( 'Address 1', 'give' ),
-			'address2'  => esc_html__( 'Address 2', 'give' ),
-			'city'      => esc_html__( 'City', 'give' ),
-			'state'     => esc_html__( 'State', 'give' ),
-			'country'   => esc_html__( 'Country', 'give' ),
-			'zip'       => esc_html__( 'Zip / Postal Code', 'give' ),
-			'form_id'   => esc_html__( 'Form ID', 'give' ),
-			'form_name' => esc_html__( 'Form Name', 'give' ),
-			'amount'    => esc_html__( 'Amount', 'give' ) . ' (' . html_entity_decode( give_currency_filter( '' ) ) . ')',
-			'gateway'   => esc_html__( 'Payment Method', 'give' ),
-			'trans_id'  => esc_html__( 'Transaction ID', 'give' ),
-			'key'       => esc_html__( 'Key', 'give' ),
-			'date'      => esc_html__( 'Date', 'give' ),
-			'user'      => esc_html__( 'User', 'give' ),
-			'status'    => esc_html__( 'Status', 'give' )
+			'id'        => esc_html__('ID', 'give'), // unaltered payment ID (use for querying).
+			'seq_id'    => esc_html__('Payment Number', 'give'), // sequential payment ID.
+			'email'     => esc_html__('Email', 'give'),
+			'first'     => esc_html__('First Name', 'give'),
+			'last'      => esc_html__('Last Name', 'give'),
+			'address1'  => esc_html__('Address 1', 'give'),
+			'address2'  => esc_html__('Address 2', 'give'),
+			'city'      => esc_html__('City', 'give'),
+			'state'     => esc_html__('State', 'give'),
+			'country'   => esc_html__('Country', 'give'),
+			'zip'       => esc_html__('Zip / Postal Code', 'give'),
+			'form_id'   => esc_html__('Form ID', 'give'),
+			'form_name' => esc_html__('Form Name', 'give'),
+			'amount'    => esc_html__('Amount', 'give').' ('.html_entity_decode(give_currency_filter('')).')',
+			'gateway'   => esc_html__('Payment Method', 'give'),
+			'trans_id'  => esc_html__('Transaction ID', 'give'),
+			'key'       => esc_html__('Key', 'give'),
+			'date'      => esc_html__('Date', 'give'),
+			'user'      => esc_html__('User', 'give'),
+			'status'    => esc_html__('Status', 'give')
 		);
 
-		if ( ! give_get_option( 'enable_sequential' ) ) {
-			unset( $cols['seq_id'] );
+		if ( ! give_get_option('enable_sequential')) {
+			unset($cols['seq_id']);
 		}
 
 		return $cols;
@@ -87,12 +87,12 @@ 
 			'status' => $this->status
 		);
 
-		if ( ! empty( $this->start ) || ! empty( $this->end ) ) {
+		if ( ! empty($this->start) || ! empty($this->end)) {
 
 			$args['date_query'] = array(
 				array(
-					'after'     => date( 'Y-n-d 00:00:00', strtotime( $this->start ) ),
-					'before'    => date( 'Y-n-d 23:59:59', strtotime( $this->end ) ),
+					'after'     => date('Y-n-d 00:00:00', strtotime($this->start)),
+					'before'    => date('Y-n-d 23:59:59', strtotime($this->end)),
 					'inclusive' => true
 				)
 			);
@@ -100,11 +100,11 @@ 
 		}
 
 		// Add category or tag to payment query if any.
-		if ( ! empty( $this->categories ) || ! empty( $this->tags ) ) {
+		if ( ! empty($this->categories) || ! empty($this->tags)) {
 			$form_args = array(
 				'post_type'      => 'give_forms',
 				'post_status'    => 'publish',
-				'posts_per_page' => - 1,
+				'posts_per_page' => -1,
 				'fields'         => 'ids',
 				'tax_query'      => array(
 					'relation' => 'AND',
@@ -112,75 +112,75 @@ 
 			);
 
 
-			if ( ! empty( $this->categories ) ) {
+			if ( ! empty($this->categories)) {
 				$form_args['tax_query'][] = array(
 					'taxonomy' => 'give_forms_category',
 					'terms'    => $this->categories,
 				);
 			}
 
-			if ( ! empty( $this->tags ) ) {
+			if ( ! empty($this->tags)) {
 				$form_args['tax_query'][] = array(
 					'taxonomy' => 'give_forms_tag',
 					'terms'    => $this->tags,
 				);
 			}
 
-			$forms = new WP_Query( $form_args );
+			$forms = new WP_Query($form_args);
 
-			if ( empty( $forms->posts ) ) {
+			if (empty($forms->posts)) {
 				return array();
 			}
 
 			$args['give_forms'] = $forms->posts;
 		}
 
-		$payments = give_get_payments( $args );
+		$payments = give_get_payments($args);
 
-		if ( $payments ) {
+		if ($payments) {
 
-			foreach ( $payments as $payment ) {
-				$payment_meta = give_get_payment_meta( $payment->ID );
-				$user_info    = give_get_payment_meta_user_info( $payment->ID );
-				$total        = give_get_payment_amount( $payment->ID );
-				$user_id      = isset( $user_info['id'] ) && $user_info['id'] != - 1 ? $user_info['id'] : $user_info['email'];
+			foreach ($payments as $payment) {
+				$payment_meta = give_get_payment_meta($payment->ID);
+				$user_info    = give_get_payment_meta_user_info($payment->ID);
+				$total        = give_get_payment_amount($payment->ID);
+				$user_id      = isset($user_info['id']) && $user_info['id'] != - 1 ? $user_info['id'] : $user_info['email'];
 				$products     = '';
 				$skus         = '';
 
-				if ( is_numeric( $user_id ) ) {
-					$user = get_userdata( $user_id );
+				if (is_numeric($user_id)) {
+					$user = get_userdata($user_id);
 				} else {
 					$user = false;
 				}
 
 				$data[] = array(
 					'id'        => $payment->ID,
-					'seq_id'    => give_get_payment_number( $payment->ID ),
+					'seq_id'    => give_get_payment_number($payment->ID),
 					'email'     => $payment_meta['email'],
 					'first'     => $user_info['first_name'],
 					'last'      => $user_info['last_name'],
-					'address1'  => isset( $user_info['address']['line1'] ) ? $user_info['address']['line1'] : '',
-					'address2'  => isset( $user_info['address']['line2'] ) ? $user_info['address']['line2'] : '',
-					'city'      => isset( $user_info['address']['city'] ) ? $user_info['address']['city'] : '',
-					'state'     => isset( $user_info['address']['state'] ) ? $user_info['address']['state'] : '',
-					'country'   => isset( $user_info['address']['country'] ) ? $user_info['address']['country'] : '',
-					'zip'       => isset( $user_info['address']['zip'] ) ? $user_info['address']['zip'] : '',
-					'form_id'   => isset( $payment_meta['form_id'] ) ? $payment_meta['form_id'] : '',
-					'form_name' => isset( $payment_meta['form_title'] ) ? $payment_meta['form_title'] : '',
+					'address1'  => isset($user_info['address']['line1']) ? $user_info['address']['line1'] : '',
+					'address2'  => isset($user_info['address']['line2']) ? $user_info['address']['line2'] : '',
+					'city'      => isset($user_info['address']['city']) ? $user_info['address']['city'] : '',
+					'state'     => isset($user_info['address']['state']) ? $user_info['address']['state'] : '',
+					'country'   => isset($user_info['address']['country']) ? $user_info['address']['country'] : '',
+					'zip'       => isset($user_info['address']['zip']) ? $user_info['address']['zip'] : '',
+					'form_id'   => isset($payment_meta['form_id']) ? $payment_meta['form_id'] : '',
+					'form_name' => isset($payment_meta['form_title']) ? $payment_meta['form_title'] : '',
 					'skus'      => $skus,
-					'amount'    => html_entity_decode( give_format_amount( $total ) ),
-					'gateway'   => give_get_gateway_admin_label( get_post_meta( $payment->ID, '_give_payment_gateway', true ) ),
-					'trans_id'  => give_get_payment_transaction_id( $payment->ID ),
+					'amount'    => html_entity_decode(give_format_amount($total)),
+					'gateway'   => give_get_gateway_admin_label(get_post_meta($payment->ID, '_give_payment_gateway', true)),
+					'trans_id'  => give_get_payment_transaction_id($payment->ID),
 					'key'       => $payment_meta['key'],
 					'date'      => $payment->post_date,
-					'user'      => $user ? $user->display_name : __( 'guest', 'give' ),
-					'status'    => give_get_payment_status( $payment, true )
+					'user'      => $user ? $user->display_name : __('guest', 'give'),
+					'status'    => give_get_payment_status($payment, true)
 				);
 
 			}
 
-			$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;
 
@@ -200,27 +200,27 @@ 
 
 		$status = $this->status;
 		$args   = array(
-			'start-date' => date( 'n/d/Y', strtotime( $this->start ) ),
-			'end-date'   => date( 'n/d/Y', strtotime( $this->end ) ),
+			'start-date' => date('n/d/Y', strtotime($this->start)),
+			'end-date'   => date('n/d/Y', strtotime($this->end)),
 		);
 
-		if ( 'any' == $status ) {
+		if ('any' == $status) {
 
-			$total = array_sum( (array) give_count_payments( $args ) );
+			$total = array_sum((array) give_count_payments($args));
 
 		} else {
 
-			$total = give_count_payments( $args )->$status;
+			$total = give_count_payments($args)->$status;
 
 		}
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( 30 * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = ((30 * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -234,11 +234,11 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing.
 	 */
-	public function set_properties( $request ) {
-		$this->start      = isset( $request['start'] ) ? sanitize_text_field( $request['start'] ) : '';
-		$this->end        = isset( $request['end'] ) ? sanitize_text_field( $request['end'] ) : '';
-		$this->status     = isset( $request['status'] ) ? sanitize_text_field( $request['status'] ) : 'complete';
-		$this->categories = isset( $request['give_forms_categories'] ) ? give_clean( $request['give_forms_categories'] ) : array();
-		$this->tags       = isset( $request['give_forms_tags'] ) ? give_clean( $request['give_forms_tags'] ) : array();
+	public function set_properties($request) {
+		$this->start      = isset($request['start']) ? sanitize_text_field($request['start']) : '';
+		$this->end        = isset($request['end']) ? sanitize_text_field($request['end']) : '';
+		$this->status     = isset($request['status']) ? sanitize_text_field($request['status']) : 'complete';
+		$this->categories = isset($request['give_forms_categories']) ? give_clean($request['give_forms_categories']) : array();
+		$this->tags       = isset($request['give_forms_tags']) ? give_clean($request['give_forms_tags']) : array();
 	}
 }

includes/admin/reporting/logs.php

Spacing +14 added lines, -14 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -23,7 +23,7 @@ 
  * @return void
  */
 function give_logs_view_sales() {
-	include( dirname( __FILE__ ) . '/class-sales-logs-list-table.php' );
+	include(dirname(__FILE__).'/class-sales-logs-list-table.php');
 
 	$logs_table = new Give_Sales_Log_Table();
 	$logs_table->prepare_items();
@@ -31,7 +31,7 @@ 
 
 }
 
-add_action( 'give_logs_view_sales', 'give_logs_view_sales' );
+add_action('give_logs_view_sales', 'give_logs_view_sales');
 
 
 /**
@@ -43,14 +43,14 @@ 
  * @return void
  */
 function give_logs_view_gateway_errors() {
-	include( dirname( __FILE__ ) . '/class-gateway-error-logs-list-table.php' );
+	include(dirname(__FILE__).'/class-gateway-error-logs-list-table.php');
 
 	$logs_table = new Give_Gateway_Error_Log_Table();
 	$logs_table->prepare_items();
 	$logs_table->display();
 }
 
-add_action( 'give_logs_view_gateway_errors', 'give_logs_view_gateway_errors' );
+add_action('give_logs_view_gateway_errors', 'give_logs_view_gateway_errors');
 
 /**
  * API Request Logs
@@ -62,7 +62,7 @@ 
  * @return void
  */
 function give_logs_view_api_requests() {
-	include( dirname( __FILE__ ) . '/class-api-requests-logs-list-table.php' );
+	include(dirname(__FILE__).'/class-api-requests-logs-list-table.php');
 
 	$logs_table = new Give_API_Request_Log_Table();
 	$logs_table->prepare_items();
@@ -75,12 +75,12 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_logs_api_requests_top' );
+		do_action('give_logs_api_requests_top');
 		?>
 
 		<form id="give-logs-filter" method="get" action="<?php echo 'edit.php?post_type=give_forms&page=give-tools&tab=logs'; ?>">
 			<?php
-			$logs_table->search_box( esc_html__( 'Search', 'give' ), 'give-api-requests' );
+			$logs_table->search_box(esc_html__('Search', 'give'), 'give-api-requests');
 			$logs_table->display();
 			?>
 			<input type="hidden" name="post_type" value="give_forms"/>
@@ -93,14 +93,14 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_logs_api_requests_bottom' );
+		do_action('give_logs_api_requests_bottom');
 		?>
 
 	</div>
 	<?php
 }
 
-add_action( 'give_logs_view_api_requests', 'give_logs_view_api_requests' );
+add_action('give_logs_view_api_requests', 'give_logs_view_api_requests');
 
 /**
  * Renders the Reports page views drop down
@@ -112,11 +112,11 @@ 
 	$current_section = give_get_current_setting_section();
 
 	// If there are not any event attach to action then do not show form.
-	if ( ! has_action( 'give_log_view_actions' ) ) {
+	if ( ! has_action('give_log_view_actions')) {
 		return;
 	}
 	?>
-	<form id="give-logs-filter" method="get" action="<?php echo 'edit.php?post_type=give_forms&page=give-tools&tab=logs&section=' . $current_section; ?>">
+	<form id="give-logs-filter" method="get" action="<?php echo 'edit.php?post_type=give_forms&page=give-tools&tab=logs&section='.$current_section; ?>">
 		<?php
 		/**
 		 * Fires after displaying the reports page views drop down.
@@ -125,14 +125,14 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_log_view_actions' );
+		do_action('give_log_view_actions');
 		?>
 
 		<input type="hidden" name="post_type" value="give_forms"/>
 		<input type="hidden" name="page" value="give-tools"/>
 		<input type="hidden" name="tab" value="logs"/>
 
-		<?php submit_button( esc_html__( 'Apply', 'give' ), 'secondary', 'submit', false ); ?>
+		<?php submit_button(esc_html__('Apply', 'give'), 'secondary', 'submit', false); ?>
 	</form>
 	<?php
 }

includes/deprecated/deprecated-functions.php

Spacing +10 added lines, -10 removed lines

@@ -12,7 +12,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -27,15 +27,15 @@ 
  *
  * @return bool $ret True if guest checkout is enabled, false otherwise
  */
-function give_no_guest_checkout( $form_id ) {
+function give_no_guest_checkout($form_id) {
 
 	$backtrace = debug_backtrace();
 
-	_give_deprecated_function( __FUNCTION__, '1.4.1', null, $backtrace );
+	_give_deprecated_function(__FUNCTION__, '1.4.1', null, $backtrace);
 
-	$ret = get_post_meta( $form_id, '_give_logged_in_only', true );
+	$ret = get_post_meta($form_id, '_give_logged_in_only', true);
 
-	return (bool) apply_filters( 'give_no_guest_checkout', give_is_setting_enabled( $ret ) );
+	return (bool) apply_filters('give_no_guest_checkout', give_is_setting_enabled($ret));
 }
 
 
@@ -50,15 +50,15 @@ 
 
 	$backtrace = debug_backtrace();
 
-	_give_deprecated_function( __FUNCTION__, '1.8', null, $backtrace );
+	_give_deprecated_function(__FUNCTION__, '1.8', null, $backtrace);
 
 	$views = array(
-		'sales'          => esc_html__( 'Donations', 'give' ),
-		'gateway_errors' => esc_html__( 'Payment Errors', 'give' ),
-		'api_requests'   => esc_html__( 'API Requests', 'give' ),
+		'sales'          => esc_html__('Donations', 'give'),
+		'gateway_errors' => esc_html__('Payment Errors', 'give'),
+		'api_requests'   => esc_html__('API Requests', 'give'),
 	);
 
-	$views = apply_filters( 'give_log_views', $views );
+	$views = apply_filters('give_log_views', $views);
 
 	return $views;
 }
\ No newline at end of file

includes/deprecated/deprecated-filters.php

Spacing +9 added lines, -9 removed lines

@@ -4,8 +4,8 @@ 
  */
 $give_map_deprecated_filters = give_deprecated_filters();
 
-foreach ( $give_map_deprecated_filters as $new => $old ) {
-	add_filter( $new, 'give_deprecated_filter_mapping', 10, 4 );
+foreach ($give_map_deprecated_filters as $new => $old) {
+	add_filter($new, 'give_deprecated_filter_mapping', 10, 4);
 }
 
 /**
@@ -41,20 +41,20 @@ 
  *
  * @return mixed
  */
-function give_deprecated_filter_mapping( $data, $arg_1 = '', $arg_2 = '', $arg_3 = '' ) {
+function give_deprecated_filter_mapping($data, $arg_1 = '', $arg_2 = '', $arg_3 = '') {
 	$give_map_deprecated_filters = give_deprecated_filters();
 	$filter                      = current_filter();
 
-	if ( isset( $give_map_deprecated_filters[ $filter ] ) ) {
-		if ( has_filter( $give_map_deprecated_filters[ $filter ] ) ) {
-			$data = apply_filters( $give_map_deprecated_filters[ $filter ], $data, $arg_1, $arg_2, $arg_3 );
+	if (isset($give_map_deprecated_filters[$filter])) {
+		if (has_filter($give_map_deprecated_filters[$filter])) {
+			$data = apply_filters($give_map_deprecated_filters[$filter], $data, $arg_1, $arg_2, $arg_3);
 
-			if ( ! defined( 'DOING_AJAX' ) ) {
+			if ( ! defined('DOING_AJAX')) {
 				_give_deprecated_function(
 					sprintf(
 					/* translators: %s: filter name */
-						__( 'The %s filter' ),
-						$give_map_deprecated_filters[ $filter ]
+						__('The %s filter'),
+						$give_map_deprecated_filters[$filter]
 					),
 					'1.7',
 					$filter

includes/gateways/functions.php

Spacing +66 added lines, -66 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,16 +24,16 @@ 
 	// Default, built-in gateways
 	$gateways = array(
 		'paypal' => array(
-			'admin_label'    => __( 'PayPal Standard', 'give' ),
-			'checkout_label' => __( 'PayPal', 'give' ),
+			'admin_label'    => __('PayPal Standard', 'give'),
+			'checkout_label' => __('PayPal', 'give'),
 		),
 		'manual' => array(
-			'admin_label'    => __( 'Test Donation', 'give' ),
-			'checkout_label' => __( 'Test Donation', 'give' )
+			'admin_label'    => __('Test Donation', 'give'),
+			'checkout_label' => __('Test Donation', 'give')
 		),
 	);
 
-	return apply_filters( 'give_payment_gateways', $gateways );
+	return apply_filters('give_payment_gateways', $gateways);
 
 }
 
@@ -46,24 +46,24 @@ 
  *
  * @return array $gateway_list All the available gateways
  */
-function give_get_enabled_payment_gateways( $form_id = 0 ) {
+function give_get_enabled_payment_gateways($form_id = 0) {
 
 	$gateways = give_get_payment_gateways();
 
-	$enabled = isset( $_POST['gateways'] ) ? $_POST['gateways'] : give_get_option( 'gateways' );
+	$enabled = isset($_POST['gateways']) ? $_POST['gateways'] : give_get_option('gateways');
 
 	$gateway_list = array();
 
-	foreach ( $gateways as $key => $gateway ) {
-		if ( isset( $enabled[ $key ] ) && $enabled[ $key ] == 1 ) {
-			$gateway_list[ $key ] = $gateway;
+	foreach ($gateways as $key => $gateway) {
+		if (isset($enabled[$key]) && $enabled[$key] == 1) {
+			$gateway_list[$key] = $gateway;
 		}
 	}
 
 	// Set order of payment gateway in list.
-	$gateway_list = give_get_ordered_payment_gateways( $gateway_list );
+	$gateway_list = give_get_ordered_payment_gateways($gateway_list);
 
-	return apply_filters( 'give_enabled_payment_gateways', $gateway_list, $form_id );
+	return apply_filters('give_enabled_payment_gateways', $gateway_list, $form_id);
 }
 
 /**
@@ -75,12 +75,12 @@ 
  *
  * @return boolean true if enabled, false otherwise
  */
-function give_is_gateway_active( $gateway ) {
+function give_is_gateway_active($gateway) {
 	$gateways = give_get_enabled_payment_gateways();
 
-	$ret = array_key_exists( $gateway, $gateways );
+	$ret = array_key_exists($gateway, $gateways);
 
-	return apply_filters( 'give_is_gateway_active', $ret, $gateway, $gateways );
+	return apply_filters('give_is_gateway_active', $ret, $gateway, $gateways);
 }
 
 /**
@@ -92,23 +92,23 @@ 
  *
  * @return string Gateway ID
  */
-function give_get_default_gateway( $form_id ) {
+function give_get_default_gateway($form_id) {
 
 	$give_options = give_get_settings();
-	$default      = isset( $give_options['default_gateway'] ) && give_is_gateway_active( $give_options['default_gateway'] ) ? $give_options['default_gateway'] : 'paypal';
-	$form_default = get_post_meta( $form_id, '_give_default_gateway', true );
+	$default      = isset($give_options['default_gateway']) && give_is_gateway_active($give_options['default_gateway']) ? $give_options['default_gateway'] : 'paypal';
+	$form_default = get_post_meta($form_id, '_give_default_gateway', true);
 
 	// Single Form settings varies compared to the Global default settings.
-	if ( ! empty( $form_default ) &&
+	if ( ! empty($form_default) &&
 		 $form_id !== null &&
 		 $default !== $form_default &&
 		 $form_default !== 'global' &&
-		 give_is_gateway_active( $form_default )
+		 give_is_gateway_active($form_default)
 	) {
 		$default = $form_default;
 	}
 
-	return apply_filters( 'give_default_gateway', $default );
+	return apply_filters('give_default_gateway', $default);
 }
 
 /**
@@ -120,18 +120,18 @@ 
  *
  * @return string Gateway admin label
  */
-function give_get_gateway_admin_label( $gateway ) {
+function give_get_gateway_admin_label($gateway) {
 	$gateways = give_get_payment_gateways();
-	$label    = isset( $gateways[ $gateway ] ) ? $gateways[ $gateway ]['admin_label'] : $gateway;
-	$payment  = isset( $_GET['id'] ) ? absint( $_GET['id'] ) : false;
+	$label    = isset($gateways[$gateway]) ? $gateways[$gateway]['admin_label'] : $gateway;
+	$payment  = isset($_GET['id']) ? absint($_GET['id']) : false;
 
-	if ( $gateway == 'manual' && $payment ) {
-		if ( give_get_payment_amount( $payment ) == 0 ) {
-			$label = __( 'Test Donation', 'give' );
+	if ($gateway == 'manual' && $payment) {
+		if (give_get_payment_amount($payment) == 0) {
+			$label = __('Test Donation', 'give');
 		}
 	}
 
-	return apply_filters( 'give_gateway_admin_label', $label, $gateway );
+	return apply_filters('give_gateway_admin_label', $label, $gateway);
 }
 
 /**
@@ -143,15 +143,15 @@ 
  *
  * @return string Checkout label for the gateway
  */
-function give_get_gateway_checkout_label( $gateway ) {
+function give_get_gateway_checkout_label($gateway) {
 	$gateways = give_get_payment_gateways();
-	$label    = isset( $gateways[ $gateway ] ) ? $gateways[ $gateway ]['checkout_label'] : $gateway;
+	$label    = isset($gateways[$gateway]) ? $gateways[$gateway]['checkout_label'] : $gateway;
 
-	if ( $gateway == 'manual' ) {
-		$label = __( 'Test Donation', 'give' );
+	if ($gateway == 'manual') {
+		$label = __('Test Donation', 'give');
 	}
 
-	return apply_filters( 'give_gateway_checkout_label', $label, $gateway );
+	return apply_filters('give_gateway_checkout_label', $label, $gateway);
 }
 
 /**
@@ -163,11 +163,11 @@ 
  *
  * @return array Options the gateway supports
  */
-function give_get_gateway_supports( $gateway ) {
+function give_get_gateway_supports($gateway) {
 	$gateways = give_get_enabled_payment_gateways();
-	$supports = isset( $gateways[ $gateway ]['supports'] ) ? $gateways[ $gateway ]['supports'] : array();
+	$supports = isset($gateways[$gateway]['supports']) ? $gateways[$gateway]['supports'] : array();
 
-	return apply_filters( 'give_gateway_supports', $supports, $gateway );
+	return apply_filters('give_gateway_supports', $supports, $gateway);
 }
 
 /**
@@ -180,9 +180,9 @@ 
  *
  * @return void
  */
-function give_send_to_gateway( $gateway, $payment_data ) {
+function give_send_to_gateway($gateway, $payment_data) {
 
-	$payment_data['gateway_nonce'] = wp_create_nonce( 'give-gateway' );
+	$payment_data['gateway_nonce'] = wp_create_nonce('give-gateway');
 
 	/**
 	 * Fires while loading payment gateway via AJAX.
@@ -193,7 +193,7 @@ 
 	 *
 	 * @param array $payment_data All the payment data to be sent to the gateway.
 	 */
-	do_action( "give_gateway_{$gateway}", $payment_data );
+	do_action("give_gateway_{$gateway}", $payment_data);
 }
 
 
@@ -207,34 +207,34 @@ 
  *
  * @return string $enabled_gateway The slug of the gateway
  */
-function give_get_chosen_gateway( $form_id ) {
+function give_get_chosen_gateway($form_id) {
 
-	$request_form_id = isset( $_REQUEST['give_form_id'] ) ? $_REQUEST['give_form_id'] : 0;
+	$request_form_id = isset($_REQUEST['give_form_id']) ? $_REQUEST['give_form_id'] : 0;
 
 	// Back to check if 'form-id' is present.
-	if ( empty( $request_form_id ) ) {
-		$request_form_id = isset( $_REQUEST['form-id'] ) ? $_REQUEST['form-id'] : 0;
+	if (empty($request_form_id)) {
+		$request_form_id = isset($_REQUEST['form-id']) ? $_REQUEST['form-id'] : 0;
 	}
 
-	$request_payment_mode = isset( $_REQUEST['payment-mode'] ) ? $_REQUEST['payment-mode'] : '';
+	$request_payment_mode = isset($_REQUEST['payment-mode']) ? $_REQUEST['payment-mode'] : '';
 	$chosen               = false;
 
 	// If both 'payment-mode' and 'form-id' then set for only this form.
-	if ( ! empty( $request_form_id ) && $form_id == $request_form_id ) {
+	if ( ! empty($request_form_id) && $form_id == $request_form_id) {
 		$chosen = $request_payment_mode;
-	} elseif ( empty( $request_form_id ) && $request_payment_mode ) {
+	} elseif (empty($request_form_id) && $request_payment_mode) {
 		// If no 'form-id' but there is 'payment-mode'.
 		$chosen = $request_payment_mode;
 	}
 
 	// Get the enable gateway based of chosen var.
-	if ( $chosen && give_is_gateway_active( $chosen ) ) {
-		$enabled_gateway = urldecode( $chosen );
+	if ($chosen && give_is_gateway_active($chosen)) {
+		$enabled_gateway = urldecode($chosen);
 	} else {
-		$enabled_gateway = give_get_default_gateway( $form_id );
+		$enabled_gateway = give_get_default_gateway($form_id);
 	}
 
-	return apply_filters( 'give_chosen_gateway', $enabled_gateway );
+	return apply_filters('give_chosen_gateway', $enabled_gateway);
 
 }
 
@@ -252,8 +252,8 @@ 
  *
  * @return int ID of the new log entry
  */
-function give_record_gateway_error( $title = '', $message = '', $parent = 0 ) {
-	return give_record_log( $title, $message, $parent, 'gateway_error' );
+function give_record_gateway_error($title = '', $message = '', $parent = 0) {
+	return give_record_log($title, $message, $parent, 'gateway_error');
 }
 
 /**
@@ -266,7 +266,7 @@ 
  *
  * @return int
  */
-function give_count_sales_by_gateway( $gateway_id = 'paypal', $status = 'publish' ) {
+function give_count_sales_by_gateway($gateway_id = 'paypal', $status = 'publish') {
 
 	$ret  = 0;
 	$args = array(
@@ -278,9 +278,9 @@ 
 		'fields'      => 'ids',
 	);
 
-	$payments = new WP_Query( $args );
+	$payments = new WP_Query($args);
 
-	if ( $payments ) {
+	if ($payments) {
 		$ret = $payments->post_count;
 	}
 
@@ -297,27 +297,27 @@ 
  *
  * @return array $gateways All the available gateways
  */
-function give_get_ordered_payment_gateways( $gateways ) {
+function give_get_ordered_payment_gateways($gateways) {
 
 	// Get gateways setting.
-	$gateways_setting = isset( $_POST['gateways'] ) ? $_POST['gateways'] : give_get_option( 'gateways' );
+	$gateways_setting = isset($_POST['gateways']) ? $_POST['gateways'] : give_get_option('gateways');
 
 	// Return from here if we do not have gateways setting.
-	if ( empty( $gateways_setting ) ) {
+	if (empty($gateways_setting)) {
 		return $gateways;
 	}
 
 	// Reverse array to order payment gateways.
-	$gateways_setting = array_reverse( $gateways_setting );
+	$gateways_setting = array_reverse($gateways_setting);
 
 	// Reorder gateways array
-	foreach ( $gateways_setting as $gateway_key => $value ) {
+	foreach ($gateways_setting as $gateway_key => $value) {
 
-		$new_gateway_value = isset( $gateways[ $gateway_key ] ) ? $gateways[ $gateway_key ] : '';
-		unset( $gateways[ $gateway_key ] );
+		$new_gateway_value = isset($gateways[$gateway_key]) ? $gateways[$gateway_key] : '';
+		unset($gateways[$gateway_key]);
 
-		if ( ! empty( $new_gateway_value ) ) {
-			$gateways = array_merge( array( $gateway_key => $new_gateway_value ), $gateways );
+		if ( ! empty($new_gateway_value)) {
+			$gateways = array_merge(array($gateway_key => $new_gateway_value), $gateways);
 		}
 	}
 
@@ -328,5 +328,5 @@ 
 	 *
 	 * @param array $gateways All the available gateways
 	 */
-	return apply_filters( 'give_payment_gateways_order', $gateways );
+	return apply_filters('give_payment_gateways_order', $gateways);
 }
\ No newline at end of file

includes/class-give-html-elements.php

Spacing +141 added lines, -141 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -35,7 +35,7 @@ 
 	 *
 	 * @return string       Donations dropdown.
 	 */
-	public function transactions_dropdown( $args = array() ) {
+	public function transactions_dropdown($args = array()) {
 
 		$defaults = array(
 			'name'        => 'transactions',
@@ -45,33 +45,33 @@ 
 			'selected'    => 0,
 			'chosen'      => false,
 			'number'      => 30,
-			'placeholder' => esc_html__( 'Select a transaction', 'give' )
+			'placeholder' => esc_html__('Select a transaction', 'give')
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		$payments = new Give_Payments_Query( array(
+		$payments = new Give_Payments_Query(array(
 			'number' => $args['number']
-		) );
+		));
 
 		$payments = $payments->get_payments();
 
 		$options = array();
 
 		//Provide nice human readable options.
-		if ( $payments ) {
-			$options[0] = esc_html__( 'Select a donation', 'give' );
-			foreach ( $payments as $payment ) {
+		if ($payments) {
+			$options[0] = esc_html__('Select a donation', 'give');
+			foreach ($payments as $payment) {
 
-				$options[ absint( $payment->ID ) ] = esc_html( '#' . $payment->ID . ' - ' . $payment->email . ' - ' . $payment->form_title);
+				$options[absint($payment->ID)] = esc_html('#'.$payment->ID.' - '.$payment->email.' - '.$payment->form_title);
 
 			}
 		} else {
-			$options[0] = esc_html__( 'No donations found.', 'give' );
+			$options[0] = esc_html__('No donations found.', 'give');
 		}
 
 
-		$output = $this->select( array(
+		$output = $this->select(array(
 			'name'             => $args['name'],
 			'selected'         => $args['selected'],
 			'id'               => $args['id'],
@@ -83,7 +83,7 @@ 
 			'select_atts'      => $args['select_atts'],
 			'show_option_all'  => false,
 			'show_option_none' => false
-		) );
+		));
 
 		return $output;
 	}
@@ -100,7 +100,7 @@ 
 	 *
 	 * @return string      Give forms dropdown.
 	 */
-	public function forms_dropdown( $args = array() ) {
+	public function forms_dropdown($args = array()) {
 
 		$defaults = array(
 			'name'        => 'forms',
@@ -110,31 +110,31 @@ 
 			'selected'    => 0,
 			'chosen'      => false,
 			'number'      => 30,
-			'placeholder' => esc_attr__( 'Select a Donation Form', 'give' )
+			'placeholder' => esc_attr__('Select a Donation Form', 'give')
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		$forms = get_posts( array(
+		$forms = get_posts(array(
 			'post_type'      => 'give_forms',
 			'orderby'        => 'title',
 			'order'          => 'ASC',
 			'posts_per_page' => $args['number']
-		) );
+		));
 
 		$options = array();
 
-		if ( $forms ) {
-			$options[0] = esc_attr__( 'Select a Donation Form', 'give' );
-			foreach ( $forms as $form ) {
-				$form_title = empty( $form->post_title ) ? sprintf( __( 'Untitled (#%s)', 'give' ), $form->ID ) : $form->post_title;
-				$options[ absint( $form->ID ) ] = esc_html( $form_title );
+		if ($forms) {
+			$options[0] = esc_attr__('Select a Donation Form', 'give');
+			foreach ($forms as $form) {
+				$form_title = empty($form->post_title) ? sprintf(__('Untitled (#%s)', 'give'), $form->ID) : $form->post_title;
+				$options[absint($form->ID)] = esc_html($form_title);
 			}
 		} else {
-			$options[0] = esc_html__( 'No forms found.', 'give' );
+			$options[0] = esc_html__('No forms found.', 'give');
 		}
 
-		$output = $this->select( array(
+		$output = $this->select(array(
 			'name'             => $args['name'],
 			'selected'         => $args['selected'],
 			'id'               => $args['id'],
@@ -145,7 +145,7 @@ 
 			'placeholder'      => $args['placeholder'],
 			'show_option_all'  => false,
 			'show_option_none' => false
-		) );
+		));
 
 		return $output;
 	}
@@ -162,7 +162,7 @@ 
 	 *
 	 * @return string      Donors dropdown.
 	 */
-	public function donor_dropdown( $args = array() ) {
+	public function donor_dropdown($args = array()) {
 
 		$defaults = array(
 			'name'        => 'customers',
@@ -171,38 +171,38 @@ 
 			'multiple'    => false,
 			'selected'    => 0,
 			'chosen'      => true,
-			'placeholder' => esc_attr__( 'Select a Donor', 'give' ),
+			'placeholder' => esc_attr__('Select a Donor', 'give'),
 			'number'      => 30
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		$customers = Give()->customers->get_customers( array(
+		$customers = Give()->customers->get_customers(array(
 			'number' => $args['number']
-		) );
+		));
 
 		$options = array();
 
-		if ( $customers ) {
-			$options[0] = esc_html__( 'No donor attached', 'give' );
-			foreach ( $customers as $customer ) {
-				$options[ absint( $customer->id ) ] = esc_html( $customer->name . ' (' . $customer->email . ')' );
+		if ($customers) {
+			$options[0] = esc_html__('No donor attached', 'give');
+			foreach ($customers as $customer) {
+				$options[absint($customer->id)] = esc_html($customer->name.' ('.$customer->email.')');
 			}
 		} else {
-			$options[0] = esc_html__( 'No donors found.', 'give' );
+			$options[0] = esc_html__('No donors found.', 'give');
 		}
 
-		if ( ! empty( $args['selected'] ) ) {
+		if ( ! empty($args['selected'])) {
 
 			// If a selected customer has been specified, we need to ensure it's in the initial list of customers displayed.
 
-			if ( ! array_key_exists( $args['selected'], $options ) ) {
+			if ( ! array_key_exists($args['selected'], $options)) {
 
-				$customer = new Give_Customer( $args['selected'] );
+				$customer = new Give_Customer($args['selected']);
 
-				if ( $customer ) {
+				if ($customer) {
 
-					$options[ absint( $args['selected'] ) ] = esc_html( $customer->name . ' (' . $customer->email . ')' );
+					$options[absint($args['selected'])] = esc_html($customer->name.' ('.$customer->email.')');
 
 				}
 
@@ -210,17 +210,17 @@ 
 
 		}
 
-		$output = $this->select( array(
+		$output = $this->select(array(
 			'name'             => $args['name'],
 			'selected'         => $args['selected'],
 			'id'               => $args['id'],
-			'class'            => $args['class'] . ' give-customer-select',
+			'class'            => $args['class'].' give-customer-select',
 			'options'          => $options,
 			'multiple'         => $args['multiple'],
 			'chosen'           => $args['chosen'],
 			'show_option_all'  => false,
 			'show_option_none' => false
-		) );
+		));
 
 		return $output;
 	}
@@ -239,24 +239,24 @@ 
 	 *
 	 * @return string           Categories dropdown.
 	 */
-	public function category_dropdown( $name = 'give_forms_categories', $selected = 0, $args = array() ) {
-		$categories = get_terms( 'give_forms_category', apply_filters( 'give_forms_category_dropdown', array() ) );
+	public function category_dropdown($name = 'give_forms_categories', $selected = 0, $args = array()) {
+		$categories = get_terms('give_forms_category', apply_filters('give_forms_category_dropdown', array()));
 		$options    = array();
 
-		foreach ( $categories as $category ) {
-			$options[ absint( $category->term_id ) ] = esc_html( $category->name );
+		foreach ($categories as $category) {
+			$options[absint($category->term_id)] = esc_html($category->name);
 		}
 
-		$output = $this->select( wp_parse_args(
+		$output = $this->select(wp_parse_args(
 			$args,
 			array(
 				'name'             => $name,
 				'selected'         => $selected,
 				'options'          => $options,
-				'show_option_all'  => esc_html__( 'All Categories', 'give' ),
+				'show_option_all'  => esc_html__('All Categories', 'give'),
 				'show_option_none' => false
 			)
-		) );
+		));
 
 		return $output;
 	}
@@ -275,24 +275,24 @@ 
 	 *
 	 * @return string           Tags dropdown.
 	 */
-	public function tags_dropdown( $name = 'give_forms_tags', $selected = 0, $args = array() ) {
-		$tags    = get_terms( 'give_forms_tag', apply_filters( 'give_forms_tag_dropdown', array() ) );
+	public function tags_dropdown($name = 'give_forms_tags', $selected = 0, $args = array()) {
+		$tags    = get_terms('give_forms_tag', apply_filters('give_forms_tag_dropdown', array()));
 		$options = array();
 
-		foreach ( $tags as $tag ) {
-			$options[ absint( $tag->term_id ) ] = esc_html( $tag->name );
+		foreach ($tags as $tag) {
+			$options[absint($tag->term_id)] = esc_html($tag->name);
 		}
 
-		$output = $this->select( wp_parse_args(
+		$output = $this->select(wp_parse_args(
 			$args,
 			array(
 				'name'             => $name,
 				'selected'         => $selected,
 				'options'          => $options,
-				'show_option_all'  => esc_html__( 'All Tags', 'give' ),
+				'show_option_all'  => esc_html__('All Tags', 'give'),
 				'show_option_none' => false,
 			)
-		) );
+		));
 
 		return $output;
 	}
@@ -312,25 +312,25 @@ 
 	 *
 	 * @return string               Years dropdown.
 	 */
-	public function year_dropdown( $name = 'year', $selected = 0, $years_before = 5, $years_after = 0 ) {
-		$current    = date( 'Y' );
-		$start_year = $current - absint( $years_before );
-		$end_year   = $current + absint( $years_after );
-		$selected   = empty( $selected ) ? date( 'Y' ) : $selected;
+	public function year_dropdown($name = 'year', $selected = 0, $years_before = 5, $years_after = 0) {
+		$current    = date('Y');
+		$start_year = $current - absint($years_before);
+		$end_year   = $current + absint($years_after);
+		$selected   = empty($selected) ? date('Y') : $selected;
 		$options    = array();
 
-		while ( $start_year <= $end_year ) {
-			$options[ absint( $start_year ) ] = $start_year;
-			$start_year ++;
+		while ($start_year <= $end_year) {
+			$options[absint($start_year)] = $start_year;
+			$start_year++;
 		}
 
-		$output = $this->select( array(
+		$output = $this->select(array(
 			'name'             => $name,
 			'selected'         => $selected,
 			'options'          => $options,
 			'show_option_all'  => false,
 			'show_option_none' => false
-		) );
+		));
 
 		return $output;
 	}
@@ -348,23 +348,23 @@ 
 	 *
 	 * @return string           Months dropdown.
 	 */
-	public function month_dropdown( $name = 'month', $selected = 0 ) {
+	public function month_dropdown($name = 'month', $selected = 0) {
 		$month    = 1;
 		$options  = array();
-		$selected = empty( $selected ) ? date( 'n' ) : $selected;
+		$selected = empty($selected) ? date('n') : $selected;
 
-		while ( $month <= 12 ) {
-			$options[ absint( $month ) ] = give_month_num_to_name( $month );
-			$month ++;
+		while ($month <= 12) {
+			$options[absint($month)] = give_month_num_to_name($month);
+			$month++;
 		}
 
-		$output = $this->select( array(
+		$output = $this->select(array(
 			'name'             => $name,
 			'selected'         => $selected,
 			'options'          => $options,
 			'show_option_all'  => false,
 			'show_option_none' => false
-		) );
+		));
 
 		return $output;
 	}
@@ -381,7 +381,7 @@ 
 	 *
 	 * @return string      The dropdown.
 	 */
-	public function select( $args = array() ) {
+	public function select($args = array()) {
 		$defaults = array(
 			'options'          => array(),
 			'name'             => null,
@@ -392,60 +392,60 @@ 
 			'placeholder'      => null,
 			'multiple'         => false,
 			'select_atts'      => false,
-			'show_option_all'  => esc_html__( 'All', 'give' ),
-			'show_option_none' => esc_html__( 'None', 'give' )
+			'show_option_all'  => esc_html__('All', 'give'),
+			'show_option_none' => esc_html__('None', 'give')
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		if ( $args['multiple'] ) {
+		if ($args['multiple']) {
 			$multiple = ' MULTIPLE';
 		} else {
 			$multiple = '';
 		}
 
-		if ( $args['chosen'] ) {
+		if ($args['chosen']) {
 			$args['class'] .= ' give-select-chosen';
 		}
 
-		if ( $args['placeholder'] ) {
+		if ($args['placeholder']) {
 			$placeholder = $args['placeholder'];
 		} else {
 			$placeholder = '';
 		}
 
 
-		$output = '<select name="' . esc_attr( $args['name'] ) . '" id="' . esc_attr( sanitize_key( str_replace( '-', '_', $args['id'] ) ) ) . '" class="give-select ' . esc_attr( $args['class'] ) . '"' . $multiple . ' ' . $args['select_atts'] . ' data-placeholder="' . $placeholder . '">';
+		$output = '<select name="'.esc_attr($args['name']).'" id="'.esc_attr(sanitize_key(str_replace('-', '_', $args['id']))).'" class="give-select '.esc_attr($args['class']).'"'.$multiple.' '.$args['select_atts'].' data-placeholder="'.$placeholder.'">';
 
-		if ( $args['show_option_all'] ) {
-			if ( $args['multiple'] ) {
-				$selected = selected( true, in_array( 0, $args['selected'] ), false );
+		if ($args['show_option_all']) {
+			if ($args['multiple']) {
+				$selected = selected(true, in_array(0, $args['selected']), false);
 			} else {
-				$selected = selected( $args['selected'], 0, false );
+				$selected = selected($args['selected'], 0, false);
 			}
-			$output .= '<option value="all"' . $selected . '>' . esc_html( $args['show_option_all'] ) . '</option>';
+			$output .= '<option value="all"'.$selected.'>'.esc_html($args['show_option_all']).'</option>';
 		}
 
-		if ( ! empty( $args['options'] ) ) {
+		if ( ! empty($args['options'])) {
 
-			if ( $args['show_option_none'] ) {
-				if ( $args['multiple'] ) {
-					$selected = selected( true, in_array( - 1, $args['selected'] ), false );
+			if ($args['show_option_none']) {
+				if ($args['multiple']) {
+					$selected = selected(true, in_array( -1, $args['selected'] ), false);
 				} else {
-					$selected = selected( $args['selected'], - 1, false );
+					$selected = selected($args['selected'], - 1, false);
 				}
-				$output .= '<option value="-1"' . $selected . '>' . esc_html( $args['show_option_none'] ) . '</option>';
+				$output .= '<option value="-1"'.$selected.'>'.esc_html($args['show_option_none']).'</option>';
 			}
 
-			foreach ( $args['options'] as $key => $option ) {
+			foreach ($args['options'] as $key => $option) {
 
-				if ( $args['multiple'] && is_array( $args['selected'] ) ) {
-					$selected = selected( true, in_array( $key, $args['selected'] ), false );
+				if ($args['multiple'] && is_array($args['selected'])) {
+					$selected = selected(true, in_array($key, $args['selected']), false);
 				} else {
-					$selected = selected( $args['selected'], $key, false );
+					$selected = selected($args['selected'], $key, false);
 				}
 
-				$output .= '<option value="' . esc_attr( $key ) . '"' . $selected . '>' . esc_html( $option ) . '</option>';
+				$output .= '<option value="'.esc_attr($key).'"'.$selected.'>'.esc_html($option).'</option>';
 			}
 		}
 
@@ -466,7 +466,7 @@ 
 	 *
 	 * @return string      The checkbox.
 	 */
-	public function checkbox( $args = array() ) {
+	public function checkbox($args = array()) {
 		$defaults = array(
 			'name'    => null,
 			'current' => null,
@@ -477,16 +477,16 @@ 
 			)
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
 		$options = '';
-		if ( ! empty( $args['options']['disabled'] ) ) {
+		if ( ! empty($args['options']['disabled'])) {
 			$options .= ' disabled="disabled"';
-		} elseif ( ! empty( $args['options']['readonly'] ) ) {
+		} elseif ( ! empty($args['options']['readonly'])) {
 			$options .= ' readonly';
 		}
 
-		$output = '<input type="checkbox"' . $options . ' name="' . esc_attr( $args['name'] ) . '" id="' . esc_attr( $args['name'] ) . '" class="' . $args['class'] . ' ' . esc_attr( $args['name'] ) . '" ' . checked( 1, $args['current'], false ) . ' />';
+		$output = '<input type="checkbox"'.$options.' name="'.esc_attr($args['name']).'" id="'.esc_attr($args['name']).'" class="'.$args['class'].' '.esc_attr($args['name']).'" '.checked(1, $args['current'], false).' />';
 
 		return $output;
 	}
@@ -503,22 +503,22 @@ 
 	 *
 	 * @return string      The text field.
 	 */
-	public function text( $args = array() ) {
+	public function text($args = array()) {
 		// Backwards compatibility
-		if ( func_num_args() > 1 ) {
+		if (func_num_args() > 1) {
 			$args = func_get_args();
 
 			$name  = $args[0];
-			$value = isset( $args[1] ) ? $args[1] : '';
-			$label = isset( $args[2] ) ? $args[2] : '';
-			$desc  = isset( $args[3] ) ? $args[3] : '';
+			$value = isset($args[1]) ? $args[1] : '';
+			$label = isset($args[2]) ? $args[2] : '';
+			$desc  = isset($args[3]) ? $args[3] : '';
 		}
 
 		$defaults = array(
-			'name'         => isset( $name ) ? $name : 'text',
-			'value'        => isset( $value ) ? $value : null,
-			'label'        => isset( $label ) ? $label : null,
-			'desc'         => isset( $desc ) ? $desc : null,
+			'name'         => isset($name) ? $name : 'text',
+			'value'        => isset($value) ? $value : null,
+			'label'        => isset($label) ? $label : null,
+			'desc'         => isset($desc) ? $desc : null,
 			'placeholder'  => '',
 			'class'        => 'regular-text',
 			'disabled'     => false,
@@ -526,29 +526,29 @@ 
 			'data'         => false
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
 		$disabled = '';
-		if ( $args['disabled'] ) {
+		if ($args['disabled']) {
 			$disabled = ' disabled="disabled"';
 		}
 
 		$data = '';
-		if ( ! empty( $args['data'] ) ) {
-			foreach ( $args['data'] as $key => $value ) {
-				$data .= 'data-' . $key . '="' . $value . '" ';
+		if ( ! empty($args['data'])) {
+			foreach ($args['data'] as $key => $value) {
+				$data .= 'data-'.$key.'="'.$value.'" ';
 			}
 		}
 
-		$output = '<span id="give-' . sanitize_key( $args['name'] ) . '-wrap">';
+		$output = '<span id="give-'.sanitize_key($args['name']).'-wrap">';
 
-		$output .= '<label class="give-label" for="give-' . sanitize_key( $args['name'] ) . '">' . esc_html( $args['label'] ) . '</label>';
+		$output .= '<label class="give-label" for="give-'.sanitize_key($args['name']).'">'.esc_html($args['label']).'</label>';
 
-		if ( ! empty( $args['desc'] ) ) {
-			$output .= '<span class="give-description">' . esc_html( $args['desc'] ) . '</span>';
+		if ( ! empty($args['desc'])) {
+			$output .= '<span class="give-description">'.esc_html($args['desc']).'</span>';
 		}
 
-		$output .= '<input type="text" name="' . esc_attr( $args['name'] ) . '" id="' . esc_attr( $args['name'] ) . '" autocomplete="' . esc_attr( $args['autocomplete'] ) . '" value="' . esc_attr( $args['value'] ) . '" placeholder="' . esc_attr( $args['placeholder'] ) . '" class="' . $args['class'] . '" ' . $data . '' . $disabled . '/>';
+		$output .= '<input type="text" name="'.esc_attr($args['name']).'" id="'.esc_attr($args['name']).'" autocomplete="'.esc_attr($args['autocomplete']).'" value="'.esc_attr($args['value']).'" placeholder="'.esc_attr($args['placeholder']).'" class="'.$args['class'].'" '.$data.''.$disabled.'/>';
 
 		$output .= '</span>';
 
@@ -567,15 +567,15 @@ 
 	 *
 	 * @return string      The date picker.
 	 */
-	public function date_field( $args = array() ) {
+	public function date_field($args = array()) {
 
-		if ( empty( $args['class'] ) ) {
+		if (empty($args['class'])) {
 			$args['class'] = 'give_datepicker';
-		} elseif ( ! strpos( $args['class'], 'give_datepicker' ) ) {
+		} elseif ( ! strpos($args['class'], 'give_datepicker')) {
 			$args['class'] .= ' give_datepicker';
 		}
 
-		return $this->text( $args );
+		return $this->text($args);
 	}
 
 	/**
@@ -590,7 +590,7 @@ 
 	 *
 	 * @return string      The textarea.
 	 */
-	public function textarea( $args = array() ) {
+	public function textarea($args = array()) {
 		$defaults = array(
 			'name'     => 'textarea',
 			'value'    => null,
@@ -600,21 +600,21 @@ 
 			'disabled' => false
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
 		$disabled = '';
-		if ( $args['disabled'] ) {
+		if ($args['disabled']) {
 			$disabled = ' disabled="disabled"';
 		}
 
-		$output = '<span id="give-' . sanitize_key( $args['name'] ) . '-wrap">';
+		$output = '<span id="give-'.sanitize_key($args['name']).'-wrap">';
 
-		$output .= '<label class="give-label" for="give-' . sanitize_key( $args['name'] ) . '">' . esc_html( $args['label'] ) . '</label>';
+		$output .= '<label class="give-label" for="give-'.sanitize_key($args['name']).'">'.esc_html($args['label']).'</label>';
 
-		$output .= '<textarea name="' . esc_attr( $args['name'] ) . '" id="' . esc_attr( $args['name'] ) . '" class="' . $args['class'] . '"' . $disabled . '>' . esc_attr( $args['value'] ) . '</textarea>';
+		$output .= '<textarea name="'.esc_attr($args['name']).'" id="'.esc_attr($args['name']).'" class="'.$args['class'].'"'.$disabled.'>'.esc_attr($args['value']).'</textarea>';
 
-		if ( ! empty( $args['desc'] ) ) {
-			$output .= '<span class="give-description">' . esc_html( $args['desc'] ) . '</span>';
+		if ( ! empty($args['desc'])) {
+			$output .= '<span class="give-description">'.esc_html($args['desc']).'</span>';
 		}
 
 		$output .= '</span>';
@@ -634,12 +634,12 @@ 
 	 *
 	 * @return string      The text field with ajax search.
 	 */
-	public function ajax_user_search( $args = array() ) {
+	public function ajax_user_search($args = array()) {
 
 		$defaults = array(
 			'name'         => 'user_id',
 			'value'        => null,
-			'placeholder'  => esc_attr__( 'Enter username', 'give' ),
+			'placeholder'  => esc_attr__('Enter username', 'give'),
 			'label'        => null,
 			'desc'         => null,
 			'class'        => '',
@@ -648,13 +648,13 @@ 
 			'data'         => false
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		$args['class'] = 'give-ajax-user-search ' . $args['class'];
+		$args['class'] = 'give-ajax-user-search '.$args['class'];
 
 		$output = '<span class="give_user_search_wrap">';
-		$output .= $this->text( $args );
-		$output .= '<span class="give_user_search_results hidden"><a class="give-ajax-user-cancel" aria-label="' . esc_attr__( 'Cancel', 'give' ) . '" href="#">x</a><span></span></span>';
+		$output .= $this->text($args);
+		$output .= '<span class="give_user_search_results hidden"><a class="give-ajax-user-cancel" aria-label="'.esc_attr__('Cancel', 'give').'" href="#">x</a><span></span></span>';
 		$output .= '</span>';
 
 		return $output;

includes/admin/admin-actions.php

Doc Comments +1 added lines, -1 removed lines

@@ -98,7 +98,7 @@
  *
  * @since 1.8
  *
- * @return bool
+ * @return false|null
  */
 function give_redirect_to_clean_url_admin_pages() {
 	// Give admin pages.

Spacing +26 added lines, -26 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -24,53 +24,53 @@ 
 function give_hide_subscription_notices() {
 
 	// Hide subscription notices permanently.
-	if ( ! empty( $_GET['_give_hide_license_notices_permanently'] ) ) {
+	if ( ! empty($_GET['_give_hide_license_notices_permanently'])) {
 		$current_user = wp_get_current_user();
 
 		// check previously disabled notice ids.
-		$already_dismiss_notices = ( $already_dismiss_notices = get_user_meta( $current_user->ID, '_give_hide_license_notices_permanently', true ) )
+		$already_dismiss_notices = ($already_dismiss_notices = get_user_meta($current_user->ID, '_give_hide_license_notices_permanently', true))
 			? $already_dismiss_notices
 			: array();
 
 		// Get notice id.
-		$notice_id = sanitize_text_field( $_GET['_give_hide_license_notices_permanently'] );
+		$notice_id = sanitize_text_field($_GET['_give_hide_license_notices_permanently']);
 
-		if ( ! in_array( $notice_id, $already_dismiss_notices ) ) {
+		if ( ! in_array($notice_id, $already_dismiss_notices)) {
 			$already_dismiss_notices[] = $notice_id;
 		}
 
 		// Store subscription ids.
-		update_user_meta( $current_user->ID, '_give_hide_license_notices_permanently', $already_dismiss_notices );
+		update_user_meta($current_user->ID, '_give_hide_license_notices_permanently', $already_dismiss_notices);
 
 		// Redirect user.
-		wp_safe_redirect( remove_query_arg( '_give_hide_license_notices_permanently', $_SERVER['REQUEST_URI'] ) );
+		wp_safe_redirect(remove_query_arg('_give_hide_license_notices_permanently', $_SERVER['REQUEST_URI']));
 		exit();
 	}
 
 	// Hide subscription notices shortly.
-	if ( ! empty( $_GET['_give_hide_license_notices_shortly'] ) ) {
+	if ( ! empty($_GET['_give_hide_license_notices_shortly'])) {
 		$current_user = wp_get_current_user();
 
 		// Get notice id.
-		$notice_id = sanitize_text_field( $_GET['_give_hide_license_notices_shortly'] );
+		$notice_id = sanitize_text_field($_GET['_give_hide_license_notices_shortly']);
 
 		// Transient key name.
 		$transient_key = "_give_hide_license_notices_shortly_{$current_user->ID}_{$notice_id}";
 
-		if ( Give_Cache::get( $transient_key, true ) ) {
+		if (Give_Cache::get($transient_key, true)) {
 			return;
 		}
 
 		// Hide notice for 24 hours.
-		Give_Cache::set( $transient_key, true, DAY_IN_SECONDS, true );
+		Give_Cache::set($transient_key, true, DAY_IN_SECONDS, true);
 
 		// Redirect user.
-		wp_safe_redirect( remove_query_arg( '_give_hide_license_notices_shortly', $_SERVER['REQUEST_URI'] ) );
+		wp_safe_redirect(remove_query_arg('_give_hide_license_notices_shortly', $_SERVER['REQUEST_URI']));
 		exit();
 	}
 }
 
-add_action( 'admin_init', 'give_hide_subscription_notices' );
+add_action('admin_init', 'give_hide_subscription_notices');
 
 /**
  * Load wp editor by ajax.
@@ -78,19 +78,19 @@ 
  * @since 1.8
  */
 function give_load_wp_editor() {
-	if ( ! isset( $_POST['wp_editor'] ) ) {
+	if ( ! isset($_POST['wp_editor'])) {
 		die();
 	}
 
-	$wp_editor                     = json_decode( base64_decode( $_POST['wp_editor'] ), true );
+	$wp_editor                     = json_decode(base64_decode($_POST['wp_editor']), true);
 	$wp_editor[2]['textarea_name'] = $_POST['textarea_name'];
 
-	wp_editor( $wp_editor[0], $_POST['wp_editor_id'], $wp_editor[2] );
+	wp_editor($wp_editor[0], $_POST['wp_editor_id'], $wp_editor[2]);
 
 	die();
 }
 
-add_action( 'wp_ajax_give_load_wp_editor', 'give_load_wp_editor' );
+add_action('wp_ajax_give_load_wp_editor', 'give_load_wp_editor');
 
 
 /**
@@ -109,13 +109,13 @@ 
 	);
 
 	// Get current page.
-	$current_page = isset( $_GET['page'] ) ? esc_attr( $_GET['page'] ) : '';
+	$current_page = isset($_GET['page']) ? esc_attr($_GET['page']) : '';
 
 	// Bailout.
 	if (
-		empty( $current_page )
-		|| empty( $_GET['_wp_http_referer'] )
-		|| ! in_array( $current_page, $give_pages )
+		empty($current_page)
+		|| empty($_GET['_wp_http_referer'])
+		|| ! in_array($current_page, $give_pages)
 	) {
 		return false;
 	}
@@ -125,18 +125,18 @@ 
 	 *
 	 * @since 1.8
 	 */
-	$redirect = apply_filters( "give_validate_{$current_page}", true );
+	$redirect = apply_filters("give_validate_{$current_page}", true);
 
-	if ( $redirect ) {
+	if ($redirect) {
 		// Redirect.
 		wp_redirect(
 			remove_query_arg(
-				array( '_wp_http_referer', '_wpnonce' ),
-				wp_unslash( $_SERVER['REQUEST_URI'] )
+				array('_wp_http_referer', '_wpnonce'),
+				wp_unslash($_SERVER['REQUEST_URI'])
 			)
 		);
 		exit;
 	}
 }
 
-add_action( 'admin_init', 'give_redirect_to_clean_url_admin_pages' );
+add_action('admin_init', 'give_redirect_to_clean_url_admin_pages');

includes/admin/class-admin-notices.php

Doc Comments +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
 	 *
 	 * @param WP_Admin_Bar $wp_admin_bar WP_Admin_Bar instance, passed by reference.
 	 *
-	 * @return bool
+	 * @return false|null
 	 */
 	public function give_admin_bar_menu( $wp_admin_bar ) {
 

Spacing +68 added lines, -68 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -33,9 +33,9 @@ 
 	 * @since 1.0
 	 */
 	public function __construct() {
-		add_action( 'admin_notices', array( $this, 'show_notices' ) );
-		add_action( 'give_dismiss_notices', array( $this, 'dismiss_notices' ) );
-		add_action( 'admin_bar_menu', array( $this, 'give_admin_bar_menu' ), 1000, 1 );
+		add_action('admin_notices', array($this, 'show_notices'));
+		add_action('give_dismiss_notices', array($this, 'dismiss_notices'));
+		add_action('admin_bar_menu', array($this, 'give_admin_bar_menu'), 1000, 1);
 
 		$this->notices = array(
 			'updated' => array(),
@@ -51,20 +51,20 @@ 
 	 *
 	 * @return bool
 	 */
-	public function give_admin_bar_menu( $wp_admin_bar ) {
+	public function give_admin_bar_menu($wp_admin_bar) {
 
-		if ( ! give_is_test_mode() || ! current_user_can( 'view_give_reports' ) ) {
+		if ( ! give_is_test_mode() || ! current_user_can('view_give_reports')) {
 			return false;
 		}
 
 		// Add the main siteadmin menu item.
-		$wp_admin_bar->add_menu( array(
+		$wp_admin_bar->add_menu(array(
 			'id'     => 'give-test-notice',
-			'href'   => admin_url( 'edit.php?post_type=give_forms&page=give-settings&tab=gateways' ),
+			'href'   => admin_url('edit.php?post_type=give_forms&page=give-settings&tab=gateways'),
 			'parent' => 'top-secondary',
-			'title'  => esc_html__( 'Give Test Mode Active', 'give' ),
-			'meta'   => array( 'class' => 'give-test-mode-active' ),
-		) );
+			'title'  => esc_html__('Give Test Mode Active', 'give'),
+			'meta'   => array('class' => 'give-test-mode-active'),
+		));
 
 	}
 
@@ -75,100 +75,100 @@ 
 	 */
 	public function show_notices() {
 
-		if ( ! give_test_ajax_works() && ! get_user_meta( get_current_user_id(), '_give_admin_ajax_inaccessible_dismissed', true ) && current_user_can( 'manage_give_settings' ) ) {
+		if ( ! give_test_ajax_works() && ! get_user_meta(get_current_user_id(), '_give_admin_ajax_inaccessible_dismissed', true) && current_user_can('manage_give_settings')) {
 			echo '<div class="error">';
-			echo '<p>' . __( 'Your site appears to be blocking the WordPress ajax interface. This may cause issues with Give.', 'give' ) . '</p>';
+			echo '<p>'.__('Your site appears to be blocking the WordPress ajax interface. This may cause issues with Give.', 'give').'</p>';
 			/* translators: %s: http://docs.givewp.com/ajax-blocked */
-			echo '<p>' . sprintf( __( 'Please see <a href="%s" target="_blank">this reference</a> for possible solutions.', 'give' ), esc_url( 'http://docs.givewp.com/ajax-blocked' ) ) . '</p>';
-			echo '<p><a href="' . add_query_arg( array(
+			echo '<p>'.sprintf(__('Please see <a href="%s" target="_blank">this reference</a> for possible solutions.', 'give'), esc_url('http://docs.givewp.com/ajax-blocked')).'</p>';
+			echo '<p><a href="'.add_query_arg(array(
 					'give_action' => 'dismiss_notices',
 					'give_notice' => 'admin_ajax_inaccessible',
-				) ) . '">' . __( 'Dismiss Notice', 'give' ) . '</a></p>';
+				)).'">'.__('Dismiss Notice', 'give').'</a></p>';
 			echo '</div>';
 		}
 
-		if ( isset( $_GET['give-message'] ) ) {
+		if (isset($_GET['give-message'])) {
 
 			// Donation reports errors.
-			if ( current_user_can( 'view_give_reports' ) ) {
-				switch ( $_GET['give-message'] ) {
+			if (current_user_can('view_give_reports')) {
+				switch ($_GET['give-message']) {
 					case 'donation_deleted' :
-						$this->notices['updated']['give-donation-deleted'] = __( 'The donation has been deleted.', 'give' );
+						$this->notices['updated']['give-donation-deleted'] = __('The donation has been deleted.', 'give');
 						break;
 					case 'email_sent' :
-						$this->notices['updated']['give-payment-sent'] = __( 'The donation receipt has been resent.', 'give' );
+						$this->notices['updated']['give-payment-sent'] = __('The donation receipt has been resent.', 'give');
 						break;
 					case 'refreshed-reports' :
-						$this->notices['updated']['give-refreshed-reports'] = __( 'The reports cache has been cleared.', 'give' );
+						$this->notices['updated']['give-refreshed-reports'] = __('The reports cache has been cleared.', 'give');
 						break;
 					case 'donation-note-deleted' :
-						$this->notices['updated']['give-donation-note-deleted'] = __( 'The donation note has been deleted.', 'give' );
+						$this->notices['updated']['give-donation-note-deleted'] = __('The donation note has been deleted.', 'give');
 						break;
 				}
 			}
 
 			// Give settings notices and errors.
-			if ( current_user_can( 'manage_give_settings' ) ) {
-				switch ( $_GET['give-message'] ) {
+			if (current_user_can('manage_give_settings')) {
+				switch ($_GET['give-message']) {
 					case 'settings-imported' :
-						$this->notices['updated']['give-settings-imported'] = __( 'The settings have been imported.', 'give' );
+						$this->notices['updated']['give-settings-imported'] = __('The settings have been imported.', 'give');
 						break;
 					case 'api-key-generated' :
-						$this->notices['updated']['give-api-key-generated'] = __( 'API keys have been generated.', 'give' );
+						$this->notices['updated']['give-api-key-generated'] = __('API keys have been generated.', 'give');
 						break;
 					case 'api-key-exists' :
-						$this->notices['error']['give-api-key-exists'] = __( 'The specified user already has API keys.', 'give' );
+						$this->notices['error']['give-api-key-exists'] = __('The specified user already has API keys.', 'give');
 						break;
 					case 'api-key-regenerated' :
-						$this->notices['updated']['give-api-key-regenerated'] = __( 'API keys have been regenerated.', 'give' );
+						$this->notices['updated']['give-api-key-regenerated'] = __('API keys have been regenerated.', 'give');
 						break;
 					case 'api-key-revoked' :
-						$this->notices['updated']['give-api-key-revoked'] = __( 'API keys have been revoked.', 'give' );
+						$this->notices['updated']['give-api-key-revoked'] = __('API keys have been revoked.', 'give');
 						break;
 					case 'sent-test-email' :
-						$this->notices['updated']['give-sent-test-email'] = __( 'The test email has been sent.', 'give' );
+						$this->notices['updated']['give-sent-test-email'] = __('The test email has been sent.', 'give');
 						break;
 					case 'matched-success-failure-page':
-						$this->notices['updated']['give-matched-success-failure-page'] = __( 'You cannot set the success and failed pages to the same page', 'give' );
+						$this->notices['updated']['give-matched-success-failure-page'] = __('You cannot set the success and failed pages to the same page', 'give');
 				}
 			}
 			// Payments errors.
-			if ( current_user_can( 'edit_give_payments' ) ) {
-				switch ( $_GET['give-message'] ) {
+			if (current_user_can('edit_give_payments')) {
+				switch ($_GET['give-message']) {
 					case 'note-added' :
-						$this->notices['updated']['give-note-added'] = __( 'The donation note has been added.', 'give' );
+						$this->notices['updated']['give-note-added'] = __('The donation note has been added.', 'give');
 						break;
 					case 'payment-updated' :
-						$this->notices['updated']['give-payment-updated'] = __( 'The donation has been updated.', 'give' );
+						$this->notices['updated']['give-payment-updated'] = __('The donation has been updated.', 'give');
 						break;
 				}
 			}
 
 			// Customer Notices.
-			if ( current_user_can( 'edit_give_payments' ) ) {
-				switch ( $_GET['give-message'] ) {
+			if (current_user_can('edit_give_payments')) {
+				switch ($_GET['give-message']) {
 					case 'customer-deleted' :
-						$this->notices['updated']['give-customer-deleted'] = __( 'The donor has been deleted.', 'give' );
+						$this->notices['updated']['give-customer-deleted'] = __('The donor has been deleted.', 'give');
 						break;
 
 					case 'email-added' :
-						$this->notices['updated']['give-customer-email-added'] = __( 'Donor email added', 'give' );
+						$this->notices['updated']['give-customer-email-added'] = __('Donor email added', 'give');
 						break;
 
 					case 'email-removed' :
-						$this->notices['updated']['give-customer-email-removed'] = __( 'Donor email removed', 'give' );
+						$this->notices['updated']['give-customer-email-removed'] = __('Donor email removed', 'give');
 						break;
 
 					case 'email-remove-failed' :
-						$this->notices['error']['give-customer-email-remove-failed'] = __( 'Failed to remove donor email', 'give' );
+						$this->notices['error']['give-customer-email-remove-failed'] = __('Failed to remove donor email', 'give');
 						break;
 
 					case 'primary-email-updated' :
-						$this->notices['updated']['give-customer-primary-email-updated'] = __( 'Primary email updated for donor', 'give' );
+						$this->notices['updated']['give-customer-primary-email-updated'] = __('Primary email updated for donor', 'give');
 						break;
 
 					case 'primary-email-failed' :
-						$this->notices['error']['give-customer-primary-email-failed'] = __( 'Failed to set primary email', 'give' );
+						$this->notices['error']['give-customer-primary-email-failed'] = __('Failed to set primary email', 'give');
 
 				}
 			}
@@ -176,19 +176,19 @@ 
 
 		$this->add_payment_bulk_action_notice();
 
-		if ( count( $this->notices['updated'] ) > 0 ) {
-			foreach ( $this->notices['updated'] as $notice => $message ) {
-				add_settings_error( 'give-notices', $notice, $message, 'updated' );
+		if (count($this->notices['updated']) > 0) {
+			foreach ($this->notices['updated'] as $notice => $message) {
+				add_settings_error('give-notices', $notice, $message, 'updated');
 			}
 		}
 
-		if ( count( $this->notices['error'] ) > 0 ) {
-			foreach ( $this->notices['error'] as $notice => $message ) {
-				add_settings_error( 'give-notices', $notice, $message, 'error' );
+		if (count($this->notices['error']) > 0) {
+			foreach ($this->notices['error'] as $notice => $message) {
+				add_settings_error('give-notices', $notice, $message, 'error');
 			}
 		}
 
-		settings_errors( 'give-notices' );
+		settings_errors('give-notices');
 
 	}
 
@@ -200,8 +200,8 @@ 
 	 * @return void
 	 */
 	function give_admin_addons_notices() {
-		add_settings_error( 'give-notices', 'give-addons-feed-error', __( 'There seems to be an issue with the server. Please try again in a few minutes.', 'give' ), 'error' );
-		settings_errors( 'give-notices' );
+		add_settings_error('give-notices', 'give-addons-feed-error', __('There seems to be an issue with the server. Please try again in a few minutes.', 'give'), 'error');
+		settings_errors('give-notices');
 	}
 
 
@@ -212,9 +212,9 @@ 
 	 * @return void
 	 */
 	function dismiss_notices() {
-		if ( isset( $_GET['give_notice'] ) ) {
-			update_user_meta( get_current_user_id(), '_give_' . $_GET['give_notice'] . '_dismissed', 1 );
-			wp_redirect( remove_query_arg( array( 'give_action', 'give_notice' ) ) );
+		if (isset($_GET['give_notice'])) {
+			update_user_meta(get_current_user_id(), '_give_'.$_GET['give_notice'].'_dismissed', 1);
+			wp_redirect(remove_query_arg(array('give_action', 'give_notice')));
 			exit;
 		}
 	}
@@ -229,24 +229,24 @@ 
 	 */
 	function add_payment_bulk_action_notice() {
 		if (
-			current_user_can( 'edit_give_payments' )
-			&& isset( $_GET['action'] )
-			&& ! empty( $_GET['action'] )
-			&& isset( $_GET['payment'] )
-			&& ! empty( $_GET['payment'] )
+			current_user_can('edit_give_payments')
+			&& isset($_GET['action'])
+			&& ! empty($_GET['action'])
+			&& isset($_GET['payment'])
+			&& ! empty($_GET['payment'])
 		) {
-			$payment_count = isset( $_GET['payment'] ) ? count( $_GET['payment'] ) : 0;
+			$payment_count = isset($_GET['payment']) ? count($_GET['payment']) : 0;
 
-			switch ( $_GET['action'] ) {
+			switch ($_GET['action']) {
 				case 'delete':
-					if ( $payment_count ) {
-						$this->notices['updated']['bulk_action_delete'] = sprintf( _n( 'Successfully deleted only one transaction.', 'Successfully deleted %d number of transactions.', $payment_count, 'give' ), $payment_count );
+					if ($payment_count) {
+						$this->notices['updated']['bulk_action_delete'] = sprintf(_n('Successfully deleted only one transaction.', 'Successfully deleted %d number of transactions.', $payment_count, 'give'), $payment_count);
 					}
 					break;
 
 				case 'resend-receipt':
-					if ( $payment_count ) {
-						$this->notices['updated']['bulk_action_resend_receipt'] = sprintf( _n( 'Successfully send email receipt to only one recipient.', 'Successfully send email receipts to %d recipients.', $payment_count, 'give' ), $payment_count );
+					if ($payment_count) {
+						$this->notices['updated']['bulk_action_resend_receipt'] = sprintf(_n('Successfully send email receipt to only one recipient.', 'Successfully send email receipts to %d recipients.', $payment_count, 'give'), $payment_count);
 					}
 					break;
 			}
@@ -267,7 +267,7 @@ 
 	 *
 	 * @return string
 	 */
-	public static function notice_html( $message, $type = 'updated' ) {
+	public static function notice_html($message, $type = 'updated') {
 		ob_start();
 		?>
 		<div class="<?php echo $type; ?> notice">

includes/admin/reporting/tools/class-give-tools-recount-customer-stats.php

Doc Comments +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
 	 * @access public
 	 * @since 1.5
 	 *
-	 * @return array $data The data for the CSV file
+	 * @return boolean $data The data for the CSV file
 	 */
 	public function get_data() {
 

Spacing +37 added lines, -37 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -55,32 +55,32 @@ 
 
 		$args = array(
 			'number'  => $this->per_step,
-			'offset'  => $this->per_step * ( $this->step - 1 ),
+			'offset'  => $this->per_step * ($this->step - 1),
 			'orderby' => 'id',
 			'order'   => 'DESC',
 		);
 
-		$customers = Give()->customers->get_customers( $args );
+		$customers = Give()->customers->get_customers($args);
 
-		if ( $customers ) {
+		if ($customers) {
 
-			$allowed_payment_status = apply_filters( 'give_recount_donors_donation_statuses', give_get_payment_status_keys() );
+			$allowed_payment_status = apply_filters('give_recount_donors_donation_statuses', give_get_payment_status_keys());
 
-			foreach ( $customers as $customer ) {
+			foreach ($customers as $customer) {
 
-				$attached_payment_ids = explode( ',', $customer->payment_ids );
+				$attached_payment_ids = explode(',', $customer->payment_ids);
 
 				$attached_args = array(
 					'post__in' => $attached_payment_ids,
-					'number'   => - 1,
+					'number'   => -1,
 					'status'   => $allowed_payment_status,
 				);
 
-				$attached_payments = (array) give_get_payments( $attached_args );
+				$attached_payments = (array) give_get_payments($attached_args);
 
 				$unattached_args = array(
 					'post__not_in' => $attached_payment_ids,
-					'number'       => - 1,
+					'number'       => -1,
 					'status'       => $allowed_payment_status,
 					'meta_query'   => array(
 						array(
@@ -91,29 +91,29 @@ 
 					),
 				);
 
-				$unattached_payments = give_get_payments( $unattached_args );
+				$unattached_payments = give_get_payments($unattached_args);
 
-				$payments = array_merge( $attached_payments, $unattached_payments );
+				$payments = array_merge($attached_payments, $unattached_payments);
 
 				$purchase_value = 0.00;
 				$purchase_count = 0;
 				$payment_ids    = array();
 
-				if ( $payments ) {
+				if ($payments) {
 
-					foreach ( $payments as $payment ) {
+					foreach ($payments as $payment) {
 
 						$should_process_payment = 'publish' == $payment->post_status ? true : false;
-						$should_process_payment = apply_filters( 'give_donor_recount_should_process_donation', $should_process_payment, $payment );
+						$should_process_payment = apply_filters('give_donor_recount_should_process_donation', $should_process_payment, $payment);
 
-						if ( true === $should_process_payment ) {
+						if (true === $should_process_payment) {
 
-							if ( apply_filters( 'give_customer_recount_should_increase_value', true, $payment ) ) {
-								$purchase_value += give_get_payment_amount( $payment->ID );
+							if (apply_filters('give_customer_recount_should_increase_value', true, $payment)) {
+								$purchase_value += give_get_payment_amount($payment->ID);
 							}
 
-							if ( apply_filters( 'give_customer_recount_should_increase_count', true, $payment ) ) {
-								$purchase_count ++;
+							if (apply_filters('give_customer_recount_should_increase_count', true, $payment)) {
+								$purchase_count++;
 							}
 
 						}
@@ -123,7 +123,7 @@ 
 
 				}
 
-				$payment_ids = implode( ',', $payment_ids );
+				$payment_ids = implode(',', $payment_ids);
 
 				$customer_update_data = array(
 					'purchase_count' => $purchase_count,
@@ -131,8 +131,8 @@ 
 					'payment_ids'    => $payment_ids,
 				);
 
-				$customer_instance = new Give_Customer( $customer->id );
-				$customer_instance->update( $customer_update_data );
+				$customer_instance = new Give_Customer($customer->id);
+				$customer_instance->update($customer_update_data);
 
 			}
 
@@ -152,21 +152,21 @@ 
 	public function get_percentage_complete() {
 
 		$args = array(
-			'number'  => - 1,
+			'number'  => -1,
 			'orderby' => 'id',
 			'order'   => 'DESC',
 		);
 
-		$customers = Give()->customers->get_customers( $args );
-		$total     = count( $customers );
+		$customers = Give()->customers->get_customers($args);
+		$total     = count($customers);
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( $this->per_step * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = (($this->per_step * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -180,7 +180,7 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 	}
 
 	/**
@@ -191,19 +191,19 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( esc_html__( 'You do not have permission to recount stats.', 'give' ), esc_html__( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(esc_html__('You do not have permission to recount stats.', 'give'), esc_html__('Error', 'give'), array('response' => 403));
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
 			$this->done    = true;
-			$this->message = esc_html__( 'Donor stats have been successfully recounted.', 'give' );
+			$this->message = esc_html__('Donor stats have been successfully recounted.', 'give');
 
 			return false;
 		}
@@ -213,10 +213,10 @@ 
 	 * Headers
 	 */
 	public function headers() {
-		ignore_user_abort( true );
+		ignore_user_abort(true);
 
-		if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-			set_time_limit( 0 );
+		if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+			set_time_limit(0);
 		}
 	}