WordImpress / Give

includes/class-give-db-logs.php

Spacing +71 added lines, -73 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -35,7 +35,7 @@ 
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
-		$this->table_name  = $wpdb->prefix . 'give_logs';
+		$this->table_name  = $wpdb->prefix.'give_logs';
 		$this->primary_key = 'ID';
 		$this->version     = '1.0';
 
@@ -73,8 +73,8 @@ 
 	 * @return array  Default column values.
 	 */
 	public function get_column_defaults() {
-		$log_create_date     = current_time( 'mysql', 0 );
-		$log_create_date_gmt = get_gmt_from_date( $log_create_date );
+		$log_create_date     = current_time('mysql', 0);
+		$log_create_date_gmt = get_gmt_from_date($log_create_date);
 
 		return array(
 			'ID'           => 0,
@@ -97,39 +97,39 @@ 
 	 *
 	 * @return bool|int
 	 */
-	public function add( $data = array() ) {
+	public function add($data = array()) {
 		// Valid table columns.
-		$table_columns = array_keys( $this->get_columns() );
+		$table_columns = array_keys($this->get_columns());
 
 		// Filter data.
-		foreach ( $data as $table_column => $column_data ) {
-			if ( ! in_array( $table_column, $table_columns ) ) {
-				unset( $data[ $table_column ] );
+		foreach ($data as $table_column => $column_data) {
+			if ( ! in_array($table_column, $table_columns)) {
+				unset($data[$table_column]);
 			}
 		}
 
 		// Set default values.
-		$current_log_data = wp_parse_args( $data, $this->get_column_defaults() );
+		$current_log_data = wp_parse_args($data, $this->get_column_defaults());
 
 		// Log parent should be an int.
-		$current_log_data['log_parent'] = absint( $current_log_data['log_parent'] );
+		$current_log_data['log_parent'] = absint($current_log_data['log_parent']);
 
 		// Get log.
-		$existing_log = $this->get_log_by( $current_log_data['ID'] );
+		$existing_log = $this->get_log_by($current_log_data['ID']);
 
 		// Update an existing log.
-		if ( $existing_log ) {
+		if ($existing_log) {
 
 			// Create new log data from existing and new log data.
-			$current_log_data = array_merge( $current_log_data, $existing_log );
+			$current_log_data = array_merge($current_log_data, $existing_log);
 
 			// Update log data.
-			$this->update( $current_log_data['ID'], $current_log_data );
+			$this->update($current_log_data['ID'], $current_log_data);
 
 			$log_id = $current_log_data['ID'];
 
 		} else {
-			$log_id = $this->insert( $current_log_data, 'log' );
+			$log_id = $this->insert($current_log_data, 'log');
 		}
 
 		return $log_id;
@@ -147,20 +147,20 @@ 
 	 *
 	 * @return bool|null|array
 	 */
-	public function get_log_by( $log_id = 0, $by = 'id' ) {
+	public function get_log_by($log_id = 0, $by = 'id') {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 		$log = null;
 
 		// Make sure $log_id is int.
-		$log_id = absint( $log_id );
+		$log_id = absint($log_id);
 
 		// Bailout.
-		if ( empty( $log_id ) ) {
+		if (empty($log_id)) {
 			return null;
 		}
 
-		switch ( $by ) {
+		switch ($by) {
 			case 'id':
 				$log = $wpdb->get_row(
 					$wpdb->prepare(
@@ -172,7 +172,7 @@ 
 				break;
 
 			default:
-				$log = apply_filters( "give_get_log_by_{$by}", $log, $log_id );
+				$log = apply_filters("give_get_log_by_{$by}", $log, $log_id);
 		}
 
 		return $log;
@@ -188,14 +188,14 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function get_logs( $args = array() ) {
+	public function get_logs($args = array()) {
 		global $wpdb;
-		$sql_query = $this->get_sql( $args );
+		$sql_query = $this->get_sql($args);
 		
 		// Get log.
-		if ( ! ( $logs = Give_Cache::get( 'give_logs', true, $sql_query ) ) ) {
-			$logs = $wpdb->get_results( $sql_query );
-			Give_Cache::set( 'give_logs', $logs, 3600, true, $sql_query );
+		if ( ! ($logs = Give_Cache::get('give_logs', true, $sql_query))) {
+			$logs = $wpdb->get_results($sql_query);
+			Give_Cache::set('give_logs', $logs, 3600, true, $sql_query);
 		}
 
 		return $logs;
@@ -212,21 +212,21 @@ 
 	 *
 	 * @return int
 	 */
-	public function count( $args = array() ) {
+	public function count($args = array()) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 		$args['number'] = - 1;
 		$args['fields'] = 'ID';
 		$args['count']  = true;
 
-		$sql_query = $this->get_sql( $args );
+		$sql_query = $this->get_sql($args);
 
-		if ( ! ( $count = Give_Cache::get( 'give_logs_count', true, $sql_query ) ) ) {
-			$count = $wpdb->get_var( $sql_query );
-			Give_Cache::set( 'give_logs_count', $count, 3600, true, $args );
+		if ( ! ($count = Give_Cache::get('give_logs_count', true, $sql_query))) {
+			$count = $wpdb->get_var($sql_query);
+			Give_Cache::set('give_logs_count', $count, 3600, true, $args);
 		}
 		
-		return absint( $count );
+		return absint($count);
 	}
 
 	/**
@@ -252,10 +252,10 @@ 
         PRIMARY KEY  (ID)
         ) {$charset_collate};";
 
-		require_once( ABSPATH . 'wp-admin/includes/upgrade.php' );
-		dbDelta( $sql );
+		require_once(ABSPATH.'wp-admin/includes/upgrade.php');
+		dbDelta($sql);
 
-		update_option( $this->table_name . '_db_version', $this->version );
+		update_option($this->table_name.'_db_version', $this->version);
 	}
 
 
@@ -269,7 +269,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_sql( $args = array() ) {
+	public function get_sql($args = array()) {
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
@@ -283,12 +283,12 @@ 
 			'count'   => false,
 		);
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
 		// validate params.
-		$this->validate_params( $args );
+		$this->validate_params($args);
 
-		if ( $args['number'] < 1 ) {
+		if ($args['number'] < 1) {
 			$args['number'] = 99999999999;
 		}
 
@@ -296,78 +296,78 @@ 
 		$where = '';
 
 		// Get sql query for meta.
-		if ( ! empty( $args['meta_query'] ) ) {
-			$meta_query_object = new WP_Meta_Query( $args['meta_query'] );
-			$meta_query        = $meta_query_object->get_sql( 'log', $this->table_name, 'id' );
-			$where             = implode( '', $meta_query );
+		if ( ! empty($args['meta_query'])) {
+			$meta_query_object = new WP_Meta_Query($args['meta_query']);
+			$meta_query        = $meta_query_object->get_sql('log', $this->table_name, 'id');
+			$where             = implode('', $meta_query);
 		}
 
 		$where .= ' WHERE 1=1 ';
 
 		// Set offset.
-		if ( empty( $args['offset'] ) && ( 0 < $args['paged'] ) ) {
-			$args['offset'] = $args['number'] * ( $args['paged'] - 1 );
+		if (empty($args['offset']) && (0 < $args['paged'])) {
+			$args['offset'] = $args['number'] * ($args['paged'] - 1);
 		}
 
 		// Set fields.
 		$fields = "{$this->table_name}.*";
-		if ( is_string( $args['fields'] ) && ( 'all' !== $args['fields'] ) ) {
+		if (is_string($args['fields']) && ('all' !== $args['fields'])) {
 			$fields = "{$this->table_name}.{$args['fields']}";
 		}
 
 		// Set count.
-		if ( $args['count'] ) {
+		if ($args['count']) {
 			$fields = "COUNT({$fields})";
 		}
 
 		// Specific logs.
-		if ( ! empty( $args['ID'] ) ) {
+		if ( ! empty($args['ID'])) {
 
-			if ( ! is_array( $args['ID'] ) ) {
-				$args['ID'] = explode( ',', $args['ID'] );
+			if ( ! is_array($args['ID'])) {
+				$args['ID'] = explode(',', $args['ID']);
 			}
-			$log_ids = implode( ',', array_map( 'intval', $args['ID'] ) );
+			$log_ids = implode(',', array_map('intval', $args['ID']));
 
 			$where .= " AND {$this->table_name}.ID IN( {$log_ids} ) ";
 		}
 
 		// Logs created for a specific date or in a date range
-		if ( ! empty( $args['date_query'] ) ) {
-			$date_query_object = new WP_Date_Query( $args['date_query'], "{$this->table_name}.log_date" );
-			$where             .= $date_query_object->get_sql();
+		if ( ! empty($args['date_query'])) {
+			$date_query_object = new WP_Date_Query($args['date_query'], "{$this->table_name}.log_date");
+			$where .= $date_query_object->get_sql();
 		}
 
 		// Logs create for specific parent.
-		if ( ! empty( $args['log_parent'] ) ) {
-			if ( ! is_array( $args['log_parent'] ) ) {
-				$args['log_parent'] = explode( ',', $args['log_parent'] );
+		if ( ! empty($args['log_parent'])) {
+			if ( ! is_array($args['log_parent'])) {
+				$args['log_parent'] = explode(',', $args['log_parent']);
 			}
-			$parent_ids = implode( ',', array_map( 'intval', $args['log_parent'] ) );
+			$parent_ids = implode(',', array_map('intval', $args['log_parent']));
 
 			$where .= " AND {$this->table_name}.log_parent IN( {$parent_ids} ) ";
 		}
 
 		// Logs create for specific type.
 		// is_array check is for backward compatibility.
-		if ( ! empty( $args['log_type'] ) && ! is_array( $args['log_type'] ) ) {
-			if ( ! is_array( $args['log_type'] ) ) {
-				$args['log_type'] = explode( ',', $args['log_type'] );
+		if ( ! empty($args['log_type']) && ! is_array($args['log_type'])) {
+			if ( ! is_array($args['log_type'])) {
+				$args['log_type'] = explode(',', $args['log_type']);
 			}
 
-			$log_types = implode( '\',\'', array_map( 'trim', $args['log_type'] ) );
+			$log_types = implode('\',\'', array_map('trim', $args['log_type']));
 
 			$where .= " AND {$this->table_name}.log_type IN( '{$log_types}' ) ";
 		}
 
-		$args['orderby'] = ! array_key_exists( $args['orderby'], $this->get_columns() ) ? 'log_date' : $args['orderby'];
+		$args['orderby'] = ! array_key_exists($args['orderby'], $this->get_columns()) ? 'log_date' : $args['orderby'];
 
-		$args['orderby'] = esc_sql( $args['orderby'] );
-		$args['order']   = esc_sql( $args['order'] );
+		$args['orderby'] = esc_sql($args['orderby']);
+		$args['order']   = esc_sql($args['order']);
 
 		return $wpdb->prepare(
 			"SELECT {$fields} FROM {$this->table_name} {$where} ORDER BY {$this->table_name}.{$args['orderby']} {$args['order']} LIMIT %d,%d;",
-			absint( $args['offset'] ),
-			absint( $args['number'] )
+			absint($args['offset']),
+			absint($args['number'])
 		);
 	}
 
@@ -382,13 +382,11 @@ 
 	 *
 	 * @return mixed
 	 */
-	private function validate_params( &$args ) {
+	private function validate_params(&$args) {
 		// fields params
 		$args['fields'] = 'ids' === $args['fields'] ?
-			'ID' :
-			$args['fields'];
-		$args['fields'] = array_key_exists( $args['fields'], $this->get_columns() ) ?
-			$args['fields'] :
-			'all';
+			'ID' : $args['fields'];
+		$args['fields'] = array_key_exists($args['fields'], $this->get_columns()) ?
+			$args['fields'] : 'all';
 	}
 }

includes/payments/class-payments-query.php

Spacing +148 added lines, -148 removed lines

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

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

Spacing +72 added lines, -72 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_General' ) ) :
+if ( ! class_exists('Give_Settings_General')) :
 
 	/**
 	 * Give_Settings_General.
@@ -27,7 +27,7 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'general';
-			$this->label = __( 'General', 'give' );
+			$this->label = __('General', 'give');
 
 			$this->default_tab = 'general-settings';
 
@@ -44,7 +44,7 @@ 
 			$settings        = array();
 			$current_section = give_get_current_setting_section();
 
-			switch ( $current_section ) {
+			switch ($current_section) {
 				case 'access-control':
 					$settings = array(
 						// Section 3: Access control.
@@ -54,20 +54,20 @@ 
 						),
 						array(
 							'id'      => 'session_lifetime',
-							'name'    => __( 'Session Lifetime', 'give' ),
-							'desc'    => __( 'The length of time a user\'s session is kept alive. Give starts a new session per user upon donation. Sessions allow donors to view their donation receipts without being logged in.', 'give' ),
+							'name'    => __('Session Lifetime', 'give'),
+							'desc'    => __('The length of time a user\'s session is kept alive. Give starts a new session per user upon donation. Sessions allow donors to view their donation receipts without being logged in.', 'give'),
 							'type'    => 'select',
 							'options' => array(
-								'86400'  => __( '24 Hours', 'give' ),
-								'172800' => __( '48 Hours', 'give' ),
-								'259200' => __( '72 Hours', 'give' ),
-								'604800' => __( '1 Week', 'give' ),
+								'86400'  => __('24 Hours', 'give'),
+								'172800' => __('48 Hours', 'give'),
+								'259200' => __('72 Hours', 'give'),
+								'604800' => __('1 Week', 'give'),
 							),
 						),
 						array(
 							'id'         => 'limit_display_donations',
-							'name'       => __( 'Limit Donations Displayed', 'give' ),
-							'desc'       => __( 'Adjusts the number of donations displayed to a non logged-in user when they attempt to access the Donation History page without an active session. For security reasons, it\'s best to leave this at 1-3 donations.', 'give' ),
+							'name'       => __('Limit Donations Displayed', 'give'),
+							'desc'       => __('Adjusts the number of donations displayed to a non logged-in user when they attempt to access the Donation History page without an active session. For security reasons, it\'s best to leave this at 1-3 donations.', 'give'),
 							'default'    => '1',
 							'type'       => 'number',
 							'css'        => 'width:50px;',
@@ -77,36 +77,36 @@ 
 							),
 						),
 						array(
-							'name'    => __( 'Email Access', 'give' ),
-							'desc'    => __( 'Would you like your donors to be able to access their donation history using only email? Donors whose sessions have expired and do not have an account may still access their donation history via a temporary email access link.', 'give' ),
+							'name'    => __('Email Access', 'give'),
+							'desc'    => __('Would you like your donors to be able to access their donation history using only email? Donors whose sessions have expired and do not have an account may still access their donation history via a temporary email access link.', 'give'),
 							'id'      => 'email_access',
 							'type'    => 'radio_inline',
 							'default' => 'disabled',
 							'options' => array(
-								'enabled'  => __( 'Enabled', 'give' ),
-								'disabled' => __( 'Disabled', 'give' ),
+								'enabled'  => __('Enabled', 'give'),
+								'disabled' => __('Disabled', 'give'),
 							),
 						),
 						array(
 							'id'      => 'recaptcha_key',
-							'name'    => __( 'reCAPTCHA Site Key', 'give' ),
+							'name'    => __('reCAPTCHA Site Key', 'give'),
 							/* translators: %s: https://www.google.com/recaptcha/ */
-							'desc'    => sprintf( __( 'If you would like to prevent spam on the email access form navigate to <a href="%s" target="_blank">the reCAPTCHA website</a> and sign up for an API key and paste your reCAPTCHA site key here. The reCAPTCHA uses Google\'s user-friendly single click verification method.', 'give' ), esc_url( 'http://docs.givewp.com/recaptcha' ) ),
+							'desc'    => sprintf(__('If you would like to prevent spam on the email access form navigate to <a href="%s" target="_blank">the reCAPTCHA website</a> and sign up for an API key and paste your reCAPTCHA site key here. The reCAPTCHA uses Google\'s user-friendly single click verification method.', 'give'), esc_url('http://docs.givewp.com/recaptcha')),
 							'default' => '',
 							'type'    => 'text',
 						),
 						array(
 							'id'      => 'recaptcha_secret',
-							'name'    => __( 'reCAPTCHA Secret Key', 'give' ),
-							'desc'    => __( 'Please paste the reCAPTCHA secret key here from your  reCAPTCHA API Keys panel.', 'give' ),
+							'name'    => __('reCAPTCHA Secret Key', 'give'),
+							'desc'    => __('Please paste the reCAPTCHA secret key here from your  reCAPTCHA API Keys panel.', 'give'),
 							'default' => '',
 							'type'    => 'text',
 						),
 						array(
-							'name'  => __( 'Access Control Docs Link', 'give' ),
+							'name'  => __('Access Control Docs Link', 'give'),
 							'id'    => 'access_control_docs_link',
-							'url'   => esc_url( 'http://docs.givewp.com/settings-access-control' ),
-							'title' => __( 'Access Control', 'give' ),
+							'url'   => esc_url('http://docs.givewp.com/settings-access-control'),
+							'title' => __('Access Control', 'give'),
 							'type'  => 'give_docs_link',
 						),
 						array(
@@ -117,8 +117,8 @@ 
 					break;
 
 				case 'currency-settings' :
-					$current_position_before = __( 'Before - %s&#x200e;10', 'give' );
-					$current_position_after  = __( 'After - 10%s&#x200f;', 'give' );
+					$current_position_before = __('Before - %s&#x200e;10', 'give');
+					$current_position_after  = __('After - 10%s&#x200f;', 'give');
 
 					$settings = array(
 						// Section 2: Currency
@@ -127,14 +127,14 @@ 
 							'id'   => 'give_title_general_settings_2',
 						),
 						array(
-							'name' => __( 'Currency Settings', 'give' ),
+							'name' => __('Currency Settings', 'give'),
 							'desc' => '',
 							'type' => 'give_title',
 							'id'   => 'give_title_general_settings_2',
 						),
 						array(
-							'name'    => __( 'Currency', 'give' ),
-							'desc'    => __( 'The donation currency. Note that some payment gateways have currency restrictions.', 'give' ),
+							'name'    => __('Currency', 'give'),
+							'desc'    => __('The donation currency. Note that some payment gateways have currency restrictions.', 'give'),
 							'id'      => 'currency',
 							'class'   => 'give-select-chosen',
 							'type'    => 'select',
@@ -142,51 +142,51 @@ 
 							'default' => 'USD',
 						),
 						array(
-							'name'       => __( 'Currency Position', 'give' ),
-							'desc'       => __( 'The position of the currency symbol.', 'give' ),
+							'name'       => __('Currency Position', 'give'),
+							'desc'       => __('The position of the currency symbol.', 'give'),
 							'id'         => 'currency_position',
 							'type'       => 'select',
 							'options'    => array(
 								/* translators: %s: currency symbol */
-								'before' => sprintf( $current_position_before, give_currency_symbol( give_get_currency() ) ),
+								'before' => sprintf($current_position_before, give_currency_symbol(give_get_currency())),
 								/* translators: %s: currency symbol */
-								'after'  => sprintf( __( $current_position_after, 'give' ), give_currency_symbol( give_get_currency() ) ),
+								'after'  => sprintf(__($current_position_after, 'give'), give_currency_symbol(give_get_currency())),
 							),
 							'default'    => 'before',
 							'attributes' => array(
-								'data-before-template' => sprintf( $current_position_before, '{currency_pos}' ),
-								'data-after-template' => sprintf( $current_position_after, '{currency_pos}' ),
+								'data-before-template' => sprintf($current_position_before, '{currency_pos}'),
+								'data-after-template' => sprintf($current_position_after, '{currency_pos}'),
 							),
 						),
 						array(
-							'name'    => __( 'Thousands Separator', 'give' ),
-							'desc'    => __( 'The symbol (typically , or .) to separate thousands.', 'give' ),
+							'name'    => __('Thousands Separator', 'give'),
+							'desc'    => __('The symbol (typically , or .) to separate thousands.', 'give'),
 							'id'      => 'thousands_separator',
 							'type'    => 'text',
 							'default' => ',',
 							'css'     => 'width:12em;',
 						),
 						array(
-							'name'    => __( 'Decimal Separator', 'give' ),
-							'desc'    => __( 'The symbol (usually , or .) to separate decimal points.', 'give' ),
+							'name'    => __('Decimal Separator', 'give'),
+							'desc'    => __('The symbol (usually , or .) to separate decimal points.', 'give'),
 							'id'      => 'decimal_separator',
 							'type'    => 'text',
 							'default' => '.',
 							'css'     => 'width:12em;',
 						),
 						array(
-							'name'    => __( 'Number of Decimals', 'give' ),
-							'desc'    => __( 'The number of decimal points displayed in amounts.', 'give' ),
+							'name'    => __('Number of Decimals', 'give'),
+							'desc'    => __('The number of decimal points displayed in amounts.', 'give'),
 							'id'      => 'number_decimals',
 							'type'    => 'text',
 							'default' => 2,
 							'css'     => 'width:12em;',
 						),
 						array(
-							'name'  => __( 'Currency Options Docs Link', 'give' ),
+							'name'  => __('Currency Options Docs Link', 'give'),
 							'id'    => 'currency_settings_docs_link',
-							'url'   => esc_url( 'http://docs.givewp.com/settings-currency' ),
-							'title' => __( 'Currency Settings', 'give' ),
+							'url'   => esc_url('http://docs.givewp.com/settings-currency'),
+							'title' => __('Currency Settings', 'give'),
 							'type'  => 'give_docs_link',
 						),
 						array(
@@ -202,7 +202,7 @@ 
 					$country = give_get_country();
 
 					// get the list of the states of which default country is selected.
-					$states = give_get_states( $country );
+					$states = give_get_states($country);
 
 					// Get the country list that does not have any states init.
 					$no_states_country = give_no_states_country_list();
@@ -214,46 +214,46 @@ 
 							'id'   => 'give_title_general_settings_1',
 						),
 						array(
-							'name' => __( 'General Settings', 'give' ),
+							'name' => __('General Settings', 'give'),
 							'desc' => '',
 							'type' => 'give_title',
 							'id'   => 'give_title_general_settings_1',
 						),
 						array(
-							'name'    => __( 'Success Page', 'give' ),
+							'name'    => __('Success Page', 'give'),
 							/* translators: %s: [give_receipt] */
-							'desc'    => sprintf( __( 'The page donors are sent to after completing their donations. The %s shortcode should be on this page.', 'give' ), '<code>[give_receipt]</code>' ),
+							'desc'    => sprintf(__('The page donors are sent to after completing their donations. The %s shortcode should be on this page.', 'give'), '<code>[give_receipt]</code>'),
 							'id'      => 'success_page',
 							'type'    => 'select',
-							'options' => give_cmb2_get_post_options( array(
+							'options' => give_cmb2_get_post_options(array(
 								'post_type'   => 'page',
 								'numberposts' => 999,
-							) ),
+							)),
 						),
 						array(
-							'name'    => __( 'Failed Donation Page', 'give' ),
-							'desc'    => __( 'The page donors are sent to if their donation is cancelled or fails.', 'give' ),
+							'name'    => __('Failed Donation Page', 'give'),
+							'desc'    => __('The page donors are sent to if their donation is cancelled or fails.', 'give'),
 							'id'      => 'failure_page',
 							'type'    => 'select',
-							'options' => give_cmb2_get_post_options( array(
+							'options' => give_cmb2_get_post_options(array(
 								'post_type'   => 'page',
 								'numberposts' => 999,
-							) ),
+							)),
 						),
 						array(
-							'name'    => __( 'Donation History Page', 'give' ),
+							'name'    => __('Donation History Page', 'give'),
 							/* translators: %s: [donation_history] */
-							'desc'    => sprintf( __( 'The page showing a complete donation history for the current user. The %s shortcode should be on this page.', 'give' ), '<code>[donation_history]</code>' ),
+							'desc'    => sprintf(__('The page showing a complete donation history for the current user. The %s shortcode should be on this page.', 'give'), '<code>[donation_history]</code>'),
 							'id'      => 'history_page',
 							'type'    => 'select',
-							'options' => give_cmb2_get_post_options( array(
+							'options' => give_cmb2_get_post_options(array(
 								'post_type'   => 'page',
 								'numberposts' => 999,
-							) ),
+							)),
 						),
 						array(
-							'name'    => __( 'Base Country', 'give' ),
-							'desc'    => __( 'The country your site operates from.', 'give' ),
+							'name'    => __('Base Country', 'give'),
+							'desc'    => __('The country your site operates from.', 'give'),
 							'id'      => 'base_country',
 							'type'    => 'select',
 							'options' => give_get_country_list(),
@@ -264,18 +264,18 @@ 
 						 * @since 1.8.14
 						 */
 						array(
-							'wrapper_class' => ( array_key_exists( $country, $no_states_country ) ? 'give-hidden' : '' ),
-							'name'          => __( 'Base State/Province', 'give' ),
-							'desc'          => __( 'The state/province your site operates from.', 'give' ),
+							'wrapper_class' => (array_key_exists($country, $no_states_country) ? 'give-hidden' : ''),
+							'name'          => __('Base State/Province', 'give'),
+							'desc'          => __('The state/province your site operates from.', 'give'),
 							'id'            => 'base_state',
-							'type'          => ( empty( $states ) ? 'text' : 'select' ),
+							'type'          => (empty($states) ? 'text' : 'select'),
 							'options'       => $states,
 						),
 						array(
-							'name'  => __( 'General Options Docs Link', 'give' ),
+							'name'  => __('General Options Docs Link', 'give'),
 							'id'    => 'general_options_docs_link',
-							'url'   => esc_url( 'http://docs.givewp.com/settings-general' ),
-							'title' => __( 'General Options', 'give' ),
+							'url'   => esc_url('http://docs.givewp.com/settings-general'),
+							'title' => __('General Options', 'give'),
 							'type'  => 'give_docs_link',
 						),
 						array(
@@ -290,7 +290,7 @@ 
 			 * Filter the general settings.
 			 * Backward compatibility: Please do not use this filter. This filter is deprecated in 1.8
 			 */
-			$settings = apply_filters( 'give_settings_general', $settings );
+			$settings = apply_filters('give_settings_general', $settings);
 
 			/**
 			 * Filter the settings.
@@ -299,7 +299,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;
@@ -313,12 +313,12 @@ 
 		 */
 		public function get_sections() {
 			$sections = array(
-				'general-settings'  => __( 'General', 'give' ),
-				'currency-settings' => __( 'Currency', 'give' ),
-				'access-control'    => __( 'Access Control', 'give' ),
+				'general-settings'  => __('General', 'give'),
+				'currency-settings' => __('Currency', 'give'),
+				'access-control'    => __('Access Control', 'give'),
 			);
 
-			return apply_filters( 'give_get_sections_' . $this->id, $sections );
+			return apply_filters('give_get_sections_'.$this->id, $sections);
 		}
 	}
 

includes/donors/class-give-donors-query.php

Spacing +53 added lines, -54 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -86,7 +86,7 @@ 
 	 *
 	 * @param  $args array The array of arguments that can be passed in and used for setting up this payment query.
 	 */
-	public function __construct( $args = array() ) {
+	public function __construct($args = array()) {
 		$defaults = array(
 			'number'     => 20,
 			'offset'     => 0,
@@ -104,7 +104,7 @@ 
 			// 'form'       => array(),
 		);
 
-		$this->args = wp_parse_args( $args, $defaults );
+		$this->args = wp_parse_args($args, $defaults);
 		$this->table_name      = Give()->donors->table_name;
 		$this->meta_table_name = Give()->donor_meta->table_name;
 		$this->meta_type       = Give()->donor_meta->meta_type;
@@ -146,21 +146,21 @@ 
 		 *
 		 * @param Give_Donors_Query $this Donors query object.
 		 */
-		do_action( 'give_pre_get_donors', $this );
+		do_action('give_pre_get_donors', $this);
 
-		$cache_key        = Give_Cache::get_key( 'give_donor', $this->get_sql(), false );
+		$cache_key = Give_Cache::get_key('give_donor', $this->get_sql(), false);
 
 		// Get donors from cache.
-		$this->donors = Give_Cache::get_db_query( $cache_key );
+		$this->donors = Give_Cache::get_db_query($cache_key);
 
-		if ( is_null( $this->donors  ) ) {
-			if ( empty( $this->args['count'] ) ) {
-				$this->donors = $wpdb->get_results( $this->get_sql() );
+		if (is_null($this->donors)) {
+			if (empty($this->args['count'])) {
+				$this->donors = $wpdb->get_results($this->get_sql());
 			} else {
-				$this->donors = $wpdb->get_var( $this->get_sql() );
+				$this->donors = $wpdb->get_var($this->get_sql());
 			}
 
-			Give_Cache::set_db_query( $cache_key, $this->donors );
+			Give_Cache::set_db_query($cache_key, $this->donors);
 		}
 
 
@@ -171,7 +171,7 @@ 
 		 *
 		 * @param Give_Donors_Query $this Donors query object.
 		 */
-		do_action( 'give_post_get_donors', $this );
+		do_action('give_post_get_donors', $this);
 
 		return $this->donors;
 	}
@@ -188,7 +188,7 @@ 
 	public function get_sql() {
 		global $wpdb;
 
-		if ( $this->args['number'] < 1 ) {
+		if ($this->args['number'] < 1) {
 			$this->args['number'] = 99999999999;
 		}
 
@@ -196,22 +196,22 @@ 
 
 
 		// Set offset.
-		if ( empty( $this->args['offset'] ) && ( 0 < $this->args['paged'] ) ) {
-			$this->args['offset'] = $this->args['number'] * ( $this->args['paged'] - 1 );
+		if (empty($this->args['offset']) && (0 < $this->args['paged'])) {
+			$this->args['offset'] = $this->args['number'] * ($this->args['paged'] - 1);
 		}
 
 		// Set fields.
 		$fields = "{$this->table_name}.*";
-		if ( ! empty( $this->args['fields'] ) && 'all' !== $this->args['fields'] ) {
-			if ( is_string( $this->args['fields'] ) ) {
+		if ( ! empty($this->args['fields']) && 'all' !== $this->args['fields']) {
+			if (is_string($this->args['fields'])) {
 				$fields = "{$this->table_name}.{$this->args['fields']}";
-			} elseif ( is_array( $this->args['fields'] ) ) {
-				$fields = "{$this->table_name}." . implode( " , {$this->table_name}.", $this->args['fields'] );
+			} elseif (is_array($this->args['fields'])) {
+				$fields = "{$this->table_name}.".implode(" , {$this->table_name}.", $this->args['fields']);
 			}
 		}
 
 		// Set count.
-		if ( ! empty( $this->args['count'] ) ) {
+		if ( ! empty($this->args['count'])) {
 			$fields = "COUNT({$this->table_name}.id)";
 		}
 
@@ -219,13 +219,13 @@ 
 
 		$sql = $wpdb->prepare(
 			"SELECT {$fields} FROM {$this->table_name} LIMIT %d,%d;",
-			absint( $this->args['offset'] ),
-			absint( $this->args['number'] )
+			absint($this->args['offset']),
+			absint($this->args['number'])
 		);
 
 		// $where, $orderby and order already prepared query they can generate notice if you re prepare them in above.
 		// WordPress consider LIKE condition as placeholder if start with s,f, or d.
-		$sql = str_replace( 'LIMIT', "{$where} {$orderby} {$this->args['order']} LIMIT", $sql );
+		$sql = str_replace('LIMIT', "{$where} {$orderby} {$this->args['order']} LIMIT", $sql);
 
 		return $sql;
 	}
@@ -243,15 +243,15 @@ 
 		$where = '';
 
 		// Get sql query for meta.
-		if ( ! empty( $this->args['meta_query'] ) ) {
-			$meta_query_object = new WP_Meta_Query( $this->args['meta_query'] );
+		if ( ! empty($this->args['meta_query'])) {
+			$meta_query_object = new WP_Meta_Query($this->args['meta_query']);
 			$meta_query        = $meta_query_object->get_sql(
 				$this->meta_type,
 				$this->table_name,
 				'id'
 			);
 
-			$where = implode( '', $meta_query );
+			$where = implode('', $meta_query);
 		}
 
 		$where .= 'WHERE 1=1 ';
@@ -261,7 +261,7 @@ 
 		$where .= $this->get_where_user();
 		$where .= $this->get_where_date();
 
-		return trim( $where );
+		return trim($where);
 		
 	}
 
@@ -279,17 +279,17 @@ 
 
 		$where = '';
 
-		if ( ! empty( $this->args['email'] ) ) {
+		if ( ! empty($this->args['email'])) {
 
-			if ( is_array( $this->args['email'] ) ) {
+			if (is_array($this->args['email'])) {
 
-				$emails_count       = count( $this->args['email'] );
-				$emails_placeholder = array_fill( 0, $emails_count, '%s' );
-				$emails             = implode( ', ', $emails_placeholder );
+				$emails_count       = count($this->args['email']);
+				$emails_placeholder = array_fill(0, $emails_count, '%s');
+				$emails             = implode(', ', $emails_placeholder);
 
-				$where .= $wpdb->prepare( "AND {$this->table_name}.email IN( $emails )", $this->args['email'] );
+				$where .= $wpdb->prepare("AND {$this->table_name}.email IN( $emails )", $this->args['email']);
 			} else {
-				$where .= $wpdb->prepare( "AND {$this->table_name}.email = %s", $this->args['email'] );
+				$where .= $wpdb->prepare("AND {$this->table_name}.email = %s", $this->args['email']);
 			}
 		}
 
@@ -309,11 +309,11 @@ 
 		$where = '';
 
 		// Specific donors.
-		if ( ! empty( $this->args['donor'] ) ) {
-			if ( ! is_array( $this->args['donor'] ) ) {
-				$this->args['donor'] = explode( ',', $this->args['donor'] );
+		if ( ! empty($this->args['donor'])) {
+			if ( ! is_array($this->args['donor'])) {
+				$this->args['donor'] = explode(',', $this->args['donor']);
 			}
-			$donor_ids = implode( ',', array_map( 'intval', $this->args['donor'] ) );
+			$donor_ids = implode(',', array_map('intval', $this->args['donor']));
 
 			$where .= "AND {$this->table_name}.id IN( {$donor_ids} )";
 		}
@@ -334,9 +334,9 @@ 
 		$where = '';
 
 		// Donors created for a specific date or in a date range
-		if ( ! empty( $this->args['date_query'] ) ) {
+		if ( ! empty($this->args['date_query'])) {
 			$date_query_object = new WP_Date_Query(
-				is_array( $this->args['date_query'] ) ? $this->args['date_query'] : wp_parse_args( $this->args['date_query'] ),
+				is_array($this->args['date_query']) ? $this->args['date_query'] : wp_parse_args($this->args['date_query']),
 				"{$this->table_name}.date_created"
 			);
 
@@ -371,11 +371,11 @@ 
 		$where = '';
 
 		// Donors created for a specific date or in a date range
-		if ( ! empty( $this->args['s'] ) && false !== strpos( $this->args['s'], ':' ) ) {
-			$search_parts = explode( ':', $this->args['s'] );
+		if ( ! empty($this->args['s']) && false !== strpos($this->args['s'], ':')) {
+			$search_parts = explode(':', $this->args['s']);
 
-			if ( ! empty( $search_parts[0] ) ) {
-				switch ( $search_parts[0] ) {
+			if ( ! empty($search_parts[0])) {
+				switch ($search_parts[0]) {
 					case 'name':
 						$where = "AND {$this->table_name}.name LIKE '%{$search_parts[1]}%'";
 						break;
@@ -403,11 +403,11 @@ 
 		$where = '';
 
 		// Donors create for specific wp user.
-		if ( ! empty( $this->args['user'] ) ) {
-			if ( ! is_array( $this->args['user'] ) ) {
-				$this->args['user'] = explode( ',', $this->args['user'] );
+		if ( ! empty($this->args['user'])) {
+			if ( ! is_array($this->args['user'])) {
+				$this->args['user'] = explode(',', $this->args['user']);
 			}
-			$user_ids = implode( ',', array_map( 'intval', $this->args['user'] ) );
+			$user_ids = implode(',', array_map('intval', $this->args['user']));
 
 			$where .= "AND {$this->table_name}.user_id IN( {$user_ids} )";
 		}
@@ -426,14 +426,13 @@ 
 	private function get_order_query() {
 		$table_columns = Give()->donors->get_columns();
 
-		$this->args['orderby'] = ! array_key_exists( $this->args['orderby'], $table_columns ) ?
-			'id' :
-			$this->args['orderby'];
+		$this->args['orderby'] = ! array_key_exists($this->args['orderby'], $table_columns) ?
+			'id' : $this->args['orderby'];
 
-		$this->args['orderby'] = esc_sql( $this->args['orderby'] );
-		$this->args['order']   = esc_sql( $this->args['order'] );
+		$this->args['orderby'] = esc_sql($this->args['orderby']);
+		$this->args['order']   = esc_sql($this->args['order']);
 
-		switch ( $table_columns[ $this->args['orderby'] ] ) {
+		switch ($table_columns[$this->args['orderby']]) {
 			case '%d':
 			case '%f':
 				$query = "ORDER BY {$this->table_name}.{$this->args['orderby']}+0";