WordImpress / Give

includes/admin/payments/class-payments-table.php

Doc Comments -1 removed lines

@@ -304,7 +304,6 @@
 	 * @access public
 	 * @since  1.0
 	 *
-	 * @param array $item Contains all the data of the discount code
 	 * @param string $column_name The name of the column
 	 *
 	 * @return string Column Name

Spacing +178 added lines, -178 removed lines

@@ -10,13 +10,13 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
 // Load WP_List_Table if not loaded
-if ( ! class_exists( 'WP_List_Table' ) ) {
-	require_once ABSPATH . 'wp-admin/includes/class-wp-list-table.php';
+if ( ! class_exists('WP_List_Table')) {
+	require_once ABSPATH.'wp-admin/includes/class-wp-list-table.php';
 }
 
 /**
@@ -120,37 +120,37 @@ 
 		global $status, $page;
 
 		// Set parent defaults
-		parent::__construct( array(
-			'singular' => give_get_forms_label_singular(),    // Singular name of the listed records
-			'plural'   => give_get_forms_label_plural(),        // Plural name of the listed records
+		parent::__construct(array(
+			'singular' => give_get_forms_label_singular(), // Singular name of the listed records
+			'plural'   => give_get_forms_label_plural(), // Plural name of the listed records
 			'ajax'     => false                        // Does this table support ajax?
-		) );
+		));
 
 		$this->get_payment_counts();
 		$this->process_bulk_action();
-		$this->base_url = admin_url( 'edit.php?post_type=give_forms&page=give-payment-history' );
+		$this->base_url = admin_url('edit.php?post_type=give_forms&page=give-payment-history');
 	}
 
 	public function advanced_filters() {
-		$start_date = isset( $_GET['start-date'] ) ? sanitize_text_field( $_GET['start-date'] ) : null;
-		$end_date   = isset( $_GET['end-date'] ) ? sanitize_text_field( $_GET['end-date'] ) : null;
-		$status     = isset( $_GET['status'] ) ? $_GET['status'] : '';
+		$start_date = isset($_GET['start-date']) ? sanitize_text_field($_GET['start-date']) : null;
+		$end_date   = isset($_GET['end-date']) ? sanitize_text_field($_GET['end-date']) : null;
+		$status     = isset($_GET['status']) ? $_GET['status'] : '';
 		?>
 		<div id="give-payment-filters">
 			<span id="give-payment-date-filters">
-				<label for="start-date" class="give-start-date-label"><?php _e( 'Start Date:', 'give' ); ?></label>
+				<label for="start-date" class="give-start-date-label"><?php _e('Start Date:', 'give'); ?></label>
 				<input type="text" id="start-date" name="start-date" class="give_datepicker" value="<?php echo $start_date; ?>" placeholder="mm/dd/yyyy"/>
-				<label for="end-date" class="give-end-date-label"><?php _e( 'End Date:', 'give' ); ?></label>
+				<label for="end-date" class="give-end-date-label"><?php _e('End Date:', 'give'); ?></label>
 				<input type="text" id="end-date" name="end-date" class="give_datepicker" value="<?php echo $end_date; ?>" placeholder="mm/dd/yyyy"/>
-				<input type="submit" class="button-secondary" value="<?php _e( 'Apply', 'give' ); ?>"/>
+				<input type="submit" class="button-secondary" value="<?php _e('Apply', 'give'); ?>"/>
 			</span>
-			<?php if ( ! empty( $status ) ) : ?>
-				<input type="hidden" name="status" value="<?php echo esc_attr( $status ); ?>"/>
+			<?php if ( ! empty($status)) : ?>
+				<input type="hidden" name="status" value="<?php echo esc_attr($status); ?>"/>
 			<?php endif; ?>
-			<?php if ( ! empty( $start_date ) || ! empty( $end_date ) ) : ?>
-				<a href="<?php echo admin_url( 'edit.php?post_type=give_forms&page=give-payment-history' ); ?>" class="button-secondary"><?php _e( 'Clear Filter', 'give' ); ?></a>
+			<?php if ( ! empty($start_date) || ! empty($end_date)) : ?>
+				<a href="<?php echo admin_url('edit.php?post_type=give_forms&page=give-payment-history'); ?>" class="button-secondary"><?php _e('Clear Filter', 'give'); ?></a>
 			<?php endif; ?>
-			<?php $this->search_box( __( 'Search', 'give' ), 'give-payments' ); ?>
+			<?php $this->search_box(__('Search', 'give'), 'give-payments'); ?>
 		</div>
 
 		<?php
@@ -167,25 +167,25 @@ 
 	 *
 	 * @return void
 	 */
-	public function search_box( $text, $input_id ) {
-		if ( empty( $_REQUEST['s'] ) && ! $this->has_items() ) {
+	public function search_box($text, $input_id) {
+		if (empty($_REQUEST['s']) && ! $this->has_items()) {
 			return;
 		}
 
-		$input_id = $input_id . '-search-input';
+		$input_id = $input_id.'-search-input';
 
-		if ( ! empty( $_REQUEST['orderby'] ) ) {
-			echo '<input type="hidden" name="orderby" value="' . esc_attr( $_REQUEST['orderby'] ) . '" />';
+		if ( ! empty($_REQUEST['orderby'])) {
+			echo '<input type="hidden" name="orderby" value="'.esc_attr($_REQUEST['orderby']).'" />';
 		}
-		if ( ! empty( $_REQUEST['order'] ) ) {
-			echo '<input type="hidden" name="order" value="' . esc_attr( $_REQUEST['order'] ) . '" />';
+		if ( ! empty($_REQUEST['order'])) {
+			echo '<input type="hidden" name="order" value="'.esc_attr($_REQUEST['order']).'" />';
 		}
 		?>
 		<p class="search-box">
-			<?php do_action( 'give_payment_history_search' ); ?>
+			<?php do_action('give_payment_history_search'); ?>
 			<label class="screen-reader-text" for="<?php echo $input_id ?>"><?php echo $text; ?>:</label>
 			<input type="search" id="<?php echo $input_id ?>" name="s" value="<?php _admin_search_query(); ?>"/>
-			<?php submit_button( $text, 'button', false, false, array( 'ID' => 'search-submit' ) ); ?><br/>
+			<?php submit_button($text, 'button', false, false, array('ID' => 'search-submit')); ?><br/>
 		</p>
 		<?php
 	}
@@ -199,52 +199,52 @@ 
 	 */
 	public function get_views() {
 
-		$current         = isset( $_GET['status'] ) ? $_GET['status'] : '';
-		$total_count     = '&nbsp;<span class="count">(' . $this->total_count . ')</span>';
-		$complete_count  = '&nbsp;<span class="count">(' . $this->complete_count . ')</span>';
-		$cancelled_count = '&nbsp;<span class="count">(' . $this->cancelled_count . ')</span>';
-		$pending_count   = '&nbsp;<span class="count">(' . $this->pending_count . ')</span>';
-		$refunded_count  = '&nbsp;<span class="count">(' . $this->refunded_count . ')</span>';
-		$failed_count    = '&nbsp;<span class="count">(' . $this->failed_count . ')</span>';
-		$abandoned_count = '&nbsp;<span class="count">(' . $this->abandoned_count . ')</span>';
-		$revoked_count   = '&nbsp;<span class="count">(' . $this->revoked_count . ')</span>';
+		$current         = isset($_GET['status']) ? $_GET['status'] : '';
+		$total_count     = '&nbsp;<span class="count">('.$this->total_count.')</span>';
+		$complete_count  = '&nbsp;<span class="count">('.$this->complete_count.')</span>';
+		$cancelled_count = '&nbsp;<span class="count">('.$this->cancelled_count.')</span>';
+		$pending_count   = '&nbsp;<span class="count">('.$this->pending_count.')</span>';
+		$refunded_count  = '&nbsp;<span class="count">('.$this->refunded_count.')</span>';
+		$failed_count    = '&nbsp;<span class="count">('.$this->failed_count.')</span>';
+		$abandoned_count = '&nbsp;<span class="count">('.$this->abandoned_count.')</span>';
+		$revoked_count   = '&nbsp;<span class="count">('.$this->revoked_count.')</span>';
 
 		$views = array(
-			'all'       => sprintf( '<a href="%s"%s>%s</a>', remove_query_arg( array(
+			'all'       => sprintf('<a href="%s"%s>%s</a>', remove_query_arg(array(
 				'status',
 				'paged'
-			) ), $current === 'all' || $current == '' ? ' class="current"' : '', __( 'All', 'give' ) . $total_count ),
-			'publish'   => sprintf( '<a href="%s"%s>%s</a>', esc_url( add_query_arg( array(
+			)), $current === 'all' || $current == '' ? ' class="current"' : '', __('All', 'give').$total_count),
+			'publish'   => sprintf('<a href="%s"%s>%s</a>', esc_url(add_query_arg(array(
 				'status' => 'publish',
 				'paged'  => false
-			) ) ), $current === 'publish' ? ' class="current"' : '', __( 'Completed', 'give' ) . $complete_count ),
-			'pending'   => sprintf( '<a href="%s"%s>%s</a>', esc_url( add_query_arg( array(
+			))), $current === 'publish' ? ' class="current"' : '', __('Completed', 'give').$complete_count),
+			'pending'   => sprintf('<a href="%s"%s>%s</a>', esc_url(add_query_arg(array(
 				'status' => 'pending',
 				'paged'  => false
-			) ) ), $current === 'pending' ? ' class="current"' : '', __( 'Pending', 'give' ) . $pending_count ),
-			'refunded'  => sprintf( '<a href="%s"%s>%s</a>', esc_url( add_query_arg( array(
+			))), $current === 'pending' ? ' class="current"' : '', __('Pending', 'give').$pending_count),
+			'refunded'  => sprintf('<a href="%s"%s>%s</a>', esc_url(add_query_arg(array(
 				'status' => 'refunded',
 				'paged'  => false
-			) ) ), $current === 'refunded' ? ' class="current"' : '', __( 'Refunded', 'give' ) . $refunded_count ),
-			'revoked'   => sprintf( '<a href="%s"%s>%s</a>', esc_url( add_query_arg( array(
+			))), $current === 'refunded' ? ' class="current"' : '', __('Refunded', 'give').$refunded_count),
+			'revoked'   => sprintf('<a href="%s"%s>%s</a>', esc_url(add_query_arg(array(
 				'status' => 'revoked',
 				'paged'  => false
-			) ) ), $current === 'revoked' ? ' class="current"' : '', __( 'Revoked', 'give' ) . $revoked_count ),
-			'failed'    => sprintf( '<a href="%s"%s>%s</a>', esc_url( add_query_arg( array(
+			))), $current === 'revoked' ? ' class="current"' : '', __('Revoked', 'give').$revoked_count),
+			'failed'    => sprintf('<a href="%s"%s>%s</a>', esc_url(add_query_arg(array(
 				'status' => 'failed',
 				'paged'  => false
-			) ) ), $current === 'failed' ? ' class="current"' : '', __( 'Failed', 'give' ) . $failed_count ),
-			'cancelled' => sprintf( '<a href="%s"%s>%s</a>', esc_url( add_query_arg( array(
+			))), $current === 'failed' ? ' class="current"' : '', __('Failed', 'give').$failed_count),
+			'cancelled' => sprintf('<a href="%s"%s>%s</a>', esc_url(add_query_arg(array(
 				'status' => 'cancelled',
 				'paged'  => false
-			) ) ), $current === 'cancelled' ? ' class="current"' : '', __( 'Cancelled', 'give' ) . $cancelled_count ),
-			'abandoned' => sprintf( '<a href="%s"%s>%s</a>', esc_url( add_query_arg( array(
+			))), $current === 'cancelled' ? ' class="current"' : '', __('Cancelled', 'give').$cancelled_count),
+			'abandoned' => sprintf('<a href="%s"%s>%s</a>', esc_url(add_query_arg(array(
 				'status' => 'abandoned',
 				'paged'  => false
-			) ) ), $current === 'abandoned' ? ' class="current"' : '', __( 'Abandoned', 'give' ) . $abandoned_count )
+			))), $current === 'abandoned' ? ' class="current"' : '', __('Abandoned', 'give').$abandoned_count)
 		);
 
-		return apply_filters( 'give_payments_table_views', $views );
+		return apply_filters('give_payments_table_views', $views);
 	}
 
 	/**
@@ -257,16 +257,16 @@ 
 	public function get_columns() {
 		$columns = array(
 			'cb'      => '<input type="checkbox" />', //Render a checkbox instead of text
-			'email'   => __( 'Email', 'give' ),
-			'details' => __( 'Details', 'give' ),
-			'amount'  => __( 'Amount', 'give' ),
-			'status'  => __( 'Status', 'give' ),
-			'date'    => __( 'Date', 'give' ),
-			'donor'   => __( 'Donor', 'give' ),
-			'ID'      => __( 'ID', 'give' ),
+			'email'   => __('Email', 'give'),
+			'details' => __('Details', 'give'),
+			'amount'  => __('Amount', 'give'),
+			'status'  => __('Status', 'give'),
+			'date'    => __('Date', 'give'),
+			'donor'   => __('Donor', 'give'),
+			'ID'      => __('ID', 'give'),
 		);
 
-		return apply_filters( 'give_payments_table_columns', $columns );
+		return apply_filters('give_payments_table_columns', $columns);
 	}
 
 	/**
@@ -278,12 +278,12 @@ 
 	 */
 	public function get_sortable_columns() {
 		$columns = array(
-			'ID'     => array( 'ID', true ),
-			'amount' => array( 'amount', false ),
-			'date'   => array( 'date', false )
+			'ID'     => array('ID', true),
+			'amount' => array('amount', false),
+			'date'   => array('date', false)
 		);
 
-		return apply_filters( 'give_payments_table_sortable_columns', $columns );
+		return apply_filters('give_payments_table_sortable_columns', $columns);
 	}
 
 	/**
@@ -309,32 +309,32 @@ 
 	 *
 	 * @return string Column Name
 	 */
-	public function column_default( $payment, $column_name ) {
-		switch ( $column_name ) {
+	public function column_default($payment, $column_name) {
+		switch ($column_name) {
 			case 'amount' :
-				$amount = ! empty( $payment->total ) ? $payment->total : 0;
-				$value  = give_currency_filter( give_format_amount( $amount ), give_get_payment_currency_code( $payment->ID ) );
+				$amount = ! empty($payment->total) ? $payment->total : 0;
+				$value  = give_currency_filter(give_format_amount($amount), give_get_payment_currency_code($payment->ID));
 				break;
 			case 'date' :
-				$date  = strtotime( $payment->date );
-				$value = date_i18n( get_option( 'date_format' ), $date );
+				$date  = strtotime($payment->date);
+				$value = date_i18n(get_option('date_format'), $date);
 				break;
 			case 'status' :
-				$value   = '<div class="give-donation-status status-' . sanitize_title( give_get_payment_status( $payment, true ) ) . '"><span class="give-donation-status-icon"></span> ' . give_get_payment_status( $payment, true ) . '</div>';
-				if ( $payment->mode == 'test' ) {
-					$value .= ' <span class="give-item-label give-item-label-orange give-test-mode-transactions-label" data-tooltip="' . __( 'This payment was made in test mode', 'give' ) . '">' . __( 'Test', 'give' ) . '</span>';
+				$value   = '<div class="give-donation-status status-'.sanitize_title(give_get_payment_status($payment, true)).'"><span class="give-donation-status-icon"></span> '.give_get_payment_status($payment, true).'</div>';
+				if ($payment->mode == 'test') {
+					$value .= ' <span class="give-item-label give-item-label-orange give-test-mode-transactions-label" data-tooltip="'.__('This payment was made in test mode', 'give').'">'.__('Test', 'give').'</span>';
 				}
 				break;
 			case 'details' :
-				$value = '<div class="give-payment-details-link-wrap"><a href="' . esc_url( add_query_arg( 'id', $payment->ID, admin_url( 'edit.php?post_type=give_forms&page=give-payment-history&view=view-order-details' ) ) ) . '" class="give-payment-details-link button button-small">' . __( 'View Details', 'give' ) . '</a></div>';
+				$value = '<div class="give-payment-details-link-wrap"><a href="'.esc_url(add_query_arg('id', $payment->ID, admin_url('edit.php?post_type=give_forms&page=give-payment-history&view=view-order-details'))).'" class="give-payment-details-link button button-small">'.__('View Details', 'give').'</a></div>';
 				break;
 			default:
-				$value = isset( $payment->$column_name ) ? $payment->$column_name : '';
+				$value = isset($payment->$column_name) ? $payment->$column_name : '';
 				break;
 
 		}
 
-		return apply_filters( 'give_payments_table_column', $value, $payment->ID, $column_name );
+		return apply_filters('give_payments_table_column', $value, $payment->ID, $column_name);
 	}
 
 	/**
@@ -347,40 +347,40 @@ 
 	 *
 	 * @return string Data shown in the Email column
 	 */
-	public function column_email( $payment ) {
+	public function column_email($payment) {
 
 		$row_actions = array();
 
-		$email = give_get_payment_user_email( $payment->ID );
+		$email = give_get_payment_user_email($payment->ID);
 
 		// Add search term string back to base URL
-		$search_terms = ( isset( $_GET['s'] ) ? trim( $_GET['s'] ) : '' );
-		if ( ! empty( $search_terms ) ) {
-			$this->base_url = add_query_arg( 's', $search_terms, $this->base_url );
+		$search_terms = (isset($_GET['s']) ? trim($_GET['s']) : '');
+		if ( ! empty($search_terms)) {
+			$this->base_url = add_query_arg('s', $search_terms, $this->base_url);
 		}
 
-		if ( give_is_payment_complete( $payment->ID ) && ! empty( $email ) ) {
-			$row_actions['email_links'] = '<a href="' . add_query_arg( array(
+		if (give_is_payment_complete($payment->ID) && ! empty($email)) {
+			$row_actions['email_links'] = '<a href="'.add_query_arg(array(
 					'give-action' => 'email_links',
 					'purchase_id' => $payment->ID
-				), $this->base_url ) . '">' . __( 'Resend Donation Receipt', 'give' ) . '</a>';
+				), $this->base_url).'">'.__('Resend Donation Receipt', 'give').'</a>';
 
 		}
 
-		$row_actions['delete'] = '<a href="' . wp_nonce_url( add_query_arg( array(
+		$row_actions['delete'] = '<a href="'.wp_nonce_url(add_query_arg(array(
 				'give-action' => 'delete_payment',
 				'purchase_id' => $payment->ID
-			), $this->base_url ), 'give_payment_nonce' ) . '">' . __( 'Delete', 'give' ) . '</a>';
+			), $this->base_url), 'give_payment_nonce').'">'.__('Delete', 'give').'</a>';
 
-		$row_actions = apply_filters( 'give_payment_row_actions', $row_actions, $payment );
+		$row_actions = apply_filters('give_payment_row_actions', $row_actions, $payment);
 
-		if ( empty($email) ) {
-			$email = __( '(unknown)', 'give' );
+		if (empty($email)) {
+			$email = __('(unknown)', 'give');
 		}
 
-		$value = '<span class="give-email-column-value">' . $email . '</span>' . $this->row_actions( $row_actions );
+		$value = '<span class="give-email-column-value">'.$email.'</span>'.$this->row_actions($row_actions);
 
-		return apply_filters( 'give_payments_table_column', $value, $payment->ID, 'email' );
+		return apply_filters('give_payments_table_column', $value, $payment->ID, 'email');
 	}
 
 	/**
@@ -393,7 +393,7 @@ 
 	 *
 	 * @return string Displays a checkbox
 	 */
-	public function column_cb( $payment ) {
+	public function column_cb($payment) {
 		return sprintf(
 			'<input type="checkbox" name="%1$s[]" value="%2$s" />',
 			'payment',
@@ -411,8 +411,8 @@ 
 	 *
 	 * @return string Displays a checkbox
 	 */
-	public function column_ID( $payment ) {
-		return '<span class="give-payment-id">' . give_get_payment_number( $payment->ID ) . '</span>';
+	public function column_ID($payment) {
+		return '<span class="give-payment-id">'.give_get_payment_number($payment->ID).'</span>';
 	}
 
 	/**
@@ -425,19 +425,19 @@ 
 	 *
 	 * @return string Data shown in the User column
 	 */
-	public function column_donor( $payment ) {
+	public function column_donor($payment) {
 
-		$customer_id = give_get_payment_customer_id( $payment->ID );
+		$customer_id = give_get_payment_customer_id($payment->ID);
 
-		if( ! empty( $customer_id ) ) {
-			$customer    = new Give_Customer( $customer_id );
-			$value = '<a href="' . esc_url( admin_url( "edit.php?post_type=give_forms&page=give-customers&view=overview&id=$customer_id" ) ) . '">' . $customer->name . '</a>';
+		if ( ! empty($customer_id)) {
+			$customer = new Give_Customer($customer_id);
+			$value = '<a href="'.esc_url(admin_url("edit.php?post_type=give_forms&page=give-customers&view=overview&id=$customer_id")).'">'.$customer->name.'</a>';
 		} else {
-			$email = give_get_payment_user_email( $payment->ID );
-			$value = '<a href="' . esc_url( admin_url( "edit.php?post_type=give_forms&page=give-payment-history&s=$email" ) ) . '">' . __( '(donor missing)', 'give' ) . '</a>';
+			$email = give_get_payment_user_email($payment->ID);
+			$value = '<a href="'.esc_url(admin_url("edit.php?post_type=give_forms&page=give-payment-history&s=$email")).'">'.__('(donor missing)', 'give').'</a>';
 		}
 
-		return apply_filters( 'give_payments_table_column', $value, $payment->ID, 'donor' );
+		return apply_filters('give_payments_table_column', $value, $payment->ID, 'donor');
 	}
 
 	/**
@@ -449,18 +449,18 @@ 
 	 */
 	public function get_bulk_actions() {
 		$actions = array(
-			'delete'               => __( 'Delete', 'give' ),
-			'set-status-publish'   => __( 'Set To Completed', 'give' ),
-			'set-status-pending'   => __( 'Set To Pending', 'give' ),
-			'set-status-refunded'  => __( 'Set To Refunded', 'give' ),
-			'set-status-revoked'   => __( 'Set To Revoked', 'give' ),
-			'set-status-failed'    => __( 'Set To Failed', 'give' ),
-			'set-status-cancelled' => __( 'Set To Cancelled', 'give' ),
-			'set-status-abandoned' => __( 'Set To Abandoned', 'give' ),
-			'resend-receipt'       => __( 'Resend Email Receipts', 'give' )
+			'delete'               => __('Delete', 'give'),
+			'set-status-publish'   => __('Set To Completed', 'give'),
+			'set-status-pending'   => __('Set To Pending', 'give'),
+			'set-status-refunded'  => __('Set To Refunded', 'give'),
+			'set-status-revoked'   => __('Set To Revoked', 'give'),
+			'set-status-failed'    => __('Set To Failed', 'give'),
+			'set-status-cancelled' => __('Set To Cancelled', 'give'),
+			'set-status-abandoned' => __('Set To Abandoned', 'give'),
+			'resend-receipt'       => __('Resend Email Receipts', 'give')
 		);
 
-		return apply_filters( 'give_payments_table_bulk_actions', $actions );
+		return apply_filters('give_payments_table_bulk_actions', $actions);
 	}
 
 	/**
@@ -471,61 +471,61 @@ 
 	 * @return void
 	 */
 	public function process_bulk_action() {
-		$ids    = isset( $_GET['payment'] ) ? $_GET['payment'] : false;
+		$ids    = isset($_GET['payment']) ? $_GET['payment'] : false;
 		$action = $this->current_action();
 
-		if ( ! is_array( $ids ) ) {
-			$ids = array( $ids );
+		if ( ! is_array($ids)) {
+			$ids = array($ids);
 		}
 
-		if ( empty( $action ) ) {
+		if (empty($action)) {
 			return;
 		}
 
-		foreach ( $ids as $id ) {
+		foreach ($ids as $id) {
 
 			// Detect when a bulk action is being triggered...
-			if ( 'delete' === $this->current_action() ) {
-				give_delete_purchase( $id );
+			if ('delete' === $this->current_action()) {
+				give_delete_purchase($id);
 			}
 
-			if ( 'set-status-publish' === $this->current_action() ) {
-				give_update_payment_status( $id, 'publish' );
+			if ('set-status-publish' === $this->current_action()) {
+				give_update_payment_status($id, 'publish');
 			}
 
-			if ( 'set-status-pending' === $this->current_action() ) {
-				give_update_payment_status( $id, 'pending' );
+			if ('set-status-pending' === $this->current_action()) {
+				give_update_payment_status($id, 'pending');
 			}
 
-			if ( 'set-status-refunded' === $this->current_action() ) {
-				give_update_payment_status( $id, 'refunded' );
+			if ('set-status-refunded' === $this->current_action()) {
+				give_update_payment_status($id, 'refunded');
 			}
 
-			if ( 'set-status-revoked' === $this->current_action() ) {
-				give_update_payment_status( $id, 'revoked' );
+			if ('set-status-revoked' === $this->current_action()) {
+				give_update_payment_status($id, 'revoked');
 			}
 
-			if ( 'set-status-failed' === $this->current_action() ) {
-				give_update_payment_status( $id, 'failed' );
+			if ('set-status-failed' === $this->current_action()) {
+				give_update_payment_status($id, 'failed');
 			}
 
-			if ( 'set-status-cancelled' === $this->current_action() ) {
-				give_update_payment_status( $id, 'cancelled' );
+			if ('set-status-cancelled' === $this->current_action()) {
+				give_update_payment_status($id, 'cancelled');
 			}
 
-			if ( 'set-status-abandoned' === $this->current_action() ) {
-				give_update_payment_status( $id, 'abandoned' );
+			if ('set-status-abandoned' === $this->current_action()) {
+				give_update_payment_status($id, 'abandoned');
 			}
 
-			if ( 'set-status-preapproval' === $this->current_action() ) {
-				give_update_payment_status( $id, 'preapproval' );
+			if ('set-status-preapproval' === $this->current_action()) {
+				give_update_payment_status($id, 'preapproval');
 			}
 
-			if ( 'resend-receipt' === $this->current_action() ) {
-				give_email_donation_receipt( $id, false );
+			if ('resend-receipt' === $this->current_action()) {
+				give_email_donation_receipt($id, false);
 			}
 
-			do_action( 'give_payments_table_do_bulk_action', $id, $this->current_action() );
+			do_action('give_payments_table_do_bulk_action', $id, $this->current_action());
 		}
 
 	}
@@ -543,27 +543,27 @@ 
 
 		$args = array();
 
-		if ( isset( $_GET['user'] ) ) {
-			$args['user'] = urldecode( $_GET['user'] );
-		} elseif ( isset( $_GET['s'] ) ) {
-			$is_user  = strpos( $_GET['s'], strtolower( 'user:' ) ) !== false;
-			if ( $is_user ) {
-				$args['user'] = absint( trim( str_replace( 'user:', '', strtolower( $_GET['s'] ) ) ) );
-				unset( $args['s'] );
+		if (isset($_GET['user'])) {
+			$args['user'] = urldecode($_GET['user']);
+		} elseif (isset($_GET['s'])) {
+			$is_user = strpos($_GET['s'], strtolower('user:')) !== false;
+			if ($is_user) {
+				$args['user'] = absint(trim(str_replace('user:', '', strtolower($_GET['s']))));
+				unset($args['s']);
 			} else {
-				$args['s'] = sanitize_text_field( $_GET['s'] );
+				$args['s'] = sanitize_text_field($_GET['s']);
 			}
 		}
 
-		if ( ! empty( $_GET['start-date'] ) ) {
-			$args['start-date'] = urldecode( $_GET['start-date'] );
+		if ( ! empty($_GET['start-date'])) {
+			$args['start-date'] = urldecode($_GET['start-date']);
 		}
 
-		if ( ! empty( $_GET['end-date'] ) ) {
-			$args['end-date'] = urldecode( $_GET['end-date'] );
+		if ( ! empty($_GET['end-date'])) {
+			$args['end-date'] = urldecode($_GET['end-date']);
 		}
 
-		$payment_count         = give_count_payments( $args );
+		$payment_count         = give_count_payments($args);
 		$this->complete_count  = $payment_count->publish;
 		$this->pending_count   = $payment_count->pending;
 		$this->refunded_count  = $payment_count->refunded;
@@ -572,7 +572,7 @@ 
 		$this->cancelled_count = $payment_count->cancelled;
 		$this->abandoned_count = $payment_count->abandoned;
 
-		foreach ( $payment_count as $count ) {
+		foreach ($payment_count as $count) {
 			$this->total_count += $count;
 		}
 	}
@@ -587,26 +587,26 @@ 
 	public function payments_data() {
 
 		$per_page   = $this->per_page;
-		$orderby    = isset( $_GET['orderby'] ) ? urldecode( $_GET['orderby'] ) : 'ID';
-		$order      = isset( $_GET['order'] ) ? $_GET['order'] : 'DESC';
-		$user       = isset( $_GET['user'] ) ? $_GET['user'] : null;
-		$status     = isset( $_GET['status'] ) ? $_GET['status'] : give_get_payment_status_keys();
-		$meta_key   = isset( $_GET['meta_key'] ) ? $_GET['meta_key'] : null;
-		$year       = isset( $_GET['year'] ) ? $_GET['year'] : null;
-		$month      = isset( $_GET['m'] ) ? $_GET['m'] : null;
-		$day        = isset( $_GET['day'] ) ? $_GET['day'] : null;
-		$search     = isset( $_GET['s'] ) ? sanitize_text_field( $_GET['s'] ) : null;
-		$start_date = isset( $_GET['start-date'] ) ? sanitize_text_field( $_GET['start-date'] ) : null;
-		$end_date   = isset( $_GET['end-date'] ) ? sanitize_text_field( $_GET['end-date'] ) : $start_date;
-
-		if ( ! empty( $search ) ) {
+		$orderby    = isset($_GET['orderby']) ? urldecode($_GET['orderby']) : 'ID';
+		$order      = isset($_GET['order']) ? $_GET['order'] : 'DESC';
+		$user       = isset($_GET['user']) ? $_GET['user'] : null;
+		$status     = isset($_GET['status']) ? $_GET['status'] : give_get_payment_status_keys();
+		$meta_key   = isset($_GET['meta_key']) ? $_GET['meta_key'] : null;
+		$year       = isset($_GET['year']) ? $_GET['year'] : null;
+		$month      = isset($_GET['m']) ? $_GET['m'] : null;
+		$day        = isset($_GET['day']) ? $_GET['day'] : null;
+		$search     = isset($_GET['s']) ? sanitize_text_field($_GET['s']) : null;
+		$start_date = isset($_GET['start-date']) ? sanitize_text_field($_GET['start-date']) : null;
+		$end_date   = isset($_GET['end-date']) ? sanitize_text_field($_GET['end-date']) : $start_date;
+
+		if ( ! empty($search)) {
 			$status = 'any'; // Force all payment statuses when searching
 		}
 
 		$args = array(
 			'output'     => 'payments',
 			'number'     => $per_page,
-			'page'       => isset( $_GET['paged'] ) ? $_GET['paged'] : null,
+			'page'       => isset($_GET['paged']) ? $_GET['paged'] : null,
 			'orderby'    => $orderby,
 			'order'      => $order,
 			'user'       => $user,
@@ -620,14 +620,14 @@ 
 			'end_date'   => $end_date,
 		);
 
-		if ( is_string( $search ) && false !== strpos( $search, 'txn:' ) ) {
+		if (is_string($search) && false !== strpos($search, 'txn:')) {
 
 			$args['search_in_notes'] = true;
-			$args['s']               = trim( str_replace( 'txn:', '', $args['s'] ) );
+			$args['s']               = trim(str_replace('txn:', '', $args['s']));
 
 		}
 
-		$p_query = new Give_Payments_Query( $args );
+		$p_query = new Give_Payments_Query($args);
 
 		return $p_query->get_payments();
 
@@ -647,17 +647,17 @@ 
 	 */
 	public function prepare_items() {
 
-		wp_reset_vars( array( 'action', 'payment', 'orderby', 'order', 's' ) );
+		wp_reset_vars(array('action', 'payment', 'orderby', 'order', 's'));
 
 		$columns  = $this->get_columns();
 		$hidden   = array(); // No hidden columns
 		$sortable = $this->get_sortable_columns();
 		$data     = $this->payments_data();
-		$status   = isset( $_GET['status'] ) ? $_GET['status'] : 'any';
+		$status   = isset($_GET['status']) ? $_GET['status'] : 'any';
 
-		$this->_column_headers = array( $columns, $hidden, $sortable );
+		$this->_column_headers = array($columns, $hidden, $sortable);
 
-		switch ( $status ) {
+		switch ($status) {
 			case 'publish':
 				$total_items = $this->complete_count;
 				break;
@@ -684,18 +684,18 @@ 
 				break;
 			default:
 				// Retrieve the count of the non-default-Give status
-				$count       = wp_count_posts( 'give_payment' );
+				$count       = wp_count_posts('give_payment');
 				$total_items = $count->{$status};
 		}
 
 		$this->items = $data;
 
-		$this->set_pagination_args( array(
+		$this->set_pagination_args(array(
 				'total_items' => $total_items,
 				// WE have to calculate the total number of items
 				'per_page'    => $this->per_page,
 				// WE have to determine how many items to show on a page
-				'total_pages' => ceil( $total_items / $this->per_page )
+				'total_pages' => ceil($total_items / $this->per_page)
 				// WE have to calculate the total number of pages
 			)
 		);

includes/admin/reporting/tools/class-give-tools-delete-test-transactions.php

Doc Comments +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
 	 * @since 1.5
 	 * @global object $wpdb Used to query the database using the WordPress
 	 *   Database API
-	 * @return array $data The data for the CSV file
+	 * @return boolean $data The data for the CSV file
 	 */
 	public function get_data() {
 		global $wpdb;

Spacing +53 added lines, -53 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -55,22 +55,22 @@ 
 	public function get_data() {
 		global $wpdb;
 
-		$items = $this->get_stored_data( 'give_temp_reset_ids' );
+		$items = $this->get_stored_data('give_temp_reset_ids');
 
-		if ( ! is_array( $items ) ) {
+		if ( ! is_array($items)) {
 			return false;
 		}
 
-		$offset     = ( $this->step - 1 ) * $this->per_step;
-		$step_items = array_slice( $items, $offset, $this->per_step );
+		$offset     = ($this->step - 1) * $this->per_step;
+		$step_items = array_slice($items, $offset, $this->per_step);
 
-		if ( $step_items ) {
+		if ($step_items) {
 
 			$step_ids = array(
 				'other' => array(),
 			);
 
-			foreach ( $step_items as $item ) {
+			foreach ($step_items as $item) {
 
 				$step_ids['other'][] = $item['id'];
 
@@ -78,15 +78,15 @@ 
 
 			$sql = array();
 
-			foreach ( $step_ids as $type => $ids ) {
+			foreach ($step_ids as $type => $ids) {
 
-				if ( empty( $ids ) ) {
+				if (empty($ids)) {
 					continue;
 				}
 
-				$ids = implode( ',', $ids );
+				$ids = implode(',', $ids);
 
-				switch ( $type ) {
+				switch ($type) {
 					case 'other':
 						$sql[] = "DELETE FROM $wpdb->posts WHERE id IN ($ids)";
 						$sql[] = "DELETE FROM $wpdb->postmeta WHERE post_id IN ($ids)";
@@ -97,9 +97,9 @@ 
 
 			}
 
-			if ( ! empty( $sql ) ) {
-				foreach ( $sql as $query ) {
-					$wpdb->query( $query );
+			if ( ! empty($sql)) {
+				foreach ($sql as $query) {
+					$wpdb->query($query);
 				}
 			}
 
@@ -119,16 +119,16 @@ 
 	 */
 	public function get_percentage_complete() {
 
-		$items = $this->get_stored_data( 'give_temp_reset_ids', false );
-		$total = count( $items );
+		$items = $this->get_stored_data('give_temp_reset_ids', false);
+		$total = count($items);
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( $this->per_step * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = (($this->per_step * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -142,7 +142,7 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 	}
 
 	/**
@@ -153,30 +153,30 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( __( 'You do not have permission to export data.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(__('You do not have permission to export data.', 'give'), __('Error', 'give'), array('response' => 403));
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
-			update_option( 'give_earnings_total', 0 );
-			delete_transient( 'give_earnings_total' );
-			delete_transient( 'give_estimated_monthly_stats' . true );
-			delete_transient( 'give_estimated_monthly_stats' . false );
-			$this->delete_data( 'give_temp_reset_ids' );
+			update_option('give_earnings_total', 0);
+			delete_transient('give_earnings_total');
+			delete_transient('give_estimated_monthly_stats'.true);
+			delete_transient('give_estimated_monthly_stats'.false);
+			$this->delete_data('give_temp_reset_ids');
 
 			// Reset the sequential order numbers
-			if ( give_get_option( 'enable_sequential' ) ) {
-				delete_option( 'give_last_payment_number' );
+			if (give_get_option('enable_sequential')) {
+				delete_option('give_last_payment_number');
 			}
 
 			$this->done    = true;
-			$this->message = __( 'Test transactions successfully deleted.', 'give' );
+			$this->message = __('Test transactions successfully deleted.', 'give');
 
 			return false;
 		}
@@ -186,10 +186,10 @@ 
 	 * Headers
 	 */
 	public function headers() {
-		ignore_user_abort( true );
+		ignore_user_abort(true);
 
-		if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-			set_time_limit( 0 );
+		if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+			set_time_limit(0);
 		}
 	}
 
@@ -213,26 +213,26 @@ 
 	 */
 	public function pre_fetch() {
 
-		if ( $this->step == 1 ) {
-			$this->delete_data( 'give_temp_reset_ids' );
+		if ($this->step == 1) {
+			$this->delete_data('give_temp_reset_ids');
 		}
 
-		$items = get_option( 'give_temp_reset_ids', false );
+		$items = get_option('give_temp_reset_ids', false);
 
-		if ( false === $items ) {
+		if (false === $items) {
 			$items = array();
 
-			$args = apply_filters( 'give_tools_reset_stats_total_args', array(
+			$args = apply_filters('give_tools_reset_stats_total_args', array(
 				'post_type'      => 'give_payment',
 				'post_status'    => 'any',
-				'posts_per_page' => - 1,
+				'posts_per_page' => -1,
 				//ONLY TEST MODE!!!
 				'meta_key'   => '_give_payment_mode',
 				'meta_value' => 'test'
-			) );
+			));
 
-			$posts = get_posts( $args );
-			foreach ( $posts as $post ) {
+			$posts = get_posts($args);
+			foreach ($posts as $post) {
 				$items[] = array(
 					'id'   => (int) $post->ID,
 					'type' => $post->post_type,
@@ -241,9 +241,9 @@ 
 
 			// Allow filtering of items to remove with an unassociative array for each item
 			// The array contains the unique ID of the item, and a 'type' for you to use in the execution of the get_data method
-			$items = apply_filters( 'give_reset_store_items', $items );
+			$items = apply_filters('give_reset_store_items', $items);
 
-			$this->store_data( 'give_temp_reset_ids', $items );
+			$this->store_data('give_temp_reset_ids', $items);
 		}
 
 	}
@@ -257,11 +257,11 @@ 
 	 *
 	 * @return mixed       Returns the data from the database
 	 */
-	private function get_stored_data( $key ) {
+	private function get_stored_data($key) {
 		global $wpdb;
-		$value = $wpdb->get_var( $wpdb->prepare( "SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key ) );
+		$value = $wpdb->get_var($wpdb->prepare("SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key));
 
-		return empty( $value ) ? false : maybe_unserialize( $value );
+		return empty($value) ? false : maybe_unserialize($value);
 	}
 
 	/**
@@ -274,10 +274,10 @@ 
 	 *
 	 * @return void
 	 */
-	private function store_data( $key, $value ) {
+	private function store_data($key, $value) {
 		global $wpdb;
 
-		$value = maybe_serialize( $value );
+		$value = maybe_serialize($value);
 
 		$data = array(
 			'option_name'  => $key,
@@ -291,7 +291,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -303,9 +303,9 @@ 
 	 *
 	 * @return void
 	 */
-	private function delete_data( $key ) {
+	private function delete_data($key) {
 		global $wpdb;
-		$wpdb->delete( $wpdb->options, array( 'option_name' => $key ) );
+		$wpdb->delete($wpdb->options, array('option_name' => $key));
 	}
 
 }

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

Doc Comments +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
 	 * @since 1.5
 	 * @global object $wpdb Used to query the database using the WordPress
 	 *   Database API
-	 * @return array $data The data for the CSV file
+	 * @return boolean $data The data for the CSV file
 	 */
 	public function get_data() {
 		global $give_logs, $wpdb;

Spacing +99 added lines, -99 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -55,26 +55,26 @@ 
 	public function get_data() {
 		global $give_logs, $wpdb;
 
-		$totals             = $this->get_stored_data( 'give_temp_recount_all_stats' );
-		$payment_items      = $this->get_stored_data( 'give_temp_payment_items' );
-		$processed_payments = $this->get_stored_data( 'give_temp_processed_payments' );
-		$accepted_statuses  = apply_filters( 'give_recount_accepted_statuses', array( 'publish', 'revoked' ) );
+		$totals             = $this->get_stored_data('give_temp_recount_all_stats');
+		$payment_items      = $this->get_stored_data('give_temp_payment_items');
+		$processed_payments = $this->get_stored_data('give_temp_processed_payments');
+		$accepted_statuses  = apply_filters('give_recount_accepted_statuses', array('publish', 'revoked'));
 
-		if ( false === $totals ) {
+		if (false === $totals) {
 			$totals = array();
 		}
 
-		if ( false === $payment_items ) {
+		if (false === $payment_items) {
 			$payment_items = array();
 		}
 
-		if ( false === $processed_payments ) {
+		if (false === $processed_payments) {
 			$processed_payments = array();
 		}
 
-		$all_forms = $this->get_stored_data( 'give_temp_form_ids' );
+		$all_forms = $this->get_stored_data('give_temp_form_ids');
 
-		$args = apply_filters( 'give_recount_form_stats_args', array(
+		$args = apply_filters('give_recount_form_stats_args', array(
 			'post_parent__in' => $all_forms,
 			'post_type'       => 'give_log',
 			'posts_per_page'  => $this->per_step,
@@ -82,73 +82,73 @@ 
 			'paged'           => $this->step,
 			'log_type'        => 'sale',
 			'fields'          => 'ids',
-		) );
+		));
 
-		$log_ids = $give_logs->get_connected_logs( $args, 'sale' );
+		$log_ids = $give_logs->get_connected_logs($args, 'sale');
 
-		if ( $log_ids ) {
-			$log_ids = implode( ',', $log_ids );
+		if ($log_ids) {
+			$log_ids = implode(',', $log_ids);
 
-			$payment_ids = $wpdb->get_col( "SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)" );
-			unset( $log_ids );
+			$payment_ids = $wpdb->get_col("SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)");
+			unset($log_ids);
 
-			$payment_ids = implode( ',', $payment_ids );
-			$payments    = $wpdb->get_results( "SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (" . $payment_ids . ")" );
-			unset( $payment_ids );
+			$payment_ids = implode(',', $payment_ids);
+			$payments    = $wpdb->get_results("SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (".$payment_ids.")");
+			unset($payment_ids);
 
 			//Loop through payments
-			foreach ( $payments as $payment ) {
+			foreach ($payments as $payment) {
 
 				// Prevent payments that have all ready been retrieved from a previous sales log from counting again.
-				if ( in_array( $payment->ID, $processed_payments ) ) {
+				if (in_array($payment->ID, $processed_payments)) {
 					continue;
 				}
 
 				//Verify accepted status'
-				if ( ! in_array( $payment->post_status, $accepted_statuses ) ) {
+				if ( ! in_array($payment->post_status, $accepted_statuses)) {
 					$processed_payments[] = $payment->ID;
 					continue;
 				}
 
-				$payment_item = $payment_items[ $payment->ID ];
+				$payment_item = $payment_items[$payment->ID];
 
 
-				$form_id = isset( $payment_item['id'] ) ? $payment_item['id'] : '';
+				$form_id = isset($payment_item['id']) ? $payment_item['id'] : '';
 
 				//Must have a form ID
-				if ( empty( $form_id ) ) {
+				if (empty($form_id)) {
 					continue;
 				}
 
 				//Form ID must be within $all_forms array to be validated
-				if ( ! in_array( $form_id, $all_forms ) ) {
+				if ( ! in_array($form_id, $all_forms)) {
 					continue;
 				}
 
 				//If array key doesn't exist, create it
-				if ( ! array_key_exists( $form_id, $totals ) ) {
-					$totals[ $form_id ] = array(
+				if ( ! array_key_exists($form_id, $totals)) {
+					$totals[$form_id] = array(
 						'sales'    => (int) 0,
 						'earnings' => (float) 0,
 					);
 				}
 
-				$totals[ $form_id ]['sales'] ++;
-				$totals[ $form_id ]['earnings'] += $payment_item['price'];
+				$totals[$form_id]['sales']++;
+				$totals[$form_id]['earnings'] += $payment_item['price'];
 
 				$processed_payments[] = $payment->ID;
 
 			}
 
-			$this->store_data( 'give_temp_processed_payments', $processed_payments );
-			$this->store_data( 'give_temp_recount_all_stats', $totals );
+			$this->store_data('give_temp_processed_payments', $processed_payments);
+			$this->store_data('give_temp_recount_all_stats', $totals);
 
 			return true;
 		}
 
-		foreach ( $totals as $key => $stats ) {
-			update_post_meta( $key, '_give_form_sales', $stats['sales'] );
-			update_post_meta( $key, '_give_form_earnings', $stats['earnings'] );
+		foreach ($totals as $key => $stats) {
+			update_post_meta($key, '_give_form_sales', $stats['sales']);
+			update_post_meta($key, '_give_form_earnings', $stats['earnings']);
 		}
 
 		return false;
@@ -163,20 +163,20 @@ 
 	 */
 	public function get_percentage_complete() {
 
-		$total = $this->get_stored_data( 'give_recount_all_total', false );
+		$total = $this->get_stored_data('give_recount_all_total', false);
 
-		if ( false === $total ) {
+		if (false === $total) {
 			$this->pre_fetch();
-			$total = $this->get_stored_data( 'give_recount_all_total', 0 );
+			$total = $this->get_stored_data('give_recount_all_total', 0);
 		}
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( $this->per_step * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = (($this->per_step * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -190,8 +190,8 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
-		$this->form_id = isset( $request['form_id'] ) ? sanitize_text_field( $request['form_id'] ) : false;
+	public function set_properties($request) {
+		$this->form_id = isset($request['form_id']) ? sanitize_text_field($request['form_id']) : false;
 	}
 
 	/**
@@ -202,34 +202,34 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( __( 'You do not have permission to export data.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(__('You do not have permission to export data.', 'give'), __('Error', 'give'), array('response' => 403));
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
-			$this->delete_data( 'give_recount_all_total' );
-			$this->delete_data( 'give_temp_recount_all_stats' );
-			$this->delete_data( 'give_temp_payment_items' );
-			$this->delete_data( 'give_temp_form_ids' );
-			$this->delete_data( 'give_temp_processed_payments' );
+			$this->delete_data('give_recount_all_total');
+			$this->delete_data('give_temp_recount_all_stats');
+			$this->delete_data('give_temp_payment_items');
+			$this->delete_data('give_temp_form_ids');
+			$this->delete_data('give_temp_processed_payments');
 			$this->done    = true;
-			$this->message = __( 'Donation form income amounts and donation counts stats successfully recounted.', 'give' );
+			$this->message = __('Donation form income amounts and donation counts stats successfully recounted.', 'give');
 
 			return false;
 		}
 	}
 
 	public function headers() {
-		ignore_user_abort( true );
+		ignore_user_abort(true);
 
-		if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-			set_time_limit( 0 );
+		if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+			set_time_limit(0);
 		}
 	}
 
@@ -258,76 +258,76 @@ 
 
 		global $give_logs, $wpdb;
 
-		if ( $this->step == 1 ) {
-			$this->delete_data( 'give_temp_recount_all_total' );
-			$this->delete_data( 'give_temp_recount_all_stats' );
-			$this->delete_data( 'give_temp_payment_items' );
-			$this->delete_data( 'give_temp_processed_payments' );
+		if ($this->step == 1) {
+			$this->delete_data('give_temp_recount_all_total');
+			$this->delete_data('give_temp_recount_all_stats');
+			$this->delete_data('give_temp_payment_items');
+			$this->delete_data('give_temp_processed_payments');
 		}
 
-		$accepted_statuses = apply_filters( 'give_recount_accepted_statuses', array( 'publish', 'revoked' ) );
-		$total             = $this->get_stored_data( 'give_temp_recount_all_total' );
+		$accepted_statuses = apply_filters('give_recount_accepted_statuses', array('publish', 'revoked'));
+		$total             = $this->get_stored_data('give_temp_recount_all_total');
 
-		if ( false === $total ) {
+		if (false === $total) {
 			$total         = 0;
-			$payment_items = $this->get_stored_data( 'give_temp_payment_items' );
+			$payment_items = $this->get_stored_data('give_temp_payment_items');
 
-			if ( false === $payment_items ) {
+			if (false === $payment_items) {
 				$payment_items = array();
-				$this->store_data( 'give_temp_payment_items', $payment_items );
+				$this->store_data('give_temp_payment_items', $payment_items);
 			}
 
-			$all_forms = $this->get_stored_data( 'give_temp_form_ids' );
+			$all_forms = $this->get_stored_data('give_temp_form_ids');
 
-			if ( false === $all_forms ) {
+			if (false === $all_forms) {
 				$args = array(
 					'post_status'    => 'any',
 					'post_type'      => 'give_forms',
-					'posts_per_page' => - 1,
+					'posts_per_page' => -1,
 					'fields'         => 'ids',
 				);
 
-				$all_forms = get_posts( $args );
-				$this->store_data( 'give_temp_form_ids', $all_forms );
+				$all_forms = get_posts($args);
+				$this->store_data('give_temp_form_ids', $all_forms);
 			}
 
-			$args = apply_filters( 'give_recount_form_stats_total_args', array(
+			$args = apply_filters('give_recount_form_stats_total_args', array(
 				'post_parent__in' => $all_forms,
 				'post_type'       => 'give_log',
 				'post_status'     => 'publish',
 				'log_type'        => 'sale',
 				'fields'          => 'ids',
 				'nopaging'        => true,
-			) );
+			));
 
-			$all_logs = $give_logs->get_connected_logs( $args, 'sale' );
+			$all_logs = $give_logs->get_connected_logs($args, 'sale');
 
-			if ( $all_logs ) {
-				$log_ids     = implode( ',', $all_logs );
-				$payment_ids = $wpdb->get_col( "SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)" );
-				unset( $log_ids );
+			if ($all_logs) {
+				$log_ids     = implode(',', $all_logs);
+				$payment_ids = $wpdb->get_col("SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)");
+				unset($log_ids);
 
-				$payment_ids = implode( ',', $payment_ids );
-				$payments    = $wpdb->get_results( "SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (" . $payment_ids . ")" );
-				unset( $payment_ids );
+				$payment_ids = implode(',', $payment_ids);
+				$payments    = $wpdb->get_results("SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (".$payment_ids.")");
+				unset($payment_ids);
 
-				foreach ( $payments as $payment ) {
+				foreach ($payments as $payment) {
 
-					$payment = new Give_Payment( $payment->ID );
+					$payment = new Give_Payment($payment->ID);
 					$form_id = $payment->form_id;
 
 					//If for some reason somehow the form_ID isn't set check payment meta
-					if ( empty( $payment->form_id ) ) {
+					if (empty($payment->form_id)) {
 						$payment_meta = $payment->get_meta();
-						$form_id = isset( $payment_meta['form_id'] ) ? $payment_meta['form_id'] : 0;
+						$form_id = isset($payment_meta['form_id']) ? $payment_meta['form_id'] : 0;
 					}
 
-					if ( ! in_array( $payment->post_status, $accepted_statuses ) ) {
+					if ( ! in_array($payment->post_status, $accepted_statuses)) {
 						continue;
 					}
 
-					if ( ! array_key_exists( $payment->ID, $payment_items ) ) {
-						$payment_items[ $payment->ID ] = array(
+					if ( ! array_key_exists($payment->ID, $payment_items)) {
+						$payment_items[$payment->ID] = array(
 							'id'         => $form_id,
 							'payment_id' => $payment->ID,
 							'price'      => $payment->total
@@ -336,11 +336,11 @@ 
 
 				}
 
-				$total = count( $all_logs );
+				$total = count($all_logs);
 			}
 
-			$this->store_data( 'give_temp_payment_items', $payment_items );
-			$this->store_data( 'give_recount_all_total', $total );
+			$this->store_data('give_temp_payment_items', $payment_items);
+			$this->store_data('give_recount_all_total', $total);
 		}
 
 	}
@@ -354,11 +354,11 @@ 
 	 *
 	 * @return mixed       Returns the data from the database
 	 */
-	private function get_stored_data( $key ) {
+	private function get_stored_data($key) {
 		global $wpdb;
-		$value = $wpdb->get_var( $wpdb->prepare( "SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key ) );
+		$value = $wpdb->get_var($wpdb->prepare("SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key));
 
-		return empty( $value ) ? false : maybe_unserialize( $value );
+		return empty($value) ? false : maybe_unserialize($value);
 	}
 
 	/**
@@ -371,10 +371,10 @@ 
 	 *
 	 * @return void
 	 */
-	private function store_data( $key, $value ) {
+	private function store_data($key, $value) {
 		global $wpdb;
 
-		$value = maybe_serialize( $value );
+		$value = maybe_serialize($value);
 
 		$data = array(
 			'option_name'  => $key,
@@ -388,7 +388,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -400,9 +400,9 @@ 
 	 *
 	 * @return void
 	 */
-	private function delete_data( $key ) {
+	private function delete_data($key) {
 		global $wpdb;
-		$wpdb->delete( $wpdb->options, array( 'option_name' => $key ) );
+		$wpdb->delete($wpdb->options, array('option_name' => $key));
 	}
 
 }

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

Doc Comments +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
 	 * @since 1.5
 	 * @global object $wpdb Used to query the database using the WordPress
 	 *   Database API
-	 * @return array $data The data for the CSV file
+	 * @return boolean $data The data for the CSV file
 	 */
 	public function get_data() {
 

Spacing +37 added lines, -37 removed lines

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

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

Doc Comments +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
 	 * @since 1.5
 	 * @global object $wpdb Used to query the database using the WordPress
 	 *   Database API
-	 * @return array $data The data for the CSV file
+	 * @return boolean $data The data for the CSV file
 	 */
 	public function get_data() {
 		global $give_logs, $wpdb;

Spacing +66 added lines, -66 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -62,23 +62,23 @@ 
 	public function get_data() {
 		global $give_logs, $wpdb;
 
-		$accepted_statuses = apply_filters( 'give_recount_accepted_statuses', array( 'publish', 'revoked' ) );
+		$accepted_statuses = apply_filters('give_recount_accepted_statuses', array('publish', 'revoked'));
 
-		if ( $this->step == 1 ) {
-			$this->delete_data( 'give_temp_recount_form_stats' );
+		if ($this->step == 1) {
+			$this->delete_data('give_temp_recount_form_stats');
 		}
 
-		$totals = $this->get_stored_data( 'give_temp_recount_form_stats' );
+		$totals = $this->get_stored_data('give_temp_recount_form_stats');
 
-		if ( false === $totals ) {
+		if (false === $totals) {
 			$totals = array(
 				'earnings' => (float) 0,
 				'sales'    => 0,
 			);
-			$this->store_data( 'give_temp_recount_form_stats', $totals );
+			$this->store_data('give_temp_recount_form_stats', $totals);
 		}
 
-		$args = apply_filters( 'give_recount_form_stats_args', array(
+		$args = apply_filters('give_recount_form_stats_args', array(
 			'post_parent'    => $this->form_id,
 			'post_type'      => 'give_log',
 			'posts_per_page' => $this->per_step,
@@ -86,49 +86,49 @@ 
 			'paged'          => $this->step,
 			'log_type'       => 'sale',
 			'fields'         => 'ids',
-		) );
+		));
 
-		$log_ids              = $give_logs->get_connected_logs( $args, 'sale' );
+		$log_ids              = $give_logs->get_connected_logs($args, 'sale');
 		$this->_log_ids_debug = array();
 
-		if ( $log_ids ) {
-			$log_ids     = implode( ',', $log_ids );
-			$payment_ids = $wpdb->get_col( "SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)" );
-			unset( $log_ids );
+		if ($log_ids) {
+			$log_ids     = implode(',', $log_ids);
+			$payment_ids = $wpdb->get_col("SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)");
+			unset($log_ids);
 
-			$payment_ids = implode( ',', $payment_ids );
-			$payments    = $wpdb->get_results( "SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (" . $payment_ids . ")" );
-			unset( $payment_ids );
+			$payment_ids = implode(',', $payment_ids);
+			$payments    = $wpdb->get_results("SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (".$payment_ids.")");
+			unset($payment_ids);
 
-			foreach ( $payments as $payment ) {
+			foreach ($payments as $payment) {
 
-				$payment = new Give_Payment( $payment->ID );
+				$payment = new Give_Payment($payment->ID);
 
 				//Ensure acceptible status only
-				if ( ! in_array( $payment->post_status, $accepted_statuses ) ) {
+				if ( ! in_array($payment->post_status, $accepted_statuses)) {
 					continue;
 				}
 
 				//Ensure only payments for this form are counted
-				if ( $payment->form_id != $this->form_id ) {
+				if ($payment->form_id != $this->form_id) {
 					continue;
 				}
 
 				$this->_log_ids_debug[] = $payment->ID;
 
-				$totals['sales'] ++;
+				$totals['sales']++;
 				$totals['earnings'] += $payment->total;
 
 			}
 
-			$this->store_data( 'give_temp_recount_form_stats', $totals );
+			$this->store_data('give_temp_recount_form_stats', $totals);
 
 			return true;
 		}
 
 
-		update_post_meta( $this->form_id, '_give_form_sales', $totals['sales'] );
-		update_post_meta( $this->form_id, '_give_form_earnings', $totals['earnings'] );
+		update_post_meta($this->form_id, '_give_form_sales', $totals['sales']);
+		update_post_meta($this->form_id, '_give_form_earnings', $totals['earnings']);
 
 		return false;
 
@@ -143,54 +143,54 @@ 
 	public function get_percentage_complete() {
 		global $give_logs, $wpdb;
 
-		if ( $this->step == 1 ) {
-			$this->delete_data( 'give_recount_total_' . $this->form_id );
+		if ($this->step == 1) {
+			$this->delete_data('give_recount_total_'.$this->form_id);
 		}
 
-		$accepted_statuses = apply_filters( 'give_recount_accepted_statuses', array( 'publish', 'revoked' ) );
-		$total             = $this->get_stored_data( 'give_recount_total_' . $this->form_id );
+		$accepted_statuses = apply_filters('give_recount_accepted_statuses', array('publish', 'revoked'));
+		$total             = $this->get_stored_data('give_recount_total_'.$this->form_id);
 
-		if ( false === $total ) {
+		if (false === $total) {
 			$total = 0;
-			$args  = apply_filters( 'give_recount_form_stats_total_args', array(
+			$args  = apply_filters('give_recount_form_stats_total_args', array(
 				'post_parent' => $this->form_id,
 				'post_type'   => 'give_log',
 				'post_status' => 'publish',
 				'log_type'    => 'sale',
 				'fields'      => 'ids',
 				'nopaging'    => true,
-			) );
+			));
 
-			$log_ids = $give_logs->get_connected_logs( $args, 'sale' );
+			$log_ids = $give_logs->get_connected_logs($args, 'sale');
 
-			if ( $log_ids ) {
-				$log_ids     = implode( ',', $log_ids );
-				$payment_ids = $wpdb->get_col( "SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)" );
-				unset( $log_ids );
+			if ($log_ids) {
+				$log_ids     = implode(',', $log_ids);
+				$payment_ids = $wpdb->get_col("SELECT meta_value FROM $wpdb->postmeta WHERE meta_key='_give_log_payment_id' AND post_id IN ($log_ids)");
+				unset($log_ids);
 
-				$payment_ids = implode( ',', $payment_ids );
-				$payments    = $wpdb->get_results( "SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (" . $payment_ids . ")" );
-				unset( $payment_ids );
+				$payment_ids = implode(',', $payment_ids);
+				$payments    = $wpdb->get_results("SELECT ID, post_status FROM $wpdb->posts WHERE ID IN (".$payment_ids.")");
+				unset($payment_ids);
 
-				foreach ( $payments as $payment ) {
-					if ( in_array( $payment->post_status, $accepted_statuses ) ) {
+				foreach ($payments as $payment) {
+					if (in_array($payment->post_status, $accepted_statuses)) {
 						continue;
 					}
 
-					$total ++;
+					$total++;
 				}
 			}
 
-			$this->store_data( 'give_recount_total_' . $this->form_id, $total );
+			$this->store_data('give_recount_total_'.$this->form_id, $total);
 		}
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( $this->per_step * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = (($this->per_step * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -204,8 +204,8 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
-		$this->form_id = isset( $request['form_id'] ) ? sanitize_text_field( $request['form_id'] ) : false;
+	public function set_properties($request) {
+		$this->form_id = isset($request['form_id']) ? sanitize_text_field($request['form_id']) : false;
 	}
 
 	/**
@@ -216,31 +216,31 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( __( 'You do not have permission to export data.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(__('You do not have permission to export data.', 'give'), __('Error', 'give'), array('response' => 403));
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
-			$this->delete_data( 'give_recount_total_' . $this->form_id );
-			$this->delete_data( 'give_temp_recount_form_stats' );
+			$this->delete_data('give_recount_total_'.$this->form_id);
+			$this->delete_data('give_temp_recount_form_stats');
 			$this->done    = true;
-			$this->message = sprintf( __( 'Donation counts and income amount statistics successfully recounted for "%s".', 'give' ), get_the_title( $this->form_id ) );
+			$this->message = sprintf(__('Donation counts and income amount statistics successfully recounted for "%s".', 'give'), get_the_title($this->form_id));
 
 			return false;
 		}
 	}
 
 	public function headers() {
-		ignore_user_abort( true );
+		ignore_user_abort(true);
 
-		if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-			set_time_limit( 0 );
+		if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+			set_time_limit(0);
 		}
 	}
 
@@ -268,11 +268,11 @@ 
 	 *
 	 * @return mixed       Returns the data from the database
 	 */
-	private function get_stored_data( $key ) {
+	private function get_stored_data($key) {
 		global $wpdb;
-		$value = $wpdb->get_var( $wpdb->prepare( "SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key ) );
+		$value = $wpdb->get_var($wpdb->prepare("SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key));
 
-		return empty( $value ) ? false : maybe_unserialize( $value );
+		return empty($value) ? false : maybe_unserialize($value);
 	}
 
 	/**
@@ -285,10 +285,10 @@ 
 	 *
 	 * @return void
 	 */
-	private function store_data( $key, $value ) {
+	private function store_data($key, $value) {
 		global $wpdb;
 
-		$value = maybe_serialize( $value );
+		$value = maybe_serialize($value);
 
 		$data = array(
 			'option_name'  => $key,
@@ -302,7 +302,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -314,9 +314,9 @@ 
 	 *
 	 * @return void
 	 */
-	private function delete_data( $key ) {
+	private function delete_data($key) {
 		global $wpdb;
-		$wpdb->delete( $wpdb->options, array( 'option_name' => $key ) );
+		$wpdb->delete($wpdb->options, array('option_name' => $key));
 	}
 
 }

includes/admin/reporting/tools/class-give-tools-recount-income.php

Doc Comments +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
 	 * @since 1.5
 	 * @global object $wpdb Used to query the database using the WordPress
 	 *   Database API
-	 * @return array $data The data for the CSV file
+	 * @return boolean $data The data for the CSV file
 	 */
 	public function get_data() {
 

Spacing +46 added lines, -46 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -54,50 +54,50 @@ 
 	 */
 	public function get_data() {
 
-		if ( $this->step == 1 ) {
-			$this->delete_data( 'give_temp_recount_income' );
+		if ($this->step == 1) {
+			$this->delete_data('give_temp_recount_income');
 		}
 
-		$total = get_option( 'give_temp_recount_income', false );
+		$total = get_option('give_temp_recount_income', false);
 
-		if ( false === $total ) {
+		if (false === $total) {
 			$total = (float) 0;
-			$this->store_data( 'give_temp_recount_income', $total );
+			$this->store_data('give_temp_recount_income', $total);
 		}
 
-		$accepted_statuses = apply_filters( 'give_recount_accepted_statuses', array( 'publish', 'revoked' ) );
+		$accepted_statuses = apply_filters('give_recount_accepted_statuses', array('publish', 'revoked'));
 
-		$args = apply_filters( 'give_recount_income_args', array(
+		$args = apply_filters('give_recount_income_args', array(
 			'number' => $this->per_step,
 			'page'   => $this->step,
 			'status' => $accepted_statuses,
 			'fields' => 'ids'
-		) );
+		));
 
-		$payments = give_get_payments( $args );
+		$payments = give_get_payments($args);
 
-		if ( ! empty( $payments ) ) {
+		if ( ! empty($payments)) {
 
-			foreach ( $payments as $payment ) {
+			foreach ($payments as $payment) {
 
-				$total += give_get_payment_amount( $payment );
+				$total += give_get_payment_amount($payment);
 
 			}
 
-			if ( $total < 0 ) {
+			if ($total < 0) {
 				$totals = 0;
 			}
 
-			$total = round( $total, give_currency_decimal_filter() );
+			$total = round($total, give_currency_decimal_filter());
 
-			$this->store_data( 'give_temp_recount_income', $total );
+			$this->store_data('give_temp_recount_income', $total);
 
 			return true;
 
 		}
 
-		update_option( 'give_income_total', $total );
-		set_transient( 'give_income_total', $total, 86400 );
+		update_option('give_income_total', $total);
+		set_transient('give_income_total', $total, 86400);
 
 		return false;
 
@@ -111,25 +111,25 @@ 
 	 */
 	public function get_percentage_complete() {
 
-		$total = $this->get_stored_data( 'give_recount_income_total' );
+		$total = $this->get_stored_data('give_recount_income_total');
 
-		if ( false === $total ) {
-			$args = apply_filters( 'give_recount_income_total_args', array() );
+		if (false === $total) {
+			$args = apply_filters('give_recount_income_total_args', array());
 
-			$counts = give_count_payments( $args );
-			$total  = absint( $counts->publish ) + absint( $counts->revoked );
-			$total  = apply_filters( 'give_recount_store_income_total', $total );
+			$counts = give_count_payments($args);
+			$total  = absint($counts->publish) + absint($counts->revoked);
+			$total  = apply_filters('give_recount_store_income_total', $total);
 
-			$this->store_data( 'give_recount_income_total', $total );
+			$this->store_data('give_recount_income_total', $total);
 		}
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( $this->per_step * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = (($this->per_step * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -143,7 +143,7 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 	}
 
 	/**
@@ -154,31 +154,31 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( __( 'You do not have permission to export data.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(__('You do not have permission to export data.', 'give'), __('Error', 'give'), array('response' => 403));
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
-			$this->delete_data( 'give_recount_income_total' );
-			$this->delete_data( 'give_temp_recount_income' );
+			$this->delete_data('give_recount_income_total');
+			$this->delete_data('give_temp_recount_income');
 			$this->done    = true;
-			$this->message = __( 'Give income successfully recounted.', 'give' );
+			$this->message = __('Give income successfully recounted.', 'give');
 
 			return false;
 		}
 	}
 
 	public function headers() {
-		ignore_user_abort( true );
+		ignore_user_abort(true);
 
-		if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-			set_time_limit( 0 );
+		if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+			set_time_limit(0);
 		}
 	}
 
@@ -206,11 +206,11 @@ 
 	 *
 	 * @return mixed       Returns the data from the database
 	 */
-	private function get_stored_data( $key ) {
+	private function get_stored_data($key) {
 		global $wpdb;
-		$value = $wpdb->get_var( $wpdb->prepare( "SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key ) );
+		$value = $wpdb->get_var($wpdb->prepare("SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key));
 
-		return empty( $value ) ? false : maybe_unserialize( $value );
+		return empty($value) ? false : maybe_unserialize($value);
 	}
 
 	/**
@@ -223,10 +223,10 @@ 
 	 *
 	 * @return void
 	 */
-	private function store_data( $key, $value ) {
+	private function store_data($key, $value) {
 		global $wpdb;
 
-		$value = maybe_serialize( $value );
+		$value = maybe_serialize($value);
 
 		$data = array(
 			'option_name'  => $key,
@@ -240,7 +240,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -252,9 +252,9 @@ 
 	 *
 	 * @return void
 	 */
-	private function delete_data( $key ) {
+	private function delete_data($key) {
 		global $wpdb;
-		$wpdb->delete( $wpdb->options, array( 'option_name' => $key ) );
+		$wpdb->delete($wpdb->options, array('option_name' => $key));
 	}
 
 }

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

Doc Comments +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
 	 * @since 1.5
 	 * @global object $wpdb Used to query the database using the WordPress
 	 *   Database API
-	 * @return array $data The data for the CSV file
+	 * @return boolean $data The data for the CSV file
 	 */
 	public function get_data() {
 

Indentation +2 added lines, -2 removed lines

@@ -188,8 +188,8 @@
 
 			$payment_ids = implode( ',', $payment_ids );
 			$customer->update( array( 'payment_ids'    => $payment_ids,
-			                          'purchase_count' => $purchase_count,
-			                          'purchase_value' => $pending_total
+									  'purchase_count' => $purchase_count,
+									  'purchase_value' => $pending_total
 			) );
 
 			$this->done    = true;

Spacing +70 added lines, -70 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -54,39 +54,39 @@ 
 	 */
 	public function get_data() {
 
-		$customer = new Give_Customer( $this->customer_id );
-		$payments = $this->get_stored_data( 'give_recount_customer_payments_' . $customer->id, array() );
+		$customer = new Give_Customer($this->customer_id);
+		$payments = $this->get_stored_data('give_recount_customer_payments_'.$customer->id, array());
 
-		$offset     = ( $this->step - 1 ) * $this->per_step;
-		$step_items = array_slice( $payments, $offset, $this->per_step );
+		$offset     = ($this->step - 1) * $this->per_step;
+		$step_items = array_slice($payments, $offset, $this->per_step);
 
-		if ( count( $step_items ) > 0 ) {
-			$pending_total = (float) $this->get_stored_data( 'give_stats_customer_pending_total' . $customer->id, 0 );
+		if (count($step_items) > 0) {
+			$pending_total = (float) $this->get_stored_data('give_stats_customer_pending_total'.$customer->id, 0);
 			$step_total    = 0;
 
-			$found_payment_ids = $this->get_stored_data( 'give_stats_found_payments_' . $customer->id, array() );
+			$found_payment_ids = $this->get_stored_data('give_stats_found_payments_'.$customer->id, array());
 
-			foreach ( $step_items as $payment ) {
-				$payment = get_post( $payment->ID );
+			foreach ($step_items as $payment) {
+				$payment = get_post($payment->ID);
 
-				if ( is_null( $payment ) || is_wp_error( $payment ) || 'give_payment' !== $payment->post_type ) {
+				if (is_null($payment) || is_wp_error($payment) || 'give_payment' !== $payment->post_type) {
 
-					$missing_payments   = $this->get_stored_data( 'give_stats_missing_payments' . $customer->id, array() );
+					$missing_payments   = $this->get_stored_data('give_stats_missing_payments'.$customer->id, array());
 					$missing_payments[] = $payment->ID;
-					$this->store_data( 'give_stats_missing_payments' . $customer->id, $missing_payments );
+					$this->store_data('give_stats_missing_payments'.$customer->id, $missing_payments);
 
 					continue;
 				}
 
 				$should_process_payment = 'publish' == $payment->post_status || 'revoked' == $payment->post_status ? true : false;
-				$should_process_payment = apply_filters( 'give_customer_recount_should_process_payment', $should_process_payment, $payment );
+				$should_process_payment = apply_filters('give_customer_recount_should_process_payment', $should_process_payment, $payment);
 
-				if ( true === $should_process_payment ) {
+				if (true === $should_process_payment) {
 
 					$found_payment_ids[] = $payment->ID;
 
-					if ( apply_filters( 'give_customer_recount_sholud_increase_value', true, $payment ) ) {
-						$payment_amount = give_get_payment_amount( $payment->ID );
+					if (apply_filters('give_customer_recount_sholud_increase_value', true, $payment)) {
+						$payment_amount = give_get_payment_amount($payment->ID);
 						$step_total += $payment_amount;
 					}
 
@@ -95,8 +95,8 @@ 
 			}
 
 			$updated_total = $pending_total + $step_total;
-			$this->store_data( 'give_stats_customer_pending_total' . $customer->id, $updated_total );
-			$this->store_data( 'give_stats_found_payments_' . $customer->id, $found_payment_ids );
+			$this->store_data('give_stats_customer_pending_total'.$customer->id, $updated_total);
+			$this->store_data('give_stats_found_payments_'.$customer->id, $found_payment_ids);
 
 			return true;
 		}
@@ -113,16 +113,16 @@ 
 	 */
 	public function get_percentage_complete() {
 
-		$payments = $this->get_stored_data( 'give_recount_customer_payments_' . $this->customer_id );
-		$total    = count( $payments );
+		$payments = $this->get_stored_data('give_recount_customer_payments_'.$this->customer_id);
+		$total    = count($payments);
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( $this->per_step * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = (($this->per_step * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -136,8 +136,8 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
-		$this->customer_id = isset( $request['customer_id'] ) ? sanitize_text_field( $request['customer_id'] ) : false;
+	public function set_properties($request) {
+		$this->customer_id = isset($request['customer_id']) ? sanitize_text_field($request['customer_id']) : false;
 	}
 
 	/**
@@ -148,62 +148,62 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( __( 'You do not have permission to modify this data.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(__('You do not have permission to modify this data.', 'give'), __('Error', 'give'), array('response' => 403));
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
-			$customer    = new Give_Customer( $this->customer_id );
-			$payment_ids = get_option( 'give_stats_found_payments_' . $customer->id, array() );
-			$this->delete_data( 'give_stats_found_payments_' . $customer->id );
+			$customer    = new Give_Customer($this->customer_id);
+			$payment_ids = get_option('give_stats_found_payments_'.$customer->id, array());
+			$this->delete_data('give_stats_found_payments_'.$customer->id);
 
-			$removed_payments = array_unique( get_option( 'give_stats_missing_payments' . $customer->id, array() ) );
+			$removed_payments = array_unique(get_option('give_stats_missing_payments'.$customer->id, array()));
 
 			// Find non-existing payments (deleted) and total up the purchase count
 			$purchase_count = 0;
-			foreach ( $payment_ids as $key => $payment_id ) {
-				if ( in_array( $payment_id, $removed_payments ) ) {
-					unset( $payment_ids[ $key ] );
+			foreach ($payment_ids as $key => $payment_id) {
+				if (in_array($payment_id, $removed_payments)) {
+					unset($payment_ids[$key]);
 					continue;
 				}
 
-				$payment = get_post( $payment_id );
-				if ( apply_filters( 'give_customer_recount_sholud_increase_count', true, $payment ) ) {
-					$purchase_count ++;
+				$payment = get_post($payment_id);
+				if (apply_filters('give_customer_recount_sholud_increase_count', true, $payment)) {
+					$purchase_count++;
 				}
 			}
 
-			$this->delete_data( 'give_stats_missing_payments' . $customer->id );
+			$this->delete_data('give_stats_missing_payments'.$customer->id);
 
-			$pending_total = $this->get_stored_data( 'give_stats_customer_pending_total' . $customer->id, 0 );
-			$this->delete_data( 'give_stats_customer_pending_total' . $customer->id );
-			$this->delete_data( 'give_recount_customer_stats_' . $customer->id );
-			$this->delete_data( 'give_recount_customer_payments_' . $this->customer_id );
+			$pending_total = $this->get_stored_data('give_stats_customer_pending_total'.$customer->id, 0);
+			$this->delete_data('give_stats_customer_pending_total'.$customer->id);
+			$this->delete_data('give_recount_customer_stats_'.$customer->id);
+			$this->delete_data('give_recount_customer_payments_'.$this->customer_id);
 
-			$payment_ids = implode( ',', $payment_ids );
-			$customer->update( array( 'payment_ids'    => $payment_ids,
+			$payment_ids = implode(',', $payment_ids);
+			$customer->update(array('payment_ids'    => $payment_ids,
 			                          'purchase_count' => $purchase_count,
 			                          'purchase_value' => $pending_total
-			) );
+			));
 
 			$this->done    = true;
-			$this->message = __( 'Donor stats successfully recounted.', 'give' );
+			$this->message = __('Donor stats successfully recounted.', 'give');
 
 			return false;
 		}
 	}
 
 	public function headers() {
-		ignore_user_abort( true );
+		ignore_user_abort(true);
 
-		if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-			set_time_limit( 0 );
+		if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+			set_time_limit(0);
 		}
 	}
 
@@ -230,26 +230,26 @@ 
 	 * @return void
 	 */
 	public function pre_fetch() {
-		if ( $this->step === 1 ) {
-			$allowed_payment_status = apply_filters( 'give_recount_customer_payment_statuses', give_get_payment_status_keys() );
+		if ($this->step === 1) {
+			$allowed_payment_status = apply_filters('give_recount_customer_payment_statuses', give_get_payment_status_keys());
 
 			// Before we start, let's zero out the customer's data
-			$customer = new Give_Customer( $this->customer_id );
-			$customer->update( array( 'purchase_value' => give_format_amount( 0 ), 'purchase_count' => 0 ) );
+			$customer = new Give_Customer($this->customer_id);
+			$customer->update(array('purchase_value' => give_format_amount(0), 'purchase_count' => 0));
 
-			$attached_payment_ids = explode( ',', $customer->payment_ids );
+			$attached_payment_ids = explode(',', $customer->payment_ids);
 
 			$attached_args = array(
 				'post__in' => $attached_payment_ids,
-				'number'   => - 1,
+				'number'   => -1,
 				'status'   => $allowed_post_status,
 			);
 
-			$attached_payments = give_get_payments( $attached_args );
+			$attached_payments = give_get_payments($attached_args);
 
 			$unattached_args = array(
 				'post__not_in' => $attached_payment_ids,
-				'number'       => - 1,
+				'number'       => -1,
 				'status'       => $allowed_post_status,
 				'meta_query'   => array(
 					array(
@@ -259,11 +259,11 @@ 
 				),
 			);
 
-			$unattached_payments = give_get_payments( $unattached_args );
+			$unattached_payments = give_get_payments($unattached_args);
 
-			$payments = array_merge( $attached_payments, $unattached_payments );
+			$payments = array_merge($attached_payments, $unattached_payments);
 
-			$this->store_data( 'give_recount_customer_payments_' . $customer->id, $payments );
+			$this->store_data('give_recount_customer_payments_'.$customer->id, $payments);
 		}
 	}
 
@@ -276,11 +276,11 @@ 
 	 *
 	 * @return mixed       Returns the data from the database
 	 */
-	private function get_stored_data( $key ) {
+	private function get_stored_data($key) {
 		global $wpdb;
-		$value = $wpdb->get_var( $wpdb->prepare( "SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key ) );
+		$value = $wpdb->get_var($wpdb->prepare("SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key));
 
-		return empty( $value ) ? false : maybe_unserialize( $value );
+		return empty($value) ? false : maybe_unserialize($value);
 	}
 
 	/**
@@ -293,10 +293,10 @@ 
 	 *
 	 * @return void
 	 */
-	private function store_data( $key, $value ) {
+	private function store_data($key, $value) {
 		global $wpdb;
 
-		$value = maybe_serialize( $value );
+		$value = maybe_serialize($value);
 
 		$data = array(
 			'option_name'  => $key,
@@ -310,7 +310,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -322,9 +322,9 @@ 
 	 *
 	 * @return void
 	 */
-	private function delete_data( $key ) {
+	private function delete_data($key) {
 		global $wpdb;
-		$wpdb->delete( $wpdb->options, array( 'option_name' => $key ) );
+		$wpdb->delete($wpdb->options, array('option_name' => $key));
 	}
 
 }

includes/admin/reporting/tools/class-give-tools-reset-stats.php

Doc Comments +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
 	 * @since 1.5
 	 * @global object $wpdb Used to query the database using the WordPress
 	 *   Database API
-	 * @return array $data The data for the CSV file
+	 * @return boolean $data The data for the CSV file
 	 */
 	public function get_data() {
 		global $wpdb;

Spacing +64 added lines, -64 removed lines

@@ -11,7 +11,7 @@ 
  */
 
 // Exit if accessed directly
-if ( ! defined( 'ABSPATH' ) ) {
+if ( ! defined('ABSPATH')) {
 	exit;
 }
 
@@ -55,16 +55,16 @@ 
 	public function get_data() {
 		global $wpdb;
 
-		$items = $this->get_stored_data( 'give_temp_reset_ids' );
+		$items = $this->get_stored_data('give_temp_reset_ids');
 
-		if ( ! is_array( $items ) ) {
+		if ( ! is_array($items)) {
 			return false;
 		}
 
-		$offset     = ( $this->step - 1 ) * $this->per_step;
-		$step_items = array_slice( $items, $offset, $this->per_step );
+		$offset     = ($this->step - 1) * $this->per_step;
+		$step_items = array_slice($items, $offset, $this->per_step);
 
-		if ( $step_items ) {
+		if ($step_items) {
 
 			$step_ids = array(
 				'customers'  => array(),
@@ -72,9 +72,9 @@ 
 				'other'      => array(),
 			);
 
-			foreach ( $step_items as $item ) {
+			foreach ($step_items as $item) {
 
-				switch ( $item['type'] ) {
+				switch ($item['type']) {
 					case 'customer':
 						$step_ids['customers'][] = $item['id'];
 						break;
@@ -82,8 +82,8 @@ 
 						$step_ids['give_forms'][] = $item['id'];
 						break;
 					default:
-						$item_type                = apply_filters( 'give_reset_item_type', 'other', $item );
-						$step_ids[ $item_type ][] = $item['id'];
+						$item_type                = apply_filters('give_reset_item_type', 'other', $item);
+						$step_ids[$item_type][] = $item['id'];
 						break;
 				}
 
@@ -91,17 +91,17 @@ 
 
 			$sql = array();
 
-			foreach ( $step_ids as $type => $ids ) {
+			foreach ($step_ids as $type => $ids) {
 
-				if ( empty( $ids ) ) {
+				if (empty($ids)) {
 					continue;
 				}
 
-				$ids = implode( ',', $ids );
+				$ids = implode(',', $ids);
 
-				switch ( $type ) {
+				switch ($type) {
 					case 'customers':
-						$table_name = $wpdb->prefix . 'give_customers';
+						$table_name = $wpdb->prefix.'give_customers';
 						$sql[]      = "DELETE FROM $table_name WHERE id IN ($ids)";
 						break;
 					case 'forms':
@@ -116,17 +116,17 @@ 
 						break;
 				}
 
-				if ( ! in_array( $type, array( 'customers', 'forms', 'other' ) ) ) {
+				if ( ! in_array($type, array('customers', 'forms', 'other'))) {
 					// Allows other types of custom post types to filter on their own post_type
 					// and add items to the query list, for the IDs found in their post type.
-					$sql = apply_filters( 'give_reset_add_queries_' . $type, $sql, $ids );
+					$sql = apply_filters('give_reset_add_queries_'.$type, $sql, $ids);
 				}
 
 			}
 
-			if ( ! empty( $sql ) ) {
-				foreach ( $sql as $query ) {
-					$wpdb->query( $query );
+			if ( ! empty($sql)) {
+				foreach ($sql as $query) {
+					$wpdb->query($query);
 				}
 			}
 
@@ -146,16 +146,16 @@ 
 	 */
 	public function get_percentage_complete() {
 
-		$items = $this->get_stored_data( 'give_temp_reset_ids', false );
-		$total = count( $items );
+		$items = $this->get_stored_data('give_temp_reset_ids', false);
+		$total = count($items);
 
 		$percentage = 100;
 
-		if ( $total > 0 ) {
-			$percentage = ( ( $this->per_step * $this->step ) / $total ) * 100;
+		if ($total > 0) {
+			$percentage = (($this->per_step * $this->step) / $total) * 100;
 		}
 
-		if ( $percentage > 100 ) {
+		if ($percentage > 100) {
 			$percentage = 100;
 		}
 
@@ -169,7 +169,7 @@ 
 	 *
 	 * @param array $request The Form Data passed into the batch processing
 	 */
-	public function set_properties( $request ) {
+	public function set_properties($request) {
 	}
 
 	/**
@@ -180,30 +180,30 @@ 
 	 */
 	public function process_step() {
 
-		if ( ! $this->can_export() ) {
-			wp_die( __( 'You do not have permission to export data.', 'give' ), __( 'Error', 'give' ), array( 'response' => 403 ) );
+		if ( ! $this->can_export()) {
+			wp_die(__('You do not have permission to export data.', 'give'), __('Error', 'give'), array('response' => 403));
 		}
 
 		$had_data = $this->get_data();
 
-		if ( $had_data ) {
+		if ($had_data) {
 			$this->done = false;
 
 			return true;
 		} else {
-			update_option( 'give_earnings_total', 0 );
-			delete_transient( 'give_earnings_total' );
-			delete_transient( 'give_estimated_monthly_stats' . true );
-			delete_transient( 'give_estimated_monthly_stats' . false );
-			$this->delete_data( 'give_temp_reset_ids' );
+			update_option('give_earnings_total', 0);
+			delete_transient('give_earnings_total');
+			delete_transient('give_estimated_monthly_stats'.true);
+			delete_transient('give_estimated_monthly_stats'.false);
+			$this->delete_data('give_temp_reset_ids');
 
 			// Reset the sequential order numbers
-			if ( give_get_option( 'enable_sequential' ) ) {
-				delete_option( 'give_last_payment_number' );
+			if (give_get_option('enable_sequential')) {
+				delete_option('give_last_payment_number');
 			}
 
 			$this->done    = true;
-			$this->message = __( 'Donation forms, income, donations counts, and logs successfully reset.', 'give' );
+			$this->message = __('Donation forms, income, donations counts, and logs successfully reset.', 'give');
 
 			return false;
 		}
@@ -213,10 +213,10 @@ 
 	 * Headers
 	 */
 	public function headers() {
-		ignore_user_abort( true );
+		ignore_user_abort(true);
 
-		if ( ! give_is_func_disabled( 'set_time_limit' ) && ! ini_get( 'safe_mode' ) ) {
-			set_time_limit( 0 );
+		if ( ! give_is_func_disabled('set_time_limit') && ! ini_get('safe_mode')) {
+			set_time_limit(0);
 		}
 	}
 
@@ -240,35 +240,35 @@ 
 	 */
 	public function pre_fetch() {
 
-		if ( $this->step == 1 ) {
-			$this->delete_data( 'give_temp_reset_ids' );
+		if ($this->step == 1) {
+			$this->delete_data('give_temp_reset_ids');
 		}
 
-		$items = get_option( 'give_temp_reset_ids', false );
+		$items = get_option('give_temp_reset_ids', false);
 
-		if ( false === $items ) {
+		if (false === $items) {
 			$items = array();
 
-			$give_types_for_reset = array( 'give_forms', 'give_log', 'give_payment' );
-			$give_types_for_reset = apply_filters( 'give_reset_store_post_types', $give_types_for_reset );
+			$give_types_for_reset = array('give_forms', 'give_log', 'give_payment');
+			$give_types_for_reset = apply_filters('give_reset_store_post_types', $give_types_for_reset);
 
-			$args = apply_filters( 'give_tools_reset_stats_total_args', array(
+			$args = apply_filters('give_tools_reset_stats_total_args', array(
 				'post_type'      => $give_types_for_reset,
 				'post_status'    => 'any',
-				'posts_per_page' => - 1,
-			) );
+				'posts_per_page' => -1,
+			));
 
-			$posts = get_posts( $args );
-			foreach ( $posts as $post ) {
+			$posts = get_posts($args);
+			foreach ($posts as $post) {
 				$items[] = array(
 					'id'   => (int) $post->ID,
 					'type' => $post->post_type,
 				);
 			}
 
-			$customer_args = array( 'number' => - 1 );
-			$customers     = Give()->customers->get_customers( $customer_args );
-			foreach ( $customers as $customer ) {
+			$customer_args = array('number' => -1);
+			$customers     = Give()->customers->get_customers($customer_args);
+			foreach ($customers as $customer) {
 				$items[] = array(
 					'id'   => (int) $customer->id,
 					'type' => 'customer',
@@ -277,9 +277,9 @@ 
 
 			// Allow filtering of items to remove with an unassociative array for each item
 			// The array contains the unique ID of the item, and a 'type' for you to use in the execution of the get_data method
-			$items = apply_filters( 'give_reset_store_items', $items );
+			$items = apply_filters('give_reset_store_items', $items);
 
-			$this->store_data( 'give_temp_reset_ids', $items );
+			$this->store_data('give_temp_reset_ids', $items);
 		}
 
 	}
@@ -293,11 +293,11 @@ 
 	 *
 	 * @return mixed       Returns the data from the database
 	 */
-	private function get_stored_data( $key ) {
+	private function get_stored_data($key) {
 		global $wpdb;
-		$value = $wpdb->get_var( $wpdb->prepare( "SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key ) );
+		$value = $wpdb->get_var($wpdb->prepare("SELECT option_value FROM $wpdb->options WHERE option_name = '%s'", $key));
 
-		return empty( $value ) ? false : maybe_unserialize( $value );
+		return empty($value) ? false : maybe_unserialize($value);
 	}
 
 	/**
@@ -310,10 +310,10 @@ 
 	 *
 	 * @return void
 	 */
-	private function store_data( $key, $value ) {
+	private function store_data($key, $value) {
 		global $wpdb;
 
-		$value = maybe_serialize( $value );
+		$value = maybe_serialize($value);
 
 		$data = array(
 			'option_name'  => $key,
@@ -327,7 +327,7 @@ 
 			'%s',
 		);
 
-		$wpdb->replace( $wpdb->options, $data, $formats );
+		$wpdb->replace($wpdb->options, $data, $formats);
 	}
 
 	/**
@@ -339,9 +339,9 @@ 
 	 *
 	 * @return void
 	 */
-	private function delete_data( $key ) {
+	private function delete_data($key) {
 		global $wpdb;
-		$wpdb->delete( $wpdb->options, array( 'option_name' => $key ) );
+		$wpdb->delete($wpdb->options, array('option_name' => $key));
 	}
 
 }

includes/class-give-donate-form.php

Doc Comments +2 added lines, -2 removed lines

@@ -152,7 +152,7 @@ 
 	 *
 	 * @since 1.0
 	 *
-	 * @param $key
+	 * @param string $key
 	 *
 	 * @return mixed
 	 * @throws Exception
@@ -179,7 +179,7 @@ 
 	 *
 	 * @param  array $data Array of attributes for a donation form
 	 *
-	 * @return mixed  false if data isn't passed and class not instantiated for creation, or New Form ID
+	 * @return boolean  false if data isn't passed and class not instantiated for creation, or New Form ID
 	 */
 	public function create( $data = array() ) {
 

Spacing +94 added lines, -94 removed lines

@@ -100,12 +100,12 @@ 
 	 * @param bool $_id
 	 * @param array $_args
 	 */
-	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);
 	}
 
 	/**
@@ -117,23 +117,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;
@@ -157,15 +157,15 @@ 
 	 * @return mixed
 	 * @throws Exception
 	 */
-	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 {
 
-			return new WP_Error( 'give-form-invalid-property', sprintf( __( 'Can\'t get property %s', 'give' ), $key ) );
+			return new WP_Error('give-form-invalid-property', sprintf(__('Can\'t get property %s', 'give'), $key));
 
 		}
 
@@ -181,30 +181,30 @@ 
 	 *
 	 * @return mixed  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
@@ -212,9 +212,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);
 
 	}
 
@@ -237,7 +237,7 @@ 
 	 * @return string Name of the donation form
 	 */
 	public function get_name() {
-		return get_the_title( $this->ID );
+		return get_the_title($this->ID);
 	}
 
 	/**
@@ -248,13 +248,13 @@ 
 	 */
 	public function get_price() {
 
-		if ( ! isset( $this->price ) ) {
+		if ( ! isset($this->price)) {
 
-			$this->price = get_post_meta( $this->ID, '_give_set_price', true );
+			$this->price = get_post_meta($this->ID, '_give_set_price', true);
 
-			if ( $this->price ) {
+			if ($this->price) {
 
-				$this->price = give_sanitize_amount( $this->price );
+				$this->price = give_sanitize_amount($this->price);
 
 			} else {
 
@@ -272,7 +272,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);
 	}
 
 	/**
@@ -283,14 +283,14 @@ 
 	 */
 	public function get_minimum_price() {
 
-		if ( ! isset( $this->minimum_price ) ) {
+		if ( ! isset($this->minimum_price)) {
 
-			$allow_custom_amount = get_post_meta( $this->ID, '_give_custom_amount', true );
-			$this->minimum_price = get_post_meta( $this->ID, '_give_custom_amount_minimum', true );
+			$allow_custom_amount = get_post_meta($this->ID, '_give_custom_amount', true);
+			$this->minimum_price = get_post_meta($this->ID, '_give_custom_amount_minimum', true);
 
-			if ( $allow_custom_amount != 'no' && $this->minimum_price ) {
+			if ($allow_custom_amount != 'no' && $this->minimum_price) {
 
-				$this->minimum_price = give_sanitize_amount( $this->minimum_price );
+				$this->minimum_price = give_sanitize_amount($this->minimum_price);
 
 			} else {
 
@@ -300,7 +300,7 @@ 
 
 		}
 
-		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);
 	}
 
 	/**
@@ -311,9 +311,9 @@ 
 	 */
 	public function get_prices() {
 
-		if ( ! isset( $this->prices ) ) {
+		if ( ! isset($this->prices)) {
 
-			$this->prices = get_post_meta( $this->ID, '_give_donation_levels', true );
+			$this->prices = get_post_meta($this->ID, '_give_donation_levels', true);
 
 		}
 
@@ -325,7 +325,7 @@ 
 		 * @param array $prices The array of mulit-level prices.
 		 * @param int|string 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);
 
 	}
 
@@ -337,13 +337,13 @@ 
 	 */
 	public function get_goal() {
 
-		if ( ! isset( $this->goal ) ) {
+		if ( ! isset($this->goal)) {
 
-			$this->goal = get_post_meta( $this->ID, '_give_set_goal', true );
+			$this->goal = get_post_meta($this->ID, '_give_set_goal', true);
 
-			if ( $this->goal ) {
+			if ($this->goal) {
 
-				$this->goal = give_sanitize_amount( $this->goal );
+				$this->goal = give_sanitize_amount($this->goal);
 
 			} else {
 
@@ -353,7 +353,7 @@ 
 
 		}
 
-		return apply_filters( 'give_get_set_goal', $this->goal, $this->ID );
+		return apply_filters('give_get_set_goal', $this->goal, $this->ID);
 
 	}
 
@@ -365,10 +365,10 @@ 
 	 */
 	public function is_single_price_mode() {
 
-		$option = get_post_meta( $this->ID, '_give_price_options_mode', true );
+		$option = get_post_meta($this->ID, '_give_price_options_mode', true);
 		$ret    = 0;
 
-		if ( empty( $option ) || $option === 'set' ) {
+		if (empty($option) || $option === 'set') {
 			$ret = 1;
 		}
 		
@@ -380,7 +380,7 @@ 
 		 * @param bool $ret Is donation form in single price mode?
 		 * @param int|string 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);
 
 	}
 
@@ -394,10 +394,10 @@ 
 	 */
 	public function has_variable_prices() {
 
-		$option = get_post_meta( $this->ID, '_give_price_option', true );
+		$option = get_post_meta($this->ID, '_give_price_option', true);
 		$ret    = 0;
 
-		if ( $option === 'multi' ) {
+		if ($option === 'multi') {
 			$ret = 1;
 		}
 
@@ -407,7 +407,7 @@ 
 		 * @param bool $ret Does donation form have variable prices?
 		 * @param int|string 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);
 
 	}
 
@@ -419,17 +419,17 @@ 
 	 */
 	public function get_type() {
 
-		if ( ! isset( $this->type ) ) {
+		if ( ! isset($this->type)) {
 
-			$this->type = get_post_meta( $this->ID, '_give_price_option', true );
+			$this->type = get_post_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);
 
 	}
 
@@ -441,15 +441,15 @@ 
 	 */
 	public function get_sales() {
 
-		if ( ! isset( $this->sales ) ) {
+		if ( ! isset($this->sales)) {
 
-			if ( '' == get_post_meta( $this->ID, '_give_form_sales', true ) ) {
-				add_post_meta( $this->ID, '_give_form_sales', 0 );
+			if ('' == get_post_meta($this->ID, '_give_form_sales', true)) {
+				add_post_meta($this->ID, '_give_form_sales', 0);
 			} // End if
 
-			$this->sales = get_post_meta( $this->ID, '_give_form_sales', true );
+			$this->sales = get_post_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;
 			}
@@ -469,13 +469,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;
 
@@ -495,17 +495,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;
 
@@ -527,15 +527,15 @@ 
 	 */
 	public function get_earnings() {
 
-		if ( ! isset( $this->earnings ) ) {
+		if ( ! isset($this->earnings)) {
 
-			if ( '' == get_post_meta( $this->ID, '_give_form_earnings', true ) ) {
-				add_post_meta( $this->ID, '_give_form_earnings', 0 );
+			if ('' == get_post_meta($this->ID, '_give_form_earnings', true)) {
+				add_post_meta($this->ID, '_give_form_earnings', 0);
 			}
 
-			$this->earnings = get_post_meta( $this->ID, '_give_form_earnings', true );
+			$this->earnings = get_post_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;
 			}
@@ -552,12 +552,12 @@ 
 	 * @since 1.0
 	 * @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;
 
@@ -575,16 +575,16 @@ 
 	 * @since 1.0
 	 * @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;
 
@@ -604,22 +604,22 @@ 
 	 * @since 1.0
 	 * @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 = get_post_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 = get_post_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);
 
 	}
 
@@ -635,28 +635,28 @@ 
 	 *
 	 * @return bool             The result of the update query
 	 */
-	private function update_meta( $meta_key = '', $meta_value = '' ) {
+	private function update_meta($meta_key = '', $meta_value = '') {
 
 		global $wpdb;
 
-		if ( empty( $meta_key ) || empty( $meta_value ) ) {
+		if (empty($meta_key) || empty($meta_value)) {
 			return false;
 		}
 
 		// Make sure if it needs to be serialized, we do
-		$meta_value = maybe_serialize( $meta_value );
+		$meta_value = maybe_serialize($meta_value);
 
-		if ( is_numeric( $meta_value ) ) {
-			$value_type = is_float( $meta_value ) ? '%f' : '%d';
+		if (is_numeric($meta_value)) {
+			$value_type = is_float($meta_value) ? '%f' : '%d';
 		} else {
 			$value_type = "'%s'";
 		}
 
-		$sql = $wpdb->prepare( "UPDATE $wpdb->postmeta SET meta_value = $value_type WHERE post_id = $this->ID AND meta_key = '%s'", $meta_value, $meta_key );
+		$sql = $wpdb->prepare("UPDATE $wpdb->postmeta SET meta_value = $value_type WHERE post_id = $this->ID AND meta_key = '%s'", $meta_value, $meta_key);
 
-		if ( $wpdb->query( $sql ) ) {
+		if ($wpdb->query($sql)) {
 
-			clean_post_cache( $this->ID );
+			clean_post_cache($this->ID);
 
 			return true;