WordImpress / Give

includes/admin/emails/class-new-donor-register-email.php

Spacing +34 added lines, -34 removed lines

@@ -10,11 +10,11 @@ 
  */
 
 // Exit if access directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
-if ( ! class_exists( 'Give_New_Donor_Register_Email' ) ) :
+if ( ! class_exists('Give_New_Donor_Register_Email')) :
 
 	/**
 	 * Give_New_Donor_Register_Email
@@ -31,34 +31,34 @@ 
 		 * @since   2.0
 		 */
 		public function init() {
-			$this->load( array(
+			$this->load(array(
 				'id'                    => 'new-donor-register',
-				'label'                 => __( 'New User Registration', 'give' ),
-				'description'           => __( 'Sent to designated recipient(s) when a new user registers on the site via a donation form.', 'give' ),
+				'label'                 => __('New User Registration', 'give'),
+				'description'           => __('Sent to designated recipient(s) when a new user registers on the site via a donation form.', 'give'),
 				'has_recipient_field'   => true,
 				'notification_status'   => 'enabled',
 				'has_preview_header'    => true,
-				'email_tag_context'     => array( 'donor', 'general' ),
+				'email_tag_context'     => array('donor', 'general'),
 				'form_metabox_setting'  => false,
 				'default_email_subject' => sprintf(
 					/* translators: %s: site name */
-					esc_attr__( '[%s] New User Registration', 'give' ),
-					get_bloginfo( 'name' )
+					esc_attr__('[%s] New User Registration', 'give'),
+					get_bloginfo('name')
 				),
 				'default_email_message' => $this->get_default_email_message(),
-			) );
+			));
 
 			// Setup action hook.
 			add_action(
 				"give_{$this->config['id']}_email_notification",
-				array( $this, 'setup_email_notification' ),
+				array($this, 'setup_email_notification'),
 				10,
 				2
 			);
 
 			add_filter(
 				'give_email_preview_header',
-				array( $this, 'email_preview_header' ),
+				array($this, 'email_preview_header'),
 				10,
 				2
 			);
@@ -73,9 +73,9 @@ 
 		 * @return string
 		 */
 		function get_default_email_message() {
-			$message = esc_attr__( 'New user registration on your site {sitename}:', 'give' ) . "\r\n\r\n";
-			$message .= esc_attr__( 'Username: {username}', 'give' ) . "\r\n\r\n";
-			$message .= esc_attr__( 'Email: {user_email}', 'give' ) . "\r\n";
+			$message = esc_attr__('New user registration on your site {sitename}:', 'give')."\r\n\r\n";
+			$message .= esc_attr__('Username: {username}', 'give')."\r\n\r\n";
+			$message .= esc_attr__('Email: {user_email}', 'give')."\r\n";
 
 			/**
 			 * Filter the default email message
@@ -95,7 +95,7 @@ 
 		 * @since 2.0
 		 */
 		public function setup_email_data() {
-			Give()->emails->__set( 'heading', esc_html__( 'New User Registration', 'give' ) );
+			Give()->emails->__set('heading', esc_html__('New User Registration', 'give'));
 		}
 
 
@@ -110,12 +110,12 @@ 
 		 *
 		 * @return string
 		 */
-		public function setup_email_notification( $user_id, $user_data ) {
+		public function setup_email_notification($user_id, $user_data) {
 			$this->setup_email_data();
 
-			$this->send_email_notification( array(
+			$this->send_email_notification(array(
 				'user_id' => $user_id,
-			) );
+			));
 		}
 
 
@@ -129,42 +129,42 @@ 
 		 * @param Give_New_Donor_Register_Email $email
 		 * @return string
 		 */
-		public function email_preview_header( $email_preview_header, $email ) {
+		public function email_preview_header($email_preview_header, $email) {
 			// Bailout.
-			if ( $this->config['id'] !== $email->config['id'] ) {
+			if ($this->config['id'] !== $email->config['id']) {
 				return $email_preview_header;
 			}
 
 			// Payment receipt switcher
-			$user_id = give_check_variable( give_clean( $_GET ), 'isset', 0, 'user_id' );
+			$user_id = give_check_variable(give_clean($_GET), 'isset', 0, 'user_id');
 
 			// Get payments.
 			$donors  = new Give_API();
-			$donors  = give_check_variable( $donors->get_donors(), 'empty', array(), 'donors' );
+			$donors  = give_check_variable($donors->get_donors(), 'empty', array(), 'donors');
 			$options = array();
 
 			// Default option.
-			$options[0] = esc_html__( 'No donor(s) found.', 'give' );
+			$options[0] = esc_html__('No donor(s) found.', 'give');
 
 			// Provide nice human readable options.
-			if ( $donors ) {
-				$options[0] = esc_html__( '- Select a donor -', 'give' );
-				foreach ( $donors as $donor ) {
+			if ($donors) {
+				$options[0] = esc_html__('- Select a donor -', 'give');
+				foreach ($donors as $donor) {
 					// Exclude customers for which wp user not exist.
-					if ( ! $donor['info']['user_id'] ) {
+					if ( ! $donor['info']['user_id']) {
 						continue;
 					}
-					$options[ $donor['info']['user_id'] ] = esc_html( '#' . $donor['info']['donor_id'] . ' - ' . $donor['info']['email'] );
+					$options[$donor['info']['user_id']] = esc_html('#'.$donor['info']['donor_id'].' - '.$donor['info']['email']);
 				}
 			}
 
-			$request_url_data = wp_parse_url( $_SERVER['REQUEST_URI'] );
+			$request_url_data = wp_parse_url($_SERVER['REQUEST_URI']);
 			$query            = $request_url_data['query'];
 
 			// Remove user id query param if set from request url.
-			$query = remove_query_arg( array( 'user_id' ), $query );
+			$query = remove_query_arg(array('user_id'), $query);
 
-			$request_url = home_url( '/?' . str_replace( '', '', $query ) );
+			$request_url = home_url('/?'.str_replace('', '', $query));
 			?>
 			<script type="text/javascript">
 				function change_preview() {
@@ -198,12 +198,12 @@ 
 			<div class="give_preview_email_user_id_main">
 
 				<label for="give_preview_email_user_id" class="give_preview_email_user_id_label">
-					<?php echo esc_html__( 'Preview email with a donor:', 'give' ); ?>
+					<?php echo esc_html__('Preview email with a donor:', 'give'); ?>
 				</label>
 
 				<?php
 				// The select field with 100 latest transactions
-				echo Give()->html->select( array(
+				echo Give()->html->select(array(
 					'name'             => 'preview_email_user_id',
 					'selected'         => $user_id,
 					'id'               => 'give_preview_email_user_id',
@@ -213,7 +213,7 @@ 
 					'select_atts'      => 'onchange="change_preview()"',
 					'show_option_all'  => false,
 					'show_option_none' => false,
-				) );
+				));
 				?>
 				<!-- Closing tag-->
 			</div>

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/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 +84 added lines, -84 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);
 		}
 	}
 
@@ -498,9 +498,9 @@ 
 			KEY meta_key (meta_key({$this->min_index_length}))
 			) {$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);
 	}
 }
\ No newline at end of file

includes/class-give-donate-form.php

Spacing +121 added lines, -121 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -337,11 +337,11 @@ 
 	 * @param  int|bool $_id   Post id. Default is false.
 	 * @param  array    $_args Arguments passed.
 	 */
-	public function __construct( $_id = false, $_args = array() ) {
+	public function __construct($_id = false, $_args = array()) {
 
-		$donation_form = WP_Post::get_instance( $_id );
+		$donation_form = WP_Post::get_instance($_id);
 
-		return $this->setup_donation_form( $donation_form );
+		return $this->setup_donation_form($donation_form);
 	}
 
 	/**
@@ -354,23 +354,23 @@ 
 	 *
 	 * @return bool                   If the setup was successful or not.
 	 */
-	private function setup_donation_form( $donation_form ) {
+	private function setup_donation_form($donation_form) {
 
-		if ( ! is_object( $donation_form ) ) {
+		if ( ! is_object($donation_form)) {
 			return false;
 		}
 
-		if ( ! is_a( $donation_form, 'WP_Post' ) ) {
+		if ( ! is_a($donation_form, 'WP_Post')) {
 			return false;
 		}
 
-		if ( 'give_forms' !== $donation_form->post_type ) {
+		if ('give_forms' !== $donation_form->post_type) {
 			return false;
 		}
 
-		foreach ( $donation_form as $key => $value ) {
+		foreach ($donation_form as $key => $value) {
 
-			switch ( $key ) {
+			switch ($key) {
 
 				default:
 					$this->$key = $value;
@@ -394,16 +394,16 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function __get( $key ) {
+	public function __get($key) {
 
-		if ( method_exists( $this, 'get_' . $key ) ) {
+		if (method_exists($this, 'get_'.$key)) {
 
-			return call_user_func( array( $this, 'get_' . $key ) );
+			return call_user_func(array($this, 'get_'.$key));
 
 		} else {
 
 			/* translators: %s: property key */
-			return new WP_Error( 'give-form-invalid-property', sprintf( esc_html__( 'Can\'t get property %s.', 'give' ), $key ) );
+			return new WP_Error('give-form-invalid-property', sprintf(esc_html__('Can\'t get property %s.', 'give'), $key));
 
 		}
 
@@ -419,30 +419,30 @@ 
 	 *
 	 * @return bool|int    False if data isn't passed and class not instantiated for creation, or New Form ID.
 	 */
-	public function create( $data = array() ) {
+	public function create($data = array()) {
 
-		if ( $this->id != 0 ) {
+		if ($this->id != 0) {
 			return false;
 		}
 
 		$defaults = array(
 			'post_type'   => 'give_forms',
 			'post_status' => 'draft',
-			'post_title'  => __( 'New Donation Form', 'give' ),
+			'post_title'  => __('New Donation Form', 'give'),
 		);
 
-		$args = wp_parse_args( $data, $defaults );
+		$args = wp_parse_args($data, $defaults);
 
 		/**
 		 * Fired before a donation form is created
 		 *
 		 * @param array $args The post object arguments used for creation.
 		 */
-		do_action( 'give_form_pre_create', $args );
+		do_action('give_form_pre_create', $args);
 
-		$id = wp_insert_post( $args, true );
+		$id = wp_insert_post($args, true);
 
-		$donation_form = WP_Post::get_instance( $id );
+		$donation_form = WP_Post::get_instance($id);
 
 		/**
 		 * Fired after a donation form is created
@@ -450,9 +450,9 @@ 
 		 * @param int   $id   The post ID of the created item.
 		 * @param array $args The post object arguments used for creation.
 		 */
-		do_action( 'give_form_post_create', $id, $args );
+		do_action('give_form_post_create', $id, $args);
 
-		return $this->setup_donation_form( $donation_form );
+		return $this->setup_donation_form($donation_form);
 
 	}
 
@@ -477,7 +477,7 @@ 
 	 * @return string Donation form name.
 	 */
 	public function get_name() {
-		return get_the_title( $this->ID );
+		return get_the_title($this->ID);
 	}
 
 	/**
@@ -490,7 +490,7 @@ 
 	 */
 	public function get_price() {
 
-		if ( ! isset( $this->price ) ) {
+		if ( ! isset($this->price)) {
 
 			$this->price = give_maybe_sanitize_amount(
 				give_get_meta(
@@ -500,7 +500,7 @@ 
 				)
 			);
 
-			if ( ! $this->price ) {
+			if ( ! $this->price) {
 				$this->price = 0;
 			}
 
@@ -514,7 +514,7 @@ 
 		 * @param string     $price The donation form price.
 		 * @param string|int $id    The form ID.
 		 */
-		return apply_filters( 'give_get_set_price', $this->price, $this->ID );
+		return apply_filters('give_get_set_price', $this->price, $this->ID);
 	}
 
 	/**
@@ -527,17 +527,17 @@ 
 	 */
 	public function get_minimum_price() {
 
-		if ( ! isset( $this->minimum_price ) ) {
+		if ( ! isset($this->minimum_price)) {
 
-			$this->minimum_price = give_get_meta( $this->ID, '_give_custom_amount_minimum', true );
+			$this->minimum_price = give_get_meta($this->ID, '_give_custom_amount_minimum', true);
 
-			if ( ! $this->is_custom_price_mode() ) {
+			if ( ! $this->is_custom_price_mode()) {
 				$this->minimum_price = 0;
 			}
 
 		}
 
-		return apply_filters( 'give_get_set_minimum_price', $this->minimum_price, $this->ID );
+		return apply_filters('give_get_set_minimum_price', $this->minimum_price, $this->ID);
 	}
 
 	/**
@@ -550,9 +550,9 @@ 
 	 */
 	public function get_prices() {
 
-		if ( ! isset( $this->prices ) ) {
+		if ( ! isset($this->prices)) {
 
-			$this->prices = give_get_meta( $this->ID, '_give_donation_levels', true );
+			$this->prices = give_get_meta($this->ID, '_give_donation_levels', true);
 
 		}
 
@@ -564,7 +564,7 @@ 
 		 * @param array      $prices The array of mulit-level prices.
 		 * @param int|string $ID     The ID of the form.
 		 */
-		return apply_filters( 'give_get_donation_levels', $this->prices, $this->ID );
+		return apply_filters('give_get_donation_levels', $this->prices, $this->ID);
 
 	}
 
@@ -578,21 +578,21 @@ 
 	 */
 	public function get_goal() {
 
-		if ( ! isset( $this->goal ) ) {
+		if ( ! isset($this->goal)) {
 
-			if ( 'donation' === give_get_form_goal_format( $this->ID ) ) {
-				$this->goal = give_get_meta( $this->ID, '_give_number_of_donation_goal', true );
+			if ('donation' === give_get_form_goal_format($this->ID)) {
+				$this->goal = give_get_meta($this->ID, '_give_number_of_donation_goal', true);
 			} else {
-				$this->goal = give_get_meta( $this->ID, '_give_set_goal', true );
+				$this->goal = give_get_meta($this->ID, '_give_set_goal', true);
 			}
 
-			if ( ! $this->goal ) {
+			if ( ! $this->goal) {
 				$this->goal = 0;
 			}
 
 		}
 
-		return apply_filters( 'give_get_set_goal', $this->goal, $this->ID );
+		return apply_filters('give_get_set_goal', $this->goal, $this->ID);
 
 	}
 
@@ -606,10 +606,10 @@ 
 	 */
 	public function is_single_price_mode() {
 
-		$option = give_get_meta( $this->ID, '_give_price_option', true );
+		$option = give_get_meta($this->ID, '_give_price_option', true);
 		$ret    = 0;
 
-		if ( empty( $option ) || $option === 'set' ) {
+		if (empty($option) || $option === 'set') {
 			$ret = 1;
 		}
 
@@ -621,7 +621,7 @@ 
 		 * @param bool       $ret Is donation form in single price mode?
 		 * @param int|string $ID The ID of the donation form.
 		 */
-		return (bool) apply_filters( 'give_single_price_option_mode', $ret, $this->ID );
+		return (bool) apply_filters('give_single_price_option_mode', $ret, $this->ID);
 
 	}
 
@@ -635,10 +635,10 @@ 
 	 */
 	public function is_custom_price_mode() {
 
-		$option = give_get_meta( $this->ID, '_give_custom_amount', true );
+		$option = give_get_meta($this->ID, '_give_custom_amount', true);
 		$ret    = 0;
 
-		if ( give_is_setting_enabled( $option ) ) {
+		if (give_is_setting_enabled($option)) {
 			$ret = 1;
 		}
 
@@ -650,7 +650,7 @@ 
 		 * @param bool       $ret Is donation form in custom price mode?
 		 * @param int|string $ID  The ID of the donation form.
 		 */
-		return (bool) apply_filters( 'give_custom_price_option_mode', $ret, $this->ID );
+		return (bool) apply_filters('give_custom_price_option_mode', $ret, $this->ID);
 
 	}
 
@@ -664,20 +664,20 @@ 
 	 *
 	 * @return bool
 	 */
-	public function is_custom_price( $amount ) {
+	public function is_custom_price($amount) {
 		$result = false;
-		$amount = give_maybe_sanitize_amount( $amount );
+		$amount = give_maybe_sanitize_amount($amount);
 
-		if ( $this->is_custom_price_mode() ) {
+		if ($this->is_custom_price_mode()) {
 
-			if ( 'set' === $this->get_type() ) {
-				if ( $amount !== $this->get_price() ) {
+			if ('set' === $this->get_type()) {
+				if ($amount !== $this->get_price()) {
 					$result = true;
 				}
 
-			} elseif ( 'multi' === $this->get_type() ) {
-				$level_amounts = array_map( 'give_maybe_sanitize_amount', wp_list_pluck( $this->get_prices(), '_give_amount' ) );
-				$result        = ! in_array( $amount, $level_amounts );
+			} elseif ('multi' === $this->get_type()) {
+				$level_amounts = array_map('give_maybe_sanitize_amount', wp_list_pluck($this->get_prices(), '_give_amount'));
+				$result        = ! in_array($amount, $level_amounts);
 			}
 		}
 
@@ -690,7 +690,7 @@ 
 		 *
 		 * @since 1.8.18
 		 */
-		return (bool) apply_filters( 'give_is_custom_price', $result, $amount, $this->ID );
+		return (bool) apply_filters('give_is_custom_price', $result, $amount, $this->ID);
 	}
 
 	/**
@@ -705,10 +705,10 @@ 
 	 */
 	public function has_variable_prices() {
 
-		$option = give_get_meta( $this->ID, '_give_price_option', true );
+		$option = give_get_meta($this->ID, '_give_price_option', true);
 		$ret    = 0;
 
-		if ( $option === 'multi' ) {
+		if ($option === 'multi') {
 			$ret = 1;
 		}
 
@@ -718,7 +718,7 @@ 
 		 * @param bool       $ret Does donation form have variable prices?
 		 * @param int|string $ID  The ID of the donation form.
 		 */
-		return (bool) apply_filters( 'give_has_variable_prices', $ret, $this->ID );
+		return (bool) apply_filters('give_has_variable_prices', $ret, $this->ID);
 
 	}
 
@@ -732,17 +732,17 @@ 
 	 */
 	public function get_type() {
 
-		if ( ! isset( $this->type ) ) {
+		if ( ! isset($this->type)) {
 
-			$this->type = give_get_meta( $this->ID, '_give_price_option', true );
+			$this->type = give_get_meta($this->ID, '_give_price_option', true);
 
-			if ( empty( $this->type ) ) {
+			if (empty($this->type)) {
 				$this->type = 'set';
 			}
 
 		}
 
-		return apply_filters( 'give_get_form_type', $this->type, $this->ID );
+		return apply_filters('give_get_form_type', $this->type, $this->ID);
 
 	}
 
@@ -758,23 +758,23 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_form_classes( $args ) {
+	public function get_form_classes($args) {
 
-		$float_labels_option = give_is_float_labels_enabled( $args )
+		$float_labels_option = give_is_float_labels_enabled($args)
 			? 'float-labels-enabled'
 			: '';
 
-		$form_classes_array = apply_filters( 'give_form_classes', array(
+		$form_classes_array = apply_filters('give_form_classes', array(
 			'give-form',
-			'give-form-' . $this->ID,
-			'give-form-type-' . $this->get_type(),
+			'give-form-'.$this->ID,
+			'give-form-type-'.$this->get_type(),
 			$float_labels_option,
-		), $this->ID, $args );
+		), $this->ID, $args);
 
 		// Remove empty class names.
-		$form_classes_array = array_filter( $form_classes_array );
+		$form_classes_array = array_filter($form_classes_array);
 
-		return implode( ' ', $form_classes_array );
+		return implode(' ', $form_classes_array);
 
 	}
 
@@ -789,22 +789,22 @@ 
 	 *
 	 * @return string
 	 */
-	public function get_form_wrap_classes( $args ) {
+	public function get_form_wrap_classes($args) {
 		$custom_class = array(
 			'give-form-wrap',
 		);
 
-		if ( $this->is_close_donation_form() ) {
+		if ($this->is_close_donation_form()) {
 			$custom_class[] = 'give-form-closed';
-		} else{
-			$display_option = ( isset( $args['display_style'] ) && ! empty( $args['display_style'] ) )
+		} else {
+			$display_option = (isset($args['display_style']) && ! empty($args['display_style']))
 				? $args['display_style']
-				: give_get_meta( $this->ID, '_give_payment_display', true );
+				: give_get_meta($this->ID, '_give_payment_display', true);
 
 			$custom_class[] = "give-display-{$display_option}";
 
 			// If admin want to show only button for form then user inbuilt modal functionality.
-			if ( 'button' === $display_option ) {
+			if ('button' === $display_option) {
 				$custom_class[] = 'give-display-button-only';
 			}
 		}
@@ -815,10 +815,10 @@ 
 		 *
 		 * @since 1.0
 		 */
-		$form_wrap_classes_array = (array) apply_filters( 'give_form_wrap_classes', $custom_class, $this->ID, $args );
+		$form_wrap_classes_array = (array) apply_filters('give_form_wrap_classes', $custom_class, $this->ID, $args);
 
 
-		return implode( ' ', $form_wrap_classes_array );
+		return implode(' ', $form_wrap_classes_array);
 
 	}
 
@@ -833,7 +833,7 @@ 
 	public function is_set_type_donation_form() {
 		$form_type = $this->get_type();
 
-		return ( 'set' === $form_type ? true : false );
+		return ('set' === $form_type ? true : false);
 	}
 
 	/**
@@ -847,7 +847,7 @@ 
 	public function is_multi_type_donation_form() {
 		$form_type = $this->get_type();
 
-		return ( 'multi' === $form_type ? true : false );
+		return ('multi' === $form_type ? true : false);
 
 	}
 
@@ -861,15 +861,15 @@ 
 	 */
 	public function get_sales() {
 
-		if ( ! isset( $this->sales ) ) {
+		if ( ! isset($this->sales)) {
 
-			if ( '' == give_get_meta( $this->ID, '_give_form_sales', true ) ) {
-				add_post_meta( $this->ID, '_give_form_sales', 0 );
+			if ('' == give_get_meta($this->ID, '_give_form_sales', true)) {
+				add_post_meta($this->ID, '_give_form_sales', 0);
 			} // End if
 
-			$this->sales = give_get_meta( $this->ID, '_give_form_sales', true );
+			$this->sales = give_get_meta($this->ID, '_give_form_sales', true);
 
-			if ( $this->sales < 0 ) {
+			if ($this->sales < 0) {
 				// Never let sales be less than zero.
 				$this->sales = 0;
 			}
@@ -890,13 +890,13 @@ 
 	 *
 	 * @return int|false     New number of total sales.
 	 */
-	public function increase_sales( $quantity = 1 ) {
+	public function increase_sales($quantity = 1) {
 
-		$sales       = give_get_form_sales_stats( $this->ID );
-		$quantity    = absint( $quantity );
+		$sales       = give_get_form_sales_stats($this->ID);
+		$quantity    = absint($quantity);
 		$total_sales = $sales + $quantity;
 
-		if ( $this->update_meta( '_give_form_sales', $total_sales ) ) {
+		if ($this->update_meta('_give_form_sales', $total_sales)) {
 
 			$this->sales = $total_sales;
 
@@ -917,17 +917,17 @@ 
 	 *
 	 * @return int|false     New number of total sales.
 	 */
-	public function decrease_sales( $quantity = 1 ) {
+	public function decrease_sales($quantity = 1) {
 
-		$sales = give_get_form_sales_stats( $this->ID );
+		$sales = give_get_form_sales_stats($this->ID);
 
 		// Only decrease if not already zero
-		if ( $sales > 0 ) {
+		if ($sales > 0) {
 
-			$quantity    = absint( $quantity );
+			$quantity    = absint($quantity);
 			$total_sales = $sales - $quantity;
 
-			if ( $this->update_meta( '_give_form_sales', $total_sales ) ) {
+			if ($this->update_meta('_give_form_sales', $total_sales)) {
 
 				$this->sales = $sales;
 
@@ -951,15 +951,15 @@ 
 	 */
 	public function get_earnings() {
 
-		if ( ! isset( $this->earnings ) ) {
+		if ( ! isset($this->earnings)) {
 
-			if ( '' == give_get_meta( $this->ID, '_give_form_earnings', true ) ) {
-				add_post_meta( $this->ID, '_give_form_earnings', 0 );
+			if ('' == give_get_meta($this->ID, '_give_form_earnings', true)) {
+				add_post_meta($this->ID, '_give_form_earnings', 0);
 			}
 
-			$this->earnings = give_get_meta( $this->ID, '_give_form_earnings', true );
+			$this->earnings = give_get_meta($this->ID, '_give_form_earnings', true);
 
-			if ( $this->earnings < 0 ) {
+			if ($this->earnings < 0) {
 				// Never let earnings be less than zero
 				$this->earnings = 0;
 			}
@@ -980,12 +980,12 @@ 
 	 *
 	 * @return float|false
 	 */
-	public function increase_earnings( $amount = 0 ) {
+	public function increase_earnings($amount = 0) {
 
-		$earnings   = give_get_form_earnings_stats( $this->ID );
+		$earnings   = give_get_form_earnings_stats($this->ID);
 		$new_amount = $earnings + (float) $amount;
 
-		if ( $this->update_meta( '_give_form_earnings', $new_amount ) ) {
+		if ($this->update_meta('_give_form_earnings', $new_amount)) {
 
 			$this->earnings = $new_amount;
 
@@ -1007,16 +1007,16 @@ 
 	 *
 	 * @return float|false
 	 */
-	public function decrease_earnings( $amount ) {
+	public function decrease_earnings($amount) {
 
-		$earnings = give_get_form_earnings_stats( $this->ID );
+		$earnings = give_get_form_earnings_stats($this->ID);
 
-		if ( $earnings > 0 ) {
+		if ($earnings > 0) {
 			// Only decrease if greater than zero
 			$new_amount = $earnings - (float) $amount;
 
 
-			if ( $this->update_meta( '_give_form_earnings', $new_amount ) ) {
+			if ($this->update_meta('_give_form_earnings', $new_amount)) {
 
 				$this->earnings = $new_amount;
 
@@ -1040,22 +1040,22 @@ 
 	 *
 	 * @return bool
 	 */
-	public function is_free( $price_id = false ) {
+	public function is_free($price_id = false) {
 
 		$is_free          = false;
-		$variable_pricing = give_has_variable_prices( $this->ID );
+		$variable_pricing = give_has_variable_prices($this->ID);
 
-		if ( $variable_pricing && ! is_null( $price_id ) && $price_id !== false ) {
-			$price = give_get_price_option_amount( $this->ID, $price_id );
-		} elseif ( ! $variable_pricing ) {
-			$price = give_get_meta( $this->ID, '_give_set_price', true );
+		if ($variable_pricing && ! is_null($price_id) && $price_id !== false) {
+			$price = give_get_price_option_amount($this->ID, $price_id);
+		} elseif ( ! $variable_pricing) {
+			$price = give_get_meta($this->ID, '_give_set_price', true);
 		}
 
-		if ( isset( $price ) && (float) $price == 0 ) {
+		if (isset($price) && (float) $price == 0) {
 			$is_free = true;
 		}
 
-		return (bool) apply_filters( 'give_is_free_donation', $is_free, $this->ID, $price_id );
+		return (bool) apply_filters('give_is_free_donation', $is_free, $this->ID, $price_id);
 
 	}
 
@@ -1074,7 +1074,7 @@ 
 	 */
 	public function is_close_donation_form() {
 
-		$goal_format = give_get_form_goal_format( $this->ID );
+		$goal_format = give_get_form_goal_format($this->ID);
 
 		/**
 		 * Filter the close form result.
@@ -1084,9 +1084,9 @@ 
 		$is_close_form = apply_filters(
 			'give_is_close_donation_form',
 			(
-				give_is_setting_enabled( give_get_meta( $this->ID, '_give_goal_option', true ) ) &&
-				give_is_setting_enabled( give_get_meta( $this->ID, '_give_close_form_when_goal_achieved', true ) ) &&
-				( 'donation' === $goal_format ? $this->get_goal() <= $this->get_sales() : $this->get_goal() <= $this->get_earnings() )
+				give_is_setting_enabled(give_get_meta($this->ID, '_give_goal_option', true)) &&
+				give_is_setting_enabled(give_get_meta($this->ID, '_give_close_form_when_goal_achieved', true)) &&
+				('donation' === $goal_format ? $this->get_goal() <= $this->get_sales() : $this->get_goal() <= $this->get_earnings())
 			),
 			$this->ID
 		);
@@ -1105,17 +1105,17 @@ 
 	 *
 	 * @return bool                            The result of the update query.
 	 */
-	private function update_meta( $meta_key = '', $meta_value = '' ) {
+	private function update_meta($meta_key = '', $meta_value = '') {
 
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
 		// Bailout.
-		if ( empty( $meta_key ) ) {
+		if (empty($meta_key)) {
 			return false;
 		}
 
-		if ( give_update_meta( $this->ID, $meta_key, $meta_value  ) ) {
+		if (give_update_meta($this->ID, $meta_key, $meta_value)) {
 			return true;
 		}
 

includes/payments/functions.php

Doc Comments +4 added lines, -4 removed lines

@@ -565,7 +565,7 @@ 
  *
  * @since 1.0
  *
- * @return int $earnings Earnings
+ * @return double $earnings Earnings
  */
 function give_get_earnings_by_date( $day = null, $month_num, $year = null, $hour = null ) {
 	// This is getting deprecated soon. Use Give_Payment_Stats with the get_earnings() method instead.
@@ -864,7 +864,7 @@ 
  *
  * @since 1.0
  *
- * @return array $user_info User Info Meta Values.
+ * @return string $user_info User Info Meta Values.
  */
 function give_get_payment_meta_user_info( $payment_id ) {
 	$payment = new Give_Payment( $payment_id );
@@ -881,7 +881,7 @@ 
  *
  * @since 1.0
  *
- * @return int $form_id Form ID.
+ * @return string $form_id Form ID.
  */
 function give_get_payment_form_id( $payment_id ) {
 	$payment = new Give_Payment( $payment_id );
@@ -1729,7 +1729,7 @@ 
  *
  * Retrieves the form title and appends the level name if present.
  *
- * @param int|Give_Payment $donation Donation Data Object.
+ * @param Give_Payment $donation Donation Data Object.
  * @param array            $args     a. only_level = If set to true will only return the level name if multi-level enabled.
  *                                   b. separator  = The separator between the Form Title and the Donation Level.
  *

Spacing +381 added lines, -383 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -44,15 +44,15 @@ 
  *
  * @return array $payments Payments retrieved from the database
  */
-function give_get_payments( $args = array() ) {
+function give_get_payments($args = array()) {
 
 	// Fallback to post objects to ensure backwards compatibility.
-	if ( ! isset( $args['output'] ) ) {
+	if ( ! isset($args['output'])) {
 		$args['output'] = 'posts';
 	}
 
-	$args     = apply_filters( 'give_get_payments_args', $args );
-	$payments = new Give_Payments_Query( $args );
+	$args     = apply_filters('give_get_payments_args', $args);
+	$payments = new Give_Payments_Query($args);
 
 	return $payments->get_payments();
 }
@@ -67,48 +67,48 @@ 
  *
  * @return mixed
  */
-function give_get_payment_by( $field = '', $value = '' ) {
+function give_get_payment_by($field = '', $value = '') {
 
-	if ( empty( $field ) || empty( $value ) ) {
+	if (empty($field) || empty($value)) {
 		return false;
 	}
 
-	switch ( strtolower( $field ) ) {
+	switch (strtolower($field)) {
 
 		case 'id':
-			$payment = new Give_Payment( $value );
+			$payment = new Give_Payment($value);
 			$id      = $payment->ID;
 
-			if ( empty( $id ) ) {
+			if (empty($id)) {
 				return false;
 			}
 
 			break;
 
 		case 'key':
-			$payment = give_get_payments( array(
+			$payment = give_get_payments(array(
 				'meta_key'       => '_give_payment_purchase_key',
 				'meta_value'     => $value,
 				'posts_per_page' => 1,
 				'fields'         => 'ids',
-			) );
+			));
 
-			if ( $payment ) {
-				$payment = new Give_Payment( $payment[0] );
+			if ($payment) {
+				$payment = new Give_Payment($payment[0]);
 			}
 
 			break;
 
 		case 'payment_number':
-			$payment = give_get_payments( array(
+			$payment = give_get_payments(array(
 				'meta_key'       => '_give_payment_number',
 				'meta_value'     => $value,
 				'posts_per_page' => 1,
 				'fields'         => 'ids',
-			) );
+			));
 
-			if ( $payment ) {
-				$payment = new Give_Payment( $payment[0] );
+			if ($payment) {
+				$payment = new Give_Payment($payment[0]);
 			}
 
 			break;
@@ -117,7 +117,7 @@ 
 			return false;
 	}// End switch().
 
-	if ( $payment ) {
+	if ($payment) {
 		return $payment;
 	}
 
@@ -133,9 +133,9 @@ 
  *
  * @return int|bool Payment ID if payment is inserted, false otherwise.
  */
-function give_insert_payment( $payment_data = array() ) {
+function give_insert_payment($payment_data = array()) {
 
-	if ( empty( $payment_data ) ) {
+	if (empty($payment_data)) {
 		return false;
 	}
 
@@ -146,25 +146,25 @@ 
 	 *
 	 * @param array $payment_data Arguments passed.
 	 */
-	$payment_data = apply_filters( 'give_pre_insert_payment', $payment_data );
+	$payment_data = apply_filters('give_pre_insert_payment', $payment_data);
 
 	$payment    = new Give_Payment();
-	$gateway    = ! empty( $payment_data['gateway'] ) ? $payment_data['gateway'] : '';
-	$gateway    = empty( $gateway ) && isset( $_POST['give-gateway'] ) ? $_POST['give-gateway'] : $gateway;
-	$form_id    = isset( $payment_data['give_form_id'] ) ? $payment_data['give_form_id'] : 0;
-	$price_id   = give_get_payment_meta_price_id( $payment_data );
-	$form_title = isset( $payment_data['give_form_title'] ) ? $payment_data['give_form_title'] : get_the_title( $form_id );
+	$gateway    = ! empty($payment_data['gateway']) ? $payment_data['gateway'] : '';
+	$gateway    = empty($gateway) && isset($_POST['give-gateway']) ? $_POST['give-gateway'] : $gateway;
+	$form_id    = isset($payment_data['give_form_id']) ? $payment_data['give_form_id'] : 0;
+	$price_id   = give_get_payment_meta_price_id($payment_data);
+	$form_title = isset($payment_data['give_form_title']) ? $payment_data['give_form_title'] : get_the_title($form_id);
 
 	// Set properties.
 	$payment->total          = $payment_data['price'];
-	$payment->status         = ! empty( $payment_data['status'] ) ? $payment_data['status'] : 'pending';
-	$payment->currency       = ! empty( $payment_data['currency'] ) ? $payment_data['currency'] : give_get_currency( $payment_data['give_form_id'], $payment_data );
+	$payment->status         = ! empty($payment_data['status']) ? $payment_data['status'] : 'pending';
+	$payment->currency       = ! empty($payment_data['currency']) ? $payment_data['currency'] : give_get_currency($payment_data['give_form_id'], $payment_data);
 	$payment->user_info      = $payment_data['user_info'];
 	$payment->gateway        = $gateway;
 	$payment->form_title     = $form_title;
 	$payment->form_id        = $form_id;
 	$payment->price_id       = $price_id;
-	$payment->donor_id       = ( ! empty( $payment_data['donor_id'] ) ? $payment_data['donor_id'] : '' );
+	$payment->donor_id       = ( ! empty($payment_data['donor_id']) ? $payment_data['donor_id'] : '');
 	$payment->user_id        = $payment_data['user_info']['id'];
 	$payment->email          = $payment_data['user_email'];
 	$payment->first_name     = $payment_data['user_info']['first_name'];
@@ -172,8 +172,8 @@ 
 	$payment->email          = $payment_data['user_info']['email'];
 	$payment->ip             = give_get_ip();
 	$payment->key            = $payment_data['purchase_key'];
-	$payment->mode           = ( ! empty( $payment_data['mode'] ) ? (string) $payment_data['mode'] : ( give_is_test_mode() ? 'test' : 'live' ) );
-	$payment->parent_payment = ! empty( $payment_data['parent'] ) ? absint( $payment_data['parent'] ) : '';
+	$payment->mode           = ( ! empty($payment_data['mode']) ? (string) $payment_data['mode'] : (give_is_test_mode() ? 'test' : 'live'));
+	$payment->parent_payment = ! empty($payment_data['parent']) ? absint($payment_data['parent']) : '';
 
 	// Add the donation.
 	$args = array(
@@ -181,19 +181,19 @@ 
 		'price_id' => $payment->price_id,
 	);
 
-	$payment->add_donation( $payment->form_id, $args );
+	$payment->add_donation($payment->form_id, $args);
 
 
 	// Set date if present.
-	if ( isset( $payment_data['post_date'] ) ) {
+	if (isset($payment_data['post_date'])) {
 		$payment->date = $payment_data['post_date'];
 	}
 
 	// Handle sequential payments.
-	if ( give_get_option( 'enable_sequential' ) ) {
+	if (give_get_option('enable_sequential')) {
 		$number          = give_get_next_payment_number();
-		$payment->number = give_format_payment_number( $number );
-		update_option( 'give_last_payment_number', $number );
+		$payment->number = give_format_payment_number($number);
+		update_option('give_last_payment_number', $number);
 	}
 
 	// Save payment.
@@ -207,10 +207,10 @@ 
 	 * @param int   $payment_id   The payment ID.
 	 * @param array $payment_data Arguments passed.
 	 */
-	do_action( 'give_insert_payment', $payment->ID, $payment_data );
+	do_action('give_insert_payment', $payment->ID, $payment_data);
 
 	// Return payment ID upon success.
-	if ( ! empty( $payment->ID ) ) {
+	if ( ! empty($payment->ID)) {
 		return $payment->ID;
 	}
 
@@ -226,10 +226,10 @@ 
  *
  * @return bool|int
  */
-function give_create_payment( $payment_data ) {
+function give_create_payment($payment_data) {
 
-	$form_id  = intval( $payment_data['post_data']['give-form-id'] );
-	$price_id = isset( $payment_data['post_data']['give-price-id'] ) ? $payment_data['post_data']['give-price-id'] : '';
+	$form_id  = intval($payment_data['post_data']['give-form-id']);
+	$price_id = isset($payment_data['post_data']['give-price-id']) ? $payment_data['post_data']['give-price-id'] : '';
 
 	// Collect payment data.
 	$insert_payment_data = array(
@@ -240,7 +240,7 @@ 
 		'date'            => $payment_data['date'],
 		'user_email'      => $payment_data['user_email'],
 		'purchase_key'    => $payment_data['purchase_key'],
-		'currency'        => give_get_currency( $form_id, $payment_data ),
+		'currency'        => give_get_currency($form_id, $payment_data),
 		'user_info'       => $payment_data['user_info'],
 		'status'          => 'pending',
 		'gateway'         => 'paypal',
@@ -253,10 +253,10 @@ 
 	 *
 	 * @param array $insert_payment_data
 	 */
-	$insert_payment_data = apply_filters( 'give_create_payment', $insert_payment_data );
+	$insert_payment_data = apply_filters('give_create_payment', $insert_payment_data);
 
 	// Record the pending payment.
-	return give_insert_payment( $insert_payment_data );
+	return give_insert_payment($insert_payment_data);
 }
 
 /**
@@ -269,12 +269,12 @@ 
  *
  * @return bool
  */
-function give_update_payment_status( $payment_id, $new_status = 'publish' ) {
+function give_update_payment_status($payment_id, $new_status = 'publish') {
 
 	$updated = false;
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	if ( $payment && $payment->ID > 0 ) {
+	if ($payment && $payment->ID > 0) {
 
 		$payment->status = $new_status;
 		$updated         = $payment->save();
@@ -295,45 +295,45 @@ 
  *
  * @return void
  */
-function give_delete_donation( $payment_id = 0, $update_donor = true ) {
-	$payment = new Give_Payment( $payment_id );
+function give_delete_donation($payment_id = 0, $update_donor = true) {
+	$payment = new Give_Payment($payment_id);
 
 	// Bailout.
-	if ( ! $payment->ID ) {
+	if ( ! $payment->ID) {
 		return;
 	}
 
-	$amount   = give_donation_amount( $payment_id );
+	$amount   = give_donation_amount($payment_id);
 	$status   = $payment->post_status;
-	$donor_id = give_get_payment_donor_id( $payment_id );
-	$donor    = new Give_Donor( $donor_id );
+	$donor_id = give_get_payment_donor_id($payment_id);
+	$donor    = new Give_Donor($donor_id);
 
 	// Only undo donations that aren't these statuses.
-	$dont_undo_statuses = apply_filters( 'give_undo_donation_statuses', array(
+	$dont_undo_statuses = apply_filters('give_undo_donation_statuses', array(
 		'pending',
 		'cancelled',
-	) );
+	));
 
-	if ( ! in_array( $status, $dont_undo_statuses ) ) {
-		give_undo_donation( $payment_id );
+	if ( ! in_array($status, $dont_undo_statuses)) {
+		give_undo_donation($payment_id);
 	}
 
 	// Only undo donations that aren't these statuses.
-	$status_to_decrease_stats = apply_filters( 'give_decrease_donor_statuses', array( 'publish' ) );
+	$status_to_decrease_stats = apply_filters('give_decrease_donor_statuses', array('publish'));
 
-	if ( in_array( $status, $status_to_decrease_stats ) ) {
+	if (in_array($status, $status_to_decrease_stats)) {
 
 		// Only decrease earnings if they haven't already been decreased (or were never increased for this payment).
-		give_decrease_total_earnings( $amount );
+		give_decrease_total_earnings($amount);
 
 		// @todo: Refresh only range related stat cache
 		give_delete_donation_stats();
 
-		if ( $donor->id && $update_donor ) {
+		if ($donor->id && $update_donor) {
 
 			// Decrement the stats for the donor.
 			$donor->decrease_donation_count();
-			$donor->decrease_value( $amount );
+			$donor->decrease_value($amount);
 
 		}
 	}
@@ -345,18 +345,18 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_payment_delete', $payment_id );
+	do_action('give_payment_delete', $payment_id);
 
-	if ( $donor->id && $update_donor ) {
+	if ($donor->id && $update_donor) {
 		// Remove the payment ID from the donor.
-		$donor->remove_payment( $payment_id );
+		$donor->remove_payment($payment_id);
 	}
 
 	// Remove the payment.
-	wp_delete_post( $payment_id, true );
+	wp_delete_post($payment_id, true);
 
 	// Remove related sale log entries.
-	Give()->logs->delete_logs( $payment_id );
+	Give()->logs->delete_logs($payment_id);
 
 	/**
 	 * Fires after payment deleted.
@@ -365,7 +365,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_payment_deleted', $payment_id );
+	do_action('give_payment_deleted', $payment_id);
 }
 
 /**
@@ -380,20 +380,20 @@ 
  *
  * @return void
  */
-function give_undo_donation( $payment_id ) {
+function give_undo_donation($payment_id) {
 
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	$maybe_decrease_earnings = apply_filters( 'give_decrease_earnings_on_undo', true, $payment, $payment->form_id );
-	if ( true === $maybe_decrease_earnings ) {
+	$maybe_decrease_earnings = apply_filters('give_decrease_earnings_on_undo', true, $payment, $payment->form_id);
+	if (true === $maybe_decrease_earnings) {
 		// Decrease earnings.
-		give_decrease_form_earnings( $payment->form_id, $payment->total );
+		give_decrease_form_earnings($payment->form_id, $payment->total);
 	}
 
-	$maybe_decrease_donations = apply_filters( 'give_decrease_donations_on_undo', true, $payment, $payment->form_id );
-	if ( true === $maybe_decrease_donations ) {
+	$maybe_decrease_donations = apply_filters('give_decrease_donations_on_undo', true, $payment, $payment->form_id);
+	if (true === $maybe_decrease_donations) {
 		// Decrease donation count.
-		give_decrease_donation_count( $payment->form_id );
+		give_decrease_donation_count($payment->form_id);
 	}
 
 }
@@ -410,21 +410,21 @@ 
  *
  * @return object $stats Contains the number of payments per payment status.
  */
-function give_count_payments( $args = array() ) {
+function give_count_payments($args = array()) {
 	// Backward compatibility.
-	if ( ! empty( $args['start-date'] ) ) {
+	if ( ! empty($args['start-date'])) {
 		$args['start_date'] = $args['start-date'];
-		unset( $args['start-date'] );
+		unset($args['start-date']);
 	}
 
-	if ( ! empty( $args['end-date'] ) ) {
+	if ( ! empty($args['end-date'])) {
 		$args['end_date'] = $args['end-date'];
-		unset( $args['end-date'] );
+		unset($args['end-date']);
 	}
 
-	if ( ! empty( $args['form_id'] ) ) {
+	if ( ! empty($args['form_id'])) {
 		$args['give_forms'] = $args['form_id'];
-		unset( $args['form_id'] );
+		unset($args['form_id']);
 	}
 
 	// Extract all donations
@@ -432,7 +432,7 @@ 
 	$args['group_by']    = 'post_status';
 	$args['count']       = 'true';
 
-	$donations_obj   = new Give_Payments_Query( $args );
+	$donations_obj   = new Give_Payments_Query($args);
 	$donations_count = $donations_obj->get_payment_by_group();
 
 	/**
@@ -440,7 +440,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	return (object) apply_filters( 'give_count_payments', $donations_count, $args, $donations_obj );
+	return (object) apply_filters('give_count_payments', $donations_count, $args, $donations_obj);
 }
 
 
@@ -453,11 +453,11 @@ 
  *
  * @return bool $exists True if payment exists, false otherwise.
  */
-function give_check_for_existing_payment( $payment_id ) {
+function give_check_for_existing_payment($payment_id) {
 	$exists  = false;
-	$payment = new Give_Payment( $payment_id );
+	$payment = new Give_Payment($payment_id);
 
-	if ( $payment_id === $payment->ID && 'publish' === $payment->status ) {
+	if ($payment_id === $payment->ID && 'publish' === $payment->status) {
 		$exists = true;
 	}
 
@@ -475,41 +475,41 @@ 
  *
  * @return bool|mixed True if payment status exists, false otherwise.
  */
-function give_get_payment_status( $payment, $return_label = false ) {
+function give_get_payment_status($payment, $return_label = false) {
 
-	if ( is_numeric( $payment ) ) {
+	if (is_numeric($payment)) {
 
-		$payment = new Give_Payment( $payment );
+		$payment = new Give_Payment($payment);
 
-		if ( ! $payment->ID > 0 ) {
+		if ( ! $payment->ID > 0) {
 			return false;
 		}
 
 	}
 
-	if ( ! is_object( $payment ) || ! isset( $payment->post_status ) ) {
+	if ( ! is_object($payment) || ! isset($payment->post_status)) {
 		return false;
 	}
 
 	$statuses = give_get_payment_statuses();
 
-	if ( ! is_array( $statuses ) || empty( $statuses ) ) {
+	if ( ! is_array($statuses) || empty($statuses)) {
 		return false;
 	}
 
 	// Get payment object if not already given.
-	$payment = $payment instanceof Give_Payment ? $payment : new Give_Payment( $payment->ID );
+	$payment = $payment instanceof Give_Payment ? $payment : new Give_Payment($payment->ID);
 
-	if ( array_key_exists( $payment->status, $statuses ) ) {
-		if ( true === $return_label ) {
+	if (array_key_exists($payment->status, $statuses)) {
+		if (true === $return_label) {
 			// Return translated status label.
-			return $statuses[ $payment->status ];
+			return $statuses[$payment->status];
 		} else {
 			// Account that our 'publish' status is labeled 'Complete'
 			$post_status = 'publish' === $payment->status ? 'Complete' : $payment->post_status;
 
 			// Make sure we're matching cases, since they matter
-			return array_search( strtolower( $post_status ), array_map( 'strtolower', $statuses ) );
+			return array_search(strtolower($post_status), array_map('strtolower', $statuses));
 		}
 	}
 
@@ -525,18 +525,18 @@ 
  */
 function give_get_payment_statuses() {
 	$payment_statuses = array(
-		'pending'     => __( 'Pending', 'give' ),
-		'publish'     => __( 'Complete', 'give' ),
-		'refunded'    => __( 'Refunded', 'give' ),
-		'failed'      => __( 'Failed', 'give' ),
-		'cancelled'   => __( 'Cancelled', 'give' ),
-		'abandoned'   => __( 'Abandoned', 'give' ),
-		'preapproval' => __( 'Pre-Approved', 'give' ),
-		'processing'  => __( 'Processing', 'give' ),
-		'revoked'     => __( 'Revoked', 'give' ),
+		'pending'     => __('Pending', 'give'),
+		'publish'     => __('Complete', 'give'),
+		'refunded'    => __('Refunded', 'give'),
+		'failed'      => __('Failed', 'give'),
+		'cancelled'   => __('Cancelled', 'give'),
+		'abandoned'   => __('Abandoned', 'give'),
+		'preapproval' => __('Pre-Approved', 'give'),
+		'processing'  => __('Processing', 'give'),
+		'revoked'     => __('Revoked', 'give'),
 	);
 
-	return apply_filters( 'give_payment_statuses', $payment_statuses );
+	return apply_filters('give_payment_statuses', $payment_statuses);
 }
 
 /**
@@ -549,10 +549,10 @@ 
  * @return array $payment_status All the available payment statuses.
  */
 function give_get_payment_status_keys() {
-	$statuses = array_keys( give_get_payment_statuses() );
-	asort( $statuses );
+	$statuses = array_keys(give_get_payment_statuses());
+	asort($statuses);
 
-	return array_values( $statuses );
+	return array_values($statuses);
 }
 
 /**
@@ -567,44 +567,44 @@ 
  *
  * @return int $earnings Earnings
  */
-function give_get_earnings_by_date( $day = null, $month_num, $year = null, $hour = null ) {
+function give_get_earnings_by_date($day = null, $month_num, $year = null, $hour = null) {
 	// This is getting deprecated soon. Use Give_Payment_Stats with the get_earnings() method instead.
 
 	global $wpdb;
-	$meta_table = __give_v20_bc_table_details( 'payment' );
+	$meta_table = __give_v20_bc_table_details('payment');
 
 	$args = array(
 		'post_type'              => 'give_payment',
 		'nopaging'               => true,
 		'year'                   => $year,
 		'monthnum'               => $month_num,
-		'post_status'            => array( 'publish' ),
+		'post_status'            => array('publish'),
 		'fields'                 => 'ids',
 		'update_post_term_cache' => false,
 	);
-	if ( ! empty( $day ) ) {
+	if ( ! empty($day)) {
 		$args['day'] = $day;
 	}
 
-	if ( isset( $hour ) ) {
+	if (isset($hour)) {
 		$args['hour'] = $hour;
 	}
 
-	$args = apply_filters( 'give_get_earnings_by_date_args', $args );
-	$key  = Give_Cache::get_key( 'give_stats', $args );
+	$args = apply_filters('give_get_earnings_by_date_args', $args);
+	$key  = Give_Cache::get_key('give_stats', $args);
 
-	if ( ! empty( $_GET['_wpnonce'] ) && wp_verify_nonce( $_GET['_wpnonce'], 'give-refresh-reports' ) ) {
+	if ( ! empty($_GET['_wpnonce']) && wp_verify_nonce($_GET['_wpnonce'], 'give-refresh-reports')) {
 		$earnings = false;
 	} else {
-		$earnings = Give_Cache::get( $key );
+		$earnings = Give_Cache::get($key);
 	}
 
-	if ( false === $earnings ) {
-		$donations = get_posts( $args );
+	if (false === $earnings) {
+		$donations = get_posts($args);
 		$earnings  = 0;
-		if ( $donations ) {
-			$donations      = implode( ',', $donations );
-			$earning_totals = $wpdb->get_var( "SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN ({$donations})" );
+		if ($donations) {
+			$donations      = implode(',', $donations);
+			$earning_totals = $wpdb->get_var("SELECT SUM(meta_value) FROM $wpdb->postmeta WHERE meta_key = '_give_payment_total' AND post_id IN ({$donations})");
 
 			/**
 			 * Filter The earnings by dates.
@@ -615,13 +615,13 @@ 
 			 * @param array $donations      Donations lists.
 			 * @param array $args           Donation query args.
 			 */
-			$earnings = apply_filters( 'give_get_earnings_by_date', $earning_totals, $donations, $args );
+			$earnings = apply_filters('give_get_earnings_by_date', $earning_totals, $donations, $args);
 		}
 		// Cache the results for one hour.
-		Give_Cache::set( $key, $earnings, HOUR_IN_SECONDS );
+		Give_Cache::set($key, $earnings, HOUR_IN_SECONDS);
 	}
 
-	return round( $earnings, 2 );
+	return round($earnings, 2);
 }
 
 /**
@@ -636,7 +636,7 @@ 
  *
  * @return int $count Sales
  */
-function give_get_sales_by_date( $day = null, $month_num = null, $year = null, $hour = null ) {
+function give_get_sales_by_date($day = null, $month_num = null, $year = null, $hour = null) {
 
 	// This is getting deprecated soon. Use Give_Payment_Stats with the get_sales() method instead.
 	$args = array(
@@ -644,14 +644,14 @@ 
 		'nopaging'               => true,
 		'year'                   => $year,
 		'fields'                 => 'ids',
-		'post_status'            => array( 'publish' ),
+		'post_status'            => array('publish'),
 		'update_post_meta_cache' => false,
 		'update_post_term_cache' => false,
 	);
 
-	$show_free = apply_filters( 'give_sales_by_date_show_free', true, $args );
+	$show_free = apply_filters('give_sales_by_date_show_free', true, $args);
 
-	if ( false === $show_free ) {
+	if (false === $show_free) {
 		$args['meta_query'] = array(
 			array(
 				'key'     => '_give_payment_total',
@@ -662,33 +662,33 @@ 
 		);
 	}
 
-	if ( ! empty( $month_num ) ) {
+	if ( ! empty($month_num)) {
 		$args['monthnum'] = $month_num;
 	}
 
-	if ( ! empty( $day ) ) {
+	if ( ! empty($day)) {
 		$args['day'] = $day;
 	}
 
-	if ( isset( $hour ) ) {
+	if (isset($hour)) {
 		$args['hour'] = $hour;
 	}
 
-	$args = apply_filters( 'give_get_sales_by_date_args', $args );
+	$args = apply_filters('give_get_sales_by_date_args', $args);
 
-	$key = Give_Cache::get_key( 'give_stats', $args );
+	$key = Give_Cache::get_key('give_stats', $args);
 
-	if ( ! empty( $_GET['_wpnonce'] ) && wp_verify_nonce( $_GET['_wpnonce'], 'give-refresh-reports' ) ) {
+	if ( ! empty($_GET['_wpnonce']) && wp_verify_nonce($_GET['_wpnonce'], 'give-refresh-reports')) {
 		$count = false;
 	} else {
-		$count = Give_Cache::get( $key );
+		$count = Give_Cache::get($key);
 	}
 
-	if ( false === $count ) {
-		$donations = new WP_Query( $args );
+	if (false === $count) {
+		$donations = new WP_Query($args);
 		$count     = (int) $donations->post_count;
 		// Cache the results for one hour.
-		Give_Cache::set( $key, $count, HOUR_IN_SECONDS );
+		Give_Cache::set($key, $count, HOUR_IN_SECONDS);
 	}
 
 	return $count;
@@ -703,19 +703,19 @@ 
  *
  * @return bool $ret True if complete, false otherwise.
  */
-function give_is_payment_complete( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_is_payment_complete($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	$ret = false;
 
-	if ( $payment->ID > 0 ) {
+	if ($payment->ID > 0) {
 
-		if ( (int) $payment_id === (int) $payment->ID && 'publish' == $payment->status ) {
+		if ((int) $payment_id === (int) $payment->ID && 'publish' == $payment->status) {
 			$ret = true;
 		}
 	}
 
-	return apply_filters( 'give_is_payment_complete', $ret, $payment_id, $payment->post_status );
+	return apply_filters('give_is_payment_complete', $ret, $payment_id, $payment->post_status);
 }
 
 /**
@@ -741,50 +741,50 @@ 
  *
  * @return float $total Total earnings.
  */
-function give_get_total_earnings( $recalculate = false ) {
+function give_get_total_earnings($recalculate = false) {
 
-	$total      = get_option( 'give_earnings_total', 0 );
-	$meta_table = __give_v20_bc_table_details( 'payment' );
+	$total      = get_option('give_earnings_total', 0);
+	$meta_table = __give_v20_bc_table_details('payment');
 
 	// Calculate total earnings.
-	if ( ! $total || $recalculate ) {
+	if ( ! $total || $recalculate) {
 		global $wpdb;
 
 		$total = (float) 0;
 
-		$args = apply_filters( 'give_get_total_earnings_args', array(
+		$args = apply_filters('give_get_total_earnings_args', array(
 			'offset' => 0,
-			'number' => - 1,
-			'status' => array( 'publish' ),
+			'number' => -1,
+			'status' => array('publish'),
 			'fields' => 'ids',
-		) );
+		));
 
-		$payments = give_get_payments( $args );
-		if ( $payments ) {
+		$payments = give_get_payments($args);
+		if ($payments) {
 
 			/**
 			 * If performing a donation, we need to skip the very last payment in the database,
 			 * since it calls give_increase_total_earnings() on completion,
 			 * which results in duplicated earnings for the very first donation.
 			 */
-			if ( did_action( 'give_update_payment_status' ) ) {
-				array_pop( $payments );
+			if (did_action('give_update_payment_status')) {
+				array_pop($payments);
 			}
 
-			if ( ! empty( $payments ) ) {
-				$payments = implode( ',', $payments );
-				$total    += $wpdb->get_var( "SELECT SUM(meta_value) FROM {$meta_table['name']} WHERE meta_key = '_give_payment_total' AND {$meta_table['column']['id']} IN({$payments})" );
+			if ( ! empty($payments)) {
+				$payments = implode(',', $payments);
+				$total += $wpdb->get_var("SELECT SUM(meta_value) FROM {$meta_table['name']} WHERE meta_key = '_give_payment_total' AND {$meta_table['column']['id']} IN({$payments})");
 			}
 		}
 
-		update_option( 'give_earnings_total', $total, 'no' );
+		update_option('give_earnings_total', $total, 'no');
 	}
 
-	if ( $total < 0 ) {
+	if ($total < 0) {
 		$total = 0; // Don't ever show negative earnings.
 	}
 
-	return apply_filters( 'give_total_earnings', round( $total, give_get_price_decimals() ), $total );
+	return apply_filters('give_total_earnings', round($total, give_get_price_decimals()), $total);
 }
 
 /**
@@ -796,10 +796,10 @@ 
  *
  * @return float $total Total earnings.
  */
-function give_increase_total_earnings( $amount = 0 ) {
+function give_increase_total_earnings($amount = 0) {
 	$total = give_get_total_earnings();
 	$total += $amount;
-	update_option( 'give_earnings_total', $total );
+	update_option('give_earnings_total', $total);
 
 	return $total;
 }
@@ -813,13 +813,13 @@ 
  *
  * @return float $total Total earnings.
  */
-function give_decrease_total_earnings( $amount = 0 ) {
+function give_decrease_total_earnings($amount = 0) {
 	$total = give_get_total_earnings();
 	$total -= $amount;
-	if ( $total < 0 ) {
+	if ($total < 0) {
 		$total = 0;
 	}
-	update_option( 'give_earnings_total', $total );
+	update_option('give_earnings_total', $total);
 
 	return $total;
 }
@@ -835,10 +835,10 @@ 
  *
  * @return mixed $meta Payment Meta.
  */
-function give_get_payment_meta( $payment_id = 0, $meta_key = '_give_payment_meta', $single = true ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_meta($payment_id = 0, $meta_key = '_give_payment_meta', $single = true) {
+	$payment = new Give_Payment($payment_id);
 
-	return $payment->get_meta( $meta_key, $single );
+	return $payment->get_meta($meta_key, $single);
 }
 
 /**
@@ -851,10 +851,10 @@ 
  *
  * @return mixed Meta ID if successful, false if unsuccessful.
  */
-function give_update_payment_meta( $payment_id = 0, $meta_key = '', $meta_value = '', $prev_value = '' ) {
-	$payment = new Give_Payment( $payment_id );
+function give_update_payment_meta($payment_id = 0, $meta_key = '', $meta_value = '', $prev_value = '') {
+	$payment = new Give_Payment($payment_id);
 
-	return $payment->update_meta( $meta_key, $meta_value, $prev_value );
+	return $payment->update_meta($meta_key, $meta_value, $prev_value);
 }
 
 /**
@@ -866,8 +866,8 @@ 
  *
  * @return array $user_info User Info Meta Values.
  */
-function give_get_payment_meta_user_info( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_meta_user_info($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->user_info;
 }
@@ -883,8 +883,8 @@ 
  *
  * @return int $form_id Form ID.
  */
-function give_get_payment_form_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_form_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->form_id;
 }
@@ -898,8 +898,8 @@ 
  *
  * @return string $email User email.
  */
-function give_get_payment_user_email( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_user_email($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->email;
 }
@@ -913,11 +913,11 @@ 
  *
  * @return bool $is_guest_payment If the payment is associated with a user (false) or not (true)
  */
-function give_is_guest_payment( $payment_id ) {
-	$payment_user_id  = give_get_payment_user_id( $payment_id );
-	$is_guest_payment = ! empty( $payment_user_id ) && $payment_user_id > 0 ? false : true;
+function give_is_guest_payment($payment_id) {
+	$payment_user_id  = give_get_payment_user_id($payment_id);
+	$is_guest_payment = ! empty($payment_user_id) && $payment_user_id > 0 ? false : true;
 
-	return (bool) apply_filters( 'give_is_guest_payment', $is_guest_payment, $payment_id );
+	return (bool) apply_filters('give_is_guest_payment', $is_guest_payment, $payment_id);
 }
 
 /**
@@ -929,8 +929,8 @@ 
  *
  * @return int $user_id User ID.
  */
-function give_get_payment_user_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_user_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->user_id;
 }
@@ -944,8 +944,8 @@ 
  *
  * @return int $payment->customer_id Donor ID.
  */
-function give_get_payment_donor_id( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_donor_id($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->customer_id;
 }
@@ -959,8 +959,8 @@ 
  *
  * @return string $ip User IP.
  */
-function give_get_payment_user_ip( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_user_ip($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->ip;
 }
@@ -974,8 +974,8 @@ 
  *
  * @return string $date The date the payment was completed.
  */
-function give_get_payment_completed_date( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_completed_date($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->completed_date;
 }
@@ -989,8 +989,8 @@ 
  *
  * @return string $gateway Gateway.
  */
-function give_get_payment_gateway( $payment_id ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_gateway($payment_id) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->gateway;
 }
@@ -1004,8 +1004,8 @@ 
  *
  * @return string $currency The currency code.
  */
-function give_get_payment_currency_code( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_currency_code($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->currency;
 }
@@ -1019,10 +1019,10 @@ 
  *
  * @return string $currency The currency name.
  */
-function give_get_payment_currency( $payment_id = 0 ) {
-	$currency = give_get_payment_currency_code( $payment_id );
+function give_get_payment_currency($payment_id = 0) {
+	$currency = give_get_payment_currency_code($payment_id);
 
-	return apply_filters( 'give_payment_currency', give_get_currency_name( $currency ), $payment_id );
+	return apply_filters('give_payment_currency', give_get_currency_name($currency), $payment_id);
 }
 
 /**
@@ -1034,8 +1034,8 @@ 
  *
  * @return string $key Donation key.
  */
-function give_get_payment_key( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_key($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->key;
 }
@@ -1051,8 +1051,8 @@ 
  *
  * @return string $number Payment order number.
  */
-function give_get_payment_number( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_number($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->number;
 }
@@ -1066,23 +1066,23 @@ 
  *
  * @return string      The formatted payment number.
  */
-function give_format_payment_number( $number ) {
+function give_format_payment_number($number) {
 
-	if ( ! give_get_option( 'enable_sequential' ) ) {
+	if ( ! give_get_option('enable_sequential')) {
 		return $number;
 	}
 
-	if ( ! is_numeric( $number ) ) {
+	if ( ! is_numeric($number)) {
 		return $number;
 	}
 
-	$prefix  = give_get_option( 'sequential_prefix' );
-	$number  = absint( $number );
-	$postfix = give_get_option( 'sequential_postfix' );
+	$prefix  = give_get_option('sequential_prefix');
+	$number  = absint($number);
+	$postfix = give_get_option('sequential_postfix');
 
-	$formatted_number = $prefix . $number . $postfix;
+	$formatted_number = $prefix.$number.$postfix;
 
-	return apply_filters( 'give_format_payment_number', $formatted_number, $prefix, $number, $postfix );
+	return apply_filters('give_format_payment_number', $formatted_number, $prefix, $number, $postfix);
 }
 
 /**
@@ -1096,17 +1096,17 @@ 
  */
 function give_get_next_payment_number() {
 
-	if ( ! give_get_option( 'enable_sequential' ) ) {
+	if ( ! give_get_option('enable_sequential')) {
 		return false;
 	}
 
-	$number           = get_option( 'give_last_payment_number' );
-	$start            = give_get_option( 'sequential_start', 1 );
+	$number           = get_option('give_last_payment_number');
+	$start            = give_get_option('sequential_start', 1);
 	$increment_number = true;
 
-	if ( false !== $number ) {
+	if (false !== $number) {
 
-		if ( empty( $number ) ) {
+		if (empty($number)) {
 
 			$number           = $start;
 			$increment_number = false;
@@ -1115,24 +1115,24 @@ 
 	} else {
 
 		// This case handles the first addition of the new option, as well as if it get's deleted for any reason.
-		$payments     = new Give_Payments_Query( array(
+		$payments = new Give_Payments_Query(array(
 			'number'  => 1,
 			'order'   => 'DESC',
 			'orderby' => 'ID',
 			'output'  => 'posts',
 			'fields'  => 'ids',
-		) );
+		));
 		$last_payment = $payments->get_payments();
 
-		if ( ! empty( $last_payment ) ) {
+		if ( ! empty($last_payment)) {
 
-			$number = give_get_payment_number( $last_payment[0] );
+			$number = give_get_payment_number($last_payment[0]);
 
 		}
 
-		if ( ! empty( $number ) && $number !== (int) $last_payment[0] ) {
+		if ( ! empty($number) && $number !== (int) $last_payment[0]) {
 
-			$number = give_remove_payment_prefix_postfix( $number );
+			$number = give_remove_payment_prefix_postfix($number);
 
 		} else {
 
@@ -1141,13 +1141,13 @@ 
 		}
 	}// End if().
 
-	$increment_number = apply_filters( 'give_increment_payment_number', $increment_number, $number );
+	$increment_number = apply_filters('give_increment_payment_number', $increment_number, $number);
 
-	if ( $increment_number ) {
-		$number ++;
+	if ($increment_number) {
+		$number++;
 	}
 
-	return apply_filters( 'give_get_next_payment_number', $number );
+	return apply_filters('give_get_next_payment_number', $number);
 }
 
 /**
@@ -1159,25 +1159,25 @@ 
  *
  * @return string The new Payment number without prefix and postfix.
  */
-function give_remove_payment_prefix_postfix( $number ) {
+function give_remove_payment_prefix_postfix($number) {
 
-	$prefix  = give_get_option( 'sequential_prefix' );
-	$postfix = give_get_option( 'sequential_postfix' );
+	$prefix  = give_get_option('sequential_prefix');
+	$postfix = give_get_option('sequential_postfix');
 
 	// Remove prefix.
-	$number = preg_replace( '/' . $prefix . '/', '', $number, 1 );
+	$number = preg_replace('/'.$prefix.'/', '', $number, 1);
 
 	// Remove the postfix.
-	$length      = strlen( $number );
-	$postfix_pos = strrpos( $number, $postfix );
-	if ( false !== $postfix_pos ) {
-		$number = substr_replace( $number, '', $postfix_pos, $length );
+	$length      = strlen($number);
+	$postfix_pos = strrpos($number, $postfix);
+	if (false !== $postfix_pos) {
+		$number = substr_replace($number, '', $postfix_pos, $length);
 	}
 
 	// Ensure it's a whole number.
-	$number = intval( $number );
+	$number = intval($number);
 
-	return apply_filters( 'give_remove_payment_prefix_postfix', $number, $prefix, $postfix );
+	return apply_filters('give_remove_payment_prefix_postfix', $number, $prefix, $postfix);
 
 }
 
@@ -1195,16 +1195,16 @@ 
  *
  * @return string $amount Fully formatted donation amount.
  */
-function give_donation_amount( $donation, $format_args = array() ) {
+function give_donation_amount($donation, $format_args = array()) {
 	/* @var Give_Payment $donation */
-	if ( ! ( $donation instanceof Give_Payment ) ) {
-		$donation = new Give_Payment( absint( $donation ) );
+	if ( ! ($donation instanceof Give_Payment)) {
+		$donation = new Give_Payment(absint($donation));
 	}
 
 	$amount           = $donation->total;
 	$formatted_amount = $amount;
 
-	if ( is_bool( $format_args ) ) {
+	if (is_bool($format_args)) {
 		$format_args = array(
 			'currency' => (bool) $format_args,
 			'amount'   => (bool) $format_args,
@@ -1227,27 +1227,25 @@ 
 		)
 	);
 
-	if ( $format_args['amount'] || $format_args['currency'] ) {
+	if ($format_args['amount'] || $format_args['currency']) {
 
-		if ( $format_args['amount'] ) {
+		if ($format_args['amount']) {
 
 			$formatted_amount = give_format_amount(
 				$amount,
-				! is_array( $format_args['amount'] ) ?
+				! is_array($format_args['amount']) ?
 					array(
 						'sanitize' => false,
 						'currency' => $donation->currency,
-					) :
-					$format_args['amount']
+					) : $format_args['amount']
 			);
 		}
 
-		if ( $format_args['currency'] ) {
+		if ($format_args['currency']) {
 			$formatted_amount = give_currency_filter(
 				$formatted_amount,
-				! is_array( $format_args['currency'] ) ?
-					array( 'currency_code' => $donation->currency ) :
-					$format_args['currency']
+				! is_array($format_args['currency']) ?
+					array('currency_code' => $donation->currency) : $format_args['currency']
 			);
 		}
 	}
@@ -1262,7 +1260,7 @@ 
 	 * @param int    $donation_id      Donation ID.
 	 * @param string $type             Donation amount type.
 	 */
-	return apply_filters( 'give_donation_amount', (string) $formatted_amount, $amount, $donation, $format_args );
+	return apply_filters('give_donation_amount', (string) $formatted_amount, $amount, $donation, $format_args);
 }
 
 /**
@@ -1279,10 +1277,10 @@ 
  *
  * @return array Fully formatted payment subtotal.
  */
-function give_payment_subtotal( $payment_id = 0 ) {
-	$subtotal = give_get_payment_subtotal( $payment_id );
+function give_payment_subtotal($payment_id = 0) {
+	$subtotal = give_get_payment_subtotal($payment_id);
 
-	return give_currency_filter( give_format_amount( $subtotal, array( 'sanitize' => false ) ), array( 'currency_code' => give_get_payment_currency_code( $payment_id ) ) );
+	return give_currency_filter(give_format_amount($subtotal, array('sanitize' => false)), array('currency_code' => give_get_payment_currency_code($payment_id)));
 }
 
 /**
@@ -1296,8 +1294,8 @@ 
  *
  * @return float $subtotal Subtotal for payment (non formatted).
  */
-function give_get_payment_subtotal( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_subtotal($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->subtotal;
 }
@@ -1311,8 +1309,8 @@ 
  *
  * @return string The donation ID.
  */
-function give_get_payment_transaction_id( $payment_id = 0 ) {
-	$payment = new Give_Payment( $payment_id );
+function give_get_payment_transaction_id($payment_id = 0) {
+	$payment = new Give_Payment($payment_id);
 
 	return $payment->transaction_id;
 }
@@ -1327,15 +1325,15 @@ 
  *
  * @return bool|mixed
  */
-function give_set_payment_transaction_id( $payment_id = 0, $transaction_id = '' ) {
+function give_set_payment_transaction_id($payment_id = 0, $transaction_id = '') {
 
-	if ( empty( $payment_id ) || empty( $transaction_id ) ) {
+	if (empty($payment_id) || empty($transaction_id)) {
 		return false;
 	}
 
-	$transaction_id = apply_filters( 'give_set_payment_transaction_id', $transaction_id, $payment_id );
+	$transaction_id = apply_filters('give_set_payment_transaction_id', $transaction_id, $payment_id);
 
-	return give_update_payment_meta( $payment_id, '_give_payment_transaction_id', $transaction_id );
+	return give_update_payment_meta($payment_id, '_give_payment_transaction_id', $transaction_id);
 }
 
 /**
@@ -1348,10 +1346,10 @@ 
  *
  * @return int $purchase Donation ID.
  */
-function give_get_donation_id_by_key( $key ) {
+function give_get_donation_id_by_key($key) {
 	global $wpdb;
 
-	$meta_table = __give_v20_bc_table_details( 'payment' );
+	$meta_table = __give_v20_bc_table_details('payment');
 
 	$purchase = $wpdb->get_var(
 		$wpdb->prepare(
@@ -1367,7 +1365,7 @@ 
 		)
 	);
 
-	if ( $purchase != null ) {
+	if ($purchase != null) {
 		return $purchase;
 	}
 
@@ -1385,13 +1383,13 @@ 
  *
  * @return int $purchase Donation ID.
  */
-function give_get_purchase_id_by_transaction_id( $key ) {
+function give_get_purchase_id_by_transaction_id($key) {
 	global $wpdb;
-	$meta_table = __give_v20_bc_table_details( 'payment' );
+	$meta_table = __give_v20_bc_table_details('payment');
 
-	$purchase = $wpdb->get_var( $wpdb->prepare( "SELECT {$meta_table['column']['id']} FROM {$meta_table['name']} WHERE meta_key = '_give_payment_transaction_id' AND meta_value = %s LIMIT 1", $key ) );
+	$purchase = $wpdb->get_var($wpdb->prepare("SELECT {$meta_table['column']['id']} FROM {$meta_table['name']} WHERE meta_key = '_give_payment_transaction_id' AND meta_value = %s LIMIT 1", $key));
 
-	if ( $purchase != null ) {
+	if ($purchase != null) {
 		return $purchase;
 	}
 
@@ -1408,23 +1406,23 @@ 
  *
  * @return array $notes Donation Notes
  */
-function give_get_payment_notes( $payment_id = 0, $search = '' ) {
+function give_get_payment_notes($payment_id = 0, $search = '') {
 
-	if ( empty( $payment_id ) && empty( $search ) ) {
+	if (empty($payment_id) && empty($search)) {
 		return false;
 	}
 
-	remove_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
-	remove_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10 );
+	remove_action('pre_get_comments', 'give_hide_payment_notes', 10);
+	remove_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10);
 
-	$notes = get_comments( array(
+	$notes = get_comments(array(
 		'post_id' => $payment_id,
 		'order'   => 'ASC',
 		'search'  => $search,
-	) );
+	));
 
-	add_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
-	add_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2 );
+	add_action('pre_get_comments', 'give_hide_payment_notes', 10);
+	add_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2);
 
 	return $notes;
 }
@@ -1440,8 +1438,8 @@ 
  *
  * @return int The new note ID
  */
-function give_insert_payment_note( $payment_id = 0, $note = '' ) {
-	if ( empty( $payment_id ) ) {
+function give_insert_payment_note($payment_id = 0, $note = '') {
+	if (empty($payment_id)) {
 		return false;
 	}
 
@@ -1453,14 +1451,14 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_pre_insert_payment_note', $payment_id, $note );
+	do_action('give_pre_insert_payment_note', $payment_id, $note);
 
-	$note_id = wp_insert_comment( wp_filter_comment( array(
+	$note_id = wp_insert_comment(wp_filter_comment(array(
 		'comment_post_ID'      => $payment_id,
 		'comment_content'      => $note,
 		'user_id'              => is_admin() ? get_current_user_id() : 0,
-		'comment_date'         => current_time( 'mysql' ),
-		'comment_date_gmt'     => current_time( 'mysql', 1 ),
+		'comment_date'         => current_time('mysql'),
+		'comment_date_gmt'     => current_time('mysql', 1),
 		'comment_approved'     => 1,
 		'comment_parent'       => 0,
 		'comment_author'       => '',
@@ -1469,7 +1467,7 @@ 
 		'comment_author_email' => '',
 		'comment_type'         => 'give_payment_note',
 
-	) ) );
+	)));
 
 	/**
 	 * Fires after payment note inserted.
@@ -1480,7 +1478,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_insert_payment_note', $note_id, $payment_id, $note );
+	do_action('give_insert_payment_note', $note_id, $payment_id, $note);
 
 	return $note_id;
 }
@@ -1495,8 +1493,8 @@ 
  *
  * @return bool True on success, false otherwise.
  */
-function give_delete_payment_note( $comment_id = 0, $payment_id = 0 ) {
-	if ( empty( $comment_id ) ) {
+function give_delete_payment_note($comment_id = 0, $payment_id = 0) {
+	if (empty($comment_id)) {
 		return false;
 	}
 
@@ -1508,9 +1506,9 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_pre_delete_payment_note', $comment_id, $payment_id );
+	do_action('give_pre_delete_payment_note', $comment_id, $payment_id);
 
-	$ret = wp_delete_comment( $comment_id, true );
+	$ret = wp_delete_comment($comment_id, true);
 
 	/**
 	 * Fires after donation note deleted.
@@ -1520,7 +1518,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_post_delete_payment_note', $comment_id, $payment_id );
+	do_action('give_post_delete_payment_note', $comment_id, $payment_id);
 
 	return $ret;
 }
@@ -1535,32 +1533,32 @@ 
  *
  * @return string
  */
-function give_get_payment_note_html( $note, $payment_id = 0 ) {
+function give_get_payment_note_html($note, $payment_id = 0) {
 
-	if ( is_numeric( $note ) ) {
-		$note = get_comment( $note );
+	if (is_numeric($note)) {
+		$note = get_comment($note);
 	}
 
-	if ( ! empty( $note->user_id ) ) {
-		$user = get_userdata( $note->user_id );
+	if ( ! empty($note->user_id)) {
+		$user = get_userdata($note->user_id);
 		$user = $user->display_name;
 	} else {
-		$user = __( 'System', 'give' );
+		$user = __('System', 'give');
 	}
 
-	$date_format = give_date_format() . ', ' . get_option( 'time_format' );
+	$date_format = give_date_format().', '.get_option('time_format');
 
-	$delete_note_url = wp_nonce_url( add_query_arg( array(
+	$delete_note_url = wp_nonce_url(add_query_arg(array(
 		'give-action' => 'delete_payment_note',
 		'note_id'     => $note->comment_ID,
 		'payment_id'  => $payment_id,
-	) ), 'give_delete_payment_note_' . $note->comment_ID );
+	)), 'give_delete_payment_note_'.$note->comment_ID);
 
-	$note_html = '<div class="give-payment-note" id="give-payment-note-' . $note->comment_ID . '">';
+	$note_html = '<div class="give-payment-note" id="give-payment-note-'.$note->comment_ID.'">';
 	$note_html .= '<p>';
-	$note_html .= '<strong>' . $user . '</strong>&nbsp;&ndash;&nbsp;<span style="color:#aaa;font-style:italic;">' . date_i18n( $date_format, strtotime( $note->comment_date ) ) . '</span><br/>';
+	$note_html .= '<strong>'.$user.'</strong>&nbsp;&ndash;&nbsp;<span style="color:#aaa;font-style:italic;">'.date_i18n($date_format, strtotime($note->comment_date)).'</span><br/>';
 	$note_html .= $note->comment_content;
-	$note_html .= '&nbsp;&ndash;&nbsp;<a href="' . esc_url( $delete_note_url ) . '" class="give-delete-payment-note" data-note-id="' . absint( $note->comment_ID ) . '" data-payment-id="' . absint( $payment_id ) . '" aria-label="' . __( 'Delete this donation note.', 'give' ) . '">' . __( 'Delete', 'give' ) . '</a>';
+	$note_html .= '&nbsp;&ndash;&nbsp;<a href="'.esc_url($delete_note_url).'" class="give-delete-payment-note" data-note-id="'.absint($note->comment_ID).'" data-payment-id="'.absint($payment_id).'" aria-label="'.__('Delete this donation note.', 'give').'">'.__('Delete', 'give').'</a>';
 	$note_html .= '</p>';
 	$note_html .= '</div>';
 
@@ -1578,18 +1576,18 @@ 
  *
  * @return void
  */
-function give_hide_payment_notes( $query ) {
-	if ( version_compare( floatval( get_bloginfo( 'version' ) ), '4.1', '>=' ) ) {
-		$types = isset( $query->query_vars['type__not_in'] ) ? $query->query_vars['type__not_in'] : array();
-		if ( ! is_array( $types ) ) {
-			$types = array( $types );
+function give_hide_payment_notes($query) {
+	if (version_compare(floatval(get_bloginfo('version')), '4.1', '>=')) {
+		$types = isset($query->query_vars['type__not_in']) ? $query->query_vars['type__not_in'] : array();
+		if ( ! is_array($types)) {
+			$types = array($types);
 		}
 		$types[]                           = 'give_payment_note';
 		$query->query_vars['type__not_in'] = $types;
 	}
 }
 
-add_action( 'pre_get_comments', 'give_hide_payment_notes', 10 );
+add_action('pre_get_comments', 'give_hide_payment_notes', 10);
 
 /**
  * Exclude notes (comments) on give_payment post type from showing in Recent Comments widgets
@@ -1601,15 +1599,15 @@ 
  *
  * @return array $clauses Updated comment clauses.
  */
-function give_hide_payment_notes_pre_41( $clauses, $wp_comment_query ) {
-	if ( version_compare( floatval( get_bloginfo( 'version' ) ), '4.1', '<' ) ) {
+function give_hide_payment_notes_pre_41($clauses, $wp_comment_query) {
+	if (version_compare(floatval(get_bloginfo('version')), '4.1', '<')) {
 		$clauses['where'] .= ' AND comment_type != "give_payment_note"';
 	}
 
 	return $clauses;
 }
 
-add_filter( 'comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2 );
+add_filter('comments_clauses', 'give_hide_payment_notes_pre_41', 10, 2);
 
 
 /**
@@ -1622,15 +1620,15 @@ 
  *
  * @return string $where
  */
-function give_hide_payment_notes_from_feeds( $where, $wp_comment_query ) {
+function give_hide_payment_notes_from_feeds($where, $wp_comment_query) {
 	global $wpdb;
 
-	$where .= $wpdb->prepare( ' AND comment_type != %s', 'give_payment_note' );
+	$where .= $wpdb->prepare(' AND comment_type != %s', 'give_payment_note');
 
 	return $where;
 }
 
-add_filter( 'comment_feed_where', 'give_hide_payment_notes_from_feeds', 10, 2 );
+add_filter('comment_feed_where', 'give_hide_payment_notes_from_feeds', 10, 2);
 
 
 /**
@@ -1644,32 +1642,32 @@ 
  *
  * @return array|object Array of comment counts.
  */
-function give_remove_payment_notes_in_comment_counts( $stats, $post_id ) {
+function give_remove_payment_notes_in_comment_counts($stats, $post_id) {
 	global $wpdb, $pagenow;
 
-	if ( 'index.php' != $pagenow ) {
+	if ('index.php' != $pagenow) {
 		return $stats;
 	}
 
 	$post_id = (int) $post_id;
 
-	if ( apply_filters( 'give_count_payment_notes_in_comments', false ) ) {
+	if (apply_filters('give_count_payment_notes_in_comments', false)) {
 		return $stats;
 	}
 
-	$stats = Give_Cache::get_group( "comments-{$post_id}", 'counts' );
+	$stats = Give_Cache::get_group("comments-{$post_id}", 'counts');
 
-	if ( ! is_null( $stats ) ) {
+	if ( ! is_null($stats)) {
 		return $stats;
 	}
 
 	$where = 'WHERE comment_type != "give_payment_note"';
 
-	if ( $post_id > 0 ) {
-		$where .= $wpdb->prepare( ' AND comment_post_ID = %d', $post_id );
+	if ($post_id > 0) {
+		$where .= $wpdb->prepare(' AND comment_post_ID = %d', $post_id);
 	}
 
-	$count = $wpdb->get_results( "SELECT comment_approved, COUNT( * ) AS num_comments FROM {$wpdb->comments} {$where} GROUP BY comment_approved", ARRAY_A );
+	$count = $wpdb->get_results("SELECT comment_approved, COUNT( * ) AS num_comments FROM {$wpdb->comments} {$where} GROUP BY comment_approved", ARRAY_A);
 
 	$total    = 0;
 	$approved = array(
@@ -1679,30 +1677,30 @@ 
 		'trash'        => 'trash',
 		'post-trashed' => 'post-trashed',
 	);
-	foreach ( (array) $count as $row ) {
+	foreach ((array) $count as $row) {
 		// Don't count post-trashed toward totals.
-		if ( 'post-trashed' != $row['comment_approved'] && 'trash' != $row['comment_approved'] ) {
+		if ('post-trashed' != $row['comment_approved'] && 'trash' != $row['comment_approved']) {
 			$total += $row['num_comments'];
 		}
-		if ( isset( $approved[ $row['comment_approved'] ] ) ) {
-			$stats[ $approved[ $row['comment_approved'] ] ] = $row['num_comments'];
+		if (isset($approved[$row['comment_approved']])) {
+			$stats[$approved[$row['comment_approved']]] = $row['num_comments'];
 		}
 	}
 
 	$stats['total_comments'] = $total;
-	foreach ( $approved as $key ) {
-		if ( empty( $stats[ $key ] ) ) {
-			$stats[ $key ] = 0;
+	foreach ($approved as $key) {
+		if (empty($stats[$key])) {
+			$stats[$key] = 0;
 		}
 	}
 
 	$stats = (object) $stats;
-	Give_Cache::set_group( "comments-{$post_id}", $stats, 'counts' );
+	Give_Cache::set_group("comments-{$post_id}", $stats, 'counts');
 
 	return $stats;
 }
 
-add_filter( 'wp_count_comments', 'give_remove_payment_notes_in_comment_counts', 10, 2 );
+add_filter('wp_count_comments', 'give_remove_payment_notes_in_comment_counts', 10, 2);
 
 
 /**
@@ -1715,9 +1713,9 @@ 
  *
  * @return string $where Modified where clause.
  */
-function give_filter_where_older_than_week( $where = '' ) {
+function give_filter_where_older_than_week($where = '') {
 	// Payments older than one week.
-	$start = date( 'Y-m-d', strtotime( '-7 days' ) );
+	$start = date('Y-m-d', strtotime('-7 days'));
 	$where .= " AND post_date <= '{$start}'";
 
 	return $where;
@@ -1737,13 +1735,13 @@ 
  *
  * @return string $form_title Returns the full title if $only_level is false, otherwise returns the levels title.
  */
-function give_get_donation_form_title( $donation, $args = array() ) {
+function give_get_donation_form_title($donation, $args = array()) {
 
-	if ( ! $donation instanceof Give_Payment ) {
-		$donation = new Give_Payment( $donation );
+	if ( ! $donation instanceof Give_Payment) {
+		$donation = new Give_Payment($donation);
 	}
 
-	if( ! $donation->ID ) {
+	if ( ! $donation->ID) {
 		return '';
 	}
 
@@ -1752,7 +1750,7 @@ 
 		'separator'  => '',
 	);
 
-	$args = wp_parse_args( $args, $defaults );
+	$args = wp_parse_args($args, $defaults);
 
 	$form_id     = $donation->form_id;
 	$price_id    = $donation->price_id;
@@ -1773,39 +1771,39 @@ 
 		, false
 	);
 
-	$form_title_html = Give_Cache::get_db_query( $cache_key );
+	$form_title_html = Give_Cache::get_db_query($cache_key);
 
-	if ( is_null( $form_title_html ) ) {
-		if ( true === $only_level ) {
+	if (is_null($form_title_html)) {
+		if (true === $only_level) {
 			$form_title = '';
 		}
 
 		$form_title_html = $form_title;
 
-		if ( 'custom' === $price_id ) {
+		if ('custom' === $price_id) {
 
-			$custom_amount_text = give_get_meta( $form_id, '_give_custom_amount_text', true );
-			$level_label        = ! empty( $custom_amount_text ) ? $custom_amount_text : __( 'Custom Amount', 'give' );
+			$custom_amount_text = give_get_meta($form_id, '_give_custom_amount_text', true);
+			$level_label        = ! empty($custom_amount_text) ? $custom_amount_text : __('Custom Amount', 'give');
 
 			// Show custom amount level only in backend otherwise hide it.
-			if( 'set' === give_get_meta( $form_id, '_give_price_option', true ) && ! is_admin()  ) {
+			if ('set' === give_get_meta($form_id, '_give_price_option', true) && ! is_admin()) {
 				$level_label = '';
 			}
 
-		} elseif ( give_has_variable_prices( $form_id ) ) {
-			$level_label = give_get_price_option_name( $form_id, $price_id, $donation->ID, false );
+		} elseif (give_has_variable_prices($form_id)) {
+			$level_label = give_get_price_option_name($form_id, $price_id, $donation->ID, false);
 		}
 
 		// Only add separator if there is a form title.
 		if (
-			! empty( $form_title_html ) &&
-			! empty( $level_label )
+			! empty($form_title_html) &&
+			! empty($level_label)
 		) {
 			$form_title_html .= " {$separator} ";
 		}
 
 		$form_title_html .= "<span class=\"donation-level-text-wrap\">{$level_label}</span>";
-		Give_Cache::set_db_query( $cache_key, $form_title_html );
+		Give_Cache::set_db_query($cache_key, $form_title_html);
 	}
 
 	/**
@@ -1813,7 +1811,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	return apply_filters( 'give_get_donation_form_title', $form_title_html, $donation->payment_meta, $donation );
+	return apply_filters('give_get_donation_form_title', $form_title_html, $donation->payment_meta, $donation);
 }
 
 /**
@@ -1826,19 +1824,19 @@ 
  *
  * @return string $price_id
  */
-function give_get_price_id( $form_id, $price ) {
+function give_get_price_id($form_id, $price) {
 	$price_id = null;
 
-	if ( give_has_variable_prices( $form_id ) ) {
+	if (give_has_variable_prices($form_id)) {
 
-		$levels = give_get_meta( $form_id, '_give_donation_levels', true );
+		$levels = give_get_meta($form_id, '_give_donation_levels', true);
 
-		foreach ( $levels as $level ) {
+		foreach ($levels as $level) {
 
-			$level_amount = give_maybe_sanitize_amount( $level['_give_amount'] );
+			$level_amount = give_maybe_sanitize_amount($level['_give_amount']);
 
 			// Check that this indeed the recurring price.
-			if ( $level_amount == $price ) {
+			if ($level_amount == $price) {
 
 				$price_id = $level['_give_id']['level_id'];
 				break;
@@ -1846,13 +1844,13 @@ 
 			}
 		}
 
-		if ( is_null( $price_id ) && give_is_custom_price_mode( $form_id ) ) {
+		if (is_null($price_id) && give_is_custom_price_mode($form_id)) {
 			$price_id = 'custom';
 		}
 	}
 
 	// Price ID must be numeric or string.
-	$price_id = ! is_numeric( $price_id ) && ! is_string( $price_id ) ? 0 : $price_id;
+	$price_id = ! is_numeric($price_id) && ! is_string($price_id) ? 0 : $price_id;
 
 	/**
 	 * Filter the price id
@@ -1862,7 +1860,7 @@ 
 	 * @param string $price_id
 	 * @param int    $form_id
 	 */
-	return apply_filters( 'give_get_price_id', $price_id, $form_id );
+	return apply_filters('give_get_price_id', $price_id, $form_id);
 }
 
 /**
@@ -1878,10 +1876,10 @@ 
  *
  * @return string
  */
-function give_get_form_dropdown( $args = array(), $echo = false ) {
-	$form_dropdown_html = Give()->html->forms_dropdown( $args );
+function give_get_form_dropdown($args = array(), $echo = false) {
+	$form_dropdown_html = Give()->html->forms_dropdown($args);
 
-	if ( ! $echo ) {
+	if ( ! $echo) {
 		return $form_dropdown_html;
 	}
 
@@ -1898,17 +1896,17 @@ 
  *
  * @return string|bool
  */
-function give_get_form_variable_price_dropdown( $args = array(), $echo = false ) {
+function give_get_form_variable_price_dropdown($args = array(), $echo = false) {
 
 	// Check for give form id.
-	if ( empty( $args['id'] ) ) {
+	if (empty($args['id'])) {
 		return false;
 	}
 
-	$form = new Give_Donate_Form( $args['id'] );
+	$form = new Give_Donate_Form($args['id']);
 
 	// Check if form has variable prices or not.
-	if ( ! $form->ID || ! $form->has_variable_prices() ) {
+	if ( ! $form->ID || ! $form->has_variable_prices()) {
 		return false;
 	}
 
@@ -1916,24 +1914,24 @@ 
 	$variable_price_options = array();
 
 	// Check if multi donation form support custom donation or not.
-	if ( $form->is_custom_price_mode() ) {
-		$variable_price_options['custom'] = _x( 'Custom', 'custom donation dropdown item', 'give' );
+	if ($form->is_custom_price_mode()) {
+		$variable_price_options['custom'] = _x('Custom', 'custom donation dropdown item', 'give');
 	}
 
 	// Get variable price and ID from variable price array.
-	foreach ( $variable_prices as $variable_price ) {
-		$variable_price_options[ $variable_price['_give_id']['level_id'] ] = ! empty( $variable_price['_give_text'] ) ? $variable_price['_give_text'] : give_currency_filter( give_format_amount( $variable_price['_give_amount'], array( 'sanitize' => false ) ) );
+	foreach ($variable_prices as $variable_price) {
+		$variable_price_options[$variable_price['_give_id']['level_id']] = ! empty($variable_price['_give_text']) ? $variable_price['_give_text'] : give_currency_filter(give_format_amount($variable_price['_give_amount'], array('sanitize' => false)));
 	}
 
 	// Update options.
-	$args = array_merge( $args, array(
+	$args = array_merge($args, array(
 		'options' => $variable_price_options,
-	) );
+	));
 
 	// Generate select html.
-	$form_dropdown_html = Give()->html->select( $args );
+	$form_dropdown_html = Give()->html->select($args);
 
-	if ( ! $echo ) {
+	if ( ! $echo) {
 		return $form_dropdown_html;
 	}
 
@@ -1952,14 +1950,14 @@ 
  *
  * @return string
  */
-function give_get_payment_meta_price_id( $payment_meta ) {
+function give_get_payment_meta_price_id($payment_meta) {
 
-	if ( isset( $payment_meta['give_price_id'] ) ) {
+	if (isset($payment_meta['give_price_id'])) {
 		$price_id = $payment_meta['give_price_id'];
-	} elseif ( isset( $payment_meta['price_id'] ) ) {
+	} elseif (isset($payment_meta['price_id'])) {
 		$price_id = $payment_meta['price_id'];
 	} else {
-		$price_id = give_get_price_id( $payment_meta['give_form_id'], $payment_meta['price'] );
+		$price_id = give_get_price_id($payment_meta['give_form_id'], $payment_meta['price']);
 	}
 
 	/**
@@ -1970,6 +1968,6 @@ 
 	 * @param string $price_id
 	 * @param array  $payment_meta
 	 */
-	return apply_filters( 'give_get_payment_meta_price_id', $price_id, $payment_meta );
+	return apply_filters('give_get_payment_meta_price_id', $price_id, $payment_meta);
 
 }

includes/class-give-db-form-meta.php

Spacing +3 added lines, -3 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -50,7 +50,7 @@ 
 		/* @var WPDB $wpdb */
 		global $wpdb;
 
-		$wpdb->formmeta    = $this->table_name = $wpdb->prefix . 'give_formmeta';
+		$wpdb->formmeta    = $this->table_name = $wpdb->prefix.'give_formmeta';
 		$this->primary_key = 'meta_id';
 		$this->version     = '1.0';
 
@@ -84,6 +84,6 @@ 
 	 * @return bool
 	 */
 	protected function is_custom_meta_table_active() {
-		return give_has_upgrade_completed( 'v20_move_metadata_into_new_table' );
+		return give_has_upgrade_completed('v20_move_metadata_into_new_table');
 	}
 }

includes/admin/tools/class-settings-logs.php

Spacing +13 added lines, -13 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_Logs' ) ) :
+if ( ! class_exists('Give_Settings_Logs')) :
 
 	/**
 	 * Give_Settings_Logs.
@@ -34,7 +34,7 @@ 
 		 */
 		public function __construct() {
 			$this->id    = 'logs';
-			$this->label = __( 'Logs', 'give' );
+			$this->label = __('Logs', 'give');
 
 			$this->default_tab = 'sales';
 
@@ -50,7 +50,7 @@ 
 		 */
 		public function get_settings() {
 			// Get settings.
-			$settings = apply_filters( 'give_settings_logs', array(
+			$settings = apply_filters('give_settings_logs', array(
 				array(
 					'id'         => 'give_tools_logs',
 					'type'       => 'title',
@@ -58,7 +58,7 @@ 
 				),
 				array(
 					'id'   => 'logs',
-					'name' => __( 'Log', 'give' ),
+					'name' => __('Log', 'give'),
 					'type' => 'logs',
 
 				),
@@ -67,7 +67,7 @@ 
 					'type'       => 'sectionend',
 					'table_html' => false,
 				),
-			) );
+			));
 
 			/**
 			 * Filter the settings.
@@ -76,7 +76,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;
@@ -90,15 +90,15 @@ 
 		 */
 		public function get_sections() {
 			$sections = array(
-				'sales'          => __( 'Donations', 'give' ),
-				'gateway_errors' => __( 'Payment Errors', 'give' ),
-				'api_requests'   => __( 'API Requests', 'give' ),
-				'updates'   => __( 'Updates', 'give' ),
+				'sales'          => __('Donations', 'give'),
+				'gateway_errors' => __('Payment Errors', 'give'),
+				'api_requests'   => __('API Requests', 'give'),
+				'updates'   => __('Updates', 'give'),
 			);
 
-			$sections = apply_filters( 'give_log_views', $sections );
+			$sections = apply_filters('give_log_views', $sections);
 
-			return apply_filters( 'give_get_sections_' . $this->id, $sections );
+			return apply_filters('give_get_sections_'.$this->id, $sections);
 		}
 	}
 

includes/admin/tools/logs/logs.php

Spacing +26 added lines, -26 removed lines

@@ -10,7 +10,7 @@ 
  */
 
 // Exit if accessed directly.
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -22,7 +22,7 @@ 
  */
 function give_get_logs_tab() {
 
-	require( GIVE_PLUGIN_DIR . 'includes/admin/tools/logs/logs.php' );
+	require(GIVE_PLUGIN_DIR.'includes/admin/tools/logs/logs.php');
 
 	// Get current section.
 	$current_section = $_GET['section'] = give_get_current_setting_section();
@@ -32,7 +32,7 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( "give_logs_view_{$current_section}" );
+	do_action("give_logs_view_{$current_section}");
 }
 
 
@@ -46,7 +46,7 @@ 
  */
 function give_logs_view_sales() {
 
-	include GIVE_PLUGIN_DIR . 'includes/admin/tools/logs/class-sales-logs-list-table.php';
+	include GIVE_PLUGIN_DIR.'includes/admin/tools/logs/class-sales-logs-list-table.php';
 
 	$logs_table = new Give_Sales_Log_Table();
 	$logs_table->prepare_items();
@@ -59,7 +59,7 @@ 
 		 *
 		 * @since 1.8.12
 		 */
-		do_action( 'give_logs_donations_top' );
+		do_action('give_logs_donations_top');
 
 		$logs_table->display(); ?>
 		<input type="hidden" name="post_type" value="give_forms"/>
@@ -73,14 +73,14 @@ 
 		 *
 		 * @since 1.8.12
 		 */
-		do_action( 'give_logs_donations_bottom' );
+		do_action('give_logs_donations_bottom');
 		?>
 
 	</div>
 	<?php
 }
 
-add_action( 'give_logs_view_sales', 'give_logs_view_sales' );
+add_action('give_logs_view_sales', 'give_logs_view_sales');
 
 /**
  * Update Logs
@@ -90,7 +90,7 @@ 
  * @return void
  */
 function give_logs_view_updates() {
-	include( GIVE_PLUGIN_DIR . 'includes/admin/tools/logs/class-update-logs-list-table.php' );
+	include(GIVE_PLUGIN_DIR.'includes/admin/tools/logs/class-update-logs-list-table.php');
 
 	$logs_table = new Give_Update_Log_Table();
 	$logs_table->prepare_items();
@@ -103,7 +103,7 @@ 
 		 *
 		 * @since 2.0.1
 		 */
-		do_action( 'give_logs_update_top' );
+		do_action('give_logs_update_top');
 
 		$logs_table->display(); ?>
 		<input type="hidden" name="post_type" value="give_forms"/>
@@ -117,14 +117,14 @@ 
 		 *
 		 * @since 2.0.1
 		 */
-		do_action( 'give_logs_update_bottom' );
+		do_action('give_logs_update_bottom');
 		?>
 
 	</div>
 	<?php
 }
 
-add_action( 'give_logs_view_updates', 'give_logs_view_updates' );
+add_action('give_logs_view_updates', 'give_logs_view_updates');
 
 /**
  * Gateway Error Logs
@@ -135,7 +135,7 @@ 
  * @return void
  */
 function give_logs_view_gateway_errors() {
-	include( GIVE_PLUGIN_DIR . 'includes/admin/tools/logs/class-gateway-error-logs-list-table.php' );
+	include(GIVE_PLUGIN_DIR.'includes/admin/tools/logs/class-gateway-error-logs-list-table.php');
 
 	$logs_table = new Give_Gateway_Error_Log_Table();
 	$logs_table->prepare_items();
@@ -148,7 +148,7 @@ 
 		 *
 		 * @since 1.8.12
 		 */
-		do_action( 'give_logs_payment_error_top' );
+		do_action('give_logs_payment_error_top');
 
 		$logs_table->display(); ?>
 		<input type="hidden" name="post_type" value="give_forms"/>
@@ -162,14 +162,14 @@ 
 		 *
 		 * @since 1.8.12
 		 */
-		do_action( 'give_logs_payment_error_bottom' );
+		do_action('give_logs_payment_error_bottom');
 		?>
 
 	</div>
 	<?php
 }
 
-add_action( 'give_logs_view_gateway_errors', 'give_logs_view_gateway_errors' );
+add_action('give_logs_view_gateway_errors', 'give_logs_view_gateway_errors');
 
 /**
  * API Request Logs
@@ -180,7 +180,7 @@ 
  * @return void
  */
 function give_logs_view_api_requests() {
-	include( GIVE_PLUGIN_DIR . 'includes/admin/tools/logs/class-api-requests-logs-list-table.php' );
+	include(GIVE_PLUGIN_DIR.'includes/admin/tools/logs/class-api-requests-logs-list-table.php');
 
 	$logs_table = new Give_API_Request_Log_Table();
 	$logs_table->prepare_items();
@@ -190,9 +190,9 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_logs_api_requests_top' );
+	do_action('give_logs_api_requests_top');
 
-	$logs_table->search_box( esc_html__( 'Search', 'give' ), 'give-api-requests' );
+	$logs_table->search_box(esc_html__('Search', 'give'), 'give-api-requests');
 	$logs_table->display();
 	?>
 	<input type="hidden" name="post_type" value="give_forms"/>
@@ -206,9 +206,9 @@ 
 	 *
 	 * @since 1.0
 	 */
-	do_action( 'give_logs_api_requests_bottom' );
+	do_action('give_logs_api_requests_bottom');
 }
-add_action( 'give_logs_view_api_requests', 'give_logs_view_api_requests' );
+add_action('give_logs_view_api_requests', 'give_logs_view_api_requests');
 
 /**
  * Renders the log views drop down.
@@ -220,11 +220,11 @@ 
 	$current_section = give_get_current_setting_section();
 
 	// If there are not any event attach to action then do not show form.
-	if ( ! has_action( 'give_log_view_actions' ) ) {
+	if ( ! has_action('give_log_view_actions')) {
 		return;
 	}
 	?>
-	<form id="give-logs-filter" method="get" action="<?php echo 'edit.php?post_type=give_forms&page=give-tools&tab=logs&section=' . $current_section; ?>">
+	<form id="give-logs-filter" method="get" action="<?php echo 'edit.php?post_type=give_forms&page=give-tools&tab=logs&section='.$current_section; ?>">
 		<?php
 		/**
 		 * Fires after displaying the reports page views drop down.
@@ -233,14 +233,14 @@ 
 		 *
 		 * @since 1.0
 		 */
-		do_action( 'give_log_view_actions' );
+		do_action('give_log_view_actions');
 		?>
 
 		<input type="hidden" name="post_type" value="give_forms"/>
 		<input type="hidden" name="page" value="give-tools"/>
 		<input type="hidden" name="tab" value="logs"/>
 
-		<?php submit_button( esc_html__( 'Apply', 'give' ), 'secondary', 'submit', false ); ?>
+		<?php submit_button(esc_html__('Apply', 'give'), 'secondary', 'submit', false); ?>
 	</form>
 	<?php
 }
@@ -252,7 +252,7 @@ 
  *
  * @return string
  */
-function give_tools_set_form_method( $method ) {
+function give_tools_set_form_method($method) {
 	return 'get';
 }
-add_filter( 'give-tools_form_method_tab_logs', 'give_tools_set_form_method', 10 );
+add_filter('give-tools_form_method_tab_logs', 'give_tools_set_form_method', 10);