WordImpress / Give

includes/class-give-background-updater.php

Spacing +37 added lines, -37 removed lines

@@ -11,7 +11,7 @@ 
  * @category Class
  * @author   WordImpress
  */
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -54,12 +54,12 @@ 
 	 * and data exists in the queue.
 	 */
 	public function handle_cron_healthcheck() {
-		if ( $this->is_process_running() ) {
+		if ($this->is_process_running()) {
 			// Background process already running.
 			return;
 		}
 
-		if ( $this->is_queue_empty() ) {
+		if ($this->is_queue_empty()) {
 			// No data to process.
 			$this->clear_scheduled_event();
 
@@ -73,8 +73,8 @@ 
 	 * Schedule fallback event.
 	 */
 	protected function schedule_event() {
-		if ( ! wp_next_scheduled( $this->cron_hook_identifier ) ) {
-			wp_schedule_event( time() + 10, $this->cron_interval_identifier, $this->cron_hook_identifier );
+		if ( ! wp_next_scheduled($this->cron_hook_identifier)) {
+			wp_schedule_event(time() + 10, $this->cron_interval_identifier, $this->cron_hook_identifier);
 		}
 	}
 
@@ -90,8 +90,8 @@ 
 	 *
 	 * @return mixed
 	 */
-	protected function task( $update ) {
-		if ( empty( $update ) ) {
+	protected function task($update) {
+		if (empty($update)) {
 			return false;
 		}
 
@@ -105,7 +105,7 @@ 
 				'update_info' => $update,
 				'step'        => 1,
 				'update'      => 1,
-				'heading'     => sprintf( 'Update %s of {update_count}', 1 ),
+				'heading'     => sprintf('Update %s of {update_count}', 1),
 				'percentage'  => $give_updates->percentage,
 			)
 		);
@@ -113,30 +113,30 @@ 
 		// Continuously skip update if previous update does not complete yet.
 		if (
 			$resume_update['update_info']['id'] !== $update['id'] &&
-			! give_has_upgrade_completed( $resume_update['update_info']['id'] )
+			! give_has_upgrade_completed($resume_update['update_info']['id'])
 		) {
 			$batch = Give_Updates::$background_updater->get_all_batch();
-			$batch_data_count = count( $batch->data );
+			$batch_data_count = count($batch->data);
 
-			if ( ! empty( $batch ) &&  1 === $batch_data_count ) {
-				if ( ! empty( $update['depend'] ) ) {
+			if ( ! empty($batch) && 1 === $batch_data_count) {
+				if ( ! empty($update['depend'])) {
 
 					$give_updates   = Give_Updates::get_instance();
-					$all_updates    = $give_updates->get_updates( 'database', 'all' );
-					$all_update_ids = wp_list_pluck( $all_updates, 'id' );
+					$all_updates    = $give_updates->get_updates('database', 'all');
+					$all_update_ids = wp_list_pluck($all_updates, 'id');
 
-					foreach ( $update['depend'] as $depend ) {
-						if ( give_has_upgrade_completed( $depend ) ) {
+					foreach ($update['depend'] as $depend) {
+						if (give_has_upgrade_completed($depend)) {
 							continue;
 						}
 
-						if ( in_array( $depend, $all_update_ids ) ) {
-							array_unshift( $batch->data, $all_updates[ array_search( $depend, $all_update_ids ) ] );
+						if (in_array($depend, $all_update_ids)) {
+							array_unshift($batch->data, $all_updates[array_search($depend, $all_update_ids)]);
 						}
 					}
 
-					if( $batch_data_count !== count( $batch->data ) ) {
-						update_option( $batch->key, $batch->data );
+					if ($batch_data_count !== count($batch->data)) {
+						update_option($batch->key, $batch->data);
 						$this->dispatch();
 
 						wp_die();
@@ -150,15 +150,15 @@ 
 
 		// Set params.
 		$resume_update['update_info'] = $update;
-		$give_updates->step           = absint( $resume_update['step'] );
-		$give_updates->update         = absint( $resume_update['update'] );
-		$is_parent_update_completed   = $give_updates->is_parent_updates_completed( $update );
+		$give_updates->step           = absint($resume_update['step']);
+		$give_updates->update         = absint($resume_update['update']);
+		$is_parent_update_completed   = $give_updates->is_parent_updates_completed($update);
 
 
 		// Skip update if dependency update does not complete yet.
-		if ( empty( $is_parent_update_completed ) ) {
+		if (empty($is_parent_update_completed)) {
 			// @todo: set error when you have only one update with invalid dependency
-			if ( ! is_null( $is_parent_update_completed ) ) {
+			if ( ! is_null($is_parent_update_completed)) {
 				return $update;
 			}
 
@@ -169,7 +169,7 @@ 
 		Give_Cache::disable();
 
 		// Run update.
-		if ( is_array( $update['callback'] ) ) {
+		if (is_array($update['callback'])) {
 			$update['callback'][0]->$update['callback'][1]();
 		} else {
 			$update['callback']();
@@ -178,21 +178,21 @@ 
 		// Set update info.
 		$doing_upgrade_args = array(
 			'update_info'      => $update,
-			'step'             => ++ $give_updates->step,
+			'step'             => ++$give_updates->step,
 			'update'           => $give_updates->update,
-			'heading'          => sprintf( 'Update %s of %s', $give_updates->update, get_option( 'give_db_update_count' ) ),
+			'heading'          => sprintf('Update %s of %s', $give_updates->update, get_option('give_db_update_count')),
 			'percentage'       => $give_updates->percentage,
 			'total_percentage' => $give_updates->get_db_update_processing_percentage(),
 		);
 
 		// Cache upgrade.
-		update_option( 'give_doing_upgrade', $doing_upgrade_args );
+		update_option('give_doing_upgrade', $doing_upgrade_args);
 
 		// Enable cache.
 		Give_Cache::enable();
 
 		// Check if current update completed or not.
-		if ( give_has_upgrade_completed( $update['id'] ) ) {
+		if (give_has_upgrade_completed($update['id'])) {
 			return false;
 		}
 
@@ -208,9 +208,9 @@ 
 	protected function complete() {
 		parent::complete();
 
-		delete_option( 'give_db_update_count' );
-		delete_option( 'give_doing_upgrade' );
-		add_option( 'give_show_db_upgrade_complete_notice', 1, '', 'no' );
+		delete_option('give_db_update_count');
+		delete_option('give_doing_upgrade');
+		add_option('give_show_db_upgrade_complete_notice', 1, '', 'no');
 	}
 
 	/**
@@ -219,18 +219,18 @@ 
 	 * @return int
 	 */
 	protected function get_memory_limit() {
-		if ( function_exists( 'ini_get' ) ) {
-			$memory_limit = ini_get( 'memory_limit' );
+		if (function_exists('ini_get')) {
+			$memory_limit = ini_get('memory_limit');
 		} else {
 			// Sensible default.
 			$memory_limit = '128M';
 		}
 
-		if ( ! $memory_limit || '-1' === $memory_limit ) {
+		if ( ! $memory_limit || '-1' === $memory_limit) {
 			// Unlimited, set to 32GB.
 			$memory_limit = '32000M';
 		}
 
-		return intval( $memory_limit ) * 1024 * 1024;
+		return intval($memory_limit) * 1024 * 1024;
 	}
 }

includes/admin/upgrades/class-give-updates.php

Spacing +130 added lines, -136 removed lines

@@ -80,34 +80,34 @@ 
 	 *
 	 * @param array $args
 	 */
-	public function register( $args ) {
+	public function register($args) {
 		$args_default = array(
 			'id'       => '',
 			'version'  => '',
 			'callback' => '',
 		);
 
-		$args = wp_parse_args( $args, $args_default );
+		$args = wp_parse_args($args, $args_default);
 
 		// You can only register database upgrade.
 		$args['type'] = 'database';
 
 		// Bailout.
 		if (
-			empty( $args['id'] ) ||
-			empty( $args['version'] ) ||
-			empty( $args['callback'] ) ||
-			! is_callable( $args['callback'] )
+			empty($args['id']) ||
+			empty($args['version']) ||
+			empty($args['callback']) ||
+			! is_callable($args['callback'])
 		) {
 			return;
 		}
 
 		// Change depend param to array.
-		if ( isset( $args['depend'] ) && is_string( $args['depend'] ) ) {
-			$args['depend'] = array( $args['depend'] );
+		if (isset($args['depend']) && is_string($args['depend'])) {
+			$args['depend'] = array($args['depend']);
 		}
 
-		$this->updates[ $args['type'] ][] = $args;
+		$this->updates[$args['type']][] = $args;
 	}
 
 	/**
@@ -118,7 +118,7 @@ 
 	 * @return static
 	 */
 	static function get_instance() {
-		if ( is_null( self::$instance ) ) {
+		if (is_null(self::$instance)) {
 			self::$instance = new self();
 		}
 
@@ -136,24 +136,24 @@ 
 		/**
 		 * Load file
 		 */
-		require_once GIVE_PLUGIN_DIR . 'includes/class-give-background-updater.php';
-		require_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/upgrade-functions.php';
+		require_once GIVE_PLUGIN_DIR.'includes/class-give-background-updater.php';
+		require_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/upgrade-functions.php';
 
 		self::$background_updater = new Give_Background_Updater();
 
 		/**
 		 * Setup hooks.
 		 */
-		add_action( 'init', array( $this, '__register_upgrade' ), 9999 );
-		add_action( 'give_set_upgrade_completed', array( $this, '__flush_resume_updates' ), 9999 );
-		add_action( 'wp_ajax_give_db_updates_info', array( $this, '__give_db_updates_info' ) );
-		add_action( 'wp_ajax_give_run_db_updates', array( $this, '__give_start_updating' ) );
-		add_action( 'admin_init', array( $this, '__redirect_admin' ) );
-		add_action( 'admin_notices', array( $this, '__show_notice' ) );
-
-		if ( is_admin() ) {
-			add_action( 'admin_init', array( $this, '__change_donations_label' ), 9999 );
-			add_action( 'admin_menu', array( $this, '__register_menu' ), 9999 );
+		add_action('init', array($this, '__register_upgrade'), 9999);
+		add_action('give_set_upgrade_completed', array($this, '__flush_resume_updates'), 9999);
+		add_action('wp_ajax_give_db_updates_info', array($this, '__give_db_updates_info'));
+		add_action('wp_ajax_give_run_db_updates', array($this, '__give_start_updating'));
+		add_action('admin_init', array($this, '__redirect_admin'));
+		add_action('admin_notices', array($this, '__show_notice'));
+
+		if (is_admin()) {
+			add_action('admin_init', array($this, '__change_donations_label'), 9999);
+			add_action('admin_menu', array($this, '__register_menu'), 9999);
 		}
 	}
 
@@ -167,12 +167,12 @@ 
 		$addons         = give_get_plugins();
 		$plugin_updates = get_plugin_updates();
 
-		foreach ( $addons as $key => $info ) {
-			if ( 'active' != $info['Status'] || 'add-on' != $info['Type'] || empty( $plugin_updates[ $key ] ) ) {
+		foreach ($addons as $key => $info) {
+			if ('active' != $info['Status'] || 'add-on' != $info['Type'] || empty($plugin_updates[$key])) {
 				continue;
 			}
 
-			$this->updates['plugin'][] = array_merge( $info, (array) $plugin_updates[ $key ] );
+			$this->updates['plugin'][] = array_merge($info, (array) $plugin_updates[$key]);
 		}
 	}
 
@@ -184,7 +184,7 @@ 
 	 * @access public
 	 */
 	public function __register_upgrade() {
-		if ( ! is_admin() ) {
+		if ( ! is_admin()) {
 			return;
 		}
 
@@ -193,7 +193,7 @@ 
 		 *
 		 * @since 1.8.12
 		 */
-		do_action( 'give_register_updates', $this );
+		do_action('give_register_updates', $this);
 	}
 
 	/**
@@ -206,21 +206,20 @@ 
 		global $menu;
 
 		// Bailout.
-		if ( empty( $menu ) || ! $this->get_total_update_count() ) {
+		if (empty($menu) || ! $this->get_total_update_count()) {
 			return;
 		}
 
-		foreach ( $menu as $index => $menu_item ) {
-			if ( 'edit.php?post_type=give_forms' !== $menu_item[2] ) {
+		foreach ($menu as $index => $menu_item) {
+			if ('edit.php?post_type=give_forms' !== $menu_item[2]) {
 				continue;
 			}
 
-			$menu[ $index ][0] = sprintf(
+			$menu[$index][0] = sprintf(
 				'%1$s <span class="update-plugins"><span class="plugin-count give-update-progress-count">%2$s%3$s</span></span>',
-				__( 'Donations', 'give' ),
+				__('Donations', 'give'),
 				$this->is_doing_updates() ?
-					$this->get_db_update_processing_percentage() :
-					$this->get_total_update_count(),
+					$this->get_db_update_processing_percentage() : $this->get_total_update_count(),
 				$this->is_doing_updates() ? '%' : ''
 			);
 
@@ -236,7 +235,7 @@ 
 	 */
 	public function __register_menu() {
 		// Bailout.
-		if( ! give_test_ajax_works() ) {
+		if ( ! give_test_ajax_works()) {
 			return;
 		}
 
@@ -244,17 +243,17 @@ 
 		$this->__register_plugin_addon_updates();
 
 		// Bailout.
-		if ( ! $this->get_total_update_count() ) {
+		if ( ! $this->get_total_update_count()) {
 			// Show complete update message if still on update setting page.
-			if ( isset( $_GET['page'] ) && 'give-updates' === $_GET['page'] ) {
+			if (isset($_GET['page']) && 'give-updates' === $_GET['page']) {
 				// Upgrades
 				add_submenu_page(
 					'edit.php?post_type=give_forms',
-					esc_html__( 'Give Updates Complete', 'give' ),
-					__( 'Updates', 'give' ),
+					esc_html__('Give Updates Complete', 'give'),
+					__('Updates', 'give'),
 					'manage_give_settings',
 					'give-updates',
-					array( $this, 'render_complete_page' )
+					array($this, 'render_complete_page')
 				);
 			}
 
@@ -264,18 +263,17 @@ 
 		// Upgrades
 		add_submenu_page(
 			'edit.php?post_type=give_forms',
-			esc_html__( 'Give Updates', 'give' ),
+			esc_html__('Give Updates', 'give'),
 			sprintf(
 				'%1$s <span class="update-plugins"><span class="plugin-count give-update-progress-count">%2$s%3$s</span></span>',
-				__( 'Updates', 'give' ),
+				__('Updates', 'give'),
 				$this->is_doing_updates() ?
-					$this->get_db_update_processing_percentage() :
-					$this->get_total_update_count(),
+					$this->get_db_update_processing_percentage() : $this->get_total_update_count(),
 				$this->is_doing_updates() ? '%' : ''
 			),
 			'manage_give_settings',
 			'give-updates',
-			array( $this, 'render_page' )
+			array($this, 'render_page')
 		);
 	}
 
@@ -290,13 +288,13 @@ 
 		// Show db upgrade completed notice.
 		if (
 			! wp_doing_ajax() &&
-			current_user_can( 'manage_give_settings' ) &&
-			get_option( 'give_show_db_upgrade_complete_notice' ) &&
-			! isset( $_GET['give-db-update-completed'] )
+			current_user_can('manage_give_settings') &&
+			get_option('give_show_db_upgrade_complete_notice') &&
+			! isset($_GET['give-db-update-completed'])
 		) {
-			delete_option( 'give_show_db_upgrade_complete_notice' );
+			delete_option('give_show_db_upgrade_complete_notice');
 
-			wp_redirect( add_query_arg( array( 'give-db-update-completed' => 'give_db_upgrade_completed' ) ) );
+			wp_redirect(add_query_arg(array('give-db-update-completed' => 'give_db_upgrade_completed')));
 			exit();
 		}
 	}
@@ -310,28 +308,28 @@ 
 	 */
 	public function __show_notice() {
 		// Bailout.
-		if ( ! current_user_can( 'manage_give_settings' ) || $this->is_doing_updates() ) {
+		if ( ! current_user_can('manage_give_settings') || $this->is_doing_updates()) {
 			return;
 		}
 
 		// Run DB updates.
-		if ( ! empty( $_GET['give-run-db-update'] ) ) {
+		if ( ! empty($_GET['give-run-db-update'])) {
 			$this->run_db_update();
 		}
 
 
 		// Bailout.
-		if ( isset( $_GET['page'] ) && 'give-updates' === $_GET['page'] ) {
+		if (isset($_GET['page']) && 'give-updates' === $_GET['page']) {
 			return;
 		}
 
 		// Show notice if ajax is not working.
-		if ( ! give_test_ajax_works() ) {
+		if ( ! give_test_ajax_works()) {
 			Give()->notices->register_notice(
 				array(
 					'id'          => 'give_db_upgrade_ajax_inaccessible',
 					'type'        => 'error',
-					'description' => __( 'Give needs to upgrade the database but cannot because AJAX is not functioning properly. Please contact your host and ask them to ensure admin-ajax.php is accessible.', 'give' ),
+					'description' => __('Give needs to upgrade the database but cannot because AJAX is not functioning properly. Please contact your host and ask them to ensure admin-ajax.php is accessible.', 'give'),
 					'show'        => true,
 				)
 			);
@@ -340,46 +338,46 @@ 
 		}
 
 		// Show db upgrade completed notice.
-		if ( ! empty( $_GET['give-db-update-completed'] ) ) {
-			Give()->notices->register_notice( array(
+		if ( ! empty($_GET['give-db-update-completed'])) {
+			Give()->notices->register_notice(array(
 				'id'          => 'give_db_upgrade_completed',
 				'type'        => 'updated',
-				'description' => __( 'Give database updates completed successfully. Thank you for updating to the latest version!', 'give' ),
+				'description' => __('Give database updates completed successfully. Thank you for updating to the latest version!', 'give'),
 				'show'        => true,
-			) );
+			));
 
 			// Start update.
-		} elseif ( ! empty( $_GET['give-run-db-update'] ) ) {
+		} elseif ( ! empty($_GET['give-run-db-update'])) {
 			$this->run_db_update();
 
 			// Show run the update notice.
-		} elseif ( $this->get_total_new_db_update_count() ) {
+		} elseif ($this->get_total_new_db_update_count()) {
 			ob_start();
 			?>
 			<p>
-				<strong><?php _e( 'Database Update', 'give' ); ?></strong>
-				&nbsp;&#8211;&nbsp;<?php _e( 'GiveWP needs to update your database to the latest version. The following process will make updates to your site\'s database. Please create a complete backup before proceeding.', 'give' ); ?>
+				<strong><?php _e('Database Update', 'give'); ?></strong>
+				&nbsp;&#8211;&nbsp;<?php _e('GiveWP needs to update your database to the latest version. The following process will make updates to your site\'s database. Please create a complete backup before proceeding.', 'give'); ?>
 			</p>
 			<p class="submit">
-				<a href="<?php echo esc_url( add_query_arg( array( 'give-run-db-update' => 1 ), admin_url( 'edit.php?post_type=give_forms&page=give-updates' ) ) ); ?>" class="button button-primary give-run-update-now">
-					<?php _e( 'Run the updater', 'woocommerce' ); ?>
+				<a href="<?php echo esc_url(add_query_arg(array('give-run-db-update' => 1), admin_url('edit.php?post_type=give_forms&page=give-updates'))); ?>" class="button button-primary give-run-update-now">
+					<?php _e('Run the updater', 'woocommerce'); ?>
 				</a>
 			</p>
 			<script type="text/javascript">
 				jQuery('.give-run-update-now').click('click', function () {
-					return window.confirm('<?php echo esc_js( __( 'It is strongly recommended that you backup your database before proceeding. Do you want to run the update now?', 'give' ) ); ?>'); // jshint ignore:line
+					return window.confirm('<?php echo esc_js(__('It is strongly recommended that you backup your database before proceeding. Do you want to run the update now?', 'give')); ?>'); // jshint ignore:line
 				});
 			</script>
 			<?php
 			$desc_html = ob_get_clean();
 
 
-			Give()->notices->register_notice( array(
+			Give()->notices->register_notice(array(
 				'id'          => 'give_upgrade_db',
 				'type'        => 'updated',
 				'dismissible' => false,
 				'description' => $desc_html,
-			) );
+			));
 		}
 	}
 
@@ -390,7 +388,7 @@ 
 	 * @access public
 	 */
 	public function render_complete_page() {
-		include_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/views/upgrades-complete.php';
+		include_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/views/upgrades-complete.php';
 	}
 
 	/**
@@ -400,7 +398,7 @@ 
 	 * @access public
 	 */
 	public function render_page() {
-		include_once GIVE_PLUGIN_DIR . 'includes/admin/upgrades/views/upgrades.php';
+		include_once GIVE_PLUGIN_DIR.'includes/admin/upgrades/views/upgrades.php';
 	}
 
 	/**
@@ -411,26 +409,26 @@ 
 	 */
 	private function run_db_update() {
 		// Bailout.
-		if ( $this->is_doing_updates() || ! $this->get_total_new_db_update_count() ) {
+		if ($this->is_doing_updates() || ! $this->get_total_new_db_update_count()) {
 			return;
 		}
 
-		$updates = $this->get_updates( 'database', 'new' );
+		$updates = $this->get_updates('database', 'new');
 
-		foreach ( $updates as $update ) {
-			self::$background_updater->push_to_queue( $update );
+		foreach ($updates as $update) {
+			self::$background_updater->push_to_queue($update);
 		}
 
-		add_option( 'give_db_update_count', count( $updates ), '', 'no' );
+		add_option('give_db_update_count', count($updates), '', 'no');
 
-		add_option( 'give_doing_upgrade', array(
+		add_option('give_doing_upgrade', array(
 			'update_info' => $updates[0],
 			'step'        => 1,
 			'update'      => 1,
-			'heading'     => sprintf( 'Update %s of %s', 1, count( $updates ) ),
+			'heading'     => sprintf('Update %s of %s', 1, count($updates)),
 			'percentage'  => 0,
 			'total_percentage'  => 0,
-		), '', 'no' );
+		), '', 'no');
 
 		self::$background_updater->save()->dispatch();
 	}
@@ -444,11 +442,11 @@ 
 	 */
 	public function __flush_resume_updates() {
 		//delete_option( 'give_doing_upgrade' );
-		update_option( 'give_version', preg_replace( '/[^0-9.].*/', '', GIVE_VERSION ) );
+		update_option('give_version', preg_replace('/[^0-9.].*/', '', GIVE_VERSION));
 
 		// Reset counter.
 		$this->step = $this->percentage = 0;
-		++ $this->update;
+		++$this->update;
 	}
 
 
@@ -463,7 +461,7 @@ 
 	public function __give_start_updating() {
 		// Check permission.
 		if (
-			! current_user_can( 'manage_give_settings' ) ||
+			! current_user_can('manage_give_settings') ||
 			$this->is_doing_updates()
 		) {
 			wp_send_json_error();
@@ -471,7 +469,7 @@ 
 
 		// @todo: validate nonce
 		// @todo: set http method to post
-		if ( empty( $_POST['run_db_update'] ) ) {
+		if (empty($_POST['run_db_update'])) {
 			wp_send_json_error();
 		}
 
@@ -490,21 +488,21 @@ 
 	 * @return string
 	 */
 	public function __give_db_updates_info() {
-		$update_info   = get_option( 'give_doing_upgrade' );
+		$update_info   = get_option('give_doing_upgrade');
 		$response_type = '';
 
-		if ( empty( $update_info ) && ! $this->get_pending_db_update_count() ) {
-			$update_info   = array(
-				'message'    => __( 'Give database updates completed successfully. Thank you for updating to the latest version!', 'give' ),
-				'heading'    => __( 'Updates Completed.', 'give' ),
+		if (empty($update_info) && ! $this->get_pending_db_update_count()) {
+			$update_info = array(
+				'message'    => __('Give database updates completed successfully. Thank you for updating to the latest version!', 'give'),
+				'heading'    => __('Updates Completed.', 'give'),
 				'percentage' => 0,
 			);
 			$response_type = 'success';
 
-			delete_option( 'give_show_db_upgrade_complete_notice' );
+			delete_option('give_show_db_upgrade_complete_notice');
 		}
 
-		$this->send_ajax_response( $update_info, $response_type );
+		$this->send_ajax_response($update_info, $response_type);
 	}
 
 	/**
@@ -516,7 +514,7 @@ 
 	 * @param        $data
 	 * @param string $type
 	 */
-	public function send_ajax_response( $data, $type = '' ) {
+	public function send_ajax_response($data, $type = '') {
 		$default = array(
 			'message'    => '',
 			'heading'    => '',
@@ -526,24 +524,24 @@ 
 		);
 
 		// Set data.
-		$data = wp_parse_args( $data, $default );
+		$data = wp_parse_args($data, $default);
 
 		// Enable cache.
 		Give_Cache::enable();
 
-		switch ( $type ) {
+		switch ($type) {
 			case 'success':
-				wp_send_json_success( $data );
+				wp_send_json_success($data);
 				break;
 
 			case 'error':
-				wp_send_json_error( $data );
+				wp_send_json_error($data);
 				break;
 
 			default:
-				wp_send_json( array(
+				wp_send_json(array(
 					'data' => $data,
-				) );
+				));
 				break;
 		}
 	}
@@ -557,12 +555,12 @@ 
 	 * @param $total
 	 * @param $current_total
 	 */
-	public function set_percentage( $total, $current_total ) {
+	public function set_percentage($total, $current_total) {
 		// Set percentage.
-		$this->percentage = $total ? ( ( $current_total ) / $total ) * 100 : 0;
+		$this->percentage = $total ? (($current_total) / $total) * 100 : 0;
 
 		// Verify percentage.
-		$this->percentage = ( 100 < $this->percentage ) ? 100 : $this->percentage;
+		$this->percentage = (100 < $this->percentage) ? 100 : $this->percentage;
 	}
 
 	/**
@@ -575,23 +573,23 @@ 
 	 *
 	 * @return bool|null
 	 */
-	public function is_parent_updates_completed( $update ) {
+	public function is_parent_updates_completed($update) {
 		// Bailout.
-		if ( empty( $update['depend'] ) ) {
+		if (empty($update['depend'])) {
 			return true;
 		}
 
 		// Check if dependency is valid or not.
-		if ( ! $this->has_valid_dependency( $update ) ) {
-			error_log( print_r( 'exit 2', true ) . "\n", 3, WP_CONTENT_DIR . '/debug_new.log' );
+		if ( ! $this->has_valid_dependency($update)) {
+			error_log(print_r('exit 2', true)."\n", 3, WP_CONTENT_DIR.'/debug_new.log');
 			return null;
 		}
 
 		$is_dependency_completed = true;
 
-		foreach ( $update['depend'] as $depend ) {
+		foreach ($update['depend'] as $depend) {
 
-			if ( ! give_has_upgrade_completed( $depend ) ) {
+			if ( ! give_has_upgrade_completed($depend)) {
 				$is_dependency_completed = false;
 				break;
 			}
@@ -608,7 +606,7 @@ 
 	 * @return bool
 	 */
 	public function is_doing_updates() {
-		return (bool) get_option( 'give_doing_upgrade' );
+		return (bool) get_option('give_doing_upgrade');
 	}
 
 
@@ -622,7 +620,7 @@ 
 	 *
 	 * @return bool
 	 */
-	public function has_valid_dependency( $update ) {
+	public function has_valid_dependency($update) {
 		$is_valid_dependency = true;
 		// $update_ids          = wp_list_pluck( $this->get_updates( 'database', 'all' ), 'id' );
 		//
@@ -648,32 +646,32 @@ 
 	 *
 	 * @return array
 	 */
-	public function get_updates( $update_type = '', $status = 'all' ) {
+	public function get_updates($update_type = '', $status = 'all') {
 		// return all updates.
-		if ( empty( $update_type ) ) {
+		if (empty($update_type)) {
 			return $this->updates;
 		}
 
 		// Get specific update.
-		$updates = ! empty( $this->updates[ $update_type ] ) ? $this->updates[ $update_type ] : array();
+		$updates = ! empty($this->updates[$update_type]) ? $this->updates[$update_type] : array();
 
 		// Bailout.
-		if ( empty( $updates ) ) {
+		if (empty($updates)) {
 			return $updates;
 		}
 
-		switch ( $status ) {
+		switch ($status) {
 			case 'new':
 				// Remove already completed updates.
 				$completed_updates = give_get_completed_upgrades();
 
-				if ( ! empty( $completed_updates ) ) {
-					foreach ( $updates as $index => $update ) {
-						if ( in_array( $update['id'], $completed_updates ) ) {
-							unset( $updates[ $index ] );
+				if ( ! empty($completed_updates)) {
+					foreach ($updates as $index => $update) {
+						if (in_array($update['id'], $completed_updates)) {
+							unset($updates[$index]);
 						}
 					}
-					$updates = array_values( $updates );
+					$updates = array_values($updates);
 				}
 
 				break;
@@ -690,7 +688,7 @@ 
 	 * @return int
 	 */
 	public function get_total_plugin_update_count() {
-		return count( $this->get_updates( 'plugin' ) );
+		return count($this->get_updates('plugin'));
 	}
 
 	/**
@@ -705,7 +703,7 @@ 
 		$db_update_count     = $this->get_pending_db_update_count();
 		$plugin_update_count = $this->get_total_plugin_update_count();
 
-		return ( $db_update_count + $plugin_update_count );
+		return ($db_update_count + $plugin_update_count);
 	}
 
 	/**
@@ -717,7 +715,7 @@ 
 	 * @return int
 	 */
 	public function get_pending_db_update_count() {
-		return count( $this->get_updates( 'database', 'new' ) );
+		return count($this->get_updates('database', 'new'));
 	}
 
 	/**
@@ -729,7 +727,7 @@ 
 	 * @return int
 	 */
 	public function get_total_db_update_count() {
-		return count( $this->get_updates( 'database', 'all' ) );
+		return count($this->get_updates('database', 'all'));
 	}
 
 	/**
@@ -742,8 +740,7 @@ 
 	 */
 	public function get_total_new_db_update_count() {
 		return $this->is_doing_updates() ?
-			get_option( 'give_db_update_count' ) :
-			$this->get_pending_db_update_count();
+			get_option('give_db_update_count') : $this->get_pending_db_update_count();
 	}
 
 	/**
@@ -755,11 +752,10 @@ 
 	 * @return int
 	 */
 	public function get_running_db_update() {
-		$current_update = get_option( 'give_doing_upgrade' );
+		$current_update = get_option('give_doing_upgrade');
 
 		return $this->is_doing_updates() ?
-			$current_update['update'] :
-			1;
+			$current_update['update'] : 1;
 	}
 
 	/**
@@ -771,23 +767,21 @@ 
 	 */
 	public function get_db_update_processing_percentage() {
 		// Bailout.
-		if ( ! $this->get_total_new_db_update_count() ) {
+		if ( ! $this->get_total_new_db_update_count()) {
 			return 0;
 		}
 
-		$resume_update            = get_option( 'give_doing_upgrade' );
-		$update_count_percentages = ( ( $this->get_running_db_update() - 1 ) / $this->get_total_new_db_update_count() ) * 100;
-		$update_percentage_share  = ( 1 / $this->get_total_new_db_update_count() ) * 100;
-		$upgrade_percentage       = ( ( $resume_update['percentage'] * $update_percentage_share ) / 100 );
+		$resume_update            = get_option('give_doing_upgrade');
+		$update_count_percentages = (($this->get_running_db_update() - 1) / $this->get_total_new_db_update_count()) * 100;
+		$update_percentage_share  = (1 / $this->get_total_new_db_update_count()) * 100;
+		$upgrade_percentage       = (($resume_update['percentage'] * $update_percentage_share) / 100);
 
 		$final_percentage = $update_count_percentages + $upgrade_percentage;
 
 		return $this->is_doing_updates() ?
-			( absint( $final_percentage ) ?
-				absint( $final_percentage ) :
-				round( $final_percentage, 2 )
-			) :
-			0;
+			(absint($final_percentage) ?
+				absint($final_percentage) : round($final_percentage, 2)
+			) : 0;
 	}
 }
 

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/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";

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' :
-					$currency_position_before = __( 'Before - %s&#x200e;10', 'give' );
-					$currency_position_after  = __( 'After - 10%s&#x200f;', 'give' );
+					$currency_position_before = __('Before - %s&#x200e;10', 'give');
+					$currency_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( $currency_position_before, give_currency_symbol( give_get_currency() ) ),
+								'before' => sprintf($currency_position_before, give_currency_symbol(give_get_currency())),
 								/* translators: %s: currency symbol */
-								'after'  => sprintf( $currency_position_after, give_currency_symbol( give_get_currency() ) ),
+								'after'  => sprintf($currency_position_after, give_currency_symbol(give_get_currency())),
 							),
 							'default'    => 'before',
 							'attributes' => array(
-								'data-before-template' => sprintf( $currency_position_before, '{currency_pos}' ),
-								'data-after-template' => sprintf( $currency_position_after, '{currency_pos}' ),
+								'data-before-template' => sprintf($currency_position_before, '{currency_pos}'),
+								'data-after-template' => sprintf($currency_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/class-give-db-meta.php

Doc Comments +1 added lines, -1 removed lines

@@ -277,7 +277,7 @@
 	/**
 	 * Rename query clauses for new meta table
 	 *
-	 * @param $clause
+	 * @param string $clause
 	 * @param $filter
 	 *
 	 * @return mixed

Spacing +81 added lines, -81 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -75,40 +75,40 @@ 
 	 */
 	function __construct() {
 		// Bailout.
-		if ( empty( $this->supports ) || ! $this->is_custom_meta_table_active() ) {
+		if (empty($this->supports) || ! $this->is_custom_meta_table_active()) {
 			return;
 		}
 
-		if ( in_array( 'add_post_metadata', $this->supports ) ) {
-			add_filter( 'add_post_metadata', array( $this, '__add_meta' ), 0, 5 );
+		if (in_array('add_post_metadata', $this->supports)) {
+			add_filter('add_post_metadata', array($this, '__add_meta'), 0, 5);
 		}
 
-		if ( in_array( 'get_post_metadata', $this->supports ) ) {
-			add_filter( 'get_post_metadata', array( $this, '__get_meta' ), 0, 4 );
+		if (in_array('get_post_metadata', $this->supports)) {
+			add_filter('get_post_metadata', array($this, '__get_meta'), 0, 4);
 		}
 
-		if ( in_array( 'update_post_metadata', $this->supports ) ) {
-			add_filter( 'update_post_metadata', array( $this, '__update_meta' ), 0, 5 );
+		if (in_array('update_post_metadata', $this->supports)) {
+			add_filter('update_post_metadata', array($this, '__update_meta'), 0, 5);
 		}
 
-		if ( in_array( 'delete_post_metadata', $this->supports ) ) {
-			add_filter( 'delete_post_metadata', array( $this, '__delete_meta' ), 0, 5 );
+		if (in_array('delete_post_metadata', $this->supports)) {
+			add_filter('delete_post_metadata', array($this, '__delete_meta'), 0, 5);
 		}
 
-		if ( in_array( 'posts_where', $this->supports ) ) {
-			add_filter( 'posts_where', array( $this, '__rename_meta_table_name_in_query' ), 99999, 2 );
+		if (in_array('posts_where', $this->supports)) {
+			add_filter('posts_where', array($this, '__rename_meta_table_name_in_query'), 99999, 2);
 		}
 
-		if ( in_array( 'posts_join', $this->supports ) ) {
-			add_filter( 'posts_join', array( $this, '__rename_meta_table_name_in_query' ), 99999, 2 );
+		if (in_array('posts_join', $this->supports)) {
+			add_filter('posts_join', array($this, '__rename_meta_table_name_in_query'), 99999, 2);
 		}
 
-		if ( in_array( 'posts_groupby', $this->supports ) ) {
-			add_filter( 'posts_groupby', array( $this, '__rename_meta_table_name_in_query' ), 99999, 2 );
+		if (in_array('posts_groupby', $this->supports)) {
+			add_filter('posts_groupby', array($this, '__rename_meta_table_name_in_query'), 99999, 2);
 		}
 
-		if ( in_array( 'posts_orderby', $this->supports ) ) {
-			add_filter( 'posts_orderby', array( $this, '__rename_meta_table_name_in_query' ), 99999, 2 );
+		if (in_array('posts_orderby', $this->supports)) {
+			add_filter('posts_orderby', array($this, '__rename_meta_table_name_in_query'), 99999, 2);
 		}
 	}
 
@@ -126,16 +126,16 @@ 
 	 * @return  mixed                 Will be an array if $single is false. Will be value of meta data field if $single
 	 *                                is true.
 	 */
-	public function get_meta( $id = 0, $meta_key = '', $single = false ) {
-		$id = $this->sanitize_id( $id );
+	public function get_meta($id = 0, $meta_key = '', $single = false) {
+		$id = $this->sanitize_id($id);
 
 		// Bailout.
-		if ( ! $this->is_valid_post_type( $id ) ) {
+		if ( ! $this->is_valid_post_type($id)) {
 			return $this->check;
 		}
 
-		if ( $this->raw_result ) {
-			if ( ! ( $value = get_metadata( $this->meta_type, $id, $meta_key, false ) ) ) {
+		if ($this->raw_result) {
+			if ( ! ($value = get_metadata($this->meta_type, $id, $meta_key, false))) {
 				$value = '';
 			}
 
@@ -143,7 +143,7 @@ 
 			$this->raw_result = false;
 
 		} else {
-			$value = get_metadata( $this->meta_type, $id, $meta_key, $single );
+			$value = get_metadata($this->meta_type, $id, $meta_key, $single);
 		}
 
 		return $value;
@@ -165,18 +165,18 @@ 
 	 *
 	 * @return  int|bool                  False for failure. True for success.
 	 */
-	public function add_meta( $id = 0, $meta_key = '', $meta_value, $unique = false ) {
-		$id = $this->sanitize_id( $id );
+	public function add_meta($id = 0, $meta_key = '', $meta_value, $unique = false) {
+		$id = $this->sanitize_id($id);
 
 		// Bailout.
-		if ( ! $this->is_valid_post_type( $id ) ) {
+		if ( ! $this->is_valid_post_type($id)) {
 			return $this->check;
 		}
 
-		$meta_id = add_metadata( $this->meta_type, $id, $meta_key, $meta_value, $unique );
+		$meta_id = add_metadata($this->meta_type, $id, $meta_key, $meta_value, $unique);
 
-		if ( $meta_id ) {
-			$this->delete_cache( $id );
+		if ($meta_id) {
+			$this->delete_cache($id);
 		}
 
 		return $meta_id;
@@ -202,18 +202,18 @@ 
 	 *
 	 * @return  int|bool                  False on failure, true if success.
 	 */
-	public function update_meta( $id = 0, $meta_key = '', $meta_value, $prev_value = '' ) {
-		$id = $this->sanitize_id( $id );
+	public function update_meta($id = 0, $meta_key = '', $meta_value, $prev_value = '') {
+		$id = $this->sanitize_id($id);
 
 		// Bailout.
-		if ( ! $this->is_valid_post_type( $id ) ) {
+		if ( ! $this->is_valid_post_type($id)) {
 			return $this->check;
 		}
 
-		$meta_id = update_metadata( $this->meta_type, $id, $meta_key, $meta_value, $prev_value );
+		$meta_id = update_metadata($this->meta_type, $id, $meta_key, $meta_value, $prev_value);
 
-		if ( $meta_id ) {
-			$this->delete_cache( $id );
+		if ($meta_id) {
+			$this->delete_cache($id);
 		}
 
 		return $meta_id;
@@ -236,18 +236,18 @@ 
 	 *
 	 * @return  bool                  False for failure. True for success.
 	 */
-	public function delete_meta( $id = 0, $meta_key = '', $meta_value = '', $delete_all = '' ) {
-		$id = $this->sanitize_id( $id );
+	public function delete_meta($id = 0, $meta_key = '', $meta_value = '', $delete_all = '') {
+		$id = $this->sanitize_id($id);
 
 		// Bailout.
-		if ( ! $this->is_valid_post_type( $id ) ) {
+		if ( ! $this->is_valid_post_type($id)) {
 			return $this->check;
 		}
 
-		$is_meta_deleted = delete_metadata( $this->meta_type, $id, $meta_key, $meta_value, $delete_all );
+		$is_meta_deleted = delete_metadata($this->meta_type, $id, $meta_key, $meta_value, $delete_all);
 
-		if ( $is_meta_deleted ) {
-			$this->delete_cache( $id );
+		if ($is_meta_deleted) {
+			$this->delete_cache($id);
 		}
 
 		return $is_meta_deleted;
@@ -264,10 +264,10 @@ 
 	 *
 	 * @return string
 	 */
-	public function __rename_meta_table_name_in_query( $clause, $wp_query ) {
+	public function __rename_meta_table_name_in_query($clause, $wp_query) {
 		// Add new table to sql query.
-		if ( $this->is_post_type_query( $wp_query ) && ! empty( $wp_query->meta_query->queries ) ) {
-			$clause = $this->__rename_meta_table_name( $clause, current_filter() );
+		if ($this->is_post_type_query($wp_query) && ! empty($wp_query->meta_query->queries)) {
+			$clause = $this->__rename_meta_table_name($clause, current_filter());
 		}
 
 		return $clause;
@@ -282,39 +282,39 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function __rename_meta_table_name( $clause, $filter ){
+	public function __rename_meta_table_name($clause, $filter) {
 		global $wpdb;
 
-		$clause = str_replace( "{$wpdb->postmeta}.post_id", "{$this->table_name}.{$this->meta_type}_id", $clause );
-		$clause = str_replace( $wpdb->postmeta, $this->table_name, $clause );
+		$clause = str_replace("{$wpdb->postmeta}.post_id", "{$this->table_name}.{$this->meta_type}_id", $clause);
+		$clause = str_replace($wpdb->postmeta, $this->table_name, $clause);
 
-		switch( $filter ) {
+		switch ($filter) {
 			case 'posts_join':
-				$joins = array( 'INNER JOIN', 'LEFT JOIN' );
+				$joins = array('INNER JOIN', 'LEFT JOIN');
 
-				foreach ( $joins as $join ) {
-					if( false !== strpos( $clause, $join ) ) {
-						$clause = explode( $join, $clause );
+				foreach ($joins as $join) {
+					if (false !== strpos($clause, $join)) {
+						$clause = explode($join, $clause);
 
-						foreach ( $clause as $key => $clause_part ) {
-							if( empty( $clause_part ) ) {
+						foreach ($clause as $key => $clause_part) {
+							if (empty($clause_part)) {
 								continue;
 							}
 
-							preg_match( '/' . $wpdb->prefix . 'give_' . $this->meta_type . 'meta AS (.*) ON/', $clause_part, $alias_table_name );
+							preg_match('/'.$wpdb->prefix.'give_'.$this->meta_type.'meta AS (.*) ON/', $clause_part, $alias_table_name);
 
-							if( isset( $alias_table_name[1] ) ) {
-								$clause[$key] = str_replace( "{$alias_table_name[1]}.post_id", "{$alias_table_name[1]}.{$this->meta_type}_id", $clause_part );
+							if (isset($alias_table_name[1])) {
+								$clause[$key] = str_replace("{$alias_table_name[1]}.post_id", "{$alias_table_name[1]}.{$this->meta_type}_id", $clause_part);
 							}
 						}
 
-						$clause = implode( "{$join} ", $clause );
+						$clause = implode("{$join} ", $clause);
 					}
 				}
 				break;
 
 			case 'posts_where':
-				$clause = str_replace( array( 'mt2.post_id', 'mt1.post_id' ), array( "mt2.{$this->meta_type}_id", "mt1.{$this->meta_type}_id" ), $clause );
+				$clause = str_replace(array('mt2.post_id', 'mt1.post_id'), array("mt2.{$this->meta_type}_id", "mt1.{$this->meta_type}_id"), $clause);
 				break;
 		}
 
@@ -332,19 +332,19 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function is_post_type_query( $wp_query ) {
+	protected function is_post_type_query($wp_query) {
 		$status = false;
 
 		// Check if it is payment query.
-		if ( ! empty( $wp_query->query['post_type'] ) ) {
+		if ( ! empty($wp_query->query['post_type'])) {
 			if (
-				is_string( $wp_query->query['post_type'] ) &&
+				is_string($wp_query->query['post_type']) &&
 				$this->post_type === $wp_query->query['post_type']
 			) {
 				$status = true;
 			} elseif (
-				is_array( $wp_query->query['post_type'] ) &&
-				in_array( $this->post_type, $wp_query->query['post_type'] )
+				is_array($wp_query->query['post_type']) &&
+				in_array($this->post_type, $wp_query->query['post_type'])
 			) {
 				$status = true;
 			}
@@ -363,8 +363,8 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function is_valid_post_type( $ID ) {
-		return $ID && ( $this->post_type === get_post_type( $ID ) );
+	protected function is_valid_post_type($ID) {
+		return $ID && ($this->post_type === get_post_type($ID));
 	}
 
 	/**
@@ -390,8 +390,8 @@ 
 	 *
 	 * @return void
 	 */
-	private function delete_cache( $id, $meta_type = '' ) {
-		$meta_type = empty( $meta_type ) ? $this->meta_type : $meta_type;
+	private function delete_cache($id, $meta_type = '') {
+		$meta_type = empty($meta_type) ? $this->meta_type : $meta_type;
 
 		$group = array(
 			// 'form'    => 'give-forms',
@@ -401,8 +401,8 @@ 
 			// 'log'     => 'give-logs',
 		);
 
-		if ( array_key_exists( $meta_type, $group ) ) {
-			Give_Cache::delete_group( $id, $group[ $meta_type ] );
+		if (array_key_exists($meta_type, $group)) {
+			Give_Cache::delete_group($id, $group[$meta_type]);
 		}
 	}
 
@@ -417,8 +417,8 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function __call( $name, $arguments ) {
-		switch ( $name ) {
+	public function __call($name, $arguments) {
+		switch ($name) {
 			case '__add_meta':
 				$this->check = $arguments[0];
 				$id          = $arguments[1];
@@ -427,11 +427,11 @@ 
 				$unique      = $arguments[4];
 
 				// Bailout.
-				if ( ! $this->is_valid_post_type( $id ) ) {
+				if ( ! $this->is_valid_post_type($id)) {
 					return $this->check;
 				}
 
-				return $this->add_meta( $id, $meta_key, $meta_value, $unique );
+				return $this->add_meta($id, $meta_key, $meta_value, $unique);
 
 			case '__get_meta':
 				$this->check = $arguments[0];
@@ -440,13 +440,13 @@ 
 				$single      = $arguments[3];
 
 				// Bailout.
-				if ( ! $this->is_valid_post_type( $id ) ) {
+				if ( ! $this->is_valid_post_type($id)) {
 					return $this->check;
 				}
 
 				$this->raw_result = true;
 
-				return $this->get_meta( $id, $meta_key, $single );
+				return $this->get_meta($id, $meta_key, $single);
 
 			case '__update_meta':
 				$this->check = $arguments[0];
@@ -455,11 +455,11 @@ 
 				$meta_value  = $arguments[3];
 
 				// Bailout.
-				if ( ! $this->is_valid_post_type( $id ) ) {
+				if ( ! $this->is_valid_post_type($id)) {
 					return $this->check;
 				}
 
-				return $this->update_meta( $id, $meta_key, $meta_value );
+				return $this->update_meta($id, $meta_key, $meta_value);
 
 			case '__delete_meta':
 				$this->check = $arguments[0];
@@ -469,11 +469,11 @@ 
 				$delete_all  = $arguments[3];
 
 				// Bailout.
-				if ( ! $this->is_valid_post_type( $id ) ) {
+				if ( ! $this->is_valid_post_type($id)) {
 					return $this->check;
 				}
 
-				return $this->delete_meta( $id, $meta_key, $meta_value, $delete_all );
+				return $this->delete_meta($id, $meta_key, $meta_value, $delete_all);
 		}
 	}
 }
\ No newline at end of file